- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: do not attempt to start websocket proxy at...
by sbonazzo＠redhat.com 09 Dec '13

09 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: do not attempt to start websocket proxy at development mode ...................................................................... packaging: setup: do not attempt to start websocket proxy at development mode Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/config/websocket_proxy.py 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40f061ec6cf658e1d8ec5c3663698ceca7569b38 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: Sandro Bonazzola <sbonazzo(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]: webadmin : Retain bricks refresh
by kmayilsa＠redhat.com 09 Dec '13

09 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : Retain bricks refresh ...................................................................... webadmin : Retain bricks refresh On clicking retain brick button from remove-brick status pop up,after succesfull execution of the command the pop up was not getting refreshed and hence the stopped at time was not getting displayed.But,on closing the pop up and reopening it again,with the new pop up the stopped at time was getting displayed. Hence,this patch fixes it by refreshing the pop up after performing retain-brick option. Change-Id: Iddc56eec6f9e456c6fdcf6dd45152b58fb0878ed Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1037583 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeRebalanceStatusModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeRebalanceStatusPopupView.java 3 files changed, 8 insertions(+), 13 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21966 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iddc56eec6f9e456c6fdcf6dd45152b58fb0878ed Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: update firewall for all services
by sbonazzo＠redhat.com 09 Dec '13

09 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: update firewall for all services ...................................................................... packaging: setup: update firewall for all services Update the firewall for websocket_proxy and postgresql not only during setup. Always ask whether to update the firewall and do not keep the answer in the postinstall answer file, to allow users to keep their manual changes on upgrades. Output manual configuration instructions only for supported firewall managers. Bug-Url: https://bugzilla.redhat.com/1024707 Bug-Url: https://bugzilla.redhat.com/1029020 Bug-Url: https://bugzilla.redhat.com/1023316 Change-Id: If3c1a634b2e8539ebd604205b5487290c8d8a1a9 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py A packaging/setup/ovirt_engine_setup/firewall_manager_base.py A packaging/setup/plugins/ovirt-engine-common/network/__init__.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_firewalld.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_human.py A packaging/setup/plugins/ovirt-engine-common/network/firewall_manager_iptables.py A packaging/setup/plugins/ovirt-engine-common/network/process_firewalld_services.py M packaging/setup/plugins/ovirt-engine-remove/network/__init__.py A packaging/setup/plugins/ovirt-engine-remove/network/firewall_manager.py D packaging/setup/plugins/ovirt-engine-remove/network/firewalld.py M packaging/setup/plugins/ovirt-engine-setup/config/websocket_proxy.py M packaging/setup/plugins/ovirt-engine-setup/legacy/__init__.py D packaging/setup/plugins/ovirt-engine-setup/legacy/firewall_manager.py M packaging/setup/plugins/ovirt-engine-setup/network/firewall_manager.py M packaging/setup/plugins/ovirt-engine-setup/provisioning/postgres.py 16 files changed, 954 insertions(+), 393 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20737 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3c1a634b2e8539ebd604205b5487290c8d8a1a9 Gerrit-PatchSet: 38 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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

Change in ovirt-engine[master]: gluster: fix typo in gluster hooks event option names
by kmayilsa＠redhat.com 09 Dec '13

09 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: fix typo in gluster hooks event option names ...................................................................... gluster: fix typo in gluster hooks event option names Replace "GLuster" with "Gluster" in LocalizedEnums.properties file for gluster hook add, remove and update options. Change-Id: I5577938b32f26507e6044b1f4d273588a8e36ce9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1036035 Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Timothy Asir: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21950 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5577938b32f26507e6044b1f4d273588a8e36ce9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: db: fix dal test config file
by emesika＠redhat.com 08 Dec '13

08 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: db: fix dal test config file ...................................................................... db: fix dal test config file Match the conf file to the common settings. (http://www.ovirt.org/OVirt_Engine_Development_Environment#Enable_DAO_Tests) Change-Id: I5ffc320576dbce8709b7e90afa1d08445011cc75 Signed-off-by: Kobi Ianko <kobi(a)redhat.com> --- M backend/manager/modules/dal/src/test/filters/pgsql.properties 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22172 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ffc320576dbce8709b7e90afa1d08445011cc75 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Fix typo
by iheim＠redhat.com 08 Dec '13

08 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Fix typo ...................................................................... engine: Fix typo Thiss patch fixes a minor spelling issue in MoM policy synchronization; Cluster--> Host --> "Synch Mom Policy" should be "Sync MoM Policy". Change-Id: I325c14e8b5c3fc97d6293d80a7f05d68fe89a1b9 Bug-Url: https://bugzilla.redhat.com/1039275 Signed-off-by: Doron Fediuck <dfediuck(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I325c14e8b5c3fc97d6293d80a7f05d68fe89a1b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: Adding Speed optimization option to the scheduler
by iheim＠redhat.com 08 Dec '13

08 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Adding Speed optimization option to the scheduler ...................................................................... engine: Adding Speed optimization option to the scheduler Add to cluster entity optimization_type field. this field is a binary representation of cluster optimization possibilities. The first option is optimize for speed, allows to skip host weighing in case there are more than configurable amount of pending scheduling requests. Bug-Url: https://bugzilla.redhat.com/1014696 Change-Id: I0784f89648093e650be73fcc3d850a4854d53c1a Reviewed-on: http://gerrit.ovirt.org/#/c/19270/ Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.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/queries/ConfigurationValues.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/scheduling/OptimizationType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml A packaging/dbscripts/upgrade/03_03_1030_add_vds_group_optimization_field.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/dbscripts/vds_groups_sp.sql M packaging/etc/engine-config/engine-config.properties 17 files changed, 218 insertions(+), 15 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22173 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0784f89648093e650be73fcc3d850a4854d53c1a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: agent: configurable host score factors
by dfediuck＠redhat.com 08 Dec '13

08 Dec '13

Doron Fediuck has submitted this change and it was merged. Change subject: agent: configurable host score factors ...................................................................... agent: configurable host score factors Allow configuration of host score factors in (usually) /etc/ovirt-hosted-engine-ha/agent.conf Change-Id: If485283c133eb517c9a3036ff42d1cd962a1697d Bug-Url: https://bugzilla.redhat.com/1037654 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt-hosted-engine-ha.spec.in M ovirt_hosted_engine_ha/agent/Makefile.am A ovirt_hosted_engine_ha/agent/agent.conf M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/hosted_engine.py 5 files changed, 69 insertions(+), 17 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, but someone else must approve Greg Padgett: Verified Doron Fediuck: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If485283c133eb517c9a3036ff42d1cd962a1697d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: scheduling: allow placement overbooking in cluster l...
by gchaplik＠redhat.com 08 Dec '13

08 Dec '13

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: scheduling: allow placement overbooking in cluster level ...................................................................... engine: scheduling: allow placement overbooking in cluster level Adding a cluster optimization to enable parallel VM scheduling requests for cluster (skip lock), in case pending requests are greater than configurable threshold. By default this feature is hidden from the user (unless setting config.SchedulerAllowOverBooking to true). Bug-Url: https://bugzilla.redhat.com/1014697 Change-Id: If52abf9974ab71b1f8d52ca268dac8dc39b89dc8 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/exclude-filters.xml M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.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/queries/ConfigurationValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/scheduling/OptimizationType.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 14 files changed, 187 insertions(+), 7 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19272 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If52abf9974ab71b1f8d52ca268dac8dc39b89dc8 Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Adding Speed optimization option to the scheduler
by gchaplik＠redhat.com 08 Dec '13

08 Dec '13

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: Adding Speed optimization option to the scheduler ...................................................................... engine: Adding Speed optimization option to the scheduler Add to cluster entity optimization_type field. this field is a binary representation of cluster optimization possibilities. The first option is optimize for speed, allows to skip host weighing in case there are more than configurable amount of pending scheduling requests. Bug-Url: https://bugzilla.redhat.com/1014696 Change-Id: I0784f89648093e650be73fcc3d850a4854d53c1a Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.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/queries/ConfigurationValues.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/scheduling/OptimizationType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml A packaging/dbscripts/upgrade/03_04_0180_add_vds_group_optimization_field.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/dbscripts/vds_groups_sp.sql M packaging/etc/engine-config/engine-config.properties 17 files changed, 229 insertions(+), 23 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19270 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0784f89648093e650be73fcc3d850a4854d53c1a Gerrit-PatchSet: 20 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0