October 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-cli[cli_3.6]: cli: Release 3.6.9.1
by juan.hernandez＠redhat.com 05 Oct '16

05 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Release 3.6.9.1 ...................................................................... cli: Release 3.6.9.1 Change-Id: I645149467c5579bfe0fc83a9a8ddb3f68a2b2a63 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 5 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65110 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I645149467c5579bfe0fc83a9a8ddb3f68a2b2a63 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-cli[cli_3.6]: Use alternate PKIResourceServlet to fetch CA certificate
by juan.hernandez＠redhat.com 05 Oct '16

05 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use alternate PKIResourceServlet to fetch CA certificate ...................................................................... Use alternate PKIResourceServlet to fetch CA certificate Since oVirt 4.0 there is no ca.crt located in root of the apache, but there is running servlet to fetch the CA certificate. This patch changes the behaviour so we first try to fetch the CA certificate from the root of the apache and if that don't exists, we try to fetch it from new PKIResourceServlet servlet. Change-Id: I5da25e4be36bee56c5ec9ee0dabead574dedada1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1362222 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> (cherry picked from commit 40ecfc7c921dd1e320810d81bb01ba91e2526f1e) --- M src/ovirtcli/platform/posix/spice.py 1 file changed, 12 insertions(+), 7 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65109 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5da25e4be36bee56c5ec9ee0dabead574dedada1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-cli[master]: Use alternate PKIResourceServlet to fetch CA certificate
by juan.hernandez＠redhat.com 05 Oct '16

05 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use alternate PKIResourceServlet to fetch CA certificate ...................................................................... Use alternate PKIResourceServlet to fetch CA certificate Since oVirt 4.0 there is no ca.crt located in root of the apache, but there is running servlet to fetch the CA certificate. This patch changes the behaviour so we first try to fetch the CA certificate from the root of the apache and if that don't exists, we try to fetch it from new PKIResourceServlet servlet. Change-Id: I5da25e4be36bee56c5ec9ee0dabead574dedada1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1362222 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M src/ovirtcli/platform/posix/spice.py 1 file changed, 12 insertions(+), 7 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65093 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5da25e4be36bee56c5ec9ee0dabead574dedada1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> 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-guest-agent[master]: agent: apply virtual-guest tuning profile on installation
by vfeenstr＠redhat.com 05 Oct '16

05 Oct '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: agent: apply virtual-guest tuning profile on installation ...................................................................... agent: apply virtual-guest tuning profile on installation Change-Id: I51691439193ec6da881459a5e7c9f798aa9f618c Signed-off-by: Gal Hammer <ghammer(a)redhat.com> Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent.spec 1 file changed, 29 insertions(+), 0 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65107 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51691439193ec6da881459a5e7c9f798aa9f618c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: aaa: Consolidate SSO session validation requests
by tnisan＠redhat.com 05 Oct '16

05 Oct '16

Tal Nisan has submitted this change and it was merged. Change subject: aaa: Consolidate SSO session validation requests ...................................................................... aaa: Consolidate SSO session validation requests Engine makes multiple queries to SSO to validate each exisitng session. This could cause performance issue when there are thousands of sessions. Combining all requests into one request to fetch all the statuses in one request can resolve the performance issue. When SSO clears an engine session associated with a token by invoking TerminateSessionsForToken SSO_ACCESS_TOKEN_PARAMETER value in the map is null. This can lead to problems when equality is checked with the contents of the map for the token. Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17 Bug-Url: https://bugzilla.redhat.com/1379805 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/SsoOAuthServiceUtils.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/aaa/SessionDataContainerTest.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/search/DirectorySearch.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoConstants.java 5 files changed, 58 insertions(+), 25 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65095 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(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-setup[master]: packaging: gluster: fix a typo in packages plugin
by sbonazzo＠redhat.com 05 Oct '16

05 Oct '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: gluster: fix a typo in packages plugin ...................................................................... packaging: gluster: fix a typo in packages plugin Fixed gluster-server -> glusterfs-server typo. Change-Id: I2b5f78a8bcea79fa6e4745c42d82c22325dff06b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1377851 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/gr-he-setup/system/packages.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65104 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b5f78a8bcea79fa6e4745c42d82c22325dff06b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: aaa: Checkstyles fixes for SessionDataContainerTest
by mperina＠redhat.com 04 Oct '16

04 Oct '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Checkstyles fixes for SessionDataContainerTest ...................................................................... aaa: Checkstyles fixes for SessionDataContainerTest Fix checkstyles issue in SessionDataContainerTest Change-Id: I72c228bf427e6f1eafdcc999d8bb3048a67960e9 Bug-Url: https://bugzilla.redhat.com/1379805 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/aaa/SessionDataContainerTest.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65099 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72c228bf427e6f1eafdcc999d8bb3048a67960e9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: el7: switch to 4.1 candidate repo
by sbonazzo＠redhat.com 04 Oct '16

04 Oct '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: el7: switch to 4.1 candidate repo ...................................................................... el7: switch to 4.1 candidate repo CentOS Virt SIG 4.1 repositories have been created. Moving master to 4.1 Change-Id: I3ad76ae6d76e047aa913736b2827959961802c0b --- M configure.ac M ovirt-el7-deps.repo.in 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65094 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ad76ae6d76e047aa913736b2827959961802c0b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release 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

Change in ovirt-engine[master]: webadmin: Add TODO for future fixing
by awels＠redhat.com 04 Oct '16

04 Oct '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Add TODO for future fixing ...................................................................... webadmin: Add TODO for future fixing Change-Id: Ie7826661bd69cdbecdfd9b27c3f002539a3abb24 Related-To: https://bugzilla.redhat.com/1368101 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/UploadImagePopupView.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/65092 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7826661bd69cdbecdfd9b27c3f002539a3abb24 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Enable HTTP compression by default
by omachace＠redhat.com 04 Oct '16

04 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Enable HTTP compression by default ...................................................................... Enable HTTP compression by default To improve performance of SDK, this patch enable compression by default. Change-Id: I27fdbc6bdde6c4a906d95f644c8421737755e358 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1370485 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/src/main/java/org/ovirt/engine/sdk4/ConnectionBuilder.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65088 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27fdbc6bdde6c4a906d95f644c8421737755e358 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0