Don't require secrets for running tests (#45)

* Don't require secrets for running tests

* make PRs work a little bit better
This commit is contained in:
Rob Landers
2022-10-20 14:44:44 +02:00
committed by GitHub
parent 3ff6cd370c
commit 4a52e0eb35
2 changed files with 6 additions and 13 deletions
+4 -11
View File
@@ -25,7 +25,7 @@ jobs:
push: false
pull: true
target: builder
tags: ${{secrets.REGISTRY_LOGIN_SERVER}}/${{ secrets.REGISTRY_USERNAME }}/frankenphp:${{ github.sha }}-builder
tags: frankenphp:${{ github.sha }}-builder
builder: ${{ steps.buildx.outputs.name }}
cache-from: type=gha
cache-to: type=gha,mode=max
@@ -34,7 +34,7 @@ jobs:
- name: Run tests
run: |
docker load -i /tmp/.builder.tar
docker run --rm ${{secrets.REGISTRY_LOGIN_SERVER}}/${{ secrets.REGISTRY_USERNAME }}/frankenphp:${{ github.sha }}-builder "go test"
docker run --rm frankenphp:${{ github.sha }}-builder "go test"
push-image:
runs-on: ubuntu-latest
steps:
@@ -42,20 +42,13 @@ jobs:
with:
submodules: recursive
- name: Docker Login
uses: docker/login-action@v2
with:
registry: ${{secrets.REGISTRY_LOGIN_SERVER}}
username: ${{secrets.REGISTRY_USERNAME}}
password: ${{secrets.REGISTRY_PASSWORD}}
- name: Docker meta
id: meta
uses: docker/metadata-action@v4
with:
# list of Docker images to use as base name for tags
images: |
${{ secrets.REGISTRY_LOGIN_SERVER }}/${{ secrets.REGISTRY_REPO }}/frankenphp
frankenphp
# generate Docker tags based on the following events/attributes
tags: |
type=schedule
@@ -80,7 +73,7 @@ jobs:
with:
context: ./
file: Dockerfile
push: true
push: false
pull: true
target: final
platforms: linux/amd64,linux/arm64