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

bug: Allows to render net properties for outputs #1298

Merged
merged 2 commits into from
Aug 14, 2024

Conversation

dex4er
Copy link
Contributor

@dex4er dex4er commented Aug 13, 2024

What this PR does / why we need it:

It renders attributes from networking field and fixes DNSResolver.

Which issue(s) this PR fixes:

Fixes #1297

Does this PR introduced a user-facing change?

No

Additional documentation, usage docs, etc.:

dex4er added 2 commits August 13, 2024 21:03
Signed-off-by: Piotr Roszatycki <[email protected]>
Signed-off-by: Piotr Roszatycki <[email protected]>
@dex4er dex4er force-pushed the bugfix/networking branch from 5845760 to 6b540ce Compare August 13, 2024 19:03
@dex4er dex4er changed the title Bugfix/networking bug: Can't set net.dns.resolver Aug 13, 2024
@dex4er dex4er changed the title bug: Can't set net.dns.resolver bug: Allows to render net properties for putputs Aug 13, 2024
@dex4er dex4er changed the title bug: Allows to render net properties for putputs bug: Allows to render net properties for outputs Aug 13, 2024
@dex4er
Copy link
Contributor Author

dex4er commented Aug 13, 2024

You can test with image public.ecr.aws/dex4er/fluent-operator:pr-1298

The configuration renders now to:

[Output]
    Name    opensearch
    Match    *
    Host    redacted
    Port    443
    Buffer_Size    10M
    Type    _doc
    Logstash_Format    true
    Logstash_Prefix    redacted
    Logstash_DateFormat    %Y%m%d
    Time_Key    @timestamp
    Time_Key_Format    %Y-%m-%dT%H:%M:%S
    Time_Key_Nanos    false
    Include_Tag_Key    true
    Tag_Key    log_name
    Generate_ID    false
    Write_Operation    create
    Replace_Dots    true
    Trace_Output    false
    Trace_Error    true
    Current_Time_Index    false
    Suppress_Type_Name    true
    tls    On
    net.dns.resolver    LEGACY

and fluent-bit actually starts as expected and I see logs in OpenSearch index.

@benjaminhuo benjaminhuo merged commit 3d77840 into fluent:master Aug 14, 2024
9 of 10 checks passed
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.

bug: Can't set net.dns.resolver
2 participants