-
Notifications
You must be signed in to change notification settings - Fork 4.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
rebase to kube 1.11 GA #20164
rebase to kube 1.11 GA #20164
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k 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 |
@liggitt pkg/api tests look suspicious since we went to jsoniter |
/test cmd |
note to self, test-integration is probably failing on the both namespaced and not removal. Try to push through instead of repicking. |
another note to self, default GC quota list |
hmm, yes, those are exactly the things ugorji let in before that bit us when we tried to drop it. I had forgotten, but past me was mean enough to add tests to not let me forget. This requires more thought on what the best path out of this mess is. xref https://bugzilla.redhat.com/show_bug.cgi?id=1525162 |
50577e6
to
999676d
Compare
verify /retest |
/test verify |
4abd6d8
to
95a270e
Compare
…wards compatibility Drop in some future release
95a270e
to
3506509
Compare
Only change is Jordan's. Tagging |
@smarterclayton I've got one more critical backwards compatibility fix that I'll finish tomorrow and you'll be good to upgrade |
Creepy. no changes.