On 04/16/2015 05:14 PM, Martin Basti wrote:
On 16/04/15 13:53, Martin Babinsky wrote:
On 04/16/2015 01:34 PM, Martin Babinsky wrote:
On 04/15/2015 04:17 PM, Martin Basti wrote:
These patches keep usage of IPA server address as NTP server in NTP
configuration on clients, in case that no NTP servers were
user, and no NTP servers were resolved from SRV records. This will
ensure backward compatibility, as IPA does not configure NTP SRV
for each domain automatically.
PATCH 224 NACK
PATCH 225 ACK
Patch 224 you keep the original destination (dest="ntp_server")
for --ntp-server option, but in patch 226 the code attempts to get the
server names from options.ntpservers resulting in:
Traceback (most recent call last):
File "/sbin/ipa-client-install", line 2932, in <module>
File "/sbin/ipa-client-install", line 2913, in main
rval = install(options, env, fstore, statestore)
File "/sbin/ipa-client-install", line 2342, in install
AttributeError: Values instance has no attribute 'ntp_servers'
So please fix this.
Naming the destination 'ntp_servers' (plural form) seems best because we
now store multiple values.
Also, if renaming "option.ntp_server" to "option.ntp_servers", do not
forget to change also these lines in "ipa-client-install":
2852 if options.ntp_server:
2853 ntp_servers = options.ntp_server
(line numbers after applying patches 224-226)
Stupid me, thank you
Updated patches attached.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code