Ok for me. Just letting you know that I addressed all the comments so everything should be ready for merge.
BR Jaime. -----Original Message----- From: Abhijit Kumbhare <[email protected]> To: Yi Yang (杨燚)-云服务集团 <[email protected]>, Daniel De La Rosa <ddelar [email protected]>, [email protected] Cc: [email protected] <[email protected]>, [email protected] <[email protected] m>, [email protected] <[email protected]>, [email protected] <[email protected]>, [email protected] <[email protected]>, [email protected] <[email protected]>, [email protected] <abelur @linuxfoundation.org>, [email protected] <integrat [email protected]>, [email protected] <sfc-de [email protected]>, [email protected] <[email protected]>, [email protected] <[email protected]> Subject: Re: [integration-dev] SFC issue Date: Tue, 12 Mar 2019 11:08:52 -0700 Jamo, Jaime, Yi, Daniel & folks, Since we have roughly a solution on hand but it may take a few days to vet it out and to avoid the cycle of one more round of project approvals - I suggest we defer this issue to SR1 unless we do a respin of the Neon build for the BGPCEP issue. What do you folks think? Thanks, Abhijit On Mon, Mar 11, 2019 at 6:38 PM Abhijit Kumbhare <[email protected] > wrote: > Thanks for your comments Yi. > > On Mon, Mar 11, 2019 at 6:11 PM Yi Yang (杨燚)-云服务集团 <[email protected] > om> wrote: > > Abhijit, I think it will be better that Jaime himself decides how > > to proceed this, Jaime is current sfc committer. Jaime still need > > to fix comments in these two changes. > > > > 发件人: Abhijit Kumbhare [mailto:[email protected]] > > 发送时间: 2019年3月12日 1:59 > > 收件人: Jaime Caamano <[email protected]> > > 抄送: Abhijit Kumbhare <[email protected]>; Dayavanti > > Gopal Kamath <[email protected]>; Faseela K <fase > > [email protected]>; [email protected]; david.suarez.fuen > > [email protected]; Yi Yang (杨燚)-云服务集团 <[email protected]>; abelur@lin > > uxfoundation.org; [email protected]; sfc-dev@l > > ists.opendaylight.org; [email protected]; shague@redhat. > > com; [email protected]; Luis Gomez Palacios <[email protected]> > > 主题: Re: [integration-dev] SFC issue > > > > Thanks Jaime! When Yi/other SFC committer comes online - perhaps he > > can merge the change - and we can make another build? > > > > On Mon, Mar 11, 2019 at 10:46 AM Jaime Caamaño Ruiz <jcaamano@suse. > > de> wrote: > > > +Luis > > > > > > Tested patches [1] & [2] with sandbox job [3]. > > > > > > Patch [1] deals with an issue in the validator that correctly > > > changes > > > some error scenarios from HTTP 500 to HTTP 400. I suggest the > > > patches > > > to be included pre-SR1 if possible, since this is an API kinda > > > change. > > > > > > [1] https://git.opendaylight.org/gerrit/#/c/80769/ > > > [2] https://git.opendaylight.org/gerrit/#/c/80766/ > > > [3] https://jenkins.opendaylight.org/sandbox/job/jaicaa-sfc-csit- > > > 3node-docker-full-deploy-all-neon/2/ > > > > > > BR > > > Jaime. > > > > > > -----Original Message----- > > > From: Abhijit Kumbhare <[email protected]> > > > To: Dayavanti Gopal Kamath <[email protected]>, > > > Faseela K <[email protected]>, [email protected] > > > <bradya > > > [email protected]>, [email protected] > > > <david.suarez.fu > > > [email protected]>, Yi Yang -云服务集团 (杨燚) <[email protected]>, > > > [email protected] <[email protected]>, > > > integration- > > > [email protected] <[email protected] > > > g>, > > > [email protected] <[email protected]>, > > > [email protected] <[email protected]>, > > > [email protected] <[email protected]>, [email protected] <vthapa > > > r@redh > > > at.com>, [email protected] > > > Subject: SFC issue > > > Date: Mon, 11 Mar 2019 17:23:08 +0000 > > > > > > Adding SFC and integration lists: > > > > > > Thanks Yi! > > > > > > So, SFC folks (Brady, Jaime, etc.) & Jamo - from Yi's email this > > > issue > > > does seem "non-critical" - doesn't it? And we should mark the > > > issue in > > > the spreadsheet to be "Okay" and pick up the CSIT test changes > > > proposed > > > by Yi for SR1? > > > > > > Descriptions of HTTP codes: > > > > > > 400 Bad Request > > > The server cannot or will not process the request due to an > > > apparent > > > client error (e.g., malformed request syntax, size too large, > > > invalid > > > request message framing, or deceptive request routing). > > > > > > 404 Not Found > > > The requested resource could not be found but may be available in > > > the > > > future. Subsequent requests by the client are permissible. > > > > > > Yi - the change you make the following change in the check to be > > > from > > > 404 to 400: > > > > > > "Should Be Equal As Strings ${resp.status_code} 404" > > > > > > Also question - should we wild card this check to catch both HTTP > > > codes > > > "400" and "404"? > > > > > > > > > On 3/10/19, 8:33 PM, "Yi Yang -云服务集团 (杨燚)" <[email protected] > > > > > > > wrote: > > > > > > Thanks Jamo, I checked test log, it seems http return status > > > code > > > is changed, http return status code is 404 before, but not it is > > > 400 > > > now, sfc has not been changed very long, I submitted a gerrit > > > change ht > > > tps://git.opendaylight.org/gerrit/#/c/80756/ and pushed a sandbox > > > check > > > for sfc-csit-3node-docker-full-deploy-all-neon, you can take it > > > to > > > avoid sfc is excluded in neon release. > > > > > > -----邮件原件----- > > > 发件人: Abhijit Kumbhare [mailto:[email protected]] > > > 发送时间: 2019年3月11日 10:06 > > > 收件人: Jamo Luhrsen <[email protected]>; Yi Yang (杨燚)-云服务集团 > > > <yangyi > > > [email protected]>; Dayavanti Gopal Kamath <dayavanti.gopal.kamath@er > > > icsson > > > .com>; [email protected]; [email protected]; ddelarosa@luminan > > > etwork > > > s.com; Faseela K <[email protected]>; abelur@linuxfoundation > > > .org; > > > [email protected]; [email protected]; [email protected] > > > om; br > > > [email protected]; [email protected]; jcaama > > > no@sus > > > e.com > > > 抄送: Nishant Saurabh <[email protected]> > > > 主题: Re: 答复: SFC project - maintenance mode or active > > > > > > Understood. > > > > > > On 3/10/19, 6:06 PM, "Jamo Luhrsen" <[email protected]> > > > wrote: > > > > > > Yi, > > > > > > you need to use the logs server to see the logs, not the > > > links > > > from jenkins. I know it's > > > not intuitive, but it helps our jenkins server keep up > > > with > > > things to offload the logs. > > > here: > > > > > > https://logs.opendaylight.org/releng/vex-yul-odl-jenkins- > > > 1/sfc- > > > csit-3node-docker-full-deploy-all-neon/201/robot- > > > plugin/log.html.gz > > > > > > As for the "non-critical" tag you saw Abhijit, that is > > > just > > > default for robot to > > > show that. We don't use critical vs non-critical, so you > > > can > > > ignore it. > > > > > > As for only 2 failures not being a big deal, that may be > > > true, > > > but the reason we > > > are discussing is that those 2 failures are not there in > > > fluorine, so it's a regression > > > in neon and we try to avoid having regressions between > > > releases. > > > > > > Thanks to everyone for taking time out on their weekend. > > > > > > JamO > > > > > > On 3/10/19 5:33 PM, Yi Yang (杨燚)-云服务集团 wrote: > > > > I don't intend to fix it, but only 2 of 32 test cases > > > are > > > failed, so they aren't big issues, anybody is still active in sfc > > > project? I think it is more reasonable to delay them to neon SR1. > > > > > > > > I can check the issues, but I'm not sure I can fix them > > > because I didn't develop it very long. > > > > > > > > BTW, https://jenkins.opendaylight.org/releng/job/sfc-cs > > > it-3no > > > de-docker-full-deploy-all-neon/201/robot/report/report.html and > > > https:/ > > > /jenkins.opendaylight.org/releng/job/sfc-csit-3node-docker-full- > > > deploy- > > > all-neon/201/robot/report/log.html can't be accessed, can you > > > help send > > > correct links for failure log? > > > > > > > > -----邮件原件----- > > > > 发件人: Abhijit Kumbhare [mailto:abhijit.kumbhare@ericsson > > > .com] > > > > 发送时间: 2019年3月11日 8:18 > > > > 收件人: Jamo Luhrsen <[email protected]>; Dayavanti > > > Gopal > > > Kamath <[email protected]>; Luis Gomez Palacios > > > <ecel > > > [email protected]>; Anil Vishnoi <[email protected]>; Daniel De La > > > Rosa > > > <[email protected]>; Faseela K <[email protected] > > > >; > > > Anil Belur <[email protected]>; Sam Hague <shague@redhat > > > .com>; > > > Vishal Thapar <[email protected]>; Prem Sankar Gopannan <pgopann > > > an@lum > > > inanetworks.com>; Brady Johnson <[email protected]>; Yi > > > Yang > > > (杨燚)-云服务集团 <[email protected]>; David Suarez Fuentes > > > <david.suarez.fu > > > [email protected]>; Jaime Caamaño <[email protected]> > > > > 抄送: Nishant Saurabh <[email protected]> > > > > 主题: Re: SFC project - maintenance mode or active > > > > > > > > Adding some existing SFC members including Brady, > > > David, > > > Jaime and Yi Yang. I know Yi is an advisory committer. Assuming > > > Daya's > > > team can fix it at a later date (SR1 or some other SR and revive > > > SFC), > > > can one of you guys also check how serious is the SFC CSIT > > > failure? I > > > had already sent an email about this - with a chance for SFC to > > > respond > > > by Tuesday (but earlier will be better) before we have to move it > > > out > > > of Neon. > > > > > > > > The sign-off sheet is here: https://docs.google.com/spr > > > eadshe > > > ets/d/1sIoApv2fFp0wJcPK7m6lQDtjbUonRANrs5upSeD_DyI/edit#gid=14754 > > > 2669 > > > > > > > > The CSIT failure that Jamo mentions is here: https://je > > > nkins. > > > opendaylight.org/releng/job/sfc-csit-3node-docker-full-deploy- > > > all- > > > neon/201/. > > > > > > > > > > > > > > > > > _______________________________________________ > > > integration-dev mailing list > > > [email protected] > > > https://lists.opendaylight.org/mailman/listinfo/integration-dev _______________________________________________ sfc-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/sfc-dev
