Change in ovirt-engine[master]: engine: Run Once dialog ingores emtpy boot options (#857848)

tjelinek at redhat.com tjelinek at redhat.com
Wed Oct 3 07:51:03 UTC 2012


Tomas Jelinek has submitted this change and it was merged.

Change subject: engine: Run Once dialog ingores emtpy boot options (#857848)
......................................................................


engine: Run Once dialog ingores emtpy boot options (#857848)

https://bugzilla.redhat.com/857848

When you set the linux boot options in Edit dialog,
than in Run Once dialog remove this options, the run once will
use the boot options set up in the Edit dialog.

The problem was in RunVmCommand.refreshBootParameters method
which expected, that when the parameter is empty, it is run VM,
when it is not empty, it is run once VM, which is not true in
this case.

Fixed by moving the run once specific code to the RunVmOnceCommand.

This patch contains also the following refactorings:
- the refreshBootSequenceParameter inlined to the
  refreshBootParameters
- setLastStartTime removed from the refreshBootSequenceParameter
  and moved to places where the refreshBootSequenceParameter
  has been called

Change-Id: I32e59783d9c4d8e8b61ec96c6bae5577fc59756c
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java
2 files changed, 29 insertions(+), 31 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I32e59783d9c4d8e8b61ec96c6bae5577fc59756c
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek 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