[NETVIRT-1296] Transaction leak in VpnOpStatusListener Created: 04/Jun/18 Updated: 04/Oct/18 Resolved: 04/Oct/18 |
|
| Status: | Resolved |
| Project: | netvirt |
| Component/s: | vpnmanager |
| Affects Version/s: | Oxygen-SR2 |
| Fix Version/s: | Oxygen-SR3, Fluorine, Neon |
| Type: | Bug | Priority: | High |
| Reporter: | Michael Vorburger | Assignee: | Josh Hershberg |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
NETVIRT-1089 found this in vpnmanager which needs to be plugged: DataBroker : newWriteOnlyTransaction()
89x transactions opened here, which are not closed:
(...) org.opendaylight.controller.md.sal.binding.impl.BindingDOMDataBrokerAdapter.newWriteOnlyTransaction(BindingDOMDataBrokerAdapter.java:69)
(...) org.opendaylight.netvirt.vpnmanager.VpnOpStatusListener.lambda$update$5(VpnOpStatusListener.java:219) org.opendaylight.infrautils.jobcoordinator.internal.JobCoordinatorImpl$MainTask.runWithUncheckedExceptionLogging(JobCoordinatorImpl.java:416)
|
| Comments |
| Comment by Michael Vorburger [ 04/Jun/18 ] |
|
shague the new automatic assignment is cool and worked in |
| Comment by Michael Vorburger [ 05/Jun/18 ] |
|
jluhrsen in chat with skitt he rightfully requested we set the right Affects Version here - are your NETVIRT-1089 CSIT jobs where I copy/pasted above from running on master (Fluorin) or stable/oxygen? |
| Comment by Jamo Luhrsen [ 05/Jun/18 ] |
|
they are Fluorine. The job name contains the branch if you are ever curious when looking at the |
| Comment by Michael Vorburger [ 30/Jul/18 ] |
|
https://git.opendaylight.org/gerrit/#/c/71799/ must have fixed this on master, so this would need a cherry-pick of that from master to Oxygen... I'm therefore removing Fluorine from the Fix Version and keeping only Oxygen SR3. |