Skip to content

Commit

Permalink
fix: Disable telemetry adaptive sampling (#6849)
Browse files Browse the repository at this point in the history
* disable adaptive sampling

* using Microsoft.ApplicationInsights.AspNetCore.Extensions
  • Loading branch information
JhontSouth authored Sep 17, 2024
1 parent c11b7fd commit 6d584df
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -209,6 +209,7 @@ internal static void AddBotRuntimeTelemetry(this IServiceCollection services, IC
}
else
{
telemetrySettings.Options.EnableAdaptiveSampling = false;
services.AddApplicationInsightsTelemetry(telemetrySettings.Options);
services.TryAddSingleton<IBotTelemetryClient, BotTelemetryClient>();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
using System;
using System.Linq;
using Microsoft.ApplicationInsights;
using Microsoft.ApplicationInsights.AspNetCore.Extensions;
using Microsoft.ApplicationInsights.Extensibility;
using Microsoft.Bot.Builder.ApplicationInsights;
using Microsoft.Bot.Configuration;
Expand Down Expand Up @@ -57,7 +58,14 @@ public static IServiceCollection AddBotApplicationInsights(this IServiceCollecti
IBotTelemetryClient telemetryClient = null;
if (appInsightService != null)
{
services.AddApplicationInsightsTelemetry(appInsightService.InstrumentationKey);
// Create Application Insights options and disable adaptive sampling
var aiOptions = new ApplicationInsightsServiceOptions
{
EnableAdaptiveSampling = false,
InstrumentationKey = appInsightService.InstrumentationKey
};

services.AddApplicationInsightsTelemetry(aiOptions);
telemetryClient = new BotTelemetryClient(new TelemetryClient());
}
else
Expand Down Expand Up @@ -88,9 +96,19 @@ public static IServiceCollection AddBotApplicationInsights(this IServiceCollecti
CreateBotTelemetry(services);

IBotTelemetryClient telemetryClient = null;

if (!string.IsNullOrWhiteSpace(instrumentationKey))
{
services.AddApplicationInsightsTelemetry(instrumentationKey);
// Create Application Insights options and disable adaptive sampling
var aiOptions = new ApplicationInsightsServiceOptions
{
EnableAdaptiveSampling = false,
InstrumentationKey = instrumentationKey
};

// Add Application Insights telemetry with custom options
services.AddApplicationInsightsTelemetry(aiOptions);

telemetryClient = new BotTelemetryClient(new TelemetryClient());
}
else
Expand Down Expand Up @@ -122,7 +140,14 @@ public static IServiceCollection AddBotApplicationInsights(this IServiceCollecti
// Start Application Insights
if (instrumentationKey != null)
{
services.AddApplicationInsightsTelemetry(instrumentationKey);
// Create Application Insights options and disable adaptive sampling
var aiOptions = new ApplicationInsightsServiceOptions
{
EnableAdaptiveSampling = false,
InstrumentationKey = instrumentationKey
};

services.AddApplicationInsightsTelemetry(aiOptions);
}

// Register the BotTelemetryClient
Expand Down

0 comments on commit 6d584df

Please sign in to comment.