Skip to content

Commit

Permalink
Code review changes
Browse files Browse the repository at this point in the history
  • Loading branch information
ioanaciont committed Jul 19, 2017
1 parent 2a6aba4 commit 02ebc9b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ func main() {
consumerAddrs := app.String(cli.StringOpt{
Name: "consumer_addr",
Value: "",
Desc: "Comma separated kafka proxy hosts for message consuming.",
Desc: "Comma separated kafka hosts for message consuming.",
EnvVar: "KAFKA_ADDRS",
})
consumerGroupID := app.String(cli.StringOpt{
Expand Down
4 changes: 2 additions & 2 deletions resources/healthchecks.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,10 @@ func (hc *NotificationsPushHealthcheck) GTG(w http.ResponseWriter, req *http.Req
func (hc *NotificationsPushHealthcheck) Check() fthealth.Check {
return fthealth.Check{
BusinessImpact: "Notifications about newly modified/published content will not reach this app, nor will they reach its clients.",
Name: "MessageQueueProxyReachable",
Name: "MessageQueueReachable",
PanicGuide: "https://sites.google.com/a/ft.com/universal-publishing/ops-guides/notifications-push",
Severity: 1,
TechnicalSummary: "Message queue proxy is not reachable/healthy",
TechnicalSummary: "Message queue is not reachable/healthy",
Checker: hc.checkAggregateMessageQueueReachable,
}
}
Expand Down

0 comments on commit 02ebc9b

Please sign in to comment.