Comment 0 for bug 1571935

In master branch, CI sanity has failed because of below check-in:
https://review.opencontrail.org/#/c/15523/

This has unearthed an existing bug in SM provisioning which caused service_provider to be set twice in neutron.conf

The reason for this is described below

Neutron DEFAULT config brings in two values for that key as seen below:
# --- Reference implementations ---
service_provider=LOADBALANCER:Haproxy:neutron.services.loadbalancer.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default
service_provider=VPN:openswan:neutron.services.vpn.service_drivers.ipsec.IPsecVPNDriver:default

SM/Puppet uses augeas which sets BOTH of those keys to configured value.

When above check-in got merged it returned ALL values of the key from neutron.conf instead of previously returning a hard-coded value. Since it expects only one value, this causes neutron plugin to fail.

This was handled in fab by first deleting both keys and they setting once.
We are now doing same in SM.