August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: LocationMatch add RHEVManagerWeb
by juan.hernandez＠redhat.com 09 Aug '13

09 Aug '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: LocationMatch add RHEVManagerWeb ...................................................................... packaging: LocationMatch add RHEVManagerWeb RHEV-H node do http request for registration into http://<engine>/RHEVManagerWeb/... Change-Id: I559866860398f5d176808938747aae04305fc4dd Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M packaging/conf/ovirt-engine-proxy.conf.v2.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17848 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I559866860398f5d176808938747aae04305fc4dd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: tools: Fix engine-manage-domains logging config
by yzaslavs＠redhat.com 08 Aug '13

08 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fix engine-manage-domains logging config ...................................................................... tools: Fix engine-manage-domains logging config Fixes engine-manage-domains logging config under JBoss EAP. The problem started when JBoss EAP reimplemented log4j classes in JBoss Log Manager, but logging configuration initialization from system property log4j.configuration has not been included in those reimplemented classes. This patch initializes log4j configuration for engine-manage-domains. Change-Id: I882cc4b6d859a630e3dcf2022f37c41135f25571 Bug-Url: https://bugzilla.redhat.com/994191 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17842 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I882cc4b6d859a630e3dcf2022f37c41135f25571 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Fix engine-manage-domains logging config
by yzaslavs＠redhat.com 08 Aug '13

08 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fix engine-manage-domains logging config ...................................................................... tools: Fix engine-manage-domains logging config Fixes engine-manage-domains logging config under JBoss EAP. The problem started when JBoss EAP reimplemented log4j classes in JBoss Log Manager, but logging configuration initialization from system property log4j.configuration has not been included in those reimplemented classes. This patch initializes log4j configuration for engine-manage-domains. Change-Id: I3bb21f27182b68246b0485782ddafa64e387d38a Bug-Url: https://bugzilla.redhat.com/994191 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bb21f27182b68246b0485782ddafa64e387d38a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(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-host-deploy[master]: vdsm: packages: ignore errors when starting network service
by Alon Bar-Lev 08 Aug '13

08 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: packages: ignore errors when starting network service ...................................................................... vdsm: packages: ignore errors when starting network service Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=990980 Change-Id: Ib46f1cec395c69e860318101fd76370d5598bf57 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17831 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib46f1cec395c69e860318101fd76370d5598bf57 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: vdsm: pacakges: stop network manager before starting network
by Alon Bar-Lev 08 Aug '13

08 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: pacakges: stop network manager before starting network ...................................................................... vdsm: pacakges: stop network manager before starting network Change-Id: Iade5655bcbe54cfc1d8b75eeced26aaaa799dbc8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 12 insertions(+), 12 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17635 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iade5655bcbe54cfc1d8b75eeced26aaaa799dbc8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>

1 0

Change in mom[master]: mom: Fix balloon libvirt interface exception
by agl＠us.ibm.com 08 Aug '13

08 Aug '13

Adam Litke has submitted this change and it was merged. Change subject: mom: Fix balloon libvirt interface exception ...................................................................... mom: Fix balloon libvirt interface exception xml_domain.findall function in _getGuaranteedMemory throws SyntaxError("cannot use absolute path on element"), so this patch fixes this by not using absolute path. Change-Id: Iea9dc8cf0ac51de9747a0ebc938761e8ce7135ed Signed-off-by: MeiLiu <liumbj(a)linux.vnet.ibm.com> Reviewed-on: http://gerrit.ovirt.org/17776 Reviewed-by: Mark Wu <wudxw(a)linux.vnet.ibm.com> Reviewed-by: Adam Litke <agl(a)us.ibm.com> --- M mom/HypervisorInterfaces/libvirtInterface.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Looks good to me, approved Mark Wu: Looks good to me, but someone else must approve Mei Liu: Verified -- To view, visit http://gerrit.ovirt.org/17776 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea9dc8cf0ac51de9747a0ebc938761e8ce7135ed Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Mei Liu <liumbj(a)linux.vnet.ibm.com> Gerrit-Reviewer: Adam Litke <agl(a)us.ibm.com> Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Mei Liu <liumbj(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: engine: add nice_level to vm_static update procedure
by lhornyak＠redhat.com 08 Aug '13

08 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: add nice_level to vm_static update procedure ...................................................................... engine: add nice_level to vm_static update procedure Looks like a missing mapping. Change-Id: I0b4241c3ab218cc8540781891f660d4de24b65f2 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17651 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b4241c3ab218cc8540781891f660d4de24b65f2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: call connectStorageServer when add a domain
by amureini＠redhat.com 08 Aug '13

08 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: call connectStorageServer when add a domain ...................................................................... core: call connectStorageServer when add a domain If a storage domain is created with an already existing storage connection, it's possible that the storage connection is not mounted on the host via which the storage domain creation is done, thus resulting a failure in domain creation. This patch adds a connectStorageServer call in domain creation flow to make sure the connection is valid and mount it before the call to create the StorageDomain. Change-Id: Ib93e681a36f439277054bf22d8ecb319e330d14f Bug-Url: https://bugzilla.redhat.com/991797 Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddStorageDomainCommand.java 1 file changed, 31 insertions(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17680 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib93e681a36f439277054bf22d8ecb319e330d14f Gerrit-PatchSet: 4 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> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Add snakeyaml dependency
by ofrenkel＠redhat.com 08 Aug '13

08 Aug '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: Add snakeyaml dependency ...................................................................... core: Add snakeyaml dependency Cloud-Init requires snakeyml in VDS broker, so adding the dependency to it. Change-Id: I6bb434c6f7051a457ca51fb7d4c323a90e830931 Bug-Url: https://bugzilla.redhat.com/994129 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M ear/src/main/resources/META-INF/MANIFEST.MF 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17780 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6bb434c6f7051a457ca51fb7d4c323a90e830931 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine : Updating host's cluster leads to cloning the host
by yzaslavs＠redhat.com 08 Aug '13

08 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: engine : Updating host's cluster leads to cloning the host ...................................................................... engine : Updating host's cluster leads to cloning the host When trying to move host from one gluster cluster to another, under certain conditions the operation fails with an internal server error and the host appears in both clusters. This happens only when the host in the target or source cluster was up during canDoAction and is down during execution of glusterHostRemove or glusterHostAdd restulting in NPE. Change-Id: I1bda0efedd62bf70ae8670861dc3a2ee70ff77dc Bug-Url: https://bugzilla.redhat.com/988033 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java 1 file changed, 24 insertions(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17358 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1bda0efedd62bf70ae8670861dc3a2ee70ff77dc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(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