December 2016 - Engine-commits - oVirt List Archives

Change in ovirt-release[ovirt-4.1]: node: add sysstat
by Code Review 20 Dec '16

20 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: node: add sysstat ...................................................................... node: add sysstat Include sysstat so `sar` is available, and upstream stays in sync with downstream. Change-Id: I6f9dfd7b8b83835e5de404804c7e3e57bf559c8b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1382843 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M ovirt-release41.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68609 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f9dfd7b8b83835e5de404804c7e3e57bf559c8b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Ryan Barry <rbarry(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[master]: Fix typos in pom.xml
by Code Review 20 Dec '16

20 Dec '16

>From Allon Mureinik <amureini(a)redhat.com>: Allon Mureinik has submitted this change and it was merged. Change subject: Fix typos in pom.xml ...................................................................... Fix typos in pom.xml Change-Id: I59665693619f68b13b785c6dc236ccb871a0dbf6 Signed-off-by: Benny Zlotnik <bzlotnik(a)redhat.com> --- M pom.xml 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Benny Zlotnik: Verified Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/68806 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I59665693619f68b13b785c6dc236ccb871a0dbf6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Benny Zlotnik <bzlotnik(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Benny Zlotnik <bzlotnik(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: check that DC and cluster are selected upon attach...
by Code Review 20 Dec '16

20 Dec '16

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: webadmin: check that DC and cluster are selected upon attach disk to VM ...................................................................... webadmin: check that DC and cluster are selected upon attach disk to VM When trying to attach a disk to a new VM via the new VM popup and there is not available data center in the system the operation fails with a UI exception Change-Id: Ic2c260152b6cd88553351a507c3dc7db2a8a9f47 Bug-Url: https://bugzilla.redhat.com/1364152 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImageLineModel.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Tal Nisan: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68739 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2c260152b6cd88553351a507c3dc7db2a8a9f47 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ruby-sdk: Build version 4.1 from branch sdk_4.1
by Code Review 20 Dec '16

20 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: ruby-sdk: Build version 4.1 from branch sdk_4.1 ...................................................................... ruby-sdk: Build version 4.1 from branch sdk_4.1 The sdk_4.1 branch has been created for the Ruby SDK. This patch adds the job to build from that branch and updates the 4.1 RPM publisher so that it takes the packages produced by the new job. Change-Id: Id39acdb0b9a6727fd968fb68f9219ac1fea54786 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-sdk-ruby/ovirt-engine-sdk-ruby_standard.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.1.yaml 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68760 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id39acdb0b9a6727fd968fb68f9219ac1fea54786 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: enabling on-demand builds for various projects
by Code Review 20 Dec '16

20 Dec '16

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: enabling on-demand builds for various projects ...................................................................... enabling on-demand builds for various projects Using the new triggers for 'on comment added'. Updated projects: - ovirt-engine - vdsm - vdsm-jsonrpc-java - ovirt-engine-dashboard Fixes: https://ovirt-jira.atlassian.net/browse/OVIRT-920 Change-Id: I756d49e0779e7d5420493a64ca1c7730c50651a5 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-dashboard/ovirt-engine-dashboard_standard.yaml M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml M jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_standard.yaml M jobs/confs/projects/vdsm/vdsm_standard.yaml 4 files changed, 4 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me, approved; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68759 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I756d49e0779e7d5420493a64ca1c7730c50651a5 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-4.1]: rpm: Build artifacts having EL7 buildable
by Code Review 20 Dec '16

20 Dec '16

>From Vinzenz Feenstra <vfeenstr(a)redhat.com>: Vinzenz Feenstra has submitted this change and it was merged. Change subject: rpm: Build artifacts having EL7 buildable ...................................................................... rpm: Build artifacts having EL7 buildable Change-Id: If6d7fe77edc72db5b5442351b5fcb39cf32fef25 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> (cherry picked from commit 165fff1854b70f8dfff19bbc0c43462e6c930e8b) --- M automation/build-artifacts.sh M ovirt-guest-agent.spec 2 files changed, 17 insertions(+), 36 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/68774 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6d7fe77edc72db5b5442351b5fcb39cf32fef25 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Use ids to calculate intersections
by Code Review 20 Dec '16

20 Dec '16

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use ids to calculate intersections ...................................................................... restapi: Use ids to calculate intersections Currently when the user uses together the 'filter' and 'search' mechanisms, the server performs two queries: one with filter only, and one with search only. Then the it calculates and returns the intersection. But this intersection is calculated assuming that the objects returned by both queries can be reliably compared using the 'equals' methods. This isn't true, in general. The net result is that many times the intersection is empty, when it should't. To avoid that issue this patch changes the server so that it compares the objects by id, instead of using the 'equals' method. Change-Id: Id6fd395ca3d3aea7ab2dcdcdf2438e4c487cfc1f Bug-Url: https://bugzilla.redhat.com/1402799 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCollectionResource.java 1 file changed, 49 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/67526 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6fd395ca3d3aea7ab2dcdcdf2438e4c487cfc1f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[master]: rpm: Build artifacts having EL7 buildable
by Code Review 20 Dec '16

20 Dec '16

>From Vinzenz Feenstra <vfeenstr(a)redhat.com>: Vinzenz Feenstra has submitted this change and it was merged. Change subject: rpm: Build artifacts having EL7 buildable ...................................................................... rpm: Build artifacts having EL7 buildable Change-Id: If6d7fe77edc72db5b5442351b5fcb39cf32fef25 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M automation/build-artifacts.sh M ovirt-guest-agent.spec 2 files changed, 17 insertions(+), 36 deletions(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified; Looks good to me, approved Milan Zamazal: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/67980 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6d7fe77edc72db5b5442351b5fcb39cf32fef25 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Populate V3 product_info.version.full_version
by Code Review 20 Dec '16

20 Dec '16

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: core: Populate V3 product_info.version.full_version ...................................................................... core: Populate V3 product_info.version.full_version In version 3 of the API the full version of the product is potentially available in two places: product_info.version.full_version product_info.full_version Version 3 of the engine used to populate the second one, but the version 3 compatibility layer of version 4 of the engine populates only the second. This means that clients that still use version 3 of the API won't be able to get this information. To avoid that, this patch changes the V3 compatibility layer so that it populates both places with the same value. Change-Id: If8a60ba2ef333dc87cc113959f5bdb3f35426b09 Bug-Url: https://bugzilla.redhat.com/1405386 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3ProductInfoOutAdapter.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/68596 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8a60ba2ef333dc87cc113959f5bdb3f35426b09 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine 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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Add IQN randomization
by Code Review 20 Dec '16

20 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add IQN randomization ...................................................................... node: Add IQN randomization Enable the new imgbsae-generate-iqn.service to generate a random IQN name if none is defined. Change-Id: I92a7e586de5bbd02a722ea8da4ef1aac330b490b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1393661 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master.spec.in 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/66593 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92a7e586de5bbd02a722ea8da4ef1aac330b490b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0