Skip to content

Commit 1b92dcf

Browse files
committed
wip
1 parent 059eba8 commit 1b92dcf

File tree

6 files changed

+26
-18
lines changed

6 files changed

+26
-18
lines changed

Directory.Packages.props

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
<PackageVersion Include="AWSSDK.SQS" Version="4.0.0.1" />
2626
<PackageVersion Include="AWSSDK.S3" Version="4.0.7.14" />
2727
<PackageVersion Include="Elastic.OpenTelemetry" Version="1.1.0" />
28-
<PackageVersion Include="OpenTelemetry.Instrumentation.AspNetCore" Version="1.12.0" />
28+
<PackageVersion Include="OpenTelemetry.Instrumentation.AspNetCore" Version="1.13.0" />
2929
<PackageVersion Include="KubernetesClient" Version="17.0.14" />
3030
<PackageVersion Include="Elastic.Aspire.Hosting.Elasticsearch" Version="9.3.0" />
3131
<PackageVersion Include="Elastic.Clients.Elasticsearch" Version="9.1.4" />
@@ -76,10 +76,10 @@
7676
<ItemGroup>
7777
<PackageVersion Include="Microsoft.Extensions.Http.Resilience" Version="9.7.0" />
7878
<PackageVersion Include="Microsoft.Extensions.ServiceDiscovery" Version="9.4.0" />
79-
<PackageVersion Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.12.0" />
80-
<PackageVersion Include="OpenTelemetry.Extensions.Hosting" Version="1.12.0" />
81-
<PackageVersion Include="OpenTelemetry.Instrumentation.Http" Version="1.12.0" />
82-
<PackageVersion Include="OpenTelemetry.Instrumentation.Runtime" Version="1.12.0" />
79+
<PackageVersion Include="OpenTelemetry.Exporter.OpenTelemetryProtocol" Version="1.13.1" />
80+
<PackageVersion Include="OpenTelemetry.Extensions.Hosting" Version="1.13.1" />
81+
<PackageVersion Include="OpenTelemetry.Instrumentation.Http" Version="1.13.0" />
82+
<PackageVersion Include="OpenTelemetry.Instrumentation.Runtime" Version="1.13.0" />
8383
</ItemGroup>
8484
<!-- Test packages -->
8585
<ItemGroup>
@@ -99,4 +99,4 @@
9999
</PackageVersion>
100100
<PackageVersion Include="xunit.v3" Version="2.0.2" />
101101
</ItemGroup>
102-
</Project>
102+
</Project>

src/api/Elastic.Documentation.Api.Core/AskAi/AskAiUsecase.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ public async Task<Stream> AskAi(AskAiRequest askAiRequest, Cancel ctx)
2929
};
3030
var inputMessagesJson = JsonSerializer.Serialize(inputMessages, ApiJsonContext.Default.InputMessageArray);
3131
_ = activity?.SetTag("gen_ai.input.messages", inputMessagesJson);
32+
logger.LogInformation("AskAI input message: {InputMessage}", askAiRequest.Message);
3233
logger.LogInformation("Streaming AskAI response");
3334
var rawStream = await askAiGateway.AskAi(askAiRequest, ctx);
3435
// The stream transformer will handle disposing the activity when streaming completes

src/api/Elastic.Documentation.Api.Infrastructure/Adapters/AskAi/StreamTransformerBase.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,7 @@ private async Task ProcessStreamAsync(PipeReader reader, PipeWriter writer, Acti
212212
case AskAiEvent.ChunkComplete chunkComplete:
213213
{
214214
outputMessageParts.Add(new MessagePart("text", chunkComplete.FullContent));
215+
Logger.LogInformation("AskAI output message: {OutputMessage}", chunkComplete.FullContent);
215216
break;
216217
}
217218
case AskAiEvent.ConversationEnd:

src/api/Elastic.Documentation.Api.Infrastructure/OpenTelemetryExtensions.cs

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,18 +3,14 @@
33
// See the LICENSE file in the project root for more information
44

55
using Elastic.OpenTelemetry;
6-
using Microsoft.AspNetCore.Builder;
76
using Microsoft.Extensions.Hosting;
8-
using Microsoft.Extensions.Logging;
97
using OpenTelemetry;
10-
using OpenTelemetry.Instrumentation.AspNetCore;
11-
using OpenTelemetry.Instrumentation.Http;
128
using OpenTelemetry.Metrics;
139
using OpenTelemetry.Trace;
1410

1511
namespace Elastic.Documentation.Api.Infrastructure;
1612

17-
public static partial class OpenTelemetryExtensions
13+
public static class OpenTelemetryExtensions
1814
{
1915
/// <summary>
2016
/// Configures Elastic OpenTelemetry (EDOT) for the Docs API.
@@ -28,12 +24,13 @@ public static TBuilder AddDocsApiOpenTelemetry<TBuilder>(
2824
{
2925
var options = new ElasticOpenTelemetryOptions
3026
{
27+
SkipOtlpExporter = true, // Disable OTLP exporter
3128
SkipInstrumentationAssemblyScanning = true // Disable instrumentation assembly scanning for AOT
3229
};
33-
_ = builder.AddElasticOpenTelemetry(edotBuilder =>
30+
_ = builder.AddElasticOpenTelemetry(options, edotBuilder =>
3431
{
3532
_ = edotBuilder
36-
.WithElasticDefaults(options)
33+
.WithElasticLogging()
3734
.WithElasticTracing(tracing =>
3835
{
3936
_ = tracing
@@ -42,18 +39,13 @@ public static TBuilder AddDocsApiOpenTelemetry<TBuilder>(
4239
.AddAspNetCoreInstrumentation()
4340
.AddHttpClientInstrumentation();
4441
})
45-
.WithElasticLogging()
4642
.WithElasticMetrics(metrics =>
4743
{
4844
_ = metrics
4945
.AddAspNetCoreInstrumentation()
5046
.AddHttpClientInstrumentation();
5147
});
5248
});
53-
54-
Console.WriteLine("[OTEL] Elastic OpenTelemetry configured successfully");
55-
Console.WriteLine($"[OTEL] Service: {builder.Environment.ApplicationName}");
56-
Console.WriteLine($"[OTEL] Environment: {builder.Environment.EnvironmentName}");
5749
return builder;
5850
}
5951
}

src/api/Elastic.Documentation.Api.Lambda/appsettings.development.json

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,12 @@
44
"Default": "Information",
55
"Microsoft.AspNetCore": "Warning"
66
}
7+
},
8+
"Elastic": {
9+
"OpenTelemetry": {
10+
"LogLevel": "Debug",
11+
"LogTargets": "stdout",
12+
"SkipInstrumentationAssemblyScanning": true
13+
}
714
}
815
}

src/api/Elastic.Documentation.Api.Lambda/appsettings.json

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,12 @@
55
"Microsoft.AspNetCore": "Warning"
66
}
77
},
8+
"Elastic": {
9+
"OpenTelemetry": {
10+
"LogLevel": "Information",
11+
"LogTargets": "stdout",
12+
"SkipInstrumentationAssemblyScanning": true
13+
}
14+
},
815
"AllowedHosts": "*"
916
}

0 commit comments

Comments
 (0)