Skip to content

Conversation

@toph-allen
Copy link
Collaborator

Instruments the Usage Metrics Dashboard to send OTel signals if its environment is configured appropriately. To learn more about required environment variables, see Collecting Telemetry Data docs for R's otelsdk package.

  • Uses the otel package to manually track spans in certain functions, as well as app initialization.
  • Uses a dev version of Shiny 1.2.0, which supports auto-instrumentation for all reactive operations.
  • Note that sending instrumentation requires the otelsdk R package, which has a dependency on the protobuf binary, so this version of the content won't run on a Connect server without that binary installed.

@jonkeane
Copy link
Collaborator

Note that sending instrumentation requires the otelsdk R package, which has a dependency on the protobuf binary, so this version of the content won't run on a Connect server without that binary installed.

Oh how I wish we had a private / dev feed of our extensions we could use for circumstances like this.

@jonkeane
Copy link
Collaborator

Are you still looking for feedback on this? Are we planning to merge this here into connect-extensions and release it? Or is this something we will try out experimentally and deploy it manually where we need to? Either is fine, but I have very different thoughts about what we might need to do depending on which direction we're taking.

@toph-allen
Copy link
Collaborator Author

Are you still looking for feedback on this? Are we planning to merge this here into connect-extensions and release it? Or is this something we will try out experimentally and deploy it manually where we need to? Either is fine, but I have very different thoughts about what we might need to do depending on which direction we're taking.

For now, I'd like to leave the PR open and use it to investigate app startup times as Connect is updated.

I'm happy to merge it in the future, but first I'd like to see what it's like to use these metrics, and I think that experience (plus other knowledge & experience w/ OTel I'm gaining working on Connect) will tell me a lot about their naming, density, etc.

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.

3 participants