- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: REST api for stop rebalance of volume
by mpastern＠redhat.com 15 Oct '13

15 Oct '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: REST api for stop rebalance of volume ...................................................................... restapi: REST api for stop rebalance of volume REST api for stop rebalance of volume Change-Id: I12c4d83e2c7e8c89dd9f820bc26ef369a4cb0490 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/gluster/GlusterVolumeRebalanceParameters.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/gluster/GlusterVolumeResource.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/BackendGlusterVolumeResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumeResourceTest.java 5 files changed, 44 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19167 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I12c4d83e2c7e8c89dd9f820bc26ef369a4cb0490 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Prevent empty MAC Address range
by masayag＠redhat.com 15 Oct '13

15 Oct '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Prevent empty MAC Address range ...................................................................... engine: Prevent empty MAC Address range The patch prevents a user from entering a mac address range which might result in an empty pool, for example a range which contains multicast mac addresses only. Change-Id: I24113379e8fbbc15a63bdb1be4e6c719d7cb764f Bug-Url: https://bugzilla.redhat.com/1011912 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManager.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/MacAddressRangeUtils.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelper.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/MacAddressPoolRangesValueHelperTest.java 4 files changed, 73 insertions(+), 73 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20155 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24113379e8fbbc15a63bdb1be4e6c719d7cb764f Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: build: ovirt-hosted-engine-setup-1.0.0-0.6.beta1
by sbonazzo＠redhat.com 15 Oct '13

15 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-1.0.0-0.6.beta1 ...................................................................... build: ovirt-hosted-engine-setup-1.0.0-0.6.beta1 Change-Id: I800909067a75ff4b15b7c3f28af022d35d358a68 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 12 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20167 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I800909067a75ff4b15b7c3f28af022d35d358a68 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.0]: vdsm: use startMonitoringDomain
by sbonazzo＠redhat.com 15 Oct '13

15 Oct '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: use startMonitoringDomain ...................................................................... vdsm: use startMonitoringDomain Use startMonitoringDomain VDSM API for monitoring hosted engine storage domain in order to support shared:exclusive images without a pool. Use getVdsStats for waiting the start of the domain monitor. Updated vdsm minimum version required according to VDSM API documentation for startMonitoringDomain. Change-Id: I3099081b79d461685901a1a32e9d526bd3bb758a Requires: http://gerrit.ovirt.org/19762 Requires: http://gerrit.ovirt.org/20099 Bug-Url: https://bugzilla.redhat.com/1015136 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in M src/ovirt_hosted_engine_setup/tasks.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 3 files changed, 53 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3099081b79d461685901a1a32e9d526bd3bb758a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: agent: fix arguments to vdsClient migrate invocation
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: fix arguments to vdsClient migrate invocation ...................................................................... agent: fix arguments to vdsClient migrate invocation The run_vds_client_command() function expects a boolean parameter for ssl state but was being sent a text string, causing migration to fail. Change-Id: I25aaaa3d6cc32a8e36e6366a24cf2db43c0039b0 Bug-Url: https://bugzilla.redhat.com/1015721 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 7 insertions(+), 5 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19873 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25aaaa3d6cc32a8e36e6366a24cf2db43c0039b0 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: lib: fix acceptance of keyword arguments for vdsm api
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: lib: fix acceptance of keyword arguments for vdsm api ...................................................................... lib: fix acceptance of keyword arguments for vdsm api A prior refactoring broke keyword argument usage in calls to the run_vds_client_cmd() method. This fix allows for both argument lists and keyword arguments, and automatically converts the keyword arguments to a dict that is accepted by the vdsm api. Callers should be careful to use the proper parameter form as required by the vdsm api. Change-Id: I9931d6c87b0798152bf344ddd8b9328ce01565a8 Bug-Url: https://bugzilla.redhat.com/1015721 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/vds_client.py 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19872 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9931d6c87b0798152bf344ddd8b9328ce01565a8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: client: API to retrieve local host score
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: client: API to retrieve local host score ...................................................................... client: API to retrieve local host score Add client API to retrieve score of local host. Part of a series of patches to enable maintenance mode for the HA hosts and engine VM. Change-Id: I49043a33e927393941e63a89e4e93519431f3695 Bug-Url: https://bugzilla.redhat.com/1015724 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/client/client.py 1 file changed, 28 insertions(+), 0 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20128 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49043a33e927393941e63a89e4e93519431f3695 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: lib, agent, client: change host_id key to int
by gpadgett＠redhat.com 14 Oct '13

14 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: lib, agent, client: change host_id key to int ...................................................................... lib, agent, client: change host_id key to int get_stats_from_storage() previously returned a dict with the host_id as a string. With all valid host_id values being integers, it makes the api cleaner to return an integer. Change-Id: I56b204080391cf09b2606316d9195bf12905034f Bug-Url: https://bugzilla.redhat.com/1015724 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/lib/brokerlink.py 3 files changed, 9 insertions(+), 9 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20127 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56b204080391cf09b2606316d9195bf12905034f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: insert formatting options with default value.
by amureini＠redhat.com 14 Oct '13

14 Oct '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: insert formatting options with default value. ...................................................................... engine: insert formatting options with default value. Add formatting options that were not in the configuration file, but were used by the formatter with their default values anyway. So adding those formatting options with same default value gives better visibility. Change-Id: Iba5ae1aa8154a8a778735749419ac8184f001d36 Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M config/engine-code-format.xml 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba5ae1aa8154a8a778735749419ac8184f001d36 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: remove handling GetStorageDomainListError after getI...
by amureini＠redhat.com 14 Oct '13

14 Oct '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: remove handling GetStorageDomainListError after getImagesList ...................................................................... engine: remove handling GetStorageDomainListError after getImagesList getImagesList API used by GetImagesListCommand can't return GetStorageDomainListError therefore special handling of this error is removed. Change-Id: I56d639d83bd24e6b3939933998d17c5498670b5a Signed-off-by: Sergey Gotliv <sgotliv(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/GetImagesListVDSCommand.java 1 file changed, 6 insertions(+), 22 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19858 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56d639d83bd24e6b3939933998d17c5498670b5a 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: Daniel Erez <derez(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