On 04/30/2013 02:32 PM, Tomas Babej wrote:
On 04/11/2013 09:57 PM, Rob Crittenden wrote:
Tomas Babej wrote:

In ipa-replica-manage commands, we enforce that hostnames we work
with are resolvable. However, this caused errors while deleting
or disconnecting a ipa / winsync replica, if that replica was down
and authoritative server for itself.



I'm not sure this is going to do the right thing either. A lot of
these commands take the an argument as the remote master to run things
on, so we'd really only be validating one of the names. Not sure how
that helps us.

Actually, the patch tried to adress that. I carefully reviewed the
effort, now we should be consistent in validating all the names.

What if we honor the --force flag for DNS lookup failures instead? Or,
since that could override it and do other things, a --no-lookup flag


I added a --no-lookup flag for ipa-replica-manage that disables host
existence check.

Sending both patches rebased.


The nolookup argument is never passed to get_ruv() when it is called by list_ruv, get_rid_by_host, clean_ruv, abort_clean_ruv. Some of these don't take the argument but are called with it. Lint error: install/tools/ipa-replica-manage:1188: [E1121, main] Too many positional arguments for function call

nolookup is also not passed to list_clean_ruv(), re_initialize(), force_sync(), show_DNA_ranges() etc.

Git complains about some extra whitespace:
Applying: Enforce host existence only where needed in ipa-replica-manage
/home/pviktori/freeipa/.git/rebase-apply/patch:234: new blank line at EOF.
warning: 1 line adds whitespace errors.


Freeipa-devel mailing list

Reply via email to