* Move publish of go stream perf test docker image to its own workflow

Author: lukebakken

.github/workflows/build_and_test.yml

@@ -9,6 +9,9 @@ on:
       - main
   workflow_dispatch:
 
+permissions:
+  contents: read
+
 jobs:
   docker-build:
     runs-on: ubuntu-latest
@@ -93,23 +96,3 @@ jobs:
       - name: Run Ginkgo test suite
         shell: pwsh
         run: go.exe run -mod=mod github.com/onsi/ginkgo/v2/ginkgo -r --procs=$env:NUMBER_OF_PROCESSORS --compilers=$env:NUMBER_OF_PROCESSORS --randomize-all --randomize-suites --cover --coverprofile=coverage.txt --covermode=atomic --race --trace --tags debug --timeout=5m
-  publish:
-    runs-on: ubuntu-latest 
-    needs: test
-    steps:
-      - uses: docker/setup-buildx-action@e468171a9de216ec08956ac3ada2f0791b6bd435 # v3.11.1
-      - uses: docker/login-action@5e57cd118135c172c3672efd75eb46360885c0ef # v3.6.0
-        with:
-          username: ${{ secrets.DOCKERHUB_USERNAME }}
-          password: ${{ secrets.DOCKERHUB_TOKEN }}
-      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
-      - name: Publish Docker Image
-        run: |
-          set -x
-          VERSION=latest
-          export VERSION
-          if [[ ! $GITHUB_REF =~ "/tags/" ]]
-          then
-            VERSION=dev
-          fi
-          make perf-test-docker-push

.github/workflows/perf-test-publish.yml

@@ -0,0 +1,30 @@
+name: Publish Go Stream Perf Test
+
+on:
+  push:
+    branches:
+      - main
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest 
+    steps:
+      - uses: docker/setup-buildx-action@e468171a9de216ec08956ac3ada2f0791b6bd435 # v3.11.1
+      - uses: docker/login-action@5e57cd118135c172c3672efd75eb46360885c0ef # v3.6.0
+        with:
+          username: ${{ secrets.DOCKERHUB_USERNAME }}
+          password: ${{ secrets.DOCKERHUB_TOKEN }}
+      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
+      - name: Publish Docker Image
+        run: |
+          set -eux
+          export VERSION=latest
+          if [[ ! $GITHUB_REF =~ "/tags/" ]]
+          then
+            VERSION=dev
+          fi
+          make perf-test-docker-push

Makefile

@@ -57,7 +57,7 @@ perf-test-docker-build: perf-test-build
 perf-test-docker-push: perf-test-docker-build
 	$(BUILDKIT) push pivotalrabbitmq/go-stream-perf-test:$(VERSION)
 
-RABBITMQ_OCI ?= rabbitmq:3-management
+RABBITMQ_OCI ?= rabbitmq:4-management
 BUILDKIT_RUN_ARGS ?= --pull always
 .PHONY: rabbitmq-server
 rabbitmq-server:
@@ -67,7 +67,6 @@ rabbitmq-server:
 		-e RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS="-rabbitmq_stream advertised_host localhost" \
 		rabbitmq-tls-test
 
-
 rabbitmq-ha-proxy:
 	cd compose/ha_tls; rm -rf tls-gen;
 	cd compose/ha_tls; git clone https://github.com/michaelklishin/tls-gen tls-gen; cd tls-gen/basic; make
@@ -87,10 +86,7 @@ rabbitmq-server-tls:
 		-v  $(shell pwd)/compose/tls/conf/:/etc/rabbitmq/ -v $(shell pwd)/compose/tls/tls-gen/basic/result/:/certs \
 		-e RABBITMQ_SERVER_ADDITIONAL_ERL_ARGS="-rabbitmq_stream advertised_host localhost" \
 		--pull always \
-		docker.io/rabbitmq:3-management
+		docker.io/rabbitmq:4-management
 
 local-release:
 	goreleaser release --skip-publish --rm-dist --skip=validate
-
-
-