Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip pod cleanup in case of pod creation failed #37671

Merged
merged 5 commits into from
Feb 26, 2024

Conversation

dirrao
Copy link
Collaborator

@dirrao dirrao commented Feb 24, 2024

When the pod creation failed, the pod cleanup is causing nested exception.
The PR will skip pod cleanup in case of pod creation failed.

@dirrao dirrao closed this Feb 24, 2024
@dirrao dirrao reopened this Feb 24, 2024
@potiuk
Copy link
Member

potiuk commented Feb 25, 2024

Looks good, but k8s test fails.

@dirrao
Copy link
Collaborator Author

dirrao commented Feb 26, 2024

Looks good, but k8s test fails.

Its fixed.

@potiuk potiuk merged commit c6ba13a into apache:main Feb 26, 2024
64 checks passed
alejorodriguez96 pushed a commit to alejorodriguez96/airflow that referenced this pull request Feb 26, 2024
abhishekbhakat pushed a commit to abhishekbhakat/my_airflow that referenced this pull request Mar 5, 2024
utkarsharma2 pushed a commit to astronomer/airflow that referenced this pull request Apr 22, 2024
@dirrao dirrao deleted the k8_pod_operator_cleanup_fix branch May 19, 2024 05:14
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 provider related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants