Skip to content

[AI Connector] Adds new temperature parameter to AI Connector config schema#239626

Merged
qn895 merged 9 commits intoelastic:mainfrom
qn895:ai-add-temperature-to-connector
Oct 28, 2025
Merged

[AI Connector] Adds new temperature parameter to AI Connector config schema#239626
qn895 merged 9 commits intoelastic:mainfrom
qn895:ai-add-temperature-to-connector

Conversation

@qn895
Copy link
Copy Markdown
Member

@qn895 qn895 commented Oct 17, 2025 •
edited
Loading

Summary

Part of #239181. This PR adds new temperature parameter to AI Connector config schema.

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.
@qn895 qn895 requested review from a team as code owners October 17, 2025 20:05
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 20, 2025

Pinging @elastic/ml-ui (:ml)
@peteharverson peteharverson added the Feature: AI Infra Relating to the AI Assistant flow and any work impacting/involving the AI/Inference Connector label Oct 20, 2025
@qn895 qn895 mentioned this pull request Oct 20, 2025
Merged
10 tasks
@qn895 qn895 requested a review from a team as a code owner October 21, 2025 13:28
@qn895 qn895 added backport:version Backport to applied version labels release_note:enhancement labels Oct 21, 2025
@qn895 qn895 force-pushed the ai-add-temperature-to-connector branch from 4003547 to 0935d44 Compare October 24, 2025 19:52
qn895 added 2 commits October 24, 2025 14:57
@qn895
Update snapshots
a5c45cb
@qn895
Merge remote-tracking branch 'upstream/main' into ai-add-temperature-…
a040eea 
…to-connector

# Conflicts:
#	x-pack/platform/plugins/shared/stack_connectors/common/inference/schema.ts
@qn895
Copy link
Copy Markdown
Member Author

qn895 commented Oct 24, 2025

@elasticmachine merge upstream
alvarezmelissa87
alvarezmelissa87 approved these changes Oct 24, 2025
View reviewed changes
Copy link
Copy Markdown
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 ⚡
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 24, 2025

There are no new commits on the base branch.
@qn895
Copy link
Copy Markdown
Member Author

qn895 commented Oct 27, 2025

@elasticmachine merge upstream
@qn895 qn895 enabled auto-merge (squash) October 27, 2025 15:05
spong
spong approved these changes Oct 27, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@spong spong left a comment

Choose a reason for hiding this comment

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

Initial schema changes LGTM! 👍
@qn895
Copy link
Copy Markdown
Member Author

qn895 commented Oct 27, 2025

@elasticmachine merge upstream
@qn895
Copy link
Copy Markdown
Member Author

qn895 commented Oct 28, 2025

@elasticmachine merge upstream
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 28, 2025

merge conflict between base and head
@qn895 qn895 removed request for a team, joeypoon, tomsonpl and vitaliidm October 28, 2025 05:02
qn895 added 3 commits October 28, 2025 00:17
@qn895
Merge remote-tracking branch 'upstream/main' into ai-add-temperature-…
33dd479 
…to-connector

# Conflicts:
#	x-pack/platform/plugins/shared/actions/server/integration_tests/__snapshots__/connector_types.test.ts.snap
#	x-pack/platform/plugins/shared/stack_connectors/common/bedrock/schema.ts
#	x-pack/platform/plugins/shared/stack_connectors/common/gemini/schema.ts
#	x-pack/platform/plugins/shared/stack_connectors/common/inference/schema.ts
#	x-pack/platform/plugins/shared/stack_connectors/common/openai/schema.ts
@qn895
Fix schema merge
483e6fb
@qn895
Update snapshots
ef7a993
@qn895 qn895 merged commit 5b15604 into elastic:main Oct 28, 2025
14 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Oct 28, 2025

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @qn895
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 29, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Oct 29, 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 239626 locally
cc: @qn895
@qn895 qn895 removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Oct 29, 2025
@qn895
Copy link
Copy Markdown
Member Author

qn895 commented Oct 29, 2025

This PR doesn't need backporting 👍
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Oct 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 239626 locally
cc: @qn895
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: AI Infra Relating to the AI Assistant flow and any work impacting/involving the AI/Inference Connector :ml release_note:enhancement Team:Fleet Team label for Observability Data Collection Fleet team v9.3.0

8 participants

@qn895 @elasticmachine @kibanamachine @spong @darnautov @alvarezmelissa87 @viduni94 @peteharverson