GitLab

Fixes #8442
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>

* Add names and email addresses to MAINTAINERS.md

This is so people can more easily reach the right people (e.g. for security
reports), and because we do it like this in other repos.

I did add parentheses to the formatting (different than in other
MAINTAINERS.md) to make it more readable, this this repo is more complex.
Signed-off-by: Julius Volz <julius.volz@gmail.com>

* Update Björn's email address
Signed-off-by: Julius Volz <julius.volz@gmail.com>

Update web UI package.json

Update several NPM version pinnings.
Signed-off-by: Ben Kochie <superq@gmail.com>

Update vendoring for 2.25 release

Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

Remove the inline Go vendoring per Prometheus 2020 Dev Summit[0].

[0]: https://docs.google.com/document/d/1yuaPKLDvhJNXMF1ubsOOm5kE2_6dvCxHowBQIDs0KdU/edit

Signed-off-by: Ben Kochie <superq@gmail.com>
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

* Consider status code 429 as recoverable errors to avoid resharding.
* Adds support for Retry-After in backoff logic in remote storage.
Signed-off-by: Harkishen-Singh <harkishensingh@hotmail.com>

Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

* Increase golint timeout 1m -> 2m
Signed-off-by: w1mvy <w1mvypine@gmail.com>

[fix] remote_storage :  rename storage/remote/{write_hander.go => write_handler.go} 

Signed-off-by: fuling <fuling.lgz@alibaba-inc.com>

Add optional name property to testgroup for better test failure output

* Accept matchers in querier LabelValues()
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* create matcher to only select metrics which have searched label
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* test case for merge querier with matchers
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* test LabelValues with matchers on head
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* add test for LabelValues on block
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* formatting fix
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Add comments
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* add missing lock release
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* remove unused parameter
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Benchmarks for LabelValues() methods on block/head
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Better comment
Co-authored-by: Julien Pivotto <roidelapluie@gmail.com>
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* update comment
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* minor refactor make code cleaner
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* better comments
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* fix expected errors in test
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Deleting parameter which can only be empty
Co-authored-by: Ganesh Vernekar <15064823+codesome@users.noreply.github.com>
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* fix comments
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* remove unnecessary lock
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* only lookup label value if label name was looked up
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Return error when there is one
Co-authored-by: Ganesh Vernekar <15064823+codesome@users.noreply.github.com>
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Call .Get() on decoder before checking errors
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* only lock head.symMtx when necessary
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* remove unnecessary delete()
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* re-use code instead of duplicating it
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Consistently return error from LabelValueFor()
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* move helper func from util.go to querier.go
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* Fix test expectation
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>
Co-authored-by: Ganesh Vernekar <15064823+codesome@users.noreply.github.com>

* ensure result de-duplication and sorting works
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>

* return named error from LabelValueFor()
Signed-off-by: Mauro Stettler <mauro.stettler@gmail.com>
Co-authored-by: Julien Pivotto <roidelapluie@gmail.com>
Co-authored-by: Ganesh Vernekar <15064823+codesome@users.noreply.github.com>

* Add start() and end() pre-processors for @ modifier
Signed-off-by: Ganesh Vernekar <cs15btech11018@iith.ac.in>

* Fix reviews
Signed-off-by: Ganesh Vernekar <cs15btech11018@iith.ac.in>

* Fix review comments
Signed-off-by: Ganesh Vernekar <cs15btech11018@iith.ac.in>

* Fix review comments
Signed-off-by: Ganesh Vernekar <cs15btech11018@iith.ac.in>

Signed-off-by: misha <DL-OTTCloudPlatform-Nova@bskyb.internal>

* Combine NewHead() args into a HeadOptions struct
Signed-off-by: Dustin Hooten <dustinhooten@gmail.com>

* remove overrides params
Signed-off-by: Dustin Hooten <dustinhooten@gmail.com>

* address pr feedback
Signed-off-by: Dustin Hooten <dustinhooten@gmail.com>

Fix label name leak into class name

[FEATURE] Accept remote_write requests in the Prometheus server

Similar to https://github.com/prometheus/prometheus/pull/7902

, this could lead
to style bugs for label names that correspond to styled CSS class names.
Signed-off-by: Julius Volz <julius.volz@gmail.com>

* [UI] Add a test for duplicated keys in EndpointLink component

I've noticed that I'm getting warnings about multiple children with the same key on /targets page.
This adds a test that fails when that happens.
Signed-off-by: Łukasz Mierzwa <l.mierzwa@gmail.com>

* [UI] Fix duplicated keys on /targets page

Since any URI we render on /targets page can have multi-value params we should use both name and value as components keys.
Signed-off-by: Łukasz Mierzwa <l.mierzwa@gmail.com>

- Remove unrelated changes
- Refactor code out of the API module - that is already getting pretty crowded.
- Don't track reference for AddFast in remote write.  This has the potential to consume unlimited server-side memory if a malicious client pushes a different label set for every series.  For now, its easier and safer to always use the 'slow' path.
- Return 400 on out of order samples.
- Use remote.DecodeWriteRequest in the remote write adapters.
- Put this behing the 'remote-write-server' feature flag
- Add some (very) basic docs.
- Used named return & add test for commit error propagation
Signed-off-by: Tom Wilkie <tom.wilkie@gmail.com>

Update github.com/gogo/protobuf to v1.3.2

Fix for CVE-2021-3121
Signed-off-by: Simon Pasquier <spasquie@redhat.com>

Add fuzzing badge in readme

Fix CI (Windows Testing)

Signed-off-by: mikeee <hey@mike.ee>

Signed-off-by: mikeee <hey@mike.ee>

Signed-off-by: mikeee <hey@mike.ee>

From https://google.github.io/oss-fuzz/getting-started/new-project-guide/

Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

Signed-off-by: fuling <fuling.lgz@alibaba-inc.com>

Windows CI: downgrade golang

Latest releases has a change in behaviour or a bug:

bash: go: command not found
bash: line 1: go: command not found
bash: line 1: go: command not found
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

* API: Fix global URL when external address has no port
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

* Update test
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

* Handle IPv6 correctly.
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>

Update golangci-lint to 1.36.0

[remote storage] remove sendWriteRequestWithBackoff() "s" and "req" param

* remote_write: allow passing along custom HTTP headers
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>

* add warning
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>

* remote_write: add header valadtion
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>

* extend tests for bad remote write headers
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>

* remote_write: add note about the authorization header
Signed-off-by: Nandor Kracser <bonifaido@gmail.com>

In the previous version, 1.18.0, the "megacheck" linter paid attention
to the '//lint:ignore' comment, but that is no longer there.

Newer version pay attention to '//nolint:<linter>,<linter>,...'
comments, optionally followed by a "second" comment introduced by '//'.

Update the directives to use this style.

This is related to prometheus/blackbox_exporter#738 and
prometheus/blackbox_exporter#745.
Signed-off-by: Marcelo E. Magallon <marcelo.magallon@grafana.com>

Signed-off-by: fuling <fuling.lgz@alibaba-inc.com>