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