Skip to content

Conversation

pquentin
Copy link
Member

@pquentin pquentin requested review from gmjehovich and a team September 30, 2025 07:05
@pquentin pquentin added the skip-backport This pull request should not be backported label Sep 30, 2025
Copy link
Contributor

github-actions bot commented Sep 30, 2025

Following you can find the validation changes against the target branch for the API.

API Status Request Response
ilm.explain_lifecycle 🔴 → 🟢 5/5 4/5 → 5/5

You can validate this API yourself by using the make validate target.

Copy link
Contributor

@nielsbauman nielsbauman left a comment

Choose a reason for hiding this comment

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

LGTM with one small comment regarding availability. Thanks!

age?: Duration
/**
* @availability stack since=9.2.0 stability=stable
* @availability serverless stability=stable
Copy link
Contributor

Choose a reason for hiding this comment

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

ILM is not available as a whole in serverless. Not sure what value this should have, but it shouldn't be serverless.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, of course, thanks! I fixed it because it was wrong. Thankfully, it never made it to the serverless OpenAPI file.

@pquentin pquentin merged commit 48e4926 into main Sep 30, 2025
8 checks passed
@pquentin pquentin deleted the fix-ilm-explain-lifecycle branch September 30, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-backport This pull request should not be backported specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants