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

Release 1.2 cherrypick batch #23024

Merged
merged 8 commits into from
Mar 16, 2016
Merged

Conversation

eparis
Copy link
Contributor

@eparis eparis commented Mar 15, 2016

#22989: Fix non-masquerade CIDR
#22942: Allow up to 39 PVs attached to AWS node
#22969: Disable kubectl validation for upgrade e2e
#22993: Bumped Heapster to stable version 1.0.0
#22948: Separate GetOldRS and GetNewRS in deployment controller (get and mutate) and deployment util (get only)
#23012: Make fullyLabeledReplicas optional in rc status
#23005: Trusty: Fix heapster breakage caused by #22940
#23020: Updating cadvisor deps to v0.22.2

k8s-merge-robot and others added 7 commits March 15, 2016 18:46
…_plus

Auto commit by PR queue bot
(cherry picked from commit d0437a1)
Allow up to 39 PVs attached to AWS node
(cherry picked from commit 8a95274)
Auto commit by PR queue bot
(cherry picked from commit 52b061a)
Bumped Heapster to stable version 1.0.0
(cherry picked from commit 70036b3)
Auto commit by PR queue bot
(cherry picked from commit 4b3e2b5)
Make fullyLabeledReplicas optional in rc status
(cherry picked from commit 6170326)
Auto commit by PR queue bot
(cherry picked from commit 8313504)
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

@k8s-github-robot
Copy link

Labelling this PR as size/XL

@k8s-github-robot k8s-github-robot added kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Mar 15, 2016
@eparis eparis added cla: yes and removed kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Mar 15, 2016
@eparis eparis assigned bgrant0607 and unassigned mikedanese Mar 15, 2016
@eparis eparis added this to the v1.2 milestone Mar 15, 2016
@bgrant0607
Copy link
Member

Thanks @eparis. Looking more carefully this time :-)

@eparis
Copy link
Contributor Author

eparis commented Mar 15, 2016

We've got bots for that :)

Updating cadvisor deps to v0.22.2
(cherry picked from commit 57bd9d4)
@k8s-github-robot
Copy link

Labelling this PR as size/XL

@k8s-github-robot k8s-github-robot added kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Mar 15, 2016
@eparis
Copy link
Contributor Author

eparis commented Mar 16, 2016

and now with cadvisor update

@bgrant0607
Copy link
Member

Looks like we got them this time! Thanks much. LGTM

@bgrant0607 bgrant0607 added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed cla: no labels Mar 16, 2016
@k8s-bot
Copy link

k8s-bot commented Mar 16, 2016

GCE e2e build/test passed for commit 32aaa5a.

@k8s-bot
Copy link

k8s-bot commented Mar 16, 2016

GCE e2e build/test failed for commit fa979b3.

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.

@eparis
Copy link
Contributor Author

eparis commented Mar 16, 2016

[Fail] Kubectl client Kubectl describe [It] should check if kubectl describe prints relevant information for rc and pods [Conformance] 
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/examples.go:506

[Fail] Kubectl client Kubectl expose [It] should create services for rc [Conformance] 
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/examples.go:506

@eparis
Copy link
Contributor Author

eparis commented Mar 16, 2016

@k8s-bot e2e test this flake: #IGNORE
because i think it is a flake, but hard to be sure...

@bgrant0607
Copy link
Member

@eparis Why are we running examples test?

@bgrant0607
Copy link
Member

I guess there are some conformance tests mixed in there.

@k8s-bot
Copy link

k8s-bot commented Mar 16, 2016

GCE e2e build/test passed for commit fa979b3.

@bgrant0607
Copy link
Member

Green now.

@eparis A couple kubectl tests flaked in head around the same time. I can't see the test artifacts yet, though, so I can't determine for certain that it's correlated.

@bgrant0607
Copy link
Member

Merging.

bgrant0607 added a commit that referenced this pull request Mar 16, 2016
Release 1.2 cherrypick batch
@bgrant0607 bgrant0607 merged commit 4a2f57b into kubernetes:release-1.2 Mar 16, 2016
@ncdc ncdc mentioned this pull request Mar 16, 2016
85 tasks
shyamjvs pushed a commit to shyamjvs/kubernetes that referenced this pull request Dec 1, 2016
shouhong pushed a commit to shouhong/kubernetes that referenced this pull request Feb 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants