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

Knylander grafana ts 413 metrics query doc update #4247

Closed

Conversation

knylander-grafana
Copy link
Contributor

What this PR does:

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

knylander-grafana and others added 24 commits September 26, 2024 10:07
Co-authored-by: Jennifer Villa <[email protected]>
Co-authored-by: Joe Elliott <[email protected]>
* Add deprecation for metrics summary

* Update docs/sources/tempo/api_docs/metrics-summary.md

* Update docs/sources/tempo/api_docs/metrics-summary.md

* Apply suggestions from code review

Co-authored-by: Jennifer Villa <[email protected]>

---------

Co-authored-by: Jennifer Villa <[email protected]>
* add definition of maintainer emeritus and place annanay and daniel in this category

Signed-off-by: Joe Elliott <[email protected]>

* codeowners

Signed-off-by: Joe Elliott <[email protected]>

---------

Signed-off-by: Joe Elliott <[email protected]>
…4210)

* feat: Use resource attribute too, rename property from span_multiplier_key to multiplier_key

* chore: update CHANGELOG.md

* fix: apply review

* fix: minor
Signed-off-by: Joe Elliott <[email protected]>
It fixes an issue with adaptive metrics: when a series is new the init sample ends in the same aggregation bucket as the span one, and the later one is downsampled. The fix consists of backdating the initial one to the end of the previous minute
* First working draft of cost attribution usage tracker

* Add missing tracker name label, more efficient batch proportioning, cleanup

* Reduce series hashing

* Fix user-configurable overrides tests for new json element

* lint

* Add per-tenant override for max cardinality

* lint, review feedback

* Default to not enabled, cleanup test config

* Explicitly check for usage_metrics handler

* review feedback

* Update tracker to support many-to-one mapping with relabel

* lint

* New behavior for missing and overflow

* Fix issue where subsequent spans would incorrectly reuse the series of the previous span if they were missing values

* Revert maps back to slices now that we can depend on a dimension always having a value

* Please ignore benchmark profiles

* Tweak config to have specific cost attribution tracker section. Update manifest and config index

* lint

* changelog

* Update api docs for new endpoint

* Review feedback

* review feedback

* Swap loop order for a tad more performance
fix: skip exemplars for instant queries (#4204)

Currently, we are computing the exemplars for query_range and  instant queries. Since exemplars are not supported for instant queries (same as Prometheus) we can save that computing time.

It also fixes a bug where the exemplars query param was not being honored

Co-authored-by: Martin Disibio <[email protected]>
* todos

* more todos and print inuse stats

* Benchmark report heapinuse, ensure cleanup between benchmarks

* Improve memory usage by changing histograms to precompute all labels for all sub-series instead of during each collection

* changelog
…un-times (#3852)

* chore: compatibility with AWS lambda new runtime

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Mario <[email protected]>

* doc: remove additional newline

---------

Co-authored-by: Mario <[email protected]>
* Upgrade OTel to reduce allocs

Signed-off-by: Joe Elliott <[email protected]>

* changelog + vendor

Signed-off-by: Joe Elliott <[email protected]>

* update mod

Signed-off-by: Joe Elliott <[email protected]>

* actually catch the desired change

Signed-off-by: Joe Elliott <[email protected]>

---------

Signed-off-by: Joe Elliott <[email protected]>
Bumps [anchore/sbom-action](https://github.com/anchore/sbom-action) from 0.17.4 to 0.17.5.
- [Release notes](https://github.com/anchore/sbom-action/releases)
- [Changelog](https://github.com/anchore/sbom-action/blob/main/RELEASE.md)
- [Commits](anchore/sbom-action@v0.17.4...v0.17.5)

---
updated-dependencies:
- dependency-name: anchore/sbom-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/add-to-project](https://github.com/actions/add-to-project) from 1.0.0 to 1.0.2.
- [Release notes](https://github.com/actions/add-to-project/releases)
- [Commits](actions/add-to-project@v1.0.0...v1.0.2)

---
updated-dependencies:
- dependency-name: actions/add-to-project
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@knylander-grafana
Copy link
Contributor Author

Messed up test.

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.

7 participants