-
Notifications
You must be signed in to change notification settings - Fork 54
Dump requests histogram + filter namespaces #66
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
Conversation
Look interesting, let me check that and give you feedback asap. |
@caruccio Please run EDIT: I fixed the lint in this commit : 6efcc90 |
Yeah, makes sense. I see you already did all this on #67!Feel free to merge that and close this one. |
@caruccio So can you just sherry-pick my changes on your branch so I can merge it? Thanks. |
@rchakode just did a full merge. cherry pick was also conflicting on my side. |
@caruccio |
This PR does two things:
requests
histogram based on suggestions from Allows accounting based on resource.requests instead real usage #64