Skip to content

Commit 10357ea

Browse files
gabriel-samfirathaJeztah
authored andcommitted
Address some timeout issues in the Windows CI
This change disables Windows Defender real-time monitoring on the test workers, and increases the test timeout to 20 minutes (default is 10). The Windows Defender real time monitoring feature scans any newly created files for malitious contents. This takes up a lot of CPU when expanding image archives, which contain lots of files. The CI has been timing out due to the fact that tests take longer than 10 minutes. This change should address that issue. Signed-off-by: Gabriel Adrian Samfira <[email protected]> (cherry picked from commit c7bdcdf) Signed-off-by: Sebastiaan van Stijn <[email protected]>
1 parent 977ce8e commit 10357ea

2 files changed

Lines changed: 10 additions & 1 deletion

File tree

.github/workflows/windows-periodic.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,11 @@ jobs:
162162
163163
- name: RunIntegrationTests
164164
run: |
165-
ssh -i $HOME/.ssh/id_rsa ${{ env.SSH_OPTS }} azureuser@${{ env.VM_PUB_IP }} "sh.exe -c 'cd /c/containerd && (make integration | tee /c/Logs/integration.log)'"
165+
ssh -i $HOME/.ssh/id_rsa ${{ env.SSH_OPTS }} azureuser@${{ env.VM_PUB_IP }} "sh.exe -s" << EOF
166+
cd /c/containerd
167+
export EXTRA_TESTFLAGS="-timeout=20m"
168+
make integration | tee /c/Logs/integration.log
169+
EOF
166170
ssh -i $HOME/.ssh/id_rsa ${{ env.SSH_OPTS }} azureuser@${{ env.VM_PUB_IP }} "sh.exe -c 'cat /c/Logs/integration.log | go-junit-report.exe > /c/Logs/junit_00.xml'"
167171
168172
- name: PrepareRepoList

script/setup/prepare_env_windows.ps1

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
# Prepare windows environment for building and running containerd tests
22

3+
# Disable Windows Defender real time monitoring. Real time monitoring consumes a lot of
4+
# CPU and slows down tests as images are unarchived, and is not really needed in a short
5+
# lived test environment.
6+
Set-MpPreference -DisableRealtimeMonitoring:$true
7+
38
$PACKAGES= @{ mingw = "10.2.0"; git = ""; golang = "1.19.6"; make = ""; nssm = "" }
49

510
Write-Host "Downloading chocolatey package"

0 commit comments

Comments
 (0)