Re: [PR] [1/2] sdks/python: enrich data with CloudSQL [PostgreSQL, MySQL, SQLServer] [beam]

["mohamedawnallah (via GitHub)" <gi...@apache.org>]

mohamedawnallah commented on PR #34398:
URL: https://github.com/apache/beam/pull/34398#issuecomment-3029451083

   Hey @claudevdm, sorry for the delay on my side — and thanks for unblocking me by providing the CloudSQL PostgreSQL credentials 🙏
   
   I've submitted a patch that fixes one of the failing post-commit CI tests related to PostgreSQL. However, the `beam_PreCommit_Python_Transforms` checks are still failing because the `ALLOYDB_PASSWORD` environment variable is not available for the CloudSQL PostgreSQL DB.
   
   I attempted to provide the workflow with `ALLOYDB_PASSWORD` env variable, but the change isn't reflected in the CI run. This is due to the fact that the workflow `beam_PreCommit_Python_Transforms` is triggered using `pull_request_target`, which means it only uses the workflow configuration from the main branch of the repository—not from the PR branch itself.
   https://github.com/apache/beam/blob/fc0617571ed44e59726e7e2e30e6b32a2a7c98cf/.github/workflows/beam_PreCommit_Python_Transforms.yml#L52-L55
   
   CI run for reference:
   https://github.com/apache/beam/actions/runs/15950834049/job/44990600693?pr=34398a
   
   How am I supposed to move this forward?


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org