DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut
@ 2019-04-29 14:32 David Marchand
  2019-04-29 14:32 ` David Marchand
  2019-04-30 10:15 ` Gaëtan Rivet
  0 siblings, 2 replies; 6+ messages in thread
From: David Marchand @ 2019-04-29 14:32 UTC (permalink / raw)
  To: dev; +Cc: thomas, aconole, gaetan.rivet, stable

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(-)

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut
  2019-04-29 14:32 [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut David Marchand
@ 2019-04-29 14:32 ` David Marchand
  2019-04-30 10:15 ` Gaëtan Rivet
  1 sibling, 0 replies; 6+ messages in thread
From: David Marchand @ 2019-04-29 14:32 UTC (permalink / raw)
  To: dev; +Cc: thomas, aconole, gaetan.rivet, stable

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(-)

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut
  2019-04-29 14:32 [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut David Marchand
  2019-04-29 14:32 ` David Marchand
@ 2019-04-30 10:15 ` Gaëtan Rivet
  2019-04-30 10:15   ` Gaëtan Rivet
  2019-05-04 20:56   ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
  1 sibling, 2 replies; 6+ messages in thread
From: Gaëtan Rivet @ 2019-04-30 10:15 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, thomas, aconole, stable

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
> 

-- 
Gaëtan Rivet
6WIND

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut
  2019-04-30 10:15 ` Gaëtan Rivet
@ 2019-04-30 10:15   ` Gaëtan Rivet
  2019-05-04 20:56   ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
  1 sibling, 0 replies; 6+ messages in thread
From: Gaëtan Rivet @ 2019-04-30 10:15 UTC (permalink / raw)
  To: David Marchand; +Cc: dev, thomas, aconole, stable

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 1807 bytes --]

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
> 

-- 
Gaëtan Rivet
6WIND

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [dpdk-stable] [PATCH] test: clean remaining trace of removed devargs ut
  2019-04-30 10:15 ` Gaëtan Rivet
  2019-04-30 10:15   ` Gaëtan Rivet
@ 2019-05-04 20:56   ` Thomas Monjalon
  2019-05-04 20:56     ` Thomas Monjalon
  1 sibling, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2019-05-04 20:56 UTC (permalink / raw)
  To: David Marchand; +Cc: stable, Gaëtan Rivet, dev, aconole

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [dpdk-stable] [PATCH] test: clean remaining trace of removed devargs ut
  2019-05-04 20:56   ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
@ 2019-05-04 20:56     ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2019-05-04 20:56 UTC (permalink / raw)
  To: David Marchand; +Cc: stable, Gaëtan Rivet, dev, aconole

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="UTF-8", Size: 733 bytes --]

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




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-05-04 20:56 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-29 14:32 [dpdk-dev] [PATCH] test: clean remaining trace of removed devargs ut David Marchand
2019-04-29 14:32 ` David Marchand
2019-04-30 10:15 ` Gaëtan Rivet
2019-04-30 10:15   ` Gaëtan Rivet
2019-05-04 20:56   ` [dpdk-dev] [dpdk-stable] " Thomas Monjalon
2019-05-04 20:56     ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).