June 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: packaging: Remove dependency on fop on F23+
by oourfali＠redhat.com 15 Jun '16

15 Jun '16

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: Remove dependency on fop on F23+ ...................................................................... packaging: Remove dependency on fop on F23+ Removes dependency on fop on F23+, because only fop 2.0 is available and its API is incompatible with fop 1.1 we need (adn which is available on EL7). Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a Bug-Url: https://bugzilla.redhat.com/1345986 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 23 insertions(+), 13 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/59184 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: Adding dao tests to std ci
by mperina＠redhat.com 15 Jun '16

15 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: Adding dao tests to std ci ...................................................................... Adding dao tests to std ci Basically this patch is all about running dao tests inside a mock. For this, a postgresql DB should be installed and run inside the mock. Added postgres packages to .packages files Created a new script, dao-tests.sh. This script is being called only if files that are relevant to DB had been changed. The script will pick a port randomly and will update pg_hba.conf file to allow DB access. All the rest is the same as how the tests were running when not in std ci. Change-Id: If01639ebd0e346e0bd724a3b9ec68b44e7e9e765 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M automation/check-merged.packages M automation/check-merged.sh M automation/check-patch.packages A automation/check-patch.packages.el6 M automation/check-patch.sh A automation/dao-tests.sh 6 files changed, 119 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Eyal Edri: Looks good to me, but someone else must approve Gil Shinar: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58633 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If01639ebd0e346e0bd724a3b9ec68b44e7e9e765 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(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: Roy Golan <rgolan(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[master]: packaging: Remove dependency on fop on F23+
by mperina＠redhat.com 14 Jun '16

14 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: packaging: Remove dependency on fop on F23+ ...................................................................... packaging: Remove dependency on fop on F23+ Removes dependency on fop on F23+, because only fop 2.0 is available and its API is incompatible with fop 1.1 we need (adn which is available on EL7). Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a Bug-Url: https://bugzilla.redhat.com/1345986 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 23 insertions(+), 13 deletions(-) Approvals: 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/59084 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc04fb488ef2f5874bd1d80744390efd5b94017a Gerrit-PatchSet: 3 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: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Don't figure out NIC profile during V3 update
by juan.hernandez＠redhat.com 14 Jun '16

14 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Don't figure out NIC profile during V3 update ...................................................................... restapi: Don't figure out NIC profile during V3 update Currently the V3 compatibility layer tries to figure out an appropriate NIC profile when a VM NIC is added or updated and no NIC profile is explicitly provided. This works fine for adding, but during updates it may select the wrong profile, thus randomly changing the network that the NIC is attached to. To avoid that this patch changes the V3 adapter so that it only tries to figure out the profile when the "network" or "port_mirroring" attributes are included in the update, as any change to those attributes requires a change of the VNIC profile. In addition, to make things less random, the patch also changes the logic to select the profile so that it sorts the candidates alphabetically by name and chooses the first one. Change-Id: Iaa953fcc200a1e175975e3dade9dc3a67d234f27 Bug-Url: https://bugzilla.redhat.com/1342054 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/helpers/V3NICHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/servers/V3VmNicServer.java 2 files changed, 8 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58529 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa953fcc200a1e175975e3dade9dc3a67d234f27 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: fix serialization of pool related parameter classes
by tnisan＠redhat.com 14 Jun '16

14 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: core: fix serialization of pool related parameter classes ...................................................................... core: fix serialization of pool related parameter classes Fields in the parameters class should have setters and getters. Change-Id: Iae19a2056ba31164c65744ea016a00329e6abcf2 Bug-Url: https://bugzilla.redhat.com/1345853 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmPoolWithVmsParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmPoolOperationParameters.java 2 files changed, 25 insertions(+), 16 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/59144 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae19a2056ba31164c65744ea016a00329e6abcf2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: fixing 4.0 publisher with 4.0 vdsm rpms
by eedri＠redhat.com 14 Jun '16

14 Jun '16

Eyal Edri has submitted this change and it was merged. Change subject: fixing 4.0 publisher with 4.0 vdsm rpms ...................................................................... fixing 4.0 publisher with 4.0 vdsm rpms Change-Id: Ie6746b9d9561d2467fdcc8b677f2d421cd39cbe3 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eyal Edri: Verified; Ready for merge Jenkins CI: Passed CI tests Francesco Romani: Looks good to me -- To view, visit https://gerrit.ovirt.org/59168 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6746b9d9561d2467fdcc8b677f2d421cd39cbe3 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Add model HTML
by tnisan＠redhat.com 14 Jun '16

14 Jun '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Add model HTML ...................................................................... restapi: Add model HTML This patch modifies the build process of the API so that it also builds the HTML documentation of the model. The documentation will be available from the API server, in the "/ovirt-engine/api/model" URL. Change-Id: I869e6f2b8a6366d9f07231b74f6fe03427a93f08 Bug-Url: https://bugzilla.redhat.com/1346218 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit d3022cc3126bb24afbe0fbe0af88d46426c62e73) --- M backend/manager/modules/restapi/interface/definition/pom.xml M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml 2 files changed, 38 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59129 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I869e6f2b8a6366d9f07231b74f6fe03427a93f08 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: build: version bump after branching
by stirabos＠redhat.com 14 Jun '16

14 Jun '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: version bump after branching ...................................................................... build: version bump after branching Change-Id: I7f57d4d4d970f041d7a27f1afc704cd55f13059e Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f57d4d4d970f041d7a27f1afc704cd55f13059e Gerrit-PatchSet: 1 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add ovirt-hosted-engine-setup stable branch for 4.0
by dcaro＠redhat.com 14 Jun '16

14 Jun '16

David Caro has submitted this change and it was merged. Change subject: Add ovirt-hosted-engine-setup stable branch for 4.0 ...................................................................... Add ovirt-hosted-engine-setup stable branch for 4.0 Configure ovirt-hosted-engine_stardard job to build also from a stable branch for 4.0. Adding it to the 4.0 nightly repo composer. Change-Id: I1955bf3913e0856b3ffb2c9bf81d00cfb4fe9d37 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M jobs/confs/projects/ovirt-hosted-engine-setup/ovirt-hosted-engine_stardard.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Simone Tiraboschi: Verified; Ready for review Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/59150 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1955bf3913e0856b3ffb2c9bf81d00cfb4fe9d37 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Add label comboboxes to VM & host dialogs
by awels＠redhat.com 14 Jun '16

14 Jun '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Add label comboboxes to VM & host dialogs ...................................................................... webadmin: Add label comboboxes to VM & host dialogs This patch adds read-only comboboxes to the general tab of the add/edit dialog boxes for VMs and hosts. The boxes are populated with all available affinity labels. For existing VMs and hosts, labels that have been set through the REST API are shown as selected. This is phase 1 of this feature. Phase 2 will introduce the ability to change selected labels via the UI. Until that time, labels can be modified via the REST API. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1338799 Change-Id: Iad36512b423a20b3ad7ad9e01ff43882555fcdb6 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants.properties M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.ui.xml 10 files changed, 165 insertions(+), 4 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/57903 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad36512b423a20b3ad7ad9e01ff43882555fcdb6 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0