-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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 gosigar in beats #4316
Update gosigar in beats #4316
Conversation
8f66462
to
4ac4513
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WFG
@@ -5,6 +5,7 @@ This project adheres to [Semantic Versioning](http://semver.org/). | |||
## [Unreleased] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we make a release to use in this project?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tagged the release with v0.3.0
.
If you use govendor fetch github.com/elastic/gosigar/...@v0.3.0
it will update all gosigar and add the tag info to the vendor file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
7747f2f
to
fddb96a
Compare
Can you add something into the beats changelog to indicate the change to the memory metrics due to |
a35b450
to
47ca374
Compare
@andrewkroh Done, thanks for the reminder. |
Update Beats to include the latest changes from the
elastic/gosigar
library, especially the fix: elastic/gosigar#71.This closes #4202