From 4063aa32b4d2b1c20b8b77607cc3a4ebbe523c52 Mon Sep 17 00:00:00 2001 From: Paul Belanger Date: Fri, 4 Aug 2017 12:44:19 -0400 Subject: [PATCH] Remove configure-mirrors from unittests/pre Right now, the configure-mirrors role is very openstack-centric. At the moment, I don't believe it is generic enough to be used by zuulv3 users. Additionally, it is completely possible users don't have mirror infrastrucuture in place and just want to use upstream directly. For now, remove this from our default zuulv3 jobs until we can better make this generic. In the case of openstack-infra, we have added this role into our base job which we have shadowed. Change-Id: Ic846d158c5458126468b0de2861d8e56b0d39c3e Signed-off-by: Paul Belanger --- playbooks/unittests/pre.yaml | 3 --- 1 file changed, 3 deletions(-) diff --git a/playbooks/unittests/pre.yaml b/playbooks/unittests/pre.yaml index 435e765e6..45364e5e2 100644 --- a/playbooks/unittests/pre.yaml +++ b/playbooks/unittests/pre.yaml @@ -1,8 +1,5 @@ - hosts: all roles: - - role: configure-mirrors - # TODO(mordred) When we have site-local variables, these should go there - mirror_fqdn: "mirror.{{ nodepool.region | lower }}.{{ nodepool.provider | lower }}.openstack.org" - role: bindep bindep_profile: test bindep_dir: "{{ zuul_work_dir }}"