Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: fixing a double call of get_images_list ...................................................................... storage: fixing a double call of get_images_list Using just result = cli.getImagesList(self._sdUUID) instead of result = cli.getImagesList(cli.getImagesList(self._sdUUID)) It's iSCSI specific since getImagesList always fails on NFS due to rhbz#1274622 but in that case scanning the directory on the mount point is enough while on iSCSI we have to rely just on getImagesList since before preparing the images we don't have a link to the volumes. Change-Id: I2baf5d24d794486b713fbafdea4049e0b885671f Bug-Url: https://bugzilla.redhat.com/1297457 Signed-off-by: Simone Tiraboschi <stirabos@redhat.com> --- M ovirt_hosted_engine_ha/lib/image.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Nir Soffer: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/53937 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2baf5d24d794486b713fbafdea4049e0b885671f 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: Nir Soffer <nsoffer@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo@redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>