smengcl commented on a change in pull request #2565:
URL: https://github.com/apache/ozone/pull/2565#discussion_r695915073
##########
File path:
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/recon/ReconConfigKeys.java
##########
@@ -40,6 +40,12 @@ private ReconConfigKeys() {
"ozone.recon.http-address";
public static final String OZONE_RECON_HTTPS_ADDRESS_KEY =
"ozone.recon.https-address";
+ public static final String OZONE_RECON_ADDRESS_DEFAULT =
+ "recon:9891";
Review comment:
Recon RPC address default host should not be `recon`. This default value
isn't just for the testing environment. `0.0.0.0` should allow Recon server to
start up normally and listen on all interfaces.
But we should expect admins to set this address properly for **clients**.
```suggestion
"0.0.0.0:9891";
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
