Revert "[Fix] Correctly pass subset parameter to cmd_subset (#58601)"

This reverts commit 3dc25dc4ca.

changelog/58600.fixed

@@ -1 +0,0 @@
-Fix --subset command line option, and support old 'sub' parameter name in cmd_subset for backwards compatibility

salt/cli/salt.py

@@ -173,7 +173,7 @@ class SaltCMD(salt.utils.parsers.SaltCMDOptionParser):
         try:
             if self.options.subset:
                 cmd_func = self.local_client.cmd_subset
-                kwargs["subset"] = self.options.subset
+                kwargs["sub"] = self.options.subset
                 kwargs["cli"] = True
             else:
                 cmd_func = self.local_client.cmd_cli

salt/client/__init__.py

@@ -487,10 +487,6 @@ class LocalClient(object):
             >>> SLC.cmd_subset('*', 'test.ping', subset=1)
             {'jerry': True}
         """
-
-        # Support legacy parameter name:
-        subset = kwargs.pop("sub", subset)
-
         minion_ret = self.cmd(tgt, "sys.list_functions", tgt_type=tgt_type, **kwargs)
         minions = list(minion_ret)
         random.shuffle(minions)

tests/unit/test_client.py

@@ -1,6 +1,8 @@
+# -*- coding: utf-8 -*-
 """
     :codeauthor: Mike Place <mp@saltstack.com>
 """
+from __future__ import absolute_import, print_function, unicode_literals
 
 import salt.utils.platform
 from salt import client
@@ -96,29 +98,6 @@ class LocalClientTestCase(TestCase, SaltClientTestCaseMixin):
         ):
             with patch("salt.client.LocalClient.cmd_cli") as cmd_cli_mock:
                 self.client.cmd_subset("*", "first.func", subset=1, cli=True)
-                try:
-                    cmd_cli_mock.assert_called_with(
-                        ["minion2"],
-                        "first.func",
-                        (),
-                        progress=False,
-                        kwarg=None,
-                        tgt_type="list",
-                        full_return=False,
-                        ret="",
-                    )
-                except AssertionError:
-                    cmd_cli_mock.assert_called_with(
-                        ["minion1"],
-                        "first.func",
-                        (),
-                        progress=False,
-                        kwarg=None,
-                        tgt_type="list",
-                        full_return=False,
-                        ret="",
-                    )
-                self.client.cmd_subset("*", "first.func", sub=1, cli=True)
                 try:
                     cmd_cli_mock.assert_called_with(
                         ["minion2"],