Change in ovirt-hosted-engine-setup[master]: packaging: setup: check for stale VM before creation

sbonazzo at redhat.com sbonazzo at redhat.com
Fri Oct 25 12:39:49 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: check for stale VM before creation
......................................................................


packaging: setup: check for stale VM before creation

Added a check for stale Hosted Engine VM before creating it.
In normal execution flow, this should never happen.
However, it is possible that the user starts again the VM
manually after having installed the OS and forget to poweroff
(destroy) the VM.
Now the user will be warned to properly poweroff the VM
started manually in order to continue the setup process.
The poweroff is not issued automatically because the VM can
be in a stale state after a failure and the user must be
aware that the VM is still around in order to debug the issue.

Bug-Url: https://bugzilla.redhat.com/1010980
Change-Id: I8d5dfc17e10e677959e903f03515a124d5e97611
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M src/plugins/ovirt-hosted-engine-setup/vm/runvm.py
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Greg Padgett: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/20545
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d5dfc17e10e677959e903f03515a124d5e97611
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Leonid Natapov <leonidnatapov at gmail.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list