Commit Graph

104 Commits

Author SHA1 Message Date
Nikita Shulga
ad79d0dd4b Add ciflow/trunk label (#69575)
Summary:
Which includes all workflows but periodic ones

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69575

Reviewed By: seemethere

Differential Revision: D32932850

Pulled By: malfet

fbshipit-source-id: 80b58fb3a0d5f8dbc527124be5bf25bd716448b8
2021-12-17 09:57:46 -08:00
Xida Chen
531da0c43b change asan test shard to 3 (#69843)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/68261

This PR changes the number of test shard from 2-->3 for all Asan test, aiming to improve the run time for Asan tests.

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69843

Reviewed By: janeyx99

Differential Revision: D33160771

Pulled By: xidachen

fbshipit-source-id: dba1d318cc49b923e18704839471d8753cc00eca
2021-12-16 07:22:03 -08:00
Andrey Talman
77a4b89411 Adding windows cuda 11.5 workflows (#69377)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/69081

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69377

Reviewed By: ngimel

Differential Revision: D33076022

Pulled By: atalman

fbshipit-source-id: aeb2791fc15d7b491976f57a74c1989c6ca61b81
2021-12-13 20:49:02 -08:00
Andrey Talman
c309637923 Making cuda 11.5 workflows periodic (#69323)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/68259

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69323

Reviewed By: gchanan, malfet

Differential Revision: D32812346

Pulled By: atalman

fbshipit-source-id: 081f40802997cfb986742f1621eee4b4565660f0
2021-12-07 13:14:07 -08:00
Andrey Talman
e0fb228e03 Revert of adding windows CUDA 11.5 workflow (#69365)
Summary:
This is partial revert of bb522c9d7a to revert addition of workflows for CUDA 11.5 windows that fails

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69365

Reviewed By: suo

Differential Revision: D32831418

Pulled By: atalman

fbshipit-source-id: 184346d22623f88594312a4ce2e4d29cc67e8338
2021-12-03 08:00:16 -08:00
Eli Uriegas
f786b03f98 ci: Migrate docs push to GHA (#69172)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/69172

Migrates the docs push jobs to Github Actions by implementing a simple
WITH_PUSH switch to do the actual push.

Adds 2 new workflows for GHA:
* linux-docs (on trunk)
* linux-docs-push (on schedule)

linux-docs-push is the only workflow that actually gets access to
credentials so it should be relatively safe.

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D32767239

Pulled By: seemethere

fbshipit-source-id: 5b100f986cf4023c323f4f96f0fe7942fec49ad2
2021-12-02 15:06:57 -08:00
Andrey Talman
bb522c9d7a Enabling CUDA 11.5 for binary builds, Adding windows workflows for CUDA 11.5 (#69262)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/68259

Pull Request resolved: https://github.com/pytorch/pytorch/pull/69262

Reviewed By: malfet

Differential Revision: D32804850

Pulled By: atalman

fbshipit-source-id: abac45ad1d49ec7e0e7df6cb9a22a46fbcd905a2
2021-12-02 13:04:43 -08:00
Andrey Talman
f1a3512b78 Adding Linux cuda 11.5 workflows (#68745)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/68960

Pull Request resolved: https://github.com/pytorch/pytorch/pull/68745

Reviewed By: janeyx99

Differential Revision: D32707491

Pulled By: atalman

fbshipit-source-id: 100facfdcc0fc2f68e203a696856852faa25ee08
2021-11-29 16:21:00 -08:00
Nikita Shulga
ee59a09772 Implement sharding for MacOS jobs (#68784)
Summary:
Do not run distributed tests as part of separate shard, but keep it inside one of the two shards (to limit concurrency problems)
Fixes https://github.com/pytorch/pytorch/issues/68260

Pull Request resolved: https://github.com/pytorch/pytorch/pull/68784

Reviewed By: seemethere, janeyx99

Differential Revision: D32653440

Pulled By: malfet

fbshipit-source-id: ebe5bbc30bdf67e930f2c766c920932700f3a4e4
2021-11-29 09:31:42 -08:00
Eli Uriegas
17ba936da0 .github: Migrate XLA tests to GHA (#64320)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/64320

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D30684490

Pulled By: seemethere

fbshipit-source-id: 5d2657f9aa4c7082591239a5bb095cc85d2cde66
2021-11-29 08:30:57 -08:00
Nikita Shulga
c5f63f859e Add slow path to getCustomClassTypeImpl (#68717)
Summary:
This fixes custom class registration issue when `typeid` is not guaranteed to be unique across multiple libraries, which is the case for libc++ runtime on MacOS 11 in particular for M1
From [libcxx/include/typeinfo](78d6a7767e/include/typeinfo (L139)):
```
// -------------------------------------------------------------------------- //
//                          NonUniqueARMRTTIBit
// -------------------------------------------------------------------------- //
// This implementation of type_info does not assume always a unique copy of
// the RTTI for a given type inside a program. It packs the pointer to the
// type name into a uintptr_t and reserves the high bit of that pointer (which
// is assumed to be free for use under the ABI in use) to represent whether
// that specific copy of the RTTI can be assumed unique inside the program.
// To implement equality-comparison of type_infos, we check whether BOTH
// type_infos are guaranteed unique, and if so, we simply compare the addresses
// of their type names instead of doing a deep string comparison, which is
// faster. If at least one of the type_infos can't guarantee uniqueness, we
// have no choice but to fall back to a deep string comparison.
```

But `std::type_index` hash is computed always assuming that implementation is unique
By adding a slow path this problem can be fixed in those scenarios.

Fixes https://github.com/pytorch/pytorch/issues/68039

Pull Request resolved: https://github.com/pytorch/pytorch/pull/68717

Reviewed By: seemethere

Differential Revision: D32605187

Pulled By: malfet

fbshipit-source-id: 8d50e56885b8c97dad3bc34a69c47ef879456dd1
2021-11-24 15:00:47 -08:00
Eli Uriegas
7b958fbec4 ci: Build periodic jobs with DEBUG=1 (#67192)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67192

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: albanD, janeyx99

Differential Revision: D31902447

Pulled By: seemethere

fbshipit-source-id: 1d1cca8b5ac84b1c23ab73e2d973bfb7bffa8982
2021-11-15 14:51:06 -08:00
Jiakai Liu
3dc0754c53 [pytorch][mobile] deprecate the LLVM-based static analyzer (#68180)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/68180

Since we've open sourced the tracing-based selective build, we can deprecate the
op-dependency-graph-based selective build and the static analyzer tool that
produces the dependency graph.
ghstack-source-id: 143108377

Test Plan: CIs

Reviewed By: seemethere

Differential Revision: D32358467

fbshipit-source-id: c61523706b85a49361416da2230ec1b035b8b99c
2021-11-11 16:37:08 -08:00
Eli Uriegas
8d025bbc2d .github: Migrate macOS workflows to GHA (#67717)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67717

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D32287733

Pulled By: seemethere

fbshipit-source-id: 8df6b20aada818ad39895ef87dc280098e09707b
2021-11-09 15:46:05 -08:00
Slava Kovalevskyi
1ffd43cf0c generated-pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single-full-jit migrated to GHA (#67695)
Summary:
in scope of: https://github.com/pytorch/pytorch/issues/67301. Main changes:
* generated-pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single-full-jit deleted from circle
* pytorch_android_gradle_custom_build_single removed since it is no longer used
* generated-pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single-full-jit added to GHA

Pull Request resolved: https://github.com/pytorch/pytorch/pull/67695

Reviewed By: malfet, seemethere, ejguan

Differential Revision: D32115620

Pulled By: b0noI

fbshipit-source-id: 113d48303c090303ae13512819bac2f069a2913f
2021-11-03 10:29:37 -07:00
Eli Uriegas
9e97ccbd7a .github: Migrate iOS workflows to GHA (#67645)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67645

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D32104367

Pulled By: seemethere

fbshipit-source-id: 08ff043ed5d0b434322f1f3f20dce2a4f5fa88c1
2021-11-02 14:38:43 -07:00
Slava Kovalevskyi
eb1b8a2160 pytorch_android_gradle_custom_build_single migrated from Circle to GHA. (#67577)
Summary:
in scope of: https://github.com/pytorch/pytorch/issues/67301. Main changes:
* pytorch_android_gradle_custom_build_single removed from the circle (however template is still there since it is used by another similar workflow: pytorch-linux-xenial-py3-clang5-android-ndk-r19c-gradle-custom-build-single-full-jit, which will be migrated next)
* new GHA workflow added: pytorch_android_gradle_custom_build_single

Pull Request resolved: https://github.com/pytorch/pytorch/pull/67577

Reviewed By: malfet, mruberry

Differential Revision: D32087709

Pulled By: b0noI

fbshipit-source-id: f9581558ddc1453b63264bf19fe5a4c245b7c007
2021-11-02 10:21:03 -07:00
Eli Uriegas
d0bc01fac2 ci: Migrate hardcoded docker builds to GHA (#67455)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67455

Migrates docker builds that don't have dependent jobs within the pytorch
repository to our new GHA docker build job

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet, janeyx99

Differential Revision: D31997671

Pulled By: seemethere

fbshipit-source-id: 9d6f58fa8ea8731cf12457fe64dc65e70f3d9f25
2021-10-28 14:50:05 -07:00
Nikita Shulga
0c93c8e39a Disable linux-xenial-cuda10.2 config (#67344)
Summary:
linux-xenial-cuda10.2 and linux-bionic-cuda10.2 are very similar, no
need to run both configs

Moved all auxiliary builds from xenial to bionic

Pull Request resolved: https://github.com/pytorch/pytorch/pull/67344

Reviewed By: seemethere, janeyx99

Differential Revision: D31964850

Pulled By: malfet

fbshipit-source-id: d07ce266c843c7fd69b281e678c4247b0bf6da20
2021-10-28 10:10:13 -07:00
Eli Uriegas
81d188101f .github: Use 4xlarge instances for linux gpu (#67264)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67264

Downgrades linux gpu instances from 4xlarge -> 8xlarge

We were seeing capacity issues in terms of scaling 8xlarge instances,
downgrading this to 4xlarge (which only have a single gpu) to see if
that helps resolve some of the capacity issues we were seeing

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: janeyx99

Differential Revision: D31933488

Pulled By: seemethere

fbshipit-source-id: b41922ebb675e663cb035cd3795bc9bae94dcac7
2021-10-26 16:17:33 -07:00
Eli Uriegas
7052c41899 .github: Add workflow to build all docker images (#67215)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/67215

We were regularly seeing gaps in our docker image builds due to specific
workflows not being run when docker builds occurred on PRs, this should
remove that ambiguity and ensure that all docker builds be re-built if a
rebuild is deemed necessary

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D31910422

Pulled By: seemethere

fbshipit-source-id: f346e64f1857e35a995c49bf30521a3acd8af0b1
2021-10-26 11:14:04 -07:00
Alex Zhao
ac948f4f35 .github: Migrate linux-xenial-py3.6-gcc7 to GHA (#67072)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/66888

cc seemethere

Pull Request resolved: https://github.com/pytorch/pytorch/pull/67072

Reviewed By: seemethere

Differential Revision: D31900833

Pulled By: zhaoalex

fbshipit-source-id: 93f8995611169d991f90e07e8c13e08182969577
2021-10-25 11:40:12 -07:00
Nikita Shulga
77beccaedb Do not build PyTorch with caffe2 by default (#66658)
Summary:
CAFFE2 has been deprecated for a while, but still included in every PyTorch build.
We should stop building it by default, although CI should still validate that caffe2 code is buildable.

Build even fewer dependencies when compiling mobile builds without Caffe2
Introduce `TEST_CAFFE2` in torch.common.utils
Skip `TestQuantizedEmbeddingOps` and `TestJit.test_old_models_bc`  is code is compiled without Caffe2
Should be landed after https://github.com/pytorch/builder/pull/864

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66658

Reviewed By: driazati, seemethere, janeyx99

Differential Revision: D31669156

Pulled By: malfet

fbshipit-source-id: 1cc45e2d402daf913a4685eb9f841cc3863e458d
2021-10-21 20:32:47 -07:00
Nikita Shulga
9c4d7d96db Address feedback from #66673 (#66905)
Summary:
Specify both `build_generates_artifacts` and `exclude_tests` properties as suggested in https://github.com/pytorch/pytorch/pull/66673#pullrequestreview-783667960

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66905

Reviewed By: seemethere

Differential Revision: D31779742

Pulled By: malfet

fbshipit-source-id: 21f5543f3b767f38132be8c7e163455f39ff893f
2021-10-19 18:27:45 -07:00
Nikita Shulga
cbd7bac914 Migrate clang5-mobile build to GHA (#66673)
Summary:
`linux-xenial-py3-clang5-mobile-build`, `linux-xenial-py3-clang5-mobile-custom-build-dynamic`, `linux-xenial-py3-clang5-mobile-custom-build-dynamic` and `linux-xenial-py3-clang5-mobile-code-analysis` are just the flavors of regular linux build job with no tests.
`linux-xenial-py3-clang5-mobile-code-analysis` is the master only job

`code-analysis` job is dispatch to `.jenkins/pytorch/build-mobile-code-analysis.sh` in
583217fe37/.jenkins/pytorch/build.sh (L23-L25)
and all `mobile-build` jobs are dispatched to `.jenkins/pytorch/build-mobile.sh` in
583217fe37/.jenkins/pytorch/build.sh (L19-L21)

Rename `is_libtorch` `CIWorkflow` property into `build_generates_artifacts` and change defaults from False to True
Both libtorch and mobile build jobs do not generate build artifacts

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66673

Reviewed By: janeyx99

Differential Revision: D31674434

Pulled By: malfet

fbshipit-source-id: 24d05d55366202cd4d9c25ecab429cb8f670ded0
2021-10-19 14:13:29 -07:00
Eli Uriegas
8492e6bc6a .github: scheduled -> schedule, fix periodic (#66531)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/66531

The github.event_name should be schedule not scheduled

Reference, https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#schedule

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D31598136

Pulled By: seemethere

fbshipit-source-id: 4d67f7731b21e05dabc8f54b4ebf9a5d2d3a4e1e
2021-10-12 19:46:01 -07:00
Jane Xu
d921891f57 GHA: Stop skipping periodic jobs (#66264)
Summary:
they have been skipped for too long
![image](https://user-images.githubusercontent.com/31798555/136433267-f35c0507-23ab-4348-be43-78d299c3d654.png)

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66264

Reviewed By: dagitses, malfet, seemethere

Differential Revision: D31478705

Pulled By: janeyx99

fbshipit-source-id: 1324b123e3f8646e5cd671af4c1850398a6f6e3b
2021-10-12 14:39:47 -07:00
Eli Uriegas
3b4b1b2d23 .github: Remove confusing ciflow_config.enabled variable (#66260)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/66260

Every workflow has ciflow enabled so this is not needed anymore

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: dagitses, janeyx99

Differential Revision: D31493340

Pulled By: seemethere

fbshipit-source-id: 8718fe5d22f4be6e0900962576782a9f23162a39
2021-10-11 09:39:31 -07:00
Nikita Shulga
ebe530a9cd Periodic jobs should not have CIFLOW_DEFAULT label (#66300)
Summary:
Noticed that `periodic-pytorch-linux-xenial-cuda10.2-cudnn7-py3-gcc7-slow-gradcheck` job has a `ciflow/default`, but does not have a `ciflow/scheduled` label
Added asserts to enforce that jobs with non-trival is_scheduled property do not have default and do have scheduled labesl

Rename `periodic-pytorch-linux-xenial-cuda10.2-cudnn7-py3-gcc7-slow-gradcheck` to `periodic-linux-xenial-cuda10.2-py3-gcc7-slow-gradcheck`

Fixes #{issue number}

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66300

Reviewed By: seemethere

Differential Revision: D31493323

Pulled By: malfet

fbshipit-source-id: 194c1d7a4e659847d94a547b87a0d7d08e66406d
2021-10-07 19:57:32 -07:00
Eli Uriegas
84c5970a77 ci: Migrate slow_gradcheck to GHA (#65730)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/65730

This should close out the door on migrating all scheduled workflows we have for CircleCI

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

cc ezyang seemethere malfet pytorch/pytorch-dev-infra

Test Plan: Imported from OSS

Reviewed By: albanD

Differential Revision: D31225188

Pulled By: seemethere

fbshipit-source-id: 4c49e88ec017edc30e07325dbc613ff54dd164d8
2021-10-06 14:29:14 -07:00
Nikita Shulga
6d4d636d66 [GHA] Rectify trigger_action_only flag (#66209)
Summary:
No longer needed, as PR can be opened/reopened with specific label

Fixes https://github.com/pytorch/pytorch/issues/66110

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66209

Reviewed By: seemethere

Differential Revision: D31436292

Pulled By: malfet

fbshipit-source-id: 5b6e0875bec261862017dfe0eb3a5ec57fb8c705
2021-10-06 10:46:10 -07:00
Eli Uriegas
d2021e5e68 ci: Migrate vulkan builds to GHA (#66044)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/66044

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet

Differential Revision: D31370889

Pulled By: seemethere

fbshipit-source-id: 399f5f0c184f7856dcddb138c357f1374706e676
2021-10-05 16:11:36 -07:00
Nikita Shulga
1932bc69e9 Move GHA to ONNX (#65975)
Summary:
- Delete CircleCI ONNX config
- Add sharded ONNX job to the list of generated workflows
- Move ONNX runtime installation from `pytorch-job-specs.yml` to `.jenkins/caffe2/test.sh`
- Limit MKLDNN to AVX2 ISA while running  Caffe2 tests

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65975

Reviewed By: seemethere

Differential Revision: D31327206

Pulled By: malfet

fbshipit-source-id: 15aa53e4481e846c62b4ee2db5c03047d68679a4
2021-10-05 09:31:57 -07:00
Jane Xu
69da4b4381 GHA: make obvious when we are running smoke tests to user (#66011)
Summary:
This PR clarifies what's run on PRs by explicitly stating when it runs smoke tests for windows CUDA and makes the logic so that user defined labels override other workflow logic.

1. Move smoke tests to its own config.

2. Make sure that when a user specifies a ciflow label that is not the default, the workflow runs as if it is on trunk.

Pull Request resolved: https://github.com/pytorch/pytorch/pull/66011

Test Plan:
the default on PRs would generate this matrix (default replaced by smoke_tests)
![image](https://user-images.githubusercontent.com/31798555/135672182-64454ea3-ff43-4746-b8e4-09b0b28e9d33.png)
But when retriggered with a label, it looks like (note that there's no smoke_tests config):
![image](https://user-images.githubusercontent.com/31798555/135672601-5aa9a268-bc76-40f1-80c6-62b3fac6601d.png)

Reviewed By: VitalyFedyunin, seemethere

Differential Revision: D31355130

Pulled By: janeyx99

fbshipit-source-id: fed58ade4235b58176e1d1a24101aea0bea83aa4
2021-10-04 07:53:17 -07:00
Nikita Shulga
6a30d83596 Move ASAN to GHA (#65846)
Summary:
- Introduce `ciflow/sanitizers` label
- Modify asan pattern in `.jenkins/pytorch/build.sh`
- Produce wheel in `.jenkins/pytorch/build-asan.sh`
- Increase stack size hard limit to 82Mb in test docker containers

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65846

Reviewed By: seemethere

Differential Revision: D31282654

Pulled By: malfet

fbshipit-source-id: f73e692899cc9bbe106ececc26f1fe430dfeae9d
2021-09-30 09:49:52 -07:00
Nikita Shulga
5950240bdf Stop Win+CUDA-10.2 builds (#65649)
Summary:
See https://github.com/pytorch/pytorch/issues/65612 and https://github.com/pytorch/pytorch/issues/25393

Fixes https://github.com/pytorch/pytorch/issues/65648

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65649

Reviewed By: janeyx99

Differential Revision: D31189692

Pulled By: malfet

fbshipit-source-id: 6ec0548d5833f3428d882071d26c357d89b0a9ba
2021-09-29 15:41:23 -07:00
Eli Uriegas
2f712c452e .github: Remove confusing on_pull_request variable (#65731)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/65731

It originally had purpose but after ciflow was introduced every PR had
on_pull_request set so it's not really as useful as it once was

Also removes the equally as confusing only_build_on_pull_request
variable as well

This change should produce no functional changes in our generated workflows

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

cc ezyang seemethere malfet pytorch/pytorch-dev-infra

Test Plan: Imported from OSS

Reviewed By: janeyx99

Differential Revision: D31225398

Pulled By: seemethere

fbshipit-source-id: 7bd8e8175794ab7d09b0632321bf52538435e858
2021-09-29 11:56:13 -07:00
Nikita Shulga
72b27bde83 [CIFlow] Modify workflow trigger logic (#65733)
Summary:
CIFLow workflows should always run on push event
On pull-request workflow should run if label conditions are met or if
no `ciflow/` labels are associated with it, workflow is enabled by
default

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65733

Reviewed By: zhouzhuojie

Differential Revision: D31251278

Pulled By: malfet

fbshipit-source-id: 31ce745cb224df7c6fec1682ec4180513e3dadf3
2021-09-28 19:19:49 -07:00
Nikita Shulga
e1340d4282 [GHA] Small refactors (#65647)
Summary:
Introduce `main` method in generate_ci_workflows
Check that all `ciflow/` labels start with the same prefix
Move `ciflow_should_run` defenition to common.yml.j2

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65647

Reviewed By: janeyx99

Differential Revision: D31189537

Pulled By: malfet

fbshipit-source-id: 7cc47f63fb334c57f450034b931ff5bae1c0ed8b
2021-09-27 13:14:49 -07:00
Nikita Shulga
e331beef20 Delete code coverage jobs from CI (#65362)
Summary:
As it does not seem useful to the lots of peope, see https://fb.workplace.com/groups/1144215345733672/posts/2062909540530910

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65362

Reviewed By: janeyx99, bdhirsh

Differential Revision: D31061945

Pulled By: malfet

fbshipit-source-id: 912ed92cc901a370a40448f1127c3ba43640ac43
2021-09-22 05:38:35 -07:00
Jane Xu
9afdf017dc Add force_on_cpu test to win cuda10.2 on GHA (#65094)
Summary:
Part of migrating from Circle.

Once we get a successful force_on_cpu test, we can move it to trunk only.

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65094

Reviewed By: seemethere

Differential Revision: D31086289

Pulled By: janeyx99

fbshipit-source-id: e1d135cc844d51f0b243b40efb49edca277d9de8
2021-09-21 11:14:15 -07:00
zhouzhuojie
2abf3594d5 Fix unassigned ciflow trigger (#65354)
Summary:
Fixes https://github.com/pytorch/pytorch/issues/65250#issuecomment-923120764

this is a limitation of github action triggers, it's hard to introduce condition before the workflow, that's why we intentionally pick the rare event ("unassigned"). The fix I think for people didn't opt-in ciflow and manually unassign, is to run all the CI (otherwise we introduce new condition on this and not worth to make things even complex).

`unassigned` event payload looks like this, just to make sure `github.event.assignee.login` is pointing to the right location.

```
  {
    "action": "unassigned",
    "assignee": {
      "avatar_url": "https://avatars.githubusercontent.com/u/658840?v=4",
      "events_url": "https://api.github.com/users/zhouzhuojie/events{/privacy}",
      "followers_url": "https://api.github.com/users/zhouzhuojie/followers",
      "following_url": "https://api.github.com/users/zhouzhuojie/following{/other_user}",
      "gists_url": "https://api.github.com/users/zhouzhuojie/gists{/gist_id}",
      "gravatar_id": "",
      "html_url": "https://github.com/zhouzhuojie",
      "id": 658840,
      "login": "zhouzhuojie",
      "node_id": "MDQ6VXNlcjY1ODg0MA==",
      "organizations_url": "https://api.github.com/users/zhouzhuojie/orgs",
      "received_events_url": "https://api.github.com/users/zhouzhuojie/received_events",
      "repos_url": "https://api.github.com/users/zhouzhuojie/repos",
      "site_admin": false,
      "starred_url": "https://api.github.com/users/zhouzhuojie/starred{/owner}{/repo}",
      "subscriptions_url": "https://api.github.com/users/zhouzhuojie/subscriptions",
      "type": "User",
      "url": "https://api.github.com/users/zhouzhuojie"
    },
```

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65354

Reviewed By: malfet, seemethere, janeyx99

Differential Revision: D31060212

Pulled By: zhouzhuojie

fbshipit-source-id: ce815cc96e8a00016646d6f02f0917169fa652dc
2021-09-20 12:33:23 -07:00
Nikita Shulga
67570a60ba Disable ParallelTBB (#65092)
Summary:
As ParallelTBB's `at::get_thread_num` is not compatible with general model used by OpenMP and ParallelNative (where it is an contiguous thread index within parallel loop), see https://github.com/pytorch/pytorch/issues/64571#issuecomment-914691883

More examples of similar regressions: https://github.com/pytorch/pytorch/runs/3612142217

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65092

Reviewed By: zhouzhuojie

Differential Revision: D30995936

Pulled By: malfet

fbshipit-source-id: db145b6a850d794f2c954f59f30249b291473e36
2021-09-16 12:38:45 -07:00
Eli Uriegas
3d5923366d .github: Enable only specific workflows for canary (#65099)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/65099

Utilizes ciflow to enable only specific workflows for
pytorch/pytorch-canary to reduce noise on that specific repository

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: jbschlosser

Differential Revision: D30973691

Pulled By: seemethere

fbshipit-source-id: 371765535b42a00bd72c2551c4faebf733d759f0
2021-09-15 16:53:12 -07:00
Eli Uriegas
59c486f2f3 ci: Disable jit legacy on circleci, enable on gha (#65106)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/65106

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

cc ezyang seemethere malfet lg20987 pytorch/pytorch-dev-infra

Test Plan: Imported from OSS

Reviewed By: malfet, janeyx99

Differential Revision: D30976186

Pulled By: seemethere

fbshipit-source-id: 8958f821eab9aa284496c57915894ed70f6b2fff
2021-09-15 16:11:38 -07:00
Jane Xu
3f27c1ae78 Replace windows 10.2 smoke tests on PRs to be 11.3 (#65090)
Summary:
As we default to linux CUDA 11.3 on PRs, we should do the same thing with Windows (instead of having 10.2 be the default). This means that 10.2 will now be master only, and 11.3 windows smoke tests will run on every PR.

This also copies over the "run smoke tests only" config--removing that will be in a separate PR once there's more certain decision making.

Pull Request resolved: https://github.com/pytorch/pytorch/pull/65090

Reviewed By: seemethere

Differential Revision: D30968382

Pulled By: janeyx99

fbshipit-source-id: c73f9a2cc800b678909365c4d80627d29fc09f94
2021-09-15 16:01:07 -07:00
driazati
ee554e2e96 Re-land Fix test report uploading (#64958)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/64958

This is a re-do of #64846 which was missing a path prefix for windows test reports

Test Plan: Imported from OSS

Reviewed By: seemethere

Differential Revision: D30915253

Pulled By: driazati

fbshipit-source-id: d14d0a64d2f8aabc335db9c4d0d2b63512887c66
2021-09-13 18:36:26 -07:00
Eli Uriegas
914e3a861a Revert D30878101: [pytorch][PR] Fix test report uploading
Test Plan: revert-hammer

Differential Revision:
D30878101 (fba40bfc1a)

Original commit changeset: 0730f17fa3f4

fbshipit-source-id: dad89e68b4daf656dd0b592bc9b2758f00af38c6
2021-09-13 15:24:44 -07:00
driazati
fba40bfc1a Fix test report uploading (#64846)
Summary:
Previously we just weren't uploading Windows test report XML files to S3, only to GitHub actions. This was different than Linux where we use both (though maybe we can kill the GHA upload in a follow up PR since I don't think it's very useful anymore). This factors it all out into a macro so they both do the same thing. This also fixes the naming of uploaded files to include info about the job name (the full config, so they can be matched to the job visually or by the included job id).

See https://hud.pytorch.org/pr/64846 for results

Pull Request resolved: https://github.com/pytorch/pytorch/pull/64846

Reviewed By: seemethere

Differential Revision: D30878101

Pulled By: driazati

fbshipit-source-id: 0730f17fa3f46a32c131f52669084c3103b0e616
2021-09-13 13:22:54 -07:00
Eli Uriegas
ff18195df9 .github: Upgrade windows CUDA 10.1 -> 10.2 (#64658)
Summary:
Pull Request resolved: https://github.com/pytorch/pytorch/pull/64658

We don't release 10.1 anymore so let's bump to 10.2

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

Test Plan: Imported from OSS

Reviewed By: malfet, janeyx99

Differential Revision: D30811178

Pulled By: seemethere

fbshipit-source-id: c504ebf7f0d4c0d6229319d774f808b4ba0facd9
2021-09-08 14:43:33 -07:00