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

feat: cedana-otel helm values.yaml #31

Merged
merged 3 commits into from
Nov 1, 2024
Merged

feat: cedana-otel helm values.yaml #31

merged 3 commits into from
Nov 1, 2024

Conversation

rahulk789
Copy link
Contributor

Describe your changes

This PR is ought to allow users to install otel through the official helm chart of otel with our specific otel configuration.

Caveats (if any)

Please merge only after rigorous testing.

Issue Ticket ID

Checklist before requesting a review

  • Code compiles correctly.
  • Self-review to ensure, the PR only introduces the changes it's intended to? E.g. No extraneous formatting changes.
  • Created tests which fail without the change (if possible).
  • Rest of the tests are passing.
  • Is this a breaking change? If yes, please describe areas affected and request reviews from developer stakeholders.
  • Extended the README / documentation, if changes have resulted in it being out of date?

@nravic nravic merged commit 26fbdc0 into main Nov 1, 2024
2 checks passed
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