Skip to content

Remove plan_label, show tenant name, treat NetBox VMID as authoritative

Aiho Tehisaru requested to merge feature/vmid-source-of-truth-and-tenant-name into trunk
  • Drop the plan_label custom field and its CPU/RAM/Disk default table entirely (model.DeviceSpec.PlanLabel, netbox.planDefaults/ applyPlanDefaults) — every device now needs explicit cpu/ram_gb/ disk_gb custom fields in NetBox.
  • Device page's "Client" field now shows the tenant's NetBox Name instead of the numeric Klient_ID (model.DeviceSpec.TenantName). Generated Proxmox VM notes are unchanged: they still write only ClientID, never a tenant's real name.
  • NetBox is now treated as the source of truth for VMID: a device with a non-zero VMID declared in NetBox but no matching live VM on Proxmox now gets a real BlockerVMIDMissingOnProxmox (a constant that existed but was never actually wired up) instead of silently being offered a create_vm action, which would have created a brand-new VM under a different, auto-assigned VMID and drifted further from what NetBox declares. Only a genuinely new device (VMID == 0) still gets an automatic create_vm proposal.

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com Claude-Session: https://claude.ai/code/session_01U9buZPRQy7YXLodkSPFFik

Merge request reports