-
Notifications
You must be signed in to change notification settings - Fork 1.8k
in_tail: fix memory leak when using generic unicode conversion (backport #10781) #10785
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
Conversation
Signed-off-by: Phillip Adair Stewart Whelan <[email protected]>
Caution Review failedFailed to post review comments. WalkthroughAdds new input/output features, Windows TLS support, proxy env ignore, metrics latency histogram, Prometheus textfile input, Node/Windows exporter collectors, Azure Kusto auth modes, CloudWatch entity handling, assorted bug fixes, tests, packaging/version bumps, and CI workflow tweaks. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant User
participant FB as Fluent Bit
participant PT as Prometheus Textfile In
participant FS as Filesystem
participant CMT as CMetrics
User->>FB: Configure in_prometheus_textfile (paths, scrape_interval)
FB->>PT: Init plugin (bind config, register collector)
loop Every scrape_interval
PT->>FS: Glob/Read matching files
FS-->>PT: File contents
PT->>CMT: cmt_decode_prometheus_create(...)
alt Decode OK
PT->>FB: flb_input_metrics_append(cmt)
else Decode Error/Empty
PT-->>FB: Skip/log
end
end
sequenceDiagram
autonumber
participant Cfg as Config
participant K as Azure Kusto Out
participant Auth as Auth Selector
participant MSI as Managed Identity
participant WI as Workload Identity
participant SP as Service Principal
Cfg->>K: auth_type, tenant_id, client_id, secret/token_file
K->>Auth: get_azure_kusto_token()
alt WORKLOAD_IDENTITY
Auth->>WI: flb_azure_workload_identity_token_get(...)
WI-->>Auth: access_token or error
else MANAGED_IDENTITY
Auth->>MSI: flb_azure_msiauth_token_get()
MSI-->>Auth: access_token or error
else SERVICE_PRINCIPAL
Auth->>SP: OAuth2 client_credentials
SP-->>Auth: access_token or error
end
Auth-->>K: Token or failure
sequenceDiagram
autonumber
participant IN as Input
participant ENG as Engine
participant OUT as Output
participant MET as CMetrics
IN->>ENG: Create chunk (record create_time)
ENG->>OUT: Dispatch task
OUT-->>ENG: Result (FLB_OK/ERR)
alt FLB_OK and OUT has cmt_latency
ENG->>MET: cmt_histogram_observe(latency_seconds, labels=input,chunk)
end
sequenceDiagram
autonumber
participant Rec as Record
participant CW as CloudWatch Out
participant Ent as Entity Builder
Rec-->>CW: msgpack map
alt add_entity enabled
CW->>Ent: parse_entity/update_or_create_entity(...)
Ent-->>CW: entity state
CW->>CW: remove aws_entity* keys from maps
CW-->>AWS: PutLogEvents payload with entity block
else
CW-->>AWS: PutLogEvents payload (no entity)
end
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Summary
Backport of #10781.
Enter
[N/A]
in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
Summary by CodeRabbit
New Features
Improvements
Bug Fixes
Tests