Dan Kenigsberg has submitted this change and it was merged. Change subject: schema: temporarily skip getMigrationStatus ......................................................................
schema: temporarily skip getMigrationStatus Commit b8df0ff261c unveiled an existing inconsistency betwen the return value of VM.getMigrationStatus() API and the schema. As result, schema validation tests, thus master, are broken. Investigation on Engine side suggests that Engine does not really cares about new keys, so we have the chance to properly fix, cleanup and modernize this verb with no hurry. However, to quickly unbreak master, this patch just skips the schema validation of the getMigrationStatus verb, until it is fully worked on. Change-Id: I86b15d626314e679b7fecb6d40cb9d950086fe27 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/39062 Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M tests/schemaValidationTest.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/39062 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86b15d626314e679b7fecb6d40cb9d950086fe27 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Shahar Havivi <[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
