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

[WIP] Semaphore CI #2823

Closed
wants to merge 5 commits into from
Closed

Conversation

DavidGregory084
Copy link
Member

@DavidGregory084 DavidGregory084 commented Apr 30, 2019

This PR is one possible option to fix #2319. It adds configuration for Semaphore CI. This looks like a good option but it's no so clear how or when they plan to support open source, which is strange because I believe Semaphore version 1 did so. The free plan is limited to $20 of build minutes per month at $0.00050/sec, which is ridiculously low.

If you guys think it's worth it I will contact them to try and clarify their policy on open source.

Compared with Travis it is very fast (about 15-16 min per build).

The configuration is well documented here and it was reasonably straightforward to set up.

The execution model is easy to understand; each block runs sequentially and jobs within a block run in parallel.

There is also a matrix feature which can be used to declare a build matrix using multiple environment variables if needed.

The only niggle I would mention is that it is not possible to add unspecified keys at the top level to use to hold the scala versions so I had to use env_vars blocks for that purpose.

From what I have seen in evaluating different services the flakiest test step which often struggles to run within memory limits is the Applicative.monoid.combineAll test, which frequently hits the GC overhead limit. I will raise that as a separate issue.

Screenshot 2019-04-30 at 16 55 41

@codecov-io
Copy link

codecov-io commented Apr 30, 2019

Codecov Report

Merging #2823 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2823   +/-   ##
=======================================
  Coverage   94.15%   94.15%           
=======================================
  Files         368      368           
  Lines        6914     6914           
  Branches      296      296           
=======================================
  Hits         6510     6510           
  Misses        404      404

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4cccb66...9359b5e. Read the comment docs.

@fbrdaric
Copy link

@DavidGregory084 I'm Filip from Semaphore Customer Success team. I'm glad to see you're trying out Semaphore! The full OS support for Semaphore 2.0 is planned to be shipped in the upcoming weeks. It'll include clear guidelines on supporting OS projects in terms of consumption. I'd be happy to share more details around this via email. If you're interested, please reach out to customersuccess@semaphoreci.com. I might be able to help you out right away :).

@larsrh
Copy link
Contributor

larsrh commented Oct 20, 2020

Since nobody has worked on this for quite a while, I'm closing all old CI-related PRs.

@larsrh larsrh closed this Oct 20, 2020
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

Successfully merging this pull request may close these issues.

Evaluate moving to Circle CI
4 participants