May 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: history: Updated calculation of cpu usage
by oourfali＠redhat.com 14 May '14

14 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: history: Updated calculation of cpu usage ...................................................................... history: Updated calculation of cpu usage Updated calculations of system_cpu_usage_percent and user_cpu_usage_percent according to the number of cpus. Change-Id: Idfae05895d8450e6d5f450fbf2a9e934e65a5785 Bug-Url: https://bugzilla.redhat.com/1078897 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql 1 file changed, 15 insertions(+), 5 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27559 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfae05895d8450e6d5f450fbf2a9e934e65a5785 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: history: update user and system cpu usage percent
by ydary＠redhat.com 14 May '14

14 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: history: update user and system cpu usage percent ...................................................................... history: update user and system cpu usage percent Updated retroactivly user and system cpu usage percent, according to the number of cpus of the vm. Change-Id: Ia47326f94b0b1b696d0bcaf13b5f6df0093b9196 Bug-Url: https://bugzilla.redhat.com/1078897 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- A packaging/dbscripts/upgrade/03_05_0020_update_cpu_usage_percent.sql 1 file changed, 369 insertions(+), 0 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27556 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia47326f94b0b1b696d0bcaf13b5f6df0093b9196 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: performance improvement
by masayag＠redhat.com 14 May '14

14 May '14

Moti Asayag has submitted this change and it was merged. Change subject: core: performance improvement ...................................................................... core: performance improvement * initialized ArrayList to avoid copying arrays (tradeof is bigger memory footprint during initialization) * changed transforming macAddress stored in 'long' to String; loop is used instead of regex. --- this changes cuts CPU time approx to 50% of original. --- Note: code can be suboptimal when looking for LOT OF macs but range offers to little of them: in that case unneccesarily big array alocated and then trown away for no reason. But this should not be an issue. Change-Id: I09f87ebd1ea17d09a974ce6ca3e26bd2454a72fc Bug-Url: https://bugzilla.redhat.com/1063064 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/MacAddressRangeUtils.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/MacAddressRangeUtilsTest.java 2 files changed, 39 insertions(+), 3 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Martin Mucha: Verified -- To view, visit http://gerrit.ovirt.org/25940 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09f87ebd1ea17d09a974ce6ca3e26bd2454a72fc Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: history: added dwh_users_history_view
by oourfali＠redhat.com 14 May '14

14 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: history: added dwh_users_history_view ...................................................................... history: added dwh_users_history_view Added dwh_users_history_view and added console_user_id to dwh_vm_history_view. Added _create_date and _update_date to users table. Change-Id: I1995308951b445bbaa5ab6d944d51f64368c44fc Bug-Url: https://bugzilla.redhat.com/1091687 Bug-Url: https://bugzilla.redhat.com/1018416 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql A packaging/dbscripts/upgrade/03_05_0440_add_create_and_update_dates_to_users_table.sql M packaging/dbscripts/user_sp.sql 3 files changed, 22 insertions(+), 5 deletions(-) Approvals: Shirly Radco: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27339 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1995308951b445bbaa5ab6d944d51f64368c44fc Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: reports: moved to using only ce edition dashboards in webadmin
by ydary＠redhat.com 14 May '14

14 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: moved to using only ce edition dashboards in webadmin ...................................................................... reports: moved to using only ce edition dashboards in webadmin Change-Id: I0c62986ace8f7b04a038d4b0c461580639335bed Bug-Url: https://bugzilla.redhat.com/1096841 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- R packaging/conf/reports.xml M packaging/setup/ovirt_engine_setup/reportsconstants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py 3 files changed, 17 insertions(+), 29 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Yedidyah Bar David: Looks good to me, but someone else must approve Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27644 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c62986ace8f7b04a038d4b0c461580639335bed Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-release[master]: packaging: provide mirrorlist for oVirt repo
by sbonazzo＠redhat.com 14 May '14

14 May '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: provide mirrorlist for oVirt repo ...................................................................... packaging: provide mirrorlist for oVirt repo Change-Id: Ia2e3dfba22bcb7354212a6f2cd55d040fe8dbee9 Bug-Url: https://bugzilla.redhat.com/1097307 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-3.4/configure.ac M ovirt-release-3.4/ovirt-release34.spec.in M ovirt-release-3.4/ovirt.repo.in 3 files changed, 6 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Kiril Nesenko: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27654 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2e3dfba22bcb7354212a6f2cd55d040fe8dbee9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Kiril Nesenko <knesenko(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-engine[master]: core: Numa engine/vdsm integration patch
by gchaplik＠redhat.com 14 May '14

