Skip to content

fix(ci): bump docker-build-push to v3.2.1 to fix release job summary crash - #2973

Merged
Erik Osterman (Cloud Posse) (osterman) merged 1 commit into
mainfrom
aknysh/docker-build-push-v3.2.1
Aug 21, 2026
Merged

Erik Osterman (Cloud Posse) (osterman) merged 1 commit into
mainfrom
aknysh/docker-build-push-v3.2.1

Conversation

@aknysh

@aknysh Andriy Knysh (aknysh) commented Aug 20, 2026

Copy link
Copy Markdown
Member

what

  • Bump the Docker Build step in .github/workflows/build.yml from cloudposse/github-action-docker-build-push v3.1.0 (02993d67) to v3.2.1 (ff59bd5).

why

  • The release / Build and push Docker image for Atmos CLI job has been failing on the post-build Docker Inspect summary step with jq: error (at inspect.json:79): Cannot iterate over null (null) → exit code 5, even though the image builds and pushes fine.
  • Root cause was in the action itself: its summary step fed docker inspect fields (.Config.Entrypoint, .Config.Cmd, .Config.Env, .RootFS.Layers) straight into jq's join/.[]/to_entries, all of which iterate. Atmos's image is FROM debian:trixie-slim with no ENTRYPOINT, so .Config.Entrypoint is null and jq aborts; under the default bash -e shell that fails the whole step.
  • Fixed upstream in fix: guard summary jq against null Entrypoint/Cmd/Env github-action-docker-build-push#111 (guards each iterating expression with // []), released as v3.2.1. Verified ff59bd5 contains all four guards. This bump pins Atmos to that release so the release job's summary no longer crashes.

references

Summary by CodeRabbit

  • Chores
    • Updated the Docker build workflow to use a newer action version, improving build pipeline maintenance and reliability.

@aknysh
Andriy Knysh (aknysh) requested a review from a team as a code owner August 20, 2026 22:42
@aknysh Andriy Knysh (aknysh) added the no-release Do not create a new release (wait for additional code changes) label Aug 20, 2026
@atmos-pro

atmos-pro Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Tip

Atmos Pro  

No affected stacks workflow was detected for this pull request.
If this is expected, no action is needed.
Learn More. Ask AI.

@github-actions

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

.github/workflows/build.yml

PackageVersionLicenseIssue Type
cloudposse/github-action-docker-build-pushff59bd5b5c0c953aef60f5eb9b80178fff407597NullUnknown License
Allowed Licenses: MIT, MIT-0, Apache-2.0, BSD-2-Clause, BSD-2-Clause-Views, BSD-3-Clause, ISC, MPL-2.0, 0BSD, Unlicense, CC0-1.0, CC-BY-3.0, CC-BY-4.0, CC-BY-SA-3.0, Python-2.0, OFL-1.1, LicenseRef-scancode-generic-cla, LicenseRef-scancode-unknown-license-reference, LicenseRef-scancode-unicode, LicenseRef-scancode-google-patent-license-golang
Excluded from license check: pkg:golang/github.com/antlr4-go/antlr/v4, pkg:golang/github.com/google/cel-go, pkg:golang/golang.org/x/image, pkg:golang/modernc.org/libc, pkg:golang/github.com/opencontainers/go-digest, pkg:npm/pako, pkg:npm/sax

Scanned Files

  • .github/workflows/build.yml

@mergify

mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Important

Cloud Posse Engineering Team Review Required

This 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 #pr-reviews channel.

@mergify mergify Bot added the needs-cloudposse Needs Cloud Posse assistance label Aug 20, 2026
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 25407967-df6a-46b4-8fcb-bb7d8c227571

📥 Commits

Reviewing files that changed from the base of the PR and between 8148b1d and f006295.

📒 Files selected for processing (1)
  • .github/workflows/build.yml

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The Docker build workflow updates cloudposse/github-action-docker-build-push from v3.1.0 to v3.2.1.

Changes

Docker build workflow

Layer / File(s) Summary
Update Docker build action
.github/workflows/build.yml
The Docker build and push step now uses cloudposse/github-action-docker-build-push v3.2.1.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f0062

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: osterman, johncblandii, goruha

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the CI action update and the release job summary crash it fixes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch aknysh/docker-build-push-v3.2.1

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@atmos-pro

atmos-pro Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Tip

Atmos Pro  

No affected stacks workflow was detected for this pull request.
If this is expected, no action is needed.
Learn More. Ask AI.

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.43%. Comparing base (8148b1d) to head (f006295).

Additional details and impacted files

Impacted file tree graph

@@            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     
Flag Coverage Δ
unittests 83.43% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 6 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Merged via the queue into main with commit 56a3c3c Aug 21, 2026
134 checks passed
@osterman
Erik Osterman (Cloud Posse) (osterman) deleted the aknysh/docker-build-push-v3.2.1 branch August 21, 2026 00:07
@atmos-pro

atmos-pro Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Tip

Atmos Pro  

No affected stacks workflow was detected for this pull request.
If this is expected, no action is needed.
Learn More. Ask AI.

@mergify mergify Bot removed the needs-cloudposse Needs Cloud Posse assistance label Aug 21, 2026
@github-actions

Copy link
Copy Markdown

These changes were released in v1.227.0-test.9.

@github-actions

Copy link
Copy Markdown

These changes were released in v1.227.0-test.10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-release Do not create a new release (wait for additional code changes) size/xs Extra small size PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants