fix the pylint hang issue
This commit is contained in:
parent
ab3601d340
commit
52204b8e32
@ -162,7 +162,7 @@ mkdir -p ${WORKSPACE}/bazel-ci_build-cache
|
|||||||
# By default we cleanup - remove the container once it finish running (--rm)
|
# 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
|
# 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).
|
# 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 \
|
-v ${WORKSPACE}/bazel-ci_build-cache:${WORKSPACE}/bazel-ci_build-cache \
|
||||||
-e "CI_BUILD_HOME=${WORKSPACE}/bazel-ci_build-cache" \
|
-e "CI_BUILD_HOME=${WORKSPACE}/bazel-ci_build-cache" \
|
||||||
-e "CI_BUILD_USER=$(id -u -n)" \
|
-e "CI_BUILD_USER=$(id -u -n)" \
|
||||||
|
@ -126,6 +126,22 @@ do_pylint() {
|
|||||||
|
|
||||||
PYLINT_BIN="python3 -m 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
|
if [[ "$1" == "--incremental" ]]; then
|
||||||
PYTHON_SRC_FILES=$(get_py_files_to_check --incremental)
|
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
|
# 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
|
# 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
|
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
|
pip2 install pylint==1.6.4
|
||||||
pip3 install pylint==1.6.4
|
pip3 install pylint==2.5.3
|
||||||
|
|
||||||
# pycodestyle tests require the following:
|
# pycodestyle tests require the following:
|
||||||
pip2 install pycodestyle
|
pip2 install pycodestyle
|
||||||
|
Loading…
Reference in New Issue
Block a user