-
Notifications
You must be signed in to change notification settings - Fork 105
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: adding matrix build #1319
Conversation
you tested this in another branch right ? can i see the outcome there ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not delete the binary signing logic in the workflow
Nevermind - please ignore this. |
|
||
- name: Unit test (check ledger support) | ||
run: | | ||
./bin/axelard-* version --long |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
./bin/axelard-* version --long | |
./bin/axelard-"$OS"-"$ARCH"-"$SEMVER" version --long | grep "build_tags: ledger" |
Don't you want to grep
for ledger
in the build_tags and check if it's present? And just to be precise
ab23065
to
0231561
Compare
Co-authored-by: kalid <kalidax@axelar.network>
Description
Todos
Steps to Test
Run the https://github.com/axelarnetwork/axelar-core/blob/feat/build-matrix/.github/workflows/build-docker-image-and-binaries.yaml . It is a job triggered manually.
select branch feat/build-matrix.
ideally create a S3 and an ECR for testing purpose.
Enter a semantic tag, you should not use an existing tag (VERY IMPORTANT), if you do so you will overwrite the existing binaries/images. You can check the latest version in the S3 axelar-releases.
Expected Behavior
The job will run on 5 os/architectures (3 Linux, 2 Darwin).
The build of container images will occur only on Ubuntu/amd64 that is the only job that will run all the steps
In the 4 remaining jobs, all container related steps will be skipped.
Other Notes
To ensure that Ledger is supported, you can check in the test step of each job see https://github.com/axelarnetwork/axelar-core/runs/5248523237?check_suite_focus=true#step:13:10 for example.
you should see "build_tags: ledger"