This definitely cleans things up a lot! Reviewed-by: Nicholas Pratte <npra...@iol.unh.edu>
<snip> > @@ -217,9 +218,7 @@ def get_supported_capabilities( > ) > if cls.capabilities_to_check: > capabilities_to_check_map = cls._get_decorated_capabilities_map() > - with TestPmdShell( > - sut_node, privileged=True, disable_device_start=True > - ) as testpmd_shell: > + with TestPmdShell() as testpmd_shell: This is probably a needless concern, but should this change technically be in the last patch in order for it to stand on its own? Not sure if that really matters or not, but figured I'd bring it up.
