Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

Docker #417

Merged
merged 4 commits into from
Sep 16, 2015
Merged

Docker #417

merged 4 commits into from
Sep 16, 2015

Conversation

LinuxBozo
Copy link
Contributor

@jmarin Here's the docker setup for the pilot. This includes a very basic nginx setup that should be good enough, but there is always room for tweaking. It uses a proxy_pass setup to the API so we get around CORS issues. We do the same thing in production currently. Total image size weighs in at around 10MB. Not too shabby.

@jmarin
Copy link
Contributor

jmarin commented Aug 28, 2015

@LinuxBozo I am able to build the other containers (api, mongodb). There is an issue remaining with this one. When I run grunt dist on your docker branch I get the following:

Running "ngAnnotate:dist" (ngAnnotate) task
Warning: Cannot assign to read only property '$methodName' of false Use --force to continue.

Aborted due to warnings.

The dist folder ends up empty. I see the same error when running the build-docker-container.sh script

@LinuxBozo
Copy link
Contributor Author

@jmarin try with nodejs v0.10 installed.. not sure that we ever did compatibility with v0.12 and the pilot after looking over the travis setup.. Looks like we only did with engine and api..

wpears added a commit that referenced this pull request Sep 16, 2015
@wpears wpears merged commit 4bd6025 into cfpb:develop Sep 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants