- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[master]: packaging: setup: floor max memory available to 0, if negative.
by sbonazzo＠redhat.com 18 Nov '15

18 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: floor max memory available to 0, if negative. ...................................................................... packaging: setup: floor max memory available to 0, if negative. Change-Id: I409d9633a4246d40398fd7b07f70a29d9fb38adf Bug-Url: https://bugzilla.redhat.com/1268373 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/memory.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/47799 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I409d9633a4246d40398fd7b07f70a29d9fb38adf Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: packaging: setup: validate available memory for virtual machine
by sbonazzo＠redhat.com 18 Nov '15

18 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: validate available memory for virtual machine ...................................................................... packaging: setup: validate available memory for virtual machine Change-Id: I87d60c3f1149be2d33bf187934b5a31c17138edf Bug-Url: https://bugzilla.redhat.com/1268373 Bug-Url: https://bugzilla.redhat.com/1283141 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/vm/boot_disk.py M src/plugins/ovirt-hosted-engine-setup/vm/memory.py 3 files changed, 123 insertions(+), 81 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47782 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87d60c3f1149be2d33bf187934b5a31c17138edf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Add GET and HEAD methods to the system resource
by juan.hernandez＠redhat.com 18 Nov '15

18 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add GET and HEAD methods to the system resource ...................................................................... restapi: Add GET and HEAD methods to the system resource The "SystemResource" interface generated by the JAX-RS code generator doesn't include the following two methods: @GET Response get(); @HEAD Response head(); Instead of that the methods and the annotations are part of the implementation class. In previous versions of Resteasy this worked correctly, but in the newer version included in WildFly the annotations aren't recognized. As a result requests to /ovirt-engine/api return nothing. To solve this issue this patch changes the JAX-RS code generator so that it explicitly adds these two methods to the "SystemResource" interface. Change-Id: I7a0dfa3bb7e3d7a3087580e79d2fe576a2daec34 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java M backend/manager/modules/restapi/metamodel/tool/src/main/java/org/ovirt/api/metamodel/tool/JaxrsGenerator.java 2 files changed, 22 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Sharon Naftaly: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48727 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a0dfa3bb7e3d7a3087580e79d2fe576a2daec34 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: proxy for import vm not in ppc cluster
by tnisan＠redhat.com 18 Nov '15

18 Nov '15

Tal Nisan has submitted this change and it was merged. Change subject: core: proxy for import vm not in ppc cluster ...................................................................... core: proxy for import vm not in ppc cluster Block import VM from external provider if the proxy host resides in PPC cluster or if there is no host that is up and in cluster with supported compatibility version and also not a PPC cluster (therefore no host in the data center can serve as proxy). Change-Id: I0277803f04aeb121507ec9d7e7efde20d1f03e03 Bug-Url: https://bugzilla.redhat.com/1280264 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromExternalProviderCommand.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 4 files changed, 6 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Passed CI tests Shahar Havivi: Looks good to me, approved Michal Skrivanek: Verified -- To view, visit https://gerrit.ovirt.org/48429 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0277803f04aeb121507ec9d7e7efde20d1f03e03 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: proxy for import VM in cluster 3.6 or higher
by tnisan＠redhat.com 18 Nov '15

18 Nov '15

Tal Nisan has submitted this change and it was merged. Change subject: core: proxy for import VM in cluster 3.6 or higher ...................................................................... core: proxy for import VM in cluster 3.6 or higher Block import VM from external provider if the selected proxy host is in cluster with compatibility version lower than 3.6 or if there is no host that is up and resides in cluster with compatibility version which is 3.6 or higher in the data center (therefore no host is capable to serve as proxy). Change-Id: I5aad27d1bde487338c889af6c37d4c2af386652f Bug-Url: https://bugzilla.redhat.com/1280264 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromExternalProviderCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/FeatureSupported.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/errors/EngineMessage.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 8 files changed, 49 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/48428 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aad27d1bde487338c889af6c37d4c2af386652f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: logging: improving logging capabilities
by msivak＠redhat.com 18 Nov '15

18 Nov '15

Martin Sivák has submitted this change and it was merged. Change subject: logging: improving logging capabilities ...................................................................... logging: improving logging capabilities Improving logging capabilities if consumed by an external tool with its own logging framework. Change-Id: I6eaa9289aae64b1ffab6ff0c071d21d9289ea93d Bug-Url: https://bugzilla.redhat.com/1278725 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/storage_backends.py 1 file changed, 82 insertions(+), 43 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48344 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6eaa9289aae64b1ffab6ff0c071d21d9289ea93d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Use version 3.0.3 of the Findbugs plugin
by sbonazzo＠redhat.com 18 Nov '15

18 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: core: Use version 3.0.3 of the Findbugs plugin ...................................................................... core: Use version 3.0.3 of the Findbugs plugin Version 3.0.3 of the Findbugs Maven plugin has been recently released, and it added support for multiple filter files. This means that we no longer need to use version 2.x of the plugin and force it to use version 3.x of Findbugs itself. Change-Id: Ic9249810603c6219c68e39e974913eae7343612a Related-To: https://gerrit.ovirt.org/46403 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/extensions-api-root/pom.xml M backend/manager/modules/restapi/metamodel/pom.xml M pom.xml 3 files changed, 5 insertions(+), 26 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48684 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic9249810603c6219c68e39e974913eae7343612a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Triggering on draft-published too
by dcaro＠redhat.com 18 Nov '15

18 Nov '15

David Caro has submitted this change and it was merged. Change subject: Triggering on draft-published too ...................................................................... Triggering on draft-published too Now all the triggers that happened on patch-created, are also happening on draft published, as we missed some patchsets that were published but had no code changes Change-Id: Iad84fa6e04d8e1561ad1958fc7dc44789eca807d Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/jenkins/jenkins_check_yaml.yaml M jobs/confs/yaml/triggers/gerrit.yaml M jobs/confs/yaml/triggers/standard.yaml 3 files changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/48716 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad84fa6e04d8e1561ad1958fc7dc44789eca807d Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: build: post ovirt-engine-3.6.1 branching
by tnisan＠redhat.com 18 Nov '15

18 Nov '15

Tal Nisan has submitted this change and it was merged. Change subject: build: post ovirt-engine-3.6.1 branching ...................................................................... build: post ovirt-engine-3.6.1 branching Change-Id: I909f507d7b12a583bd726defce3d0e01d35d466b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M backend/manager/dependencies/common/pom.xml M backend/manager/dependencies/pom.xml M backend/manager/dependencies/tools/pom.xml M backend/manager/extensions-tool/pom.xml M backend/manager/modules/aaa/pom.xml M backend/manager/modules/auth-plugin/pom.xml M backend/manager/modules/bll/pom.xml M backend/manager/modules/branding/pom.xml M backend/manager/modules/builtin-extensions/pom.xml M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/dal/pom.xml M backend/manager/modules/docs/pom.xml M backend/manager/modules/extensions-manager/pom.xml M backend/manager/modules/logger/pom.xml M backend/manager/modules/pom.xml M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml M backend/manager/modules/restapi/interface/common/pom.xml M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/restapi/interface/pom.xml M backend/manager/modules/restapi/jaxrs/pom.xml M backend/manager/modules/restapi/pom.xml M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/webapp/pom.xml M backend/manager/modules/root/pom.xml M backend/manager/modules/scheduler/pom.xml M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/services/pom.xml M backend/manager/modules/utils/pom.xml M backend/manager/modules/uutils/pom.xml M backend/manager/modules/vdsbroker/pom.xml M backend/manager/modules/welcome/pom.xml M backend/manager/pom.xml M backend/manager/tools/pom.xml M backend/pom.xml M build-tools-root/checkstyles/pom.xml M build-tools-root/jboss-modules-maven-plugin/pom.xml M build-tools-root/ovirt-checkstyle-extension/pom.xml M build-tools-root/pom.xml M build/validations/pom.xml M ear/pom.xml M frontend/pom.xml M frontend/webadmin/modules/frontend-symbols/pom.xml M frontend/webadmin/modules/frontend/pom.xml M frontend/webadmin/modules/gwt-common/pom.xml M frontend/webadmin/modules/gwt-extension/pom.xml M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/uicommonweb/pom.xml M frontend/webadmin/modules/uicompat/pom.xml M frontend/webadmin/modules/userportal-gwtp/pom.xml M frontend/webadmin/modules/webadmin/pom.xml M frontend/webadmin/pom.xml M mavenmake/pom.xml M pom.xml M version.mak 55 files changed, 55 insertions(+), 55 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48709 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I909f507d7b12a583bd726defce3d0e01d35d466b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in jenkins[master]: ovirt-engine_standard: added 3.6.1 builders
by sbonazzo＠redhat.com 18 Nov '15

18 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine_standard: added 3.6.1 builders ...................................................................... ovirt-engine_standard: added 3.6.1 builders Change-Id: I81cff54a1d1b7fe62724e230bfdaa768211f75ef Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/48718 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81cff54a1d1b7fe62724e230bfdaa768211f75ef Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0