ci: Set --nocleanup for Windows functional tests #23043

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2109-ciWinCleanup changing 1 files +1 −1

MarcoFalke commented at 7:42 AM on September 20, 2021: member

Example: https://cirrus-ci.com/task/6292402450202624

ci: Set --nocleanup for Windows functional tests

This avoids intermittent issues in the CI Task when a test passes
successfully. For example:

PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\ContainerAdministrator\\AppData\\Local\\Temp\\test_runner_₿_🏃_20210920_072037\\feature_versionbits_warning_89\\alert.txt'

fac67b393b

fanquake added the label Tests on Sep 20, 2021
hebasto commented at 2:51 PM on September 20, 2021: member

What is the time required to complete the task with both (qt and vcpkg) caches invalidated after reducing the memory usage?
MarcoFalke force-pushed on Sep 20, 2021
MarcoFalke commented at 2:56 PM on September 20, 2021: member

What is the time required to complete the task with both (qt and vcpkg) caches invalidated after reducing the memory usage?

I'd say there should be no effect, but I didn't test that. Removed commit for now.
hebasto commented at 3:01 PM on September 20, 2021: member

ACK fac67b393ba4ff2f6698955b5f35433dec64ae51
fanquake merged this on Sep 21, 2021
fanquake closed this on Sep 21, 2021
MarcoFalke deleted the branch on Sep 21, 2021
hebasto commented at 1:21 PM on October 7, 2021: member

@MarcoFalke

Removed commit for now.

Re-introduced in #23220 after testing in #23217.
MarcoFalke referenced this in commit 02ccf1086e on Oct 15, 2021
DrahtBot locked this on Oct 30, 2022

Contributors

MarcoFalke

hebasto

Labels

Tests

Linked (view graph)

#23217 [DO NOT MERGE] ci: Test required resources for native Windows task #23220 ci: Reduce Windows memory for faster scheduling