Skip to content
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

Reduced set: Safe-deployment enhancements #223

Merged
merged 6 commits into from
Oct 31, 2023
Merged

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
src/core/src/core_logic/ExecutionConfig.py Show resolved Hide resolved
src/core/src/core_logic/ExecutionConfig.py Outdated Show resolved Hide resolved
@kjohn-msft kjohn-msft merged commit 7d90cff into master Oct 31, 2023
5 checks passed
@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.

3 participants