diff options
| author | Jay Faulkner <jay@jvf.cc> | 2016-03-30 17:54:47 +0000 |
|---|---|---|
| committer | Jay Faulkner <jay@jvf.cc> | 2016-03-30 23:38:50 +0000 |
| commit | b176f29b3d56fdd1f29d0298b6c277a7fe9c8e23 (patch) | |
| tree | 41e75bb3e691c88b38b99a364b02a6a0eb1b33bd /ironic_python_agent/extensions/base.py | |
| parent | d1060c0469a474fa76992f3d8ce53f2e29f86301 (diff) | |
| download | ironic-python-agent-b176f29b3d56fdd1f29d0298b6c277a7fe9c8e23.tar.gz | |
Improve __str__ method of BaseCommandResult
Fixes nits raised in review for Change ID
I3ca0382f05e88700c56d3c7f4b4ecf7b8fc3bc0c
Remove useless id field, fix capitalization and grammar.
Change-Id: I7976993d1ac9354e035a5e7dba6ef8cc392537e3
Diffstat (limited to 'ironic_python_agent/extensions/base.py')
| -rw-r--r-- | ironic_python_agent/extensions/base.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/ironic_python_agent/extensions/base.py b/ironic_python_agent/extensions/base.py index 5e49fdc7..77c23694 100644 --- a/ironic_python_agent/extensions/base.py +++ b/ironic_python_agent/extensions/base.py @@ -56,10 +56,11 @@ class BaseCommandResult(encoding.SerializableComparable): self.command_result = None def __str__(self): - return ("Command ID: %(id)s, Name: %(name)s " - "Params: %(params)s Status: %(status)s (%(result)s)" % - {"id": self.id, "name": self.command_name, - "params": self.command_params, "status": self.command_status, + return ("Command name: %(name)s, " + "params: %(params)s, status: %(status)s, result: " + "%(result)s." % + {"name": self.command_name, "params": self.command_params, + "status": self.command_status, "result": self.command_result}) def is_done(self): |