Skip to content

Add support for service account when using litellm#295

Open
luis5tb wants to merge 2 commits into
RHEcosystemAppEng:mainfrom
luis5tb:worktree-litellm-sa
Open

Add support for service account when using litellm#295
luis5tb wants to merge 2 commits into
RHEcosystemAppEng:mainfrom
luis5tb:worktree-litellm-sa

Conversation

@luis5tb

@luis5tb luis5tb commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Allow using a service account credentials JSON file (GOOGLE_APPLICATION_CREDENTIALS)
as a secure alternative to GOOGLE_API_KEY when accessing Vertex AI models, both
via direct Gemini and LiteLLM with vertex_ai/* providers.

- Add google_application_credentials setting to config
- Set GOOGLE_APPLICATION_CREDENTIALS env var in _setup_environment()
- Mount GCP SA key in agent deployment (conditional on gcpServiceAccountKey)
- Make gcp-sa-secret conditional on key presence (not handler-only)
- Fix handler deployment to also be conditional on key presence
- Default cloudLocation to global (aligns with Python settings default)
- Update docs, .env.example, secrets.yaml.example, and OpenShift README

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@luis5tb luis5tb force-pushed the worktree-litellm-sa branch from d0150bc to 5033377 Compare June 12, 2026 07:23
The marketplace handler on OpenShift is a standalone service that does
not need GCP service account credentials, Gemini API keys, or access
to the Google Commerce Procurement API. Remove GOOGLE_APPLICATION_CREDENTIALS,
GOOGLE_API_KEY, GOOGLE_CLOUD_PROJECT, and the gcp-sa-key volume mount
from the handler deployment template.

Also add gcp-sa-secret checksum to the agent deployment and clean up
GOOGLE_APPLICATION_CREDENTIALS in tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@luis5tb luis5tb force-pushed the worktree-litellm-sa branch from 5033377 to 0d11a7a Compare June 12, 2026 08:19
@luis5tb luis5tb changed the title [WIP] Add support for service account when using litellm Add support for service account when using litellm Jun 12, 2026
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.

1 participant