diff options
| author | John L. Villalovos <john.l.villalovos@intel.com> | 2016-11-08 11:08:21 -0800 |
|---|---|---|
| committer | John L. Villalovos <john.l.villalovos@intel.com> | 2016-11-08 11:31:18 -0800 |
| commit | 0b8763e33221c585207ec33caf05adfba7fad867 (patch) | |
| tree | ba6704af7f54c1ca9854a991cd14f6d10b9fc1d8 /ironic_python_agent/extensions/flow.py | |
| parent | fb3a56d185e9ee22df44c2cb83324b354f9973e5 (diff) | |
| download | ironic-python-agent-0b8763e33221c585207ec33caf05adfba7fad867.tar.gz | |
Update to hacking 0.12.0 and use new checks
Use hacking 0.12.0
Use the new checks that are available:
[H106] Don’t put vim configuration in source files.
[H203] Use assertIs(Not)None to check for None.
[H904] Delay string interpolations at logging calls.
Fix code so tests pass.
Change-Id: If22ad272c332f30624ce10861408d377908b152b
Depends-On: I2aa44b62f900d4dfd67701b01eadd0523fbfaf07
Diffstat (limited to 'ironic_python_agent/extensions/flow.py')
| -rw-r--r-- | ironic_python_agent/extensions/flow.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ironic_python_agent/extensions/flow.py b/ironic_python_agent/extensions/flow.py index a080747a..8ef17ca0 100644 --- a/ironic_python_agent/extensions/flow.py +++ b/ironic_python_agent/extensions/flow.py @@ -36,10 +36,10 @@ class FlowExtension(base.BaseAgentExtension, base.ExecuteCommandMixin): def start_flow(self, flow=None): for task in flow: for method, params in task.items(): - LOG.info("Executing method %s for now" % method) + LOG.info("Executing method %s for now", method) result = self.execute_command(method, **params) result.join() - LOG.info("%s method's execution is done" % method) + LOG.info("%s method's execution is done", method) if result.command_status == base.AgentCommandStatus.FAILED: raise errors.CommandExecutionError( "%s was failed" % method |