From: Andrew Schoen Date: Wed, 30 Aug 2017 20:55:16 +0000 (-0500) Subject: ceph-osd: add support for --journal vg/lv for lvm osds X-Git-Tag: v3.0.0rc5~3^2~4 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=fcba9d17f003182294d0ab9804594ef2cd2a6b42;p=ceph-ansible.git ceph-osd: add support for --journal vg/lv for lvm osds This also updates the tests Signed-off-by: Andrew Schoen --- diff --git a/roles/ceph-osd/tasks/scenarios/lvm.yml b/roles/ceph-osd/tasks/scenarios/lvm.yml index a1c5994c9..bd2936a2f 100644 --- a/roles/ceph-osd/tasks/scenarios/lvm.yml +++ b/roles/ceph-osd/tasks/scenarios/lvm.yml @@ -2,3 +2,11 @@ - name: use ceph-volume to create filestore osds with dedicated journals command: "ceph-volume lvm create --filestore --data {{ item.data_vg }}/{{ item.data }} --journal {{ item.journal }}" with_items: "{{ lvm_volumes }}" + when: + - item.journal_vg is not defined + +- name: use ceph-volume to create filestore osds with dedicated lv journals + command: "ceph-volume lvm create --filestore --data {{ item.data_vg }}/{{ item.data }} --journal {{item.journal_vg }}/{{ item.journal }}" + with_items: "{{ lvm_volumes }}" + when: + - item.journal_vg is defined diff --git a/tests/functional/centos/7/lvm-osds/group_vars/all b/tests/functional/centos/7/lvm-osds/group_vars/all index 799b778da..2bcf30c1f 100644 --- a/tests/functional/centos/7/lvm-osds/group_vars/all +++ b/tests/functional/centos/7/lvm-osds/group_vars/all @@ -1,6 +1,7 @@ --- -ceph_stable: True +ceph_origin: repository +ceph_repository: community cluster: ceph public_network: "192.168.3.0/24" cluster_network: "192.168.4.0/24" @@ -12,9 +13,13 @@ osd_scenario: lvm copy_admin_key: true # test-volume is created by tests/functional/lvm_setup.yml from /dev/sdb lvm_volumes: - - data: test_volume - journal: /dev/sdc + - data: data-lv1 + journal: /dev/sdc1 data_vg: test_group + - data: data-lv2 + journal: journal1 + data_vg: test_group + journal_vg: journals os_tuning_params: - { name: kernel.pid_max, value: 4194303 } - { name: fs.file-max, value: 26234859 }