Skip to content

Conversation

@rane-rajasi
Copy link
Contributor

@rane-rajasi rane-rajasi commented Jul 13, 2023

Changes include:

  • adding /etc/apt/sources.list.d/ as well to the security sources list
  • Removed error raised on this command failure (NOTE: This comes from the ask of not blocking the operation on this error but issuing a warning to the customer)

This should take care of the change attempted in: #176

@rane-rajasi rane-rajasi requested review from a team, feng-j678, kjohn-msft and najams and removed request for a team July 13, 2023 16:24
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #208 (e0549e5) into master (f3b7c7b) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
- Coverage   90.12%   90.11%   -0.01%     
==========================================
  Files          90       90              
  Lines       14445    14442       -3     
==========================================
- Hits        13018    13015       -3     
  Misses       1427     1427              
Flag Coverage Δ
python27 90.11% <100.00%> (-0.01%) ⬇️
python39 90.11% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ore/src/package_managers/AptitudePackageManager.py 89.60% <100.00%> (-0.07%) ⬇️
src/core/tests/library/LegacyEnvLayerExtensions.py 94.71% <100.00%> (ø)

@kjohn-msft kjohn-msft self-assigned this Jul 13, 2023
@kjohn-msft kjohn-msft merged commit 05fb0aa into master Jul 17, 2023
@kjohn-msft kjohn-msft deleted the rarane-aptsourceslistchanges branch July 17, 2023 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants