Change in ovirt-engine[master]: core: Set read only in the CDA instead the constructor

[mlipchuk at redhat.com]

Maor Lipchuk 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
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachDiskToVmCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib70637b4fc29a95bb73aa57a6f3f0c74d4d16ca1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server