[Merge] ~nteodosio/update-manager:jammy-ua into update-manager:main

Nathan Teodosio mp+459155 at code.launchpad.net
Mon Jan 22 11:41:20 UTC 2024


Nathan Teodosio has proposed merging ~nteodosio/update-manager:jammy-ua into update-manager:main.

Requested reviews:
  Sebastien Bacher (seb128)
Related bugs:
  Bug #1990450 in update-manager (Ubuntu): "Show potential Ubuntu Pro updates and apt news messages"
  https://bugs.launchpad.net/ubuntu/+source/update-manager/+bug/1990450

For more details, see:
https://code.launchpad.net/~nteodosio/update-manager/+git/update-manager/+merge/459155

Fix the crash caused by incomplete response of the 'ua security-status' when ubuntu-advantage-tools (UAT) is an older version.

As the documentation[1] states, this call returns no exceptions, so in my opinion it is simpler to switch to it than patching our old call to security-status.

However, this crash would also occur by using the version 27 of the API, so the effective fix here is to force ubuntu-advantage-tools >= 30.

Note that we already use the API in Noble[2].

[1] https://canonical-ubuntu-pro-client.readthedocs-hosted.com/en/latest/references/api/#u-pro-packages-updates-v1
[2] https://code.launchpad.net/~nteodosio/update-manager/+git/update-manager/+merge/456088
-- 
Your team Ubuntu Core Development Team is subscribed to branch update-manager:main.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 31842 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20240122/eff1bcca/attachment-0001.diff>


More information about the Ubuntu-reviews mailing list