Skip to content

Update dskit to allow tracing of HTTP headers #11655

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

Merged
merged 9 commits into from
Jun 9, 2025

Conversation

colega
Copy link
Contributor

@colega colega commented Jun 6, 2025

What this PR does

Updates dskit to latest version and documents the new flag & jsonnet config to enable proper tracing of HTTP headers.

Which issue(s) this PR fixes or relates to

Ref: #2708
Ref: grafana/dskit#709

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

colega added 3 commits June 6, 2025 09:52
Updates dskit to latest version and documents the new flag & jsonnet config to enable proper tracing of HTTP headers.

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Copy link
Contributor

github-actions bot commented Jun 6, 2025

💻 Deploy preview deleted.

colega added 4 commits June 6, 2025 09:58
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega colega changed the title Update dskit to allow tracing of HTTP headers. Update dskit to allow tracing of HTTP headers Jun 6, 2025
@colega colega marked this pull request as ready for review June 6, 2025 15:01
@colega colega requested review from stevesg, a team and tacole02 as code owners June 6, 2025 15:01
…-tracing-of-http-headers-

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega colega force-pushed the update-dskit-to-allow-tracing-of-http-headers- branch from 11637e0 to 82e68ac Compare June 6, 2025 15:02
@colega colega enabled auto-merge (squash) June 6, 2025 15:25
…-tracing-of-http-headers-

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@colega colega merged commit 86dba05 into main Jun 9, 2025
32 checks passed
@colega colega deleted the update-dskit-to-allow-tracing-of-http-headers- branch June 9, 2025 08:41
mimir-github-bot bot pushed a commit that referenced this pull request Jun 9, 2025
* Update dskit to allow tracing of HTTP headers.

Updates dskit to latest version and documents the new flag & jsonnet config to enable proper tracing of HTTP headers.

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* Remove dashes from jsonnet flags

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* make build-jsonnet-tests

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* Update CHANGELOG.md PR number

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* Update dskit to correct version and update docs

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* make reference-help

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

* Disable in jsonnet by default

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>

---------

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
(cherry picked from commit 86dba05)
dimitarvdimitrov pushed a commit that referenced this pull request Jun 9, 2025
* Update dskit to allow tracing of HTTP headers.

Updates dskit to latest version and documents the new flag & jsonnet config to enable proper tracing of HTTP headers.



* Remove dashes from jsonnet flags



* make build-jsonnet-tests



* Update CHANGELOG.md PR number



* Update dskit to correct version and update docs



* make reference-help



* Disable in jsonnet by default



---------


(cherry picked from commit 86dba05)

Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Co-authored-by: Oleg Zaytsev <mail@olegzaytsev.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants