Skip to content
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

Updated ilm docs as per the issue #118148

Merged
merged 9 commits into from
Dec 17, 2024

Conversation

satyyam11
Copy link
Contributor

This PR updates the Elasticsearch ILM tutorial for the newer screenshot and the primary shard update in the text.

Copy link
Contributor

github-actions bot commented Dec 6, 2024

Documentation preview:

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Dec 6, 2024
@leemthompo leemthompo added the >docs General docs changes label Dec 11, 2024
@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Dec 11, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@leemthompo
Copy link
Contributor

@elasticmachine test this

@leemthompo
Copy link
Contributor

run docs-build

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@satyyam11
Copy link
Contributor Author

hi can I do anything?

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.

Thanks! This LGTM. @dakrone you happy with this?

@leemthompo leemthompo added the auto-backport Automatically create backport pull requests when merged label Dec 13, 2024
@leemthompo
Copy link
Contributor

@elasticmachine test this

@leemthompo
Copy link
Contributor

run docs-build

@leemthompo
Copy link
Contributor

@elasticmachine test this

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 also, thanks!

@satyyam11
Copy link
Contributor Author

Will it self merge or do I need to do something?
Also can I work on more issues?
I know Java and Springboot

@dakrone dakrone added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Dec 16, 2024
@dakrone
Copy link
Member

dakrone commented Dec 17, 2024

buildkite test this

@dakrone
Copy link
Member

dakrone commented Dec 17, 2024

@elasticmachine update branch

@dakrone
Copy link
Member

dakrone commented Dec 17, 2024

Will it self merge or do I need to do something?

Don't worry about this, I'll take care of merging it.

Also can I work on more issues? I know Java and Springboot

We don't use spring, but you're welcome to work on other issues!

@dakrone
Copy link
Member

dakrone commented Dec 17, 2024

buildkite test this

@elasticsearchmachine elasticsearchmachine merged commit 41c3dde into elastic:main Dec 17, 2024
7 checks passed
dakrone pushed a commit to dakrone/elasticsearch that referenced this pull request Dec 17, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.
dakrone pushed a commit to dakrone/elasticsearch that referenced this pull request Dec 17, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.16
8.17
8.x

dakrone pushed a commit to dakrone/elasticsearch that referenced this pull request Dec 17, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.
elasticsearchmachine pushed a commit that referenced this pull request Dec 17, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.

Co-authored-by: Satyam Mishra <satyyam.misshra1121@gmail.com>
elasticsearchmachine pushed a commit that referenced this pull request Dec 17, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.

Co-authored-by: Satyam Mishra <satyyam.misshra1121@gmail.com>
elasticsearchmachine pushed a commit that referenced this pull request Dec 18, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.

Co-authored-by: Satyam Mishra <satyyam.misshra1121@gmail.com>
rjernst pushed a commit to rjernst/elasticsearch that referenced this pull request Dec 18, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.
navarone-feekery pushed a commit to navarone-feekery/elasticsearch that referenced this pull request Dec 26, 2024
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.
sarog pushed a commit to sarog/elasticsearch that referenced this pull request Jan 22, 2025
This PR updates the Elasticsearch ILM tutorial for the newer screenshot
and the primary shard update in the text.

Co-authored-by: Satyam Mishra <satyyam.misshra1121@gmail.com>
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 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team v8.16.0 v8.17.0 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants