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

Dynamic named tunnel VLANs #3314

Closed
cglewis opened this issue Oct 30, 2019 · 0 comments · Fixed by #3476
Closed

Dynamic named tunnel VLANs #3314

cglewis opened this issue Oct 30, 2019 · 0 comments · Fixed by #3476

Comments

@cglewis
Copy link
Member

cglewis commented Oct 30, 2019

Currently it is required to create a tunnel with an unused VLAN. That's great, as it's unlikely a network is going to have completely saturated all VLANs. However, it presents the problem of having to specify one to reserve manually and hope that the operator or application didn't use a VLAN already in use.

Since Faucet will have full knowledge of the VLANs already in use, I propose using unqiue names for the IDs and behind the scenes Faucet picks an unused VLAN, which may change upon config reload (since reloading could introduce new VLANs).

Currently this looks like where the VLAN number is specified:

acls:
  tunnel-to-host1:
    - rule:
      action:
      output:
        tunnel:
          type: 'vlan'
          tunnel_id: 901
          dp: x230
          port: 1
        allow: 1

Or this where the name of the VLAN is specified and defined:

acls:
  tunnel-to-host1:
    - rule:
      action:
      output:
        tunnel:
          type: 'vlan'
          tunnel_id: foo
          dp: x230
          port: 1
        allow: 1
vlans:
  foo:
    vid: 100

What I'm proposing as an option (or replacement, maybe) would be to be able to do this (or something like this):

acls:
  tunnel-to-host1:
    - rule:
      action:
      output:
        tunnel:
          type: 'vlan'
          tunnel_id: foo
          dp: x230
          port: 1
        allow: 1

And behind the scenes Faucet would pick a vid not in use.

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

Successfully merging a pull request may close this issue.

2 participants