Dan Kenigsberg has submitted this change and it was merged. Change subject: hsm: fix setStoragePoolDescription execution ......................................................................
hsm: fix setStoragePoolDescription execution In commit ab6014b the pool metadata operations were moved to the pool backend, while the call to setStoragePoolDescription was left as it was, leading to an attribute error. StoragePoolMemoryBackend does not maintain any information about the pool so setStoragePoolDescription it is not supported. It is safe to ignore the setStoragePoolDescription request because in StoragePoolMemoryBackend there's no place where to persist the value and engine never tries to read this information. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1096212 Change-Id: Ic85ecd5310d15947631224a21f962526eec91180 Signed-off-by: Federico Simoncelli <fsimo...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/27540 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> Tested-by: Dan Kenigsberg <dan...@redhat.com> (cherry picked from commit f49c989a7d909b7b66f8d925edf3dad1f16c2a2b) Reviewed-on: http://gerrit.ovirt.org/27543 Reviewed-by: Allon Mureinik <amure...@redhat.com> --- M vdsm/storage/hsm.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Federico Simoncelli: Verified Allon Mureinik: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27543 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic85ecd5310d15947631224a21f962526eec91180 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.4 Gerrit-Owner: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches