Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Add property to request/dependency telemetry to indicate standard metrics + remove request metrics from Autocollection #124

Merged
merged 12 commits into from
Oct 2, 2020

Conversation

lzchen
Copy link
Contributor

@lzchen lzchen commented Oct 1, 2020

To indicate to the backend to not automatically generate standard metrics based off of request/dependency telemetry, set a flag in properties of the envelope that we will be doing the aggregation in the SDK.

This PR also removes "request metrics" from AutoCollection since we will be using standard metrics for request related metrics instead.

Part 1 of [#123]

@lzchen lzchen requested a review from hectorhdzg as a code owner October 1, 2020 22:10
@lzchen lzchen changed the title Add property to request/dependency telemetry to indicate standard metrics Add property to request/dependency telemetry to indicate standard metrics + remove request metrics from Autocollection Oct 1, 2020
@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2020

Codecov Report

Merging #124 into master will increase coverage by 1.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   91.88%   93.00%   +1.11%     
==========================================
  Files          21       21              
  Lines        1196     1201       +5     
  Branches      149      149              
==========================================
+ Hits         1099     1117      +18     
+ Misses         70       56      -14     
- Partials       27       28       +1     
Impacted Files Coverage Δ
.../src/azure_monitor/sdk/auto_collection/__init__.py 100.00% <ø> (ø)
..._monitor/sdk/auto_collection/dependency_metrics.py 93.25% <ø> (ø)
...monitor/sdk/auto_collection/performance_metrics.py 100.00% <ø> (ø)
...tor/src/azure_monitor/sdk/auto_collection/utils.py 100.00% <ø> (ø)
...monitor/src/azure_monitor/export/trace/__init__.py 97.97% <100.00%> (+0.17%) ⬆️
...nitor/sdk/auto_collection/live_metrics/__init__.py 100.00% <100.00%> (ø)
...ure_monitor/sdk/auto_collection/request_metrics.py 89.21% <100.00%> (+12.29%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c06ddf1...1f019df. Read the comment docs.

Copy link
Member

@hectorhdzg hectorhdzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lzchen lzchen merged commit 132024c into microsoft:master Oct 2, 2020
@lzchen lzchen deleted the update branch October 2, 2020 18:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants