[ML] Add Groq inference service#138251
Conversation
🔍 Preview links for changed docs |
ℹ️ Important: Docs version tagging👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version. We use applies_to tags to mark version-specific features and changes. Expand for a quick overviewWhen to use applies_to tags:✅ At the page level to indicate which products/deployments the content applies to (mandatory) What NOT to do:❌ Don't remove or replace information that applies to an older version 🤔 Need help?
|
ed7dc20 to
1d8c585
Compare
1d8c585 to
27e55c0
Compare
|
Pinging @elastic/ml-core (Team:ML) |
|
@elasticmachine test this please |
|
@elasticmachine test this please |
davidkyle
left a comment
There was a problem hiding this comment.
Looks great I left a couple of minor comments.
CI failed due to the use of a forbidden reflection API call in a test I pushed a change to remove that. You can run the precommit target locally to check for these sorts of problems with
./gradlew :x-pack:plugin:inference:spotlessApply :x-pack:plugin:inference:precommit
To run the tests its
./gradlew :x-pack:plugin:inference:test
...gin/inference/src/main/java/org/elasticsearch/xpack/inference/services/groq/GroqService.java
Outdated
Show resolved
Hide resolved
...gin/inference/src/main/java/org/elasticsearch/xpack/inference/services/groq/GroqService.java
Outdated
Show resolved
Hide resolved
...gin/inference/src/main/java/org/elasticsearch/xpack/inference/services/groq/GroqService.java
Outdated
Show resolved
Hide resolved
...gin/inference/src/main/java/org/elasticsearch/xpack/inference/services/groq/GroqService.java
Outdated
Show resolved
Hide resolved
...gin/inference/src/main/java/org/elasticsearch/xpack/inference/services/groq/GroqService.java
Outdated
Show resolved
Hide resolved
...csearch/xpack/inference/services/groq/completion/GroqChatCompletionServiceSettingsTests.java
Outdated
Show resolved
Hide resolved
...csearch/xpack/inference/services/groq/completion/GroqChatCompletionServiceSettingsTests.java
Outdated
Show resolved
Hide resolved
...sticsearch/xpack/inference/services/groq/completion/GroqChatCompletionTaskSettingsTests.java
Outdated
Show resolved
Hide resolved
|
Related Kibana issue for the icon elastic/kibana#243696 |
Co-authored-by: David Kyle <david.kyle@elastic.co>
…e and task setting tests to support new AbstractWireSerializingTestCase base class.
|
@elasticmachine test this please |
|
@davidkyle I see a conflict in |
|
@elasticmachine test this please |
I resolved it by accepting the incoming change then ran The PR is approved, I'll merge it once the UI change elastic/kibana#243696 is underway so that the icon is not missing when groq appears in Elastic serverless. |
|
@elasticmachine test this please |
|
@elasticmachine run elasticsearch-ci/bwc-snapshots-part3 |
|
@elasticmachine test this please |
|
@elasticmachine test this please |
…: add groq icon (#244962) ## Summary Fixes #243696 Groq will be added to the Inference API in elastic/elasticsearch#138251 with the service name `groq`. This PR adds the `groq` icon to the providers list shown in the AI Connector and inference endpoint creation providers list and updates telemetry. <img width="893" height="577" alt="image" src="https://github.com/user-attachments/assets/7e03144f-9f32-42d9-b423-7f79d48020a3" /> <img width="894" height="416" alt="image" src="https://github.com/user-attachments/assets/fc4d4ed4-8ea4-4f4b-8440-a0a7e9c1cb8b" /> ### 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: Elastic Machine <elasticmachine@users.noreply.github.com>
…: add groq icon (elastic#244962) ## Summary Fixes elastic#243696 Groq will be added to the Inference API in elastic/elasticsearch#138251 with the service name `groq`. This PR adds the `groq` icon to the providers list shown in the AI Connector and inference endpoint creation providers list and updates telemetry. <img width="893" height="577" alt="image" src="https://github.com/user-attachments/assets/7e03144f-9f32-42d9-b423-7f79d48020a3" /> <img width="894" height="416" alt="image" src="https://github.com/user-attachments/assets/fc4d4ed4-8ea4-4f4b-8440-a0a7e9c1cb8b" /> ### 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: Elastic Machine <elasticmachine@users.noreply.github.com>
Summary
Testing
Create Groq chat completion endpoint
Stream chat completion
Stream chat completion with tool calling