February 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Remove JBoss specific logging from bll
by amureini＠redhat.com 20 Feb '13

20 Feb '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove JBoss specific logging from bll ...................................................................... core: Remove JBoss specific logging from bll Removed the usage of JBoss specific implementation from bll tests' logging configuration and replaced it with standard log4j configuration for better portability and simplicity. Change-Id: I6988196004df741f37ce4971327c005e068cca5b Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/pom.xml M backend/manager/modules/bll/src/test/resources/log4j.xml 2 files changed, 9 insertions(+), 26 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12168 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6988196004df741f37ce4971327c005e068cca5b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(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: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Remove unused JARs from bll's pom.xml
by amureini＠redhat.com 20 Feb '13

20 Feb '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove unused JARs from bll's pom.xml ...................................................................... core: Remove unused JARs from bll's pom.xml Removed a bunch of JAR dependencies from bll's pom.xml file. These dependencies were probably required when we had integration tests on an embedded JBoss. Since these tests are long gone, there is no reason to hang on to their dependencies. Change-Id: I3312c854bf9da0236f42ad559015ff71d1a08aea Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/pom.xml 1 file changed, 0 insertions(+), 55 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12167 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3312c854bf9da0236f42ad559015ff71d1a08aea Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(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: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Make connection timeout configurable
by dfediuck＠redhat.com 20 Feb '13

20 Feb '13

Doron Fediuck has submitted this change and it was merged. Change subject: core: Make connection timeout configurable ...................................................................... core: Make connection timeout configurable The timeout that we currently use to connect to hosts is by default very large, as it is the same we use for VDSM operations. The default is 180 seconds but waiting that long for connections is not very reasonable, as establishing a connection is a quick operation, specially in a high speed local network. This large timeout severely affects the high availability support, as it means that we need to wait at least 6 minutes (we need at least 2 failures in a row) when a host fails and starts dropping packets before we can declare it failed and start the recovery process. This patch introduces two new parameters that control the time that it takes to react when there are connection issues: 1. vdsConnectionTimeout (default is 3 minutes) Timeout for establishment of connections with hosts. This should be quite small, a few seconds at most, as it the TCP handshake with hosts should be very quick in most networks. At the moment we are using 3 minutes in order to preserve the behaviour before this modification, but this will probably be changed to a much smaller value (2 seconds, for example) in the future. 2. vdsRetries (default is 3) The number of times to retry host operations when there are IO errors. The default value is 3 to match the default value of the XML RPC library that we use, so there is no change in behaviour. Bug-Url: https://bugzilla.redhat.com/863211 Change-Id: I55a49ae5d655c2105c5840decec81ae712e40c32 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M backend/manager/tools/src/main/conf/engine-config.properties 6 files changed, 72 insertions(+), 5 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55a49ae5d655c2105c5840decec81ae712e40c32 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com>

1 0

Change in ovirt-engine[engine_3.1]: restapi: if filter: header is set, use query instead of sear...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: if filter: header is set, use query instead of search (#882946) ...................................................................... restapi: if filter: header is set, use query instead of search (#882946) When the filter option is specified the api query should not throw a insufficient permissions exception. Instead the query should return a list of objects that the user has permissions for. Change-Id: I00eeb96515cb50431f73f8fc66d2730b143f14af Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=882946 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Verified -- To view, visit http://gerrit.ovirt.org/12088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I00eeb96515cb50431f73f8fc66d2730b143f14af Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.1 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]: restapi: if filter: header is set, use query instead of sear...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: if filter: header is set, use query instead of search (#869334) ...................................................................... restapi: if filter: header is set, use query instead of search (#869334) When the filter option is specified the api query should not throw a insufficient permissions exception. Instead the query should return a list of objects that the user has permissions for. Change-Id: I0051d10458fe85cf6660f09ce4efca1aa7c2d952 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=869334 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- 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/BackendVmPoolsResource.java 2 files changed, 13 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12087 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0051d10458fe85cf6660f09ce4efca1aa7c2d952 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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[engine_3.2]: restapi: if filter: header is set, use query instead of sear...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: if filter: header is set, use query instead of search (#869334) ...................................................................... restapi: if filter: header is set, use query instead of search (#869334) When the filter option is specified the api query should not throw a insufficient permissions exception. Instead the query should return a list of objects that the user has permissions for. Change-Id: I0e459fc315753e88e8a1f232a206303568b44492 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=869334 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12085 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e459fc315753e88e8a1f232a206303568b44492 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 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]: restapi : Add /clusters sub-collection under /datacenters/xx...
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi : Add /clusters sub-collection under /datacenters/xxx(#674521) ...................................................................... restapi : Add /clusters sub-collection under /datacenters/xxx(#674521) Adds datacenters/clusters links to see specific data center clusters. Change-Id: I6b27ef501324d890efbe0e8b914d9db47653f146 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=674521 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DataCenterResource.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/BackendClustersResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClusterResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClustersResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCentersResource.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClusterResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterClustersResourceTest.java 9 files changed, 693 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11618 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b27ef501324d890efbe0e8b914d9db47653f146 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine 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-cli[master]: cli: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: prepare for the next development iteration ...................................................................... cli: prepare for the next development iteration Change-Id: If26472c88ecc2e004743d2f2690afa3ecfb5cd12 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12224 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If26472c88ecc2e004743d2f2690afa3ecfb5cd12 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-sdk[master]: sdk: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: prepare for the next development iteration ...................................................................... sdk: prepare for the next development iteration Change-Id: I2af7c2e2d331abdc028877f71ef959f8a7d7b44c Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12223 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2af7c2e2d331abdc028877f71ef959f8a7d7b44c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk 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[cli_3.2]: cli: prepare for the next development iteration
by mpastern＠redhat.com 20 Feb '13

20 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: prepare for the next development iteration ...................................................................... cli: prepare for the next development iteration Change-Id: I6e050073aad7c385bb120101a1a460ce83cca23e Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12222 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e050073aad7c385bb120101a1a460ce83cca23e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0