- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: tools: Proper error testing in EngineConfigCLITest
by amureini＠redhat.com 06 May '13

06 May '13

Allon Mureinik has submitted this change and it was merged. Change subject: tools: Proper error testing in EngineConfigCLITest ...................................................................... tools: Proper error testing in EngineConfigCLITest Converted the error tests in EngineConfigCLITest to the proper JUnit form (with the expect parameter on the @Test annotation), instead of the awkward explicit fail before the catch block. Since the test methods have meaningful names, this change also allowed removing the System.out.println calls. Change-Id: Iad25eae4fd7b4535cc1b2d9570f142d6996b79d0 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/tools/src/test/java/org/ovirt/engine/core/config/EngineConfigCLIParserTest.java 1 file changed, 4 insertions(+), 20 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14424 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad25eae4fd7b4535cc1b2d9570f142d6996b79d0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine:Sync gluster hooks
by sanjal＠redhat.com 06 May '13

06 May '13

Shireesh Anjal has submitted this change and it was merged. Change subject: engine:Sync gluster hooks ...................................................................... engine:Sync gluster hooks This patch adds a periodic job to sync gluster hooks on servers in the cluster with the hooks stored in database. Every execution of the job - Retrieves the list of hooks from each server in the cluster - Compares the list with the saved data in database - If different, inserts or updates the database - Changes conflict status in db, if any status, content or missing conflict detected. Added GlusterHooksEnabled to GlusterFeatureSupported. Change-Id: I2766edd4e810677a200cf5c45d334cabef5f2924 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/dbscripts/gluster_hooks_sp.sql M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterHookSyncJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterJobsManager.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterHookSyncJobTest.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/businessentities/gluster/GlusterHookEntity.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/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterHooksDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterHooksDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 13 files changed, 661 insertions(+), 18 deletions(-) Approvals: Shireesh Anjal: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13943 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2766edd4e810677a200cf5c45d334cabef5f2924 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Allow editing a bootable disk
by amureini＠redhat.com 05 May '13

05 May '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Allow editing a bootable disk ...................................................................... core: Allow editing a bootable disk Fixed a couple of bugs in getOtherVmDisks(Guid) to allow editing a bootable disk: 1. Once the list of disks is produced, it should be saved in the cache map 2. The break statement should be inside the if, not after it. Change-Id: I111de3ff5710d29d1301c0c47e12755d88e016b9 Bug-Url: https://bugzilla.redhat.com/959761 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Looks good to me, approved Allon Mureinik: Verified Liron Ar: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14439 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I111de3ff5710d29d1301c0c47e12755d88e016b9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Sergey Gotliv <sgotliv(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Added encodingvalidator.sh to validation list
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Added encodingvalidator.sh to validation list ...................................................................... packaging: Added encodingvalidator.sh to validation list This patch updates encodingvalidator.sh utility and adds it to the list of executed validators in validatedb.sh generic script. Change-Id: Ic89088c7738eb379b2bc88d8d59193d4ed20a295 Bug-Url: https://bugzilla.redhat.com/920565 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M Makefile M backend/manager/tools/dbutils/encodingvalidator.sh M backend/manager/tools/dbutils/validatedb.sh 3 files changed, 11 insertions(+), 8 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14389 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic89088c7738eb379b2bc88d8d59193d4ed20a295 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-dwh[master]: history: changed vm ip cloumn to text (#953148)
by ydary＠redhat.com 05 May '13

05 May '13

