Skip to content

Add docker workflow - #61

Merged
dianab-cl merged 4 commits into
mainfrom
diana/fou-1343-dogfood-remote-signer-and-standalone-attestor-on-besu-to
Aug 20, 2026
Merged

Add docker workflow#61
dianab-cl merged 4 commits into
mainfrom
diana/fou-1343-dogfood-remote-signer-and-standalone-attestor-on-besu-to

Conversation

@dianab-cl

Copy link
Copy Markdown
Contributor

No description provided.

@dianab-cl
dianab-cl requested a review from a team as a code owner August 19, 2026 15:38
@linear-code

linear-code Bot commented Aug 19, 2026

Copy link
Copy Markdown

FOU-1343

@greptile-apps

greptile-apps Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR adds a multi-architecture GitHub Actions workflow for building and publishing Docker images and makes the container’s default command overridable.

  • Builds amd64 and arm64 images, uploads their digests, and publishes a combined manifest.
  • Supports manual ref selection and optional PKCS#11 dependencies.
  • Pins third-party actions to immutable commit SHAs.
  • Sanitizes and validates image tags while reserving latest for main-branch pushes.
  • Splits the Docker image’s entrypoint and default arguments.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/docker.yaml Adds a pinned, multi-platform image publication workflow with digest merging and validated tag selection; the previously reported workflow issues are resolved.
Dockerfile Separates the executable entrypoint from its default arguments so callers can override the container command without replacing the executable.

Sequence Diagram

sequenceDiagram
  participant E as GitHub Event
  participant B as Platform Build Jobs
  participant R as GHCR
  participant A as Digest Artifacts
  participant M as Manifest Job
  E->>B: Trigger amd64 and arm64 builds
  B->>R: Push images by digest
  B->>A: Upload digest markers
  A->>M: Download platform digests
  M->>M: Sanitize and validate tag
  M->>R: Publish multi-platform manifest
  M->>R: Inspect published manifest
Loading

Reviews (2): Last reviewed commit: "After code review" | Re-trigger Greptile

Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml Outdated
Comment thread .github/workflows/docker.yaml
@dianab-cl

Copy link
Copy Markdown
Contributor Author

@greptile review

@dianab-cl
dianab-cl added this pull request to the merge queue Aug 20, 2026
Merged via the queue into main with commit e3dbb27 Aug 20, 2026
11 checks passed
@dianab-cl
dianab-cl deleted the diana/fou-1343-dogfood-remote-signer-and-standalone-attestor-on-besu-to branch August 20, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants