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

Invalid Probes #12

Closed
Fissium opened this issue Nov 15, 2024 · 1 comment · Fixed by #14
Closed

Invalid Probes #12

Fissium opened this issue Nov 15, 2024 · 1 comment · Fixed by #14
Assignees
Labels
bug Something isn't working

Comments

@Fissium
Copy link

Fissium commented Nov 15, 2024

Describe the bug a clear and concise description of what the bug is.

liveness, rediness and startup probes are invalid:

livenessProbe:
  enabled: true
  failureThreshold: 3
  initialDelaySeconds: 180
  periodSeconds: 10
  successThreshold: 1
  timeoutSeconds: 5

What's your helm version?

3.16.3

What's your kubectl version?

1.31.2

Chart version

1.1.4

Explain bug

The probes configuration includes an invalid enabled field, which is not recognized in Kubernetes. This causes deployment errors.

What you expected to happen?

No response

How to reproduce it?

No response

Enter the changed values of values.yaml?

No response

Enter the command that you execute when chart failing/misfunctioning

I use argocd

More info

No response

@Fissium Fissium added the bug Something isn't working label Nov 15, 2024
ialejandro added a commit that referenced this issue Nov 15, 2024
* Fix typo on NW_INT default value
* Improve probes

Closes #12
Closes #13
@ialejandro ialejandro mentioned this issue Nov 15, 2024
1 task
@ialejandro
Copy link
Member

On the next helm release, you can modify your custom probes. Idk if by default / works or not. You can disable or modify your custom probes (path, port, etc)

ialejandro added a commit that referenced this issue Nov 15, 2024
* Fix typo on NW_INT default value
* Improve probes
* Update docs

Closes #12
Closes #13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants