Remove unnecessary overrides of service variables

Use the existing aodh namespaced variables instead of overriding and
providing a generic 'service_' variable for use with the service setup
tasks. The existing aodh_service_setup.yml file can be directly replaced
by aodh_service_add.yml.

Change-Id: Ic5fa09d7fd886c4def544c87fa87002c303a4da4

tasks/aodh_service_add.yml

@@ -1,89 +0,0 @@
----
-# Copyright 2015, Rackspace US, Inc.
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-- name: Ensure aodh service
-  keystone:
-    command: "ensure_service"
-    endpoint: "{{ keystone_service_adminurl }}"
-    login_user: "{{ keystone_admin_user_name }}"
-    login_password: "{{ keystone_auth_admin_password }}"
-    login_project_name: "{{ keystone_admin_tenant_name }}"
-    service_name: "{{ service_name }}"
-    service_type: "{{ service_type }}"
-    description: "{{ service_description }}"
-    insecure: "{{ keystone_service_adminuri_insecure }}"
-  register: add_service
-  until: add_service|success
-  retries: 5
-  delay: 2
-
-- name: Ensure aodh user
-  keystone:
-    command: "ensure_user"
-    endpoint: "{{ keystone_service_adminurl }}"
-    login_user: "{{ keystone_admin_user_name }}"
-    login_password: "{{ keystone_auth_admin_password }}"
-    login_project_name: "{{ keystone_admin_tenant_name }}"
-    user_name: "{{ service_user_name }}"
-    tenant_name: "{{ service_tenant_name }}"
-    role_name: "{{ aodh_role_name }}"
-    password: "{{ aodh_service_password }}"
-    insecure: "{{ keystone_service_adminuri_insecure }}"
-  register: add_service
-  when: not aodh_service_in_ldap | bool
-  until: add_service|success
-  retries: 5
-  delay: 10
-
-- name: Ensure aodh user to admin role
-  keystone:
-    command: "ensure_user_role"
-    endpoint: "{{ keystone_service_adminurl }}"
-    login_user: "{{ keystone_admin_user_name }}"
-    login_password: "{{ keystone_auth_admin_password }}"
-    login_project_name: "{{ keystone_admin_tenant_name }}"
-    user_name: "{{ aodh_service_user_name }}"
-    tenant_name: "{{ aodh_service_project_name }}"
-    role_name: "{{ aodh_role_name }}"
-    insecure: "{{ keystone_service_adminuri_insecure }}"
-  register: add_admin_role
-  when: not aodh_service_in_ldap | bool
-  until: add_admin_role|success
-  retries: 5
-  delay: 10
-
-# Create an endpoint
-- name: Ensure aodh endpoint
-  keystone:
-    command: "ensure_endpoint"
-    endpoint: "{{ keystone_service_adminurl }}"
-    login_user: "{{ keystone_admin_user_name }}"
-    login_password: "{{ keystone_auth_admin_password }}"
-    login_project_name: "{{ keystone_admin_tenant_name }}"
-    region_name: "{{ service_region }}"
-    service_name: "{{ service_name }}"
-    service_type: "{{ service_type }}"
-    insecure: "{{ keystone_service_adminuri_insecure }}"
-    endpoint_list:
-      - url: "{{ service_publicurl }}"
-        interface: "public"
-      - url: "{{ service_adminurl }}"
-        interface: "admin"
-      - url: "{{ service_internalurl }}"
-        interface: "internal"
-  register: add_service
-  until: add_service|success
-  retries: 5
-  delay: 10

tasks/aodh_service_setup.yml

@@ -13,15 +13,77 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-- include: aodh_service_add.yml
-  vars:
-    service_user_name: "{{ aodh_service_user_name }}"
-    service_tenant_name: "{{ aodh_service_tenant_name }}"
+- name: Ensure aodh service
+  keystone:
+    command: "ensure_service"
+    endpoint: "{{ keystone_service_adminurl }}"
+    login_user: "{{ keystone_admin_user_name }}"
+    login_password: "{{ keystone_auth_admin_password }}"
+    login_project_name: "{{ keystone_admin_tenant_name }}"
     service_name: "{{ aodh_service_name }}"
     service_type: "{{ aodh_service_type }}"
-    service_region: "{{aodh_service_region }}"
-    service_description: "{{ aodh_service_description }}"
-    service_password: "{{ aodh_service_password }}"
-    service_internalurl: "{{ aodh_service_internalurl }}"
-    service_publicurl: "{{ aodh_service_publicurl }}"
-    service_adminurl: "{{ aodh_service_adminurl }}"
+    description: "{{ aodh_service_description }}"
+    insecure: "{{ keystone_service_adminuri_insecure }}"
+  register: add_service
+  until: add_service|success
+  retries: 5
+  delay: 2
+
+- name: Ensure aodh user
+  keystone:
+    command: "ensure_user"
+    endpoint: "{{ keystone_service_adminurl }}"
+    login_user: "{{ keystone_admin_user_name }}"
+    login_password: "{{ keystone_auth_admin_password }}"
+    login_project_name: "{{ keystone_admin_tenant_name }}"
+    user_name: "{{ aodh_service_user_name }}"
+    tenant_name: "{{ aodh_service_tenant_name }}"
+    role_name: "{{ aodh_role_name }}"
+    password: "{{ aodh_service_password }}"
+    insecure: "{{ keystone_service_adminuri_insecure }}"
+  register: add_service
+  when: not aodh_service_in_ldap | bool
+  until: add_service|success
+  retries: 5
+  delay: 10
+
+- name: Ensure aodh user to admin role
+  keystone:
+    command: "ensure_user_role"
+    endpoint: "{{ keystone_service_adminurl }}"
+    login_user: "{{ keystone_admin_user_name }}"
+    login_password: "{{ keystone_auth_admin_password }}"
+    login_project_name: "{{ keystone_admin_tenant_name }}"
+    user_name: "{{ aodh_service_user_name }}"
+    tenant_name: "{{ aodh_service_project_name }}"
+    role_name: "{{ aodh_role_name }}"
+    insecure: "{{ keystone_service_adminuri_insecure }}"
+  register: add_admin_role
+  when: not aodh_service_in_ldap | bool
+  until: add_admin_role|success
+  retries: 5
+  delay: 10
+
+# Create an endpoint
+- name: Ensure aodh endpoint
+  keystone:
+    command: "ensure_endpoint"
+    endpoint: "{{ keystone_service_adminurl }}"
+    login_user: "{{ keystone_admin_user_name }}"
+    login_password: "{{ keystone_auth_admin_password }}"
+    login_project_name: "{{ keystone_admin_tenant_name }}"
+    region_name: "{{ aodh_service_region }}"
+    service_name: "{{ aodh_service_name }}"
+    service_type: "{{ aodh_service_type }}"
+    insecure: "{{ keystone_service_adminuri_insecure }}"
+    endpoint_list:
+      - url: "{{ aodh_service_publicurl }}"
+        interface: "public"
+      - url: "{{ aodh_service_adminurl }}"
+        interface: "admin"
+      - url: "{{ aodh_service_internalurl }}"
+        interface: "internal"
+  register: add_service
+  until: add_service|success
+  retries: 5
+  delay: 10