elastic / elastic-agent

Elastic Agent - single, unified way to add monitoring for logs, metrics, and other types of data to a host.
Other
124 stars 133 forks source link

Mark the Elastic Agent Helm Chart as beta #5485

Open ycombinator opened 1 week ago

ycombinator commented 1 week ago

Describe the enhancement:

In https://github.com/elastic/elastic-agent/pull/5331, we added a Helm Chart for Elastic Agent. For the moment, we would like to mark this chart as beta.

Describe a specific use case for the enhancement or feature:

To set appropriate expectations around stability and supportability of the Helm Chart with its users.

What is the definition of done?

TBD — we need to first decide how best to mark the chart as beta.

Related: https://github.com/elastic/elastic-agent/issues/5486

ycombinator commented 1 week ago

Copied from https://github.com/elastic/elastic-agent/pull/5331#issuecomment-2338320617, where @pkoutsovasilis said:

Just thinking out loud the available options here:

  1. I assume that we can push the helm chart with a beta suffix at the version and update the NOTES.txt accordingly here https://helm.elastic.co/
  2. Somebody that wants to try this helm chart can clone the repo and install the helm chart from the folder deploy/helm/elastic-agent
  3. We could utilise github pages on this repo and commit directly the helm chart package.tgz (that points to the SNAPSHOT agent image) [my least favourite option]

that's why a discussion about the Helm chart release process is wise to have before any next step 🙂

ycombinator commented 1 week ago

Copied from https://github.com/elastic/elastic-agent/pull/5331#issuecomment-2338484088, where @swiatekm said:

Helm Charts use semver 2.0 for versioning, so it's simple enough to just designate our releases as betas by setting it to va.b.c-beta.d or something similar. This makes Helm automatically treat it as a prerelease. This will cause them not to show up in searches without the --devel flag, amongst other consequences.

ycombinator commented 1 week ago

cc: @cmacknz

swiatekm commented 6 days ago

I like @pkoutsovasilis's option 1. I don't see any reason to make things complicated if we already have a Helm repo we can push to.