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

install_check: require IPv6 stack to be enabled #545

Closed

Conversation

tkrizek
Copy link
Contributor

@tkrizek tkrizek commented Mar 7, 2017

Add checks to install and replica install to verify IPv6 stack
is enabled. IPv6 is required by some IPA parts (AD, conncheck, ...).

https://pagure.io/freeipa/issue/6608

edit: please appy to ipa-4-4 and master

@tkrizek tkrizek force-pushed the t6608-check-ipv6-stack-enabled branch from 5851ab9 to 82068f1 Compare March 7, 2017 14:18
@abbra
Copy link
Contributor

abbra commented Mar 7, 2017

how the /proc check going to play with containers?

@tkrizek
Copy link
Contributor Author

tkrizek commented Mar 7, 2017

We tested it with @MartinBasti and /proc is mounted in container.

@HonzaCholasta
Copy link
Contributor

Nitpick: the "install_check" prefix in the subject does not say anything, there are currently 7 install_checks in IPA. Please use something more descriptive, such as "server install".

Add checks to install and replica install to verify IPv6 stack
is enabled. IPv6 is required by some IPA parts (AD, conncheck, ...).

https://pagure.io/freeipa/issue/6608
@tkrizek tkrizek force-pushed the t6608-check-ipv6-stack-enabled branch from 82068f1 to c6da7b9 Compare March 8, 2017 09:08
@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Mar 9, 2017
@tkrizek
Copy link
Contributor Author

tkrizek commented Mar 9, 2017

master:

  • ecb4503 server install: require IPv6 stack to be enabled

@tkrizek tkrizek added the pushed Pull Request has already been pushed label Mar 9, 2017
@tkrizek tkrizek closed this Mar 9, 2017
@tkrizek
Copy link
Contributor Author

tkrizek commented Mar 9, 2017

I had to rebase for ipa-4-4: #562

@tkrizek tkrizek deleted the t6608-check-ipv6-stack-enabled branch September 15, 2017 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
4 participants