amoghrajesh commented on PR #35146: URL: https://github.com/apache/airflow/pull/35146#issuecomment-1777279997
> 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 kerberos init container in case kerberos is enabled - 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. Completely agree with you. We should add this framework that naturally works out of box for such use cases. I will start working on this here -- 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]
