Add additional Coaxial cable type to choices.py #7659

Closed
opened 2025-12-29 20:26:32 +01:00 by adam · 7 comments
Owner

Originally created by @davidabuckner on GitHub (Feb 17, 2023).

NetBox version

v3.4.4

Feature type

Change to existing functionality

Proposed functionality

Add the following section to choices.py

# Coaxial
TYPE_DOCSIS = 'docsis'
TYPE_BNC = "bnc'


    (
        'Coaxial',
        (
            (TYPE_DOCSIS, 'DOCSIS'),
			(TYPE_BNC, 'BNC'),
        )
    ),

Use case

This is to support devices such as a Grandmaster clock with GPS antennas

Database changes

none

External dependencies

None

Originally created by @davidabuckner on GitHub (Feb 17, 2023). ### NetBox version v3.4.4 ### Feature type Change to existing functionality ### Proposed functionality Add the following section to choices.py # Coaxial TYPE_DOCSIS = 'docsis' TYPE_BNC = "bnc' ( 'Coaxial', ( (TYPE_DOCSIS, 'DOCSIS'), (TYPE_BNC, 'BNC'), ) ), ### Use case This is to support devices such as a Grandmaster clock with GPS antennas ### Database changes none ### External dependencies None
adam added the type: feature label 2025-12-29 20:26:32 +01:00
adam closed this issue 2025-12-29 20:26:32 +01:00
Author
Owner

@davidabuckner commented on GitHub (Feb 17, 2023):

Not sure how it added the extra tab in
(TYPE_DOCSIS, 'DOCSIS'),
(TYPE_BNC, 'BNC'),

Should read
(TYPE_DOCSIS, 'DOCSIS'),
(TYPE_BNC, 'BNC'),

@davidabuckner commented on GitHub (Feb 17, 2023): Not sure how it added the extra tab in (TYPE_DOCSIS, 'DOCSIS'), (TYPE_BNC, 'BNC'), Should read (TYPE_DOCSIS, 'DOCSIS'), (TYPE_BNC, 'BNC'),
Author
Owner

@jeremystretch commented on GitHub (Feb 17, 2023):

DOCSIS is a protocol, and BNC is a connector type. Neither of these would make sense to add as a cable type.

@jeremystretch commented on GitHub (Feb 17, 2023): DOCSIS is a protocol, and BNC is a connector type. Neither of these would make sense to add as a cable type.
Author
Owner

@davidabuckner commented on GitHub (Feb 17, 2023):

Okay even weirder that it removed the tab in the comment.

@davidabuckner commented on GitHub (Feb 17, 2023): Okay even weirder that it removed the tab in the comment.
Author
Owner

@davidabuckner commented on GitHub (Feb 17, 2023):

@jeremystretch I thought that was odd, but the coaxial section under interfaces shows the DOCIS type as an option.
Maybe replace DOCIS with BNC?

@davidabuckner commented on GitHub (Feb 17, 2023): @jeremystretch I thought that was odd, but the coaxial section under interfaces shows the DOCIS type as an option. Maybe replace DOCIS with BNC?
Author
Owner

@davidabuckner commented on GitHub (Feb 17, 2023):

@jeremystretch Okay I see now where I was confused. BNC is already listed under "front port" types. I guess that makes sense since interface type isn't a "connector". Thanks for helping me through my senior moment.

@davidabuckner commented on GitHub (Feb 17, 2023): @jeremystretch Okay I see now where I was confused. BNC is already listed under "front port" types. I guess that makes sense since interface type isn't a "connector". Thanks for helping me through my senior moment.
Author
Owner

@davidabuckner commented on GitHub (Feb 17, 2023):

Only issue now is that I have to add a rear port when I'm creating a front port. That doesn't make sense when I'm adding a port to a device to connect something external.

@davidabuckner commented on GitHub (Feb 17, 2023): Only issue now is that I have to add a rear port when I'm creating a front port. That doesn't make sense when I'm adding a port to a device to connect something external.
Author
Owner

@jeremystretch commented on GitHub (Mar 16, 2023):

You can add rear ports without associating a front port. The front/rear declaration is only for differentiating between mapped ports.

I'm going to close this out as the FR is untenable.

@jeremystretch commented on GitHub (Mar 16, 2023): You can add rear ports without associating a front port. The front/rear declaration is only for differentiating between mapped ports. I'm going to close this out as the FR is untenable.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#7659