Change task timeout limit to 3 days

appengine/swarming/server/task_request.py

 # coding: utf-8
 # Copyright 2014 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 """Tasks definition.
 
 Each user request creates a new TaskRequest. The TaskRequest instance saves the
 metadata of the request, e.g. who requested it, when why, etc. It links to the
 actual data of the request in a TaskProperties. The TaskProperties represents
@@ skipping 57 matching lines @@
 from server import config
 from server import task_pack
 import acl
 import cipd
 
 
 # Maximum acceptable priority value, which is effectively the lowest priority.
 MAXIMUM_PRIORITY = 255
 
 
-# One day in seconds. Add 10s to account for small jitter.
-_ONE_DAY_SECS = 24*60*60 + 10
+# Three days in seconds. Add 10s to account for small jitter.
+_THREE_DAY_SECS = 3*24*60*60 + 10
 
 
 # Seven day in seconds. Add 10s to account for small jitter.
 _SEVEN_DAYS_SECS = 7*24*60*60 + 10
 
 
 # Minimum value for timeouts.
 _MIN_TIMEOUT_SECS = 1 if utils.is_local_dev_server() else 30
 
 
@@ skipping 130 matching lines @@
 def _validate_task_run_id(_prop, value):
   """Validates a task_id looks valid without fetching the entity."""
   if not value:
     return None
   task_pack.unpack_run_result_key(value)
   return value
 
 
 def _validate_timeout(prop, value):
   """Validates timeouts in seconds in TaskProperties."""
-  if value and not (_MIN_TIMEOUT_SECS <= value <= _ONE_DAY_SECS):
+  if value and not (_MIN_TIMEOUT_SECS <= value <= _THREE_DAY_SECS):
     # pylint: disable=W0212
     raise datastore_errors.BadValueError(
         '%s (%ds) must be 0 or between %ds and one day' %

[M-A Ruel, 2017/07/21 14:35:56]

update

             (prop._name, value, _MIN_TIMEOUT_SECS))
 
 
 def _validate_tags(prop, value):
   """Validates TaskRequest.tags."""
   _validate_length(prop, value, 1024)
   if ':' not in value:
     # pylint: disable=W0212
     raise datastore_errors.BadValueError(
         '%s must be key:value form, not %s' % (prop._name, value))
@@ skipping 770 matching lines @@
   init_new_request(request, allow_high_priority, original_secret_bytes)
   return request
 
 
 def validate_priority(priority):
   """Throws ValueError if priority is not a valid value."""
   if 0 > priority or MAXIMUM_PRIORITY < priority:
     raise datastore_errors.BadValueError(
         'priority (%d) must be between 0 and %d (inclusive)' %
         (priority, MAXIMUM_PRIORITY))