Comments (4)
It is because the startup probe is not checking for .Values.nginx.enabled like the readiness probe.
The startupProbe is set eventhrough there is no http port defined in the case where nginx is enabled.
Thereby the kubelet can't find a port named "http" and then kubelet tries to parse the string as an integer which fails.
The if-statement around the startupProbe should look something like this:
{{- if and .Values.startupProbe.enabled (not .Values.nginx.enabled) }}
from helm.
Just from reading the template it looks like the named port http
should only be defined for one container. Does it work for you when you use 80
instead in the startup probe?
from helm.
For which value do you want me to specify 80
? Doing something like
startupProbe:
enabled: true
httpGet:
path: /status.php
port: 80
httpHeaders:
- name: Host
value: ...
doesn't override the values from the template
from helm.
1afd7dd fixed this issue but it would be nice to have Startup Probes even if using nginx. Closing this issue anyway
from helm.
Related Issues (20)
- Helm chart 4.6.8 does not work when using an external database for NextCloud. "key" is not defined. HOT 1
- Cannot disable packaged PostgreSQL HOT 2
- Feature: Use CronJob resources instead of a container with crond HOT 9
- PHP Memory Limit should be configured properly by default HOT 8
- Enabling `.persistence.enabled` leads to `/var/www/html/lib` and probably other files not being created HOT 5
- chown permission in nfs pvc HOT 2
- Error: Accessing site insecurely via HTTP | Config Question HOT 3
- Add ExtraVolumes for Cronjob Container HOT 2
- Feature: Add Valkey as an optional subchart to be an alternative to redis HOT 1
- Feature: Add test for enabling redis to ci
- Feature: Add test for metrics in CI
- Feature: Add OnlyOffice as a subchart
- Feature: Add Collabora as a subchart
- New S3 Configuration breaks prior S3 configuration and is missing uploadPartSize HOT 7
- Feature: Add test for adding extra configs in CI
- failed to create resource: Deployment.apps "nextcloud" is invalid: [spec.template.spec.containers[0].env[22].valueFrom.secretKeyRef.key: Required value, spec.template.spec.containers[1].env[22].valueFrom.secretKeyRef.key: Required value] HOT 1
- Changing nextcloud.host vars results in crashloopbackoff HOT 1
- The smb protocol is not available
- Configure datadir & external database properly
- Upgrade via helm on K8s cluster not working HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from helm.