discourse / prometheus_exporter

A framework for collecting and aggregating prometheus metrics
MIT License
525 stars 153 forks source link

Fix Github Actions workflow for publish_docker #303

Closed pcallewaert closed 5 months ago

pcallewaert commented 5 months ago

Hi 👋

This PR fixes 2 issues with the current workflow:

It's not that easy to test, but I tested this out on my fork (with some workarounds) and the publish_docker job triggered now successfully: https://github.com/pcallewaert/prometheus_exporter/actions/runs/7637020510

This should fix https://github.com/discourse/prometheus_exporter/issues/301

sosedoff commented 5 months ago

Interesting. When i worked on the release flow i also used my own fork for testing, and it was working initially. Suspecting there were slight changes in the action that we missed since no new versions of this gem were published for a while (about a year?).

SamSaffron commented 5 months ago

I am more than happy to try it out!