chore(multiple samples): bump TensorFlow and downgrade python/beam on Dockerfiles#14384
chore(multiple samples): bump TensorFlow and downgrade python/beam on Dockerfiles#14384XrossFox wants to merge 2 commits into
Conversation
…n Dockerfiles - Update TensorFlow to 2.21.0 and Apache Beam to 2.74.0 across samples. - Modify Dockerfiles to use Python 3.13 and matching beam images due to compatibility with the updated CUDA base image (`12.5.1`). - Remove pytest version constraints in `requirements-test.txt`
product-auto-label
Bot
added
samples
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates several Dockerfiles and dependency files to upgrade TensorFlow to 2.21.0, Apache Beam to 2.74.0, and transition Python environments from 3.14 to 3.13. Feedback was provided on three Dockerfiles pointing out that installing python3-distutils via apt-get is redundant and should be removed, as distutils is no longer present in Python 3.13 and the package only installs for the system default Python 3.10.
| # Install Python and other system dependencies. | ||
| && apt-get install -y --no-install-recommends \ | ||
| curl g++ python3.14-dev python3.14-venv python3-distutils \ | ||
| curl g++ python3.13-dev python3.13-venv python3-distutils \ |
There was a problem hiding this comment.
With the upgrade to Python 3.13, the distutils package has been completely removed from the standard library (deprecated since Python 3.10 and removed in Python 3.12). Installing python3-distutils via apt-get is redundant as it only installs the package for the system default Python (Python 3.10 on Ubuntu 22.04) and does not provide it for Python 3.13. It can be safely removed.
curl g++ python3.13-dev python3.13-venv \
| # Install Python and other system dependencies. | ||
| && apt-get install -y --no-install-recommends \ | ||
| curl g++ python3.14-dev python3.14-venv python3-distutils \ | ||
| curl g++ python3.13-dev python3.13-venv python3-distutils \ |
There was a problem hiding this comment.
With the upgrade to Python 3.13, the distutils package has been completely removed from the standard library (deprecated since Python 3.10 and removed in Python 3.12). Installing python3-distutils via apt-get is redundant as it only installs the package for the system default Python (Python 3.10 on Ubuntu 22.04) and does not provide it for Python 3.13. It can be safely removed.
curl g++ python3.13-dev python3.13-venv \
| # Install Python and other system dependencies. | ||
| && apt-get install -y --no-install-recommends \ | ||
| curl g++ python3.14-dev python3.14-venv python3-distutils \ | ||
| curl g++ python3.13-dev python3.13-venv python3-distutils \ |
There was a problem hiding this comment.
With the upgrade to Python 3.13, the distutils package has been completely removed from the standard library (deprecated since Python 3.10 and removed in Python 3.12). Installing python3-distutils via apt-get is redundant as it only installs the package for the system default Python (Python 3.10 on Ubuntu 22.04) and does not provide it for Python 3.13. It can be safely removed.
curl g++ python3.13-dev python3.13-venv \
XrossFox
force-pushed
the
package-update-tensorflow
branch
from
0da0935 to
ebdd931
Compare
2 participants
Description
Fixing dependabot alerts:
12.5.1).requirements-test.txtFixes b/531809402
Checklist
Testing
Compliance & Style
Post-Approval Actions