Skip to content

Commit

Permalink
Migrate TPU tests to GitHub actions (#14687)
Browse files Browse the repository at this point in the history
* Migrate TPU tests to GitHub actions

* No working dir

* Keep _target

* Dont skip draft

* CHECK_SLEEP

* Not yet

* Remove recurrent cleanup script

* Set secrets

* a step cannot have both the `uses` and `run` keys

* Version $PYTHON_VER was not found in the local cache

* can't load package ... ($GOPATH not set)

* The `set-env` command is disabled

* Try updating go

* Match timeout

* simplify path

* More cleanup

* Install coverage. Unmark draft

* Update .github/workflows/ci-pytorch-test-tpu.yml

* DEBUG echo

* Revert "DEBUG echo"

This reverts commit 4011856.

* More debug

* SSH

* Im stupid

* Remove always()

* Forgot some

Co-authored-by: Jirka Borovec <[email protected]>
Co-authored-by: Luca Antiga <[email protected]>
  • Loading branch information
3 people authored Oct 21, 2022
1 parent b556713 commit 375ab53
Show file tree
Hide file tree
Showing 13 changed files with 152 additions and 286 deletions.
150 changes: 0 additions & 150 deletions .circleci/config.yml

This file was deleted.

1 change: 0 additions & 1 deletion .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

# CI/CD and configs
/.github/ @borda @carmocca @akihironitta @otaj
/.circleci/ @borda @carmocca @akihironitta @otaj
/.azure/ @borda @carmocca @akihironitta @otaj
/dockers/ @borda @carmocca @akihironitta @otaj
*.yml @borda @carmocca @akihironitta @otaj
Expand Down
15 changes: 7 additions & 8 deletions .github/checkgroup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,6 @@ subprojects:

# SECTION: pytorch_lightning

- id: "CI: CircleCI"
paths:
- ".circleci/**"
- ".github/workflows/ci-circleci.yml"
checks:
- "test-on-tpus"

- id: "pytorch_lightning"
paths:
# all examples don't need to be added because they aren't used in CI, but these are
Expand Down Expand Up @@ -58,7 +51,7 @@ subprojects:
- "pl-slow (macOS-11, 3.7, 1.11)"
- "pl-slow (ubuntu-20.04, 3.7, 1.11)"
- "pl-slow (windows-2022, 3.7, 1.11)"
# TODO: since this job cannot run on forks, it cannot be required or it will block all PL PRs from forks
# TODO: since this job has intermittent availability, it cannot be required or it will block all PL PRs from forks
#- "test-on-tpus"

- id: "pytorch_lightning: CPU"
Expand Down Expand Up @@ -110,6 +103,12 @@ subprojects:
checks:
- "pytorch-lightning (IPUs)"

- id: "pytorch-lightning: TPU"
paths:
- ".github/workflows/tpu-tests.yml"
checks:
- "test-on-tpus"

- id: "pytorch_lightning: Docs"
paths:
- "docs/source-pytorch/**"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
| pytorch-lightning (HPUs) | .azure-pipelines/hpu-tests.yml | Run only HPU-specific tests. | HPU |
| pytorch-lightning (GPUs) | .azure-pipelines/gpu-tests-pytorch.yml | Run all CPU and GPU-specific tests, standalone, and examples. Each standalone test needs to be run in separate processes to avoid unwanted interactions between test cases. | GPU |
| PyTorchLightning.Benchmark | .azure-pipelines/gpu-benchmark.yml | Run speed/memory benchmarks for parity with pure PyTorch. | GPU |
| test-on-tpus | .circleci/config.yml | Run only TPU-specific tests. | TPU |
| test-on-tpus | .github/workflows/tpu-tests.yml | Run only TPU-specific tests. | TPU |

- \*Accelerators used in CI

Expand Down
36 changes: 0 additions & 36 deletions .github/workflows/ci-circleci.yml

This file was deleted.

2 changes: 1 addition & 1 deletion .github/workflows/ci-pytorch-dockers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ jobs:
strategy:
fail-fast: false
matrix:
# the config used in '.circleci/config.yml`'
# the config used in '.github/workflows/tpu-tests.yml'
python_version: ["3.7"]
xla_version: ["1.12"]
steps:
Expand Down
43 changes: 0 additions & 43 deletions .github/workflows/events-recurrent.yml

This file was deleted.

Loading

0 comments on commit 375ab53

Please sign in to comment.