Skip to content
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

[ci] Properly test mac and linux in GitHub Actions CI #9130

Closed
wants to merge 4 commits into from

Conversation

SamChou19815
Copy link
Contributor

@SamChou19815 SamChou19815 commented Mar 18, 2024

Currently, the mac and linux tests are passing, but if we look at the logs, they are failing for every single test. It turns out that we are not even downloading the artifacts from previous build jobs. We also need to set the executable bit to actually make the tests runnable (this is not needed in circle ci jobs and also not needed for windows builds, so it's unclear why, which needs further investigation)

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@SamChou19815 SamChou19815 changed the title [ci] Enable tool_test_win [ci] Properly test mac and linux in GitHub Actions CI Mar 18, 2024
@SamChou19815 SamChou19815 marked this pull request as ready for review March 18, 2024 18:53
@facebook-github-bot
Copy link
Contributor

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

@SamChou19815
Copy link
Contributor Author

We also need to set the executable bit to actually make the tests runnable (this is not needed in circle ci jobs and also not needed for windows builds, so it's unclear why, which needs further investigation)

Note for myself:

actions/upload-artifact#38

@facebook-github-bot
Copy link
Contributor

@SamChou19815 merged this pull request in b0efcf1.

@SamChou19815 SamChou19815 deleted the SamChou19815-patch-1 branch March 20, 2024 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants