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

[Feature]: Support Jaeger-v2 Operator #2411

Closed
yurishkuro opened this issue Jan 8, 2024 · 2 comments
Closed

[Feature]: Support Jaeger-v2 Operator #2411

yurishkuro opened this issue Jan 8, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Jan 8, 2024

Requirement

Once Jaeger-v2 (jaegertracing/jaeger#4843) reaches production readiness and feature parity, we should support it in the operator.

Problem

Jaeger-v2 is based on OpenTelemetry Collector and uses completely different configuration mechanism:

  • all configuration is done via config files, not CLI flags or env vars (although some env vars are supported by telemetry SDKs)
  • it is a single binary that can be deployed into different roles (collector, query, etc.) also based on configuration, but in this case with structural config changes, not just tweaking parameters

Proposal

I would like to hear what maintainers of this repo think about the best approach. Can we come up with a roadmap of how jaeger-v2 will be supported by the operator? Is there some common approach we can use with the OTEL operator?

Open questions

  • Is it worth maintaining both helm charts and operator?
@dipankardas011
Copy link

not sure if we actually need helm charts as we have to deploy via the kubernetes operator method

still need to have a discussion with otel operator folks

dipankardas011 added a commit to dipankardas011/jaeger-operator that referenced this issue Mar 3, 2024
only the default is tested
TODO: need to add configmap support

related to Issue jaegertracing#2411

Signed-off-by: Dipankar Das <[email protected]>
dipankardas011 added a commit to dipankardas011/jaeger-operator that referenced this issue Mar 7, 2024
only the default is tested
TODO: need to add configmap support

related to Issue jaegertracing#2411

Signed-off-by: Dipankar Das <[email protected]>
@yurishkuro yurishkuro changed the title [Feature]: Support Jaeger-v2 [Feature]: Support Jaeger-v2 Operator Aug 31, 2024
@yurishkuro
Copy link
Member Author

Closing as a duplicate of #2717 (which was moved from Jaeger repo)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants