Skip to content

Send expected errors to expected bucket#7650

Merged
jridgewell merged 2 commits intoampproject:masterfrom
jridgewell:errortracker-expected-errors
Feb 21, 2017
Merged

Send expected errors to expected bucket#7650
jridgewell merged 2 commits intoampproject:masterfrom
jridgewell:errortracker-expected-errors

Conversation

@jridgewell
Copy link
Copy Markdown
Contributor

I want them out of my error logs.

Copy link
Copy Markdown
Member

@cramforce cramforce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update dashboards.

@jridgewell
Copy link
Copy Markdown
Contributor Author

Please update dashboards.

@erwinmombay, how do I do that? Actually, you'll probably need to do that since I don't have admin permissions.

@erwinmombay
Copy link
Copy Markdown
Member

@jridgewell will do.

@jridgewell
Copy link
Copy Markdown
Contributor Author

Merge afterwards, please.

errorType += "-canary"
isCanary = true;
}
if r.URL.Query().Get("ex") == "1" {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please increment version number in app.yaml

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@erwinmombay
Copy link
Copy Markdown
Member

ive updated the filters on the metrics

@jridgewell jridgewell merged commit 9d412e4 into ampproject:master Feb 21, 2017
mrjoro pushed a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
* Send expected errors to expected bucket

* Increment errortracker version
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.

3 participants