- Engine-commits - oVirt List Archives

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

19 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: Ie81f13a96502236bdfa7fc537e8fa3a10481cf36 Bug-Url: https://bugzilla.redhat.com/1078897 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/dbscripts/create_dwh_views.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 16 insertions(+), 6 deletions(-) Approvals: Shirly Radco: Verified Eli Mesika: Looks good to me, approved Yaniv Dary: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/27683 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie81f13a96502236bdfa7fc537e8fa3a10481cf36 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: common: fix de-serialization issues
by gchaplik＠redhat.com 19 May '14

19 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: common: fix de-serialization issues ...................................................................... common: fix de-serialization issues Handle List of VmNumaNodeis in VM entity. Change-Id: Iaf63772f358a6358a4d15fe474c81ba907bf61f9 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27806 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf63772f358a6358a4d15fe474c81ba907bf61f9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: common: add missing 'override' annotations
by gchaplik＠redhat.com 19 May '14

19 May '14

Gilad Chaplik has submitted this change and it was merged. Change subject: common: add missing 'override' annotations ...................................................................... common: add missing 'override' annotations For VM BE. Change-Id: I324306d46b84d34237ba68a303d6178bf6619e6c Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified Omer Frenkel: Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27807 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I324306d46b84d34237ba68a303d6178bf6619e6c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: extmgr: use jboss module service load method
by yzaslavs＠redhat.com 19 May '14

19 May '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: extmgr: use jboss module service load method ...................................................................... extmgr: use jboss module service load method this is leftover from legacy code, now that I look at the jboss module api there is no reason why not to get this via the module. Change-Id: I5e0f7391dd34e17be1cc321f3114fac815093546 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/extensions-manager/src/main/java/org/ovirt/engine/core/extensions/mgr/ExtensionsManager.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27843 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e0f7391dd34e17be1cc321f3114fac815093546 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: reports: br22b report - added comments to queries
by ydary＠redhat.com 19 May '14

19 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: br22b report - added comments to queries ...................................................................... reports: br22b report - added comments to queries Changed the code structure so it will be more readable and added commets. Change-Id: I714da9cffbd626865deb0bcbf503afd1fe88cfcb Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/embedded_reports/Inventory_Dashboard/Host_OS_Break_Down_BR22B_for_inventory_files/Host_OS_Break_Down_BR22B_for_inventory_jrxml.data 1 file changed, 213 insertions(+), 157 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27827 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I714da9cffbd626865deb0bcbf503afd1fe88cfcb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports 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

1 0

Change in ovirt-engine[master]: webadmin : Existing gluster cluster import iptables config
by kmayilsa＠redhat.com 19 May '14

19 May '14

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Existing gluster cluster import iptables config ...................................................................... webadmin : Existing gluster cluster import iptables config This patch enables iptables rules configuration, when importing existing gluster cluster configurations Change-Id: Ib4a391cb1f912c4c052ff183de6463edcad4e0a9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1051019 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/MultipleHostsModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml 6 files changed, 34 insertions(+), 3 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26267 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4a391cb1f912c4c052ff183de6463edcad4e0a9 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: gluster: check gluster daemon status in sync-job
by oourfali＠redhat.com 19 May '14

19 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: gluster: check gluster daemon status in sync-job ...................................................................... gluster: check gluster daemon status in sync-job In sync-job, peer list will be retrieved always from the first UP server. There are chances that gluster daemon in other servers may be down. In this case, peer status for those servers will DISCONNECTED. When a host is found in UP state in Database and peer status is 'DISCONNECTED', that host will be moved to Non-Operational status. 'peer status' command will be executed on that server to make sure the gluster daemon is actually down before moving to Non-Operational. Change-Id: I3e0f661cb496e741f7c06df42ce3b55037a87e28 Bug-Url: https://bugzilla.redhat.com/1056997 Bug-Url: https://bugzilla.redhat.com/1097736 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java 1 file changed, 34 insertions(+), 20 deletions(-) Approvals: Kanagaraj M: Looks good to me, but someone else must approve Sahina Bose: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27838 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e0f661cb496e741f7c06df42ce3b55037a87e28 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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

Change in ovirt-engine[ovirt-engine-3.4]: core: VdsNotRespondingTreatment Job is not marked as finished.
by oourfali＠redhat.com 19 May '14

19 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: VdsNotRespondingTreatment Job is not marked as finished. ...................................................................... core: VdsNotRespondingTreatment Job is not marked as finished. When a hypervisor gets into non responding state and it is fenced the job VdsNotRespondingTreatment is not marked as finished and get stuck in the tasks list. Change-Id: I956c8f8d51e8c9569013973044494a8c95a58454 Bug-Url: https://bugzilla.redhat.com/1075500 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Liran Zelkha: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27826 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I956c8f8d51e8c9569013973044494a8c95a58454 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: history: changed network_name column
by ydary＠redhat.com 19 May '14

19 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: history: changed network_name column ...................................................................... history: changed network_name column Changed network_name column to logical_network_name in the database tables, views and ETL process. Change-Id: Ic3d2f8815a7dbb48552c1c7b4162c570ea3f43b1 Bug-Url: https://bugzilla.redhat.com/891073 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M etl_export/src/ovirt_engine_dwh/configurationsync_3_5/ConfigurationSync.java M packaging/dbscripts/create_views_3_0.sql M packaging/dbscripts/create_views_3_1.sql M packaging/dbscripts/create_views_3_2.sql M packaging/dbscripts/create_views_3_3.sql M packaging/dbscripts/create_views_3_4.sql M packaging/dbscripts/create_views_3_5.sql A packaging/dbscripts/upgrade/03_05_0040_change_network_name_column.sql M tos_project/process/ConfigurationSync_3.5.item M tos_project/process/ConfigurationSync_3.5.properties 10 files changed, 180 insertions(+), 176 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27789 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3d2f8815a7dbb48552c1c7b4162c570ea3f43b1 Gerrit-PatchSet: 4 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[ovirt-engine-3.4]: core: Batch updates might create a database deadlock
by oourfali＠redhat.com 19 May '14

19 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Batch updates might create a database deadlock ...................................................................... core: Batch updates might create a database deadlock Batch updates might create a database deadlock in systems under load Sorting the batch collection by id should solve this problem Change-Id: I1f1868c7a76ea50257f40929573e36ad304c353c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1098962 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/MassOperationsGenericDaoDbFacade.java 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Looks good to me, approved Liran Zelkha: Verified Yair Zaslavsky: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/27594 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1f1868c7a76ea50257f40929573e36ad304c353c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0