fix(ci): bump docker-build-push to v3.2.1 to fix release job summary crash - #2973
Conversation
|
Tip Atmos Pro
No affected stacks workflow was detected for this pull request. |
Dependency ReviewThe following issues were found:
License Issues.github/workflows/build.yml
Scanned Files
|
|
Important Cloud Posse Engineering Team Review RequiredThis pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes. To expedite this process, reach out to us on Slack in the |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe Docker build workflow updates ChangesDocker build workflow
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized workflow update pins the Docker build step to the upstream release intended to prevent the release-summary crash; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Tip Atmos Pro
No affected stacks workflow was detected for this pull request. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2973 +/- ##
==========================================
- Coverage 83.44% 83.43% -0.01%
==========================================
Files 1923 1923
Lines 188072 188072
==========================================
- Hits 156928 156918 -10
- Misses 23227 23235 +8
- Partials 7917 7919 +2
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Tip Atmos Pro
No affected stacks workflow was detected for this pull request. |
|
These changes were released in v1.227.0-test.9. |
|
These changes were released in v1.227.0-test.10. |
what
Docker Buildstep in.github/workflows/build.ymlfromcloudposse/github-action-docker-build-pushv3.1.0 (02993d67) to v3.2.1 (ff59bd5).why
release / Build and push Docker image for Atmos CLIjob has been failing on the post-build Docker Inspect summary step withjq: error (at inspect.json:79): Cannot iterate over null (null)→ exit code 5, even though the image builds and pushes fine.docker inspectfields (.Config.Entrypoint,.Config.Cmd,.Config.Env,.RootFS.Layers) straight into jq'sjoin/.[]/to_entries, all of which iterate. Atmos's image isFROM debian:trixie-slimwith noENTRYPOINT, so.Config.Entrypointisnulland jq aborts; under the defaultbash -eshell that fails the whole step.// []), released as v3.2.1. Verifiedff59bd5contains all four guards. This bump pins Atmos to that release so the release job's summary no longer crashes.references
Summary by CodeRabbit