diff options
| author | MacroFake <falke.marco@gmail.com> | 2022-11-02 15:00:35 +0100 |
|---|---|---|
| committer | MacroFake <falke.marco@gmail.com> | 2022-11-02 15:00:40 +0100 |
| commit | 5274f324375fd31cf8507531fbc612765d03092f (patch) | |
| tree | c4bc2fef5e73dde1415ed47a637102fa96870754 /.python-version | |
| parent | 39f026b1ec17ef8100457ef46a1e4980767c0fe2 (diff) | |
| parent | 201b9a02fd9885ce40999e383cf75b8354d6ef26 (diff) | |
| download | bitcoin-5274f324375fd31cf8507531fbc612765d03092f.tar.xz | |
Merge bitcoin/bitcoin#26417: test: fix intermittent failure in feature_index_prune.py
201b9a02fd9885ce40999e383cf75b8354d6ef26 test: fix intermittent failure in feature_index_prune.py (Martin Zumsande)
Pull request description:
I can't reproduce the error from #26630 locally, but from analying the logs I think the problem is the following:
After calling `sync_blocks`, we didn't check that the indexes have caught up to the tip before performing the manual pruning. This could possibly lead to prune blockers with a lower height than the expected 2489, which do appear in the logs of the failed CI runs, e.g.
- `2022-10-27T21:14:17.703920Z [C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\src\validation.cpp:2395] [FlushStateToDisk] [prune] coinstatsindex limited pruning to height 2488` ([Cirrus](https://cirrus-ci.com/task/5443742333665280?logs=functional_tests#L2506))
So, this should be fixed by a call to `sync_index`.
Fixes #26330
ACKs for top commit:
brunoerg:
crACK 201b9a02fd9885ce40999e383cf75b8354d6ef26
Tree-SHA512: fb7023c9eb2ba6d0e69e059a401453cbdf63abc6804543dffcf36ba9f93c9cd13209e57aa5536d94b2e420c9d4cd0b1a7eff1adadd19aa7b3c33f592502e1bc0
Diffstat (limited to '.python-version')
0 files changed, 0 insertions, 0 deletions