fix(fluent-bit): Listen on IPv6+IPv4

[jplitza]

This fixes #355

[lilioid]

I can confirm that this works. A fluent-bit process started with http_listen: "::" listens on both IPv4 (so is compatible to the old behavior) as well as IPv6.

I would like to see this merged as well :)

[genestack-okunitsyn]

It would be great to have it merged, very helpful!

[stevehipwell]

Thanks for the PR @jplitza, please could you make sure you follow the contribution guide.

RE this PR, if you want to make a major change like this you 100% need to open an issue to discuss it (I can't see this change being approved in this state). The current chart will allow you to override http_listen by setting config.service, if you want to make it more customizable you could add a new value but you'd also need to check if 0.0.0.0 is hardcoded anywhere else in the chart and keep that in sync.

If you look at the fluent-bit-aggregator & fluent-bit-collector charts you'll see that this can already be set via config.service.http_listen. I'd strongly recommend that you switch over to one of these charts.

[jplitza]

please could you make sure you follow the contribution guide.

I did at the time of opening the PR.

a major change like this

How is this a major change? It is backwards compatible, because :: accepts IPv4 as well as IPv6.

but you'd also need to check if 0.0.0.0 is hardcoded anywhere else in the chart

It isn't. (Okay, I should have changed the readme as well.)

I'd strongly recommend that you switch over to one of these charts.

That's what we did by now. But again, when I opened the PR, that chart didn't exist. So...

I can't see this change being approved in this state

I'm closing this.