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

Fix bug when getting new RC of a deployment #18331

Merged
merged 1 commit into from
Dec 10, 2015

Conversation

janetkuo
Copy link
Member

@janetkuo janetkuo commented Dec 8, 2015

Similar to #18321.
cc @nikhiljindal @lavalamp @kubernetes/sig-config

@janetkuo janetkuo added kind/bug Categorizes issue or PR as related to a bug. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. area/app-lifecycle labels Dec 8, 2015
@janetkuo janetkuo added this to the v1.2 milestone Dec 8, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/XS

@k8s-github-robot k8s-github-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 8, 2015
@nikhiljindal
Copy link
Contributor

Sorry I have made the same mistake everywhere :)
Thanks for fixing these.

We should also cherrypick these to release-1.1.

@nikhiljindal nikhiljindal added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 8, 2015
@janetkuo
Copy link
Member Author

janetkuo commented Dec 8, 2015

@nikhiljindal okay I will cherrypick both. :)

@janetkuo janetkuo assigned nikhiljindal and unassigned lavalamp Dec 8, 2015
@k8s-bot
Copy link

k8s-bot commented Dec 8, 2015

GCE e2e build/test failed for commit 72bd4e8.

@janetkuo
Copy link
Member Author

janetkuo commented Dec 8, 2015

@k8s-bot e2e test this

@k8s-bot
Copy link

k8s-bot commented Dec 8, 2015

GCE e2e test build/test passed for commit 72bd4e8.

@k8s-github-robot
Copy link

@k8s-bot test this

Tests are more than 48 hours old. Re-running tests.

@k8s-bot
Copy link

k8s-bot commented Dec 10, 2015

GCE e2e test build/test passed for commit 72bd4e8.

nikhiljindal added a commit that referenced this pull request Dec 10, 2015
Fix bug when getting new RC of a deployment
@nikhiljindal nikhiljindal merged commit 9d1f07d into kubernetes:master Dec 10, 2015
j3ffml added a commit that referenced this pull request Dec 11, 2015
…31-upstream-release-1.1

Automated cherry pick of #18331 upstream release 1.1
@janetkuo janetkuo added the kind/flake Categorizes issue or PR as related to a flaky test. label Mar 7, 2016
shyamjvs pushed a commit to shyamjvs/kubernetes that referenced this pull request Dec 1, 2016
…ck-of-#18331-upstream-release-1.1

Automated cherry pick of kubernetes#18331 upstream release 1.1
shouhong pushed a commit to shouhong/kubernetes that referenced this pull request Feb 14, 2017
…ck-of-#18331-upstream-release-1.1

Automated cherry pick of kubernetes#18331 upstream release 1.1
openshift-publish-robot pushed a commit to openshift/kubernetes that referenced this pull request Jan 31, 2018
Automatic merge from submit-queue (batch tested with PRs 18225, 18351, 18331, 18340, 18326).

UPSTREAM: 57854: fix bug of swallowing missing merge key error

fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1529659

Origin-commit: db03c4980ab2a7c4d1cd85ca0cf9bb1b3e5411d7
openshift-publish-robot pushed a commit to openshift/kubernetes that referenced this pull request Feb 27, 2018
Automatic merge from submit-queue (batch tested with PRs 18225, 18351, 18331, 18340, 18326).

UPSTREAM: 57854: fix bug of swallowing missing merge key error

fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1529659

Origin-commit: db03c4980ab2a7c4d1cd85ca0cf9bb1b3e5411d7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/app-lifecycle kind/bug Categorizes issue or PR as related to a bug. kind/flake Categorizes issue or PR as related to a flaky test. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants