Monday, 2020-01-27

*** hoonetorg has quit IRC01:38
*** hoonetorg has joined #openstack-swift01:51
*** openstackgerrit has quit IRC02:04
*** openstackgerrit has joined #openstack-swift02:08
openstackgerritThiago da Silva proposed openstack/swift master: s3api: Implement object versioning API  https://review.opendev.org/67368202:08
*** psachin has joined #openstack-swift03:35
*** tkajinam has quit IRC07:02
*** tkajinam has joined #openstack-swift07:04
openstackgerritMerged openstack/swift master: container-sync: Sync static links similar to how we sync SLOs  https://review.opendev.org/69813907:24
*** tkajinam_ has joined #openstack-swift07:52
*** tkajinam has quit IRC07:55
*** rdejoux has joined #openstack-swift08:13
*** tesseract has joined #openstack-swift08:13
*** cjloader has quit IRC08:16
*** tkajinam_ has quit IRC08:18
*** rpittau|afk is now known as rpittau08:18
*** pcaruana has joined #openstack-swift08:25
openstackgerritMerged openstack/swift master: New Object Versioning mode  https://review.opendev.org/68238209:16
*** ccamacho has joined #openstack-swift09:25
*** rpittau is now known as rpittau|bbl11:21
*** tomha has joined #openstack-swift12:07
*** rpittau|bbl is now known as rpittau12:55
*** psachin has quit IRC13:29
*** jvisser has joined #openstack-swift14:41
*** paladox is now known as paladox|UKInEU14:56
*** cjloader has joined #openstack-swift15:08
*** theintern_ has joined #openstack-swift15:50
*** theintern_ has quit IRC16:07
*** theintern_ has joined #openstack-swift16:19
*** gyee has joined #openstack-swift16:29
*** zaitcev has joined #openstack-swift16:29
*** ChanServ sets mode: +v zaitcev16:29
*** ccamacho has quit IRC16:34
*** theintern_ has quit IRC16:37
*** theintern_ has joined #openstack-swift16:49
openstackgerritTim Burke proposed openstack/swift master: Add unittest2 back to lower-constraints.txt  https://review.opendev.org/70435616:50
timburkethanks for fixing up p 673682, tdasilva!16:51
patchbothttps://review.opendev.org/#/c/673682/ - swift - s3api: Implement object versioning API - 51 patch sets16:51
*** rdejoux has quit IRC16:59
timburkehmm... based on my read of the code, i'm pretty sure https://github.com/openstack/swift/blob/2.23.0/swift/common/middleware/s3api/s3request.py#L535-L536 are currently wsgi strings... should probably be native, though...17:05
*** paladox|UKInEU is now known as paladox17:09
*** gregwork has joined #openstack-swift17:15
*** theintern_ has quit IRC17:23
*** tesseract has quit IRC17:38
*** rpittau is now known as rpittau|afk18:18
*** gregwork has quit IRC19:25
openstackgerritTim Burke proposed openstack/swift master: s3api: use native strings in s3api.auth_details access_key  https://review.opendev.org/70441320:13
*** gyee has quit IRC20:27
*** gyee has joined #openstack-swift20:28
*** openstackstatus has joined #openstack-swift20:28
*** ChanServ sets mode: +v openstackstatus20:28
openstackgerritTim Burke proposed openstack/swift master: Middleware that allows a user to have quoted Etags  https://review.opendev.org/70005620:53
openstackgerritRomain LE DISEZ proposed openstack/swift master: proxy-logging: add fields ttfb and pid  https://review.opendev.org/70442420:56
openstackgerritRomain LE DISEZ proposed openstack/swift master: SSYNC: log body of errors in SSYNC subrequests  https://review.opendev.org/70443021:14
openstackgerritRomain LE DISEZ proposed openstack/swift master: Mark a container reported if account was reclaimed  https://review.opendev.org/70443521:29
claygrledisez: with the patches! 👍21:32
rledisezclayg: just going through the patches we run in production but that are not proposed yet21:34
rledisezreally small stuff, but useful I think21:34
claygundoubtly!!!21:42
timburkelove 'em!21:57
timburkei'm a little nervous about the possibility of containers *with data* in accounts that have been reclaimed though...21:58
timburkeidk that there's an obvious right thing to do in that case though...21:59
openstackgerritTim Burke proposed openstack/swift master: s3api: use native strings in s3api.auth_details access_key  https://review.opendev.org/70441322:01
rlediseztimburke: actually it does not delete the data, just the container db (as the account db was already deleted). but the datafile stays there (I admit it might be harder to find the objects)22:02
timburkerledisez, but if the container db has objects in it, it won't get reclaimed, right? so you've not only got the dark .data files lurking on your object nodes but also these dark .db files...22:06
timburkebut since the account db's *actually* gone, there's nothing to signal that any of this should've been deleted :-/22:07
timburkelike, the 500 in account/server.py's no good. but there are two "reasonable" things we could do to fix it -- 404 (as in this patch) or pop the db into existence (like we would if it started with auto_account_prefix)22:08
timburkeer, auto_create_account_prefix -- you know what i mean22:09
*** rcernin has joined #openstack-swift22:10
mattoliveraumorning22:16
timburkemaybe it'd make sense to quarantine if we've done all those PUTs, nobody has an account DB, and the container db is not marked deleted?22:37
timburkeidk22:37
*** diablo_rojo has quit IRC22:42
rledisezi think the right thing to do is to delete these objects. dark data is bad, we should fight it :)22:53
rledisezwhat about re-creating the account in a deleted state so that the reaper will get another chance to clean it?22:53
*** tkajinam has joined #openstack-swift22:55
timburkeseems reasonable -- if we do it in the account-server as part of the container-update flow, we'll want to be careful about choosing timestamps in case one of the other replicas has a db23:19
tdasilvarledisez, timburke: was wondering if it would make sense to add a "deprecation flag" to /info requests. I'm trying to think of effective ways of communicating deprecation plans to API users and operators and was thinking that highlighting current deprecations (or plans) for a given option could be a good way23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!