Pass build env python limits to config object

readthedocs/doc_builder/config.py

@@ -2,6 +2,7 @@
                                       load as load_config)
 
 
+from .constants import BUILD_IMAGES, DOCKER_IMAGE
 from readthedocs.projects.exceptions import ProjectImportError
 
 
@@ -49,20 +50,20 @@ def extra_requirements(self):
 
     @property
     def python_interpreter(self):
-        if 'version' in self._yaml_config.get('python', {}):
-            ver = self._yaml_config['python']['version']
-            if str(ver).startswith('2'):
-                return 'python'
-            else:
-                return 'python3'
-        else:
-            return self._project.python_interpreter
+        ver = self.python_version
+        if ver in [2, 3]:
+            # Get the highest version of the major series version if user only
+            # gave us a version of '2', or '3'
+            ver = max(filter(
+                lambda x: x < ver + 1,
+                self._yaml_config.get_valid_python_versions(),
+            ))
+        return 'python{0}'.format(ver)
 
     @property
     def python_version(self):
         if 'version' in self._yaml_config.get('python', {}):
-            ver = self._yaml_config['python']['version']
-            return ver
+            return self._yaml_config['python']['version']
         else:
             if self._project.python_interpreter == 'python':
                 return 2
@@ -125,20 +126,36 @@ def load_yaml_config(version):
     """
 
     checkout_path = version.project.checkout_path(version.slug)
+    env_config = {}
+
+    # Get build image to set up the python version validation. Pass in the
+    # build image python limitations to the loaded config so that the versions
+    # can be rejected at validation
+    build_image = BUILD_IMAGES.get(
+        version.project.container_image,
+        BUILD_IMAGES.get(DOCKER_IMAGE, None),
+    )
+    if build_image:
+        env_config = {
+            'python': build_image['python'],
+        }
+
     try:
+        sphinx_env_config = env_config.copy()
+        sphinx_env_config.update({
+            'output_base': '',
+            'type': 'sphinx',
+            'name': version.slug,
+        })
         config = load_config(
             path=checkout_path,
-            env_config={
-                'output_base': '',
-                'type': 'sphinx',
-                'name': version.slug,
-            },
+            env_config=sphinx_env_config,
         )[0]
     except InvalidConfig:  # This is a subclass of ConfigError, so has to come first
         raise
     except ConfigError:
         config = BuildConfig(
-            env_config={},
+            env_config=env_config,
             raw_config={},
             source_file='empty',
             source_position=0,

readthedocs/doc_builder/constants.py

@@ -15,6 +15,7 @@
 
 PDF_RE = re.compile('Output written on (.*?)')
 
+# Docker
 DOCKER_SOCKET = getattr(settings, 'DOCKER_SOCKET', 'unix:///var/run/docker.sock')
 DOCKER_VERSION = getattr(settings, 'DOCKER_VERSION', 'auto')
 DOCKER_IMAGE = getattr(settings, 'DOCKER_IMAGE', 'rtfd-build')
@@ -25,3 +26,16 @@
 DOCKER_OOM_EXIT_CODE = 137
 
 DOCKER_HOSTNAME_MAX_LEN = 64
+
+# Build images
+BUILD_IMAGES = {
+    'readthedocs/build:14.04': {
+        'python': {'supported_versions': [2, 2.7, 3, 3.3]},
+    },
+    'readthedocs/build:16.04': {
+        'python': {'supported_versions': [2, 2.7, 3, 3.5]},
+    },
+    'readthedocs/build:beta': {
+        'python': {'supported_versions': [2, 2.7, 3, 3.3, 3.4, 3.5, 3.6]},
+    },
+}

readthedocs/projects/tasks.py

@@ -18,6 +18,7 @@
 from django.conf import settings
 from django.core.urlresolvers import reverse
 from django.utils.translation import ugettext_lazy as _
+from readthedocs_build.config import ConfigError
 
 from readthedocs.builds.constants import (LATEST,
                                           BUILD_STATE_CLONING,
@@ -132,7 +133,12 @@ def run(self, pk, version_pk=None, build_pk=None, record=True, docker=False,
                     status_code=423
                 )
 
-            self.config = load_yaml_config(version=self.version)
+            try:
+                self.config = load_yaml_config(version=self.version)
+            except ConfigError as e:
+                raise BuildEnvironmentError(
+                    'Problem parsing YAML configuration. {0}'.format(str(e))
+                )
 
         if self.setup_env.failure or self.config is None:
             self._log('Failing build because of setup failure: %s' % self.setup_env.failure)

requirements/pip.txt

@@ -5,7 +5,7 @@ docutils==0.11
 Sphinx==1.3.5
 Pygments==2.0.2
 mkdocs==0.14.0
-git+https://github.com/rtfd/readthedocs-build.git@db4ad19df4f432bfbbd56d05964c58b6634356f3#egg=readthedocs-build-2.0.6.dev
+git+https://github.com/rtfd/readthedocs-build.git@31082ec0e7c6597c8e05c0e821271aaacd715dc7#egg=readthedocs-build-2.0.7.dev
 django==1.8.16
 
 django-tastypie==0.12.2