Skip to content

Add LiteLlm + TemporalModel integration test#1431

Merged
tconley1428 merged 4 commits intomainfrom
maplexu/adk-litellm-integration
Apr 10, 2026
Merged

Add LiteLlm + TemporalModel integration test#1431
tconley1428 merged 4 commits intomainfrom
maplexu/adk-litellm-integration

Conversation

@xumaple
Copy link
Copy Markdown
Contributor

@xumaple xumaple commented Apr 9, 2026

Summary

  • Added an integration test (test_litellm_model) that runs a LiteLlm-backed agent through a full Temporal workflow using a fake litellm custom provider (no API key needed).
  • Added litellm to dev dependencies.

@xumaple xumaple requested a review from a team as a code owner April 9, 2026 20:10
@xumaple xumaple force-pushed the maplexu/adk-litellm-integration branch from 50f99e2 to 654c80e Compare April 9, 2026 20:44
@xumaple xumaple force-pushed the maplexu/adk-litellm-integration branch from 654c80e to ffe8223 Compare April 9, 2026 21:16
@xumaple xumaple marked this pull request as draft April 9, 2026 21:18
xumaple and others added 4 commits April 9, 2026 21:25
Add litellm and httpx to the GoogleAdkPlugin sandbox passthrough
modules. Without this, any LiteLlm-backed model crashes inside the
workflow sandbox because litellm transitively imports httpx which
fails sandbox restrictions.

Add an integration test proving LiteLlm works with TemporalModel
through the full Temporal workflow path, using a fake litellm custom
provider that requires no API key.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@xumaple xumaple force-pushed the maplexu/adk-litellm-integration branch from 4362000 to 316bf50 Compare April 10, 2026 01:26
@xumaple xumaple marked this pull request as ready for review April 10, 2026 04:03
Copy link
Copy Markdown
Contributor

@tconley1428 tconley1428 left a comment

Choose a reason for hiding this comment

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

Update the PR description and title first.

@xumaple xumaple changed the title Add LiteLlm + TemporalModel integration test and sandbox fix Add LiteLlm + TemporalModel integration test Apr 10, 2026
@tconley1428 tconley1428 merged commit 79a140b into main Apr 10, 2026
17 checks passed
@tconley1428 tconley1428 deleted the maplexu/adk-litellm-integration branch April 10, 2026 16:30
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.

2 participants