caas-team / sparrow

A monitoring tool to gather infrastructure network information
Apache License 2.0
6 stars 4 forks source link

Push to MTR directly #127

Closed puffitos closed 2 months ago

puffitos commented 2 months ago

Motivation

We don't need to depend to MTR doing its mirroring for the image. We can directly push the image here once build and be done with it.

Mirroring is a deferred process, which hasn't been working lately (at least of the sparrow repository). Switching to a push based approach, given the small volume of traffic on this repository, is the simplest solution to move & test quickly.

Changes

Changed the CI actions to push the OCI artrifacts to MTR as well.

For additional information look at the commits.

Tests done

None needed.

TODO

puffitos commented 2 months ago

@y-eight it was a combination of multiple things... Typos, missing secrets, the go-releaser job doesn't allow one to add extra tags (the whole image, not just the tag in the end) so no mtr.devops.telekom.de image was available and so on. Now eveything works, with a small caveat: the charts are now being pushed to /charts/sparrow instead of just /chart (it's not possible AFAIK to define the repository, as helm always adds the name of the release to the repository you're uploading to)

@lvlcn-t I think that's not a problem. As long as MTR is public, I think it's something we may do. If it isn't anymore, we can remove the pushing and do some other kind of trick to mirror the images.