sadanand48 commented on PR #7229: URL: https://github.com/apache/ozone/pull/7229#issuecomment-2370803237
@dombizita I agree it is kind of redundant to pass the option because passing the correct config does the same. It is expected that this command use the config for the Ozone datanodes and run this on the datanode but say the command is run within a gateway/client host which also has the DN role but the default shell config points to the client config not the DN config and won't have the required value for `hdds.datanode.dir` . Introducing the -path arg provides some clarity to the developer trying this command. I have also asked @ArafatKhan2198 to modify the command description to communicate that the command will only print containers inside a particular storage dir and if no path is provided, it reads from the first storage dir. -- 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]