Yaniv Dary has submitted this change and it was merged. Change subject: history: changed vm ip cloumn to text (#953148) ...................................................................... history: changed vm ip cloumn to text (#953148) This is to match the engine type and allow almost unlimited ip amount per vm. Bug-Url: https://bugzilla.redhat.com/953148 Change-Id: Ifdf493916b22cd53fc36b36ec128add43e655941 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.2.item M data-warehouse/history_etl/tos_project/process/AggregationToDaily_3.2.properties M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.2.item M data-warehouse/history_etl/tos_project/process/AggregationToHourly_3.2.properties M data-warehouse/history_etl/tos_project/process/StatisticsSync_3.2.item M data-warehouse/history_etl/tos_project/process/StatisticsSync_3.2.properties M data-warehouse/historydbscripts_postgres/create_views_3_0.sql M data-warehouse/historydbscripts_postgres/create_views_3_1.sql A data-warehouse/historydbscripts_postgres/upgrade/03_02_0070_changed_vm_ip_column_to_text.sql 9 files changed, 26 insertions(+), 22 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14437 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdf493916b22cd53fc36b36ec128add43e655941 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh 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]: Adding fencing parameters to log print
by yzaslavs＠redhat.com 05 May '13

05 May '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: Adding fencing parameters to log print ...................................................................... Adding fencing parameters to log print Change-Id: Ic14f3ae9e302ffaf8e4fc883f2a177864e512c4f Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14436 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic14f3ae9e302ffaf8e4fc883f2a177864e512c4f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: complete the transition of packaging filesystem l...
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: complete the transition of packaging filesystem like structure ...................................................................... packaging: complete the transition of packaging filesystem like structure move files within the packaging/fedora/setup. Change-Id: I637d1659a528ad4a0e7539faaa80cb1d9a80c154 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile R packaging/conf/iptables.default R packaging/conf/nfs.sysconfig R packaging/conf/ovirt-engine-proxy.conf.in R packaging/conf/ovirt-engine-root-redirect.conf.in 5 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14435 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I637d1659a528ad4a0e7539faaa80cb1d9a80c154 Gerrit-PatchSet: 2 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: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: move spec to root
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: move spec to root ...................................................................... packaging: move spec to root spec file is long not fedora only, usually projects that contains spec have this within their root, no reason why we cannot behave the same. Change-Id: I4e022efd8a709ed939b2fca5d7374723c3530277 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore M Makefile R ovirt-engine.spec.in D packaging/fedora/spec/ovirt-engine.spec.tmpl 4 files changed, 5 insertions(+), 6 deletions(-) Approvals: Alon Bar-Lev: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14431 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e022efd8a709ed939b2fca5d7374723c3530277 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: refactored firewalld support
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: refactored firewalld support ...................................................................... packaging: refactored firewalld support Added one service configuration template file for each service that should be allowed by firewalld under /usr/share/ovirt-engine/firewalld/*. During firewalld configuration the template files will be processed by processTemplate with provided ports. Then the service files will be copied to /etc/firewalld/services and enabled for the active zones. The ovirt-aio service will be enabled only if the plugin is installed and enabled. Change-Id: If6a60e1667182f926f399e22bbfbb939e7171cb0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M Makefile M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-setup.py D packaging/fedora/setup/firewalld.ovirt.xml M packaging/fedora/setup/output_messages.py M packaging/fedora/setup/plugins/all_in_one_100.py M packaging/fedora/spec/ovirt-engine.spec.in A packaging/firewalld/aio/ovirt-aio.xml.in A packaging/firewalld/base/ovirt-http.xml.in A packaging/firewalld/base/ovirt-https.xml.in A packaging/firewalld/base/ovirt-nfs.xml.in 11 files changed, 115 insertions(+), 44 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13638 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If6a60e1667182f926f399e22bbfbb939e7171cb0 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: quota - subtabs resizable columns
by gchaplik＠redhat.com 05 May '13

05 May '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: quota - subtabs resizable columns ...................................................................... webadmin: quota - subtabs resizable columns Invoking enableColumnResizing() in all quota related subtabs, and set fixed width for the sub-tab columns. Quota subtab under datacenters main tab was already covered in another patch. Change-Id: Ib4702514c4316a4e2022018d6439e437d326caac Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/quota/SubTabQuotaClusterView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/quota/SubTabQuotaStorageView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/quota/SubTabQuotaUserView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/user/SubTabUserQuotaView.java 4 files changed, 20 insertions(+), 34 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14278 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4702514c4316a4e2022018d6439e437d326caac Gerrit-PatchSet: 2 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>

1 0