-
Notifications
You must be signed in to change notification settings - Fork 25k
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
Add missing timeouts to rest-api-spec ILM APIs #118837
Conversation
Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination) |
Pinging @elastic/es-data-management (Team:Data Management) |
Hi @pquentin, I've created a changelog YAML for you. |
docs/changelog/118837.yaml
Outdated
@@ -0,0 +1,5 @@ | |||
pr: 118837 | |||
summary: Add missing timeouts to rest-api-spec ILM APIs | |||
area: "Cluster Coordination" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me but these omissions pre-date #109112 and definitely belong to ILM rather than Cluster Coordination
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with David's comment about changing the area in the changelog
I have noticed those when reviewing elastic/elasticsearch-specification#3297. I tagged Data Management because those are ILM APIs but also Cluster Coordination as @DaveCTurner worked on those timeouts this year.