http_timeout config option coupling

Bug #1686280 reported by clayg on 2017-04-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Low
Rajat Sharma

Bug Description

The reconstructor uses the http_timeout config option for both for getting REPLICATE responses [1] and also in SSYNC protocol state changes [2].

It would be better if they were independently tunable.

maybe

self.replicate_http_timeout = conf.get('replicate_http_timeout', conf.get('http_timeout'))
self.ssync_http_timeout = conf.get('ssync_http_timeout', conf.get('http_timeout'))

sorta preference backwards compat thing going on? The replicator probably has the same issue.

1. https://github.com/openstack/swift/blob/da9cd166099a1f29b63e681a68de6da84eed679c/swift/obj/reconstructor.py#L545
2. https://github.com/openstack/swift/blob/da9cd166099a1f29b63e681a68de6da84eed679c/swift/obj/ssync_sender.py#L366

Rajat Sharma (tajar29) on 2017-10-26
Changed in swift:
assignee: nobody → Rajat Sharma (tajar29)
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers