Dan Kenigsberg has submitted this change and it was merged. Change subject: FakeLVM: Simplify storage of lv metadata ......................................................................
FakeLVM: Simplify storage of lv metadata Nir pointed out that storing the LV metadata as a dictionary of dictionaries (first level is vg_name and second level is lv_name) can be error prone. Instead, let's store in a single level dictionary using the tuple (vg_name, lv_name) as the key. Change-Id: I0e6c0c9af11ecdc88d294848cf244d0d639ec65d Signed-off-by: Adam Litke <ali...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/48215 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer <nsof...@redhat.com> --- M tests/storagefakelib.py 1 file changed, 8 insertions(+), 9 deletions(-) Approvals: Adam Litke: Verified Nir Soffer: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48215 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e6c0c9af11ecdc88d294848cf244d0d639ec65d Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
