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

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new fd839bbf33 Chart: Decrease default gitSync wait (#27625)
fd839bbf33 is described below

commit fd839bbf33b4633742f33ea644015e59df75f62f
Author: Jed Cunningham <[email protected]>
AuthorDate: Wed Nov 16 12:10:17 2022 -0800

    Chart: Decrease default gitSync wait (#27625)
---
 chart/newsfragments/27625.significant.rst | 4 ++++
 chart/values.schema.json                  | 4 ++--
 chart/values.yaml                         | 5 ++++-
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/chart/newsfragments/27625.significant.rst 
b/chart/newsfragments/27625.significant.rst
new file mode 100644
index 0000000000..2fe472459e
--- /dev/null
+++ b/chart/newsfragments/27625.significant.rst
@@ -0,0 +1,4 @@
+Default ``dags.gitSync.wait`` reduced to ``5`` seconds
+
+The default for ``dags.gitSync.wait`` has been reduced from ``60`` seconds to 
``5`` seconds to reduce the likelihood of DAGs
+becoming inconsistent between Airflow components. This will, however, increase 
traffic to the remote git repository.
diff --git a/chart/values.schema.json b/chart/values.schema.json
index b5b2ac304f..235e7e4bd4 100644
--- a/chart/values.schema.json
+++ b/chart/values.schema.json
@@ -5244,9 +5244,9 @@
                             "default": "tests/dags"
                         },
                         "wait": {
-                            "description": "Interval between git sync attempts 
in seconds.",
+                            "description": "Interval between git sync attempts 
in seconds. High values are more likely to cause DAGs to become out of sync 
between different components. Low values cause more traffic to the remote git 
repository.",
                             "type": "integer",
-                            "default": 60
+                            "default": 5
                         },
                         "containerName": {
                             "description": "Git sync container name.",
diff --git a/chart/values.yaml b/chart/values.yaml
index db0fca8648..1fa61ae77e 100644
--- a/chart/values.yaml
+++ b/chart/values.yaml
@@ -1825,8 +1825,11 @@ dags:
     # knownHosts: |
     #    <host1>,<ip1> <key1>
     #    <host2>,<ip2> <key2>
+
     # interval between git sync attempts in seconds
-    wait: 60
+    # high values are more likely to cause DAGs to become out of sync between 
different components
+    # low values cause more traffic to the remote git repository
+    wait: 5
     containerName: git-sync
     uid: 65533
 

Reply via email to