Re: [ovs-dev] [PATCH ovn 2/2] tests: Remove table numbers from test action parsing.

2024-02-06 Thread Ales Musil
On Tue, Feb 6, 2024 at 6:49 PM Xavier Simonart wrote: > Hi Ales > > Thanks for the review. > I'll add NEXT(x, y), with x being ingress or egress, and y being lflow > table number, and use them where necessary. > I'll post a v2 when the 26 first patches from the series get merged so > that ovs

Re: [ovs-dev] [PATCH ovn 2/2] tests: Remove table numbers from test action parsing.

2024-02-06 Thread Xavier Simonart
Hi Ales Thanks for the review. I'll add NEXT(x, y), with x being ingress or egress, and y being lflow table number, and use them where necessary. I'll post a v2 when the 26 first patches from the series get merged so that ovs robot can run happily. With this and your 26 patches, we should be

Re: [ovs-dev] [PATCH ovn 2/2] tests: Remove table numbers from test action parsing.

2024-02-06 Thread Ales Musil
On Tue, Feb 6, 2024 at 3:45 PM Xavier Simonart wrote: > This patch uses the recently introduced macros defining openflow table > numbers. > > Signed-off-by: Xavier Simonart > --- > Hi Xavier, thank you for the follow up to remove the rest of hardcoded table numbers. There are a few places

Re: [ovs-dev] [PATCH ovn 2/2] tests: Remove table numbers from test action parsing.

2024-02-06 Thread 0-day Robot
Bleep bloop. Greetings Xavier Simonart, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (tests/ovn.at). error: could not build fake

[ovs-dev] [PATCH ovn 2/2] tests: Remove table numbers from test action parsing.

2024-02-06 Thread Xavier Simonart
This patch uses the recently introduced macros defining openflow table numbers. Signed-off-by: Xavier Simonart --- tests/ovn-macros.at | 4 + tests/ovn.at| 230 +++- 2 files changed, 122 insertions(+), 112 deletions(-) diff --git