Skip to content

Conversation

@jstudler
Copy link
Collaborator

If there is a mismatch of .spec.prefixLength and the prefix length of the Prefix in NetBox that would be restored, NetBox Operator will now longer restore it.

Fixes #82

@jstudler jstudler self-assigned this Jul 16, 2025
@jstudler jstudler requested a review from faebr July 16, 2025 14:23
@jstudler jstudler added the bug Something isn't working label Jul 16, 2025
Copy link
Collaborator

@faebr faebr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected and I like the tests but the error message can be misleading. Also I think the operator logs should not print the stack traces on such an error since its not an unexpected code error. But this might be out of scope of this mr

@jstudler jstudler merged commit 8af64cb into main Jul 16, 2025
12 checks passed
@jstudler jstudler deleted the fix/restore-pxc-to-check-prefix-length branch July 18, 2025 13:06
vaishutin pushed a commit to vaishutin/netbox-operator that referenced this pull request Aug 17, 2025
…-community#373)

If there is a mismatch of .spec.prefixLength and the prefix length of the Prefix in NetBox that would be restored, NetBox Operator will now longer restore it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restoring PrefixClaim ignores prefixLength

3 participants