Merge pull request #1521 from deis/1354-delete-logs

fix(controller): remove application logs on delete

Author: Matthew Fisher

controller/api/models.py

@@ -139,6 +139,10 @@ def create(self, *args, **kwargs):
     def delete(self, *args, **kwargs):
         for c in self.container_set.all():
             c.destroy()
+        # delete application logs stored by deis/logger
+        path = os.path.join(settings.DEIS_LOG_DIR, self.id + '.log')
+        if os.path.exists(path):
+            os.remove(path)
         return super(App, self).delete(*args, **kwargs)
 
     def deploy(self, release, initial=False):

controller/bin/boot

@@ -48,6 +48,9 @@ done
 
 cd /app
 
+# give the deis user permissions to write to the volume mount
+chown -R deis:deis /var/log/deis
+
 # run an idempotent database migration
 sudo -E -u deis ./manage.py syncdb --migrate --noinput