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

Hercules CI doesn't build PRs from external projects #626

Closed
phaer opened this issue Aug 31, 2023 · 5 comments
Closed

Hercules CI doesn't build PRs from external projects #626

phaer opened this issue Aug 31, 2023 · 5 comments

Comments

@phaer
Copy link
Member

phaer commented Aug 31, 2023

...but those checks are expected by github, so merging is blocked.

See e.g. #620 which never triggered a Hercules CI build according to https://hercules-ci.com/github/nix-community/dream2nix but waits for one since I have approved the checks to run.

I have permission to allow the checks, but not to check merge queue and ci settings, so we'll need at least @DavHau. Might be a good idea to ask roberth and/or mic92 about it if we are stuck? Not pinging them until clarified :)

@DavHau
Copy link
Member

DavHau commented Sep 2, 2023

I guess we have to migrate away from hercules. It doesn't seem to work with community contributions.
It won't submit to the merge queue until the checks have passed, but they cannot because hercules blocks.
If I remove the hercules checks from being required, it submits to the queue, but then it also doesn't execute the checks in the queue anymore.

Even if we could make the queue work and execute the checks there, it's still very suboptimal UX. Users should get quick feedback for failing builds on every commit, not just when the PR is submitted to the queue.
Therefore, if that is not possible it doesn't make sense in my opinion to use hercules.

@phaer
Copy link
Member Author

phaer commented Sep 3, 2023

Sound's good to me, but do we have an alternative ready? Back to Github Actions? Trying Garnix? Others?

Even if we could make the queue work and execute the checks there, it's still very suboptimal UX.

Totally agree, getting rid of that would be great :)

@DavHau
Copy link
Member

DavHau commented Sep 3, 2023

opened hercules-ci/support#70

@zowoq
Copy link

zowoq commented Sep 4, 2023

Even if we could make the queue work and execute the checks there, it's still very suboptimal UX.

I agree the the UX isn't great but if you'd like an interim solution something like nix-community/infra#793 should work.

We've also talked a little about deploying buildbot which would support external PRs but I couldn't say when it would be available.

@phaer
Copy link
Member Author

phaer commented Dec 8, 2023

Fixed with the switch to buildbot

@phaer phaer closed this as completed Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants