Ceph OSD journal partitions with number above 9 are not allocated to OSDs

Bug #1339833 reported by Dmitry Borodaenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Dmitry Borodaenko
5.0.x
Fix Released
Medium
Dmitry Borodaenko

Bug Description

If there's more than 9 partitions on the Ceph OSD Journal device, double-digit partitions (10+) will not be allocated to OSDs.

The problem is in this line of ceph_osd Facter snippet:
https://github.com/stackforge/fuel-library/blob/master/deployment/puppet/ceph/lib/facter/ceph_osd.rb#L11

"ls /dev/#{d}?" should be "ls /dev/#{d}?*"

Changed in fuel:
importance: Undecided → Critical
importance: Critical → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Meg McRoberts (dreidellhasa) wrote :

Documented as "Fixed in 5.0.1" in 5.0.1 Release Notes.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/106444

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/105836
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=41e2659661d82887442ba1b392a8bc3af93589c3
Submitter: Jenkins
Branch: master

commit 41e2659661d82887442ba1b392a8bc3af93589c3
Author: Dmitry Borodaenko <email address hidden>
Date: Wed Jul 9 11:24:16 2014 -0700

    Enumerate Ceph OSD partitions with multi-digit index

    When there's more than 6 OSDs per OSD Journal device, some OSD journal
    partitions end up having double-digit partition index number. This patch
    modifies the ls command used to enumerate journal partitions to account for
    that multi-digit partition index numbers.

    Change-Id: I654b5c070324887f9ffd41e6eb5e231851b97404
    Closes-Bug: #1339833

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.0)

Reviewed: https://review.openstack.org/106444
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=9aff1a68740a42363dc912d43b99a378fbe98408
Submitter: Jenkins
Branch: stable/5.0

commit 9aff1a68740a42363dc912d43b99a378fbe98408
Author: Dmitry Borodaenko <email address hidden>
Date: Wed Jul 9 11:24:16 2014 -0700

    Enumerate Ceph OSD partitions with multi-digit index

    When there's more than 6 OSDs per OSD Journal device, some OSD journal
    partitions end up having double-digit partition index number. This patch
    modifies the ls command used to enumerate journal partitions to account for
    that multi-digit partition index numbers.

    Change-Id: I654b5c070324887f9ffd41e6eb5e231851b97404
    Closes-Bug: #1339833

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/5.1.x
Revision history for this message
Dennis Dmitriev (ddmitriev) wrote :

{"build_id": "2014-09-17_21-40-34", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "11", "auth_required": true, "api": "1.0", "nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d", "production": "docker", "fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd", "astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13", "feature_groups": ["mirantis"], "release": "5.1", "release_versions": {"2014.1.1-5.1": {"VERSION": {"build_id": "2014-09-17_21-40-34", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "11", "api": "1.0", "nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d", "production": "docker", "fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd", "astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13", "feature_groups": ["mirantis"], "release": "5.1", "fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"}}}, "fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"}

Changed in fuel:
status: Fix Committed → 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.