- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: change default migration policy of 4.0 clusters
by tjelinek＠redhat.com 15 Jun '16

15 Jun '16

Tomas Jelinek has submitted this change and it was merged. Change subject: core: change default migration policy of 4.0 clusters ...................................................................... core: change default migration policy of 4.0 clusters Change-Id: Ie8e1de4ace55b0b02be60b76e940d3f346101e50 Bug-Url: https://bugzilla.redhat.com/1346194 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- A packaging/dbscripts/upgrade/04_00_0810_change_cluster_default_policy.sql 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59154 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8e1de4ace55b0b02be60b76e940d3f346101e50 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

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 14 Jun '16

14 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