Francesco Romani has posted comments on this change. Change subject: vm: storage: drop legacy drive config support ......................................................................
Patch Set 2: (1 comment) https://gerrit.ovirt.org/#/c/60323/2/vdsm/virt/vm.py File vdsm/virt/vm.py: Line 607 Line 608 Line 609 Line 610 Line 611 > It "drives" list in the conf not used any more? Is it documented in the sch IIRC this dates back to Engine 3.3 days. It was never present in the old JSON schema (checked ovirt3.3 and 3.4) I think we don't use this since ages, but I'll doublecheck again just to be sure. -- To view, visit https://gerrit.ovirt.org/60323 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib7f719c8e89fd976cbccdc6e93d124b5e34c9ce7 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
