search

avocado-framework / avocado-vt

Avocado VT Plugin
https://avocado-vt.readthedocs.org/
Other
83 stars 241 forks source link

env_process: Configure SMT on Power by a Setuper #3967

Closed bgartzi closed 2 weeks ago

bgartzi commented 1 month ago

Refactor the preprocess/postprocess steps in virttest.env_process configuring the PowerPC systems SMT into a Setuper subclass. Use that class later on env_process and register it in the setup_manager.

ID: 2436

bgartzi commented 1 month ago

I just force-pushed a little change removing the creation of the migration_setup variable, which it wasn't used in this scope.

I noticed that the migration_setup variable shouldn't exist even in env_process, so I opened another PR proposing a cleanup. See https://github.com/avocado-framework/avocado-vt/pull/3968

bgartzi commented 1 month ago

Hey @YongxueHong, could you review this patch please?

bgartzi commented 1 month ago

Rebased onto the latest master to avoid possible conflicts.

YongxueHong commented 1 month ago

Hi @cliping I am not sure whether it will affect the virsh migration case. Could you help to review it? Thank you in advance. CC @dzhengfy @chunfuwen