Remove 3.14 ML container reference from snapshot action#38049
Remove 3.14 ML container reference from snapshot action#38049jrmccluskey merged 1 commit intoapache:masterfrom
Conversation
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Assigning reviewers: R: @damccorm for label build. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
| - "python:container:ml:py311:docker" | ||
| - "python:container:ml:py312:docker" | ||
| - "python:container:ml:py313:docker" | ||
| - "python:container:ml:py314:docker" |
There was a problem hiding this comment.
I'm confused - don't we need to be publishing these snapshots?
There was a problem hiding this comment.
there isn't an ML container to publish at present, as the ml_base set of dependencies cannot be used in Python 3.14 yet (main blocker is that there isn't a tensorflow release that is happy with 3.14, or at least there isn't one that pip is happy with.)
Remove erroneous ML 3.14 container gradle command from snapshot publishing. It doesn't affect the other containers, but it would make the workflow appear to be failing when run.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, commentfixes #<ISSUE NUMBER>instead.CHANGES.mdwith noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.