December 2015 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Add jenkins builders for ovirt-node-ng
by dcaro＠redhat.com 17 Dec '15

17 Dec '15

David Caro has submitted this change and it was merged. Change subject: Add jenkins builders for ovirt-node-ng ...................................................................... Add jenkins builders for ovirt-node-ng Add a nightly build in a F23 mock root. Maybe change this to just checking syntax later if the number of builds becomes oppressive Also add a nightly builder Change-Id: I26e48703c7df8289646d796485029316576f5cab Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- A jobs/confs/projects/ovirt-node-ng/ovirt-node-ng.yaml A jobs/confs/yaml/scms/ovirt-node-ng-scms.yaml 2 files changed, 29 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/50558 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26e48703c7df8289646d796485029316576f5cab Gerrit-PatchSet: 8 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add jenkins builders for imgbased
by dcaro＠redhat.com 17 Dec '15

17 Dec '15

David Caro has submitted this change and it was merged. Change subject: Add jenkins builders for imgbased ...................................................................... Add jenkins builders for imgbased Trigger on changes, build for Fedora and EL (comments in imbased indicate that there's a bug on EL7 presently, but it's being installed on EL images, so that may be old) The linters explicitly check for python3-pyflakes, etc, though. And they're called from the RPM specfile. Are these eventually going to run on EL7? Change-Id: Icb6e015464108cb0715381f69a9ce9d410ee657d Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- A jobs/confs/projects/imgbased/imgbased.yaml A jobs/confs/yaml/scms/imgbased.yaml 2 files changed, 45 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Verified Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/50560 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb6e015464108cb0715381f69a9ce9d410ee657d Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: Add host-node subpackage
by sbonazzo＠redhat.com 17 Dec '15

17 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add host-node subpackage ...................................................................... Add host-node subpackage Add a ovirt-release??-host-node subpackage for next-gen node as an equivalent to fedora-release-??, for instance, so node can be easily identified Add a subdir to ovirt-release to hold the required files Change-Id: Ib029f0de2c45a15d36ffb06c078802881d5438e0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1285024 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release-master/Makefile.am M ovirt-release-master/configure.ac A ovirt-release-master/host-node/Makefile.am A ovirt-release-master/host-node/firewalld/zones/ovirthostnode.xml A ovirt-release-master/host-node/systemd/system-preset/98-ovirt-host-node.preset M ovirt-release-master/ovirt-release-master.spec.in 6 files changed, 136 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Ryan Barry: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/50227 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib029f0de2c45a15d36ffb06c078802881d5438e0 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: 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

Change in jenkins[master]: ovirt-engine: drop 3.6.1 jobs
by sbonazzo＠redhat.com 17 Dec '15

17 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: drop 3.6.1 jobs ...................................................................... ovirt-engine: drop 3.6.1 jobs oVirt 3.6.1 has been released, so its branch is now dead. Dropping jobs for 3.6.1 branch. Change-Id: I004b7160e1bdedf59a44be360d5212898c9b39ea Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml M jobs/confs/projects/ovirt-engine/ovirt-engine_upgrade-db.yaml M jobs/confs/projects/ovirt-engine/ovirt-engine_upgrade-engine.yaml 3 files changed, 0 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Barak Korren: Looks good to me Jenkins CI: Passed CI tests David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/50692 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I004b7160e1bdedf59a44be360d5212898c9b39ea Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(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]: backend: Fix sound device handling on ppc desktops
by ahadas＠redhat.com 17 Dec '15

17 Dec '15

Arik Hadas has submitted this change and it was merged. Change subject: backend: Fix sound device handling on ppc desktops ...................................................................... backend: Fix sound device handling on ppc desktops When adding a VM to pool via rest API we automatically fall back to enabling sound device only based on the VM's desktop type. Extracted check already present in AddVmCommand into VmDeviceUtils and used it to replace all ocurrences of setSoundDeviceEnabled(vmType == DESKTOP). Change-Id: I6459f5cfb51b4dbdd53af6e7138e690ec4bb9c80 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1291626 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddVmTemplateCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/UpdateVmCommandTest.java 7 files changed, 54 insertions(+), 28 deletions(-) Approvals: Jenkins CI: Passed CI tests Martin Betak: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6459f5cfb51b4dbdd53af6e7138e690ec4bb9c80 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: removing a stale comment from CopyImageVDSCommand
by derez＠redhat.com 17 Dec '15

17 Dec '15

Daniel Erez has submitted this change and it was merged. Change subject: core: removing a stale comment from CopyImageVDSCommand ...................................................................... core: removing a stale comment from CopyImageVDSCommand Although probably a useful comment at the time, it has gotten obsolete a while now. Hence, tidying up and removing from code base. Change-Id: I48c48fde09595af0abaeac6fbc8e8e4418e7bd71 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/CopyImageVDSCommand.java 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Daniel Erez: Verified Jenkins CI: Passed CI tests Einav Cohen: Looks good to me, but someone else must approve Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50664 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I48c48fde09595af0abaeac6fbc8e8e4418e7bd71 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add job to build the Ruby gem libxml-ruby
by sbonazzo＠redhat.com 17 Dec '15

17 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add job to build the Ruby gem libxml-ruby ...................................................................... Add job to build the Ruby gem libxml-ruby This is required by the Ruby SDK. Change-Id: I32d8ebea2648bcb1c2795372d8eca902b202547b Related-To: https://bugzilla.redhat.com/1291365 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A jobs/confs/projects/rubygem-libxml-ruby/rubygem-libxml-ruby.yaml 1 file changed, 20 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50460 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32d8ebea2648bcb1c2795372d8eca902b202547b Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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]: Add job to build the Ruby SDK
by sbonazzo＠redhat.com 17 Dec '15

17 Dec '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add job to build the Ruby SDK ...................................................................... Add job to build the Ruby SDK This patch adds the SCM configuration and the stardard-ci jobs to build the Ruby SDK. Change-Id: I58fec73a8c33a578e86d726fe63543ef31803668 Related-To: https://bugzilla.redhat.com/1291365 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A jobs/confs/projects/ovirt-engine-sdk-ruby/ovirt-engine-sdk-ruby_build-artifacts.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml A jobs/confs/yaml/scms/ovirt-engine-sdk-ruby.yaml 3 files changed, 42 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50461 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58fec73a8c33a578e86d726fe63543ef31803668 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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]: restapi: Default display type for PPC
by ahadas＠redhat.com 17 Dec '15

17 Dec '15

Arik Hadas has submitted this change and it was merged. Change subject: restapi: Default display type for PPC ...................................................................... restapi: Default display type for PPC Default display type and graphics is selected for PPC in REST Add VM based on osinfo-defaults.properties. Change-Id: I9d0606b47e267526b071d63883d0eca16ece65b8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1253263 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java 1 file changed, 25 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Marek Libra: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/49412 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d0606b47e267526b071d63883d0eca16ece65b8 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(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-3.6]: webadmin: Info icon for Threads p. C. in Add VM dialog
by piotr.kliczewski＠gmail.com 17 Dec '15

17 Dec '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Info icon for Threads p. C. in Add VM dialog ...................................................................... webadmin: Info icon for Threads p. C. in Add VM dialog Info icon with description is displayed when setting simultaneous multithreading in Add/Edit VM/Template/Instance type dialogs. Change-Id: I6e67e158ac6b24ed6ea93ea376cda5a25346efd6 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267165 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationMessages.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/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java 4 files changed, 45 insertions(+), 17 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/50107 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e67e158ac6b24ed6ea93ea376cda5a25346efd6 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> 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