Skip to content

ci: rm collect-test-data#110182

Closed
joshuarli wants to merge 2 commits intomasterfrom
remove-collect-test-data
Closed

ci: rm collect-test-data#110182
joshuarli wants to merge 2 commits intomasterfrom
remove-collect-test-data

Conversation

@joshuarli
Copy link
Member

this powers devinfra-flakiness (the old flaky test triage system) which has since been paused because i'm working on burning down flaky tests with a much better and more actionable system so at present it doesn't provide much value beyond noise

@joshuarli joshuarli requested review from a team as code owners March 7, 2026 01:10
@joshuarli joshuarli requested a review from rbro112 March 7, 2026 01:11
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 7, 2026
@joshuarli joshuarli requested a review from a team as a code owner March 7, 2026 01:17
--json-report \
--json-report-file=".artifacts/pytest.monolith-dbs.json" \
--json-report-omit=log \
--junit-xml=.artifacts/monolith-dbs.junit.xml \
Copy link
Member Author

Choose a reason for hiding this comment

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

we don't need these junit xmls anymore either; unused

devservices logs
fi

- name: Collect test data
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't this also remove the devinfra metrics reporting we rely on?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah - forgot about that

@joshuarli joshuarli closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants