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

httpGet liveness and readiness probes in helm-chart deployment #1903

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gaal-dev
Copy link

Please, use httpGet liveness and readiness probes in helm-chart deployment instead of radial/busyboxplus:curl

helm/mockserver/values.yaml
service:
test:
image: radial/busyboxplus:curl

The Docker image https://hub.docker.com/r/radial/busyboxplus is 9 years old

Docker complains that its format is deprecated
`$ docker pull radial/busyboxplus
Using default tag: latest

What's next:
View a summary of image vulnerabilities and recommendations → docker scout quickview radial/busyboxplus
Error response from daemon: [DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/radial/busyboxplus:latest to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at https://docs.docker.com/go/deprecated-image-specs/`

https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/
My simple change:
readinessProbe: httpGet: path: /status port: 1080 livenessProbe: httpGet: path: /status port: 1080

Using service-test.yaml also has a negative side effect when MockServer starts and tests pass, but the curl container still cannot start and everything crashes after some timeout (30 minutes in my case). I had this scenario on a Jenkins server with K8S.

@gaal-dev gaal-dev force-pushed the helm-chart-deployment-probes branch 2 times, most recently from 6047a1d to d627c34 Compare September 12, 2024 11:22
dependabot bot and others added 2 commits September 12, 2024 13:32
Bumps [maven-enforcer-plugin](https://github.com/apache/maven-enforcer) from 3.1.0 to 3.2.1.
- [Release notes](https://github.com/apache/maven-enforcer/releases)
- [Commits](apache/maven-enforcer@enforcer-3.1.0...enforcer-3.2.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-enforcer-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@gaal-dev gaal-dev force-pushed the helm-chart-deployment-probes branch from d627c34 to 05c3e59 Compare September 12, 2024 11:32
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

Successfully merging this pull request may close these issues.

1 participant