> 3. Hidden dependencies on analytics in OC
>    
>    AFAIK, svc-monitor relies (or used to rely) on analytics data to learn 
>    which vRouters are live and able to receive a service instance. Chances 
>    are, there are other places like this in the code. If we make analytics 
>    pluggable, we must be careful not to break these.
>    
>    SI v1 API has been deprecated for some time, so it's probably as good 
>    moment as any to start discussing how to get rid of it (which means we 
>    need to figure out SNAT, LB and other "base" services that make use of 
>    the fact that contrail can spawn processes/VMs/containers by itself)


Might be going a little bit off-topic here but the choice to depend services on 
“analytics data” IMHO is not a good one.
Contrail-controller knows which vrouters are alive and should be the source of 
truth for anything critical.

Using a service that has a very indirect knowledge about the aliveness of 
vrouters feels very bad to me.
e.g. the node-manager is pushing data about the aliveness that goes through a 
collector, kafka and eventually cassandra. (might not be exactly like this but 
you get the idea)
Now you are going to base decisions on that info via contrail-api getting that 
from cassandra.
A hiccup in any of those systems and bad stuff happens.

E.g we have seen stuff break because analytics was having issues.
There are some bug fixes for that now but e.g. seeing all load-balancer 
instances removed because analytics is giving faulty or no results makes you 
very sad.

TLDR: If this would mean there are no longer any dependencies on analytics that 
would make me very happy.

Cheers,
Robert

_______________________________________________
Dev mailing list
[email protected]
http://lists.opencontrail.org/mailman/listinfo/dev_lists.opencontrail.org

Reply via email to