Skip to content
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

Add configuration for different instrumentation levels #62

Open
caalador opened this issue Aug 26, 2022 · 0 comments
Open

Add configuration for different instrumentation levels #62

caalador opened this issue Aug 26, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@caalador
Copy link
Collaborator

We should have properties for controlling what instumentation is used.

One property to consider could be vaadin.instrumentation.level with the levels of trace, detailed and info where trace would make spans for all the things, detailed for all rpc requests and info more of an overview.

Some true/false properties could be vaadin.instrumentation.request.handler, vaadin.instrumentation.rpc.sync, vaadin.instrumentation.rpc.server.event. Disabling some of these should still not make the traces bad, just collect less details.

All properties added need to add documentation in the PR.

@sissbruecker sissbruecker self-assigned this Aug 29, 2022
caalador pushed a commit that referenced this issue Aug 30, 2022
Allows to configure a global trace level option that instrumentations can check for whether they should add some detail to a trace or not. The option supports three levels `minimum`, `default`, `maximum`. The default is `default`. The option can be changed using any of the configuration mechanism supported by OpenTelemetry, for example environment variables or system properties. For example, to change the trace level with a system property, add the following argument to the java process: `-Dotel.instrumentation.vaadin.trace-level=maximum`.

This PR is only intended to add the basic option mechanism, and adapts the `UidlRequestHandlerInstrumentation` to demonstrate its use. The remaining instrumentations will be changed in a separate PR.

Part of #62
@heruan heruan added the enhancement New feature or request label Feb 21, 2023
@sissbruecker sissbruecker removed their assignment Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants