May 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: core: Fix Storage allocation validation for live merge
by tnisan＠redhat.com 05 May '16

05 May '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix Storage allocation validation for live merge ...................................................................... core: Fix Storage allocation validation for live merge Currently, same storage allocation validation is used both for cold merge and live merge. However, while for cold merge we check allocation for disks and their child snapshots, for live merge we only want to check allocation for disks. This patch fixes the validation accordingly. Change-Id: I7312f35a536ee8999a1ac73f84f7144e8a2fefa2 Bug-Url: https://bugzilla.redhat.com/1318050 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> Signed-off-by: Ala Hino <ahino(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandTest.java 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Ala Hino: Verified -- To view, visit https://gerrit.ovirt.org/57001 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7312f35a536ee8999a1ac73f84f7144e8a2fefa2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cleanup updateInterfaceStatistics
by ahadas＠redhat.com 04 May '16

04 May '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup updateInterfaceStatistics ...................................................................... core: cleanup updateInterfaceStatistics Minor cleanup in (mostly) VmAnalyzer#updateInterfaceStatistics. Change-Id: I317d37b3ad32888cd4d0af95c9c4ae472e421dc9 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java 1 file changed, 20 insertions(+), 21 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/56982 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I317d37b3ad32888cd4d0af95c9c4ae472e421dc9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core, db: Better convention for Image Upload URI configuration
by mlipchuk＠redhat.com 04 May '16

04 May '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core, db: Better convention for Image Upload URI configuration ...................................................................... core, db: Better convention for Image Upload URI configuration The existing configuration for URIs related to Image Upload was not clear, and also split some settings between the engine and proxy. This change: 1. Moves the image daemon port setting to the engine where it belongs. 2. Makes SSL enablement more explicit. 3. Updates the proxy port setting to match the default listening port of the proxy. 4. Uses a more standardized address:port format for the proxy address, which we can later extend to included multiple addresses if desired. Change-Id: Id489f2526513a70991432af7235c7d42b5f16107 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/UploadImageCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 3 files changed, 25 insertions(+), 9 deletions(-) Approvals: Greg Padgett: Verified Jenkins CI: Passed CI tests Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56231 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id489f2526513a70991432af7235c7d42b5f16107 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Moving the image_ng jobs to monitor the maser of ovirt-syste...
by tlitovsk＠redhat.com 04 May '16

04 May '16

Anatoly Litovsky has submitted this change and it was merged. Change subject: Moving the image_ng jobs to monitor the maser of ovirt-system-tests ...................................................................... Moving the image_ng jobs to monitor the maser of ovirt-system-tests Change-Id: Ied66e2236afa61678df8a06e17586ebd34dffd79 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/yaml/templates/system_tests_templates.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests David Caro: Looks good to me Anatoly Litovsky: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/57060 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied66e2236afa61678df8a06e17586ebd34dffd79 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Enforce ascii characters in HTTP headers
by juan.hernandez＠redhat.com 04 May '16

04 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: Enforce ascii characters in HTTP headers ...................................................................... Enforce ascii characters in HTTP headers Since python-pycurl-7.19.0, which is in rhel7 support only string type in headers, we need to explicitly encode to ascii characters in headers Change-Id: I504444815ef5d72c13426ec9e23a92691111b274 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/lib/ovirtsdk4/__init__.py M sdk/tests/server.py M sdk/tests/test_connection_create.py 3 files changed, 45 insertions(+), 3 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/57014 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I504444815ef5d72c13426ec9e23a92691111b274 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Add general exception Error to SDK
by juan.hernandez＠redhat.com 04 May '16

04 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add general exception Error to SDK ...................................................................... Add general exception Error to SDK This patch add general exception Error to SDK, which is thrown always when some exception happend in SDK. Change-Id: I1cdca6cefea230e669cf1f10a8ca71c4addc3b58 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java M sdk/src/main/java/org/ovirt/engine/sdk4/ConnectionBuilder.java A sdk/src/main/java/org/ovirt/engine/sdk4/Error.java M sdk/src/main/java/org/ovirt/engine/sdk4/HttpClient.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/ConnectionBuilder42.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/ConnectionBuilder45.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpClient42.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpClient45.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/HttpConnection.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/SsoUtils.java M sdk/src/main/java/org/ovirt/engine/sdk4/internal/services/ServiceImpl.java M sdk/src/test/java/org/ovirt/engine/sdk4/ConnectionCreateTest.java M sdk/src/test/java/org/ovirt/engine/sdk4/ConnectionTimeoutTest.java 13 files changed, 111 insertions(+), 80 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56951 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1cdca6cefea230e669cf1f10a8ca71c4addc3b58 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Create documentation for generated services
by juan.hernandez＠redhat.com 04 May '16

04 May '16

Juan Hernandez has submitted this change and it was merged. Change subject: Create documentation for generated services ...................................................................... Create documentation for generated services This patch adds documentation to generated services Change-Id: Id608d90f3bcc5b87dac8e8ae34978e677bb4a8ce Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java 1 file changed, 53 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57024 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id608d90f3bcc5b87dac8e8ae34978e677bb4a8ce Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(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]: build: post - ovirt-hosted-engine-ha-1.3.5.5
by msivak＠redhat.com 04 May '16

04 May '16

Martin Sivák has submitted this change and it was merged. Change subject: build: post - ovirt-hosted-engine-ha-1.3.5.5 ...................................................................... build: post - ovirt-hosted-engine-ha-1.3.5.5 Change-Id: Ied13ced416638ef8afa42e7d18a08654bc17a207 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 9 insertions(+), 6 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57046 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied13ced416638ef8afa42e7d18a08654bc17a207 Gerrit-PatchSet: 1 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-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: build: ovirt-hosted-engine-ha-1.3.5.5
by msivak＠redhat.com 04 May '16

04 May '16

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.3.5.5 ...................................................................... build: ovirt-hosted-engine-ha-1.3.5.5 Change-Id: Ibb850a83921d370c9b9fc417c6bf4ff63fd53a77 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 8 insertions(+), 5 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57045 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb850a83921d370c9b9fc417c6bf4ff63fd53a77 Gerrit-PatchSet: 1 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]: webadmin: Automatically set VM hostname
by tjelinek＠redhat.com 04 May '16

04 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Automatically set VM hostname ...................................................................... webadmin: Automatically set VM hostname VMInit hostname is automatically populated by vm_name when CloudInit/SysPrep is enabled. Change-Id: Icff118ef3c97fb3ae177aa5760a262b3d729b87b Bug-Url: https://bugzilla.redhat.com/1270483 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/TemplateVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInitModel.java 5 files changed, 64 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/56950 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icff118ef3c97fb3ae177aa5760a262b3d729b87b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0