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

replica prepare: fix wrong IPA CA nickname in replica file #622

Closed
wants to merge 1 commit into from
Closed

replica prepare: fix wrong IPA CA nickname in replica file #622

wants to merge 1 commit into from

Conversation

HonzaCholasta
Copy link
Contributor

@HonzaCholasta HonzaCholasta commented Mar 20, 2017

Lookup IPA CA subject and pass it to CertDB when creating dscert.p12 and
httpcert.p12, otherwise a generic nickname will be used for the IPA CA
certificate instead of "$REALM IPA CA".

This fixes replica install on domain level 0 from a replica file created
using ipa-replica-install on IPA 4.5.

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

Lookup IPA CA subject and pass it to CertDB when creating dscert.p12 and
httpcert.p12, otherwise a generic nickname will be used for the IPA CA
certificate instead of "$REALM IPA CA".

This fixes replica install on domain level 0 from a replica file created
using ipa-replica-install on IPA 4.5.

https://pagure.io/freeipa/issue/6777
@martbab martbab self-assigned this Mar 20, 2017
@martbab
Copy link
Contributor

martbab commented Mar 22, 2017

I ran into some issues with CA deployment, but they vanished after re-provisioning my test env. Otherwise it works.

@martbab martbab added the ack Pull Request approved, can be merged label Mar 22, 2017
@martbab
Copy link
Contributor

martbab commented Mar 22, 2017

master:

  • 9939aa5 replica prepare: fix wrong IPA CA nickname in replica file
    ipa-4-5:

  • df60e88 replica prepare: fix wrong IPA CA nickname in replica file

@martbab martbab added the pushed Pull Request has already been pushed label Mar 22, 2017
@martbab martbab closed this Mar 22, 2017
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