opendevreview | Takashi Kajinami proposed openstack/swift master: Add missing services to sample rsyslog.conf https://review.opendev.org/c/openstack/swift/+/841673 | 02:48 |
---|---|---|
opendevreview | Tim Burke proposed openstack/swift master: Add --test-config option to WSGI servers https://review.opendev.org/c/openstack/swift/+/833124 | 03:57 |
opendevreview | Tim Burke proposed openstack/swift master: Add a swift-reload command https://review.opendev.org/c/openstack/swift/+/833174 | 03:57 |
opendevreview | Tim Burke proposed openstack/swift master: systemd: Send STOPPING/RELOADING notifications https://review.opendev.org/c/openstack/swift/+/837633 | 03:57 |
opendevreview | Tim Burke proposed openstack/swift master: Add abstract sockets for process notifications https://review.opendev.org/c/openstack/swift/+/837641 | 03:57 |
opendevreview | Alistair Coles proposed openstack/swift master: sharder: ensure that misplaced tombstone rows are moved https://review.opendev.org/c/openstack/swift/+/841612 | 07:38 |
acoles | mattoliver: ^^^ do you have any recollection if there was a reason to process misplaced object rows in order undeleted followed by deleted? | 07:42 |
mattoliver | I was going to say, in case a delete was issued after a put, but the delete hits first. But that's only an issue if a delete isn't logged if the object isn't already there? Surely it is. Maybe it was to make sure objects that were there got to its destination first, to minimise missing objects from listing. | 08:19 |
mattoliver | Maybe we should treat it more like a journal, and move objects in row order (include deleted) from the start. | 08:20 |
mattoliver | If there were alot of deletes, like in an expired objects container then maybe so it looks like it makes progress if you move the deleted=0 first 🤷♂️ | 08:21 |
afaranha_ | mattoliver, hey, I see that the tempurl patches are in a good shape, thanks for that https://review.opendev.org/c/openstack/swift/+/525771 are we currently just waiting for reviews? | 08:51 |
afaranha_ | do you need any help with the patches? | 08:51 |
mattoliver | Yeah, I'll double back to them and check, but yeah it's waiting on reviews I believe. I'll poke people about it in the next meeting if it isn't reviewed by then. | 08:57 |
mattoliver | If you want to double check and review them too (if you haven't already) that'll be great! (On my phone so don't have them handy to check). | 08:57 |
afaranha_ | thanks, I'll do that, and ask my colleague | 08:58 |
opendevreview | Merged openstack/swift master: memcached: Give callers the option to accept errors https://review.opendev.org/c/openstack/swift/+/839448 | 16:44 |
opendevreview | Tim Burke proposed openstack/swift master: container: Add delimiter-depth query param https://review.opendev.org/c/openstack/swift/+/829605 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!