Skip to content

Commit 33d58d3

Browse files
Merge pull request #105 from Kellybnd/doc-update
Register Activity Source with Open Telemetry so SetTag() values show up
2 parents 649e1cd + b3c554a commit 33d58d3

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

complete/Api/Program.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@
1919
builder.Services.AddNwsManager();
2020

2121
builder.Services.AddOpenTelemetry()
22-
.WithMetrics(m => m.AddMeter("NwsManagerMetrics"));
22+
.WithMetrics(m => m.AddMeter("NwsManagerMetrics"))
23+
.WithTracing(m => m.AddSource("NwsManager"));
2324

2425
var app = builder.Build();
2526

workshop/6-telemetry.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,12 @@ Our custom metrics track several key aspects of the weather service:
6868
- **Cache Performance**: Cache hit and miss rates
6969
- **Distributed Tracing**: Activity source for following requests
7070

71-
3. Register the custom meter in Program.cs:
71+
3. Register the custom meter and activity source in Program.cs:
7272

7373
```csharp
7474
builder.Services.AddOpenTelemetry()
75-
.WithMetrics(m => m.AddMeter("NwsManagerMetrics"));
75+
.WithMetrics(m => m.AddMeter("NwsManagerMetrics"))
76+
.WithTracing(m => m.AddSource("NwsManager"));
7677
```
7778

7879
## Implementing Telemetry in NwsManager

0 commit comments

Comments
 (0)