Add HLRC docs for Explain LIfecycle#35803
Conversation
Adds HLRC documentation for the Explain Lifecycle API.
|
Pinging @elastic/es-core-features |
talevy
left a comment
There was a problem hiding this comment.
Looks great! just had one suggestion
| assertNotNull(response); | ||
|
|
||
| // tag::ilm-explain-lifecycle-response | ||
| Map<String, IndexLifecycleExplainResponse> indices = |
There was a problem hiding this comment.
we should add the example response for otherIndex here too
There was a problem hiding this comment.
I'm not sure what would be different in terms of what we show here - it will just have different values (isManaged = false, everything else is null), which we don't show in the docs. Do you just mean we should add indices.get("other_index") to show that you can do it?
There was a problem hiding this comment.
I was thinking of adding anchored comments discussing how things will be null or something. Maybe adding it in the description?
There was a problem hiding this comment.
Yeah, with adding it to the description makes more sense to me - I'll also add some asserts to that effect.
Adds HLRC documentation for the Explain Lifecycle API.
Adds HLRC documentation for the Explain Lifecycle API.