Skip to content

Commit

Permalink
bors: Remove ci/hydra-eval from required statuses
Browse files Browse the repository at this point in the history
Recently we've seen several Bors merge failures from ci/hydra-eval
that terminate with

    error: unable to fork: Cannot allocate memory

This works around the issue (#1888) by removing ci/hydra-eval from
statuses required for Bors to merge.

This status will instead be added as a requirement to the GitHub
branch protection rules of this repo.

If the Hydra eval fails the first time around for the Bors merge
commit, Bors won't fail because eval isn't required for Bors
anymore. Hydra will automatically fix the eval, likely on the second
pass, and then build the job at which point Bors will get the build
notification and merge assuming the build is fine (which it should be
if the PR CI status is already passed).
  • Loading branch information
rvl committed Jul 17, 2020
1 parent 53805cc commit ffd104f
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion bors.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
status = [
"buildkite/cardano-wallet",
"ci/hydra-eval",
"ci/hydra-build:required",
]
timeout_sec = 7200
Expand Down

0 comments on commit ffd104f

Please sign in to comment.