-
Notifications
You must be signed in to change notification settings - Fork 115
Drop project_routing from query parameters
#5835
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
base: main
Are you sure you want to change the base?
Conversation
|
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 |
quux00
left a comment
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.
Left a few suggestions.
| }, | ||
| "async_search.submit": { | ||
| "request": [ | ||
| "Request: missing json spec query parameter 'project_routing'" |
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.
I don't understand what this file is reporting here.
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.
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 |
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.
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?
ES-13725: We previously assumed that
project_routingcould 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.