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 removal of kernel 4.9 support #23822

Merged
merged 1 commit into from Feb 16, 2023

Conversation

nbusseneau
Copy link
Member

We removed support for kernel 4.9 in the CI via
bbda112, as part of ongoing work to bump the minimal supprted kernel version (see 1).

We have rotated / expanded the Jenkins test jobs as follow:

  • Changed: K8s 1.16 on Kernel 4.19 (instead of 4.9, triggered on /test).
  • Changed: K8s 1.17-1.24 on Kernel 4.19 (instead of 4.9, triggered on /test-missed-k8s).

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

We removed support for kernel 4.9 in the CI via
bbda112, as part of ongoing work to
bump the minimal supprted kernel version (see [1]).

We have rotated / expanded the Jenkins test jobs as follow:

- Changed: K8s 1.16 on Kernel 4.19 (instead of 4.9, triggered on `/test`).
- Changed: K8s 1.17-1.24 on Kernel 4.19 (instead of 4.9, triggered on `/test-missed-k8s`).

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

[1]: cilium#22116

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 Feb 16, 2023
@nbusseneau nbusseneau requested a review from a team as a code owner February 16, 2023 14:37
@nbusseneau
Copy link
Member Author

nbusseneau commented Feb 16, 2023

/test

EDIT: /test correctly triggered the expected jobs (i.e. now k8s-1.16-kernel-4.19 instead of k8s-1.16-kernel-4.9).

@nbusseneau
Copy link
Member Author

nbusseneau commented Feb 16, 2023

/test-missed-k8s

EDIT: /test-missed-k8s correctly triggered the expected jobs (i.e. K8s 1.17-1.27 on 4.19 instead of 4.9).

@nbusseneau
Copy link
Member Author

nbusseneau commented Feb 16, 2023

EDIT: it's done.

@aanm @joestringer The following PR checks need to be marked as required in GH settings for master branches:

  • k8s-1.16-kernel-4.19 (test-1.16-4.19)

The following need to be unmarked:

  • k8s-1.16-kernel-4.9 (test-1.16-4.9)

@nbusseneau
Copy link
Member Author

We don't need to wait for any CI run to complete, I triggered the CI only to verify trigger phrases and also have GH pick up the new jobs (so that we can mark them as required), but this change is completely independent of CI and only requires a reviewer's approval.

@nbusseneau nbusseneau requested a review from a team February 16, 2023 14:56
@pchaigno pchaigno merged commit 7d108b9 into cilium:master Feb 16, 2023
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 release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants