Skip to content

Feature: add connector worker runtime#251

Merged
smithaldon1 merged 1 commit into
mainfrom
feat/issue-238-connector-worker-runtime
May 27, 2026
Merged

Feature: add connector worker runtime#251
smithaldon1 merged 1 commit into
mainfrom
feat/issue-238-connector-worker-runtime

Conversation

@smithaldon1

Copy link
Copy Markdown
Member

Summary

  • Add a long-running read-only connector worker entrypoint for enabled connection profiles.
  • Wire Docker Compose connector-worker to the new runtime and poll interval env.
  • Add read-only MQTT source support, OPC UA browse-path resolution, fixture corrections, docs, and learning log updates.

Closes #238

Test Evidence

  • .venv/bin/python -m pytest services/ingestion/tests/test_connector_worker.py services/ingestion/tests/test_opcua_read_only_adapter.py services/ingestion/tests/test_mqtt_read_only_adapter.py services/ingestion/tests/test_bacnet_read_only_adapter.py services/simulator/tests/test_docker_compose_runtime_docs.py: 39 passed before final fixture expansion; final focused suite with fixture tests: 49 passed; MQTT focused suite: 12 passed.
  • make test-integration: 104 passed, 2 skipped.
  • make lint: All checks passed.
  • make typecheck: compileall completed successfully.
  • make test: 280 passed, 4 skipped.
  • docker compose -f infra/docker/docker-compose.yml config: passed.
  • Compose smoke with Demo-Factory running separately: connector-worker healthy; logs showed OPC UA profiles=1 records=5 emitted_events=5; Postgres factory_events count reached 20 before cleanup.

Docs Impact

  • Updated ingestion README, Docker Compose runtime docs, troubleshooting env list, .env.example, and docs/LEARNING_LOG.md.

Follow-up Work

  • MQTT currently records numeric Sparkplug metrics when they arrive; boolean/string metrics are logged as nonnumeric until source-to-event mappings support non-process-measurement event types.
  • BACnet remains disabled in the Demo-Factory fixture by default per prior Docker Desktop variability.

@smithaldon1 smithaldon1 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and approved for merge

@smithaldon1 smithaldon1 merged commit f2183ff into main May 27, 2026
3 checks passed
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.

Feature: Add connector worker runtime

1 participant