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

Mark the Elastic Agent Helm Chart as beta #5485

Closed
ycombinator opened this issue Sep 9, 2024 · 5 comments · Fixed by #6293
Closed

Mark the Elastic Agent Helm Chart as beta #5485

ycombinator opened this issue Sep 9, 2024 · 5 comments · Fixed by #6293
Assignees

Comments

@ycombinator
Copy link
Contributor

ycombinator commented Sep 9, 2024

Describe the enhancement:

In #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: #5486

@ycombinator
Copy link
Contributor Author

Copied from #5331 (comment), 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
Copy link
Contributor Author

Copied from #5331 (comment), 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
Copy link
Contributor Author

cc: @cmacknz

@swiatekm
Copy link
Contributor

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.

@ycombinator
Copy link
Contributor Author

Seems like we have consensus on this option:

push the helm chart with a beta suffix at the version and update the NOTES.txt accordingly here https://helm.elastic.co/

@pkoutsovasilis Could you please close this issue when the above is done?

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 a pull request may close this issue.

3 participants