summaryrefslogtreecommitdiffstats
path: root/ipalib/plugins/ping.py
diff options
context:
space:
mode:
authorPetr Viktorin <pviktori@redhat.com>2012-06-21 08:20:26 -0400
committerMartin Kosek <mkosek@redhat.com>2013-02-21 16:26:09 +0100
commit7336a176b43989b9d459a2536af88f89e849213f (patch)
tree46c2ab141dc545e8f0c148871508fe7c952d22e5 /ipalib/plugins/ping.py
parent246bc3f3eaccf2a84310df84dc85fe455c24ac65 (diff)
downloadfreeipa-7336a176b43989b9d459a2536af88f89e849213f.tar.gz
freeipa-7336a176b43989b9d459a2536af88f89e849213f.tar.xz
freeipa-7336a176b43989b9d459a2536af88f89e849213f.zip
Add the version option to all Commands
Several Commands were missing the 'version' option. Add it to those that were missing it. Do not remove the version option before calling commands. This means methods such as execute(), forward(), run() receive it. Several of these needed `**options` added to their signatures. Commands in the Cert plugin passed any unknown options to the underlying functions, these are changed to pass what's needed explicitly. Some commands in DNS and Batch plugins now pass version to commands they call. When the option is not given, fill it in automatically. (In a subsequent commit, a warning will be added in this case). Note that the public API did not change: all RPC calls already accepted a version option. There's no need for an API version bump (even though API.txt changes substantially). Design page: http://freeipa.org/page/V3/Messages Tickets: https://fedorahosted.org/freeipa/ticket/2732 https://fedorahosted.org/freeipa/ticket/3294
Diffstat (limited to 'ipalib/plugins/ping.py')
-rw-r--r--ipalib/plugins/ping.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/ipalib/plugins/ping.py b/ipalib/plugins/ping.py
index 0da07e0b4..e9dc28fea 100644
--- a/ipalib/plugins/ping.py
+++ b/ipalib/plugins/ping.py
@@ -58,7 +58,7 @@ class ping(Command):
output.summary,
)
- def execute(self):
+ def execute(self, **options):
"""
A possible enhancement would be to take an argument and echo it
back but a fixed value works for now.