openstack/rally-openstack
Code Issues Proposed changes

Remove atomic timer from nova.bind_actions method

Browse Source 
Method `_bind_actions` creates an inner rally entity for further
launching different kind of actions.
It is redundant to calculate how much time takes initialization of one
inner class + calling several inner methods. It takes ms.

Change-Id: Id5224138e3dcdd6815e7ee00ae5b16200fb7f2d9
This commit is contained in:
Andrey Kurilin 2018-04-24 18:45:34 +03:00
parent 911663cda1
commit 6158c1139c
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
CHANGELOG.rst
View File

@ -38,6 +38,14 @@ Changed
Also, the CI is extended to check ability to build Docker image for any of Also, the CI is extended to check ability to build Docker image for any of
changes. changes.
Removed
~~~~~~~
* Calculation of the duration for "nova.bind_actions" action. It shows
only duration of initialization Rally inner class and can be easily
misunderstood as some kind of "Nova operation".
Affects 1 inner scenario "NovaServers.boot_and_bounce_server".
Fixed Fixed
~~~~~ ~~~~~

1
rally_openstack/scenarios/nova/utils.py
View File

@ -1078,7 +1078,6 @@ class NovaScenario(scenario.OpenStackScenario):
""" """
return self.admin_clients("nova").aggregates.delete(aggregate) return self.admin_clients("nova").aggregates.delete(aggregate)
@atomic.action_timer("nova.bind_actions")
def _bind_actions(self): def _bind_actions(self):
actions = ["hard_reboot", "soft_reboot", "stop_start", actions = ["hard_reboot", "soft_reboot", "stop_start",
"rescue_unrescue", "pause_unpause", "suspend_resume", "rescue_unrescue", "pause_unpause", "suspend_resume",