2021-05-03 18:13:30 |
Lucas Albuquerque Medeiros de Moura |
bug |
|
|
added bug |
2021-05-04 13:34:14 |
Lucas Albuquerque Medeiros de Moura |
description |
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system unauthenticated source file for both services. Therefore, we would display a message like that in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machine will not gain much from esm-apps. Therefore, on ESM machines we should alert about the existence of esm-infra |
[Impact]
When users are running an ESM distro and looking at motd message they may be trigger by an alert generated by update-notifier that both esm-infra and esm-apps could be enable in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that
users will be receiving alerts for esm-apps that are not necessary
[Test case]
To reproduce the issue, you can:
1. Launch a xenial container
2. create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. install ubuntu-advantage-client from this ppa:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear in the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing here the condition to output esm-apps alerts. If that condition is not right we may miss to display those messages to users that
need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source file for both services. Therefore, we would display a message like that in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machine will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
|
2021-05-04 13:36:00 |
Lucas Albuquerque Medeiros de Moura |
summary |
update-notifier should not alert users about esm-infra in a ESM machine |
update-notifier should not alert users about esm-apps in a ESM machine |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
nominated for series |
|
Ubuntu Bionic |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
bug task added |
|
update-notifier (Ubuntu Bionic) |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
nominated for series |
|
Ubuntu Focal |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
bug task added |
|
update-notifier (Ubuntu Focal) |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
nominated for series |
|
Ubuntu Xenial |
|
2021-05-14 18:36:58 |
Lucas Albuquerque Medeiros de Moura |
bug task added |
|
update-notifier (Ubuntu Xenial) |
|
2021-05-14 18:37:08 |
Lucas Albuquerque Medeiros de Moura |
update-notifier (Ubuntu): assignee |
|
Lucas Albuquerque Medeiros de Moura (lamoura) |
|
2021-05-20 18:12:02 |
Launchpad Janitor |
update-notifier (Ubuntu): status |
New |
Fix Released |
|
2021-05-21 03:33:59 |
Bryce Harrington |
description |
[Impact]
When users are running an ESM distro and looking at motd message they may be trigger by an alert generated by update-notifier that both esm-infra and esm-apps could be enable in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that
users will be receiving alerts for esm-apps that are not necessary
[Test case]
To reproduce the issue, you can:
1. Launch a xenial container
2. create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. install ubuntu-advantage-client from this ppa:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear in the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing here the condition to output esm-apps alerts. If that condition is not right we may miss to display those messages to users that
need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source file for both services. Therefore, we would display a message like that in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machine will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
[Impact]
When users are running an ESM distro and looking at the motd message they may be triggered by an alert generated by update-notifier that both esm-infra and esm-apps could be enabled in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that users will be receiving alerts for esm-apps that are not necessary.
[Test case]
To reproduce the issue, you can:
1. Launch a xenial container
2. create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. install ubuntu-advantage-client from this ppa:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear in the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing here the condition to output esm-apps alerts. If that condition is not right we may miss to display those messages to users that
need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source file for both services. Therefore, we would display a message like that in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machine will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
|
2021-05-21 03:38:15 |
Bryce Harrington |
description |
[Impact]
When users are running an ESM distro and looking at the motd message they may be triggered by an alert generated by update-notifier that both esm-infra and esm-apps could be enabled in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that users will be receiving alerts for esm-apps that are not necessary.
[Test case]
To reproduce the issue, you can:
1. Launch a xenial container
2. create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. install ubuntu-advantage-client from this ppa:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear in the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing here the condition to output esm-apps alerts. If that condition is not right we may miss to display those messages to users that
need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source file for both services. Therefore, we would display a message like that in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machine will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
[Impact]
When users are running an ESM distro and looking at the MOTD they may be surprised by an alert generated by update-notifier that both esm-infra and esm-apps could be enabled in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that users will be receiving alerts about esm-apps unnecessarily.
[Test case]
To reproduce the issue:
1. Launch a xenial container
2. Create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. Install ubuntu-advantage-client from this PPA:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear at the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing the condition for when to output esm-apps alerts. If that condition is not right, we may miss displaying the messages to users that need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source files for both services. Therefore, we would display a message like this in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machines will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
|
2021-05-21 19:02:04 |
Lucas Albuquerque Medeiros de Moura |
description |
[Impact]
When users are running an ESM distro and looking at the MOTD they may be surprised by an alert generated by update-notifier that both esm-infra and esm-apps could be enabled in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that users will be receiving alerts about esm-apps unnecessarily.
[Test case]
To reproduce the issue:
1. Launch a xenial container
2. Create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. Install ubuntu-advantage-client from this PPA:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear at the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are only changing the condition for when to output esm-apps alerts. If that condition is not right, we may miss displaying the messages to users that need them.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source files for both services. Therefore, we would display a message like this in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machines will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
== Changelog ==
* data/apt_check.py
- do not show esm-apps alerts on esm distros |
[Impact]
When users are running an ESM distro and looking at the MOTD they may be surprised by an alert generated by update-notifier that both esm-infra and esm-apps could be enabled in the machine to receive future updates. However, esm-apps will not bring much to an ESM distro. This means that users will be receiving alerts about esm-apps unnecessarily.
[Test case]
To reproduce the issue:
1. Launch a xenial container
2. Create the file /etc/ubuntu-advantage/uaclient.conf with the following content:
# Ubuntu-Advantage client config file.
contract_url: 'https://contracts.canonical.com'
security_url: 'https://ubuntu.com/security'
data_dir: /var/lib/ubuntu-advantage
log_level: debug
log_file: /var/log/ubuntu-advantage.log
features:
allow_beta: true
3. Install ubuntu-advantage-client from this PPA:
https://code.launchpad.net/~ua-client/+archive/ubuntu/daily
PS: Make sure to not allow the installation to override the config file you have just created.
4. Update the version of update-notifier to the one in proposed
5. Run `/usr/lib/update-notifier/apt-check --human-readable`
6. Verify that both esm-infra and esm-apps alerts appear at the end of the message
To verify that the error is fixed:
1. Run the past scenario until step 3
2. Install the new update-notifier from this ppa:
https://launchpad.net/~lamoura/+archive/ubuntu/update-notifier-test-ppa/
3. Run `/usr/lib/update-notifier/apt-check --human-readable`
4. Verify that only esm-infra is now showing an alert message at the end
[Where problems could occur]
We are changing some aspects of the package that could generate problems:
1) We are changing the condition for when to output esm-apps alerts. If that condition is not right, we may miss displaying the messages to users that need them.
2) We are adding a new dependency to the package, lsb-release. But since the package was already relying on that dependency, we are just fixing a packaging mistake here.
3) Instead of relying on `lsb_release` for distro-info, we are now getting that info directly from `/etc/os-release`. Since is an attempt to not rely on subprocess python calls on that script. However, since we are adding more python code, logic errors can not happen because of that new chunk of code.
4) Since we are touching translatable message, translation errors can happen because of the modifications we are providing here.
5) We are now adding a ESM Apps headers for the case where users now have
esm-apps source files in the system. If our logic is incorrect, we could be displaying that info on situations that are not needed.
However, we believe that all of the risks mentioned above are manageable and should not block the changes we are bringing to the package.
[Discussion]
When running /usr/lib/update-notifier/apt-check on a ESM machine, we could alert the user about the existence of both esm-infra and esm-apps if the system has unauthenticated source files for both services. Therefore, we would display a message like this in the system:
--------------------------
Enable UA Apps: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
Enable UA Infra: ESM to receive additional future security updates.
See https://ubuntu.com/security/esm or run: sudo ua status
----------------------------------
This is not ideal, since ESM machines will not gain much from esm-apps.
Also, this messaging may hide the importance of using esm-infra on ESM
machines, since the users will not know beforehand which service to
prioritize.
Furthermore, we are also bringing back the message:
0 updates can be applied immediately
When the system has no packages to install. This is because users found that omission confusing, since they thought that maybe something went wrong with motd if display no messages when the system is up to date.
Additionally, we are adding an esm-apps header to the `apt-check` message if the system is a LTS one with esm-apps source files on it. The header should behave exactly like the one we have for esm-infra:
UA Apps: Extended Security Maintenance (ESM) is (enabled|disabled).
We doing this to be consistent with the message we have on ESM distros
Finally, we are no longer relying on `lsb_release` on `apt-check` to get distro info, like name and version. We are parsing `/etc/os-release` for that now. The rationale for this change is some errors we have saw regarding the use of `subprocess.check_output`. Since we can parse the os-release file directly, we believe it is safer to do so than to rely on shell out for getting that info.
== Changelog ==
* debian/control: add lsb-release to the update-notifier-common depends as it is used by apt-cdrom-check. (LP: #1927996)
* data/apt_check.py
- Only show esm-apps alerts on lts and non-esm distros (LP: #1926990)
- Update esm-infra alert for distros on ESM mode
- Show message with number of upgradable packages even if that
number is zero (LP: #1926819)
- Get distro name and version directly from /etc/os-release
- Show esm-apps status header when running on LTS distro that
has not yet entered Extended Security Maintenance |
|
2021-05-31 13:37:26 |
Lucas Albuquerque Medeiros de Moura |
update-notifier (Ubuntu Xenial): assignee |
|
Lucas Albuquerque Medeiros de Moura (lamoura) |
|
2021-05-31 13:37:31 |
Lucas Albuquerque Medeiros de Moura |
update-notifier (Ubuntu Bionic): assignee |
|
Lucas Albuquerque Medeiros de Moura (lamoura) |
|
2021-05-31 13:37:33 |
Lucas Albuquerque Medeiros de Moura |
update-notifier (Ubuntu Focal): assignee |
|
Lucas Albuquerque Medeiros de Moura (lamoura) |
|
2021-05-31 17:16:25 |
Łukasz Zemczak |
update-notifier (Ubuntu Hirsute): status |
New |
Fix Committed |
|
2021-05-31 17:16:26 |
Łukasz Zemczak |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2021-05-31 17:16:29 |
Łukasz Zemczak |
bug |
|
|
added subscriber SRU Verification |
2021-05-31 17:16:31 |
Łukasz Zemczak |
tags |
|
verification-needed verification-needed-hirsute |
|
2021-05-31 17:25:41 |
Łukasz Zemczak |
update-notifier (Ubuntu Focal): status |
New |
Fix Committed |
|
2021-05-31 17:25:45 |
Łukasz Zemczak |
tags |
verification-needed verification-needed-hirsute |
verification-needed verification-needed-focal verification-needed-hirsute |
|
2021-05-31 17:36:48 |
Łukasz Zemczak |
update-notifier (Ubuntu Bionic): status |
New |
Fix Committed |
|
2021-05-31 17:36:53 |
Łukasz Zemczak |
tags |
verification-needed verification-needed-focal verification-needed-hirsute |
verification-needed verification-needed-bionic verification-needed-focal verification-needed-hirsute |
|
2021-05-31 17:38:47 |
Łukasz Zemczak |
update-notifier (Ubuntu Xenial): status |
New |
Fix Committed |
|
2021-05-31 17:38:52 |
Łukasz Zemczak |
tags |
verification-needed verification-needed-bionic verification-needed-focal verification-needed-hirsute |
verification-needed verification-needed-bionic verification-needed-focal verification-needed-hirsute verification-needed-xenial |
|
2021-06-07 13:39:21 |
Lucas Albuquerque Medeiros de Moura |
tags |
verification-needed verification-needed-bionic verification-needed-focal verification-needed-hirsute verification-needed-xenial |
verification-done verification-done-bionic verification-done-focal verification-done-hirsute verification-done-xenial |
|
2021-06-08 07:56:07 |
Launchpad Janitor |
update-notifier (Ubuntu Hirsute): status |
Fix Committed |
Fix Released |
|
2021-06-08 07:56:21 |
Łukasz Zemczak |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|
2021-06-08 07:58:20 |
Launchpad Janitor |
update-notifier (Ubuntu Focal): status |
Fix Committed |
Fix Released |
|
2021-06-08 07:59:00 |
Launchpad Janitor |
update-notifier (Ubuntu Bionic): status |
Fix Committed |
Fix Released |
|
2021-06-08 17:19:37 |
Launchpad Janitor |
update-notifier (Ubuntu Xenial): status |
Fix Committed |
Fix Released |
|
2022-09-22 16:48:02 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lamoura/ubuntu/+source/update-notifier/+git/update-notifier/+merge/430317 |
|
2022-09-22 16:48:57 |
Launchpad Janitor |
merge proposal linked |
|
https://code.launchpad.net/~lamoura/ubuntu/+source/update-notifier/+git/update-notifier/+merge/430318 |
|