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

Bug #1444557 reported by Mark Vanderwiel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Fix Released
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"

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cookbook-openstack-object-storage (master)

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)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-object-storage (master)

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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