- Notifications
You must be signed in to change notification settings - Fork45
fix: win_psDscAdapter targeted module cache refreshes#787
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
3 tasks
3 tasks
Now also fixes#807 |
6c16fae
to6fd7145
CompareSteveL-MSFT requested changesMay 17, 2025
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
SteveL-MSFT requested changesMay 18, 2025
Uh oh!
There was an error while loading.Please reload this page.
SteveL-MSFT approved these changesMay 19, 2025
Merged via the queue intoPowerShell:main with commitMay 19, 2025
35dabc9
4 checks passed
Uh oh!
There was an error while loading.Please reload this page.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading.Please reload this page.
PR Summary
Fixes#745 by ensuring a full
Get-DscResource
refresh is triggered when$env:PSModulePath
changes.Also reorders the cache staleness checks to prioritize detecting
PSModulePath
differences before evaluating stale resource entries.Fixes#807 by ensuring a full cache refresh is triggered when cache is cold. It now builds a list
$namedModules
which determines whether targeted module refresh occurs, using this list if it does.PR Context
When
$env:PSModulePath
changes, the cache may be rebuilt incorrectly if$Module
remains set. The module environment may have changed outside the scope defined by$Module
, but the refresh will only consider the provided modules.When a module is deleted, the stale entries check will detect the change and initiate a cache refresh, skipping the check for stale paths. If the removed module is not in
$Module
, the stale entries get captured in$existingDscResourceCacheEntries
and remain in the refreshed cache.A change in
PSModulePath
should result in a full cache refresh and take priority over stale resource entries.