okpy / ok

ok.py supports programming projects by running tests, tracking progress, and assisting in debugging.
Apache License 2.0
335 stars 84 forks source link

Create custom docker image for worker #1298

Closed c-w closed 6 years ago

c-w commented 6 years ago

AppService for Containers seems to have an issue with the custom run command for the Docker image. According to the logs, the worker's AppService was starting an instance of the OK server instead of the worker. As a simple work-around, this change introduces a Dockerfile for the OK worker which bakes the correct CMD into the image. This does create some level of redundancy, but on the bright side it also means that in the future we could publish a more optimized image for the worker, e.g. excluding the frontend assets, etc.

Also: pass-through auto-grader URL in ARM template

c-w commented 6 years ago

CC @marrobi since this affects the ARM templates.

c-w commented 6 years ago

Going to close this since we faced a few issues with running workers in AppService. Going to open a new pull request soon which moves the compute from AppService to Azure Kubernetes Service. Apologies for the back and forth.