Change in ovirt-host-deploy[ovirt-host-deploy-1.6]: node: remove a duplicate decorator

Code Review gerrit at ovirt.org
Thu Feb 9 16:57:40 UTC 2017


>From Sandro Bonazzola <sbonazzo at redhat.com>:

Sandro Bonazzola has submitted this change and it was merged.

Change subject: node: remove a duplicate decorator
......................................................................


node: remove a duplicate decorator

A plugin method has two otopi plugin decorators;
python will call first the decorator instance closest
to the method name and then the external one.
Due to otopi method condition handling, the second
decorator (the external one) will completely replace
the conditions set by the first decorator instance.
Removing it.

Change-Id: Icd638c905d59b526c7a7383776757038d99cc292
Bug-Url: https://bugzilla.redhat.com/1414265
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
(cherry picked from commit 680c0672e1c91cfa437d3a975edfef53fb7e9dcc)
---
M src/plugins/ovirt-host-deploy/vdsm/config.py
1 file changed, 0 insertions(+), 6 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Simone Tiraboschi: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/72044
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd638c905d59b526c7a7383776757038d99cc292
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: ovirt-host-deploy-1.6
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list