[PATCH v3 06/12] dst: add basic capability support

Dean Marx dmarx at iol.unh.edu
Tue Sep 3 18:03:45 CEST 2024


On Wed, Aug 21, 2024 at 10:53 AM Juraj Linkeš <juraj.linkes at pantheon.tech>
wrote:

> A test case or suite may require certain capabilities to be present in
> the tested environment. Add the basic infrastructure for checking the
> support status of capabilities:
> * The Capability ABC defining the common capability API
> * Extension of the TestProtocol with required capabilities (each test
>   suite or case stores the capabilities it requires)
> * Integration with the runner which calls the new APIs to get which
>   capabilities are supported.
>
> Signed-off-by: Juraj Linkeš <juraj.linkes at pantheon.tech>
>

Looks all good to me, it was interesting to see how you've used abstract
methods in the Capability class. The only thing I noticed was it seems like
you wrote "dst" instead of "dts" in the commit message, otherwise:

Reviewed-by: Dean Marx <dmarx at iol.unh.edu>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/dev/attachments/20240903/687f1ead/attachment.htm>


More information about the dev mailing list