Skip to content

refactor: remove Fluent Bit Lua remnants and dead goldens#2331

Merged
rafaelwestphal merged 1 commit into
ops_agent_3.0from
westphalrafael/remove-fluentbit-lua-remnants
Jun 4, 2026
Merged

refactor: remove Fluent Bit Lua remnants and dead goldens#2331
rafaelwestphal merged 1 commit into
ops_agent_3.0from
westphalrafael/remove-fluentbit-lua-remnants

Conversation

@rafaelwestphal
Copy link
Copy Markdown
Contributor

@rafaelwestphal rafaelwestphal commented Jun 4, 2026

Description

Since Fluent Bit has been decommissioned in favor of OpenTelemetry, this PR cleans up the remaining Lua-related code and dead Fluent Bit configuration golden files from the codebase.
Changes:

  • Deleted confgenerator/add_lua_script.go which defined unused Lua script constants (addLogNameLuaFunction and addLogNameLuaScriptContents).
  • Removed the unused LuaQuote helper function from confgenerator/filter/internal/ast/ast.go.
  • Removed the unused sourceVar field (which stored Lua variable names for copied values) from the ModifyField struct in confgenerator/logging_modify_fields.go.
  • Removed the unused eventLogV2SeverityParserLua constant from confgenerator/logging_receivers.go.
  • Updated test comments in confgenerator/confgenerator_test.go to remove obsolete Fluent Bit/Lua references.
  • Deleted all remaining fluent_bit_main.conf and fluent_bit_parser.conf golden files from confgenerator/testdata/goldens/.

Related issue

How has this been tested?

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes.
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

@rafaelwestphal rafaelwestphal changed the base branch from master to ops_agent_3.0 June 4, 2026 01:21
@rafaelwestphal rafaelwestphal force-pushed the westphalrafael/remove-fluentbit-lua-remnants branch 2 times, most recently from e7e75b7 to 6be95d7 Compare June 4, 2026 01:25
This change removes the unused  function from the AST package, the  constant from logging receivers, and the entire  file which defined unused Lua script constants.
It also removes the unused  field from  struct and updates comments.
Finally, it deletes all remaining  and  golden files from the testdata, as Fluent Bit has been decommissioned.
@rafaelwestphal rafaelwestphal force-pushed the westphalrafael/remove-fluentbit-lua-remnants branch from 6be95d7 to c6b9d96 Compare June 4, 2026 12:55
@rafaelwestphal rafaelwestphal marked this pull request as ready for review June 4, 2026 13:15
@rafaelwestphal rafaelwestphal changed the title Westphalrafael/remove fluentbit lua remnants refactor: remove Fluent Bit Lua remnants and dead goldens Jun 4, 2026
@rafaelwestphal rafaelwestphal merged commit 0d1b6ba into ops_agent_3.0 Jun 4, 2026
71 of 75 checks passed
@rafaelwestphal rafaelwestphal deleted the westphalrafael/remove-fluentbit-lua-remnants branch June 4, 2026 16:56
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.

1 participant