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

Docker, Docker everywhere #32

Merged
merged 3 commits into from
Jul 2, 2015
Merged

Docker, Docker everywhere #32

merged 3 commits into from
Jul 2, 2015

Conversation

akx
Copy link
Contributor

@akx akx commented Jun 30, 2015

This PR adds a basic Dockerfile that can be used to boot up an "evaluation" instance of Shoop.

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/762/
Test FAILed.

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/763/
Test PASSed.

@JsseL
Copy link

JsseL commented Jun 30, 2015

PR or commit messages doesn't say much, but looks like we get somekind of docker support for Shoop. That sounds nice.

@suutari-ai
Copy link
Contributor

Build our resources before build_py (that copies files)

Please add better explanation to this commit message to explain why the order has to be changed. There is some stuff in parenthesis that does not belong to title line, but rather you should have a paragraph or two explaining what used to happen and how this commit changes it. We used quite a lot of time to debug this and commit message is a good place to log the (relevant) things we found out while debugging it.

akx added 2 commits June 30, 2015 13:26
It turns out that the `build_py` command actually also copies data files
into the build directory.  This naturally means we'll have to do the
Node.js based builds before `build_py` is allowed to copy those artifacts
further.

Refs SHOOP-994
Much quicker `migrate` (since there's, well, no `migrate`) for Docker
and testing purposes.

Refs SHOOP-983
@akx
Copy link
Contributor Author

akx commented Jun 30, 2015

Fixed (to some degree), @suutari-ai

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/764/
Test PASSed.

@@ -0,0 +1,11 @@
FROM node:0.12.5
RUN apt-get update && apt-get install -y --no-install-recommends python3-minimal python3-virtualenv python3-pip python3-dev python3-pil
RUN python3 -m virtualenv -p /usr/bin/python3 --system-site-packages /var/www/shoop/venv
Copy link
Contributor

Choose a reason for hiding this comment

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

Why system-site-packages?

Also, why not use "python3 -m venv" rather than "virtualenv" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why system-site-packages?

python3-pil

Copy link
Contributor

Choose a reason for hiding this comment

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

OK then. I usually let PIL compile itself, but maybe installing it from deb packages is better idea for the Docker image.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also, venv doesn't seem to work:

Command '['/var/www/shoop/venv/bin/python3', '-Im', 'ensurepip', '--upgrade', '--default-pip']' returned non-zero exit status 1

and I don't really have the inclination to figure out why when virtualenv does the trick :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use virtualenv then. Dunno what's the base image of node:0.12.5, but if it's Ubuntu, then probably the reason is this bug: https://bugs.launchpad.net/ubuntu/+source/python3.4/+bug/1290847

@suutari-ai
Copy link
Contributor

@akx, your commit message:

Build our resources before original build tasks

It turns out that the build_py command actually also copies data files
into the build directory. This naturally means we'll have to do the
Node.js based builds before build_py is allowed to copy those artifacts
further.

Refs SHOOP-994

Exactly what I meant. Thanks! 👍

@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/765/
Test FAILed.

Yay, it docks a demo shop!

Refs SHOOP-983
@anders-jenkins
Copy link

Refer to this link for build results (access rights to CI server needed):
http://spartacus.aintra:8080//job/shoop-ng-pull-requests/766/
Test PASSed.

@suutari-ai
Copy link
Contributor

Good work, @akx. Thanks!

suutari-ai added a commit that referenced this pull request Jul 2, 2015
Docker, Docker everywhere
@suutari-ai suutari-ai merged commit 737666d into shuup:master Jul 2, 2015
@akx akx deleted the setup-up-up branch July 2, 2015 09:27
tulimaki added a commit to tulimaki/shuup that referenced this pull request Jan 3, 2017
Admin: Update wizard address form (CLOUD-65)
maximlion pushed a commit to maximlion/shuup that referenced this pull request Jun 3, 2019
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.

None yet

4 participants