-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: Add {otel}
support
#4269
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add {otel}
support
#4269
Conversation
Introduces bindOtel() and withOtelShiny() to enable OpenTelemetry tracing for reactive objects and observers. Adds options shiny.otel.bindall and shiny.otel.graphlocked for automatic binding and graph locking. Updates reactiveVal, reactiveValues, reactive, and observe to support automatic OpenTelemetry binding. Documentation and NAMESPACE updated accordingly.
Included r-lib/otelsdk in the Remotes field to enable installation from GitHub. This supports dependencies required for development or testing.
bindOtel()
bindOtel()
support
This comment was marked as outdated.
This comment was marked as outdated.
* main: preemptively run revdepcheck. 5 failures. fix(scss): Remove updated scss files and restore R sass compilation (#4272)
…cent one is evaluated last
…ession_start` span
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...after few remaining nits
@@ -0,0 +1,469 @@ | |||
# Personal debugging function ------------------------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove before merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am going to merge with in it. And perform testing in a followup PR.
bindOtel()
support{otel}
support
…ls. Only relevant calculations
Fixes: #4258
Not implementing #4265 -
bindOtel(x)
,withOtel(expr, ..., bind = "all")
Fixes #4268 -
options(shiny.otel.bind = "all")
(and"none"
)Fixes #4267 - ExtendedTask
Fixes #4282 -
options(shiny.otel.bind = "session")
Fixes #4266 -
options(shiny.otel.bind = "reactive_update")
Fixes #4268 -
options(shiny.otel.bind = "reactivity")
/options(shiny.otel.bind = "all")
News entry: