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

chore: makefile for building binary and upload to S3 #1

Merged
merged 2 commits into from
Jul 31, 2023
Merged

Conversation

honeyAcorn
Copy link
Collaborator

@honeyAcorn honeyAcorn commented Jul 28, 2023

We are going to upload the rezolus binary to S3 via pipelines.
This way we can use the rezolus and run it in ms2 or mr2 whenever we need them.

There is another PR in pipeline-of-pipelines that uses the makefile here:
https://github.com/momentohq/pipeline-of-pipelines/pull/642
So this makefile will need to be merged first before that PR can be merged.

pipeline-build-arm64v8:
/sbin/bpftool btf dump file /sys/kernel/btf/vmlinux format c > src/common/bpf/vmlinux.h \
&& cargo build --release --features bpf \
&& ./package.sh arm64v8/rezolus.tar.gz target/release/rezolus \

Choose a reason for hiding this comment

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

package.sh appears to be missing contents 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

🤦 so embarrassed. Fixed now 4ef393e

&& ./package.sh amd64/rezolus.tar.gz target/release/rezolus \

pipeline-build-arm64v8:
/sbin/bpftool btf dump file /sys/kernel/btf/vmlinux format c > src/common/bpf/vmlinux.h \

Choose a reason for hiding this comment

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

you don't need to do this bpftool thing for amd64?

Copy link
Collaborator Author

@honeyAcorn honeyAcorn Jul 31, 2023

Choose a reason for hiding this comment

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

According to Brian, he said this is only for apple chip, but if it goes wrong then I will come back and fix it! Thanks.

@honeyAcorn honeyAcorn merged commit 5e73d3b into main Jul 31, 2023
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