- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[ovirt-engine-3.6]: engine: Disallow setting a non-mgmt net as the host gateway
by piotr.kliczewski＠gmail.com 17 Dec '15

17 Dec '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: Disallow setting a non-mgmt net as the host gateway ...................................................................... engine: Disallow setting a non-mgmt net as the host gateway Disallow setting a non-management network as the host gateway in case of a cluster level that doesn't support MultipleGateways feature. Before the patch a non-management network on a host that serves as the management network in another cluster could be set as the host gateway. Change-Id: Icc8374af5d635a0c8e70116ccb8993d433ebc35e Bug-Url: https://bugzilla.redhat.com/1290433 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/NetworkAttachmentValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/NetworkAttachmentValidatorTest.java 2 files changed, 29 insertions(+), 22 deletions(-) Approvals: Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/50526 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc8374af5d635a0c8e70116ccb8993d433ebc35e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Release ovirt-hosted-engine-ha-1.3.3.4-1
by msivak＠redhat.com 17 Dec '15

17 Dec '15

Martin Sivák has submitted this change and it was merged. Change subject: Release ovirt-hosted-engine-ha-1.3.3.4-1 ...................................................................... Release ovirt-hosted-engine-ha-1.3.3.4-1 Change-Id: Id06b4d5a326a337053ad74353be7407cec6ddba1 Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, but someone else must approve; Passed CI tests Martin Sivák: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50672 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id06b4d5a326a337053ad74353be7407cec6ddba1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0