On 06/18/2012 10:43 PM, Rob Crittenden wrote:
Add some additional logging to ipa-upgradeconfig and have it update
/var/log/ipaupgrade.log so we can see what an upgrade has already done.

rob


I don't think the indentation works too well:

2012-06-19T12:52:47Z INFO Verifying that root certificate is published
2012-06-19T12:52:47Z DEBUG   Certificate file exists
2012-06-19T12:52:47Z INFO Verifying that CA proxy configuration is correct
2012-06-19T12:52:47Z DEBUG   No CA detected in /etc/pki-ca
2012-06-19T12:52:47Z INFO Verifying that KDC configuration is using ipa-kdb backend
2012-06-19T12:52:47Z DEBUG   dbmodules already updated in /etc/krb5.conf


It would be nice to add a message right after logging setup saying that ipa-upgradeconfig is being run.


The --debug flag only affects console output, standard_logging_setup always sets the file handler to debug level. There's no need for the flag with >/dev/null.


Otherwise the patch works great.

--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to