-
Notifications
You must be signed in to change notification settings - Fork 7
Patch to include check if system supports architecture of lokl image #27
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
base: master
Are you sure you want to change the base?
Conversation
|
Awesome, thanks for sending this @nevehallon. I'll check this and the reported issue out soon |
|
@nevehallon - the check in Would a |
|
To elaborate, sorry. That path doesn't seem to exist outside of Debian-based distros or others which have specifically installed |
|
@nevehallon what architecture was your Lubuntu running on? Arm64? |
It's running on amd64 |
uname doesn't provide achitecture information. At least not on my OS... |
|
Try 'uname -a', that usually ends with arch on *buntus |
|
I see. |
|
Or, perhaps a more concise check would be running |
|
That looks perfect! Just tested on my M1 mac and a remote OpenBSD machine: leon@Leons-MacBook-Pro ~ % docker buildx ls
NAME/NODE DRIVER/ENDPOINT STATUS PLATFORMS
default * docker
default default running linux/arm64, linux/amd64, linux/ppc64le, linux/s390x, linux/386, linux/arm/v7, linux/arm/v6
leon@Leons-MacBook-Pro ~ % arch
arm64
leon@Leons-MacBook-Pro ~ % ssh someopenbsd
someopenbsd$ arch
OpenBSD.amd64That Will need to finish my current task before can properly look into this, but if you want to adjust the PR, I can test the scripts against a variety of architecture VPS's later to try and reproduce the issue. (There is an argument to just show the arch with OpenBSD's arch command, might be similar on other BSDs, too. Noting, that OpenBSD has no ability to run Docker/Lokl yet, but I believe it might work fine on FreeBSD). |
|
Wasn't sure if I should include
@leonstafford Let me know if this adjustment works. |
|
Many thanks @nevehallon! It will take me a bit of time to get to test this on a few systems, but looks great! |
In relation to https://github.com/leonstafford/lokl/issues/34
Performing check in the case that there is a timeout.