From 0f56d394878425c9d5ce913898d3d7ffdcc88645 Mon Sep 17 00:00:00 2001
From: Tomas Kopecek <tkopecek@redhat.com>
Date: Nov 23 2020 10:03:38 +0000
Subject: remove deprecated --ca option


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

---

diff --git a/builder/kojid b/builder/kojid
index ce2c297..278430f 100755
--- a/builder/kojid
+++ b/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'):
diff --git a/cli/koji b/cli/koji
index badabcb..05ae44f 100755
--- a/cli/koji
+++ b/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)
diff --git a/koji/__init__.py b/koji/__init__.py
index 9ab87d0..2245242 100644
--- a/koji/__init__.py
+++ b/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
 
 
diff --git a/tests/test_cli/test_call.py b/tests/test_cli/test_call.py
index a459dcc..b8d191f 100644
--- a/tests/test_cli/test_call.py
+++ b/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'
         }
diff --git a/util/koji-gc b/util/koji-gc
index 0c8af22..e84e15a 100755
--- a/util/koji-gc
+++ b/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:
diff --git a/util/kojira b/util/kojira
index 0758cb5..bca7d8d 100755
--- a/util/kojira
+++ b/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')
diff --git a/vm/kojivmd b/vm/kojivmd
index e50debd..d6e4029 100755
--- a/vm/kojivmd
+++ b/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'):