Skip to content

Add "already checked" to failed pods in K8sPodOperator#11368

Merged
dimberman merged 1 commit intoapache:masterfrom
astronomer:already-checked-k8spodoperator
Oct 9, 2020
Merged

Add "already checked" to failed pods in K8sPodOperator#11368
dimberman merged 1 commit intoapache:masterfrom
astronomer:already-checked-k8spodoperator

Conversation

@dimberman
Copy link
Contributor

@dimberman dimberman commented Oct 9, 2020

To prevent reattaching to already failed (and processed) pods, we add an "already_checked" label to any pod that completes the K8sPodOperator.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@boring-cyborg boring-cyborg bot added the k8s label Oct 9, 2020
@dimberman dimberman requested review from ashb and kaxil October 9, 2020 00:46
@dimberman
Copy link
Contributor Author

dimberman commented Oct 9, 2020

addresses #11005

@dimberman dimberman added this to the Airflow 1.10.13 milestone Oct 9, 2020
@dimberman dimberman linked an issue Oct 9, 2020 that may be closed by this pull request
@dimberman dimberman requested review from potiuk and turbaszek October 9, 2020 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add annotation "already_checked=True" on failed pods.

2 participants