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

mlh: update Jenkins jobs following 1.23 support #18028

Merged
merged 1 commit into from
Nov 26, 2021

Conversation

nbusseneau
Copy link
Member

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

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>
@nbusseneau nbusseneau added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Nov 26, 2021
@nbusseneau nbusseneau requested a review from aanm November 26, 2021 17:46
@nbusseneau nbusseneau requested a review from a team as a code owner November 26, 2021 17:46
@nbusseneau
Copy link
Member Author

nbusseneau commented Nov 26, 2021

/test

Job 'Cilium-PR-K8s-1.22-kernel-4.19' has 1 failure but they might be new flake since it also hit 1 known flake: #17881 (97.29)

Job 'Cilium-PR-K8s-GKE' failed and has not been observed before, so may be related to your PR:

Click to show.

Test Name

K8sDatapathConfig Host firewall Check connectivity with IPv6 disabled

Failure Output

FAIL: Pods are not ready in time: timed out waiting for pods with filter  to be ready: 4m0s timeout expired

If it is a flake, comment /mlh new-flake Cilium-PR-K8s-GKE so I can create a new GitHub issue to track it.

@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Nov 26, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from master in 1.11.0 Nov 26, 2021
Copy link
Member

@aanm aanm left a comment

Choose a reason for hiding this comment

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

This will be backported to 1.11 as is, right?

@nbusseneau
Copy link
Member Author

nbusseneau commented Nov 26, 2021

Not as-is, but needs manual backporting of similar changes (after we merge #18027). Re-adding label :)

Note to Top Hat: we added the backport label to not forget about it, but the changes from this PR will not apply cleanly on 1.11. First, we need #18027 merged (to pick up changes from #18008 on 1.11), then we need manual actions on Jenkins (that's on me) and then we need similar changes made on 1.11 MLH config (I can do it right after Jenkins, that'll be easier). Just a heads up that you don't need to backport it.

@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.11.0 Nov 26, 2021
@nbusseneau nbusseneau added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 26, 2021
@aanm aanm merged commit 93d4a62 into cilium:master Nov 26, 2021
@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 30, 2021
@nbusseneau nbusseneau added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels Nov 30, 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 Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.11.0
Backport done to v1.11
Development

Successfully merging this pull request may close these issues.

None yet

2 participants