Change in ovirt-hosted-engine-ha[master]: storage: Added call to repoStats before storage recovery

Code Review gerrit at ovirt.org
Wed Aug 9 12:11:12 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged. ( https://gerrit.ovirt.org/80454 )

Change subject: storage: Added call to repoStats before storage recovery
......................................................................


storage: Added call to repoStats before storage recovery

Storage domain recovery procedure is expensive, so it is
better to verify domain availability with cheap repoStats
call before trying to recover it.

Change-Id: I7d662e12f532868ace957bea3f21576796887e50
Bug-Url: https://bugzilla.redhat.com/1337914
Signed-off-by: Denis Chaplygin <dchaplyg at redhat.com>
---
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/lib/storage_server.py
2 files changed, 28 insertions(+), 2 deletions(-)

Approvals:
  Denis Chaplygin: Verified
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Andrej Krejcir: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/80454
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7d662e12f532868ace957bea3f21576796887e50
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list