ci(base-cluster/artifacthub-images): Update ArtifactHUB images#1577
ci(base-cluster/artifacthub-images): Update ArtifactHUB images#1577teutonet-bot merged 1 commit intomainfrom
Conversation
WalkthroughThis change updates Docker image version strings for several components in the Helm chart annotations, including upgrades for Grafana Tempo, Memcached, and Grafana Alloy images. No logic, control flow, or exported entities were modified; only version metadata in the Helm chart was changed. Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
charts/base-cluster/Chart.yaml (3)
28-33: Consider pinning the new Tempo tags to immutable digestsThe six updated
bitnami/grafana-tempo:2.8.1-debian-12-r1entries are still tag-only references, whereas many other images in the list are pinned with@sha256:digests.
A digest lock prevents accidental pulls of a mutated tag and keeps the SBOM on ArtifactHUB reproducible.- - image: docker.io/bitnami/grafana-tempo:2.8.1-debian-12-r1 + - image: docker.io/bitnami/grafana-tempo:2.8.1-debian-12-r1@sha256:<tempo-digest>Same applies to lines 29-33.
If you need the digest values I can script-fetch them.
40-40: Memcached bump LGTM, but digest consistency would help
bitnami/memcached:1.6.38-debian-12-r4is a straight patch release; no breaking changes expected.
For the same reproducibility reasons mentioned above, consider pinning it to a digest.
24-85: Long-term maintainability: generate this list instead of manual editsManually curating ~60 image lines is error-prone and easy to fall out of sync (as seen with the missing metrics-generator earlier).
Consider:
- A post-render script that extracts images from the rendered manifests (
yq e '.spec.template.spec.containers[].image') and rewritesChart.yamlbefore commit.- Or a CI guard that diff-checks the list against the rendered output and fails if they diverge.
This keeps ArtifactHUB data accurate without manual toil.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
charts/base-cluster/Chart.yaml(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: generateDiffCommentBody
- GitHub Check: check licenses
- GitHub Check: lint helm chart (base-cluster)
- GitHub Check: wait-for-checks
🔇 Additional comments (2)
charts/base-cluster/Chart.yaml (2)
28-33: Nice catch adding themetrics-generatorentryThe list previously omitted the
grafana-tempo-metrics-generatorStatefulSet, which could confuse ArtifactHUB’s image scanner. The new line 33 fixes the gap.
52-52: Alloy upgrade OK – watch for config deprecations
grafana/alloyjumped fromv1.7.5→v1.9.2. The Alloy release notes mention config-field renames (batch_size→max_batch_size, etc.). Double-check the Helm values rendered fortelemetry-collectorto avoid runtime errors.
🤖 I have diffed this beep boop"/$namespace/$kind/$name.yaml" for normal resources
|
Updates the ArtifactHUB images to the really deployed ones.
Summary by CodeRabbit