Skip to content

feat: Remove Prometheus metric about build info#104

Merged
minhtule merged 2 commits intomasterfrom
feat/remove-build-metrics
Aug 11, 2025
Merged

feat: Remove Prometheus metric about build info#104
minhtule merged 2 commits intomasterfrom
feat/remove-build-metrics

Conversation

@minhtule
Copy link
Copy Markdown
Contributor

Changes

Remove build info metric from Go code and Grafana dashboard

@minhtule minhtule requested review from clement0010 and ekampf August 11, 2025 05:14
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 16871480624

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.05%) to 92.344%

Totals Coverage Status
Change from base Build 16840451160: -0.05%
Covered Lines: 1363
Relevant Lines: 1476

💛 - Coveralls

@minhtule minhtule merged commit 204d9c2 into master Aug 11, 2025
11 of 12 checks passed
@minhtule minhtule deleted the feat/remove-build-metrics branch August 11, 2025 15:30
clement0010 added a commit that referenced this pull request Aug 29, 2025
## Changes
- Fix `configmap-grafana-dashboard` snapshot test

## Notes
Unfortunately, `helm-unittest@v1.0.0` is having a [known
issue](helm-unittest/helm-unittest#712) with
`matchSnapshot`, that is why the `test-helm` CI did not fail in this
[PR](#104). I
only found this issue locally because I was using an `helm-unittest`
older version.
minhtule added a commit that referenced this pull request Dec 30, 2025
…#104)

## Changes

- Add `Resource.Type` field to GAT
- Use the resource type to determine the inner transport protocol
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