ceph
/
ceph-ansible
mirror of https://github.com/ceph/ceph-ansible.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

osd: do not enable osd@id unit file 

ceph-disk is responsable for enabling the unit file if needed. Actually
since https://github.com/ceph/ceph/pull/12241 it seems that it's not
even needed. On an event of a restart, udev rules will be trigger and
they will ceph-disk activate the device too so the 'enabled' is not
needed.

Closes: https://github.com/ceph/ceph-ansible/issues/1142
Signed-off-by: Sébastien Han <seb@redhat.com>
pull/1715/head
Sébastien Han 2017-07-26 16:46:57 +02:00
parent fad9d0caec
commit 8ac7d2e4c9
2 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
roles/ceph-osd/tasks/activate_osds.yml
View File

@ -95,6 +95,5 @@
service: service:
name: ceph-osd@{{ item }} name: ceph-osd@{{ item }}
state: started state: started
enabled: yes
with_items: "{{ (osd_id|default({})).stdout_lines|default([]) }}" with_items: "{{ (osd_id|default({})).stdout_lines|default([]) }}"
changed_when: false changed_when: false

5
tests/functional/tests/osd/test_osds.py
View File

@ -22,11 +22,6 @@ class TestOSDs(object):
for osd in node["osds"]: for osd in node["osds"]:
assert Service("ceph-osd@%s" % osd).is_running assert Service("ceph-osd@%s" % osd).is_running
def test_osd_services_are_enabled(self, node, Service):
# TODO: figure out way to paramaterize node['osds'] for this test
for osd in node["osds"]:
assert Service("ceph-osd@%s" % osd).is_enabled
@pytest.mark.no_docker @pytest.mark.no_docker
def test_osd_are_mounted(self, node, MountPoint): def test_osd_are_mounted(self, node, MountPoint):
# TODO: figure out way to paramaterize node['osd_ids'] for this test # TODO: figure out way to paramaterize node['osd_ids'] for this test