Skip to content

Document FluentdConfig Helm chart template#334

Open
promptless-for-oss wants to merge 2 commits into
kube-logging:masterfrom
Promptless:promptless/document-fluentdconfig-helm-template
Open

Document FluentdConfig Helm chart template#334
promptless-for-oss wants to merge 2 commits into
kube-logging:masterfrom
Promptless:promptless/document-fluentdconfig-helm-template

Conversation

@promptless-for-oss
Copy link
Copy Markdown
Contributor

Open this suggestion in Promptless to view citations and reasoning process

Documents the new FluentdConfig Helm chart template that creates a FluentdConfig resource when logging.enabled is true. Adds Helm as a third configuration method and provides examples for configuring Fluentd via Helm values.

Trigger Events


Tip: Tag @Promptless in GitHub PR comments to guide documentation changes during code review 🐙

Add documentation for configuring Fluentd via Helm chart values. The Helm
chart now creates a FluentdConfig resource when logging.enabled is true
and logging.fluentdDisabled is false, using values from logging.fluentd.

- Update "Ways to configure Fluentd" section to list Helm as a third option
- Add new "Configuring Fluentd with Helm" section with examples
- Update Helm chart note to mention FluentdConfig and FluentbitAgent resources
For the detailed list of available parameters, see {{% xref "/docs/configuration/crds/v1beta1/fluentd_types.md" %}}.

### Configuring Fluentd with Helm {#helm}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR #2246 adds the charts/logging-operator/templates/logging/fluend.yaml template that creates a FluentdConfig resource from logging.fluentd Helm values when logging is enabled.

Source: kube-logging/logging-operator#2246

The PR #2247 implementation changed during review. The FluentdConfig
creation is now opt-in via logging.fluentdConfig.create (default: false),
rather than automatic when logging.enabled=true. Updated documentation
to accurately reflect this behavior with corrected examples.
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