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 import/assign OOB addresses #17071

Closed
andy-lawson opened this issue Aug 2, 2024 · 5 comments · Fixed by #17975
Closed

Bulk import/assign OOB addresses #17071

andy-lawson opened this issue Aug 2, 2024 · 5 comments · Fixed by #17975
Assignees
Labels
complexity: low Requires minimal effort to implement netbox status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@andy-lawson
Copy link

andy-lawson commented Aug 2, 2024

NetBox version

v3.7.5

Feature type

New functionality

Proposed functionality

Add the capability to add or assign device out-of-band IP addresses by bulk import or update.

Add an "is_oob" field to the IP address bulk import feature

Use case

  • When importing IPs, it is currently possible to assign those IPs as the "Primary" IP for a device.
  • Out-of-band IP status cannot be assigned in this manner during the bulk import or subsequent bulk update of IPs or devices.
  • Out-of-band IP status is frequently overlooked because it cannot be bulk added or updated.

Database changes

None.

External dependencies

None.

Edit for clarity

@andy-lawson andy-lawson added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels Aug 2, 2024
@andy-lawson
Copy link
Author

May be a duplicate of 16801, but that issue has closed for no update.

@arthanson
Copy link
Collaborator

@andy-lawson please modify your original post above to clearly indicate what field(s) you're proposing be added to which bulk import forms.

@arthanson arthanson added status: revisions needed This issue requires additional information to be actionable and removed status: needs triage This issue is awaiting triage by a maintainer labels Aug 2, 2024
@andy-lawson
Copy link
Author

Updated as requested.

@arthanson arthanson added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation complexity: low Requires minimal effort to implement and removed status: revisions needed This issue requires additional information to be actionable labels Aug 5, 2024
@arthanson arthanson removed their assignment Aug 5, 2024
@pl0xym0r
Copy link
Contributor

pl0xym0r commented Oct 31, 2024

Was working on it and was wondering : if implemeted on bulk_import, shouldn't be added on IPAddressForm too to keep consistency ? It could be a box to check (like : "make it primary") under :

image

edit : I think that feature is asked on #17889

@jeremystretch jeremystretch added the netbox label Nov 1, 2024 — with Linear
@M00SE80
Copy link

M00SE80 commented Nov 6, 2024

I too would like to see this functionality added.

Currently there is an option to make an address the primary using the is_primary field under the bulk add IP Address form.
It would be good to have something like this for the OOB here also as @andy-lawson has suggested like is_oob

@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 Dec 9, 2024
jeremystretch added a commit that referenced this issue Dec 9, 2024
* add is_oob parameter on bulk_import ipaddress

* Tweak wording

---------

Co-authored-by: Jeremy Stretch <jstretch@netboxlabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: low Requires minimal effort to implement netbox status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants