December 2013 - Engine-commits - oVirt List Archives

Change in jenkins[master]: Add new option to the answer file
by eedri＠redhat.com 15 Dec '13

15 Dec '13

Eyal Edri has submitted this change and it was merged. Change subject: Add new option to the answer file ...................................................................... Add new option to the answer file Affected jobs: upgrade_engine_params Added option OVESETUP_CONFIG/updateFirewall introduced on master d6fa02b. Change-Id: I711fc94e445526cd777c3d2c8c79d36d900164ce Signed-off-by: Kiril Nesenko <knesenko(a)redhat.com> Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/ovirt_engine_upgrade_params/answer.file.otopi 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me, approved Kiril Nesenko: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/22372 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I711fc94e445526cd777c3d2c8c79d36d900164ce Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-reports[master]: ovirt-reports: Dashboard results an error, when reports is n...
by ydary＠redhat.com 15 Dec '13

15 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: ovirt-reports: Dashboard results an error, when reports is not deployed ...................................................................... ovirt-reports: Dashboard results an error, when reports is not deployed Reports tab shows error page when accessed after a server restart before the reports web app has been deployed A new Status servlet has been added to Reports app which is used by the webadmin to check the status of the report webapp before showing the reports web app in reports tab Change-Id: I13d0345962226d383f1d795174b6a98089f35e7b Bug-Url: https://bugzilla.redhat.com/1037667 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- A reports/jars/ReportsStatus/pom.xml A reports/jars/ReportsStatus/src/main/java/org/ovirt/status/StatusServlet.java M reports/jars/pom.xml M server-customizations/WEB-INF/web.xml 4 files changed, 119 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, but someone else must approve Ravi Nori: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22280 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I13d0345962226d383f1d795174b6a98089f35e7b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packging: Pass full log path to database schema creation script
by ydary＠redhat.com 15 Dec '13

15 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packging: Pass full log path to database schema creation script ...................................................................... packging: Pass full log path to database schema creation script Don't pass a relative path to database schema creation script, as it will create the log file under current directory. Change-Id: Iceb095d05473f41da4fa56eadc6e60da14056ac2 Bug-Url: https://bugzilla.redhat.com/1040026 Signed-off-by: Ofer Schreiber <oschreib(a)redhat.com> --- M packaging/ovirt-engine-dwh-setup.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22339 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iceb095d05473f41da4fa56eadc6e60da14056ac2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: setup: handle answerfile correctly
by ydary＠redhat.com 15 Dec '13

15 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: handle answerfile correctly ...................................................................... packaging: setup: handle answerfile correctly Previously if CREATE_READONLY_USER was specified in answer file, the readonly user was not created. Now if CREATE_READONLY_USER is set to yes or true it will be created using validated values from: - READONLY_SECURE - READONLY_USER - READONLY_PASS Change-Id: I1598f4bfaf82b1713f6b0e4ee5b61d1053542702 Bug-Url: https://bugzilla.redhat.com/1040029 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/ovirt-engine-dwh-setup.py 1 file changed, 29 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22369 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1598f4bfaf82b1713f6b0e4ee5b61d1053542702 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[ovirt-3.3]: build: ovirt-engine-dwh-3.3.2
by ydary＠redhat.com 15 Dec '13

15 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-dwh-3.3.2 ...................................................................... build: ovirt-engine-dwh-3.3.2 Fix error: line 31: Illegal sequence ".." in: Release: 0.2..20131213083001.fc19 Change-Id: I0571c8c61b7ee5866b6f72170d12033180325949 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M version.mak 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22365 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0571c8c61b7ee5866b6f72170d12033180325949 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-reports[ovirt-3.3]: build: ovirt-engine-reports-3.3.2
by ydary＠redhat.com 15 Dec '13

15 Dec '13

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-reports-3.3.2 ...................................................................... build: ovirt-engine-reports-3.3.2 Fix Illegal sequence ".." in: Release: 0.2..20131213083001.fc19 Change-Id: Ib3ab4051429c1d13e25b7d15f205364abf411449 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M version.mak 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22366 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib3ab4051429c1d13e25b7d15f205364abf411449 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: SearchBox doesnt display Templates in search for Users
by emesika＠redhat.com 15 Dec '13

15 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: engine: SearchBox doesnt display Templates in search for Users ...................................................................... engine: SearchBox doesnt display Templates in search for Users Searching for Users who have permissions on a template does not show any results, always returns empty list of users. Change-Id: Ibc15f4144e129a5dc36e9ee52cb0033b143a16a3 Bug-Url: https://bugzilla.redhat.com/1005836 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjectAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java 2 files changed, 5 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved Ravi Nori: Verified -- To view, visit http://gerrit.ovirt.org/22373 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc15f4144e129a5dc36e9ee52cb0033b143a16a3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: changed the loggers names to use hierachy
by gpadgett＠redhat.com 14 Dec '13

14 Dec '13

Greg Padgett has submitted this change and it was merged. Change subject: changed the loggers names to use hierachy ...................................................................... changed the loggers names to use hierachy - with that change it's possible to set the log level for the whole package, so users like vdsm can set the required log level without knowing the exact logger name Change-Id: I40fb7aca813800dd445e858406034736817cac9c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1032667 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/agent.py M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/broker/broker.py M ovirt_hosted_engine_ha/broker/listener.py M ovirt_hosted_engine_ha/broker/monitor.py M ovirt_hosted_engine_ha/broker/notifications.py M ovirt_hosted_engine_ha/broker/storage_broker.py M ovirt_hosted_engine_ha/broker/submonitor_base.py M ovirt_hosted_engine_ha/broker/submonitors/cpu_load_no_engine.py M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py M ovirt_hosted_engine_ha/broker/submonitors/mem_free.py M ovirt_hosted_engine_ha/broker/submonitors/mem_load.py M ovirt_hosted_engine_ha/broker/submonitors/mgmt_bridge.py M ovirt_hosted_engine_ha/broker/submonitors/ping.py M ovirt_hosted_engine_ha/broker/test/test_monitor.py M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/lib/brokerlink.py 17 files changed, 24 insertions(+), 23 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40fb7aca813800dd445e858406034736817cac9c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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

13 Dec '13

Itamar Heim 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 Reviewed-on: http://gerrit.ovirt.org/#/c/19272/ 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(+), 9 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22174 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If52abf9974ab71b1f8d52ca268dac8dc39b89dc8 Gerrit-PatchSet: 4 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-engine[master]: frontend: Add secure-channels to .vv file
by tjelinek＠redhat.com 13 Dec '13

13 Dec '13

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Add secure-channels to .vv file ...................................................................... frontend: Add secure-channels to .vv file This patch adds secure-channels line to SPICE .vv descriptor. Change-Id: Ie3868ed99c915e0444c94f502075bbe1e8b05339 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/SpiceNativeImpl.java 1 file changed, 13 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22346 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3868ed99c915e0444c94f502075bbe1e8b05339 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0