[BGPCEP-919] exceptions and failures in CSIT Created: 29/Jul/20  Updated: 27/Aug/20

Status: In Review
Project: bgpcep
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Low
Reporter: Jamo Luhrsen Assignee: Venkatrangan Govindarajan
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

New test cases were added that are consisently failing on some exceptions
like this and this .

This job used to be stable and was not showing up on our CSIT failures when checking
for a release. It would be nice to have the failures resolved. Hopefully the exceptions
themselves can be addressed. Otherwise I think we have a way to put the exceptions
in to an allow list so they will be ignored, but that as a last resort.

 Comments   
Comment by Daniel De La Rosa [ 19/Aug/20 ]

gvrangan any updates on this issues? jluhrsen ecelgpand I would like to know

Comment by Venkatrangan Govindarajan [ 25/Aug/20 ]

danieldelarosa

The patch https://git.opendaylight.org/gerrit/c/integration/test/+/91759 has to be merged. That will fix most of the test failures.

There is one more failure because this patch is not merged https://git.opendaylight.org/gerrit/c/bgpcep/+/91971

Comment by Daniel De La Rosa [ 25/Aug/20 ]

Thanks gvrangan... jluhrsen and/or rovarga odd22 might be able to help with the last patch

Comment by Olivier Dugeon [ 26/Aug/20 ]

Patch https://git.opendaylight.org/gerrit/c/integration/test/+/91759 has been already merged. I also approved (+2) patch https://git.opendaylight.org/gerrit/c/bgpcep/+/91971 but, I have not the right to merge it (Note that Robert also validated this patch). So, is there someone with right authorisation who could merge it?

Comment by Venkatrangan Govindarajan [ 26/Aug/20 ]

Olivier, The branch is frozen, I think Super Committers can merge.

Comment by Daniel De La Rosa [ 26/Aug/20 ]

jluhrsen can you merge that last patch? 

https://git.opendaylight.org/gerrit/c/bgpcep/+/91971

 

Thank you all

 

Comment by Jamo Luhrsen [ 26/Aug/20 ]

this patch says BGPCEP-906, not 919. what's the story here?

Comment by Venkatrangan Govindarajan [ 27/Aug/20 ]

Yes, the fix is pending for merge, but tests were added much before. You can check the master (silicon) is green where all patches are merged. So you can skip as this is not a critical bug. and consier these two failures as "IGNORE" as the patches for fix are already in place.

Generated at Wed Feb 07 19:14:29 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.