ansible-galaxy tidy up arg parse with better validation (#59957)

* ansible-galaxy tidy up arg parse with better validation

* Add support back in for -v before sub aprser

* Added deprecation warning for manually parsed verbosity

changelogs/fragments/galaxy-argspec-verbosity.yaml

@@ -0,0 +1,2 @@
+deprecated_features:
+- Deprecated setting the verbosity before the sub command for ``ansible-galaxy`` and ``ansible-vault``. Set the verbosity level after the sub command, e.g. do ``ansible-galaxy init -v`` and not ``ansible-galaxy -v init``.

lib/ansible/cli/__init__.py

@@ -341,6 +341,16 @@ def post_process_args(self, options):
             else:
                 options.inventory = C.DEFAULT_HOST_LIST
 
+        # Dup args set on the root parser and sub parsers results in the root parser ignoring the args. e.g. doing
+        # 'ansible-galaxy -vvv init' has no verbosity set but 'ansible-galaxy init -vvv' sets a level of 3. To preserve
+        # back compat with pre-argparse changes we manually scan and set verbosity based on the argv values.
+        if self.parser.prog in ['ansible-galaxy', 'ansible-vault'] and not options.verbosity:
+            verbosity_arg = next(iter([arg for arg in self.args if arg.startswith('-v')]), None)
+            if verbosity_arg:
+                display.deprecated("Setting verbosity before the arg sub command is deprecated, set the verbosity "
+                                   "after the sub command", "2.13")
+                options.verbosity = verbosity_arg.count('v')
+
         return options
 
     def parse(self):