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
gh/workflows: Use cilium-cli GHA to install CLI exec #25228
Conversation
5286297
to
fbd6c8b
Compare
ba288c8
to
5125666
Compare
bbb556b
to
8f08c42
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding Tobias as reviewer as well.
873bbb0
to
a483cb0
Compare
@@ -72,6 +72,7 @@ env: | |||
k8s_version: 1.23 | |||
# renovate: datasource=github-releases depName=cilium/cilium-cli | |||
cilium_cli_version: v0.13.2 | |||
cilium_cli_ci_version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really need to set this in every workflow? Can't we just keep with.relase-version
when using the action below and rely on the with.ci-version
default value being empty?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's to make sed'ing easier (described in the description). I'd like to avoid setting it too, but didn't come up with any nice solution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Non blocking comment
Removed ready-to-merge pending resolution of #25228 (comment) . |
This reduces not only the boilerplate, but allows with a few cmds to change the CLI vsn to test WIP changes. For example, to use a custom CLI CI build for all workflows: $ CI_BUILD_VSN=foobar # Docker Image Version of CI Cilium CLI $ git grep -l cilium_cli_version: | xargs sed -i 's/cilium_cli_version: .*/cilium_cli_version:/g' $ git grep -l cilium_cli_ci_version: | xargs sed -i 's/cilium_cli_ci_version: .*/cilium_cli_ci_version: $CI_BUILD_VSN/g' Signed-off-by: Martynas Pumputis <m@lambda.lt>
@joestringer The comment has been addressed. |
This reduces not only the boilerplate, but allows with a few cmds to change the CLI vsn to test WIP changes. For example, to use a custom CLI CI build for all workflows:
Depends on cilium/cilium-cli#1563