Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm: Missing hostId parameter on reconstructMaster verb ......................................................................
vdsm: Missing hostId parameter on reconstructMaster verb Json schema was not consistent with API.py for StoragePool class and reconstructMaster method. There was missing hostId parameter. Please note that there is parameter ordering difference for coresponding xmlrpc poolReconstructMaster method. Change-Id: I0695a21601b8f0765bc24c3cf273f1ba161a40a9 Signed-off-by: pkliczewski <[email protected]> Bug-Url: https://bugzilla.redhat.com/1115466 Reviewed-on: http://gerrit.ovirt.org/29510 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Saggi Mizrahi <[email protected]> --- M vdsm/rpc/vdsmapi-schema.json 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Verified Nir Soffer: Looks good to me, but someone else must approve Saggi Mizrahi: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0695a21601b8f0765bc24c3cf273f1ba161a40a9 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
