-
Notifications
You must be signed in to change notification settings - Fork 11
Feature: Adding support for Azure Linux 2.0 in Tdnf Package Manager #311
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
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 pull request expands the Tdnf Package Manager support to include Azure Linux 2.0 (Common Base Linux Mariner) by updating platform checks and related constants.
- Updated installation logic in TdnfPackageManager.py to include both Azure Linux and Common Base Linux Mariner.
- Modified EnvLayer.py to correctly detect the new distribution scenario.
- Added a new constant in Constants.py for Common Base Linux Mariner.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/core/src/package_managers/TdnfPackageManager.py | Updated patch logic to include Common Base Linux Mariner in the distribution check. |
| src/core/src/bootstrap/EnvLayer.py | Revised package manager detection to account for the new distribution name. |
| src/core/src/bootstrap/Constants.py | Added a constant for Common Base Linux Mariner. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #311 +/- ##
=======================================
Coverage 93.12% 93.13%
=======================================
Files 103 103
Lines 17550 17555 +5
=======================================
+ Hits 16344 16349 +5
Misses 1206 1206
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Minor inline
This release includes: [x] Engg. hygiene: Remove TelemetryWriter related log noise [#312](#312) [x] Bugfix: Mitigate external Ubuntu Pro Client issue [#308](#308) [x] Feature: Adding support for Azure Linux 2.0 in Tdnf Package Manager [#311](#311) [x] Eng. sys: Upgrade CICD pipeline from Python 3.9 to Python 3.12 [#309](#309) [x] Coverage: Increase code coverage - TimerManager and ServiceManager [#307](#307) [x] Bugfix: Unit tests broken in Python 3.12 [#306](#306) [x] Feature: Adding Azure Linux 3.0 Base Support [#293](#293) [x] Bugfix: Retry Handler to Prevent Unbounded Retries while trying to Mitigate YUM Update Errors [#303](#303) [x] BugFix: CentOS VMs not installing patches during Auto Patching [#298](#298) [x] Bugfix: Auto-assessment - Restricting execution permissions to root user/ owner [#299](#299)
Expanding Tdnf Package Manager to support Azure Linux 2.0
Inn VM Test Logs:
[x] Azure Linux 2.0 (Common Base Linux):
- 4.core.log
- 4.status.txt
[x] Pre-Existing Azure Linux 3.0 code compatibility test
- 1.core.log
- 1.status.txt