Skip to content

Conversation

@kjohn-msft
Copy link
Collaborator

  • Fixes a bug in extension handler where healthstoreId wasn't being passed to Core
  • Service-originating CloudType support was added explicitly
  • Derivation of max patch publish date from healthstoreId added to core
  • Removed transitional dependency on maintenanceRunId and moved over fully to healthstoreId - fixed tests to align
  • Apt package manager enhanced to benefit from repo snapshots.
  • Patch installer enhanced to use a simplified execution model for strict SDP where flexibilty exists.
  • StatusHandler enhanced to show single informational messages if only that message exists with no actual errors. (LT request)

This is a partial slice of a much larger change-set that will be reviewed separately due to complexity.

@codecov
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #223 (2babf8e) into master (b50ef62) will increase coverage by 0.04%.
The diff coverage is 98.33%.

@@            Coverage Diff             @@
##           master     #223      +/-   ##
==========================================
+ Coverage   90.39%   90.43%   +0.04%     
==========================================
  Files          90       90              
  Lines       14810    14987     +177     
==========================================
+ Hits        13387    13554     +167     
- Misses       1423     1433      +10     
Flag Coverage Δ
python27 90.43% <98.33%> (+0.04%) ⬆️
python39 90.43% <98.33%> (+0.04%) ⬆️

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

Files Coverage Δ
src/core/src/bootstrap/Constants.py 98.67% <100.00%> (+0.01%) ⬆️
src/core/src/core_logic/ExecutionConfig.py 85.13% <100.00%> (+0.96%) ⬆️
src/core/src/core_logic/PatchInstaller.py 98.61% <100.00%> (+1.94%) ⬆️
...ore/src/package_managers/AptitudePackageManager.py 90.44% <100.00%> (+0.79%) ⬆️
src/core/src/service_interfaces/StatusHandler.py 95.30% <100.00%> (+0.49%) ⬆️
src/core/tests/Test_ConfigurePatchingProcessor.py 99.04% <100.00%> (+<0.01%) ⬆️
src/core/tests/Test_CoreMain.py 99.88% <100.00%> (+<0.01%) ⬆️
src/core/tests/Test_PatchInstaller.py 99.46% <100.00%> (-0.35%) ⬇️
src/core/tests/library/LegacyEnvLayerExtensions.py 94.80% <100.00%> (ø)
src/core/tests/library/RuntimeCompositor.py 100.00% <100.00%> (ø)
... and 8 more

... and 6 files with indirect coverage changes

feng-j678
feng-j678 previously approved these changes Oct 31, 2023
@kjohn-msft kjohn-msft merged commit 7d90cff into master Oct 31, 2023
@kjohn-msft kjohn-msft deleted the kjohn-minmerge branch October 31, 2023 23:53
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