-
Notifications
You must be signed in to change notification settings - Fork 46
Dockerize dev environment #44
Comments
Heads up, this will be a bit of a brain dump.... So what would we actually like here from a user perspective? To be able to run:
and have two containers spun up, one for the registry server, one for mongodb and have them linked together? Would we want this to be a kind of testbed where the data is ephemeral and disappears after the containers go away or keep it around even if we remove the containers? What does the Vagrant solution do? Finally, some people prefer their docker separated from their code. We could have another repository under the jsonresume org or we can put the necessary files in this repo - any preferences? |
Please note: I'm coming from a position of ignorance as far as Docker goes 😛 Running Personally I think I'm OK with it being an ephemeral test-bed if that's easier, but having it persistent would be cool. Vagrant saves the state of your VM. Unless you destroy and re-create the VM, the data in Mongo is kept (even if you re-provision, I think). What are the arguments for/against separating/integrating the docker stuff? |
I would be very surprised if it is difficult to make the data persistent - mostly depends on how mongo stores its data. Arguments around separating are basically the usual coupling ones, that you end up having both build and deployment artifacts in the same repo. No big deal either way I think. Probably best to keep things together for now and split out later if necessary? |
I managed to simplify the Dockerfile by using
This works, but I then also discoverd that the connection to the theme server is hardcoded but because the theme server doesn't have CORS enabled, this will fail. I solved it by copying /lib/theme.js from the themeserver repository and adding the corresponding routes to server.js/ This also resulted in errors because the themes are grabbed from the / route by the editor. By updating the routes to use /theme, I now have a working combined registry-server and theme-server. For details, check this commit: codefornl@b20864f |
@williammartin
The text was updated successfully, but these errors were encountered: