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

workflows: move cilium_cli_version definition to set-env-variables action #29237

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

jibi
Copy link
Member

@jibi jibi commented Nov 17, 2023

No description provided.

@jibi jibi added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Nov 17, 2023
@jibi jibi force-pushed the pr/jibi/move-cli-version-to-env-var branch from 3369d03 to 0cf809b Compare November 17, 2023 11:26
@jibi jibi requested a review from tklauser November 17, 2023 11:26
@jibi jibi marked this pull request as ready for review November 17, 2023 11:28
@jibi jibi requested review from a team as code owners November 17, 2023 11:28
@jibi jibi requested a review from nbusseneau November 17, 2023 11:28
@jibi jibi force-pushed the pr/jibi/move-cli-version-to-env-var branch from 90199e4 to 0a9da3a Compare November 17, 2023 11:31
.github/renovate.json5 Outdated Show resolved Hide resolved
@jibi jibi force-pushed the pr/jibi/move-cli-version-to-env-var branch 3 times, most recently from 395f3f3 to e77e96d Compare November 17, 2023 12:29
@jibi
Copy link
Member Author

jibi commented Nov 17, 2023

/test

@jibi jibi requested a review from tklauser November 17, 2023 12:41
Copy link
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

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

LGTM with the third commit removed before merge. Thanks!

@jibi
Copy link
Member Author

jibi commented Nov 17, 2023

example of successful run https://github.com/cilium/cilium/actions/runs/6904072712

@jibi jibi force-pushed the pr/jibi/move-cli-version-to-env-var branch from e77e96d to 3825539 Compare November 17, 2023 14:38
@michi-covalent
Copy link
Contributor

i'm picking up this pull request. excited to resolve all these conflicts 🥰

@michi-covalent michi-covalent force-pushed the pr/jibi/move-cli-version-to-env-var branch from 3825539 to 3294907 Compare November 20, 2023 16:34
@michi-covalent michi-covalent force-pushed the pr/jibi/move-cli-version-to-env-var branch from 3294907 to e44562f Compare November 20, 2023 16:43
@michi-covalent
Copy link
Contributor

jibi and others added 2 commits November 20, 2023 17:21
…tion

Co-authored-by: Tobias Klauser <tobias@isovalent.com>
Signed-off-by: Gilberto Bertin <jibi@cilium.io>
Assign .github/renovate.json5 to @cilium/github-sec and
@cilium/ci-structure.

Signed-off-by: Gilberto Bertin <jibi@cilium.io>
Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent force-pushed the pr/jibi/move-cli-version-to-env-var branch from 46bd7d4 to a8ae167 Compare November 20, 2023 17:22
@michi-covalent
Copy link
Contributor

/test

@michi-covalent
Copy link
Contributor

manually setting ready-to-merge label since these failure are expected 🚀 🙏

@michi-covalent michi-covalent added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 20, 2023
@julianwiedmann julianwiedmann merged commit 4274f45 into main Nov 21, 2023
208 of 213 checks passed
@julianwiedmann julianwiedmann deleted the pr/jibi/move-cli-version-to-env-var branch November 21, 2023 06:00
michi-covalent added a commit that referenced this pull request Nov 22, 2023
Define CILIUM_CLI_RELEASE_REPO environment variable in set-env-variables
action, and use that to specify the repository parameter for cilium-cli
action to make it easier for downstream projects to override it.

Ref: #29237

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@michi-covalent michi-covalent added the affects/v1.14 This issue affects v1.14 branch label Nov 22, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 22, 2023
github-merge-queue bot pushed a commit that referenced this pull request Nov 23, 2023
Define CILIUM_CLI_RELEASE_REPO environment variable in set-env-variables
action, and use that to specify the repository parameter for cilium-cli
action to make it easier for downstream projects to override it.

Ref: #29237

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
sayboras pushed a commit that referenced this pull request Nov 25, 2023
[ upstream commit 3f694f8 ]

Define CILIUM_CLI_RELEASE_REPO environment variable in set-env-variables
action, and use that to specify the repository parameter for cilium-cli
action to make it easier for downstream projects to override it.

Ref: #29237

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Signed-off-by: Tam Mach <tam.mach@isovalent.com>
michi-covalent added a commit that referenced this pull request Nov 29, 2023
Looks like #29237 missed tests-e2e-upgrade.yaml.

Fixes: 7267f94 ("workflows: move cilium_cli_version definition to set-env-variables action")

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
michi-covalent added a commit that referenced this pull request Nov 29, 2023
Looks like #29237 missed tests-e2e-upgrade.yaml.

Fixes: 42e1a4a ("workflows: move cilium_cli_version definition to set-env-variables action")

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
github-merge-queue bot pushed a commit that referenced this pull request Nov 30, 2023
Looks like #29237 missed tests-e2e-upgrade.yaml.

Fixes: 42e1a4a ("workflows: move cilium_cli_version definition to set-env-variables action")

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
pjablonski123 pushed a commit to pjablonski123/cilium that referenced this pull request Dec 15, 2023
Define CILIUM_CLI_RELEASE_REPO environment variable in set-env-variables
action, and use that to specify the repository parameter for cilium-cli
action to make it easier for downstream projects to override it.

Ref: cilium#29237

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
pjablonski123 pushed a commit to pjablonski123/cilium that referenced this pull request Dec 15, 2023
Looks like cilium#29237 missed tests-e2e-upgrade.yaml.

Fixes: 42e1a4a ("workflows: move cilium_cli_version definition to set-env-variables action")

Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
@jibi jibi added needs-backport/1.12 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 and removed needs-backport/1.12 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Jan 9, 2024
@jibi jibi mentioned this pull request Jan 11, 2024
9 tasks
@jibi jibi 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 Jan 11, 2024
@github-actions github-actions bot added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects/v1.14 This issue affects v1.14 branch area/CI Continuous Integration testing issue or flake backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. 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

6 participants