tanvipenumudy commented on code in PR #3864:
URL: https://github.com/apache/ozone/pull/3864#discussion_r1004040931


##########
hadoop-ozone/dist/src/shell/ozone/ozone-functions.sh:
##########
@@ -2512,6 +2525,23 @@ function ozone_parse_args
           ozone_exit_with_usage 1
         fi
       ;;
+      --validate)
+        shift
+        OZONE_VALIDATE_CLASSPATH=true
+        if [[ "${1}" == "classpath" || "${1}" == "--daemon" ]]; then
+          ((OZONE_PARSE_COUNTER=OZONE_PARSE_COUNTER+1))
+        elif [[ "${1}" == "continue" && "${2}" == "--daemon" ]]; then

Review Comment:
   Thank you for reviewing @adoroszlai, noted these points.
   
   Regarding the ordering of the options - just wanted to check if we should be 
going by the following usage convention or not:
   
   ```
   Usage: ozone [OPTIONS] SUBCOMMAND [SUBCOMMAND OPTIONS]
   
     OPTIONS is none or any of:
   
   --buildpaths                       attempt to add class files from build tree
   --config dir                       Ozone config directory
   --daemon (start|status|stop)       operate on a daemon
   --debug                            turn on shell script debug mode
   --help                             usage information
   --hostnames list[,of,host,names]   hosts to use in worker mode
   --hosts filename                   list of hosts to use in worker mode
   --jvmargs arguments                append JVM options to any existing 
options defined in the OZONE_OPTS environment variable. Any defined in 
OZONE_CLIENT_OPTS will be append after these jvmargs
   --loglevel level                   set the log4j level for this command
   --validate (continue)              validates if all jars as indicated in the 
corresponding OZONE_RUN_ARTIFACT_NAME classpath file are present
   --workers                          turn on worker mode
   
     SUBCOMMAND is one of:
   
   
       Admin Commands:
   
   daemonlog     get/set the log level for each daemon
   jmxget        get JMX exported values from NameNode or DataNode.
   
       Client Commands:
   
   admin         Ozone admin tool
   auditparser   runs audit parser tool
   classpath     prints the class path needed for running ozone commands
   debug         Ozone debug tool
   dtutil        operations related to delegation tokens
   envvars       display computed Hadoop environment variables
   freon         runs an ozone data generator
   fs            run a filesystem command on Ozone file system. Equivalent to 
'hadoop fs'
   genconf       generate minimally required ozone configs and output to 
ozone-site.xml in specified path
   getconf       get ozone config values from configuration
   insight       tool to get runtime operation information
   s3            command line interface for s3 related operations
   sh            command line interface for object store operations
   tenant        command line interface for multi-tenant related operations
   version       print the version
   
       Daemon Commands:
   
   csi           run the standalone CSI daemon
   datanode      run a HDDS datanode
   om            Ozone Manager
   recon         run the Recon service
   s3g           run the S3 compatible REST gateway
   scm           run the Storage Container Manager service
   
   SUBCOMMAND may print help when invoked w/o parameters or with -h.
   ```
   
   Usage: `ozone [OPTIONS] SUBCOMMAND [SUBCOMMAND OPTIONS]`
   
   - `[OPTIONS]` = `--validate`
   - `SUBCOMMAND` = `classpath`
   - `[SUBCOMMAND OPTIONS]` `ozone-manager`
   
   Command: `ozone --validate classpath ozone-manager`
   



-- 
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]

Reply via email to