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

Add USB type to Front/Rear Ports choices #16926

Closed
Brian-Chow opened this issue Jul 16, 2024 · 1 comment · Fixed by #16951
Closed

Add USB type to Front/Rear Ports choices #16926

Brian-Chow opened this issue Jul 16, 2024 · 1 comment · Fixed by #16951
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@Brian-Chow
Copy link
Contributor

NetBox version

v4.0.7

Feature type

Data model extension

Proposed functionality

I propose to add USB choices for Front/Rear ports.
Changes proposed include modifying dcim/choices.py to add USB front and rear ports.
No other data or UI changes required beyond the addition of USB choices

Use case

Use Cases:

  • Patch Panels with modular/keystone ports and pass-through USB keystones.
  • USB WiFi and Network Adapters

Specific Scenario:
Firewalla sells a 1u rack adapter/shelf for their Firewalla GOLD line of firewall appliances. It has an optional USB Wifi adapter that they provide keystone slots to move from the rear of the firewalla appliance to the front of the 1u rack adapter.

General other benefits:
Netbox already supports USB cable types for consoles, but does not have a way to model using a usb cable and a patch panel to relocate the port from the rear/inside the rack to the front/outside of the rack. These cables should be treated as any other patch cable.
This modification could also support USB attached celluar backup modems, and USB attached SMS gateways.

Code changes already completed in fork, just waiting on approval to submit a PR.

Database changes

n/a -- only requires adding lines to dcim/choices.py

External dependencies

n/a

@Brian-Chow Brian-Chow added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels Jul 16, 2024
@jeffgdotorg jeffgdotorg removed their assignment Jul 19, 2024
@jeffgdotorg jeffgdotorg added status: accepted This issue has been accepted for implementation and removed status: needs triage This issue is awaiting triage by a maintainer labels Jul 19, 2024
@jeffgdotorg
Copy link
Contributor

Thanks for your interest in improving NetBox!

I've moved this issue to accepted status and assigning it to you for implementation, since you've expressed interest in working it through to a PR.

Brian-Chow added a commit to Brian-Chow/netbox that referenced this issue Jul 19, 2024
Brian-Chow added a commit to Brian-Chow/netbox that referenced this issue Jul 19, 2024
Brian-Chow added a commit to Brian-Chow/netbox that referenced this issue Aug 14, 2024
…ions-physical differences. Updated human readable labels on USB front/rear ports to match style of usb console ports
Brian-Chow added a commit to Brian-Chow/netbox that referenced this issue Aug 20, 2024
jeremystretch pushed a commit that referenced this issue Sep 9, 2024
…ble choices (#16951)

* Closes #16926 Adds various USB cable types to front/rear ports and cable choices

* Closes #16926 Changes USB cable types to reflect versions-physical differences. Updated human readable labels on USB front/rear ports to match style of usb console ports

* Closes #16926 Removes USB cable type choices in favor of single, generic 'USB' cable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants