diff mbox series

[ovs-dev,v4,3/4] system-common-macros: Check for drop action in datapath.

Message ID 20230804195847.1168385-4-eric@garver.life
State Superseded
Headers show
Series dpif: probe support for OVS_ACTION_ATTR_DROP | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Eric Garver Aug. 4, 2023, 7:58 p.m. UTC
Signed-off-by: Eric Garver <eric@garver.life>
---
 tests/system-common-macros.at | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Simon Horman Aug. 7, 2023, 12:35 p.m. UTC | #1
On Fri, Aug 04, 2023 at 03:58:46PM -0400, Eric Garver wrote:

nit: I would slightly prefer if there was a patch description here.
     Or, alternatively, perhaps this patch could be squashed into
     the following one.

> Signed-off-by: Eric Garver <eric@garver.life>
> ---
>  tests/system-common-macros.at | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at
> index 0077a8609c02..afc0df00a329 100644
> --- a/tests/system-common-macros.at
> +++ b/tests/system-common-macros.at
> @@ -359,3 +359,7 @@ m4_define([OVS_CHECK_IPROUTE_ENCAP],
>  # OVS_CHECK_CT_CLEAR()
>  m4_define([OVS_CHECK_CT_CLEAR],
>      [AT_SKIP_IF([! grep -q "Datapath supports ct_clear action" ovs-vswitchd.log])])
> +
> +# OVS_CHECK_DROP_ACTION()
> +m4_define([OVS_CHECK_DROP_ACTION],
> +    [AT_SKIP_IF([! grep -q "Datapath supports drop action" ovs-vswitchd.log])])
> -- 
> 2.39.0
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
diff mbox series

Patch

diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at
index 0077a8609c02..afc0df00a329 100644
--- a/tests/system-common-macros.at
+++ b/tests/system-common-macros.at
@@ -359,3 +359,7 @@  m4_define([OVS_CHECK_IPROUTE_ENCAP],
 # OVS_CHECK_CT_CLEAR()
 m4_define([OVS_CHECK_CT_CLEAR],
     [AT_SKIP_IF([! grep -q "Datapath supports ct_clear action" ovs-vswitchd.log])])
+
+# OVS_CHECK_DROP_ACTION()
+m4_define([OVS_CHECK_DROP_ACTION],
+    [AT_SKIP_IF([! grep -q "Datapath supports drop action" ovs-vswitchd.log])])