Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: setup: avoid restart and destroy VM options at the add_host ...

sbonazzo at redhat.com sbonazzo at redhat.com
Tue Oct 27 07:20:44 UTC 2015


Sandro Bonazzola has submitted this change and it was merged.

Change subject: setup: avoid restart and destroy VM options at the add_host plugin
......................................................................


setup: avoid restart and destroy VM options at the add_host plugin

Restart and destroy VM options make sense only if on the first
host and at initial stages.
If already at add_host plugin we have simply to add the host to the
engine.
So:
- reordering options cause only 1 and 2 are always significative
- avoiding options 3  and 4 if already at add_host

Change-Id: I8d57245ccfc5c1f66bf03699acb3f197c9068ff1
Bug-Url: https://bugzilla.redhat.com/1271511
Signed-off-by: Simone Tiraboschi <stirabos at redhat.com>
(cherry picked from commit 291160e36b78edbcc83e0dc9a75535fc52872294)
---
M src/ovirt_hosted_engine_setup/check_liveliness.py
1 file changed, 13 insertions(+), 6 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Simone Tiraboschi: Verified
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d57245ccfc5c1f66bf03699acb3f197c9068ff1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: ovirt-hosted-engine-setup-1.3
Gerrit-Owner: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list