[Openvpn-devel,S] Change in openvpn[master]: GHA: Enable t_server_null tests

Message ID 73512253abd82a644d72f5c297ac252926f6e0d4-HTML@gerrit.openvpn.net
State Superseded
Headers show
Series [Openvpn-devel,S] Change in openvpn[master]: GHA: Enable t_server_null tests | expand

Commit Message

flichtenheld (Code Review) Sept. 11, 2024, 2:33 p.m. UTC
Attention is currently required from: plaisthos.

Hello plaisthos,

I'd like you to do a code review.
Please visit

    http://gerrit.openvpn.net/c/openvpn/+/743?usp=email

to review the following change.


Change subject: GHA: Enable t_server_null tests
......................................................................

GHA: Enable t_server_null tests

Change-Id: I86203b8f9a6d3cfc5e56d3ce9452af694fd11011
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
---
M .github/workflows/build.yaml
1 file changed, 16 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/43/743/1

Patch

diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
index 8f0a7b5..361d457 100644
--- a/.github/workflows/build.yaml
+++ b/.github/workflows/build.yaml
@@ -172,8 +172,11 @@ 
         run: ./configure --with-crypto-library=${{matrix.ssllib}} ${{matrix.extraconf}} --enable-werror
       - name: make all
         run: make -j3
+      - name: configure checks
+        if: ${{ matrix.extraconf != '--disable-management' }}
+        run: echo 'RUN_SUDO="sudo -E"' >tests/t_server_null.rc
       - name: make check
-        run: make check VERBOSE=1
+        run: make -j3 check VERBOSE=1
 
   ubuntu-clang-asan:
     strategy:
@@ -199,8 +202,10 @@ 
         run: CFLAGS="-fsanitize=address,undefined -fno-sanitize-recover=all  -fno-omit-frame-pointer -O2" CC=clang ./configure --with-crypto-library=${{matrix.ssllib}} --enable-werror
       - name: make all
         run: make -j3
+      - name: configure checks
+        run: echo 'RUN_SUDO="sudo -E"' >tests/t_server_null.rc
       - name: make check
-        run: make check VERBOSE=1
+        run: make -j3 check VERBOSE=1
 
   macos:
     strategy:
@@ -258,8 +263,10 @@ 
         run: ./configure --enable-werror ${{matrix.configureflags}} ${{matrix.configuressllib}}
       - name: make all
         run: make -j4
+      - name: configure checks
+        run: echo 'RUN_SUDO="sudo -E"' >tests/t_server_null.rc
       - name: make check
-        run: make check VERBOSE=1
+        run: make -j4 check VERBOSE=1
 
   msvc:
       strategy:
@@ -369,8 +376,10 @@ 
         run: ./configure --with-crypto-library=openssl ${{matrix.configureflags}} --enable-werror
       - name: make all
         run: make -j3
+      - name: configure checks
+        run: echo 'RUN_SUDO="sudo -E"' >tests/t_server_null.rc
       - name: make check
-        run: make check VERBOSE=1
+        run: make -j3 check VERBOSE=1
 
   mbedtls3:
     strategy:
@@ -422,5 +431,7 @@ 
         run: ./configure --with-crypto-library=mbedtls
       - name: make all
         run: make -j3
+      - name: configure checks
+        run: echo 'RUN_SUDO="sudo -E"' >tests/t_server_null.rc
       - name: make check
-        run: make check VERBOSE=1
+        run: make -j3 check VERBOSE=1