gitlab-ci: Implement head.hackage jobs
Depends upon !478 (merged) .
Edited by Ben Gamari
Merge request reports
Activity
This sounds cool, although I can't very easily tell what exactly is being tested here just from the
.gitlab-ci.ymlfile alone. Is there documentation somewhere that describes this in more detail?Also, the
hackagepart of this MR's pipeline was skipped, so it's hard to tell if this is working properly or not.Also, the
hackagepart of this MR's pipeline was skipped, so it's hard to tell if this is working properly or not.Oops, I think this was simply because CI was down. Now that it's back up, the
hackagejob failed with this error:Running with gitlab-runner 11.5.0 (v11.5.0) on ghc-ci-1 e7d5528a Using Docker executor with image debian/jessie ... Pulling docker image debian/jessie ... ERROR: Job failed: Error response from daemon: pull access denied for debian/jessie, repository does not exist or may require 'docker login' (executor_docker.go:168:0s)added 2 commits
@RyanGlScott and I discussed options for deploying this on IRC. There were a few ideas floated:
- Run on all MRs
- Run on all MRs that touch certain sets of files (e.g.
libraries/base,compiler/typecheck,compiler/parser) - Manually run hackage job on MRs during review. This unfortunately will require reviewers to repeatedly trigger the job every time new commits are pushed to the branch. Moreover, the manual job can only be triggered after all of its dependencies have passed. I suspect this will quickly grow tiresome.
- Make this hackage job run on all MRs bearing a given label. This would avoid the need to manually trigger the job but will require a small patch to GitLab which I am pushing upstream.
In my opinion the last option is best.
Edited by Ben Gamariadded 10 commits
-
741f8346...b1c7ffaf - 6 commits from branch
master - 142ca680 - gitlab-ci: Always build fedora27
- b46ab517 - gitlab-ci: Implement head.hackage jobs
- 46e896b2 - gitlab-ci: A bit of reorganization
- 6cd30122 - XXX: Disable irrelevant jobs
Toggle commit list-
741f8346...b1c7ffaf - 6 commits from branch
Please register or sign in to reply