Re: [Freeipa-devel] [PATCH] Use DNS forwarders in /etc/named.conf

2009-09-07 Thread Martin Nagy
On Mon, 2009-09-07 at 07:54 +1000, David O'Brien wrote:
 Martin Nagy wrote:
  On Fri, 2009-09-04 at 09:50 +1000, David O'Brien wrote:

  Martin Nagy wrote:
  
  Hi,
  This patch adds options --forwarder and --no-forwarders. At least one of
  them must be used if you are doing a setup with DNS server. They are
  also mutually exclusive. The --forwarder option can be used more than
  once to specify more servers. If the installer runs in interactive mode,
  it will prompt the user if none of these option was given at the command
 
  Martin

  Something like ipa-server-install --setup-dns --forwarder IP 
  --forwarder IP ?
  You can't use a separated list of IP addresses to save space and typing?
 
  That may go against coding style or practice, I don't know...
 
  /me has user hat on  :-)
  
 
  Hm, well, we use a python module to handle option parsing [1] and this
  is the way it handles lists. We could allow the IP addresses to be
  separated by, say, ',' but this also has the risk that the user will do
  --forwarder 1.2.3.4, 2.3.4.5 which of course wouldn't work (the last
  IP is treated as a positional argument).
 
  Martin
 
  [1] http://docs.python.org/library/optparse.html
 

 ok, well if it's SOP to handle options this way I'm not about to suggest 
 otherwise[1], but if there were a user-friendly way of doing it that 
 didn't bend or break any rules I'd make more noise.
 
 cheers
 David
 
 [1] One of the goals in ECS' style guide is to adhere as much as 
 possible to industry standards and standard practice, not invent new 
 ones that Red Hat likes better.

I'll patch it to also accept comma-separated IP addresses, it's not that
big a deal :)

Martin

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


Re: [Freeipa-devel] [PATCH] Use DNS forwarders in /etc/named.conf

2009-09-06 Thread David O'Brien

Martin Nagy wrote:

On Fri, 2009-09-04 at 09:50 +1000, David O'Brien wrote:
  

Martin Nagy wrote:


Hi,
This patch adds options --forwarder and --no-forwarders. At least one of
them must be used if you are doing a setup with DNS server. They are
also mutually exclusive. The --forwarder option can be used more than
once to specify more servers. If the installer runs in interactive mode,
it will prompt the user if none of these option was given at the command

Martin


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
  
Something like ipa-server-install --setup-dns --forwarder IP 
--forwarder IP ?

You can't use a separated list of IP addresses to save space and typing?

That may go against coding style or practice, I don't know...

/me has user hat on  :-)



Hm, well, we use a python module to handle option parsing [1] and this
is the way it handles lists. We could allow the IP addresses to be
separated by, say, ',' but this also has the risk that the user will do
--forwarder 1.2.3.4, 2.3.4.5 which of course wouldn't work (the last
IP is treated as a positional argument).

Martin

[1] http://docs.python.org/library/optparse.html

  
ok, well if it's SOP to handle options this way I'm not about to suggest 
otherwise[1], but if there were a user-friendly way of doing it that 
didn't bend or break any rules I'd make more noise.


cheers
David

[1] One of the goals in ECS' style guide is to adhere as much as 
possible to industry standards and standard practice, not invent new 
ones that Red Hat likes better.


--
David O'Brien
IPA Content Author
Red Hat Asia Pacific
+61 7 3514 8189

http://freeipa.org/page/DocumentationPortal
http://git.fedorahosted.org/git/ipadocs.git

He who asks is a fool for five minutes, but he who does not ask remains a fool 
forever.
 ~ Chinese proverb

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


Re: [Freeipa-devel] [PATCH] Use DNS forwarders in /etc/named.conf

2009-09-04 Thread Martin Nagy
On Fri, 2009-09-04 at 09:50 +1000, David O'Brien wrote:
 Martin Nagy wrote:
  Hi,
  This patch adds options --forwarder and --no-forwarders. At least one of
  them must be used if you are doing a setup with DNS server. They are
  also mutually exclusive. The --forwarder option can be used more than
  once to specify more servers. If the installer runs in interactive mode,
  it will prompt the user if none of these option was given at the command
 
  Martin
  
 
  ___
  Freeipa-devel mailing list
  Freeipa-devel@redhat.com
  https://www.redhat.com/mailman/listinfo/freeipa-devel
 Something like ipa-server-install --setup-dns --forwarder IP 
 --forwarder IP ?
 You can't use a separated list of IP addresses to save space and typing?
 
 That may go against coding style or practice, I don't know...
 
 /me has user hat on  :-)

Hm, well, we use a python module to handle option parsing [1] and this
is the way it handles lists. We could allow the IP addresses to be
separated by, say, ',' but this also has the risk that the user will do
--forwarder 1.2.3.4, 2.3.4.5 which of course wouldn't work (the last
IP is treated as a positional argument).

Martin

[1] http://docs.python.org/library/optparse.html

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