change: remove red from possible colors when streaming logs (#1121)

This change also adds a workaround in tox.ini to handle
boto/botocore@e87e7a7

Author: laurenyu

src/sagemaker/logs.py

@@ -1,4 +1,4 @@
-# Copyright 2017-2018 Amazon.com, Inc. or its affiliates. All Rights Reserved.
+# Copyright 2017-2019 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License"). You
 # may not use this file except in compliance with the License. A copy of
@@ -31,7 +31,9 @@ class ColorWrap(object):
     cell).
     """
 
-    _stream_colors = [31, 32, 33, 34, 35, 36]
+    # For what color each number represents, see
+    # https://misc.flogisoft.com/bash/tip_colors_and_formatting#colors
+    _stream_colors = [34, 35, 32, 36, 33]
 
     def __init__(self, force=False):
         """Initialize the class.

tox.ini

@@ -57,7 +57,9 @@ passenv =
 
 # {posargs} can be passed in by additional arguments specified when invoking tox.
 # Can be used to specify which tests to run, e.g.: tox -- -s
+# TODO: Remove pip install python-dateutil==2.8.0 once botocore no longer requires a pinned version
 commands =
+    pip install python-dateutil==2.8.0
     coverage run --source sagemaker -m pytest {posargs}
     {env:IGNORE_COVERAGE:} coverage report --fail-under=90 --omit */tensorflow/tensorflow_serving/*
 extras = test
@@ -112,7 +114,9 @@ deps =
 # pip install requirements.txt is separate as RTD does it in separate steps
 # having the requirements.txt installed in deps above results in Double Requirement exception
 # https://github.com/pypa/pip/issues/988
+# TODO: Remove pip install python-dateutil==2.8.0 once botocore no longer requires a pinned version
 commands =
+    pip install python-dateutil==2.8.0
     pip install --exists-action=w -r requirements.txt
     sphinx-build -T -W -b html -d _build/doctrees-readthedocs -D language=en . _build/html