From c369bd2c69d1cf237d1f279626de057e8b182b1c Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Wed, 4 Apr 2018 21:33:23 -0400 Subject: [PATCH] Add bubblewrap for zuul-executor The zuul-executor used bubblewrap for isolation of ansible-playbook. Add it into windmill for now, but we really should make this an ansible-role. Change-Id: I9a459b5a073052e85fcd80bebd142db1bc217593 Signed-off-by: Paul Belanger --- playbooks/zuul-executor.yaml | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/playbooks/zuul-executor.yaml b/playbooks/zuul-executor.yaml index 54d7410..ade8a7d 100644 --- a/playbooks/zuul-executor.yaml +++ b/playbooks/zuul-executor.yaml @@ -20,6 +20,23 @@ include_role: name: openstack.zuul + # TODO(pabelanger): If we want to support xenial, we'll need to use this ppa + # since bubblewrap doesn't exist. + - name: Add openstack-infra PPA for bubblewrap + become: yes + apt_repository: + repo: ppa:openstack-ci-core/bubblewrap + when: + - ansible_distribution_release | lower == "xenial" + + # TODO(pabelanger): I'm thinking we should likely create + # ansible-role-bubblewrap to allow user to better manage this dependency. + - name: Ensure bubblewrap is installed. + become: yes + package: + name: bubblewrap + state: installed + - name: Setup openstack.logrotate role. include_role: name: openstack.logrotate