Skip to content

fix(storage): node upsert meshtastic_* fields + API v3 client paths#111

Merged
pskillen merged 2 commits into
mainfrom
bot-110/paddy/fix-store-node-startup-errors
May 22, 2026
Merged

fix(storage): node upsert meshtastic_* fields + API v3 client paths#111
pskillen merged 2 commits into
mainfrom
bot-110/paddy/fix-store-node-startup-errors

Conversation

@pskillen

@pskillen pskillen commented May 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Send meshtastic_location_source, meshtastic_channel_utilization, and meshtastic_air_util_tx on node upsert (fixes startup store_node 400s in [ERROR] StorageAPI.store_node on startup #110).
  • Default STORAGE_API_VERSION=3 with /api/v3/packets/{nodenum}/ for ingest, nodes, and bot-version.
  • Keep STORAGE_API_VERSION=2 for /api/packets/... when paired with older configs.
  • Include node id in StorageAPI.store_node HTTP error logs.

Closes #110

Related: pskillen/meshflow-api#339, pskillen/meshflow-api#340

Deploy meshflow-api #340 before merging so v3 routes and v2 legacy aliases exist in production.

Testing performed

  • pytest test/meshtastic/test_serializers.py test/test_bot_version_report.py -v
  • Manual: restart bot against API with #340; confirm no startup store_node flood

Node upsert used legacy location_source/channel_utilization keys that
meshflow-api no longer maps, causing startup store_node 400s. Coalesce
null metrics and log the node id on HTTP errors.

Closes #110
Use /api/v3/packets/{nodenum}/ for ingest, nodes, and bot-version.
Keep v2 URL prefix when STORAGE_API_VERSION=2. Related to meshflow-api#339.
@pskillen pskillen changed the title fix(storage): send meshtastic_* fields on node upsert fix(storage): node upsert meshtastic_* fields + API v3 client paths May 22, 2026
@pskillen pskillen merged commit 78fab21 into main May 22, 2026
3 checks passed
@pskillen pskillen deleted the bot-110/paddy/fix-store-node-startup-errors branch May 22, 2026 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ERROR] StorageAPI.store_node on startup

2 participants