dangernoodle-io / TaipanMiner / 28264259603

90%

push

github

feat: publish run metrics to mosquitto by default (push_telemetry) (TA-455) (#586)

* feat: add fleetlib/mqtt.py shared helper (connect_and_publish, subscribe_and_wait)

extract reusable paho-mqtt primitives from suites/telemetry.py into a
shared module; paho remains a soft dep (lazy import; _client_factory
injection bypasses import for offline tests); build_tls_context centralises
ssl.SSLContext construction from pem strings

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat: rename push_influxdb -> push_telemetry; implement MQTT publish; refactor telemetry.py to use shared mqtt helper (TA-455)

- ResultSet.push_influxdb (no-op stub) removed; push_telemetry publishes
  per-result metrics as json to fleettest/<suite>/<board>
- payload: {suite, host, board, ts (iso), status, metrics:{...}}
- never raises: publish failures are warnings; exit code unchanged
- suites/telemetry.py _mqtt_broker_verify now delegates to
  fleetlib.mqtt.subscribe_and_wait + build_tls_context (no duplicate logic)
- test_telemetry.py: patch target updated to fleetlib.mqtt.ssl

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* feat: wire push_telemetry into fleet.py suite completion; add --metrics-mqtt-url / --no-publish-metrics flags (TA-455)

- default ON after every suite run when a broker is configured
  (--metrics-mqtt-url / BB_TEST_METRICS_BROKER / BB_TEST_RECEIVER fallback)
- --no-publish-metrics opt-out flag
- no broker configured: skip with one-line note on stderr (non-fatal)
- publish exception: warning only, exit code unchanged
- --metrics-topic PREFIX overrides default 'fleettest' topic prefix

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test: push_telemetry offline tests — topic/payload shape, opt-out, no broker skip, exception safety, rename assertion (TA-455)

14 new tests; mock paho client injected via _client_factory — no broker
required; full suite: 526 passed

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
... (continued)

2047 of 2628 branches covered (77.89%)

Branch coverage included in aggregate %.

3453 of 3507 relevant lines covered (98.46%)

37.53 hits per line