Skip to content

bug(infra): Fix backend-with-coverage job to have odiff binary#109922

Merged
rbro112 merged 2 commits intomasterfrom
ryan/fix_backend-with-coverage_job_to_have_odiff_binary
Mar 5, 2026
Merged

bug(infra): Fix backend-with-coverage job to have odiff binary#109922
rbro112 merged 2 commits intomasterfrom
ryan/fix_backend-with-coverage_job_to_have_odiff_binary

Conversation

@rbro112
Copy link
Copy Markdown
Member

@rbro112 rbro112 commented Mar 4, 2026

Last week #109381 was merged which added the odiff binary to the backend CI workflow as it was a new required dependency for CI tests.

Unfortunately, we missed adding this to the backend-with-coverage job which is used to upload selective testing coverage metrics. This caused all runs of backend with coverage to fail, resulting in no coverage data being uploaded.

This fixes the backend-with-coverage workflow to install the odiff binary which should fix tests.

Manual run to confirm this fixes: https://github.com/getsentry/sentry/actions/runs/22696037548

@rbro112 rbro112 requested a review from a team as a code owner March 4, 2026 23:40
Copy link
Copy Markdown
Member Author

rbro112 commented Mar 4, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@rbro112 rbro112 changed the title Fix backend-with-coverage job to have odiff binary bug(infra): Fix backend-with-coverage job to have odiff binary Mar 4, 2026
@rbro112 rbro112 merged commit 99d52ff into master Mar 5, 2026
74 checks passed
@rbro112 rbro112 deleted the ryan/fix_backend-with-coverage_job_to_have_odiff_binary branch March 5, 2026 00:59
JonasBa pushed a commit that referenced this pull request Mar 5, 2026
Last week #109381 was merged
which added the odiff binary to the backend CI workflow as it was a new
required dependency for CI tests.

Unfortunately, we missed adding this to the `backend-with-coverage` job
which is used to upload selective testing coverage metrics. This caused
all runs of backend with coverage to fail, resulting in no coverage data
being uploaded.

This fixes the `backend-with-coverage` workflow to install the odiff
binary which should fix tests.

Manual run to confirm this fixes:
https://github.com/getsentry/sentry/actions/runs/22696037548
@github-actions github-actions bot locked and limited conversation to collaborators Mar 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants