Skip to content

Speed up alpine build - #703

Closed
LaurentGoderre wants to merge 2 commits into
nodejs:masterfrom
LaurentGoderre:alpine-ccache
Closed

Speed up alpine build#703
LaurentGoderre wants to merge 2 commits into
nodejs:masterfrom
LaurentGoderre:alpine-ccache

Conversation

@LaurentGoderre

Copy link
Copy Markdown
Member

Fixes#693

@LaurentGoderre

Copy link
Copy Markdown
MemberAuthor

/cc @tianon

@LaurentGoderre

Copy link
Copy Markdown
MemberAuthor

BTW @SimenB@PeterDaveHello@chorrell the build errors. For some reasons the build output doesn't get to Travis. This could be caused by build being so verbose that the maximum Travis log length is reached.

Since we are using multi-stage, we could redirect the build output to file in the Dockerfile and if we need to access that log we can get if from the intermediary image.

What do you fellows think?

@chorrell

Copy link
Copy Markdown
Contributor

I’m fine with reducing the verbosity of the logs a bit

@LaurentGoderre

Copy link
Copy Markdown
MemberAuthor

@LaurentGoderre

Copy link
Copy Markdown
MemberAuthor

@chorrell maybe it's not the log length.....it's so weird. Is it possible that the docker process detaches?

@LaurentGoderre
LaurentGoderreforce-pushed the alpine-ccache branch 2 times, most recently from c3a2607 to 925390eCompareApril 26, 2018 16:58
@LaurentGoderre

Copy link
Copy Markdown
MemberAuthor

Something is weird with v9 alipine. It seems to fail often. I restarted just that one here and it's all green https://travis-ci.org/LaurentGoderre/docker-node/builds/371661398

PeterDaveHello

This comment was marked as off-topic.

@LaurentGoderre
LaurentGoderreforce-pushed the alpine-ccache branch 5 times, most recently from 9503084 to 7201187CompareMay 15, 2018 18:03
PeterDaveHello

This comment was marked as off-topic.

PeterDaveHello

This comment was marked as off-topic.

@LaurentGoderre
LaurentGoderreforce-pushed the alpine-ccache branch 7 times, most recently from 251ee87 to e45115fCompareMay 22, 2018 03:40
Sign up for freeto 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.

3 participants

@LaurentGoderre@chorrell@PeterDaveHello