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

Update examples to work with global stats instance #339

Merged
merged 7 commits into from
Feb 12, 2019

Conversation

mayurkale22
Copy link
Member

This PR contains updated examples based on CHAGELOG -> https://github.com/census-instrumentation/opencensus-node/blob/master/CHANGELOG.md#new-code

Also, added ReadMe for the Stackdriver Stats(Metrics) exporter.

@mayurkale22
Copy link
Member Author

mayurkale22 commented Feb 12, 2019

+@DazWilkin for review.

@codecov-io
Copy link

codecov-io commented Feb 12, 2019

Codecov Report

Merging #339 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #339   +/-   ##
=======================================
  Coverage   94.98%   94.98%           
=======================================
  Files         120      120           
  Lines        8332     8332           
  Branches      741      741           
=======================================
  Hits         7914     7914           
  Misses        418      418

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 95be16e...23036b4. Read the comment docs.

export GOOGLE_APPLICATION_CREDENTIALS=path/to/your/credential.json
```

Instance the exporter on your application.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: did you mean "Install the exporter ..."?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copied from top example (Trace Exporter), amended both the sentence.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I did.

@mayurkale22
Copy link
Member Author

+@alexamies for review (especially web_client_monitoring example)

@mayurkale22 mayurkale22 force-pushed the update_example branch 2 times, most recently from 5867924 to 371169a Compare February 12, 2019 07:49
@alexamies
Copy link
Contributor

+@alexamies for review (especially web_client_monitoring example)

Looks good. Thanks for making the changes in the web_client_monitoring example.

@mayurkale22 mayurkale22 merged commit cb97d3f into census-instrumentation:master Feb 12, 2019
@mayurkale22 mayurkale22 deleted the update_example branch February 12, 2019 18:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants