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

helm, ipam, virt: allow persistent ips on helm based installations #4322

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maiqueb
Copy link
Contributor

@maiqueb maiqueb commented Apr 30, 2024

- What this PR does and why is it needed

- Special notes for reviewers

- How to verify it

- Description for the changelog

@maiqueb
Copy link
Contributor Author

maiqueb commented Apr 30, 2024

@flavio-fernandes o/

I still didn't test this locally since the kind-helm script depends on docker (I use podman ...).

I'd still appreciate if you could point out something visibly broken.

`docker build` is an alias for `docker buildx build`, which does not
automatically load the generated images into `docker images` - for that,
it needs to use the `--load` or `--output=type=docker` options.

When explicitly doing `kind load docker-image`, the image must be
available on docker images.

Signed-off-by: Miguel Duarte Barroso <[email protected]>
@tssurya tssurya added feature/multi-networks Issues related to secondary networks, L3, L2, localnet installation Issues related to installing ovn-kubernetes CNI labels Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/multi-networks Issues related to secondary networks, L3, L2, localnet installation Issues related to installing ovn-kubernetes CNI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants