Handle VLAN updates - Githubissues

atlanticwave-sdx / pce

Path Computation Element for AtlanticWave SDX.

0 stars 3 forks source link

Handle VLAN updates #123

Open sajith opened 1 year ago

sajith commented 1 year ago

With PR #120, we will add a way to assign VLAN to connection requests, but it is only an initial sketch. We're missing a design and implementation to handle updates.

This is related to this TODO item:

https://github.com/atlanticwave-sdx/pce/blob/1e19555bdc5b0d6925132ff8c4eaa2c90ddc3917/src/sdx/pce/topology/temanager.py#L94-L99

italovalcy commented 2 weeks ago

Some discussion needs to be done here to better define the use case and expected behavior:

What happens is that the OXP modifies the VLAN range available for SDX, and L2VPN services are already using the removed VLAN.

Should we just ignore that change?
Mark the VLAN as removed and allow existing L2VPN to exist, but not allow new L2VPN service be created with that VLAN? (after freeing up the existing one)?
Remove the existing L2VPN and mark them with error status?

YufengXin commented 2 weeks ago

My opinion is to distinguish between management function and control function: this is a case of management function, which I think should be defined as out of scope for now.