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: http input keepaliveTimeout in CRD transforms to keepLive_timeout, should be keepalive_timeout #965

Closed
applike-ss opened this issue Oct 27, 2023 · 0 comments · Fixed by #968

Comments

@applike-ss
Copy link

applike-ss commented Oct 27, 2023

Describe the issue

The http input in the CRD has a keepaliveTimeout property which the operator makes keepLive_timeout in the fluent config secret.
However it should be keepalive_timeout as per the docs here: https://docs.fluentd.org/input/http

To Reproduce

Create a fluentd.fluent.io/v1alpha1 Fluentd with one http globalInputs and specify the keepaliveTimeout property.

Expected behavior

Should translate to keepalive_timeout in the compiled fluentd config

Your Environment

- Fluent Operator version: v2.4.0
- Container Runtime: doesn't matter in this case
- Operating system: doesn't matter in this case
- Kernel version: doesn't matter in this case

How did you install fluent operator?

helm chart from https://fluent.github.io/helm-charts - shouldn't matter though

Additional context

output in the logs:

2023-10-27 09:28:29 +0000 [warn]: fluent/log.rb:383:warn: parameter 'keepLive_timeout' in <source>
  @type http
  bind "0.0.0.0"
  body_size_limit 32MB
  keepLive_timeout 10s
  port 15001
</source> is not used.
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 a pull request may close this issue.

1 participant