Skip to content

Enable autopy-server to replicate across nodes - #1317

Merged
colinschoen merged 1 commit into
masterfrom
bug/c-w/azure-flask-admin-for-autopy
Aug 2, 2018
Merged

Enable autopy-server to replicate across nodes#1317
colinschoen merged 1 commit into
masterfrom
bug/c-w/azure-flask-admin-for-autopy

Conversation

@c-w

@c-w c-w commented Aug 1, 2018

Copy link
Copy Markdown
Contributor

Currently the autopy-server stores the autograder files on the local
file-system via flask-admin. In Kubernetes, the file-system is managed
via a persistent volume claim. However, the volume can only attach to a
single node which means that all autopy-server pods have to be scheduled
to the same node which limits fault tolerance and scalability.

This change switches the autopy-server to use the new direct Azure Blob
Storage integration for flask-admin which stores the autograder files in
a non-local storage. As such, we can remove the persistent volume claim
and autopy-server pods can now be scheduled to arbitrary nodes.

This issue was found by @tmbgreaves.

Currently the autopy-server stores the autograder files on the local
file-system via flask-admin. In Kubernetes, the file-system is managed
via a persistent volume claim. However, the volume can only attach to a
single node which means that all autopy-server pods have to be scheduled
to the same node which limits fault tolerance and scalability.

This change switches the autopy-server to use the new direct Azure Blob
Storage integration for flask-admin which stores the autograder files in
a non-local storage. As such, we can remove the persistent volume claim
and autopy-server pods can now be scheduled to arbitrary nodes.

@colinschoen colinschoen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@colinschoen
colinschoen merged commit 2458f96 into master Aug 2, 2018
@colinschoen
colinschoen deleted the bug/c-w/azure-flask-admin-for-autopy branch August 2, 2018 01:34
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.

2 participants