On 1.7.2014 13:51, Martin Basti wrote:
On Tue, 2014-07-01 at 12:23 +0200, Petr Spacek wrote:
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.
Make DNSSEC subsections per IPA version is IMO better, than change
warnings in code per release.
I thought that this warning will go away completely in the next version.
Please keep in mind that we need to keep the link alive 'forever'. For this
reason I would like to have version number in the URL so we don't have to mix
instructions for IPA 4.0, 4.1, 4.2, 5.0 etc. on single page.
Another option is to put the page into /usr/share/doc... and serve it through
local Apache.
--
Petr^2 Spacek
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
