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