On 1.7.2014 14:04, Martin Kosek wrote:
On 07/01/2014 01:59 PM, Petr Spacek wrote:
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.

Please do not over-engineer this, we just want to pass couple simple commands.

Could we just add the steps to "$ ipa help dns" online help + to the release
notes + to DNS page on wiki? This should allow people interested in the manual
steps to find the information without requiring us to invent some machinery for
that help.

WebUI users can't run 'ipa help dns' command :-) Maybe it is okay because they will have to open terminal anyway (to generate dnssec keys etc.).

Maybe we can have 'ipa help dnssec' command.

I'm personally okay with that as long as it contains the original text.

--
Petr^2 Spacek

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to