Merge pull request #42938 from kvignesh1420:sanity-fix
PiperOrigin-RevId: 330646777 Change-Id: I618227cbb0457bd36fdd2bffcfd5ff996b083325
This commit is contained in:
commit
054738d534
tensorflow/tools/ci_build
@ -162,7 +162,7 @@ mkdir -p ${WORKSPACE}/bazel-ci_build-cache
|
||||
# By default we cleanup - remove the container once it finish running (--rm)
|
||||
# and share the PID namespace (--pid=host) so the process inside does not have
|
||||
# pid 1 and SIGKILL is propagated to the process inside (jenkins can kill it).
|
||||
${DOCKER_BINARY} run --rm --pid=host \
|
||||
${DOCKER_BINARY} run --rm --name ${DOCKER_IMG_NAME} --pid=host \
|
||||
-v ${WORKSPACE}/bazel-ci_build-cache:${WORKSPACE}/bazel-ci_build-cache \
|
||||
-e "CI_BUILD_HOME=${WORKSPACE}/bazel-ci_build-cache" \
|
||||
-e "CI_BUILD_USER=$(id -u -n)" \
|
||||
|
@ -126,6 +126,22 @@ do_pylint() {
|
||||
|
||||
PYLINT_BIN="python3 -m pylint"
|
||||
|
||||
echo ""
|
||||
echo "check whether pylint is available or not."
|
||||
echo ""
|
||||
${PYLINT_BIN} --version
|
||||
if [[ $? -eq 0 ]]
|
||||
then
|
||||
echo ""
|
||||
echo "pylint available, proceeding with pylint sanity check."
|
||||
echo ""
|
||||
else
|
||||
echo ""
|
||||
echo "pylint not available." >&2
|
||||
echo ""
|
||||
return 1
|
||||
fi
|
||||
|
||||
if [[ "$1" == "--incremental" ]]; then
|
||||
PYTHON_SRC_FILES=$(get_py_files_to_check --incremental)
|
||||
|
||||
|
@ -96,13 +96,17 @@ pip3 install py-cpuinfo
|
||||
|
||||
# pylint==1.6.4 requires python-astroid (>= 1.4.5) requires lazy-object-proxy
|
||||
# Latest version of lazy-object-proxy (1.4.2) fails to install from source
|
||||
# when using setuptools 39.1.0
|
||||
# when using setuptools 39.1.0.
|
||||
# NOTE: Using the updated version of pylint for python3 as python2 is EOL,
|
||||
# thus using the updated version of lazy-object-proxy==1.4.3
|
||||
pip2 install lazy-object-proxy==1.4.1
|
||||
pip3 install lazy-object-proxy==1.4.1
|
||||
pip3 install lazy-object-proxy==1.4.3
|
||||
|
||||
# pylint tests require the following:
|
||||
# pylint tests require the following version. pylint==1.6.4 hangs erratically,
|
||||
# thus using the updated version of 2.5.3 only for python3 as python2 is EOL
|
||||
# and this version is not available.
|
||||
pip2 install pylint==1.6.4
|
||||
pip3 install pylint==1.6.4
|
||||
pip3 install pylint==2.5.3
|
||||
|
||||
# pycodestyle tests require the following:
|
||||
pip2 install pycodestyle
|
||||
|
Loading…
Reference in New Issue
Block a user