June 2014 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[ovirt-host-deploy-1.2]: gluster: Restart nrpe nervice during host deploy.
by Alon Bar-Lev 18 Jun '14

18 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: gluster: Restart nrpe nervice during host deploy. ...................................................................... gluster: Restart nrpe nervice during host deploy. This patch restarts the nrpe service if nrpe configuration is modified during host deploy. This is done for reflecting the config changes. Change-Id: I7d1164d97ce9b95de0107990733dc7ecb1fe3974 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110623 Signed-off-by: ndarshan <dnarayan(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/gluster/nrpe.py 2 files changed, 16 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Darshan N: Verified -- To view, visit http://gerrit.ovirt.org/28902 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d1164d97ce9b95de0107990733dc7ecb1fe3974 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.2 Gerrit-Owner: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: fix entity type setting
by ahadas＠redhat.com 18 Jun '14

18 Jun '14

Arik Hadas has submitted this change and it was merged. Change subject: core: fix entity type setting ...................................................................... core: fix entity type setting When invoking RemoveImage from CreateImageTemplate, we used to set the entity type to VM in the EntityInfo of the RemoveImage command instead of Disk. This patch fix it. Change-Id: I657b63fd5474be91c5eed4db876cc2942aa375ab Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateImageTemplateCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Looks good to me, but someone else must approve Arik Hadas: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28889 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I657b63fd5474be91c5eed4db876cc2942aa375ab Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: [RFE] add support for custom fencing
by emesika＠redhat.com 18 Jun '14

18 Jun '14

Eli Mesika has submitted this change and it was merged. Change subject: core: [RFE] add support for custom fencing ...................................................................... core: [RFE] add support for custom fencing Up to now if a customer wants to add an additional fencing device, he had to tweak the oVirt fencing configuration values while those changes were overridden when engine-setup upgrades oVirt. This patch adds a custom configuration key for each of the relevant fencing configuration keys such that the customer can set those keys using the engin-config utility and the corresponding values are merged with the original after validating that their formats are valid. Change-Id: Ie887b9025d0813c4d101a886614364127aea75a0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=878662 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetFenceConfigurationValueQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 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/BackendResource.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/FenceConfigHelper.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java A backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/FenceConfigValueHelper.java A backend/manager/tools/src/test/java/org/ovirt/engine/core/config/entity/helper/FenceConfigValueHelperTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 13 files changed, 290 insertions(+), 13 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28621 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie887b9025d0813c4d101a886614364127aea75a0 Gerrit-PatchSet: 8 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: gluster: Restart nrpe nervice during host deploy.
by Alon Bar-Lev 18 Jun '14

18 Jun '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: gluster: Restart nrpe nervice during host deploy. ...................................................................... gluster: Restart nrpe nervice during host deploy. This patch restarts the nrpe service if nrpe configuration is modified during host deploy. This is done for reflecting the config changes. Change-Id: I7d1164d97ce9b95de0107990733dc7ecb1fe3974 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110623 Signed-off-by: ndarshan <dnarayan(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/gluster/nrpe.py 2 files changed, 16 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Darshan N: Verified -- To view, visit http://gerrit.ovirt.org/28890 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d1164d97ce9b95de0107990733dc7ecb1fe3974 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Fix MAC pool initalization
by mkolesni＠redhat.com 18 Jun '14

18 Jun '14

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Fix MAC pool initalization ...................................................................... engine: Fix MAC pool initalization Template NICs have no MAC address, so need to avoid null addresses. Since this code will be replaced soon, just added check that MAC is not null. Change-Id: If318029b6ae16a8b02a135afd641d3fd0955cbaf Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/MacPoolManagerRanges.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Mike Kolesnik: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28894 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If318029b6ae16a8b02a135afd641d3fd0955cbaf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-guest-agent[master]: sudoers: fix path to ovirt-shutdown-wrappers.sh
by vfeenstr＠redhat.com 18 Jun '14

18 Jun '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: sudoers: fix path to ovirt-shutdown-wrappers.sh ...................................................................... sudoers: fix path to ovirt-shutdown-wrappers.sh Thanks to John Xue for reporting the issue. Change-Id: I6964e654cff6e0c602ff87ee5502fac2d9db7705 Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M scripts/sudoers.ovirt-guest-agent 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28842 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6964e654cff6e0c602ff87ee5502fac2d9db7705 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Disable support for LinuxBridge plugin
by masayag＠redhat.com 18 Jun '14

18 Jun '14

Moti Asayag has submitted this change and it was merged. Change subject: engine: Disable support for LinuxBridge plugin ...................................................................... engine: Disable support for LinuxBridge plugin The Linux Bridge plug-in is deprecated in the Havana release, and will be removed in the Icehouse release. Therefore the engine should not support it. Therefore it should be removed from the ovirt-integration as well. Change-Id: Iaa05188a5cba9987754058b2492329a6134e52e4 Bug-Url: https://bugzilla.redhat.com/1102018 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/OpenstackNetworkProviderProperties.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/NeutronPluginTranslator.java 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Lior Vernia: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/28790 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa05188a5cba9987754058b2492329a6134e52e4 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: ui: big line spacing in Sysprep/Custom Script editor
by shavivi＠redhat.com 18 Jun '14

18 Jun '14

Shahar Havivi has submitted this change and it was merged. Change subject: ui: big line spacing in Sysprep/Custom Script editor ...................................................................... ui: big line spacing in Sysprep/Custom Script editor https://bugzilla.redhat.com/1109779 Change-Id: Iad212dff08897835677b8fd58b4018367b23bdf4 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/css/BaseVmInitStyle.css 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Shahar Havivi: Verified -- To view, visit http://gerrit.ovirt.org/28882 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad212dff08897835677b8fd58b4018367b23bdf4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core, restapi: removed separator inconsistency..
by emesika＠redhat.com 18 Jun '14

18 Jun '14

Eli Mesika has submitted this change and it was merged. Change subject: core,restapi: removed separator inconsistency.. ...................................................................... core,restapi: removed separator inconsistency.. restapi: removed separator inconsistency, removed duplicates. Change-Id: Id2c72e0527f084e1453da7f720a23fe84dd94db6 Bug-Url: https://bugzilla.redhat.com/1104080 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SearchQueryTest.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/CommonConstants.java M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/QueryHelper.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjects.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 5 files changed, 101 insertions(+), 75 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Liran Zelkha: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28343 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2c72e0527f084e1453da7f720a23fe84dd94db6 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: set device to "lun" on VmInfoBuilder
by amureini＠redhat.com 18 Jun '14

18 Jun '14

Allon Mureinik has submitted this change and it was merged. Change subject: core: set device to "lun" on VmInfoBuilder ...................................................................... core: set device to "lun" on VmInfoBuilder When starting a virtual machine with plugged lun disk using the virtio interface set de device type to "lun" instead of "disk". The rationale behind this patch is to provide the same behavior as when the disk is hotplugged to a running virtual machine: provide the capability of using generic scsi commands on the device inside the virtual machine. Change-Id: I7ceffe36e1145d4783c94f62043dfad694fdd290 Bug-Url: https://bugzilla.redhat.com/1097622 Signed-off-by: Xavi Francisco <xfrancis(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Xavi Francisco: Verified Allon Mureinik: Looks good to me, but someone else must approve Omer Frenkel: Looks good to me, approved Liron Ar: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28843 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7ceffe36e1145d4783c94f62043dfad694fdd290 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xavi Francisco <xfrancis(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Xavi Francisco <xfrancis(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0