-
Notifications
You must be signed in to change notification settings - Fork 11
APT: Patch Mode set to disabled in ConfigurePatching when os patch config file does not exist #216
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
…nfig file does not exist
Codecov Report
@@ Coverage Diff @@
## master #216 +/- ##
==========================================
+ Coverage 90.19% 90.21% +0.01%
==========================================
Files 90 90
Lines 14483 14539 +56
==========================================
+ Hits 13063 13116 +53
- Misses 1420 1423 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
Please fill in description to describe current behavior and desired behavior. |
kjohn-msft
left a comment
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.
LGTM, minor questions inline
done |
Issue:
'/etc/apt/apt.conf.d/20auto-upgrades' not existing on a VM has been observed to be an indication that default auto updates are not configured. The config file is only created the very first time auto updates are configured on the machine. Hence, non existence of the file can be safely considered as default auto updates are disabled. This also aligns with the implementation patterns used in YUM and Zypper package managers where a config not existing is considered as default updates disabled.
Desired behavior from this PR:
If '/etc/apt/apt.conf.d/20auto-upgrades' is not found: