Change in vdsm[master]: Don't change the name of a secured class

[Federico Simoncelli]

Federico Simoncelli has uploaded a new change for review.

Change subject: Don't change the name of a secured class
......................................................................

Don't change the name of a secured class

The variable 'name' used to name the secured classes was overridden
by the 'name' variable used to cycle over the methods.
The result was that the secured classes were renamed to their last
method name, eg: storage.sp.StoragePool => storage.sp.setMaxHostId.

Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
---
M vdsm/storage/securable.py
1 file changed, 3 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/74/974/1
--
To view, visit http://gerrit.ovirt.org/974
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: Don't change the name of a secured class
......................................................................


Don't change the name of a secured class

The variable 'name' used to name the secured classes was overridden
by the 'name' variable used to cycle over the methods.
The result was that the secured classes were renamed to their last
method name, eg: storage.sp.StoragePool => storage.sp.setMaxHostId.

Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
---
M vdsm/storage/securable.py
1 file changed, 3 insertions(+), 5 deletions(-)

Approvals:
  Federico Simoncelli: Verified
  Dan Kenigsberg: Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/974
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Federico Simoncelli]

Federico Simoncelli has posted comments on this change.

Change subject: Don't change the name of a secured class
......................................................................


Patch Set 1: Verified

--
To view, visit http://gerrit.ovirt.org/974
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: Don't change the name of a secured class
......................................................................


Patch Set 1: Looks good to me, approved

--
To view, visit http://gerrit.ovirt.org/974
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7d34b14391a0940e1ca73b8d736877f49b1c47f5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com&gt;