<div dir="ltr">Hello, my OVN gurus!<div><br></div><div>I've been looking at ACLs with logging.</div><div>Using Meters is a great way to keep the ovn-controllers from getting overwhelmed with ACL log events.</div><div>Since multiple ACL rows with logging enabled can refer to the same Meter, I ran a little experiment</div><div>to better understand how that behaves [1].</div><div><br></div><div>My findings were not ideal. A 'noisy' ACL match can consume all the events allowed by the</div><div>meter, shadowing logs for other ACL(s) that also use the same meter. The thought of maintaining</div><div>a meter row per ACL at the NB side seems like overkill. A much better approach would be to</div><div>leverage northd to take care of this on behalf of the ACL.</div><div><br></div><div>Following the same line of thought that Dumitru/Numan had when splitting the SB port group per</div><div>datapath [2], I would like to pursue a change where a single meter on the NB may represent</div><div>multiple meters at the SB side.</div><div><br></div><div>In summary, I would love to hear your opinion on this. I can't imagine a case where ACLs</div><div>sharing the same meter would benefit from getting smothered by a noisy neighbor. But maybe</div><div>I'm not being creative enough. ;) Adding a boolean in the ACL to explicitly say "protect my log</div><div>events from other users of the same Meter" would allow for both cases and that is what I'm</div><div>thinking of [3] at the moment.</div><div><br></div><div>What do you think?</div><div><br></div><div>Thanks,</div><div><br></div><div>-- flaviof</div><div><br></div><div>[1]: <a href="https://github.com/flavio-fernandes/ovsdbapp_playground/blob/acl_meter_issue/scripts/acl_meter.sh">https://github.com/flavio-fernandes/ovsdbapp_playground/blob/acl_meter_issue/scripts/acl_meter.sh</a></div><div>[2]: <a href="https://github.com/ovn-org/ovn/commit/0db5cbf65283fb41a7d28e5d0ad2f8ac14725d73">https://github.com/ovn-org/ovn/commit/0db5cbf65283fb41a7d28e5d0ad2f8ac14725d73</a><br>[3]: <a href="https://github.com/flavio-fernandes/ovn/commit/f265b01fcb7866bdc697eccc151b7dedebd0085b">https://github.com/flavio-fernandes/ovn/commit/f265b01fcb7866bdc697eccc151b7dedebd0085b</a></div></div>