fixup! [profiling] dump profile info option

aptos-labs · Nov 3, 2023 · 9a08729 · 9a08729
1 parent 421dec5
commit 9a08729
Showing 1 changed file with 2 additions and 84 deletions.
diff --git a/.github/workflows/workflow-run-execution-performance.yaml b/.github/workflows/workflow-run-execution-performance.yaml
@@ -55,83 +55,6 @@ jobs:
         id: determine_file_changes
         uses: ./.github/actions/file-change-determinator
 
-  # Run sequential execution performance tests
-  sequential-execution-performance:
-    needs: file_change_determinator
-    timeout-minutes: 30
-    runs-on: ${{ inputs.RUNNER_NAME }}
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          ref: ${{ inputs.GIT_SHA }}
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - uses: aptos-labs/aptos-core/.github/actions/rust-setup@main
-        with:
-          GIT_CREDENTIALS: ${{ secrets.GIT_CREDENTIALS }}
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - name: Run sequential execution benchmark in performance build mode
-        shell: bash
-        run: testsuite/sequential_execution_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - run: echo "Skipping sequential execution performance!"
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed == 'true' || !inputs.IS_FULL_RUN }}
-
-  # Run parallel execution performance tests
-  parallel-execution-performance:
-    needs: file_change_determinator
-    timeout-minutes: 60
-    runs-on: ${{ inputs.RUNNER_NAME }}
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          ref: ${{ inputs.GIT_SHA }}
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - uses: aptos-labs/aptos-core/.github/actions/rust-setup@main
-        with:
-          GIT_CREDENTIALS: ${{ secrets.GIT_CREDENTIALS }}
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - name: Run parallel execution benchmark in performance build mode
-        shell: bash
-        run: testsuite/parallel_execution_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - run: echo "Skipping parallel execution performance!"
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed == 'true' || !inputs.IS_FULL_RUN }}
-
-  # Run single node execution performance tests
-  single-node-performance:
-    needs: file_change_determinator
-    timeout-minutes: 60
-    runs-on: ${{ inputs.RUNNER_NAME }}
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          ref: ${{ inputs.GIT_SHA }}
-        if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
-
-      - uses: aptos-labs/aptos-core/.github/actions/rust-setup@main
-        with:
-          GIT_CREDENTIALS: ${{ secrets.GIT_CREDENTIALS }}
-        if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
-
-      - name: Run single node execution benchmark in performance build mode
-        shell: bash
-        run: TABULATE_INSTALL=lib-only pip install tabulate && testsuite/single_node_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && !inputs.IS_FULL_RUN }}
-
-      - name: Run full version of the single node execution benchmark in performance build mode
-        shell: bash
-        run: TABULATE_INSTALL=lib-only pip install tabulate && FLOW=CONTINUOUS testsuite/single_node_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.IS_FULL_RUN }}
-
-      - run: echo "Skipping single node execution performance! Unrelated changes detected."
-        if: needs.file_change_determinator.outputs.only_docs_changed == 'true'
-
   single-node-performance-profile:
     needs: file_change_determinator
     timeout-minutes: 60
@@ -147,21 +70,16 @@ jobs:
           GIT_CREDENTIALS: ${{ secrets.GIT_CREDENTIALS }}
         if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
 
-      - name: Run single node execution benchmark in performance build mode
-        shell: bash
-        run: TABULATE_INSTALL=lib-only pip install tabulate && testsuite/single_node_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.PROFILE }}
-
       - name: Run full version of the single node execution benchmark in performance build mode
         shell: bash
         run: TABULATE_INSTALL=lib-only pip install tabulate && FLOW=CONTINUOUS PROFILE=1 testsuite/single_node_performance.py
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.PROFILE }}
+        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' }}
 
       - name: Upload profile results
         uses: actions/upload-artifact@v3
         with:
           path: profiling_results
-        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' && inputs.PROFILE }}
+        if: ${{ needs.file_change_determinator.outputs.only_docs_changed != 'true' }}
 
       - run: echo "Skipping single node execution performance! Unrelated changes detected."
         if: needs.file_change_determinator.outputs.only_docs_changed == 'true'