-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AI Connector] Migrates AI inference Connector to use a shared components from '@kbn/inference-endpoint-ui-common' #204885
[AI Connector] Migrates AI inference Connector to use a shared components from '@kbn/inference-endpoint-ui-common' #204885
Conversation
…tor through flyout
…ents from '@kbn/inference-endpoint-ui-common'
…hared-ui-components # Conflicts: # .github/CODEOWNERS # package.json # tsconfig.base.json # x-pack/.i18nrc.json # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/additional_options_fields.tsx # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/configuration/configuration_field.tsx # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/configuration/configuration_form_items.tsx # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/configuration/configuration_utils.ts # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/alibaba_cloud.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/amazon_bedrock.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/anthropic.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/azure_ai_studio.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/azure_open_ai.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/cohere.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/elastic.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/google_ai_studio.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/hugging_face.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/ibm_watsonx.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/mistral.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/open_ai.svg # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/render_service_provider/service_provider.test.tsx # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/render_service_provider/service_provider.tsx # x-pack/packages/kbn-inference-endpoint-ui-common/src/components/providers/selectable.tsx # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/alibaba_cloud.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/amazon_bedrock.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/anthropic.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/azure_ai_studio.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/azure_open_ai.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/cohere.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/elastic.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/google_ai_studio.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/hugging_face.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/ibm_watsonx.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/mistral.svg # x-pack/platform/packages/shared/kbn-inference-endpoint-ui-common/src/components/providers/assets/images/open_ai.svg # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/additional_options_fields.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/providers/get_providers.test.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/providers/render_service_provider/service_provider.test.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/providers/render_service_provider/service_provider.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/providers/selectable/index.test.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/inference/providers/selectable/index.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_field.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_form_items.tsx # x-pack/platform/plugins/shared/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_utils.ts # x-pack/plugins/stack_connectors/public/connector_types/inference/additional_options_fields.tsx # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/alibaba_cloud.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/amazon_bedrock.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/anthropic.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/azure_ai_studio.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/azure_open_ai.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/cohere.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/elastic.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/google_ai_studio.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/hugging_face.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/ibm_watsonx.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/mistral.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/assets/images/open_ai.svg # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/render_service_provider/service_provider.test.tsx # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/render_service_provider/service_provider.tsx # x-pack/plugins/stack_connectors/public/connector_types/inference/providers/selectable/index.tsx # x-pack/plugins/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_field.tsx # x-pack/plugins/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_form_items.tsx # x-pack/plugins/stack_connectors/public/connector_types/lib/dynamic_config/connector_configuration_utils.ts # x-pack/solutions/search/plugins/search_inference_endpoints/public/hooks/use_add_endpoint.test.tsx # x-pack/solutions/search/plugins/search_inference_endpoints/public/hooks/use_add_endpoint.ts # x-pack/solutions/search/plugins/search_inference_endpoints/public/hooks/use_providers.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/lib/add_inference_endpoint.test.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/lib/add_inference_endpoint.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/lib/fetch_inference_services.test.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/lib/fetch_inference_services.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/routes.ts # x-pack/solutions/search/plugins/search_inference_endpoints/server/utils/unflatten_object.ts # yarn.lock
…ulNaumenko/kibana into ai-connector-uses-shared-ui-components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the improvements!
</> | ||
); | ||
return !isLoading ? ( | ||
<InferenceServiceFormFields providers={providers ?? []} isEdit={isEdit} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since you're using the shared components now and it looks like the shared components have their own routes for getting the services configuration (hard-coded for now), should we also remove the server side routes in the stack_connectors folder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ulNaumenko/kibana into ai-connector-uses-shared-ui-components
💔 Build Failed
Failed CI StepsHistory
cc @YulNaumenko |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ents from '@kbn/inference-endpoint-ui-common' (elastic#204885) 1. Migrated stack-connector `.inference` to use share UI components elastic#203204 2. Extended package `@kbn/inference-endpoint-ui-common` to support edit mode for the form of the connector, by adding the optional property `isEdit` to `InferenceServiceFormFields` component 3. Resolves flaky timing out test elastic#205129 --------- Co-authored-by: Samiul Monir <[email protected]> Co-authored-by: kibanamachine <[email protected]>
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
1 similar comment
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
.inference
to use share UI components [Search] Add Inference endpoint Flyout in Inference Management UI #203204@kbn/inference-endpoint-ui-common
to support edit mode for the form of the connector, by adding the optional propertyisEdit
toInferenceServiceFormFields
component