seongsoocho | Hi All. I'm using object versioning and container sync at the same time. And when the object has a delete mark in its metadata, the container-sync failed because the remote proxy server response 400 bad request and retry forever.. ( delete marker : Content-Type: application/x-deleted;swift_versions_deleted=1) My guess is that this code returns 400. | 01:01 |
---|---|---|
seongsoocho | https://github.com/openstack/swift/blob/master/swift/proxy/controllers/obj.py#L85. It is a kind of bug? | 01:01 |
timburke__ | seongsoocho, yeah, sounds like a bug :-( sorry | 01:04 |
timburke__ | fix is most likely going to be in https://github.com/openstack/swift/blob/master/swift/common/middleware/container_sync.py | 01:05 |
seongsoocho | timburke__: Ok. then I will report this our launchpad and try to fix it | 01:05 |
timburke__ | oh -- or maybe in https://github.com/openstack/swift/blob/master/swift/container/sync.py to translate that to a DELETE instead of a PUT... | 01:09 |
timburke__ | been a bit since i thought about container sync | 01:09 |
timburke__ | like https://github.com/openstack/swift/blob/master/swift/container/sync.py#L535 maybe should just be `if row['deleted'] or row['content_type'] == DELETE_MARKER_CONTENT_TYPE:` | 01:11 |
timburke__ | great bug to have a probe test for | 01:11 |
seongsoocho | oh. your suggestion is more better. | 01:12 |
opendevreview | Alistair Coles proposed openstack/swift master: s3api: add API support for optional bucket inventory https://review.opendev.org/c/openstack/swift/+/835938 | 17:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!