Skip to content

Commit

Permalink
API for exposing query analysis (#3276)
Browse files Browse the repository at this point in the history
* Add API signature

* Implement AnalyzeQuery (wip)

* Implement AnalyzeQuery, part 2 (wip)

* Fix bug

* Fix bug

* Reorder query scopes

* Add queries series stat

* Add queried series stat (take 2)

* Improve naming, query matchers handling

* Simplify querier logic, add deduplication flag

* Add basic validation

* Add basic validation (fix compile error)

* Only return query scopes that are in use

* Revert "Only return query scopes that are in use"

This reverts commit 649c2cce206e95d01ab0a80b6f15450d96b38d97.

* Improve readability, add unit tests, fix broken unit tests

* Remove unused fields in proto spec

* Revert undesired change

* Improve method name

* Revert undesired change

* Improve naming consistency

* Remove unused function

* Speed up reference-help target

* Add per-tenant overrides for query analysis

* Change default config value for query_analysis_series_enabled

* Update reference help

* Improve naming consistency, fix flaky test

* Revert change to ebpf/testdata
  • Loading branch information
aleks-p authored May 6, 2024
1 parent 5aed1e5 commit f4f2c43
Show file tree
Hide file tree
Showing 39 changed files with 4,070 additions and 463 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ clean: ## Delete intermediate build artifacts

.PHONY: reference-help
reference-help: ## Generates the reference help documentation.
reference-help: build
reference-help: go/bin
@(./pyroscope -h || true) > cmd/pyroscope/help.txt.tmpl
@(./pyroscope -help-all || true) > cmd/pyroscope/help-all.txt.tmpl

Expand Down
539 changes: 395 additions & 144 deletions api/gen/proto/go/ingester/v1/ingester.pb.go

Large diffs are not rendered by default.

Loading

0 comments on commit f4f2c43

Please sign in to comment.