Change in ovirt-hosted-engine-setup[master]: packaging: setup: better handling of dirty storage

[sbonazzo at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server