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

Including CPU & Memory tracking function #1

Closed
mijikim opened this issue Jul 29, 2015 · 5 comments
Closed

Including CPU & Memory tracking function #1

mijikim opened this issue Jul 29, 2015 · 5 comments

Comments

@mijikim
Copy link

mijikim commented Jul 29, 2015

Hi, I think it will be great if minigun run command produces cpu & memory usage report.

I am testing this by incorporating usage module into minigun-core module and supplying pid in the JSON file that 'minigun run' command will read. If pid is not given, run command should work without collecting cpu & memory data. Would appreciate your opinion on this!

@hassy
Copy link
Member

hassy commented Jul 30, 2015

Hello @mijikim. Is this for CPU and RAM usage of Minigun itself or of the target? I think that either way, one should be using external tools to monitor the processes, e.g. ps if you're watching Minigun, or collectd or New Relic on your servers. There may be an option to talk to the agent on the host that is running the target process to get the info, but that wouldn't be something I'd prioritise myself right now, although if you wanted to write a plugin for Minigun to do this, I'd be happy to help.

@mijikim
Copy link
Author

mijikim commented Jul 30, 2015

@hassy This will be for cpu and ram of the target. The use case will be me running a node application on my machine and collecting data of that application by supplying pid. And I am testing usage module to get the data. So instead of embedding this in minigun-core, you suggest a plugin for this feature?

@hassy
Copy link
Member

hassy commented Jul 31, 2015

@mijikim I'm guessing you want to correlate the exact point in the load-test with where CPU/RAM usage spikes? Yes, it would be really useful to have this data available in the reports that Minigun generates, but for this feature to be built right into Minigun it would need to:

  1. Be easy to set up (easier than just using another tool like ps or collectd or a New Relic agent etc)
  2. Work with targets that are not on the same machine (because most tests run against targets on remote hosts, not localhost to localhost)
  3. Work with targets written in any language, not just Node (so it cannot rely on the usage module)

But if it were an optional plugin just for testing Node.js apps running locally, then the above doesn't necessarily need to apply.

If you want to give it a go, I'd be happy to help (we'd need to figure out a plugin interface for Minigun).

@mijikim
Copy link
Author

mijikim commented Jul 31, 2015

@hassy I see your point and agree that this should be a plugin. I am happy to create a plugin for a purpose to record cpu&memory on targets that are on the same machine. I would greatly appreciate your help as this will be my first time contributing to open source project :)

@hassy
Copy link
Member

hassy commented Aug 4, 2015

@mijikim I'll need to have a think about a plugin interface for Minigun. Please add suggestions and ideas to: #3 :)

@hassy hassy closed this as completed Aug 4, 2015
tomgco pushed a commit to tomgco/minigun that referenced this issue Jan 6, 2016
hassy added a commit that referenced this issue Jan 16, 2023
hassy pushed a commit that referenced this issue May 2, 2023
hassy added a commit that referenced this issue Jun 6, 2023
* feat: aggregate metrics by scenario name

* Update README.md

Add section for aggregateByName usage

Co-authored-by: Hassy Veldstra <h@veldstra.org>
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

No branches or pull requests

2 participants