From 948405752ad7a33199274aceaaa007a6f9c07f51 Mon Sep 17 00:00:00 2001
From: Terry Kong <terryk@nvidia.com>
Date: Wed, 26 Mar 2025 16:51:24 -0700
Subject: [PATCH] ci: skip functional until more capacity available and/or
 tests speed up

Signed-off-by: Terry Kong <terryk@nvidia.com>
---
 .github/workflows/cicd-main.yml | 42 +++++++++++++++++----------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/.github/workflows/cicd-main.yml b/.github/workflows/cicd-main.yml
index 2d2255e7f2..7902c02146 100644
--- a/.github/workflows/cicd-main.yml
+++ b/.github/workflows/cicd-main.yml
@@ -165,23 +165,25 @@ jobs:
     secrets:
       HF_TOKEN: ${{ secrets.HF_TOKEN }}
 
-  functional-tests:
-    name: ${{ matrix.test_case }}
-    needs: [build-container, pre-flight]
-    uses: ./.github/workflows/_run_test.yml
-    if: ${{ needs.pre-flight.outputs.run_ci == 'true' }}
-    strategy:
-      matrix:
-        test_case:
-          - sft.sh
-          - grpo.sh
-    with:
-      # TODO: For now, allow these to fail since the checks are not robust.
-      IS_OPTIONAL: true
-      RUNNER: self-hosted-azure
-      TIMEOUT: 15
-      SCRIPT: |
-        cd ${REINFORCER_REPO_DIR}
-        uv run bash ./tests/functional/${{ matrix.test_case }}
-    secrets:
-      HF_TOKEN: ${{ secrets.HF_TOKEN }}
\ No newline at end of file
+  # TODO: Temporarily disable functional tests until we have more capacity and tests run quicker
+  #       Related: https://github.com/NVIDIA/reinforcer/pull/27
+  #functional-tests:
+  #  name: ${{ matrix.test_case }}
+  #  needs: [build-container, pre-flight]
+  #  uses: ./.github/workflows/_run_test.yml
+  #  if: ${{ needs.pre-flight.outputs.run_ci == 'true' }}
+  #  strategy:
+  #    matrix:
+  #      test_case:
+  #        - sft.sh
+  #        - grpo.sh
+  #  with:
+  #    # TODO: For now, allow these to fail since the checks are not robust.
+  #    IS_OPTIONAL: true
+  #    RUNNER: self-hosted-azure
+  #    TIMEOUT: 15
+  #    SCRIPT: |
+  #      cd ${REINFORCER_REPO_DIR}
+  #      uv run bash ./tests/functional/${{ matrix.test_case }}
+  #  secrets:
+  #    HF_TOKEN: ${{ secrets.HF_TOKEN }}
\ No newline at end of file