Fix explicit bucket histogram aggregation by ocelotl · Pull Request #3429 · open-telemetry/opentelemetry-python
Navigation Menu
{{ message }}
open-telemetry / opentelemetry-python Public
- Notifications You must be signed in to change notification settings
- Fork 836
Merged
Fix explicit bucket histogram aggregation#3429
ocelotl merged 11 commits intoopen-telemetry:mainfrom
Fix explicit bucket histogram aggregation#3429
ocelotl merged 11 commits intoopen-telemetry:mainfrom
Conversation
Copy link Copy Markdown
Contributor
ocelotl
added
the
metrics
label
ocelotl
requested a review
from a team
ocelotl
self-assigned this
ocelotl
force-pushed
the
issue_3407
branch
2 times, most recently
from
2ca4a99 to
c9fa0ce
Compare
ocelotl
force-pushed
the
issue_3407
branch
from
3cb8351 to
5fbdfcb
Compare
ocelotl
force-pushed
the
issue_3407
branch
from
5fbdfcb to
3bde9aa
Compare
ocelotl
mentioned this pull request
pmcollins reviewed Jan 30, 2024
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py Show resolved Hide resolved
ocelotl
force-pushed
the
issue_3407
branch
from
3bde9aa to
4046eb4
Compare
ocelotl
requested a review
from pmcollins
pmcollins approved these changes Feb 7, 2024
Copy link Copy Markdown
Member
pmcollins
left a comment
pmcollins
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking care of this. Added some non-blocking comments.
opentelemetry-sdk/tests/metrics/integration_test/test_explicit_bucket_histogram_aggregation.py
Outdated
Show resolved
Hide resolved
opentelemetry-sdk/tests/metrics/integration_test/test_explicit_bucket_histogram_aggregation.py Outdated Show resolved Hide resolved
opentelemetry-sdk/tests/metrics/integration_test/test_explicit_bucket_histogram_aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/tests/metrics/integration_test/test_explicit_bucket_histogram_aggregation.py Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py Show resolved Hide resolved
ocelotl
enabled auto-merge (squash)
ocelotl added 11 commits
February 7, 2024 17:38This reverts commit f1c6683.
ocelotl
force-pushed
the
issue_3407
branch
from
344fd10 to
0166a5e
Compare
ocelotl
merged commit
941e355
into
open-telemetry:main
This was referenced
Feb 10, 2024
colincadams
mentioned this pull request
ocelotl
mentioned this pull request
ocelotl
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment