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

Output max CPU utilization on summarize-benchmark script JSON output format #330

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

aemous
Copy link
Contributor

@aemous aemous commented Jan 22, 2025

Description of changes:

  • Add max cpu and standard deviation of max cpu to the JSON output format. This is already included in the table output format.
  • Fix a rare bug in the benchmark script where the process is non-zombie during the if-check on line 72, but becomes a zombie in the try-block in lines 79-81.

Description of tests:

  • Ran the benchmark and summarize scripts and confirmed the max CPU metric (and standard deviation) are now output in JSON output mode.
  • After many executions of the benchmark script, I no longer observe the rare bug of an unhandled ZombieProcess exception thrown in the try block.

@aemous aemous merged commit 9a71d52 into boto:develop Jan 24, 2025
39 checks passed
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.

2 participants