[AIRFLOW-XXX] Fix docstring minor issues in airflow/kubernetes/ (#6708)

* Correct typo in pod_launcher

* Remove param `secrets` in docstring since it's not specified/used

* Always use triple double quotes around docstrings 

https://www.python.org/dev/peps/pep-0257/
For consistency, always use """triple double quotes""" around docstrings
This commit is contained in:
Xiaodong 2019-12-03 07:08:31 +08:00 коммит произвёл GitHub
Родитель 91eb87112b
Коммит 80295af28d
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
3 изменённых файлов: 3 добавлений и 5 удалений

Просмотреть файл

@ -47,12 +47,12 @@ try:
return cfg
def _get_client_with_patched_configuration(cfg: Optional[Configuration]) -> client.CoreV1Api:
'''
"""
This is a workaround for supporting api token refresh in k8s client.
The function can be replace with `return client.CoreV1Api()` once the
upstream client supports token refresh.
'''
"""
if cfg:
return client.CoreV1Api(api_client=ApiClient(configuration=cfg))
else:

Просмотреть файл

@ -69,8 +69,6 @@ class PodGenerator:
:type envs: Dict[str, str]
:param cmds: The command to be run on the pod
:type cmds: List[str]
:param secrets: Secrets to be launched to the pod
:type secrets: List[airflow.kubernetes.models.secret.Secret]
:param image_pull_policy: Specify a policy to cache or always pull an image
:type image_pull_policy: str
:param image_pull_secrets: Any image pull secrets to be given to the pod.

Просмотреть файл

@ -228,7 +228,7 @@ class PodLauncher(LoggingMixin):
return None
def process_status(self, job_id, status):
"""Process status infomration for the JOB"""
"""Process status information for the JOB"""
status = status.lower()
if status == PodStatus.PENDING:
return State.QUEUED