Change in vdsm[master]: BZ#807642 Use Securable as base class

danken at redhat.com danken at redhat.com
Mon Apr 2 13:02:00 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#807642 Use Securable as base class
......................................................................


BZ#807642 Use Securable as base class

Using Securable as base class allow us to take the best of metaclasses
(being able to modifying methods at import time) and the best of
subclassing (explicit inherited methods such as __init__, _setSafe and
_setUnsafe).
It's also now possible to instantiate attributes per object (__init__)
instead of per class and therefore fix BZ#807642.

Change-Id: I2309e2cfc0291309023238ed91527884ec879d64
---
M vdsm/storage/securable.py
M vdsm/storage/sp.py
2 files changed, 34 insertions(+), 34 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, but someone else must approve
  Saggi Mizrahi: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2309e2cfc0291309023238ed91527884ec879d64
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list