Skip to content

fix: compress OTLP profile uploads - #352

Merged
korniltsev-grafanista merged 1 commit into
grafana:mainfrom
arnabnandy7:fix/otlp-upload-compression
Aug 18, 2026
Merged

fix: compress OTLP profile uploads#352
korniltsev-grafanista merged 1 commit into
grafana:mainfrom
arnabnandy7:fix/otlp-upload-compression

Conversation

@arnabnandy7

Copy link
Copy Markdown
Contributor

Summary

Fixes #346 by compressing OTLP profile uploads with gzip.

  • Reuses the existing compressionLevelJFR configuration.
  • Adds Content-Encoding: gzip when compression is enabled.
  • Preserves application/x-protobuf as the content type.
  • Keeps NO_COMPRESSION behavior unchanged.
  • Updates the exporter test to verify the header and decompressed payload.

Testing

./gradlew :agent:test --tests io.pyroscope.javaagent.impl.PyroscopeExporterTest --rerun-tasks

Result: BUILD SUCCESSFUL

Signed-off-by: Arnab Nandy <arnab_nandy7@yahoo.com>
@arnabnandy7
arnabnandy7 requested review from a team as code owners August 16, 2026 12:39
@arnabnandy7

Copy link
Copy Markdown
Contributor Author

@korniltsev-grafanista @korniltsev to review

@arnabnandy7

Copy link
Copy Markdown
Contributor Author

@korniltsev-grafanista @korniltsev please re-run the failed tests, and there are 3 other reviews need your attention. let me know if you can share your inputs there as well. Thanks again for your review and approval.

@korniltsev-grafanista
korniltsev-grafanista merged commit b9df4dd into grafana:main Aug 18, 2026
93 of 229 checks passed
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.

OTLP profile uploads are not compressed

2 participants