Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: correctly parse stroage domain info
......................................................................
packaging: setup: correctly parse stroage domain info
previously the getStorageDomainInfo parser assumed that
remotePath was always defined in the dictionary returned
by the call to getStorageDomainInfo.
The key is not defined for iSCSI domains so now the
parser check for remotePath availability before
trying to use its value.
Change-Id: I7e8db0f31dadd7072680fb1564514be065ea5431
Bug-Url:
https://bugzilla.redhat.com/1005767
Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
1 file changed, 1 insertion(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/19115
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7e8db0f31dadd7072680fb1564514be065ea5431
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server