dready avatar dready committed ccff446

option to email only if job failed

Comments (0)

Files changed (2)

src/factory/signals.py

 from django.conf import settings
 
 def send_build_email(instance, **kwargs):
-    if not getattr(settings, 'FF_EMAIL_BUILD_RESULTS', False):
-        return
-
     if not instance.executed:
         return
 
     if not instance.fabfile_recipe.notify:
         return
 
+    email_results = getattr(settings, 'FF_EMAIL_BUILD_RESULTS', False)
+    if not email_results or (email_results is "failed" and instance.success):
+        return
+
     subject = '%s Build Results (%s)' % (instance, "success" if instance.success else "failure")
     msg = """
     Tasks: %(task)s

src/project/settings.py

     pass
 
 
-FF_EMAIL_BUILD_RESULTS = True
+FF_EMAIL_BUILD_RESULTS = "failed"
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.