[object-storage] ring_script provider parse_ring_output method out of date

Bug #1444557 reported by Mark Vanderwiel on 2015-04-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Medium
Mark Vanderwiel

Bug Description

Currrent output from swift-ring-builder:

[root@swift168 rings]# su swift -c 'swift-ring-builder /etc/swift/ring-workspace/rings/account.builder'
/etc/swift/ring-workspace/rings/account.builder, build version 0
262144 partitions, 3.000000 replicas, 0 regions, 0 zones, 0 devices, 0.00 balance, 0.00 dispersion
The minimum number of hours before a partition can be reassigned is 1
The overload factor is 0.00% (0.000000)

The method parse_ring_output does not handle the now optional "0.00 dispersion" and does not handle the "overload factor"

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

Changed in openstack-chef:
status: New → In Progress
Changed in openstack-chef:
importance: Undecided → Medium
Changed in openstack-chef:
assignee: Mark Vanderwiel (vanderwl) → Ma Wen Cheng (mars914)
Changed in openstack-chef:
assignee: Ma Wen Cheng (mars914) → Mark Vanderwiel (vanderwl)

Reviewed: https://review.openstack.org/174012
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-object-storage/commit/?id=4f778233cffafdc839697b713aaae0dd18cc5cc8
Submitter: Jenkins
Branch: master

commit 4f778233cffafdc839697b713aaae0dd18cc5cc8
Author: Mark Vanderwiel <email address hidden>
Date: Wed Apr 15 12:59:24 2015 -0500

    Fix ring data parser to allow for optional dispersion field

    Swift was changed to add an option dispersion field to the output
    of the swift-ring-builder command. Also a line was added for the
    overload factor, need to handle that as well.
    Added a new spec for testing provider methods.

    Change-Id: I1ae650e1514d69dc9c26b6054837f00fb4eb7bcb
    Closes-Bug: #1444557

Changed in openstack-chef:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers