Skip to content

[DOCS] Add CCR docs to the Elasticsearch Reference Guide#45704

Merged
jrodewig merged 9 commits intoelastic:masterfrom
jrodewig:add-ccr-to-es-ref
Sep 17, 2019
Merged

[DOCS] Add CCR docs to the Elasticsearch Reference Guide#45704
jrodewig merged 9 commits intoelastic:masterfrom
jrodewig:add-ccr-to-es-ref

Conversation

@jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Aug 19, 2019

Changes

  • Changes "Administering Elasticsearch" to "Set up a cluster for high availability," including renaming and moving several related files.
  • Moves a several intro paragraphs from "Back up a cluster's data" to "Set up a cluster for high availability."
  • Include cross-cluster replication (CCR) docs in "Set up a cluster for high availability."
    Previously, cross-cluster replication (CCR) documentation was located only in
    the Stack Overview. This inclusion requires level-offsets, which will be removed in a later PR.
  • Adds a redirect for "Administering Elasticsearch."

Relates to #45703

Preview

http://elasticsearch_45704.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/xpack-ccr.html

@jrodewig jrodewig added >docs General docs changes :Distributed/CCR Issues around the Cross Cluster State Replication features v8.0.0 v7.3.0 v7.4.0 labels Aug 19, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@jrodewig jrodewig requested a review from jasontedor August 19, 2019 16:01
Previously, cross-cluster replication (CCR) documentation was located in
the Stack Overview:
https://www.elastic.co/guide/en/elastic-stack-overview/master/xpack-ccr.html

This adds CCR documentation to the Elasticsearch Reference Guide with a
level offset for headings.

The level offset and CCR Stack Overview docs will be removed in later
commits.
@lcawl
Copy link
Contributor

lcawl commented Aug 26, 2019

This PR builds successfully for me. I was a little surprised, however, at the location chosen for this content. I thought it might be closer (in location and title) to the CCS content (https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-cross-cluster-search.html).

@jrodewig
Copy link
Contributor Author

jrodewig commented Aug 26, 2019

I chose Administering Elasticsearch because that section seemed more relevant for common use cases:

  • Disaster recovery in case a primary cluster fails. A secondary cluster can serve as a hot backup
  • Geo-proximity so that reads can be served locally

Aside from both requiring remote clusters, the concepts don't seem particularly dependent on or tied to each other.

That said, I'm happy to relocate and open to any suggestions. It's better to move once than twice.

@lcawl
Copy link
Contributor

lcawl commented Aug 26, 2019

I'm not up to speed on all the Elasticsearch content reorganization plans, I just thought since there's a "Search across clusters" at the top level, it would be nice to have something like "Replicate across clusters" too. As a plus, that way you don't lose the sub-pages under the CCR "Overview".

I'm fine with whatever you decide, however.

@jrodewig
Copy link
Contributor Author

jrodewig commented Aug 26, 2019

d883beb relocates the CCR chapter to the top level, following "Search across clusters"

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

We need to be consistent about where we put admin topics--having some under "Administration" and some at the top level is confusing. CCS has both an admin configuration element and a search usage element, where CCR is more purely an admin topic. If we opt to bump the admin topics to the top level, I think we need to group them. Putting CCR in between the Query DSL and Scripting and Mapping sections doesn't seem like the right place. One option might be to split the info on how to write a query to search multiple clusters from how to set up remotes and enable cross-cluster search and cross-cluster replication.

@jrodewig
Copy link
Contributor Author

jrodewig commented Aug 27, 2019

We already document remote clusters separately: https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-remote-clusters.html

The only admin piece of CCS is a very brief setup example here:
https://www.elastic.co/guide/en/elasticsearch/reference/master/modules-cross-cluster-search.html#ccs-remote-cluster-setup

I can remove that setup if wanted, but I think it helps give the user a start-to-finish CCS setup.
We refer to a few other settings in the CCS docs, but those are directly related to search, not admin.

@jrodewig jrodewig added the WIP label Aug 27, 2019
@jrodewig
Copy link
Contributor Author

Spoke to @debadair. I'm going to hold off on merging this and any other architecture-related PRs pending some more team discussion.

Adding a WIP label to indicate that.

jrodewig added a commit to elastic/beats that referenced this pull request Sep 18, 2019
elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
jrodewig added a commit to elastic/beats that referenced this pull request Sep 18, 2019
…3729)

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
jrodewig added a commit to elastic/beats that referenced this pull request Sep 20, 2019
…3730)

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
lcawl pushed a commit to lcawl/elasticsearch that referenced this pull request Dec 6, 2019
@lcawl lcawl added the v7.2.2 label Dec 6, 2019
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…) (elastic#13729)

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…) (elastic#13730)

elastic/elasticsearch#45704 and elastic/stack-docs#464 moves the
cross-cluster replication (CCR) docs from the Elastic Stack Overview to
the Elasticsearch Reference Guide.

This updates any links pointing to the old Elastic Stack Overview docs.
We've set up redirects so these links won't fail in the meantime. This
just removes a step for users.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/CCR Issues around the Cross Cluster State Replication features >docs General docs changes v7.2.2 v7.3.0 v7.5.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants