Conversation
Collaborator
Author
This was referenced May 15, 2025
danpiths
approved these changes
May 15, 2025
Pratham-Mishra04
force-pushed
the
05-15-fix_vertex_ai_provider_cred_input_changes
branch
from
c58ad8c to
763035f
Compare
Pratham-Mishra04
changed the base branch from
graphite-base/38
to
05-13-feat_readme_additions
This was referenced May 17, 2025
Contributor
Merge activity
|
akshaydeo
changed the base branch from
05-13-feat_readme_additions
to
graphite-base/38
akshaydeo
force-pushed
the
05-15-fix_vertex_ai_provider_cred_input_changes
branch
from
763035f to
e9e8a5f
Compare
Pratham-Mishra04
deleted the
05-15-fix_vertex_ai_provider_cred_input_changes
branch
akshaydeo
added a commit
that referenced
this pull request
Nov 17, 2025
…ex provider (#38) # Use Vertex credentials directly instead of reading from file This PR changes the Vertex provider to accept authentication credentials directly as a string rather than reading them from a file path. This simplifies configuration and deployment by: 1. Replacing `auth_credential_path` with `auth_credentials` in the Vertex meta configuration 2. Modifying the provider interface to use `GetAuthCredentials()` instead of `GetAuthCredentialPath()` 3. Updating the Vertex provider to use the credentials directly instead of reading from a file 4. Updating test configurations and example config to use the new approach This change allows credentials to be passed directly as environment variables or configuration values without requiring file system access.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use Vertex credentials directly instead of reading from file
This PR changes the Vertex provider to accept authentication credentials directly as a string rather than reading them from a file path. This simplifies configuration and deployment by:
auth_credential_pathwithauth_credentialsin the Vertex meta configurationGetAuthCredentials()instead ofGetAuthCredentialPath()This change allows credentials to be passed directly as environment variables or configuration values without requiring file system access.