Change in ovirt-engine[master]: webadmin: AbstractDiskModel - setDiskProfilesList()

Code Review gerrit at ovirt.org
Tue Mar 7 09:09:02 UTC 2017


>From Liron Aravot <laravot at redhat.com>:

Liron Aravot has submitted this change and it was merged.

Change subject: webadmin: AbstractDiskModel - setDiskProfilesList()
......................................................................


webadmin: AbstractDiskModel - setDiskProfilesList()

When setDiskProfilesList() is executed, the disk profile set on the disk
is considered when updating the disk profile list.
However, for disk models that doesn't represnt disk which were already 
actually created the profile set in the model isn't the actual disk
profile - therefore if the domain is changed we can update it to the 
newly selected domain profile and not consider it as set profile.

Change-Id: I24d357cdd324817834b8ee2902ec7d3c9add99b6
Bug-Url: https://bugzilla.redhat.com/1360355
Signed-off-by: Liron Aravot <laravot at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/AbstractDiskModel.java
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Liron Aravot: Verified



-- 
To view, visit https://gerrit.ovirt.org/73465
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I24d357cdd324817834b8ee2902ec7d3c9add99b6
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list