- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: Fix of disablityHint
by tjelinek＠redhat.com 05 May '16

05 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Fix of disablityHint ...................................................................... webadmin: Fix of disablityHint Components based on AbstractValidatedWidgetWithLabel can be disabled (greyed out) with optional tooltip. This tooltip was not removed when re-enabled. This patch ensures tooltip to be removed when `setEnabled(true)` is called. Change-Id: I628c06427f1f7fd1632a018e01224bb2c19e726f Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AbstractValidatedWidgetWithLabel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57050 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I628c06427f1f7fd1632a018e01224bb2c19e726f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Cluster and ServerCpu toString()
by tjelinek＠redhat.com 05 May '16

05 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: Cluster and ServerCpu toString() ...................................................................... core: Cluster and ServerCpu toString() toString() methods added to improve debug experience. Change-Id: Ie5fcf4efa0622bcc3c268498407e9f3f450db2aa Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Cluster.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ServerCpu.java 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57049 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie5fcf4efa0622bcc3c268498407e9f3f450db2aa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: aaa: add new kerberos configuration to jaas
by tnisan＠redhat.com 05 May '16

05 May '16

Tal Nisan has submitted this change and it was merged. Change subject: aaa: add new kerberos configuration to jaas ...................................................................... aaa: add new kerberos configuration to jaas Adds new <security-domain> configuration to JBoss configuration file which allows use of ticket cache and/or keytab to login. Change-Id: Ic36734c65793674fa0b37a7baf0f59b7fc27287b Bug-Url: https://bugzilla.redhat.com/1327041 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 656e11ec1c83718f4425be424d98c2402b066260) --- M packaging/services/ovirt-engine/ovirt-engine.conf.in M packaging/services/ovirt-engine/ovirt-engine.py M packaging/services/ovirt-engine/ovirt-engine.xml.in 3 files changed, 53 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic36734c65793674fa0b37a7baf0f59b7fc27287b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(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]: aaa: exttool: use JBoss JAAS configuration by default
by tnisan＠redhat.com 05 May '16

05 May '16

Tal Nisan has submitted this change and it was merged. Change subject: aaa: exttool: use JBoss JAAS configuration by default ...................................................................... aaa: exttool: use JBoss JAAS configuration by default To be consistent with behaviour of aaa-ldap running in JBoss and extension-tool running outside of JBoss, we provide temporary configuration of JAAS to exttool, which reflect configuration of <security-domains> in JBoss. Change-Id: Idf816a065cc4e4b0a3dc5b378bacd4c2aa214dfb Bug-Url: https://bugzilla.redhat.com/1327041 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit ab4a52b18ee3b75bfafaec6dc1f87f4182e9ec5b) --- M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/core/ExtensionsToolExecutor.java M packaging/bin/ovirt-engine-extensions-tool.sh 2 files changed, 76 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56121 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf816a065cc4e4b0a3dc5b378bacd4c2aa214dfb Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(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.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/57002 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.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>

1 0

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