diff options
author | Martin Nagy <mnagy@redhat.com> | 2008-08-15 18:08:01 +0200 |
---|---|---|
committer | Martin Nagy <mnagy@redhat.com> | 2008-09-11 23:34:01 +0200 |
commit | 885103c32127d10250564e25c5895464fb366f9e (patch) | |
tree | 5db92cd0d4282b3e1aacbfc04c9d076a0d515bec /ipa-radius-admintools | |
parent | 57669ba43224eee0d90556aeea03d14873b4bd7f (diff) | |
download | freeipa-885103c32127d10250564e25c5895464fb366f9e.tar.gz freeipa-885103c32127d10250564e25c5895464fb366f9e.tar.xz freeipa-885103c32127d10250564e25c5895464fb366f9e.zip |
Rework config.py and change cli tools. Maintain order of IPA servers from command line, config and DNS. Parse options before detecting IPA configuration. Don't ignore rest of the options if one is missing in ipa.conf. Drop the --usage options, we will rely on --help. Fixes: 458869, 459070, 458980, 459234
Diffstat (limited to 'ipa-radius-admintools')
-rw-r--r-- | ipa-radius-admintools/ipa-addradiusclient | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-addradiusprofile | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-delradiusclient | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-delradiusprofile | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-findradiusclient | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-findradiusprofile | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-modradiusclient | 10 | ||||
-rw-r--r-- | ipa-radius-admintools/ipa-modradiusprofile | 10 |
8 files changed, 48 insertions, 32 deletions
diff --git a/ipa-radius-admintools/ipa-addradiusclient b/ipa-radius-admintools/ipa-addradiusclient index 1db571a71..4f97c648a 100644 --- a/ipa-radius-admintools/ipa-addradiusclient +++ b/ipa-radius-admintools/ipa-addradiusclient @@ -81,13 +81,15 @@ def main(): opt_parser.set_usage("Usage: %s [options] %s" % (distinguished_attr, os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error('missing %s' % (distinguished_attr)) - ip_addr = args[1] + ipa.config.init_config(options) + + ip_addr = args[0] pairs[distinguished_attr] = ip_addr # Get pairs from a file or stdin diff --git a/ipa-radius-admintools/ipa-addradiusprofile b/ipa-radius-admintools/ipa-addradiusprofile index 66db52267..aa3bf9c1f 100644 --- a/ipa-radius-admintools/ipa-addradiusprofile +++ b/ipa-radius-admintools/ipa-addradiusprofile @@ -77,13 +77,15 @@ def main(): opt_parser.set_usage("Usage: %s [options] %s" % (distinguished_attr, os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error('missing %s' % (distinguished_attr)) - uid = args[1] + ipa.config.init_config(options) + + uid = args[0] user_profile = not options.shared pairs[distinguished_attr] = uid diff --git a/ipa-radius-admintools/ipa-delradiusclient b/ipa-radius-admintools/ipa-delradiusclient index dd26e8ac9..ee5e2e797 100644 --- a/ipa-radius-admintools/ipa-delradiusclient +++ b/ipa-radius-admintools/ipa-delradiusclient @@ -46,13 +46,15 @@ def main(): help="detailed help information") opt_parser.set_usage("Usage: %s [options] Client-IP-Address" % (os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error("missing Client-IP-Address") - ip_addr = args[1] + ipa.config.init_config(options) + + ip_addr = args[0] try: ipa_client = ipaclient.IPAClient() diff --git a/ipa-radius-admintools/ipa-delradiusprofile b/ipa-radius-admintools/ipa-delradiusprofile index f77d01747..d5be88a76 100644 --- a/ipa-radius-admintools/ipa-delradiusprofile +++ b/ipa-radius-admintools/ipa-delradiusprofile @@ -48,13 +48,15 @@ def main(): help="detailed help information") opt_parser.set_usage("Usage: %s [options] UID" % (os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error("missing UID") - uid = args[1] + ipa.config.init_config(options) + + uid = args[0] user_profile = not options.shared # Per user profiles are pre-created (i.e. objectclass radiusprofile is always added for each user) diff --git a/ipa-radius-admintools/ipa-findradiusclient b/ipa-radius-admintools/ipa-findradiusclient index ade4bd397..12bb9bcfe 100644 --- a/ipa-radius-admintools/ipa-findradiusclient +++ b/ipa-radius-admintools/ipa-findradiusclient @@ -56,15 +56,17 @@ def main(): opt_parser.add_option("-h", "--help", action="callback", callback=help_option_callback, help="detailed help information") - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() opt_parser.set_usage("Usage: %s [options] Client-IP-Address [Client-IP-Address ...]" % (os.path.basename(sys.argv[0]))) - if len(args) < 2: + if len(args) < 1: opt_parser.error("missing Client-IP-Address(es)") - ip_addrs = args[1:] + ipa.config.init_config(options) + + ip_addrs = args try: ipa_client = ipaclient.IPAClient() diff --git a/ipa-radius-admintools/ipa-findradiusprofile b/ipa-radius-admintools/ipa-findradiusprofile index ba714068a..01efa2c03 100644 --- a/ipa-radius-admintools/ipa-findradiusprofile +++ b/ipa-radius-admintools/ipa-findradiusprofile @@ -58,15 +58,17 @@ def main(): opt_parser.add_option("-h", "--help", action="callback", callback=help_option_callback, help="detailed help information") - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() opt_parser.set_usage("Usage: %s [options] UID [UID ...]" % (os.path.basename(sys.argv[0]))) - if len(args) < 2: + if len(args) < 1: opt_parser.error("missing UID(es)") - uids = args[1:] + ipa.config.init_config(options) + + uids = args user_profile = not options.shared try: diff --git a/ipa-radius-admintools/ipa-modradiusclient b/ipa-radius-admintools/ipa-modradiusclient index 9e973b121..2a37929e9 100644 --- a/ipa-radius-admintools/ipa-modradiusclient +++ b/ipa-radius-admintools/ipa-modradiusclient @@ -82,13 +82,15 @@ def main(): opt_parser.set_usage("Usage: %s [options] %s" % (distinguished_attr, os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error('missing %s' % (distinguished_attr)) - ip_addr = args[1] + ipa.config.init_config(options) + + ip_addr = args[0] # Verify entity previously exists and get current values ipa_client = ipaclient.IPAClient() diff --git a/ipa-radius-admintools/ipa-modradiusprofile b/ipa-radius-admintools/ipa-modradiusprofile index 405abcfcc..20586be58 100644 --- a/ipa-radius-admintools/ipa-modradiusprofile +++ b/ipa-radius-admintools/ipa-modradiusprofile @@ -78,13 +78,15 @@ def main(): opt_parser.set_usage("Usage: %s [options] %s" % (distinguished_attr, os.path.basename(sys.argv[0]))) - args = ipa.config.init_config(sys.argv) - options, args = opt_parser.parse_args(args) + ipa.config.add_standard_options(opt_parser) + options, args = opt_parser.parse_args() - if len(args) < 2: + if len(args) < 1: opt_parser.error('missing %s' % (distinguished_attr)) - uid = args[1] + ipa.config.init_config(options) + + uid = args[0] user_profile = not options.shared # Verify entity previously exists and get current values |