Skip to content

Conversation

@pawankartik-elastic
Copy link
Contributor

ES-13725: We previously assumed that project_routing could appear either as a query parameter or within the request body. Now that we restrict it to just the request body, this PR makes the suitable adjustments within the specs.

@pawankartik-elastic pawankartik-elastic added specification skip-backport This pull request should not be backported labels Dec 11, 2025
@github-actions
Copy link
Contributor

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

No changes detected.

You can validate these APIs yourself by using the make validate target.

@pawankartik-elastic pawankartik-elastic marked this pull request as ready for review December 11, 2025 17:53
Copy link
Contributor

@quux00 quux00 left a comment

Choose a reason for hiding this comment

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

Left a few suggestions.

},
"async_search.submit": {
"request": [
"Request: missing json spec query parameter 'project_routing'"
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand what this file is reporting here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is exactly why I asked you what commands you ran. I see similar changes in your PR as well, and I wasn't really sure what's all this about.

* Supported in serverless only.
* @availability serverless stability=stable visibility=feature_flag feature_flag=serverless.cross_project.enabled
*/
project_routing?: ProjectRouting
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see project_routing in the body for EQL or SQL, but I'm pretty sure support for them was added in the code. @luigidellaquila is that right?

If yes, @pawankartik-elastic, can you add those here?

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.

3 participants