Merge pull request pulp#2711 from dkliban/check-for-out-of-space-down…

…load

Adds a check for 'No space left on device' on download failures

server/pulp/plugins/util/publish_step.py

@@ -10,6 +10,8 @@
 import time
 import traceback
 import uuid
+import errno
+import signal
 
 from pulp.common import error_codes
 from pulp.common.plugins import reporting_constants, importer_constants
@@ -1147,6 +1149,8 @@ def download_failed(self, report):
         """
         self.progress_failures += 1
         self.report_progress()
+        if os.strerror(errno.ENOSPC) in report.error_msg:
+            os.kill(os.getpid(), signal.SIGKILL)
 
     def cancel(self):
         """

server/test/unit/plugins/util/test_publish_step.py

@@ -1023,6 +1023,7 @@ def test_download_succeeded(self):
     def test_download_failed(self):
         dlstep = publish_step.DownloadStep('fake-step')
         mock_report = Mock()
+        mock_report.error_msg = '404 encountered'
         mock_report_progress = Mock()
         dlstep.report_progress = mock_report_progress
         dlstep.download_failed(mock_report)