Skip to content

Add docs for index component selector api convention#128786

Merged
jbaiera merged 6 commits intoelastic:mainfrom
jbaiera:docs-index-selector
Jun 26, 2025
Merged

Add docs for index component selector api convention#128786
jbaiera merged 6 commits intoelastic:mainfrom
jbaiera:docs-index-selector

Conversation

@jbaiera
Copy link
Member

@jbaiera jbaiera commented Jun 2, 2025

Adds documentation for index component selectors to the reference docs.

This documentation references and will be referenced by new docs located at elastic/docs-content#1368
@jbaiera jbaiera added >docs General docs changes :StorageEngine/Data streams Data streams and their lifecycles v8.19.0 v9.1.0 labels Jun 2, 2025
@elasticsearchmachine elasticsearchmachine added Team:Data Management (obsolete) DO NOT USE. This team no longer exists. Team:Docs Meta label for docs team labels Jun 2, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Jun 2, 2025

Pinging @elastic/es-docs (Team:Docs)
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Jun 2, 2025

Pinging @elastic/es-data-management (Team:Data Management)
@jbaiera jbaiera requested review from dakrone and kilfoyle June 2, 2025 21:28
@jbaiera jbaiera mentioned this pull request Jun 2, 2025
Merged
dakrone
dakrone approved these changes Jun 2, 2025
View reviewed changes
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

The general shape and structure LGTM. I left some suggestions, but this is a hard topic to talk about with simplicity, so I wouldn't take any of them as dogmatic.
kilfoyle
kilfoyle reviewed Jun 3, 2025
View reviewed changes
kilfoyle
kilfoyle reviewed Jun 3, 2025
View reviewed changes
kilfoyle
kilfoyle approved these changes Jun 3, 2025
View reviewed changes
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! 🚀
Added just a few small suggestions.
@jbaiera @kilfoyle
Update docs/reference/elasticsearch/rest-apis/api-conventions.md
9b905de 
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jun 25, 2025 •
edited
Loading

🔍 Preview links for changed docs:

🔔 The preview site may take up to 3 minutes to finish building. These links will become live once it completes.
@jbaiera @kilfoyle
Update docs/reference/elasticsearch/rest-apis/api-conventions.md
758b78d 
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
dakrone
dakrone approved these changes Jun 26, 2025
View reviewed changes
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM still, thanks Jimmy!
@jbaiera jbaiera merged commit e54cd63 into elastic:main Jun 26, 2025
8 of 9 checks passed
@jbaiera jbaiera deleted the docs-index-selector branch June 26, 2025 20:47
@reakaleek
Copy link
Member

reakaleek commented Jun 26, 2025 •
edited
Loading

The docs-build is failing on main because docs-content://manage-data/data-store/data-streams/failure-store.md does not exit yet.
@jbaiera
Copy link
Member Author

jbaiera commented Jun 26, 2025

Sorry! This PR was tied up with elastic/docs-content#1368 and needed both to be merged to work. Looks like both are now.
@reakaleek
Copy link
Member

reakaleek commented Jun 26, 2025

Np. @shainaraskas temporarily removed the crosslink in elastic/docs-content#1368 so that the page officially exists. Now the docs-build is green again. ✌️
smalyshev pushed a commit to smalyshev/elasticsearch that referenced this pull request Jun 27, 2025
@jbaiera @kilfoyle @smalyshev
Add docs for index component selector api convention (elastic#128786)
acdc25c 
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@georgewallace georgewallace added the auto-backport Automatically create backport pull requests when merged label Jul 1, 2025
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jul 3, 2025
@jbaiera @kilfoyle @mridula-s109
Add docs for index component selector api convention (elastic#128786)
e1d3629 
Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@shainaraskas shainaraskas mentioned this pull request Jul 22, 2025
Merged
This was referenced Aug 11, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged >docs General docs changes :StorageEngine/Data streams Data streams and their lifecycles Team:Data Management (obsolete) DO NOT USE. This team no longer exists. Team:Docs Meta label for docs team v8.19.0 v9.2.0

7 participants

@jbaiera @elasticsearchmachine @reakaleek @dakrone @kilfoyle @georgewallace @shainaraskas