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

Initial CircleCI 2.0 configuration #1117

Merged
merged 4 commits into from Sep 11, 2017
Merged

Initial CircleCI 2.0 configuration #1117

merged 4 commits into from Sep 11, 2017

Conversation

noirbizarre
Copy link
Contributor

@noirbizarre noirbizarre commented Sep 9, 2017

This PR try to enable CircleCI 2.0 and workflows.
The resuling build goes from around 15 minutes to between 7 and 8 minutes by using workflows (Python dependencies and tests run at the same time than assets testing and compilation)

CircleCI 2.0 does not allows to use tmpfs (to speed up the build) neither to use docker-compose with direct port access to reuse the existing docker-compose.yml file.

It requires #1118 and #1119 to make the build pass (these issues only appears with a fresh/clean dependencies cache)

@noirbizarre noirbizarre requested a review from a team September 11, 2017 08:22
Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably already saw that, but just FYI about tmpfs: https://discuss.circleci.com/t/docker-tmpfs-support/10416/3

@noirbizarre
Copy link
Contributor Author

Yes, I already saw that 😒
Let's hope CircleCI improve that (tmpfs brings a lot of spped to the test)

@noirbizarre
Copy link
Contributor Author

BTY, this PR also fixes CircleCI cache issue

@noirbizarre noirbizarre merged commit bcc63f5 into opendatateam:master Sep 11, 2017
@noirbizarre noirbizarre deleted the circleci2 branch September 11, 2017 09:42
noirbizarre added a commit that referenced this pull request Sep 11, 2017
Initial CircleCI 2.0 configuration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants