Bug 1889711: bump Prometheus to v2.22.2 #62
Merged
Conversation
This file contains 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
Signed-off-by: Drumil Patel <drumilpatel720@gmail.com>
…tions (prometheus#7860) Signed-off-by: Vijay Samuel <vjsamuel@ebay.com>
We're using Prometheus behind a reverse proxy which authenticates everything based on cookies. Without this attribute the cookie is not send with the request for the manifest.json Signed-off-by: Stefan Bueringer <sbueringer@gmail.com>
Signed-off-by: johncming <johncming@yahoo.com>
…t directory (prometheus#7850) * web: Close and exhaust all response bodies in tests Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu> * Remove empty snapshot directory Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu> * Fix lint Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
…metheus#7871) * Update the storage.md to fix typo, and some grammar/format issue Signed-off-by: Luke Chen <showuon@gmail.com>
The documentation referenced "data volume containers", which were superseded by named volume support in Docker several years ago. There were to bind-mounting examples in the docs that are effectively doing the same thing, but the description of the second was somewhat erroneous. Signed-off-by: Lars Kellogg-Stedman <lars@redhat.com>
Signed-off-by: Jaga Santagostino <jagasantagostino@gmail.com>
Signed-off-by: johncming <johncming@yahoo.com>
…ion section (prometheus#7875) Signed-off-by: Luke Chen <showuon@gmail.com>
…s#6477) Signed-off-by: Ganesh Vernekar <cs15btech11018@iith.ac.in>
* Unnecessary go routine spawn. * Remove unnecessary local variable creation. Signed-off-by: iurii <iurii@coins.ph> Co-authored-by: iurii <iurii@coins.ph>
Signed-off-by: johncming <johncming@yahoo.com>
Upgrade to promu 0.6.0. Signed-off-by: Ben Kochie <superq@gmail.com>
…us#7881) This should make the `NewReadClient()` exported outside the `remote` package Signed-off-by: Anand Sanmukhani <asanmukh@redhat.com>
Signed-off-by: Drumil Patel <drumilpatel720@gmail.com>
`promtool update rules` is no longer available after Prometheus 2.5. Document this to avoid confusion such as in prometheus#6988. Signed-off-by: Christian Hoffmann <mail@hoffmann-christian.info>
The collapse all option as suggested in the corresponding issue has been implemented in this PR. Signed-off-by: Deepjyoti Mondal <djmdeveloper060796@gmail.com>
Update `ALERTS` example be more meta-syntactic. Signed-off-by: Justin Walz <justin.walz10@gmail.com>
Signed-off-by: BurtonQin <bobbqqin@gmail.com>
Merge release 2.21 into master
* Fix EQL when using = inside braces. Signed-off-by: Harkishen-Singh <harkishensingh@hotmail.com> * EQL => EQLC and ASSIGN => EQL Signed-off-by: Harkishen-Singh <harkishensingh@hotmail.com> * Aligned yacc code. Signed-off-by: Harkishen-Singh <harkishensingh@hotmail.com>
The remote read client needs to use the nethttp.Transport wrapper in order for spans to be instrumented properly. Signed-off-by: Chris Marchbanks <csmarchbanks@gmail.com>
* Default to bigger remote_write sends Raise the default MaxSamplesPerSend to amortise the cost of remote calls across more samples. Lower MaxShards to keep the expected max memory usage within reason. Signed-off-by: Bryan Boreham <bryan@weave.works> * Change default Capacity to 2500 To maintain ratio with MaxSamplesPerSend Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
prometheus#7915) Signed-off-by: Xiaochao Dong (@damnever) <dxc.wolf@gmail.com>
Merge release 2.21 in master
Signed-off-by: Luke Chen <showuon@gmail.com>
* web: Remove APIv2 Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
Signed-off-by: venkatbvc <chalapathirao@gmail.com>
v2.22.1 Signed-off-by: Simon Pasquier <spasquie@redhat.com>
simonpasquier
changed the title
|
@simonpasquier: This pull request references Bugzilla bug 1889711, which is valid. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
1 similar comment
|
@simonpasquier: This pull request references Bugzilla bug 1889711, which is valid. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/hold needs #63 first |
openshift-ci-robot
added
the
do-not-merge/hold
|
/retest |
|
/unhold |
openshift-ci-robot
removed
the
do-not-merge/hold
Manager.reload takes the mutex that would make it safe, however releases it before the goroutines spawned are finished with it. Thus more explicit locking of scrapePool.Sync/stop/reload is needed. Signed-off-by: Brian Brazil <brian.brazil@robustperception.io>
Signed-off-by: Frederic Branczyk <fbranczyk@gmail.com>
Cut v2.22.2
|
/hold v2.22.2 is out :) |
openshift-ci-robot
added
the
do-not-merge/hold
v2.22.2 Signed-off-by: Simon Pasquier <spasquie@redhat.com>
simonpasquier
changed the title
|
@simonpasquier: This pull request references Bugzilla bug 1889711, which is valid. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/hold cancel |
openshift-ci-robot
removed
the
do-not-merge/hold
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juzhao, lilic, simonpasquier The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
sdodson
added
the
cherry-pick-approved
|
@simonpasquier: All pull requests linked via external trackers have merged: Bugzilla bug 1889711 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
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.
Backport of #61, #63 and #65 .