- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: renamed confusing OVA label
by tjelinek＠redhat.com 17 May '16

17 May '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: renamed confusing OVA label ...................................................................... webadmin: renamed confusing OVA label The Virtual Appliance (OVA) was quite confusing since it was not obvious that it applies only to VMware one. Change-Id: Ib1015646128cbb1fd9b25fd50ed344c63fefc98d Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved Michal Skrivanek: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/57404 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib1015646128cbb1fd9b25fd50ed344c63fefc98d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(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[master]: engine: disable IPv6 out-of-sync reporting
by alkaplan＠redhat.com 17 May '16

17 May '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: disable IPv6 out-of-sync reporting ...................................................................... engine: disable IPv6 out-of-sync reporting Currently VDSM reports "dhcpv6=true" when an IPv6 address was assigned dynamically. Similar logic worked for IPv4 where the only method to get the address dynamically is DHCP indeed. However, IPv6 address could be assigned dynamically by the stateless autoconfiguration protocol too, which makes the described logic not reliable. The described VDSM bug causes the engine marking a network as out-of-sync when the "autoconf" boot protocol was requested - VDSM applies the requested correctly, but GetCaps reports that erroneously. Reporting out-of-sync IPv6 configuration is disabled temporary. It's planned to be re-enabled after v4.0-beta is released. Change-Id: I5d3e8de70e24f21ede03a875da4de2526145d033 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NetworkInSyncWithVdsNetworkInterface.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NetworkInSyncWithVdsNetworkInterfaceTest.java 2 files changed, 40 insertions(+), 27 deletions(-) Approvals: Marcin Mirecki: Looks good to me, but someone else must approve Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/57274 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d3e8de70e24f21ede03a875da4de2526145d033 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: lsm: RFE: Remove auto-generated snapshot
by laravot＠redhat.com 17 May '16

17 May '16

Liron Aravot has submitted this change and it was merged. Change subject: lsm: RFE: Remove auto-generated snapshot ...................................................................... lsm: RFE: Remove auto-generated snapshot After live storage migration (LSM), a snapshot is created and the uesr has to manually remove it after LSM completes. This patch automatically removes the snapshot after LSM completes. Change-Id: I88055726c5939a91fe84005bdc86946319fea2e1 Bug-Url: https://bugzilla.redhat.com/1317434 Signed-off-by: Ala Hino <ahino(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RemoveSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/lsm/LiveMigrateVmDisksCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/LiveMigrateVmDisksParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/RemoveSnapshotParameters.java 4 files changed, 56 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Ala Hino: Verified Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/57302 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88055726c5939a91fe84005bdc86946319fea2e1 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: gluster: add VDS Command for glusterVolumeHealInfo
by sabose＠redhat.com 17 May '16

17 May '16

Sahina Bose has submitted this change and it was merged. Change subject: gluster: add VDS Command for glusterVolumeHealInfo ...................................................................... gluster: add VDS Command for glusterVolumeHealInfo Add a VDS Command for glusterVolumeHealInfo to monitor self heal for gluster volume. Bug-Url: https://bugzilla.redhat.com/1205641 Change-Id: I2bfa52bcc692ec2ef85443fd70dbb6ba08234bc3 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineError.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/AbstractGlusterBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeHealInfoVDSCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumesHealInfoReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 9 files changed, 132 insertions(+), 0 deletions(-) Approvals: Sahina Bose: Looks good to me, approved; Passed CI tests Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/57076 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bfa52bcc692ec2ef85443fd70dbb6ba08234bc3 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: gluster: entity changes for self heal monitoring
by sabose＠redhat.com 17 May '16

17 May '16

Sahina Bose has submitted this change and it was merged. Change subject: gluster: entity changes for self heal monitoring ...................................................................... gluster: entity changes for self heal monitoring Adding new columns for gluster self heal monitoring. Following columns will be added to gluster_volume_bricks. unsynced_entries - No.of unsynced entries in the brick. unsynced_entires_history - History of unsynced entries in the brick. It will be a list of comma separated values. Bug-Url: https://bugzilla.redhat.com/1205641 Change-Id: Icc6a4346fa06bb78f39e0ed08f65b8ac0527df83 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/gluster/GlusterFeatureSupported.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDaoImpl.java M packaging/dbscripts/gluster_volumes_sp.sql A packaging/dbscripts/upgrade/04_00_0660_add_gluster_self_heal_monitoring_related_columns_to_volume_and_bricks.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 8 files changed, 161 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Looks good to me, approved Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/57075 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc6a4346fa06bb78f39e0ed08f65b8ac0527df83 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(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: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: spec: Add postfix as requirement
by sbonazzo＠redhat.com 17 May '16

17 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: spec: Add postfix as requirement ...................................................................... spec: Add postfix as requirement Ship at least one mail server with the distro. Change-Id: I3df531bcaed947b1e5e6bfedc981b6fd0acf7948 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1301966 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release36.spec.in 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 Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57386 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3df531bcaed947b1e5e6bfedc981b6fd0acf7948 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(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: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-release[ovirt-4.0]: spec: Add postfix as requirement
by sbonazzo＠redhat.com 17 May '16

17 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: spec: Add postfix as requirement ...................................................................... spec: Add postfix as requirement Ship at least one mail server with the distro. Change-Id: I3df531bcaed947b1e5e6bfedc981b6fd0acf7948 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1301966 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release40.spec.in 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 Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3df531bcaed947b1e5e6bfedc981b6fd0acf7948 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(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: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-release[master]: node: Add mailx requirement
by sbonazzo＠redhat.com 17 May '16

17 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Add mailx requirement ...................................................................... node: Add mailx requirement Adding mailx to be able to read emails from daemon/agents. Change-Id: I952d56574af11ea4748c9d4c33c704b095bbd08e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1335503 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release-master.spec.in 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 Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57384 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I952d56574af11ea4748c9d4c33c704b095bbd08e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(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: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: Output end of stderr when schema.sh fails
by sbonazzo＠redhat.com 17 May '16

17 May '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Output end of stderr when schema.sh fails ...................................................................... packaging: setup: Output end of stderr when schema.sh fails Change-Id: I46cd919d467fe66a4e68d15e4fc3f182afa591cc Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py 1 file changed, 11 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/57546 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I46cd919d467fe66a4e68d15e4fc3f182afa591cc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-dwh[master]: History: Added log message - Application Settings
by Shirly Radco 17 May '16

17 May '16

Shirly Radco has submitted this change and it was merged. Change subject: History: Added log message - Application Settings ...................................................................... History: Added log message - Application Settings Added log message with the Application Settings, so it will be more clear if a change was made to the dwh settings for debugging. Change-Id: Ia7eb6dbac1c936502ad85cac7570bd439554778d Bug-Url: https://bugzilla.redhat.com/1324440 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/ovirt_engine_dwh/historyetl_4_0/contexts/Default.properties M etl_export/src/main/java/ovirt_engine_dwh/historyetl_4_0/HistoryETL.java M etl_export/src/main/resources/ovirt_engine_dwh/historyetl_4_0/contexts/Default.properties M tos_project/process/HistoryETL_4.0.item M tos_project/process/HistoryETL_4.0.properties 5 files changed, 1,029 insertions(+), 489 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/57459 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7eb6dbac1c936502ad85cac7570bd439554778d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0