Cannot specify replicaton network in swift-object-device

[Mark Kirkwood]

There appears to be no way to separate storage and replication networks in swift-object-device and friends.

My naive attempt was this:

class profiles::swift::stor {

  $storage_ip = hiera('profiles::swift::storage_ip')
  $replication_ip = hiera('profiles::swift::replication_ip')

  @@ring_object_device { "${storage_ip}:6000R${replication_ip}:6010/${name}":
      region => hiera('profiles::swift::region', 1),
      zone => hiera('profiles::swift::zone', 1),
      weight => hiera('profiles::swift::weight', 1),
  }

...while this sets up the two networks ok in the ring, it confuses the provider utterly and it tries to add the device repeatedly even if it is already there (I'm guessing - not a ruby pro - that it is comparing the storage part of the ring builder output to the entire passed in storage + replication string). If I've understood the problem properly this might be easy to correct!

[Mark Kirkwood (mark-kirkwood)]

I decided it was time to learn a little ruby. I think pull request https://github.com/openstack/puppet-swift/pull/7 fixes the issue by amending the validators and provider for ring_xxx_device.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Alex Schultz (<email address hidden>) on branch: master
Review: https://review.openstack.org/302483
Reason: Since this is in merge conflict and hasn't been touched in over a year, abandoning per our policy. Abandoning per our policy https://docs.openstack.org/puppet-openstack-guide/latest/contributor/reviews.html#abandonment