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

travis-ci: mitigate slow builds #1142

Merged
merged 2 commits into from Aug 5, 2017

Conversation

Projects
None yet
4 participants
@grondo
Copy link
Contributor

grondo commented Aug 4, 2017

This PR adds -j2 to many of the make commands during travis-ci builds (both builds of dependencies and make distcheck for the code itself)

This helps mitigate the problem described in #1141.

grondo added some commits Aug 4, 2017

travis-ci: add -j2 to make in dep builder
Try to get dependencies to build a little faster with make -j 2.
@coveralls

This comment has been minimized.

Copy link

coveralls commented Aug 4, 2017

Coverage Status

Coverage increased (+0.004%) to 77.923% when pulling efec906 on grondo:travis-mitigation into 38fed2d on flux-framework:master.

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Aug 4, 2017

Codecov Report

Merging #1142 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1142      +/-   ##
==========================================
+ Coverage   77.55%   77.56%   +<.01%     
==========================================
  Files         157      157              
  Lines       28680    28679       -1     
==========================================
+ Hits        22243    22244       +1     
+ Misses       6437     6435       -2
Impacted Files Coverage Δ
src/common/libflux/keepalive.c 86.66% <0%> (-6.67%) ⬇️
src/common/libflux/ev_flux.c 97.56% <0%> (-2.44%) ⬇️
src/broker/modservice.c 79.61% <0%> (-0.98%) ⬇️
src/connectors/local/local.c 86.82% <0%> (-0.78%) ⬇️
src/common/libflux/dispatch.c 85.18% <0%> (-0.75%) ⬇️
src/common/libflux/reactor.c 93.18% <0%> (-0.29%) ⬇️
src/common/libflux/handle.c 83.16% <0%> (-0.25%) ⬇️
src/common/libcompat/handle.c 84.47% <0%> (-0.24%) ⬇️
src/modules/connector-local/local.c 73.88% <0%> (-0.19%) ⬇️
src/common/libkvs/kvs.c 65.08% <0%> (+0.25%) ⬆️
... and 5 more
@garlick

This comment has been minimized.

Copy link
Member

garlick commented Aug 5, 2017

Looks good - thanks!

@garlick garlick merged commit 0abcd67 into flux-framework:master Aug 5, 2017

4 checks passed

codecov/patch Coverage not affected when comparing 38fed2d...efec906
Details
codecov/project 77.56% (+<.01%) compared to 38fed2d
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.004%) to 77.923%
Details

@grondo grondo referenced this pull request Aug 23, 2017

Closed

0.8.0 Release #1160

@grondo grondo deleted the grondo:travis-mitigation branch Apr 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.