Skip to content

Conversation

@PeterStolz
Copy link
Contributor

I opened fluent/fluent-bit#8348 thinking that there was no option to set uri/path for the loki output plugin, but when checking how to implement it I found out it already exists

https://github.com/fluent/fluent-bit/blob/cd497e38dc8e4e958756b449aab34801da486ad2/plugins/out_loki/loki.c#L1741-L1748

Signed-off-by: Peter Stolz <50801264+PeterStolz@users.noreply.github.com>
@PeterStolz PeterStolz marked this pull request as draft January 10, 2024 13:41
@PeterStolz
Copy link
Contributor Author

Nevermind. While it exists in source there is some validation inbetween preventing me from using it directly

@PeterStolz PeterStolz closed this Jan 10, 2024
@PeterStolz
Copy link
Contributor Author

Nevermind I did not run the latest version of fluent-bit. Setting uri works fine.

@PeterStolz PeterStolz reopened this Jan 10, 2024
Signed-off-by: Peter Stolz <50801264+PeterStolz@users.noreply.github.com>
@PeterStolz PeterStolz marked this pull request as ready for review January 10, 2024 13:56
Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified via docker run --rm -it fluent/fluent-bit:2.2.1 -o loki --help

@patrick-stephens patrick-stephens merged commit 9509165 into fluent:master Jan 11, 2024
TomlinfreeGit pushed a commit to TomlinfreeGit/fluent-bit-docs that referenced this pull request Oct 28, 2025
* Added documentation for uri parameter

Signed-off-by: Peter Stolz <50801264+PeterStolz@users.noreply.github.com>

* Document Tls parameter for loki.md

Signed-off-by: Peter Stolz <50801264+PeterStolz@users.noreply.github.com>

---------

Signed-off-by: Peter Stolz <50801264+PeterStolz@users.noreply.github.com>
Signed-off-by: Tom <yao.lin@siemens.com>
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