-
Notifications
You must be signed in to change notification settings - Fork 10
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
CircleCI MacOS builds Inop #27
Comments
@faustbrian any ideas why this could be? |
Related Issue also discussed in ArkEcosystemArchive/cpp-client#55 |
#46, and #47 (submitted myself) builds and passes all tests including macOS. The following three PR's (also submitted by me) in Cpp-Client trigger macOS builds which pass all tests including macOS:
The following PR from @ciband did not trigger macOS builds: Overall, macOS builds seem to fail sporadically. PR's from other users continue to not trigger macOS builds. |
The issue is simply that @ciband is not on a paid plain on CircleCI and because he works on a fork the tests won't run for macOS. I've added him as a contributor 1-2 weeks ago so he could work in branches so the macOS tests would work. |
I would think that circleci should run all PRs against the ArkEcosystem repo with the ArkEcosystem account, not the Github user who may or may not have a circleci account. This is how TravisCI worked (I believe). Are we sure there isn't some circleci config that we might be missing? As far as working directly in ArkEcosystem branches, I don't like that idea. The beauty of the GitHub workflow is the PR system from forks. We do a similar work flow in my day job with Gerrit. Obviously, I can work in the Ark repo and still do PRs, but we shouldn't have/need to. Ideal state should be full PR checks to run for any PR for anyone who wants to contribute. Not trying stir anything up, but that is my 2 sats. If I had the rights, I would attempt to investigate. |
Once you fork the repository you get your own CircleCI repository for which you have no paid macOS plan which results in those tests not running/failing. We use CircleCI instead of TravisCI because we had tons of timeout issues, even on the paid plan, and the performance of TravisCI is a joke compared to CircleCI which in a lot of repositories results in saving several minutes of runtime. |
@faustbrian I don't have a paid plan and am able to trigger macOS builds though. Am I added anywhere in CI that allows this? |
Probably because you are a member of the ArkEcosystem organisation. You can go to any of cibands failed test macOS runs and it will have a big, fat yellow banner that says |
Describe the bug
When CircleCI builds are triggered by PRs, the MacOS builds are not firing.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I would expect the macOS builds to work like they did in TravisCI.
Screenshots
N/A
Server (please complete the following information):
N/A
Additional context
cpp-client does not seem to have this issue. I assume it is something administrative.
The text was updated successfully, but these errors were encountered: