diff --git a/doc/source/roles/role-ceilometerdb_size.rst b/doc/source/roles/role-ceilometerdb_size.rst deleted file mode 100644 index 9ea910838..000000000 --- a/doc/source/roles/role-ceilometerdb_size.rst +++ /dev/null @@ -1,7 +0,0 @@ -================= -ceilometerdb_size -================= - -.. ansibleautoplugin:: - :role: roles/ceilometerdb_size - diff --git a/playbooks/ceilometerdb-size.yaml b/playbooks/ceilometerdb-size.yaml deleted file mode 100644 index 6cbd67857..000000000 --- a/playbooks/ceilometerdb-size.yaml +++ /dev/null @@ -1,20 +0,0 @@ ---- -- hosts: undercloud - vars: - metadata: - name: Events Database Size Check (DEPRECATED) - description: | - The undercloud's events database can grow to a substantial - size if event_time_to_live is set to a negative value (infinite limit). - - This validation checks event_time_to_live and fails if the variable is - set to a negative value or if it has no custom setting - (their value is -1 by default). - - WARNING! This validation is considered as deprecated since Train release - groups: - - pre-deployment - event_ttl_check: "event_time_to_live" - panko_config_file: "/var/lib/config-data/puppet-generated/panko/etc/panko/panko.conf" - roles: - - ceilometerdb_size diff --git a/releasenotes/notes/ceilometerdb-size-validation-removal-a5c42940eac102df.yaml b/releasenotes/notes/ceilometerdb-size-validation-removal-a5c42940eac102df.yaml new file mode 100644 index 000000000..d5417120e --- /dev/null +++ b/releasenotes/notes/ceilometerdb-size-validation-removal-a5c42940eac102df.yaml @@ -0,0 +1,5 @@ +--- +others: + - | + The ceilometerdb-size is deprecated since the Train cycle and it's now time + to retire it. diff --git a/roles/ceilometerdb_size/defaults/main.yml b/roles/ceilometerdb_size/defaults/main.yml deleted file mode 100644 index 37c3a19fc..000000000 --- a/roles/ceilometerdb_size/defaults/main.yml +++ /dev/null @@ -1,3 +0,0 @@ ---- -event_ttl_check: "event_time_to_live" -panko_config_file: "/var/lib/config-data/puppet-generated/panko/etc/panko/panko.conf" diff --git a/roles/ceilometerdb_size/molecule/default/Dockerfile b/roles/ceilometerdb_size/molecule/default/Dockerfile deleted file mode 100644 index e0534b4d1..000000000 --- a/roles/ceilometerdb_size/molecule/default/Dockerfile +++ /dev/null @@ -1,37 +0,0 @@ -# Molecule managed -# Copyright 2019 Red Hat, Inc. -# All Rights Reserved. -# -# 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. - - -{% if item.registry is defined %} -FROM {{ item.registry.url }}/{{ item.image }} -{% else %} -FROM {{ item.image }} -{% endif %} - -RUN if [ $(command -v apt-get) ]; then apt-get update && apt-get install -y python sudo bash ca-certificates && apt-get clean; \ - elif [ $(command -v dnf) ]; then dnf makecache && dnf --assumeyes install sudo python*-devel python*-dnf bash {{ item.pkg_extras | default('') }} && dnf clean all; \ - elif [ $(command -v yum) ]; then yum makecache fast && yum install -y python sudo yum-plugin-ovl python-setuptools bash {{ item.pkg_extras | default('') }} && sed -i 's/plugins=0/plugins=1/g' /etc/yum.conf && yum clean all; \ - elif [ $(command -v zypper) ]; then zypper refresh && zypper install -y python sudo bash python-xml {{ item.pkg_extras | default('') }} && zypper clean -a; \ - elif [ $(command -v apk) ]; then apk update && apk add --no-cache python sudo bash ca-certificates {{ item.pkg_extras | default('') }}; \ - elif [ $(command -v xbps-install) ]; then xbps-install -Syu && xbps-install -y python sudo bash ca-certificates {{ item.pkg_extras | default('') }} && xbps-remove -O; fi - -{% for pkg in item.easy_install | default([]) %} -# install pip for centos where there is no python-pip rpm in default repos -RUN easy_install {{ pkg }} -{% endfor %} - - -CMD ["sh", "-c", "while true; do sleep 10000; done"] diff --git a/roles/ceilometerdb_size/molecule/default/converge.yml b/roles/ceilometerdb_size/molecule/default/converge.yml deleted file mode 100644 index 6c61a39a2..000000000 --- a/roles/ceilometerdb_size/molecule/default/converge.yml +++ /dev/null @@ -1,26 +0,0 @@ ---- -# Copyright 2019 Red Hat, Inc. -# All Rights Reserved. -# -# 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: Converge - hosts: all - gather_facts: false - - tasks: - - name: Warn developers about the lack of molecule testing - fail: - msg: >- - This role needs molecule tests! diff --git a/roles/ceilometerdb_size/molecule/default/molecule.yml b/roles/ceilometerdb_size/molecule/default/molecule.yml deleted file mode 100644 index d4dfe7fd9..000000000 --- a/roles/ceilometerdb_size/molecule/default/molecule.yml +++ /dev/null @@ -1,48 +0,0 @@ ---- -driver: - name: podman - -log: true - -platforms: - - name: ubi8 - hostname: ubi8 - image: ubi8/ubi-init - registry: - url: registry.access.redhat.com - dockerfile: Dockerfile - pkg_extras: python*-setuptools ruby python*-pyyaml - volumes: - - /etc/ci/mirror_info.sh:/etc/ci/mirror_info.sh:ro - - /etc/pki/rpm-gpg:/etc/pki/rpm-gpg - - /opt/yum.repos.d:/etc/yum.repos.d:rw - environment: &env - http_proxy: "{{ lookup('env', 'http_proxy') }}" - https_proxy: "{{ lookup('env', 'https_proxy') }}" - ulimits: &ulimit - - host - -provisioner: - name: ansible - inventory: - hosts: - all: - hosts: - ubi8: - ansible_python_interpreter: /usr/bin/python3 - log: true - env: - ANSIBLE_STDOUT_CALLBACK: yaml - ANSIBLE_LIBRARY: "${ANSIBLE_LIBRARY:-/usr/share/ansible/plugins/modules}" - -scenario: - test_sequence: - - destroy - - create - - prepare - - converge - - verify - - destroy - -verifier: - name: testinfra diff --git a/roles/ceilometerdb_size/tasks/main.yml b/roles/ceilometerdb_size/tasks/main.yml deleted file mode 100644 index 4577141a3..000000000 --- a/roles/ceilometerdb_size/tasks/main.yml +++ /dev/null @@ -1,42 +0,0 @@ ---- -- name: Deprecation WARNING - warn: - msg: >- - Panko has been deprecated since the Train Release. - The 'ceilometerdb-size' validation is now considered as deprecated too. - -- name: Ensure we get needed facts - setup: - gather_subset: - - '!all' - - '!any' - - '!min' - - env - -- name: Check if telemetry services are enabled - validations_read_ini: - path: "{{ ansible_env.HOME }}/undercloud.conf" - section: DEFAULT - key: enable_telemetry - register: enable_telemetry - become: true - -- name: End play if telemetry is not enabled - meta: end_play - when: not enable_telemetry.value|bool - -- name: Get event ttl from panko configuration file - become: true - validations_read_ini: - path: "{{ panko_config_file }}" - section: database - key: "{{ event_ttl_check }}" - ignore_missing_file: true - register: event_check_result - -- name: Fail if event_ttl_check is set to -1 - fail: - msg: >- - '{{ panko_config_file }}[database]/{{ event_ttl_check }}' is - set to -1 or not set - when: event_check_result.value|int|default(-1) < 0 diff --git a/roles/ceilometerdb_size/vars/main.yml b/roles/ceilometerdb_size/vars/main.yml deleted file mode 100644 index 1b56ccad0..000000000 --- a/roles/ceilometerdb_size/vars/main.yml +++ /dev/null @@ -1,12 +0,0 @@ ---- -metadata: - name: Events Database Size Check - description: > - The undercloud's events database can grow to a substantial - size if event_time_to_live is set to a negative value (infinite limit). - - This validation checks event_time_to_live and fails if the variable is - set to a negative value or if it has no custom setting - (their value is -1 by default). - groups: - - pre-deployment diff --git a/zuul.d/molecule.yaml b/zuul.d/molecule.yaml index e1fd8b486..a50dfc82a 100644 --- a/zuul.d/molecule.yaml +++ b/zuul.d/molecule.yaml @@ -52,14 +52,6 @@ vars: tripleo_validations_role_name: collect_flavors_and_verify_profiles voting: false -- job: - files: - - ^roles/ceilometerdb_size/.* - name: tripleo-validations-centos-8-molecule-ceilometerdb_size - parent: tripleo-validations-centos-8-base - vars: - tripleo_validations_role_name: ceilometerdb_size - voting: false - job: files: - ^roles/neutron_sanity_check/.*