On Tue, Mar 22, 2022 at 12:31 PM Mark Michelson <[email protected]> wrote: > > Hi everyone, > > Last month, commit d7514abe1 (acl-log: Log the direction (logical > pipeline) of the matching ACL.) was added to the main branch of OVN. > This commit was made because of the issue report at > https://bugzilla.redhat.com/show_bug.cgi?id=1992641 . > > At the time the change was added, it was not considered for a backport > to OVN 21.12 because it modifies the contents of ACL logs. The fear was > that if people have written parsers for ACL logs, the addition of a new > field could cause issues. This is not a very friendly thing to do during > a minor update. > > However, the change was made in an attempt to correct a deficiency in > the ACL logs. Without the change, it appears that there are duplicate > ACL logs since there is nothing to distinguish the pipeline on which the > ACL matched. Also, there is no documented contract about ACL log format; > they are intended to be human readable, and that's all. > > My question is if the community would be OK with backporting this fix to > OVN 21.12 in light of the change to the ACL log contents. Please let me > know what you think.
Hi Mark, No objections from me. Numan > > Thanks, > Mark Michelson > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
