[Freeipa-devel] [PATCH 0411] upgrade: log to ipaupgrade.log if ipa is not installed

[Martin Kosek]

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)