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

Update weaveworks/common for histogram buckets and logging changes #1669

Closed
wants to merge 2 commits into from

Conversation

bboreham
Copy link

@bboreham bboreham commented Jan 5, 2018

Update weaveworks/common to include weaveworks/common#81 "Extend histogram buckets for a server up to 100 seconds"
Previously we would clamp at 10 seconds, giving a misleading impression for things that can take a lot longer to respond.

Since I updated to latest, this also brings in:
- weaveworks/common#83 "Let servers set any gRPC options they like "
- weaveworks/common#84 "Don't log request body and headers for 502 errors"

Previously we would clamp at 10 seconds, giving a misleading
impression for things that can take a lot longer to respond.
This brings in the merged commits for:
- #81 "Extend histogram buckets for a server up to 100 seconds"
- #83 "Let servers set any gRPC options they like "
- #84 "Don't log request body and headers for 502 errors"
(#82 was relaced by #83)
@bboreham bboreham changed the title Extend default histogram buckets up to 100 seconds Update weaveworks/common for histogram buckets and logging changes Jan 28, 2018
@rade
Copy link
Member

rade commented Feb 4, 2018

What is happening with this?

@bboreham
Copy link
Author

Overtaken by #1742

@bboreham bboreham closed this Apr 17, 2018
@bboreham bboreham deleted the bigger-buckets branch April 17, 2018 13:54
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