Change in vdsm[master]: engine.py: return False on cert error screen

dougsland at redhat.com dougsland at redhat.com
Wed Feb 20 19:38:31 UTC 2013


Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: engine.py: return False on cert error screen
......................................................................

engine.py: return False on cert error screen

If we cannot download the engine certificate we should
only show the error screen and return.

Change-Id: I3e3f03365e77244c5521a11d92041371088b4362
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=895912
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M vdsm_reg/engine.py.in
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/44/12244/1

diff --git a/vdsm_reg/engine.py.in b/vdsm_reg/engine.py.in
index b506dd5..c4d46ae 100644
--- a/vdsm_reg/engine.py.in
+++ b/vdsm_reg/engine.py.in
@@ -335,6 +335,7 @@
                         "Failed downloading @ENGINENAME@ certificate",
                         buttons=['Ok'])
                     self.ncs.reset_screen_colors()
+                    return False
             # Stopping vdsm-reg may fail but its ok - its in the case when the
             # menus are run after installation
             deployUtil._logExec([constants.EXT_SERVICE, 'vdsm-reg', 'stop'])


--
To view, visit http://gerrit.ovirt.org/12244
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3e3f03365e77244c5521a11d92041371088b4362
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list