Merge pull request #1 from teamhephy/feature/exceptions-on-release-api

Adding exception field on release api

Author: Cryptophobia

rootfs/api/migrations/0026_release_exception.py

@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.2 on 2017-08-28 14:59
+from __future__ import unicode_literals
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('api', '0025_app_procfile_structure'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='release',
+            name='exception',
+            field=models.TextField(blank=True, null=True),
+        ),
+    ]

rootfs/api/models/build.py

@@ -72,6 +72,8 @@ def create(self, user, *args, **kwargs):
             if 'new_release' in locals():
                 new_release.failed = True
                 new_release.summary = "{} deployed {} which failed".format(self.owner, str(self.uuid)[:7])  # noqa
+                # Get the exception that has occured
+                new_release.exception = "error: {}".format(str(e))
                 new_release.save()
             else:
                 self.delete()

rootfs/api/models/release.py

@@ -24,6 +24,7 @@ class Release(UuidAuditedModel):
     version = models.PositiveIntegerField()
     summary = models.TextField(blank=True, null=True)
     failed = models.BooleanField(default=False)
+    exception = models.TextField(blank=True, null=True)
 
     config = models.ForeignKey('Config', on_delete=models.CASCADE)
     build = models.ForeignKey('Build', null=True, on_delete=models.CASCADE)
@@ -243,6 +244,8 @@ def rollback(self, user, version=None):
             if 'new_release' in locals():
                 new_release.failed = True
                 new_release.summary = "{} performed roll back to a release that failed".format(self.owner)  # noqa
+                # Get the exception that has occured
+                new_release.exception = "error: {}".format(str(e))
                 new_release.save()
             raise DeisException(str(e)) from e
 

rootfs/api/tests/test_release.py

@@ -114,7 +114,7 @@ def test_response_data(self, mock_requests):
         response = self.client.get(url)
         for key in response.data.keys():
             self.assertIn(key, ['uuid', 'owner', 'created', 'updated', 'app', 'build', 'config',
-                                'summary', 'version', 'failed'])
+                                'summary', 'version', 'failed', 'exception'])
         expected = {
             'owner': self.user.username,
             'app': app_id,

rootfs/api/views.py

@@ -289,6 +289,8 @@ def post_save(self, config):
             if hasattr(self, 'release'):
                 self.release.failed = True
                 self.release.summary = "{} deployed a config that failed".format(self.request.user)  # noqa
+                # Get the exception that has occured
+                self.release.exception = "error: {}".format(str(e))
                 self.release.save()
             else:
                 config.delete()