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

Use only internal network and skip assigning fips #619

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

darmach
Copy link
Contributor

@darmach darmach commented Sep 10, 2024

Added new variable infra_use_floatingip allowing to skip allocation of floating IPs using only infra network for deployment and Magnum communication.

  • infra_use_floatingip defaults to True to retain current defaults
  • abovementioned parameter set to False should be used along with infra_network_id pointing to internal provider network accessible from controllers that is used both for nodes and Magnum internal connectivity. capi_cluster_apiserver_floating_ip might also be set to False in the environment.

Replaces #597

Added new variable `infra_use_floatingip` allowing to skip allocation of
floating IPs using only infra network for deployment and Magnum
communication.

 - infra_use_floatingip defaults to `True` to retain current defaults
 - abovementioned parameter set to `False` should be used along with
   `infra_network_id` pointing to internal provider network accessible
   from controllers that is used both for nodes and Magnum internal
   connectivity. `capi_cluster_apiserver_floating_ip` might also be set
   to `False` in the environment.
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

LGTM

@mkjpryor mkjpryor merged commit 773ad7b into main Sep 10, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants