Skip to content

fix(dataflow): update pipeline_with_dependencies deps for dependabot and fix sample bugs - #14648

Open
XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:dependabot-alerts-pymongo
Open

XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:dependabot-alerts-pymongo

Conversation

@XrossFox

@XrossFox XrossFox commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Description

  • Upgrade dependencies and Python runtime (3.13 / Apache Beam 2.77.0) to resolve Dependabot alerts.
  • Fix sample build and test issues discovered during verification (update Dockerfile/packaging, add .gcloudignore, and normalize bucket URI in conftest.py).
image

Fixes b/573677639

Checklist

Testing

  • I have tested this change on a live environment and verified it works as intended.

Compliance & Style


Post-Approval Actions

  • Please merge this PR for me once it is approved

…and fix sample bugs

- Upgrade dependencies and Python runtime (3.13 / Apache Beam 2.77.0) to resolve Dependabot alerts.
- Fix sample build and test issues discovered during verification (update Dockerfile/packaging, add .gcloudignore, and normalize bucket URI in conftest.py).
@product-auto-label product-auto-label Bot added api: dataflow Issues related to the Dataflow API. samples Issues that are directly related to samples. labels Oct 15, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Python version from 3.14 to 3.13 and upgrades the Apache Beam SDK to version 2.77.0 across the pipeline_with_dependencies flex template files, including the Dockerfile, pyproject.toml, and requirements files. It also introduces a .gcloudignore file and modifies conftest.py to strip the 'gs://' prefix from the GOOGLE_CLOUD_BUCKET environment variable. Regarding the feedback, it is recommended to use .removeprefix('gs://').rstrip('/') in conftest.py to make the bucket name parsing cleaner and more robust against trailing slashes.

Comment thread dataflow/conftest.py

@angelcaamal angelcaamal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@XrossFox
XrossFox marked this pull request as ready for review October 20, 2026 23:49
@XrossFox
XrossFox requested review from a team as code owners October 20, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: dataflow Issues related to the Dataflow API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants