some more logging statements to follow user around the ui a little more

subiquity/controllers/welcome.py

@@ -45,7 +45,7 @@ class WelcomeController(BaseController):
             self.done(self.answers['lang'])
 
     def done(self, code):
-        log.debug("WelcomeController.done %s next-screen")
+        log.debug("WelcomeController.done %s next-screen", code)
         self.signal.emit_signal('l10n:language-selected', code)
         self.model.switch_language(code)
         self.signal.emit_signal('next-screen')

subiquity/ui/views/filesystem/guided.py

@@ -77,9 +77,11 @@ class GuidedFilesystemView(BaseView):
             excerpt=text))
 
     def manual(self, btn):
+        log.debug("GuidedFilesystemView.manual")
         self.controller.manual()
 
     def guided(self, btn, method):
+        log.debug("GuidedFilesystemView.guided")
         self.controller.guided(method)
 
     def cancel(self, btn=None):

subiquity/ui/views/keyboard.py

@@ -474,7 +474,8 @@ class KeyboardView(BaseView):
         self.controller.cancel()
 
     def select_layout(self, sender, layout):
-        log.debug("select_layout %s", layout)
+        if sender is not None:
+            log.debug("select_layout %s", layout)
         opts = []
         default_i = -1
         layout_items = enumerate(self.model.variants[layout].items())

subiquity/ui/views/proxy.py

@@ -28,7 +28,7 @@ from subiquitycore.ui.form import (
 )
 
 
-log = logging.getLogger('subiquity.installpath')
+log = logging.getLogger('subiquity.ui.views.proxy')
 
 proxy_help = _("If you need to use a HTTP proxy to access the outside world, "
                "enter the proxy information here. Otherwise, leave this blank."

subiquity/ui/views/snaplist.py

@@ -196,6 +196,8 @@ class SnapInfoView(WidgetWrap):
 
     def state_change(self, sender, state, selection):
         if state:
+            log.debug(
+                "selecting %s from %s", self.snap.name, selection.channel)
             self.parent.snap_boxes[self.snap.name].set_state(True)
             self.parent.to_install[self.snap.name] = selection
 
@@ -349,10 +351,12 @@ class SnapCheckBox(CheckBox):
 
     def state_change(self, sender, new_state):
         if new_state:
+            log.debug("selecting %s", self.snap.name)
             self.parent.to_install[self.snap.name] = SnapSelection(
                 channel='stable',
                 is_classic=self.snap.confinement == "classic")
         else:
+            log.debug("unselecting %s", self.snap.name)
             self.parent.to_install.pop(self.snap.name, None)
 
 

subiquitycore/tasksequence.py

@@ -216,7 +216,7 @@ class TaskSequence:
         self.run_in_bg(self.curtask._bg_run, self._call_end)
 
     def _call_end(self, fut):
-        log.exception("%s ended", self.stage)
+        log.debug("%s ended", self.stage)
         if self.canceled:
             return
         self.task_complete_or_failed_called = False

subiquitycore/ui/views/network.py

@@ -379,6 +379,7 @@ class NetworkView(BaseView):
         return rows
 
     def _create_bond(self, sender=None):
+        log.debug("_create_bond")
         self.show_stretchy_overlay(BondStretchy(self))
 
     def show_network_error(self, action, info=None):

subiquitycore/ui/views/network_configure_manual_interface.py

@@ -94,7 +94,6 @@ class NetworkConfigForm(Form):
                                 help=_("Domains, comma separated"))
 
     def clean_subnet(self, subnet):
-        log.debug("clean_subnet %r", subnet)
         if '/' not in subnet:
             if self.ip_version == 4:
                 example = "xx.xx.xx.xx/yy"
@@ -241,12 +240,16 @@ class EditNetworkStretchy(Stretchy):
                 'nameservers': list(map(str, form.nameservers.value)),
                 'searchdomains': form.searchdomains.value,
             }
+            log.debug(
+                "EditNetworkStretchy %s manual result=%s",
+                self.ip_version, result)
             self.device.config.pop('nameservers', None)
             self.device.add_network(self.ip_version, result)
         elif self.method_form.method.value == "dhcp":
+            log.debug("EditNetworkStretchy %s dhcp", self.ip_version)
             self.device.config['dhcp{v}'.format(v=self.ip_version)] = True
         else:
-            pass
+            log.debug("EditNetworkStretchy %s, disabled", self.ip_version)
         self.parent.controller.apply_config()
         self.parent.update_link(self.device)
         self.parent.remove_overlay()
@@ -297,6 +300,9 @@ class AddVlanStretchy(Stretchy):
             0, 0)
 
     def done(self, sender):
+        log.debug(
+            "AddVlanStretchy.done %s %s",
+            self.device.name, self.form.vlan.value)
         self.parent.remove_overlay()
         dev = self.parent.controller.add_vlan(
             self.device, self.form.vlan.value)
@@ -309,7 +315,6 @@ class AddVlanStretchy(Stretchy):
 
 class ViewInterfaceInfo(Stretchy):
     def __init__(self, parent, device):
-        log.debug('ViewInterfaceInfo: {}'.format(device))
         self.parent = parent
         if device.info is not None:
             result = yaml.dump(
@@ -463,6 +468,7 @@ class BondStretchy(Stretchy):
             0, 0)
 
     def done(self, sender):
+        log.debug("BondStretchy.done result=%s", self.form.as_data())
         touched_devices = set()
         get_netdev_by_name = self.parent.model.get_netdev_by_name
         if self.existing: