You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "AutomationDev85 (via GitHub)" <gi...@apache.org> on 2025/10/20 08:01:11 UTC
[PR] Move triggerer from pod-log-reader-role to pod-launcher-role [airflow]
AutomationDev85 opened a new pull request, #56872:
URL: https://github.com/apache/airflow/pull/56872
# Overview
We are preparing an update to the KubernetesPodTriggerer workflow to align its startup behavior with that of the synchronous workflow. As part of this effort, we are introducing this preparation PR.
This PR moves the triggerer to the pod-launcher-role, enabling it to read events from pods during the startup phase. Another reason for this change is to grant the triggerer the rights to clean up pods if it detects that the state of an Airflow task was set to failed by the user. This cleanup behavior will be introduced in a future PR.
We welcome your feedback on this change!
# Details of change:
* Move KubernetesPodTriggerer to the pod-launcher-role.
* Remove KubernetesPodTriggerer from the pod-reader-role, as pod-launcher-role already provides the necessary rights.
--
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
Re: [PR] Move triggerer from pod-log-reader-role to pod-launcher-role [airflow]
Posted by "AutomationDev85 (via GitHub)" <gi...@apache.org>.
AutomationDev85 commented on PR #56872:
URL: https://github.com/apache/airflow/pull/56872#issuecomment-3425291166
@jscheffl Thanks for the feedback! I fixed and extend the unit tests. Regarding the explicit error error. I will add during the rework of this PR https://github.com/apache/airflow/pull/56875 more detailed error message about permission issues and the helm chart.
--
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
Re: [PR] Move triggerer from pod-log-reader-role to pod-launcher-role [airflow]
Posted by "jscheffl (via GitHub)" <gi...@apache.org>.
jscheffl merged PR #56872:
URL: https://github.com/apache/airflow/pull/56872
--
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
Re: [PR] Move triggerer from pod-log-reader-role to pod-launcher-role [airflow]
Posted by "jscheffl (via GitHub)" <gi...@apache.org>.
jscheffl commented on PR #56872:
URL: https://github.com/apache/airflow/pull/56872#issuecomment-3423711802
Oh helm tests fail... I assume you can fix this? Else let me/us know.
@jedcunningham how do we handle this (did we have such in the past?) if a feature in K8s provider package has a dependency to Helm deployment? Any special care in release notes needed?
@AutomationDev85 considering people might have not upgraded the Helm and would use the async mode in future, can an explicit error being generated pointing to the root cause other than a plain HTTP 403 from K8s API?
--
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