Brandon Lo <b...@iol.unh.edu> writes: > Hi Tomek, > > I do not think we have a specific reason for disabling vhost. I can > look into enabling it and get back to you on that.
I did a test, but I had to hack up the dockerfile and compile script a bit. I removed the '--without-vhost' (I also modified it to pull dpdk and SPDK from their respective 'main'/'master' branches). The build completed successfully. I will update the BZ with this information. Can you try testing with the '--without-vhost' removed from the compile_spdk.sh line, and if that is successful we can modify the job and re-enable? > Thanks, > Brandon > > On Fri, Feb 12, 2021 at 4:18 AM Zawadzki, Tomasz > <tomasz.zawad...@intel.com> wrote: >> >> Hi, >> >> As Aaron noted this was result of rte_ethdev depending on rte_net. >> On SPDK side rte_net was included in default compilation with vhost >> component, but since vhost is disabled in the UNH lab tests with >> SPDK it showed up as missing. A fix for that is now submitted to >> SPDK: >> https://review.spdk.io/gerrit/c/spdk/spdk/+/6398 >> I'll let you know when it makes it's way to the v21.01.x branch. >> >> @Brandon Lo General question, is there particular reason to disable >> vhost [configure argument '--without-vhost'] in the UNH lab tests >> with SPDK ? >> I don't recall any identified issue and enabling it would increase >> the coverage of the compilation tests. >> >> Thanks, >> Tomek >> >> > -----Original Message----- >> > From: Aaron Conole <acon...@redhat.com> >> > Sent: Thursday, February 11, 2021 3:02 PM >> > To: Brandon Lo <b...@iol.unh.edu> >> > Cc: Zawadzki, Tomasz <tomasz.zawad...@intel.com>; Lincoln Lavoie >> > <lylav...@iol.unh.edu>; dpdk...@iol.unh.edu; c...@dpdk.org; dev@dpdk.org; >> > s...@lists.01.org >> > Subject: Re: [dpdklab] Re: [dpdk-ci] [dpdk-dev] [CI] SPDK compilation >> > failures >> > @ DPDK community lab >> > >> > Brandon Lo <b...@iol.unh.edu> writes: >> > >> > > Hi again everyone, >> > > >> > > I have checked the pipelines with SPDK branch v21.01.x on the main DPDK >> > branch. >> > > It still seems to have an issue with compilation, and I have attached >> > > a log of a Fedora SPDK compilation. >> > > There are some undefined references to "rte_ether_unformat_addr" >> > > I will continue to look into this. If you have any ideas on how to fix >> > > this, please let me know. >> > >> > Looks like rte_ethdev depends on rte_net - maybe I missed something. >> > >> > Brandon, can we disable this test for the time being since it's been >> > failing for >> > a while now? Can you also send me the container image / definitions you're >> > using so that I can help work on this? >> > >> > > Thanks, >> > > Brandon >> > > >> > > On Tue, Feb 9, 2021 at 11:07 AM Brandon Lo <b...@iol.unh.edu> wrote: >> > >> >> > >> Hi everyone, >> > >> >> > >> I will adjust the branches and watch over the first few pipelines to >> > >> make sure everything goes smoothly. >> > >> >> > >> Thanks for the update, >> > >> Brandon >> > >> >> > >> On Tue, Feb 9, 2021 at 10:13 AM Aaron Conole <acon...@redhat.com> >> > wrote: >> > >> > >> > >> > "Zawadzki, Tomasz" <tomasz.zawad...@intel.com> writes: >> > >> > >> > >> > > Hi Lincoln, >> > >> > > >> > >> > > >> > >> > > >> > >> > > That patch in question is now merged to branch v21.01.x. >> > >> > > >> > >> > >> > >> > Good to know - I do still see a failure in the IOL job (even from a >> > >> > few hours ago). I suppose the lab side might need some adjustment, >> > too? >> > >> > >> > >> > > >> > >> > > The builds performed for latest SPDK and SPDK LTS, against >> > >> > > dpdk-main branch seem to be passing. Would love to hear if this >> > >> > > is what you are seeing on your end too. >> > >> > > >> > >> > > >> > >> > > >> > >> > > Thanks, >> > >> > > >> > >> > > Tomek >> > >> > > >> > >> > > >> > >> > > >> > >> > > From: Lincoln Lavoie <lylav...@iol.unh.edu> >> > >> > > Sent: Monday, February 8, 2021 5:21 PM >> > >> > > To: Zawadzki, Tomasz <tomasz.zawad...@intel.com> >> > >> > > Cc: Aaron Conole <acon...@redhat.com>; Brandon Lo >> > >> > > <b...@iol.unh.edu>; dpdk...@iol.unh.edu; c...@dpdk.org; >> > >> > > dev@dpdk.org; s...@lists.01.org >> > >> > > Subject: Re: [dpdk-ci] [dpdk-dev] [CI] SPDK compilation failures >> > >> > > @ DPDK community lab >> > >> > > >> > >> > > >> > >> > > >> > >> > > Thanks Tomek, >> > >> > > >> > >> > > >> > >> > > >> > >> > > Can you let us know when the merge happens and we'll make sure >> > >> > > the next set of builds pass or see what the next failure is. :-P >> > >> > > >> > >> > > >> > >> > > >> > >> > > Cheers, >> > >> > > Lincoln >> > >> > > >> > >> > > >> > >> > > >> > >> > > On Mon, Feb 8, 2021 at 11:03 AM Zawadzki, Tomasz >> > <tomasz.zawad...@intel.com> wrote: >> > >> > > >> > >> > > Hi Aaron, >> > >> > > >> > >> > > Thank you for reporting this ! >> > >> > > >> > >> > > This is an issue with rte_power now depending on rte_ethdev, which >> > was resolved on latest SPDK. >> > >> > > >> > >> > > I believe that UNH lab verifies DPDK patches against SPDK branch >> > >> > > for latest release. Which after the very recent SPDK release, >> > >> > > would be v21.01.x: >> > >> > > https://github.com/spdk/spdk/tree/v21.01.x >> > >> > > >> > >> > > The fix has been backported to that branch and should be merged >> > shortly: >> > >> > > https://review.spdk.io/gerrit/c/spdk/spdk/+/6320 >> > >> > > >> > >> > > Thanks, >> > >> > > Tomek >> > >> > > >> > >> > > > -----Original Message----- >> > >> > > > From: dev <dev-boun...@dpdk.org> On Behalf Of Aaron Conole > >> > >> > > Sent: Monday, February 8, 2021 4:21 PM > To: Brandon Lo >> > >> > > <b...@iol.unh.edu> > Cc: dpdk...@iol.unh.edu; c...@dpdk.org; >> > >> > > dev@dpdk.org; s...@lists.01.org > Subject: [dpdk-dev] [CI] SPDK >> > >> > > compilation failures @ DPDK community lab > > Greetings, > > >> > >> > > I've noticed that recently SPDK compilation in the UNH community >> > >> > > lab seems > to be failing, and I don't see an obvious reason for >> > >> > > the >> > failure. >> > >> > > > The logs haven't been too helpful - it appears that there is a >> > >> > > symbol that isn't > available when linking. >> > >> > > > >> > >> > > > Job details (for example): >> > >> > > > https://lab.dpdk.org/results/dashboard/results/results- >> > >> > > > >> > >> > > >> > uploads/test_runs/2363efb43157465db3228c34c00ebd57/log_upload_fil >> > >> > > e/20 > 21/2/dpdk_f6f2d2240153_15524_2021-02-04_22-59-59_NA.zip >> > >> > > > >> > >> > > > Is it possible to turn on more verbose logging during the >> > >> > > compilation of > SPDK? Maybe show the arguments to the compiler >> > for the specific object? >> > >> > > > Maybe the SPDK folks can see something obviously wrong? >> > >> > > > >> > >> > > > Thanks, >> > >> > > > -Aaron >> > >> > >> > >> >> > >> >> > >> -- >> > >> >> > >> Brandon Lo >> > >> >> > >> UNH InterOperability Laboratory >> > >> >> > >> 21 Madbury Rd, Suite 100, Durham, NH 03824 >> > >> >> > >> b...@iol.unh.edu >> > >> >> > >> www.iol.unh.edu >>
