[Freeipa-devel] [PATCH 0083] Add DNSSEC experimental support warning message

[Petr Spacek]

On 1.7.2014 12:20, Martin Kosek wrote:
> On 07/01/2014 10:55 AM, Petr Spacek wrote:
>> On 1.7.2014 10:49, Petr Viktorin wrote:
>>> On 07/01/2014 10:43 AM, Petr Spacek wrote:
>>>> On 30.6.2014 17:10, Martin Basti wrote:
>>>>> On Mon, 2014-06-30 at 16:57 +0200, Petr Spacek wrote:
>>>>>> On 30.6.2014 14:33, Martin Basti wrote:
>>>>>>> On Mon, 2014-06-30 at 12:49 +0200, Martin Basti wrote:
>>>>>>>> Patch attached.
>>>>>>
>>>>>> It works for me.
>>>>>>
>>>>>> Please change the string little bit, I have realized that we should
>>>>>> ensure
>>>>>> that file permissions are correct:
>>>>>>
>>>>>> chown named: *
>>>>>> chmod u= *
>>>>>>
>>>>>> (the chmod part new)
>>>>>>
>>>>>> Thanks!
>>>>>>
>>>>>
>>>>> Updated patch attached
>>>>
>>>> I'm really sorry, I had to change the message once again :-)
>>>>
>>>> None of us noticed that chmod command was completely incorrect. I'm
>>>> attaching fixed patch as an apology.
>>>>
>>>> It works for me when applied to master
>>>> (50c30c8401c21d43414404bd5caa157196449e4c).
>>>>
>>>> Functional self-ACK :-)
>>>>
>>>> IMHO it can be pushed if Python-review is okay.
>>>
>>> Once again, please define new message classes in messages.py instead of just
>>> using PublicMessage with a custom string.
>>>
>>> Also, these messages will work for console output, but I'm not sure
>>> pre-wrapped text would look good in web UI.
>>> I'm not sold on the idea of giving instructions in warning messages. Would a
>>> link to some documentation be better?
>>
>> Well, the idea was to provide copy&paste instructions directly in the console,
>> not speaking about problems with URLs downstream.
>>
>> If you insist on URL ... here it is:
>> http://www.freeipa.org/page/Releases/4.0.0#Experimental_DNSSEC_Support
>>
>
> Please use something more stable, like
>
> http://www.freeipa.org/page/DNSSEC
>
> which we would use as a gathering place for information about FreeIPA and DNSSEC.

IMHO this particular warning should point to version-specific information.

I'm not opposing to /page/DNSSEC idea in general but this warning should point 
to very specific steps which will be valid only to very specific version of 
FreeIPA.

-- 
Petr^2 Spacek