Skip to content

Commit

Permalink
Mark as non-resilient, reduce severity
Browse files Browse the repository at this point in the history
  • Loading branch information
Tamas Molnar committed Mar 2, 2018
1 parent 85f779a commit c7d8448
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions healthcheck.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ func (h *HealthCheck) readQueueCheck() fthealth.Check {
return fthealth.Check{
ID: "read-message-queue-proxy-reachable",
Name: "Read Message Queue Proxy Reachable",
Severity: 1,
Severity: 2,
BusinessImpact: "Related content from published Next videos will not be processed, clients will not see them within content.",
TechnicalSummary: "Read message queue proxy is not reachable/healthy",
PanicGuide: h.panicGuide,
Expand All @@ -59,7 +59,7 @@ func (h *HealthCheck) writeQueueCheck() fthealth.Check {
return fthealth.Check{
ID: "write-message-queue-proxy-reachable",
Name: "Write Message Queue Proxy Reachable",
Severity: 1,
Severity: 2,
BusinessImpact: "Related content from published Next videos will not be processed, clients will not see them within content.",
TechnicalSummary: "Write message queue proxy is not reachable/healthy",
PanicGuide: h.panicGuide,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ metadata:
chartVersion: "{{ .Chart.Version | trunc 63 }}"
app: {{.Values.service.name}}
visualize: "true"
hasHealthcheck: "{{ .Values.service.hasHealthcheck }}"
hasHealthcheck: "{{ .Values.service.hasHealthcheck }}"
isResilient: "{{ .Values.service.isResilient }}"
spec:
ports:
- port: 8080
Expand Down
1 change: 1 addition & 0 deletions helm/upp-next-video-content-collection-mapper/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
service:
name: "" # The name of the service, should be defined in the specific app-configs folder.
hasHealthcheck: "true"
isResilient: "false"
replicaCount: 2
image:
repository: coco/upp-next-video-content-collection-mapper
Expand Down

0 comments on commit c7d8448

Please sign in to comment.