Skip to content

feat(ci_visibility): add rq_compressed and rs_compressed tags to request metrics#17699

Draft
gnufede wants to merge 1 commit intomainfrom
telemetry/compression-tags
Draft

feat(ci_visibility): add rq_compressed and rs_compressed tags to request metrics#17699
gnufede wants to merge 1 commit intomainfrom
telemetry/compression-tags

Conversation

@gnufede
Copy link
Copy Markdown
Member

@gnufede gnufede commented Apr 23, 2026

Summary

  • Adds rq_compressed and rs_compressed boolean tags to the request count metric
  • Tracks whether HTTP request/response bodies were gzip-compressed
  • Request compression state flows from BackendConnector._do_single_request through BackendResult.is_gzip_request to telemetry

Test plan

  • Unit test for both compression tags present
  • Unit test for request-only compression
  • Unit test for response-only compression
  • Integration tests for gzip request detection in BackendConnector
  • All existing test_http.py assertions updated with compressed_request= parameter

Motivation

Per CI Visibility telemetry RFC: since gzip support was added for backend requests/responses, we need tags to distinguish compressed vs uncompressed payloads on metrics like endpoint_payload.requests, git_requests.search_commits, etc.

🤖 Generated with Claude Code

…est metrics

Tracks whether HTTP request and response bodies were gzip-compressed by
adding rq_compressed and rs_compressed tags to the request count metric.
The request compression state flows from BackendConnector through
BackendResult to the telemetry recording layer.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gnufede gnufede requested a review from a team as a code owner April 23, 2026 12:00
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

Codeowners resolved as

ddtrace/testing/internal/http.py                                        @DataDog/ci-app-libraries
ddtrace/testing/internal/telemetry.py                                   @DataDog/ci-app-libraries
tests/testing/internal/test_http.py                                     @DataDog/ci-app-libraries
tests/testing/internal/test_telemetry.py                                @DataDog/ci-app-libraries

@gnufede gnufede marked this pull request as draft April 23, 2026 12:02
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.

1 participant