Change in vdsm[master]: schema: temporarily skip getMigrationStatus

danken at redhat.com danken at redhat.com
Wed Mar 25 10:37:32 UTC 2015


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 <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/39062
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
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 <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list