dheerenmohta commented on PR #72479: URL: https://github.com/apache/airflow/pull/72479#issuecomment-5759627002
Makes sense, thanks for the pointer. Went through `contributing-docs/29_helm_chart_development.rst`'s chart-vs-Kustomize decision criteria: ServiceMonitor support needs the Prometheus Operator CRDs, which the chart doesn't install, and it's an optional/environment-specific concern — that lands squarely under "belongs in Kustomize," not the core chart. Happy to close this and follow up with an overlay under `chart/kustomize-overlays/`, modeled on the existing `keda`/`kerberos` ones (`kustomization.yaml` + `README.rst` + `STATUS.yaml`). Let me know if that's the right shape, or if there's a reason to hold off given the overlay work is still early. --- Drafted-by: Claude Code (Sonnet 5) (no human review before posting) -- 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]
