Skip to content

Commit 78ebd3c

Browse files
Camyllpytorchmergebot
authored andcommitted
Revert commit that removed windows testing in VS2019-> update (pytorch#146920)
This reverts commit b57b38b. This commit removed windows testing for the VS build and needs to be added back in with the updated VS2022 build Fixes #ISSUE_NUMBER Pull Request resolved: pytorch#146920 Approved by: https://github.com/seemethere, https://github.com/huydhn, https://github.com/atalman, https://github.com/malfet
1 parent df5e232 commit 78ebd3c

File tree

3 files changed

+63
-6
lines changed

3 files changed

+63
-6
lines changed

.github/scripts/test_trymerge.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -603,12 +603,12 @@ def test_get_classifications(self, *args: Any) -> None:
603603
].classification
604604
== "BROKEN_TRUNK"
605605
)
606-
# self.assertTrue(
607-
# checks[
608-
# "trunk / win-vs2022-cpu-py3 / test (default, 2, 3, windows.4xlarge.nonephemeral)"
609-
# ].classification
610-
# == "FLAKY"
611-
# )
606+
self.assertTrue(
607+
checks[
608+
"trunk / win-vs2019-cpu-py3 / test (default, 2, 3, windows.4xlarge.nonephemeral)"
609+
].classification
610+
== "FLAKY"
611+
)
612612
self.assertTrue(
613613
checks[
614614
"pull / linux-jammy-py3.8-gcc11 / test (distributed, 1, 2, linux.2xlarge)"

.github/workflows/pull.yml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,25 @@ jobs:
362362
test-matrix: ${{ needs.linux-focal-py3_9-clang9-xla-build.outputs.test-matrix }}
363363
secrets: inherit
364364

365+
win-vs2022-cpu-py3-build:
366+
# don't run build twice on main
367+
if: github.event_name == 'pull_request'
368+
name: win-vs2022-cpu-py3
369+
uses: ./.github/workflows/_win-build.yml
370+
needs: get-label-type
371+
with:
372+
build-environment: win-vs2022-cpu-py3
373+
cuda-version: cpu
374+
sync-tag: win-cpu-build
375+
runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral"
376+
test-matrix: |
377+
{ include: [
378+
{ config: "default", shard: 1, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
379+
{ config: "default", shard: 2, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
380+
{ config: "default", shard: 3, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
381+
]}
382+
secrets: inherit
383+
365384
linux-focal-cpu-py3_10-gcc9-bazel-test:
366385
name: linux-focal-cpu-py3.10-gcc9-bazel-test
367386
uses: ./.github/workflows/_bazel-build-test.yml

.github/workflows/trunk.yml

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,44 @@ jobs:
125125
test-matrix: ${{ needs.macos-py3-arm64-build.outputs.test-matrix }}
126126
secrets: inherit
127127

128+
win-vs2022-cpu-py3-build:
129+
name: win-vs2022-cpu-py3
130+
uses: ./.github/workflows/_win-build.yml
131+
needs: get-label-type
132+
with:
133+
build-environment: win-vs2022-cpu-py3
134+
cuda-version: cpu
135+
sync-tag: win-cpu-build
136+
runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral"
137+
test-matrix: |
138+
{ include: [
139+
{ config: "default", shard: 1, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
140+
{ config: "default", shard: 2, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
141+
{ config: "default", shard: 3, num_shards: 3, runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral" },
142+
]}
143+
secrets: inherit
144+
145+
win-vs2022-cpu-py3-test:
146+
name: win-vs2022-cpu-py3
147+
uses: ./.github/workflows/_win-test.yml
148+
needs:
149+
- win-vs2022-cpu-py3-build
150+
- target-determination
151+
with:
152+
build-environment: win-vs2022-cpu-py3
153+
cuda-version: cpu
154+
test-matrix: ${{ needs.win-vs2022-cpu-py3-build.outputs.test-matrix }}
155+
secrets: inherit
156+
157+
win-vs2022-cuda12_1-py3-build:
158+
name: win-vs2022-cuda12.1-py3
159+
uses: ./.github/workflows/_win-build.yml
160+
needs: get-label-type
161+
with:
162+
build-environment: win-vs2022-cuda12.1-py3
163+
cuda-version: "12.1"
164+
runner: "${{ needs.get-label-type.outputs.label-type }}windows.4xlarge.nonephemeral"
165+
secrets: inherit
128166

129167
linux-focal-rocm6_3-py3_10-build:
130168
if: ${{ startsWith(github.event.ref, 'refs/tags/ciflow/trunk') }}

0 commit comments

Comments
 (0)