You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by as...@apache.org on 2021/04/01 13:47:20 UTC

[airflow] 05/11: More proper default value for namespace in K8S cleanup-pods CLI (#15060)

This is an automated email from the ASF dual-hosted git repository.

ash pushed a commit to branch v2-0-test
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit e66c870cda62441c93d18ec87074346fe11d20b4
Author: Xiaodong DENG <xd...@apache.org>
AuthorDate: Mon Mar 29 00:00:15 2021 +0200

    More proper default value for namespace in K8S cleanup-pods CLI (#15060)
    
    Currently the default value for namespace is always 'default'.
    
    However, `conf.get('kubernetes', 'namespace')` may be a more proper
    default value for namespace in this case
    
    (cherry picked from commit b8cf46a12fba5701d9ffc0b31aac8375fbca37f9)
---
 airflow/cli/cli_parser.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/airflow/cli/cli_parser.py b/airflow/cli/cli_parser.py
index e62346a..c33a854 100644
--- a/airflow/cli/cli_parser.py
+++ b/airflow/cli/cli_parser.py
@@ -678,8 +678,8 @@ ARG_OPTION = Arg(
 # kubernetes cleanup-pods
 ARG_NAMESPACE = Arg(
     ("--namespace",),
-    default='default',
-    help="Kubernetes Namespace",
+    default=conf.get('kubernetes', 'namespace'),
+    help="Kubernetes Namespace. Default value is `[kubernetes] namespace` in configuration.",
 )
 
 ALTERNATIVE_CONN_SPECS_ARGS = [