Re: [ovs-dev] [PATCH ovn v2 2/2] missed during test simplification.

2024-04-04 Thread Mark Michelson
Thanks Jacob and Ales. I fixed the subject line and fixed the typo below (as well as s/referances/references/) and pushed both patches in the series to main. On 3/28/24 05:04, Ales Musil wrote: On Tue, Mar 26, 2024 at 5:32 PM Jacob Tanenbaum wrote: Hi Jacob, thank you for the v2. As 0-day

Re: [ovs-dev] [PATCH ovn v2 2/2] missed during test simplification.

2024-03-28 Thread Ales Musil
On Tue, Mar 26, 2024 at 5:32 PM Jacob Tanenbaum wrote: Hi Jacob, thank you for the v2. As 0-day suggests the commit subject should stat with capital letter. the testing was changed to remove the hardcoded table numbers from > flows. Two referances to table=28 where missed, this patch corrects

Re: [ovs-dev] [PATCH ovn v2 2/2] missed during test simplification.

2024-03-26 Thread 0-day Robot
Bleep bloop. Greetings Jacob Tanenbaum, 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. checkpatch: WARNING: The subject summary should start with a capital. Subject: missed during test

[ovs-dev] [PATCH ovn v2 2/2] missed during test simplification.

2024-03-26 Thread Jacob Tanenbaum
the testing was changed to remove the hardcoded table numbers from flows. Two referances to table=28 where missed, this patch corrects that and adds the correct variable name to the flow. Fixes: f614335abca1 ("tests: Remove table numbers from "action parsing".") Signed-off-by: Jacob Tanenbaum