-
Notifications
You must be signed in to change notification settings - Fork 15.1k
Add webserver config helm #50288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add webserver config helm #50288
Conversation
This update introduces conditional mounting of webserver configuration to the API server deployment. It ensures compatibility when `webserverConfig` or `webserverConfigConfigMapName` is specified. Additional logic for handling related volumes and volume mounts is included.
Corrected the indentation of the podAntiAffinity block in the api-server deployment YAML. This ensures proper configuration parsing and alignment with Kubernetes manifests formatting standards.
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
@jedcunningham @hussein-awala @dstandish Hello, Thank you. |
Although the Helm chart's values.yaml allows configuration for webserverConfig,
there is no corresponding section in the api-server-deployment to apply this configuration.
As a result, even if webserverConfig is added—for example, for LDAP or other authentication settings—it is not properly applied.
To address this, we have added a manifest that includes the necessary configuration.