Add ARM support - Githubissues

napalmz commented 3 years ago

I have used this program for 2 years on x86 but now, for reasons of reliability, I have to migrate it to an ARM (Rpi). Since the NGINX part works on ARM (SWAG image by Linuxserver.io), I was hoping to be able to keep the SSO part as well (instead of splitting it on another x86 machine). Thank you

Luzifer commented 3 years ago

This is already the case: If you have a look at the releases there is already a nginx-sso_linux_arm.tar.gz download included. That one should run perfectly fine on a Raspi…

napalmz commented 3 years ago

Thank you for your quick answer. Is there also a docker image available? I can't find the right tag...

Luzifer commented 3 years ago

Hm no, currently there is only an amd64 Docker image available…

Will have a look how to do this the best way and whether my Jenkins (running amd64) can properly provide one… 🤔

Luzifer commented 3 years ago

Had a look into this: I'm not able to provide an arm64v8 image as I don't have a build-system for that architecture. If you have a look at the Dockerfile, it must install bash and ca-certificates but I don't see a chance to install those packages without apk being able to run inside the container.

I could write a Dockerfile on a theoretical base (I cannot test it) to build an arm64v8 container but you would need to run that build yourself.

napalmz commented 3 years ago

I tried by myself yesterday but the generated image couldn’t run properly. If you can provide a proper Dockerfile I can build it on my Rpi and test it. Thank you

Luzifer commented 3 years ago

Please have a look at #66 - That Dockerfile should give you a running image for arm64v8 architecture.

napalmz commented 2 years ago

Having an armv7l (therefore 32bit) I created my own specific dockerfile starting from yours. I was able to get an usable image. Unfortunately I changed the dumb-init installation and therefore I also had to change the docker-start.sh, because now the path is no longer /usr/local/bin/dumb-init but /usr/bin/dumb-init. Perhaps with a simple cp command it could be solved without modifying the file.

Here's the code:

FROM golang:alpine as builder

ADD . /go/src/github.com/Luzifer/nginx-sso
WORKDIR /go/src/github.com/Luzifer/nginx-sso

ENV CGO_ENABLED=0 \
    GOOS=linux \
    GOARCH=arm

RUN set -ex \
 && apk add --update \
      git \
 && go install \
      -ldflags "-X main.version=$(git describe --tags || git rev-parse --short HEAD || echo dev)" \
      -mod=readonly

FROM alpine:latest

LABEL maintainer "Knut Ahlers <knut@ahlers.me>"

RUN set -ex \
 && apk --no-cache add \
      bash \
      ca-certificates \
      curl \
      dumb-init \
 && apk --no-cache del --purge \
      curl

COPY --from=builder /go/bin/nginx-sso                                     /usr/local/bin/
COPY --from=builder /go/src/github.com/Luzifer/nginx-sso/config.yaml      /usr/local/share/nginx-sso/
COPY --from=builder /go/src/github.com/Luzifer/nginx-sso/docker-start.sh  /usr/local/bin/
COPY --from=builder /go/src/github.com/Luzifer/nginx-sso/frontend/*       /usr/local/share/nginx-sso/frontend/

EXPOSE 8082
VOLUME ["/data"]

ENTRYPOINT ["/usr/local/bin/docker-start.sh"]
CMD ["--"]

# vim: set ft=Dockerfile: