On Tue, Feb 02, 2021 at 09:58:22PM +0100, Philippe Mathieu-Daudé wrote: > Follow the inclusive terminology from the "Conscious Language in your > Open Source Projects" guidelines [*] and replace the words "blacklist" > and "whitelist" appropriately. > > [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md > > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > tests/qemu-iotests/149 | 14 +++++++------- > tests/qemu-iotests/149.out | 8 ++++---- > 2 files changed, 11 insertions(+), 11 deletions(-) > > diff --git a/tests/qemu-iotests/149 b/tests/qemu-iotests/149 > index 328fd05a4c9..b1d3f5fad67 100755 > --- a/tests/qemu-iotests/149 > +++ b/tests/qemu-iotests/149 > @@ -500,7 +500,7 @@ configs = [ > > ] > > -blacklist = [ > +denylist = [ > # We don't have a cast-6 cipher impl for QEMU yet > "cast6-256-xts-plain64-sha1", > "cast6-128-xts-plain64-sha1", > @@ -510,17 +510,17 @@ blacklist = [ > "twofish-192-xts-plain64-sha1", > ]
"skiplist" better describes the purpose of this. > > -whitelist = [] > +allowlist = [] > if "LUKS_CONFIG" in os.environ: > - whitelist = os.environ["LUKS_CONFIG"].split(",") > + allowlist = os.environ["LUKS_CONFIG"].split(",") And "filterlist" > > for config in configs: > - if config.name in blacklist: > - iotests.log("Skipping %s in blacklist" % config.name) > + if config.name in denylist: > + iotests.log("Skipping %s in denylist" % config.name) > continue > > - if len(whitelist) > 0 and config.name not in whitelist: > - iotests.log("Skipping %s not in whitelist" % config.name) > + if len(allowlist) > 0 and config.name not in allowlist: > + iotests.log("Skipping %s not in allowlist" % config.name) > continue > > test_once(config, qemu_img=False) Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|