Skip to content

[8.18] [Security Solution] Onboarding check inference endpoint (#217150)#217335

Merged
semd merged 1 commit intoelastic:8.18from
semd:backport/8.18/pr-217150
Apr 7, 2025
Merged

[8.18] [Security Solution] Onboarding check inference endpoint (#217150)#217335
semd merged 1 commit intoelastic:8.18from
semd:backport/8.18/pr-217150

Conversation

@semd
Copy link
Contributor

@semd semd commented Apr 7, 2025

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

## Summary

This PR checks that the inference endpoint exists before showing the
inference connector in the list of selectable connectors.

Also, it removes code duplication by centralizing the implementation in
the /common/connectors directory of the onboarding cards

![siem_migrations_connector_card](https://github.com/user-attachments/assets/95cb0a3b-e55b-4626-be80-08520bd58ea4)

![assistant_connector_card](https://github.com/user-attachments/assets/99ddd69d-18c7-41be-bec0-45524d718d08)

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 7b934f9)

# Conflicts:
#	x-pack/solutions/security/plugins/security_solution/tsconfig.json
@semd semd requested a review from kibanamachine as a code owner April 7, 2025 14:04
@semd semd added the backport This PR is a backport of another PR label Apr 7, 2025
@semd semd enabled auto-merge (squash) April 7, 2025 14:04
@semd semd merged commit dd7dae0 into elastic:8.18 Apr 7, 2025
11 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 7029 7030 +1

Async chunks

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

id before after diff
securitySolution 9.1MB 9.1MB -98.0B

Page load bundle

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

id before after diff
securitySolution 86.0KB 85.9KB -142.0B

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants