On 29.01.2016 12:12, Martin Kosek wrote:
On 01/29/2016 10:48 AM, Martin Basti wrote:
Missing record in ipaupgrade.log that upgrade failed because IPA is not
installed, causes harder time to debugging upgrade from log.

Patch attached.
I am thinking that in these general catch-all clauses, it could be also useful
to print the stack trace in the DEBUG log, especially for the cases when the
exception is re-raised to some general one, like here:

     try:
         data_upgrade.create_instance()
...
     except RuntimeError:
         raise RuntimeError('IPA upgrade failed.', 1)

It is already logged to DEBUG log in data_upgrade instance.

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

Reply via email to