Skip to content

Extend CI scripts to test for external LLM endpoints - #1140

Merged
mkbhanda merged 2 commits into
opea-project:mainfrom
devpramod:external-llm-ci
Jul 3, 2025
Merged

mkbhanda merged 2 commits into
opea-project:mainfrom
devpramod:external-llm-ci

Conversation

@devpramod

Copy link
Copy Markdown
Contributor

Description

This PR extends the CI scripts to support testing with external LLM endpoints. The changes add configuration for external LLM services in the Helm E2E workflow, allowing charts with external-llm-values.yaml files to be tested against external LLM providers. The implementation includes environment variable mapping for different chart types (ChatQnA, DocSum, CodeGen etc) and supports configurable LLM server hosts, ports, models, and API keys.

Issues

n/a

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)

Dependencies

No new 3rd party dependencies introduced.

Tests

Describe the tests that you ran to verify your changes.
The changes extend existing Helm E2E tests to support external LLM endpoint testing. Charts with external-llm-values.yaml files will now be tested with the configured external LLM services using the appropriate environment variables and Helm chart arguments.

@devpramod
devpramod requested review from mkbhanda and poussa as code owners July 1, 2025 21:33
Comment thread .github/workflows/_helm-e2e.yaml
@eero-t
eero-t requested a review from lianhao July 2, 2025 08:40
@eero-t

eero-t commented Jul 2, 2025

Copy link
Copy Markdown
Collaborator

pre-commit.ci fails due to trailing whitespace, please fix that: sed -i 's/ *$//' <file>

devpramod added 2 commits July 3, 2025 13:53
Signed-off-by: devpramod <pramod.pai@intel.com>
Signed-off-by: devpramod <pramod.pai@intel.com>

@eero-t eero-t left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's use this non-ideal solution for now and align later, when we see fit.

OK, approved. CI is going AWOL for few weeks, so we're in a bit of hurry.

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.

4 participants