Change in ovirt-engine[ovirt-engine-3.6]: core: Fix Storage allocation validation for live merge
by tnisan@redhat.com
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>
8 years, 6 months
Change in ovirt-engine[master]: core: cleanup updateInterfaceStatistics
by ahadas@redhat.com
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>
8 years, 6 months
Change in ovirt-engine[master]: core, db: Better convention for Image Upload URI configuration
by mlipchuk@redhat.com
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>
8 years, 6 months
Change in jenkins[master]: Moving the image_ng jobs to monitor the maser of ovirt-syste...
by tlitovsk@redhat.com
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>
8 years, 6 months
Change in ovirt-engine-sdk[master]: Enforce ascii characters in HTTP headers
by juan.hernandez@redhat.com
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>
8 years, 6 months
Change in ovirt-engine-sdk-java[master]: Add general exception Error to SDK
by juan.hernandez@redhat.com
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>
8 years, 6 months
Change in ovirt-engine-sdk[master]: Create documentation for generated services
by juan.hernandez@redhat.com
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>
8 years, 6 months
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
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>
8 years, 6 months
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
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>
8 years, 6 months
Change in ovirt-engine[master]: webadmin: Automatically set VM hostname
by tjelinek@redhat.com
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>
8 years, 6 months