Skip to content

metrics_service: populate histogram summary sample sum v1.26#32706

Closed
sschepens wants to merge 1 commit intoenvoyproxy:release/v1.26from
sschepens:metrics-service-add-sample-sum-1-26
Closed

metrics_service: populate histogram summary sample sum v1.26#32706
sschepens wants to merge 1 commit intoenvoyproxy:release/v1.26from
sschepens:metrics-service-add-sample-sum-1-26

Conversation

@sschepens
Copy link
Copy Markdown
Contributor

@sschepens sschepens commented Mar 5, 2024

backport of #32666

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Sebastian Schepens <sebastian.schepens@mercadolibre.com>
@sschepens sschepens changed the base branch from main to release/v1.26 March 5, 2024 13:31
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Mar 5, 2024
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @htuch

🐱

Caused by: #32706 was opened by sschepens.

see: more, trace.

@phlax
Copy link
Copy Markdown
Member

phlax commented Mar 5, 2024

@sschepens is this not required? afaict the original change is on the 1.26 branch

@phlax
Copy link
Copy Markdown
Member

phlax commented Mar 5, 2024

nm - i see other pr

@sschepens
Copy link
Copy Markdown
Contributor Author

sschepens commented Mar 5, 2024

@phlax yea, I accidentally opened this against main and pulled in a lot of reviewers, thought just better to open a new clean one.

@sschepens sschepens deleted the metrics-service-add-sample-sum-1-26 branch March 5, 2024 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants