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

Centralize configuration of kind version/image in GitHub Action workflows #30916

Merged

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Feb 22, 2024

Let's define kind-related variables (i.e., version, k8s image and k8s version) inside the set-env-variables action. This ensures consistency across workflows, simplify version bumps as well as the introduction of new workflows depending on them. One extra byproduct is that renovate updates will also stop requesting reviews from all the different teams owning each specific workflow. Additionally, let's also stop waiting for kind clusters to become ready, as they will never, because no CNI is present at that point.

Note: a few pull-request based workflows are currently failing, because they use the set-env-variables checked out from the main branch, which doesn't contain the additional variables. This will be fixed after merging this PR.

Centralize configuration of kind version/image in GitHub Action workflows

@giorio94 giorio94 added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Feb 22, 2024
@giorio94
Copy link
Member Author

/test

@giorio94 giorio94 force-pushed the pr/giorio94/main/centralize-kind-image-definition branch from 07682e8 to eb106cf Compare February 23, 2024 09:09
@giorio94 giorio94 added backport/author The backport will be carried out by the author of the PR. needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch kind/cleanup This includes no functional changes. labels Feb 23, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.13.13 Feb 23, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.15.2 Feb 23, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.14.8 Feb 23, 2024
@giorio94 giorio94 force-pushed the pr/giorio94/main/centralize-kind-image-definition branch 2 times, most recently from 7822bc7 to a5d74aa Compare February 23, 2024 10:20
@giorio94 giorio94 marked this pull request as ready for review February 23, 2024 10:24
@giorio94 giorio94 requested review from a team as code owners February 23, 2024 10:24
@giorio94
Copy link
Member Author

/ci-e2e-upgrade

@giorio94
Copy link
Member Author

Converting back to draft. The image seems not to be correctly picked by LVH-based workflows.

@giorio94 giorio94 marked this pull request as draft February 23, 2024 10:57
Let's define kind-related variables (i.e., version, k8s image and k8s
version) inside the set-env-variables action. One all consumers will
have been migrated through the subsequent commit, this will ensure
consistency across workflows, simplify version bumps as well as the
introduction of new workflows depending on them. One extra byproduct
is that renovate updates will also stop requesting reviews from all
the different teams owning each specific workflow.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 mentioned this pull request Mar 6, 2024
3 tasks
@giorio94 giorio94 added backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. and removed needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Mar 6, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.15 in 1.15.2 Mar 6, 2024
@giorio94 giorio94 mentioned this pull request Mar 6, 2024
3 tasks
@giorio94 giorio94 added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Mar 6, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.14 in 1.14.8 Mar 6, 2024
@giorio94 giorio94 mentioned this pull request Mar 6, 2024
3 tasks
@giorio94 giorio94 added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Mar 6, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.13 in 1.13.13 Mar 6, 2024
@jrajahalme jrajahalme added this to Backport pending to v1.15 in 1.15.3 Mar 13, 2024
@jrajahalme jrajahalme removed this from Backport pending to v1.15 in 1.15.2 Mar 13, 2024
@thorn3r thorn3r added this to Backport pending to v1.13 in 1.13.14 Mar 13, 2024
@thorn3r thorn3r removed this from Backport pending to v1.13 in 1.13.13 Mar 13, 2024
@thorn3r thorn3r added this to Backport pending to v1.14 in 1.14.9 Mar 13, 2024
@thorn3r thorn3r removed this from Backport pending to v1.14 in 1.14.8 Mar 13, 2024
@github-actions github-actions bot added backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Mar 16, 2024
@jrajahalme jrajahalme moved this from Backport pending to v1.15 to Backport done to v1.15 in 1.15.3 Mar 26, 2024
@thorn3r thorn3r moved this from Backport pending to v1.13 to Backport done to v1.13 in 1.13.14 Mar 26, 2024
@jrajahalme jrajahalme moved this from Backport pending to v1.14 to Backport done to v1.14 in 1.14.9 Mar 26, 2024
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/author The backport will be carried out by the author of the PR. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. kind/cleanup This includes no functional changes. 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.13.14
Backport done to v1.13
1.14.9
Backport done to v1.14
1.15.3
Backport done to v1.15
Development

Successfully merging this pull request may close these issues.

None yet

9 participants