-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Improved the quality of output when the test fails #105860
Conversation
@NikhilSharmaWe: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@NikhilSharmaWe: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hi @NikhilSharmaWe. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: NikhilSharmaWe The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Can you combine more of these changes into a single PR? A good granularity is to go by OWNERS file. In this case, everything under |
Should I combine this and #105774 PR together. |
Yes, please do. |
Done #105875 |
There are also other places that should be fixed:
|
Sorry, I missed that the other PR is outside of |
The above command tells that how many changes needed, can you tell me that how can I find in which files the changes needed or it has to be done manually. |
|
hey @NikhilSharmaWe Actually I was making the req. changes locally and it would be really helpful if you can please specify which all files you are changing so we can collaborate our work |
@NikhilSharmaWe: @chetak123 makes a good point here. To avoid duplication of efforts we assign issues to people and then no-one else should work on the same thing. I hadn't checked who the assignee is for #105678, sorry. @chetak123: do you want to take the lead on splitting up that issue by OWNERS? I don't think we need sub-issues for that. Just having a comment with a proposed split should be enough. Then others can comment to sign up and you can edit your comment to keep track of who's doing what. |
That would be good |
/ok-to-test |
@NikhilSharmaWe: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@NikhilSharmaWe: there are syntax errors in this PR. |
@NikhilSharmaWe will you be revisiting this PR? or can we close it? |
@dims Apologies for the inconvenience , I will keep better track of my PR's in future. |
What type of PR is this?
Enhancement
What this PR does / why we need it:
Improved the quality of output when the test fails
Which issue(s) this PR fixes:
Part of #105678
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: