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

Update CNI to 1.2.0 #23320

Merged
merged 2 commits into from
Jan 26, 2023
Merged

Update CNI to 1.2.0 #23320

merged 2 commits into from
Jan 26, 2023

Conversation

michi-covalent
Copy link
Contributor

[ upstream commit a1046bc ]

Run images/scripts/update-cni-version.sh 1.2.0 to update the CNI version.

Ref: https://github.com/containernetworking/plugins/releases/tag/v1.2.0
Signed-off-by: Michi Mutsuzaki michi@isovalent.com

@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.12 This PR represents a backport for Cilium 1.12.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Jan 25, 2023
@michi-covalent michi-covalent temporarily deployed to release-base-images January 25, 2023 01:24 — with GitHub Actions Inactive
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Jan 25, 2023
@michi-covalent michi-covalent mentioned this pull request Jan 25, 2023
Merged
@michi-covalent michi-covalent added release-note/misc This PR makes changes that have no direct user impact. release-blocker/1.12 This issue will prevent the release of the next version of Cilium. labels Jan 25, 2023
@auto-committer auto-committer bot temporarily deployed to release-base-images January 25, 2023 01:38 Inactive
@michi-covalent
Copy link
Contributor Author

/test-backport-1.12

@michi-covalent
Update CNI to 1.2.0
3256c09 
[ upstream commit a1046bc ]

Run `images/scripts/update-cni-version.sh 1.2.0` to update the CNI
version.

Ref: https://github.com/containernetworking/plugins/releases/tag/v1.2.0
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent temporarily deployed to release-base-images January 25, 2023 07:31 — with GitHub Actions Inactive
images: update cilium-{runtime,builder}
653d312 
Signed-off-by: Cilium Imagebot <noreply@cilium.io>
@auto-committer auto-committer bot temporarily deployed to release-base-images January 25, 2023 07:59 Inactive
@michi-covalent michi-covalent marked this pull request as ready for review January 25, 2023 23:21
@michi-covalent michi-covalent requested review from a team as code owners January 25, 2023 23:21
@michi-covalent
Copy link
Contributor Author

michi-covalent commented Jan 25, 2023 •
edited by maintainer-s-little-helper bot

/test-backport-1.12

Job 'Cilium-PR-K8s-1.17-kernel-4.9' hit: #23264 (98.50% similarity)

Job 'Cilium-PR-K8s-1.20-kernel-4.9' failed:

Click to show.

Test Name

K8sServicesTest Checks E/W loadbalancing (ClusterIP, NodePort from inside cluster, etc) Tests NodePort inside cluster (kube-proxy) with externalTrafficPolicy=Local

Failure Output

FAIL: Request from k8s1 to service tftp://[fd04::12]:30061/hello failed

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.20-kernel-4.9 so I can create one.

@maintainer-s-little-helper maintainer-s-little-helper bot mentioned this pull request Jan 26, 2023
Closed
@michi-covalent
Copy link
Contributor Author

test-1.17-4.9: a known flake: #22474

@michi-covalent
Copy link
Contributor Author

/test-1.17-4.9

@joestringer
Copy link
Member

k8s-1.20-kernel-4.9 hit several failures: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.20-kernel-4.9/1554/

All of these failures seem to point to a sudden loss of DNS. Given that none of the other workflows exhibited this issue, I presume it must either be a problem with the connectivity in the VM when it ran, or otherwise some sort of transient test issue.

@joestringer
Copy link
Member

/test-1.20-4.9

@michi-covalent
Copy link
Contributor Author

michi-covalent commented Jan 26, 2023 •
edited by maintainer-s-little-helper bot

/mlh new-flake Cilium-PR-K8s-1.20-kernel-4.9

👍 created #23369

@michi-covalent
Copy link
Contributor Author

/test-1.20-4.9

@michi-covalent michi-covalent merged commit 328039d into v1.12 Jan 26, 2023
@michi-covalent michi-covalent deleted the pr/michi/backport-cni-112 branch January 26, 2023 09:32
@michi-covalent michi-covalent mentioned this pull request Jan 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joestringer joestringer joestringer approved these changes

@nathanjsweet nathanjsweet Awaiting requested review from nathanjsweet nathanjsweet is a code owner automatically assigned from cilium/api

Assignees
No one assigned
Labels
backport/1.12 This PR represents a backport for Cilium 1.12.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. kind/community-contribution This was a contribution made by a community member. release-blocker/1.12 This issue will prevent the release of the next version of Cilium. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@michi-covalent @joestringer