openstack/openstack-ansible-ceph_client
Code Issues Proposed changes

Remove ceph_yum_repo_url leftovers

Browse Source 
With [1] the ceph_yum_repo_url was deprecated in favor of unified
ceph_repo_url variable, though it was not fully removed from
redhat vars, resulting in "variable not defined" error.

Also a type was made for defenition of yum repos at the same patch
that's being addressed now.

[1] https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/929645

Change-Id: I971754edcf7cc58e67b91f726c4f16aa97374bb2
This commit is contained in:
Dmitriy Rabotyagov 2024-10-28 18:25:01 +01:00
parent 07d41f641f
commit 08ed9d9873
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tasks/ceph_preinstall_dnf.yml
View File

@ -59,7 +59,7 @@
gpgkey: "{{ item.gpgkey | default(omit) }}"
name: "{{ item.name }}"
priority: "{{ item.priority | default(omit) }}"
state: "{{ item.enabled | default('present') }}"
state: "{{ item.state | default('present') }}"
with_items: "{{ ceph_repos }}"
register: install_repo
until: install_repo is success

4
vars/redhat.yml
View File

@ -47,7 +47,7 @@ _ceph_repos:
- name: ceph
description: "Ceph packages for $basearch"
file: ceph
baseurl: "{{ ceph_yum_repo_url }}/el$releasever/$basearch"
baseurl: "{{ ceph_repo_url }}/el$releasever/$basearch"
gpgcheck: yes
enabled: yes
priority: 50
@ -55,7 +55,7 @@ _ceph_repos:
- name: ceph-noarch
description: "Ceph noarch packages"
file: ceph
baseurl: "{{ ceph_yum_repo_url }}/el$releasever/noarch"
baseurl: "{{ ceph_repo_url }}/el$releasever/noarch"
gpgcheck: yes
enabled: yes
priority: 50