-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
in_calyptia_fleet: support multiple files for configurations. #8092
Conversation
fix conflict pls |
There is also #7969 that is relevant to fleet, especially for windows. |
5ead325
to
18dc15c
Compare
18dc15c
to
a1275a8
Compare
a1275a8
to
c19ce47
Compare
The branch has been rebased again off of master and all conflicts fixed. |
@edsiper all tests besides the log timing test have passed. |
There is still an out of scope memory leak but it seems to be a single allocation over a long time frame:
That is the end of a run of about 4 hours with several configuration updates. |
@pwhelan pls fix the conflict; re the leak maybe its related to |
…m json. Signed-off-by: Phillip Whelan <[email protected]>
Signed-off-by: Phillip Whelan <[email protected]>
…ion on windows. Signed-off-by: Phillip Whelan <[email protected]>
Signed-off-by: Phillip Whelan <[email protected]>
…e 'new' symlink exists. Signed-off-by: Phillip Whelan <[email protected]>
…ithout files. Signed-off-by: Phillip Whelan <[email protected]>
Signed-off-by: Phillip Whelan <[email protected]>
Summary
Add support for extra fleet files. These go along with the main configuration and can be referred to using relative paths. The intention is to use them for:
But they are not restricted to just those uses. They can also be used as general files for plugins.
Enter
[N/A]
in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
ok-package-test
label to test for all targets (requires maintainer to do).Documentation
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.