zuul/zuul-jobs
Code Issues Proposed changes

Fix CentOS wheel mirror URL

Browse Source 
For CentOS we are currently using default values for "wheel_mirror" from
roles/configure-mirrors/defaults/main.yaml, which means that the URL
uses the full distribution version, which is wrong, we should be using
the major version instead.

As an example, centos-8 jobs are trying to use
  https://mirror.bhs1.ovh.opendev.org/wheel/centos-8.3-x86_64
When they should be using
  https://mirror.bhs1.ovh.opendev.org/wheel/centos-8-x86_64

Which makes some jobs fail to find alembic wheel files and then they
fail to build from source because the CI was expecting the wheel mirror
to have it.

This patch fixes this by creating a Centos specific "wheel_mirror"
variable.

Change-Id: I11199cf943a3070fd62bf486fe2b06381db4b04d
This commit is contained in:
Gorka Eguileor 2021-01-19 12:55:55 +01:00
parent 57a2f59f5a
commit c9cad38eeb
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
roles/configure-mirrors/vars/CentOS.yaml
View File

@ -1,2 +1,3 @@
package_mirror: "{{ http_or_https }}://{{ mirror_fqdn }}/{{ ansible_distribution | lower }}" package_mirror: "{{ http_or_https }}://{{ mirror_fqdn }}/{{ ansible_distribution | lower }}"
epel_mirror: "{{ http_or_https }}://{{ mirror_fqdn }}/epel" epel_mirror: "{{ http_or_https }}://{{ mirror_fqdn }}/epel"
wheel_mirror: "{{ http_or_https }}://{{ mirror_fqdn }}/wheel/{{ ansible_distribution | lower }}-{{ ansible_distribution_major_version }}-{{ ansible_architecture | lower }}"