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

loader: invalidate the import cachefor extra modules #221

Merged
merged 1 commit into from
Mar 13, 2020

Conversation

aplanas
Copy link
Contributor

@aplanas aplanas commented Mar 12, 2020

Because we are mangling with importlib, we can find from time to
time an invalidation issue with sys.path_importer_cache, that
requires the removal of FileFinder that remain None for the
extra_module_dirs

(cherry picked from commit 0fb8e707a45d5caf40759e8b4943590d6fce5046)

Because we are mangling with importlib, we can find from time to
time an invalidation issue with sys.path_importer_cache, that
requires the removal of FileFinder that remain None for the
extra_module_dirs

(cherry picked from commit 0fb8e707a45d5caf40759e8b4943590d6fce5046)
@brejoc
Copy link
Member

brejoc commented Mar 13, 2020

@aplanas Is this already merged at upstream? Is there a pull request?

Update: Ah, never mind. Of course it's merged and I found the PR: saltstack/salt#53167

@aplanas
Copy link
Contributor Author

aplanas commented Mar 13, 2020

Is a new commit for this one: saltstack/salt#53167

The rest of the batch is already in our package

@brejoc brejoc merged commit fa1bc07 into openSUSE:openSUSE-3000 Mar 13, 2020
@brejoc
Copy link
Member

brejoc commented Mar 13, 2020

@aplanas Do we also need this for 2019.2.3?

@aplanas
Copy link
Contributor Author

aplanas commented Mar 13, 2020

@aplanas Do we also need this for 2019.2.3?

You are right. Yes.

@aplanas aplanas deleted the fix_loader branch March 13, 2020 15:17
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.

2 participants