Add default aggregation by ocelotl · Pull Request #2543 · 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
Conversation
ocelotl
self-assigned this
ocelotl
requested a review
from a team
srikanthccv reviewed Mar 25, 2022
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py Show resolved Hide resolved
lzchen approved these changes Mar 25, 2022
Copy link Copy Markdown
Contributor
lzchen
left a comment
lzchen
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CHANGELOG?
ocelotl
force-pushed
the
issue_2537
branch
from
489d34e to
f4e66fb
Compare
Copy link Copy Markdown
Contributor Author
ocelotl
commented
Mar 25, 2022
ocelotl commented
Mar 25, 2022CHANGELOG?
Added changelog entry
ocelotl
requested a review
from srikanthccv
srikanthccv approved these changes Mar 26, 2022
ocelotl
force-pushed
the
issue_2537
branch
from
c19042d to
9cfe02a
Compare
aabmass approved these changes Mar 28, 2022
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py
Outdated
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py Outdated Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py
Outdated
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/aggregation.py Outdated Show resolved Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/view.py
Outdated
Show resolved
Hide resolved
opentelemetry-sdk/src/opentelemetry/sdk/_metrics/view.py Outdated Show resolved Hide resolved
ocelotl added 5 commits
March 28, 2022 16:24
ocelotl
force-pushed
the
issue_2537
branch
from
4745ccf to
8e475ff
Compare
ocelotl
merged commit
9087d23
into
open-telemetry:main
ocelotl
deleted the
issue_2537
branch
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