- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.3]: backend: unlock imported glance image on create errors
by iheim＠redhat.com 24 Sep '13

24 Sep '13

Itamar Heim has submitted this change and it was merged. Change subject: backend: unlock imported glance image on create errors ...................................................................... backend: unlock imported glance image on create errors When the serial execution of tasks for ImportRepoImageCommand fails in the first handler we should make sure to remove the disk as well. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1006400 Change-Id: I49054d36eef780d0902f7dcd448ca44797f42916 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19519 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49054d36eef780d0902f7dcd448ca44797f42916 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: backend: prevent destroy command on glance domains
by iheim＠redhat.com 24 Sep '13

24 Sep '13

Itamar Heim has submitted this change and it was merged. Change subject: backend: prevent destroy command on glance domains ...................................................................... backend: prevent destroy command on glance domains Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1006878 Change-Id: Idfb6f61d63d2bff7a3131eab82d24a9f7c4645c3 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ForceRemoveStorageDomainCommand.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19520 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfb6f61d63d2bff7a3131eab82d24a9f7c4645c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3.0]: engine: Clearing address of vm disk device if it's interface...
by oschreib＠redhat.com 24 Sep '13

24 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: engine: Clearing address of vm disk device if it's interface was changed. ...................................................................... engine: Clearing address of vm disk device if it's interface was changed. Updating disk interface its like detach and attach disk again, therefore address of the vm device representing this disk must be cleared. Change-Id: I62605c490da909447f77513e7691d76ddd24ff26 Bug-Url: https://bugzilla.redhat.com/994247 Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmDiskCommandTest.java 2 files changed, 53 insertions(+), 10 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Ofer Schreiber: Looks good to me, approved Sergey Gotliv: Verified -- To view, visit http://gerrit.ovirt.org/19465 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I62605c490da909447f77513e7691d76ddd24ff26 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.0 Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Add vnic profile action groups to roles tree
by masayag＠redhat.com 24 Sep '13

24 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: webadmin: Add vnic profile action groups to roles tree ...................................................................... webadmin: Add vnic profile action groups to roles tree The vnic profiles action groups were added to the roles tree in order to better describe the actions supported by an existing roles or to enable creation of a custom role which includes any of these action groups. Change-Id: I56ba098f8b0b8e1f94aa7ec82b3612a6a81365d6 Bug-Url: https://bugzilla.redhat.com/1002483 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 3 files changed, 36 insertions(+), 16 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19495 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56ba098f8b0b8e1f94aa7ec82b3612a6a81365d6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Refactor RoleTreeView
by masayag＠redhat.com 24 Sep '13

24 Sep '13

Moti Asayag has submitted this change and it was merged. Change subject: webadmin: Refactor RoleTreeView ...................................................................... webadmin: Refactor RoleTreeView RoleTreeView had too long and complicated method with too many indentation levels. The patch reduces the complexity and allows new entities to be added more easily to the Action Groups tree. Change-Id: I82431f3e96be13d3815e9e89130f4cd674394bc8 Related-To: https://bugzilla.redhat.com/1002483 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java 1 file changed, 207 insertions(+), 341 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19494 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82431f3e96be13d3815e9e89130f4cd674394bc8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: backend: prevent destroy command on glance domains
by tnisan＠redhat.com 24 Sep '13

24 Sep '13

Tal Nisan has submitted this change and it was merged. Change subject: backend: prevent destroy command on glance domains ...................................................................... backend: prevent destroy command on glance domains Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1006878 Change-Id: Idfb6f61d63d2bff7a3131eab82d24a9f7c4645c3 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ForceRemoveStorageDomainCommand.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19372 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfb6f61d63d2bff7a3131eab82d24a9f7c4645c3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: backend: unlock imported glance image on create errors
by tnisan＠redhat.com 24 Sep '13

24 Sep '13

Tal Nisan has submitted this change and it was merged. Change subject: backend: unlock imported glance image on create errors ...................................................................... backend: unlock imported glance image on create errors When the serial execution of tasks for ImportRepoImageCommand fails in the first handler we should make sure to remove the disk as well. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1006400 Change-Id: I49054d36eef780d0902f7dcd448ca44797f42916 Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49054d36eef780d0902f7dcd448ca44797f42916 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: enable start/stop rebalance based on status
by kmayilsa＠redhat.com 24 Sep '13

24 Sep '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: enable start/stop rebalance based on status ...................................................................... webadmin: enable start/stop rebalance based on status 'Start' rebalance action will be enabled if the task status of the selected volume(s) is not 'STARTED'. 'Stop' rebalance action will be enabled if only one volume is selected and the task is REBALANCE and the status is 'STARTED'. Change-Id: Id01a605cfd3a59f3e3b584144f8f7589f0922f1e Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 1 file changed, 21 insertions(+), 7 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18917 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id01a605cfd3a59f3e3b584144f8f7589f0922f1e Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Fix NPE in UpdateVdsGroupCommand
by rgolan＠redhat.com 24 Sep '13

24 Sep '13

Roy Golan has submitted this change and it was merged. Change subject: engine: Fix NPE in UpdateVdsGroupCommand ...................................................................... engine: Fix NPE in UpdateVdsGroupCommand Fixed an NPE when updating a cluster Change-Id: Ibc2fd532e4955d9c2d4632ad8d5c2f47b6615838 Bug-Url: https://bugzilla.redhat.com/1011435 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19506 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc2fd532e4955d9c2d4632ad8d5c2f47b6615838 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: set multi monitor only when compatible on import
by shavivi＠redhat.com 24 Sep '13

24 Sep '13

Shahar Havivi has submitted this change and it was merged. Change subject: engine: set multi monitor only when compatible on import ...................................................................... engine: set multi monitor only when compatible on import Set true only for windows on 3.3 and higher, Also fix tests Change-Id: I8a5ae4a6367fd5e4184f42b514f9be6e42879c26 Bug-Url: https://bugzilla.redhat.com/1007784 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmReader.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfManagerTest.java 3 files changed, 10 insertions(+), 3 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19473 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a5ae4a6367fd5e4184f42b514f9be6e42879c26 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0