From 9deb72e02e63325cbdd9a1d098eb216ee566c6d6 Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Mon, 29 May 2017 08:40:17 -0400 Subject: [PATCH] Remove git-review from DIB elements Looking at the git history for git-review, it isn't clear where we are actually using it. Additionally, codesearch.o.o seems to show some projects already managing git-review as a requirement. Ensure everybody is okay with this change by sending a ML post too. Change-Id: I53a45736de74b842c5c01982bb153f04bff5622c Signed-off-by: Paul Belanger --- .../elements/infra-package-needs/install.d/10-pip-packages | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/nodepool/elements/infra-package-needs/install.d/10-pip-packages b/nodepool/elements/infra-package-needs/install.d/10-pip-packages index 3f1588ce19..ebcc3e3966 100755 --- a/nodepool/elements/infra-package-needs/install.d/10-pip-packages +++ b/nodepool/elements/infra-package-needs/install.d/10-pip-packages @@ -20,9 +20,7 @@ if [ ${DIB_DEBUG_TRACE:-0} -gt 0 ]; then fi set -e -# TODO(fungi): switch jobs to use /usr/git-review-env/bin/git-review -packages='git-review==1.25.0 ' -packages+='tox ' +packages='tox' for package in $packages ; do pip install $package