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 support for Molex Power Connectors #12984

Closed
engageant opened this issue Jun 23, 2023 · 11 comments · Fixed by #14909
Closed

Add support for Molex Power Connectors #12984

engageant opened this issue Jun 23, 2023 · 11 comments · Fixed by #14909
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@engageant
Copy link

NetBox version

v3.5.4

Feature type

Data model extension

Proposed functionality

Add Molex Micro-Fit 2x2 to power port inlet type.

Use case

Various devices utilize various Molex connectors for power input. For example, Cradlepoint E300 and IBR900-1200M routers and the Cisco 891 router all use a 2x2 Molex Micro-Fit connector.

Database changes

None required.

External dependencies

None.

@engageant engageant added the type: feature Introduction of new functionality to the application label Jun 23, 2023
@DanSheps DanSheps added the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Jun 26, 2023
@jeremystretch jeremystretch added status: under review Further discussion is needed to determine this issue's scope and/or implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Jun 27, 2023
@jeremystretch
Copy link
Member

Are these devices connecting directly to a power source via these connectors? Typically Molex connectors are used to connect an external power supply, which itself houses the device's actual power port.

@DanSheps
Copy link
Member

DanSheps commented Jun 27, 2023

They are external PSU. I have used 881's (similar to the 891's) before, it is a stupid connector.

However, if someone wanted to get pedantic they could model the power adapter as a separate device.

@engageant
Copy link
Author

The Cradlepoints specifically can be hardwired to a vehicle ; the cable in this case just carries direct power and a couple GPIO lines.

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Do not attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Sep 26, 2023
@DanSheps DanSheps added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation pending closure Requires immediate attention to avoid being closed for inactivity labels Oct 2, 2023
@DanSheps
Copy link
Member

DanSheps commented Oct 2, 2023

Going to make this available for implementation as it does appear to be able to support direct connection to a DC power source.

Should likely add additional types for other devices (for example, Nexus 9k is typically a Molex Mini-FIT)

Copy link
Contributor

github-actions bot commented Jan 1, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Do not attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Jan 1, 2024
@arcticash
Copy link
Contributor

Can this feature be reviewed again for implementation, looks like there is already a solution under #13014

The use case I have is although typically you are right these devices will use an included external power supply to convert from 240vAC -> 48vDC, there is also a use case i have in certain environments such as industrial controls cabinets where they will be connected directly to a 48v power source, using the Molex headers. Not a showstopper but it would be good to model the connector accurately if only for my own sanity, rather than just using the "other" connector type 😄

@DanSheps
Copy link
Member

DanSheps commented Jan 9, 2024

Unfortunately no one has volunteered for this. The linked PR was closed as this issue needed to be re-evaluated.

We would need a new PR going forward

@arcticash
Copy link
Contributor

I'd be happy to work on a new PR to implement this, if implementing the new connector types is the way forward

@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 pending closure Requires immediate attention to avoid being closed for inactivity labels Jan 19, 2024
@abhi1693
Copy link
Member

@arcticash All yours!

@arcticash
Copy link
Contributor

Is anyone able to trigger the workflow on the pull request, just noticed that it's still pending :)

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
5 participants