Skip to content

fix(influx): keep InfluxDB ingest independent of MQTT brokers - #68

Merged
stakach merged 2 commits into
masterfrom
fix/influx-ingest-isolation
Oct 27, 2026
Merged

stakach merged 2 commits into
masterfrom
fix/influx-ingest-isolation

Conversation

@stakach

@stakach stakach commented Oct 27, 2026

Copy link
Copy Markdown
Member

Adds end to end InfluxDB ingest specs (Redis → StatusEvents → InfluxDB, read back with Flux) covering every payload format, and fixes what they found:

  • A dead MQTT broker blocked InfluxDB. Each message was retried ~30s while disconnected, filling the publisher queue and blocking broadcast for every sink. Publishing is now skipped while the client is closed; reconnecting already republishes all state.
  • Resyncs were written to InfluxDB, recording a duplicate point for every status on each broker reconnect. Resyncs now go only to sinks that retain state.
  • Buffered points could wait indefinitely under a steady trickle, and were dropped on stop. They now commit by age and on stop.
  • Default filters never matched: they ran on raw JSON, where strings are quoted. They now apply to parsed field and tag values.

Review

  • Full suite: 78 examples, 0 failures.
  • New specs fail on master for the dead-broker, resync, trickle, stop and filter cases.
  • Not covered: a half-open broker connection still applies backpressure until keepalive notices (~60–90s), and a slow InfluxDB can likewise slow MQTT.

🤖 Generated with Claude Code

- skip MQTT retries while disconnected, so a dead broker no longer fills its
  queue and blocks every other sink; reconnecting republishes state
- send state resyncs only to sinks that retain state, not InfluxDB
- commit buffered points by age and on stop
- apply the default filters to parsed values, so they actually match
- add end to end InfluxDB ingest specs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added the type: bug something isn't working label Oct 27, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added type: bug something isn't working and removed type: bug something isn't working labels Oct 27, 2026
@stakach
stakach merged commit 5713986 into master Oct 27, 2026
10 checks passed
@stakach
stakach deleted the fix/influx-ingest-isolation branch October 27, 2026 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant