Skip to content

Conversation

@andywerner
Copy link

No description provided.

@andywerner andywerner marked this pull request as ready for review April 16, 2021 09:00
Copy link
Contributor

@amrsoll amrsoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So what happened to psutils? It would have been easy to install it through the usb keys as well.

Comment on lines +430 to +433
"wait_for_cpu_below() cpu_load: %s, cpu_max: %s, cpu_load > cpu_max: %s",
cpu_load,
cpu_max,
cpu_load > cpu_max,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make it a debug log and without the function name

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@amrsoll
Copy link
Contributor

amrsoll commented Apr 16, 2021

But why not use psutil then?

@ahmed-mrbeam ahmed-mrbeam force-pushed the develop branch 2 times, most recently from 202baee to bcd6b27 Compare September 28, 2021 13:05
@ahmed-mrbeam ahmed-mrbeam force-pushed the develop branch 2 times, most recently from c49d8eb to 5d6659c Compare October 1, 2021 10:15
@ahmed-mrbeam ahmed-mrbeam force-pushed the develop branch 2 times, most recently from 1f28493 to 6c61b35 Compare December 8, 2021 15:55
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.

3 participants