clintropolis commented on a change in pull request #10763:
URL: https://github.com/apache/druid/pull/10763#discussion_r557725191
##########
File path: integration-tests/README.md
##########
@@ -70,6 +74,28 @@ The file must contain one property per line, the key must
start with `druid_` an
Note that when bringing up docker containers through mvn and
-Doverride.config.path is provided, additional
Druid routers for security group integration test (permissive tls, no client
auth tls, custom check tls) will not be started.
+### Troubleshooting
+Early in the integration test, the scripts need to generate some RAS keys. If
the key generation fails because of a an error then check that
Review comment:
typos: `RAS` -> `RSA`? `.. fails because of a an error ..` -> `.. fails
because of an error`
##########
File path: integration-tests/README.md
##########
@@ -70,6 +74,28 @@ The file must contain one property per line, the key must
start with `druid_` an
Note that when bringing up docker containers through mvn and
-Doverride.config.path is provided, additional
Druid routers for security group integration test (permissive tls, no client
auth tls, custom check tls) will not be started.
+### Troubleshooting
Review comment:
nit: I would suggest putting this section much further down in the file,
probably near the end
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org
