Fix KV device path formatting #1759

Merged

mfreeman451 merged 1 commit from refs/pull/1759/head into 849-update-devices_mv-materialized-view-to-include-relationship-metadata

2025-06-19 17:25:45 +00:00

mfreeman451 commented

2025-06-19 17:25:27 +00:00

(Migrated from github.com)

Owner

Imported from GitHub pull request.

Original GitHub pull request: #984
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/984
Original created: 2025-06-19T17:25:27Z
Original updated: 2025-06-19T17:25:48Z
Original head: carverauto/serviceradar:codex/update-key-path-format-in-serviceradar-sync
Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata
Original merged: 2025-06-19T17:25:45Z by @mfreeman451

Summary

adjust writeToKV to build KV keys without colons
parse device IDs of form ip:agent:poller
test new path transformation logic

Testing

go test ./pkg/sync/... ./pkg/kv/... ./pkg/config/...
go test ./...

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

Imported from GitHub pull request. Original GitHub pull request: #984 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/984 Original created: 2025-06-19T17:25:27Z Original updated: 2025-06-19T17:25:48Z Original head: carverauto/serviceradar:codex/update-key-path-format-in-serviceradar-sync Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata Original merged: 2025-06-19T17:25:45Z by @mfreeman451 --- ## Summary - adjust `writeToKV` to build KV keys without colons - parse device IDs of form `ip:agent:poller` - test new path transformation logic ## Testing - `go test ./pkg/sync/... ./pkg/kv/... ./pkg/config/...` - `go test ./...` ------ https://chatgpt.com/codex/tasks/task_e_6854450c9750832094af182acb14f4dd

Rows
Columns