Skip to content

Commit 23fc1b9

Browse files
author
Elod Illes
committed
[CI] Replace deprecated regex and remove Centos8
Latest Zuul drops the following warnings: All regular expressions must conform to RE2 syntax, but an expression using the deprecated Perl-style syntax has been detected. Adjust the configuration to conform to RE2 syntax. The RE2 syntax error is: invalid perl operator: (?! This patch replaces the 'irrelevant-files' to 'files' with explicitly listing the pattern which files should be the tests run against. NOTE(elod.illes): this patch is extended with the removal of CentOS 8 Stream based jobs as that OS is End of Life, hence removed from available nodesets. Change-Id: If287e800fb9ff428dbe6f9c4c046627f22afe3df (cherry picked from commit 9b77bae) (cherry picked from commit 8223e6a) (cherry picked from commit 510a27b) (cherry picked from commit 197b14d) (cherry picked from commit ffc252e)
1 parent 398b453 commit 23fc1b9

File tree

1 file changed

+16
-45
lines changed

1 file changed

+16
-45
lines changed

.zuul.yaml

Lines changed: 16 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -197,24 +197,11 @@
197197
parent: devstack-tempest
198198
description: |
199199
Run tempest compute API tests using LVM image backend. This only runs
200-
against nova/virt/libvirt/* changes.
201-
# Copy irrelevant-files from nova-dsvm-multinode-base and then exclude
202-
# anything that is not in nova/virt/libvirt/* or nova/privsep/*.
203-
irrelevant-files:
204-
- ^(?!.zuul.yaml)(?!nova/virt/libvirt/)(?!nova/privsep/).*$
205-
- ^api-.*$
206-
- ^(test-|)requirements.txt$
207-
- ^.*\.rst$
208-
- ^.git.*$
209-
- ^doc/.*$
210-
- ^nova/hacking/.*$
211-
- ^nova/locale/.*$
212-
- ^nova/tests/.*$
213-
- ^nova/test.py$
214-
- ^releasenotes/.*$
215-
- ^setup.cfg$
216-
- ^tools/.*$
217-
- ^tox.ini$
200+
against nova/virt/libvirt/*, nova/privsep/* and .zuul.yaml changes.
201+
files:
202+
- ^nova/virt/libvirt/.*$
203+
- ^nova/privsep/.*$
204+
- .zuul.yaml
218205
vars:
219206
# We use the "all" environment for tempest_test_regex and
220207
# tempest_exclude_regex.
@@ -253,22 +240,11 @@
253240
# NOTE(chateaulav): due to constraints with no IDE support for aarch64,
254241
# tests have been limited to eliminate any items that are incompatible.
255242
# This is to be re-evaluated as greater support is added and defined.
256-
irrelevant-files:
257-
- ^(?!.zuul.yaml)(?!nova/virt/libvirt/)(?!nova/objects/)(?!nova/scheduler/).*$
258-
- ^api-.*$
259-
- ^(test-|)requirements.txt$
260-
- ^.*\.rst$
261-
- ^.git.*$
262-
- ^doc/.*$
263-
- ^nova/hacking/.*$
264-
- ^nova/locale/.*$
265-
- ^nova/policies/.*$
266-
- ^nova/tests/.*$
267-
- ^nova/test.py$
268-
- ^releasenotes/.*$
269-
- ^setup.cfg$
270-
- ^tools/.*$
271-
- ^tox.ini$
243+
files:
244+
- ^nova/virt/libvirt/.*$
245+
- ^nova/objects/.*$
246+
- ^nova/scheduler/.*$
247+
- .zuul.yaml
272248
vars:
273249
tox_envlist: all
274250
tempest_test_regex: ^tempest\.(api\.compute\.servers|scenario\.test_network_basic_ops)
@@ -632,11 +608,12 @@
632608
- nova-ceph-multistore:
633609
irrelevant-files: *nova-base-irrelevant-files
634610
- neutron-linuxbridge-tempest:
635-
irrelevant-files:
611+
files:
636612
# NOTE(mriedem): This job has its own irrelevant-files section
637613
# so that we only run it on changes to networking and libvirt/vif
638614
# code; we don't need to run this on all changes.
639-
- ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$
615+
- ^nova/network/.*$
616+
- nova/virt/libvirt/vif.py
640617
- nova-live-migration
641618
- nova-live-migration-ceph
642619
- nova-lvm
@@ -680,11 +657,6 @@
680657
- barbican-tempest-plugin-simple-crypto:
681658
irrelevant-files: *nova-base-irrelevant-files
682659
voting: false
683-
- tempest-integrated-compute-centos-8-stream:
684-
irrelevant-files: *nova-base-irrelevant-files
685-
- tempest-centos8-stream-fips:
686-
irrelevant-files: *nova-base-irrelevant-files
687-
voting: false
688660
gate:
689661
jobs:
690662
- nova-live-migration
@@ -697,11 +669,12 @@
697669
- nova-ceph-multistore:
698670
irrelevant-files: *nova-base-irrelevant-files
699671
- neutron-linuxbridge-tempest:
700-
irrelevant-files:
672+
files:
701673
# NOTE(mriedem): This job has its own irrelevant-files section
702674
# so that we only run it on changes to networking and libvirt/vif
703675
# code; we don't need to run this on all changes.
704-
- ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$
676+
- ^nova/network/.*$
677+
- nova/virt/libvirt/vif.py
705678
- tempest-integrated-compute:
706679
irrelevant-files: *policies-irrelevant-files
707680
- nova-grenade-multinode:
@@ -710,8 +683,6 @@
710683
irrelevant-files: *nova-base-irrelevant-files
711684
- openstacksdk-functional-devstack:
712685
irrelevant-files: *nova-base-irrelevant-files
713-
- tempest-integrated-compute-centos-8-stream:
714-
irrelevant-files: *nova-base-irrelevant-files
715686
experimental:
716687
jobs:
717688
- ironic-tempest-bfv:

0 commit comments

Comments
 (0)