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

Clarify high availability recommendations in Elasticsearch orchestration docs #8151

Merged
merged 3 commits into from
Oct 25, 2024

Conversation

stefnestor
Copy link
Contributor

👋 howdy, team! Can we call out that our recommendation for updates applying to highly available clusters should be multiple nodes per data tier and not just multiple nodes. (Multiple nodes would potentially keep responsiveness as long as master-eligible, but this avoids status:red search/ingestion issues.)

👋 howdy, team! Can we call out that our recommendation for updates applying to highly available clusters should be multiple nodes per data tier and not just multiple nodes. (Multiple nodes would potentially keep responsiveness as long as master-eligible, but this avoids `status:red` search/ingestion issues.)
@stefnestor stefnestor added >enhancement Enhancement of existing functionality >docs Documentation labels Oct 22, 2024
@stefnestor stefnestor requested a review from a team October 22, 2024 16:31
…chestration.asciidoc

Co-authored-by: Stef Nestor <26751266+stefnestor@users.noreply.github.com>
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@stefnestor
Copy link
Contributor Author

stefnestor commented Oct 22, 2024

Thanks, @kilfoyle ! While I believe this is true (as it is true for other Elastic platforms like self-hosted and ESS/ECE), my team had some uncertainty to confirm this is also true for ECK. Could we flag this requiring Dev sign-off? (Alternative, ECK could temporarily remove _tier_preference from indices during rolling restarts, which'd be news to me that I'd definitely want to hear about 🙂.) 🙏🏽

@kilfoyle
Copy link
Contributor

Could we flag this requiring Dev sign-off?

Yup! If my memory serves, I don't think I can merge anything in this repo without the ECK team first giving it a thumbs up. :-)

@kilfoyle
Copy link
Contributor

@elasticmachine run docs-build

Co-authored-by: Peter Brachwitz <peter.brachwitz@elastic.co>
@thbkrkr thbkrkr changed the title (Doc+) Rolling changes high availability is per ES data tier Clarify high availability recommendations in Elasticsearch orchestration docs Oct 25, 2024
@thbkrkr thbkrkr merged commit f2243bc into main Oct 25, 2024
5 checks passed
@thbkrkr thbkrkr deleted the stefnestor-patch-2 branch October 25, 2024 13:40
thbkrkr pushed a commit that referenced this pull request Oct 25, 2024
…ion docs (#8151)

This updates the orchestration documentation to clarify high availability setup recommendations.

---------
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
Co-authored-by: Peter Brachwitz <peter.brachwitz@elastic.co>
thbkrkr added a commit that referenced this pull request Oct 28, 2024
…ion docs (#8151) (#8154)

This updates the orchestration documentation to clarify high availability setup recommendations.

---------
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
Co-authored-by: Peter Brachwitz <peter.brachwitz@elastic.co>
Co-authored-by: Stef Nestor <26751266+stefnestor@users.noreply.github.com>
@rhr323 rhr323 removed the >enhancement Enhancement of existing functionality label Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants