-
Notifications
You must be signed in to change notification settings - Fork 354
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
ci: Djanicek/infraeng 487/circle test runner #8977
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8977 +/- ##
=======================================
Coverage 47.49% 47.49%
=======================================
Files 1168 1168
Lines 176315 176315
Branches 2353 2350 -3
=======================================
Hits 83732 83732
Misses 92425 92425
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable
99b37b7
to
053390a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks good. i'd like to find a way to run a single test manually as well
053390a
to
28cd47e
Compare
* use circle test runner instead of splitter * fix xargs syntax
Description
INFRAENG-487
Circle-ci lets us re-run failed tests individually if we use the circle ci runner instead of splitting and running separately.
https://circleci.com/docs/rerun-failed-tests/
This makes it faster and cheaper to re-run failures on circle-ci since individual tests can be re-run instead of whole jobs.
Test Plan
pipeline passed and all the E2E tests took the same amount of time. Spot checking most of them, they all ran the same number of tests as main.
https://app.circleci.com/pipelines/github/determined-ai/determined/52143/workflows/311b97d9-4e74-4c18-90c2-f03abd7f3cf6/jobs/2331410
We'll have to be careful of long-running nightly e2e. I'm running a sample to insure they work. https://app.circleci.com/pipelines/github/determined-ai/determined/52143/workflows/2e6ceb8a-4d72-4ae3-9f3c-73e85358f7ee
Checklist
docs/release-notes/
.See Release Note for details.