osnailyfacter and openstack classes pass around *_address variables using them in different ways

[Andrew Woodward]

though osnailyfacter and openstack classes we have variables for

internal_address
public_address
management_address

in site.pp they are set to this node's IP addresses
in openstack::cluster_ha we re-write them to be the public_vip and management_vip
in openstack::cluster* we gain service_address which could be equal to the management_vip or controller management_address.

in openstack::compute they are used as the same from site.pp

This makes it confusing to follow some of the manifests around.

I propose that we add new parameters in site.pp for
service_internal_address
service_management_address
service_public_address

These should be set to the value of the relevant address depending on usage of ha mode or simple mode.

[Vladimir Kuklin (vkuklin)]

We do not use site.pp anymore and openstack classes