Change in ovirt-hosted-engine-setup[master]: packaging: setup: correctly parse stroage domain info

[sbonazzo at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov at gmail.com>
Gerrit-Reviewer: Lev Veyde <lveyde at gmail.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server