6 Commits

Author SHA1 Message Date
1e8d8a93d9 feat(query-tee): improvements to skip samples outside comparison window (#15794) 2025-01-22 10:47:09 +05:30
b5462b6639 chore: remove copied loop vars (#14383) 2024-10-04 10:38:36 -06:00
18cef217fc chore: Linting updates in prep for Go 1.23 (#14071) 2024-09-06 13:12:14 -04:00
364226108a chore: fix flaky querytee proxy test (#10367)
**What this PR does / why we need it**:
`Test_ProxyEndpoint_SummaryMetrics` does not wait for `compareResponses`
call to finish which causes the test to fail as the expected metric
response does not match

this pr updates the test to wait for `responses_compared_total` to be
set to 1 before asserting `missingMetrics`

**Which issue(s) this PR fixes**:
Fixes #<issue number>

**Special notes for your reviewer**:

**Checklist**
- [X] Reviewed the
[`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md)
guide (**required**)
- [ ] Documentation added
- [x] Tests updated
- [ ] `CHANGELOG.md` updated
- [ ] If the change is worth mentioning in the release notes, add
`add-to-release-notes` label
- [ ] Changes that require user attention or interaction to upgrade are
documented in `docs/sources/setup/upgrade/_index.md`
- [ ] For Helm chart changes bump the Helm chart version in
`production/helm/loki/Chart.yaml` and update
`production/helm/loki/CHANGELOG.md` and
`production/helm/loki/README.md`. [Example
PR](d10549e3ec)
2023-08-28 15:33:02 +05:30
5ba154a6de Report missing metrics in querytee. (#10304)
**What this PR does / why we need it**:
Sometimes we want to not just fail a querytee but also report results on
the comparison between expected and actual results. That's why querytee
is extended with special metrics to give more insights into the
differences.

**Checklist**
- [x] Reviewed the
[`CONTRIBUTING.md`](https://github.com/grafana/loki/blob/main/CONTRIBUTING.md)
guide (**required**)
- [ ] Documentation added
- [x] Tests updated
- [ ] `CHANGELOG.md` updated
- [ ] If the change is worth mentioning in the release notes, add
`add-to-release-notes` label
- [ ] Changes that require user attention or interaction to upgrade are
documented in `docs/sources/setup/upgrade/_index.md`
- [ ] For Helm chart changes bump the Helm chart version in
`production/helm/loki/Chart.yaml` and update
`production/helm/loki/CHANGELOG.md` and
`production/helm/loki/README.md`. [Example
PR](d10549e3ec)
2023-08-24 12:59:47 +02:00
66bf9f6474 Copy cortex/tools/querytee package dependency into Loki (#5165)
* Fork Cortex `tools/querytee`.

* Use forked querytee instead of Cortex one.

* Add `loki/tools` to list of possible paths.

* Add opts to querytee proxy.
2022-01-19 14:56:48 +05:30