fix(chart): standardize tracesSamplerArg comment#102
Conversation
🤖 Generated with [Claude Code](https://claude.com/claude-code) Issue: LFXV2-2568 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
This PR standardizes a single Helm chart comment. It expands the previous one-line description of the tracesSamplerArg OTEL value into the canonical three-line form used by lfx-v2-email-service, adding a (default: "") footer. I verified via code search that the new wording matches the canonical source (charts/lfx-v2-email-service/values.yaml) exactly. This is a documentation-only change with no functional impact.
Changes:
- Replaces the single-line
tracesSamplerArgcomment with the canonical 3-line form. - Adds detail about which samplers use the argument and the
(default: "")footer.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
tracesSamplerArgHelm chart comment to match the canonical wording fromlfx-v2-email-service(default: "")footerTest plan
values.yamlmatches canonical format🤖 Generated with Claude Code