Skip to content

Conversation

@DataBoyTX
Copy link
Contributor

@DataBoyTX DataBoyTX commented May 22, 2024

I copied this from Percy's wiki page, removing any internal references to servers, dev envs, etc.

@DataBoyTX DataBoyTX requested review from aucahuasi and lmeyerov May 22, 2024 21:47
Copy link
Contributor

@aucahuasi aucahuasi left a comment

Choose a reason for hiding this comment

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

I think we can rename the doc to telemetry.md instead of Open-Telemetry.

@lmeyerov
Copy link
Contributor

  • i renamed to telemetry.md
  • I made a quick pass on text

@aucahuasi :

  1. I thought commands need to run via ./release ... to work the new services, not docker compsoe

  2. Maybe we need to experiment & chat (so after this PR), but to turn on the local UIs, we should show how to setup Caddy for limiting access to admins, and pointers to how to look at logs vs metrics vs traces. Likely can enlist @dess890 for writing up the latter once we've got smooth?

@lmeyerov
Copy link
Contributor

fixed, merging

@aucahuasi next time feel free to edit directly in cases like this , you can hit . in github and it'll go into edit mode

@lmeyerov lmeyerov merged commit 48397c6 into master May 23, 2024
@lmeyerov lmeyerov deleted the tcook-open-telemetry branch May 23, 2024 04:48
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.

4 participants