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: Security: CVEs in suggested (EOL) image for fluentd #1166

Closed
max-allan opened this issue May 14, 2024 · 3 comments
Closed

bug: Security: CVEs in suggested (EOL) image for fluentd #1166

max-allan opened this issue May 14, 2024 · 3 comments
Labels
good first issue Good for newcomers

Comments

@max-allan
Copy link

Describe the issue

From the helm values file:

    repository: "kubesphere/fluentd"
    tag: "v1.15.3"

That image has (according to Grype) a lot of vulnerabilities:

 3 critical, 5 high, 22 medium, 1 low, 0 negligible

Also, fluentd 1.15.3 is EOL.

Can we update the image to 1.16 or 1.17? Will the operator work with newer versions?

Anywhere that image is referenced will need updating, not just that location in the chart.

In addition, fluent-bit 2.2.2 is EOL in a few weeks time.

To Reproduce

See the values file

Expected behavior

Current/supported versions of fluentd and fluent-bit are used by default.

Your Environment

- Fluent Operator version: 2.8.0
- Container Runtime: any
- Operating system: any
- Kernel version: any

How did you install fluent operator?

Helm

Additional context

No response

@benjaminhuo benjaminhuo added the good first issue Good for newcomers label May 15, 2024
@benjaminhuo
Copy link
Member

@max-allan Good point, we do need hands to upgrade and test new version of fluentd and fluentbit

@SvenThies
Copy link
Contributor

@max-allan can you confirm that this issue is closed with #1199. Can this issue be closed?

@benjaminhuo
Copy link
Member

@max-allan can you confirm that this issue is closed with #1199. Can this issue be closed?

Yes, fluentd was upgraded to 1.17 in https://github.com/fluent/fluent-operator/blob/master/charts/fluent-operator/values.yaml#L383

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants