Skip to content

Commit ffc252e

Browse files
author
Elod Illes
committed
[CI] Replace deprecated regex
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. Change-Id: If287e800fb9ff428dbe6f9c4c046627f22afe3df (cherry picked from commit 9b77bae) (cherry picked from commit 8223e6a) (cherry picked from commit 510a27b) (cherry picked from commit 197b14d)
1 parent 41cdd39 commit ffc252e

File tree

1 file changed

+16
-38
lines changed

1 file changed

+16
-38
lines changed

.zuul.yaml

Lines changed: 16 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -224,24 +224,11 @@
224224
parent: devstack-tempest
225225
description: |
226226
Run tempest compute API tests using LVM image backend. This only runs
227-
against nova/virt/libvirt/* changes.
228-
# Copy irrelevant-files from nova-dsvm-multinode-base and then exclude
229-
# anything that is not in nova/virt/libvirt/* or nova/privsep/*.
230-
irrelevant-files:
231-
- ^(?!.zuul.yaml)(?!nova/virt/libvirt/)(?!nova/privsep/).*$
232-
- ^api-.*$
233-
- ^(test-|)requirements.txt$
234-
- ^.*\.rst$
235-
- ^.git.*$
236-
- ^doc/.*$
237-
- ^nova/hacking/.*$
238-
- ^nova/locale/.*$
239-
- ^nova/tests/.*$
240-
- ^nova/test.py$
241-
- ^releasenotes/.*$
242-
- ^setup.cfg$
243-
- ^tools/.*$
244-
- ^tox.ini$
227+
against nova/virt/libvirt/*, nova/privsep/* and .zuul.yaml changes.
228+
files:
229+
- ^nova/virt/libvirt/.*$
230+
- ^nova/privsep/.*$
231+
- .zuul.yaml
245232
vars:
246233
# We use the "all" environment for tempest_test_regex and
247234
# tempest_exclude_regex.
@@ -280,22 +267,11 @@
280267
# NOTE(chateaulav): due to constraints with no IDE support for aarch64,
281268
# tests have been limited to eliminate any items that are incompatible.
282269
# This is to be re-evaluated as greater support is added and defined.
283-
irrelevant-files:
284-
- ^(?!.zuul.yaml)(?!nova/virt/libvirt/)(?!nova/objects/)(?!nova/scheduler/).*$
285-
- ^api-.*$
286-
- ^(test-|)requirements.txt$
287-
- ^.*\.rst$
288-
- ^.git.*$
289-
- ^doc/.*$
290-
- ^nova/hacking/.*$
291-
- ^nova/locale/.*$
292-
- ^nova/policies/.*$
293-
- ^nova/tests/.*$
294-
- ^nova/test.py$
295-
- ^releasenotes/.*$
296-
- ^setup.cfg$
297-
- ^tools/.*$
298-
- ^tox.ini$
270+
files:
271+
- ^nova/virt/libvirt/.*$
272+
- ^nova/objects/.*$
273+
- ^nova/scheduler/.*$
274+
- .zuul.yaml
299275
vars:
300276
tox_envlist: all
301277
tempest_test_regex: ^tempest\.(api\.compute\.servers|scenario\.test_network_basic_ops)
@@ -670,11 +646,12 @@
670646
- nova-ceph-multistore:
671647
irrelevant-files: *nova-base-irrelevant-files
672648
- neutron-linuxbridge-tempest:
673-
irrelevant-files:
649+
files:
674650
# NOTE(mriedem): This job has its own irrelevant-files section
675651
# so that we only run it on changes to networking and libvirt/vif
676652
# code; we don't need to run this on all changes.
677-
- ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$
653+
- ^nova/network/.*$
654+
- nova/virt/libvirt/vif.py
678655
- nova-live-migration
679656
- nova-live-migration-ceph
680657
- nova-lvm
@@ -734,11 +711,12 @@
734711
- nova-ceph-multistore:
735712
irrelevant-files: *nova-base-irrelevant-files
736713
- neutron-linuxbridge-tempest:
737-
irrelevant-files:
714+
files:
738715
# NOTE(mriedem): This job has its own irrelevant-files section
739716
# so that we only run it on changes to networking and libvirt/vif
740717
# code; we don't need to run this on all changes.
741-
- ^(?!nova/network/.*)(?!nova/virt/libvirt/vif.py).*$
718+
- ^nova/network/.*$
719+
- nova/virt/libvirt/vif.py
742720
- tempest-integrated-compute:
743721
irrelevant-files: *policies-irrelevant-files
744722
- nova-grenade-multinode:

0 commit comments

Comments
 (0)