diff mbox series

[1/2] tests: fix parallel tests

Message ID 20210623042027.336625-2-dja@axtens.net
State Accepted
Headers show
Series Unbreak testing | expand

Commit Message

Daniel Axtens June 23, 2021, 4:20 a.m. UTC 
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 <dja@axtens.net>
---
 tools/docker/entrypoint.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/tools/docker/entrypoint.sh b/tools/docker/entrypoint.sh
index 8f7ea4f70219..5450a535e956 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
 }