Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update k8s #1324

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.

Sign up for GitHub

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

Jump to bottom
Merged
runcom merged 1 commit into from
Feb 22, 2018
Merged

Update k8s #1324

runcom merged 1 commit into from
Feb 22, 2018

Conversation

@mrunalp
Copy link
Member

@mrunalp mrunalp commented Feb 12, 2018

This PR updates k8s dependency to latest master.
I have created a separate commit for non-vendor changes for easier review which can be squashed before merging.

@mrunalp mrunalp requested a review from runcom as a code owner February 12, 2018 20:15
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 12, 2018
@rhatdan
Copy link
Contributor

rhatdan commented Feb 12, 2018

LGTM

@mrunalp mrunalp force-pushed the update_k8s branch 2 times, most recently from 656e241 to 2d3a70f Compare February 12, 2018 23:55
@mrunalp
Copy link
Member Author

mrunalp commented Feb 13, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 13, 2018

/test integration_rhel

@mrunalp
Copy link
Member Author

mrunalp commented Feb 13, 2018

/test e2e

1 similar comment
@rhatdan
Copy link
Contributor

rhatdan commented Feb 13, 2018

/test e2e

runcom
runcom reviewed Feb 13, 2018
View reviewed changes
vendor.conf Outdated
@@ -1,4 +1,4 @@
k8s.io/kubernetes a48f11c2257d84b0bec89864025508b0ef626b4f https://github.com/kubernetes/kubernetes
k8s.io/kubernetes 7488d1c9210e60aef9ad49f07cb5d8a24152db88 https://github.com/kubernetes/kubernetes
k8s.io/client-go master https://github.com/kubernetes/client-go
Copy link
Member

@runcom runcom Feb 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very likely we need to bump everything related to k8s as well

Copy link
Member Author

@mrunalp mrunalp Feb 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did update most of these. Let us switch back to commit ids. I'll update the PR.

@mrunalp mrunalp force-pushed the update_k8s branch 2 times, most recently from 64905df to 6a937d0 Compare February 13, 2018 19:14
@mrunalp
Copy link
Member Author

mrunalp commented Feb 13, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 13, 2018

Networking related tests are failing. Investigating.

@mrunalp
Copy link
Member Author

mrunalp commented Feb 14, 2018

/retest

3 similar comments
@mrunalp
Copy link
Member Author

mrunalp commented Feb 14, 2018

/retest

@mrunalp
Copy link
Member Author

mrunalp commented Feb 14, 2018

/retest

@mrunalp
Copy link
Member Author

mrunalp commented Feb 15, 2018

/retest

@mrunalp
Copy link
Member Author

mrunalp commented Feb 17, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 17, 2018

/test e2e_rhel

@mrunalp
Copy link
Member Author

mrunalp commented Feb 17, 2018

/test all

@mrunalp mrunalp changed the title WIP: Update k8s Update k8s Feb 17, 2018
@mrunalp
Copy link
Member Author

mrunalp commented Feb 17, 2018

/retest

1 similar comment
@mrunalp
Copy link
Member Author

mrunalp commented Feb 18, 2018

/retest

@mrunalp
Copy link
Member Author

mrunalp commented Feb 20, 2018

/test e2e

@mrunalp
Copy link
Member Author

mrunalp commented Feb 20, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 20, 2018

/test e2e

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test e2e_rhel

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test all

@mrunalp
Update code for latest k8s
5f7ac28 
Signed-off-by: Mrunal Patel <[email protected]>
@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test e2e_rhel

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test all

@mrunalp
Copy link
Member Author

mrunalp commented Feb 21, 2018

/test e2e_rhel

@mrunalp
Copy link
Member Author

mrunalp commented Feb 22, 2018

@runcom This is green!

@runcom runcom merged commit 1c54023 into cri-o:master Feb 22, 2018
@wking wking mentioned this pull request Feb 24, 2018
Closed
@wking wking mentioned this pull request Mar 6, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@runcom runcom runcom left review comments

Assignees

No one assigned

Labels

size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mrunalp @rhatdan @runcom @k8s-ci-robot