storage_domain is inconsistent between cname_lookup and domain_remap

Bug #1664647 reported by Romain LE DISEZ
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Low
Unassigned

Bug Description

On middleware cname_lookup, storage_domain is a list of valid domains for the Swift cluster.

On middleware domain_remap, storage_domain is a single valid domain for the Swift cluster.

It seems inconsistent. If an operator is using multiple domains for cname_lookup, it is highly probable that it will want to use multiple domains with domain_remap.

domain_remap should accept a list of domains in storage_domain parameter.

Revision history for this message
Romain LE DISEZ (rledisez) wrote :
Changed in swift:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to swift (master)

Reviewed: https://review.openstack.org/435767
Committed: https://git.openstack.org/cgit/openstack/swift/commit/?id=5c93d6f238ce2614cc7b058e584e2273c2079603
Submitter: Jenkins
Branch: master

commit 5c93d6f238ce2614cc7b058e584e2273c2079603
Author: Romain LE DISEZ <email address hidden>
Date: Sat Feb 18 10:19:20 2017 +0100

    Accept storage_domain as a list in domain_remap

    Middleware domain_remap can work with cname_lookup middleware. This last
    middleware accept that storage_domain is a list of domains. To be
    consistent, domain_remap should have the same behavior.

    Closes-Bug: #1664647

    Change-Id: Iacc6619968cc7c677bf63e0b8d101a20c86ce599

Changed in swift:
status: Confirmed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/swift 2.14.0

This issue was fixed in the openstack/swift 2.14.0 release.

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.