April 2014 - Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: add compatiblePort() to src
by dougsland＠redhat.com 05 Apr '14

05 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: add compatiblePort() to src ...................................................................... engine_page: add compatiblePort() to src compatiblePort() is defined vdsm/vdsm_reg/engine.py.in (previous version of engine_page.py) delivered in %{python_sitelib}/ovirt_config_setup/engine.py. Instead of to keep this old file in vdsm tree just for one function we should move the function to the new project ovirt-node-plugin-vdsm and remove such file. Change-Id: I8eec07e576dce8b3ce8c064897a2e47c674891f6 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 22 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26439 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8eec07e576dce8b3ce8c064897a2e47c674891f6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: Display correctly proto for Manage by
by dougsland＠redhat.com 05 Apr '14

05 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: Display correctly proto for Manage by ...................................................................... engine_page: Display correctly proto for Manage by On status page, we have the "Manage by" string to show to users which oVirt Engine the node is registered. This patch will add a validation which protocol it used during the registration (http or https) and put it in the display. Change-Id: Iecfc53bab7e5f6ab226ded7f00bd6d140b6846b1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1065401 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26438 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iecfc53bab7e5f6ab226ded7f00bd6d140b6846b1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: Remove Retrieve Certificate button
by dougsland＠redhat.com 05 Apr '14

05 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: Remove Retrieve Certificate button ...................................................................... engine_page: Remove Retrieve Certificate button ovirt-node-plugin-vdsm should popup to users the certificate when they register via https to accept it or not. This patch will add such schema and will remove the unneeded Retrieve Certificate button. Change-Id: I9f0e47242380c511e8ed672eeb0f541616696f8e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1081042 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 72 insertions(+), 33 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26436 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9f0e47242380c511e8ed672eeb0f541616696f8e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: network is required to register tab
by dougsland＠redhat.com 05 Apr '14

05 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: network is required to register tab ...................................................................... engine_page: network is required to register tab oVirt Engine tab will only show the options for register once the oVirt Node has the network setup. Change-Id: Ic2563804673954234f3c29fed80ca54fe195d35a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1064206 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 45 insertions(+), 18 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26437 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2563804673954234f3c29fed80ca54fe195d35a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: optimize system tree queries.
by awels＠redhat.com 04 Apr '14

04 Apr '14

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: optimize system tree queries. ...................................................................... webadmin: optimize system tree queries. - Previously the system tree would sequencially call methods to retrieve the data once the previous call was finished. This causes at least 7 distinct queries one after another. This patch optimizes the queries to 3 distinct queries and some in parallel. Change-Id: I304585d93de970030095919614ec13b2f46fc0ef Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeModel.java 1 file changed, 181 insertions(+), 167 deletions(-) Approvals: Alexander Wels: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25763 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I304585d93de970030095919614ec13b2f46fc0ef Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: snapshot subtab action bar
by awels＠redhat.com 04 Apr '14

04 Apr '14

Alexander Wels has submitted this change and it was merged. Change subject: userportal,webadmin: snapshot subtab action bar ...................................................................... userportal,webadmin: snapshot subtab action bar - When using the splitter to resize the snapshot section of the snapshot sub tab the action bar would not automatically create the dropdown menu when not enough room existed. This patch fixes that problem in both user portal and web admin Change-Id: I8f587170fffba3bc4d27a0144cd02f4ed2a2aa52 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1071387 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/action/AbstractActionPanel.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/action/DropdownActionButton.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmSnapshotListModelTable.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmSnapshotListModelTable.ui.xml 4 files changed, 11 insertions(+), 7 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25667 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8f587170fffba3bc4d27a0144cd02f4ed2a2aa52 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: add compatiblePort() to src
by dougsland＠redhat.com 04 Apr '14

04 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: add compatiblePort() to src ...................................................................... engine_page: add compatiblePort() to src compatiblePort() is defined vdsm/vdsm_reg/engine.py.in (previous version of engine_page.py) delivered in %{python_sitelib}/ovirt_config_setup/engine.py. Instead of to keep this old file in vdsm tree just for one function we should move the function to the new project ovirt-node-plugin-vdsm and remove such file. Change-Id: I8eec07e576dce8b3ce8c064897a2e47c674891f6 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 22 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8eec07e576dce8b3ce8c064897a2e47c674891f6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Joey Boggs <jboggs(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: Revert "spec: start/enable glusterd"
by dougsland＠redhat.com 04 Apr '14

04 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: Revert "spec: start/enable glusterd" ...................................................................... Revert "spec: start/enable glusterd" This reverts commit 1701ecc3b9d9573da41f66a7e5a493a1317d03db. Should be done via cluster level. Change-Id: I32fb3247213cc99484a442e4df91910bd196695e Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 1 insertion(+), 37 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26416 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32fb3247213cc99484a442e4df91910bd196695e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: allow automation for cluster
by sbonazzo＠redhat.com 04 Apr '14

04 Apr '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: allow automation for cluster ...................................................................... packaging: setup: allow automation for cluster Allow automation for cluster selection. Introduce a second answer file saving as last step of closeup stage, storing the cluster name selected for reusing it on additional hosts. Change-Id: I699dda9d1286b8d6f2e251a5f0ffcaa43725da21 Related-To: http://bugzilla.redhat.com/1059950 Related-To: http://bugzilla.redhat.com/1043906 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/core/answerfile.py M src/plugins/ovirt-hosted-engine-setup/engine/add_host.py 3 files changed, 81 insertions(+), 32 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26355 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I699dda9d1286b8d6f2e251a5f0ffcaa43725da21 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Meital Bourvine <mbourvin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: added numpy as a deps
by sbonazzo＠redhat.com 04 Apr '14

04 Apr '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: added numpy as a deps ...................................................................... packaging: added numpy as a deps This patch adds python library numpy as a dependency of ovirt-websocket-proxy package. The library is an optional dependency of python-websockify and can lead to improved performance of console clients using websocket proxy (noVNC, SPICE-HTML5). Change-Id: I496416c660bef09f628fe8344774e8b555a06e34 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1045042 --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved Frank Kobzik: Verified -- To view, visit http://gerrit.ovirt.org/26371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I496416c660bef09f628fe8344774e8b555a06e34 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0