21:00:19 #startmeeting swift 21:00:20 Meeting started Wed Oct 3 21:00:19 2018 UTC and is due to finish in 60 minutes. The chair is notmyname. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:00:21 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:00:23 The meeting name has been set to 'swift' 21:00:24 who's here for the swift team meeting? 21:00:26 o/ 21:00:33 o/ 21:00:37 o/ 21:00:51 o/ 21:00:57 welcome 21:01:03 hi 21:01:08 #link https://wiki.openstack.org/wiki/Meetings/Swift 21:01:27 hi o/ 21:01:32 again, not a super full agenda, so let's see if it's another short one :-) 21:02:07 in lieu of any specific topics, let's look at the priority reviews page 21:02:08 #link https://wiki.openstack.org/wiki/Swift/PriorityReviews 21:02:33 #topic priority reviews 21:03:05 mattoliverau: you've been reviewing the keymaster patches, I think 21:03:15 I have 21:03:45 anything to report or anything to bring up with those? 21:04:11 There looking good. A little more work, doc and tests, but otherwise should land soon 21:04:17 cool 21:05:19 m_kazuhiro or rledisez: any updates for task queue work? 21:05:34 (sorry, got distracted from the keyboard for a moment) 21:05:49 nothing, in this week. 21:05:54 I don't see any reviews on https://review.openstack.org/#/c/601950/ 21:05:55 patch 601950 - swift - Enable to configure object-expirer in object-serve... - 4 patch sets 21:06:10 rledisez: are you (or other ovh) planning on reviewing that patch? 21:07:33 notmyname: i'm planning to review it 21:07:43 great :-) 21:07:43 thanks 21:08:24 there are 6 open s3api patches from timburke. these are all patches we were carrying internally for swift3 (I think many/all had been proposed, just not landed) 21:08:43 tdasilva: were you wanting someone else to review https://review.openstack.org/#/c/575829/ ? 21:08:44 patch 575829 - swift - Change PUT bucket conflict error - 8 patch sets 21:08:55 same question to zaitcev on https://review.openstack.org/#/c/604208/ 21:08:55 patch 604208 - swift - s3api: Increase max body size for Delete Multiple ... - 3 patch sets 21:09:28 What was the question? 21:09:58 zaitcev: ah, sorry. you just came in. I was asking if there's something else you're waiting for on https://review.openstack.org/#/c/604208/ 21:09:59 patch 604208 - swift - s3api: Increase max body size for Delete Multiple ... - 3 patch sets 21:10:03 i have a little nit comment with my +2 and was kind of waiting from tburke's feedback to see if he just preferred that we merged as is or if he wanted to update 21:11:04 tdasilva: ok. I'll let you and timburke work it out then :-) 21:11:29 yeah, i was torn about breaking it out as a separate test entirely -- i *did* intentionally push the maybe-skipped parts of the test to the end, so if an earlier part *fails* you'd still know... 21:11:34 zaitcev: if you're not looking for more changes (you added a +2 already) or if you want someone else to look, that's fine. just trying to see if there's something that can land or not 21:11:56 Well, it's S3 21:12:21 Although, the subject of the change is not one of the dusty corners like freeform ACLs 21:13:02 in the interest of transparency, the s3api patches are a bit of a priority for swiftstack. since we've been shipping swift3 for a while, we want to get the s3api middleware to a point where we can drop-in replace it before we go adding new s3api features 21:13:03 I thought Tim and Kota owned the field. 21:13:46 zaitcev: it's ok. and timur reviewed it too (and positively), so that's a good sign. 21:14:03 oooh, I see. Tim was the author 21:14:35 zaitcev: if it makes you feel any better about it, all of the s3api patches that are on the priority reviews page are ones that we are running in production. doesn't mean they don't have bugs, but they probably don't do terrible things either 21:14:50 that means i should visit it :P 21:15:06 kota_: I approved it just now... 21:15:11 ( timburke will probably remind me that, yes, it is actually possible to run code in prod that does terrible things ;-) 21:15:15 thanks zaitcev! 21:15:23 zaitcev: great, thanks! 21:15:27 kota_: don't worry! there are more you can look at too :-) 21:15:52 mattoliverau: thanks for reviewing https://review.openstack.org/#/c/507808 21:15:53 patch 507808 - swift - Add ability to undelete an account. - 19 patch sets 21:16:03 I know the authors have been working on it for a while 21:16:40 mattoliverau: looks like you offered to work with them on making it something that's a little better operationally 21:17:16 rledisez: this patch would be nice to have you or someone from ovh looking at too. it's mostly around ops procedures 21:17:20 Yeah, and theyve responded, so I'll loop back round on it 21:17:43 Yeah +100 21:17:47 mattoliverau: looks like you've also got a -1 on timburke's client patch at https://review.openstack.org/#/c/577584/ 21:17:48 patch 577584 - python-swiftclient - Stop leaking quite so many connections - 1 patch set 21:17:58 ...and tests fail 21:18:05 yeah, i should really go back and address that... 21:18:14 timburke: I'm guessing this is one you're not looking at right now... :-) 21:18:15 Man, I've been sleep deprived and seems busy :p 21:18:56 mattoliverau: seems to me that you're embarrassing us all with how much you're reviewing! making the rest of us look bad! nice work :-) 21:19:15 +1!! 21:19:27 ok, that's all the specific patches listed on the priority reviews page 21:19:36 rledisez: anything to share this week about losf? 21:20:14 notmyname: no, nothing in particular. alexuyer will focus on ops tools 21:20:24 ok 21:20:36 m_kazuhiro: anything new to share on the task queue work? 21:21:17 No. The patch in priority review is the latest. 21:21:22 ok 21:21:32 kota_: mattoliverau: anything else from you this week? 21:22:07 Nope 21:22:23 kota_: ? 21:22:50 Hopefully he's off reviewing s3api patches :p 21:22:53 lol 21:23:20 from the swiftstack side, beyond the s3api patches, we've been mostly focused on integrating existing features (eg kmip, sharding, etc) 21:24:02 ah 21:24:03 I got word form the openstack foundation that there will be a project onboarding session 21:24:22 i just want to make sure the status of LOSF work to rledisez 21:24:46 Oh speaking of encryption, I have the multi key KMS keymaster which would be good to get it. 21:24:54 I wasn't expecting it, but I'll work on preparing for it (anything you can share mattoliverau will be helpful) 21:24:55 *in 21:24:58 mattoliverau: cool 21:25:11 Kk 21:25:23 kota_: he said there wasn't anthing new to share this week (losf) 21:25:35 kota_: yes, for now everything is fine for us on the "upstream side", we are focusing on moving our production to the latest LOSF version (that is a bit ahead of what's on gerrit) 21:25:54 ok, last call for any other topics. anything else to bring up this week? 21:25:54 i see, thanks 21:26:51 all right 21:26:59 thanks for your work on swift 21:27:02 #endmeeting