Change in ovirt-hosted-engine-setup[master]: Adapt to VDSM 3.7 StorageDomainInfo API

sbonazzo at redhat.com sbonazzo at redhat.com
Tue May 5 06:38:42 UTC 2015


Sandro Bonazzola has submitted this change and it was merged.

Change subject: Adapt to VDSM 3.7 StorageDomainInfo API
......................................................................


Adapt to VDSM 3.7 StorageDomainInfo API

In VDSM 3.7 StorageDomainInfo returns a string for
the "type" key in the result dictionary.
The string should be ones between the names of the constants in
the enum type StorageDomainType defined in vdsm/rpc/vdsmapi-schema.json

The VDSM schema is missing GLUSTERFS but it's using it anyway.

Change-Id: Id5f45d088c943c62e0e5761cca1a565616b2dcc5
See-Also: https://bugzilla.redhat.com/1215039
Bug-Url: https://bugzilla.redhat.com/1213307
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M src/ovirt_hosted_engine_setup/constants.py
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
2 files changed, 15 insertions(+), 2 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve
  Jenkins CI: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id5f45d088c943c62e0e5761cca1a565616b2dcc5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list