Skip to content
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

Startup probe errored: strconv.Atoi: parsing "http": invalid syntax #63

Closed
sseneca opened this issue Jan 14, 2021 · 4 comments
Closed

Comments

@sseneca
Copy link

sseneca commented Jan 14, 2021

Startup probe doesn't work for me.

Could be because of this line, which looks similar to this issue.

@webwurst
Copy link
Contributor

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?

@sseneca
Copy link
Author

sseneca commented Feb 27, 2021

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

@ErikLundJensen
Copy link

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) }}

@sseneca
Copy link
Author

sseneca commented Apr 12, 2021

1afd7dd fixed this issue but it would be nice to have Startup Probes even if using nginx. Closing this issue anyway

@sseneca sseneca closed this as completed Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants