Lee-W commented on code in PR #74451:
URL: https://github.com/apache/airflow/pull/74451#discussion_r4273397813
##########
providers/common/ai/docs/connections/pydantic_ai_vertex.rst:
##########
@@ -57,8 +57,10 @@ Model
mean Vertex AI Express Mode credentials (see "Credentials" below), so its
presence alone cannot tell the two platforms apart. To reach the
Generative Language API instead, prefix the model explicitly with
- ``google:`` -- that spelling routes to a different provider regardless of
- which fields this connection sets.
+ ``google:``. On this connection type that spelling requires ``API Key``
Review Comment:
Fixed both. The Azure page now says a model prefixed with another platform
works only if that provider class accepts the forwarded fields, otherwise
`get_conn()` raises. The Vertex page and `PydanticAIVertexHook` docstring now
say `google:` takes the API key and Custom Endpoint URL (and reads
`GOOGLE_API_KEY` without a key), and rejects the Vertex-only fields.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
