June 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: removing uicommon from build (pom)
by gchaplik＠redhat.com 11 Jun '12

11 Jun '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: removing uicommon from build (pom) ...................................................................... webadmin: removing uicommon from build (pom) & removing uiQuery references Change-Id: Ie0dc666dedd43a720f5c64ff77b132f885cf1a0a Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/DataProvider.java 3 files changed, 0 insertions(+), 33 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5219 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0dc666dedd43a720f5c64ff77b132f885cf1a0a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: upgrade sql file name sequence number issue fix
by ofrenkel＠redhat.com 11 Jun '12

11 Jun '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: upgrade sql file name sequence number issue fix ...................................................................... engine: upgrade sql file name sequence number issue fix Change-Id: Ic4ea0cdc5c98b2320c537c72b510bdac7b1a361c Signed-off-by: Selvasundaram <sesubram(a)redhat.com> --- R backend/manager/dbscripts/upgrade/03_01_1220_add_volume_brick_order_column.sql 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5247 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4ea0cdc5c98b2320c537c72b510bdac7b1a361c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: VMs created by vmpool fail to run(#830789)
by oourfali＠redhat.com 11 Jun '12

11 Jun '12

Oved Ourfali has submitted this change and it was merged. Change subject: core: VMs created by vmpool fail to run(#830789) ...................................................................... core: VMs created by vmpool fail to run(#830789) https://bugzilla.redhat.com/show_bug.cgi?id=830789 Verify that when a balloon device is created it has the model variable in its specParams map set to 'virtio' Change-Id: I17592c17d66f5280a052f30efe0221bec9573ca0 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java 2 files changed, 9 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Looks good to me, approved Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5246 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17592c17d66f5280a052f30efe0221bec9573ca0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Differentiate Virtual/Actual size in disks
by derez＠redhat.com 11 Jun '12

11 Jun '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Differentiate Virtual/Actual size in disks ...................................................................... webadmin: Differentiate Virtual/Actual size in disks Differentiate between 'Virtual Size' and 'Actual Size' in disks: * Renamed 'Provisioned Size' to 'Virtual Size' * Added 'Virtual Size' column to: ** Storage -> Templates sub-tab ** Templates -> Disks sub-tab Change-Id: If7abf7d6b0b49d906247500e1a4a3a1b4f36002b Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/label/DiskSizeLabel.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/storage/SubTabStorageTemplateView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/template/SubTabTemplateDiskView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/storage/TemplatesTree.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/template/DisksTree.java 7 files changed, 21 insertions(+), 7 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5213 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7abf7d6b0b49d906247500e1a4a3a1b4f36002b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: display SD status in Storage sub-tab
by derez＠redhat.com 11 Jun '12

11 Jun '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: display SD status in Storage sub-tab ...................................................................... webadmin: display SD status in Storage sub-tab https://bugzilla.redhat.com/819307 Display StorageDomainSharedStatus in Templates -> storage sub-tab Change-Id: I8b42f9d31c12d12123fb0590600ef70d02489868 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/template/DisksTree.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5212 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8b42f9d31c12d12123fb0590600ef70d02489868 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Removing default transport type in volume(#830495)
by ofrenkel＠redhat.com 11 Jun '12

11 Jun '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Removing default transport type in volume(#830495) ...................................................................... engine: Removing default transport type in volume(#830495) https://bugzilla.redhat.com/830495 Removed the default populated transport type from the GlusterVolumeEntity. An empty trasport type check is already in place in the CreateVolume command. Change-Id: Ibee994efad0ad0783dac5cd283f8d288ec00239c Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeEntity.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Kanagaraj M: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5227 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibee994efad0ad0783dac5cd283f8d288ec00239c Gerrit-PatchSet: 3 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: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Skipping Cluster CPU validation in GlusterOnly mode
by ofrenkel＠redhat.com 11 Jun '12

11 Jun '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Skipping Cluster CPU validation in GlusterOnly mode ...................................................................... engine: Skipping Cluster CPU validation in GlusterOnly mode while adding a cluster in GluterOnly mode, CPU checks will not be performed. Change-Id: I45b1019bd28c0b48e0fd277437bec260d5a06b84 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Kanagaraj M: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5165 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I45b1019bd28c0b48e0fd277437bec260d5a06b84 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: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Gluster Volume Bricks resource implementation
by ofrenkel＠redhat.com 11 Jun '12

11 Jun '12

Omer Frenkel has submitted this change and it was merged. Change subject: restapi: Gluster Volume Bricks resource implementation ...................................................................... restapi: Gluster Volume Bricks resource implementation Implementation of BackendGlusterBricksResource - get - Add Brick - Remove Brick JUnit test case for above api. Change-Id: I0ab5fc0dd5ef9c2d3eadbc7e78876079c61f46cb Signed-off-by: Dhandapani <dgopal(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/gluster/GlusterCoreUtil.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/gluster/GlusterBricksResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResourceTest.java 10 files changed, 514 insertions(+), 47 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Dhandapani Gopal: Verified -- To view, visit http://gerrit.ovirt.org/4909 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ab5fc0dd5ef9c2d3eadbc7e78876079c61f46cb Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Dhandapani Gopal <dgopal(a)redhat.com> Gerrit-Reviewer: Dhandapani Gopal <dgopal(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Adding brick order to Gluster volume bricks
by ofrenkel＠redhat.com 11 Jun '12

11 Jun '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Adding brick order to Gluster volume bricks ...................................................................... engine: Adding brick order to Gluster volume bricks Introduced the brick order field in GlusterBrickEntity and in gluster_volume_bricks table. Add / update the brick order. Update the volume type and replica/stripe count while adding/removing bricks. Change-Id: I6bce4cf0b75c49b4ab20494c48a012b0bd3d55f6 Signed-off-by: Selvasundaram <sesubram(a)redhat.com> --- M backend/manager/dbscripts/gluster_volumes_sp.sql A backend/manager/dbscripts/upgrade/03_01_1170_add_volume_brick_order_column.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterVolumeRemoveBricksCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java 12 files changed, 204 insertions(+), 39 deletions(-) Approvals: Selvasundaram: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4939 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6bce4cf0b75c49b4ab20494c48a012b0bd3d55f6 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Dhandapani Gopal <dgopal(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Selvasundaram <sesubram(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Disable global transaction at InitVdsOnUp
by mkublin＠redhat.com 11 Jun '12

11 Jun '12

Michael Kublin has submitted this change and it was merged. Change subject: core: Disable global transaction at InitVdsOnUp ...................................................................... core: Disable global transaction at InitVdsOnUp We don't need to open transaction during InitVdsOnUp, all internal commands are called with out or inside own transaction. Change-Id: I0841a171512a8529315076f1bdd676d25b88b7f0 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5229 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0841a171512a8529315076f1bdd676d25b88b7f0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0