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
108 stars 18 forks source link

Architecture mismatch with the latest version 1.5.0 #13

Closed CypZ closed 1 year ago

CypZ commented 1 year ago

Describe The Bug: With the latest version 1.5.0 I had to fix the version in the docker compose file (jwetzell/guacamole:1.5.0-arm64), because there is a mismatch of the architecture in the Docker Image file (jwetzell/guacamole:arm64)

To Reproduce: Dockerfile jwetzell/guacamole:arm64 line 27:amd64 (instead of arm64) Dockerfile jwetzell/guacamole:1.5.0-arm64 line 27 armhf is correct.

Expected behavior: Dockerfile jwetzell/guacamole:arm64 line 27:armhf or arm64

Environment:

jwetzell commented 1 year ago

I think I have corrected this now. @CypZ

CypZ commented 1 year ago

confirmed working, thanks

CypZ commented 1 year ago

Again, same error tag=arm64 and tag=1.5.1-arm64

Bring my prod dowwn each and every time...

jwetzell commented 1 year ago

Ok, I will take a look, rebuild, and reupload

CypZ commented 1 year ago

hint: typo in the build pipeline ? Confusion between amd64 and arm64 ?

jwetzell commented 1 year ago

I think what happen was I built it using the arm image but didn't change the ARCH environment variable so it still installed the amd64 version of guacamole/s6-overlay, etc.

jwetzell commented 1 year ago

alright @CypZ lets try this again. New images should be pushed as well as 1.5.2 which recently came out.

CypZ commented 1 year ago

yes, (I only tested) the latest image with the tag arm64 (1.5.2 as of today) has now the correct arch.

Finger crossed for the next version ;)

jwetzell commented 1 year ago

I've hopefully laid things out so I can't screw this up next time.