johnhoran commented on code in PR #61778:
URL: https://github.com/apache/airflow/pull/61778#discussion_r2817005490


##########
providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/triggers/pod.py:
##########
@@ -183,7 +184,7 @@ async def run(self) -> AsyncIterator[TriggerEvent]:
                 event = await self._wait_for_container_completion()
             yield event
             return
-        except PodLaunchTimeoutException as e:
+        except (PodLaunchTimeoutException, PodLaunchFailedException) as e:

Review Comment:
   Actually scratch the seperate PR, I've added it to this one.  I had assumed 
`ImagePullBackOff` would hide the reason for the failure, but on testing it 
looks like it preserves it into the message block.  
   ```yaml
   state:
     waiting:
       message: 'Back-off pulling image "ngnx:1.14.2": ErrImagePull: Error 
response
         from daemon: pull access denied for ngnx, repository does not exist or 
may
         require ''docker login'': denied: requested access to the resource is 
denied'
       reason: ImagePullBackOff
   ```



-- 
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]

Reply via email to