On Mon, 21 Apr 2025 17:21:34 +0530 <kirankum...@marvell.com> wrote: > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c > index 998527590f..c4359e965d 100644 > --- a/app/test-pmd/cmdline_flow.c > +++ b/app/test-pmd/cmdline_flow.c > @@ -652,6 +652,7 @@ enum index { > ACTION_METER_STATE, > ACTION_OF_DEC_NW_TTL, > ACTION_OF_POP_VLAN, > + ACTION_OF_POP_ETAG, > ACTION_OF_PUSH_VLAN, > ACTION_OF_PUSH_VLAN_ETHERTYPE, > ACTION_OF_SET_VLAN_VID,
Putting it in the middle of VLAN stuff seems odd. Maybe put these in alphabetical order.