Re: [Freeipa-devel] [PATCH] 163+164 Fix DNS zone --allow-dynupdate option behavior

2011-11-09 Thread Alexander Bokovoy
On Wed, 09 Nov 2011, Martin Kosek wrote:
 Patch 164 fixes dnszone-mod --allow-dynupdate behavior which  disables
 the zone dynamic updates value whenever a dnszone-mod command is run and
 --allow-dynupdate options is not set.
As discussed on IRC, please rename the CLI option name to 
--dynamic-update.

Otherwise, ACK.

Please also file a ticket for 3.0 for gathering all renames of CLI 
options where we feel they are unclear or unfriendly.

 I introduced a Param.encode() function (patch 163) to our framework to
 help encoding Python native values to LDAP representation more
 effectively. True/False to LDAP's TRUE/FALSE in this case. Encoding
 functions are executed in a server context only.
Ack.

-- 
/ Alexander Bokovoy

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


Re: [Freeipa-devel] [PATCH] 163+164 Fix DNS zone --allow-dynupdate option behavior

2011-11-09 Thread Martin Kosek
On Wed, 2011-11-09 at 16:14 +0200, Alexander Bokovoy wrote:
 On Wed, 09 Nov 2011, Martin Kosek wrote:
  Patch 164 fixes dnszone-mod --allow-dynupdate behavior which  disables
  the zone dynamic updates value whenever a dnszone-mod command is run and
  --allow-dynupdate options is not set.
 As discussed on IRC, please rename the CLI option name to 
 --dynamic-update.
 
 Otherwise, ACK.

Renamed and pushed to master.

Endi, Petr, I created a ticket to implement this new behavior in WebUI
too:

https://fedorahosted.org/freeipa/ticket/2084

 
 Please also file a ticket for 3.0 for gathering all renames of CLI 
 options where we feel they are unclear or unfriendly.

https://fedorahosted.org/freeipa/ticket/2083

 
  I introduced a Param.encode() function (patch 163) to our framework to
  help encoding Python native values to LDAP representation more
  effectively. True/False to LDAP's TRUE/FALSE in this case. Encoding
  functions are executed in a server context only.
 Ack.
 

Pushed to master.

Martin

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