docs(modbus-tests): sweep #2 was a fixture-cycling false alarm (all profiles green) #448

2026-07-15T05:13:04-04:00

dohertj2 commented

2026-07-15 05:13:04 -04:00

Integration-sweep follow-up #2. The Modbus non-standard 8F/5F/3F/8F was NOT pymodbus drift (image runs the pinned 3.13.0) and NOT an OtOpcUa regression — it was a fixture-cycling artifact: profile-gated compose services share :5020 and 'docker compose down' (no --profile) doesn't stop the running one, so later profile runs hit the stale sim (proven: 'mitsubishi' served DL205's 0xCAFE). Bringing each up cleanly: dl205 16/16, mitsubishi 13/13, s7_1500 10/10, exception_injection 17/17. No code change — documented the gotcha + reliable-cycle recipe in Docker/README.md and corrected the sweep record.

dohertj2 added 1 commit 2026-07-15 05:13:06 -04:00

docs(modbus-tests): document profile-cycling gotcha; sweep #2 was a false alarm

v2-ci / build (pull_request) Successful in 4m13s

Details

v2-ci / unit-tests (pull_request) Failing after 9m41s

Details

00f39c849e

The 2026-07 integration sweep flagged the Modbus non-standard profiles as
8F/5F/3F/8F and guessed 'pymodbus 4.x drift'. Re-investigation shows that was
wrong on both counts:

- The image runs the PINNED pymodbus 3.13.0 (verified via import pymodbus;
  __version__). The '...removed in v4' log is 3.13.0's forward-deprecation
  warning, misread as running 4.x.
- The failures were a fixture-cycling artifact: each profile is a separate
  profile-gated compose service sharing :5020, and a plain 'docker compose down'
  (no --profile) does NOT stop the running profile container. It kept holding
  :5020; the next '--profile up' container silently failed to bind (compose
  still says 'Started'), so every later profile run hit the STALE sim. Proven by
  a raw Modbus probe: the 'mitsubishi' sim served DL205's 0xCAFE at HR0.

After force-removing all modbus containers and bringing up each profile cleanly
(verifying it actually published :5020), all four pass: dl205 16/16,
mitsubishi 13/13, s7_1500 10/10, exception_injection 17/17. No OtOpcUa or test
code change needed.

Added a profile-cycling gotcha + reliable-cycle recipe to Docker/README.md and
corrected the sweep record. Integration-sweep follow-up #2.

dohertj2 merged commit ab1644daa4 into master

2026-07-15 05:13:14 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dohertj2/lmxopcua#448