November 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: REST API to stop migrate on brick
by kmayilsa＠redhat.com 14 Nov '13

14 Nov '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi: REST API to stop migrate on brick ...................................................................... restapi: REST API to stop migrate on brick REST API for stopping the already started migrate brick asynchronous task on a gluster volume. Usage: - api/clusters/<cluster-Id>/glustervolumes/<volume-id>/bricks/stopmigrate Input Parameters: <action> <bricks> <brick> <name><Brick1-Name></name> </brick> <brick> <name><Brick2-Name></name> </brick> </bricks> </action> Change-Id: I0bba03cf04bed8041cb4e25cac0748afbeed2261 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- 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/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.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/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResourceTest.java 6 files changed, 80 insertions(+), 12 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19411 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0bba03cf04bed8041cb4e25cac0748afbeed2261 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(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]: restapi: Migrates the brick data and mark it for deletion
by kmayilsa＠redhat.com 14 Nov '13

14 Nov '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi: Migrates the brick data and mark it for deletion ...................................................................... restapi: Migrates the brick data and mark it for deletion Migrates the brick data to other existing bricks and marks the brick for deletion. Usage: - api/clusters/<cluster-Id>/glustervolumes/<volume-id>/bricks/migrate Input Parameters: <action> <bricks> <brick> <name><Brick1-Name></name> </brick> <brick> <name><Brick2-Name></name> </brick> </bricks> </action> Change-Id: Iaa88f3402a25316d1717ee914aa77856fbe802db Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- 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.yaml 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/gluster/BackendGlusterBricksResourceTest.java 5 files changed, 147 insertions(+), 33 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19400 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa88f3402a25316d1717ee914aa77856fbe802db Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(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]: gluster: compare GlusterAsyncTask in equals of Entities
by ofrenkel＠redhat.com 14 Nov '13

14 Nov '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster: compare GlusterAsyncTask in equals of Entities ...................................................................... gluster: compare GlusterAsyncTask in equals of Entities Added GlusterAsyncTask comparison to GlusterVolumeEntity's and GlusterBrickEntity's equals method. Change-Id: If85362b23057532ba69fe2b9f6d0e9c5d5753e2b Bug-Url: https://bugzilla.redhat.com/1025013 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- 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/businessentities/gluster/GlusterVolumeEntity.java 2 files changed, 7 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21077 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If85362b23057532ba69fe2b9f6d0e9c5d5753e2b 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: Ramesh N <rnachimu(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[ovirt-engine-3.3]: packaging: setup: generate answerfile readable by user only
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: generate answerfile readable by user only ...................................................................... packaging: setup: generate answerfile readable by user only fix file mode of previous stored answer files at state. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1028748 Change-Id: I3c1ca00b7a56216d7051c7625c5e983d422caab9 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/answerfile.py M packaging/setup/plugins/ovirt-engine-setup/upgrade/__init__.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/answerfile_fixup.py 3 files changed, 65 insertions(+), 0 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21201 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c1ca00b7a56216d7051c7625c5e983d422caab9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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.1]: core: allow identical path for different local domains
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: core: allow identical path for different local domains ...................................................................... core: allow identical path for different local domains StorageServerConnectionCommandBase -> isConnWithSameDetailsExists: Added a specific validation for local domains connections; allowing identical path for local domains residing on different data-centers. Note: as a future work, we should add a unique identification for a local domains connections (e.g. to differentiate between local connections retrieved from the rest-api). Change-Id: I4628d7fa442b959d5582e44386f31518b98852e7 Bug-Url: https://bugzilla.redhat.com/1023739 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageServerConnectionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/StorageServerConnectionCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/AddStorageServerConnectionCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/UpdateStorageServerConnectionCommandTest.java 5 files changed, 87 insertions(+), 25 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21255 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4628d7fa442b959d5582e44386f31518b98852e7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3.1]: Engine: pass default displays head as string
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: Engine: pass default displays head as string ...................................................................... Engine: pass default displays head as string New VM store the heads value as integer, which pass to vdsm to treat it as String. Change-Id: I938e7a9285282f72c6b08b23b804cd84f6f8d060 Bug-Url: https://bugzilla.redhat.com/1018760 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21253 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I938e7a9285282f72c6b08b23b804cd84f6f8d060 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3.1]: UI: support multi monitor only for cluster 3.3 and up
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: UI: support multi monitor only for cluster 3.3 and up ...................................................................... UI: support multi monitor only for cluster 3.3 and up When the cluster is less then 3.3 or non Linux OS we set the SinglePCI checkbox to false and hide it. Change-Id: I5f65dd3d5ca1409402b9b0b93e1585f4bfa47a64 Bug-Url: https://bugzilla.redhat.com/1009391 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 2 files changed, 32 insertions(+), 10 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21252 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f65dd3d5ca1409402b9b0b93e1585f4bfa47a64 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: taskcleaner.sh '-l' option does not prod...
by ofrenkel＠redhat.com 14 Nov '13

14 Nov '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: taskcleaner.sh '-l' option does not prod... ...................................................................... core: taskcleaner.sh '-l' option does not prod... Adding -L to the psql call to log command and its result. Change-Id: Iee255ff5f929d795790088d5cf3475db0b81e04a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1029177 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/setup/dbutils/taskcleaner.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21215 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee255ff5f929d795790088d5cf3475db0b81e04a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: search vms os match and ignore case
by ofrenkel＠redhat.com 14 Nov '13

14 Nov '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: search vms os match and ignore case ...................................................................... core: search vms os match and ignore case search on os matched startsWith instead of equalsIgnoreCase * Added unit test coverage Change-Id: I739cad296389ef0356e204ed00b2546bd4d13c92 Signed-off-by: Roy Golan <rgolan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1003804 --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/OsValueAutoCompleter.java A backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/OsValueAutoCompleterTest.java 2 files changed, 43 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I739cad296389ef0356e204ed00b2546bd4d13c92 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Put back vNIC listener after flush
by lvernia＠redhat.com 14 Nov '13

14 Nov '13

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Put back vNIC listener after flush ...................................................................... webadmin: Put back vNIC listener after flush Since pressing the "Advanced Mode" button in the Add VM dialog causes widgets to flush (without closing the dialog), the vNIC widget listener should be put back to still be able to handle events after the button was pressed. Change-Id: I7cc1583bd629e0cf217d7e941f6fd018032831fb Bug-Url: https://bugzilla.redhat.com/1029339 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/profile/ProfilesInstanceTypeEditor.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21242 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7cc1583bd629e0cf217d7e941f6fd018032831fb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0