This repository has been archived by the owner. It is now read-only.
Permalink
Browse files

Avoid setting footer to none (#931)

sometimes the footer info_message was being set after clear_status was
called.
if we just clear it instead of setting it to None then this will still work.

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>
  • Loading branch information...
1 parent 38a702c commit 1419917de3801340712fe52ca817c24c38685e63 @mikemccracken mikemccracken committed with battlemidget May 18, 2016
Showing with 2 additions and 2 deletions.
  1. +2 −2 cloudinstall/gui.py
View
@@ -188,8 +188,7 @@ def set_openstack_rel(self, release):
self.frame.header.set_openstack_rel(release)
def clear_status(self):
- self.frame.footer = None
- self.frame.set_footer(self.frame.footer)
+ self.frame.footer.clear()
def render_services_view(self, nodes, juju_state, maas_state, config):
self.services_view = ServicesView(nodes, juju_state, maas_state,
@@ -265,6 +264,7 @@ def show_exception_message(self, ex):
else:
msg = ("A fatal error has occurred: {}\n".format(ex.args[0]))
log.error(msg)
+ log.exception(ex)
self.frame.body = ErrorView(ex.args[0])
AlarmMonitor.remove_all()

0 comments on commit 1419917

Please sign in to comment.