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@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@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: automation@ovirt.org