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

Confusion on helm chart source/repository #1376

Open
joshuabaird opened this issue Oct 15, 2024 · 3 comments
Open

Confusion on helm chart source/repository #1376

joshuabaird opened this issue Oct 15, 2024 · 3 comments

Comments

@joshuabaird
Copy link
Collaborator

Describe the issue

I'm confused (and others are too) about where the source for the "official" fluent-operator Helm chart lives.

There are two chart sources published:

The first chart (in the fluent-operator repo) is included in the documentation for installing fluent-operator. This chart doesn't seem to be regularly updated properly (the appVersion is out of sync with the current app version, etc) nor is it published to any Helm repository.

The second chart (in the fluent/helm-charts repo) does appear to be updated and is published to fluent.github.io/helm-charts.

This is extremely confusing for both maintainers and users of the Helm chart. As it stands currently, it seems that contributors are expected to manually sync changes between the two charts (which doesn't always happen). Secondly, the first Helm chart is not published to a repo which makes it more difficult to use.

Should we do do the following?

How did you install fluent operator?

No response

Additional context

No response

@joshuabaird
Copy link
Collaborator Author

@wenchajun @benjaminhuo @wanjunlei Any opinions or thoughts?

@benjaminhuo
Copy link
Member

Currently, the first one is the most updated which needs to be synced to the second whenever a new release is out.
You can consider the first a dev branch while the second is a release branch.

Some kind of workflow is need to automatically sync the dev helm chart to the second repo whenever a release is out

@joshuabaird
Copy link
Collaborator Author

joshuabaird commented Oct 16, 2024

I'll take a look at how fluent-bit and fluentd handle this, but I do think it's confusing the chart is published in two places.

Furthermore, if the second one is intended to be a "release" helm chart, our "install" documentation shoukd reference that chart/repo and not a tarball from this repo. Agreed?

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

2 participants