Software Updater

Merge lp://qastaging/~renanrodrigo/update-manager/ss into lp://qastaging/update-manager

  1. ss
  2. Merge into main
Proposed by Renan Rodrigo
Status: Merged
Merged at revision: 2951
Proposed branch: lp://qastaging/~renanrodrigo/update-manager/ss
Merge into: lp://qastaging/update-manager
Diff against target: 86 lines (+13/-36)
1 file modified
ubuntu-security-status (+13/-36)
To merge this branch: bzr merge lp://qastaging/~renanrodrigo/update-manager/ss
Reviewer Review Type Date Requested Status
Brian Murray Approve
Lucas Albuquerque Medeiros de Moura (community) Approve
Review via email: mp+430698@code.qastaging.launchpad.net

Commit message

ubuntu-security-status: Don't show ESM Apps information when the service is disabled, and prefer 'pro security-status' when it is available.

Description of the change

Due to changes in the Pro launch strategy, the tooling should not advertise ESM-Apps even when it goes out of beta. If ubuntu-advantage-tools has made 'pro' available to the system, delegate the 'ubuntu-security-status' call to 'pro security-status' to make sure it reflects the Pro Client output.

To post a comment you must log in.
Revision history for this message
Lucas Albuquerque Medeiros de Moura (lamoura) :
lp://qastaging/~renanrodrigo/update-manager/ss updated
2951. By Renan Rodrigo

Prefer redirecting ubuntu-security-status to pro security-status if it exists

Revision history for this message
Renan Rodrigo (renanrodrigo) wrote :

Thanks Lucas for the review, I updated the code to the safer version.

Revision history for this message
Lucas Albuquerque Medeiros de Moura (lamoura) :
review: Approve
Revision history for this message
Brian Murray (brian-murray) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: