Dan Kenigsberg has submitted this change and it was merged. Change subject: securable: refactor the securable implementation ......................................................................
securable: refactor the securable implementation In order to remove the storage pool metadata we need more flexibility in defining the isSafe method. In fact the different backends (with/without metadata) will rely on the StoragePool SPMness to protect their methods. In this patch: - transform the Securable base class and MetaSecurable in a decorator Change-Id: Id5a8be7536748481746795e27701fbecc7c3318c Signed-off-by: Federico Simoncelli <[email protected]> Reviewed-on: http://gerrit.ovirt.org/22115 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-by: Ayal Baron <[email protected]> --- M tests/main.py M vdsm/storage/securable.py M vdsm/storage/sp.py 3 files changed, 115 insertions(+), 62 deletions(-) Approvals: Ayal Baron: Looks good to me, approved Nir Soffer: Looks good to me, but someone else must approve Federico Simoncelli: Verified Dan Kenigsberg: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22115 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id5a8be7536748481746795e27701fbecc7c3318c Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
