> -----Original Message-----
> From: Thomas Monjalon <[email protected]>
> Sent: Friday, February 2, 2024 10:55
> To: [email protected]; [email protected]; Oleksandr Kolomeiets
> <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> Ori Kam <[email protected]>; Dariusz Sosnowski <[email protected]>
> Subject: Re: [PATCH v2] app/testpmd: support updating flow rule actions
>
> 01/02/2024 10:59, Oleksandr Kolomeiets:
> > "flow actions_update" updates a flow rule specified by a rule ID with
> > a new action list by making a call to "rte_flow_actions_update()":
> >
> > flow actions_update {port_id} {rule_id}
> > actions {action} [/ {action} [...]] / end [user_id]
> >
> > Creating, updating and destroying a flow rule:
> >
> > testpmd> flow create 0 group 1 pattern eth / end actions drop / end
> > Flow rule #0 created
> > testpmd> flow actions_update 0 0 actions queue index 1 / end
> > Flow rule #0 updated with new actions
> > testpmd> flow destroy 0 rule 0
> > Flow rule #0 destroyed
>
> Why not a simple "flow update" command name?
+1. This would also make it consistent with async version of this command -
"flow queue {port_id} update ...".
