Change in ovirt-engine[master]: core: Add disk device after disk is added

ofrenkel at redhat.com ofrenkel at redhat.com
Sun Aug 24 12:23:53 UTC 2014


Omer Frenkel has submitted this change and it was merged.

Change subject: core: Add disk device after disk is added
......................................................................


core: Add disk device after disk is added

Currently, AddDiskCommand adds the device to the DB,
before the disk is there, this cause the boot order not to take the new
disk into consideration.

In this patch the addition of the device is moved after the disk is
created by the disk is in the db.

Change-Id: I2f2fc8e6771ed2b352f2bde7b4e313481df76b90
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1131018
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java
1 file changed, 14 insertions(+), 10 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, but someone else must approve
  Omer Frenkel: Verified
  Liron Aravot: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2f2fc8e6771ed2b352f2bde7b4e313481df76b90
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list