From Tal Nisan <tnisan@redhat.com>:
Tal Nisan 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@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/73674 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24d357cdd324817834b8ee2902ec7d3c9add99b6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>