Skip to content

Conversation

@eedugon
Copy link
Contributor

@eedugon eedugon commented Dec 17, 2025

This PR adds documentation for Elastic Stack Helm chart (provided by ECK) users to know how to upgrade and apply configuration changes.

As you'll see, certain parts of the content are not purely about ECK and Elastic Stack specifics, but a bit about how Helm works and educates the reader in areas that are not purely ours.

I've tried to fit the content to our exact domain, but giving enough information to understand the basics.

Closes #3591

After admin-docs review we will need to find devs to review the content.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

Vale Linting Results

Summary: 1 suggestion found

💡 Suggestions (1)
File Line Rule Message
deploy-manage/deploy/cloud-on-k8s/managing-deployments-using-helm-chart.md 226 Elastic.Capitalization 'Add Ingress to the' should use sentence-style capitalization.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

although I don't know too much about kubernetes / helm, none of this struck me as superfluous information. I think you did a good job of adding the right amount of info but keeping it task-based for the ECK context.

approving because this is nearly perfect - whatever you do from here is mergeable from my pov.

one thing that might be missing: do I need a specific version of the helm chart to access later versions of the stack? e.g. is upgrading the helm chart version a prerequisite? If that is the case, I believe it is not clear.

Comment on lines +22 to +24
::::{note}
If you deploy your {{stack}} resources using our Helm chart, refer to [](/deploy-manage/deploy/cloud-on-k8s/managing-deployments-using-helm-chart.md#k8s-upgrade-modify-helm) for details on how to perform upgrades with Helm.
::::
Copy link
Collaborator

@shainaraskas shainaraskas Dec 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this note kind of works, but I don't like how it's splitting the instructions and example, and it is burying an up-front choice in the middle of the procedure

consider instead:

  1. introducing that there are two ways to do it in the intro for the page
  2. making two H2s, one for each: e.g. "Perform the upgrade in the cluster spec file" "Perform the upgrade using a Helm chart"

not sure if there is a better way to word the headings, because it's less about the method of performing the upgrade, than the context of the configuration ... some alternative headings might be:

"For orchestrators using spec files"
"For orchestrators using Helm charts"

orchestrators / operators / environments / deployments / clusters ... whatever feels the most accurate

you see what I'm getting at :)

Copy link
Contributor Author

@eedugon eedugon Dec 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see your point, and I agree.... however I didn't want to do extra changes on documents that are already established and feel very simple (in a good way).

Take in mind the following and let me know your thoughts.... :)

Currently we have this:

image
  • The doc intro feels accurate for all ECK users (helm vs no helm) as it starts with an intro about upgrading in general with links to preparations, etc.

  • Then when it comes to the "procedure" what we show is that technically upgrading is super simple, just change the version of your spec. It's true that we don't mention helm based installations earlier, but considering how small the procedure is and that we have the note right after I think it should be fine.

  • I also think it's good to "force" a helm-user to read the standard procedure, because at the end of the day, what they have to achieve through helm is the same (change the version on the final spec file).

What I think we could really to do improve this is moving upwards these two paragraphs that are currently at the end:
image

They are relevant to all users and those feel a big hidden at the end of a big example. Maybe those 2 paragraphs could be in the intro. What do you think? :)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think moving up those paragraphs makes sense, but I still think that forcing helm chart users to read through this procedure is a little confusing, especially because the easiest way to upgrade is via a flag on a helm command, which might impact the underlying spec file, but it's not the same thing as editing the spec file directly.

won't block this but that is my opinion

Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
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.

Upgrade cluster on ECK doc is not considering Helm based deployments

3 participants