On Fri, Mar 31, 2023 at 02:10:35PM +0200, Dumitru Ceara wrote: > On 3/31/23 13:52, Simon Horman wrote: > > On Tue, Mar 21, 2023 at 01:59:06PM -0400, Mark Michelson wrote: > >> Prior to this commit, ACLs were evaluated and acted on in a single > >> stage. With this commit, evaluation of ACLs and acting on an ACL's > >> decision are separated into two stages. > >> > >> The acl_eval stage checks the ACL match and will set a bit to indicate > >> the verdict of the ACL. The acl_action stage then checks the relevant > >> bits to determine how to proceed. If no ACLs are matched, then the > >> default ACL action is taken. > >> > >> A couple of notes about updated tests: > >> - For test cases where I just had to increment a table number, I changed > >> the check so the table numbers are masked. This should prevent similar > >> changes from being needed later. > >> - The port security test changes may seem odd. The issue here is that > >> the ls_out_apply_port_sec table number changed from 9 to 10. This > >> means that this table's flows now sort to a lower position than > >> before. This is why the check had to change for this test. > >> > >> Signed-off-by: Mark Michelson <[email protected]> > > > > Hi Mark, > > > > this does not appear to apply on top of main any more. > > Hi Simon, > > I didn't review this series yet but something that helps when dealing > with patches that don't apply on top of main anymore is checking the > branch 0-day bot created from the patchwork series: > > https://github.com/ovsrobot/ovn/tree/series_347327 > > It's: https://github.com/ovsrobot/ovn/tree/series_<series-id>
Thanks, I did not know that :) _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
