Skip to content

Commit

Permalink
fix(refinery): Update health check (#359)
Browse files Browse the repository at this point in the history
## Which problem is this PR solving?

Updates health checks to use new Refinery health check endpoint. Also
removes any default values and sets some new values based on how
Refinery now operates
  • Loading branch information
TylerHelmuth authored May 7, 2024
1 parent 837251e commit d526b71
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
2 changes: 1 addition & 1 deletion charts/refinery/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: refinery
description: Chart to deploy Honeycomb Refinery
type: application
version: 3.0.0-hnyinternal.0
version: 3.0.0-hnyinternal.1
appVersion: 3.0.0
keywords:
- refinery
Expand Down
9 changes: 3 additions & 6 deletions charts/refinery/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -113,15 +113,12 @@ spec:
path: /alive
port: data
initialDelaySeconds: 10
periodSeconds: 10
failureThreshold: 3
periodSeconds: 3
readinessProbe:
httpGet:
path: /alive
path: /ready
port: data
initialDelaySeconds: 0
periodSeconds: 3
failureThreshold: 5
periodSeconds: 1
resources:
{{- toYaml .Values.resources | nindent 12 }}
volumes:
Expand Down
1 change: 1 addition & 0 deletions charts/refinery/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ resources:
memory: 500Mi

image:
# temp repo value to get CI to pass
repository: tylerhelmuthhc/refinery
pullPolicy: IfNotPresent
# Overrides the image tag whose default is the chart appVersion.
Expand Down

0 comments on commit d526b71

Please sign in to comment.