Skip to content

[Backport 7.1.x] fix helm manifests to support 2.16 jaeger configs#845

Merged
DaedalusG merged 1 commit into7.1.xfrom
backport-841-to-7.1.x
Mar 19, 2026
Merged

[Backport 7.1.x] fix helm manifests to support 2.16 jaeger configs#845
DaedalusG merged 1 commit into7.1.xfrom
backport-841-to-7.1.x

Conversation

@sourcegraph-release-bot
Copy link
Collaborator

Backport 1b6f92b from #841

closes PLAT-485

Update manifests to support 2.16 jaeger configurations style

This follows changes to the sourcegraph jaeger base image https://github.com/sourcegraph/sourcegraph/pull/10912

Test plan

See testing from original PR, this version of the charts was tested via kind

Tested by generating a template of the jaeger deployment and running

helm unittest charts/sourcegraph -f 'tests/otelCollectorJaeger_test.yaml' -f 'tests/otelAgentHostPort_test.yaml' 2>&1

closes PLAT-485

Update manifests to support 2.16 jaeger configurations style

This follows changes to the sourcegraph jaeger base image
sourcegraph/sourcegraph#10912

### Test plan
See testing from original PR, this version of the charts was tested via
`kind`

Tested by generating a template of the jaeger deployment and running
```
helm unittest charts/sourcegraph -f 'tests/otelCollectorJaeger_test.yaml' -f 'tests/otelAgentHostPort_test.yaml' 2>&1
```

<!--
As part of SOC2/GN-104 and SOC2/GN-105 requirements, all pull requests
are REQUIRED to
provide a "test plan". A test plan is a loose explanation of what you
have done or
implemented to test this, as outlined in our Testing principles and
guidelines:

https://docs.sourcegraph.com/dev/background-information/testing_principles
  Write your test plan here after the "Test plan" header.
-->

---------

Co-authored-by: Michael Lin <mlzc@hey.com>
(cherry picked from commit 1b6f92b)
@DaedalusG DaedalusG merged commit 43fbf49 into 7.1.x Mar 19, 2026
3 checks passed
@DaedalusG DaedalusG deleted the backport-841-to-7.1.x branch March 19, 2026 23:06
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.

2 participants