Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: upgrade: reimplementing getImagesList cause vdsm one is broken

[sbonazzo at redhat.com]

Sandro Bonazzola has submitted this change and it was merged.

Change subject: upgrade: reimplementing getImagesList cause vdsm one is broken
......................................................................


upgrade: reimplementing getImagesList cause vdsm one is broken

VDSM getImagesList raises an 'list index out of range' exception
if called on a storage domain witch is not connected to any SP.

Change-Id: I3b014029e9a820ea6bbb038786502778e00ea679
Bug-Url: https://bugzilla.redhat.com/1276650
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
(cherry picked from commit 1776e0843895f0e3b3c512bda5e2454305b9c4d2)
---
M ovirt_hosted_engine_ha/lib/heconflib.py
M ovirt_hosted_engine_ha/lib/upgrade.py
2 files changed, 31 insertions(+), 4 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3b014029e9a820ea6bbb038786502778e00ea679
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: ovirt-hosted-engine-ha-1.3
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: automation at ovirt.org