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

server install: fix external CA install #318

Closed
wants to merge 1 commit into from
Closed

server install: fix external CA install #318

wants to merge 1 commit into from

Conversation

HonzaCholasta
Copy link
Contributor

@HonzaCholasta HonzaCholasta commented Dec 7, 2016

Replace the dual definitions of domain_name, dm_password and admin_password
knobs in server install with single definitions using the original names
without the 'new_' prefix.

This fixes the options read from the installer option cache in step 2 of
external CA install to use the correct knob names.

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

Replace the dual definitions of domain_name, dm_password and admin_password
knobs in server install with single definitions using the original names
without the 'new_' prefix.

This fixes the options read from the installer option cache in step 2 of
external CA install to use the correct knob names.

https://fedorahosted.org/freeipa/ticket/6392
@flo-renaud flo-renaud self-assigned this Dec 7, 2016
@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Dec 8, 2016
@flo-renaud
Copy link
Contributor

Works as expected.

@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Dec 8, 2016
@MartinBasti
Copy link
Contributor

@MartinBasti MartinBasti closed this Dec 8, 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
3 participants