amoghrajesh commented on code in PR #62129:
URL: https://github.com/apache/airflow/pull/62129#discussion_r2826402691
##########
providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/executors/kubernetes_executor_utils.py:
##########
@@ -618,6 +660,19 @@ def delete_pod(self, pod_name: str, namespace: str) ->
None:
# If the pod is already deleted
if str(e.status) != "404":
raise
+ self._delete_workload_secret(f"airflow-workload-{pod_name}", namespace)
+
+ def _delete_workload_secret(self, secret_name: str, namespace: str) ->
None:
Review Comment:
That actually made sense, I had to look up ownerRefs and it totally was
worth it, handled it in 0cb0b19b1c8b0a780ca491fe63a8f9ecab728298 by making a
patch API call for the secret
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
