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.

Thanks,
Mark Michelson

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to