21:00:05 #startmeeting swift 21:00:06 Meeting started Wed Jul 25 21:00:05 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:07 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:00:09 The meeting name has been set to 'swift' 21:00:13 who's here for the swift team meeting? 21:00:16 o/ 21:00:31 hi 21:00:35 o/ 21:00:39 hello 21:01:04 timburke: clayg: courtesy ping 21:01:13 o/ 21:01:24 notmyname: thanks! 21:01:38 welcome, everyone 21:01:53 should be a quick meeting, i hope 21:02:28 there's a couple of patches to go over, and I wanted to make sure were all thinking about the same things (especially now that kota_ and timburke are back online) 21:02:38 #link https://wiki.openstack.org/wiki/Meetings/Swift 21:02:53 first, python-swiftclient 3.6.0 has been tagged 21:03:07 nice 21:03:11 this is what will be in openstack's rocky release 21:03:13 \o/ 21:03:38 it's got a couple of cool things in it. thanks for your help over the past week to get stuff landed 21:04:16 next up, our regular checkin for https://review.openstack.org/#/c/427911/ 21:04:17 patch 427911 - swift - Replace MIME with PUT+POST for EC and Encryption 21:04:24 heh 21:04:40 zaitcev: IIRC last you said about it in irc was, "looks great, let's land it" (or something close to that) :-) 21:04:47 Yes 21:05:23 Sorry I said I'd take a look, I started yesterday, as it's been a crazy busy week. Promise to look this week. 21:05:27 as a reminder to us all, this patch unblocks future work on replacing the object servers with something different than what we have today 21:05:27 Well, there was an addendum: Alistair wanted to change use Content-Type. I added to his method +1, I forgot the number. It's on Etherpad. 21:05:50 #link https://etherpad.openstack.org/p/swift-put-post 21:06:23 Personally I think that his patch is not making anything worse, but I don't share his opposition to the /./v1/ 21:06:35 same as matt, i just finished some un-related work today that was tying me up, so I should be able to start focusing on this patch tomorrow 21:06:48 tdasilva: mattoliverau: thanks! 21:06:49 If it helps to land the whole shebang, I'm okay with the Content-Type. 21:08:26 let's see what tdasilva and mattoliverau say 21:08:31 let's 21:08:50 next patch... https://review.openstack.org/#/c/447129 21:08:51 patch 447129 - swift - Configure diskfile per storage policy 21:09:01 looks like rledisez uploaded a new patch set that addressed comments 21:09:23 acoles liked it a while back. kota_ has looked at it too 21:09:29 oh and m_kazuhiro 21:09:48 i looked at the patch in this week, then I was realizing I'd like to hear tdasilva's opinion 21:10:24 that is because it enables us to use custom diskfile so i thought similar thing like swift-on-file. 21:10:24 kota_: about something specifically in the patch? 21:10:30 ah, got it 21:10:32 I understand 21:10:44 kota_: saw your msg earlier today, I will take a look...thanks for the heads up 21:11:00 in particular, the point is how to configure the diskfile. 21:11:21 thanks tdasilva. 21:11:51 kota_: while tdasilva is looking at the put+post patch, could you (and m_kazuhiro) look at rledisez's patch again to make sure it's ok? 21:12:11 other than the configuration format, it looks fine so far. let me check the last change yesterday by rledisez 21:12:19 sure 21:12:27 thanks 21:13:31 notmyname: ok 21:13:35 timburke reminded me of https://review.openstack.org/#/c/585568/ earlier today. this fixes probtests that we'd been seeing 21:13:36 patch 585568 - swift - Lower the required version of ipaddress 21:13:46 I'l also like to suggest making the probe tests voting in the gate 21:14:48 looks like the probe test failures were the result of a dependency issue 21:15:03 zaitcev: could you take a quick look at that patch and land it if it looks good? 21:15:15 introduced in https://review.openstack.org/#/c/569404/ 21:15:16 patch 569404 - swift - IP Range restrictions in temp urls (MERGED) 21:15:45 i just was seeing the ipaddress dependency in the diskfile configuration patch now :-o 21:15:50 timburke: wow. who approved that?!?! ;-) 21:15:58 some idiot ;-) 21:16:04 The probe tests do cover things we can't cover elsewhere. So +1 for voting, so kind as there passing all the time. I'd admit I don't run them as often as I should. 21:16:29 *so long as there passing... 21:16:35 mattoliverau: I know, right? that's why I'd prefer them voting. to cover the lack of running them locally all the time 21:17:01 Really, guys? No support for IPv6? 21:17:22 add it! :-) 21:17:59 Hey, I thought you guys might be interested in this blog by freenode staff member Bryan 'kloeri' Ostergaard https://bryanostergaard.com/ 21:17:59 or maybe this blog by freenode staff member Matthew 'mst' Trout https://MattSTrout.com/ 21:18:15 Hey, I thought you guys might be interested in this blog by freenode staff member Bryan 'kloeri' Ostergaard https://bryanostergaard.com/ 21:18:15 or maybe this blog by freenode staff member Matthew 'mst' Trout https://MattSTrout.com/ 21:18:19 I'm sure those are totally legit domains 21:18:23 lol 21:18:35 Do +m and add +v to us all? 21:18:48 I don't htink I can do that in this channel 21:18:59 zaitcev: eh? we've got tests for it: https://review.openstack.org/#/c/569404/17/test/unit/common/middleware/test_tempurl.py@1453 21:19:00 patch 569404 - swift - IP Range restrictions in temp urls (MERGED) 21:19:35 It's a massive bot attack, I saw that thing flooding other channels on unrelated networks like IRChighway 21:19:46 the examples in the docs should probably include some ipv6 though 21:19:55 timburke: thanks 21:20:12 * kota_ is off for a while, baby crying 21:20:20 yeah, I think we're just about done 21:20:27 that's all the patches I wanted to bring up 21:20:38 kota_: m_kazuhiro: will you both be at the denver ptg? 21:20:42 zaitcev: and you? 21:20:51 mattoliverau: you're not going to be available then 21:21:09 notmyname: I'll be at denver 21:21:12 Nope, 10 the is the babies due date 21:21:20 10th 21:21:23 m_kazuhiro: do you know if kota_ will be there too? 21:21:53 I hope the train line is all fixed :) 21:22:51 notmyname: yes for Denver 21:23:01 great. I'm looking forward to seeing people there 21:23:12 is there anything else to bring up in the meeting this week? 21:23:12 thinking of patches, i wanted to thank mattoliverau for looking at https://review.openstack.org/#/c/578075/ -- now i can rebase https://review.openstack.org/#/c/577874/ and start working on a multi-key kmip keymaster... 21:23:12 patch 578075 - swift - Add keymaster to fetch root secret from KMIP service (MERGED) 21:23:14 patch 577874 - swift - Add support for multiple root encryption secrets 21:23:27 timburke: \o/ 21:23:53 Nps, thanks for acoles first made it easy to test 21:23:58 I'd like to cancel next week's meeting. I will not be available 21:24:05 *gist 21:24:20 Man auto complete is failing me today 21:24:41 back 21:25:02 I'll be there, saying about denver ptg. 21:25:13 cool 21:25:30 I think that's about it for this week 21:25:39 thank you for your work on swift 21:25:55 #endmeeting