Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20044 )
Change subject: [build-support] unset *_proxy env variables for Python tests ...................................................................... [build-support] unset *_proxy env variables for Python tests The environment of a Jenkins node might have HTTP/HTTPS proxy configured to proxy HTTP requests even for 127.0.0.0/8 network and other quirks. Since Python Kudu tests don't assume any proxying of HTTP/HTTPS requests to Kudu components listening on 127.0.0.0/8 and other addresses, let's explicitly unset environment variables that might affect behavior of the Python libraries like urllib/urllib2 [1][2]. The motivation for this change was observing failures of the new Python tests introduced with 8c546ae7f560deebb32cb04815377f74470643c6 when running pre-commit Jenkins jobs on nodes of some peculiar network configuration. [1] https://docs.python.org/3/library/urllib.request.html [2] https://docs.python.org/2/library/urllib2.html Change-Id: I5a4d785e1eddf3b66c897a23f6b6c5d0b92392de Reviewed-on: http://gerrit.cloudera.org:8080/20012 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 459207ff5c103e7d4d60d974504eb8633e188d15) Reviewed-on: http://gerrit.cloudera.org:8080/20044 Reviewed-by: Yingchun Lai <[email protected]> --- M build-support/jenkins/build-and-test.sh 1 file changed, 12 insertions(+), 1 deletion(-) Approvals: Alexey Serbin: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20044 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I5a4d785e1eddf3b66c897a23f6b6c5d0b92392de Gerrit-Change-Number: 20044 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
