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

[Martin Basti]

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.