Skip to content

[DX-3611] notify channel if memory leak test fails#21877

Open
Tofel wants to merge 3 commits intodevelopfrom
dx-3611-soak-tests-notification
Open

[DX-3611] notify channel if memory leak test fails#21877
Tofel wants to merge 3 commits intodevelopfrom
dx-3611-soak-tests-notification

Conversation

@Tofel
Copy link
Copy Markdown
Contributor

@Tofel Tofel commented Apr 7, 2026

Comment on lines +149 to +194
name: Notify about test Failure
if: failure()
needs: [soak]
environment:
name: integration
deployment: false
runs-on: ubuntu-latest
steps:
- name: Send slack notification for failed migration tests
id: send-slack-notification
uses: slackapi/slack-github-action@485a9d42d3a73031f12ec201c457e2162c45d02d # v2.0.0
with:
errors: "true"
method: chat.postMessage
token: ${{ secrets.QA_SLACK_API_KEY }}
payload: |
{
"channel": "C0117GGJB6Y",
"text": "CRE Memory Leak Test Failed. Investigation is required.",
"blocks": [
{
"type": "section",
"text": {
"type": "mrkdwn",
"text": "*:rotating_light: CRE Memory Leak Test Failed. Investigation is required. :rotating_light:*"
}
},
{
"type": "section",
"text": {
"type": "mrkdwn",
"text": "Alerting <@U033CGL3CDT>, CRE Memory Leak Test failed for commit <${{ github.server_url }}/${{ github.repository }}/commit/${{ github.sha }}|${{ github.sha }}> on run ID <${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}|${{ github.run_id }}>. Please investigate."
}
},
{
"type": "divider"
},
{
"type": "section",
"text": {
"type": "mrkdwn",
"text": "Check the Docker logs and the alloc.pprof file to investigate the issue."
}
}
]
}
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

✅ No conflicts with other open PRs targeting develop

@Tofel Tofel marked this pull request as ready for review April 7, 2026 14:50
@Tofel Tofel requested review from a team as code owners April 7, 2026 14:50
Copilot AI review requested due to automatic review settings April 7, 2026 14:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a Slack notification path to the CRE PoR memory leak soak workflow so a channel/user is alerted when the soak test run fails.

Changes:

  • Set workflow-level permissions: contents: read.
  • Add a notify-test-failure job that posts a Slack message when the workflow fails.

@Tofel Tofel force-pushed the dx-3611-soak-tests-notification branch from 0c2c81f to a38f5a0 Compare April 7, 2026 15:09
@skudasov skudasov self-requested a review April 7, 2026 15:10
skudasov
skudasov previously approved these changes Apr 7, 2026
@Tofel Tofel requested a review from skudasov April 7, 2026 15:31
@Tofel Tofel enabled auto-merge April 7, 2026 15:32
@trunk-io
Copy link
Copy Markdown

trunk-io bot commented Apr 7, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
Test_CRE_V2_EVM_Read_StateQueries/[v2]_EVM_Read_(state-queries)_-_workflow-gateway-capabilities/Read_EVMReadBalance/on_chain_2337 The test failed because it timed out while waiting for the transaction to be mined, indicating a possible issue with transaction processing or netw... Logs ↗︎
Test_CRE_V2_EVM_Read_StateQueries/[v2]_EVM_Read_(state-queries)_-_workflow-gateway-capabilities/Read_EVMReadHeaders The test failed during a state query operation, but the specific cause of failure is not indicated in the provided logs. Logs ↗︎
Test_CRE_V2_EVM_Read_StateQueries/[v2]_EVM_Read_(state-queries)_-_workflow-gateway-capabilities The test failed without providing specific error details. Logs ↗︎
Test_CRE_V2_EVM_Read_StateQueries/[v2]_EVM_Read_(state-queries)_-_workflow-gateway-capabilities/Read_EVMReadNotFoundTx The test failed during a state query operation because it did not find the expected transaction. Logs ↗︎

... and 4 more

View Full Report ↗︎Docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants