Hi Andrew, I think the patch is for SSWAN plugin only, the VPP code is untouched. I don’t see any impact this patch will bring to IPsec source code, I cc’d Fan & Radu for suggestion.
Regards Kai From: vpp-dev@lists.fd.io <vpp-dev@lists.fd.io> On Behalf Of Andrew Yourtchenko Sent: Friday, October 14, 2022 11:04 AM To: vpp-dev@lists.fd.io Subject: Re: [vpp-dev] VPP SSWAN plugin git cherry pick to stable 22.10 Hi Kai, Having a second look at it - the plugin is offloading IPSec; So I would like some of the CSIT folks/IPSec maintainers to chime in with confirmation that they are comfortable with this… I think I might need to rescind the previous “no risk” assessment. --a On 14 Oct 2022, at 11:50, Ji, Kai <kai...@intel.com<mailto:kai...@intel.com>> wrote: Hi Andrew, Thank you for the reply, I’m agree with you, the https://gerrit.fd.io/r/c/vpp/+/36183 is way too big and risky for stable 22.10 now. In this case, can I kindly to ask you consider to merge VPP SSWAN plugin (https://gerrit.fd.io/r/c/vpp/+/36552) for 22.10 stable ? This patches only update the plugin and test script itself, and no dependences on vlibapi refactor and library fix. Regards Kai From: vpp-dev@lists.fd.io<mailto:vpp-dev@lists.fd.io> <vpp-dev@lists.fd.io<mailto:vpp-dev@lists.fd.io>> On Behalf Of Andrew Yourtchenko Sent: Friday, October 14, 2022 9:57 AM To: vpp-dev@lists.fd.io<mailto:vpp-dev@lists.fd.io> Subject: Re: [vpp-dev] VPP SSWAN plugin git cherry pick to stable 22.10 Kaj, *release manager hat on* we are past RC2 milestone - which means only critical fixes for CSIT tests go in. if the commit was just the plugin itself, in principle might be possible to entertain the idea, since a separate plugin is relatively low risk. But https://gerrit.fd.io/r/c/vpp/+/36183 is *far* too much risk to back port at this point in time, and is again not a fix per se. So unfortunately I will have to say “no” on this one. --a On 14 Oct 2022, at 10:43, Ji, Kai <kai...@intel.com<mailto:kai...@intel.com>> wrote: Hello We recently added fix plugin for vpp-sswan (https://gerrit.fd.io/r/c/vpp/+/36552) and I wonder if is it too late for this patch to be picked to stable 22.10 ? To make things more complicate, there is also a fix linked library patch for vpp-sswan https://gerrit.fd.io/r/c/vpp/+/37388 where need to be applied on top of vlibapi refactor (https://gerrit.fd.io/r/c/vpp/+/36183). Otherwise, the fix patch is no longer required if the vlibapi refactor is out of scope of stable 22.10. Regards Kai Ji --------------------------------------------------------------------- Intel Research and Development Ireland Ltd Co. Reg. #308263 Collinstown Industrial Park, Leixlip, County Kildare, Ireland
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#22037): https://lists.fd.io/g/vpp-dev/message/22037 Mute This Topic: https://lists.fd.io/mt/94322720/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/leave/1480452/21656/631435203/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
