Re: [dpdk-ci] [PATCH 1/2] create_new_execution_file_from_tags: add test specifier

[Ali Alnubani <alialnu@oss.nvidia.com>]

> -----Original Message-----
> From: ci <ci-bounces@dpdk.org> On Behalf Of ohilyard@iol.unh.edu
> Sent: Friday, September 24, 2021 7:05 PM
> To: ci@dpdk.org
> Cc: aconole@redhat.com; Owen Hilyard <ohilyard@iol.unh.edu>
> Subject: [dpdk-ci] [PATCH 1/2] create_new_execution_file_from_tags: add
> test specifier
> 
> From: Owen Hilyard <ohilyard@iol.unh.edu>
> 
> BREAKING CHANGE: --tags syntax changed
> 
> Due to limitations of the argparse library, the tags syntax needed
> to be redone when the ability to specify individual tests was added.
> As such, the syntax has gone from "--tags core documentation" to "--tag
> core --tag documentation". This is because argparse does not allow
> multiple greedy variadic arguments in all versions.
> 
> added syntax to force adding a specific test
> 
> syntax: "--test nic_single_core_perf --test hello_world"
> 
> This flag was added to help facilitate periodic testing. The list of
> tests in the template file will be ignored by this argument. This allows
> a more automated way to run tests which should not be run under normal
> circumstances, such as rte_flow, which is lightly supported in many
> NICs.
> 
> Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
> ---

I suggest splitting this patch into 2 (one for modifying the tag usage, and one for adding the --test arg).

Regards,
Ali