Ghanshyam Maan
3e1ec7a5d6
tempest-tox-plugin-sanity-check job detect all the tempest plugins even they are outside of openstack official projects. This caused us a lot of issue where non-openstack plugins (in x/ namespace) broke this job in regular interval. The number of non-openstack plugins in NON_ACTIVE_LIST shows the fact: - https://github.com/openstack/tempest/blob/a45d14bead8545840c644b869ba80bd60974a1b9/tools/generate-tempest-plugins-list.py#L37 Another point is that we do not have control to fix/maintain the non-openstack plugins so if they are broken then we need to add them in NON_ACTIVE_LIST and keep monitor when they are green. Solving this by limiting the scope of tempest-tox-plugin-sanity-check job to test the openstack official projects plugins only. Change-Id: Ib9939de038f857498090fe0d9ba4211c64a5ab04 Signed-off-by: Ghanshyam Maan <gmaan.os14@gmail.com>
113 lines
4.0 KiB
Python
113 lines
4.0 KiB
Python
#! /usr/bin/env python
|
|
|
|
# Copyright 2016 Hewlett Packard Enterprise Development Company, L.P.
|
|
#
|
|
# 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.
|
|
|
|
# This script is intended to be run as part of a periodic proposal bot
|
|
# job in OpenStack infrastructure.
|
|
#
|
|
# In order to function correctly, the environment in which the
|
|
# script runs must have
|
|
# * network access to the review.opendev.org Gerrit API
|
|
# working directory
|
|
# * network access to https://opendev.org/openstack
|
|
|
|
import json
|
|
import re
|
|
import sys
|
|
|
|
import urllib3
|
|
from urllib3.util import retry
|
|
|
|
# List of openstack projects having tempest plugin stale or unmaintained for
|
|
# a long time(6 months or more)
|
|
NON_ACTIVE_LIST = [
|
|
'openstack/networking-generic-switch',
|
|
# networking-spp is missing neutron-tempest-plugin as a dep plus
|
|
# test-requirements.txt is nested in a openstack dir and sanity script
|
|
# doesn't count with such scenario yet
|
|
'openstack/neutron-dynamic-routing',
|
|
# As tests have been migrated to neutron-tempest-plugin:
|
|
# https://review.opendev.org/#/c/637718/
|
|
'openstack/neutron-vpnaas',
|
|
]
|
|
|
|
url = 'https://review.opendev.org/projects/'
|
|
|
|
# This is what a project looks like
|
|
'''
|
|
"openstack-attic/akanda": {
|
|
"id": "openstack-attic%2Fakanda",
|
|
"state": "READ_ONLY"
|
|
},
|
|
'''
|
|
|
|
http = urllib3.PoolManager(cert_reqs='CERT_REQUIRED')
|
|
retries = retry.Retry(status_forcelist=[500], backoff_factor=1.0)
|
|
|
|
|
|
def has_tempest_plugin(proj):
|
|
try:
|
|
r = http.request('GET', "https://opendev.org/%s/raw/branch/"
|
|
"master/setup.cfg" % proj, retries=retries)
|
|
if r.status == 404:
|
|
return False
|
|
except urllib3.exceptions.MaxRetryError as err:
|
|
# We should not ignore non 404 errors.
|
|
raise err
|
|
p = re.compile(r'^tempest\.test_plugins', re.M)
|
|
if p.findall(r.data.decode('utf-8')):
|
|
return True
|
|
else:
|
|
False
|
|
|
|
|
|
if len(sys.argv) > 1 and sys.argv[1] == 'nonactivelist':
|
|
for non_active_plugin in NON_ACTIVE_LIST:
|
|
print(non_active_plugin)
|
|
# We just need NON_ACTIVE_LIST when we use this `nonactivelist` option.
|
|
# So, this exits here.
|
|
sys.exit()
|
|
|
|
r = http.request('GET', url, retries=retries)
|
|
# Gerrit prepends 4 garbage octets to the JSON, in order to counter
|
|
# cross-site scripting attacks. Therefore we must discard it so the
|
|
# json library won't choke.
|
|
content = r.data.decode('utf-8')[4:]
|
|
projects = sorted(json.loads(content))
|
|
|
|
# Retrieve openstack projects having no deployment tool repo (such
|
|
# as deb, puppet, ansible, etc.), infra repos, ui or spec namespace
|
|
# as those namespaces do not contains tempest plugins.
|
|
projects_list = [i for i in projects if
|
|
i.startswith('openstack/') and not (
|
|
i.startswith('openstack/ansible-') or
|
|
i.startswith('openstack/charm-') or
|
|
i.startswith('openstack/cookbook-openstack-') or
|
|
i.startswith('openstack/devstack-') or
|
|
i.startswith('openstack/fuel-') or
|
|
i.startswith('openstack/deb-') or
|
|
i.startswith('openstack/puppet-') or
|
|
i.startswith('openstack/openstack-ansible-') or
|
|
i.endswith('-ui') or
|
|
i.endswith('-specs'))]
|
|
|
|
found_plugins = list(filter(has_tempest_plugin, projects_list))
|
|
|
|
# We have tempest plugins not only in 'openstack/' namespace but also the
|
|
# other name spaces such as 'airship/', 'x/', etc.
|
|
# So, we print all of them here.
|
|
for project in found_plugins:
|
|
print(project)
|