-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Bug: Mask fwupd
timer on ubuntu images
#12225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
We're running on VMs in Azure and the fwupd package is not needed. Leaving it enable means periodic refreshes show in network traffic and firewall logs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR ensures the fwupd
package is uninstalled from Ubuntu VM images and adds a test to verify its removal.
- Added a Pester test to assert that
fwupd
is not present on the box. - Updated the environment setup script to purge
fwupd
if installed and disable any remaining timer/config.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
images/ubuntu/scripts/tests/System.Tests.ps1 | Added a test block to verify fwupd is removed |
images/ubuntu/scripts/build/configure-environment.sh | Purges fwupd package and disables its timer/config |
fwupd
from ubuntu imagesfwupd
timer on ubuntu images
@subir0071 I think this failed CI on Ubuntu 22.04 I see:
Build: 24.04 was fine 🟢 |
* Bug: Fully remove `fwupd` from ubuntu images We're running on VMs in Azure and the fwupd package is not needed. Leaving it enable means periodic refreshes show in network traffic and firewall logs. * Update check method * Simplify test
* Bug: Fully remove `fwupd` from ubuntu images We're running on VMs in Azure and the fwupd package is not needed. Leaving it enable means periodic refreshes show in network traffic and firewall logs. * Update check method * Simplify test Co-authored-by: Lawrence Gripper <lawrencegripper@github.com>
We're running on VMs in Azure and the fwupd package is not needed.
Leaving it enable means periodic refreshes show in network traffic and firewall logs.
The current check has a bug, meaning the timer isn't masked.
In the current
fwupd
version there isn't adaemon.conf
file present so theif
condition doesn't run and doesn't mask the systemd unit.Related issue:
Check list