Skip to content

Update instrumentation.md#1046

Merged
beorn7 merged 2 commits intomasterfrom
RichiH-patch-2
Nov 28, 2019
Merged

Update instrumentation.md#1046
beorn7 merged 2 commits intomasterfrom
RichiH-patch-2

Conversation

@RichiH
Copy link
Member

@RichiH RichiH commented Jun 4, 2018

Try to trigger rebuild; might fix #1045

@RichiH
Copy link
Member Author

RichiH commented Jun 4, 2018

Changed the first thing I could find to nitpick.

@RichiH
Copy link
Member Author

RichiH commented Jun 4, 2018

Heh, DCO is an issue with in-web changes.

Try to trigger rebuild; might fix #1045

Signed-off-by: Richard Hartmann <richih@debian.org>

The short answer is to instrument everything. Every library, subsystem and
service should have at least a few metrics to give you a rough idea of how it is
service should expose at least a few metrics to give you a rough idea of how it is
Copy link
Contributor

Choose a reason for hiding this comment

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

Instrumentation and exposition are separate concerns, libraries should not be exposing metrics. They should register them in a registry.

Signed-off-by: Richard Hartmann <richih@debian.org>
@RichiH
Copy link
Member Author

RichiH commented Jun 10, 2018

This has been fixed by our world-visible host.

@RichiH RichiH closed this Jun 10, 2018
@RichiH
Copy link
Member Author

RichiH commented Jun 10, 2018

Erm, wrong PR. Disregard my confusion.

@RichiH RichiH reopened this Jun 10, 2018
aylei pushed a commit to aylei/docs that referenced this pull request Oct 28, 2019
Changed URL structure of how-to/get-started.
Fixed s/sample/example/ bug introduced in DITA
fixed categories
@beorn7
Copy link
Member

beorn7 commented Nov 28, 2019

DCO is fine as per human inspection.

@beorn7 beorn7 merged commit db8964b into master Nov 28, 2019
@beorn7 beorn7 deleted the RichiH-patch-2 branch November 28, 2019 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Instrumentation page is broken

3 participants