| opendevreview | Jianjian Huo proposed openstack/swift master: Container-server: add container full namespaces GET https://review.opendev.org/c/openstack/swift/+/890470 | 01:23 |
|---|---|---|
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: wsgi: bad request syntax response missing txn-id https://review.opendev.org/c/openstack/swift/+/887904 | 03:50 |
| opendevreview | Jianjian Huo proposed openstack/swift master: Container-server: add container full namespaces GET https://review.opendev.org/c/openstack/swift/+/890470 | 04:19 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber https://review.opendev.org/c/openstack/swift/+/891009 | 04:21 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber https://review.opendev.org/c/openstack/swift/+/891009 | 04:21 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: s3api: Support GET/HEAD request with PartNumber https://review.opendev.org/c/openstack/swift/+/891009 | 04:22 |
| opendevreview | Alistair Coles proposed openstack/swift master: s3api: set swift.backend_path when returning 422 https://review.opendev.org/c/openstack/swift/+/891088 | 17:57 |
| reid_g | Hello, which proc is actually responsible for deleting container db files when an account is delete? The doc mentions db_replicator but is that the container-replicator? | 18:50 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: wsgi: bad request syntax response missing txn-id https://review.opendev.org/c/openstack/swift/+/887904 | 18:53 |
| reid_g | Also what does swift-container-info mean when it says Delete:No but has Delete Timestamp: 2023-06-05T22:44:56.660350 (1686005096.66035) and Reported Delete Timestamp: 2023-06-05T22:44:56.660350 (1686005096.66035) | 18:54 |
| timburke | reid_g, yeah, the account and container replicators both subclass from https://github.com/openstack/swift/blob/master/swift/common/db_replicator.py | 18:58 |
| timburke | so it'd be a container-replicator process that eventually runs https://github.com/openstack/swift/blob/master/swift/common/db_replicator.py#L637-L639 to unlink off disk | 18:58 |
| timburke | that'll only happen after the container DB has been marked deleted via a DELETE request; if the whole account has been marked for deletion, that'd be coming from the account-reaper | 18:59 |
| timburke | the delay (called reclaim_age) is to make sure any objects that might have been written have had a chance to get reported to the container so you don't end up with dark data that ties up space in your cluster but users can't actually find | 19:01 |
| timburke | as for the swift-container-info output -- is there a put timestamp that's after the delete timestamp? are there still object rows in the DB? | 19:02 |
| reid_g | it shows Reported Object Count: 3 | 19:06 |
| timburke | that's basically the thrust of it in https://github.com/openstack/swift/blob/master/swift/container/backend.py#L827-L838 -- there might be some added nuance for sharded containers | 19:06 |
| reid_g | I went and found the objects and manually deleted them. Will the container still be updated? | 19:06 |
| timburke | "manually deleted them" like ssh to the server and rm, or like issue a DELETE (either directly to the object-server or via a proxy/internal-client)? | 19:08 |
| reid_g | went to server and rm them | 19:08 |
| timburke | then no -- but you ought to be able to similarly rm the db files | 19:09 |
| reid_g | Should I issue deletes via api? Will that work if the account is gone already? (this is going back to the questions last week where I have some containers that aren't being cleaned up) | 19:11 |
| opendevreview | ASHWIN A NAIR proposed openstack/swift master: unit tests: improve FakeSwift handling of HEAD with query param https://review.opendev.org/c/openstack/swift/+/889785 | 20:04 |
| opendevreview | Merged openstack/swift master: py3: apply str_to_wsgi to includes in _get_shard_ranges https://review.opendev.org/c/openstack/swift/+/889113 | 22:59 |
| opendevreview | Merged openstack/swift master: sharder: use ContainerBroker.has_other_shard_ranges() method https://review.opendev.org/c/openstack/swift/+/889042 | 22:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!