Skip to content

Commit 086fe49

Browse files
ramilbakhshyievabhinavdangeti
authored andcommitted
Gate all PR builds behind linux-x86_64-cmake in GitHub Actions (facebookresearch#3476)
Summary: Pull Request resolved: facebookresearch#3476 The long tail will still be the RAFT build but we can save on cost if the build has errors by incurring ~10m penalty added by waiting for the basic cmake build to complete. Both GPU and RAFT builds will start together so this will take less time overall to complete. Reviewed By: algoriddle Differential Revision: D57707298 fbshipit-source-id: 3589842e9bda9ebca9b25e089e6177fe96b6a0f5
1 parent f46dba1 commit 086fe49

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/build.yml

+7-1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ jobs:
4646
- uses: ./.github/actions/build_cmake
4747
linux-x86_64-AVX2-cmake:
4848
name: Linux x86_64 AVX2 (cmake)
49+
needs: linux-x86_64-cmake
4950
runs-on: ubuntu-latest
5051
steps:
5152
- name: Checkout
@@ -56,6 +57,7 @@ jobs:
5657
linux-x86_64-AVX512-cmake:
5758
name: Linux x86_64 AVX512 (cmake)
5859
if: false # TODO: enable when GitHub Actions adds AVX-512 hosts
60+
needs: linux-x86_64-cmake
5961
runs-on: ubuntu-latest
6062
steps:
6163
- name: Checkout
@@ -65,7 +67,7 @@ jobs:
6567
opt_level: avx512
6668
linux-x86_64-GPU-cmake:
6769
name: Linux x86_64 GPU (cmake)
68-
needs: linux-x86_64-AVX2-cmake
70+
needs: linux-x86_64-cmake
6971
runs-on: 4-core-ubuntu-gpu-t4
7072
steps:
7173
- name: Checkout
@@ -75,6 +77,7 @@ jobs:
7577
gpu: ON
7678
linux-x86_64-GPU-w-RAFT-cmake:
7779
name: Linux x86_64 GPU w/ RAFT (cmake)
80+
needs: linux-x86_64-cmake
7881
runs-on: 4-core-ubuntu-gpu-t4
7982
steps:
8083
- name: Checkout
@@ -85,6 +88,7 @@ jobs:
8588
raft: ON
8689
linux-x86_64-conda:
8790
name: Linux x86_64 (conda)
91+
needs: linux-x86_64-cmake
8892
runs-on: ubuntu-latest
8993
steps:
9094
- name: Checkout
@@ -95,6 +99,7 @@ jobs:
9599
- uses: ./.github/actions/build_conda
96100
windows-x86_64-conda:
97101
name: Windows x86_64 (conda)
102+
needs: linux-x86_64-cmake
98103
runs-on: windows-2019
99104
steps:
100105
- name: Checkout
@@ -105,6 +110,7 @@ jobs:
105110
- uses: ./.github/actions/build_conda
106111
linux-arm64-conda:
107112
name: Linux arm64 (conda)
113+
needs: linux-x86_64-cmake
108114
runs-on: 2-core-ubuntu-arm
109115
steps:
110116
- name: Checkout

0 commit comments

Comments
 (0)