You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/01/14 17:21:39 UTC
[GitHub] [druid] loquisgon opened a new pull request #10763: Update README.md -- troubleshooting for multiple IPs
loquisgon opened a new pull request #10763:
URL: https://github.com/apache/druid/pull/10763
Add a troubleshooting note for the case when RSA generation fails because using a DNS that returns IP addresses for unresolved domains.
This PR has:
- [X ] been self-reviewed.
----------------------------------------------------------------
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] clintropolis commented on a change in pull request #10763: Update README.md -- troubleshooting for multiple IPs
Posted by GitBox <gi...@apache.org>.
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org