Merge ~chad.smith/ubuntu/+source/update-notifier:update-notifier-esm-support-hirsute into ubuntu/+source/update-notifier:ubuntu/hirsute-proposed
Status: | Approved |
---|---|
Approved by: | Bryce Harrington |
Approved revision: | 73a133fef94e01ab004c04d1f284a930b1163177 |
Proposed branch: | ~chad.smith/ubuntu/+source/update-notifier:update-notifier-esm-support-hirsute |
Merge into: | ubuntu/+source/update-notifier:ubuntu/hirsute-proposed |
Diff against target: |
885 lines (+521/-143) 4 files modified
data/apt_check.py (+186/-83) debian/changelog (+17/-0) debian/control (+2/-0) tests/test_motd.py (+316/-60) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Bryce Harrington (community) | Approve | ||
Lucas Albuquerque Medeiros de Moura | Pending | ||
Brian Murray | Pending | ||
Review via email:
|
Description of the change
Currently, the apt-check script is configured to only handle package count for ESM Infra. We are now updating the logic to also handle ESM Apps packages as well.
Furthermore, we are also updating the messaging that is created in apt-check. We are advertising ESM Apps if the service is disabled and only messaging about ESM Infra if the distro is already on ESM mode.
Finally, this PR also fixes LP #1883315, since we now check to see if the system has esm-infra or esm-apps before performing a package count for package with esm origins
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
I'd reviewed the focal/bionic/xenial versions of this branch previously. Lucas had added some code docs for some of the new functions that should be included here as well, so they're preserved in the devel tree going forward.
I'd also appreciate a summary of any other known differences between this MP and the focal/bionic/xenial one. Offhand this looks the same as that but I didn't do a strict comparison.