test: Remove windows workaround in authproxy #27418

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2304-test-win-no-work- changing 1 files +0 −6

maflcko commented at 11:27 AM on April 4, 2023: member

I wonder if the windows issues have also been fixed by bumping the server timeout in commit 88134fcee998849d9569368d6cefbfa487dc82d1.

I guess the only way to find out and try.

Note that even with the workaround, the issue would still happen occasionally: https://github.com/bitcoin/bitcoin/issues/18623
test: Remove windows workaround in authproxy fa584b4d01
DrahtBot commented at 11:27 AM on April 4, 2023: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
DrahtBot added the label Tests on Apr 4, 2023
fanquake commented at 12:58 PM on April 4, 2023: member

Concept ACK
maflcko commented at 5:13 PM on April 4, 2023: member

re-ran the windows test 12 times with only one failure: https://cirrus-ci.com/task/6093556574584832 . Which looks like an unrelated issue.
fanquake approved
fanquake commented at 10:52 AM on April 5, 2023: member

ACK fa584b4d01ca055488a10b280f45dca323b474d4
fanquake merged this on Apr 5, 2023
fanquake closed this on Apr 5, 2023
maflcko deleted the branch on Apr 5, 2023
sidhujag referenced this in commit fe8f9e6cb2 on Apr 5, 2023
bitcoin locked this on Apr 4, 2024

Type	Reviewers
ACK	fanquake

Contributors

Labels

Linked (view graph)

#18623 Investiage RPC connection lifetime issues

test: Remove windows workaround in authproxy #27418

Reviews