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
proxy: label kube_proxy test with more unique label #95337
Conversation
/cc @aojea |
/retest |
it LGTM but we should use the same name/tag in all tests that are kube-proxy only, so people can filter them out with their regexes, and not have to use a complicated regex @jayunit100 You were exploring this recently, any idea/suggestion? |
@aojea @jayunit100 Please take a look at issue#95368 to identify all the tests we should label for kube-proxy |
8aa6560
to
46f6232
Compare
/retest |
/lgtm |
/retest passing locally with kind |
Sgtm but one tiny change suggestion before we pull the lever: Since This might cause regression in some test suite skip/focus jobs people have though. Is "kube-proxy Networking" a safer string then "kube-proxy" so that we guarantee not Breaking existing focus strings ? No strong opinion either way if y'all wanna merge that's cool, just a thought. |
they should be using [sig-network] , right? |
/retest |
/assign @andrewsykim for approval |
Agree with this, and folks should be using |
🤔 this tag only applies to the 2 test that are contained within it, the [sig-network] tag is added here kubernetes/test/e2e/network/framework.go Lines 22 to 24 in 515f834
|
Good point |
The kube_proxy SIGDescribe previously only had Network in the title and made it more difficult to select just the test cases in the kube_proxy file and would end up running anything with Network in the text area of SIGDescribe e2e tests. Signed-off-by: Christopher M. Luciano <cmluciano@us.ibm.com>
46f6232
to
22b8cd5
Compare
/retest |
1 similar comment
/retest |
/lgtm |
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.
/approve
/lgtm
/triage accepted |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim, cmluciano 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 |
1 similar comment
/retest |
/retest Review the full test history for this PR. Silence the bot with an |
1 similar comment
/retest Review the full test history for this PR. Silence the bot with an |
Signed-off-by: Christopher M. Luciano cmluciano@us.ibm.com
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
The kube_proxy SIGDescribe previously only had Network in the title
and made it more difficult to select just the test cases in the
kube_proxy file and would end up running anything with Network in the
text area of SIGDescribe e2e tests.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: