Monday, 2021-07-12

opendevreviewMerged openstack/swift master: Sharder: Add root_audit overlap stats  https://review.opendev.org/c/openstack/swift/+/79899409:47
opendevreviewTim Burke proposed openstack/swift master: squash: reconciler: PPI aware reconciler  https://review.opendev.org/c/openstack/swift/+/80049415:00
opendevreviewDouglas Mendizábal proposed openstack/swift master: WIP/DNM: Add FIPS CI jobs  https://review.opendev.org/c/openstack/swift/+/79605717:52
zaitcevtimburke: I'm looking for reviews on patch 743797, what do you think would be a good trade for me to review? What's hot? Down the list at https://wiki.openstack.org/wiki/Swift/PriorityReviews ?18:14
timburkezaitcev, i wouldn't mind somebody taking a look at https://review.opendev.org/c/openstack/swift/+/792490 -- i think it slipped off mattoliver's radar ;-)19:29
zaitcevtimburke: ok19:31
timburkezaitcev, on your patch, am i right to think that we're going to have a ton of not just dark data (objects), but dark databases as well?19:32
zaitcevtimburke: there are going to be some, although I am not sure it's a ton of them. These containers are gnocchi sacks.19:33
zaitcevtimburke: Remember Romain's patch that tried to re-attach them somewhere? It should not leave any laying around, if it succeeds of course. But it was hard to make guarantees.19:34
zaitcevtimburke: so my official course is to say "screw this, let's bulletproof account server" and then rely on watchers to keep dark objects in check.19:35
timburkehow much of this could be avoided if we just had auditors quarantine data when the hash dir doesn't match the expected hash given the path from metadata?19:54
timburkethe 500 in the account server is definitely still worth addressing -- i'm just thinking that the updater change may not be necessary if we quarantined the DBs19:56
zaitcevI'm reading the CORS docs and I'm remembering it all.21:14

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!