Handle 207 from Armis bulk updates #1892

Merged
mfreeman451 merged 1 commit from refs/pull/1892/head into bug/armis_updater_cant_update_armis 2025-06-25 16:12:28 +00:00
mfreeman451 commented 2025-06-25 16:11:47 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #1147
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/1147
Original created: 2025-06-25T16:11:47Z
Original updated: 2025-06-25T16:12:31Z
Original head: carverauto/serviceradar:codex/fix-armis-updater-failure-with-status-207
Original base: bug/armis_updater_cant_update_armis
Original merged: 2025-06-25T16:12:28Z by @mfreeman451

Summary

  • allow 207 Multi-Status responses from Armis bulk APIs
  • update tests for the new status handling

Testing

  • go test ./pkg/sync/integrations/armis -run TestDefaultArmisUpdater_UpdateDeviceStatus -count=1

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

Imported from GitHub pull request. Original GitHub pull request: #1147 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/1147 Original created: 2025-06-25T16:11:47Z Original updated: 2025-06-25T16:12:31Z Original head: carverauto/serviceradar:codex/fix-armis-updater-failure-with-status-207 Original base: bug/armis_updater_cant_update_armis Original merged: 2025-06-25T16:12:28Z by @mfreeman451 --- ## Summary - allow 207 Multi-Status responses from Armis bulk APIs - update tests for the new status handling ## Testing - `go test ./pkg/sync/integrations/armis -run TestDefaultArmisUpdater_UpdateDeviceStatus -count=1` ------ https://chatgpt.com/codex/tasks/task_e_685c1e3b4fc483209e650495b933a1af
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!1892
No description provided.