On 4/27/21 1:00 PM, Adrian Moreno wrote:
> I guess this was a cut-and-paste error from set_mpls_ttl
>
> Signed-off-by: Adrian Moreno <[email protected]>
> ---
> lib/ofp-actions.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/ofp-actions.c b/lib/ofp-actions.c
> index 0342a228b..91d810860 100644
> --- a/lib/ofp-actions.c
> +++ b/lib/ofp-actions.c
> @@ -3850,7 +3850,7 @@ static void
> format_SET_MPLS_TC(const struct ofpact_mpls_tc *a,
> const struct ofpact_format_params *fp)
> {
> - ds_put_format(fp->s, "%sset_mpls_ttl(%s%"PRIu8"%s)%s",
> + ds_put_format(fp->s, "%sset_mpls_tc(%s%"PRIu8"%s)%s",
> colors.paren, colors.end, a->tc,
> colors.paren, colors.end);
> }
>
Hi. Thanks for the fix!
Could you, please, add a unit test for this issue?
You can use following commit as a reference:
0062a04d8701 ("tests: Add parse-flow tests for MPLS fields.")
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
