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

Update k8s tests and libraries to v1.23.0-rc.0 #18008

Merged
merged 2 commits into from Nov 26, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented Nov 25, 2021

Signed-off-by: André Martins andre@cilium.io

Add support for k8s 1.23.0

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 25, 2021
@aanm aanm added the release-note/major This PR introduces major new functionality to Cilium. label Nov 25, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 25, 2021
@nbusseneau nbusseneau self-requested a review November 25, 2021 13:45
@aanm aanm force-pushed the pr/update-k8s-version branch 2 times, most recently from cedad81 to 3e03fd3 Compare November 25, 2021 17:05
Copy link
Member

@nbusseneau nbusseneau left a comment

Choose a reason for hiding this comment

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

Code changes LGTM. I am making changes to Jenkins to add 1.23 and enable testing via /test-1.23-4.9.

Once this is actually merged or close to merging (depending if we want to test the rotated jobs before or after merging), we need to rotate the /test (master branch) Jenkins triggers and Jenkins pipelines. This entails:

  • Adding Kernel 4.9 / K8s 1.23 testing
  • Testing Kernel 4.19 on 1.22 instead of 1.21
  • Testing Kernel 5.4 on 1.21 instead of 1.20
  • Rolling back 1.21 to 4.9 kernel
  • Updating the Table of Truth:tm:
  • Update MLH config
  • After updating triggers on Jenkins, trigger /test at least once so that the jobs get registered on GH, then update GH PR status marked as required.

Please ping me on Slack when merging 🚀

@nbusseneau
Copy link
Member

/test-1.23-4.9

@aanm
Copy link
Member Author

aanm commented Nov 25, 2021

/test-1.23-4.9

Copy link
Member

@nbusseneau nbusseneau left a comment

Choose a reason for hiding this comment

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

Missed in previous review:

  • Updating supported K8s versions in Documentation/contributing/testing/e2e.rst.
  • Updating default K8s version in jenkinsfiles/ginkgo-kernel.Jenkinsfile.

Documentation/contributing/testing/e2e.rst Outdated Show resolved Hide resolved
@aanm
Copy link
Member Author

aanm commented Nov 25, 2021

/test-1.23-4.9

@aanm aanm force-pushed the pr/update-k8s-version branch 2 times, most recently from c395a60 to 7bc89fa Compare November 25, 2021 23:35
@aanm
Copy link
Member Author

aanm commented Nov 25, 2021

/test-1.23-4.9

@aanm
Copy link
Member Author

aanm commented Nov 26, 2021

/test-upstream

@aanm
Copy link
Member Author

aanm commented Nov 26, 2021

/test

Job 'Cilium-PR-K8s-1.21-kernel-4.19' hit: #17881 (98.18% similarity)

@aanm aanm marked this pull request as ready for review November 26, 2021 01:26
@aanm aanm requested review from a team as code owners November 26, 2021 01:26
@aanm aanm requested a review from a team November 26, 2021 01:26
@aanm aanm requested a review from a team as a code owner November 26, 2021 01:26
@aanm
Copy link
Member Author

aanm commented Nov 26, 2021

Fixed all comments mentioned by Quentin. Doesn't require a new CI run. Merging since previously it was all green

@aanm aanm merged commit bb6ef27 into cilium:master Nov 26, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Nov 26, 2021
@qmonnet qmonnet mentioned this pull request Nov 26, 2021
12 tasks
nbusseneau added a commit to nbusseneau/cilium that referenced this pull request Nov 26, 2021
Following merge of cilium#18008, we now support K8s 1.22 and have rotated the
Jenkins test jobs as follow:

- Changed: Kernel 4.9 testing on K8s 1.23 (instead of 1.22)
- Changed: Kernel 4.19 testing on K8s 1.22 (instead of 1.21)
- Changed: Kernel 5.4 testing on K8s 1.21 (instead of 1.20)
- Added: Kernel 4.9 testing on K8s 1.21

See the Table of Truth:tm: for up to date status on all trigger phrases:
https://docs.google.com/spreadsheets/d/1TThkqvVZxaqLR-Ela4ZrcJ0lrTJByCqrbdCjnI32_X0/edit#gid=0

Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.11.0 Nov 26, 2021
aanm pushed a commit that referenced this pull request Nov 26, 2021
Following merge of #18008, we now support K8s 1.22 and have rotated the
Jenkins test jobs as follow:

- Changed: Kernel 4.9 testing on K8s 1.23 (instead of 1.22)
- Changed: Kernel 4.19 testing on K8s 1.22 (instead of 1.21)
- Changed: Kernel 5.4 testing on K8s 1.21 (instead of 1.20)
- Added: Kernel 4.9 testing on K8s 1.21

See the Table of Truth:tm: for up to date status on all trigger phrases:
https://docs.google.com/spreadsheets/d/1TThkqvVZxaqLR-Ela4ZrcJ0lrTJByCqrbdCjnI32_X0/edit#gid=0

Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@joestringer joestringer added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Dec 3, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.11 in 1.11.0 Dec 3, 2021
tklauser added a commit to cilium/cilium-cli that referenced this pull request Dec 7, 2021
This pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
tklauser added a commit to cilium/cilium-cli that referenced this pull request Dec 7, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
tklauser added a commit to cilium/cilium-cli that referenced this pull request Dec 7, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
michi-covalent pushed a commit to cilium/cilium-cli that referenced this pull request Dec 7, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
tklauser added a commit to cilium/hubble that referenced this pull request Dec 8, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)).

Signed-off-by: Tobias Klauser <tobias@cilium.io>
tklauser added a commit to cilium/hubble that referenced this pull request Dec 8, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)).

Signed-off-by: Tobias Klauser <tobias@cilium.io>
tklauser added a commit to cilium/hubble that referenced this pull request Dec 8, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)).

Signed-off-by: Tobias Klauser <tobias@cilium.io>
rolinh pushed a commit to cilium/hubble that referenced this pull request Dec 8, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)).

Signed-off-by: Tobias Klauser <tobias@cilium.io>
rolinh pushed a commit to cilium/hubble that referenced this pull request Dec 8, 2021
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)).

Signed-off-by: Tobias Klauser <tobias@cilium.io>
aditighag pushed a commit to aditighag/cilium-cli that referenced this pull request Apr 21, 2023
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium/cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
michi-covalent pushed a commit to michi-covalent/cilium that referenced this pull request May 30, 2023
This also pulls in k8s 1.23 (tagged as v0.24.0-alpha.0, see
cilium#18008 (comment)) and
thus requires some changes to corev1.Probe and corev1.Lifecycle
instantiations.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. release-note/major This PR introduces major new functionality to Cilium.
Projects
No open projects
1.11.0
Backport done to v1.11
Development

Successfully merging this pull request may close these issues.

None yet

7 participants