dstandish commented on code in PR #25787:
URL: https://github.com/apache/airflow/pull/25787#discussion_r992503951
##########
airflow/providers/cncf/kubernetes/hooks/kubernetes.py:
##########
@@ -56,15 +56,15 @@ class KubernetesHook(BaseHook):
:doc:`/connections/kubernetes`
:param conn_id: The :ref:`kubernetes connection
<howto/connection:kubernetes>`
- to Kubernetes cluster.
+ to Kubernetes cluster
:param client_configuration: Optional dictionary of client configuration
params.
- Passed on to kubernetes client.
+ Passed on to kubernetes client
:param cluster_context: Optionally specify a context to use (e.g. if you
have multiple
- in your kubeconfig.
- :param config_file: Path to kubeconfig file.
- :param in_cluster: Set to ``True`` if running from within a kubernetes
cluster.
- :param disable_verify_ssl: Set to ``True`` if SSL verification should be
disabled.
- :param disable_tcp_keepalive: Set to ``True`` if you want to disable
keepalive logic.
+ in your kubeconfig
+ :param config_file: Path to kubeconfig file
+ :param in_cluster: Set to ``True`` if running from within a kubernetes
cluster
+ :param disable_verify_ssl: Set to ``True`` if SSL verification should be
disabled
+ :param disable_tcp_keepalive: Set to ``True`` if you want to disable
keepalive logic
"""
Review Comment:
well it's certainly not worth debating so, as you wish. but going forward
you might consider disabling those specific warnings since... they will pop up
all over the codebase
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
