- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-java[master]: packaging: spec: fixed runtime dependencies
by juan.hernandez＠redhat.com 14 Nov '14

14 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: fixed runtime dependencies ...................................................................... packaging: spec: fixed runtime dependencies Fixed runtime dependencies for EL7. Change-Id: Id816bb7eef91fe807932416263b12f95ac509678 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine-sdk-java/ovirt-engine-sdk-java.spec.in 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35154 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id816bb7eef91fe807932416263b12f95ac509678 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: iscsi: checking image size against VG free space
by sbonazzo＠redhat.com 14 Nov '14

14 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: iscsi: checking image size against VG free space ...................................................................... iscsi: checking image size against VG free space On block devices we are preallocating the disk image so we need to check if we have enough free space before starting. We need to validate the image size before creating the VG so we cannot simply check the VG free space but we need a rough estimate of its overhead Change-Id: I17f861a053f6ac38983967ffb07ecca9ff9b8de1 Bug-Url: https://bugzilla.redhat.com/1142710 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit 624186e372d4e80e36b60312b8b79c7a3fd2bc7d) --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py M src/plugins/ovirt-hosted-engine-setup/vm/image.py 4 files changed, 74 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit http://gerrit.ovirt.org/35151 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17f861a053f6ac38983967ffb07ecca9ff9b8de1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: iscsi: checking image size against VG free space
by sbonazzo＠redhat.com 14 Nov '14

14 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: iscsi: checking image size against VG free space ...................................................................... iscsi: checking image size against VG free space On block devices we are preallocating the disk image so we need to check if we have enough free space before starting. We need to validate the image size before creating the VG so we cannot simply check the VG free space but we need a rough estimate of its overhead Change-Id: I17f861a053f6ac38983967ffb07ecca9ff9b8de1 Bug-Url: https://bugzilla.redhat.com/1142710 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py M src/plugins/ovirt-hosted-engine-setup/vm/image.py 4 files changed, 74 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34940 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17f861a053f6ac38983967ffb07ecca9ff9b8de1 Gerrit-PatchSet: 13 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine, userportal, webadmin: Event log->Event list/pane
by awels＠redhat.com 13 Nov '14

13 Nov '14

Alexander Wels has submitted this change and it was merged. Change subject: engine,userportal,webadmin: Event log->Event list/pane ...................................................................... engine,userportal,webadmin: Event log->Event list/pane - Replaced all user facing instances of Event log with either Event list or Event pane based on if they showed up exclusively in the UI or not. Change-Id: I582ceaa91b934446812dfad77dd86068beed0125 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1162824 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/CancelMigrateVDSCommand.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 5 files changed, 10 insertions(+), 10 deletions(-) Approvals: Alexander Wels: Verified Einav Cohen: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35147 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I582ceaa91b934446812dfad77dd86068beed0125 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: iscsi: detecting empty LUN list
by sbonazzo＠redhat.com 13 Nov '14

13 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: iscsi: detecting empty LUN list ...................................................................... iscsi: detecting empty LUN list Detecting and reporting an empty LUN list to provide a better user experience. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1133937 Change-Id: Iab293c172746a36779f8c2623c1c2542bf5aaca0 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit 5d231c6b29a14eaf9b3a646de4e65aabb1122245) --- M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py 1 file changed, 12 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit http://gerrit.ovirt.org/35141 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab293c172746a36779f8c2623c1c2542bf5aaca0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: iscsi: detecting empty LUN list
by sbonazzo＠redhat.com 13 Nov '14

13 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: iscsi: detecting empty LUN list ...................................................................... iscsi: detecting empty LUN list Detecting and reporting an empty LUN list to provide a better user experience. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1133937 Change-Id: Iab293c172746a36779f8c2623c1c2542bf5aaca0 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/iscsi.py 1 file changed, 12 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit http://gerrit.ovirt.org/35138 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab293c172746a36779f8c2623c1c2542bf5aaca0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: upgrade: allow to add host to updated cluster
by sbonazzo＠redhat.com 13 Nov '14

13 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: upgrade: allow to add host to updated cluster ...................................................................... upgrade: allow to add host to updated cluster Fix "Answer file lacks lockspace UUIDs, please use an answer file generated from the same version you are using on this additional host" Error while adding a host 3.5 based to a NFS cluster upgraded from 3.4. Change-Id: Id77da1857d379910b41764133eb5c0b82c5a3054 Bug-Url: https://bugzilla.redhat.com/1163385 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 8e42128172acf65aedec6668faa16cde0c34319c) --- M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35137 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id77da1857d379910b41764133eb5c0b82c5a3054 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: upgrade: allow to add host to updated cluster
by sbonazzo＠redhat.com 13 Nov '14

13 Nov '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: upgrade: allow to add host to updated cluster ...................................................................... upgrade: allow to add host to updated cluster Fix "Answer file lacks lockspace UUIDs, please use an answer file generated from the same version you are using on this additional host" Error while adding a host 3.5 based to a NFS cluster upgraded from 3.4. Change-Id: Id77da1857d379910b41764133eb5c0b82c5a3054 Bug-Url: https://bugzilla.redhat.com/1163385 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/35104 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id77da1857d379910b41764133eb5c0b82c5a3054 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: changed the list of fields which can not be overri...
by tjelinek＠redhat.com 13 Nov '14

13 Nov '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: changed the list of fields which can not be overridden ...................................................................... webadmin: changed the list of fields which can not be overridden There are the following types of fields in instance types: - the ones which are taken from it but the user can override them (e.g. defaults) - the ones which the user can not override This patch reduces the num of fields which can not be overridden to contain only the really most important ones. This avoids the situation that the user will have to have instance types like: "big with soundcard", "big with smardcard and soundcard" etc. The list is now significantly smaller, contains only: - memory size - num of sockets/cores per socket - HA - migration model/downtime - priority - balloon/min allocated memory The specific UI changes include: - remove the chain icon from fields which are not considered to be "marked" anymore - remove the specific fields from InstanceTypeAttachDetachManager.detachableModels so it will not detach the VM from instance type if changed Change-Id: I7fc831f0a0656f8eaa92fe3509b03c8ac1b71fb9 Bug-Url: https://bugzilla.redhat.com/1149663 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- 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/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/instancetypes/InstanceTypeAttachDetachManager.java 3 files changed, 14 insertions(+), 59 deletions(-) Approvals: Tomas Jelinek: Verified Gilad Chaplik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34916 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fc831f0a0656f8eaa92fe3509b03c8ac1b71fb9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Remove empty Vds hostname condition
by emesika＠redhat.com 13 Nov '14

13 Nov '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Remove empty Vds hostname condition ...................................................................... core: Remove empty Vds hostname condition Removes condition which test if Vds hostname is empty. But hostname cannot be empty, as there is no way how to install or communicate with empty hostname host. Change-Id: I639e9aea7ddbe9c6952a05d1077bbaf06930adcc Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java 1 file changed, 2 insertions(+), 8 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved Sahina Bose: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/35088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I639e9aea7ddbe9c6952a05d1077bbaf06930adcc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0