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.

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

vendor: Update k8s dependencies and tests to 1.21.3 #16608

Merged
merged 1 commit into from Jul 23, 2021
Merged

vendor: Update k8s dependencies and tests to 1.21.3 #16608

merged 1 commit into from Jul 23, 2021

Conversation

christarazi
Copy link
Member

@christarazi christarazi commented Jun 21, 2021 •
edited

Also updates the k8s version for other branches to 1.18.20, 1.19.13, and
1.20.9.

Signed-off-by: Chris Tarazi chris@isovalent.com

@christarazi christarazi added sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. needs-backport/1.10 release-note/misc This PR makes changes that have no direct user impact. labels Jun 21, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jun 21, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.2 Jun 21, 2021
@christarazi

This comment has been minimized.

Sign in to view
@christarazi christarazi marked this pull request as ready for review June 21, 2021 19:17
@christarazi christarazi requested review from a team as code owners June 21, 2021 19:17
@christarazi christarazi requested a review from nebril June 21, 2021 19:17
@christarazi christarazi requested a review from rolinh June 21, 2021 19:17
@christarazi

This comment has been minimized.

Sign in to view
@christarazi

This comment has been minimized.

Sign in to view
@christarazi

This comment has been minimized.

Sign in to view
@christarazi

This comment has been minimized.

Sign in to view
@christarazi

This comment has been minimized.

Sign in to view
@aanm aanm added this to Needs backport from master in 1.10.3 Jul 2, 2021
@aanm aanm removed this from Needs backport from master in 1.10.2 Jul 2, 2021
@christarazi
Copy link
Member Author

christarazi commented Jul 7, 2021 •
edited

test-me-please

Edit:

@aanm aanm added this to Needs backport from master in 1.10.4 Jul 15, 2021
@aanm aanm removed this from Needs backport from master in 1.10.3 Jul 15, 2021
@aanm aanm added the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label Jul 19, 2021
@aanm
Copy link
Member

aanm commented Jul 19, 2021

@christarazi a new k8s version has been released so it might make sense to upgrade to that one and rebase the PR against master.

@christarazi
vendor: Update k8s dependencies and tests to 1.21.3
4dc0eff 
Also updates the k8s version for other branches to 1.18.20, 1.19.13, and
1.20.9.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi changed the title vendor: Update k8s dependencies and tests to 1.21.2 vendor: Update k8s dependencies and tests to 1.21.3 Jul 19, 2021
@christarazi
Copy link
Member Author

test-me-please

@christarazi christarazi removed the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label Jul 19, 2021
@christarazi christarazi mentioned this pull request Jul 23, 2021
Merged
@christarazi
Copy link
Member Author

christarazi commented Jul 23, 2021 •
edited

Smoke Test with IPv6 / conformance-test-ipv6 flaked by timing out on connectivity check pods timing out: https://github.com/cilium/cilium/pull/16608/checks?check_run_id=3108750036

It's difficult to tell why. I opened #16968 in order to get insights (hubble flows) as to why those pods' connectivity were broken.

Otherwise this is ready to merge.

@christarazi christarazi added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 23, 2021
@aanm aanm merged commit 059c4f5 into cilium:master Jul 23, 2021
@christarazi christarazi deleted the pr/christarazi/k8s-1.21.2 branch July 23, 2021 23:49
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.10.4 Jul 26, 2021
@jibi jibi mentioned this pull request Jul 26, 2021
Merged
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.10 in 1.10.4 Jul 27, 2021
@joestringer joestringer mentioned this pull request Sep 1, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rolinh rolinh rolinh approved these changes

@nebril nebril nebril approved these changes

@aanm aanm aanm approved these changes

Assignees
No one assigned
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers.
Projects
No open projects
1.10.4
Backport done to v1.10
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@christarazi @aanm @rolinh @nebril @jibi