Added calling module api test in the job, and fixed the consolidation condition

Signed-off-by: Hanxi Zhang <hanxizh@amazon.com>

Author: hanxizh9910

.github/workflows/daily.yml

@@ -1931,6 +1931,11 @@ jobs:
       - name: module api test
         if: true && !contains(github.event.inputs.skiptests, 'modules')
         run: CFLAGS='-Werror' ./runtest-moduleapi --log-req-res --no-latency --dont-clean --force-resp3 --dont-pre-clean --verbose --dump-logs ${{github.event.inputs.test_args}}
+      - name: Upload module api test failures
+        if: always() && !contains(github.event.inputs.skiptests, 'modules') && steps.module-api-test.outcome != 'skipped'
+        uses: ./.github/actions/upload-test-failures
+        with:
+          job-name: ${{ github.job }}-moduleapi
       - name: sentinel tests
         if: true && !contains(github.event.inputs.skiptests, 'sentinel')
         run: ./runtest-sentinel --log-req-res --dont-clean --force-resp3 ${{github.event.inputs.cluster_test_args}}
@@ -1945,7 +1950,7 @@ jobs:
         run: ./utils/req-res-log-validator.py --verbose --fail-missing-reply-schemas ${{ (!contains(github.event.inputs.skiptests, 'valkey') && !contains(github.event.inputs.skiptests, 'module') && !contains(github.event.inputs.skiptests, 'sentinel') && !contains(github.event.inputs.skiptests, 'cluster')) && (inputs.test_args || github.event.inputs.test_args || '') == '' && (inputs.cluster_test_args || github.event.inputs.cluster_test_args || '') == '' && '--fail-commands-not-all-hit' || '' }}
   consolidate-test-failures:
     runs-on: ubuntu-latest
-    if: always() && !contains(github.event.inputs.skiptests, 'valkey')
+    if: always()
     permissions:
       actions: write
     needs: