On 28.7.2016 16:55, Petr Spacek wrote:
On 28.7.2016 16:44, Jan Cholasta wrote:
On 28.7.2016 16:37, Petr Spacek wrote:
On 28.7.2016 16:35, Jan Cholasta wrote:
On 28.7.2016 16:20, Petr Spacek wrote:
install: Call hostnamectl set-hostname only if --hostname option is used
This commit also splits hostname backup and configuration into two separate
functions. This allows us to backup hostname without setting it at the
Note that you can set ca_host in cfg unconditionally, the value is only valid
during install and is not written anywhere.
I prefer not to set it so the code blows up when we attempt to touch variables
we should reference in particular setups. I.e. Take this as a first step
towards api.env without invalid values :-)
OK. Use the proper condition then ("if setup_ca:").
Oh, I'm probably blind. Here is revised version.
But you used "if not setup_ca:" rather than "if setup_ca:" :-)
(In my stash I have a patch which removes nonsense defaults from
ipalib.constants. To be pushed when we a new git branch for 4.4...)
server-install: Fix --hostname option to always override api.env values
Attempts to compare local hostname with user-provided values are error
prone as we found out in #5794. This patch removes comparison and makes
the env values deterministic.
Jan, this patch set should fix problems you have seen in containers.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code