- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: core: Fix clone VM from snapshot with Cinder disks
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix clone VM from snapshot with Cinder disks ...................................................................... core: Fix clone VM from snapshot with Cinder disks Change-Id: Ib46e1bd24002e914b902c017c44ad8d8131d3b0c Bug-Url: https://bugzilla.redhat.com/1377442 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/64291 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib46e1bd24002e914b902c017c44ad8d8131d3b0c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: Fix creation of template from VM with Cinder disks
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix creation of template from VM with Cinder disks ...................................................................... core: Fix creation of template from VM with Cinder disks Change-Id: I8275dcbe4e3295fda4797870aaa7b455a87b0a1a Bug-Url: https://bugzilla.redhat.com/1378402 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64289 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8275dcbe4e3295fda4797870aaa7b455a87b0a1a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: Fix clone VM from snapshot with Cinder disks
by tnisan＠redhat.com 22 Sep '16

22 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix clone VM from snapshot with Cinder disks ...................................................................... core: Fix clone VM from snapshot with Cinder disks Change-Id: Ib46e1bd24002e914b902c017c44ad8d8131d3b0c Bug-Url: https://bugzilla.redhat.com/1377442 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64288 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib46e1bd24002e914b902c017c44ad8d8131d3b0c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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-sdk-java[master]: Add example of add group from a directory service
by omachace＠redhat.com 22 Sep '16

22 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of add group from a directory service ...................................................................... Add example of add group from a directory service This patch adds example of adding a group from a directory service. Change-Id: I87b6decade678238c6b463fdbc911ce950df2dea Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/AddGroup.java 1 file changed, 61 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63778 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87b6decade678238c6b463fdbc911ce950df2dea Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java 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: Martin Peřina <mperina(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[master]: Add example of add group from a directory service
by omachace＠redhat.com 22 Sep '16

22 Sep '16

Ondra Machacek has submitted this change and it was merged. Change subject: Add example of add group from a directory service ...................................................................... Add example of add group from a directory service This patch adds example of adding a group from a directory service. Change-Id: If80b5207dd23b3d06ca78b2f2d57f14fa9b02a48 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- A sdk/examples/add_group.py 1 file changed, 54 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63777 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If80b5207dd23b3d06ca78b2f2d57f14fa9b02a48 Gerrit-PatchSet: 3 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: Martin Peřina <mperina(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-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