Re: [PR] Fail the task if the xcom sidecar container does not start for a long time [airflow]

Sat, 21 Sep 2024 03:44:12 -0700 


bitomukesh commented on PR #40909:
URL: https://github.com/apache/airflow/pull/40909#issuecomment-2365140209

   <div>
   <h2>Changelist by Bito</h2>
       <p>This pull request implements the following key changes.</p>
   
   <div>
   <table width="100%">
   <tbody>
     <tr>
       <th>Key Change</th>
       <th>Files Impacted</th>
     </tr>
   
   <tr>
   <td width="30%">
       <b>Feature Improvement</b> - Add timeout for XCOM sidecar container 
startup
   </td>
   <td width="70%">
   
   
   <div>
   
   
-&nbsp;[`pod.py`](https://github.com/apache/airflow/pull/40909/files#diff-93707635f12195e7b94d14cf99675542592a28a6eb552895fb3ff37d42713740)
 - Added startup_timeout parameter to await_xcom_sidecar_container_start method 
call
   <div>
   
   
-&nbsp;[`pod_manager.py`](https://github.com/apache/airflow/pull/40909/files#diff-6900da9281d8404b14da0815f2b37350f3148b1b63449928b952744e6711e7e7)
 - Implemented timeout logic for XCOM sidecar container startup
   <div>
   
   
-&nbsp;[`test_pod.py`](https://github.com/apache/airflow/pull/40909/files#diff-ee74008348722f2718d4af25f47bd3314f3369edb0aa495337367744535a0079)
 - Updated test to include startup_timeout parameter in method call
   </div>
   </td>
   </tr>
   
   </tbody>
   </table>
   </div>
   </div>


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