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: Tentative release date for the changes related: Fluent-bit init container configurations in fluent-operator helm chart- #1340

Open
btalakola opened this issue Sep 12, 2024 · 8 comments

Comments

@btalakola
Copy link
Contributor

btalakola commented Sep 12, 2024

Describe the issue

Hi Team,

#1320
This PR allows us to configure the initContainers in fluentbit custom resource. Fluentbit CRD schema already has the option to configure initContainers. So just exposing the same in values.yaml.

Can you please provide the tentative release date for the above changes.

To Reproduce

Hi Team,

#1320
This PR allows us to configure the initContainers in fluentbit custom resource. Fluentbit CRD schema already has the option to configure initContainers. So just exposing the same in values.yaml.

Can you please provide the tentative release date for the above changes.

Expected behavior

Hi Team,

#1320
This PR allows us to configure the initContainers in fluentbit custom resource. Fluentbit CRD schema already has the option to configure initContainers. So just exposing the same in values.yaml.

Can you please provide the tentative release date for the above changes.

Your Environment

 Fluent Operator version: v3.1.0
- Container Runtime: containerd
- Operating system: Ubuntu 22.04.4 LTS

How did you install fluent operator?

Deploy fluent-bit using fluent-operator helm chart.

Additional context

No response

@btalakola btalakola changed the title bug: Fluent-bit init container configurations in fluent-operator helm chart- bug: Tentative release date for the changes related: Fluent-bit init container configurations in fluent-operator helm chart- Sep 12, 2024
@benjaminhuo
Copy link
Member

@wenchajun are you able to release a new version v3.2.0 of fluent operator?
cc @wanjunlei @Gentleelephant

@benjaminhuo
Copy link
Member

@cw-Guo Would you help to release a new version of fluent operator following https://github.com/fluent/fluent-operator/blob/master/RELEASE.md#how-to-cut-a-new-release ?

@reegnz
Copy link
Contributor

reegnz commented Sep 16, 2024

I'd also be grateful for this chart improvement to be released soon. Right now I have to work around this issue by having forked the chart, which is sub-optimal.

@btalakola
Copy link
Contributor Author

HI Team, could you please provide an update on this?

@cw-Guo
Copy link
Collaborator

cw-Guo commented Sep 21, 2024

@btalakola please stay tuned. #1343 we are working on a release. cc @benjaminhuo

@reegnz
Copy link
Contributor

reegnz commented Sep 23, 2024

Seems like a version bump was missed over here: https://github.com/fluent/fluent-operator/blob/v3.2.0/charts/fluent-operator/Chart.yaml#L18
Probably not a game-changer as the charts are released from a different repo. But you might want to incorporate bumping those versions into the release procedure document.

@btalakola
Copy link
Contributor Author

Hi Team,
There is an indentation issue for the initContainers configuration in fluent-bit custom resource in the release 3.2.0
https://github.com/fluent/fluent-operator/blob/release-3.2/charts/fluent-operator/templates/fluentbit-fluentBit.yaml#L116
Could you please check it?

@harshvora10101
Copy link

Fix for the indentation issue : #1360

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

No branches or pull requests

5 participants