Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More metrics #408

Merged
merged 2 commits into from
Dec 13, 2016
Merged

More metrics #408

merged 2 commits into from
Dec 13, 2016

Conversation

talex5
Copy link
Contributor

@talex5 talex5 commented Dec 12, 2016

Report number of tags, branches and open PRs

We recently had a case of a user accidentally pushing a large number of
tags to their repository and wondering why the CI was so busy building
things. It would have been easier to see what the cause was with these
metrics.

Report metrics for log messages

It's useful to see e.g. graphs of warning and error log messages.

This metric uses multiple labels (level and src), and includes a fix to make that work.

Thomas Leonard added 2 commits December 12, 2016 20:06
We recently had a case of a user accidentally pushing a large number of
tags to their repository and wondering why the CI was so busy building
things. It would have been easier to see what the cause was with these
metrics.

Signed-off-by: Thomas Leonard <thomas.leonard@docker.com>
It's useful to see e.g. graphs of warning and error log messages.

This metric uses multiple labels (level and src), and includes a fix to
make that work.

Signed-off-by: Thomas Leonard <thomas.leonard@docker.com>
@talex5
Copy link
Contributor Author

talex5 commented Dec 13, 2016

Travis failures seem unrelated (no deps changed in this PR):

The following dependencies couldn't be met:

  • comparelib -> type_conv < 112.02.00

@talex5 talex5 merged commit c9153d4 into moby:master Dec 13, 2016
@talex5 talex5 deleted the more-metrics branch December 13, 2016 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant