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

Code Review gerrit at ovirt.org
Thu Feb 9 16:49:45 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>
---
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/71599
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Icd638c905d59b526c7a7383776757038d99cc292
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika 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: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list