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]>
> 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 <
>> [email protected]>; [email protected];
>> [email protected]; Yi Yang (杨燚)-云服务集团 <[email protected]>;
>> [email protected]; [email protected];
>> [email protected]; [email protected];
>> [email protected]; [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 <[email protected]>
>> 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]>,
>> [email protected] <[email protected]>,
>> [email protected] <[email protected]>,
>> [email protected] <[email protected]>, [email protected] <vthapar@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@ericsson
>> .com <dayavanti.gopal.kamath@ericsson%0b.com>>; [email protected];
>> [email protected]; ddelarosa@luminanetwork
>> s.com; Faseela K <[email protected]>; [email protected];
>> [email protected]; [email protected]; [email protected]; br
>> [email protected]; [email protected]; jcaamano@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-csit-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:[email protected]]
>>         > 发送时间: 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 <[email protected]>;
>> Vishal Thapar <[email protected]>; Prem Sankar Gopannan <pgopannan@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/spreadshe
>> ets/d/1sIoApv2fFp0wJcPK7m6lQDtjbUonRANrs5upSeD_DyI/edit#gid=147542669
>>         >
>>         > The CSIT failure that Jamo mentions is here: https://jenkins.
>> 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

Reply via email to