MI-DPLA / combine

Combine /kämˌbīn/ - Metadata Aggregator Platform
MIT License
26 stars 11 forks source link

add restart Django manage.py management command #369

Closed ghukill closed 5 years ago

ghukill commented 5 years ago

Would be a subset of update - perhaps, they should work in tandem?

antmoth commented 5 years ago

Can you expand on the context for this? @ghukill

ghukill commented 5 years ago

Idea was a Django management command that would restart Django app, Celery, and Livy (and possibility refresh session). Currently these actions happen when python manage.py update is run, but they are a bit piecemeal.

The Docker deployment has added a bit of complexity to this as the Django app, Celery, and Livy are all in separate containers, and thus can't readily be restarted from the context of a Django management command.

However, probably would still have utility in a non-Docker environment.

antmoth commented 5 years ago

Not useful to any actual users we are aware of.