Skip to content

Comments

[8.17] [Inference API] Fix model field in OpenAI Upgrade IT (#119362)#119371

Merged
elasticsearchmachine merged 1 commit intoelastic:8.17from
maxhniebergall:backport/8.17/pr-119362
Dec 30, 2024
Merged

[8.17] [Inference API] Fix model field in OpenAI Upgrade IT (#119362)#119371
elasticsearchmachine merged 1 commit intoelastic:8.17from
maxhniebergall:backport/8.17/pr-119362

Conversation

@maxhniebergall
Copy link
Contributor

Backports the following commits to 8.17:

* Fix NPE by getting request once and defaulting to empty list

* Fix error in OpenAIUpgrade test which used the wrong field name for certain versions

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
@maxhniebergall maxhniebergall added :ml Machine learning >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:ML Meta label for the ML team labels Dec 30, 2024
@elasticsearchmachine elasticsearchmachine merged commit df39798 into elastic:8.17 Dec 30, 2024
@maxhniebergall maxhniebergall deleted the backport/8.17/pr-119362 branch December 30, 2024 20:43
sarog pushed a commit to portsbuild/elasticsearch that referenced this pull request Jan 22, 2025
…elastic#119371)

* Fix NPE by getting request once and defaulting to empty list

* Fix error in OpenAIUpgrade test which used the wrong field name for certain versions

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :ml Machine learning Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests v8.17.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants