Skip to content

cli: report missing TunnelIds extension for devices without users#3560

Merged
elitegreg merged 1 commit intomainfrom
gm/resource-verify-user-count-fix
Apr 23, 2026
Merged

cli: report missing TunnelIds extension for devices without users#3560
elitegreg merged 1 commit intomainfrom
gm/resource-verify-user-count-fix

Conversation

@elitegreg
Copy link
Copy Markdown
Contributor

Summary

  • Fix doublezero resource verify to report missing TunnelIds resource extensions for all devices, not just those with users
  • Previously the ExtensionNotFound discrepancy was suppressed when a device had no users, which hid unallocated extensions from operators

Testing Verification

  • Added test_verify_tunnel_ids_reports_missing_extension_for_device_without_users covering a device with no users and no TunnelIds extension; asserts the ExtensionNotFound discrepancy is emitted

Previously `doublezero resource verify` suppressed the
`ExtensionNotFound` discrepancy for a device's `TunnelIds` resource
extension when the device had no users, which hid unallocated
extensions. Always report the discrepancy so operators can spot missing
extensions regardless of user count.
@elitegreg elitegreg enabled auto-merge (squash) April 21, 2026 19:30
@elitegreg elitegreg requested a review from vihu April 22, 2026 18:01
@elitegreg elitegreg merged commit b1eb7b9 into main Apr 23, 2026
36 checks passed
@elitegreg elitegreg deleted the gm/resource-verify-user-count-fix branch April 23, 2026 17:07
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.

2 participants