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

Fix KRA install tests #136

Closed
wants to merge 3 commits into from
Closed

Fix KRA install tests #136

wants to merge 3 commits into from

Conversation

MartinBasti
Copy link
Contributor

@MartinBasti MartinBasti commented Oct 4, 2016

  • fix replica installation of kra as single step in tests
  • in test_installation test suite with domain level 0, some KRA tests must be skipped because does not work under domain level 0 by design
  • I added KRA tests into replication_layout test suite to test how KRA install works with more replicas and various layouts (needed mainly for domain level 0)

https://fedorahosted.org/freeipa/ticket/6088

@MartinBasti MartinBasti changed the title [WIP] Fix KRA install tests Fix KRA install tests Oct 5, 2016
@pvomacka pvomacka self-assigned this Oct 6, 2016
@MartinBasti
Copy link
Contributor Author

Self NACK, I should not remove tests, I have to fix tasks.py to install replica properly with KRA, because we actually support --setup-kra option

KRA should be tested with warious replication topologies as well, mainly
in domain level 0

https://fedorahosted.org/freeipa/ticket/6088
Currently tests were not testing one-step installation of KRA together
with replica. Adding --setup-kra to replica installation instead of
calling ipa-kra-install after.

https://fedorahosted.org/freeipa/ticket/6088
When master is without KRA installed, on domain level0 we cannot install
KRA replicas, thus this tests must be skipped on DL0

https://fedorahosted.org/freeipa/ticket/6088
@pvomacka pvomacka added the ack Pull Request approved, can be merged label Oct 26, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Oct 26, 2016
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
2 participants