From c7386a8168474ed034e2516ee738759f3d34f582 Mon Sep 17 00:00:00 2001 From: Ghanshyam Mann Date: Sat, 28 Nov 2020 15:52:39 -0600 Subject: [PATCH] Remove retired Searchlight support Searchlight project is retiring in Wallaby cycle[1]. This commit removes the ansible roles of Searchlight project before its code is removed. Needed-By: https://review.opendev.org/c/openstack/searchlight/+/764526 [1] http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018637.html Change-Id: I85aab66376ea4f1376c2705066ba3c7e5645644f --- README.rst | 1 - ansible/group_vars/all.yml | 4 - ansible/inventory/all-in-one | 10 -- ansible/inventory/multinode | 10 -- ansible/roles/barbican/defaults/main.yml | 2 +- ansible/roles/common/defaults/main.yml | 2 - ansible/roles/common/tasks/config.yml | 1 - .../cron-logrotate-searchlight.conf.j2 | 3 - ansible/roles/glance/defaults/main.yml | 2 +- ansible/roles/haproxy/tasks/precheck.yml | 13 -- ansible/roles/horizon/defaults/main.yml | 1 - ansible/roles/horizon/tasks/config.yml | 1 - ansible/roles/neutron/defaults/main.yml | 2 +- ansible/roles/nova-cell/defaults/main.yml | 2 +- .../roles/nova-cell/templates/nova.conf.j2 | 10 +- ansible/roles/nova/defaults/main.yml | 2 +- ansible/roles/nova/templates/nova.conf.j2 | 10 +- ansible/roles/placement/defaults/main.yml | 2 +- ansible/roles/searchlight/defaults/main.yml | 107 ------------- ansible/roles/searchlight/handlers/main.yml | 30 ---- ansible/roles/searchlight/tasks/bootstrap.yml | 2 - .../searchlight/tasks/bootstrap_service.yml | 20 --- .../searchlight/tasks/check-containers.yml | 16 -- ansible/roles/searchlight/tasks/check.yml | 1 - ansible/roles/searchlight/tasks/config.yml | 82 ---------- .../roles/searchlight/tasks/copy-certs.yml | 6 - .../searchlight/tasks/deploy-containers.yml | 2 - ansible/roles/searchlight/tasks/deploy.yml | 11 -- .../roles/searchlight/tasks/loadbalancer.yml | 7 - ansible/roles/searchlight/tasks/main.yml | 2 - ansible/roles/searchlight/tasks/precheck.yml | 24 --- ansible/roles/searchlight/tasks/pull.yml | 11 -- .../roles/searchlight/tasks/reconfigure.yml | 2 - ansible/roles/searchlight/tasks/register.yml | 7 - ansible/roles/searchlight/tasks/stop.yml | 6 - ansible/roles/searchlight/tasks/upgrade.yml | 9 -- .../templates/searchlight-api.json.j2 | 24 --- .../templates/searchlight-listener.json.j2 | 24 --- .../searchlight/templates/searchlight.conf.j2 | 147 ------------------ ansible/site.yml | 18 --- etc/kolla/globals.yml | 2 - etc/kolla/passwords.yml | 2 - ...ve-searchlight-roles-7fde75ff93812b09.yaml | 8 + tests/templates/inventory.j2 | 10 -- 44 files changed, 18 insertions(+), 640 deletions(-) delete mode 100644 ansible/roles/common/templates/cron-logrotate-searchlight.conf.j2 delete mode 100644 ansible/roles/searchlight/defaults/main.yml delete mode 100644 ansible/roles/searchlight/handlers/main.yml delete mode 100644 ansible/roles/searchlight/tasks/bootstrap.yml delete mode 100644 ansible/roles/searchlight/tasks/bootstrap_service.yml delete mode 100644 ansible/roles/searchlight/tasks/check-containers.yml delete mode 100644 ansible/roles/searchlight/tasks/check.yml delete mode 100644 ansible/roles/searchlight/tasks/config.yml delete mode 100644 ansible/roles/searchlight/tasks/copy-certs.yml delete mode 100644 ansible/roles/searchlight/tasks/deploy-containers.yml delete mode 100644 ansible/roles/searchlight/tasks/deploy.yml delete mode 100644 ansible/roles/searchlight/tasks/loadbalancer.yml delete mode 100644 ansible/roles/searchlight/tasks/main.yml delete mode 100644 ansible/roles/searchlight/tasks/precheck.yml delete mode 100644 ansible/roles/searchlight/tasks/pull.yml delete mode 100644 ansible/roles/searchlight/tasks/reconfigure.yml delete mode 100644 ansible/roles/searchlight/tasks/register.yml delete mode 100644 ansible/roles/searchlight/tasks/stop.yml delete mode 100644 ansible/roles/searchlight/tasks/upgrade.yml delete mode 100644 ansible/roles/searchlight/templates/searchlight-api.json.j2 delete mode 100644 ansible/roles/searchlight/templates/searchlight-listener.json.j2 delete mode 100644 ansible/roles/searchlight/templates/searchlight.conf.j2 create mode 100644 releasenotes/notes/remove-searchlight-roles-7fde75ff93812b09.yaml diff --git a/README.rst b/README.rst index b462534bf7..1a9b64294d 100644 --- a/README.rst +++ b/README.rst @@ -69,7 +69,6 @@ Kolla-Ansible deploys containers for the following OpenStack projects: - `Qinling `__ - `Rally `__ - `Sahara `__ -- `Searchlight `__ - `Senlin `__ - `Solum `__ - `Swift `__ diff --git a/ansible/group_vars/all.yml b/ansible/group_vars/all.yml index e0eb0d0efc..ccdc98ff91 100644 --- a/ansible/group_vars/all.yml +++ b/ansible/group_vars/all.yml @@ -458,8 +458,6 @@ redis_sentinel_port: "26379" sahara_api_port: "8386" -searchlight_api_port: "9393" - senlin_internal_fqdn: "{{ kolla_internal_fqdn }}" senlin_external_fqdn: "{{ kolla_external_fqdn }}" senlin_api_port: "8778" @@ -619,7 +617,6 @@ enable_horizon_neutron_vpnaas: "{{ enable_neutron_vpnaas | bool }}" enable_horizon_octavia: "{{ enable_octavia | bool }}" enable_horizon_qinling: "{{ enable_qinling | bool }}" enable_horizon_sahara: "{{ enable_sahara | bool }}" -enable_horizon_searchlight: "{{ enable_searchlight | bool }}" enable_horizon_senlin: "{{ enable_senlin | bool }}" enable_horizon_solum: "{{ enable_solum | bool }}" enable_horizon_tacker: "{{ enable_tacker | bool }}" @@ -678,7 +675,6 @@ enable_qinling: "no" enable_rally: "no" enable_redis: "no" enable_sahara: "no" -enable_searchlight: "no" enable_senlin: "no" enable_skydive: "no" enable_solum: "no" diff --git a/ansible/inventory/all-in-one b/ansible/inventory/all-in-one index 28dd2fda4e..1dd2cd3e41 100644 --- a/ansible/inventory/all-in-one +++ b/ansible/inventory/all-in-one @@ -212,9 +212,6 @@ control [rally:children] control -[searchlight:children] -control - [octavia:children] control @@ -620,13 +617,6 @@ senlin [senlin-health-manager:children] senlin -# Searchlight -[searchlight-api:children] -searchlight - -[searchlight-listener:children] -searchlight - # Octavia [octavia-api:children] octavia diff --git a/ansible/inventory/multinode b/ansible/inventory/multinode index d2e11d9091..a9bede9755 100644 --- a/ansible/inventory/multinode +++ b/ansible/inventory/multinode @@ -230,9 +230,6 @@ control [rally:children] control -[searchlight:children] -control - [octavia:children] control @@ -638,13 +635,6 @@ senlin [senlin-health-manager:children] senlin -# Searchlight -[searchlight-api:children] -searchlight - -[searchlight-listener:children] -searchlight - # Octavia [octavia-api:children] octavia diff --git a/ansible/roles/barbican/defaults/main.yml b/ansible/roles/barbican/defaults/main.yml index ccdf4d7a9f..491d9c3fe5 100644 --- a/ansible/roles/barbican/defaults/main.yml +++ b/ansible/roles/barbican/defaults/main.yml @@ -146,7 +146,7 @@ barbican_ks_roles: #################### barbican_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool }}" + enabled: "{{ enable_ceilometer | bool }}" barbican_enabled_notification_topics: "{{ barbican_notification_topics | selectattr('enabled', 'equalto', true) | list }}" diff --git a/ansible/roles/common/defaults/main.yml b/ansible/roles/common/defaults/main.yml index f59c78a586..3da29c7c8a 100644 --- a/ansible/roles/common/defaults/main.yml +++ b/ansible/roles/common/defaults/main.yml @@ -168,8 +168,6 @@ fluentd_input_openstack_services: enabled: "{{ enable_rally | bool }}" - name: sahara enabled: "{{ enable_sahara | bool }}" - - name: searchlight - enabled: "{{ enable_searchlight | bool }}" - name: senlin enabled: "{{ enable_senlin | bool }}" - name: solum diff --git a/ansible/roles/common/tasks/config.yml b/ansible/roles/common/tasks/config.yml index 1d98c74ef5..3ae0ef9a84 100644 --- a/ansible/roles/common/tasks/config.yml +++ b/ansible/roles/common/tasks/config.yml @@ -213,7 +213,6 @@ - { name: "rabbitmq", enabled: "{{ enable_rabbitmq | bool }}" } - { name: "rally", enabled: "{{ enable_rally | bool }}" } - { name: "sahara", enabled: "{{ enable_sahara | bool }}" } - - { name: "searchlight", enabled: "{{ enable_searchlight | bool }}" } - { name: "senlin", enabled: "{{ enable_senlin | bool }}" } - { name: "skydive", enabled: "{{ enable_skydive | bool }}" } - { name: "solum", enabled: "{{ enable_solum | bool }}" } diff --git a/ansible/roles/common/templates/cron-logrotate-searchlight.conf.j2 b/ansible/roles/common/templates/cron-logrotate-searchlight.conf.j2 deleted file mode 100644 index da916664ae..0000000000 --- a/ansible/roles/common/templates/cron-logrotate-searchlight.conf.j2 +++ /dev/null @@ -1,3 +0,0 @@ -"/var/log/kolla/searchlight/*.log" -{ -} diff --git a/ansible/roles/glance/defaults/main.yml b/ansible/roles/glance/defaults/main.yml index 0e6191ba9b..9bd0984347 100644 --- a/ansible/roles/glance/defaults/main.yml +++ b/ansible/roles/glance/defaults/main.yml @@ -96,7 +96,7 @@ glance_ks_users: #################### glance_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool }}" + enabled: "{{ enable_ceilometer | bool }}" glance_enabled_notification_topics: "{{ glance_notification_topics | selectattr('enabled', 'equalto', true) | list }}" diff --git a/ansible/roles/haproxy/tasks/precheck.yml b/ansible/roles/haproxy/tasks/precheck.yml index 0e79dd44af..19940a0577 100644 --- a/ansible/roles/haproxy/tasks/precheck.yml +++ b/ansible/roles/haproxy/tasks/precheck.yml @@ -780,19 +780,6 @@ - haproxy_stat.find('sahara_api') == -1 - haproxy_vip_prechecks -- name: Checking free port for Searchlight API HAProxy - wait_for: - host: "{{ kolla_internal_vip_address }}" - port: "{{ searchlight_api_port }}" - connect_timeout: 1 - timeout: 1 - state: stopped - when: - - enable_searchlight | bool - - inventory_hostname in groups['haproxy'] - - haproxy_stat.find('searchlight_api') == -1 - - haproxy_vip_prechecks - - name: Checking free port for Senlin API HAProxy wait_for: host: "{{ kolla_internal_vip_address }}" diff --git a/ansible/roles/horizon/defaults/main.yml b/ansible/roles/horizon/defaults/main.yml index 6876cc37fa..ba74ab417c 100644 --- a/ansible/roles/horizon/defaults/main.yml +++ b/ansible/roles/horizon/defaults/main.yml @@ -25,7 +25,6 @@ horizon_services: ENABLE_OCTAVIA: "{{ 'yes' if enable_horizon_octavia | bool else 'no' }}" ENABLE_QINLING: "{{ 'yes' if enable_horizon_qinling | bool else 'no' }}" ENABLE_SAHARA: "{{ 'yes' if enable_horizon_sahara | bool else 'no' }}" - ENABLE_SEARCHLIGHT: "{{ 'yes' if enable_horizon_searchlight | bool else 'no' }}" ENABLE_SENLIN: "{{ 'yes' if enable_horizon_senlin | bool else 'no' }}" ENABLE_SOLUM: "{{ 'yes' if enable_horizon_solum | bool else 'no' }}" ENABLE_TACKER: "{{ 'yes' if enable_horizon_tacker | bool else 'no' }}" diff --git a/ansible/roles/horizon/tasks/config.yml b/ansible/roles/horizon/tasks/config.yml index 1f5e8ff45f..d8316e1e23 100644 --- a/ansible/roles/horizon/tasks/config.yml +++ b/ansible/roles/horizon/tasks/config.yml @@ -42,7 +42,6 @@ - { name: "octavia", enabled: "{{ enable_horizon_octavia }}" } - { name: "qinling", enabled: "{{ enable_horizon_qinling }}" } - { name: "sahara", enabled: "{{ enable_horizon_sahara }}" } - - { name: "searchlight", enabled: "{{ enable_horizon_searchlight }}" } - { name: "senlin", enabled: "{{ enable_horizon_senlin }}" } - { name: "solum", enabled: "{{ enable_horizon_solum }}" } - { name: "tacker", enabled: "{{ enable_horizon_tacker }}" } diff --git a/ansible/roles/neutron/defaults/main.yml b/ansible/roles/neutron/defaults/main.yml index f449844f57..7e46c4326e 100644 --- a/ansible/roles/neutron/defaults/main.yml +++ b/ansible/roles/neutron/defaults/main.yml @@ -636,7 +636,7 @@ neutron_service_plugins: "{{ service_plugins | selectattr('enabled', 'equalto', #################### neutron_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool or enable_neutron_infoblox_ipam_agent | bool }}" + enabled: "{{ enable_ceilometer | bool or enable_neutron_infoblox_ipam_agent | bool }}" - name: notifications_designate enabled: "{{ enable_designate | bool }}" - name: vitrage_notifications diff --git a/ansible/roles/nova-cell/defaults/main.yml b/ansible/roles/nova-cell/defaults/main.yml index 0060ffb5f7..0385ea2017 100644 --- a/ansible/roles/nova-cell/defaults/main.yml +++ b/ansible/roles/nova-cell/defaults/main.yml @@ -467,7 +467,7 @@ nova_cell_conductor_has_api_database: "yes" #################### nova_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool or enable_neutron_infoblox_ipam_agent | bool }}" + enabled: "{{ enable_ceilometer | bool or enable_neutron_infoblox_ipam_agent | bool }}" - name: notifications_designate enabled: "{{ enable_designate | bool }}" - name: vitrage_notifications diff --git a/ansible/roles/nova-cell/templates/nova.conf.j2 b/ansible/roles/nova-cell/templates/nova.conf.j2 index 81852d5e70..78abab8538 100644 --- a/ansible/roles/nova-cell/templates/nova.conf.j2 +++ b/ansible/roles/nova-cell/templates/nova.conf.j2 @@ -26,7 +26,7 @@ compute_driver = libvirt.LibvirtDriver # Though my_ip is not used directly, lots of other variables use $my_ip my_ip = {{ api_interface_address }} -{% if enable_ceilometer | bool or enable_searchlight | bool or enable_designate | bool %} +{% if enable_ceilometer | bool or enable_designate | bool %} instance_usage_audit = True instance_usage_audit_period = hour {% if enable_watcher | bool %} @@ -217,15 +217,9 @@ cafile = {{ openstack_cacert }} valid_interfaces = internal [notifications] -{% if enable_ceilometer | bool or enable_searchlight | bool or enable_designate | bool or enable_neutron_infoblox_ipam_agent | bool %} +{% if enable_ceilometer | bool or enable_designate | bool or enable_neutron_infoblox_ipam_agent | bool %} notify_on_state_change = vm_and_task_state {% endif %} -{% if not enable_searchlight | bool %} -notification_format = unversioned -{% else %} -notification_format = both -{% endif %} - {% if enable_osprofiler | bool %} [profiler] diff --git a/ansible/roles/nova/defaults/main.yml b/ansible/roles/nova/defaults/main.yml index 55fc167004..dd15f220db 100644 --- a/ansible/roles/nova/defaults/main.yml +++ b/ansible/roles/nova/defaults/main.yml @@ -224,7 +224,7 @@ nova_ks_users: #################### nova_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool or enable_neutron_infoblox_ipam_agent | bool }}" + enabled: "{{ enable_ceilometer | bool or enable_neutron_infoblox_ipam_agent | bool }}" - name: notifications_designate enabled: "{{ enable_designate | bool }}" - name: vitrage_notifications diff --git a/ansible/roles/nova/templates/nova.conf.j2 b/ansible/roles/nova/templates/nova.conf.j2 index 2795c14759..74540fdac4 100644 --- a/ansible/roles/nova/templates/nova.conf.j2 +++ b/ansible/roles/nova/templates/nova.conf.j2 @@ -13,7 +13,7 @@ allow_resize_to_same_host = true # Though my_ip is not used directly, lots of other variables use $my_ip my_ip = {{ api_interface_address }} -{% if enable_ceilometer | bool or enable_searchlight | bool or enable_designate | bool %} +{% if enable_ceilometer | bool or enable_designate | bool %} instance_usage_audit = True instance_usage_audit_period = hour {% endif %} @@ -176,15 +176,9 @@ cafile = {{ openstack_cacert }} valid_interfaces = internal [notifications] -{% if enable_ceilometer | bool or enable_searchlight | bool or enable_designate | bool or enable_neutron_infoblox_ipam_agent | bool %} +{% if enable_ceilometer | bool or enable_designate | bool or enable_neutron_infoblox_ipam_agent | bool %} notify_on_state_change = vm_and_task_state {% endif %} -{% if not enable_searchlight | bool %} -notification_format = unversioned -{% else %} -notification_format = both -{% endif %} - {% if enable_osprofiler | bool %} [profiler] diff --git a/ansible/roles/placement/defaults/main.yml b/ansible/roles/placement/defaults/main.yml index 1bcfabdb1a..5a88570666 100644 --- a/ansible/roles/placement/defaults/main.yml +++ b/ansible/roles/placement/defaults/main.yml @@ -84,7 +84,7 @@ openstack_placement_auth: "{{ openstack_auth }}" #################### placement_notification_topics: - name: notifications - enabled: "{{ enable_ceilometer | bool or enable_searchlight | bool or enable_neutron_infoblox_ipam_agent | bool }}" + enabled: "{{ enable_ceilometer | bool or enable_neutron_infoblox_ipam_agent | bool }}" - name: notifications_designate enabled: "{{ enable_designate | bool }}" diff --git a/ansible/roles/searchlight/defaults/main.yml b/ansible/roles/searchlight/defaults/main.yml deleted file mode 100644 index b09411b9f7..0000000000 --- a/ansible/roles/searchlight/defaults/main.yml +++ /dev/null @@ -1,107 +0,0 @@ ---- -project_name: "searchlight" - -searchlight_services: - searchlight-api: - container_name: searchlight_api - group: searchlight-api - enabled: true - image: "{{ searchlight_api_image_full }}" - volumes: "{{ searchlight_api_default_volumes + searchlight_api_extra_volumes }}" - dimensions: "{{ searchlight_api_dimensions }}" - haproxy: - searchlight_api: - enabled: "{{ enable_searchlight }}" - mode: "http" - external: false - port: "{{ searchlight_api_port }}" - searchlight_api_external: - enabled: "{{ enable_searchlight }}" - mode: "http" - external: true - port: "{{ searchlight_api_port }}" - searchlight-listener: - container_name: searchlight_listener - group: searchlight-listener - enabled: true - image: "{{ searchlight_listener_image_full }}" - volumes: "{{ searchlight_listener_default_volumes + searchlight_listener_extra_volumes }}" - dimensions: "{{ searchlight_listener_dimensions }}" - -#################### -# Elasticsearch -#################### -searchlight_elasticsearch_url: "{{ elasticsearch_address | put_address_in_context('url') }}:{{ elasticsearch_port }}" - -#################### -# Docker -#################### -searchlight_install_type: "{{ kolla_install_type }}" -searchlight_tag: "{{ openstack_tag }}" - -searchlight_listener_image: "{{ docker_registry ~ '/' if docker_registry else '' }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ searchlight_install_type }}-searchlight-listener" -searchlight_listener_tag: "{{ searchlight_tag }}" -searchlight_listener_image_full: "{{ searchlight_listener_image }}:{{ searchlight_listener_tag }}" - -searchlight_api_image: "{{ docker_registry ~ '/' if docker_registry else '' }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ searchlight_install_type }}-searchlight-api" -searchlight_api_tag: "{{ searchlight_tag }}" -searchlight_api_image_full: "{{ searchlight_api_image }}:{{ searchlight_api_tag }}" - -searchlight_api_dimensions: "{{ default_container_dimensions }}" -searchlight_listener_dimensions: "{{ default_container_dimensions }}" - -searchlight_api_default_volumes: - - "{{ node_config_directory }}/searchlight-api/:{{ container_config_directory }}/:ro" - - "/etc/localtime:/etc/localtime:ro" - - "{{ '/etc/timezone:/etc/timezone:ro' if ansible_os_family == 'Debian' else '' }}" - - "kolla_logs:/var/log/kolla/" -searchlight_listener_default_volumes: - - "{{ node_config_directory }}/searchlight-listener/:{{ container_config_directory }}/:ro" - - "/etc/localtime:/etc/localtime:ro" - - "{{ '/etc/timezone:/etc/timezone:ro' if ansible_os_family == 'Debian' else '' }}" - - "kolla_logs:/var/log/kolla/" - -searchlight_extra_volumes: "{{ default_extra_volumes }}" -searchlight_api_extra_volumes: "{{ searchlight_extra_volumes }}" -searchlight_listener_extra_volumes: "{{ searchlight_extra_volumes }}" - -#################### -# OpenStack -#################### -searchlight_admin_endpoint: "{{ admin_protocol }}://{{ kolla_internal_fqdn | put_address_in_context('url') }}:{{ searchlight_api_port }}" -searchlight_internal_endpoint: "{{ internal_protocol }}://{{ kolla_internal_fqdn | put_address_in_context('url') }}:{{ searchlight_api_port }}" -searchlight_public_endpoint: "{{ public_protocol }}://{{ kolla_external_fqdn | put_address_in_context('url') }}:{{ searchlight_api_port }}" - -searchlight_logging_debug: "{{ openstack_logging_debug }}" - -searchlight_keystone_user: "searchlight" - -openstack_searchlight_auth: "{{ openstack_auth }}" - - -#################### -# Notifications -#################### -searchlight_notification_topics: - - name: notifications - enabled: "{{ enable_ceilometer | bool }}" - -searchlight_enabled_notification_topics: "{{ searchlight_notification_topics | selectattr('enabled', 'equalto', true) | list }}" - -#################### -# Keystone -#################### -searchlight_ks_services: - - name: "searchlight" - type: "search" - description: "Openstack Index Service" - endpoints: - - {'interface': 'admin', 'url': '{{ searchlight_admin_endpoint }}'} - - {'interface': 'internal', 'url': '{{ searchlight_internal_endpoint }}'} - - {'interface': 'public', 'url': '{{ searchlight_public_endpoint }}'} - -searchlight_ks_users: - - project: "service" - user: "{{ searchlight_keystone_user }}" - password: "{{ searchlight_keystone_password }}" - role: "admin" diff --git a/ansible/roles/searchlight/handlers/main.yml b/ansible/roles/searchlight/handlers/main.yml deleted file mode 100644 index dc94797b67..0000000000 --- a/ansible/roles/searchlight/handlers/main.yml +++ /dev/null @@ -1,30 +0,0 @@ ---- -- name: Restart searchlight-api container - vars: - service_name: "searchlight-api" - service: "{{ searchlight_services[service_name] }}" - become: true - kolla_docker: - action: "recreate_or_restart_container" - common_options: "{{ docker_common_options }}" - name: "{{ service.container_name }}" - image: "{{ service.image }}" - volumes: "{{ service.volumes }}" - dimensions: "{{ service.dimensions }}" - when: - - kolla_action != "config" - -- name: Restart searchlight-listener container - vars: - service_name: "searchlight-listener" - service: "{{ searchlight_services[service_name] }}" - become: true - kolla_docker: - action: "recreate_or_restart_container" - common_options: "{{ docker_common_options }}" - name: "{{ service.container_name }}" - image: "{{ service.image }}" - volumes: "{{ service.volumes }}" - dimensions: "{{ service.dimensions }}" - when: - - kolla_action != "config" diff --git a/ansible/roles/searchlight/tasks/bootstrap.yml b/ansible/roles/searchlight/tasks/bootstrap.yml deleted file mode 100644 index 2115ef6239..0000000000 --- a/ansible/roles/searchlight/tasks/bootstrap.yml +++ /dev/null @@ -1,2 +0,0 @@ ---- -- import_tasks: bootstrap_service.yml diff --git a/ansible/roles/searchlight/tasks/bootstrap_service.yml b/ansible/roles/searchlight/tasks/bootstrap_service.yml deleted file mode 100644 index 53c44a3838..0000000000 --- a/ansible/roles/searchlight/tasks/bootstrap_service.yml +++ /dev/null @@ -1,20 +0,0 @@ ---- -- name: Running Searchlight bootstrap container - vars: - searchlight_api: "{{ searchlight_services['searchlight-api'] }}" - become: true - kolla_docker: - action: "start_container" - common_options: "{{ docker_common_options }}" - detach: False - environment: - KOLLA_BOOTSTRAP: - KOLLA_CONFIG_STRATEGY: "{{ config_strategy }}" - image: "{{ searchlight_api.image }}" - labels: - BOOTSTRAP: - name: "bootstrap_searchlight" - restart_policy: no - volumes: "{{ searchlight_api.volumes }}" - run_once: True - delegate_to: "{{ groups[searchlight_api.group][0] }}" diff --git a/ansible/roles/searchlight/tasks/check-containers.yml b/ansible/roles/searchlight/tasks/check-containers.yml deleted file mode 100644 index c544b70dde..0000000000 --- a/ansible/roles/searchlight/tasks/check-containers.yml +++ /dev/null @@ -1,16 +0,0 @@ ---- -- name: Check searchlight containers - become: true - kolla_docker: - action: "compare_container" - common_options: "{{ docker_common_options }}" - name: "{{ item.value.container_name }}" - image: "{{ item.value.image }}" - volumes: "{{ item.value.volumes }}" - dimensions: "{{ item.value.dimensions }}" - when: - - inventory_hostname in groups[item.value.group] - - item.value.enabled | bool - with_dict: "{{ searchlight_services }}" - notify: - - "Restart {{ item.key }} container" diff --git a/ansible/roles/searchlight/tasks/check.yml b/ansible/roles/searchlight/tasks/check.yml deleted file mode 100644 index ed97d539c0..0000000000 --- a/ansible/roles/searchlight/tasks/check.yml +++ /dev/null @@ -1 +0,0 @@ ---- diff --git a/ansible/roles/searchlight/tasks/config.yml b/ansible/roles/searchlight/tasks/config.yml deleted file mode 100644 index 5b661eb6d7..0000000000 --- a/ansible/roles/searchlight/tasks/config.yml +++ /dev/null @@ -1,82 +0,0 @@ ---- -- name: Ensuring config directories exist - file: - path: "{{ node_config_directory }}/{{ item.key }}" - state: "directory" - owner: "{{ config_owner_user }}" - group: "{{ config_owner_group }}" - mode: "0770" - become: true - when: - - inventory_hostname in groups[item.value.group] - - item.value.enabled | bool - with_dict: "{{ searchlight_services }}" - -- name: Check if policies shall be overwritten - stat: - path: "{{ item }}" - delegate_to: localhost - run_once: True - register: searchlight_policy - with_first_found: - - files: "{{ supported_policy_format_list }}" - paths: - - "{{ node_custom_config }}/searchlight/" - skip: true - -- name: Set searchlight policy file - set_fact: - searchlight_policy_file: "{{ searchlight_policy.results.0.stat.path | basename }}" - searchlight_policy_file_path: "{{ searchlight_policy.results.0.stat.path }}" - when: - - searchlight_policy.results - -- include_tasks: copy-certs.yml - when: - - kolla_copy_ca_into_containers | bool - -- name: Copying over config.json files for services - template: - src: "{{ item.key }}.json.j2" - dest: "{{ node_config_directory }}/{{ item.key }}/config.json" - mode: "0660" - become: true - when: - - item.value.enabled | bool - - inventory_hostname in groups[item.value.group] - with_dict: "{{ searchlight_services }}" - notify: - - Restart {{ item.key }} container - -- name: Copying over searchlight.conf - vars: - service_name: "{{ item.key }}" - merge_configs: - sources: - - "{{ role_path }}/templates/searchlight.conf.j2" - - "{{ node_custom_config }}/global.conf" - - "{{ node_custom_config }}/searchlight.conf" - - "{{ node_custom_config }}/searchlight/{{ inventory_hostname }}/searchlight.conf" - dest: "{{ node_config_directory }}/{{ item.key }}/searchlight.conf" - mode: "0660" - become: true - when: - - item.value.enabled | bool - - inventory_hostname in groups[item.value.group] - with_dict: "{{ searchlight_services }}" - notify: - - Restart {{ item.key }} container - -- name: Copying over existing policy file - template: - src: "{{ searchlight_policy_file_path }}" - dest: "{{ node_config_directory }}/{{ item.key }}/{{ searchlight_policy_file }}" - mode: "0660" - become: true - when: - - searchlight_policy_file is defined - - inventory_hostname in groups[item.value.group] - - item.value.enabled | bool - with_dict: "{{ searchlight_services }}" - notify: - - Restart {{ item.key }} container diff --git a/ansible/roles/searchlight/tasks/copy-certs.yml b/ansible/roles/searchlight/tasks/copy-certs.yml deleted file mode 100644 index 39d605bcd2..0000000000 --- a/ansible/roles/searchlight/tasks/copy-certs.yml +++ /dev/null @@ -1,6 +0,0 @@ ---- -- name: "Copy certificates and keys for {{ project_name }}" - import_role: - role: service-cert-copy - vars: - project_services: "{{ searchlight_services }}" diff --git a/ansible/roles/searchlight/tasks/deploy-containers.yml b/ansible/roles/searchlight/tasks/deploy-containers.yml deleted file mode 100644 index eb24ab5c7a..0000000000 --- a/ansible/roles/searchlight/tasks/deploy-containers.yml +++ /dev/null @@ -1,2 +0,0 @@ ---- -- import_tasks: check-containers.yml diff --git a/ansible/roles/searchlight/tasks/deploy.yml b/ansible/roles/searchlight/tasks/deploy.yml deleted file mode 100644 index d793a349da..0000000000 --- a/ansible/roles/searchlight/tasks/deploy.yml +++ /dev/null @@ -1,11 +0,0 @@ ---- -- import_tasks: register.yml - -- import_tasks: config.yml - -- import_tasks: check-containers.yml - -- import_tasks: bootstrap.yml - -- name: Flush handlers - meta: flush_handlers diff --git a/ansible/roles/searchlight/tasks/loadbalancer.yml b/ansible/roles/searchlight/tasks/loadbalancer.yml deleted file mode 100644 index 5f38f2a068..0000000000 --- a/ansible/roles/searchlight/tasks/loadbalancer.yml +++ /dev/null @@ -1,7 +0,0 @@ ---- -- name: "Configure haproxy for {{ project_name }}" - import_role: - name: haproxy-config - vars: - project_services: "{{ searchlight_services }}" - tags: always diff --git a/ansible/roles/searchlight/tasks/main.yml b/ansible/roles/searchlight/tasks/main.yml deleted file mode 100644 index bc5d1e6257..0000000000 --- a/ansible/roles/searchlight/tasks/main.yml +++ /dev/null @@ -1,2 +0,0 @@ ---- -- include_tasks: "{{ kolla_action }}.yml" diff --git a/ansible/roles/searchlight/tasks/precheck.yml b/ansible/roles/searchlight/tasks/precheck.yml deleted file mode 100644 index fd1a2f2da5..0000000000 --- a/ansible/roles/searchlight/tasks/precheck.yml +++ /dev/null @@ -1,24 +0,0 @@ ---- -- import_role: - name: service-precheck - vars: - service_precheck_services: "{{ searchlight_services }}" - service_name: "{{ project_name }}" - -- name: Get container facts - become: true - kolla_container_facts: - name: - - searchlight_api - register: container_facts - -- name: Checking free port for Searchlight API - wait_for: - host: "{{ api_interface_address }}" - port: "{{ searchlight_api_port }}" - connect_timeout: 1 - timeout: 1 - state: stopped - when: - - container_facts['searchlight_api'] is not defined - - inventory_hostname in groups['searchlight-api'] diff --git a/ansible/roles/searchlight/tasks/pull.yml b/ansible/roles/searchlight/tasks/pull.yml deleted file mode 100644 index 6cfb58d99c..0000000000 --- a/ansible/roles/searchlight/tasks/pull.yml +++ /dev/null @@ -1,11 +0,0 @@ ---- -- name: Pulling searchlight images - become: true - kolla_docker: - action: "pull_image" - common_options: "{{ docker_common_options }}" - image: "{{ item.value.image }}" - when: - - inventory_hostname in groups[item.value.group] - - item.value.enabled | bool - with_dict: "{{ searchlight_services }}" diff --git a/ansible/roles/searchlight/tasks/reconfigure.yml b/ansible/roles/searchlight/tasks/reconfigure.yml deleted file mode 100644 index 5b10a7e111..0000000000 --- a/ansible/roles/searchlight/tasks/reconfigure.yml +++ /dev/null @@ -1,2 +0,0 @@ ---- -- import_tasks: deploy.yml diff --git a/ansible/roles/searchlight/tasks/register.yml b/ansible/roles/searchlight/tasks/register.yml deleted file mode 100644 index 6750024289..0000000000 --- a/ansible/roles/searchlight/tasks/register.yml +++ /dev/null @@ -1,7 +0,0 @@ ---- -- import_role: - name: service-ks-register - vars: - service_ks_register_auth: "{{ openstack_searchlight_auth }}" - service_ks_register_services: "{{ searchlight_ks_services }}" - service_ks_register_users: "{{ searchlight_ks_users }}" diff --git a/ansible/roles/searchlight/tasks/stop.yml b/ansible/roles/searchlight/tasks/stop.yml deleted file mode 100644 index 7ad29acfd8..0000000000 --- a/ansible/roles/searchlight/tasks/stop.yml +++ /dev/null @@ -1,6 +0,0 @@ ---- -- import_role: - name: service-stop - vars: - project_services: "{{ searchlight_services }}" - service_name: "{{ project_name }}" diff --git a/ansible/roles/searchlight/tasks/upgrade.yml b/ansible/roles/searchlight/tasks/upgrade.yml deleted file mode 100644 index 6ba9f99799..0000000000 --- a/ansible/roles/searchlight/tasks/upgrade.yml +++ /dev/null @@ -1,9 +0,0 @@ ---- -- import_tasks: config.yml - -- import_tasks: check-containers.yml - -- import_tasks: bootstrap_service.yml - -- name: Flush handlers - meta: flush_handlers diff --git a/ansible/roles/searchlight/templates/searchlight-api.json.j2 b/ansible/roles/searchlight/templates/searchlight-api.json.j2 deleted file mode 100644 index 4dc10528e5..0000000000 --- a/ansible/roles/searchlight/templates/searchlight-api.json.j2 +++ /dev/null @@ -1,24 +0,0 @@ -{ - "command": "searchlight-api", - "config_files": [ - { - "source": "{{ container_config_directory }}/searchlight.conf", - "dest": "/etc/searchlight/searchlight.conf", - "owner": "searchlight", - "perm": "0600" - }{% if searchlight_policy_file is defined %}, - { - "source": "{{ container_config_directory }}/{{ searchlight_policy_file }}", - "dest": "/etc/searchlight/{{ searchlight_policy_file }}", - "owner": "searchlight", - "perm": "0600" - }{% endif %} - ], - "permissions": [ - { - "path": "/var/log/kolla/searchlight", - "owner": "searchlight:searchlight", - "recurse": true - } - ] -} diff --git a/ansible/roles/searchlight/templates/searchlight-listener.json.j2 b/ansible/roles/searchlight/templates/searchlight-listener.json.j2 deleted file mode 100644 index 88a631a15d..0000000000 --- a/ansible/roles/searchlight/templates/searchlight-listener.json.j2 +++ /dev/null @@ -1,24 +0,0 @@ -{ - "command": "searchlight-listener", - "config_files": [ - { - "source": "{{ container_config_directory }}/searchlight.conf", - "dest": "/etc/searchlight/searchlight.conf", - "owner": "searchlight", - "perm": "0600" - }{% if searchlight_policy_file is defined %}, - { - "source": "{{ container_config_directory }}/{{ searchlight_policy_file }}", - "dest": "/etc/searchlight/{{ searchlight_policy_file }}", - "owner": "searchlight", - "perm": "0600" - }{% endif %} - ], - "permissions": [ - { - "path": "/var/log/kolla/searchlight", - "owner": "searchlight:searchlight", - "recurse": true - } - ] -} diff --git a/ansible/roles/searchlight/templates/searchlight.conf.j2 b/ansible/roles/searchlight/templates/searchlight.conf.j2 deleted file mode 100644 index 1c51fbcc61..0000000000 --- a/ansible/roles/searchlight/templates/searchlight.conf.j2 +++ /dev/null @@ -1,147 +0,0 @@ -[DEFAULT] -debug = {{ searchlight_logging_debug }} - -log_dir = /var/log/kolla/searchlight -transport_url = {{ rpc_transport_url }} - -[api] -port = {{ searchlight_api_port }} -bind_host = {{ api_interface_address }} -public_endpoint = {{ searchlight_public_endpoint }} -workers = {{ openstack_service_workers }} - -[elasticsearch] -hosts = {{ searchlight_elasticsearch_url }} - -[listener] -workers = {{ openstack_service_workers }} -notifications_pool = searchlight-listener - -[paste_deploy] -flavor = keystone - -[keystone_authtoken] -www_authenticate_uri = {{ keystone_internal_url }} -auth_url = {{ keystone_admin_url }} -project_domain_name = {{ default_project_domain_name }} -project_name = service -user_domain_name = {{ default_user_domain_name }} -username = {{ searchlight_keystone_user }} -password = {{ searchlight_keystone_password }} -auth_type = password -cafile = {{ openstack_cacert }} - -memcache_security_strategy = ENCRYPT -memcache_secret_key = {{ memcache_secret_key }} -memcached_servers = {% for host in groups['memcached'] %}{{ 'api' | kolla_address(host) | put_address_in_context('memcache') }}:{{ memcached_port }}{% if not loop.last %},{% endif %}{% endfor %} - -[oslo_messaging_notifications] -transport_url = {{ notify_transport_url }} -{% if searchlight_enabled_notification_topics %} -driver = messagingv2 -topics = {{ searchlight_enabled_notification_topics | map(attribute='name') | join(',') }} -{% else %} -driver = noop -{% endif %} - -{% if om_enable_rabbitmq_tls | bool %} -[oslo_messaging_rabbit] -ssl = true -ssl_ca_file = {{ om_rabbitmq_cacert }} -{% endif %} - -{% if searchlight_policy_file is defined %} -[oslo_policy] -policy_file = {{ searchlight_policy_file }} -{% endif %} - -[service_credentials] -auth_uri = {{ keystone_internal_url }} -auth_url = {{ keystone_admin_url }} -region_name = {{ openstack_region_name }} -project_domain_name = default -project_name = service -user_domain_name = default -username = {{ searchlight_keystone_user }} -password = {{ searchlight_keystone_password }} -auth_type = password -auth_plugin = password -cafile = {{ openstack_cacert }} - -memcache_security_strategy = ENCRYPT -memcache_secret_key = {{ memcache_secret_key }} -memcached_servers = {% for host in groups['memcached'] %}{{ 'api' | kolla_address(host) | put_address_in_context('memcache') }}:{{ memcached_port }}{% if not loop.last %},{% endif %}{% endfor %} - -[resource_plugin:os_cinder_volume] -enabled = {{ enable_cinder | bool }} - -[resource_plugin:os_cinder_snapshot] -enabled = {{ enable_cinder | bool }} - -[resource_plugin:os_designate_zone] -enabled = {{ enable_designate | bool }} - -[resource_plugin:os_designate_recordset] -enabled = {{ enable_designate | bool }} - -[resource_plugin:os_swift_account] -enabled = {{ enable_swift | bool }} - -[resource_plugin:os_swift_container] -enabled = {{ enable_swift | bool }} - -[resource_plugin:os_swift_object] -enabled = {{ enable_swift | bool }} - -[resource_plugin:os_nova_server] -enabled = {{ enable_nova | bool }} - -[resource_plugin:os_nova_hypervisor] -enabled = {{ enable_nova | bool }} - -[resource_plugin:os_nova_flavor] -enabled = {{ enable_nova | bool }} -notifications_topics_exchanges = versioned_notifications,nova - -[resource_plugin:os_nova_servergroup] -enabled = {{ enable_nova | bool }} - -[resource_plugin:os_glance_image] -enabled = {{ enable_glance | bool }} - -[resource_plugin:os_glance_metadef] -enabled = {{ enable_glance | bool }} - -[resource_plugin:os_neutron_net] -enabled = {{ enable_neutron | bool }} - -[resource_plugin:os_neutron_port] -enabled = {{ enable_neutron | bool }} - -[resource_plugin:os_neutron_floatingip] -enabled = {{ enable_neutron | bool }} - -[resource_plugin:os_neutron_security_group] -enabled = {{ enable_neutron | bool }} - -[resource_plugin:os_ironic_chassis] -enabled = {{ enable_ironic | bool }} -notifications_topics_exchanges = ironic_versioned_notifications,ironic - -[resource_plugin:os_ironic_node] -enabled = {{ enable_ironic | bool }} -notifications_topics_exchanges = ironic_versioned_notifications,ironic - -[resource_plugin:os_ironic_port] -enabled = {{ enable_ironic | bool }} -notifications_topics_exchanges = ironic_versioned_notifications,ironic - -{# TODO(blallau): enabling osprofiler when fixed in searchlight #} - -{# {% if enable_osprofiler | bool %} #} -{# [profiler] #} -{# enabled = true #} -{# trace_sqlalchemy = true #} -{# hmac_keys = {{ osprofiler_secret }} #} -{# connection_string = {{ osprofiler_backend_connection_string }} #} -{# {% endif %} #} diff --git a/ansible/site.yml b/ansible/site.yml index 6b5745a9cc..2b315e996e 100644 --- a/ansible/site.yml +++ b/ansible/site.yml @@ -67,7 +67,6 @@ - enable_rally_{{ enable_rally | bool }} - enable_redis_{{ enable_redis | bool }} - enable_sahara_{{ enable_sahara | bool }} - - enable_searchlight_{{ enable_searchlight | bool }} - enable_senlin_{{ enable_senlin | bool }} - enable_skydive_{{ enable_skydive | bool }} - enable_solum_{{ enable_solum | bool }} @@ -320,11 +319,6 @@ tasks_from: loadbalancer tags: sahara when: enable_sahara | bool - - include_role: - name: searchlight - tasks_from: loadbalancer - tags: searchlight - when: enable_searchlight | bool - include_role: name: senlin tasks_from: loadbalancer @@ -1102,18 +1096,6 @@ tags: senlin, when: enable_senlin | bool } -- name: Apply role searchlight - gather_facts: false - hosts: - - searchlight-api - - searchlight-listener - - '&enable_searchlight_True' - serial: '{{ kolla_serial|default("0") }}' - roles: - - { role: searchlight, - tags: searchlight, - when: enable_searchlight | bool } - - name: Apply role tacker gather_facts: false hosts: diff --git a/etc/kolla/globals.yml b/etc/kolla/globals.yml index 932a1c03bf..31abfa5299 100644 --- a/etc/kolla/globals.yml +++ b/etc/kolla/globals.yml @@ -314,7 +314,6 @@ #enable_horizon_octavia: "{{ enable_octavia | bool }}" #enable_horizon_qinling: "{{ enable_qinling | bool }}" #enable_horizon_sahara: "{{ enable_sahara | bool }}" -#enable_horizon_searchlight: "{{ enable_searchlight | bool }}" #enable_horizon_senlin: "{{ enable_senlin | bool }}" #enable_horizon_solum: "{{ enable_solum | bool }}" #enable_horizon_tacker: "{{ enable_tacker | bool }}" @@ -373,7 +372,6 @@ #enable_rally: "no" #enable_redis: "no" #enable_sahara: "no" -#enable_searchlight: "no" #enable_senlin: "no" #enable_skydive: "no" #enable_solum: "no" diff --git a/etc/kolla/passwords.yml b/etc/kolla/passwords.yml index 80fafec360..3d20098fcd 100644 --- a/etc/kolla/passwords.yml +++ b/etc/kolla/passwords.yml @@ -172,8 +172,6 @@ octavia_keystone_password: octavia_ca_password: octavia_client_ca_password: -searchlight_keystone_password: - tacker_database_password: tacker_keystone_password: diff --git a/releasenotes/notes/remove-searchlight-roles-7fde75ff93812b09.yaml b/releasenotes/notes/remove-searchlight-roles-7fde75ff93812b09.yaml new file mode 100644 index 0000000000..c109592f24 --- /dev/null +++ b/releasenotes/notes/remove-searchlight-roles-7fde75ff93812b09.yaml @@ -0,0 +1,8 @@ +--- +upgrade: + - | + The ``Searchlight`` project is no longer maintained and + `retired since Wallaby cycle`__ . Its support and roles + are also removed since Wallaby cycle. + + __ http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018637.html diff --git a/tests/templates/inventory.j2 b/tests/templates/inventory.j2 index 86b3213b54..280c7d182c 100644 --- a/tests/templates/inventory.j2 +++ b/tests/templates/inventory.j2 @@ -244,9 +244,6 @@ control [rally:children] control -[searchlight:children] -control - [octavia:children] control @@ -652,13 +649,6 @@ senlin [senlin-health-manager:children] senlin -# Searchlight -[searchlight-api:children] -searchlight - -[searchlight-listener:children] -searchlight - # Octavia [octavia-api:children] octavia