Skip to content

Conversation

brlbil
Copy link
Contributor

@brlbil brlbil commented Oct 24, 2024

This PR adds Ariane extra-args to run all GKE versions with the argument versions=all.
Normally, these run only on scheduled workflow runs.
Also, this PR adds an argument for the GKE channel, if channel= versions given it
is filtered for that channel, and channel=none disables filtering. If no argument is given the default channel REGULAR is used.

The purpose of adding versions=all is to make it easy to test the running of all versions,
and the purpose of adding channel= is that scheduled tests fail without filtering versions with stable channels, especially during version updates on GKE. But at the same time, we want to keep the ability to run tests without channel filtering too.

Runs with arguments:
versions=all https://github.com/cilium/cilium/actions/runs/11515943731
channel=stable https://github.com/cilium/cilium/actions/runs/11515905320

@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 Oct 24, 2024
@brlbil brlbil added area/CI Continuous Integration testing issue or flake area/CI-improvement Topic or proposal to improve the Continuous Integration workflow ci/flake This is a known failure that occurs in the tree. Please investigate me! release-note/ci This PR makes changes to the CI. labels Oct 24, 2024
@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 Oct 24, 2024
@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from 143ee3d to 781b6b3 Compare October 24, 2024 11:52
@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-external-workloads {"runAllVersion":"yes"}

@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from 781b6b3 to 8117068 Compare October 24, 2024 11:56
@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-external-workloads {"runAllVersion":"yes"}

@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-external-workloads runAllVersions

@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-external-workloads

@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from 8117068 to bdeaefb Compare October 24, 2024 13:32
@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-external-workloads channel=stable

@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from bdeaefb to 668f5bd Compare October 24, 2024 14:49
@brlbil
Copy link
Contributor Author

brlbil commented Oct 24, 2024

/ci-gke channel=stable

@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch 2 times, most recently from 00f1a46 to 59c7b0d Compare October 25, 2024 08:03
@brlbil
Copy link
Contributor Author

brlbil commented Oct 25, 2024

/ci-external-workloads channel=stable

@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from 59c7b0d to 557880a Compare October 25, 2024 09:09
@brlbil
Copy link
Contributor Author

brlbil commented Oct 25, 2024

/ci-gke channel=stable

@brlbil
Copy link
Contributor Author

brlbil commented Oct 25, 2024

/ci-external-workloads versions=all

This commit adds Ariane extra-args to run all GKE versions with the argument versions=all.
Normally, these run only on scheduled workflow runs.
Also, this commit adds an argument for the GKE channel, if channel=<name> versions given it
is filtered for that channel, and channel=none disables filtering. If no argument is given the default channel REGULAR is used.

Signed-off-by: Birol Bilgin <birol@cilium.io>
@brlbil brlbil force-pushed the pr/brlbil/gke-version-args branch from 557880a to 1ad192b Compare October 25, 2024 10:32
@brlbil
Copy link
Contributor Author

brlbil commented Oct 25, 2024

/test

@brlbil brlbil marked this pull request as ready for review October 25, 2024 11:04
@brlbil brlbil requested review from a team as code owners October 25, 2024 11:04
@brlbil brlbil requested a review from giorio94 October 25, 2024 11:04
@brlbil brlbil requested review from aanm and nebril October 25, 2024 11:04
@brlbil brlbil added backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. backport/1.15 This PR represents a backport for Cilium 1.15.x of a PR that was merged to main. backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. and removed backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. backport/1.15 This PR represents a backport for Cilium 1.15.x of a PR that was merged to main. backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. labels Oct 25, 2024
@brlbil
Copy link
Contributor Author

brlbil commented Oct 25, 2024

/ci-gke versions=all channel=stable

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 29, 2024
@aanm aanm added this pull request to the merge queue Oct 29, 2024
Merged via the queue into main with commit fc4101d Oct 29, 2024
278 checks passed
@aanm aanm deleted the pr/brlbil/gke-version-args branch October 29, 2024 09:13
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 area/CI-improvement Topic or proposal to improve the Continuous Integration workflow ci/flake This is a known failure that occurs in the tree. Please investigate me! 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
None yet
Development

Successfully merging this pull request may close these issues.

4 participants