July 2012 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: reports: input controls now cast all uuids to strings
by ydary＠redhat.com 11 Jul '12

11 Jul '12

Yaniv Dary has submitted this change and it was merged. Change subject: reports: input controls now cast all uuids to strings ...................................................................... reports: input controls now cast all uuids to strings This is to increase compatibility with any db type implementation of uuids Change-Id: Ie9c1c30cde5a789fae71252c2d5d3374a6930b0c Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/cluster_ic/all_cluster_with_vms/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/cluster_ic/all_clusters_with_hosts/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_entities/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_hosts/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_storage/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_vms/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/host_ic/Hosts_List.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/storage_ic/P_StorageDomain_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/vm_ic/VM_List.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_ListID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/P_StorageDomain_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_VM_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_VM_ListID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/webadmin_dashboards_ic/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/webadmin_dashboards_ic/P_DataCenter_ID.xml 21 files changed, 32 insertions(+), 32 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6156 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9c1c30cde5a789fae71252c2d5d3374a6930b0c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Don't put .jar files in separate lib directories
by oschreib＠redhat.com 11 Jul '12

11 Jul '12

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Don't put .jar files in separate lib directories ...................................................................... packaging: Don't put .jar files in separate lib directories At the moment we are copying the tools .jar files to specific directories under /usr/share/ovirt-engine directory, but the scripts don't use that, but the /usr/share/java/ovirt-engine directory. Change-Id: I8bdad169b2a69782455023c983a955b9cb23131b Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile 1 file changed, 3 insertions(+), 21 deletions(-) Approvals: Juan Hernandez: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6153 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bdad169b2a69782455023c983a955b9cb23131b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Move JBoss modules from Makefile to spec
by oschreib＠redhat.com 11 Jul '12

11 Jul '12

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Move JBoss modules from Makefile to spec ...................................................................... packaging: Move JBoss modules from Makefile to spec Most of the JBoss modules currently installed by the Makefile only needed by the JBoss package available in Fedora right now, so they are better placed in the spec file for the Fedora RPMS. Change-Id: Ie4d4dd6d29c4db8486764d242d9c89ee8ecc0cba Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M packaging/fedora/spec/ovirt-engine.spec.in 2 files changed, 64 insertions(+), 57 deletions(-) Approvals: Juan Hernandez: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6060 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4d4dd6d29c4db8486764d242d9c89ee8ecc0cba Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: cleanup
by mpastern＠redhat.com 11 Jul '12

11 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: cleanup ...................................................................... cli: cleanup Change-Id: I92c6897e3ebbe2e61aa807f0ca9120e4705a0b54 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/connect.py M src/ovirtcli/command/help.py M src/ovirtcli/main.py M src/ovirtcli/options.py M src/ovirtcli/shell/cmdshell.py 5 files changed, 4 insertions(+), 38 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6142 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92c6897e3ebbe2e61aa807f0ca9120e4705a0b54 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: implement generic parameters processing for /list/ comm...
by mpastern＠redhat.com 11 Jul '12

11 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement generic parameters processing for /list/ command ...................................................................... cli: implement generic parameters processing for /list/ command Change-Id: I4f70d6945cdda4bd377509b9f64649bd7488d919 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/command.py 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6134 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f70d6945cdda4bd377509b9f64649bd7488d919 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine-cli[master]: cli: implement debug mode
by mpastern＠redhat.com 11 Jul '12

11 Jul '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: implement debug mode ...................................................................... cli: implement debug mode Change-Id: I48cf5c42fe91dbcd0994195579647a8b6a26c2e3 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/command/connect.py M src/ovirtcli/context.py M src/ovirtcli/options.py 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ori Liel: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6133 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I48cf5c42fe91dbcd0994195579647a8b6a26c2e3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Don't install postgres-ds.xml
by oschreib＠redhat.com 11 Jul '12

11 Jul '12

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Don't install postgres-ds.xml ...................................................................... packaging: Don't install postgres-ds.xml This file is JBoss 5 specific, and no longer used by the engine or by the setup scripts. Change-Id: Ida2bb6ec4a51d1b7e0fe5ae1eca5122bd7c01a79 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile D packaging/fedora/setup/resources/postgres/postgres-ds.xml 2 files changed, 0 insertions(+), 14 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5902 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ida2bb6ec4a51d1b7e0fe5ae1eca5122bd7c01a79 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-dwh[master]: etl: fixed duplicate row issues (#838962)
by ydary＠redhat.com 11 Jul '12

11 Jul '12

Yaniv Dary has submitted this change and it was merged. Change subject: etl: fixed duplicate row issues (#838962) ...................................................................... etl: fixed duplicate row issues (#838962) Due to wrong data selection data was missing or two row were created for each hour\day. This patch fixes the selects. https://bugzilla.redhat.com/838962 Change-Id: I8f41a017ae5ebf91f230881485850e41a4f2eb53 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M data-warehouse/history_etl/etl_sources/historyETLProcedure/src/main/java/ovirt_engine_dwh/aggregationtodaily_3_1/AggregationToDaily.java M data-warehouse/history_etl/etl_sources/historyETLProcedure/src/main/java/ovirt_engine_dwh/aggregationtohourly_3_1/AggregationToHourly.java M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.1.item M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.1.properties M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.1.item M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.1.properties M data-warehouse/history_etl/tos_project/talend.project 7 files changed, 502 insertions(+), 502 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Yaniv Dary: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/6150 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8f41a017ae5ebf91f230881485850e41a4f2eb53 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Moving acquireLock to InternalCanDoAction
by mkublin＠redhat.com 11 Jul '12

11 Jul '12

Michael Kublin has submitted this change and it was merged. Change subject: core: Moving acquireLock to InternalCanDoAction ...................................................................... core: Moving acquireLock to InternalCanDoAction The following patch will move acquireLock inside InternalCanDoAction, these is done in order to promise of acquring lock during running command via MutltipleActionRunner Change-Id: Ibf805bb9984016abda411f2d9cf70be6832e4753 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/6149 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf805bb9984016abda411f2d9cf70be6832e4753 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Replace spring .jar files with symlinks
by oschreib＠redhat.com 11 Jul '12

11 Jul '12

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Replace spring .jar files with symlinks ...................................................................... packaging: Replace spring .jar files with symlinks We are currently including the required spring framework .jar files inside the lib directory of the engine .ear file. This patch replaces those .jar files with symbolic links to the actual locations of the .jar files provided by the springframework package. Change-Id: Idd9381cba8dd04b5533bf775caa9c250e6b8f936 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 18 insertions(+), 7 deletions(-) Approvals: Juan Hernandez: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5948 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd9381cba8dd04b5533bf775caa9c250e6b8f936 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0