On 07/21/2016 05:22 PM, Petr Vobornik wrote:
On 07/19/2016 09:27 AM, Petr Vobornik wrote:
On 07/19/2016 08:01 AM, Jan Cholasta wrote:

On 18.7.2016 18:50, Florence Blanc-Renaud wrote:
On 07/15/2016 04:29 PM, Petr Vobornik wrote:
ipa-ca-install said that it used
but in fact it used

This patch unites it to ipaserver-ca-install.log

It was chosen because ipa-ca-install can be also used on
master on CA-less -> CA conversion.

Term "server" is valid for both master and replica.


Looks good to me.

Does not look so good to me, "ipareplica-ca-install.log" is in fact the
original file name used since ipa-ca-install was introduced (in commit
8a32bb3746802a29b2655e4ad2cbbba8481e1eaf), so why the switch to


Ideally it would be ipa-ca-install.log but for backwards compatibility,
let's stick with one which we have. AFAIK the framework(run_script
methodĂș doesn't support switching the log name which is printed in error
message depending on usage. Therefore the universal was chosen -
ipaserver-ca-install.log. It was introduced by your commit

And I see, that I used wrong ticket number in the commit. Correct is

Proper solution might be to rework main and __main__ function in
ipa-ca-install but IMO we spent too much time on this already.

Updated patch attach - it uses the other log.

Correct me if I'm wrong but the ticket URL should be https://fedorahosted.org/freeipa/ticket/6086

Martin^3 Babinsky

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to