Daniel Sheppard
Daniel Sheppard
This is now ready. I stupidly missed an UnsupportedCablePath in the test instead of AbortRequest
Merged in main. Started clean for me either way but hopefully this corrects whatever you are hitting.
> I agree with Jeremy's prior comment that I think we need to validate it before save. Seems like we would want to create the connection graph in memory and...
I think catching that is a separate issue and I think we can wrap that in this PR as it is similar. Let me go add that and see if...
> also, do you have an easier repro scenario for the original cable error? Unfortunately no
@alehaa I am not so sure about the approach, I think something more agnostic that can be called **after** initialization of all apps are done but not specifically tied jobs...
> As a heads up, I'm currently working on a virtual circuit implementation (packaged in core, not as a plugin). Virtual circuits run on physical circuits or provider networks, but...
> We provision most our L2vpn on virtual interfaces (sub-interfaces), they end being interconnected/meshed together through the L2vpn construct. The termination is not meant to be the VTEP, but the...
Was pretty sure I fixed this once already, is this a regression?
Similar: #11478 looks like we (I) was supposed to fix this and went off in a tangent