dohertj2/lmxopcua
1
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

Galaxy probe initial ScanState lost when platforms are registered before the event pump starts #865

New Issue
Closed
opened 2026-11-11 12:14:09 -05:00 by dohertj2 · 1 comment
dohertj2 commented 2026-11-11 12:14:09 -05:00
Owner
Copy Link
Copy Source

Found by the #614 read-only live run against VD03 (2026-11-11). The event pump starts only on the first Read or Subscribe. Platforms that the probe watcher registers before then never receive their initial ScanState value. ScanState rarely changes, so those platforms stay missing from GetHostStatuses().

Live evidence: in normal order, only the transport entry appeared after 30 s. With the pump started before Discover, all 5 hosts appeared (CTRX_DEV, DevWinPlatform and DevAppEngine Running; JoePlatform and JoeAppEngine Unknown).

This compounds the platform-seed bug: once the seed works, a background seed would always hit this race, and a driver with no authored tags would never start the pump at all.

Fix: start the pump, or replay or read the current value, when probe advises are registered. Compare with the #859 early-event parking.

Found by the #614 read-only live run against VD03 (2026-11-11). The event pump starts only on the first Read or Subscribe. Platforms that the probe watcher registers before then never receive their initial `ScanState` value. ScanState rarely changes, so those platforms stay missing from `GetHostStatuses()`. **Live evidence:** in normal order, only the transport entry appeared after 30 s. With the pump started before Discover, all 5 hosts appeared (CTRX_DEV, DevWinPlatform and DevAppEngine Running; JoePlatform and JoeAppEngine Unknown). This compounds the platform-seed bug: once the seed works, a background seed would always hit this race, and a driver with no authored tags would never start the pump at all. **Fix:** start the pump, or replay or read the current value, when probe advises are registered. Compare with the #859 early-event parking.
dohertj2 added the priority:p2type:bug labels 2026-11-11 12:14:09 -05:00
dohertj2 commented 2026-11-11 14:53:54 -05:00
Author
Owner
Copy Link
Copy Source

Done. The event stream opens before probe or advise registration, and the ResolveOrPark race and the replay-over-dead-stream bug in ReconnectSupervisor are fixed. Seven review rounds. Live, read-only on VD03: GetHostStatuses lists all 5 hosts before any Discover/Read/Subscribe. The remaining race needs a gateway attach acknowledgement: mxaccessgw#227. Pushed to master in e857135e..7a27ef1f; full suite green (20 projects, 0 failures, 0 warnings).

Done. The event stream opens before probe or advise registration, and the ResolveOrPark race and the replay-over-dead-stream bug in ReconnectSupervisor are fixed. Seven review rounds. Live, read-only on VD03: GetHostStatuses lists all 5 hosts before any Discover/Read/Subscribe. The remaining race needs a gateway attach acknowledgement: mxaccessgw#227. Pushed to master in `e857135e..7a27ef1f`; full suite green (20 projects, 0 failures, 0 warnings).
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
auto-managed
queue management
 blocked
Blocked by dependency or external hold
 cross-driver
queue management
 cross-repo
Needs coordinated change in another family repo (wire contract etc.)
 driver/abcip
queue management
 driver/ablegacy
queue management
 driver/focas
queue management
 driver/mqtt
MQTT / Sparkplug B broker-subscribe driver
 driver/opcuaclient
queue management
 driver/s7
queue management
 driver/twincat
queue management
 migrated
Migrated from a markdown backlog (2026-11 standardization)
 phase/1
queue management
 phase/2
queue management
 phase/3
queue management
 phase/4
queue management
 phase/5
queue management
 phase/6
queue management
 priority:p0
Must-have / blocking
 priority:p1
High priority
 priority:p2
Normal priority
 priority:p3
Low priority / nice to have
 queue/blocked
queue management
 queue/done
queue management
 queue/failed
queue management
 queue/in-progress
queue management
 queue/queued
queue management
 type:bug
Defect in shipped behaviour
 type:debt
Technical debt / hygiene
 type:deploy-gate
Rollout/deploy/migration step not yet done on an environment
 type:docs
Documentation work
 type:feature
Feature implementation
 type:followup
Follow-up deferred from a plan, PR or gate
 type:infra
Infrastructure and scaffolding
 type:review-finding
Unfixed finding from an architecture/code review
 type:test
Test work
No Label priority:p2 type:bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dohertj2 (dohertj2)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#865
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?