June 2014 - Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: add exception for ENETUNREACH
by dougsland＠redhat.com 06 Jun '14

06 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: add exception for ENETUNREACH ...................................................................... engine_page: add exception for ENETUNREACH Due the change to use vdscli (xmlrpc) to communicate with VDSM to verify the networks available, it can cause an exception of Network is unreachable if there is no (ovirtmgmt) interface to communicate with vdsm. This patch handle the exception. Change-Id: I77db22b372820968a5af86d65fa1c40574a7ebd1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1104009 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28427 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77db22b372820968a5af86d65fa1c40574a7ebd1 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: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: add exception for ENETUNREACH
by dougsland＠redhat.com 05 Jun '14

05 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: add exception for ENETUNREACH ...................................................................... engine_page: add exception for ENETUNREACH Due the change to use vdscli (xmlrpc) to communicate with VDSM to verify the networks available, it can cause an exception of Network is unreachable if there is no (ovirtmgmt) interface to communicate with vdsm. This patch handle the exception. Change-Id: I77db22b372820968a5af86d65fa1c40574a7ebd1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1104009 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77db22b372820968a5af86d65fa1c40574a7ebd1 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: hooks: Adding hooks for ovirt-node-upgrade
by dougsland＠redhat.com 05 Jun '14

05 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: hooks: Adding hooks for ovirt-node-upgrade ...................................................................... hooks: Adding hooks for ovirt-node-upgrade Currently ovirt-node offers ovirt-node-upgrade tool to upgrade node images. This patch adds the pre-upgrade script for vdsm tasks before the upgrade. Change-Id: I986fe6edb8f5619dcbca33cccab080245654fd02 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1079087 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M Makefile.am M configure.ac A hooks/Makefile.am A hooks/README A hooks/pre-upgrade/01-vdsm M ovirt-node-plugin-vdsm.spec.in 6 files changed, 65 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28424 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I986fe6edb8f5619dcbca33cccab080245654fd02 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[master]: hooks: Adding hooks for ovirt-node-upgrade
by dougsland＠redhat.com 05 Jun '14

05 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: hooks: Adding hooks for ovirt-node-upgrade ...................................................................... hooks: Adding hooks for ovirt-node-upgrade Currently ovirt-node offers ovirt-node-upgrade tool to upgrade node images. This patch adds the pre-upgrade script for vdsm tasks before the upgrade. Change-Id: I986fe6edb8f5619dcbca33cccab080245654fd02 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1079087 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M Makefile.am M configure.ac A hooks/Makefile.am A hooks/README A hooks/pre-upgrade/01-vdsm M ovirt-node-plugin-vdsm.spec.in 6 files changed, 65 insertions(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Joey Boggs: Verified -- To view, visit http://gerrit.ovirt.org/28251 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I986fe6edb8f5619dcbca33cccab080245654fd02 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: engine_page: remove dep. from ovirt_config_setup.engine
by dougsland＠redhat.com 05 Jun '14

05 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: remove dep. from ovirt_config_setup.engine ...................................................................... engine_page: remove dep. from ovirt_config_setup.engine ovirt_config_setup.engine is not available anymore. This patch removes dependency. Change-Id: Ia04f6fb1d3822828f543cfa4472d9e224162b5ff Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1102995 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/config.py.in M src/engine_page.py 2 files changed, 88 insertions(+), 19 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28423 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia04f6fb1d3822828f543cfa4472d9e224162b5ff 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: 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]: engine_page: remove dep. from ovirt_config_setup.engine
by dougsland＠redhat.com 05 Jun '14

05 Jun '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: remove dep. from ovirt_config_setup.engine ...................................................................... engine_page: remove dep. from ovirt_config_setup.engine ovirt_config_setup.engine is not available anymore. This patch removes dependency. Change-Id: Ia04f6fb1d3822828f543cfa4472d9e224162b5ff Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1102995 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/config.py.in M src/engine_page.py 2 files changed, 88 insertions(+), 19 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Ryan Barry: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/28232 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia04f6fb1d3822828f543cfa4472d9e224162b5ff Gerrit-PatchSet: 4 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: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: context-sensitive help popups use scrollbars
by vszocs＠redhat.com 05 Jun '14

05 Jun '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin: context-sensitive help popups use scrollbars ...................................................................... webadmin: context-sensitive help popups use scrollbars Changed all context-sensitive help popup windows to force use of scrollbars. Firefox and Chrome were using scrollbars automatically, but Internet Explorer was not. Now all three use scrollbars. Change-Id: If188b744d3535b0dfc747578d8615e92094891b0 Bug-Url: https://bugzilla.redhat.com/1085943 Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/AbstractModelBoundPopupPresenterWidget.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Greg Sheremeta: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28247 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If188b744d3535b0dfc747578d8615e92094891b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(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]: webadmin, userportal: Add workaround-related info in UI code
by vszocs＠redhat.com 05 Jun '14

05 Jun '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin,userportal: Add workaround-related info in UI code ...................................................................... webadmin,userportal: Add workaround-related info in UI code This patch adds information on notable workarounds related to 3rd party code (GWT, GWT-Platform) by means of Java comments in following format: // TODO-GWT: description goes here When upgrading versions of 3rd party dependencies, comments on notable workarounds will make it easier to review existing UI code during the version upgrade. Change-Id: I336da10b2931825a7b44c24b663e18593f489979 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/presenter/DynamicTabContainerPresenter.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/MenuBar.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uicommon/model/SystemTreeModelProvider.java 4 files changed, 8 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Alexander Wels: Looks good to me, but someone else must approve Vojtech Szocs: Verified -- To view, visit http://gerrit.ovirt.org/24995 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I336da10b2931825a7b44c24b663e18593f489979 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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]: webadmin : exposing the profile entities to frontend
by kmayilsa＠redhat.com 05 Jun '14

05 Jun '14

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : exposing the profile entities to frontend ...................................................................... webadmin : exposing the profile entities to frontend Change-Id: Ifee3944d2ce60c4a65bdd915de7939e470e55a97 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28136 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifee3944d2ce60c4a65bdd915de7939e470e55a97 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: added IDs to locale boxes, welcome pag...
by vszocs＠redhat.com 05 Jun '14

05 Jun '14

Vojtech Szocs has submitted this change and it was merged. Change subject: userportal, webadmin: added IDs to locale boxes, welcome page links ...................................................................... userportal, webadmin: added IDs to locale boxes, welcome page links Added element IDs to some elements to help with Selenium tests: * locale drop-down in welcome page * locale drop-down in UserPortal & WebAdmin login screen * all links in welcome page template Change-Id: Ic456286d03a906d3cc8c224050504a0f4e4b2d6c Signed-off-by: Greg Sheremeta <gshereme(a)redhat.com> --- M backend/manager/modules/welcome/src/main/webapp/WEB-INF/ovirt-engine.jsp M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/AbstractLoginFormView.java M packaging/branding/ovirt.brand/welcome_page.template 3 files changed, 6 insertions(+), 5 deletions(-) Approvals: Greg Sheremeta: Verified Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28333 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic456286d03a906d3cc8c224050504a0f4e4b2d6c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gshereme(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