Conversation
iameli
left a comment
There was a problem hiding this comment.
Sure but you can't rely on it being there any time soon, we don't roll pods much. What would you think of using this in catalyst-api instead? We're going to need something like it for the load balancing work. https://github.com/cloudfoundry/gosigar
4e8eac9 to
1485ebb
Compare
1485ebb to
f41ee2c
Compare
|
@emranemran Do we still need this with all the system monitoring stuff landing in catabalancer? |
yes! this helps with per-process cpu/mem usage during specific operations in catalyst-api and the inter-parsing of these logs with catalyst-api helps narrow down issues quicker. |
f41ee2c to
3995922
Compare
|
hmm do we think we don't need to reference it by kernel version? |
Dockerfile
Outdated
| gstreamer1.0-tools gstreamer1.0-plugins-good gstreamer1.0-plugins-base gstreamer1.0-plugins-bad \ | ||
| procps \ | ||
| vnstat \ | ||
| linux-tools-common linux-tools-generic linux-tools-5.4.0-148-generic \ |
There was a problem hiding this comment.
linux-tools-generic alone should be enough. it should pick up correct linux-tools-<version>-generic based on installed kernel version.
There was a problem hiding this comment.
Got it, updating.
3995922 to
e53e270
Compare
|
Tests failing though, you. should rebase |
e53e270 to
2beaf4e
Compare
toprunning on our pods and get access to better info.