-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Made changes to build and test multi-arch Beam SDK container images #27311
Conversation
Run Python Dataflow ValidatesContainer |
Codecov Report
@@ Coverage Diff @@
## master #27311 +/- ##
==========================================
- Coverage 70.89% 70.89% -0.01%
==========================================
Files 860 860
Lines 104635 104635
==========================================
- Hits 74184 74183 -1
- Misses 28898 28899 +1
Partials 1553 1553
Flags with carried forward coverage won't be shown. Click here to find out more. see 5 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Assigning reviewers. If you would like to opt out of this review, comment R: @tvalentyn for label python. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
…script for validatescontainer
…escontainer test.
Run Python Dataflow ValidatesContainer |
… images built during testing will succeed.
Run Python Dataflow ValidatesContainer |
…tCommit_Java_Examples_Dataflow_V2 test suite for diff Java versions.
Reminder, please take a look at this pr: @tvalentyn @lostluck @Abacn |
Changes were broken down into other PRs. |
The pull request adds the following changes:
-Ppush-containers
build parameter to push and save multi-arch containers as a manifest list.Related issues:
Explicitly install tensorflow-cpu-aws: tensorflow/tensorflow#60751
Save multi-arch images: #26902
Design doc: https://docs.google.com/document/d/1ikbEJNsFH1D9HqiMqiVyyMlNpDgSqxXK22nUoetzW6I/edit?usp=sharing