21:00:19 <notmyname> #startmeeting swift
21:00:20 <openstack> 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 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
21:00:23 <openstack> The meeting name has been set to 'swift'
21:00:24 <notmyname> who's here for the swift team meeting?
21:00:26 <timburke> o/
21:00:33 <m_kazuhiro> o/
21:00:37 <kota_> o/
21:00:51 <mattoliverau> o/
21:00:57 <notmyname> welcome
21:01:03 <tdasilva> hi
21:01:08 <notmyname> #link https://wiki.openstack.org/wiki/Meetings/Swift
21:01:27 <rledisez> hi o/
21:01:32 <notmyname> again, not a super full agenda, so let's see if it's another short one :-)
21:02:07 <notmyname> in lieu of any specific topics, let's look at the priority reviews page
21:02:08 <notmyname> #link https://wiki.openstack.org/wiki/Swift/PriorityReviews
21:02:33 <notmyname> #topic priority reviews
21:03:05 <notmyname> mattoliverau: you've been reviewing the keymaster patches, I think
21:03:15 <mattoliverau> I have
21:03:45 <notmyname> anything to report or anything to bring up with those?
21:04:11 <mattoliverau> There looking good. A little more work, doc and tests, but otherwise should land soon
21:04:17 <notmyname> cool
21:05:19 <notmyname> m_kazuhiro or rledisez: any updates for task queue work?
21:05:34 <notmyname> (sorry, got distracted from the keyboard for a moment)
21:05:49 <m_kazuhiro> nothing, in this week.
21:05:54 <notmyname> I don't see any reviews on https://review.openstack.org/#/c/601950/
21:05:55 <patchbot> patch 601950 - swift - Enable to configure object-expirer in object-serve... - 4 patch sets
21:06:10 <notmyname> rledisez: are you (or other ovh) planning on reviewing that patch?
21:07:33 <rledisez> notmyname: i'm planning to review it
21:07:43 <notmyname> great :-)
21:07:43 <notmyname> thanks
21:08:24 <notmyname> 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 <notmyname> tdasilva: were you wanting someone else to review https://review.openstack.org/#/c/575829/ ?
21:08:44 <patchbot> patch 575829 - swift - Change PUT bucket conflict error - 8 patch sets
21:08:55 <notmyname> same question to zaitcev on https://review.openstack.org/#/c/604208/
21:08:55 <patchbot> patch 604208 - swift - s3api: Increase max body size for Delete Multiple ... - 3 patch sets
21:09:28 <zaitcev> What was the question?
21:09:58 <notmyname> 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 <patchbot> patch 604208 - swift - s3api: Increase max body size for Delete Multiple ... - 3 patch sets
21:10:03 <tdasilva> 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 <notmyname> tdasilva: ok. I'll let you and timburke work it out then :-)
21:11:29 <timburke> 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 <notmyname> 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 <zaitcev> Well, it's S3
21:12:21 <zaitcev> Although, the subject of the change is not one of the dusty corners like freeform ACLs
21:13:02 <notmyname> 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 <zaitcev> I thought Tim and Kota owned the field.
21:13:46 <notmyname> zaitcev: it's ok. and timur reviewed it too (and positively), so that's a good sign.
21:14:03 <zaitcev> oooh, I see. Tim was the author
21:14:35 <notmyname> 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 <kota_> that means i should visit it :P
21:15:06 <zaitcev> kota_: I approved it just now...
21:15:11 <notmyname> ( timburke will probably remind me that, yes, it is actually possible to run code in prod that does terrible things ;-)
21:15:15 <timburke> thanks zaitcev!
21:15:23 <kota_> zaitcev: great, thanks!
21:15:27 <notmyname> kota_: don't worry! there are more you can look at too :-)
21:15:52 <notmyname> mattoliverau: thanks for reviewing https://review.openstack.org/#/c/507808
21:15:53 <patchbot> patch 507808 - swift - Add ability to undelete an account. - 19 patch sets
21:16:03 <notmyname> I know the authors have been working on it for a while
21:16:40 <notmyname> mattoliverau: looks like you offered to work with them on making it something that's a little better operationally
21:17:16 <notmyname> 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 <mattoliverau> Yeah, and theyve responded, so I'll loop back round on it
21:17:43 <mattoliverau> Yeah +100
21:17:47 <notmyname> mattoliverau: looks like you've also got a -1 on timburke's client patch at https://review.openstack.org/#/c/577584/
21:17:48 <patchbot> patch 577584 - python-swiftclient - Stop leaking quite so many connections - 1 patch set
21:17:58 <notmyname> ...and tests fail
21:18:05 <timburke> yeah, i should really go back and address that...
21:18:14 <notmyname> timburke: I'm guessing this is one you're not looking at right now... :-)
21:18:15 <mattoliverau> Man, I've been sleep deprived and seems busy :p
21:18:56 <notmyname> 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 <tdasilva> +1!!
21:19:27 <notmyname> ok, that's all the specific patches listed on the priority reviews page
21:19:36 <notmyname> rledisez: anything to share this week about losf?
21:20:14 <rledisez> notmyname: no, nothing in particular. alexuyer will focus on ops tools
21:20:24 <notmyname> ok
21:20:36 <notmyname> m_kazuhiro: anything new to share on the task queue work?
21:21:17 <m_kazuhiro> No. The patch in priority review is the latest.
21:21:22 <notmyname> ok
21:21:32 <notmyname> kota_: mattoliverau: anything else from you this week?
21:22:07 <mattoliverau> Nope
21:22:23 <notmyname> kota_: ?
21:22:50 <mattoliverau> Hopefully he's off reviewing s3api patches :p
21:22:53 <notmyname> lol
21:23:20 <notmyname> from the swiftstack side, beyond the s3api patches, we've been mostly focused on integrating existing features (eg kmip, sharding, etc)
21:24:02 <kota_> ah
21:24:03 <notmyname> I got word form the openstack foundation that there will be a project onboarding session
21:24:22 <kota_> i just want to make sure the status of LOSF work to rledisez
21:24:46 <mattoliverau> Oh speaking of encryption, I have the multi key KMS keymaster which would be good to get it.
21:24:54 <notmyname> I wasn't expecting it, but I'll work on preparing for it (anything you can share mattoliverau will be helpful)
21:24:55 <mattoliverau> *in
21:24:58 <notmyname> mattoliverau: cool
21:25:11 <mattoliverau> Kk
21:25:23 <notmyname> kota_: he said there wasn't anthing new to share this week (losf)
21:25:35 <rledisez> 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 <notmyname> ok, last call for any other topics. anything else to bring up this week?
21:25:54 <kota_> i see, thanks
21:26:51 <notmyname> all right
21:26:59 <notmyname> thanks for your work on swift
21:27:02 <notmyname> #endmeeting