CI provider fixes for ansible-test. (#71929)
* Make Azure Pipelines resource_prefix lowercase. * Make classification of CI files consistent. * Update package-data sanity test for AZP.
This commit is contained in:
parent
4f990e7615
commit
92b66e3e31
5 changed files with 16 additions and 2 deletions
|
@ -0,0 +1,2 @@
|
||||||
|
bugfixes:
|
||||||
|
- ansible-test - The ``resource_prefix`` variable provided to tests running on Azure Pipelines is now converted to lowercase to match other CI providers.
|
|
@ -0,0 +1,2 @@
|
||||||
|
bugfixes:
|
||||||
|
- ansible-test - Change classification using ``--changed`` now consistently handles common configuration files for supported CI providers.
|
|
@ -73,7 +73,7 @@ class AzurePipelines(CIProvider):
|
||||||
except KeyError as ex:
|
except KeyError as ex:
|
||||||
raise MissingEnvironmentVariable(name=ex.args[0])
|
raise MissingEnvironmentVariable(name=ex.args[0])
|
||||||
|
|
||||||
prefix = re.sub(r'[^a-zA-Z0-9]+', '-', prefix)
|
prefix = re.sub(r'[^a-zA-Z0-9]+', '-', prefix).lower()
|
||||||
|
|
||||||
return prefix
|
return prefix
|
||||||
|
|
||||||
|
|
|
@ -377,6 +377,16 @@ class PathMapper:
|
||||||
|
|
||||||
minimal = {}
|
minimal = {}
|
||||||
|
|
||||||
|
if os.path.sep not in path:
|
||||||
|
if filename in (
|
||||||
|
'azure-pipelines.yml',
|
||||||
|
'shippable.yml',
|
||||||
|
):
|
||||||
|
return all_tests(self.args) # test infrastructure, run all tests
|
||||||
|
|
||||||
|
if is_subdir(path, '.azure-pipelines'):
|
||||||
|
return all_tests(self.args) # test infrastructure, run all tests
|
||||||
|
|
||||||
if is_subdir(path, '.github'):
|
if is_subdir(path, '.github'):
|
||||||
return minimal
|
return minimal
|
||||||
|
|
||||||
|
@ -873,7 +883,6 @@ class PathMapper:
|
||||||
|
|
||||||
if path in (
|
if path in (
|
||||||
'setup.py',
|
'setup.py',
|
||||||
'shippable.yml',
|
|
||||||
):
|
):
|
||||||
return all_tests(self.args) # broad impact, run all tests
|
return all_tests(self.args) # broad impact, run all tests
|
||||||
|
|
||||||
|
|
|
@ -21,6 +21,7 @@ def assemble_files_to_ship(complete_file_list):
|
||||||
# All files which are in the repository except these:
|
# All files which are in the repository except these:
|
||||||
ignore_patterns = (
|
ignore_patterns = (
|
||||||
# Developer-only tools
|
# Developer-only tools
|
||||||
|
'.azure-pipelines/*',
|
||||||
'.github/*',
|
'.github/*',
|
||||||
'.github/*/*',
|
'.github/*/*',
|
||||||
'changelogs/fragments/*',
|
'changelogs/fragments/*',
|
||||||
|
|
Loading…
Reference in a new issue