confluentinc / ducktape

System integration and performance tests
16 stars 98 forks source link

Add the missing psutil dependency #420

Closed skorotkov closed 1 week ago

skorotkov commented 2 weeks ago

Now the psutil module used not only in the unit tests.

So move it to requirements.txt

Fixes the https://github.com/confluentinc/ducktape/issues/419

confluent-cla-assistant[bot] commented 2 weeks ago

:tada: All Contributor License Agreements have been signed. Ready to merge.
:white_check_mark: skorotkov
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

cla-assistant[bot] commented 2 weeks ago

CLA assistant check
All committers have signed the CLA.

sprsi commented 2 weeks ago

@skorotkov Was it throwing some error without this change?

skorotkov commented 2 weeks ago

Yes, see the https://github.com/confluentinc/ducktape/issues/419

ivandasch commented 2 weeks ago

@tsrivatsavs Could you please take a look? Also, the patch looks good to me