Add configurable NATS stream for Sync #1747

Merged

mfreeman451 merged 1 commit from refs/pull/1747/head into 800-live-armis-poc-integration-into-serviceradar

2025-06-18 19:46:53 +00:00

mfreeman451 commented

2025-06-18 19:45:47 +00:00

(Migrated from github.com)

Owner

Imported from GitHub pull request.

Original GitHub pull request: #971
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/971
Original created: 2025-06-18T19:45:47Z
Original updated: 2025-06-18T19:47:10Z
Original head: carverauto/serviceradar:codex/define-stream-and-subject-in-nats-jetstream-config
Original base: 800-live-armis-poc-integration-into-serviceradar
Original merged: 2025-06-18T19:46:53Z by @mfreeman451

Summary

allow Sync to specify stream_name and subject in config
validate new options and use them when creating JetStream context
prefix domain on publish if set
update default config and docs
adjust unit tests for new fields

Testing

go test ./pkg/sync -run Test

https://chatgpt.com/codex/tasks/task_e_685315a0e77c83208b8d61f8c748216d

Imported from GitHub pull request. Original GitHub pull request: #971 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/971 Original created: 2025-06-18T19:45:47Z Original updated: 2025-06-18T19:47:10Z Original head: carverauto/serviceradar:codex/define-stream-and-subject-in-nats-jetstream-config Original base: 800-live-armis-poc-integration-into-serviceradar Original merged: 2025-06-18T19:46:53Z by @mfreeman451 --- ## Summary - allow Sync to specify `stream_name` and `subject` in config - validate new options and use them when creating JetStream context - prefix domain on publish if set - update default config and docs - adjust unit tests for new fields ## Testing - `go test ./pkg/sync -run Test` ------ https://chatgpt.com/codex/tasks/task_e_685315a0e77c83208b8d61f8c748216d