test: clean remaining trace of removed devargs ut

Message ID 1556548365-22148-1-git-send-email-david.marchand@redhat.com (mailing list archive)
State Accepted, archived
Headers
Series test: clean remaining trace of removed devargs ut |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS

Commit Message

David Marchand April 29, 2019, 2:32 p.m. UTC
  This test has been removed by the commit 83945fbd7c49 ("test: remove
devargs unit tests") which left some trace in meson and reintroduced in
autotest by the second commit 9eabcb682493 ("test: update autotest
list").

Fixes: 83945fbd7c49 ("test: remove devargs unit tests")
Fixes: 9eabcb682493 ("test: update autotest list")
Cc: stable@dpdk.org

Signed-off-by: David Marchand <david.marchand@redhat.com>
---
 app/test/autotest_data.py | 6 ------
 app/test/meson.build      | 1 -
 2 files changed, 7 deletions(-)
  

Comments

Gaëtan Rivet April 30, 2019, 10:15 a.m. UTC | #1
Hello David,

On Mon, Apr 29, 2019 at 04:32:45PM +0200, David Marchand wrote:
> This test has been removed by the commit 83945fbd7c49 ("test: remove
> devargs unit tests") which left some trace in meson and reintroduced in
> autotest by the second commit 9eabcb682493 ("test: update autotest
> list").
> 
> Fixes: 83945fbd7c49 ("test: remove devargs unit tests")
> Fixes: 9eabcb682493 ("test: update autotest list")
> Cc: stable@dpdk.org
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>

AFAICS, those are the only remaining references to the devargs autotest,

Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>

> ---
>  app/test/autotest_data.py | 6 ------
>  app/test/meson.build      | 1 -
>  2 files changed, 7 deletions(-)
> 
> diff --git a/app/test/autotest_data.py b/app/test/autotest_data.py
> index 72c56e5..c9c39d0 100644
> --- a/app/test/autotest_data.py
> +++ b/app/test/autotest_data.py
> @@ -303,12 +303,6 @@
>          "Report":  None,
>      },
>      {
> -        "Name":    "Devargs autotest",
> -        "Command": "devargs_autotest",
> -        "Func":    default_autotest,
> -        "Report":  None,
> -    },
> -    {
>          "Name":    "Link bonding autotest",
>          "Command": "link_bonding_autotest",
>          "Func":    default_autotest,
> diff --git a/app/test/meson.build b/app/test/meson.build
> index 80cdea5..a56f60b 100644
> --- a/app/test/meson.build
> +++ b/app/test/meson.build
> @@ -196,7 +196,6 @@ fast_non_parallel_test_names = [
>          'cryptodev_openssl_asym_autotest',
>          'cryptodev_sw_mvsam_autotest',
>          'delay_us_sleep_autotest',
> -        'devargs_autotest',
>          'distributor_autotest',
>          'eventdev_common_autotest',
>          'eventdev_octeontx_autotest',
> -- 
> 1.8.3.1
>
  
Thomas Monjalon May 4, 2019, 8:56 p.m. UTC | #2
30/04/2019 12:15, Gaëtan Rivet:
> Hello David,
> 
> On Mon, Apr 29, 2019 at 04:32:45PM +0200, David Marchand wrote:
> > This test has been removed by the commit 83945fbd7c49 ("test: remove
> > devargs unit tests") which left some trace in meson and reintroduced in
> > autotest by the second commit 9eabcb682493 ("test: update autotest
> > list").
> > 
> > Fixes: 83945fbd7c49 ("test: remove devargs unit tests")
> > Fixes: 9eabcb682493 ("test: update autotest list")
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> 
> AFAICS, those are the only remaining references to the devargs autotest,
> 
> Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>

Applied, thanks
  

Patch

diff --git a/app/test/autotest_data.py b/app/test/autotest_data.py
index 72c56e5..c9c39d0 100644
--- a/app/test/autotest_data.py
+++ b/app/test/autotest_data.py
@@ -303,12 +303,6 @@ 
         "Report":  None,
     },
     {
-        "Name":    "Devargs autotest",
-        "Command": "devargs_autotest",
-        "Func":    default_autotest,
-        "Report":  None,
-    },
-    {
         "Name":    "Link bonding autotest",
         "Command": "link_bonding_autotest",
         "Func":    default_autotest,
diff --git a/app/test/meson.build b/app/test/meson.build
index 80cdea5..a56f60b 100644
--- a/app/test/meson.build
+++ b/app/test/meson.build
@@ -196,7 +196,6 @@  fast_non_parallel_test_names = [
         'cryptodev_openssl_asym_autotest',
         'cryptodev_sw_mvsam_autotest',
         'delay_us_sleep_autotest',
-        'devargs_autotest',
         'distributor_autotest',
         'eventdev_common_autotest',
         'eventdev_octeontx_autotest',