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

add logic to handle downloading requestor binaries on linux arm (aarch64) #44

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

krunch3r76
Copy link

@krunch3r76 krunch3r76 commented Jul 17, 2023

currently requestor installations are available via manual install but with a little bash scripting modification this can be automated via the as-requestor script. the modifications i have made should produces as-requestor (as-provider) production and development bash installation scripts via the standard gen.py to automatically work for a requestor installation on linux with an arm based cpu (aarch64) but to emit an error as expected on provider installs.

the only file modified is the template install.sh and specifically only one function is modified, detect_dist(), so that:

  1. detect_dist() now tests for _cputype reported as aarch64 and fails if it is a provider install
  2. detect_dust() post processes _ostype script variable in linux where cputype is aarch64 so that '_aarch64' is concatenated to 'linux' to correctly resolve to the canonical naming scheme used for linux arm variants e.g:
  • golem-requestor-linux-v0.12.2.tar.gz
  • golem-requestor-linux_aarch64-v0.12.2.tar.gz

note, the given code makes it easy to support provider installs on linux arm when available simply by commenting or removing the logic that tests for "provider"

this pull shall close issue #45 and #24

@krunch3r76 krunch3r76 changed the title add logic to handle downloading requestor binaries on arm (aarch64) add logic to handle downloading requestor binaries on linux arm (aarch64) Jul 17, 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.

1 participant