feat: use EUC for AI generation functions when connection_id is not present #2272
Conversation
product-auto-label
bot
added
size: m
sycai
changed the title
tswast
left a comment
tswast
left a comment
There was a problem hiding this comment.
doctest failures appear to be related
FAILED bigframes/bigquery/_operations/ai.py::bigframes.bigquery._operations.ai.generate
FAILED bigframes/bigquery/_operations/ai.py::bigframes.bigquery._operations.ai.generate_bool
FAILED bigframes/bigquery/_operations/ai.py::bigframes.bigquery._operations.ai.generate_double
FAILED bigframes/bigquery/_operations/ai.py::bigframes.bigquery._operations.ai.generate_int
Yeah those look like permission failures. I guess our doc runners don't have EUC permissions for AI functions. |
|
cl/828515544 may be a useful example. that's where I granted the samples tests users permissions. that shouldn't affect doctest, though. doctest should use the compute engine service account from bigframes-dev. |
Would doctest be considered a background job (I'm not very familiar with these setups)? If that's true, then we will need to explicitly provide a connection_id too 🧐. |
sycai
added
the
do not merge
|
Do not merge this PR until related failures in doc tests and notebook tests are gone |
Fixes #460856043 🦕