Skip to content
This repository has been archived by the owner on May 29, 2018. It is now read-only.

Commit

Permalink
Exclude router from list of erroring apps
Browse files Browse the repository at this point in the history
Exclude the `router` application from the list of erroring apps. The
router acts as a reverse proxy for all requests to all other
applications, so it's not useful to show it in this dashboard.

Do this by changing the query to only use logs that are tagged with
`application`, which excludes logs from the router.

I also think this is more eloquent than using all logs not tagged
`nginx`.
  • Loading branch information
mattbostock committed Jan 21, 2016
1 parent c912eee commit 0a51101
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dashboards/top_10_erroring_apps.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"query": {
"list": {
"0": {
"query": "@fields.status: [500 TO 599] AND NOT @tags:nginx AND NOT @fields.application:\"govuk-cdn-logs-monitor\"",
"query": "@fields.status: [500 TO 599] AND @tags:application AND NOT @fields.application:\"govuk-cdn-logs-monitor\"",
"alias": "",
"color": "#7EB26D",
"id": 0,
Expand Down

0 comments on commit 0a51101

Please sign in to comment.