Skip to content

fix: pass daemon collector image tag from chart version#888

Merged
gmlexx merged 1 commit into
mainfrom
fix-main-builds
Mar 26, 2026
Merged

fix: pass daemon collector image tag from chart version#888
gmlexx merged 1 commit into
mainfrom
fix-main-builds

Conversation

@gmlexx
Copy link
Copy Markdown
Collaborator

@gmlexx gmlexx commented Mar 26, 2026

Fixing installation from latest main builds

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to ensure the OpenTelemetry daemon collector image tag used in multi-cluster deployments (regional/child) tracks the Helm chart version, reducing drift between chart releases and deployed collector images.

Changes:

  • Set opentelemetry-kube-stack.collectors.daemon.image.tag to the chart version in the regional MultiClusterService template.
  • Set opentelemetry-kube-stack.collectors.daemon.image.tag to the chart version in the child MultiClusterService template (non-Istio branch).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
charts/kof-regional/templates/regional-multi-cluster-service.yaml Injects daemon collector image tag into the kof-collectors values payload using the chart version.
charts/kof-child/templates/child-multi-cluster-service.yaml Injects daemon collector image tag into the child collectors values payload (currently only for non-Istio).

Comment thread charts/kof-child/templates/child-multi-cluster-service.yaml
Comment thread charts/kof-regional/templates/regional-multi-cluster-service.yaml Outdated
Comment thread charts/kof-child/templates/child-multi-cluster-service.yaml Outdated
@gmlexx gmlexx merged commit e2a517d into main Mar 26, 2026
19 of 21 checks passed
@gmlexx gmlexx deleted the fix-main-builds branch March 26, 2026 14:39
@github-project-automation github-project-automation Bot moved this to Done in k0rdent Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants