View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003620 | GNUnet | transport service | public | 2015-01-26 23:52 | 2018-06-07 00:25 |
Reporter | Christian Grothoff | Assigned To | Christian Grothoff | ||
Priority | urgent | Severity | block | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
Product Version | Git master | ||||
Target Version | 0.11.0pre66 | Fixed in Version | 0.11.0pre66 | ||
Summary | 0003620: ./test_transport_api_slow_ats fails | ||||
Description | Amatus added this testcase to reproduce a peer connectivity issue. Should be investigated. | ||||
Tags | No tags attached. | ||||
|
01:24 <grothoffhome> verification works (as it should, doesn't need ATS), peer1 ATS suggestion _also_ seems to work, then peer 2 makes the ATS suggestion (so ATS service works), but transport drops it on the floor saying "I don't know that neighbour". 01:25 <grothoffhome> So the problem seems to be that transport 2 times out the 'neighbour' entry *before* ATS makes the suggestion (as ATS is delayed), and then is somehow unable to recover from there.01:25 <grothoffhome> So the problem seems to be that transport 2 times out the 'neighbour' entry *before* ATS makes the suggestion (as ATS is delayed), and then is somehow unable to recover from there. 01:28 <grothoffhome> So I suspect one (or more) of the timeouts in gnunet-service-transport_neighbours.c is to blame, as we timeout that entry while the (plugin-level) session is still active. 01:28 <grothoffhome> At that time, the plugin-level session continues to exist, effectively robbing peer1 from the opportunity to reconnect. 01:28 <grothoffhome> (as it is already connected) |
|
Ok, changed the logic a bit here to make transport _create_ a neighbor entry when ATS suggests an address (if none exists already). The timeouts are still an issue, as I think we have connection establishment still a bit backwards, but this should theoretically fix the testcase. Still, we might want to have a more general prohibition on neighbour entries being destroyed while plugin sessions are still active. |
|
Key problem that seems to be left: ATS makes address suggestion, but transport service "forgets" it, and ATS doesn't make another one. |
|
Test seems to now pass. |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-01-26 23:52 | Christian Grothoff | New Issue | |
2015-01-26 23:52 | Christian Grothoff | Status | new => assigned |
2015-01-26 23:52 | Christian Grothoff | Assigned To | => Matthias Wachs |
2015-02-03 01:28 | Christian Grothoff | Note Added: 0008840 | |
2015-02-03 01:30 | Christian Grothoff | Note Edited: 0008840 | |
2015-02-03 15:39 | Christian Grothoff | Note Added: 0008841 | |
2015-02-03 16:41 | Christian Grothoff | Note Added: 0008842 | |
2015-02-10 01:03 | Christian Grothoff | Assigned To | Matthias Wachs => Christian Grothoff |
2015-02-10 01:04 | Christian Grothoff | Note Added: 0008850 | |
2015-02-10 01:04 | Christian Grothoff | Status | assigned => resolved |
2015-02-10 01:04 | Christian Grothoff | Fixed in Version | => 0.11.0pre66 |
2015-02-10 01:04 | Christian Grothoff | Resolution | open => fixed |
2018-06-07 00:25 | Christian Grothoff | Status | resolved => closed |