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

Support for port-specific VLAN tags #629

Open
tardoe opened this issue Apr 6, 2020 · 14 comments · May be fixed by #630
Open

Support for port-specific VLAN tags #629

tardoe opened this issue Apr 6, 2020 · 14 comments · May be fixed by #630

Comments

@tardoe
Copy link

tardoe commented Apr 6, 2020

Is your feature request related to a problem? Please describe.
Given the modern fabric solution of presenting multiple services on a single port, we'd like the option to tag a given peering VLAN onto a fabric port with a customer-selected VLAN ID. This type of port-specific VLAN hand-off is common place in modern switching platforms and is easily automated with the correct source-of-truth information.

Describe the solution you'd like
During provisioning, when a user selects "Use 802.1q framing" they are presented with an additional field (portSpecificVlanId) where they can select a VLAN ID to hand the peering VLAN off on.

Note: we currently have this partially working in our own fork, some work is required to align with master and updated all the associated function (e.g. provisioning YAML output etc.)

@nickhilliard
Copy link
Member

this is under internal discussion. it's not especially easy to implement, but it's fairly high on our wish-list.

@tardoe
Copy link
Author

tardoe commented Apr 6, 2020 via email

@tardoe tardoe linked a pull request Apr 8, 2020 that will close this issue
3 tasks
@tardoe
Copy link
Author

tardoe commented Jul 10, 2020

@nickhilliard any further information about internal discussions about this one?

@tardoe
Copy link
Author

tardoe commented Oct 10, 2020

@nickhilliard any further info about this one? The PR has been submitted but yet to be accepted. Is there any information from your internal discussions I could combine into the PR to get this feature merged?

@nickhilliard
Copy link
Member

not yet unfortunately. We've been working on a framework upgrade (doctrine->eloquent) over the last several months. This has a huge change footprint, so for the moment we're staying away from feature-adds and anything which needs database schema modifications.

@tardoe
Copy link
Author

tardoe commented Oct 12, 2020

Ah okay, this makes sense. Perhaps we can look at this feature as a way to test the benefits of the new framework?

Is there a timeline on a dev cut of the framework migrations?

@nickhilliard
Copy link
Member

Hopefully reasonably soon, but that's a vague answer. @barryo?

@barryo
Copy link
Member

barryo commented Oct 19, 2020

The change is very big and encompasses the entire database layer. I just caught up with @yannrobin there and he's powering through it. In fact most objects are done but the trickier ones are left: auth, users, some of interfaces, patch panels. I think an estmiate of four weeks would be reasonable (but that's not me giving @yannrobin an arbitrary deadline here, just trying to put a reasonable guess on it).

@tardoe
Copy link
Author

tardoe commented Oct 20, 2020

Thanks all, this is very exciting. Let me know if there is anything you need (testing etc.) closer to release time that we can assist with.

@barryo
Copy link
Member

barryo commented Oct 20, 2020

Thanks @tardoe - we'll likely do this as a beta release. We'll have it live at INEX first so others will know it's in production use but the beta tag will reflect the stong possibility of UI bugs, etc. We'll be looking for people to give it a go then. Thanks!

@tardoe
Copy link
Author

tardoe commented Jun 2, 2021

@barryo Any update on this one?

@tardoe
Copy link
Author

tardoe commented Feb 24, 2022

@barryo @nickhilliard any update on this?

@shaneshort
Copy link

shaneshort commented Jul 20, 2022

@tardoe @barryo have you had any progress here? This is something we'd like to implement and we've been waiting patiently for this to be implemented

@lostdave
Copy link

Has there been any movement on this @nickhilliard or @barryo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants