-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Fix kubectl create to create to all resources in a url. #22009
Fix kubectl create to create to all resources in a url. #22009
Conversation
@kubernetes/rh-cluster-infra |
Labelling this PR as size/S |
GCE e2e build/test passed for commit f4dd51479f0bc8ebf8003384f6a0b73250e338f2. |
The author of this PR is not in the whitelist for merge, can one of the admins add the 'ok-to-merge' label? |
it seems it needs LGTM too. |
cc @kubernetes/kubectl |
@smarterclayton Do you have time to take a quick look? |
Needs a test |
f4dd514
to
c1ea4d6
Compare
Updated it with a test case. |
Labelling this PR as size/M |
GCE e2e build/test passed for commit c1ea4d645eb96ef1f63759e0b922b0f369537b36. |
@aveshagarwal can you also exercise create in test-cmd.sh? |
c1ea4d6
to
ad6dfa0
Compare
Added a test case to hack/test-cmd.sh and updated PR. |
GCE e2e build/test failed for commit ad6dfa0. Please reference the list of currently known flakes when examining this failure. If you request a re-test, you must reference the issue describing the flake. |
Flakes:
|
GCE e2e build/test passed for commit ad6dfa0. |
LGTM |
@bgrant0607 please tag this as lgtm |
@Kargakis Done. Thanks for the review. |
Fix kubectl create to create to all resources in a url.
Fixes #18751 .