You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "SamWheating (via GitHub)" <gi...@apache.org> on 2023/08/24 15:37:37 UTC

[GitHub] [airflow] SamWheating commented on issue #33697: skip_on_exit_code parameter in KPO does not take effect

SamWheating commented on issue #33697:
URL: https://github.com/apache/airflow/issues/33697#issuecomment-1691922679

   I think that this is due to the `last_state` block being all `None` in that container status object. Here's the code which is used to determine if we should skip based on the exit code:
   https://github.com/apache/airflow/blob/7141c42cc33000fe506a41679409d0a260d63eb0/airflow/providers/cncf/kubernetes/operators/pod.py#L723-L734
   
   So its relying on the exit code being available under `base_container_status.last_state.terminated.exit_code`, which it wasn't in your case 🤔 I'm not completely familiar with the expected structure of these objects but I'll do some digging and see if this is something we can fix airflow-side.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org