February 2017 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Add ovirt-vdsmfake project to std CI
by Code Review 20 Feb '17

20 Feb '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: Add ovirt-vdsmfake project to std CI ...................................................................... Add ovirt-vdsmfake project to std CI Fixes: https://github.com/oVirt/ovirt-vdsmfake/issues/7 Change-Id: I35fb6569753f75d2e1e9ad909281effe75994468 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- A jobs/confs/projects/ovirt-vdsmfake/ovirt-vdsmfake_standard.yaml A jobs/confs/yaml/scms/ovirt-vdsmfake.yaml 2 files changed, 35 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me, approved; Ready for merge Barak Korren: Verified; Looks good to me; Ready for review Gil Shinar: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72664 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35fb6569753f75d2e1e9ad909281effe75994468 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(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: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[virt-sig]: centos-release-ovirt41: new package
by Code Review 20 Feb '17

20 Feb '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: centos-release-ovirt41: new package ...................................................................... centos-release-ovirt41: new package Change-Id: I69001f5d7a74c49e5b866eb6682d65d6445132e9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A centos-release-ovirt41/COPYING A centos-release-ovirt41/CentOS-oVirt-4.1.repo A centos-release-ovirt41/centos-release-ovirt41.spec 3 files changed, 404 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72479 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69001f5d7a74c49e5b866eb6682d65d6445132e9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: virt-sig Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alan Pevec <apevec(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Richard Megginson <rmeggins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: webadmin: Prevent error in VM NUMA pinning dialog
by Code Review 20 Feb '17

20 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Prevent error in VM NUMA pinning dialog ...................................................................... webadmin: Prevent error in VM NUMA pinning dialog When dragging an icon from any part of the VM NUMA pinning dialog other than the 'unassigned virtual nodes' element, an exception is thrown causing an error to be displayed in the UI. This patch prevents the exception, avoiding the error. Change-Id: Iebc2adabb216a1b65d8ad31a2d6d9700e4baae4b Bug-Url: https://bugzilla.redhat.com/1382807 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/numa/DragTargetScrollPanel.java 1 file changed, 13 insertions(+), 6 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/72706 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebc2adabb216a1b65d8ad31a2d6d9700e4baae4b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.0]: Add quota limit update example
by Code Review 20 Feb '17

20 Feb '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add quota limit update example ...................................................................... Add quota limit update example This patch adds an example that shows how to update a storage quota limit. Change-Id: I401e55f2f1d368b18a775e0c3034fbaa2bda3e48 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 0e22954600443e1c6d4aab44d519c8ce51552ad8) (cherry picked from commit 537b51f7832c1b3d9929ada77e76b82d33d1d0fa) --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/UpdateQuotaLimits.java 1 file changed, 135 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72684 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I401e55f2f1d368b18a775e0c3034fbaa2bda3e48 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: 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[master]: webadmin: Modify minimum threshold of Quota.
by Code Review 20 Feb '17

20 Feb '17

>From Alexander Wels <awels(a)redhat.com>: Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Modify minimum threshold of Quota. ...................................................................... webadmin: Modify minimum threshold of Quota. When Add the Quota of DataCenter if set the threshold to 0, the AddQuotaCommand will modify to the default value 80, but Edit the Quota can set the threshold to 0. Refer to the minimum grace, modify the minimum threshold of Quota to 1. Change-Id: I69fa419c6b6ddad1923e660b87a083c1d436c90a Signed-off-by: Tony Wang <Sline798(a)gmail.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/quota/QuotaPopupView.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Tony Wang: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72411 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69fa419c6b6ddad1923e660b87a083c1d436c90a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tony Wang <sline798(a)gmail.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Tony Wang <sline798(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Prevent error in VM NUMA pinning dialog
by Code Review 20 Feb '17

20 Feb '17

>From Alexander Wels <awels(a)redhat.com>: Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Prevent error in VM NUMA pinning dialog ...................................................................... webadmin: Prevent error in VM NUMA pinning dialog When dragging an icon from any part of the VM NUMA pinning dialog other than the 'unassigned virtual nodes' element, an exception is thrown causing an error to be displayed in the UI. This patch prevents the exception, avoiding the error. Change-Id: Iebc2adabb216a1b65d8ad31a2d6d9700e4baae4b Bug-Url: https://bugzilla.redhat.com/1382807 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/numa/DragTargetScrollPanel.java 1 file changed, 13 insertions(+), 6 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Phillip Bailey: Verified -- To view, visit https://gerrit.ovirt.org/72235 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebc2adabb216a1b65d8ad31a2d6d9700e4baae4b Gerrit-PatchSet: 3 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: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(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: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: core: warning on low swap only once a day
by Code Review 20 Feb '17

20 Feb '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: warning on low swap only once a day ...................................................................... core: warning on low swap only once a day One should not be warned every 30 minutes on low swap. Once a day should be enough. Change-Id: I75fced4446695fc5b9e2d539e6e4b50e444b89fd Bug-Url: https://bugzilla.redhat.com/1424787 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Oved Ourfali: Verified -- To view, visit https://gerrit.ovirt.org/72688 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I75fced4446695fc5b9e2d539e6e4b50e444b89fd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: vdsbroker: Code style cleanup for TransportFactory
by Code Review 20 Feb '17

20 Feb '17

>From Roy Golan <rgolan(a)redhat.com>: Roy Golan has submitted this change and it was merged. Change subject: vdsbroker: Code style cleanup for TransportFactory ...................................................................... vdsbroker: Code style cleanup for TransportFactory In preparation for more refactorings, start with styling this class so it will be readable. Change-Id: Ief8ae9ebaf80d7285c7d33a9dee225096443f815 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/TransportFactory.java 1 file changed, 36 insertions(+), 15 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/71837 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief8ae9ebaf80d7285c7d33a9dee225096443f815 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(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: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[sdk_4.1]: Add quota limit update example
by Code Review 20 Feb '17

20 Feb '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add quota limit update example ...................................................................... Add quota limit update example This patch adds an example that shows how to update a storage quota limit. Change-Id: I401e55f2f1d368b18a775e0c3034fbaa2bda3e48 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 0e22954600443e1c6d4aab44d519c8ce51552ad8) --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/UpdateQuotaLimits.java 1 file changed, 135 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72681 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I401e55f2f1d368b18a775e0c3034fbaa2bda3e48 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(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[sdk_4.0]: Add quota limit update example
by Code Review 20 Feb '17

20 Feb '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add quota limit update example ...................................................................... Add quota limit update example This patch adds an example that shows how to update a storage quota limit. Change-Id: I73a4f1924b289cbf882df4f2bff9c8d2c68d0007 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit a7305aaf26ab8cdad0b5b43913c0437dd76ffcd1) (cherry picked from commit e78e0bc36f259f65681105b83993632cbb5a420d) --- A sdk/examples/update_quota_limits.py 1 file changed, 103 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72680 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73a4f1924b289cbf882df4f2bff9c8d2c68d0007 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.0 Gerrit-Owner: Ondra Machacek <omachace(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