potiuk commented on PR #35146:
URL: https://github.com/apache/airflow/pull/35146#issuecomment-1777080700

   Also one more request here comment. Since our helm chart already supports 
kerberos as an option, I think we should also **actually** modify the Helm 
chart of ours to add kerboros init container in case kerberos is enabled -not - 
that would be not only good (the token would be refreshed when airflow starts) 
but also serve as an example to others who want to use it.  
   
   This would be perfect if it comes together, however we have to be careful to 
add this init container only if airflow >= 2.8.0  - this is a new feature that 
will come there (we do not release new features in patchlevel versions) and 
such init container will only work when airflow is >= 2.8.0. We have a few 
similar conditionals in our helm chart already - doing similar things.


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