Issue in elasticsearch-serverless api doc. - operation-indices-rollover

[nehasaini1608]

[Bug Report] Incorrect Rollover API Examples and Recommendations for Elasticsearch Serverless
https://www.elastic.co/docs/api/doc/elasticsearch-serverless/operation/operation-indices-rollover

Description:
The current Elasticsearch Serverless Rollover API documentation contains examples and guidance that are not applicable in the Serverless context.

- Bug 1: Example uses unsupported rollover conditions

Issue:
The documentation provides an example using rollover conditions, such as:

POST my-data-stream/_rollover
{
"conditions": {
"max_age": "7d",
"max_docs": 1000,
"max_primary_shard_size": "50gb",
"max_primary_shard_docs": "2000"
}
}

However, when executed in Elasticsearch Serverless, this returns an error:
"rollover with conditions is not supported in serverless mode"

Suggested Fix:
Update the example to exclude conditions, as conditional rollover is not supported in Serverless.

- Bug2) Inaccurate Tip about ILM
ssue:
The page includes the tip:

"It is recommended to use the index lifecycle rollover action to automate rollovers."

However, Index Lifecycle Management (ILM) is not supported in Elasticsearch Serverless.

Suggested Fix:
Remove or revise this tip to avoid confusion for Serverless users.

[georgewallace]

@nehasaini1608 Thank you for bringing this to our attention. Currently we generate the serverless apis and the regular apis from the same source. We will update the source to ensure that it calls out the differences and fix the example.

[ketkee-aryamane]

Requested Neha to let us know if it's fine that users send an empty POST after we remove the conditions object in the request sample for serverless.
cc: @szabosteve @georgewallace

[szabosteve]

FYI, I think you cannot add an example without a body to the examples pool.

[georgewallace]

Since the spec builds out both stateful and server less together. We will need to update the description of the operation to call out the nuance about it not being available in serverless. We can make those updates once you hear back.

[szabosteve]

You can also add a comment to the existing example that the example body is not applicable to Serverless or something similar. On top of the callout that George suggested.

[ketkee-aryamane]

Since the spec builds out both stateful and server less together. We will need to update the description of the operation to call out the nuance about it not being available in serverless. We can make those updates once you hear back.

Right @georgewallace , I wasn't going to get rid of that block, but when we call out stating one can't send the conditions block for serverless, it would mean that the POST bodywould be empty, hence the question.

[ketkee-aryamane]

You can also add a comment to the existing example that the example body is not applicable to Serverless or something similar. On top of the callout that George suggested.

yes, absolutely @szabosteve

[ketkee-aryamane]

@georgewallace , Neha got back saying the empty POST works for her. But she is unsure whether it's fine to keep it empty. And Istvan confirmed that we can't add an example without a body to the examples pool. Could I check this with any developer to be sure?

[georgewallace]

We can check with the dev tools team this week