-
Notifications
You must be signed in to change notification settings - Fork 196
Update the Connect to a remote cluster page #4682
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
Conversation
✅ Vale Linting ResultsNo issues found on modified lines! The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale. |
🔍 Preview links for changed docs |
…nding on their deployment type
shainaraskas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ultimately, I think that this ccr tutorial should just be one page, so this specific fix is not ideal in the long term. however, it fixes the root cause of the issue and is good enough for me.
some link syntax suggestions, and some suggestions on the remaining content scope. approving to unblock but you should edit a bit
deploy-manage/tools/cross-cluster-replication/_connect_to_a_remote_cluster.md
Outdated
Show resolved
Hide resolved
deploy-manage/tools/cross-cluster-replication/_connect_to_a_remote_cluster.md
Outdated
Show resolved
Hide resolved
eedugon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should update the first paragraph of the connect_to_a_remote_cluster doc as mentioned in one of the threads.
The reason is to ensure the reader understands remote clusters is a separate feature (than CCR) and needs to be configured first, before being able to replicate indices, etc. The way the sentence looks now feels ambiguous.
deploy-manage/tools/cross-cluster-replication/_connect_to_a_remote_cluster.md
Outdated
Show resolved
Hide resolved
deploy-manage/tools/cross-cluster-replication/_connect_to_a_remote_cluster.md
Outdated
Show resolved
Hide resolved
eedugon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks much better! Small comment to ensure local & remote, source & destination do not cause misunderstandings :)
Summary
Fixes #3473
These changes update the purpose of the page from what used to be a self-managed guide to a landing page that instead links out to instructions for setting up remote clusters for different deployment types.
A preview of the updated page
Generative AI disclosure