search

kytos-ng / topology

Kytos Main Topology Network Application (NApp)
https://kytos-ng.github.io/api/topology.html
MIT License
0 stars 8 forks source link

Consistency check to try to activate links if both ports are UP #188

Closed viniarck closed 9 months ago

viniarck commented 9 months ago

@italovalcy & @RenataFrez have caught a case in prod where a link was down but interfaces were UP. We're still analyzing and trying to reproduce, but we should also introduce a consistency check in the meantime here to avoid topology inconsistencies for these cases when convergence needs to happen.

Consistency shouldn't be absolutely required in the first place, and we'll fix the root cause in the future, but all things considered, it's worth having a consistency in this part here to try to fix this case if it ever happens, since it can leave actual network paths unavailable when they should be available, and in certain cases this might be critical if there's no other paths. The consistency check is supposed to be very simple and do not cause too much IO, and easy to maintain.

viniarck commented 9 months ago

I'll go ahead and not implement this now, one day we still might, in the meantime only https://github.com/kytos-ng/of_core/pull/133 is expected to land.