diff options
| author | Russell Haering <russellhaering@gmail.com> | 2014-04-13 17:45:18 -0700 |
|---|---|---|
| committer | Russell Haering <russellhaering@gmail.com> | 2014-04-15 10:25:12 -0700 |
| commit | bf746732268c073b4749df1a1e35a762d9f8a816 (patch) | |
| tree | ac370486adb19fa88ae07f2a2d366c8dcaf1db5c /ironic_python_agent/extensions/flow.py | |
| parent | dcd6e3e0dcadda84dcbf478c71460e4287e01c7b (diff) | |
| download | ironic-python-agent-bf746732268c073b4749df1a1e35a762d9f8a816.tar.gz | |
Decorated commands no longer take command_name
Prior to the (sync|async)_command decorators, extension methods had to
construct command results themselves, which required that they have
the name of the command being executed.
When the async_command decorator was introduced, we wanted to maintain
the current command interface, so that a command could be made
asynchronous simply by adding the decorator.
The introduction of the sync_command decorator allows us to satisfy
this objective, without requiring decorated methods to accept a
command_name. This patch modifies both decorators to no longer pass
the command_name argument, and modifies existing command methods not
to expect one.
Closes-Bug: #1308021
Change-Id: I1b33bb7519588cdcb79c6f4d2b10c1e5e8c0cca3
Diffstat (limited to 'ironic_python_agent/extensions/flow.py')
| -rw-r--r-- | ironic_python_agent/extensions/flow.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ironic_python_agent/extensions/flow.py b/ironic_python_agent/extensions/flow.py index 617eebac..3c1b8980 100644 --- a/ironic_python_agent/extensions/flow.py +++ b/ironic_python_agent/extensions/flow.py @@ -51,7 +51,7 @@ class FlowExtension(base.BaseAgentExtension, base.ExecuteCommandMixin): ) @base.async_command(_validate_exts) - def start_flow(self, command_name, flow=None): + def start_flow(self, flow=None): for task in flow: for method, params in task.items(): LOG.info("Executing method %s for now" % method) |