Charts should be published and indexed on merge to master

[max-rocket-internet]

It's just confusing for everyone otherwise.

For example:
#46
#36

[jaypipes]

Not sure about every merge to master, but I suppose I could support auto-publish/index when a new tag is pushed to the master repos of the underlying projects

[max-rocket-internet]

No, I mean like someone makes a PR to update a chart in this repo. The PR is reviewed, the chart version is bumped and the the PR is merged. At this point, we can see it in master but we can't use it as it hasn't been published yet. That is what those 2 issues above are about.

My point is: once a PR has been approved and merged, it should be published and available to use via Helm immediately. That's how the main stable charts repo does it. It's confusing if people have to wait until a tag is created for this whole repo. See what I mean?

[stefanprodan]

It takes many hours for the stable charts to be published on GPC. The current publishing flow allows us to bundle several PRs in a release and have a change log in GitHub releases. I think it's the reviewer responsibility to do a release or not after the merge and also document the changes.

[max-rocket-internet]

It takes many hours for the stable charts to be published on GPC

It takes less than a minute in our CI with many more charts, feel free to copy it: https://github.com/deliveryhero/helm-charts/blob/master/.github/workflows/push-chart.yaml

😃

[max-rocket-internet]

@jaypipes @M00nF1sh @fawadkhaliq can we address what I mention in this issue?

It's so frustrating having to push to get a chart update merged and THEN have to wait/push for a new release to be made.

Another example is #345

Not sure about every merge to master

Why not? That's a totally normal work flow. The old Helm stable repo did this and I don't know how bitnami do it but a PR merged 2 hours ago is already in their index 😅

[max-rocket-internet]

@kishorj @bwagner5

Look at all these issues that could be prevented:

#272
#281
#247
#244

[max-rocket-internet]

And more

#185
#193

I think you get my point 🙂

[jwenz723]

I would love to see this functionality built in as well. My PR, #472, got merged 11 days ago by @fawadkhaliq. I still cannot make use of the features in my PR because a released version of the helm chart has not been published yet.

[kazu]

when will the newer version including the aws-for-fluent-bit chart (> 0.1.7) publish?
these PR already is merged to master

#446
#492
#353
#491