Skip to content

docs: add automated remediation guardrails to retry docs#67705

Merged
potiuk merged 6 commits into
apache:mainfrom
vividbaek:docs/automated-remediation-guardrails
Jun 22, 2026
Merged

docs: add automated remediation guardrails to retry docs#67705
potiuk merged 6 commits into
apache:mainfrom
vividbaek:docs/automated-remediation-guardrails

Conversation

@vividbaek

Copy link
Copy Markdown
Contributor

Add a short section to the Dag-level retry how-to describing guardrails for automated remediation Dags.

The new section clarifies that pools can limit concurrent recovery work, but they do not limit how often the same recovery action is attempted across Dag runs. It also suggests using small cooldown markers and maximum automatic action counts before requiring manual review.

The section includes a caveat that Airflow Variables may be sufficient for small guard values, but should not be used as a high-volume state store or as a strongly consistent locking mechanism.

This is a documentation-only change. No runtime behavior is affected.


Was generative AI tooling used to co-author this PR?
  • Yes — GLM 5.1

Generated-by: GLM 5.1 following the guidelines

@potiuk

potiuk commented Jun 1, 2026

Copy link
Copy Markdown
Member

@vividbaek A few things need addressing before review — see our Pull Request quality criteria.

  • Build docs. See docs.

No rush.


Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this two-stage triage process so that our maintainers' limited time is spent where it matters most: the conversation with you.

@vividbaek

Copy link
Copy Markdown
Contributor Author

Sorry for the confusion, and apologies for not checking this more carefully earlier.
I re-ran the docs spellcheck locally with ANSWER=n breeze build-docs apache-airflow --spellcheck-only --one-pass-only, added cooldown to spelling_wordlist.txt, and the build now passes.
The earlier failure was due to the spellchecker word list, not the docs change itself. Please re-run CI.

@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jun 8, 2026
@potiuk potiuk merged commit 3e65810 into apache:main Jun 22, 2026
72 checks passed
@vividbaek vividbaek deleted the docs/automated-remediation-guardrails branch June 23, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:documentation ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants