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

quickstarts: add C++ metrics #401

Merged
merged 1 commit into from
Oct 22, 2018
Merged

quickstarts: add C++ metrics #401

merged 1 commit into from
Oct 22, 2018

Conversation

odeke-em
Copy link
Member

@odeke-em odeke-em commented Oct 20, 2018

Added a full quickstart tutorial for C++
metrics with the Prometheus exporter.

Updates #287
Updates #343

@g-easy
Copy link
Contributor

g-easy commented Oct 22, 2018

Is there a way to see what this looks like rendered?

Copy link
Contributor

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

LGTM

@songy23
Copy link
Contributor

songy23 commented Oct 22, 2018

@g-easy The new web page can be built with the commands here: https://github.com/census-instrumentation/opencensus-website#contributing.

Screenshots:
cpp-quickstart-1
cpp-quickstart-2

@odeke-em
Copy link
Member Author

@g-easy in regards to

Is there a way to see what this looks like rendered?

I provided you with a URL of a tunnelled and rendered website offline :)

Thank you @helly25 and @songy23 for the review! @g-easy am going to merge this for now to get it off my plate but also to ensure that you can view it rendered on the website. Please feel free to send corrections and updates.

Added a full quickstart tutorial for C++
metrics with the Prometheus exporter and
also updates the C++ tracing quickstart
with some grammatical fixes and added
references.

Updates #287
Updates #343
@odeke-em odeke-em force-pushed the c++-quickstart-metrics branch from 750cf53 to 3d12272 Compare October 22, 2018 17:13
@odeke-em odeke-em merged commit 3d12272 into master Oct 22, 2018
@odeke-em odeke-em deleted the c++-quickstart-metrics branch October 22, 2018 20:14
@g-easy
Copy link
Contributor

g-easy commented Oct 23, 2018

I'm sorry I ran out of time for this yesterday. I'm reviewing the live version now.

@odeke-em
Copy link
Member Author

Not a problem @g-easy, thanks for taking a look at it!

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.

5 participants