-
Notifications
You must be signed in to change notification settings - Fork 233
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
convert circleci workflows to github actions #533
convert circleci workflows to github actions #533
Conversation
Do you suggest we merge as is and iterate, or iterate here first? For Windows, which tar/gzip are on the path? For Linux/Mac its all in the OCaml zlib stuff, which I know @shayne-fletcher debugged. Context for Shayne: We are switching to GitHub actions more broadly and @robandpdx is helping us. |
the issue appears to be a known ocaml 5.1.0 bug that's fixed in 5.1.1. PR https://github.com/robandpdx-org/buck2/pull/25 upgrades to 5.1.1 in the opam init action. |
got that wrong. neither sufficient or necessary to upgrade to 5.1.1. in fact #534 is seen to fix it. |
Summary: fix link error on macos x86_64 noted in facebook#533. test results showing the fix works here: https://github.com/shayne-fletcher/buck2/actions/runs/7515602243 Test Plan: Imported from GitHub, without a `Test Plan:` line. https://www.internalfb.com/sandcastle/workflow/4386506037060688764 Reviewed By: ndmitchell Differential Revision: D52802355 Pulled By: shayne-fletcher
Summary: fix link error on macos x86_64 noted in #533. test results showing the fix works here: https://github.com/shayne-fletcher/buck2/actions/runs/7515602243 Pull Request resolved: #534 Test Plan: Imported from GitHub, without a `Test Plan:` line. https://www.internalfb.com/sandcastle/workflow/4386506037060688764 Reviewed By: ndmitchell Differential Revision: D52802355 Pulled By: shayne-fletcher fbshipit-source-id: 542c72ec81f26f5ec85ed032cd41b5a900f261a6
@ndmitchell Since the workflow is triggered by push to main, it's probably easier to merge first, then iterate. Iterating in a fork is also a good option, and suited me well during my work on this. On windows bash, we have...
... and pwsh...
|
@shayne-fletcher Thank you! I've rebased my fork and am running the workflow again now. I'll report back. |
@shayne-fletcher Progress. New errors with I'm happy to keep iterating with your guidance. You are also welcome to push to the branch in my fork and PR/merge to my forks main as well. |
the 'buck2-examples' branch https://github.com/shayne-fletcher/buck2/tree/buck2-examples is based off 'robanpdx-org/convert-facebook-buck2-to-actions-20240105-214735' on this branch the "Build and test" workflow runs the 'macos-build-examples' and 'linux-build-examples' jobs. see results here. everything up to here works. in particular, the go examples on macos and the conan examples on linux now succeed. i suggest incorporating the relevant fixes from buck2-examples into convert-facebook-buck2-to-actions-20240105-214735. |
@shayne-fletcher Do we want to discard the jobs removed in this change? Let me know please. |
hi @robandpdx. i think with that last push in which we got the linux-build-examples and macos-build-examples working that a good strategy now is to land this excellent work and iterate further in place.
so, in short yes. i see two ways forward, (1) sync your branch with the changes in https://github.com/shayne-fletcher/buck2/tree/buck2-examples including the change that removes the jobs that don't work yet and i import it and get it landed or (2) abandon your branch and i import https://github.com/shayne-fletcher/buck2/tree/buck2-examples directly. in either case, we can aim to have the linux & mac build examples working and we can resume work to get the other jobs going from there. how does all this sit with you and if you are agreeable which of (1) and (2) do you prefer? (should mention i had been planning to do this today) |
@shayne-fletcher With the linux and macos examples working, the only job not working is the windows examples. So we can keep the build and test jobs for linux, macos, and windows seen working here. That said, I'm happy to land these 2 jobs that build the examples, then follow up with another PR for the build and test jobs for linux, macos, and windows. This PR is ready now with option 1. Building now here. |
@shayne-fletcher has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Pull Request resolved: #533 Test Plan: https://github.com/robandpdx-org/buck2/actions/runs/7585649935 Reviewed By: ndmitchell Differential Revision: D52910227 Pulled By: shayne-fletcher fbshipit-source-id: f2a8964cc2d92614fb3cf77b7792223e308db13e
@robandpdx merged in 965dcc2. thank-you! |
This pull request converts the CircleCI workflows to GitHub actions workflows.
Notes
I could not get the "Increase open file descriptor limit" job to work on macos.
sudo
is not available on macos runners.Errors
The following jobs have errors that I am not able to solve. Someone with better knowledge fo the project will need to adress these error.
linux-build-examples error
Error
macos-build-examples error
Error
windows-build-examples error
Error
Due the the errors above, I was not able to test workflow steps that followed these error.
Here is the latest workflow run in my fork.
https://fburl.com/workplace/f6mz6tmw