Don't break cloud-init by uninstalling python packages, prefer pip to apt for more packages
Change-Id: Iabd2baf899641e66181d9342bc3883534d7b6862
diff --git a/packer/provision/basebuild.sh b/packer/provision/basebuild.sh
index 3b8c6a1..c433d2d 100644
--- a/packer/provision/basebuild.sh
+++ b/packer/provision/basebuild.sh
@@ -19,9 +19,6 @@
apt-get update
apt-get install -y apt-transport-https software-properties-common
- # set up ansible repo
- apt-add-repository -y ppa:ansible/ansible
-
# set up git backports repo
add-apt-repository -y ppa:git-core/ppa
@@ -87,7 +84,6 @@
apt-get install -y \
"docker-ce=17.06*" \
apt-transport-https \
- ansible \
build-essential \
bzip2 \
curl \
@@ -97,7 +93,6 @@
git \
golang-1.12-go \
graphviz \
- httpie \
jq \
kafkacat \
"kubeadm=1.12.7-*" \
@@ -122,23 +117,22 @@
# end of apt-get install list
# remove apt installed incompatible python tools
+ # NOTE: Python3 versions are not removed, as cloud-init depends on them
apt-get -y remove \
python-enum34 \
python-cryptography \
python-openssl \
python-ndg-httpsclient \
python-requests \
- python3-requests \
python-six \
- python3-six \
- python-urllib3 \
- python3-urllib3
+ python-urllib3
# install python modules
# upgrade pip or other installations may fail in unusual ways
pip install --upgrade pip
pip install \
Jinja2 \
+ ansible \
ansible-lint \
astroid==1.* \
coverage \
@@ -147,8 +141,10 @@
docker-compose==1.20.1 \
docker==3.2.1 \
gitpython \
+ git-review \
graphviz \
grpcio-tools \
+ httpie==1.0.3 \
isort \
linkchecker \
more-itertools==5.0.0 \
diff --git a/packer/provision/baseline.sh b/packer/provision/baseline.sh
index ed76c9d..a966a45 100644
--- a/packer/provision/baseline.sh
+++ b/packer/provision/baseline.sh
@@ -341,7 +341,7 @@
# --- START LFTOOLS DEPS
# Used by various scripts to push patches to Gerrit
- ensure_ubuntu_install git-review
+ # ensure_ubuntu_install git-review
# Needed to parse OpenStack commands used by opendaylight-infra stack commands
# to initialize Heat template based systems.