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

CI build speed-ups #2574

Closed
wants to merge 18 commits into from
Closed

CI build speed-ups #2574

wants to merge 18 commits into from

Conversation

bboreham
Copy link
Collaborator

@bboreham bboreham commented Jun 8, 2017

This takes it down to around 20 minutes, although scope is currently running with 3 "containers".

Ref #2271

bboreham added 9 commits June 7, 2017 10:17
If its stdout is a tty it will do this.  'RUN_FLAGS' is defaulted to
'-ti' so that control-C works, but we don't need that on CircleCI.
So we are using a version pre-installed by CircleCI
We don't really care if it completes: there's nothing we would do
differently if it doesn't.
@bboreham
Copy link
Collaborator Author

bboreham commented Jun 8, 2017

Netlify failure is caused by me bumping the Ruby version; it needs an associated change in the Netlify box
https://github.com/weaveworks/website-next/issues/173

bboreham added 5 commits June 8, 2017 12:43
we already used it in the 'pre' section
(a) seems reasonable to use the same tag for testing as for running
(b) this avoids rebuilding all the libraries for the different tag set
@rade
Copy link
Member

rade commented Jun 13, 2017

Is there anything left to do here?

@bboreham
Copy link
Collaborator Author

The changes in the tools directory need to be done in the build-tools repo.
The Netlify build needs to be updated.
It would be nice to know why the unit tests are still taking so long.

@2opremio
Copy link
Contributor

@bboreham Are you planning to finish up this work anytime soon? Otherwise I would like to close it (I am cleaning up the lingering PRs)

@bboreham
Copy link
Collaborator Author

We need to move to CircleCI 2.0, which will perhaps provoke larger changes.
I think I'll leave this open as a constant reminder.

@bboreham
Copy link
Collaborator Author

Closing in favour of #3333 (which runs in about 14 minutes)

@bboreham bboreham closed this Aug 27, 2018
@bboreham bboreham deleted the tweak-circle branch January 13, 2020 12:00
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.

3 participants