jwetzell / docker-guacamole

A self-contained guacamole docker container for x64 and ARM. Remotely connect over SSH, RDP or VNC using HTML5.
https://hub.docker.com/r/oznu/guacamole/
GNU General Public License v3.0
104 stars 17 forks source link

arm64 broken in multi-platform image #50

Closed divinehawk closed 2 months ago

divinehawk commented 2 months ago

Describe The Bug:

Docker image does not work on Raspberry pi (arm64).

The arm64 image appears to have an x86_64 binary: bin/execlineb: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, stripped

To Reproduce: docker run \ -p 8080:8080 \ -v /opt/guacamole:/config \ jwetzell/guacamole:latest

Expected behavior: jwetzell/guacamole:arm64 works jwetzell/guacamole:1.5.5 does not

Logs:

exec /init: exec format error

Environment:

jwetzell commented 2 months ago

Thanks for reporting this, I'll look into what went wrong with those tags. Might take a second for me to get an ARM64 platform up for testing.

divinehawk commented 2 months ago

Adding: ARG TARGETPLATFORM to Dockerfile.base fixes it on my local system. Not sure if github requires it.

jwetzell commented 2 months ago

HA, good catch. Not sure how I missed then when I split things into the base/main image. I'll look into making that change and I've got 64-bit Pi setup so I can test the change.

jwetzell commented 2 months ago

@divinehawk I think I've got it built and pushed properly now. If you could check the jwetzell/guacamole:latest tag that would be great. I've tested on a 64-bit Pi and now don't get the exec format error.

divinehawk commented 2 months ago

It works now. Thanks!

jwetzell commented 2 months ago

Perfect! This has been rolled back into the 1.5.5 tag as well as the latest.