Bug 1585010 - Remove apb refresh as an option (#301)

* Bug 1585010 - Remove apb refresh as an option

* Comment don't delete!

* Fix flake8

Author: dymurray

src/apb/cli.py

@@ -22,8 +22,8 @@
     'bootstrap': 'Tell Ansible Service Broker to reload APBs from the container repository',
     'test': 'Test the APB',
     'run': 'Run APB',
-    'refresh': 'Refresh all the service-catalog data',
     'version': 'Get current version of APB tool'
+    # 'refresh': 'Refresh all the service-catalog data',
 }
 
 
@@ -562,60 +562,62 @@ def subcmd_relist_parser(subcmd):
     )
     return
 
-
-def subcmd_refresh_parser(subcmd):
-    """ version subcommand """
-    subcmd.add_argument(
-        '--secure',
-        action='store_true',
-        dest='verify',
-        help=u'Verify SSL connection to Ansible Service Broker',
-        default=False
-    )
-    subcmd.add_argument(
-        '--ca-path',
-        action='store',
-        dest='cert',
-        help=u'CA cert to use for verifying SSL connection to Ansible Service Broker',
-        default=None
-    )
-    subcmd.add_argument(
-        '--username',
-        '-u',
-        action='store',
-        default=None,
-        dest='basic_auth_username',
-        help=u'Specify the basic auth username to be used'
-    )
-    subcmd.add_argument(
-        '--password',
-        '-p',
-        action='store',
-        default=None,
-        dest='basic_auth_password',
-        help=u'Specify the basic auth password to be used'
-    )
-    subcmd.add_argument(
-        '--broker',
-        action='store',
-        dest='broker',
-        help=u'Route to the Ansible Service Broker'
-    )
-    subcmd.add_argument(
-        '--no-relist',
-        action='store_true',
-        dest='no_relist',
-        help=u'Do not relist the catalog after pushing an apb to the broker',
-        default=False
-    )
-    subcmd.add_argument(
-        '--broker-name',
-        action='store',
-        dest='broker_name',
-        help=u'Name of the ServiceBroker k8s resource',
-        default=u'ansible-service-broker'
-    )
-    return
+# Bug 1585010 - Removing apb refresh as a valid option since it currently depends on kubectl being installed
+# We need to revisit this using the dynamic client as opposed to doing a subprocess call.
+#
+# def subcmd_refresh_parser(subcmd):
+#     """ version subcommand """
+#     subcmd.add_argument(
+#         '--secure',
+#         action='store_true',
+#         dest='verify',
+#         help=u'Verify SSL connection to Ansible Service Broker',
+#         default=False
+#     )
+#     subcmd.add_argument(
+#         '--ca-path',
+#         action='store',
+#         dest='cert',
+#         help=u'CA cert to use for verifying SSL connection to Ansible Service Broker',
+#         default=None
+#     )
+#     subcmd.add_argument(
+#         '--username',
+#         '-u',
+#         action='store',
+#         default=None,
+#         dest='basic_auth_username',
+#         help=u'Specify the basic auth username to be used'
+#     )
+#     subcmd.add_argument(
+#         '--password',
+#         '-p',
+#         action='store',
+#         default=None,
+#         dest='basic_auth_password',
+#         help=u'Specify the basic auth password to be used'
+#     )
+#     subcmd.add_argument(
+#         '--broker',
+#         action='store',
+#         dest='broker',
+#         help=u'Route to the Ansible Service Broker'
+#     )
+#     subcmd.add_argument(
+#         '--no-relist',
+#         action='store_true',
+#         dest='no_relist',
+#         help=u'Do not relist the catalog after pushing an apb to the broker',
+#         default=False
+#     )
+#     subcmd.add_argument(
+#         '--broker-name',
+#         action='store',
+#         dest='broker_name',
+#         help=u'Name of the ServiceBroker k8s resource',
+#         default=u'ansible-service-broker'
+#     )
+#     return
 
 
 def subcmd_version_parser(subcmd):