Skip to content

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Mar 14, 2025

Relates to elastic/elasticsearch#124798
.. in particular elastic/elasticsearch#124798 (comment)

This PR adds new sections to the Unassigned shards troubleshooting page based on content derived from https://www.elastic.co/guide/en/elasticsearch/reference/8.18/cluster-allocation-explain.html#cluster-allocation-explain-api-examples. Two of those sections are linked from the Elasticsearch logs, so I've added custom anchors for increased link permanency

We can continue to copy in additional examples in subsequent PRs if we need to close this quickly for linking purposes.

@leemthompo
Copy link
Contributor

We can continue to copy in additional examples in subsequent PRs if we need to close this quickly for linking purposes.

Thanks @lcawl! I guess the most important thing is to ensure we honor the contract of whatever we commit in elastic/elasticsearch#124798. We need to merge that EOD tomorrow, but we also have time to update the actual pages afterwards

Copy link
Contributor

@leemthompo leemthompo 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

@marciw marciw left a comment

Choose a reason for hiding this comment

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

guess this is still a draft but +1 approving just in case 🙂

@lcawl lcawl marked this pull request as ready for review March 18, 2025 00:51
@lcawl lcawl enabled auto-merge (squash) March 18, 2025 00:52
@lcawl lcawl merged commit 316b747 into elastic:main Mar 18, 2025
3 of 4 checks passed
@lcawl lcawl deleted the cluster-explain branch March 18, 2025 01:08
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.

3 participants