Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(github): support arm in the official release #2007

Closed
wants to merge 2 commits into from

Conversation

sideninja
Copy link

@sideninja sideninja commented Sep 30, 2024

Add build variant for linux/arm platform.

The PR modifies the goreleaser config to support building for arm architecture. It also modifies the release-official action for halovisor to support building for the arm.

issue: #1984

Tested goreleaser build locally:

goreleaser release --snapshot --skip publish --clean

  • skipping announce, publish and validate...
  • cleaning distribution directory
  • loading environment variables
  • getting and validating git state
    • git state                                      commit=fabe4ac6fe1ceea6252d651b0d70697ca0fa6411 branch=gregor/multi-platform-release current_tag=v0.8.0 previous_tag=v0.7.0 dirty=false
    • pipe skipped                                   reason=disabled during snapshot mode
  • parsing tag
  • setting defaults
  • snapshotting
    • building snapshot...                           version=0.8.0-SNAPSHOT-fabe4ac
  • ensuring distribution directory
  • setting up metadata
  • writing release metadata
  • loading go mod information
  • build prerequisites
  • building binaries
    • building                                       binary=dist/halo_linux_amd64_v1/halo
    • building                                       binary=dist/relayer_linux_amd64_v1/relayer
    • building                                       binary=dist/relayer_darwin_amd64_v1/relayer
    • building                                       binary=dist/halo_linux_arm64/halo
    • building                                       binary=dist/halo_darwin_arm64/halo
    • building                                       binary=dist/halo_darwin_amd64_v1/halo
    • building                                       binary=dist/relayer_linux_arm64/relayer
    • building                                       binary=dist/relayer_darwin_arm64/relayer
    • building                                       binary=dist/monitor_linux_amd64_v1/monitor
    • building                                       binary=dist/monitor_linux_arm64/monitor
    • building                                       binary=dist/monitor_darwin_amd64_v1/monitor
    • building                                       binary=dist/monitor_darwin_arm64/monitor
    • building                                       binary=dist/omni_linux_amd64_v1/omni
    • building                                       binary=dist/omni_linux_arm64/omni
    • building                                       binary=dist/omni_darwin_amd64_v1/omni
    • building                                       binary=dist/omni_darwin_arm64/omni
    • building                                       binary=dist/anvilproxy_linux_amd64_v1/anvilproxy
    • building                                       binary=dist/anvilproxy_linux_arm64/anvilproxy
    • took: 38s
  • archives
    • creating                                       archive=dist/omni_Linux_arm64.tar.gz
    • creating                                       archive=dist/omni_Darwin_x86_64.tar.gz
    • creating                                       archive=dist/omni_Darwin_arm64.tar.gz
    • creating                                       archive=dist/omni_Linux_x86_64.tar.gz
    • took: 15s
  • calculating checksums
  • docker images

  • writing artifacts metadata
  • release succeeded after 53s
  • thanks for using goreleaser!

@sideninja
Copy link
Author

sideninja commented Sep 30, 2024

@corverroos
Follow up: should we modify the goreleaser to also include the halovisor and make it more consistent? why is halovisor part of the release action and not included in goreleaser config?

todo: need to validate the whole action against a fork.

@sideninja sideninja self-assigned this Sep 30, 2024
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.

1 participant