Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add job link to race test notifications #16018

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

erikburt
Copy link
Collaborator

@erikburt erikburt commented Jan 21, 2025

Changes

Include the job link in race test failure notifications

Notes

Github doesn't natively support getting the job id, and it is especially bad within matrix jobs.

gh run --repo $GH_REPO view $GH_RUN_ID --json jobs --jq '.jobs[] | select(.name | contains("${{ matrix.type.cmd }}")) | .url'

This command was adapted from: https://stackoverflow.com/questions/59073850/github-actions-get-url-of-test-build, this works because we know a substring of our race tests job so we can do select(.name | contains("${{ matrix.type.cmd }}").

Testing

Echo'd the run_url here: https://github.com/smartcontractkit/chainlink/actions/runs/12896302500/job/35959051383?pr=16018#step:4:14


RE-3429

@erikburt erikburt self-assigned this Jan 21, 2025
@erikburt erikburt force-pushed the fix/ci-core-race-test-link branch 2 times, most recently from 045392a to 5d3a086 Compare January 21, 2025 21:15
@smartcontractkit smartcontractkit deleted a comment from github-actions bot Jan 21, 2025
@erikburt erikburt force-pushed the fix/ci-core-race-test-link branch 3 times, most recently from 781e7d3 to 3b58c7a Compare January 21, 2025 21:34
@erikburt erikburt force-pushed the fix/ci-core-race-test-link branch from 3b58c7a to f72c760 Compare January 21, 2025 21:40
@erikburt erikburt marked this pull request as ready for review January 21, 2025 21:41
@erikburt erikburt requested review from a team as code owners January 21, 2025 21:41
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@erikburt erikburt enabled auto-merge January 21, 2025 22:08
Comment on lines +293 to +295
GH_REPO: ${{ github.repository }}
GH_RUN_ID: ${{ github.run_id }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@erikburt erikburt added this pull request to the merge queue Jan 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 22, 2025
@jmank88 jmank88 added this pull request to the merge queue Jan 22, 2025
Merged via the queue into develop with commit a05a947 Jan 22, 2025
102 checks passed
@jmank88 jmank88 deleted the fix/ci-core-race-test-link branch January 22, 2025 01:09
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.

3 participants