Skip to content

Add startup probe for Kerberos worker sidecars - #71221

Open
aaron-y-chen wants to merge 1 commit into
apache:mainfrom
aaron-y-chen:kerberos-sidecar-startup-probe
Open

Add startup probe for Kerberos worker sidecars#71221
aaron-y-chen wants to merge 1 commit into
apache:mainfrom
aaron-y-chen:kerberos-sidecar-startup-probe

Conversation

@aaron-y-chen

Copy link
Copy Markdown
Contributor

related: #35154

Summary

Add a configurable klist -s startup probe to Kerberos worker sidecars as a prerequisite for adopting native Kubernetes sidecars.

File Purpose and design principle
chart/values.yaml Defines default-enabled probe settings for Celery and Kubernetes workers. The approximately 60-second failure window allows time to obtain the initial Kerberos credentials.
chart/values.schema.json Validates the new settings and intentionally exposes only enablement and timing controls. The probe type and command remain fixed to preserve its Kerberos-specific semantics.
chart/templates/workers/worker-deployment.yaml Renders the klist -s exec probe for the Celery worker Kerberos sidecar. Users of custom images without klist can disable it.
chart/files/pod-template-file.kubernetes-helm-yaml Applies the same probe contract to the KubernetesExecutor worker pod template.
chart/tests/helm_tests/security/test_kerberos.py Verifies the Celery worker probe's default, customized, and disabled rendering.
chart/tests/helm_tests/airflow_aux/test_pod_template_file.py Verifies the same three cases for the KubernetesExecutor pod template.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: [GPT 5.6-sol] following the guidelines

@boring-cyborg boring-cyborg Bot added the area:helm-chart Airflow Helm Chart label Aug 6, 2026
@aaron-y-chen
aaron-y-chen marked this pull request as ready for review August 6, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant