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
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
