- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Fix default value of "insecure" connection parameter
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Fix default value of "insecure" connection parameter ...................................................................... Fix default value of "insecure" connection parameter By default the connections are secured, and the certificates of the server are checked, but the documentation of the "insecure" parameter suggest the contrary. This patch fixes that inconsistency. Change-Id: Ie1f1528ac095b7a3b336af8c01dbb4edfb8cf62a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirtsdk4/http.rb 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54921 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie1f1528ac095b7a3b336af8c01dbb4edfb8cf62a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: utils: Add missing javax.activation.api dependency
by mperina＠redhat.com 18 Mar '16

18 Mar '16

Martin Peřina has submitted this change and it was merged. Change subject: utils: Add missing javax.activation.api dependency ...................................................................... utils: Add missing javax.activation.api dependency Adds missing javax.activation.api dependency into module.xml which caused error during engine deploy on WildFly 10.0.0.GA. Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/utils/src/main/modules/org/ovirt/engine/core/utils/main/module.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54913 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I91dcc6a2062b10fbdc1737d4cf5267cbdd936fb1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Use only query parameters
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use only query parameters ...................................................................... Use only query parameters Currently the SDK has to send parameters as both query and matrix parameters, as the server some times requires query and some times matrix. But the server has recently been fixed to support both, for all paramers, so the SDK no longer needs to send both. This patch changes the SDK so that it only uses query parameters. Change-Id: I41f0aa2a0bb4b8b20e71a64769a352ea508c7583 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ServicesGenerator.java M sdk/lib/ovirtsdk4/http.py 2 files changed, 7 insertions(+), 27 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54780 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41f0aa2a0bb4b8b20e71a64769a352ea508c7583 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Use only query parameters
by juan.hernandez＠redhat.com 18 Mar '16

18 Mar '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use only query parameters ...................................................................... Use only query parameters Currently the SDK has to send parameters as both query and matrix parameters, as the server some times requires query and some times matrix. But the server has recently been fixed to support both, for all paramers, so the SDK no longer needs to send both. This patch changes the SDK so that it only uses query parameters. Change-Id: Ib17aaaab7b5660619c2ac69ee2f5c737e61d9ce3 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java M sdk/lib/ovirtsdk4/http.rb M sdk/spec/connection_spec.rb 3 files changed, 4 insertions(+), 98 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54781 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib17aaaab7b5660619c2ac69ee2f5c737e61d9ce3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: node: Add distag to node packages
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add distag to node packages ...................................................................... node: Add distag to node packages Make the Node packages only available on el7. Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release36.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54904 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 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-release[ovirt-4.0]: node: Add distag to node packages
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add distag to node packages ...................................................................... node: Add distag to node packages Make the Node packages only available on el7. Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release40.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54903 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 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-hosted-engine-setup[master]: Revert "add_host: SDK call update"
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "add_host: SDK call update" ...................................................................... Revert "add_host: SDK call update" After Id6fad7867102c9e121c4c0ddfb4be7bba43bb9bd will still rely on SDK from 3.6 since SDK 4.0 is still under heavy refactoring. Reverting this to 3.6 status for compatibility reasons. This reverts commit b61609627ad5147551b585d1365dc9ffd9498351. Bug-Url: https://bugzilla.redhat.com/1318652 Change-Id: I30a02672bdcfa3c1984472594bfc91363122f78f Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54893 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30a02672bdcfa3c1984472594bfc91363122f78f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(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-release[master]: node: Add distag to node packages
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add distag to node packages ...................................................................... node: Add distag to node packages Make the Node packages only available on el7. Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54879 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3fd3e9f4f2b11bcbbe03180084f09c896f583726 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(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-hosted-engine-setup[master]: Revert "add_host: from cpu.set_id to cpu.set_type"
by sbonazzo＠redhat.com 18 Mar '16

18 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "add_host: from cpu.set_id to cpu.set_type" ...................................................................... Revert "add_host: from cpu.set_id to cpu.set_type" After Id6fad7867102c9e121c4c0ddfb4be7bba43bb9bd will still rely on SDK from 3.6 since SDK 4.0 is still under heavy refactoring. Reverting this to 3.6 status for compatibility reasons. This reverts commit 4744f6c6fe48de23e81cac9691b9c6533cd3ec55. Bug-Url: https://bugzilla.redhat.com/1318652 Change-Id: I7951b3b5343e8533bdb4832bdbeb8954440017dd Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54895 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7951b3b5343e8533bdb4832bdbeb8954440017dd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(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]: core: StorageDomainValidator functional interface
by amureini＠redhat.com 17 Mar '16

17 Mar '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: StorageDomainValidator functional interface ...................................................................... core: StorageDomainValidator functional interface Use lambdas as functional interfaces instead of anonymous implementations, as per Java 8 and the project's standards. Change-Id: I16bc9d1814585a468407d9df3abe28b9fdbe594d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainValidator.java 1 file changed, 31 insertions(+), 39 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/54852 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I16bc9d1814585a468407d9df3abe28b9fdbe594d Gerrit-PatchSet: 2 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0