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

[fluent-operator] upgrade fluent-operator from v2.7.0 to v2.8.0 #510

Merged
merged 5 commits into from
Jun 13, 2024

Conversation

elsnepal
Copy link
Contributor

@elsnepal elsnepal commented May 23, 2024

This change is to bring v2.8.0 changes from https://github.com/fluent/fluent-operator/tree/v2.8.0/charts

@patrick-stephens
Copy link
Contributor

Can you fix the DCO as that is required for merge?
Also a quick description on what you did to help future people :)

@jhoblitt
Copy link

I would really like to see this go in.

Also, is there any reason why a gha can't be used to publish releases directly from https://github.com/fluent/fluent-operator/tree/master/charts/fluent-operator when it is tagged? E.g. using an action to publish it to gh-pages in that repo (yes, the repo url would change) That would make the diff between tags more useful.

@elsnepal
Copy link
Contributor Author

elsnepal commented Jun 3, 2024

I would really like to see this go in.

Also, is there any reason why a gha can't be used to publish releases directly from https://github.com/fluent/fluent-operator/tree/master/charts/fluent-operator when it is tagged? E.g. using an action to publish it to gh-pages in that repo (yes, the repo url would change) That would make the diff between tags more useful.

The way i see, we shouldn't even have duplicate copy and helm chart to reference to cross repo where the original chart lives.

https://github.com/fluent/helm-charts/blob/gh-pages/index.yaml#L4718C5-L4719C110 Assuming this index could be looking at operator repo?

@jhoblitt
Copy link

jhoblitt commented Jun 3, 2024

I used this in a project a few weeks ago for pushing chart releases to gh-pages. It is bare-bones but it works. https://github.com/helm/chart-releaser-action

@elsnepal
Copy link
Contributor Author

@patrick-stephens any chance to get this out?

@berowil
Copy link

berowil commented Jun 10, 2024

Would it be possible to release 2.8.0 in the near future? :)

@berowil
Copy link

berowil commented Jun 11, 2024

This change is to bring v2.8.0 changes from https://github.com/fluent/fluent-operator/tree/v2.8.0/charts

Could you please also fix "Lint and Test Charts Checks"? Thank you!

elsnepal and others added 4 commits June 12, 2024 08:01
Signed-off-by: Ashish Nepal <a.nepal@elsevier.com>
Signed-off-by: Ashish Nepal <a.nepal@elsevier.com>
Signed-off-by: Sébastien Larivière <sebastien.lariviere@goto.com>
Signed-off-by: Ashish Nepal <a.nepal@elsevier.com>
Signed-off-by: Sébastien Larivière <sebastien.lariviere@goto.com>
Signed-off-by: Ashish Nepal <a.nepal@elsevier.com>
@benjaminhuo
Copy link
Member

fluent operator v2.9.0 is out, supporting fluent-bit v3.0.7 and fluentd 1.17.0
https://github.com/fluent/fluent-operator/releases/tag/v2.9.0

@wenchajun would you help to sync the chart to this official repo?

@benjaminhuo benjaminhuo merged commit 53722cf into fluent:main Jun 13, 2024
2 checks passed
@benjaminhuo
Copy link
Member

fluent operator v2.9.0 is out, supporting fluent-bit v3.0.7 and fluentd 1.17.0 https://github.com/fluent/fluent-operator/releases/tag/v2.9.0

@wenchajun would you help to sync the chart to this official repo?

fluent operator 2.9.0 will be updated soon by @wenchajun

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.

None yet

7 participants