Skip to content
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

let kubectl label use PATCH instead of PUT #15651

Merged
merged 1 commit into from
Oct 16, 2015

Conversation

caesarxuchao
Copy link
Member

Similar to #15597, this PR lets kubectl label use PATCH instead of PUT
Fix #15333

We can revert the retry function introduced to test-cmd.sh in #15325 after this one and #15597 are merged.

@bgrant0607 @janetkuo @jlowdermilk

@k8s-github-robot
Copy link

Labelling this PR as size/M

@k8s-github-robot k8s-github-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 14, 2015
@k8s-bot
Copy link

k8s-bot commented Oct 14, 2015

GCE e2e test build/test passed for commit 873c1d28e3b1082cf0b3aff38ba9dfbd950a0f67.

@j3ffml j3ffml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 14, 2015
@caesarxuchao
Copy link
Member Author

@k8s-bot test this please

@k8s-bot
Copy link

k8s-bot commented Oct 15, 2015

GCE e2e test build/test passed for commit 873c1d28e3b1082cf0b3aff38ba9dfbd950a0f67.

@k8s-github-robot
Copy link

PR changed after LGTM, removing LGTM.

@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 15, 2015
@k8s-bot
Copy link

k8s-bot commented Oct 15, 2015

GCE e2e test build/test passed for commit b5a3ab6.

@caesarxuchao caesarxuchao added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 15, 2015
@caesarxuchao
Copy link
Member Author

add back "lgtm" since it's a rebase

@caesarxuchao
Copy link
Member Author

@k8s-oncall @bgrant0607 it seems the merge bot is not working. Shall I go ahead merging this?

@bgrant0607
Copy link
Member

@caesarxuchao It's up to the build cop.

piosz added a commit that referenced this pull request Oct 16, 2015
let kubectl label use PATCH instead of PUT
@piosz piosz merged commit c261325 into kubernetes:master Oct 16, 2015
RiRa12621 added a commit to RiRa12621/kubernetes that referenced this pull request Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants