Skip to content
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

Bulk upload of Location fails; parent uniqueness check not limited to site #14064

Closed
ravinald opened this issue Oct 17, 2023 · 4 comments · Fixed by #15217
Closed

Bulk upload of Location fails; parent uniqueness check not limited to site #14064

ravinald opened this issue Oct 17, 2023 · 4 comments · Fixed by #15217
Assignees
Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@ravinald
Copy link

ravinald commented Oct 17, 2023

NetBox version

v3.6.4

Python version

3.10

Steps to Reproduce

  1. Create Site Site A
  2. Create Site Site B
  3. Create Location Floor 1 in Site A with slug floor-1
  4. Create Location Floor 1 in Site B with slug floor-1
  5. Navigate to Locations > bulk import
  6. Paste the following to get the error
site,parent,name,slug,status
Site B,Floor 1,Loc A,loc-a,active
  1. Delete Location Floor 1 in Site A
  2. Paste the same again with success

Expected Behavior

A new Location, Loc A, with a parent location of Floor 1 in site Site B should have been imported

Observed Behavior

It appears the query for locations to check for duplicates is not limiting to the defined Site. Instead, the following error is returned:

Record 1 parent: "Floor 1" is not a unique value for this field; multiple objects were found

@ravinald ravinald added the type: bug A confirmed report of unexpected behavior in the application label Oct 17, 2023
@abhi1693 abhi1693 added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation severity: low Does not significantly disrupt application functionality, or a workaround is available labels Oct 18, 2023
@kprince28
Copy link
Contributor

Can anyone please assign this issue to me ?

@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Nov 3, 2023
@jeremystretch
Copy link
Member

@kprince28 have you made any progress on this?

@kprince28
Copy link
Contributor

@jeremystretch I am currently contemplating the best approach to resolve the issue. The error seems to originate from Django's side.

@jeremystretch
Copy link
Member

@kprince28 ok, I'm going to open this back up for a new owner in case someone else would like to work on it. If you reach the point where you have a PR ready for review, please let me know and I'll reassign it to you.

@jeremystretch jeremystretch added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation and removed status: accepted This issue has been accepted for implementation labels Jan 8, 2024
@abhi1693 abhi1693 self-assigned this Feb 21, 2024
@abhi1693 abhi1693 added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Feb 21, 2024
abhi1693 added a commit that referenced this issue Feb 21, 2024
jeremystretch pushed a commit that referenced this issue Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants