swarming: isolate defaults

appengine/swarming/server/task_request.py

 # coding: utf-8
 # Copyright 2014 The LUCI Authors. All rights reserved.
 # Use of this source code is governed by the Apache v2.0 license 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 76 matching lines @@
 # The world started on 2010-01-01 at 00:00:00 UTC. The rationale is that using
 # EPOCH (1970) means that 40 years worth of keys are wasted.
 #
 # Note: This creates a 'naive' object instead of a formal UTC object. Note that
 # datetime.datetime.utcnow() also return naive objects. That's python.
 _BEGINING_OF_THE_WORLD = datetime.datetime(2010, 1, 1, 0, 0, 0, 0)
 
 
 # Used for isolated files.
 _HASH_CHARS = frozenset('0123456789abcdef')
-_NAMESPACE_RE = re.compile(r'^[a-z0-9A-Z\-._]+$')
+NAMESPACE_RE = re.compile(r'^[a-z0-9A-Z\-._]+$')
 
 
 ### Properties validators must come before the models.
 
 
 def _validate_isolated(prop, value):
   if value:
     if not _HASH_CHARS.issuperset(value) or len(value) != 40:
       raise datastore_errors.BadValueError(
           '%s must be lowercase hex of length 40, not %s' % (prop._name, value))
@@ skipping 10 matching lines @@
     if parsed.netloc.endswith('appspot.com'):
       if parsed.scheme != 'https':
         raise datastore_errors.BadValueError(
             '%s must be https://, not %s' % (prop._name, value))
     elif parsed.scheme not in ('http', 'https'):
         raise datastore_errors.BadValueError(
             '%s must be https:// or http://, not %s' % (prop._name, value))
 
 
 def _validate_namespace(prop, value):
-  if not _NAMESPACE_RE.match(value):
+  if not NAMESPACE_RE.match(value):
     raise datastore_errors.BadValueError('malformed %s' % prop._name)
 
 
 def _validate_dict_of_strings(prop, value):
   """Validates TaskProperties.env."""
   if not all(
          isinstance(k, unicode) and isinstance(v, unicode)
          for k, v in value.iteritems()):
     # pylint: disable=W0212
     raise TypeError('%s must be a dict of strings' % prop._name)
@@ skipping 584 matching lines @@
   _put_request(request)
   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))