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

templates for dualstack (ipv4/ipv6) control plane #3446

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

templates for dualstack (ipv4/ipv6) control plane #3446

wants to merge 4 commits into from

Conversation

spacefito
Copy link
Contributor

Contains templates for dualstack
Allows for disabling firewall
Adds scripts to generate input model but not deploy to ecp

@spacefito spacefito added the wip label May 22, 2019
Copy link
Contributor

@italiahardik italiahardik left a comment

Choose a reason for hiding this comment

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

Dualstack support is already there but with this change, it seems we are trying to run mysql & rabbitmq services on ipv4. If so, update the commit message with details will be helpful.

Adolfo Duarte and others added 2 commits May 23, 2019 16:44
when ipv6 is being used for management
all services should be place in that network
by default. They should not be placed in the
ipv4 network.
Copy link
Contributor

@nicolasbock nicolasbock left a comment

Choose a reason for hiding this comment

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

Thanks for adding the example input file!

@spacefito spacefito closed this Jun 14, 2019
@spacefito
Copy link
Contributor Author

closed by mistake

@spacefito spacefito reopened this Jun 14, 2019
@spacefito spacefito added the wip label Jun 14, 2019
@spacefito
Copy link
Contributor Author

Making this a WIP again, there is too much changes to the input model templates.
I don't feel confortable at this point. We need to make sure we dont' break other stuff

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants