fluent-bit
fluent-bit copied to clipboard
fluent
Fixing bug in flb_plugin.c: key -> value
Sorry, there is nothing to fill there, it's obvious bug. Not sure about the fix though, just used exactly the same syntax as in the very next function flb_plugin_load_config_file
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:
- [N/A] Example configuration file for the change
- [N/A] Debug log output from testing the change
- [N/A] Attached Valgrind output that shows no leaks or memory corruption was found
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
- [N/A] Run local packaging test showing all targets (including any new ones) build.
- [N/A] Set
ok-package-testlabel to test for all targets (requires maintainer to do).
Documentation
- [N/A] Documentation required for this feature
Backporting
- [N/A] Backport to latest stable release.
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.
@edsiper @leonardo-albertovich @fujimotos @koleini please take a quick look on this one-line fix.
@edsiper @leonardo-albertovich @fujimotos @koleini plugin loading from file is still broken, please take a look at this fix
@edsiper @leonardo-albertovich @fujimotos @koleini this one-liner fixes plugin loading (which are specified via config file, while via command line works ok), it's kinda important IMO, please take a look
@edsiper @leonardo-albertovich @fujimotos @koleini may I kindly ask you to review this one line fix of a typo?
@edsiper @leonardo-albertovich @fujimotos @koleini please take a look 🙏 I can't update FB to newest version because of this tiny bug...