[CONTROLLER-1765] trace:transactions should suppress false TransactionChain positives, by white list, or showing only x2+ ? Created: 31/Aug/17 Updated: 25/Jul/23 Resolved: 31/Jul/18 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | mdsal |
| Affects Version/s: | None |
| Fix Version/s: | Fluorine |
| Type: | Improvement | ||
| Reporter: | Michael Vorburger | Assignee: | Michael Vorburger |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||||||||||||||
| Description |
|
The new tool from |
| Comments |
| Comment by Michael Vorburger [ 31/Aug/17 ] |
|
+Tom, if you want to chime in here: I'm leaning towards implementing the solution to suppress all open TransactionChains that are only x1 and only start showing x2+ from trace:transactions... but ONLY for TransactionChain, NOT for the Read[Write]Transaction! Maybe with a -a kind of CLI flag, to be able to see "all", if you really want to. Thoughts, approvals, objections? |
| Comment by Michael Vorburger [ 11/Apr/18 ] |
|
|
| Comment by Michael Vorburger [ 28/May/18 ] |
|
https://git.opendaylight.org/gerrit/#/c/72362/ is adding sorting, which helps in spotting the bad from the harmless more easily. |
| Comment by Michael Vorburger [ 30/Jul/18 ] |
|
This has come up in NETVIRT-1089 again... it's hard for a fully automated (CSIT) test to programmatically know / distinguish when there is a real problem. I'll therefore add a new option to that CLI command to suppress leaks with less than N (option) open TXs. |
| Comment by Michael Vorburger [ 30/Jul/18 ] |
|
https://git.opendaylight.org/gerrit/#/c/74652/ should do the trick... Once that is merged, jluhrsen can use it in NETVIRT-1089 with some "sensible value" (7 ?) for the new minOpenTransactions option to trace:transactions, and we're done here. |