- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: post ovirt-hosted-engine-setup-1.3.2.2-2
by sbonazzo＠redhat.com 12 Jan '16

12 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.3.2.2-2 ...................................................................... build: post ovirt-hosted-engine-setup-1.3.2.2-2 Change-Id: I5219a5eb988936fb566e0acefba3c9d00e1aaa00 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 9 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51718 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5219a5eb988936fb566e0acefba3c9d00e1aaa00 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[master]: cloud_init: local variable 'address' referenced before assig...
by sbonazzo＠redhat.com 12 Jan '16

12 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: cloud_init: local variable 'address' referenced before assignment ...................................................................... cloud_init: local variable 'address' referenced before assignment Assign var address to None before trying to manage it. Change-Id: I47358e9686b50bb4057306092a33d29438bce2b7 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/vm/cloud_init.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51677 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47358e9686b50bb4057306092a33d29438bce2b7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(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: Roy Golan <rgolan(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-3.6]: core: device: avoid warning about Graphics device
by tnisan＠redhat.com 12 Jan '16

12 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: core: device: avoid warning about Graphics device ...................................................................... core: device: avoid warning about Graphics device Libvirt gives no address to Graphics devices, on purpose, so better not to emit misleasing warning about them. Change-Id: Iafdc3566f625988e462e50cf1685bdc5330743bd Bug-Url: https://bugzilla.redhat.com/1273965 Signed-off-by: Francesco Romani <fromani(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VmDeviceCommonUtils.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java 2 files changed, 15 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/51707 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iafdc3566f625988e462e50cf1685bdc5330743bd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(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 ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: ovirt-hosted-engine-setup-1.3.2.2-2
by sbonazzo＠redhat.com 12 Jan '16

12 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-1.3.2.2-2 ...................................................................... build: ovirt-hosted-engine-setup-1.3.2.2-2 Change-Id: I94b1e3b8ea587f99ad4e173e88fcd9d93381fe4b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 10 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51717 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94b1e3b8ea587f99ad4e173e88fcd9d93381fe4b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: Revert "build: post ovirt-hosted-engine-setup-1.3.2.2"
by sbonazzo＠redhat.com 12 Jan '16

12 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "build: post ovirt-hosted-engine-setup-1.3.2.2" ...................................................................... Revert "build: post ovirt-hosted-engine-setup-1.3.2.2" This reverts commit a6170c372f0aa80703a31a594ea709557585eb98. Change-Id: I55027ffaa42355267a9aa1c3e53aed613545c3d3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 6 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51716 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55027ffaa42355267a9aa1c3e53aed613545c3d3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Fix check if selected dedicated host supports NUMA
by vszocs＠redhat.com 12 Jan '16

12 Jan '16

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: Fix check if selected dedicated host supports NUMA ...................................................................... webadmin: Fix check if selected dedicated host supports NUMA Since multiple dedicated hosts are supported we have to use the first selected host of the dedicated host list to detect if numa is supported and listen to the right multiselect event to detect selection changes. Additional checks make sure that only one host is selected. Change-Id: I69a06d01b70ba38bd1a8a7a636172616218207b2 Bug-Url: https://bugzilla.redhat.com/1279178 Bug-Url: https://bugzilla.redhat.com/1289967 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 3 files changed, 38 insertions(+), 12 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve Roman Mohr: Verified Alexander Wels: Looks good to me, approved Vojtech Szocs: Looks good to me, but someone else must approve; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51380 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69a06d01b70ba38bd1a8a7a636172616218207b2 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: db: add rename table function
by emesika＠redhat.com 12 Jan '16

12 Jan '16

Eli Mesika has submitted this change and it was merged. Change subject: db: add rename table function ...................................................................... db: add rename table function This patch series removes the old "vds Group" terminology and replace it all over the code with the term cluster. Up to this patch there was a mix of both "vds group" and cluster that makes the code inconsistent and hard to follow Change-Id: I02b1f21b51e05a3f1d53c127ebda63e0b2fead73 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/common_sp.sql 1 file changed, 29 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51106 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02b1f21b51e05a3f1d53c127ebda63e0b2fead73 Gerrit-PatchSet: 17 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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]: build: ovirt-hosted-engine-ha-1.3.3.7
by msivak＠redhat.com 12 Jan '16

12 Jan '16

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.3.3.7 ...................................................................... build: ovirt-hosted-engine-ha-1.3.3.7 Change-Id: Ie897d643c4de866504da207d7ccbcdecad9fc7d6 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 9 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51713 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie897d643c4de866504da207d7ccbcdecad9fc7d6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: upgrade: fixing for iSCSI additional hosts
by msivak＠redhat.com 12 Jan '16

12 Jan '16

Martin Sivák has submitted this change and it was merged. Change subject: upgrade: fixing for iSCSI additional hosts ...................................................................... upgrade: fixing for iSCSI additional hosts On 3.5 -> 3.6 we need to call getVolumesList to scan all the available volumes and check if some other host already created the configuration volume. After a reboot ovirt-ha-agent calls prepareImage only on metadata, lockspace and if available on the configuration image but not on the engine image. So, only after a reboot and only on iSCSI, getVolumesList will report an error regarding the engine image. filtering out know images from the scan, using metadata image to check the images path, avoid breaking on getVolumesList errors Change-Id: I724835de6498fc7098d03499b246fa69b5bf09fe Bug-Url: https://bugzilla.redhat.com/1297457 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/upgrade.py 1 file changed, 57 insertions(+), 26 deletions(-) Approvals: Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/51722 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I724835de6498fc7098d03499b246fa69b5bf09fe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: upgrade: fixing for iSCSI additional hosts
by msivak＠redhat.com 12 Jan '16

12 Jan '16

Martin Sivák has submitted this change and it was merged. Change subject: upgrade: fixing for iSCSI additional hosts ...................................................................... upgrade: fixing for iSCSI additional hosts On 3.5 -> 3.6 we need to call getVolumesList to scan all the available volumes and check if some other host already created the configuration volume. After a reboot ovirt-ha-agent calls prepareImage only on metadata, lockspace and if available on the configuration image but not on the engine image. So, only after a reboot and only on iSCSI, getVolumesList will report an error regarding the engine image. filtering out know images from the scan, using metadata image to check the images path, avoid breaking on getVolumesList errors Change-Id: I724835de6498fc7098d03499b246fa69b5bf09fe Bug-Url: https://bugzilla.redhat.com/1297457 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt_hosted_engine_ha/lib/upgrade.py 1 file changed, 57 insertions(+), 26 deletions(-) Approvals: Martin Sivák: Looks good to me, approved; Passed CI tests Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/51658 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I724835de6498fc7098d03499b246fa69b5bf09fe Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Artyom Lukianov <alukiano(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(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