Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: better handling of dirty storage
......................................................................
packaging: setup: better handling of dirty storage
Better handling of dirty storage. Now it checks the output
of getStorageDomainInfo to understand if the storage domain is really
there, otherwise it tries to create again.
The aim is to better handle retry on dirty storage where the VG is
there but it doesn't contain a valid storageDomain.
Change-Id: I7b9c05d26db929c408943455c7c9ac15764be3d4
Bug-Url:
https://bugzilla.redhat.com/1155637
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/storage/storage.py
1 file changed, 10 insertions(+), 8 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Verified
--
To view, visit
http://gerrit.ovirt.org/36811
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7b9c05d26db929c408943455c7c9ac15764be3d4
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server