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

Struggle using hercules for public repo #70

Open
DavHau opened this issue Sep 3, 2023 · 2 comments
Open

Struggle using hercules for public repo #70

DavHau opened this issue Sep 3, 2023 · 2 comments

Comments

@DavHau
Copy link

DavHau commented Sep 3, 2023

The dream2nix repo was migrated to using hercules, but we have a problem where PRs of external contributor are blocked by the hercules jobs not executing.

Github does have a feature to require maintainer confirmation before CI runs on PRs of first time contributors, but this gets ignored by hercules. Even when confirming, the jobs won't run.

I heard a recommendation to use merge queues, but I don't see this as a viable alternative. Running status checks should be a cheap operation that doesn't require much human intervention. Queuing a PR for merge requires the review to be finished thereby making running status checks an expensive operation.
This seems to go against some principle of continuous integration which is to get quick feedback on the quality of a change.
Practically speaking, if I see that a PR fails some tests, I might decide to spend my time instead reviewing higher quality PRs, but I cannot do that if can't easily get a CI status.

Is there a simple way to fix it which I'm missing?
Does hercules plan anytime soon to integrate with github's feature that allows maintainers to confirm CI runs for external contributors?

@roberth
Copy link
Member

roberth commented Sep 3, 2023

A good solution to this just hasn't been implemented yet.
Hercules CI has been focused on a paying customer archetype that's doing CI/CD on private repos. In this scenario forks don't exist, or fork PRs are rare.
Nonetheless I want to help the OSS community, and this seems to be the most important issue after the (almost solved now) slowdowns from what would be mass rebuilds if it wasn't for cache.nixos.org.

@roberth
Copy link
Member

roberth commented Oct 4, 2023

Besides the undesirable human intervention, merge queues don't offer a way to try without merging or bypass the approvals requirement (if that's been configured).

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

2 participants