Oved Ourfali has submitted this change and it was merged.

Change subject: core: Set read only in the CDA instead the constructor
......................................................................


core: Set read only in the CDA instead the constructor

Setting read only property in the disk at the CDA phase in the constructor to 
avoid NPE
when there is no disk in the DB.

Change-Id: Ib70637b4fc29a95bb73aa57a6f3f0c74d4d16ca1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128430
Signed-off-by: Maor Lipchuk <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachDiskToVmCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Maor Lipchuk: Verified
  Vered Volansky: Looks good to me, but someone else must approve
  Liron Aravot: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib70637b4fc29a95bb73aa57a6f3f0c74d4d16ca1
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Liron Aravot <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to