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 MPO cable types #13375

Closed
romchi opened this issue Aug 4, 2023 · 5 comments
Closed

add MPO cable types #13375

romchi opened this issue Aug 4, 2023 · 5 comments
Labels
pending closure Requires immediate attention to avoid being closed for inactivity topic: cabling type: feature Introduction of new functionality to the application

Comments

@romchi
Copy link

romchi commented Aug 4, 2023

NetBox version

v3.5.7

Feature type

Data model extension

Proposed functionality

Сan you add cable types such as MPO/MTP (Multiple-fiber Push-On/Pull-off).

The multimode cable does not provide a complete understanding of which type is being used. MPO/MTP is like a separate type of cable, which must be distinguished from ordinary patches at the level of DAC, multimode and singlemode.

Proposed cable types to be added:

  • MPO/MTP
  • MPO/MTP (breakout)
  • MPO/MTP (conversion)
  • MPO/MTP (OS2)
  • MPO/MTP (OM3)
  • MPO/MTP (OM4)
  • MPO/MTP (OM5)

Use case

Will allow a more correct description of the physical connection scheme.

Database changes

Proposed cable types to be added:

  • MPO/MTP
  • MPO/MTP (breakout)
  • MPO/MTP (conversion)
  • MPO/MTP (OS2)
  • MPO/MTP (OM3)
  • MPO/MTP (OM4)
  • MPO/MTP (OM5)

External dependencies

None

@romchi romchi added the type: feature Introduction of new functionality to the application label Aug 4, 2023
@DanSheps
Copy link
Member

DanSheps commented Aug 7, 2023

I agree, to a certain extent, however I would not agree with this as a whole. Instead, MPO/MTP should be the following:

MPO Type A OS2
MPO Type A OM3
MPO Type A OM4
MPO Type A OM5
MPO Type B OS2
MPO Type B OM3
MPO Type B OM4
MPO Type B OM5
MPO Type C OS2
MPO Type C OM3
MPO Type C OM4
MPO Type C OM5

Breakout should be modelled with splices, I think, and should not be a cable type in and of itself as our cable model is not setup to handle that type of objects.

Copy link
Contributor

github-actions bot commented Nov 6, 2023

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 Nov 6, 2023
Copy link
Contributor

github-actions bot commented Dec 6, 2023

This issue has been automatically closed due to lack of activity. In an effort to reduce noise, please do not comment any further. Note that the core maintainers may elect to reopen this issue at a later date if deemed necessary.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 6, 2023
@wohali
Copy link

wohali commented Dec 6, 2023

Surely given #13447 it'd be nice to keep this open until it lands? Bad bot.

@DanSheps
Copy link
Member

DanSheps commented Dec 6, 2023

Honestly, I think it is better to model the cable type (OS1/2, OM1/2/3/4/5) as a cable and the MPO type (A/B/C) as the connector type.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pending closure Requires immediate attention to avoid being closed for inactivity topic: cabling type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

4 participants