ApplicationInsights-Java icon indicating copy to clipboard operation
ApplicationInsights-Java copied to clipboard

Is there a way to sample overrides Servicebus requests?

Open Charlie441324219 opened this issue 11 months ago • 18 comments

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

Screenshot 2024-03-13 at 11 49 15 AM

Charlie441324219 avatar Mar 13 '24 16:03 Charlie441324219

@Charlie441324219 thanks for reaching out. you should be able to sample out servicebus requests.

can you pls share your json config regarding sampling overrides? this is how you can find out servicebus request's attributes.

image

heyams avatar Mar 13 '24 17:03 heyams

Hi @heyams , thank you for your reply, this is my config to sample out servicebus request

        {
          "telemetryType": "request",
          "attributes": [
            {
              "key": "http.url",
              "value": ".*servicebus.windows.net.*",
              "matchType": "regexp"
            }
          ],
          "percentage": 0
        }

Charlie441324219 avatar Mar 13 '24 17:03 Charlie441324219

@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.

heyams avatar Mar 13 '24 17:03 heyams

Sure @heyams , this is my full config

{
  "sampling": {
    "percentage": 100
  },
  "jmxMetrics": [
    {
      "name": "Heap Memory Usage-used",
      "objectName": "java.lang:type=Memory",
      "attribute": "HeapMemoryUsage.used"
    }
  ],
  "instrumentation": {
    "logging": {
      "level": "ALL"
    }
  },
  "preview": {
    "captureLoggingLevelAsCustomDimension": true,
    "sampling": {
      "overrides": [
        {
          "telemetryType": "dependency",
          "percentage": 0
        },
        {
          "telemetryType": "request",
          "attributes": [
            {
              "key": "http.url",
              "value": ".*servicebus.windows.net.*",
              "matchType": "regexp"
            }
          ],
          "percentage": 0
        }
      ]
    }
  },
  "selfDiagnostics": {
    "destination": "console",
    "level": "DEBUG"
  }
}

Charlie441324219 avatar Mar 13 '24 17:03 Charlie441324219

@Charlie441324219 applicationinsights.log please, you may share a link with me somewhere? your config looks good to me though.

heyams avatar Mar 13 '24 17:03 heyams

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 key and value based on applicationinsights.log ?

Charlie441324219 avatar Mar 13 '24 18:03 Charlie441324219

image

heyams avatar Mar 13 '24 18:03 heyams

from exporting span I can see

attributes=AttributesMap{data={peer.address=stage-membership-credit.servicebus.windows.net, thread.name=boundedElastic-19, message_bus.destination=credit-sync/subscriptions/quickScreenAttachments, applicationinsights.internal.item_count=1, thread.id=130, x-opt-enqueued-time=1710354044, az.namespace=Microsoft.ServiceBus}, capacity=128, totalAddedValues=7}, totalAttributeCount=7, events=[], totalRecordedEvents=0, links=[], totalRecordedLinks=0, status=ImmutableStatusData{statusCode=OK, description=}, hasEnded=true}

so I tried with

        {
          "telemetryType": "request",
          "attributes": [
            {
              "key": "peer.address",
              "value": ".*servicebus.windows.net",
              "matchType": "regexp"
            }
          ],
          "percentage": 0
        }

but still not work

Charlie441324219 avatar Mar 13 '24 18:03 Charlie441324219

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.

heyams avatar Mar 13 '24 18:03 heyams

Thank you @heyams , I will raise a ticket to MSFT via company.

Charlie441324219 avatar Mar 13 '24 18:03 Charlie441324219

@Charlie441324219 hey Charlie, there was an upstream fix for azure-core 1.36. your issue seems related to https://github.com/microsoft/ApplicationInsights-Java/issues/3147.

can you confirm if you're using azure-core 1.36+?

heyams avatar Mar 13 '24 19:03 heyams

@Charlie441324219 I believe you're running very old version of Azure ServiceBus where we used different set of attributes and configuration options. With newer version of servicebus and the agent you'd be able to you'd be able to override sampling settings. It also contains a LOT of perf and reliability improvements.

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.

lmolkova avatar Mar 14 '24 06:03 lmolkova

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 ?

Charlie441324219 avatar Mar 21 '24 15:03 Charlie441324219

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. By disabling Azure SDK instrumentation in Application Insights config you disable all requests and dependencies from all Azure SDKs. By setting the flag on Service Bus client options, you disable one specific instance of this client.

You'll still get logs based on your logging provider configuration.

lmolkova avatar Mar 21 '24 16:03 lmolkova

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. By disabling Azure SDK instrumentation in Application Insights config you disable all requests and dependencies from all Azure SDKs. By setting the flag on Service Bus client options, you disable one specific instance of this client.

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.

Charlie441324219 avatar Mar 22 '24 18:03 Charlie441324219

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.

lmolkova avatar Mar 22 '24 20:03 lmolkova

Thank you @lmolkova , I will go ahead with disabling all Azure SDK instrumentation

Charlie441324219 avatar Mar 26 '24 18:03 Charlie441324219

@Charlie441324219 thanks! pls let me know if i can close this issue now.

heyams avatar Mar 26 '24 18:03 heyams