[NETCONF-463] Exception in ListenerAdapter: java.lang.UnsupportedOperationException: Cannot create XMLStreamReader or XMLEventReader from a org.codehaus.stax2.io.Stax2ByteArraySource Created: 04/Sep/17 Updated: 15/Mar/19 Resolved: 13/Sep/17 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | restconf-nb |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Josh Hershberg | Assignee: | Stephen Kitt |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Attachments: |
|
| External issue ID: | 9092 |
| Priority: | High |
| Description |
|
Stack trace below and logs attached. Marking this as critical since it is killing some pretty core functionality for us. Netvirt's neutron ml2 driver, networking-odl, relies by default on websocket notifications for port statuses. 2017-09-04 08:21:20,536 | ERROR | n-dispatcher-202 | DataChangeListener | 205 - org.opendaylight.controller.sal-clustering-commons - 1.6.0.SNAPSHOT | Error notifying listener org.opendaylight.netconf.sal.streams.listeners.ListenerAdapter@798a509 |
| Comments |
| Comment by Josh Hershberg [ 04/Sep/17 ] |
|
Attachment karaf.log.gz has been added with description: karaf.log |
| Comment by Jamo Luhrsen [ 04/Sep/17 ] |
|
It's believe this bug is causing openstack instances 4+ minutes to become |
| Comment by Kit Lou [ 05/Sep/17 ] |
|
Is this a blocker for nitrogen? Please confirm. Thanks! |
| Comment by Robert Varga [ 05/Sep/17 ] |
|
Yes, it is. It boils down to jackson not being able to parse a string XML – this is so not needed. We already have the data in NormalizedNode format, hence instead of transcoding XML text, that should be thrown away and normal NormalizedNode->JSON encoding should be done. That will ditch jackson, rendering the issue impossible. |
| Comment by Kit Lou [ 06/Sep/17 ] |
|
Do we have an ETA on when we can merge a patch into nitrogen? Thanks! |
| Comment by Stephen Kitt [ 07/Sep/17 ] |
|
https://git.opendaylight.org/gerrit/62811 is a workaround, but it might not be appropriate because of the bundle refresh issues we had with org.json in the past. I'm working on a proper serialization which will avoid the XML-to-JSON issue altogether, but it will take a bit more time. |
| Comment by Stephen Kitt [ 07/Sep/17 ] |
|
https://git.opendaylight.org/gerrit/62849 for Nitrogen. |
| Comment by Sam Hague [ 12/Sep/17 ] |
|
(In reply to Stephen Kitt from comment #6) The fix looks good. It was verified in sandbox. |
| Comment by Stephen Kitt [ 13/Sep/17 ] |
|
The workaround has been merged. https://bugs.opendaylight.org/show_bug.cgi?id=9154 tracks the JSON serialisation rewrite. |