add option to use Docker instead of VM #16

Merged
merenber merged 11 commits from docker into v1 2 years ago
Owner
There is no content yet.
merenber added 3 commits 2 years ago
merenber requested review from a268wang 2 years ago
merenber requested review from r345liu 2 years ago
merenber added 1 commit 2 years ago
f72c6665f9 add nslcd for mail and coffee
merenber added 1 commit 2 years ago
2814adcb7c don't setup auth on coffee in CI
merenber added 1 commit 2 years ago
dca957d61a update OU
Owner

Why not use docker-compose to set up the networks, volumes, etc? Also, why not start ceod automatically in docker, but to use sleep infinity and attach? That seems like bunch of manual steps that can be automated.

Why not use docker-compose to set up the networks, volumes, etc? Also, why not start ceod automatically in docker, but to use sleep infinity and attach? That seems like bunch of manual steps that can be automated.
Poster
Owner

We could use docker-compose - I just felt like writing a shell script (feel free to modify).
As for why we sleep infinity, it's because you're going to need to manually restart ceod if you change the source code. If ceod was the foreground process, this would require killing and re-creating the container. I figured it would be easier to just manually start ceod so that we can Ctrl-C and restart it.

We *could* use docker-compose - I just felt like writing a shell script (feel free to modify). As for why we `sleep infinity`, it's because you're going to need to manually restart ceod if you change the source code. If ceod was the foreground process, this would require killing and re-creating the container. I figured it would be easier to just manually start ceod so that we can Ctrl-C and restart it.
merenber added 1 commit 2 years ago
ae34b6b7ce add more UWLDAP users
Poster
Owner

I updated the VM so that we can use the same ceod_dev.ini file in both:
2438b1c1d9

I updated the VM so that we can use the same ceod_dev.ini file in both: https://git.uwaterloo.ca/csc/syscom-dev-environment/-/commit/2438b1c1d9d270aedb79b388e912df584853a319
merenber added 1 commit 2 years ago
35e577b866 use python image for coffee
Owner

Maybe we can have a simple superviser that restarts ceod on signal? Normally ceod restarts by itself anyways so manual restart would only be needed occasionally (using docker kill -sighup or similiar). Main thing is that I feel having to attach to each individual manually is too much manual work especially when docker can start services by itself.

I can make a pr to this branch with docker-compose and the superviser thing if you want, although I don't want to install docker on my computer right now. Doe any csc machines have docker installed?

Maybe we can have a simple superviser that restarts ceod on signal? Normally ceod restarts by itself anyways so manual restart would only be needed occasionally (using docker kill -sighup or similiar). Main thing is that I feel having to attach to each individual manually is too much manual work especially when docker can start services by itself. I can make a pr to this branch with docker-compose and the superviser thing if you want, although I don't want to install docker on my computer right now. Doe any csc machines have docker installed?
Poster
Owner

I can make a pr to this branch with docker-compose and the superviser thing if you want

Sure, that'd be great.

Doe any csc machines have docker installed?

Unfortunately none of the general-use machines have Docker installed - we haven't figured out a way to do that securely.

> I can make a pr to this branch with docker-compose and the superviser thing if you want Sure, that'd be great. > Doe any csc machines have docker installed? Unfortunately none of the general-use machines have Docker installed - we haven't figured out a way to do that securely.
merenber added 1 commit 2 years ago
f873d04139 Use docker-compose to manage containers. (#19)
merenber added 1 commit 2 years ago
d7e9de5f61 install virtualenv on host
merenber added 1 commit 2 years ago
0d408e4ed3 Merge branch 'v1' into docker
merenber merged commit de18a9f293 into v1 2 years ago
merenber deleted branch docker 2 years ago

Reviewers

a268wang was requested for review 2 years ago
r345liu was requested for review 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as de18a9f293.
Sign in to join this conversation.
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: public/pyceo#16
Loading…
There is no content yet.