Skip to content

Conversation

@cratelyn
Copy link
Member

@cratelyn cratelyn commented Sep 16, 2025

#3308 introduced tower
middleware to the outbound proxy that records the size and number of
response body's data frames.

this commit introduces an equivalent layer to the inbound proxy's
prometheus metrics.

this is based on #4166.

@cratelyn cratelyn self-assigned this Sep 16, 2025
@cratelyn cratelyn force-pushed the kate/app-inbound.metrics-layer-add-response-body-data branch 2 times, most recently from 4394a72 to 9b37841 Compare September 16, 2025 15:08
cratelyn added a commit that referenced this pull request Sep 16, 2025
<#3308> introduced tower
middleware to the outbound proxy that records the size and number of
response body's data frames.

this commit introduces an equivalent layer to the inbound proxy's
prometheus metrics.

NB: this is based upon
<#4165>

Signed-off-by: katelyn martin <[email protected]>
@cratelyn cratelyn force-pushed the kate/app-inbound.metrics-layer-add-response-body-data branch from 9b37841 to d094b1f Compare September 16, 2025 15:29
@cratelyn cratelyn changed the base branch from main to kate/app-inbound.metrics-layer-inbound-metrics-param September 16, 2025 15:29
@cratelyn cratelyn marked this pull request as ready for review September 16, 2025 15:35
@cratelyn cratelyn requested a review from a team as a code owner September 16, 2025 15:35
Base automatically changed from kate/app-inbound.metrics-layer-inbound-metrics-param to main September 16, 2025 18:44
<#3308> introduced tower
middleware to the outbound proxy that records the size and number of
response body's data frames.

this commit introduces an equivalent layer to the inbound proxy's
prometheus metrics.

NB: this is based upon
<#4165>

Signed-off-by: katelyn martin <[email protected]>
@cratelyn cratelyn force-pushed the kate/app-inbound.metrics-layer-add-response-body-data branch from d094b1f to b1cb019 Compare September 16, 2025 19:00
@cratelyn
Copy link
Member Author

force-pushed atop main now that #4166 has merged.

@cratelyn cratelyn enabled auto-merge (squash) September 16, 2025 19:01
@cratelyn cratelyn merged commit 3996fa3 into main Sep 16, 2025
15 checks passed
@cratelyn cratelyn deleted the kate/app-inbound.metrics-layer-add-response-body-data branch September 16, 2025 19:07
cratelyn added a commit that referenced this pull request Sep 22, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4165
* #4166
* #4174
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4165
* #4166
* #4174
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4165
* #4166
* #4174
* #4127
* #4186

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4188
* #4187
* #4186
* #4174
* #4165
* #4166
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4188
* #4187
* #4186
* #4174
* #4165
* #4166
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4188
* #4187
* #4186
* #4174
* #4165
* #4166
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 24, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4188
* #4187
* #4186
* #4174
* #4165
* #4166
* #4127

Signed-off-by: katelyn martin <[email protected]>
cratelyn added a commit that referenced this pull request Sep 26, 2025
this commit introduces an additional layer of telemetry to the inbound
proxy's http router.

either http and grpc metrics are used, depending upon the policy that
authorized a given request.

this is based upon #4174, which refactored the
request body telemetry middleware to be metrics agnostic.

see:
* #4189
* #4188
* #4187
* #4186
* #4174
* #4165
* #4166
* #4127

Signed-off-by: katelyn martin <[email protected]>
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.

3 participants