Skip to content

[ML] AI Connector/Inference endpoints creation UI: Adds icon for Contextual AI#236951

Merged
KodeRad merged 7 commits intoelastic:mainfrom
KodeRad:ml-ai-connector-contextualai-icon-add
Oct 1, 2025
Merged

[ML] AI Connector/Inference endpoints creation UI: Adds icon for Contextual AI#236951
KodeRad merged 7 commits intoelastic:mainfrom
KodeRad:ml-ai-connector-contextualai-icon-add

Conversation

@KodeRad
Copy link
Contributor

@KodeRad KodeRad commented Sep 30, 2025

Summary

Implements #236743

Before:
Screenshot 2025-09-30 at 14 05 52

After:
Screenshot 2025-09-30 at 14 08 46

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.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.
@KodeRad KodeRad self-assigned this Sep 30, 2025
@KodeRad KodeRad added release_note:enhancement :ml backport:skip This PR does not require backporting Team:ML Team label for ML (also use :ml) t// Feature:Inference UI ML Inference endpoints UI and AI connector labels Sep 30, 2025
@KodeRad KodeRad marked this pull request as ready for review September 30, 2025 12:59
@KodeRad KodeRad requested a review from a team as a code owner September 30, 2025 12:59
@elasticmachine
Copy link
Contributor

elasticmachine commented Sep 30, 2025

Pinging @elastic/ml-ui (:ml)
@KodeRad KodeRad changed the title [ML] AI Connector/Inference endpoints creation UI: Adds icon forContextual AI Sep 30, 2025
@KodeRad KodeRad added the v9.2.0 label Sep 30, 2025
peteharverson
peteharverson approved these changes Sep 30, 2025
View reviewed changes
Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested and LGTM
alvarezmelissa87
alvarezmelissa87 approved these changes Sep 30, 2025
View reviewed changes
Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 left a comment

Choose a reason for hiding this comment

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

LGTM ⚡
@KodeRad KodeRad requested a review from a team as a code owner September 30, 2025 16:48
@KodeRad KodeRad requested a review from rbrtj October 1, 2025 10:30
rbrtj
rbrtj approved these changes Oct 1, 2025
View reviewed changes
Copy link
Contributor

@rbrtj rbrtj left a comment

Choose a reason for hiding this comment

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

Collector changes LGTM 👍
ddillinger
ddillinger approved these changes Oct 1, 2025
View reviewed changes
Copy link
Contributor

@ddillinger ddillinger left a comment

Choose a reason for hiding this comment

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

telemetry schema ✅
@KodeRad KodeRad enabled auto-merge (squash) October 1, 2025 14:38
@KodeRad KodeRad merged commit 875bfd8 into elastic:main Oct 1, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 1, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #10 / security APIs - Session Lifespan Session Lifespan cleanup should properly clean up session expired because of lifespan when providers override global session config

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
indexManagement 645 646 +1
searchInferenceEndpoints 152 153 +1
searchPlayground 417 418 +1
securitySolution 8287 8288 +1
stackConnectors 368 369 +1
total +5

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/inference-endpoint-ui-common 166 170 +4

Async chunks

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

id before after diff
indexManagement 687.9KB 689.6KB +1.7KB
searchInferenceEndpoints 113.4KB 115.1KB +1.7KB
searchPlayground 229.7KB 231.4KB +1.7KB
securitySolution 11.1MB 11.1MB +48.0B
stackConnectors 714.0KB 715.7KB +1.7KB
total +6.8KB
Unknown metric groups

API count

id before after diff
@kbn/inference-endpoint-ui-common 166 170 +4

History

cc @KodeRad
@peteharverson peteharverson mentioned this pull request Oct 6, 2025
Closed
rylnd pushed a commit to rylnd/kibana that referenced this pull request Oct 17, 2025
@KodeRad
[ML] AI Connector/Inference endpoints creation UI: Adds icon for Cont…
6985db5 
…extual AI (elastic#236951)

## Summary

Implements elastic#236743

Before: 
<img width="720" height="622" alt="Screenshot 2025-09-30 at 14 05 52"
src="https://github.com/user-attachments/assets/44d989d9-07de-4a5a-9417-5fc02ea89688"
/>

After:
<img width="718" height="584" alt="Screenshot 2025-09-30 at 14 08 46"
src="https://github.com/user-attachments/assets/941c1180-ecb3-4f2c-9f87-07f1b5257695"
/>


### 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.

### Identify risks

Does this PR introduce any risks? For example, consider risks like hard
to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified
risk. Invite stakeholders and evaluate how to proceed before merging.

- [ ] [See some risk
examples](https://github.com/elastic/kibana/blob/main/RISK_MATRIX.mdx)
- [ ] ...
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 Feature:Inference UI ML Inference endpoints UI and AI connector :ml release_note:enhancement Team:ML Team label for ML (also use :ml) t// v9.2.0

6 participants

@KodeRad @elasticmachine @ddillinger @alvarezmelissa87 @peteharverson @rbrtj