Exceptions in the state machine (D-Bus) should result in UpdateAvailableStatus with an error_reason

[errors.ubuntu.com bug bridge]

The Ubuntu Error Tracker has been receiving reports about a problem regarding system-image. This problem was most recently seen with version 1.9.1-0ubuntu1, the problem page at https://errors.ubuntu.com/problem/99f4f3e3ccd3d0349d82e550e0b8bd7af389feea contains more details.

[Barry Warsaw (barry)]

This looks like a network error coming from u-d-m. There's not much system-image can do if ubuntu-download-manager can't make a network connection.

[Sebastien Bacher (seb128)]

@Barry: can you trap the exception?

The current situation is that when it happens users get a whoopsie error dialog in the middle if their screen ... we can't consider that a normal user experience.

Either it's a real issue and we fix it, or it's a transient issue which is fine to ignore and we don't make whoopsie bother the user to report it...

[Barry Warsaw (barry)]

Service._check_for_update() should catch exceptions that occur in the state machine (such as this one) and return UpdateAvailableStatus() with an appropriate error_reason.

[Launchpad Janitor (janitor)]

This bug was fixed in the package system-image - 2.2-0ubuntu1

---------------
system-image (2.2-0ubuntu1) trusty; urgency=medium

  [ Barry Warsaw ]
  * New upstream release.
    - LP: #1284217 - When CheckForUpdate() is called a second time, while
      an auto-download is in progress, but after the first check is
      complete, we send an UpdateAvailableStatus signal with the cached
      information.
    - LP: #1287919 - Close a race condition when manually downloading and
      issuing multiple CheckForUpdate calls.
    - LP: #1278589 - Support disabling either HTTP or HTTPS services for
      custom system image servers.
    - Allow the channel.ini file to override the [service] section.
    - LP: #1287287 - Do not do atomic renames of temporary download files;
      ubuntu-download-manager now supports this by default.
    - LP: #1250817 - Exceptions in the state machine are caught and
      logged, with an appropriate error message added to
      UpdateAvailableStatus signals. These exceptions do not percolate up
      to the GLib main loop.
    - LP: #1279532 - During testing, pass the log dir argument to
      ubuntu-download-manager.
  * d/rules: Add override_dh_python3 rule to set shebang line to
    /usr/bin/python3. (LP: #1283277)
  * d/patches/lp1284217.patch: Removed; applied upstream.
 -- Ubuntu daily release <email address hidden> Mon, 17 Mar 2014 17:28:16 +0000

[Sebastien Bacher (seb128)]

Reopening, that's still an issue with 2.2, see e.g https://errors.ubuntu.com/oops/3a22e620-af85-11e3-8908-2c768aafd08c

"Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/systemimage/dbus.py", line 86, in _check_for_update
    self._update = self._api.check_for_update()
  File "/usr/lib/python3/dist-packages/systemimage/api.py", line 96, in check_for_update
    self._state.run_until('download_files')
  File "/usr/lib/python3/dist-packages/systemimage/state.py", line 175, in run_until
    step()
  File "/usr/lib/python3/dist-packages/systemimage/state.py", line 209, in _get_blacklist_1
    'image-master', 'gpg/image-master.tar.xz', 'archive-master')
  File "/usr/lib/python3/dist-packages/systemimage/keyring.py", line 107, in get_keyring
    (ascxz_src, ascxz_dst),
  File "/usr/lib/python3/dist-packages/systemimage/download.py", line 205, in get_files
    fd, path = tempfile.mkstemp(suffix='.tmp', prefix='', dir=head)
  File "/usr/lib/python3.4/tempfile.py", line 409, in mkstemp
    return _mkstemp_inner(dir, prefix, suffix, flags)
  File "/usr/lib/python3.4/tempfile.py", line 339, in _mkstemp_inner
    fd = _os.open(file, flags, 0o600)
FileNotFoundError: [Errno 2] No such file or directory: '/android/cache/recovery/btxk4vie.tmp'"

[Barry Warsaw (barry)]

Please attach a new /var/log/system-image/client.log file and provide some detail on how you reproduced this. Thanks!