Skip to content

fix: add qemu#81

Merged
varonix0 merged 1 commit into
mainfrom
add-qemu
Dec 5, 2025
Merged

fix: add qemu#81
varonix0 merged 1 commit into
mainfrom
add-qemu

Conversation

@varonix0
Copy link
Copy Markdown
Member

@varonix0 varonix0 commented Dec 5, 2025

Description 📣

Added qemu for linux static binary, to resolve failed releases

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

@varonix0 varonix0 self-assigned this Dec 5, 2025
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Dec 5, 2025

Greptile Overview

Greptile Summary

Added docker/setup-qemu-action@v3 to the goreleaser job to enable cross-platform emulation for building ARM64 Docker images on AMD64 GitHub runners.

  • Enables QEMU emulation before Docker Buildx setup, which is required for the existing --platform=linux/arm64 builds defined in .goreleaser.yaml
  • Resolves release failures for multi-architecture Docker images

Confidence Score: 5/5

  • This PR is safe to merge - it adds standard infrastructure setup for cross-platform Docker builds
  • The change is minimal, follows established patterns for multi-arch Docker builds, and correctly positions QEMU setup before Docker Buildx. No security concerns or breaking changes.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
.github/workflows/release_build_infisical_cli.yml 5/5 Added QEMU setup step before Docker Buildx to enable cross-platform ARM64 Docker image builds on AMD64 runners. Standard and correct approach for multi-arch builds.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@varonix0 varonix0 merged commit d00ed82 into main Dec 5, 2025
4 checks passed
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.

2 participants