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

VLANs API contains all VLANs for every device. #822

Open
ichilton opened this issue Dec 12, 2022 · 1 comment
Open

VLANs API contains all VLANs for every device. #822

ichilton opened this issue Dec 12, 2022 · 1 comment

Comments

@ichilton
Copy link

ISSUE TYPE

Bug Report

VERSION
define( 'APPLICATION_VERSION', '6.3.0' );
define( 'APPLICATION_VERDATE', '2022110200' );
SUMMARY

The provisioning API for VLANs contains all private vlans and not just that should be on each switch.

STEPS TO REPRODUCE

https:///api/v4/provisioner/vlans/switch-name/{switchname}.yaml
...whichever switch you use, it gives the full list of VLANs.

EXPECTED RESULTS

It should only include VLANs which are either not private or are private and have an interface on that switch.

ACTUAL RESULTS

It shows the same list of all VLANs, for every device.

@barryo
Copy link
Member

barryo commented Dec 19, 2022

Hey @ichilton - most IXs using IXP Manager are running layer2 / stp. I.e. every switch needs to know about every VLAN regardless of whether there's an access port there or not.

We'd probably need a new api endpoint or an optional qualifier on this one to limit to access ports.

You could also generate this from other api endpoints such as the list of ports for example.

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

No branches or pull requests

2 participants