aws / sagemaker-mxnet-training-toolkit

Toolkit for running MXNet training scripts on SageMaker. Dockerfiles used for building SageMaker MXNet Containers are at https://github.com/aws/deep-learning-containers.
Apache License 2.0
60 stars 55 forks source link

fix: Fix issue in installation of sagemaker-containers #140

Closed saimidu closed 4 years ago

saimidu commented 4 years ago

Description of changes: Fix issue in installation of sagemaker-containers by constraining the version of inotify-simple being used in py2 dockerfiles to inotify-simple<1.3.

The following is the sequence of dependencies that causes this issue: sagemaker-mxnet-training > sagemaker-containers > gevent > inotify-simple > enum34

Installation of inotify-simple==1.3.0 results in the following error:

Collecting inotify-simple
  Downloading https://files.pythonhosted.org/packages/d5/b9/e8893eb0cf4ea05f091ca833d4129c5cbb193535fd4962fb5bb23c54b79f/inotify_simple-1.3.0.tar.gz
    ERROR: Command errored out with exit status 1:
     command: /usr/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-myOcJF/inotify-simple/setup.py'"'"'; __file__='"'"'/tmp/pip-install-myOcJF/inotify-simple/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-myOcJF/inotify-simple/pip-egg-info
         cwd: /tmp/pip-install-myOcJF/inotify-simple/
    Complete output (22 lines):
    /usr/local/lib/python2.7/dist-packages/pkg_resources/py2_warn.py:22: UserWarning: Setuptools will stop working on Python 2
    ************************************************************
    You are running Setuptools on Python 2, which is no longer
    supported and
    >>> SETUPTOOLS WILL STOP WORKING <<<
    in a subsequent release (no sooner than 2020-04-20).
    Please ensure you are installing
    Setuptools using pip 9.x or later or pin to `setuptools<45`
    in your environment.
    If you have done those things and are still encountering
    this message, please comment in
    https://github.com/pypa/setuptools/issues/1458
    about the steps that led to this unsupported combination.
    ************************************************************
      sys.version_info < (3,) and warnings.warn(pre + "*" * 60 + msg + "*" * 60)
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-myOcJF/inotify-simple/setup.py", line 3, in <module>
        from inotify_simple import __version__
      File "inotify_simple.py", line 3, in <module>
        from enum import Enum, IntEnum
    ImportError: No module named enum
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sagemaker-bot commented 4 years ago

AWS CodeBuild CI Report

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository