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

feat: add multiarch build and release #278

Merged
merged 2 commits into from
Mar 22, 2022
Merged

feat: add multiarch build and release #278

merged 2 commits into from
Mar 22, 2022

Conversation

kalidax
Copy link
Contributor

@kalidax kalidax commented Mar 2, 2022

This job will

  • Build and push an image to dockerhub.

  • Build tofnd binary, gpg sign it, zip it, create sha256 hashes for both zip and binary, upload files to release and S3.

  • For binary build the job will run on each combination of OS/arch defined in the matrix.

  • For the container build it will run one time only on ubuntu-18.04/amd64

  • Multiarch binary tests without docker push, upload to release and upload to s3 have been done here : https://github.com/axelarnetwork/tofnd/actions/runs/1921406385

Copy link
Contributor

@ggutoski ggutoski left a comment

Choose a reason for hiding this comment

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

Approving as codeowner but @talalashraf or @erain9 should approve before merging.

This PR subsumes #274 so we can close that after merge.

@@ -36,59 +37,102 @@ jobs:
submodules: recursive

- name: Install Cosign
if: matrix.os == 'ubuntu-18.04' && matrix.arch == 'amd64'
Copy link
Member

Choose a reason for hiding this comment

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

Why not sign all binaries?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

because it is related to docker images. Binaries are signed in gpg section

run: |
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y

- name: build binaries for Linux/MacOS
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- name: build binaries for Linux/MacOS
- name: Build binaries

Perhaps?

@kalidax
Copy link
Contributor Author

kalidax commented Mar 9, 2022

Update that will improve the build time.

@kalidax kalidax force-pushed the feat/multiarch-build branch from 2c5d941 to b9f541e Compare March 9, 2022 11:32
@kalidax kalidax requested review from ggutoski and milapsheth March 9, 2022 11:50
@talalashraf
Copy link
Contributor

@kalidax @milapsheth @ggutoski can we get this across the line ?

@ggutoski ggutoski merged commit 82793df into main Mar 22, 2022
@ggutoski ggutoski deleted the feat/multiarch-build branch March 22, 2022 00:19
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.

4 participants