Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge prometheus output #4287

Draft
wants to merge 187 commits into
base: master
Choose a base branch
from
Draft

Merge prometheus output #4287

wants to merge 187 commits into from

Conversation

codebien
Copy link
Contributor

What?

Why?

Checklist

  • I have performed a self-review of my code.
  • I have added tests for my changes.
  • I have run linter locally (make lint) and all checks pass.
  • I have run tests locally (make tests) and all tests pass.
  • I have commented on my code, particularly in hard-to-understand areas.

Related PR(s)/Issue(s)

yorugac and others added 30 commits September 13, 2021 14:47
* Add http headers support for write api.

Signed-off-by: johncming <[email protected]>

* Add http headers support for write api.

* Add http headers support for write api.
Changes bring Dockerfile more in line with that of k6 itself.
Add support for keepUrlTag
codebien and others added 27 commits September 25, 2023 14:57
A new format for passing custom HTTP headers using the same syntax as used by InfluxDB output in k6 core.
It adds options and the eventual usage for client certificates. It is useful for supporting the mTLS use case.
Check the following docs for the details:
- https://prometheus.io/docs/guides/tls-encryption
- https://www.cloudflare.com/learning/access-management/what-is-mutual-tls

---------

Co-authored-by: Raoel Oomen <[email protected]>
* Update Dashboards and Docker Compose setup

* Fix `Peak RPS query` for Dashboard without NH

* `k6 Prometheus` Dashboard: format JSON, update Stat Query option

* Dashboard NH: remove ID for local Prometheus DS

* Dashboard NH: format JSON content

* Dashboards: add Doc link and update variable descriptions

* Update the `description` of the `TestID` variable

* Docker compose: use one network

* Dashboards: refresh dynamic variables on time-range change

* Dashboards: show `HTTP` panels before `Checks`

* Dashboard: format JSON

* Dashboards: remove AnalyzeMetric panel and dynamic variable

* Dashboards: open rows

* Dashboards: Rename `Quantile` variable and add informative panel at the bottom

* k6 Prometheus: change variable regex for Trend Query

* Dashboard: use same color for `http_req_duration`. Do not fill graphs
When trying to build for 32-bit ARM (linux/arm), you get:

```
pkg/remote/client.go:120:35: cannot use 0xffffffff (untyped int constant 4294967295) as int value in argument to fmt.Errorf (overflows)
```

Pass the constant as unsigned 64-bit integer, which is what the
underlying snappy code is doing (it's casting the argument to an uint64
and comparing against 0xffffffff, which in Go means it's comparing
against uint64(0xffffffff)). Since this is just an error message, it
shouldn't matter, but let's be consistent.

---------

Signed-off-by: Marcelo E. Magallon <[email protected]>
Co-authored-by: codebien <[email protected]>
…rformance Overview (#165)

* Shared tooltip and Peak RPS calculation

* fixes

* fixes

* fixes

* Remove ID

* fix
We are moving to a fork of goja under grafana org called sobek.

More info in:
- #3772
- #3773
* feat: support sigv4 signing

---------

Signed-off-by: obanby <[email protected]>
…d-metrics-quantile-mismatch-lastnotnull

Fix K6 Prometheus Dashboard Metrics Quantile Mismatch by Changing Aggregation to lastNotNull
…daf0309a0a249514caca'

git-subtree-dir: internal/output/prometheusrw
git-subtree-mainline: cfed5fc
git-subtree-split: c40885f
@codebien codebien self-assigned this Jan 27, 2025
@CLAassistant
Copy link

CLAassistant commented Jan 27, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
14 out of 15 committers have signed the CLA.

✅ jwcastillo
✅ javaducky
✅ codebien
✅ pkwarren
✅ olegbespalov
✅ arukiidou
✅ mstoykov
✅ mem
✅ dylanjpaulson
✅ brunocascio
✅ Sea-you
✅ ppcano
✅ obanby
✅ 7olstoy
❌ Dmitry Tolstoy


Dmitry Tolstoy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.