Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Commit

Permalink
Use Ceph cluster name when setting minimum client version
Browse files Browse the repository at this point in the history
If CephClusterName is set to it's non-default value, then
I4c3d8fee6c7a0c25cca4d1e6b731060684cd28b3 will cause the
deployment to fail as per the bug this patch closes.

Change-Id: Ia5611d7cb64808ac247873023e5aa5afd07f6698
Closes-Bug: #1915197
(cherry picked from commit 8149df4)
  • Loading branch information
fultonj authored and gfidente committed Feb 10, 2021
1 parent 0efeb96 commit 5e4d71b
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion deployment/ceph-ansible/ceph-mon.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,15 @@ parameters:
type: boolean
default: false
description: Parameter used to trigger the dashboard deployment.
CephClusterName:
type: string
default: ceph
description: The Ceph cluster name.
constraints:
- allowed_pattern: "[a-zA-Z0-9]+"
description: >
The Ceph cluster name must be at least 1 character and contain only
letters and numbers.
conditions:
dashboard_enabled: {equals: [{get_param: CephEnableDashboard}, true]}
Expand Down Expand Up @@ -123,7 +132,9 @@ outputs:
register: ceph_mon_id
delegate_to: "{{ groups['ceph_mon'][0] }}"
- name: Enforce minimum Ceph clients version to Mimic
command: "{{ container_cli }} exec {{ ceph_mon_id.stdout_lines[0] }} ceph osd set-require-min-compat-client mimic"
command: "{{ container_cli }} exec {{ ceph_mon_id.stdout_lines[0] }} ceph --cluster {{ cluster }} osd set-require-min-compat-client mimic"
delegate_to: "{{ groups['ceph_mon'][0] }}"
vars:
cluster: {get_param: CephClusterName}
external_update_tasks: {get_attr: [CephBase, role_data, external_update_tasks]}
external_upgrade_tasks: {get_attr: [CephBase, role_data, external_upgrade_tasks]}

0 comments on commit 5e4d71b

Please sign in to comment.