diff options
| author | Abhijeet Kasurde <akasurde@redhat.com> | 2016-08-23 17:34:51 +0530 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-08-24 15:50:25 +0200 |
| commit | 95a594af4c99255ea4da27e609cf41b79ca7ed91 (patch) | |
| tree | 2540fa0202cbb91db85c7bc66bd5fba1e6d136d8 /ipaclient/plugins | |
| parent | 6f9a029bf5d33e6c8267cb330bd48033c5517188 (diff) | |
| download | freeipa-95a594af4c99255ea4da27e609cf41b79ca7ed91.tar.gz freeipa-95a594af4c99255ea4da27e609cf41b79ca7ed91.tar.xz freeipa-95a594af4c99255ea4da27e609cf41b79ca7ed91.zip | |
Handled empty hostname in server-del command
Fixes: https://fedorahosted.org/freeipa/ticket/6248
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Reviewed-By: Martin Basti <mbasti@redhat.com>
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'ipaclient/plugins')
| -rw-r--r-- | ipaclient/plugins/server.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/ipaclient/plugins/server.py b/ipaclient/plugins/server.py index 725a2ce95..5356dbbf5 100644 --- a/ipaclient/plugins/server.py +++ b/ipaclient/plugins/server.py @@ -3,7 +3,7 @@ # from ipaclient.frontend import MethodOverride -from ipalib import _ +from ipalib import _, errors from ipalib.plugable import Registry register = Registry() @@ -12,6 +12,10 @@ register = Registry() @register(override=True, no_fail=True) class server_del(MethodOverride): def interactive_prompt_callback(self, kw): + server_list = kw.get('cn') + if not server_list: + raise errors.RequirementError(name='cn') + self.api.Backend.textui.print_plain( _("Removing %(servers)s from replication topology, " - "please wait...") % {'servers': ', '.join(kw['cn'])}) + "please wait...") % {'servers': ', '.join(server_list)}) |