-
Notifications
You must be signed in to change notification settings - Fork 349
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
test: performance test CI work #8761
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 #8761 +/- ##
==========================================
- Coverage 47.44% 47.44% -0.01%
==========================================
Files 1046 1046
Lines 166888 166888
Branches 2239 2241 +2
==========================================
- Hits 79185 79182 -3
- Misses 87545 87548 +3
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
02dcbc5
to
bdcfcfa
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.
looks great
.circleci/real_config.yml
Outdated
command: python .circleci/scripts/wait_for_perf_migration_upload_results.py | ||
|
||
- when: | ||
condition: <<parameters.take-snapshot>> |
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.
maybe call this snapshot-after-migrations
), | ||
// This is a bad endpoint and we know it is bad. | ||
// No sense in making other endpoints be slowed before we get the fix in | ||
// https://hpe-aiatscale.atlassian.net/browse/DET-10114 |
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.
Can we ensure that a ticket is created or a comment is made in one of these tickets to re-enable this test. Although I am assume that was the plan in the ticket anyway, I just don't want us to forget.
Description
Does the infra work for perf tests.
Runs the performance tests on every commit on main and allows running on feature branches.
Also records migration timings. Writes this all to a postgres instance we will eventually hook up to our ci grafana.
Test Plan
Manual, and also we will test a bunch after this lands and adjust any issues we find.
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket