Change in ovirt-host-deploy[master]: hosted-engine: prevent a race cond. between ha-agent and vdsm
Simone Tiraboschi has submitted this change and it was merged.
Change subject: hosted-engine: prevent a race cond. between ha-agent and vdsm
......................................................................
hosted-engine: prevent a race cond. between ha-agent and vdsm
Without this, host-deploy will first start ovirt-ha-agent that will
bring up vdsm due to systemd dependencies but, just after that,
host-deploy will (re)start vdsm.
If ovirt-ha-agent is not quick enough, vdsm will be restarted in
the middle of connectStorageServer that will not complete,
because vdsm was stopped and does not continue such operations
upon restart.
On the other side, the timeout waiting for connectStorageServer
to be completed is pretty long (900 seconds) so ovirt-ha-agent
will take a lot of time to realize the issue.
In the mean time, ovirt-ha-broker will fail accessing the shared
storage and vdsm get the error and reports back a bad status
to the engine and so adding the host from the webui could fail
if ovirt-ha-agent is not fast enough.
Reversing the start order of vdsm and ovirt-ha-agent
will prevent this since ovirt-ha-agnet has Wants=vdsmd.service and
After=vdsmd.service and both are met.
Change-Id: I2e5dd0cb3c6cbfec00286b063558d379ade1def4
Bug-Url: https://bugzilla.redhat.com/1349532
Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com>
---
M src/ovirt_host_deploy/constants.py
M src/plugins/ovirt-host-common/hosted-engine/packages.py
M src/plugins/ovirt-host-deploy/vdsm/packages.py
3 files changed, 11 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Simone Tiraboschi: Verified; Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/60200
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2e5dd0cb3c6cbfec00286b063558d379ade1def4
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-host-deploy
Gerrit-Branch: master
Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan(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: gerrit-hooks <automation(a)ovirt.org>