Skip to content

Commit 18154d7

Browse files
update comment in kpo (#10656)
* update comment in kpo * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
1 parent 7db85a7 commit 18154d7

2 files changed

Lines changed: 13 additions & 15 deletions

File tree

composer/workflows/airflow_db_cleanup.py

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -196,13 +196,16 @@
196196
if AIRFLOW_VERSION < ["2", "4", "0"]:
197197
try:
198198
from airflow.models import RenderedTaskInstanceFields
199-
DATABASE_OBJECTS.append({
200-
"airflow_db_model": RenderedTaskInstanceFields,
201-
"age_check_column": RenderedTaskInstanceFields.execution_date,
202-
"keep_last": False,
203-
"keep_last_filters": None,
204-
"keep_last_group_by": None
205-
})
199+
200+
DATABASE_OBJECTS.append(
201+
{
202+
"airflow_db_model": RenderedTaskInstanceFields,
203+
"age_check_column": RenderedTaskInstanceFields.execution_date,
204+
"keep_last": False,
205+
"keep_last_filters": None,
206+
"keep_last_group_by": None,
207+
}
208+
)
206209

207210
except Exception as e:
208211
logging.error(e)

composer/workflows/kubernetes_pod_operator_c2.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,14 +78,9 @@
7878
# Entrypoint of the container, if not specified the Docker container's
7979
# entrypoint is used. The cmds parameter is templated.
8080
cmds=["echo"],
81-
# The namespace to run within Kubernetes, default namespace is
82-
# `default`. In Composer 1 there is the potential for
83-
# the resource starvation of Airflow workers and scheduler
84-
# within the Cloud Composer environment,
85-
# the recommended solution is to increase the amount of nodes in order
86-
# to satisfy the computing requirements. Alternatively, launching pods
87-
# into a custom namespace will stop fighting over resources,
88-
# and using Composer 2 will mean the environment will autoscale.
81+
# The namespace to run within Kubernetes. In Composer 2 environments
82+
# after December 2022, the default namespace is
83+
# `composer-user-workloads`.
8984
namespace="composer-user-workloads",
9085
# Docker image specified. Defaults to hub.docker.com, but any fully
9186
# qualified URLs will point to a custom repository. Supports private

0 commit comments

Comments
 (0)