Skip to content

Conversation

@bruelea
Copy link
Collaborator

@bruelea bruelea commented May 2, 2025

With this feature a check is added to only allow existing fields in the parentPrefixSelector.
If the parent prefix selector contains a key for a custom field that does not exist in NetBox, the key is ignored in the filter.

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.

Testes locally and works nice! I like the new thorough tests and the nice error message. Just 2 minor changes

bruelea and others added 3 commits May 6, 2025 09:58
Co-authored-by: Fabian <30692464+faebr@users.noreply.github.com>
@faebr faebr self-requested a review May 6, 2025 12:30
@faebr faebr self-requested a review May 6, 2025 15:28
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.

Added a small question for my understanding but looks good to me :)

@bruelea bruelea merged commit c955981 into main May 7, 2025
8 checks passed
vaishutin pushed a commit to vaishutin/netbox-operator that referenced this pull request Aug 17, 2025
* check if custom fields in parent prefix selector exist

* add unit tests

* add chainsaw test

---------

Co-authored-by: Fabian <30692464+faebr@users.noreply.github.com>
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.

3 participants