Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Remove Ollama from Integration Tests #1376

Merged
merged 3 commits into from
Apr 29, 2025
Merged

Remove Ollama from Integration Tests #1376

merged 3 commits into from
Apr 29, 2025

Conversation

lukehinds
Copy link

Need to test this by pushing

Comment on lines 379 to 385
else:
selected_providers = available_providers

# Exclude ollama provider if it's in the list
if "ollama" in selected_providers:
selected_providers.remove("ollama")
logger.info("Excluded 'ollama' provider from tests.")
Copy link
Member

Choose a reason for hiding this comment

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

I think we can leave that there as we might want to run the test locally.

The change in the workflow yml file should be sufficient to skip the ollama tests from running 👍

blkt
blkt previously approved these changes Apr 29, 2025
@rdimitrov
Copy link
Member

rdimitrov commented Apr 29, 2025

@blkt @lukehinds - I see Michelangelo liked the suggestion so I hope you don't mind that I'll push on top of this PR to revert only that change so we can merge the rest

@rdimitrov rdimitrov requested a review from blkt April 29, 2025 10:52
@rdimitrov rdimitrov merged commit 6bb71bf into main Apr 29, 2025
8 of 10 checks passed
@rdimitrov rdimitrov deleted the rem-ollama branch April 29, 2025 12:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants