April 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.1]: core, engine, aaa: RestAPI connection fail after time change
by Code Review 02 Apr '17

02 Apr '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core, engine, aaa: RestAPI connection fail after time change ...................................................................... core, engine, aaa: RestAPI connection fail after time change Configure apache http pooling connection manager to validate idle connections and retry failed requests. Change-Id: I16876681f0cbf9d4dc36ffbc6d6f9442addc1fa9 Bug-Url: https://bugzilla.redhat.com/1434326 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/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoUtils.java M backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/net/HttpClientBuilder.java M packaging/services/ovirt-engine/ovirt-engine.conf.in 4 files changed, 32 insertions(+), 0 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/74988 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I16876681f0cbf9d4dc36ffbc6d6f9442addc1fa9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(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-engine[master]: microbenchmarks: Introduce microbenchmarks using JMH
by Code Review 02 Apr '17

02 Apr '17

>From Roy Golan <rgolan(a)redhat.com>: Roy Golan has submitted this change and it was merged. Change subject: microbenchmarks: Introduce microbenchmarks using JMH ...................................................................... microbenchmarks: Introduce microbenchmarks using JMH See: http://openjdk.java.net/projects/code-tools/jmh/ Motivation: Microbenchmark when you don't know how pieces of code behave in runtime. For example, make sure encryption and decryption performs well and doesn't affect the throughput of query for vds from the db. Change: Add encryption and decryption benchmarks based on EngineEncryptionUtils Result: Run Benchmarks from your IDE or using mvn or straight java: - mvn package creates an uber jar target/benchmarks.jar - run it with java -jar target/benchmarks.jar - or run mvn test -P benchmarks Sample output of running 'mvn test -P benchmarks': Benchmark Mode Samples Score Score error Units Benchmark.decryption thrpt 50 0.099 0.002 ops/ms Benchmark.encryption thrpt 50 0.237 0.003 ops/ms Benchmark.decryption avgt 50 10.038 0.166 ms/op Benchmark.encryption avgt 50 4.419 0.239 ms/op Benchmark.decryption sample 5320 10.381 0.064 ms/op Benchmark.encryption sample 12864 4.280 0.015 ms/op Benchmark.decryption ss 50 14.313 1.430 ms Benchmark.encryption ss 50 8.725 0.683 ms Process finished with exit code 0 Change-Id: If93cfe18435f8c9fc16973296e4f5e598eb0b248 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- A backend/manager/modules/microbenchmarks/exclude-filters.xml A backend/manager/modules/microbenchmarks/pom.xml A backend/manager/modules/microbenchmarks/src/main/java/org/ovirt/engine/benchmarks/EncryptionDecryptionBenchmark.java A backend/manager/modules/microbenchmarks/src/main/resources/key.p12 M backend/manager/modules/pom.xml 5 files changed, 220 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Roy Golan: Verified; Looks good to me, approved Moti Asayag: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/74537 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If93cfe18435f8c9fc16973296e4f5e598eb0b248 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: EngineEncryptionUtils type inference
by Code Review 02 Apr '17

02 Apr '17

>From Moti Asayag <masayag(a)redhat.com>: Moti Asayag has submitted this change and it was merged. Change subject: core: EngineEncryptionUtils type inference ...................................................................... core: EngineEncryptionUtils type inference Change-Id: I5e7e1dda5b99c9802113d45dfaa5a0d7c88d1e13 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/crypt/EngineEncryptionUtils.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/75012 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e7e1dda5b99c9802113d45dfaa5a0d7c88d1e13 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0