-
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
Make KUBE_TEST_ARGS come last, so that -args can be used #107516
Make KUBE_TEST_ARGS come last, so that -args can be used #107516
Conversation
@MikeSpreitzer: 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. |
/cc @gavinfish |
@kubernetes/sig-testing |
/sig testing |
/cc @spiffxp |
71ae53e
to
e3fa01f
Compare
Undoing bot response to fat finger: |
/retest |
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
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
/assign @liggitt |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt, MikeSpreitzer 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 |
Just a small note: this technically means those other env are no longer authoritative versus KUBE_TEST_ARGS (they would have been before since they came last), which IMHO makes more sense, but is technically a sort of breaking change. If anyone's setup is actually broken by this should probably rethink how they set these env though ... |
@BenTheElder : what you are saying is that the hack of putting |
I mean the short and vmodule env / args. before this PR they always came last and would overrode any previous args. Folks should not set these except through those env hopefully |
What type of PR is this?
/kind bug
What this PR does / why we need it:
This PR re-arranges the order in which arguments are presented to
go test
bymake test-integration
, so thatKUBE_TEST_ARGS
can usefully include-args
.Which issue(s) this PR fixes:
Fixes #100611
Special notes for your reviewer:
This is a rebased version of #100657 by @gavinfish
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: