Skip to content

Conversation

shasts
Copy link
Member

@shasts shasts commented Apr 8, 2025

Clarifies the --user and --pass arguments required for ECE 3.8.0 in the simple upgrade step example.

@shasts shasts added the documentation Improvements or additions to documentation label Apr 8, 2025
@shasts shasts self-assigned this Apr 8, 2025
@shasts shasts enabled auto-merge (squash) April 9, 2025 04:46
@shasts shasts disabled auto-merge April 9, 2025 04:46
@shasts shasts enabled auto-merge (squash) April 9, 2025 04:46
Copy link
Contributor

@jakommo jakommo left a comment

Choose a reason for hiding this comment

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

Left a comment. No strong opinion if you feel otherwise

@jakommo
Copy link
Contributor

jakommo commented Apr 9, 2025

One more thing. Are the 3.8 docs built from this? My understanding was it would only apply to ECE 4.0 docs and later?

In any case, I just saw elastic/ansible-elastic-cloud-enterprise#205 (comment) and you had a good point about upgrades using the latest version of the elastic-cloud-enterprise.sh script. So I think we should back port this to earlier 3.x ECE docs as well.

@georgewallace
Copy link
Contributor

@jakommo No, this would be 4.0 only. For 3.8 we would need to make the updates in asciidoc. If this does relate to 3.8, I can absolutely cover over the contents to a PR there.

@shasts
Copy link
Member Author

shasts commented Apr 9, 2025

@georgewallace
I created https://github.com/elastic/cloud/pull/139863
Thank you, @jakommo

Copy link
Contributor

@jakommo jakommo left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the quick PR Sha!

@shasts shasts merged commit 4f17c0f into elastic:main Apr 9, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants