From Tal Nisan <tnisan(a)redhat.com>:
Tal Nisan has submitted this change and it was merged.
Change subject: engine: ovirt-node should wait for restart after upgrade
......................................................................
engine: ovirt-node should wait for restart after upgrade
The SSH host reboot command changed as follow:
1. Set the host to 'Reboot' status: status is set to reboot once the reboot
command was successfully invoked on the host.
2. Added a sleep: Instead of setting the host's status immediately to its
previous status, there should be a delay to allow the host to start the
reboot and to prevent from the host monitoring to grab the host and attempt
to move it to up.
3. Fixed a bug in which the initial host status was ignored.
4. Set the default status after reboot is 'non-responsive'. This align
the behavior of the command with the host installation flow.
In addition, the ovirt-node upgrade process will be reported as a
failed one if the expected reboot step didn't start properly.
Change-Id: If2e3b4fcafb98fa6e14da54c1c42f9d20e4d1278
Bug-Url:
https://bugzilla.redhat.com/1382691
Signed-off-by: Moti Asayag <masayag(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SshHostRebootCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/UpgradeHostInternalCommand.java
2 files changed, 19 insertions(+), 9 deletions(-)
Approvals:
Martin Peřina: Looks good to me, approved
Jenkins CI: Passed CI tests
Moti Asayag: Verified
--
To view, visit
https://gerrit.ovirt.org/67597
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If2e3b4fcafb98fa6e14da54c1c42f9d20e4d1278
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>