On 5 Sep 2025, at 11:07, Ales Musil via dev wrote:
> Add a way to set IDL txn as read only, that is useful in cases when > we want to ensure that the program doesn't write anything into the > txn. This is done via assert because this is considered a bug which > should be fixed in the IDL caller. > > Signed-off-by: Ales Musil <[email protected]> > --- > More context on why, this would be used by OVN to ensure > that certain I-P nodes do not write into the database when > they shouldn't Hi Ales, I haven’t done a full code review yet, but I do want to raise an objection regarding the assert() approach used here. To me, this doesn’t make much sense, why should the whole dbserver crash just because a client is misbehaving? It feels like this should be handled more gracefully, maybe with an error return or logging instead of taking down the entire process. What do you think? //Eelco _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
