Skip to content

Commit

Permalink
airflow-chart v1.8.8 (#451)
Browse files Browse the repository at this point in the history
* Update pre-commit hooks

* v1.8.8

* Use 1.8.8-beta1

* update sidecar triggerer args (#441)

* update sidecar triggerer args

* updated test cases

* update vendor packages (#442)

* remove rc1 for 1.8.8

---------

Co-authored-by: pgvishnuram <[email protected]>
Co-authored-by: pgvishnuram <[email protected]>
  • Loading branch information
3 people authored Aug 22, 2023
1 parent 8b5e7ce commit 9f1dad1
Show file tree
Hide file tree
Showing 5 changed files with 35 additions and 12 deletions.
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ repos:
hooks:
- id: remove-tabs
exclude_types: [makefile, binary]
- repo: https://github.com/charliermarsh/ruff-pre-commit
rev: "v0.0.284"
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: "v0.0.285"
hooks:
- id: ruff
args: [--fix, --exit-non-zero-on-fix, --ignore, E501]
Expand Down Expand Up @@ -73,7 +73,7 @@ repos:
entry: .circleci/generate_circleci_config.py
additional_dependencies: ["jinja2"]
- repo: https://github.com/pre-commit/mirrors-prettier
rev: "v3.0.1"
rev: "v3.0.2"
hooks:
- id: prettier
args: ["--print-width=135"]
Expand Down
8 changes: 4 additions & 4 deletions Chart.lock
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
dependencies:
- name: airflow
repository: https://airflow.apache.org
version: 1.8.0
digest: sha256:0b82db678ce5fb5a100b315f893eb91cc9bf9fee9067074e1b16244c8b5fa295
generated: "2023-03-16T18:31:10.54017-04:00"
repository: https://github.com/astronomer/airflow/releases/download/oss-helm-chart/1.11.1-astro
version: 1.11.1-astro
digest: sha256:6d7c55a8e643d3e50723187c70177ec8baa8610a4b27f430a28805afaaf7a05f
generated: "2023-08-22T17:01:11.74743+07:00"
6 changes: 3 additions & 3 deletions Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
# apiVersion v2 is Helm 3
apiVersion: v2
name: airflow
version: 1.8.7
version: 1.8.8
description: Helm chart to deploy the Astronomer Platform Airflow module
icon: https://airflow.apache.org/docs/apache-airflow/stable/_images/pin_large.png
keywords:
- astronomer
- airflow
dependencies:
- name: airflow
version: 1.8.0
repository: https://airflow.apache.org
version: 1.11.1-astro
repository: https://github.com/astronomer/airflow/releases/download/oss-helm-chart/1.11.1-astro
17 changes: 17 additions & 0 deletions tests/chart_tests/test_triggerer.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
from tests.chart_tests.helm_template_generator import render_chart


class TestTriggerer:
def test_triggerer_log_grommer_defaults(self):
"""Test Triggerer Log Groomer defaults."""
docs = render_chart(
values={
"airflow": {"airflowVersion": "2.4.3"},
},
show_only=["charts/airflow/templates/triggerer/triggerer-deployment.yaml"],
)
assert len(docs) == 1
assert (
"/usr/local/bin/clean-airflow-logs"
in docs[0]["spec"]["template"]["spec"]["containers"][1]["args"]
)
10 changes: 8 additions & 2 deletions values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,10 @@ airflow:
tag: 1.17.0-10
pgbouncerExporter:
repository: quay.io/astronomer/ap-pgbouncer-exporter
tag: 0.13.0-13
tag: 0.14.0-1
gitSync:
repository: quay.io/astronomer/ap-git-sync
tag: 3.6.5
tag: 3.6.8
# Airflow scheduler settings
scheduler:
livenessProbe:
Expand All @@ -63,6 +63,12 @@ airflow:
logGroomerSidecar:
args: ["/usr/local/bin/clean-airflow-logs"]

triggerer:
persistence:
enabled: false
logGroomerSidecar:
args: ["/usr/local/bin/clean-airflow-logs"]

# Airflow webserver settings
webserver:
allowPodLogReading: false
Expand Down

0 comments on commit 9f1dad1

Please sign in to comment.