ci: Adds MacOS Bazel Builder#9569
Conversation
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
clang-tidy review says "All clean, LGTM! 👍" |
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
|
Introduced in e3ee598 was for back when we had a self-hosted mac build. We may again someday but this currently means the action never runs. |
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
|
Fixed |
|
As you point out we should probably populate a bazel macos cache for the CI. Right now we're limited by the 10GB limit action cache. |
|
clang-tidy review says "All clean, LGTM! 👍" |
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
|
clang-tidy review says "All clean, LGTM! 👍" |
|
@maliberty the ci failed pulling a docker container should be good to go |
|
|
||
| # 2. SAVE: Only executes if this is a push/merge directly to the master branch | ||
| - name: Save Bazel Disk Cache | ||
| if: github.ref == 'refs/heads/master' && github.event_name == 'push' |
There was a problem hiding this comment.
Do we need to check if this is being run in the The-OpenROAD-Project org and not in a fork?
There was a problem hiding this comment.
I don't think so the upstream cache can only be run by the upstream action
|
As a point of policy this will not be merge blocking and mac remains best effort. It is nearly impossible to debug build problems without an actual mac. |
2 participants
No description provided.