OSD data dir expected in /var/lib/ceph/osd

Bug #1589922 reported by Erno Kuvaja
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Erno Kuvaja

Bug Description

Ceph expects all OSDs to be in /var/lib/ceph/osd we currently overwrite this with /srv/data.

Ceph is able to deal with this by creating symbolic links to /var/lib/ceph/osd but there is really no reason to change the default. The current specified directory is also really generic by the name and does not give admin direct indication what it is used for.

As the symbolic links are created on the nodes, changing this to the default /var/lib/ceph/osd should not affect the upgrade.

Erno Kuvaja (jokke)
Changed in tripleo:
assignee: nobody → Erno Kuvaja (jokke)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

Fix proposed to branch: master
Review: https://review.openstack.org/326376

Changed in tripleo:
status: New → In Progress
Steven Hardy (shardy)
Changed in tripleo:
milestone: none → newton-2
importance: Undecided → High
Revision history for this message
Giulio Fidente (gfidente) wrote :

currently we have in /var/lib/ceph/osd a number of symlinks (eg /var/lib/ceph/osd/ceph-0) for each dir-based osd path

is this really an issue?

Revision history for this message
Steven Hardy (shardy) wrote :

There are questions around the upgrade impact of this change, deferring to n-3 and we can discuss either here or on the review.

Changed in tripleo:
milestone: newton-2 → newton-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (master)

Change abandoned by Erno Kuvaja (<email address hidden>) on branch: master
Review: https://review.openstack.org/326376
Reason: Too risky

Revision history for this message
Giulio Fidente (gfidente) wrote :

I think the only problem with this is the SELinux label applied on /srv/data.

In theory we want puppet-ceph to label the data path correctly as we don't know in advance which one it will be. This is tracked via https://bugs.launchpad.net/puppet-ceph/+bug/1611893 .

In addition to that, we're trying to add in ceph-selinux /srv/data as a default path so labeling won't be needed. Swift does the same with /srv/node path.

I think we can "close" this as soon as one the above is resolved.

Revision history for this message
Emilien Macchi (emilienm) wrote :
Changed in tripleo:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.