CI: Grafana Docker push fails

[JakobLichterfeld]

The push of new grafana image fails due to auth, @jlestel

https://github.com/teslamate-org/teslamate/actions/runs/6988327850/job/19015781415

v1.28.0

[rpearson-catdevnull]

Was just about to open an issue of not being able to see the LFP dashboards from PR #3311. This is probably related right?

edit: it will be as I’m pulling the :latest tag and that won’t have updated due to the failure 👍

[rpearson-catdevnull]

You’re missing the docker login command, which you have in your main teslamate build here, in the build script for your grafana action.

You’ll need to log in for each action due to the scope of the login not persisting between the two actions I believe. They’re separate jobs so will run on two separate action runners

[JakobLichterfeld]

Yeah, assume the same. As soon as the job failed, I switch to pre-release, so not everybody is complaining :-) GHCR images are good.

[jlestel]

@JakobLichterfeld sorry for the oversight I pushed a fix that should allow you to push correctly.

[JakobLichterfeld]

@JakobLichterfeld sorry for the oversight I pushed a fix that should allow you to push correctly.

No problem. We had already found it in a review, but we were both confused by the accidental deletion of the fork branch :-)

[JakobLichterfeld]

Will double-check tomorrow or on Monday before final releasing :-)

[JakobLichterfeld]

closed with #3491