June 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: increase blank template memory size
by lhornyak＠redhat.com 17 Jun '13

17 Jun '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: increase blank template memory size ...................................................................... engine: increase blank template memory size Installation of any linux box today requires 1 GB of RAM, setting the memory of the default template could improve user experience by bumping into less problem when doing this. Change-Id: If74889c0165a66fb76fc5081fefdb66cd4df9e83 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/dbscripts/insert_data.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14747 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If74889c0165a66fb76fc5081fefdb66cd4df9e83 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: remove memory snapshot when its not needed
by ofrenkel＠redhat.com 17 Jun '13

17 Jun '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: remove memory snapshot when its not needed ...................................................................... core: remove memory snapshot when its not needed This patch adds the removal of memory snapshot volumes in case it's not needed anymore: 1. when removing VM that has snapshots with memory 2. when snapshots with memory are removed because we revert (commit) to previous snapshot 3. on run VM when the VM reach to a point where the disks might not be synchronous with the memory state that was restored anymore (and the active snapshot is the only one that points to its memory) This patch also treat the restoration of snapshot's memory when running a VM that is previewing/being committed to a snapshot with memory in stateless mode. Change-Id: Ic087580475cd4fcfac969ddf0f69929fa247f86d Bug-Url: https://bugzilla.redhat.com/960931 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/dbscripts/snapshots_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreAllSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmOnceCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/LiveSnapshotMemoryImageBuilder.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/StatelessSnapshotMemoryImageBuilder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/SnapshotDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/SnapshotDaoDbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/SnapshotDaoTest.java 17 files changed, 263 insertions(+), 126 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15682 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic087580475cd4fcfac969ddf0f69929fa247f86d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine:Fixed FK violation on update of gluster event names
by emesika＠redhat.com 17 Jun '13

17 Jun '13

Eli Mesika has submitted this change and it was merged. Change subject: engine:Fixed FK violation on update of gluster event names ...................................................................... engine:Fixed FK violation on update of gluster event names A FK violation occurs during upgrade due to event_up name being present in event_subscriber Change-Id: If8d811be0e8eb52d3ebf2d1bfb4e391398b65abe Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/dbscripts/upgrade/03_02_0060_change_host_to_server_event_map.sql 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15706 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8d811be0e8eb52d3ebf2d1bfb4e391398b65abe Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Lee Yarwood <lyarwood(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: humble devassy <hchiramm(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: host gluster-swift sub tab
by kmayilsa＠redhat.com 17 Jun '13

17 Jun '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: host gluster-swift sub tab ...................................................................... webadmin: host gluster-swift sub tab Adding a new sub-tab "Gluster Swift" under Hosts. This will list all the Gluster Swift related services and their statuses. Start, Stop and Restart action are added to the sub-tab. Change-Id: Iacffe5b49e6dcde743fa81d5a3f694de5d8f1421 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostGlusterSwiftListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/ManagedComponents.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/PresenterModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/HostModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/place/ApplicationPlaces.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostEventPresenter.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostGlusterSwiftPresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostHardwarePresenter.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostPermissionPresenter.java A frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGlusterSwiftView.java 14 files changed, 381 insertions(+), 5 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacffe5b49e6dcde743fa81d5a3f694de5d8f1421 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: fix gluster hook conten-type detection
by kmayilsa＠redhat.com 17 Jun '13

17 Jun '13

Kanagaraj M has submitted this change and it was merged. Change subject: engine: fix gluster hook conten-type detection ...................................................................... engine: fix gluster hook conten-type detection The hook will be considered as human readable('TEXT') if mime-type has 'text' in type part. And the sub-type part will be ignored. Modified the GlusterHookContentQuery to check server hook's content type if serverId is provided. Change-Id: I191d8f233c965c6b34a652fa07b2c60ef5ee6194 Bug-Url: https://bugzilla.redhat.com/973091 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetGlusterHookContentQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterHookContentType.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterHooksListReturnForXmlRpc.java 3 files changed, 31 insertions(+), 15 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15708 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I191d8f233c965c6b34a652fa07b2c60ef5ee6194 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: gluster hooks - sync with servers action
by kmayilsa＠redhat.com 17 Jun '13

17 Jun '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: gluster hooks - sync with servers action ...................................................................... webadmin: gluster hooks - sync with servers action Added a new action "Sync" to gluster hooks sub-tab. This will invoke the GlusterHooksSyncJob internally to sync the hooks across the servers of the cluster. Change-Id: I3e825620ec15e41a2093b0f84b53a926974d66cb Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGlusterHookListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterGlusterHookView.java 3 files changed, 29 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15574 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e825620ec15e41a2093b0f84b53a926974d66cb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: renamed RefreshGlusterHook action
by kmayilsa＠redhat.com 17 Jun '13

17 Jun '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: renamed RefreshGlusterHook action ...................................................................... gluster: renamed RefreshGlusterHook action changed from "RefreshGlusterHook" to "RefreshGlusterHooks" to be inline with the bll command. Change-Id: I1a53c1d242dd18f83810f940aad3d7239a53b78f Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15573 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1a53c1d242dd18f83810f940aad3d7239a53b78f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Fix disk allocation on new VM/Template
by tnisan＠redhat.com 17 Jun '13

17 Jun '13

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix disk allocation on new VM/Template ...................................................................... webadmin: Fix disk allocation on new VM/Template When creating a new template from VM or a new VM/pool from template the disk allocation table selected the first storage domain as the default storage domain for the operation and that caused all the disks in the template/VM to be created on this storage domain regardless to the storage domain the original disks reside on Change-Id: I0a2f71c4f18ccb52362272edf56853e2fde9643f Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java 1 file changed, 16 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15576 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a2f71c4f18ccb52362272edf56853e2fde9643f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in otopi[master]: packager: yum: refresh yum instance at every transaction
by Alon Bar-Lev 17 Jun '13

17 Jun '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packager: yum: refresh yum instance at every transaction ...................................................................... packager: yum: refresh yum instance at every transaction yum has long history, especially at the suck where it holds the include/exclude data. as we need to handle runtime change in versionlock, we need to force yum to read the information again. I would have expected this information will be re-read at every new transaction... but apparently not. Change-Id: I35a39253aab15b2be27e73be6ac89fe14c9d3f2a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/otopi/packagers/yumpackager.py 1 file changed, 33 insertions(+), 16 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15780 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35a39253aab15b2be27e73be6ac89fe14c9d3f2a Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: packagers: yum: add explicit destructor to miniyum
by Alon Bar-Lev 17 Jun '13

17 Jun '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packagers: yum: add explicit destructor to miniyum ...................................................................... packagers: yum: add explicit destructor to miniyum Change-Id: Ib75f01b8dd26a9f4d0b8748b45322d5619f34ac3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/otopi/miniyum.py 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15779 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib75f01b8dd26a9f4d0b8748b45322d5619f34ac3 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0