Skip to content
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

[fluent-bit] Enabling hotReload creates an empty luascripts configmap #472

Open
rgaduput opened this issue Mar 18, 2024 · 0 comments
Open

Comments

@rgaduput
Copy link
Contributor

We have requirement where we create the ConfigMap of lua scripts outside the fluent-bit charts. The reason is that configmap-luascripts.yaml file in the chart does not support rendering.
After creating custom configMap with the required lua scripts, i have tried to enable hotReload. But because of the way chart is designed https://github.com/fluent/helm-charts/blob/main/charts/fluent-bit/templates/_pod.tpl#L94 https://github.com/fluent/helm-charts/blob/main/charts/fluent-bit/templates/configmap-luascripts.yaml#L1 when ever hotReload is enabled it tries to create a lua scripts configMap does not matter what we have at .Values.luaScripts and it overwrites my custom config mountpoint inside the pod (/fluent-bit/scripts) which causes fluent-bit to fail since the script does not exists any more at path /fluent-bit/scripts/<script.lua>.

My suggestion is to support hotReload without mixing with the lua script config.

Thanks

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

No branches or pull requests

1 participant