Change in vdsm[master]: sp.py: Assume a hostId when reconstructing master

nsoffer at redhat.com nsoffer at redhat.com
Tue Apr 12 20:12:59 UTC 2016


Nir Soffer has posted comments on this change.

Change subject: sp.py: Assume a hostId when reconstructing master
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

Waiting for Adam ack.

https://gerrit.ovirt.org/#/c/55857/3/vdsm/storage/hsm.py
File vdsm/storage/hsm.py:

Line 1843
Line 1844
Line 1845
Line 1846
Line 1847
> According to the schema this is correct change, but the schema by be wrong 
On the other hand, API.py is the only caller, and it require all the arguments like the schema, so we cannot have code calling this with out any of the parameters.


-- 
To view, visit https://gerrit.ovirt.org/55857
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaffbb563532a40f67f33fd971f55724e994c49dd
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list