Skip to content

Commit

Permalink
Fix ollama IT execution
Browse files Browse the repository at this point in the history
  • Loading branch information
Mark Pollack committed Nov 19, 2024
1 parent 20f0477 commit 4363c95
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
6 changes: 5 additions & 1 deletion .github/actions/do-integration-test/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ inputs:
model-name:
description: "SpringAI Model to integration test"
required: true
maven-properties:
description: "Maven properties to pass to the build (e.g., -Dskip.vectorstore.milvus=true -Dskip.vectorstore.neo4j=true)"
required: false
default: ""

runs:
using: "composite"
Expand Down Expand Up @@ -38,4 +42,4 @@ runs:

- name: Run ${{ inputs.model-name }} model tests
shell: bash
run: cd spring-ai && ./mvnw -pl models/spring-ai-${{ inputs.model-name }} -Pintegration-tests -Dfailsafe.rerunFailingTestsCount=2 verify
run: cd spring-ai && ./mvnw -pl models/spring-ai-${{ inputs.model-name }} -Pintegration-tests ${{ inputs.maven-properties }} -Dfailsafe.rerunFailingTestsCount=2 verify
2 changes: 1 addition & 1 deletion .github/workflows/spring-ai-integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ jobs:
ports:
- 11434:11434
env:
OLLAMA_TESTS_ENABLED: true
OLLAMA_WITH_REUSE: true
steps:
- name: Checkout the action
Expand All @@ -37,6 +36,7 @@ jobs:
uses: ./.github/actions/do-integration-test
with:
model-name: ollama
maven-properties: "-Dskip.model.ollama=false"

# Anthrohpic is overloaded :(
#
Expand Down

0 comments on commit 4363c95

Please # to comment.