View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001776 | GNUnet | DHT service | public | 2011-09-16 20:50 | 2011-09-18 14:59 |
| Reporter | Christian Grothoff | Assigned To | Christian Grothoff | ||
| Priority | urgent | Severity | major | Reproducibility | random |
| Status | closed | Resolution | fixed | ||
| Summary | 0001776: dht multipeer test fails SOMETIMES | ||||
| Description | Also, the test simply fails, but doesn't give any real error message: Sep 16 20:46:09-709463 test-dht-multipeer-30175 WARNING Found file size 3481426 for hostkeys, expect hostkeys to be size 914 Sep 16 20:46:09-709497 test-dht-multipeer-30175 WARNING Will read 3809 hostkeys from file Sep 16 20:46:09-716015 test-dht-multipeer-30175 WARNING Read 5 total peers in topology FAIL: test_dht_multipeer | ||||
| Steps To Reproduce | Run 'make check' in src/dht/ a few times (done on pixel). | ||||
| Additional Information | I first thought this related to the new HAVE_UID_FOR_TESTING and/or HAVE_REPLY_BLOOMFITLER #defines, but right now it seems to happen even with the "old" code where I keep both of those fields in the messages. | ||||
| Tags | No tags attached. | ||||
| related to | 0001770 | closed | Christian Grothoff | DHT includes unique id (uid) in each message, likely for no good reason |
| related to | 0001769 | closed | Christian Grothoff | DHT checks peer-bloomfilter for replies |
|
|
I've now seen the testcase pass AND fail with any combination of the defines from 0001770/0001769. It doesn't fail all the time, so maybe this just slipped through before? |
|
|
Ok, this is related to 0001770. The unique ID is used to create the hash into the 'recent' hash map (around line 3285), which causes trouble if UIDs are always 0. |
|
|
Fixed in SVN 16920. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2011-09-16 20:50 | Christian Grothoff | New Issue | |
| 2011-09-16 20:50 | Christian Grothoff | Status | new => assigned |
| 2011-09-16 20:50 | Christian Grothoff | Assigned To | => mrwiggles |
| 2011-09-16 20:50 | Christian Grothoff | Relationship added | related to 0001769 |
| 2011-09-16 20:51 | Christian Grothoff | Relationship added | related to 0001770 |
| 2011-09-16 21:30 | Christian Grothoff | Note Added: 0004593 | |
| 2011-09-17 22:37 | Christian Grothoff | Note Added: 0004595 | |
| 2011-09-17 22:37 | Christian Grothoff | Note Edited: 0004595 | |
| 2011-09-17 23:37 | Christian Grothoff | Note Added: 0004597 | |
| 2011-09-17 23:37 | Christian Grothoff | Status | assigned => resolved |
| 2011-09-17 23:37 | Christian Grothoff | Resolution | open => fixed |
| 2011-09-17 23:37 | Christian Grothoff | Assigned To | mrwiggles => Christian Grothoff |
| 2011-09-17 23:37 | Christian Grothoff | Status | resolved => assigned |
| 2011-09-17 23:37 | Christian Grothoff | Status | assigned => resolved |
| 2011-09-18 14:59 | Christian Grothoff | Status | resolved => closed |
