Fixed ceilometer-agent template

Changed "os_endpoint_type=internal" to
"interface = internal" to support communication
by internal urls.

Signed-off-by: Evgeny Kremenetsky <ekremenetsky@dev.rtsoft.ru>

Change-Id: Icc637f4125e76b28d85b4d253ccf9c6be4bd172c
Closes-Bug: #1700111

templates/mitaka/ceilometer.conf

@@ -9,13 +9,14 @@ debug = {{ debug }}
 verbose = {{ verbose }}
 logdir = /var/log/ceilometer
 
-{% if use_internal_endpoints -%}
-interface = internal
-{% endif -%}
-
 {% if service_host -%}
 [service_credentials]
+{% if use_internal_endpoints -%}
+auth_url = {{ auth_protocol }}://{{ auth_host }}:{{ auth_port }}
+interface = internalURL
+{% else -%}
 auth_url = {{ service_protocol }}://{{ service_host }}:{{ service_port }}
+{% endif -%}
 project_name = {{ admin_tenant_name }}
 username = {{ admin_user }}
 password = {{ admin_password }}

tests/basic_deployment.py

@@ -608,6 +608,77 @@ class CeiloAgentBasicDeployment(OpenStackAmuletDeployment):
 
         u.log.debug('OK')
 
+    def test_303_nova_ceilometer_config_check_endpoint(self):
+        """Verify endpoint option on the
+        nova-compute (ceilometer-agent) unit."""
+        u.log.debug('Checking nova ceilometer config file...')
+        sentry = self.ceil_agent_sentry
+        conf = '/etc/ceilometer/ceilometer.conf'
+
+        expected_default = {
+            'DEFAULT': {
+                'logdir': '/var/log/ceilometer'
+            },
+        }
+
+        expected_alternate = {
+            'DEFAULT': {
+                'logdir': '/var/log/ceilometer'
+            },
+            'service_credentials': {
+                'interface': 'internalURL'
+            },
+        }
+
+        juju_service = 'ceilometer-agent'
+        service = 'ceilometer-polling'
+        option = 'use-internal-endpoints'
+        set_default = {option: 'False'}
+        set_alternate = {option: 'True'}
+
+        for section, pairs in expected_default.iteritems():
+            ret = u.validate_config_data(sentry, conf, section, pairs)
+            if ret:
+                message = "ceilometer config error: {}".format(ret)
+                amulet.raise_status(amulet.FAIL, msg=message)
+
+        sleep_time = 40
+        mtime = u.get_sentry_time(sentry)
+        self.d.configure(juju_service, set_alternate)
+
+        if not u.service_restarted_since(sentry, mtime, service,
+                                         sleep_time=sleep_time,
+                                         retry_count=4, retry_sleep_time=20):
+            self.d.configure(juju_service, set_default)
+            msg = "service {} didn't restart after config change".format(
+                service)
+            amulet.raise_status(amulet.FAIL, msg=msg)
+
+        for section, pairs in expected_alternate.iteritems():
+            ret = u.validate_config_data(sentry, conf, section, pairs)
+            if ret:
+                message = "ceilometer config error: {}".format(ret)
+                amulet.raise_status(amulet.FAIL, msg=message)
+
+        mtime = u.get_sentry_time(sentry)
+        self.d.configure(juju_service, set_default)
+
+        if not u.service_restarted_since(sentry, mtime, service,
+                                         sleep_time=sleep_time,
+                                         retry_count=4, retry_sleep_time=20):
+            self.d.configure(juju_service, set_default)
+            msg = "service {} didn't restart after config change".format(
+                service)
+            amulet.raise_status(amulet.FAIL, msg=msg)
+
+        for section, pairs in expected_default.iteritems():
+            ret = u.validate_config_data(sentry, conf, section, pairs)
+            if ret:
+                message = "ceilometer config error: {}".format(ret)
+                amulet.raise_status(amulet.FAIL, msg=message)
+
+        u.log.debug('OK')
+
     def test_400_api_connection(self):
         """Simple api calls to check service is up and responding"""
         u.log.debug('Checking api functionality...')