workflows bugfix concurrency

silverqx · Apr 12, 2024 · 4b5d20c · 4b5d20c
1 parent fc47ca8
commit 4b5d20c
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 9 deletions.
diff --git a/.github/workflows/analyzers.yml b/.github/workflows/analyzers.yml
@@ -8,7 +8,7 @@ on:
       - silverqx-develop
 
 concurrency:
-  group: tinyorm
+  group: tinyorm-linux
 
 jobs:
 

diff --git a/.github/workflows/linux-qt6-drivers.yml b/.github/workflows/linux-qt6-drivers.yml
@@ -14,7 +14,7 @@ on:
       - silverqx-develop
 
 concurrency:
-  group: tinyorm
+  group: tinyorm-linux
 
 # I will not remove the build folders before a job execution it's not necessary and
 # it will be faster this way. I can still remove them manually if needed or
@@ -58,7 +58,10 @@ jobs:
         # For gcc13 with the parallel 4 I saw 15.1GB maximum memory usage from 15.6GB and that is
         # very close to the edge, so I have to decrease it to 3.
         # For clang18 it was ~11.6 from 15.6GB so parallel 8 is ok.
-        parallel=${{ matrix.compiler.key == 'gcc13' && '3' || '8' }}
+        # I must divide all parallel by 2 because I have 2 self-hosted runners on the same computer
+        # and also -1 for reserve to avoid swapping, so for clang: 8 / 2 - 1 = 3 and for gcc:
+        # 3 / 2 - 1 = 1.
+        parallel=${{ matrix.compiler.key == 'gcc13' && '1' || '3' }}
         echo "TinyParallel=$parallel" >> $GITHUB_ENV
 
         tinyormPath=$(realpath ./main)

diff --git a/.github/workflows/msvc2022-qt6-drivers.yml b/.github/workflows/msvc2022-qt6-drivers.yml
@@ -14,7 +14,7 @@ on:
       - silverqx-develop
 
 concurrency:
-  group: tinyorm
+  group: tinyorm-windows
 
 # I will not remove the build folders before a job execution it's not necessary and
 # it will be faster this way. I can still remove them manually if needed or
@@ -53,7 +53,9 @@ jobs:
         $mysqlInstallationPath = Split-Path -Parent -Path (Split-Path -Parent -Path $mysqlExePath)
         "TinyMySQLInstallationPath=$mysqlInstallationPath" >> $env:GITHUB_ENV
 
-        "TinyParallel=10" >> $env:GITHUB_ENV
+        # I must divide all parallel by 2 because I have 2 self-hosted runners on the same computer
+        # and also -1 for reserve to avoid swapping, so 10 / 2 - 1 = 4
+        "TinyParallel=4" >> $env:GITHUB_ENV
 
         $tinyormPath = Resolve-Path -Path ./main
         "TinyORMPath=$tinyormPath" >> $env:GITHUB_ENV

diff --git a/.github/workflows/vcpkg-linux.yml b/.github/workflows/vcpkg-linux.yml
@@ -12,7 +12,7 @@ on:
       - silverqx-develop
 
 concurrency:
-  group: tinyorm
+  group: tinyorm-linux
 
 # I will not remove the build folders before a job execution it's not necessary and
 # it will be faster this way. I can still remove them manually if needed or
@@ -71,7 +71,9 @@ jobs:
         [[ "${{ runner.name }}" == 'merydeye-'* ]] && isSelfHostedRunner='true' || isSelfHostedRunner='false'
         echo "TinyIsSelfHostedRunner=$isSelfHostedRunner" >> $GITHUB_ENV
 
-        [[ "$isSelfHostedRunner" == 'true' ]] && parallel=10 || parallel=2
+        # I must divide all parallel by 2 because I have 2 self-hosted runners on the same computer
+        # and also -1 for reserve to avoid swapping, so 10 / 2 - 1 = 4
+        [[ "$isSelfHostedRunner" == 'true' ]] && parallel=4 || parallel=2
         echo "TinyParallel=$parallel" >> $GITHUB_ENV
 
         tinyormPath=$(realpath ./main)

diff --git a/.github/workflows/vcpkg-windows.yml b/.github/workflows/vcpkg-windows.yml
@@ -8,7 +8,7 @@ on:
       - silverqx-develop
 
 concurrency:
-  group: tinyorm
+  group: tinyorm-windows
 
 # I will not remove the build folders before a job execution it's not necessary and
 # it will be faster this way. I can still remove them manually if needed or
@@ -66,7 +66,9 @@ jobs:
         $isSelfHostedRunner = '${{ runner.name }}'.StartsWith('merydeye-', $true, $null)
         "TinyIsSelfHostedRunner=$isSelfHostedRunner" >> $env:GITHUB_ENV
 
-        $parallel = $isSelfHostedRunner ? 10 : 2
+        # I must divide all parallel by 2 because I have 2 self-hosted runners on the same computer
+        # and also -1 for reserve to avoid swapping, so 10 / 2 - 1 = 4
+        $parallel = $isSelfHostedRunner ? 4 : 2
         "TinyParallel=$parallel" >> $env:GITHUB_ENV
 
         $tinyormPath = Resolve-Path -Path ./main