swift-proxy charm fails to add multiple devices per node

Bug #1479938 reported by JuanJo Ciarlante
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
swift-proxy (Juju Charms Collection)
Fix Released
High
JuanJo Ciarlante

Bug Description

charm: swift-proxy trunk r97
deploy: swift-proxy related to swift-storage-z{1,2,3}, with block-device="/dev/sdc /dev/sdd" each.
result: only sdc showing in the ring
expected: sdc and sdd showing in the ring.

Debugging details: swift-ring-builder output, juju setting and juju relation values:
http://paste.ubuntu.com/11968692/

Related branches

Revision history for this message
JuanJo Ciarlante (jjo) wrote :

AFAICS exists_in_ring(ring_path, node) doesn't take
into account the physical device that's being added,
thus skipping all additional devices from existing node(s).

Revision history for this message
JuanJo Ciarlante (jjo) wrote :

FYI deploying with same configs, with changes from
lp:~jjo/charms/trusty/swift-proxy/fix-multiple-devices-per-node_lp1479938
I now properly get sdc *and* sdd added to the ring:
http://paste.ubuntu.com/11969096/

Revision history for this message
David Ames (thedac) wrote :

Confirmed with revno 100 lp:trusty/swift-proxy

Changed in swift-proxy (Juju Charms Collection):
status: New → Confirmed
James Page (james-page)
Changed in swift-proxy (Juju Charms Collection):
importance: Undecided → High
milestone: none → 15.10
Changed in swift-proxy (Juju Charms Collection):
status: Confirmed → In Progress
David Britton (dpb)
Changed in swift-proxy (Juju Charms Collection):
assignee: nobody → JuanJo Ciarlante (jjo)
David Ames (thedac)
Changed in swift-proxy (Juju Charms Collection):
status: In Progress → Fix Committed
James Page (james-page)
Changed in swift-proxy (Juju Charms Collection):
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.