14 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: core: Numa engine/vdsm integration patch ...................................................................... core: Numa engine/vdsm integration patch This patch implement the communication between vdsm and ovirt engine to collect numa node information and statistics data, also set the numa tune configurations. Change-Id: I050bc9d80a90ac73b5642ccd7630dd352eba236e Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NumaUtils.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 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/VdsBrokerObjectsBuilder.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilderBase.java 6 files changed, 467 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Looks good to me, approved Xiaolei Shi: Verified -- To view, visit http://gerrit.ovirt.org/27083 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I050bc9d80a90ac73b5642ccd7630dd352eba236e Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: NUMA feature actions and queries
by gchaplik＠redhat.com 14 May '14

14 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: NUMA feature actions and queries ...................................................................... engine: NUMA feature actions and queries Queries and actions command for NUMA features. UX and Restful API development could base on this commit. The actions and queries define is on ovirt wiki page http://www.ovirt.org/Features/Detailed_NUMA_and_Virtual_NUMA Change-Id: I2f297311a49c051b7ae2daee540d96210bf1a66d Bug-Url: https://bugzilla.redhat.com/1069303 Signed-off-by: Jason Liao <chuan.liao(a)hp.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandsFactory.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/host/GetCpuStatisticsByVdsIdQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/host/GetVdsNumaNodesByVdsIdQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/AddVmNumaNodesCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/GetAllVmsWithNumaByVdsGroupIdQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/GetVmNumaNodesByVmIdQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/RemoveVmNumaNodesCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/UpdateVmNumaNodesCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VmNumaNodeOperationParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 13 files changed, 347 insertions(+), 0 deletions(-) Approvals: Jason Liao: Verified Gilad Chaplik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26810 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2f297311a49c051b7ae2daee540d96210bf1a66d Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jason Liao <chuan.liao(a)hp.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jason Liao <chuan.liao(a)hp.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: db: Numa support database implementation
by gchaplik＠redhat.com 14 May '14

14 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: db: Numa support database implementation ...................................................................... db: Numa support database implementation For more information see: http://www.ovirt.org/Features/Detailed_NUMA_and_Virtual_NUMA#Interface_betw… Change-Id: I2507c084aa214bcfb65e860b11ed7dcf02af50cc Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsCpuStatisticsDAO.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsCpuStatisticsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDynamicDAODbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsNumaNodeDAO.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsNumaNodeDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmNumaNodeDAO.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmNumaNodeDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/jdbc-resources/engine-daos.properties A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsCpuStatisticsDAOTest.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsNumaNodeDAOTest.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmNumaNodeDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M packaging/dbscripts/create_views.sql A packaging/dbscripts/numa_sp.sql A packaging/dbscripts/upgrade/03_05_0430_add_numa_tables_and_columns.sql M packaging/dbscripts/upgrade/post_upgrade/0010_add_object_column_white_list_table.sql M packaging/dbscripts/vds_sp.sql M packaging/dbscripts/vms_sp.sql 22 files changed, 2,220 insertions(+), 17 deletions(-) Approvals: Gilad Chaplik: Looks good to me, approved Xiaolei Shi: Verified -- To view, visit http://gerrit.ovirt.org/26996 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2507c084aa214bcfb65e860b11ed7dcf02af50cc Gerrit-PatchSet: 28 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jason Liao <chuan.liao(a)hp.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Numa feature entities
by gchaplik＠redhat.com 14 May '14

14 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: Numa feature entities ...................................................................... engine: Numa feature entities The entities define for NUMA support feature. See http://www.ovirt.org/Features/NUMA_and_Virtual_NUMA for the feature information. Change-Id: Ifefade432e7955503980bdc6fc5d73ea32818a95 Bug-Url: https://bugzilla.redhat.com/1069303 Signed-off-by: Bruce Shi <xiao-lei.shi(a)hp.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/AutoNumaBalanceStatus.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/CpuStatistics.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/NumaNodeStatistics.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/NumaTuneMode.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsDynamic.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsNumaNode.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatistics.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmNumaNode.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatic.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 12 files changed, 648 insertions(+), 3 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved Xiaolei Shi: Verified -- To view, visit http://gerrit.ovirt.org/23702 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifefade432e7955503980bdc6fc5d73ea32818a95 Gerrit-PatchSet: 31 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: Xiaolei Shi <xiao-lei.shi(a)hp.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0