Currently when the blacklist_file and regex string is provided,
the constructed regex looks like '^((?!black1|black2|...).)*$regex'.
This is incorrect, as it will match nothing - some string
is expected after end of the line [denoted as $ in the regex].
The proper construction is like ^(?!black1|black2|...).*(regex).*$
This solves the issue with Tempest, where using blacklist for smoke
tests is not working now, as it leads into the issue described above.
Reviewed: https:/ /review. openstack. org/522768 /git.openstack. org/cgit/ openstack/ os-testr/ commit/ ?id=96db91056ad ed65abd8e8bb531 ebcaf77b32b12a
Committed: https:/
Submitter: Zuul
Branch: master
commit 96db91056aded65 abd8e8bb531ebca f77b32b12a
Author: Szymon Datko <email address hidden>
Date: Thu Nov 23 15:54:29 2017 +0100
Fix regex builder
Currently when the blacklist_file and regex string is provided, black2| ...).)* $regex' .
the constructed regex looks like '^((?!black1|
This is incorrect, as it will match nothing - some string
is expected after end of the line [denoted as $ in the regex].
The proper construction is like ^(?!black1| black2| ...).*( regex). *$
This solves the issue with Tempest, where using blacklist for smoke
tests is not working now, as it leads into the issue described above.
Change-Id: Icdeb3c311f7eb4 14158aedb4c0304 94b419211c0
Closes-Bug: #1506215
Closes-Bug: #1595119
Closes-Bug: #1622722
Closes-Bug: #1669455