You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "oryanperl (via GitHub)" <gi...@apache.org> on 2025/02/26 10:28:53 UTC
[I] Missing initialDelaySeconds configuration of startupProbe [airflow]
oryanperl opened a new issue, #47093:
URL: https://github.com/apache/airflow/issues/47093
### Official Helm Chart version
1.15.0 (latest released)
### Apache Airflow version
2.9.3
### Kubernetes Version
v1.30.5
### Helm Chart configuration
### Docker Image customizations
_No response_
### What happened
Hi!
When running on small clusters sometimes the webserver takes longer period to start up
this causes the startupProbe to kill and restart the web server.
There is a possible workaround solution by configuring startupProbe's failureThreshold and periodSeconds but, as far as i can think of, the correct solution would be to configure initialDelaySeconds to allow slow pods to run the webserver before attempting to hearbeat it.
### What you think should happen instead
Allow initialDelaySeconds to startupProbes.
configuring initialDelaySeconds is already possible for livenessProbe and readinessProbe, so the addition is only needed for startupProbe
### How to reproduce
1. Running a small kubernetes cluster (e.g. docker desktop's default kubernetes cluster)
2. Run argocd on the cluster
```bash
kubectl create namespace argocd
kubectl apply -n argocd -f https://raw.githubusercontent.com/argoproj/argo-cd/stable/manifests/install.yaml
```
3. Have an app of apps in argocd
4. Have app of apps define a child application which uses airflow chart, i.e.:
```yaml
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
name: airflow
namespace: argocd
spec:
project: default
source:
repoURL: https://airflow.apache.org
chart: airflow
targetRevision: 1.15.0
helm:
values: |-
{{- with .Values.airflow }}
{{- toYaml . | nindent 8 }}
destination:
server: https://kubernetes.default.svc
namespace: {{ .Values.airflow.namespace }}
syncPolicy:
automated:
prune: true
selfHeal: true
syncOptions:
- CreateNamespace=true
- Replace=true
```
5. The manifest of startupProbe does not contain initialDelaySeconds
### Anything else
I Would like to sole this issue and am willing to submit a PR
### Are you willing to submit PR?
- [x] Yes I am willing to submit a PR!
### Code of Conduct
- [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
--
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: commits-unsubscribe@airflow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Missing initialDelaySeconds configuration of startupProbe [airflow]
Posted by "romsharon98 (via GitHub)" <gi...@apache.org>.
romsharon98 closed issue #47093: Missing initialDelaySeconds configuration of startupProbe
URL: https://github.com/apache/airflow/issues/47093
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Missing initialDelaySeconds configuration of startupProbe [airflow]
Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on issue #47093:
URL: https://github.com/apache/airflow/issues/47093#issuecomment-2684556217
Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Missing initialDelaySeconds configuration of startupProbe [airflow]
Posted by "oryanperl (via GitHub)" <gi...@apache.org>.
oryanperl commented on issue #47093:
URL: https://github.com/apache/airflow/issues/47093#issuecomment-2684590146
I Created a [Pull Request adding support for the manner](https://github.com/apache/airflow/pull/47094)
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org