nais / docker-build-push

MIT License
0 stars 5 forks source link

Mulighet for å bygge flere image typer og flere tags. #2

Closed mrsladek closed 6 months ago

mrsladek commented 1 year ago

Hei, Kjempe initiativ med egen action! Her kommer noe forslag fra min siden. Ville det være mulig å bygge images for forskjellige platform, få støtte for flere tags og pulle underliggende images?

Vi bruke noe slik per i dag og ser at dere bruker docker/build-push-action@v4 under panseret også så jeg regner med at det blir mulig å etablere noe sånt.

      - name: Set up QEMU
        if: inputs.build-image
        uses: docker/setup-qemu-action@v2

      - name: Set up Docker Buildx
        if: inputs.build-image
        uses: docker/setup-buildx-action@v2

      - name: Build and push
        if: inputs.build-image
        uses: docker/build-push-action@v4
        with:
          context: .
          **platforms: linux/amd64, linux/arm64** # støtte både for Intel og Apple
          **pull: true** # greit med en pull slit at underliggende images får en update
          push: ${{ inputs.push-image }}
          **tags: app:tag1, app:latest** # støtte for flere tags
          cache-from: type=gha
          cache-to: type=gha,mode=max
thokra-nav commented 1 year ago

Hei og takk for tilbakemeldingene!

Lurer på om dette er litt utenfor scopet til actionet. Per nå så er nok beste løsningen å heller gjøre det selv i egen workflow og heller bruke https://github.com/nais/login .

Vi ønsker i hovedsak å standardisere tags (støtter å legge til én for f.eks. caching eller lignende), og QEMU kan gjøre bygginga veldig treg.

Men pull legger vi til :)

thokra-nav commented 6 months ago

Løst av #35