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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Lev Veyde <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Simone Tiraboschi <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to