-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
when editing vlans with the api, vlans without a group are considered as one group, resulting in duplicate-errors #17979
Comments
the exact api call: curl -X 'PUT' response: Error: Bad request: { |
Please edit your post above to include the specific values and REST API query used in the creation of each object. This is necessary for someone to attempt to reproduce the behavior. |
Maybe a duplicate of (or at least related to) #17810 ? |
@jeremystretch here is the API call flow that will result in the error: POST ungrouped VLAN with vid 1000 -> Success with ID 1
POST ungrouped VLAN with vid 1000 -> Success with ID 2
POST ungrouped VLAN with vid 1001 -> Success with ID 3
PUT VLAN with ID 1 to update VID to 1001 -> Error: Bad request: {"non_field_errors": ["The fields group, vid must make a unique set."]}
PATCH VLAN with ID 1 to update VID to 1001 -> Error: Bad request: {"non_field_errors": ["The fields group, vid must make a unique set."]}
|
Deployment Type
Self-hosted
Triage priority
N/A
NetBox Version
v4.1.6
Python Version
3.10
Steps to Reproduce
Using the API, create a vlan without a group,
using the api, create a second vlan with the same vlan-id, this works.
using the api, create a third vlan with a different vlan-id, also without a group, this works.
using the api, change the first vlan to have the same vlan id as the second one, still without a group. This results in an error that the vlan is duplicate.
Expected Behavior
as the vlans have no group, duplicates should be allowed.
When the same steps to reproduce are executed using the Netbox Gui, it works as expected.
Observed Behavior
Error: The fields group, vid must make a unique set.
The text was updated successfully, but these errors were encountered: