Skip to content

Forward fix macOS jobs after test-infra #5086 #2980

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

huydhn
Copy link
Contributor

@huydhn huydhn commented Apr 10, 2024

After pytorch/test-infra#5086, the working directory is now set correctly, so pushd isn't needed anymore. More importantly, trying to change the directory ends up failing all macOS CI jobs because that subdirectory doesn't exist.

Copy link

pytorch-bot bot commented Apr 10, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/2980

Note: Links to docs will display an error until the docs builds have been completed.

❌ 4 New Failures

As of commit 6f967c3 with merge base 859e924 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 10, 2024
@huydhn huydhn marked this pull request as ready for review April 10, 2024 23:46
@huydhn huydhn requested a review from larryliu0820 April 10, 2024 23:47
@facebook-github-bot
Copy link
Contributor

@huydhn has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@huydhn huydhn changed the title Forward fix macOS job after test-infra #5086 Forward fix macOS jobs after test-infra #5086 Apr 10, 2024
@facebook-github-bot
Copy link
Contributor

@huydhn merged this pull request in 2fc99b0.

@kirklandsign
Copy link
Contributor

@pytorchbot cherry-pick --onto release/0.2 -c regression

@pytorchbot
Copy link
Collaborator

Cherry picking #2980

The cherry pick PR is at #3020 and it is recommended to link a regression cherry pick PR with an issue

Details for Dev Infra team Raised by workflow job

pytorchbot pushed a commit that referenced this pull request Apr 12, 2024
Summary:
After pytorch/test-infra#5086, the working directory is now set correctly, so `pushd` isn't needed anymore.  More importantly, trying to change the directory ends up failing all macOS CI jobs because that subdirectory doesn't exist.

Pull Request resolved: #2980

Reviewed By: larryliu0820

Differential Revision: D55996299

Pulled By: huydhn

fbshipit-source-id: 05758603d7628cc0a01fd577a49202d45c84e6c5
(cherry picked from commit 2fc99b0)
@mergennachin mergennachin mentioned this pull request Apr 26, 2024
@huydhn huydhn deleted the forward-fix-after-test-infra-5086 branch August 13, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants