May 2013 - Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: packaging: engine-service: fix sysv typeo
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: engine-service: fix sysv typeo ...................................................................... packaging: engine-service: fix sysv typeo Change-Id: Ie76ef19aed39b8ba8b2af646c110cfe864996ffe Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine.sysv.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14429 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie76ef19aed39b8ba8b2af646c110cfe864996ffe 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: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Added generic DB validations before the upgrade
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Added generic DB validations before the upgrade ...................................................................... packaging: Added generic DB validations before the upgrade * This patch adds a simple validatedb.sh script that will run required validation scripts. * When a new validation script is added, it should be added to the validatedb runner script as well. * During the upgrade, this script is executed and handles the validations control to corresponding scripts. * If user decides not to clear DB issues or there's any other error, the exception is raised. * Patch also introduces some code cleanups Change-Id: Ief0e65231bb42b569734cdd2fa6491b645491f98 Bug-Url: https://bugzilla.redhat.com/893459 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M Makefile M backend/manager/tools/dbutils/fkvalidator.sh M backend/manager/tools/dbutils/fkvalidator_sp.sql A backend/manager/tools/dbutils/validatedb.sh M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-upgrade.py M packaging/fedora/spec/ovirt-engine.spec.in 7 files changed, 200 insertions(+), 25 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief0e65231bb42b569734cdd2fa6491b645491f98 Gerrit-PatchSet: 18 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-engine[master]: packaging: move all packaging related to packaging
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: move all packaging related to packaging ...................................................................... packaging: move all packaging related to packaging currently we manage packaging scripts all over the tree, sometimes in different names than the file installed. this patch collects all files to be under the packaging directory using hierarchy similar to the installation and using same file naming. Change-Id: I6d0bd7b8cd5381ddb80b4b2e5ee7df65bab487bc Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore M Makefile R packaging/bin/engine-config.sh R packaging/bin/engine-encrypt-passwd.sh R packaging/bin/engine-manage-domains.sh R packaging/bin/engine-notifier-service.sh R packaging/bin/engine-notifier.sh R packaging/bin/engine-prolog.sh.in R packaging/conf/engine-config-install.properties R packaging/conf/engine.conf.defaults.in R packaging/conf/jaas.conf R packaging/etc/engine-config/engine-config.conf R packaging/etc/engine-config/engine-config.properties R packaging/etc/engine-config/log4j.xml.in R packaging/etc/engine-manage-domains/engine-manage-domains.conf R packaging/etc/engine-manage-domains/log4j.xml.in R packaging/etc/engine.conf R packaging/etc/notifier/log4j.xml.in R packaging/etc/notifier/notifier.conf R packaging/etc/pki/CreateCA.sh R packaging/etc/pki/SignReq.lock R packaging/etc/pki/SignReq.sh R packaging/etc/pki/cacert.template.in R packaging/etc/pki/cert.template.in R packaging/etc/pki/certs/.keep R packaging/etc/pki/installCA.sh R packaging/etc/pki/keys/.keep R packaging/etc/pki/openssl.conf R packaging/etc/pki/private/.keep R packaging/etc/pki/requests/.keep R packaging/etc/sysprep/sysprep.2k12x64 R packaging/etc/sysprep/sysprep.2k3 R packaging/etc/sysprep/sysprep.2k8 R packaging/etc/sysprep/sysprep.2k8x86 R packaging/etc/sysprep/sysprep.w7 R packaging/etc/sysprep/sysprep.w7x64 R packaging/etc/sysprep/sysprep.w8 R packaging/etc/sysprep/sysprep.w8x64 R packaging/etc/sysprep/sysprep.xp R packaging/etc/usbfilter.txt 40 files changed, 34 insertions(+), 34 deletions(-) Approvals: Alon Bar-Lev: Verified Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14302 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d0bd7b8cd5381ddb80b4b2e5ee7df65bab487bc Gerrit-PatchSet: 6 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: Allon Mureinik <amureini(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: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: engine.conf: add quotes to selected keys
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: engine.conf: add quotes to selected keys ...................................................................... packaging: engine.conf: add quotes to selected keys this to support potential space within value, while sourcing in shell. Change-Id: If7b0086fc2927449b54877e3cf1a18bd6e9480e0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/conf/engine.conf.defaults.in 1 file changed, 7 insertions(+), 7 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14352 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If7b0086fc2927449b54877e3cf1a18bd6e9480e0 Gerrit-PatchSet: 4 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: engine-cleanup: drop user as well
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: engine-cleanup: drop user as well ...................................................................... packaging: engine-cleanup: drop user as well currently we drop database, but we create user as well, no reason why not to attempt remove it. Change-Id: I5cc437890d7cce686329435c12d82a50ab0df4b0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-cleanup.py 2 files changed, 13 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14363 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cc437890d7cce686329435c12d82a50ab0df4b0 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: 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: enigne-service: cleanup to ease reuse
by oschreib＠redhat.com 05 May '13

05 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: enigne-service: cleanup to ease reuse ...................................................................... packaging: enigne-service: cleanup to ease reuse Change-Id: I755f550f6992851fd70e03afa1db008d6b42d9d4 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/services/ovirt-engine.py M packaging/services/ovirt-engine.systemd.in M packaging/services/ovirt-engine.sysv.in 3 files changed, 15 insertions(+), 16 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14372 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I755f550f6992851fd70e03afa1db008d6b42d9d4 Gerrit-PatchSet: 4 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0