zhengruifeng opened a new pull request, #56004:
URL: https://github.com/apache/spark/pull/56004

   Backport of #55972 to `branch-4.x`. Cherry-picked from master commit 
83b2d0723543984b074591e1c04fcd95c92c1f3e; resolved a trivial conflict on the 
`docker/build-push-action` SHA pin (kept the version already in use on 
`branch-4.x`).
   
   ### What changes were proposed in this pull request?
   
   Make the `build_infra_images_cache.yml` workflow tolerant of individual 
image build failures:
   
   - Add `continue-on-error: true` to each of the 12 `Build and push` steps so 
a failure in one does not abort the remaining builds. In particular, a failure 
of the base `./dev/infra/` image build should no longer prevent the other image 
builds from running.
   - Add a final "Fail if any image build failed" step that runs with `if: 
always()`, prints each build step's `outcome`, and exits non-zero if any was 
`failure`.
   
   ### Why are the changes needed?
   
   Today, a single image build failure aborts the workflow immediately, leaving 
the remaining cache layers stale until someone re-triggers the job. This is 
especially impactful when the first step (the `./dev/infra/` base image) fails, 
because every subsequent image build is then skipped on that run. With this 
change every image still gets a chance to build and refresh its cache on each 
run, while the overall workflow still fails if any image build did not succeed.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   YAML parses cleanly (`python3 -c "import yaml; yaml.safe_load(...)"`). 
Verified all 12 build steps received `continue-on-error: true` and that the 
final aggregator step references every build step's `outcome`.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Claude Opus 4.7)


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to