September 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-java[sdk_4.0]: [maven-release-plugin] prepare for next development iteration
by omachace＠redhat.com 22 Sep '16

22 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare for next development iteration ...................................................................... [maven-release-plugin] prepare for next development iteration Change-Id: I372ced640b6b7031ce0ee3a5f2883b335f55861b Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M automation/build-artifacts.sh M generator/pom.xml M pom.xml M sdk/pom.xml 4 files changed, 6 insertions(+), 6 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64198 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I372ced640b6b7031ce0ee3a5f2883b335f55861b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.0 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

Change in ovirt-engine[ovirt-engine-4.0]: spec: Make jboss optional
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: spec: Make jboss optional ...................................................................... spec: Make jboss optional So that using jboss-eap does not require pulling in also wildfly. Change-Id: I3ccdabb504e086092f61c1e9c50637b32c94ba09 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 6c889b9ee8c5ef8168fc5933de875c8956e485e3) --- M ovirt-engine.spec.in 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/64264 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ccdabb504e086092f61c1e9c50637b32c94ba09 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yedidyah Bar David <didi(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: aaa: Internal server error once automatically logged out
by mperina＠redhat.com 22 Sep '16

22 Sep '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Internal server error once automatically logged out ...................................................................... aaa: Internal server error once automatically logged out Session expiration on webadmin or userportal after redirecting to sso causes the session attributes to be lost resulting in a NPE. Passing app_url to sso and receiving it back from sso instead of saving app_url in session resolves the issue of session expiration. Change-Id: I44e423d52b5058add54d4ee69e09e1c54f363712 Bug-Url: https://bugzilla.redhat.com/1376003 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/servlet/SsoLoginServlet.java M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/servlet/SsoPostLoginServlet.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/servlets/OAuthAuthorizeServlet.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoConstants.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoSession.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoUtils.java M backend/manager/modules/enginesso/src/main/webapp/WEB-INF/login.jsp 7 files changed, 35 insertions(+), 6 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/64219 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44e423d52b5058add54d4ee69e09e1c54f363712 Gerrit-PatchSet: 2 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: fix pagination error when navigating the system tree
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: fix pagination error when navigating the system tree ...................................................................... webadmin: fix pagination error when navigating the system tree Changed SearchableListModel such that the current search page is reset with every search string reset. If the currently displayed page is not tracked correctly, UX elements will misbehave. The broken behavior primarily effected table lists view on main tabs. When the tab's search is updated by either navigating the system tree or by entering search terms, its list model needs to hold the correct search page information. If not, the table view pagination controls may behave as if a different page of records is being displayed than what is actually displayed. Change-Id: I17eab7bb25cbd7aecfedf259e33975947c1df892 Bug-Url: https://bugzilla.redhat.com/1326452 Signed-off-by: Scott J Dickerson <sdickers(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Vojtech Szocs: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved Scott Dickerson: Verified -- To view, visit https://gerrit.ovirt.org/64136 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17eab7bb25cbd7aecfedf259e33975947c1df892 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Scott Dickerson <sdickers(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Scott Dickerson <sdickers(a)redhat.com> 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[ovirt-engine-4.0]: core: set display and management network
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: set display and management network ...................................................................... core: set display and management network If a display network is not set by any network except the management one , set it to the management network If a migration network is not set by any network except the management one , set it to the management network Change-Id: Ia6c8435f87b48fea8afc0707920a9e100e4bd02b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1372955 Signed-off-by: emesika <emesika(a)redhat.com> Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- A packaging/dbscripts/upgrade/04_00_0920_set_display_migration_cluster_network.sql 1 file changed, 16 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/64248 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6c8435f87b48fea8afc0707920a9e100e4bd02b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Removing absolete emails
by tlitovsk＠redhat.com 22 Sep '16

22 Sep '16

Anatoly Litovsky has submitted this change and it was merged. Change subject: Removing absolete emails ...................................................................... Removing absolete emails Change-Id: I5d2d7cf9a9649327bd4c2e52845b97f11c39a5da --- M jobs/confs/projects/ovirt/system-tests.yaml M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Anatoly Litovsky: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/64267 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d2d7cf9a9649327bd4c2e52845b97f11c39a5da Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG
by sbonazzo＠redhat.com 22 Sep '16

22 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG ...................................................................... Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG With the release of OpenStack Liberty the Open vSwitch agent ini file location has changed. Includes logic to use OLD path if running OpenStack Kilo or older. OLD: /etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini NEW: /etc/neutron/plugins/ml2/openvswitch_agent.ini Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1373968 Change-Id: I6adf1078442693410d3435616e4dafb5600a77fb Signed-off-by: Tony James <tony(a)anthonyjames.org> (cherry picked from commit c0a9fda5be80966560c60c00638912d03f3248e2) --- M src/ovirt_host_deploy/constants.py 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64266 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6adf1078442693410d3435616e4dafb5600a77fb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 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-sdk-ruby[sdk_4.0]: Write correctly lists of enum values
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Write correctly lists of enum values ...................................................................... Write correctly lists of enum values Currently the SDK doesn't write correctly lists of enum values, because it tries to end the list calling the "end_element" method of the "XmlWriter" class, which doesn't exist, the correct name is "write_end". This is a bug in the code generator, fixed by this patch. Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Bug-Url: https://bugzilla.redhat.com/1378066 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 1e05f5d2aaee90ef9a39506bbd206c186423b0fe) --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java M sdk/spec/vm_writer_spec.rb 2 files changed, 33 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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]: Write correctly lists of enum values
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Write correctly lists of enum values ...................................................................... Write correctly lists of enum values Currently the SDK doesn't write correctly lists of enum values, because it tries to end the list calling the "end_element" method of the "XmlWriter" class, which doesn't exist, the correct name is "write_end". This is a bug in the code generator, fixed by this patch. Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Bug-Url: https://bugzilla.redhat.com/1378066 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java M sdk/spec/vm_writer_spec.rb 2 files changed, 33 insertions(+), 1 deletion(-) Approvals: Boris Odnopozov: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64245 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(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

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Added Probing for API version capability
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Added Probing for API version capability ...................................................................... Added Probing for API version capability This patch adds the Probe class that allows to check which API versions the oVirt server supports. Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Signed-off-by: borod108 <bodnopoz(a)redhat.com> (cherry picked from commit 9cf6f163d66e265423e8b2d65e14edbc6aba2c31) --- M sdk/Gemfile.lock M sdk/lib/ovirtsdk4.rb M sdk/lib/ovirtsdk4/http.rb A sdk/lib/ovirtsdk4/probe.rb M sdk/spec/connection_create_spec.rb A sdk/spec/probe_spec.rb M sdk/spec/spec_helper.rb 7 files changed, 318 insertions(+), 43 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64247 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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