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

core: added filesystem abstraction layer #9325

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

leonardo-albertovich
Copy link
Collaborator

This PR introduces a platform agnostic filesystem interface that implements functionality that used to be repeated in multiple plugins.

A platform agnostic change monitoring component based on native facilities (inotify, windows fs change notifications, FSEvent, etc) will be added before this is merged.

This PR does not change any of the existing code in significative ways other than renaming flb_file_read to flb_file_read_contents.

Note: This PR contains the comoponents introduced by PR 7564 in a way that can be merged immediately with no compromises and another PR will be opened to update in_tail to use the new abstraction layer.

@edsiper
Copy link
Member

edsiper commented Sep 16, 2024

@leonardo-albertovich is this one is ready pls remove the draft mode

@leonardo-albertovich
Copy link
Collaborator Author

I'd like to get a chance to add the monitoring part before we merge it which I won't be able to work on for about two weeks so I think it'd be better to leave it as a draft for the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants