-
Notifications
You must be signed in to change notification settings - Fork 201
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
Is there a way to sample overrides Servicebus requests? #3601
Comments
@Charlie441324219 thanks for reaching out. can you pls share your json config regarding sampling overrides? this is how you can find out servicebus request's attributes. |
Hi @heyams , thank you for your reply, this is my config to sample out servicebus request
|
@Charlie441324219 can you share your full applicaitoninsights.json config and your applicationinsights.log at debug logging level? we ga sampling overrides since 3.5.0 GA. our azure doc got updated. I want to make sure you use the correct config for 3.4.17. please try out 3.5.1 GA. |
Sure @heyams , this is my full config
|
@Charlie441324219 applicationinsights.log please, you may share a link with me somewhere? your config looks good to me though. |
Hi @heyams , I am afraid I cannot share that due to the company policy. could you please provide more details about how to setup the |
from exporting span I can see
so I tried with
but still not work |
i don't find attributes though. it's hard for me to help you further if you don't share your log with me. you can redact any sensitive information in the log and then share. |
Thank you @heyams , I will raise a ticket to MSFT via company. |
@Charlie441324219 hey Charlie, there was an upstream fix for azure-core 1.36. can you confirm if you're using azure-core 1.36+? |
@Charlie441324219 If you update ServiceBus version, you would be to disable distributed tracing on the client instance using the following code. TracingOptions tracingOptions = new TracingOptions().setEnabled(false);
ClientOptions clientOptions = new AmqpClientOptions().setTracingOptions(tracingOptions);
return new ServiceBusClientBuilder()
.clientOptions(clientOptions ) Without updating, I think the best option would be to disable all Azure SDK instrumentation including ServiceBus - you can do it via the config or env var - https://learn.microsoft.com/en-us/azure/azure-monitor/app/java-standalone-config#suppress-specific-autocollected-telemetry. |
Hi @lmolkova , Thank you for your info, after disabling all Azure SDK instrumentation, I cannot see servicebus request anymore. But is there any other impact for disabling Azure SDK instrumentation ? Is that possible that I lost some important log ? |
The Application Insights config (or environment variable) and the flag on tracing options only affect distributed tracing and not logs. You'll still get logs based on your logging provider configuration. |
What about exceptions from Azure SDK ? If we lost exception logs, we will have big trouble. |
Exceptions come from logs (and then are mapped to exceptions or traces with error severity if there was no exception associated with an error) and not as requests or dependencies, so this setting should not affect them. |
Thank you @lmolkova , I will go ahead with disabling all Azure SDK instrumentation |
@Charlie441324219 thanks! pls let me know if i can close this issue now. |
Is your feature request related to a problem? Please describe.
After migrating from v2 to v3, we received a huge amount of requests from Servicebus. I've tried to do sample override for Servicebus requests but I wasn't able to do so. I can't find any documentation example allowing me to do so.
Describe the solution you would like
Users would be able to do sample overrides for Servicebus requests.
Describe alternatives you have considered
NA
Additional context
I'm using AppInsights Java 3.4.17. servicebus 7.10.1
The text was updated successfully, but these errors were encountered: