Skip to content

[ML] AI Connector: Adds ability to filter providers by solution type#228116

Merged
alvarezmelissa87 merged 15 commits intoelastic:mainfrom
alvarezmelissa87:ml-ai-connector-filter-providers-by-project
Jul 28, 2025
Merged

[ML] AI Connector: Adds ability to filter providers by solution type#228116
alvarezmelissa87 merged 15 commits intoelastic:mainfrom
alvarezmelissa87:ml-ai-connector-filter-providers-by-project

Conversation

@alvarezmelissa87
Copy link
Contributor

@alvarezmelissa87 alvarezmelissa87 commented Jul 16, 2025 •
edited
Loading

Summary

Related issue: #218499

This PR adds filters for each of the project types so the list of providers can be filtered by solution/project type.
The filter will be enabled by default for the current solution/project type.

The filter will only be shown when user is in applicable space solution or in applicable serverless project. If the user is in classic mode, no filter will be shown.

For Observability solution in space:

image

NOTE:

The EuiSelectable accepts searchProps with an append property that appends nodes to the end of the search bar but it always had that odd gray space in between as shown below. That's why I decided not to use that prop. Adding the filters after the search bar directly as children of the component looked a lot better (screenshot above).

image

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.
@alvarezmelissa87 alvarezmelissa87 self-assigned this Jul 16, 2025
@alvarezmelissa87 alvarezmelissa87 added release_note:enhancement :ml backport:version Backport to applied version labels Feature:Inference UI ML Inference endpoints UI and AI connector v9.2.0 labels Jul 16, 2025
@alvarezmelissa87 alvarezmelissa87 requested a review from jcger July 16, 2025 01:15
@alvarezmelissa87 alvarezmelissa87 marked this pull request as ready for review July 17, 2025 18:57
@alvarezmelissa87 alvarezmelissa87 requested review from a team as code owners July 17, 2025 18:57
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 17, 2025

Pinging @elastic/ml-ui (:ml)
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 18, 2025

@elasticmachine merge upstream
Samiul-TheSoccerFan
Samiul-TheSoccerFan approved these changes Jul 18, 2025
View reviewed changes
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.

kbn-inference-endpoint-ui-common changes look good.
This was referenced Jul 15, 2025
Closed
Closed
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 21, 2025

@elasticmachine merge upstream
qn895
qn895 approved these changes Jul 23, 2025
View reviewed changes
Copy link
Member

@qn895 qn895 left a comment

Choose a reason for hiding this comment

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

Code LGTM 🎉
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 24, 2025

@elasticmachine merge upstream
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 24, 2025

merge conflict between base and head
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 24, 2025

/ci
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 24, 2025

@elasticmachine merge upstream
@alvarezmelissa87
Copy link
Contributor Author

alvarezmelissa87 commented Jul 25, 2025

@elasticmachine merge upstream
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 28, 2025

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
indexManagement 670 674 +4
searchInferenceEndpoints 142 146 +4
searchPlayground 395 399 +4
securitySolution 7688 7692 +4
stackConnectors 334 338 +4
total +20

Async chunks

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

id before after diff
indexManagement 679.9KB 681.0KB +1.1KB
searchInferenceEndpoints 97.6KB 98.7KB +1.1KB
searchPlayground 221.0KB 221.2KB +118.0B
securitySolution 9.8MB 9.8MB +118.0B
stackConnectors 618.3KB 620.0KB +1.7KB
total +4.1KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
stackConnectors 64.2KB 64.3KB +80.0B
triggersActionsUi 105.7KB 105.7KB +14.0B
total +94.0B

History

cc @alvarezmelissa87
@alvarezmelissa87 alvarezmelissa87 merged commit b0bb040 into elastic:main Jul 28, 2025
12 checks passed
@alvarezmelissa87 alvarezmelissa87 deleted the ml-ai-connector-filter-providers-by-project branch July 28, 2025 23:13
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 30, 2025
@kibanamachine
Copy link
Contributor

kibanamachine commented Jul 30, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 228116 locally
cc: @alvarezmelissa87
2 similar comments
@kibanamachine
Copy link
Contributor

kibanamachine commented Aug 1, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 228116 locally
cc: @alvarezmelissa87
@kibanamachine
Copy link
Contributor

kibanamachine commented Aug 4, 2025

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 228116 locally
cc: @alvarezmelissa87
@alvarezmelissa87 alvarezmelissa87 added backport:skip This PR does not require backporting and removed backport:version Backport to applied version labels labels Aug 4, 2025
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Aug 4, 2025
delanni pushed a commit to delanni/kibana that referenced this pull request Aug 5, 2025
@alvarezmelissa87 @kibanamachine
@elasticmachine @delanni
[ML] AI Connector: adds ability to filter providers by solution type (e…
ef9f66e 
…lastic#228116)

## Summary

Related issue:
elastic#218499

This PR adds filters for each of the project types so the list of
providers can be filtered by solution/project type.
The filter will be enabled by default for the current solution/project
type.

The filter will only be shown when user is in applicable space solution
or in applicable serverless project. If the user is in classic mode, no
filter will be shown.

For Observability solution in space:

<img width="944" height="829" alt="image"
src="https://github.com/user-attachments/assets/d5a29cf0-267a-4cf6-b290-835bff2dd908"
/>


## NOTE:

The `EuiSelectable` accepts `searchProps` with an `append` property that
appends nodes to the end of the search bar but it always had that odd
gray space in between as shown below. That's why I decided not to use
that prop. Adding the filters after the search bar directly as children
of the component looked a lot better (screenshot above).

<img width="916" height="315" alt="image"
src="https://github.com/user-attachments/assets/0642cb0b-9a06-40c2-97a1-d4d9007b1b9c"
/>


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

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@peteharverson peteharverson changed the title [ML] AI Connector: adds ability to filter providers by solution type Sep 24, 2025
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 v9.2.0

9 participants

@alvarezmelissa87 @elasticmachine @kibanamachine @spong @afharo @jcger @viduni94 @qn895 @Samiul-TheSoccerFan