Skip to content

[aws-for-fluent-bit] Add optional luaScripts ConfigMap and volume mount - #1365

Open
dundysm wants to merge 2 commits into
aws:masterfrom
dundysm:feat/fluent-bit-luascripts
Open

dundysm wants to merge 2 commits into
aws:masterfrom
dundysm:feat/fluent-bit-luascripts

Conversation

@dundysm

@dundysm dundysm commented Sep 2, 2026

Copy link
Copy Markdown

Issue

Fixes #1307

Description of changes

Optional luaScripts values, ConfigMap, DaemonSet mount at /fluent-bit/scripts/. Empty default is a no-op. Rolling update via checksum annotation when scripts are set.

Checklist

  • Added/modified documentation as required (such as the README.md for modified charts)
  • Incremented the chart version in Chart.yaml for the modified chart(s)
  • Manually tested. Describe what testing was done in the testing section below
  • Make sure the title of the PR is a good description that can go into the release notes

Testing

Repo make verify (scripts/validate-charts.sh, validate-chart-versions.sh, lint-charts.sh) — pass. aws-for-fluent-bit version 0.2.0 → 0.2.1. make package produced aws-for-fluent-bit-0.2.1.tgz.

helm lint stable/aws-for-fluent-bit — pass (default and with a luaScripts override).

helm template test-fb stable/aws-for-fluent-bit --namespace kube-system (default luaScripts: {}) — no luascripts ConfigMap, no /fluent-bit/scripts/ volume or volumeMount, no checksum/luascripts.

helm template with a multiline helpers.lua override — ConfigMap YAML parses; script body is in data.helpers.lua; DaemonSet has the luascripts volume, mount at /fluent-bit/scripts/, and checksum/luascripts.

kubeconform -strict -ignore-missing-schemas and kubeval --strict --ignore-missing-schemas on default and lua override renders — pass.

Did not apply to a cluster (no EKS/kind install, no log-shipping e2e).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dundysm

dundysm commented Sep 2, 2026

Copy link
Copy Markdown
Author

@PettitWesley @mforutan this adds optional luaScripts on aws-for-fluent-bit for #1307 (ConfigMap + mount at /fluent-bit/scripts/). Default {} is a no-op.

make verify / make package, helm lint/template, and kubeconform/kubeval are green on the renders.

Happy to change the values shape if you'd rather this reuse volumes / volumeMounts instead of a dedicated key.

This branch has not been deployed

No deployments
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.

Support luaScripts - add ConfigMap template and DaemonSet volume mount for Lua scripts

1 participant