Skip to content

Conversation

leemthompo
Copy link
Contributor

@leemthompo leemthompo self-assigned this Oct 6, 2025
Copy link

github-actions bot commented Oct 6, 2025

@leemthompo leemthompo requested a review from a team October 6, 2025 13:29
Comment on lines 42 to 43
4. Search for **GenAI Settings** in the global search field
5. Select your new connector from the **Default AI Connector** dropdown
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: doesn't look like this be ready for 9.2?

Copy link
Member

Choose a reason for hiding this comment

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

@leemthompo leemthompo mentioned this pull request Oct 20, 2025
1 task
@leemthompo leemthompo marked this pull request as ready for review October 20, 2025 11:37
@leemthompo leemthompo requested a review from a team as a code owner October 20, 2025 11:37
Copy link
Contributor

@kosabogi kosabogi 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

@ketkee-aryamane ketkee-aryamane left a comment

Choose a reason for hiding this comment

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

couple questions, lgtm otherwise. Approving for unblocking.


For programmatic access to connector management, refer to the [Connectors API documentation]({{kib-serverless-apis}}group/endpoint-connectors).

## Recommended models
Copy link
Contributor

Choose a reason for hiding this comment

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

How about we combine this section with the subsequent one? The opening line can be the one in this section followed by the list of the families.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

absolutely, no need for both headings 👍

GPT-4o-mini and similar smaller models are not recommended for {{agent-builder}} as they lack the necessary capabilities for reliable agent workflows.
:::

## Related pages
Copy link
Contributor

Choose a reason for hiding this comment

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

Can call it "Related resources" for consistency with other doc pages

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure 👍

Removed model selection section and added a reference to the models page for more information.
@leemthompo leemthompo enabled auto-merge (squash) October 21, 2025 07:11
@leemthompo leemthompo merged commit 222dc9e into main Oct 21, 2025
7 checks passed
@leemthompo leemthompo deleted the leemthompo/agent-builder-models branch October 21, 2025 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants