Merge pull request #2452 from kayibal/kubernetes-fixes

Kubernetes: Don't set activeDeadlineSeconds by default

luigi/contrib/kubernetes.py

@@ -186,7 +186,7 @@ def active_deadline_seconds(self):
         Time allowed to successfully schedule pods.
         See: https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion/#job-termination-and-cleanup
         """
-        return 100
+        return None
 
     @property
     def kubernetes_config(self):
@@ -329,7 +329,6 @@ def run(self):
                 }
             },
             "spec": {
-                "activeDeadlineSeconds": self.active_deadline_seconds,
                 "backoffLimit": self.backoff_limit,
                 "template": {
                     "metadata": {
@@ -339,6 +338,9 @@ def run(self):
                 }
             }
         }
+        if self.active_deadline_seconds is not None:
+            job_json['spec']['activeDeadlineSeconds'] = \
+                self.active_deadline_seconds
         # Update user labels
         job_json['metadata']['labels'].update(self.labels)
         # Add default restartPolicy if not specified