Skip to content

Commit

Permalink
More proper default value for namespace in K8S cleanup-pods CLI (#15060)
Browse files Browse the repository at this point in the history
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 b8cf46a)
  • Loading branch information
XD-DENG authored and ashb committed Apr 15, 2021
1 parent 48436ac commit 15e1baf
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions airflow/cli/cli_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -678,8 +678,8 @@ def positive_int(value):
# 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 = [
Expand Down

0 comments on commit 15e1baf

Please sign in to comment.