Ubuntu system image

Should provide an API to query the timestamp of the most recent CheckForUpdates

Bug #1280169 reported by Sebastien Bacher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu system image
Fix Released
Medium
Barry Warsaw
Ubuntu system image 2.3

Bug Description

The design on https://wiki.ubuntu.com/SoftwareUpdates specify

"If push notification of updates has not been implemented, and you visit the “Updates” screen when at least 30 minutes has passed since the last check for updates, a new check should be triggered"

To implement that we need to know when the most recent check happened though. The caching would better be done by the service because the check can be called by different clients.

Tags: client

Related branches

lp:~barry/ubuntu-system-image/systemimage2.3
Registry Administrators: Pending requested
Diff: 7348 lines (+4720/-717)
50 files modified
NEWS.rst (+36/-0)
PKG-INFO (+1/-1)
README.rst (+71/-0)
cli-manpage.rst (+33/-2)
dbus-manpage.rst (+255/-2)
debian/changelog (+52/-0)
debian/control (+2/-0)
debian/rules (+11/-25)
debian/tests/control (+6/-1)
debian/tests/dryrun (+11/-0)
debian/tests/smoketest (+4/-2)
debian/upstream/signing-key.asc (+2459/-0)
debian/watch (+1/-1)
ini-manpage.rst (+2/-2)
setup.cfg (+2/-2)
system_image.egg-info/PKG-INFO (+1/-1)
system_image.egg-info/SOURCES.txt (+2/-0)
system_image.egg-info/entry_points.txt (+1/-1)
system_image.egg-info/requires.txt (+1/-1)
systemimage/api.py (+6/-5)
systemimage/bag.py (+13/-0)
systemimage/bindings.py (+15/-10)
systemimage/config.py (+4/-3)
systemimage/dbus.py (+25/-11)
systemimage/download.py (+87/-21)
systemimage/gpg.py (+5/-5)
systemimage/helpers.py (+33/-6)
systemimage/logging.py (+18/-11)
systemimage/main.py (+104/-2)
systemimage/reboot.py (+16/-1)
systemimage/scores.py (+1/-1)
systemimage/settings.py (+11/-0)
systemimage/state.py (+16/-16)
systemimage/testing/controller.py (+10/-8)
systemimage/testing/dbus.py (+31/-62)
systemimage/testing/helpers.py (+30/-6)
systemimage/tests/data/index_25.json (+22/-0)
systemimage/tests/test_api.py (+36/-21)
systemimage/tests/test_bag.py (+19/-0)
systemimage/tests/test_channel.py (+2/-6)
systemimage/tests/test_config.py (+19/-28)
systemimage/tests/test_dbus.py (+373/-324)
systemimage/tests/test_download.py (+234/-29)
systemimage/tests/test_helpers.py (+45/-5)
systemimage/tests/test_main.py (+497/-50)
systemimage/tests/test_settings.py (+29/-2)
systemimage/tests/test_state.py (+61/-40)
systemimage/tests/test_winner.py (+3/-1)
systemimage/version.txt (+1/-1)
tox.ini (+3/-1)
Registry Administrators: Pending requested
Diff: 4058 lines (+3524/-347) (has conflicts)
35 files modified
.bzrignore (+0/-10)
PKG-INFO (+10/-0)
README.rst (+71/-0)
debian/archive-master.tar.xz.asc (+17/-0)
debian/changelog (+468/-0)
debian/compat (+1/-0)
debian/control (+62/-0)
debian/copyright (+35/-0)
debian/rules (+64/-0)
debian/source/format (+1/-0)
debian/source/include-binaries (+1/-0)
debian/system-image-common.dirs (+2/-0)
debian/system-image-common.manpages (+3/-0)
debian/system-image-common.postinst (+23/-0)
debian/system-image-common.postrm (+18/-0)
debian/tests/client.ini.in (+34/-0)
debian/tests/control (+7/-0)
debian/tests/dryrun (+11/-0)
debian/tests/prep.py (+22/-0)
debian/tests/smoketest (+13/-0)
debian/tests/unittests (+11/-0)
debian/upstream/signing-key.asc (+2459/-0)
debian/watch (+2/-0)
setup.cfg (+9/-3)
system_image.egg-info/PKG-INFO (+10/-0)
system_image.egg-info/SOURCES.txt (+141/-0)
system_image.egg-info/dependency_links.txt (+1/-0)
system_image.egg-info/entry_points.txt (+4/-0)
system_image.egg-info/requires.txt (+1/-0)
system_image.egg-info/top_level.txt (+1/-0)
systemimage/docs/Makefile (+0/-153)
systemimage/tests/data/index_25.json (+22/-0)
tools/heregpg (+0/-7)
tools/mkvenv.py (+0/-121)
tools/sd.py (+0/-53)
lp:~ps-jenkins/ubuntu-system-image/utopic-proposed
lp:ubuntu/utopic-proposed/system-image
Revision history for this message
Barry Warsaw (barry) wrote :

Note that this will require a D-Bus API change.

tags: added: client
Changed in ubuntu-system-image:
status: New → Triaged
importance: Undecided → Wishlist
Barry Warsaw (barry)
summary: - Should provide an API to query the timpestamp of the most recent
+ Should provide an API to query the timestamp of the most recent
CheckForUpdates
Barry Warsaw (barry)
Changed in ubuntu-system-image:
importance: Wishlist → Medium
Barry Warsaw (barry)
Changed in ubuntu-system-image:
assignee: nobody → Barry Warsaw (barry)
milestone: none → 2.3
status: Triaged → In Progress
Revision history for this message
Barry Warsaw (barry) wrote :

Here's what I'm planning to implement. For backward compatibility, I will add a new method to the D-Bus API. There will be no changes to existing methods. The new method will be called Information() and will return a dictionary mapping strings to strings. From https://wiki.ubuntu.com/ImageBasedUpgrades/Client

Information()
Like Info(), but returns extended information that includes the last time a CheckForUpdate() was performed. Returns a dictionary of strings-to-strings, where the following keys are currently defined:

current_build_number - The current build number, as a string.
device_name - The device name.
channel_name - The device's current channel.
last_update_date - ISO 8601 UTC format string, with second resolution, e.g. "YYYY-MM-DDTHH:MM:SS"
version_details - comma-separated key-value pairs providing additional version details, e.g. ubuntu=123,mako=456,custom=789
last_check_date - ISO 8601 UTC format string, with second resolution, e.g. "YYYY-MM-DDTHH:MM:SS"

Note that `last_check_date` is the new bit of information that closes this bug. Also note that the version_details value is different than in the Info() method. The client calling Information() must split the returned value itself.

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

Technically, `last_check_date` will be set when an UpdateAvailableStatus signal is sent, but they are logically equivalent.

Changed in ubuntu-system-image:
status: In Progress → Fix Committed
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.
Subscribing...

Other bug subscribers

Remote bug watches

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