jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
267 stars 347 forks source link

[Feature]: Add ability to set resources for the Oauth2 sidecar #506

Closed Stevenpc3 closed 9 months ago

Stevenpc3 commented 1 year ago

Requirement

As a tool developer for the organization I want to be able to deploy Jaeger with our tests dynamically. This requires pushing to the cluster in a namespace that has quotas set. Jaeger fails with Oauth2 sidecar enabled because the sidecar does not allow setting resources.

I would like to integrate this PR https://github.com/jaegertracing/helm-charts/pull/505 to allow this to happen.

Then Jaeger helm chart can be used in systems where a quota is set and limit range is not used.

Problem

Jaeger query fails come up in a kubernetes system with resource quotas with Oauth2 sidecar enabled because the sidecar does not allow setting resources.

Proposal

Enable resources to be set in the query-deploy.yaml template file. https://github.com/jaegertracing/helm-charts/pull/505

Open questions

No response

Stevenpc3 commented 9 months ago

PR was merged in #505