Skip to content

[18.0][IMP] fs_storage: Makes the cache policy of fsspec client configurable#487

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
acsone:18.0-fs-folder-no-orm-cache
Oct 20, 2025
Merged

[18.0][IMP] fs_storage: Makes the cache policy of fsspec client configurable#487
OCA-git-bot merged 1 commit intoOCA:18.0from
acsone:18.0-fs-folder-no-orm-cache

Conversation

@lmignon
Copy link
Contributor

@lmignon lmignon commented Jun 18, 2025

On the fs.storage model you can now specify if the fsspec client must be cached into the ormcache once instanciated or not. By default, the filesystem is cacheable but in some cases, like when using OAuth2 authentication, the filesystem cannot be cached because it depends on the user and the token can change.

On the fs.storage model you can now specify if the fsspec client must be cached into the ormcache once instanciated or not.
By default, the filesystem is cacheable but in some cases, like when using OAuth2 authentication, the filesystem cannot be cached because it depends on the user and the token can change.
@lmignon lmignon force-pushed the 18.0-fs-folder-no-orm-cache branch from b42dff9 to cfa306a Compare September 30, 2025 14:41
@lmignon
Copy link
Contributor Author

lmignon commented Oct 20, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-487-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 20, 2025
Signed-off-by lmignon
@OCA-git-bot
Copy link
Contributor

@lmignon your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-487-by-lmignon-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@lmignon
Copy link
Contributor Author

lmignon commented Oct 20, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-487-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 20, 2025
Signed-off-by lmignon
@OCA-git-bot
Copy link
Contributor

It looks like something changed on 18.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 18.0-ocabot-merge-pr-487-by-lmignon-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 33ca860 into OCA:18.0 Oct 20, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 3e27f0d. Thanks a lot for contributing to OCA. ❤️

bencoronel pushed a commit to factorlibre/storage that referenced this pull request Nov 19, 2025
bencoronel pushed a commit to factorlibre/storage that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request merged 🎉

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants