Liron Aravot has posted comments on this change. Change subject: hsm: add getVGInfo discard related fields ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/62800/2/vdsm/storage/hsm.py File vdsm/storage/hsm.py: Line 2807: info["vgUUID"] = str(pv.vg_uuid) Line 2808: info["pvUUID"] = str(pv.uuid) Line 2809: info["GUID"] = str(pv.guid) Line 2810: info["discard_max_bytes"] = devInfo["discard_max_bytes"] Line 2811: info["discard_zeroes_data"] = devInfo["discard_zeroes_data"] you need to update vdsm-api.yml as well. Line 2812: return info Line 2813: Line 2814: @deprecated Line 2815: @public -- To view, visit https://gerrit.ovirt.org/62800 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3cdb535923f6f5ffc961750271e8cde823ed835a Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ish...@redhat.com> Gerrit-Reviewer: Adam Litke <ali...@redhat.com> Gerrit-Reviewer: Ala Hino <ah...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Idan Shaby <ish...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <lara...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Tal Nisan <tni...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org