Skip to content

Add link to Max Shards Per Node exception message#110993

Merged
DaveCTurner merged 9 commits intoelastic:mainfrom
stefnestor:stef_errorToDoc_maxShards
Jul 22, 2024
Merged

Add link to Max Shards Per Node exception message#110993
DaveCTurner merged 9 commits intoelastic:mainfrom
stefnestor:stef_errorToDoc_maxShards

Conversation

@stefnestor
Copy link
Copy Markdown
Member

@stefnestor stefnestor commented Jul 17, 2024

👋🏽 howdy, team!

Trying to follow #105607 #110911, this links common error like ... to the related doc which is common support volume

Elasticsearch exception [type=validation_exception, reason=Validation Failed: 1: this action would add [2] total shards, but this cluster currently has [1000]/[1000] maximum shards open;]

Validation Failed: 1: this action would add [x] shards, but this cluster currently has [y]/[y] maximum shards open | maximum normal shards open

@stefnestor stefnestor added >enhancement :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. Team:Distributed Meta label for distributed team. labels Jul 17, 2024
@elasticsearchmachine elasticsearchmachine added v8.16.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Jul 17, 2024
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@elasticsearchmachine

This comment was marked as resolved.

Comment thread server/src/main/resources/org/elasticsearch/common/reference-docs-links.json Outdated
Comment thread docs/changelog/110993.yaml Outdated
@@ -0,0 +1,5 @@
pr: 110993
summary: (Err>Doc) Max Shards Per Node
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This summary will appear in the release notes - could you edit it (and ideally the PR title) to make it more release-notes-worthy?

@stefnestor stefnestor changed the title (Err>Doc) Max Shards Per Node Add link to Max Shards Per Node exception message Jul 18, 2024
@stefnestor stefnestor requested review from a team as code owners July 22, 2024 12:18
@stefnestor stefnestor force-pushed the stef_errorToDoc_maxShards branch from b7244ad to e40040b Compare July 22, 2024 14:16
Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner merged commit cc245c4 into elastic:main Jul 22, 2024
salvatore-campagna pushed a commit to salvatore-campagna/elasticsearch that referenced this pull request Jul 23, 2024
stefnestor added a commit that referenced this pull request Oct 11, 2024
👋 It appears we have two docs of similar content about max open shards. This one contains the error users search (so is what we linked the error to in #110993) but the other I believe is a placeholder doc for the health api code. Should maybe consolidate some day but in the mean time at least cross-link.
stefnestor added a commit that referenced this pull request Oct 16, 2024
* (Doc+) Cross-link max shards

👋 It appears we have two docs of similar content about max open shards. This one contains the error users search (so is what we linked the error to in #110993) but the other I believe is a placeholder doc for the health api code. Should maybe consolidate some day but in the mean time at least cross-link.

---------

Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com>
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this pull request Oct 25, 2024
* (Doc+) Cross-link max shards

👋 It appears we have two docs of similar content about max open shards. This one contains the error users search (so is what we linked the error to in elastic#110993) but the other I believe is a placeholder doc for the health api code. Should maybe consolidate some day but in the mean time at least cross-link.

---------

Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com>
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Nov 4, 2024
* (Doc+) Cross-link max shards

👋 It appears we have two docs of similar content about max open shards. This one contains the error users search (so is what we linked the error to in elastic#110993) but the other I believe is a placeholder doc for the health api code. Should maybe consolidate some day but in the mean time at least cross-link.

---------

Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com>
@stefnestor stefnestor deleted the stef_errorToDoc_maxShards branch November 15, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >enhancement external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Distributed Meta label for distributed team. v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants