You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by mi...@apache.org on 2018/08/24 02:39:13 UTC

impala git commit: IMPALA-7460 part 2: upgrade Paramiko and Fabric in extended test env

Repository: impala
Updated Branches:
  refs/heads/master 1d84d6855 -> 063d2c9d5


IMPALA-7460 part 2: upgrade Paramiko and Fabric in extended test env

Upgrade Paramiko to the latest, 2.4.1. Paramiko drastically changed its
dependencies in Paramiko 2, dropping defunct Pycrypto and using Cryptography
instead.

https://github.com/paramiko/paramiko/issues/637
https://github.com/paramiko/paramiko/pull/394

This change implicitly removes the dependency on Pycrypto.

Also upgrade Fabric to the latest 1.x version, 1.14.0.

Testing:
- This works in my development environment.
- This works in my downstream stress and query gen environments.
- This works when doing a full data load.
- Impala still builds on a variety of OSs.

Change-Id: I0636d8113be449953420e1d5773f63d7c91943e3
Reviewed-on: http://gerrit.cloudera.org:8080/11308
Reviewed-by: Philip Zeyliger <ph...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/063d2c9d
Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/063d2c9d
Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/063d2c9d

Branch: refs/heads/master
Commit: 063d2c9d554331d75dffe910a7b450bf143842fa
Parents: 1d84d68
Author: Michael Brown <mi...@cloudera.com>
Authored: Wed Aug 22 15:40:11 2018 -0700
Committer: Impala Public Jenkins <im...@cloudera.com>
Committed: Fri Aug 24 01:12:16 2018 +0000

----------------------------------------------------------------------
 infra/python/deps/extended-test-requirements.txt | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/impala/blob/063d2c9d/infra/python/deps/extended-test-requirements.txt
----------------------------------------------------------------------
diff --git a/infra/python/deps/extended-test-requirements.txt b/infra/python/deps/extended-test-requirements.txt
index c62c99a..4876ec8 100644
--- a/infra/python/deps/extended-test-requirements.txt
+++ b/infra/python/deps/extended-test-requirements.txt
@@ -16,9 +16,10 @@
 # under the License.
 
 # Requirements for use of test tooling (tests/comparison and tests/stress).
-
-# IMPALA-7460 TODO: upgrade Fabric and Paramiko
-Fabric == 1.10.2
-  paramiko == 1.15.2
-    ecdsa == 0.13
-  pycrypto == 2.6.1
+# The random query generator uses Fabric to interact with a remote host
+# running Docker.
+Fabric==1.14.0
+# Fabric depends on Paramiko. Additionally, the stress test uses
+# Paramiko directly to keep a persistent SSH connection open to each
+# Impalad host to run in-test monitoring.
+paramiko==2.4.1