View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003165 | GNUnet | namestore service | public | 2013-12-08 20:00 | 2013-12-24 20:53 |
| Reporter | Christian Grothoff | Assigned To | LRN | ||
| Priority | high | Severity | major | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | Git master | ||||
| Target Version | 0.10.0 | Fixed in Version | 0.10.0 | ||
| Summary | 0003165: namestore tests fail on W32 | ||||
| Description | https://gnunet.org/buildbot/builders/polot-win7-i386/builds/1058/steps/tests%20namestore/logs/stdio | ||||
| Tags | No tags attached. | ||||
|
|
22:37 <grothoffhome> LRN: are you aware of 0003165? 22:37 <LRN> yes 22:37 <LRN> i am baffled by it 22:38 <grothoffhome> :-(. 22:38 <LRN> Let's see if my recent changes fix anything 22:38 <LRN> because this test passes on my machine 22:38 <grothoffhome> I was baffled by the fact that this one test would fail for W32 and pass elsewhere, as I don't see how namestore should be W32-specific... 22:39 <grothoffhome> But good to know that it is passing on your system, so maybe it is something on the VM itself. 22:39 <LRN> until 5 minutes ago my system had +5 patches on top of SVN HEAD 22:39 <LRN> This might have been the reason :) 22:40 <grothoffhome> Well, the next run of the BB will tell... |
|
|
Found the problem. Tests passed on my machine, because i have somewhat more extensive pre-test cleanup procedures than buildslave does. In particular, these procedures remove namestore tmp directory, the one with test_sqlite.* database files. These files are supposed to be removed by the testcase, but it fails to do so, because files are in use by the namestore service. For some reason the testcase tries to clean them up (by using GNUNET_DISK_directory_remove()) before arm & namestore are dead. Subsequent tests fail because database from previous test leaks into them. |
|
|
Moving cleanup to the end of main() alleviates the problem for me (only tried this on one test though). |
|
|
Fixed in r31231 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2013-12-08 20:00 | Christian Grothoff | New Issue | |
| 2013-12-08 20:00 | Christian Grothoff | Status | new => assigned |
| 2013-12-08 20:00 | Christian Grothoff | Assigned To | => Matthias Wachs |
| 2013-12-08 22:39 | Christian Grothoff | Note Added: 0007789 | |
| 2013-12-08 22:40 | Christian Grothoff | Note Edited: 0007789 | |
| 2013-12-09 03:20 | LRN | Note Added: 0007793 | |
| 2013-12-09 03:20 | LRN | Note Edited: 0007793 | |
| 2013-12-09 15:41 | LRN | Note Added: 0007798 | |
| 2013-12-11 17:55 | LRN | Note Added: 0007843 | |
| 2013-12-11 17:56 | LRN | Status | assigned => resolved |
| 2013-12-11 17:56 | LRN | Fixed in Version | => Git master |
| 2013-12-11 17:56 | LRN | Resolution | open => fixed |
| 2013-12-11 17:56 | LRN | Assigned To | Matthias Wachs => LRN |
| 2013-12-11 20:10 | Christian Grothoff | Fixed in Version | Git master => 0.10.0 |
| 2013-12-24 20:53 | Christian Grothoff | Status | resolved => closed |
