Skip to content
This repository was archived by the owner on Sep 25, 2019. It is now read-only.

Conversation

lowjoel
Copy link
Member

@lowjoel lowjoel commented Jan 13, 2016

@fonglh See if this is sufficient to set up new build infra?

@fonglh
Copy link
Member

fonglh commented Jan 13, 2016

Do you still need Docker?

So to deploy this I configure the steps in the README as part of the deploy process, then use the system scripts to start?

@lowjoel
Copy link
Member Author

lowjoel commented Jan 13, 2016

Docker will be used later for spinning up containers for executing the submitted code.

This repo is actually just a daemon process that will poll Coursemology for thing to evaluate... Not sure what you're think of?

@fonglh
Copy link
Member

fonglh commented Jan 13, 2016

foreman runs and can generate upstart files.

I'm thinking through how to deploy it.

  1. Checkout repo
  2. bundle install
  3. Copy the entire thing over to the deploy machine.
  4. Run foreman for the Upstart scripts, copy to /etc/init on the deploy machine.
  5. sudo service evaluator start

Is that about right?

@lowjoel
Copy link
Member Author

lowjoel commented Jan 13, 2016

Oh, that's what you mean.

I intended for this to be deployed via gem install, but I guess your steps could be fine. We can discuss this in the ops repo.

@fonglh
Copy link
Member

fonglh commented Jan 13, 2016

Ok it seems sufficient then.

@lowjoel
Copy link
Member Author

lowjoel commented Jan 13, 2016

@allenwq can this be merged?

allenwq pushed a commit that referenced this pull request Jan 13, 2016
Define a Procfile for running the evaluator.
@allenwq allenwq merged commit 554719a into Coursemology:master Jan 13, 2016
@lowjoel lowjoel deleted the procfile branch January 13, 2016 12:15
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.

3 participants