Fix Netbox sweep config key generation #1758

Merged
mfreeman451 merged 1 commit from refs/pull/1758/head into 849-update-devices_mv-materialized-view-to-include-relationship-metadata 2025-06-19 17:07:06 +00:00
mfreeman451 commented 2025-06-19 17:06:36 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #983
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/983
Original created: 2025-06-19T17:06:36Z
Original updated: 2025-06-23T20:59:36Z
Original head: carverauto/serviceradar:codex/fix-nats-kv-key-storage-issue
Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata
Original merged: 2025-06-19T17:07:06Z by @mfreeman451

Summary

  • write sweep config using the configured agent ID instead of server name
  • warn and skip when agent ID is missing

Testing

  • go test ./...

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

Imported from GitHub pull request. Original GitHub pull request: #983 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/983 Original created: 2025-06-19T17:06:36Z Original updated: 2025-06-23T20:59:36Z Original head: carverauto/serviceradar:codex/fix-nats-kv-key-storage-issue Original base: 849-update-devices_mv-materialized-view-to-include-relationship-metadata Original merged: 2025-06-19T17:07:06Z by @mfreeman451 --- ## Summary - write sweep config using the configured agent ID instead of server name - warn and skip when agent ID is missing ## Testing - `go test ./...` ------ https://chatgpt.com/codex/tasks/task_e_6854422ece408320807cf60efd8bc1a0
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
carverauto/serviceradar!1758
No description provided.