March 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Upgrade - Error Can't start the ovirt-engine-dwhd serv...
by yzaslavs＠redhat.com 11 Mar '13

11 Mar '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Upgrade - Error Can't start the ovirt-engine-dwhd service ...................................................................... core: Upgrade - Error Can't start the ovirt-engine-dwhd service Fix column name to _update_date instead of update_date Add column to all relevant tables in fixtures.xml Change-Id: If99b9dca0f1b83f1a815d4ebb112828a135e495a Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=918152 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/upgrade/03_02_0490_add_update_column_to_statistics_tables.sql M backend/manager/modules/dal/src/test/resources/fixtures.xml 2 files changed, 40 insertions(+), 6 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12942 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If99b9dca0f1b83f1a815d4ebb112828a135e495a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[engine_3.2]: packaging: Use POM_VERSION to locate artifacts
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Use POM_VERSION to locate artifacts ...................................................................... packaging: Use POM_VERSION to locate artifacts We currently use APP_VERSION to locate artifacts in the directory where they are deployed during the build, but this doesn't work if APP_VERSION is different to the version specified in the root POM. Change-Id: I6b780190ccb2c71b40f43f4697cd04c061fca9ad Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12941 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b780190ccb2c71b40f43f4697cd04c061fca9ad Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Use POM_VERSION to locate artifacts
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Use POM_VERSION to locate artifacts ...................................................................... packaging: Use POM_VERSION to locate artifacts We currently use APP_VERSION to locate artifacts in the directory where they are deployed during the build, but this doesn't work if APP_VERSION is different to the version specified in the root POM. Change-Id: I6b780190ccb2c71b40f43f4697cd04c061fca9ad Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12939 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b780190ccb2c71b40f43f4697cd04c061fca9ad Gerrit-PatchSet: 1 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>

1 0

Change in ovirt-engine[engine_3.2]: packaging: Try nfs and nfs-server
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Try nfs and nfs-server ...................................................................... packaging: Try nfs and nfs-server When configuring the NFS service we need to try with both names, as some distributions use one of them and other distributions use the other. Change-Id: I5cdb4c1d0930a058f42d63d37fcab647391754ac Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit ec09f0381d193949c2660ed64bcad80c0f641f04) --- M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-setup.py 2 files changed, 22 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12864 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cdb4c1d0930a058f42d63d37fcab647391754ac Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[engine_3.2]: packaging: Use Id=... to check systemd services
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Use Id=... to check systemd services ...................................................................... packaging: Use Id=... to check systemd services The services managed by systemd can have an unique identifier and then several alternative names, or aliases. The NFS service, for example, has "nfs-server" as identifier but it also has "nfs" and "nfs-server" as aliases. These alternative names can be used for some operations, but not for all of them, in particular the "systemctl enable ..." operation requires the main name, it will not work with alternative names. This meas that we need to make sure that the method that checks the availability of services uses the main name, otherwise it can succeed but then later the call to "systemctl enable" can fail. Change-Id: I60ba0a5481f191c67eeb1fed6abd3f961cfbc8f4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit b2f6cbeac1cef232b560269faa50de4d030bb282) --- M packaging/fedora/setup/common_utils.py 1 file changed, 14 insertions(+), 7 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12863 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60ba0a5481f191c67eeb1fed6abd3f961cfbc8f4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[engine_3.2]: packaging: Enable features according to distribution
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Enable features according to distribution ...................................................................... packaging: Enable features according to distribution This change introduces two new "install_systemd" and "install_systemv" macros that control if the service should be controlled by systemd or by System V init scripts. These macros are enabled or disabled according to the type of distribution. Change-Id: I7b0318a6ddd4067751c796e4ab0fa150b6091204 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 4fcb3bd6f2b4720a7e497b73f0506595a24558a0) --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 32 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12862 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b0318a6ddd4067751c796e4ab0fa150b6091204 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Upgrade - Error Can't start the ovirt...
by emesika＠redhat.com 11 Mar '13

11 Mar '13

Eli Mesika has submitted this change and it was merged. Change subject: core: Upgrade - Error Can't start the ovirt... ...................................................................... core: Upgrade - Error Can't start the ovirt... Upgrade - Error Can't start the ovirt-engine-dwhd service Adding _updated field to the following tables: vds_interface_statistics vm_interface_statistics vm_statistics vds_statistics disk_image_dynamic storage_domain_dynamic This field will be updated with the current time-stamp and will be available for history queries Those fields will be included by the dwh views and used from the reports. Change-Id: Ie8f39b5cdc3771f438f9c61897cb0242e8bb6c90 Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=918152 --- M backend/manager/dbscripts/disk_image_dynamic_sp.sql M backend/manager/dbscripts/network_sp.sql M backend/manager/dbscripts/storages_san_sp.sql A backend/manager/dbscripts/upgrade/03_02_0490_add_update_column_to_statistics_tables.sql M backend/manager/dbscripts/vds_sp.sql M backend/manager/dbscripts/vms_sp.sql 6 files changed, 12 insertions(+), 6 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12919 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8f39b5cdc3771f438f9c61897cb0242e8bb6c90 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Enable features according to distribution
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Enable features according to distribution ...................................................................... packaging: Enable features according to distribution This change introduces two new "install_systemd" and "install_systemv" macros that control if the service should be controlled by systemd or by System V init scripts. These macros are enabled or disabled according to the type of distribution. Change-Id: If6b2afe381169943c5fc03f78a6c38bea790dd13 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 32 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12764 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6b2afe381169943c5fc03f78a6c38bea790dd13 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[engine_3.2]: packaging: Move passwd encryption script to tools
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Move passwd encryption script to tools ...................................................................... packaging: Move passwd encryption script to tools This script used to be in the PKI directory, but it is a tool used outside of the PKI environment, so it is better placed in the bin directory and owned by the tools package. Change-Id: Iafde9090b17dcc3ddf56de6b5117806870eeb855 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit cdab98683d8e5eb63a58fc7ca4d92b7ce2b1937f) --- M Makefile D backend/manager/conf/ca/encryptpasswd.sh A backend/manager/tools/src/main/shell/engine-encrypt-passwd.sh M packaging/fedora/setup/basedefs.py M packaging/fedora/spec/ovirt-engine.spec.in 5 files changed, 35 insertions(+), 46 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12861 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iafde9090b17dcc3ddf56de6b5117806870eeb855 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[engine_3.2]: core: Removed the ResXResourceReader class
by juan.hernandez＠redhat.com 11 Mar '13

11 Mar '13

Juan Hernandez has submitted this change and it was merged. Change subject: core: Removed the ResXResourceReader class ...................................................................... core: Removed the ResXResourceReader class Removed the class ResXResourceReader from the compat module, this class was used to load the audit log messages from the properties file, it was used only in AuditLogDirector, the logic in the class was improved and moved to a method inside AuditLogDirector Change-Id: Iacca292e7e725a8e243ad8fb57e4522ac3e082df Signed-off-by: Tal Nisan <tnisan(a)redhat.com> (cherry picked from commit 2084bd522fe50372d1318a78258014033ae95705) --- D backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/backendcompat/ResXResourceReader.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java 2 files changed, 619 insertions(+), 640 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacca292e7e725a8e243ad8fb57e4522ac3e082df Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0