-
Notifications
You must be signed in to change notification settings - Fork 55
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 issues with the Bulk API Scale Tests #1400
Conversation
…ents with time-range Signed-off-by: Saad Khan <[email protected]>
@khansaad - Have you tested it with the scale test and thanos setup? |
With However, in both cases the |
@khansaad - I think 503 is with 1k containers, can you share the build you used. |
|
@khansaad - I still see the issue with the above image ./bulk_scale_test.sh -i quay.io/khansaad/autotune_operator:time-range-fix -w 5 -r /tmp/res
|
I have re-tested this again, but haven't seen this issue. Could be intermittent though, for now we can merge this and will raise another issue if seen again |
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.
LGTM
Description
This PR fixes issues with the Bulk API scale tests
Fixes # (#1399 )
Type of change
How has this been tested?
Please describe the tests that were run to verify your changes and steps to reproduce. Please specify any test configuration required.
Test Configuration
Checklist 🎯
Additional information
Include any additional information such as links, test results, screenshots here