From d2174b9bd7cb6187e0dc6c16edf874cdce079067 Mon Sep 17 00:00:00 2001 From: Daniel Axtens Date: Wed, 23 Jun 2021 01:03:19 +1000 Subject: [PATCH] tests: fix parallel tests Parallel tests require: - the % wildcard to be in a token enclosed by backticks, not single quotes - that the user still be able to use 'test_patchwork' (so we don't want the \_ before the %) Presumably this was skipped because if you get permissions working manually but you miss part of the required permissions in the automated script, you need to delete the old db data in order to observe the issue. Amusingly postgres worked the whole time. Fixes: 6025f0e2533f ("Add parallel testing") Signed-off-by: Daniel Axtens --- tools/docker/entrypoint.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/docker/entrypoint.sh b/tools/docker/entrypoint.sh index 8f7ea4f7..5450a535 100755 --- a/tools/docker/entrypoint.sh +++ b/tools/docker/entrypoint.sh @@ -26,7 +26,7 @@ reset_data_mysql() { DROP DATABASE IF EXISTS patchwork; CREATE DATABASE patchwork CHARACTER SET utf8; GRANT ALL ON patchwork.* TO 'patchwork' IDENTIFIED BY 'password'; -GRANT ALL ON 'test_patchwork\_%'.* to 'patchwork'@'%'; +GRANT ALL ON \`test\\_patchwork%\`.* to 'patchwork'@'%'; FLUSH PRIVILEGES; EOF } -- 2.47.3