Skip to content

KubernetesPodOperator: make pod retention patching explicit#68507

Merged
jscheffl merged 1 commit into
apache:mainfrom
SameerMesiah97:KubernetesPodOperator-Patching-Logic-Explicit
Jun 14, 2026
Merged

KubernetesPodOperator: make pod retention patching explicit#68507
jscheffl merged 1 commit into
apache:mainfrom
SameerMesiah97:KubernetesPodOperator-Patching-Logic-Explicit

Conversation

@SameerMesiah97

@SameerMesiah97 SameerMesiah97 commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Description

This change is a small cleanup that was originally intended for PR #61637 but was omitted when that PR was closed.

Rationale

The pod retention logic has been made more explicit to clearly distinguish between pods that are deleted and pods that are intentionally retained due to phase-based deletion rules. No functional behavior is changed.

Backwards Compatibility

No existing defaults are modified, and no public APIs have beem removed or altered.

@boring-cyborg boring-cyborg Bot added area:providers provider:cncf-kubernetes Kubernetes (k8s) provider related issues labels Jun 13, 2026
@SameerMesiah97 SameerMesiah97 force-pushed the KubernetesPodOperator-Patching-Logic-Explicit branch from 1509a25 to 877e2fd Compare June 13, 2026 15:35
@SameerMesiah97 SameerMesiah97 marked this pull request as ready for review June 13, 2026 16:41
Comment thread providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/operators/pod.py Outdated
@SameerMesiah97 SameerMesiah97 force-pushed the KubernetesPodOperator-Patching-Logic-Explicit branch 2 times, most recently from 93ee4e2 to ad61178 Compare June 13, 2026 22:01
@SameerMesiah97 SameerMesiah97 marked this pull request as draft June 13, 2026 22:01
@SameerMesiah97 SameerMesiah97 force-pushed the KubernetesPodOperator-Patching-Logic-Explicit branch from ad61178 to f673820 Compare June 13, 2026 22:31
@SameerMesiah97 SameerMesiah97 marked this pull request as ready for review June 13, 2026 23:20
@jscheffl jscheffl merged commit c6cb72c into apache:main Jun 14, 2026
189 of 196 checks passed
pgagnon pushed a commit to pgagnon/airflow that referenced this pull request Jun 15, 2026
…ache#68507)

due to phase-based deletion rules.

Co-authored-by: Sameer Mesiah <smesiah971@gmail.com>
imrichardwu pushed a commit to imrichardwu/airflow that referenced this pull request Jun 16, 2026
…ache#68507)

due to phase-based deletion rules.

Co-authored-by: Sameer Mesiah <smesiah971@gmail.com>
dingo4dev pushed a commit to dingo4dev/airflow that referenced this pull request Jun 16, 2026
…ache#68507)

due to phase-based deletion rules.

Co-authored-by: Sameer Mesiah <smesiah971@gmail.com>
RulerChen pushed a commit to RulerChen/airflow that referenced this pull request Jun 16, 2026
…ache#68507)

due to phase-based deletion rules.

Co-authored-by: Sameer Mesiah <smesiah971@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:cncf-kubernetes Kubernetes (k8s) provider related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants