Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Fix error in opencensus to stackdriver translation (#474) #475

Merged
merged 1 commit into from
Apr 5, 2019

Conversation

jinmel
Copy link
Contributor

@jinmel jinmel commented Apr 5, 2019

  • compressedSize -> compressedSizeBytes
  • uncompressedSize -> uncompressedSizeBytes

I've told my boss that last bug fix and publishing 0.0.10 milestone will be the cure for failing opencensus instrumentation last week and it seems like I'd need to wait more

going for a beer cheers!

…ation#474)

  - compressedSize -> compressedSizeBytes
  - uncompressedSize -> uncompressedSizeBytes
@codecov-io
Copy link

Codecov Report

Merging #475 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #475      +/-   ##
=========================================
+ Coverage   94.79%   94.8%   +0.01%     
=========================================
  Files         150     150              
  Lines        9875    9876       +1     
  Branches      742     742              
=========================================
+ Hits         9361    9363       +2     
+ Misses        514     513       -1
Impacted Files Coverage Δ
src/zpages-frontend/latency-bucket-boundaries.ts 71.64% <0%> (-2.99%) ⬇️
src/http-stats.ts 100% <0%> (ø) ⬆️
test/test-stackdriver-cloudtrace-utils.ts 100% <0%> (ø) ⬆️
src/stackdriver-monitoring.ts 82.02% <0%> (+3.37%) ⬆️

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 9632157...8d48b77. Read the comment docs.

Copy link
Member

@mayurkale22 mayurkale22 left a comment

Choose a reason for hiding this comment

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

I have verified the change, thanks for the PR.

@mayurkale22
Copy link
Member

I've told my boss that last bug fix and publishing 0.0.10 milestone will be the cure for failing opencensus instrumentation last week and it seems like I'd need to wait more

I'll help you to cut the release soon (might be on monday).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants