-
Notifications
You must be signed in to change notification settings - Fork 295
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
Check deleted metrics form the test ns #4159
Conversation
✅ Deploy Preview for kubernetes-sigs-kueue canceled.
|
looks good |
47392a0
to
c68fdf9
Compare
Have removed improvements to the log verbosity from here, will create another PR just for that |
IIUC this PR also aims to improve the error messages which is great! Please post example failure messages when the tests fail by inducing a failure due to invalid asserts. Please also describe what is the actual root cause for the flakes. EDIT: oh I see you moved the error message improvement out of the PR - this is even better to understand what is the purpose of every change. Thanks! |
/retest Created an issue #4160. |
/lgtm |
LGTM label has been added. Git tree hash: 4b0718a932500915e69ee2bfe093df0dc0b3d021
|
Indeed, after de-coupling the change it becomes quite easy to see the culprit. /approve Still, please follow up with the logging improvements. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mimowo, mykysha The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind failing-test
/kind flake
What this PR does / why we need it:
Strictly check the deleted metrics from the test environment. This way the test will target only the metrics from the current test run, and ignore the metrics that might not have been cleared previously.
Which issue(s) this PR fixes:
Part of #4139
Special notes for your reviewer:
Does this PR introduce a user-facing change?