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 Ubiquiti INS-3AF-O-G PoE adapter #1190

Merged
merged 4 commits into from
Mar 29, 2023

Conversation

BarbarossaTM
Copy link
Contributor

No description provided.

@danner26
Copy link
Member

danner26 commented Mar 11, 2023

Hello @BarbarossaTM I do not think this is the correct way to model this. Instead, you should have 1 interface with PoE Mode set to Powered Device PD for the internal end (PoE in side), and the second interface with PoE Mode set to Power Sourcing PSE for the PoE out side

@danner26 danner26 added the status: revisions needed This issue requires additional information to be actionable label Mar 11, 2023
@BarbarossaTM
Copy link
Contributor Author

Hm, I can see why you would want to model it this way. For me there are two reasons for rather choosing Front/Rear Port:

a) In my understanding an Interface terminates an L2 connection, which doesn't happen here, L2+-wise this thing is totally transparent, arguebly even L1-wise.
b) I want to be able to trace from the switch to the connected WiFi backhaul gear / see it directly on the switches interfaces page / the API, which I loose if these are interfaces.

What do you think?

@danner26
Copy link
Member

Hmm I see all of your points, let me pose this to the maintainers and lets see what the collective thinks

Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
@BarbarossaTM
Copy link
Contributor Author

Hey @danner26, did the collective come up with a verdict? :-D

@danner26
Copy link
Member

We are torn on it, pretty much 50/50. So as a last ditch effort, just to cover all of my bases, I am going to pose the question to the community.. If I dont reach back out in a few days please ping me again! I am sorry for the delay

@danner26
Copy link
Member

This discussion includes the poll

@danner26
Copy link
Member

Alright we are going with what you have! The majority has decided. Thanks for waiting while we figured that out

@danner26 danner26 added status: accepted and removed status: revisions needed This issue requires additional information to be actionable labels Mar 29, 2023
@danner26 danner26 merged commit adba033 into netbox-community:master Mar 29, 2023
@BarbarossaTM
Copy link
Contributor Author

Awesome, thanks!

@BarbarossaTM BarbarossaTM deleted the Ubiquiti-PoE-adapter branch March 29, 2023 15:01
etherwrangler pushed a commit to etherwrangler/devicetype-library that referenced this pull request Jul 16, 2023
* Add Ubiquiti INS-3AF-O-G PoE adapter

Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>

* Update INS-3AF-O-G.yaml

Fixed slug

---------

Signed-off-by: Maximilian Wilhelm <max@sdn.clinic>
Co-authored-by: Daniel W. Anner <daniel.anner@danstechsupport.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants