Add a dbus API

Bug #1192585 reported by Barry Warsaw
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu system image
Fix Released
High
Barry Warsaw

Bug Description

Currently, the client is a command line tool. When the UI is implemented it will have to talk to this tool over dbus. So we need a dbus API. The client must still be short-lived though.

Tags: client
Barry Warsaw (barry)
Changed in ubuntu-system-image:
status: New → Triaged
importance: Undecided → High
importance: High → Medium
tags: added: client
Revision history for this message
Barry Warsaw (barry) wrote :

  + dbus activation service
  + check for update, but don't download anything
  + send signal if there are updates pending
  + how big is the current update
  + descriptions for all images (including translations if available)
  + do the upgrade
  + perhaps a cancel one (perhaps plumbed through to download service)
  + signals for pre-reboot success and failure (code or message)
  + apply reboot

Revision history for this message
Barry Warsaw (barry) wrote :

  + METHOD: apply reboot
  + METHOD: check for update, but don't download anything
  + METHOD: do the upgrade
  + METHOD: get details: how big and descriptions including i18n
  + METHOD: perhaps a cancel one (perhaps plumbed through to download service)
  + SIGNAL: send signal if there are updates pending
  + SIGNAL: signals for pre-reboot success and failure (code or message)
  + SIGNAL: download canceled

Barry Warsaw (barry)
Changed in ubuntu-system-image:
status: Triaged → In Progress
importance: Medium → High
assignee: nobody → Barry Warsaw (barry)
milestone: none → 0.3
Barry Warsaw (barry)
Changed in ubuntu-system-image:
milestone: 0.3 → 0.4
Barry Warsaw (barry)
Changed in ubuntu-system-image:
milestone: 0.4 → 0.5
Barry Warsaw (barry)
Changed in ubuntu-system-image:
milestone: 0.5 → 0.6
Barry Warsaw (barry)
Changed in ubuntu-system-image:
milestone: 0.6 → 0.7
Revision history for this message
Barry Warsaw (barry) wrote :

Darn, I don't remember what this one was for:

+ SIGNAL: signals for pre-reboot success and failure (code or message)

so until we need it, I'm skipping it.

Barry Warsaw (barry)
Changed in ubuntu-system-image:
status: In Progress → Fix Committed
Revision history for this message
Barry Warsaw (barry) wrote :

After further discussion and thinking, the success case is covered by the ReadyToReboot signal, which gets triggered when GetUpdate() successfully completes.

The failure case isn't really covered, although when the download fails due to a Cancel(), a Canceled() signal is issued.

So I think there is room for a UpdateFailed() signal for when the GetUpdate() fails for reasons other than being canceled. It can have an optional message string, although I'm not sure how helpful I can make that right now.

Barry Warsaw (barry)
Changed in ubuntu-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.