[OPNFLWPLUG-938] Regression in Cbench test Created: 04/Sep/17 Updated: 27/Sep/21 Resolved: 12/Oct/17 |
|
| Status: | Resolved |
| Project: | OpenFlowPlugin |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Luis Gomez | Assignee: | Jozef Bacigal |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 9088 |
| Description |
|
We have regression in cbench test in nitrogen: By the time it could well be this patch: |
| Comments |
| Comment by Luis Gomez [ 04/Sep/17 ] |
|
From the robot log it looks like controller disconnects the cbench connection just after the test starts: controller msgbuf_read() = 0: closed connection ... exiting |
| Comment by Jozef Bacigal [ 04/Sep/17 ] |
|
Luis can you please provide some LOGs at least on INFO level ? It seems log are on ERROR level for OFP. Thx |
| Comment by Luis Gomez [ 05/Sep/17 ] |
|
Right, normal logs are skipped for perf reasons, I will run once with normal logging and post log here. |
| Comment by Luis Gomez [ 05/Sep/17 ] |
|
Here is the karaf log with INFO level: |
| Comment by Jozef Bacigal [ 13/Sep/17 ] |
|
Assuming the NPE this should help. |
| Comment by Tomas Slusny [ 13/Sep/17 ] |
|
Also, this one (what was already merged on master) will help this issue: https://git.opendaylight.org/gerrit/#/c/62667/ |
| Comment by Luis Gomez [ 13/Sep/17 ] |
|
Unless I am missing something none of the above patches fixes the issue with Cbench, see karaf logs below for the patch test: https://git.opendaylight.org/gerrit/#/c/62775/ https://git.opendaylight.org/gerrit/#/c/62667/ |
| Comment by Tomas Slusny [ 13/Sep/17 ] |
|
Hmm, looks like the null pointer exception is still there, that means the auxiliary id is null and not FeaturesReply |
| Comment by Jozef Bacigal [ 14/Sep/17 ] |
|
Updated patch set. |
| Comment by Luis Gomez [ 14/Sep/17 ] |
|
Still not quite there yet: |
| Comment by Luis Gomez [ 18/Sep/17 ] |
|
Last patch in nitrogen works: |
| Comment by Luis Gomez [ 18/Sep/17 ] |
|
As discussed this morning, we should get this fix in nitrogen release so better make it blocker. |
| Comment by Luis Gomez [ 18/Sep/17 ] |
|
Nitrogen cbench test result: |
| Comment by Luis Gomez [ 04/Oct/17 ] |
|
This is now happening in carbon and oxygen, we need to cherry-pick this patch. |
| Comment by Luis Gomez [ 05/Oct/17 ] |
|
Currently oxygen works: Maybe because of this patch: So it is only Carbon failing: |
| Comment by Kit Lou [ 12/Oct/17 ] |
|
Luis, Please verify issue is fixed in latest carbon SR2 build. Thanks! |
| Comment by Luis Gomez [ 12/Oct/17 ] |
|
Yes, this is fixed now: |