[ovs-dev] [PATCH ovn] Fix the test case "ACL label usage" for northd-ddlog.

Mark Michelson mmichels at redhat.com
Fri Aug 27 18:30:54 UTC 2021


Acked-by: Mark Michelson <mmichels at redhat.com>

I went ahead and merged the change to master since it is trivial.

On 8/20/21 2:38 PM, numans at ovn.org wrote:
> From: Numan Siddique <numans at ovn.org>
> 
> Commit in the fixes tag added an extra space in the action
> causing the test case to fail.
> 
> Fixes: b5b434b4d49("Intern all the matches and actions.")
> Signed-off-by: Numan Siddique <numans at ovn.org>
> ---
>   northd/ovn_northd.dl | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/northd/ovn_northd.dl b/northd/ovn_northd.dl
> index 9cf4c373b..38a0f79d4 100644
> --- a/northd/ovn_northd.dl
> +++ b/northd/ovn_northd.dl
> @@ -2819,7 +2819,7 @@ for (&SwitchACL(.sw = sw, .acl = acl, .has_fair_meter = fair_meter)) {
>                */
>               var __action = if (acl.label != 0) {
>                   i"${rEGBIT_CONNTRACK_COMMIT()} = 1; ${rEGBIT_ACL_LABEL()} = 1; "
> -                " ${rEG_LABEL()} = ${acl.label}; ${acl_log}next;"
> +                "${rEG_LABEL()} = ${acl.label}; ${acl_log}next;"
>               } else {
>                   i"${rEGBIT_CONNTRACK_COMMIT()} = 1; ${acl_log}next;"
>               } in Flow(.logical_datapath = sw._uuid,
> 



More information about the dev mailing list