Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

blackbox-exporter: fix broken image reference #2439

Closed

Conversation

AuStien
Copy link

@AuStien AuStien commented Jun 3, 2024

Description

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.

Fixes a broken reference to the configmap-reload image.

The currently used version of the configmap-reload image (0.5.0) isn't hosted on Github (ghcr.io/jimmidyson/configmap-reload), only on Docker Hub (jimmidyson/configmap-reload). Only the versions from 0.10.0 and onwards are hosted on Github (as described in the repository's readme).

The best long term solution would be to bump configmap-reload to a newer version, but this is something I'm not comfortable doing.

Type of change

What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry

Please put a one-line changelog entry below. Later this will be copied to the changelog file.

Fix invalid reference to `configmap-reload` image

The currently used version of the configmap-reload image (0.5.0) isn't hosted
on Github (ghcr.io/jimmidyson/configmap-reload), only on
Docker Hub (jimmidyson/configmap-reload), meaning the current reference is broken.
Only the versions from 0.10.0 and onwards are hosted on Github.
The best long term solution would be to bump configmap-reload to a newer
version, but this is something I'm not comfortable doing.
@AuStien AuStien requested a review from a team as a code owner June 3, 2024 10:33
@AuStien
Copy link
Author

AuStien commented Jun 3, 2024

Never mind, problem was fixed by bumping configmap-reload to 0.13.0 in #2438.

@AuStien AuStien closed this Jun 3, 2024
@AuStien AuStien deleted the fix/configmap-reload-image branch August 16, 2024 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant