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

Makefile: use CLI options to set local images for kind-install-cilium-clustermesh #25810

Merged
merged 1 commit into from Jun 8, 2023

Conversation

thorn3r
Copy link
Contributor

@thorn3r thorn3r commented May 31, 2023

Makefile: use CLI options to set local images for kind-install-cilium-clustermesh

make-kind-clustermesh-images uses KIND_ENV to set image repositories for local images to be pushed to kind clusters, however these are not actually used when installing Cilium on the kind clusters in kind-install-cilium-clustermesh.

The hardcoded image overrides have been removed from the helm values for these clusters and replaced with CLI flags to specify using the local images for Agent and Operator.

Fixes: #issue-number

Makefile: use CLI options to set local images for kind-install-cilium-clustermesh

@thorn3r thorn3r requested review from a team as code owners May 31, 2023 23:43
@thorn3r thorn3r requested review from bimmlerd and borkmann May 31, 2023 23:43
@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 May 31, 2023
…mesh

`make-kind-clustermesh-images` uses `KIND_ENV` to set image repositories
for local images to be pushed to kind clusters, however these are not
actually used when installing Cilium on the kind clusters in
`kind-install-cilium-clustermesh`.

The hardcoded image overrides have been removed from the helm values
for these clusters and replaced with CLI flags to specify using the
local images for Agent and Operator.

Signed-off-by: Tim Horner <timothy.horner@isovalent.com>
Copy link
Member

@bimmlerd bimmlerd left a comment

Choose a reason for hiding this comment

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

noice.

@thorn3r thorn3r added the release-note/misc This PR makes changes that have no direct user impact. label Jun 1, 2023
@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 Jun 1, 2023
@thorn3r
Copy link
Contributor Author

thorn3r commented Jun 1, 2023

/test

@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 Jun 8, 2023
@dylandreimerink dylandreimerink merged commit aee97b4 into cilium:main Jun 8, 2023
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants