Skip to content

[ML] Inference endpoint creation: ensure header is handled#242300

Merged
alvarezmelissa87 merged 1 commit intoelastic:mainfrom
alvarezmelissa87:inference-endpoint-creation-fix-unhandled-headers
Nov 7, 2025
Merged

[ML] Inference endpoint creation: ensure header is handled#242300
alvarezmelissa87 merged 1 commit intoelastic:mainfrom
alvarezmelissa87:inference-endpoint-creation-fix-unhandled-headers

Conversation

@alvarezmelissa87
Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 commented Nov 7, 2025

Summary

This PR handles header field for inference endpoint creation until full support is added here #242187

Before:

Screen.Recording.2025-11-07.at.9.15.56.AM.mov

After:

Screen.Recording.2025-11-07.at.9.14.05.AM.mov

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@alvarezmelissa87 alvarezmelissa87 added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Nov 7, 2025
Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Samiul-TheSoccerFan Samiul-TheSoccerFan left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #67 / Alerting builtin alertTypes es_query rule runs correctly: threshold on ungrouped hit count < >

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
indexManagement 704.0KB 704.1KB +94.0B
searchInferenceEndpoints 119.8KB 119.9KB +91.0B
stackConnectors 733.6KB 733.7KB +92.0B
total +277.0B

cc @alvarezmelissa87

@alvarezmelissa87 alvarezmelissa87 merged commit 1dd8a00 into elastic:main Nov 7, 2025
23 checks passed
@alvarezmelissa87 alvarezmelissa87 deleted the inference-endpoint-creation-fix-unhandled-headers branch November 7, 2025 18:17
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…42300)

## Summary

This PR handles header field for inference endpoint creation until full
support is added here elastic#242187

Before:


https://github.com/user-attachments/assets/b3d41704-8240-4f4b-bc0b-f802efa794ae


After:


https://github.com/user-attachments/assets/19ac1a7f-b1d2-433b-84d9-c22b10765119


### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [ ] If a plugin configuration key changed, check if it needs to be
allowlisted in the cloud and added to the [docker
list](https://github.com/elastic/kibana/blob/main/src/dev/build/tasks/os_packages/docker_generator/resources/base/bin/kibana-docker)
- [ ] This was checked for breaking HTTP API changes, and any breaking
changes have been approved by the breaking-change committee. The
`release_note:breaking` label should be applied in these situations.
- [ ] [Flaky Test
Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was
used on any tests changed
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting :ml release_note:skip Skip the PR/issue when compiling release notes v9.3.0

4 participants