Skip to content

[tempo-distributed] standardize .helmignore#81

Merged
jkroepke merged 4 commits intografana-community:mainfrom
TheRealNoob:tempo-distributed
Feb 13, 2026
Merged

[tempo-distributed] standardize .helmignore#81
jkroepke merged 4 commits intografana-community:mainfrom
TheRealNoob:tempo-distributed

Conversation

@TheRealNoob
Copy link
Copy Markdown
Member

What this PR does / why we need it

fork of #65

Which issue this PR fixes

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [grafana])

Signed-off-by: TheRealNoob <mike1118@live.com>
jkroepke and others added 3 commits February 13, 2026 18:56
Signed-off-by: TheRealNoob <mike1118@live.com>
Signed-off-by: TheRealNoob <mike1118@live.com>
@TheRealNoob
Copy link
Copy Markdown
Member Author

TheRealNoob commented Feb 13, 2026

So I made the minimum changes to satisfy linting, but i'm really not sure about some of them. Do all of the values take in the values case-insensitive? @jkroepke made the suggestion they could be moved away from helm-docs comments to regular comments like so

- # -- ...gRPC...
+ # ...gRPC...

but i'm really not sure. I don't think there's a clean solution between these two tools.

@jkroepke jkroepke merged commit 99decf5 into grafana-community:main Feb 13, 2026
7 checks passed
TheRealNoob added a commit to TheRealNoob/grafana-community-helm-charts that referenced this pull request Feb 22, 2026
changes were made in grafana-community#81

Signed-off-by: TheRealNoob <mike1118@live.com>
QuentinBisson pushed a commit that referenced this pull request Feb 22, 2026
* [tempo-distributed] Remove helm-docs, standardize README

- Delete README.md.gotmpl
- Rewrite README.md to standardized format with OCI + HTTP install
  sections, changelog link, and no auto-generated badges/Values table
  or version footer
- Preserve Upgrading, Components, and Configuration sections
- Update dependency table versions to match Chart.yaml
- Bump version 2.3.1 → 2.3.2

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: TheRealNoob <mike1118@live.com>

* revert some values.yaml changes

changes were made in #81

Signed-off-by: TheRealNoob <mike1118@live.com>

---------

Signed-off-by: TheRealNoob <mike1118@live.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants