Skip to content

Conversation

jonathan-buttner
Copy link
Contributor

This PR adds a new optional field headers to the openai integration corresponding to this PR: elastic/elasticsearch#134960

@jonathan-buttner jonathan-buttner added specification skip-backport This pull request should not be backported labels Oct 15, 2025
Copy link
Contributor

github-actions bot commented Oct 15, 2025

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

API Status Request Response
get 🟢 317/317 → 321/321 314/314 → 318/318
index 🟢 1440/1440 → 1443/1443 1442/1442 → 1445/1445
indices.create 🔴 1373/1397 → 1385/1409 1397/1397 → 1409/1409
indices.get_mapping 🔴 198/198 → 202/202 187/198 → 191/202
indices.refresh 🟢 329/329 → 327/327 329/329 → 327/327
ml.get_job_stats 🟢 30/30 → 29/29 30/30 → 29/29
ml.put_job 🟢 65/65 → 64/64 65/65 → 64/64
search 🔴 2584/2611 → 2585/2612 2611/2611 → 2612/2612

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

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

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

LGTM

@jonathan-buttner jonathan-buttner merged commit f9b2639 into main Oct 20, 2025
8 checks passed
@jonathan-buttner jonathan-buttner deleted the ml-openai-headers branch October 20, 2025 14:09
@jonathan-buttner jonathan-buttner added backport 9.2 and removed skip-backport This pull request should not be backported labels Oct 20, 2025
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
* Adding headers for openai

* Update specification/inference/_types/CommonTypes.ts

Co-authored-by: David Kyle <[email protected]>

---------

Co-authored-by: David Kyle <[email protected]>
(cherry picked from commit f9b2639)
jonathan-buttner added a commit that referenced this pull request Oct 20, 2025
* Adding headers for openai

* Update specification/inference/_types/CommonTypes.ts

Co-authored-by: David Kyle <[email protected]>

---------

Co-authored-by: David Kyle <[email protected]>
(cherry picked from commit f9b2639)
jonathan-buttner added a commit that referenced this pull request Oct 20, 2025
* Adding headers for openai

* Update specification/inference/_types/CommonTypes.ts



---------


(cherry picked from commit f9b2639)

Co-authored-by: David Kyle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants