Skip to content

Commit

Permalink
ceph-mon: fix admin keyring creation task
Browse files Browse the repository at this point in the history
Although custom cluster name support was dropped, it breaks ceph-volume
functional testing as it uses "test" as cluster name.
The plan is to make ceph-volume use "ceph" but for now it's easier to
address the issue in this task.

Signed-off-by: Guillaume Abrioux <[email protected]>
  • Loading branch information
guits committed Apr 4, 2024
1 parent 8ebff21 commit c4b3520
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions roles/ceph-mon/tasks/deploy_monitors.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
- name: Create admin keyring
ceph_authtool:
name: client.admin
path: /etc/ceph/ceph.client.admin.keyring
path: "/etc/ceph/{{ cluster }}.client.admin.keyring"
owner: "{{ ceph_uid if containerized_deployment | bool else 'ceph' }}"
group: "{{ ceph_uid if containerized_deployment | bool else 'ceph' }}"
mode: "0400"
Expand Down Expand Up @@ -138,7 +138,7 @@
owner: "{{ ceph_uid if containerized_deployment | bool else 'ceph' }}"
group: "{{ ceph_uid if containerized_deployment | bool else 'ceph' }}"
mode: "0400"
import_keyring: /etc/ceph/ceph.client.admin.keyring
import_keyring: "/etc/ceph/{{ cluster }}.client.admin.keyring"
environment:
CEPH_CONTAINER_IMAGE: "{{ ceph_docker_registry + '/' + ceph_docker_image + ':' + ceph_docker_image_tag if containerized_deployment | bool else None }}"
CEPH_CONTAINER_BINARY: "{{ container_binary }}"
Expand Down

0 comments on commit c4b3520

Please sign in to comment.