Add ShardedPostings() support to TSDB#10421
Merged
bwplotka merged 6 commits intoprometheus:mainfrom Jan 29, 2024
Merged
Conversation
Member
|
(Visiting this during Prometheus bug scrub meeting) As I mentioned in #10420 I would be supportive to merge this if you rebase @pracucci . The code is tiny, only 2 methods, one to block, one to index reader. We could also explain in comment that it's used by e.g. Mimir at this point. We could work towards using it in Thanos or other systems and iterate together. There is one question around some edge case, but we could explain in comment and explictly not support for now #10420 (comment) |
Member
|
Discussed in the bug-scrub meeting; the feeling was we can merge this if it's just adding one or two methods to the interface. |
Contributor
Author
Yes. I will do it, thanks. |
pracucci
force-pushed
the
tsdb-query-sharding-reference-implementation
branch
2 times, most recently
from
2041b26 to
229de71
Compare
pracucci
changed the title
Contributor
Author
bwplotka
approved these changes
Jan 23, 2024
Member
There was a problem hiding this comment.
Thanks! It has more changes than I initially saw. I don't mind extra methods, but it's worth mentioning SelectHints gets extra optional fields (thus some complexity for users who do not use sharding). This is fine for me, but mentioning to be explicit.
The memSeries field worries me bit more, do we really need that extra field there? 🤔 I guess no.
Maybe prombench would tell if that's impacting anything?
Otherwise LGTM
| meta *metadata.Metadata | ||
|
|
||
| // Series labels hash to use for sharding purposes. | ||
| shardHash uint64 |
Member
There was a problem hiding this comment.
Is this field absolutely needed? It adds some overhead even for non sharded calls, wonder if we can stop that impact 🤔
This low level structure is used a lot
Contributor
Author
There was a problem hiding this comment.
Keeping the hash here significantly improve performance. It's a must for Mimir. In Mimir we haven't measured a significant extra cost to keep it, and I don't know if it really makes sense to upstream this feature without it (cause we'll have to change the implementation again in mimir-prometheus to keep the current behaviour).
Member
There was a problem hiding this comment.
We could use a build tag to have it in Mimir and omit it in Prometheus.
(e.g. shardHash is a field in a struct which is declared in a new file, and that struct is brought here as an anonymous member).
Contributor
Author
There was a problem hiding this comment.
@bboreham How would that work to populate the shardHash field? For example, right now it's passed to newMemSeries() by Head.getOrCreateWithID(). Two different newMemSeries() implementations but same signature?
Member
There was a problem hiding this comment.
Tag is too much maybe, but separate struct memSeriesShardable would work. I am fine to merge for now in this form and improve later on if we see regression 👍🏽
Contributor
Author
There was a problem hiding this comment.
separate struct memSeriesShardable would work
I would like to better understand your idea. If we have a separate struct memSeriesShardable how does it reduces the memory footprint unless we use build tags with two different versions?
I am fine to merge for now in this form
There are other comments from Bryan I would like to address first.
improve later on if we see regression
What if we run the prombench on this PR?
Member
There was a problem hiding this comment.
Two different newMemSeries() implementations but same signature?
Yes, one that doesn't compute the hash. Two implementations for most of the functions added in this PR.
Contributor
Author
There was a problem hiding this comment.
The prombench you've run shows no difference in memory, but an increased CPU usage. Given the build tag option here would have been useful to not impact the memory (at the cost of extra complexity), but memory looks not impacted anyway, WDYT about just skipping computing the shard adding a EnableSharding config option to TSDB?
My proposal is in this comment: c38d4e4
Member
There was a problem hiding this comment.
More detail on the point I made earlier: Go allocates heap objects into a size class, e.g. it will allocate 224 or 240 or 256 bytes for this struct. If it currently needs 248 and you add 8 more then no difference is observed, however it is now twice as hard to drop it to the 240 class.
I'm ok leaving that to a future PR, but build tags favouring different consumers is something I would like to investigate.
bboreham
approved these changes
Jan 23, 2024
Member
bboreham
left a comment
bboreham
left a comment
There was a problem hiding this comment.
I'm broadly fine with this, but have made some notes that came to mind as I read through it.
tsdb/head.go
Outdated
| func (h *Head) getOrCreateWithID(id chunks.HeadSeriesRef, hash uint64, lset labels.Labels) (*memSeries, bool, error) { | ||
| s, created, err := h.series.getOrSet(hash, lset, func() *memSeries { | ||
| return newMemSeries(lset, id, h.opts.IsolationDisabled) | ||
| return newMemSeries(lset, id, labels.StableHash(lset), h.opts.IsolationDisabled) |
Member
There was a problem hiding this comment.
Some extra cost here to do a second hash, but only once for each series created.
We could adjust StableHash to take the hash we already have here and return that if it is known to be the correct value.
(Which it will be for labels-as-slice and the upcoming implementation in #12304)
Contributor
Author
There was a problem hiding this comment.
WDYT about making it conditional, adding a EnableSharding config option to TSDB, like proposed in c38d4e4 ?
storage/interface.go
Outdated
| By bool // Indicate whether it is without or by. | ||
| Range int64 // Range vector selector range in milliseconds. | ||
|
|
||
| ShardIndex uint64 // Current shard index (starts from 0 and up to ShardCount-1). |
Member
There was a problem hiding this comment.
Be nice to link to a higher-level description of the sharding concept.
E.g. the comment on IndexReader interface, or somewhere in the docs.
Contributor
Author
There was a problem hiding this comment.
Rather than linking, I've tried to better explain it directly here. Would this new comment work?
e37b7f9
Member
|
/prombench main |
Member
|
Prombench metrics show more CPU usage and no difference on memory. 
|
Contributor
Author
|
/prombench cancel |
Contributor
|
@pracucci is not a org member nor a collaborator and cannot execute benchmarks. |
Member
|
/prombench restart main |
Member
|
/prombench cancel |
Contributor
|
Benchmark cancel is in progress. |
Member
|
Not sure that "restart" did what we wanted, i.e. rebuild from latest source. |
pracucci
force-pushed
the
tsdb-query-sharding-reference-implementation
branch
from
e37b7f9 to
7c4437f
Compare
Member
|
Changed my mind: |
Member
|
/prombench main |
Member
|
/prombench cancel |
Contributor
|
Benchmark cancel is in progress. |
Member
|
Stats look just the same as main: |
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
pracucci
force-pushed
the
tsdb-query-sharding-reference-implementation
branch
from
7c4437f to
cbd2eea
Compare
Member
|
Benchmarks looks good, let's make Prometheus code more useful for others 👍🏽 |
yeya24
reviewed
Feb 1, 2024
| } | ||
|
|
||
| // Check if the series belong to the shard. | ||
| if s.shardHash%shardCount != shardIndex { |
Contributor
There was a problem hiding this comment.
There are different ways to shard TSDB and shard by hash is only one way.
Are we open to make this logic a configurable function?
Contributor
Author
There was a problem hiding this comment.
Yes, definitely. We should benchmark the overhead of calling a function here inside the loop.
cstyan
pushed a commit
that referenced
this pull request
Feb 2, 2024
* consoles: exclude iowait and steal from CPU Utilisation 'iowait' and 'steal' indicate specific idle/wait states, which shouldn't be counted into CPU Utilisation. Also see prometheus-operator/kube-prometheus#796 and kubernetes-monitoring/kubernetes-mixin#667. Per the iostat man page: %idle Show the percentage of time that the CPU or CPUs were idle and the system did not have an outstanding disk I/O request. %iowait Show the percentage of time that the CPU or CPUs were idle during which the system had an outstanding disk I/O request. %steal Show the percentage of time spent in involuntary wait by the virtual CPU or CPUs while the hypervisor was servicing another virtual processor. Signed-off-by: Julian Wiedmann <[email protected]> * tsdb: shrink txRing with smaller integers 4 billion active transactions ought to be enough for anyone. Signed-off-by: Bryan Boreham <[email protected]> * tsdb: create isolation transaction slice on demand When Prometheus restarts it creates every series read in from the WAL, but many of those series will be finished, and never receive any more samples. By defering allocation of the txRing slice to when it is first needed, we save 32 bytes per stale series. Signed-off-by: Bryan Boreham <[email protected]> * add cluster variable to Overview dashboard Signed-off-by: Erik Sommer <[email protected]> * promql: simplify Native Histogram arithmetics Signed-off-by: Linas Medziunas <[email protected]> * Cut 2.49.0-rc.0 (#13270) * Cut 2.49.0-rc.0 Signed-off-by: bwplotka <[email protected]> * Removed the duplicate. Signed-off-by: bwplotka <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> * Add unit protobuf parser Signed-off-by: Arianna Vespri <[email protected]> * Go on adding protobuf parsing for unit Signed-off-by: Arianna Vespri <[email protected]> * ui: create a reproduction for #13292 Signed-off-by: machine424 <[email protected]> * Get conditional right Signed-off-by: Arianna Vespri <[email protected]> * Get VM Scale Set NIC (#13283) Calling `*armnetwork.InterfacesClient.Get()` doesn't work for Scale Set VM NIC, because these use a different Resource ID format. Use `*armnetwork.InterfacesClient.GetVirtualMachineScaleSetNetworkInterface()` instead. This needs both the scale set name and the instance ID, so add an `InstanceID` field to the `virtualMachine` struct. `InstanceID` is empty for a VM that isn't a ScaleSetVM. Signed-off-by: Daniel Nicholls <[email protected]> * Cut v2.49.0-rc.1 Signed-off-by: bwplotka <[email protected]> * Delete debugging lines, amend error message for unit Signed-off-by: Arianna Vespri <[email protected]> * Correct order in error message Signed-off-by: Arianna Vespri <[email protected]> * Consider storage.ErrTooOldSample as non-retryable Signed-off-by: Daniel Kerbel <[email protected]> * scrape_test.go: Increase scrape interval in TestScrapeLoopCache to reduce potential flakiness Signed-off-by: machine424 <[email protected]> * Avoid creating string for suffix, consider counters without _total suffix Signed-off-by: Arianna Vespri <[email protected]> * build(deps): bump github.com/prometheus/client_golang Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.17.0 to 1.18.0. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.17.0...v1.18.0) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * build(deps): bump actions/setup-node from 3.8.1 to 4.0.1 Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3.8.1 to 4.0.1. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@5e21ff4...b39b52d) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * scripts: sort file list in embed directive Otherwise the resulting string depends on find, which afaict depends on the underlying filesystem. A stable file list make it easier to detect UI changes in downstreams that need to track UI assets. Signed-off-by: Jan Fajerski <[email protected]> * Fix DataTableProps['data'] for resultType string Signed-off-by: Kevin Mingtarja <[email protected]> * Fix handling of scalar and string in isHeatmapData Signed-off-by: Kevin Mingtarja <[email protected]> * build(deps): bump github.com/influxdata/influxdb Bumps [github.com/influxdata/influxdb](https://github.com/influxdata/influxdb) from 1.11.2 to 1.11.4. - [Release notes](https://github.com/influxdata/influxdb/releases) - [Commits](influxdata/influxdb@v1.11.2...v1.11.4) --- updated-dependencies: - dependency-name: github.com/influxdata/influxdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * build(deps): bump github.com/prometheus/prometheus Bumps [github.com/prometheus/prometheus](https://github.com/prometheus/prometheus) from 0.48.0 to 0.48.1. - [Release notes](https://github.com/prometheus/prometheus/releases) - [Changelog](https://github.com/prometheus/prometheus/blob/main/CHANGELOG.md) - [Commits](v0.48.0...v0.48.1) --- updated-dependencies: - dependency-name: github.com/prometheus/prometheus dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Bump client_golang to v1.18.0 (#13373) Signed-off-by: Paschalis Tsilias <[email protected]> * Drop old inmemory samples (#13002) * Drop old inmemory samples Co-authored-by: Paschalis Tsilias <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Avoid copying timeseries when the feature is disabled Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Run gofmt Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Clarify docs Signed-off-by: Marc Tuduri <[email protected]> * Add more logging info Signed-off-by: Marc Tuduri <[email protected]> * Remove loggers Signed-off-by: Marc Tuduri <[email protected]> * optimize function and add tests Signed-off-by: Marc Tuduri <[email protected]> * Simplify filter Signed-off-by: Marc Tuduri <[email protected]> * rename var Signed-off-by: Marc Tuduri <[email protected]> * Update help info from metrics Signed-off-by: Marc Tuduri <[email protected]> * use metrics to keep track of drop elements during buildWriteRequest Signed-off-by: Marc Tuduri <[email protected]> * rename var in tests Signed-off-by: Marc Tuduri <[email protected]> * pass time.Now as parameter Signed-off-by: Marc Tuduri <[email protected]> * Change buildwriterequest during retries Signed-off-by: Marc Tuduri <[email protected]> * Revert "Remove loggers" This reverts commit 54f91df. Signed-off-by: Marc Tuduri <[email protected]> * use log level debug for loggers Signed-off-by: Marc Tuduri <[email protected]> * Fix linter Signed-off-by: Paschalis Tsilias <[email protected]> * Remove noisy debug-level logs; add 'reason' label to drop metrics Signed-off-by: Paschalis Tsilias <[email protected]> * Remove accidentally committed files Signed-off-by: Paschalis Tsilias <[email protected]> * Propagate logger to buildWriteRequest to log dropped data Signed-off-by: Paschalis Tsilias <[email protected]> * Fix docs comment Signed-off-by: Paschalis Tsilias <[email protected]> * Make drop reason more specific Signed-off-by: Paschalis Tsilias <[email protected]> * Remove unnecessary pass of logger Signed-off-by: Paschalis Tsilias <[email protected]> * Use snake_case for reason label Signed-off-by: Paschalis Tsilias <[email protected]> * Fix dropped samples metric Signed-off-by: Paschalis Tsilias <[email protected]> --------- Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> * fix(discovery): allow requireUpdate util to timeout in discovery/file/file_test.go. The loop ran indefinitely if the condition isn't met. Before, each iteration created a new timer channel which was always outpaced by the other timer channel with smaller duration. minor detail: There was a memory leak: resources of the ~10 previous timers were constantly kept. With the fix, we may keep the resources of one timer around for defaultWait but this isn't worth the changes to make it right. Signed-off-by: machine424 <[email protected]> * Merge pull request #13371 from kevinmingtarja/fix-isHeatmapData ui: fix handling of scalar and string in isHeatmapData * tsdb/{index,compact}: allow using custom postings encoding format (#13242) * tsdb/{index,compact}: allow using custom postings encoding format We would like to experiment with a different postings encoding format in Thanos so in this change I am proposing adding another argument to `NewWriter` which would allow users to change the format if needed. Also, wire the leveled compactor so that it would be possible to change the format there too. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb/compact: use a struct for leveled compactor options As discussed on Slack, let's use a struct for the options in leveled compactor. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb: make changes after Bryan's review - Make changes less intrusive - Turn the postings encoder type into a function - Add NewWriterWithEncoder() Signed-off-by: Giedrius Statkevičius <[email protected]> --------- Signed-off-by: Giedrius Statkevičius <[email protected]> * Cut 2.49.0-rc.2 Signed-off-by: bwplotka <[email protected]> * build(deps): bump actions/setup-go from 3.5.0 to 5.0.0 in /scripts (#13362) Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3.5.0 to 5.0.0. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@6edd440...0c52d54) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github/codeql-action from 2.22.8 to 3.22.12 (#13358) Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.8 to 3.22.12. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@407ffaf...012739e) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * put @Nexucis has a release shepherd (#13383) Signed-off-by: Augustin Husson <[email protected]> * Add analyze histograms command to promtool (#12331) Add `query analyze` command to promtool This command analyzes the buckets of classic and native histograms, based on data queried from the Prometheus query API, i.e. it doesn't require direct access to the TSDB files. Signed-off-by: Jeanette Tan <[email protected]> --------- Signed-off-by: Jeanette Tan <[email protected]> * included instance in all necessary descriptions Signed-off-by: Erik Sommer <[email protected]> * tsdb/compact: fix passing merge func Fixing a very small logical problem I've introduced :(. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb: add enable overlapping compaction This functionality is needed in downstream projects because they have a separate component that does compaction. Upstreaming https://github.com/grafana/mimir-prometheus/blob/7c8e9a2a76fc729e9078889782928b2fdfe240e9/tsdb/compact.go#L323-L325. Signed-off-by: Giedrius Statkevičius <[email protected]> * Cut 2.49.0 Signed-off-by: bwplotka <[email protected]> * promtool: allow setting multiple matchers to "promtool tsdb dump" command. (#13296) Conditions are ANDed inside the same matcher but matchers are ORed Including unit tests for "promtool tsdb dump". Refactor some matchers scraping utils. Signed-off-by: machine424 <[email protected]> * Fixed changelog Signed-off-by: bwplotka <[email protected]> * tsdb/main: wire "EnableOverlappingCompaction" to tsdb.Options (#13398) This added the #13393 "EnableOverlappingCompaction" parameter to the compactor code but not to the tsdb.Options. I forgot about that. Add it to `tsdb.Options` too and set it to `true` in Prometheus. Copy/paste the description from #13393 (comment) Signed-off-by: Giedrius Statkevičius <[email protected]> * Issue #13268: fix quality value in accept header Signed-off-by: Kumar Kalpadiptya Roy <[email protected]> * Cut 2.49.1 with scrape q= bugfix. Signed-off-by: bwplotka <[email protected]> * Cut 2.49.1 web package. Signed-off-by: bwplotka <[email protected]> * Restore more efficient version of NewPossibleNonCounterInfo annotation (#13022) Restore more efficient version of NewPossibleNonCounterInfo annotation Signed-off-by: Jeanette Tan <[email protected]> --------- Signed-off-by: Jeanette Tan <[email protected]> * Fix regressions introduced by #13242 Signed-off-by: Marco Pracucci <[email protected]> * fix slice copy in 1.20 (#13389) The slices package is added to the standard library in Go 1.21; we need to import from the exp area to maintain compatibility with Go 1.20. Signed-off-by: tyltr <[email protected]> * Docs: Query Basics: link to rate (#10538) Co-authored-by: Julien Pivotto <[email protected]> * chore(kubernetes): check preconditions earlier and avoid unnecessary checks or iterations Signed-off-by: machine424 <[email protected]> * Examples: link to `rate` for new users (#10535) * Examples: link to `rate` for new users Signed-off-by: Ted Robertson [email protected] Co-authored-by: Bryan Boreham <[email protected]> * promql: use natural sort in sort_by_label and sort_by_label_desc (#13411) These functions are intended for humans, as robots can already sort the results however they please. Humans like things sorted "naturally": * https://blog.codinghorror.com/sorting-for-humans-natural-sort-order/ A similar thing has been done to Grafana, which is also used by humans: * grafana/grafana#78024 * grafana/grafana#78494 Signed-off-by: Ivan Babrou <[email protected]> * TestLabelValuesWithMatchers: Add test case Signed-off-by: Arve Knudsen <[email protected]> * remove obsolete build tag Signed-off-by: tyltr <[email protected]> * Upgrade some golang dependencies for resty 2.11 Signed-off-by: Israel Blancas <[email protected]> * Native Histograms: support `native_histogram_min_bucket_factor` in scrape_config (#13222) Native Histograms: support native_histogram_min_bucket_factor in scrape_config --------- Signed-off-by: Ziqi Zhao <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> * Add warnings for histogramRate applied with isCounter not matching counter/gauge histogram (#13392) Add warnings for histogramRate applied with isCounter not matching counter/gauge histogram --------- Signed-off-by: Jeanette Tan <[email protected]> * Minor fixes to otlp vendor update script Signed-off-by: Goutham <[email protected]> * build(deps): bump github.com/hetznercloud/hcloud-go/v2 Bumps [github.com/hetznercloud/hcloud-go/v2](https://github.com/hetznercloud/hcloud-go) from 2.4.0 to 2.6.0. - [Release notes](https://github.com/hetznercloud/hcloud-go/releases) - [Changelog](https://github.com/hetznercloud/hcloud-go/blob/main/CHANGELOG.md) - [Commits](hetznercloud/hcloud-go@v2.4.0...v2.6.0) --- updated-dependencies: - dependency-name: github.com/hetznercloud/hcloud-go/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Enhanced visibility for `promtool test rules` with JSON colored formatting (#13342) * Added diff flag for unit test to improvise readability & debugging Signed-off-by: Rewanth Tammana <[email protected]> * Removed blank spaces Signed-off-by: Rewanth Tammana <[email protected]> * Fixed linting error Signed-off-by: Rewanth Tammana <[email protected]> * Added cli flags to documentation Signed-off-by: Rewanth Tammana <[email protected]> * Revert unrrelated linting fixes Signed-off-by: Rewanth Tammana <[email protected]> * Fixed review suggestions Signed-off-by: Rewanth Tammana <[email protected]> * Cleanup Signed-off-by: Rewanth Tammana <[email protected]> * Updated flag description Signed-off-by: Rewanth Tammana <[email protected]> * Updated flag description Signed-off-by: Rewanth Tammana <[email protected]> --------- Signed-off-by: Rewanth Tammana <[email protected]> * storage: skip merging when no remote storage configured Prometheus is hard-coded to use a fanout storage between TSDB and a remote storage which by default is empty. This change detects the empty storage and skips merging between result sets, which would make `Select()` sort results. Bottom line: we skip a sort unless there really is some remote storage configured. Signed-off-by: Bryan Boreham <[email protected]> * Remove csmarchbanks from remote write owners (#13432) I have not had the time to keep up with remote write and have no plans to work on it in the near future so I am withdrawing my maintainership of that part of the codebase. I continue to focus on client_python. Signed-off-by: Chris Marchbanks <[email protected]> * add more context cancellation check at evaluation time Signed-off-by: Ben Ye <[email protected]> * Optimize label values with matchers by taking shortcuts (#13426) Don't calculate postings beforehand: we may not need them. If all matchers are for the requested label, we can just filter its values. Also, if there are no values at all, no need to run any kind of logic. Also add more labelValuesWithMatchers benchmarks Signed-off-by: Oleg Zaytsev <[email protected]> * Add automatic memory limit handling Enable automatic detection of memory limits and configure GOMEMLIMIT to match. * Also includes a flag to allow controlling the reserved ratio. Signed-off-by: SuperQ <[email protected]> * Update OSSF badge link (#13433) Provide a more user friendly interface Signed-off-by: Matthieu MOREL <[email protected]> * SD Managers taking over responsibility for registration of debug metrics (#13375) SD Managers take over responsibility for SD metrics registration --------- Signed-off-by: Paulin Todev <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> * Optimize histogram iterators (#13340) Optimize histogram iterators Histogram iterators allocate new objects in the AtHistogram and AtFloatHistogram methods, which makes calculating rates over long ranges expensive. In #13215 we allowed an existing object to be reused when converting an integer histogram to a float histogram. This commit follows the same idea and allows injecting an existing object in the AtHistogram and AtFloatHistogram methods. When the injected value is nil, iterators allocate new histograms, otherwise they populate and return the injected object. The commit also adds a CopyTo method to Histogram and FloatHistogram which is used in the BufferedIterator to overwrite items in the ring instead of making new copies. Note that a specialized HPoint pool is needed for all of this to work (`matrixSelectorHPool`). --------- Signed-off-by: Filip Petkovski <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> * doc: Mark `mad_over_time` as experimental (#13440) We forgot to do that in #13059 Signed-off-by: beorn7 <[email protected]> * Change metric label for Puppetdb from 'http' to 'puppetdb' Signed-off-by: Paulin Todev <[email protected]> * mirror metrics.proto change & generate code Signed-off-by: Ziqi Zhao <[email protected]> * TestHeadLabelValuesWithMatchers: Add test case (#13414) Add test case to TestHeadLabelValuesWithMatchers, while fixing a couple of typos in other test cases. Also enclosing some implicit sub-tests in a `t.Run` call to make them explicitly sub-tests. Signed-off-by: Arve Knudsen <[email protected]> * update all go dependencies (#13438) Signed-off-by: Augustin Husson <[email protected]> * build(deps): bump the k8s-io group with 2 updates (#13454) Bumps the k8s-io group with 2 updates: [k8s.io/api](https://github.com/kubernetes/api) and [k8s.io/client-go](https://github.com/kubernetes/client-go). Updates `k8s.io/api` from 0.28.4 to 0.29.1 - [Commits](kubernetes/api@v0.28.4...v0.29.1) Updates `k8s.io/client-go` from 0.28.4 to 0.29.1 - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](kubernetes/client-go@v0.28.4...v0.29.1) --- updated-dependencies: - dependency-name: k8s.io/api dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-io - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-io ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump the go-opentelemetry-io group with 1 update (#13453) Bumps the go-opentelemetry-io group with 1 update: [go.opentelemetry.io/collector/semconv](https://github.com/open-telemetry/opentelemetry-collector). Updates `go.opentelemetry.io/collector/semconv` from 0.92.0 to 0.93.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-collector/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-collector/blob/main/CHANGELOG-API.md) - [Commits](open-telemetry/opentelemetry-collector@v0.92.0...v0.93.0) --- updated-dependencies: - dependency-name: go.opentelemetry.io/collector/semconv dependency-type: direct:production update-type: version-update:semver-minor dependency-group: go-opentelemetry-io ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (#13355) Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.3 to 4.0.0. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@a8a3f3a...c7d193f) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump bufbuild/buf-push-action (#13357) Bumps [bufbuild/buf-push-action](https://github.com/bufbuild/buf-push-action) from 342fc4cdcf29115a01cf12a2c6dd6aac68dc51e1 to a654ff18effe4641ebea4a4ce242c49800728459. - [Release notes](https://github.com/bufbuild/buf-push-action/releases) - [Commits](bufbuild/buf-push-action@342fc4c...a654ff1) --- updated-dependencies: - dependency-name: bufbuild/buf-push-action dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Labels: Add DropMetricName function, used in PromQL (#13446) This function is called very frequently when executing PromQL functions, and we can do it much more efficiently inside Labels. In the common case that `__name__` comes first in the labels, we simply re-point to start at the next label, which is nearly free. `DropMetricName` is now so cheap I removed the cache - benchmarks show everything still goes faster. Signed-off-by: Bryan Boreham <[email protected]> * tsdb: simplify internal series delete function (#13261) Lifting an optimisation from Agent code, `seriesHashmap.del` can use the unique series reference, doesn't need to check Labels. Also streamline the logic for deleting from `unique` and `conflicts` maps, and add some comments to help the next person. Signed-off-by: Bryan Boreham <[email protected]> * otlptranslator/update-copy.sh: Fix sed command lines Signed-off-by: Arve Knudsen <[email protected]> * Rollback k8s.io requirements (#13462) Rollback k8s.io Go modules to v0.28.6 to avoid forcing upgrade of Go to 1.21. This allows us to keep compatibility with the currently supported upstream Go releases. Signed-off-by: SuperQ <[email protected]> * Make update-copy.sh work for both OSX and GNU sed Signed-off-by: Arve Knudsen <[email protected]> * Name @beorn7 and @krajorama as maintainers for native histograms I have been the de-facto maintainer for native histograms from the beginning. So let's put this into MAINTAINERS.md. In addition, I hereby proposose George Krajcsovits AKA Krajo as a co-maintainer. He has contributed a lot of native histogram code, but more importantly, he has contributed substantially to reviewing other contributors' native histogram code, up to a point where I was merely rubberstamping the PRs he had already reviewed. I'm confident that he is ready to to be granted commit rights as outlined in the "Maintainers" section of the governance: https://prometheus.io/governance/#maintainers According to the same section of the governance, I will announce the proposed change on the developers mailing list and will give some time for lazy consensus before merging this PR. Signed-off-by: beorn7 <[email protected]> * ui/fix: correct url handling for stacked graphs (#13460) Signed-off-by: Yury Moladau <[email protected]> * tsdb: use cheaper Mutex on series Mutex is 8 bytes; RWMutex is 24 bytes and much more complicated. Since `RLock` is only used in two places, `UpdateMetadata` and `Delete`, neither of which are hotspots, we should use the cheaper one. Signed-off-by: Bryan Boreham <[email protected]> * Fix last_over_time for native histograms The last_over_time retains a histogram sample without making a copy. This sample is now coming from the buffered iterator used for windowing functions, and can be reused for reading subsequent samples as the iterator progresses. I would propose copying the sample in the last_over_time function, similar to how it is done for rate, sum_over_time and others. Signed-off-by: Filip Petkovski <[email protected]> * Implementation NOTE: Rebased from main after refactor in #13014 Signed-off-by: Danny Kopping <[email protected]> * Add feature flag Signed-off-by: Danny Kopping <[email protected]> * Refactor concurrency control Signed-off-by: Danny Kopping <[email protected]> * Optimising dependencies/dependents funcs to not produce new slices each request Signed-off-by: Danny Kopping <[email protected]> * Refactoring Signed-off-by: Danny Kopping <[email protected]> * Rename flag Signed-off-by: Danny Kopping <[email protected]> * Refactoring for performance, and to allow controller to be overridden Signed-off-by: Danny Kopping <[email protected]> * Block until all rules, both sync & async, have completed evaluating Updated & added tests Review feedback nits Return empty map if not indeterminate Use highWatermark to track inflight requests counter Appease the linter Clarify feature flag Signed-off-by: Danny Kopping <[email protected]> * Fix typo in CLI flag description Signed-off-by: Marco Pracucci <[email protected]> * Fixed auto-generated doc Signed-off-by: Marco Pracucci <[email protected]> * Improve doc Signed-off-by: Marco Pracucci <[email protected]> * Simplify the design to update concurrency controller once the rule evaluation has done Signed-off-by: Marco Pracucci <[email protected]> * Add more test cases to TestDependenciesEdgeCases Signed-off-by: Marco Pracucci <[email protected]> * Added more test cases to TestDependenciesEdgeCases Signed-off-by: Marco Pracucci <[email protected]> * Improved RuleConcurrencyController interface doc Signed-off-by: Marco Pracucci <[email protected]> * Introduced sequentialRuleEvalController Signed-off-by: Marco Pracucci <[email protected]> * Remove superfluous nil check in Group.metrics Signed-off-by: Marco Pracucci <[email protected]> * api: Serialize discovered and target labels into JSON directly (#13469) Converted maps into labels.Labels to avoid a lot of copying of data which leads to very high memory consumption while opening the /service-discovery endpoint in the Prometheus UI Signed-off-by: Leegin <[email protected]> * api: Serialize discovered labels into JSON directly in dropped targets (#13484) Converted maps into labels.Labels to avoid a lot of copying of data which leads to very high memory consumption while opening the /service-discovery endpoint in the Prometheus UI Signed-off-by: Leegin <[email protected]> * Add ShardedPostings() support to TSDB (#10421) This PR is a reference implementation of the proposal described in #10420. In addition to what described in #10420, in this PR I've introduced labels.StableHash(). The idea is to offer an hashing function which doesn't change over time, and that's used by query sharding in order to get a stable behaviour over time. The implementation of labels.StableHash() is the hashing function used by Prometheus before stringlabels, and what's used by Grafana Mimir for query sharding (because built before stringlabels was a thing). Follow up work As mentioned in #10420, if this PR is accepted I'm also open to upload another foundamental piece used by Grafana Mimir query sharding to accelerate the query execution: an optional, configurable and fast in-memory cache for the series hashes. Signed-off-by: Marco Pracucci <[email protected]> * storage/remote: document why two benchmarks are skipped One was silently doing nothing; one was doing something but the work didn't go up linearly with iteration count. Signed-off-by: Bryan Boreham <[email protected]> * Pod status changes not discovered by Kube Endpoints SD (#13337) * fix(discovery/kubernetes/endpoints): react to changes on Pods because some modifications can occur on them without triggering an update on the related Endpoints (The Pod phase changing from Pending to Running e.g.). --------- Signed-off-by: machine424 <[email protected]> Co-authored-by: Guillermo Sanchez Gavier <[email protected]> * Small improvements, add const, remove copypasta (#8106) Signed-off-by: Mikhail Fesenko <[email protected]> Signed-off-by: Jesus Vazquez <[email protected]> * Proposal to improve FPointSlice and HPointSlice allocation. (#13448) * Reusing points slice from previous series when the slice is under utilized * Adding comments on the bench test Signed-off-by: Alan Protasio <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * go mod tidy Signed-off-by: Nicolás Pazos <[email protected]> --------- Signed-off-by: Julian Wiedmann <[email protected]> Signed-off-by: Bryan Boreham <[email protected]> Signed-off-by: Erik Sommer <[email protected]> Signed-off-by: Linas Medziunas <[email protected]> Signed-off-by: bwplotka <[email protected]> Signed-off-by: Arianna Vespri <[email protected]> Signed-off-by: machine424 <[email protected]> Signed-off-by: Daniel Nicholls <[email protected]> Signed-off-by: Daniel Kerbel <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Jan Fajerski <[email protected]> Signed-off-by: Kevin Mingtarja <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Giedrius Statkevičius <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Jeanette Tan <[email protected]> Signed-off-by: Bartlomiej Plotka <[email protected]> Signed-off-by: Kumar Kalpadiptya Roy <[email protected]> Signed-off-by: Marco Pracucci <[email protected]> Signed-off-by: tyltr <[email protected]> Signed-off-by: Ted Robertson [email protected] Signed-off-by: Ivan Babrou <[email protected]> Signed-off-by: Arve Knudsen <[email protected]> Signed-off-by: Israel Blancas <[email protected]> Signed-off-by: Ziqi Zhao <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Signed-off-by: Goutham <[email protected]> Signed-off-by: Rewanth Tammana <[email protected]> Signed-off-by: Chris Marchbanks <[email protected]> Signed-off-by: Ben Ye <[email protected]> Signed-off-by: Oleg Zaytsev <[email protected]> Signed-off-by: SuperQ <[email protected]> Signed-off-by: Ben Kochie <[email protected]> Signed-off-by: Matthieu MOREL <[email protected]> Signed-off-by: Paulin Todev <[email protected]> Signed-off-by: Filip Petkovski <[email protected]> Signed-off-by: beorn7 <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Yury Moladau <[email protected]> Signed-off-by: Danny Kopping <[email protected]> Signed-off-by: Leegin <[email protected]> Signed-off-by: Mikhail Fesenko <[email protected]> Signed-off-by: Jesus Vazquez <[email protected]> Signed-off-by: Alan Protasio <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> Co-authored-by: Julian Wiedmann <[email protected]> Co-authored-by: Bryan Boreham <[email protected]> Co-authored-by: Erik Sommer <[email protected]> Co-authored-by: Linas Medziunas <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]> Co-authored-by: Arianna Vespri <[email protected]> Co-authored-by: machine424 <[email protected]> Co-authored-by: daniel-resdiary <[email protected]> Co-authored-by: Daniel Kerbel <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jan Fajerski <[email protected]> Co-authored-by: Kevin Mingtarja <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Marc Tudurí <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Giedrius Statkevičius <[email protected]> Co-authored-by: Augustin Husson <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> Co-authored-by: zenador <[email protected]> Co-authored-by: gotjosh <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Kumar Kalpadiptya Roy <[email protected]> Co-authored-by: Marco Pracucci <[email protected]> Co-authored-by: tyltr <[email protected]> Co-authored-by: Ted Robertson <[email protected]> Co-authored-by: Julien Pivotto <[email protected]> Co-authored-by: Matthias Loibl <[email protected]> Co-authored-by: Ivan Babrou <[email protected]> Co-authored-by: Arve Knudsen <[email protected]> Co-authored-by: Israel Blancas <[email protected]> Co-authored-by: Ziqi Zhao <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> Co-authored-by: Goutham <[email protected]> Co-authored-by: Rewanth Tammana <[email protected]> Co-authored-by: Chris Marchbanks <[email protected]> Co-authored-by: Ben Ye <[email protected]> Co-authored-by: Oleg Zaytsev <[email protected]> Co-authored-by: Matthieu MOREL <[email protected]> Co-authored-by: Paulin Todev <[email protected]> Co-authored-by: Filip Petkovski <[email protected]> Co-authored-by: Yury Molodov <[email protected]> Co-authored-by: Danny Kopping <[email protected]> Co-authored-by: Leegin <[email protected]> Co-authored-by: Guillermo Sanchez Gavier <[email protected]> Co-authored-by: Mikhail Fesenko <[email protected]> Co-authored-by: Alan Protasio <[email protected]>
cstyan
added a commit
that referenced
this pull request
Jul 4, 2024
…support + metadata in WAL) (#14395) * Remote Write 1.1: e2e benchmarks (#13102) * Remote Write e2e benchmarks Signed-off-by: Nicolás Pazos <[email protected]> * Prometheus ports automatically assigned Signed-off-by: Nicolás Pazos <[email protected]> * make dashboard editable + more modular to different job label values Signed-off-by: Callum Styan <[email protected]> * Dashboard improvements * memory stats * diffs look at counter increases Signed-off-by: Nicolás Pazos <[email protected]> * run script: absolute path for config templates Signed-off-by: Nicolás Pazos <[email protected]> * grafana dashboard improvements * show actual values of metrics * add memory stats and diff Signed-off-by: Nicolás Pazos <[email protected]> * dashboard changes Signed-off-by: Callum Styan <[email protected]> --------- Signed-off-by: Nicolás Pazos <[email protected]> Signed-off-by: Callum Styan <[email protected]> Co-authored-by: Callum Styan <[email protected]> * replace snappy encoding library Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * add new proto types Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * add decode function for new write request proto Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * add lookup table struct that is used to build the symbol table in new write request format Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * Implement code paths for new proto format Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * update example server to include handler for new format Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * Add new test client Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * tests and new -> original proto mapping util Signed-off-by: Nicolás Pazos <[email protected]> * add new proto support on receiver end Signed-off-by: Nicolás Pazos <[email protected]> * Fix test Signed-off-by: Nicolás Pazos <[email protected]> * no-brainer copypaste but more performance write support Signed-off-by: Nicolás Pazos <[email protected]> * remove some comented code Signed-off-by: Nicolás Pazos <[email protected]> * fix mocks and fixture Signed-off-by: Nicolás Pazos <[email protected]> * add basic reduce remote write handler benchmark Signed-off-by: Nicolás Pazos <[email protected]> * refactor out common code between write methods Signed-off-by: Nicolás Pazos <[email protected]> * fix: queue manager to include float histograms in new requests Signed-off-by: Nicolás Pazos <[email protected]> * add sender-side tests and fix failing ones Signed-off-by: Nicolás Pazos <[email protected]> * refactor queue manager code to remove some duplication Signed-off-by: Nicolás Pazos <[email protected]> * fix build Signed-off-by: Nicolás Pazos <[email protected]> * Improve sender benchmarks and some allocations Signed-off-by: Nicolás Pazos <[email protected]> * Use github.com/golang/snappy Signed-off-by: Nicolás Pazos <[email protected]> * cleanup: remove hardcoded fake url for testing Signed-off-by: Nicolás Pazos <[email protected]> * Add 1.1 version handling code Signed-off-by: Nicolás Pazos <[email protected]> * Remove config, update proto Signed-off-by: Nicolás Pazos <[email protected]> * gofmt Signed-off-by: Nicolás Pazos <[email protected]> * fix NewWriteClient and change new flags wording Signed-off-by: Nicolás Pazos <[email protected]> * fields rewording in handler Signed-off-by: Nicolás Pazos <[email protected]> * remote write handler to checks version header Signed-off-by: Nicolás Pazos <[email protected]> * fix typo in log Signed-off-by: Nicolás Pazos <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * Add minmized remote write proto format Co-authored-by: Marco Pracucci <[email protected]> Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * add functions for translating between new proto formats symbol table and actual prometheus labels Co-authored-by: Marco Pracucci <[email protected]> Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * add functionality for new minimized remote write request format Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * fix minor things Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * Make LabelSymbols a fixed32 Signed-off-by: Nicolás Pazos <[email protected]> * remove unused proto type Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * update tests Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * fix build for stringlabels tag Signed-off-by: Nicolás Pazos <[email protected]> * Use two uint32 to encode (offset,leng) Signed-off-by: Nicolás Pazos <[email protected]> * manually optimize varint marshaling Signed-off-by: Nicolás Pazos <[email protected]> * Use unsafe []byte->string cast to reuse buffer Signed-off-by: Nicolás Pazos <[email protected]> * fix writeRequestMinimizedFixture Signed-off-by: Nicolás Pazos <[email protected]> * remove all code from previous interning approach the 'minimized' version is now the only v1.1 version Signed-off-by: Nicolás Pazos <[email protected]> * minimally-tested exemplar support for rw 1.1 Signed-off-by: Nicolás Pazos <[email protected]> * refactor new version flag to make it easier to pick a specific format instead of having multiple flags, plus add new formats for testing Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * use exp slices for backwards compat. to go 1.20 plus add copyright header to test file Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * fix label ranging Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * Add bytes slice (instead of slice of 32bit vars) format for testing Co-authored-by: Nicolás Pazos <[email protected]> Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * test additional len and lenbytes formats Co-authored-by: Nicolás Pazos <[email protected]> Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * remove mistaken package lock changes Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * remove formats we've decided not to use Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * remove more format types we probably won't use Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * More cleanup Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * use require instead of assert in custom marshal test Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * cleanup; remove some unused functions Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * more cleanup, mostly linting fixes Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * remove package-lock.json change again Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * more cleanup, address review comments Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * fix test panic Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * fix minor lint issue + use labels Range function since it looks like the tests fail to do `range labels.Labels` on CI Signed-off-by: Callum Styan <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * new interning format based on []string indeces Co-authored-by: bwplotka <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * remove all new rw formats but the []string one also adapt tests to the new format Signed-off-by: Nicolás Pazos <[email protected]> * cleanup rwSymbolTable Signed-off-by: Nicolás Pazos <[email protected]> * add some TODOs for later Signed-off-by: Nicolás Pazos <[email protected]> * don't reserve field 3 for new proto and add TODO Signed-off-by: Nicolás Pazos <[email protected]> * fix custom marshaling Signed-off-by: Nicolás Pazos <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * additional merge fixes Signed-off-by: Nicolás Pazos <[email protected]> * lint fixes Signed-off-by: Nicolás Pazos <[email protected]> * fix server example Signed-off-by: Nicolás Pazos <[email protected]> * revert package-lock.json changes Signed-off-by: Nicolás Pazos <[email protected]> * update example prometheus version Signed-off-by: Nicolás Pazos <[email protected]> * define separate proto types for remote write 2.0 Signed-off-by: Nicolás Pazos <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * rename new proto types and move to separate pkg Signed-off-by: Nicolás Pazos <[email protected]> * update prometheus version for example Signed-off-by: Nicolás Pazos <[email protected]> * make proto Signed-off-by: Nicolás Pazos <[email protected]> * make Metadata not nullable Signed-off-by: Nicolás Pazos <[email protected]> * remove old MinSample proto message Signed-off-by: Nicolás Pazos <[email protected]> * change enum names to fit buf build recommend enum naming and lint rules Signed-off-by: Callum Styan <[email protected]> * remote: Added test for classic histogram grouping when sending rw; Fixed queue manager test delay. (#13421) Signed-off-by: bwplotka <[email protected]> * Remote write v2: metadata support in every write request (#13394) * Approach bundling metadata along with samples and exemplars Signed-off-by: Paschalis Tsilias <[email protected]> * Add first test; rebase with main Signed-off-by: Paschalis Tsilias <[email protected]> * Alternative approach: bundle metadata in TimeSeries protobuf Signed-off-by: Paschalis Tsilias <[email protected]> * update go mod to match main branch Signed-off-by: Callum Styan <[email protected]> * fix after rebase Signed-off-by: Callum Styan <[email protected]> * we're not going to modify the 1.X format anymore Signed-off-by: Callum Styan <[email protected]> * Modify AppendMetadata based on the fact that we be putting metadata into timeseries Signed-off-by: Callum Styan <[email protected]> * Rename enums for remote write versions to something that makes more sense + remove the added `sendMetadata` flag. Signed-off-by: Callum Styan <[email protected]> * rename flag that enables writing of metadata records to the WAL Signed-off-by: Callum Styan <[email protected]> * additional clean up Signed-off-by: Callum Styan <[email protected]> * lint Signed-off-by: Callum Styan <[email protected]> * fix usage of require.Len Signed-off-by: Callum Styan <[email protected]> * some clean up from review comments Signed-off-by: Callum Styan <[email protected]> * more review fixes Signed-off-by: Callum Styan <[email protected]> --------- Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Callum Styan <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> * remote write 2.0: sync with `main` branch (#13510) * consoles: exclude iowait and steal from CPU Utilisation 'iowait' and 'steal' indicate specific idle/wait states, which shouldn't be counted into CPU Utilisation. Also see https://github.com/prometheus-operator/kube-prometheus/pull/796 and https://github.com/kubernetes-monitoring/kubernetes-mixin/pull/667. Per the iostat man page: %idle Show the percentage of time that the CPU or CPUs were idle and the system did not have an outstanding disk I/O request. %iowait Show the percentage of time that the CPU or CPUs were idle during which the system had an outstanding disk I/O request. %steal Show the percentage of time spent in involuntary wait by the virtual CPU or CPUs while the hypervisor was servicing another virtual processor. Signed-off-by: Julian Wiedmann <[email protected]> * tsdb: shrink txRing with smaller integers 4 billion active transactions ought to be enough for anyone. Signed-off-by: Bryan Boreham <[email protected]> * tsdb: create isolation transaction slice on demand When Prometheus restarts it creates every series read in from the WAL, but many of those series will be finished, and never receive any more samples. By defering allocation of the txRing slice to when it is first needed, we save 32 bytes per stale series. Signed-off-by: Bryan Boreham <[email protected]> * add cluster variable to Overview dashboard Signed-off-by: Erik Sommer <[email protected]> * promql: simplify Native Histogram arithmetics Signed-off-by: Linas Medziunas <[email protected]> * Cut 2.49.0-rc.0 (#13270) * Cut 2.49.0-rc.0 Signed-off-by: bwplotka <[email protected]> * Removed the duplicate. Signed-off-by: bwplotka <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> * Add unit protobuf parser Signed-off-by: Arianna Vespri <[email protected]> * Go on adding protobuf parsing for unit Signed-off-by: Arianna Vespri <[email protected]> * ui: create a reproduction for https://github.com/prometheus/prometheus/issues/13292 Signed-off-by: machine424 <[email protected]> * Get conditional right Signed-off-by: Arianna Vespri <[email protected]> * Get VM Scale Set NIC (#13283) Calling `*armnetwork.InterfacesClient.Get()` doesn't work for Scale Set VM NIC, because these use a different Resource ID format. Use `*armnetwork.InterfacesClient.GetVirtualMachineScaleSetNetworkInterface()` instead. This needs both the scale set name and the instance ID, so add an `InstanceID` field to the `virtualMachine` struct. `InstanceID` is empty for a VM that isn't a ScaleSetVM. Signed-off-by: Daniel Nicholls <[email protected]> * Cut v2.49.0-rc.1 Signed-off-by: bwplotka <[email protected]> * Delete debugging lines, amend error message for unit Signed-off-by: Arianna Vespri <[email protected]> * Correct order in error message Signed-off-by: Arianna Vespri <[email protected]> * Consider storage.ErrTooOldSample as non-retryable Signed-off-by: Daniel Kerbel <[email protected]> * scrape_test.go: Increase scrape interval in TestScrapeLoopCache to reduce potential flakiness Signed-off-by: machine424 <[email protected]> * Avoid creating string for suffix, consider counters without _total suffix Signed-off-by: Arianna Vespri <[email protected]> * build(deps): bump github.com/prometheus/client_golang Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.17.0 to 1.18.0. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/main/CHANGELOG.md) - [Commits](https://github.com/prometheus/client_golang/compare/v1.17.0...v1.18.0) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * build(deps): bump actions/setup-node from 3.8.1 to 4.0.1 Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3.8.1 to 4.0.1. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](https://github.com/actions/setup-node/compare/5e21ff4d9bc1a8cf6de233a3057d20ec6b3fb69d...b39b52d1213e96004bfcb1c61a8a6fa8ab84f3e8) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * scripts: sort file list in embed directive Otherwise the resulting string depends on find, which afaict depends on the underlying filesystem. A stable file list make it easier to detect UI changes in downstreams that need to track UI assets. Signed-off-by: Jan Fajerski <[email protected]> * Fix DataTableProps['data'] for resultType string Signed-off-by: Kevin Mingtarja <[email protected]> * Fix handling of scalar and string in isHeatmapData Signed-off-by: Kevin Mingtarja <[email protected]> * build(deps): bump github.com/influxdata/influxdb Bumps [github.com/influxdata/influxdb](https://github.com/influxdata/influxdb) from 1.11.2 to 1.11.4. - [Release notes](https://github.com/influxdata/influxdb/releases) - [Commits](https://github.com/influxdata/influxdb/compare/v1.11.2...v1.11.4) --- updated-dependencies: - dependency-name: github.com/influxdata/influxdb dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * build(deps): bump github.com/prometheus/prometheus Bumps [github.com/prometheus/prometheus](https://github.com/prometheus/prometheus) from 0.48.0 to 0.48.1. - [Release notes](https://github.com/prometheus/prometheus/releases) - [Changelog](https://github.com/prometheus/prometheus/blob/main/CHANGELOG.md) - [Commits](https://github.com/prometheus/prometheus/compare/v0.48.0...v0.48.1) --- updated-dependencies: - dependency-name: github.com/prometheus/prometheus dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Bump client_golang to v1.18.0 (#13373) Signed-off-by: Paschalis Tsilias <[email protected]> * Drop old inmemory samples (#13002) * Drop old inmemory samples Co-authored-by: Paschalis Tsilias <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Avoid copying timeseries when the feature is disabled Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Run gofmt Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> * Clarify docs Signed-off-by: Marc Tuduri <[email protected]> * Add more logging info Signed-off-by: Marc Tuduri <[email protected]> * Remove loggers Signed-off-by: Marc Tuduri <[email protected]> * optimize function and add tests Signed-off-by: Marc Tuduri <[email protected]> * Simplify filter Signed-off-by: Marc Tuduri <[email protected]> * rename var Signed-off-by: Marc Tuduri <[email protected]> * Update help info from metrics Signed-off-by: Marc Tuduri <[email protected]> * use metrics to keep track of drop elements during buildWriteRequest Signed-off-by: Marc Tuduri <[email protected]> * rename var in tests Signed-off-by: Marc Tuduri <[email protected]> * pass time.Now as parameter Signed-off-by: Marc Tuduri <[email protected]> * Change buildwriterequest during retries Signed-off-by: Marc Tuduri <[email protected]> * Revert "Remove loggers" This reverts commit 54f91dfcae20488944162335ab4ad8be459df1ab. Signed-off-by: Marc Tuduri <[email protected]> * use log level debug for loggers Signed-off-by: Marc Tuduri <[email protected]> * Fix linter Signed-off-by: Paschalis Tsilias <[email protected]> * Remove noisy debug-level logs; add 'reason' label to drop metrics Signed-off-by: Paschalis Tsilias <[email protected]> * Remove accidentally committed files Signed-off-by: Paschalis Tsilias <[email protected]> * Propagate logger to buildWriteRequest to log dropped data Signed-off-by: Paschalis Tsilias <[email protected]> * Fix docs comment Signed-off-by: Paschalis Tsilias <[email protected]> * Make drop reason more specific Signed-off-by: Paschalis Tsilias <[email protected]> * Remove unnecessary pass of logger Signed-off-by: Paschalis Tsilias <[email protected]> * Use snake_case for reason label Signed-off-by: Paschalis Tsilias <[email protected]> * Fix dropped samples metric Signed-off-by: Paschalis Tsilias <[email protected]> --------- Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> * fix(discovery): allow requireUpdate util to timeout in discovery/file/file_test.go. The loop ran indefinitely if the condition isn't met. Before, each iteration created a new timer channel which was always outpaced by the other timer channel with smaller duration. minor detail: There was a memory leak: resources of the ~10 previous timers were constantly kept. With the fix, we may keep the resources of one timer around for defaultWait but this isn't worth the changes to make it right. Signed-off-by: machine424 <[email protected]> * Merge pull request #13371 from kevinmingtarja/fix-isHeatmapData ui: fix handling of scalar and string in isHeatmapData * tsdb/{index,compact}: allow using custom postings encoding format (#13242) * tsdb/{index,compact}: allow using custom postings encoding format We would like to experiment with a different postings encoding format in Thanos so in this change I am proposing adding another argument to `NewWriter` which would allow users to change the format if needed. Also, wire the leveled compactor so that it would be possible to change the format there too. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb/compact: use a struct for leveled compactor options As discussed on Slack, let's use a struct for the options in leveled compactor. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb: make changes after Bryan's review - Make changes less intrusive - Turn the postings encoder type into a function - Add NewWriterWithEncoder() Signed-off-by: Giedrius Statkevičius <[email protected]> --------- Signed-off-by: Giedrius Statkevičius <[email protected]> * Cut 2.49.0-rc.2 Signed-off-by: bwplotka <[email protected]> * build(deps): bump actions/setup-go from 3.5.0 to 5.0.0 in /scripts (#13362) Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3.5.0 to 5.0.0. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](https://github.com/actions/setup-go/compare/6edd4406fa81c3da01a34fa6f6343087c207a568...0c52d547c9bc32b1aa3301fd7a9cb496313a4491) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump github/codeql-action from 2.22.8 to 3.22.12 (#13358) Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2.22.8 to 3.22.12. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](https://github.com/github/codeql-action/compare/407ffafae6a767df3e0230c3df91b6443ae8df75...012739e5082ff0c22ca6d6ab32e07c36df03c4a4) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * put @nexucis has a release shepherd (#13383) Signed-off-by: Augustin Husson <[email protected]> * Add analyze histograms command to promtool (#12331) Add `query analyze` command to promtool This command analyzes the buckets of classic and native histograms, based on data queried from the Prometheus query API, i.e. it doesn't require direct access to the TSDB files. Signed-off-by: Jeanette Tan <[email protected]> --------- Signed-off-by: Jeanette Tan <[email protected]> * included instance in all necessary descriptions Signed-off-by: Erik Sommer <[email protected]> * tsdb/compact: fix passing merge func Fixing a very small logical problem I've introduced :(. Signed-off-by: Giedrius Statkevičius <[email protected]> * tsdb: add enable overlapping compaction This functionality is needed in downstream projects because they have a separate component that does compaction. Upstreaming https://github.com/grafana/mimir-prometheus/blob/7c8e9a2a76fc729e9078889782928b2fdfe240e9/tsdb/compact.go#L323-L325. Signed-off-by: Giedrius Statkevičius <[email protected]> * Cut 2.49.0 Signed-off-by: bwplotka <[email protected]> * promtool: allow setting multiple matchers to "promtool tsdb dump" command. (#13296) Conditions are ANDed inside the same matcher but matchers are ORed Including unit tests for "promtool tsdb dump". Refactor some matchers scraping utils. Signed-off-by: machine424 <[email protected]> * Fixed changelog Signed-off-by: bwplotka <[email protected]> * tsdb/main: wire "EnableOverlappingCompaction" to tsdb.Options (#13398) This added the https://github.com/prometheus/prometheus/pull/13393 "EnableOverlappingCompaction" parameter to the compactor code but not to the tsdb.Options. I forgot about that. Add it to `tsdb.Options` too and set it to `true` in Prometheus. Copy/paste the description from https://github.com/prometheus/prometheus/pull/13393#issuecomment-1891787986 Signed-off-by: Giedrius Statkevičius <[email protected]> * Issue #13268: fix quality value in accept header Signed-off-by: Kumar Kalpadiptya Roy <[email protected]> * Cut 2.49.1 with scrape q= bugfix. Signed-off-by: bwplotka <[email protected]> * Cut 2.49.1 web package. Signed-off-by: bwplotka <[email protected]> * Restore more efficient version of NewPossibleNonCounterInfo annotation (#13022) Restore more efficient version of NewPossibleNonCounterInfo annotation Signed-off-by: Jeanette Tan <[email protected]> --------- Signed-off-by: Jeanette Tan <[email protected]> * Fix regressions introduced by #13242 Signed-off-by: Marco Pracucci <[email protected]> * fix slice copy in 1.20 (#13389) The slices package is added to the standard library in Go 1.21; we need to import from the exp area to maintain compatibility with Go 1.20. Signed-off-by: tyltr <[email protected]> * Docs: Query Basics: link to rate (#10538) Co-authored-by: Julien Pivotto <[email protected]> * chore(kubernetes): check preconditions earlier and avoid unnecessary checks or iterations Signed-off-by: machine424 <[email protected]> * Examples: link to `rate` for new users (#10535) * Examples: link to `rate` for new users Signed-off-by: Ted Robertson [email protected] Co-authored-by: Bryan Boreham <[email protected]> * promql: use natural sort in sort_by_label and sort_by_label_desc (#13411) These functions are intended for humans, as robots can already sort the results however they please. Humans like things sorted "naturally": * https://blog.codinghorror.com/sorting-for-humans-natural-sort-order/ A similar thing has been done to Grafana, which is also used by humans: * https://github.com/grafana/grafana/pull/78024 * https://github.com/grafana/grafana/pull/78494 Signed-off-by: Ivan Babrou <[email protected]> * TestLabelValuesWithMatchers: Add test case Signed-off-by: Arve Knudsen <[email protected]> * remove obsolete build tag Signed-off-by: tyltr <[email protected]> * Upgrade some golang dependencies for resty 2.11 Signed-off-by: Israel Blancas <[email protected]> * Native Histograms: support `native_histogram_min_bucket_factor` in scrape_config (#13222) Native Histograms: support native_histogram_min_bucket_factor in scrape_config --------- Signed-off-by: Ziqi Zhao <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> * Add warnings for histogramRate applied with isCounter not matching counter/gauge histogram (#13392) Add warnings for histogramRate applied with isCounter not matching counter/gauge histogram --------- Signed-off-by: Jeanette Tan <[email protected]> * Minor fixes to otlp vendor update script Signed-off-by: Goutham <[email protected]> * build(deps): bump github.com/hetznercloud/hcloud-go/v2 Bumps [github.com/hetznercloud/hcloud-go/v2](https://github.com/hetznercloud/hcloud-go) from 2.4.0 to 2.6.0. - [Release notes](https://github.com/hetznercloud/hcloud-go/releases) - [Changelog](https://github.com/hetznercloud/hcloud-go/blob/main/CHANGELOG.md) - [Commits](https://github.com/hetznercloud/hcloud-go/compare/v2.4.0...v2.6.0) --- updated-dependencies: - dependency-name: github.com/hetznercloud/hcloud-go/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Enhanced visibility for `promtool test rules` with JSON colored formatting (#13342) * Added diff flag for unit test to improvise readability & debugging Signed-off-by: Rewanth Tammana <[email protected]> * Removed blank spaces Signed-off-by: Rewanth Tammana <[email protected]> * Fixed linting error Signed-off-by: Rewanth Tammana <[email protected]> * Added cli flags to documentation Signed-off-by: Rewanth Tammana <[email protected]> * Revert unrrelated linting fixes Signed-off-by: Rewanth Tammana <[email protected]> * Fixed review suggestions Signed-off-by: Rewanth Tammana <[email protected]> * Cleanup Signed-off-by: Rewanth Tammana <[email protected]> * Updated flag description Signed-off-by: Rewanth Tammana <[email protected]> * Updated flag description Signed-off-by: Rewanth Tammana <[email protected]> --------- Signed-off-by: Rewanth Tammana <[email protected]> * storage: skip merging when no remote storage configured Prometheus is hard-coded to use a fanout storage between TSDB and a remote storage which by default is empty. This change detects the empty storage and skips merging between result sets, which would make `Select()` sort results. Bottom line: we skip a sort unless there really is some remote storage configured. Signed-off-by: Bryan Boreham <[email protected]> * Remove csmarchbanks from remote write owners (#13432) I have not had the time to keep up with remote write and have no plans to work on it in the near future so I am withdrawing my maintainership of that part of the codebase. I continue to focus on client_python. Signed-off-by: Chris Marchbanks <[email protected]> * add more context cancellation check at evaluation time Signed-off-by: Ben Ye <[email protected]> * Optimize label values with matchers by taking shortcuts (#13426) Don't calculate postings beforehand: we may not need them. If all matchers are for the requested label, we can just filter its values. Also, if there are no values at all, no need to run any kind of logic. Also add more labelValuesWithMatchers benchmarks Signed-off-by: Oleg Zaytsev <[email protected]> * Add automatic memory limit handling Enable automatic detection of memory limits and configure GOMEMLIMIT to match. * Also includes a flag to allow controlling the reserved ratio. Signed-off-by: SuperQ <[email protected]> * Update OSSF badge link (#13433) Provide a more user friendly interface Signed-off-by: Matthieu MOREL <[email protected]> * SD Managers taking over responsibility for registration of debug metrics (#13375) SD Managers take over responsibility for SD metrics registration --------- Signed-off-by: Paulin Todev <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> * Optimize histogram iterators (#13340) Optimize histogram iterators Histogram iterators allocate new objects in the AtHistogram and AtFloatHistogram methods, which makes calculating rates over long ranges expensive. In #13215 we allowed an existing object to be reused when converting an integer histogram to a float histogram. This commit follows the same idea and allows injecting an existing object in the AtHistogram and AtFloatHistogram methods. When the injected value is nil, iterators allocate new histograms, otherwise they populate and return the injected object. The commit also adds a CopyTo method to Histogram and FloatHistogram which is used in the BufferedIterator to overwrite items in the ring instead of making new copies. Note that a specialized HPoint pool is needed for all of this to work (`matrixSelectorHPool`). --------- Signed-off-by: Filip Petkovski <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> * doc: Mark `mad_over_time` as experimental (#13440) We forgot to do that in https://github.com/prometheus/prometheus/pull/13059 Signed-off-by: beorn7 <[email protected]> * Change metric label for Puppetdb from 'http' to 'puppetdb' Signed-off-by: Paulin Todev <[email protected]> * mirror metrics.proto change & generate code Signed-off-by: Ziqi Zhao <[email protected]> * TestHeadLabelValuesWithMatchers: Add test case (#13414) Add test case to TestHeadLabelValuesWithMatchers, while fixing a couple of typos in other test cases. Also enclosing some implicit sub-tests in a `t.Run` call to make them explicitly sub-tests. Signed-off-by: Arve Knudsen <[email protected]> * update all go dependencies (#13438) Signed-off-by: Augustin Husson <[email protected]> * build(deps): bump the k8s-io group with 2 updates (#13454) Bumps the k8s-io group with 2 updates: [k8s.io/api](https://github.com/kubernetes/api) and [k8s.io/client-go](https://github.com/kubernetes/client-go). Updates `k8s.io/api` from 0.28.4 to 0.29.1 - [Commits](https://github.com/kubernetes/api/compare/v0.28.4...v0.29.1) Updates `k8s.io/client-go` from 0.28.4 to 0.29.1 - [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md) - [Commits](https://github.com/kubernetes/client-go/compare/v0.28.4...v0.29.1) --- updated-dependencies: - dependency-name: k8s.io/api dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-io - dependency-name: k8s.io/client-go dependency-type: direct:production update-type: version-update:semver-minor dependency-group: k8s-io ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump the go-opentelemetry-io group with 1 update (#13453) Bumps the go-opentelemetry-io group with 1 update: [go.opentelemetry.io/collector/semconv](https://github.com/open-telemetry/opentelemetry-collector). Updates `go.opentelemetry.io/collector/semconv` from 0.92.0 to 0.93.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-collector/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-collector/blob/main/CHANGELOG-API.md) - [Commits](https://github.com/open-telemetry/opentelemetry-collector/compare/v0.92.0...v0.93.0) --- updated-dependencies: - dependency-name: go.opentelemetry.io/collector/semconv dependency-type: direct:production update-type: version-update:semver-minor dependency-group: go-opentelemetry-io ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump actions/upload-artifact from 3.1.3 to 4.0.0 (#13355) Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.3 to 4.0.0. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](https://github.com/actions/upload-artifact/compare/a8a3f3ad30e3422c9c7b888a15615d19a852ae32...c7d193f32edcb7bfad88892161225aeda64e9392) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * build(deps): bump bufbuild/buf-push-action (#13357) Bumps [bufbuild/buf-push-action](https://github.com/bufbuild/buf-push-action) from 342fc4cdcf29115a01cf12a2c6dd6aac68dc51e1 to a654ff18effe4641ebea4a4ce242c49800728459. - [Release notes](https://github.com/bufbuild/buf-push-action/releases) - [Commits](https://github.com/bufbuild/buf-push-action/compare/342fc4cdcf29115a01cf12a2c6dd6aac68dc51e1...a654ff18effe4641ebea4a4ce242c49800728459) --- updated-dependencies: - dependency-name: bufbuild/buf-push-action dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Labels: Add DropMetricName function, used in PromQL (#13446) This function is called very frequently when executing PromQL functions, and we can do it much more efficiently inside Labels. In the common case that `__name__` comes first in the labels, we simply re-point to start at the next label, which is nearly free. `DropMetricName` is now so cheap I removed the cache - benchmarks show everything still goes faster. Signed-off-by: Bryan Boreham <[email protected]> * tsdb: simplify internal series delete function (#13261) Lifting an optimisation from Agent code, `seriesHashmap.del` can use the unique series reference, doesn't need to check Labels. Also streamline the logic for deleting from `unique` and `conflicts` maps, and add some comments to help the next person. Signed-off-by: Bryan Boreham <[email protected]> * otlptranslator/update-copy.sh: Fix sed command lines Signed-off-by: Arve Knudsen <[email protected]> * Rollback k8s.io requirements (#13462) Rollback k8s.io Go modules to v0.28.6 to avoid forcing upgrade of Go to 1.21. This allows us to keep compatibility with the currently supported upstream Go releases. Signed-off-by: SuperQ <[email protected]> * Make update-copy.sh work for both OSX and GNU sed Signed-off-by: Arve Knudsen <[email protected]> * Name @beorn7 and @krajorama as maintainers for native histograms I have been the de-facto maintainer for native histograms from the beginning. So let's put this into MAINTAINERS.md. In addition, I hereby proposose George Krajcsovits AKA Krajo as a co-maintainer. He has contributed a lot of native histogram code, but more importantly, he has contributed substantially to reviewing other contributors' native histogram code, up to a point where I was merely rubberstamping the PRs he had already reviewed. I'm confident that he is ready to to be granted commit rights as outlined in the "Maintainers" section of the governance: https://prometheus.io/governance/#maintainers According to the same section of the governance, I will announce the proposed change on the developers mailing list and will give some time for lazy consensus before merging this PR. Signed-off-by: beorn7 <[email protected]> * ui/fix: correct url handling for stacked graphs (#13460) Signed-off-by: Yury Moladau <[email protected]> * tsdb: use cheaper Mutex on series Mutex is 8 bytes; RWMutex is 24 bytes and much more complicated. Since `RLock` is only used in two places, `UpdateMetadata` and `Delete`, neither of which are hotspots, we should use the cheaper one. Signed-off-by: Bryan Boreham <[email protected]> * Fix last_over_time for native histograms The last_over_time retains a histogram sample without making a copy. This sample is now coming from the buffered iterator used for windowing functions, and can be reused for reading subsequent samples as the iterator progresses. I would propose copying the sample in the last_over_time function, similar to how it is done for rate, sum_over_time and others. Signed-off-by: Filip Petkovski <[email protected]> * Implementation NOTE: Rebased from main after refactor in #13014 Signed-off-by: Danny Kopping <[email protected]> * Add feature flag Signed-off-by: Danny Kopping <[email protected]> * Refactor concurrency control Signed-off-by: Danny Kopping <[email protected]> * Optimising dependencies/dependents funcs to not produce new slices each request Signed-off-by: Danny Kopping <[email protected]> * Refactoring Signed-off-by: Danny Kopping <[email protected]> * Rename flag Signed-off-by: Danny Kopping <[email protected]> * Refactoring for performance, and to allow controller to be overridden Signed-off-by: Danny Kopping <[email protected]> * Block until all rules, both sync & async, have completed evaluating Updated & added tests Review feedback nits Return empty map if not indeterminate Use highWatermark to track inflight requests counter Appease the linter Clarify feature flag Signed-off-by: Danny Kopping <[email protected]> * Fix typo in CLI flag description Signed-off-by: Marco Pracucci <[email protected]> * Fixed auto-generated doc Signed-off-by: Marco Pracucci <[email protected]> * Improve doc Signed-off-by: Marco Pracucci <[email protected]> * Simplify the design to update concurrency controller once the rule evaluation has done Signed-off-by: Marco Pracucci <[email protected]> * Add more test cases to TestDependenciesEdgeCases Signed-off-by: Marco Pracucci <[email protected]> * Added more test cases to TestDependenciesEdgeCases Signed-off-by: Marco Pracucci <[email protected]> * Improved RuleConcurrencyController interface doc Signed-off-by: Marco Pracucci <[email protected]> * Introduced sequentialRuleEvalController Signed-off-by: Marco Pracucci <[email protected]> * Remove superfluous nil check in Group.metrics Signed-off-by: Marco Pracucci <[email protected]> * api: Serialize discovered and target labels into JSON directly (#13469) Converted maps into labels.Labels to avoid a lot of copying of data which leads to very high memory consumption while opening the /service-discovery endpoint in the Prometheus UI Signed-off-by: Leegin <[email protected]> * api: Serialize discovered labels into JSON directly in dropped targets (#13484) Converted maps into labels.Labels to avoid a lot of copying of data which leads to very high memory consumption while opening the /service-discovery endpoint in the Prometheus UI Signed-off-by: Leegin <[email protected]> * Add ShardedPostings() support to TSDB (#10421) This PR is a reference implementation of the proposal described in #10420. In addition to what described in #10420, in this PR I've introduced labels.StableHash(). The idea is to offer an hashing function which doesn't change over time, and that's used by query sharding in order to get a stable behaviour over time. The implementation of labels.StableHash() is the hashing function used by Prometheus before stringlabels, and what's used by Grafana Mimir for query sharding (because built before stringlabels was a thing). Follow up work As mentioned in #10420, if this PR is accepted I'm also open to upload another foundamental piece used by Grafana Mimir query sharding to accelerate the query execution: an optional, configurable and fast in-memory cache for the series hashes. Signed-off-by: Marco Pracucci <[email protected]> * storage/remote: document why two benchmarks are skipped One was silently doing nothing; one was doing something but the work didn't go up linearly with iteration count. Signed-off-by: Bryan Boreham <[email protected]> * Pod status changes not discovered by Kube Endpoints SD (#13337) * fix(discovery/kubernetes/endpoints): react to changes on Pods because some modifications can occur on them without triggering an update on the related Endpoints (The Pod phase changing from Pending to Running e.g.). --------- Signed-off-by: machine424 <[email protected]> Co-authored-by: Guillermo Sanchez Gavier <[email protected]> * Small improvements, add const, remove copypasta (#8106) Signed-off-by: Mikhail Fesenko <[email protected]> Signed-off-by: Jesus Vazquez <[email protected]> * Proposal to improve FPointSlice and HPointSlice allocation. (#13448) * Reusing points slice from previous series when the slice is under utilized * Adding comments on the bench test Signed-off-by: Alan Protasio <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * go mod tidy Signed-off-by: Nicolás Pazos <[email protected]> --------- Signed-off-by: Julian Wiedmann <[email protected]> Signed-off-by: Bryan Boreham <[email protected]> Signed-off-by: Erik Sommer <[email protected]> Signed-off-by: Linas Medziunas <[email protected]> Signed-off-by: bwplotka <[email protected]> Signed-off-by: Arianna Vespri <[email protected]> Signed-off-by: machine424 <[email protected]> Signed-off-by: Daniel Nicholls <[email protected]> Signed-off-by: Daniel Kerbel <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Jan Fajerski <[email protected]> Signed-off-by: Kevin Mingtarja <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Giedrius Statkevičius <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Jeanette Tan <[email protected]> Signed-off-by: Bartlomiej Plotka <[email protected]> Signed-off-by: Kumar Kalpadiptya Roy <[email protected]> Signed-off-by: Marco Pracucci <[email protected]> Signed-off-by: tyltr <[email protected]> Signed-off-by: Ted Robertson [email protected] Signed-off-by: Ivan Babrou <[email protected]> Signed-off-by: Arve Knudsen <[email protected]> Signed-off-by: Israel Blancas <[email protected]> Signed-off-by: Ziqi Zhao <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Signed-off-by: Goutham <[email protected]> Signed-off-by: Rewanth Tammana <[email protected]> Signed-off-by: Chris Marchbanks <[email protected]> Signed-off-by: Ben Ye <[email protected]> Signed-off-by: Oleg Zaytsev <[email protected]> Signed-off-by: SuperQ <[email protected]> Signed-off-by: Ben Kochie <[email protected]> Signed-off-by: Matthieu MOREL <[email protected]> Signed-off-by: Paulin Todev <[email protected]> Signed-off-by: Filip Petkovski <[email protected]> Signed-off-by: beorn7 <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Yury Moladau <[email protected]> Signed-off-by: Danny Kopping <[email protected]> Signed-off-by: Leegin <[email protected]> Signed-off-by: Mikhail Fesenko <[email protected]> Signed-off-by: Jesus Vazquez <[email protected]> Signed-off-by: Alan Protasio <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> Co-authored-by: Julian Wiedmann <[email protected]> Co-authored-by: Bryan Boreham <[email protected]> Co-authored-by: Erik Sommer <[email protected]> Co-authored-by: Linas Medziunas <[email protected]> Co-authored-by: Bartlomiej Plotka <[email protected]> Co-authored-by: Arianna Vespri <[email protected]> Co-authored-by: machine424 <[email protected]> Co-authored-by: daniel-resdiary <[email protected]> Co-authored-by: Daniel Kerbel <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jan Fajerski <[email protected]> Co-authored-by: Kevin Mingtarja <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Marc Tudurí <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Giedrius Statkevičius <[email protected]> Co-authored-by: Augustin Husson <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> Co-authored-by: zenador <[email protected]> Co-authored-by: gotjosh <[email protected]> Co-authored-by: Ben Kochie <[email protected]> Co-authored-by: Kumar Kalpadiptya Roy <[email protected]> Co-authored-by: Marco Pracucci <[email protected]> Co-authored-by: tyltr <[email protected]> Co-authored-by: Ted Robertson <[email protected]> Co-authored-by: Julien Pivotto <[email protected]> Co-authored-by: Matthias Loibl <[email protected]> Co-authored-by: Ivan Babrou <[email protected]> Co-authored-by: Arve Knudsen <[email protected]> Co-authored-by: Israel Blancas <[email protected]> Co-authored-by: Ziqi Zhao <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> Co-authored-by: Björn Rabenstein <[email protected]> Co-authored-by: Goutham <[email protected]> Co-authored-by: Rewanth Tammana <[email protected]> Co-authored-by: Chris Marchbanks <[email protected]> Co-authored-by: Ben Ye <[email protected]> Co-authored-by: Oleg Zaytsev <[email protected]> Co-authored-by: Matthieu MOREL <[email protected]> Co-authored-by: Paulin Todev <[email protected]> Co-authored-by: Filip Petkovski <[email protected]> Co-authored-by: Yury Molodov <[email protected]> Co-authored-by: Danny Kopping <[email protected]> Co-authored-by: Leegin <[email protected]> Co-authored-by: Guillermo Sanchez Gavier <[email protected]> Co-authored-by: Mikhail Fesenko <[email protected]> Co-authored-by: Alan Protasio <[email protected]> * remote write 2.0 - follow up improvements (#13478) * move remote write proto version config from a remote storage config to a per remote write configuration option Signed-off-by: Callum Styan <[email protected]> * rename scrape config for metadata, fix 2.0 header var name/value (was 1.1), and more clean up Signed-off-by: Callum Styan <[email protected]> * address review comments, mostly lint fixes Signed-off-by: Callum Styan <[email protected]> * another lint fix Signed-off-by: Callum Styan <[email protected]> * lint imports Signed-off-by: Callum Styan <[email protected]> --------- Signed-off-by: Callum Styan <[email protected]> * go mod tidy Signed-off-by: Nicolás Pazos <[email protected]> * Added commmentary to RW 2.0 protocol for easier adoption and explicit semantics. (#13502) * Added commmentary to RW 2.0 protocol for easier adoption and explicit semantics. Signed-off-by: bwplotka <[email protected]> * Apply suggestions from code review Co-authored-by: Nico Pazos <[email protected]> Signed-off-by: Callum Styan <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> Signed-off-by: Callum Styan <[email protected]> Co-authored-by: Callum Styan <[email protected]> Co-authored-by: Nico Pazos <[email protected]> * prw2.0: Added support for "custom" layouts for native histogram proto (#13558) * prw2.0: Added support for "custom" layouts for native histogram. Result of the discussions: * https://github.com/prometheus/prometheus/issues/13475#issuecomment-1931496924 * https://cloud-native.slack.com/archives/C02KR205UMU/p1707301006347199 Signed-off-by: bwplotka <[email protected]> * prw2.0: Added support for "custom" layouts for native histogram. Result of the discussions: * https://github.com/prometheus/prometheus/issues/13475#issuecomment-1931496924 * https://cloud-native.slack.com/archives/C02KR205UMU/p1707301006347199 Signed-off-by: bwplotka <[email protected]> # Conflicts: # prompb/write/v2/types.pb.go * Update prompb/write/v2/types.proto Co-authored-by: George Krajcsovits <[email protected]> Signed-off-by: Bartlomiej Plotka <[email protected]> * Addressed comments, fixed test. Signed-off-by: bwplotka <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> Signed-off-by: Bartlomiej Plotka <[email protected]> Co-authored-by: George Krajcsovits <[email protected]> * first draft of content negotiation Signed-off-by: Alex Greenbank <[email protected]> * Lint Signed-off-by: Alex Greenbank <[email protected]> * Fix race in test Signed-off-by: Alex Greenbank <[email protected]> * Fix another test race Signed-off-by: Alex Greenbank <[email protected]> * Almost done with lint Signed-off-by: Alex Greenbank <[email protected]> * Fix todos around 405 HEAD handling Signed-off-by: Alex Greenbank <[email protected]> * Changes based on review comments Signed-off-by: Alex Greenbank <[email protected]> * Update storage/remote/client.go Co-authored-by: Bartlomiej Plotka <[email protected]> Signed-off-by: Alex Greenbank <[email protected]> * Latest updates to review comments Signed-off-by: Alex Greenbank <[email protected]> * latest tweaks Signed-off-by: Alex Greenbank <[email protected]> * remote write 2.0 - content negotiation remediation (#13921) * Consolidate renegotiation error into one, fix tests Signed-off-by: Alex Greenbank <[email protected]> * fix metric name and actuall increment counter Signed-off-by: Alex Greenbank <[email protected]> --------- Signed-off-by: Alex Greenbank <[email protected]> * Fixes after main sync. Signed-off-by: bwplotka <[email protected]> * [PRW 2.0] Moved rw2 proto to the full path (both package name and placement) (#13973) undefined * [PRW2.0] Remove benchmark scripts (#13949) See rationales on https://docs.google.com/document/d/1Bpf7mYjrHUhPHkie0qlnZFxzgqf_L32kM8ZOknSdJrU/edit Signed-off-by: bwplotka <[email protected]> * rw20: Update prw commentary after Callum spec review (#14136) * rw20: Update prw commentary after Callum spec review Signed-off-by: Bartlomiej Plotka <[email protected]> * Update types.proto Signed-off-by: Bartlomiej Plotka <[email protected]> --------- Signed-off-by: Bartlomiej Plotka <[email protected]> * [PRW 2.0] Updated spec proto (2.0-rc.1); deterministic v1 interop; to be sympathetic with implementation. (#14330) * [PRW 2.0] Updated spec proto (2.0-rc.1); deterministic v1 interop; to be sympathetic with implementation. Signed-off-by: bwplotka <[email protected]> * update custom marshalling Signed-off-by: bwplotka <[email protected]> * Removed confusing comments. Signed-off-by: bwplotka <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> * [PRW-2.0] (chain1) New Remote Write 2.0 Config options for 2.0-rc.1 spec. (#14335) NOTE: For simple review this change does not touch remote/ packages, only main and configs. Spec: https://prometheus.io/docs/specs/remote_write_spec_2_0 Supersedes https://github.com/prometheus/prometheus/pull/13968 Signed-off-by: bwplotka <[email protected]> * [PRW-2.0] (part 2) Removed automatic negotiation, updates for the latest spec semantics in remote pkg (#14329) * [PRW-2.0] (part2) Moved to latest basic negotiation & spec semantics. Spec: https://github.com/prometheus/docs/pull/2462 Supersedes https://github.com/prometheus/prometheus/pull/13968 Signed-off-by: bwplotka <[email protected]> # Conflicts: # config/config.go # docs/configuration/configuration.md # storage/remote/queue_manager_test.go # storage/remote/write.go # web/api/v1/api.go * Addressed comments. Signed-off-by: bwplotka <[email protected]> --------- Signed-off-by: bwplotka <[email protected]> * lint Signed-off-by: Nicolás Pazos <[email protected]> * storage/remote tests: refactor: extract function newTestQueueManager To reduce repetition. Signed-off-by: Bryan Boreham <[email protected]> Signed-off-by: Nicolás Pazos <[email protected]> * use newTestQueueManager for test Signed-off-by: Nicolás Pazos <[email protected]> * go mod tidy Signed-off-by: Nicolás Pazos <[email protected]> * [PRW 2.0] (part3) moved type specific conversions to prompb and writev2 codecs. Signed-off-by: bwplotka <[email protected]> * Added test for rwProtoMsgFlagParser; fixed TODO comment. Signed-off-by: bwplotka <[email protected]> * Renamed DecodeV2WriteRequestStr to DecodeWriteV2Request (with tests). Signed-off-by: bwplotka <[email protected]> * Addressed comments on remote_storage example, updated it for 2.0 Signed-off-by: bwplotka <[email protected]> * Fixed `--enable-feature=metadata-wal-records` docs and error when using PRW 2.0 without it. Signed-off-by: bwplotka <[email protected]> * Addressed Callum comments on custom*.go Signed-off-by: bwplotka <[email protected]> * Added TODO to genproto. Signed-off-by: bwplotka <[email protected]> * Addressed Callum comments in remote pkg. Signed-off-by: bwplotka <[email protected]> * Added metadata validation to write handler test; fixed ToMetadata. Signed-off-by: bwplotka <[email protected]> * Addressed rest of Callum comments. Signed-off-by: bwplotka <[email protected]> * Fixed writev2.FromMetadataType (was wrongly using prompb). Signed-off-by: bwplotka <[email protected]> * fix a few import whitespaces Signed-off-by: Callum Styan <[email protected]> * add a default case with an error to the example RW receiver Signed-off-by: Callum Styan <[email protected]> * more minor import whitespace chagnes Signed-off-by: Callum Styan <[email protected]> * Apply suggestions from code review Signed-off-by: Bartlomiej Plotka <[email protected]> * Update storage/remote/queue_manager_test.go Signed-off-by: Bartlomiej Plotka <[email protected]> --------- Signed-off-by: Nicolás Pazos <[email protected]> Signed-off-by: Callum Styan <[email protected]> Signed-off-by: bwplotka <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Julian Wiedmann <[email protected]> Signed-off-by: Bryan Boreham <[email protected]> Signed-off-by: Erik Sommer <[email protected]> Signed-off-by: Linas Medziunas <[email protected]> Signed-off-by: Arianna Vespri <[email protected]> Signed-off-by: machine424 <[email protected]> Signed-off-by: Daniel Nicholls <[email protected]> Signed-off-by: Daniel Kerbel <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Jan Fajerski <[email protected]> Signed-off-by: Kevin Mingtarja <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Marc Tuduri <[email protected]> Signed-off-by: Paschalis Tsilias <[email protected]> Signed-off-by: Giedrius Statkevičius <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Jeanette Tan <[email protected]> Signed-off-by: Bartlomiej Plotka <[email protected]> Signed-off-by: Kumar Kalpadiptya Roy <[email protected]> Signed-off-by: Marco Pracucci <[email protected]> Signed-off-by: tyltr <[email protected]> Signed-off-by: Ted Robertson [email protected] Signed-off-by: Ivan Babrou <[email protected]> Signed-off-by: Arve Knudsen <[email protected]> Signed-off-by: Israel Blancas <[email protected]> Signed-off-by: Ziqi Zhao <[email protected]> Signed-off-by: Björn Rabenstein <[email protected]> Signed-off-by: Goutham <[email protected]> Signed-off-by: Rewanth Tammana <[email protected]> Signed-off-by: Chris Marchbanks <[email protected]> Signed-off-by: Ben Ye <[email protected]> Signed-off-by: Oleg Zaytsev <[email protected]> Signed-off-by: SuperQ <[email protected]> Signed-off-by: Ben Kochie <[email protected]> Signed-off-by: Matthieu MOREL <[email protected]> Signed-off-by: Paulin Todev <[email protected]> Signed-off-by: Filip Petkovski <[email protected]> Signed-off-by: beorn7 <[email protected]> Signed-off-by: Augustin Husson <[email protected]> Signed-off-by: Yury Moladau <[email protected]> Signed-off-by: Danny Kopping <[email protected]> Signed-off-by: Leegin <[email protected]> Signed-off-by: Mikhail Fesenko <[email protected]> Signed-off-by: Jesus Vazquez <[email protected]> Signed-off-by: Alan Protasio <[email protected]> Signed-off-by: Alex Greenbank <[email protected]> Co-authored-by: Nicolás Pazos <[email protected]> Co-authored-by: Callum Styan <[email protected]> Co-authored-by: Nicolás Pazos <[email protected]> Co-authored-by: alexgreenbank <[email protected]> Co-authored-by: Marco Pracucci <[email protected]> Co-authored-by: Paschalis Tsilias <[email protected]> Co-authored-by: Julian Wiedmann <[email protected]> Co-authored-by: Bryan Boreham <[email protected]> Co-authored-by: Erik Sommer <[email protected]> Co-authored-by: Linas Medziunas <[email protected]> Co-authored-by: Arianna Vespri <[email protected]> Co-authored-by: machine424 <[email protected]> Co-authored-by: daniel-resdiary <[email protected]> Co-authored-by: Daniel Kerbel <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot…
fbegyn
pushed a commit
to fbegyn/prometheus
that referenced
this pull request
Jul 11, 2024
This PR is a reference implementation of the proposal described in prometheus#10420. In addition to what described in prometheus#10420, in this PR I've introduced labels.StableHash(). The idea is to offer an hashing function which doesn't change over time, and that's used by query sharding in order to get a stable behaviour over time. The implementation of labels.StableHash() is the hashing function used by Prometheus before stringlabels, and what's used by Grafana Mimir for query sharding (because built before stringlabels was a thing). Follow up work As mentioned in prometheus#10420, if this PR is accepted I'm also open to upload another foundamental piece used by Grafana Mimir query sharding to accelerate the query execution: an optional, configurable and fast in-memory cache for the series hashes. Signed-off-by: Marco Pracucci <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a reference implementation of the proposal described in #10420.
In addition to what described in #10420, in this PR I've introduced
labels.StableHash(). The idea is to offer an hashing function which doesn't change over time, and that's used by query sharding in order to get a stable behaviour over time. The implementation oflabels.StableHash()is the hashing function used by Prometheus before stringlabels, and what's used by Grafana Mimir for query sharding (because built before stringlabels was a thing).Follow up work
As mentioned in #10420, if this PR is accepted I'm also open to upload another foundamental piece used by Grafana Mimir query sharding to accelerate the query execution: an optional, configurable and fast in-memory cache for the series hashes.