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

Update operator.md #4401

Merged
merged 2 commits into from
May 25, 2024
Merged

Update operator.md #4401

merged 2 commits into from
May 25, 2024

Conversation

Tarun-Arora
Copy link
Contributor

Please update the helm charts documentation here

Thank You.

@Tarun-Arora Tarun-Arora requested review from a team as code owners May 1, 2024 04:34
Copy link

linux-foundation-easycla bot commented May 1, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: cartermp / name: Phillip Carter (747e02a)
  • ✅ login: Tarun-Arora / name: Tarun Arora (e92f2b8)

@theletterf
Copy link
Member

@Tarun-Arora Please sign the CLA.

@open-telemetry/helm-maintainers Please have a look, thank you!

@Tarun-Arora
Copy link
Contributor Author

@theletterf Done, thanks!

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

RSLGTM - waiting on language SIG approvers

@svrnm svrnm added the sig-approval-missing Co-owning SIG didn't provide an approval label May 2, 2024
@cartermp cartermp merged commit 91a476f into open-telemetry:main May 25, 2024
15 checks passed
@cartermp
Copy link
Contributor

Merging since it's a small change, it's been a while, and I verified this is the value that the charts check for.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sig:helm sig-approval-missing Co-owning SIG didn't provide an approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants