September 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: spec: use system jars
by yzaslavs＠redhat.com 22 Sep '13

22 Sep '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: packaging: spec: use system jars ...................................................................... packaging: spec: use system jars PACKAGE POM FC19 EL6 apache-commons-compress 1.4.1 1.5-1 N/A apache-commons-configuration 1.6 1.9-4 N/A postgresql-jdbc 9.1-901-1.jdbc4 9.2.1002-2 8.4.701-8(*) spring-asm 3.1.1 N/A N/A objectweb-asm N/A 3.3.1-7 N/A apache-commons-jxpath 1.3 1.3-15 N/A apache-mina 2.0.1 2.0.4-6 N/A apache-sshd 0.7.0 0.7.0-3 N/A jakarta-commons-httpclient 3.1 3.1-13 N/A quartz 2.1.2 2.1.2-7 N/A spring-ldap 1.3.1 1.3.1-5 N/A springframework-* 1.3.1 1.3.1 N/A ws-commons-util 1.0.2 1.0.1-26 1.0.1-13 xmlrpc-client 3.1.3 3.1.3-5 N/A (*) too low. objectweb-asm is used instead of spring-asm, without we get the following at server.log: Caused by: java.lang.NoClassDefFoundError: org/objectweb/asm/ClassVisitor at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.<init>(AbstractAutowireCapableBeanFactory.java:121) at org.springframework.beans.factory.support.DefaultListableBeanFactory.<init>(DefaultListableBeanFactory.java:150) at org.springframework.jdbc.support.SQLErrorCodesFactory.<init>(SQLErrorCodesFactory.java:103) at org.springframework.jdbc.support.SQLErrorCodesFactory.<clinit>(SQLErrorCodesFactory.java:68) at org.springframework.jdbc.support.SQLErrorCodeSQLExceptionTranslator.setDataSource(SQLErrorCodeSQLExceptionTranslator.java:141) at org.springframework.jdbc.support.SQLErrorCodeSQLExceptionTranslator.<init>(SQLErrorCodeSQLExceptionTranslator.java:104) at org.ovirt.engine.core.dal.dbbroker.CustomSQLErrorCodeSQLExceptionTranslator.<init>(CustomSQLErrorCodeSQLExceptionTranslator.java:10) Change-Id: I97995665ce34aeb2dd24d69b96919999d5498f43 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 64 insertions(+), 0 deletions(-) Approvals: Allon Mureinik: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19060 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I97995665ce34aeb2dd24d69b96919999d5498f43 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: JBoss modules update
by yzaslavs＠redhat.com 22 Sep '13

22 Sep '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: JBoss modules update ...................................................................... core: JBoss modules update When the JBoss modules mechanism was introduced all the dependencies where added to a single "dependencies" module. This patch separates them into different modules. Change-Id: Iec23626539af13f2ae3cd4edb1610198bef8eef4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/dependencies/pom.xml M backend/manager/dependencies/src/main/modules/com/woorea/openstack/sdk/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/commons/compress/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/commons/httpclient/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/mina/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/sshd/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/ws/commons/main/module.xml A backend/manager/dependencies/src/main/modules/org/apache/xmlrpc/main/module.xml D backend/manager/dependencies/src/main/modules/org/ovirt/engine/core/dependencies/main/module.xml A backend/manager/dependencies/src/main/modules/org/ovirt/otopi/main/module.xml A backend/manager/dependencies/src/main/modules/org/ovirt/ovirt-host-deploy/main/module.xml M backend/manager/dependencies/src/main/modules/org/postgresql/main/module.xml A backend/manager/dependencies/src/main/modules/org/quartz/main/module.xml A backend/manager/dependencies/src/main/modules/org/springframework/main/module.xml M backend/manager/modules/common/src/main/modules/org/ovirt/engine/core/common/main/module.xml M backend/manager/modules/compat/src/main/modules/org/ovirt/engine/core/compat/main/module.xml M backend/manager/modules/dal/src/main/modules/org/ovirt/engine/core/dal/main/module.xml M backend/manager/modules/searchbackend/src/main/modules/org/ovirt/engine/core/searchbackend/main/module.xml M backend/manager/modules/utils/src/main/modules/org/ovirt/engine/core/utils/main/module.xml M backend/manager/tools/src/main/modules/org/ovirt/engine/core/tools/main/module.xml M ear/src/main/resources/META-INF/MANIFEST.MF 21 files changed, 288 insertions(+), 185 deletions(-) Approvals: Allon Mureinik: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19404 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec23626539af13f2ae3cd4edb1610198bef8eef4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.1]: packaging: spec: minor cleanup
by Alon Bar-Lev 20 Sep '13

20 Sep '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: spec: minor cleanup ...................................................................... packaging: spec: minor cleanup Change-Id: I257450a0e04700c87770ddd5cd347b63af06bb22 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19421 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I257450a0e04700c87770ddd5cd347b63af06bb22 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.1]: packaging: spec: fix source urls
by Alon Bar-Lev 20 Sep '13

20 Sep '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: spec: fix source urls ...................................................................... packaging: spec: fix source urls Change-Id: I7044fc5408b99cf376a45bf3871b1661fdae73a8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19420 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7044fc5408b99cf376a45bf3871b1661fdae73a8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.1]: packaging: spec: fedora fixups
by Alon Bar-Lev 20 Sep '13

20 Sep '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: spec: fedora fixups ...................................................................... packaging: spec: fedora fixups Change-Id: I9fe1e8b3783dfccd2d02d256044744877a5d953e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9fe1e8b3783dfccd2d02d256044744877a5d953e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.1 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: packaging: spec: minor cleanup
by Alon Bar-Lev 20 Sep '13

20 Sep '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: spec: minor cleanup ...................................................................... packaging: spec: minor cleanup Change-Id: I257450a0e04700c87770ddd5cd347b63af06bb22 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19418 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I257450a0e04700c87770ddd5cd347b63af06bb22 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>

1 0

Change in ovirt-host-deploy[master]: packaging: spec: fix source urls
by Alon Bar-Lev 20 Sep '13

20 Sep '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: spec: fix source urls ...................................................................... packaging: spec: fix source urls Change-Id: I7044fc5408b99cf376a45bf3871b1661fdae73a8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy-offline.spec.in M ovirt-host-deploy.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19417 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7044fc5408b99cf376a45bf3871b1661fdae73a8 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>

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: bridge name in hosted-engine.conf
by sbonazzo＠redhat.com 20 Sep '13

20 Sep '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: bridge name in hosted-engine.conf ...................................................................... packaging: setup: bridge name in hosted-engine.conf Added bridge key in hosted-engine.conf allowing the HA agent to parse it in an easy way. The value is now assigned from environment variable instead of being hardcoded in the template Change-Id: I698d6e7609ef20dff0426b07028ca3391e82a0d7 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/core/conf.py M src/plugins/ovirt-hosted-engine-setup/vm/configurevm.py M templates/hosted-engine.conf.in M templates/vm.conf.in 4 files changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/19413 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I698d6e7609ef20dff0426b07028ca3391e82a0d7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.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]: conf: rebased bridge configuration
by gpadgett＠redhat.com 20 Sep '13

20 Sep '13

Greg Padgett has submitted this change and it was merged. Change subject: conf: rebased bridge configuration ...................................................................... conf: rebased bridge configuration Previously the bridge configuration was read from vm.conf. Now is read from hosted-engine.conf. Change-Id: I85d4c0ea98819dcf423f10900d01862a215a2f3d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/env/config.py 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified Greg Padgett: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19414 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85d4c0ea98819dcf423f10900d01862a215a2f3d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.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-setup[master]: UX: fix typos
by sbonazzo＠redhat.com 20 Sep '13

20 Sep '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: UX: fix typos ...................................................................... UX: fix typos Fixed some typos left around. Change-Id: Ib8ee2968013636e8ba7471e1b26b9788c3c5f54a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/cpu.py M src/plugins/ovirt-hosted-engine-setup/vm/image.py M src/plugins/ovirt-hosted-engine-setup/vm/memory.py 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19412 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib8ee2968013636e8ba7471e1b26b9788c3c5f54a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.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