[Freeipa-devel] [PATCH 0434] log: add timestamp to filename of logs

[Martin Kosek]

On 03/11/2016 09:55 AM, Jan Cholasta wrote:
> On 11.3.2016 09:33, Martin Kosek wrote:
>> On 03/08/2016 07:07 PM, Martin Basti wrote:
>>>
>>>
>>> On 08.03.2016 16:37, Martin Basti wrote:
>>>>
>>>>
>>>> On 08.03.2016 16:31, Martin Basti wrote:
>>>>> https://fedorahosted.org/freeipa/ticket/4501
>>>>>
>>>>> Patch attached.
>>>>>
>>>>>
>>>> Rebased patch attached.
>>>>
>>>>
>>>
>>> self-NACK
>>>
>>> Scripts print to CLI unformatted strings, it should not be so easy.
>>> See /var/log/ipaupgrade-{timestamp}.log for more information
>>
>> second-NACK. We cannot break existing log file paths. The paths are mentioned
>> in a documentation and there may be also automation around that (gathering log
>> files). So there should be always symlink from the well known location to the
>> newest timestampe'd log.
> 
> Sorry, but this is absurd. What's the point of maintaining backward
> compatibility with obsolete documentation? Following this logic, we would not
> be able to change anything ever. What we should actually do is update the
> documentation. Ditto for automation.

+1 for updating the automation and documentation. But some backward
compatibility will need to be retained, at least for the stable systems like
RHEL where *other* people may have some automation or documentation around it,
not just us.