Sync upstream Prometheus at 9cc4741#844
Merged
Merged
Conversation
memChunk is a linked list, speed up some common operations when there's no need to iterate all elements on the list. Signed-off-by: Łukasz Mierzwa <l.mierzwa@gmail.com>
Signed-off-by: Jan Fajerski <jfajersk@redhat.com>
Signed-off-by: Jan Fajerski <jfajersk@redhat.com>
Use the `-n / --noname` option to ignore non-deterministric information, such as timestamps, in gzip metadata. This is required for [reproducible builds](https://reproducible-builds.org/). Signed-off-by: Robin Candau <robincandau@protonmail.com>
Signed-off-by: Jan Fajerski <jfajersk@redhat.com>
Prepare release 3.2.0
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
chore: enable unused-parameter from revive
Signed-off-by: Julius Volz <julius.volz@gmail.com>
Fix health color for target pools with single target that is down
The `Accept` header should not include `escape=allow-utf-8` unless explicitly requested. Conveniently, there was already a test covering this header's value, it just required updating so it also asserts that this value in the header is not set in the cases we don't expect it to be set. I also converted those tests into table tests to help make failures clearer. Issue: prometheus/prometheus#15857 Signed-off-by: Matt Hughes <mhughes@uw.co.uk>
Signed-off-by: beforetech <mail@before.tech>
Short-cut common memChunk operations
Signed-off-by: Jan Fajerski <jfajersk@redhat.com>
Prepare release 3.2.1
When intentionally dropping all alerts in a batch, `SendAll` returns false, increasing the dropped_total metric. This makes it difficult to tell if there is a connection issue between Prometheus and AlertManager or a result of intentionally dropping alerts. Have `SendAll` return `true` when no batches were sent by keeping track of the number of AlertManager request attempts. If no attempts were made, then the send is successful. Fixes: #15978 Signed-off-by: Justin Cichra <jcichra@cloudflare.com>
build: Ignore timestamps recording in gzip metadata
Signed-off-by: Sebastian Schubert <basti@schubert.digital>
Signed-off-by: beorn7 <beorn@grafana.com>
…uitive Signed-off-by: Julius Volz <julius.volz@gmail.com>
doc: clarified behaviour of source_labels in relabel_config
* fix: set gogc default value when config body is empty Signed-off-by: Danial Eskandari <doneskandari@gmail.com> * refactor: explicitly check value 75 in `TestGoGCDefaultValueOnEmptyConfigBody` add GoGC assertion in `TestEmptyConfig`, also removed the no longer needed runtime config assignment in `TestEmptyGlobalBlock` Signed-off-by: Danial Eskandari <doneskandari@gmail.com> * refactor: remove `TestGoGCDefaultValueOnEmptyConfigBody` to reduce duplicate assertions Signed-off-by: Danial Eskandari <doneskandari@gmail.com> --------- Signed-off-by: Danial Eskandari <doneskandari@gmail.com>
Signed-off-by: Julius Volz <julius.volz@gmail.com>
Targets API: Remove superfluous if() and make variable names more intuitive
This is needed so we can display dropped targets in the right scrape pool in the web interface even if they have a non-default job name. See also prometheus/prometheus#16078 Fixes prometheus/prometheus#16065 Signed-off-by: Julius Volz <julius.volz@gmail.com>
Include scrape pool name for dropped targets in /api/v1/targets
Signed-off-by: co63oc <co63oc@users.noreply.github.com>
notifier: Consider alert relabeling when deciding if alerts are dropped
Bumps [eslint-plugin-react-refresh](https://github.com/ArnaudBarre/eslint-plugin-react-refresh) from 0.4.16 to 0.4.19. - [Release notes](https://github.com/ArnaudBarre/eslint-plugin-react-refresh/releases) - [Changelog](https://github.com/ArnaudBarre/eslint-plugin-react-refresh/blob/main/CHANGELOG.md) - [Commits](ArnaudBarre/eslint-plugin-react-refresh@v0.4.16...v0.4.19) --- updated-dependencies: - dependency-name: eslint-plugin-react-refresh dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.20.0 to 8.25.0. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.25.0/packages/parser) --- updated-dependencies: - dependency-name: "@typescript-eslint/parser" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Patryk Prus <p@trykpr.us>
Signed-off-by: Patryk Prus <p@trykpr.us>
Signed-off-by: Patryk Prus <p@trykpr.us>
…p some OOO series as well Signed-off-by: Patryk Prus <p@trykpr.us>
TSDB: Track count of unknown series referenced during WAL/WBL replay
Signed-off-by: Patryk Prus <p@trykpr.us>
* Use more informative error msg for timeouts on remote read Signed-off-by: Jeanette Tan <jeanette.tan@grafana.com>
…#16207) * Remove experimental out-of-order native histogram flag This feature has been available in Prometheus since September 2024, and has no known issues. Therefore proposing to remove the flag entirely and always have it on. Note that there are still two settings that need to be configured (out-of-order time window > 0 and native histograms enabled) for this feature to work. Signed-off-by: Fiona Liao <fiona.liao@grafana.com> * Update CHANGELOG Signed-off-by: Fiona Liao <fiona.liao@grafana.com> * Keep feature flag with warning Signed-off-by: Fiona Liao <fiona.liao@grafana.com> * Update CHANGELOG Signed-off-by: Fiona Liao <fiona.liao@grafana.com> * Update tsdb/head_append.go Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com> Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com> * Update CHANGELOG.md Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com> Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com> * Update tsdb/head_append.go Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com> Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com> * Additional cleanup of comments and test names Signed-off-by: Fiona Liao <fiona.liao@grafana.com> --------- Signed-off-by: Fiona Liao <fiona.liao@grafana.com> Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com> Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com>
Fix rules UI to display correct value for 'keepFiringFor'
Signed-off-by: Patryk Prus <p@trykpr.us>
Add CHANGELOG entry
…… (#16195) Let HistogramAppender.appendable return CounterResetHeader instead of boolean Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com> Signed-off-by: Björn Rabenstein <github@rabenste.in> --------- Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com> Signed-off-by: Björn Rabenstein <github@rabenste.in> Co-authored-by: Björn Rabenstein <github@rabenste.in>
Signed-off-by: Patryk Prus <p@trykpr.us>
Address data race in main_test.go
Fix flaky TestReadClient/timeout test
Signed-off-by: Patryk Prus <p@trykpr.us>
chore(discovery): disable additional test as flaky on windows
|
|
eb63c74 to
b0d6b93
Compare
aknuds1
approved these changes
Mar 19, 2025
4 tasks
This was referenced Mar 20, 2025
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
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 brings changes from prometheus/prometheus up to 2025-03-19 commit prometheus/prometheus@9cc4741
Diff: prometheus/prometheus@8356990...9cc4741