November 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Adding new test for read only disk feature.
by amureini＠redhat.com 14 Nov '13

14 Nov '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Adding new test for read only disk feature. ...................................................................... engine: Adding new test for read only disk feature. Change-Id: I4d490eda5df4387ceedf4e0acd6ccd980249b797 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, 26 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21241 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d490eda5df4387ceedf4e0acd6ccd980249b797 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(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-cli[master]: cli: release 3.4.0.1-1
by mpastern＠redhat.com 14 Nov '13

14 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: release 3.4.0.1-1 ...................................................................... cli: release 3.4.0.1-1 Change-Id: I9fae2d02730f51ba46ab924cb626f86346a67b7a Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 19 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21265 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9fae2d02730f51ba46ab924cb626f86346a67b7a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: do not align to middle multi-lined text
by mpastern＠redhat.com 14 Nov '13

14 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: do not align to middle multi-lined text ...................................................................... cli: do not align to middle multi-lined text Change-Id: Ib90316a466fead5b9a888ced72e3c6b0a097f64d Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/format/format.py 1 file changed, 11 insertions(+), 4 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21264 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib90316a466fead5b9a888ced72e3c6b0a097f64d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: move warning output to the terminal formatting
by mpastern＠redhat.com 14 Nov '13

14 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: move warning output to the terminal formatting ...................................................................... cli: move warning output to the terminal formatting Change-Id: I9e185f269b7eaec194e864d20bc24cacfdcf5a52 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/context.py 1 file changed, 13 insertions(+), 6 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21263 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e185f269b7eaec194e864d20bc24cacfdcf5a52 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: move error output to the terminal formatting
by mpastern＠redhat.com 14 Nov '13

14 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: move error output to the terminal formatting ...................................................................... cli: move error output to the terminal formatting Change-Id: I2fc445fe4044bf391ace18f72ed043dfba0d283f Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/cli/context.py M src/ovirtcli/format/format.py M src/ovirtcli/shell/engineshell.py 3 files changed, 47 insertions(+), 19 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21262 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2fc445fe4044bf391ace18f72ed043dfba0d283f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: list xxx --show all - output is not well formatted(#890...
by mpastern＠redhat.com 14 Nov '13

14 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: list xxx --show all - output is not well formatted(#890038) ...................................................................... cli: list xxx --show all - output is not well formatted(#890038) There are indentation differences between the listed devices when there are extra fields in some devices. Change-Id: Ibcaa28d3718f8dd0aceb3935ae2606713eee34e9 Bug-Url: https://bugzilla.redhat.com/890038 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M src/ovirtcli/format/text.py 1 file changed, 27 insertions(+), 5 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21240 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibcaa28d3718f8dd0aceb3935ae2606713eee34e9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: fix cloud-init userData content
by rgolan＠redhat.com 14 Nov '13

14 Nov '13

Roy Golan has submitted this change and it was merged. Change subject: core: fix cloud-init userData content ...................................................................... core: fix cloud-init userData content some changes needed to comply with cloud-init userData requirements: * add needed #cloud-config header to userData file: this fix all commands sent in userData like timezone and cloud-init configuration settings * move password from metaData to userData root password parameter should be in userData, and not in metaData. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1004829 Change-Id: I4642aaf1d9349ebbe82b7c77e562ad5ca19ca3ee Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CloudInitHandler.java 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4642aaf1d9349ebbe82b7c77e562ad5ca19ca3ee Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(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[ovirt-engine-3.3.1]: pacakging: bump rpm release to 2
by iheim＠redhat.com 14 Nov '13

14 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: pacakging: bump rpm release to 2 ...................................................................... pacakging: bump rpm release to 2 Bump release in order to force upgrade of current public RPM Bug-Url: https://bugzilla.redhat.com/1019391 Change-Id: Id831ff88d14f0d785b7f4d4cba270f3d950aa9f1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M version.mak 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21258 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id831ff88d14f0d785b7f4d4cba270f3d950aa9f1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Activate the bricks after data migration
by kmayilsa＠redhat.com 14 Nov '13

14 Nov '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi: Activate the bricks after data migration ...................................................................... restapi: Activate the bricks after data migration Activates the set of bricks of data migration is over on them. If user decide to retain the set of bricks for further use, this action can be invoked. Usage: api/clusters/<cluster-Id>/glustervolumes/<volume-id>/bricks/activate Input Parameters: <action> <bricks> <brick> <name><Brick1-Name></name> </brick> <brick> <name><Brick2-Name></name> </brick> </bricks> </action> Change-Id: Iee2e7234ac2ca78b741bac12d53fc6441d8c7d80 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/gluster/BackendGlusterBricksResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResourceTest.java 4 files changed, 49 insertions(+), 0 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee2e7234ac2ca78b741bac12d53fc6441d8c7d80 Gerrit-PatchSet: 8 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]: restapi: Commit remove brick if migrate executed
by kmayilsa＠redhat.com 14 Nov '13

14 Nov '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi: Commit remove brick if migrate executed ...................................................................... restapi: Commit remove brick if migrate executed Modified the DELETE action which checks if there was a data migration executed on the set of bricks being deleted. If so it executes the commit operation on the same set of bricks. If there was no migration executed earlier, its effectively a force remove brick. Change-Id: I7830749bb84f61dd1a80281d3abe6dc113150fd1 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- 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 M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/GlusterTestHelper.java 3 files changed, 75 insertions(+), 15 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21043 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7830749bb84f61dd1a80281d3abe6dc113150fd1 Gerrit-PatchSet: 11 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: 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