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

[Backport][ipa-4-7] Keep Dogtag's client db in external CA step 1 #2498

Closed
wants to merge 2 commits into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Oct 26, 2018

This PR was opened automatically because PR #2497 was pushed to master and backport to ipa-4-7 is required.

Don't remove /root/.dogtag/pki-tomcat when performing step 1 of external
CA installation process. Dogtag 10.6.7 changed behavior and no longer
re-creates the client database in step 2.

Fixes: https://pagure.io/freeipa/issue/7742
Signed-off-by: Christian Heimes <cheimes@redhat.com>
The install_master() function performs additional steps besides just
installing a server. It also sets up log collection and performs
additional tests.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran
Copy link
Member Author

tiran commented Oct 26, 2018

PR was ACKed automatically because this is backport of PR #2497. Wait for CI to finish before pushing. In case of questions or problems contact @tiran who is author of the original PR.

@tiran tiran added the ack Pull Request approved, can be merged label Oct 26, 2018
@tiran
Copy link
Member Author

tiran commented Oct 26, 2018

ipa-4-7:

  • 78bf80e Keep Dogtag's client db in external CA step 1
  • 6214fc5 Use tasks.install_master() in external_ca tests

@tiran tiran added the pushed Pull Request has already been pushed label Oct 26, 2018
@tiran tiran closed this Oct 26, 2018
@tiran tiran deleted the backport_pr2497_ipa-4-7 branch March 29, 2019 09:09
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
Development

Successfully merging this pull request may close these issues.

1 participant