September 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: userportal: changed two icons
by tjelinek＠redhat.com 02 Sep '13

02 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: userportal: changed two icons ...................................................................... userportal: changed two icons - Changed the disabled suspend icon in UP to be less dark - Changed the migrating icon to have a bigger up triangle Change-Id: I1e0929cbea92c198bc19df8dccfb7ec5dcddc714 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=997901 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/images/actions/icn_suspend_disabled.png M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/images/status/vm_migration.png 2 files changed, 0 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18724 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e0929cbea92c198bc19df8dccfb7ec5dcddc714 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: frontend: incorrect DC/Cluster handling on new pool
by tjelinek＠redhat.com 02 Sep '13

02 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: incorrect DC/Cluster handling on new pool ...................................................................... frontend: incorrect DC/Cluster handling on new pool Have one DC with two clusters. Make a template on the first cluster. Go to the new pool dialog and select the second cluster - it gets automatically reset to the first one. The problem was that the logic of selecting the DC/Cluster was implemented in the common place for new and edit dialog. For edit dialog it selected the DC/Cluster according to the selected pool. For new pool according to the selected template. While for edit pool it made sense to select the DC/Cluster according to edited pool it made no sense for new pool to select the DC/Cluster according to the selected template. Fixed by moving the logic specific for edit pool to the ExistingPoolModelBehavior from PoolModelBehaviorBase. Change-Id: I38e771d88fc300304942f750b6628862279a02ad Bug-Url: https://bugzilla.redhat.com/1002902 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewPoolModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java 3 files changed, 12 insertions(+), 12 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38e771d88fc300304942f750b6628862279a02ad Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: removing requirement on vdsm-gluster
by sbonazzo＠redhat.com 02 Sep '13

02 Sep '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: removing requirement on vdsm-gluster ...................................................................... removing requirement on vdsm-gluster The use the hosted engine setup does in gluster doesn't require the vdsm-gluster RPM, as the functionality of creating storage domains exists in the core of VDSM (vdsm-gluster is adding functionality to create gluster bricks and volumes, which isn't required here). Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1001161 Change-Id: I40722917a3ed4b51bf16db480c8d98fb3ff8d6ed Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/18735 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40722917a3ed4b51bf16db480c8d98fb3ff8d6ed Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: remove 'default' term from iso nfs export
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: remove 'default' term from iso nfs export ...................................................................... packaging: setup: remove 'default' term from iso nfs export confusing term for share creation based on dialog. Change-Id: I678ef0dd2ad18e72f2b7098c208d2f1bdbfdc958 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/system/exportfs.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18752 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I678ef0dd2ad18e72f2b7098c208d2f1bdbfdc958 Gerrit-PatchSet: 3 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> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: do not mark iptables unremovable if not ac...
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: do not mark iptables unremovable if not actually used ...................................................................... packaging: setup: do not mark iptables unremovable if not actually used Change-Id: I682c001567e3271fa6a10e166d9c2b83fc643995 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py 1 file changed, 18 insertions(+), 10 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18740 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I682c001567e3271fa6a10e166d9c2b83fc643995 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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: always use public http/https ports
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: always use public http/https ports ...................................................................... packaging: setup: always use public http/https ports Change-Id: I560efda78530d493bdb1cb1d7475288da9e9c7d6 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-rename/core/tools.py M packaging/setup/plugins/ovirt-engine-setup/config/tools.py M packaging/setup/plugins/ovirt-engine-setup/network/firewall.py 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18738 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I560efda78530d493bdb1cb1d7475288da9e9c7d6 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: disable yum packager at rename
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: disable yum packager at rename ...................................................................... packaging: setup: disable yum packager at rename Change-Id: I9b8e8ae5cb66dcd1bd3deb61dfbf7c2dd940a9f2 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/bin/ovirt-engine-rename 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18707 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b8e8ae5cb66dcd1bd3deb61dfbf7c2dd940a9f2 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: remove unused engine-db-install.sh script
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: remove unused engine-db-install.sh script ...................................................................... packaging: setup: remove unused engine-db-install.sh script script used by legacy setup, no need for it now. Change-Id: I67ca34f17c9676c3c3f755b11cc62bef708feb81 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- D packaging/dbscripts/engine-db-install.sh M packaging/setup/ovirt_engine_setup/constants.py 2 files changed, 0 insertions(+), 440 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18705 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67ca34f17c9676c3c3f755b11cc62bef708feb81 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: clear PGPASSWORD when interacting with dat...
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: clear PGPASSWORD when interacting with database ...................................................................... packaging: setup: clear PGPASSWORD when interacting with database when both PGPASSWORD and PGPASSFILE are specified the PGPASSWORD takes precedence, so we should unset/zero it to avoid errors when user has PGPASSWORD within environment and executes setup. Change-Id: Ie02939fe51eb4308d76a65e9eb1eb86df4a6801c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/dbscripts/dbcustomfunctions.sh M packaging/setup/ovirt_engine_setup/database.py 2 files changed, 3 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18704 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie02939fe51eb4308d76a65e9eb1eb86df4a6801c 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Spenser Shumaker <sshumake(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: modify AIA port when upgrade from legacy
by oschreib＠redhat.com 02 Sep '13

02 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: modify AIA port when upgrade from legacy ...................................................................... packaging: setup: modify AIA port when upgrade from legacy in case of moving configuration from jboss as web server and apache as web server we need to modify AIA extension URL. this will not effect past certificate, only new certificate. the engine rename script may be used to refresh the engine certificate. Change-Id: I6d955b34497e3256528180938dc67575d7812646 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/legacy/ca.py 1 file changed, 75 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18687 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d955b34497e3256528180938dc67575d7812646 Gerrit-PatchSet: 3 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> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0