Change in ovirt-engine[master]: engine:cannot start stateless VM

ofrenkel at redhat.com ofrenkel at redhat.com
Tue Oct 16 07:38:20 UTC 2012


Omer Frenkel has submitted this change and it was merged.

Change subject: engine:cannot start stateless VM
......................................................................


engine:cannot start stateless VM

1. create a vm with disk
2. edit vm -> check the stateless checkbox
3. run the vm

The machine doesn't run with NPE. The problem was that the parent command and
its parameters were set incorrectly in classes CreateAllSnapshotsFromVmCommand
and RunVmCommand. Fixed by setting the right parent command/parameters.

Change-Id: I6152049360dd0254b9bd07b2397abd9d7d56c5d8
Signed-off-by: Frantisek Kobzik <fkobzik at redhat.com>
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=863766
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
2 files changed, 7 insertions(+), 4 deletions(-)

Approvals:
  Omer Frenkel: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6152049360dd0254b9bd07b2397abd9d7d56c5d8
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <fkobzik at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Frank Kobzik <fkobzik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list