Phase 6.1 Stream E.3 partial - in-flight counter feeds CurrentBulkheadDepth #107

dohertj2 · 2026-04-19T15:04:26-04:00

dohertj2 commented

2026-04-19 15:04:26 -04:00

CapabilityInvoker now records start/complete against the optional tracker; ResilienceStatusPublisherHostedService persists the counter as CurrentBulkheadDepth (was 0 before).

Summary

DriverResilienceStatusTracker gains RecordCallStart/Complete + CurrentInFlight snapshot field. Clamps to zero on over-decrement.
CapabilityInvoker optional statusTracker ctor param + try/finally around every ExecuteAsync so the counter decrements on success, exception, or cancellation.
Publisher persists CurrentInFlight as CurrentBulkheadDepth. A future Polly-telemetry observer can replace the proxy without changing the column shape.

Test plan

8 new InFlightCounterTests: start/complete parity, nested starts, over-decrement clamp, per-host independence, 500-parallel concurrent safety, mid-call observation == 1, exception path decrements, null-tracker no-op.
Full solution dotnet test: 1243 passing (was 1235, +8).

🤖 Generated with Claude Code

CapabilityInvoker now records start/complete against the optional tracker; ResilienceStatusPublisherHostedService persists the counter as CurrentBulkheadDepth (was 0 before). ## Summary - `DriverResilienceStatusTracker` gains RecordCallStart/Complete + CurrentInFlight snapshot field. Clamps to zero on over-decrement. - `CapabilityInvoker` optional statusTracker ctor param + try/finally around every ExecuteAsync so the counter decrements on success, exception, or cancellation. - Publisher persists CurrentInFlight as CurrentBulkheadDepth. A future Polly-telemetry observer can replace the proxy without changing the column shape. ## Test plan - [x] 8 new InFlightCounterTests: start/complete parity, nested starts, over-decrement clamp, per-host independence, 500-parallel concurrent safety, mid-call observation == 1, exception path decrements, null-tracker no-op. - [x] Full solution `dotnet test`: 1243 passing (was 1235, +8). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

dohertj2 added 1 commit 2026-04-19 15:04:28 -04:00

Phase 6.1 Stream E.3 partial — in-flight counter feeds CurrentBulkheadDepth ae8f226e45

Closes the observer half of #162 that was flagged as "persisted as 0 today"
in PR #105. The Admin /hosts column refresh + FleetStatusHub SignalR push
+ red-badge visual still belong to the visual-compliance pass.

Core.Resilience:
- DriverResilienceStatusTracker gains RecordCallStart + RecordCallComplete
  + CurrentInFlight field on the snapshot record. Concurrent-safe via the
  same ConcurrentDictionary.AddOrUpdate pattern as the other recorder methods.
  Clamps to zero on over-decrement so a stray Complete-without-Start can't
  drive the counter negative.
- CapabilityInvoker gains an optional statusTracker ctor parameter. When
  wired, every ExecuteAsync / ExecuteAsync(void) wraps the pipeline call
  in try / finally that records start/complete — so the counter advances
  cleanly whether the call succeeds, cancels, or throws. Null tracker keeps
  the pre-Phase-6.1 Stream E.3 behaviour exactly.

Server.Hosting:
- ResilienceStatusPublisherHostedService persists CurrentInFlight as the
  DriverInstanceResilienceStatus.CurrentBulkheadDepth column (was 0 before
  this PR). One-line fix on both the insert + update branches.

The in-flight counter is a pragmatic proxy for Polly's internal bulkhead
depth — a future PR wiring Polly telemetry would replace it with the real
value. The shape of the column + the publisher + the Admin /hosts query
doesn't change, so the follow-up is invisible to consumers.

Tests (8 new InFlightCounterTests, all pass):
- Start+Complete nets to zero.
- Nested starts sum; Complete decrements.
- Complete-without-Start clamps to zero.
- Different hosts track independently.
- Concurrent starts (500 parallel) don't lose count.
- CapabilityInvoker observed-mid-call depth == 1 during a pending call.
- CapabilityInvoker exception path still decrements (try/finally).
- CapabilityInvoker without tracker doesn't throw.

Full solution dotnet test: 1243 passing (was 1235, +8). Pre-existing
Client.CLI Subscribe flake unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

dohertj2 merged commit 2172d49d2e into v2

2026-04-19 15:04:30 -04:00

dohertj2 referenced this issue from a commit

2026-04-19 15:04:33 -04:00

Merge pull request (#107) - in-flight counter

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#107