Update fluent-bit image to 2.0.4 #279
Conversation
iamleot
requested review from
edsiper,
naseemkullah and
stevehipwell
as code owners
|
FYI: we are shipping 2.0.4 right now |
Whoops, I've missed that! If no candidate PR is prepared for the Helm chart I can adjust this one to update to Thank you @edsiper! |
|
@edsiper is there anything which would need to change to support v2? Is YAML GA yet (if not where can I feedback on it as the use of arrays is problematic)? I'm still planning on splitting the charts out into collector and aggregator variants so major work can happen then. |
|
|
@stevehipwell YAML is still problematic with arrays, see fluent/fluent-bit#5446 When YAML goes GA it would be good to have better support in the chart for it (currently you need to override the start params and use a custom config file) |
|
|
iamleot
changed the title
There was a problem hiding this comment.
Thanks for the PR @iamleot, just one comment for you to take a look at.
|
@iamleot I'm going to try and merge a couple of other PRs before yours as the current automation is very noisy and it'd be best to get the noise out of the way before the v2 chart release. You might need to rebase once they're done but I'll comment back here when you can do it without needing to do so multiple times. |
|
Just waiting for #278. |
|
@iamleot could you rebase this now? |
Closes #274. Signed-off-by: Leonardo Taccari <leonardo@faire.ai>
|
@iamleot do you want to remove the draft status? |
iamleot
requested review from
stevehipwell
and removed request for
edsiper and
naseemkullah
Yeah, I was waiting for the GitHub Actions workflows checks to complete to avoid possible further notification spamming! I have rebased to latest If there is anything else please let me know! Thank you very much! |
Closes #274.
Signed-off-by: Leonardo Taccari leonardo@faire.ai