openflow: fix multi-part reply and drain pending msgs - #54
Conversation
Fix 'Broken pipe' errors in ovs-vswitchd.log caused by the exporter closing OpenFlow connections prematurely. - Add drainPendingMessages() helper to discard pending OVS messages - Modify getFlowStats() to loop until OFPSF_REPLY_MORE is unset - Add drain call to both getFlowStats() and GetAggregateStats() - Add unit tests for multi-part handling and drain functionality Closes: OSPRH-32382 Signed-off-by: Abhiram R N <abhiramrn@gmail.com> (cherry picked from commit 38c64db) Signed-off-by: Abhiram R N <abhiramrn@gmail.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Fix 'Broken pipe' errors in ovs-vswitchd.log caused by the exporter closing OpenFlow connections prematurely.
Closes: OSPRH-32382
(cherry picked from commit 38c64db)