-
Notifications
You must be signed in to change notification settings - Fork 244
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
Commenting project list -o json CI flake #3620
Commenting project list -o json CI flake #3620
Conversation
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
Please triage this flake tho
Codecov Report
@@ Coverage Diff @@
## master #3620 +/- ##
=======================================
Coverage 45.50% 45.50%
=======================================
Files 116 116
Lines 11712 11712
=======================================
Hits 5330 5330
Misses 5862 5862
Partials 520 520 Continue to review full report at Codecov.
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: girishramnani 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
What type of PR is this?
commenting flake test as it was previously
What does does this PR do / why we need it:
I actually uncommented a test scenario through #3546 which is again hitting the same flake as reported in #2117 (Bot closed it). So commenting the spec again and reopening the issue for triage.
Which issue(s) this PR fixes:
Fixes NA
PR acceptance criteria:
Unit test : NA
Integration test : NA
Documentation : NA
How to test changes / Special notes to the reviewer:
Flake scenario is commented as it was.