Skip to content

[pytorch][CI] add e2e mobile custom build jobs to CI#34184

Closed
ljk53 wants to merge 4 commits intogh/ljk53/112/basefrom
gh/ljk53/112/head
Closed

[pytorch][CI] add e2e mobile custom build jobs to CI#34184
ljk53 wants to merge 4 commits intogh/ljk53/112/basefrom
gh/ljk53/112/head

Conversation

@ljk53
Copy link
Copy Markdown
Contributor

@ljk53 ljk53 commented Mar 4, 2020

Stack from ghstack:

Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

Differential Revision: D20241774

Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

[ghstack-poisoned]
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Mar 4, 2020

💊 CircleCI build failures summary and remediations

As of commit 81c2de2 (more details on the Dr. CI page):


None of the build failures appear to be your fault 💚


  • 1/1 broken upstream at merge base 2b79bab since Mar 05

    Please rebase on the viable/strict branch (expand for instructions)

    If your commit is newer than viable/strict, you can try basing on an older, stable commit:

    git fetch origin viable/strict
    git rebase --onto viable/strict $(git merge-base origin/master HEAD)
    

    If your commit is older than viable/strict:

    git fetch origin viable/strict
    git rebase viable/strict
    

    Check out the recency history of this "viable master" tracking branch.


🚧 1 upstream failure recognized by patterns:

These were probably caused by upstream breakages:


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

This comment has been revised 13 times.

Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

Differential Revision: [D20241774](https://our.internmc.facebook.com/intern/diff/D20241774)

[ghstack-poisoned]
Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

Differential Revision: [D20241774](https://our.internmc.facebook.com/intern/diff/D20241774)

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 4, 2020
Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

ghstack-source-id: 70bf93b
Pull Request resolved: #34184
@ljk53 ljk53 requested review from dreiss, ezyang and xta0 March 4, 2020 06:25
@ljk53
Copy link
Copy Markdown
Contributor Author

ljk53 commented Mar 4, 2020

default mobile e2e test: 22min: https://app.circleci.com/jobs/github/pytorch/pytorch/4683871
static dispatch custom build e2e test: 25min: https://app.circleci.com/jobs/github/pytorch/pytorch/4683877
dynamic dispatch custom build e2e test: 53min: https://app.circleci.com/jobs/github/pytorch/pytorch/4683862

@ezyang is it ok to run the dynamic one on every PR? what do you think?
In return, I changed mobile_code_analysis test to be master-only.

Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

Differential Revision: [D20241774](https://our.internmc.facebook.com/intern/diff/D20241774)

[ghstack-poisoned]
ljk53 added a commit that referenced this pull request Mar 5, 2020
Summary:
Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

ghstack-source-id: 7549b00
Pull Request resolved: #34184
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@ljk53 merged this pull request in e642a65.

@facebook-github-bot facebook-github-bot deleted the gh/ljk53/112/head branch March 9, 2020 14:16
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#34184

Add mobile custom build with static dispatch & dynamic dispatch to CI.
Most of mobile code analysis CI should be covered by the custom build +
dynamic dispatch flow, so changing it to running on master only.

Test Plan: Imported from OSS

Differential Revision: D20241774

Pulled By: ljk53

fbshipit-source-id: f34c5748735c536ab6b42c8eb1429d8bbdaefd62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants