remove deprecated --ca option

Fixes: https://pagure.io/koji/issue/2192

builder/kojid

@@ -6327,7 +6327,6 @@ def get_options():
                 'xz_options': '-z6T0',
                 'task_avail_delay': 300,
                 'cert': None,
-                'ca': '',  # FIXME: Unused, remove in next major release
                 'serverca': None}
     if config.has_section('kojid'):
         for name, value in config.items('kojid'):

cli/koji

@@ -122,7 +122,6 @@ def get_options():
     parser.add_option("--keytab", help=_("specify a Kerberos keytab to use"), metavar="FILE")
     parser.add_option("--principal", help=_("specify a Kerberos principal to use"))
     parser.add_option("--cert", help=_("specify a SSL cert to use"), metavar="FILE")
-    parser.add_option("--ca", help=SUPPRESS_HELP, metavar="FILE")  # FIXME: remove in 1.24
     parser.add_option("--runas", help=_("run as the specified user (requires special privileges)"))
     parser.add_option("--user", help=_("specify user"))
     parser.add_option("--password", help=_("specify password"))
@@ -151,9 +150,6 @@ def get_options():
                       help=_("list commands"))
     (options, args) = parser.parse_args()
 
-    if options.ca:
-        koji.util.deprecated("--ca option is deprecated and will be removed in 1.24")
-
     # load local config
     try:
         result = koji.read_config(options.profile, user_config=options.configFile)

koji/__init__.py

@@ -1915,7 +1915,6 @@ def read_config(profile_name, user_config=None):
         'principal': None,
         'keytab': None,
         'cert': None,
-        'ca': '',  # FIXME: remove in next major release
         'serverca': None,
         'no_ssl_verify': False,
         'authtype': None,
@@ -1993,9 +1992,6 @@ def read_config(profile_name, user_config=None):
         else:
             result[name] = os.path.expanduser(result[name])
 
-    if result.get('ca'):
-        util.deprecated("ca option in config file is deprecated and will be removed in 1.24")
-
     return result
 
 

tests/test_cli/test_call.py

@@ -71,7 +71,6 @@ class TestCall(utils.CliTestCase):
             'method': 'ssl_login',
             'parameters': {
                 'cert': '/etc/pki/cert',
-                'ca': ['/etc/pki/clientca', '/etc/pki/serverca'],
             },
             'result': 'success'
         }

util/koji-gc

@@ -23,7 +23,7 @@ import requests
 
 import koji
 import koji.policy
-from koji.util import LazyDict, LazyValue, to_list, deprecated
+from koji.util import LazyDict, LazyValue, to_list
 
 
 def _(args):
@@ -50,7 +50,6 @@ def get_options():
     parser.add_option("--network-hack", action="store_true", default=False,
                       help=optparse.SUPPRESS_HELP)  # no longer used
     parser.add_option("--cert", help=_("Client SSL certificate file for authentication"))
-    parser.add_option("--ca", help=optparse.SUPPRESS_HELP)  # FIXME: remove in 1.24
     parser.add_option("--serverca", help=_("CA cert file that issued the hub certificate"))
     parser.add_option("-n", "--test", action="store_true", default=False,
                       help=_("test mode"))
@@ -127,7 +126,6 @@ def get_options():
         ['password', None, 'string'],
         ['noauth', None, 'boolean'],
         ['cert', None, 'string'],
-        ['ca', None, 'string'],  # FIXME: remove in next major release
         ['serverca', None, 'string'],
         ['server', None, 'string'],
         ['weburl', None, 'string'],
@@ -169,9 +167,6 @@ def get_options():
     (options, args) = parser.parse_args(values=defaults)
     options.config = config
 
-    if options.ca:
-        deprecated("--ca option is deprecated and will be removed in 1.24")
-
     # figure out actions
     actions = ('prune', 'trash', 'delete', 'salvage')
     if options.action:

util/kojira

@@ -1137,7 +1137,6 @@ def get_options():
                 'recent_tasks_lifetime': 600,
                 'sleeptime': 15,
                 'cert': None,
-                'ca': '',  # FIXME: unused, remove in next major release
                 'serverca': None,
                 'queue_file': None,
                 }
@@ -1147,8 +1146,8 @@ def get_options():
                     'max_delete_processes', 'max_repo_tasks_maven', 'dist_repo_lifetime',
                     'sleeptime', 'recent_tasks_lifetime')
         str_opts = ('topdir', 'server', 'user', 'password', 'logfile', 'principal', 'keytab',
-                    'cert', 'ca', 'serverca', 'debuginfo_tags', 'queue_file',
-                    'source_tags', 'separate_source_tags', 'ignore_tags')  # FIXME: remove ca here
+                    'cert', 'serverca', 'debuginfo_tags', 'queue_file',
+                    'source_tags', 'separate_source_tags', 'ignore_tags')
         bool_opts = ('verbose', 'debug', 'ignore_stray_repos', 'offline_retry',
                      'no_ssl_verify', 'check_external_repos')
         legacy_opts = ('with_src', 'delete_batch_size')

vm/kojivmd

@@ -139,7 +139,6 @@ def get_options():
                 'offline_retry_interval': 120,
                 'allowed_scms': '',
                 'cert': None,
-                'ca': '',  # FIXME: Remove in next major release
                 'serverca': None}
     if config.has_section('kojivmd'):
         for name, value in config.items('kojivmd'):