21:05:51 #startmeeting swift 21:05:52 Meeting started Wed May 9 21:05:51 2018 UTC and is due to finish in 60 minutes. The chair is notmyname. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:05:53 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:05:56 The meeting name has been set to 'swift' 21:05:56 whoa!! 21:05:59 !!!??? 21:06:00 clayg: Error: "!!???" is not a valid command. 21:06:05 sorry 21:06:08 we can meet here, it'll be like dublin airbnb all over again. swifters will prevail. 21:06:12 ok, meeting time is now and in here 21:06:39 #link https://wiki.openstack.org/wiki/Meetings/Swift 21:06:51 the agenda this week is simply to talk about feature/deep-review 21:06:55 notmyname: i can remove that 21:06:57 #link https://review.openstack.org/#/q/status:open+project:openstack/swift+branch:feature/deep-review 21:07:16 corvus: thank you 21:07:56 acoles: I've seen lots of new patch sets for the review banch this week. how is it going? 21:08:44 notmyname: it's ok. thanks to those who have reviewed. 21:08:58 there have not been a lot of reviews 21:09:05 (granted, it's a huge topic) 21:09:08 so far this week I have turned around review comments and pushed new patchsets daily. 21:09:20 thank you for that 21:09:26 notmyname: but, yes, we're not making much headway up the chain of patches yet 21:10:12 given who we have available for reviews, I wrote a wall of text here in IRC earlier this week :-) 21:10:17 thanks mattoliverau for helping with the docs 21:10:46 notmyname: reviewing the akick entries, i don't see any ban for your ip address. looks like one was added to #openstack-meeting for the "patchbot" nick specifically but not by address mask. also i see your notmyname nick present in #openstack-meeting 21:11:51 acoles: where should we be focusing on for now? 21:12:07 should we start with the first refactoring patch? 21:12:49 I got sidetracked with the diagrams/doc. But will do a good review over the next 2 days. Tho have looked at the code alot over the last while, and probably a little code blind, except for acoles better code that is :p 21:12:51 notmyname: that's had quite a lot of review from timburke and kota, so no, skip that if you're starting 21:12:53 aha, not in the akick list, it's a +b on the channel for your address mask 21:12:57 i'll fix that 21:13:27 nevermind, someone else already has 21:13:44 also, reminder to everyone: when leaving a comment, please include a diff with improvements 21:13:47 fungi: thanks 21:13:59 I liked notmyname's suggestion that some start at top and some start at bottom of chain, but not sure how to coordinate that 21:14:49 I started with the docs and got as far as the "beyond here it needs to be updated" marker 21:14:51 if you want to dive in, the final patch (docs) now has sufficient content to guide you through dpeloying and sharding 21:15:07 yeah, the docs look great 21:15:43 acoles: you mentioned something earlier this week about dropping some stuff from the docs. has that happened already, or is that something someone else can do? 21:16:41 notmyname: no I was hoping to work on that tomorrow - everything should stay up to the obvious 'below here is outdated' heading , but beyond that I think we can trim down the content. 21:16:55 ok 21:17:12 and I'm happy if anyone else wants to do that. 21:17:17 :-) 21:17:51 I would love to be able to do that during my tomorrow 21:18:17 it's great to have plenty of explanation but not if it slows us down by needing to review lots of text - we can always add to it on master 21:18:37 right 21:19:02 clayg: can you give a quick update of what we've seen with this feature branch on the really big DB in our test clusteR? 21:19:08 plus, I have relatively large amount of comment in code 21:19:19 notmyname: we're getting there! 21:19:42 I'm playing with some of the knobs trying to figure out the right tuning 21:19:57 a week or two ago, I said something about it being pretty slow, but it looks like it's gotten a lot better since then 21:19:59 An explaination on the shard range states woukd be good. But yeah trimmming down the other under hood makes sense 21:20:04 we're learning with a 140M row db on a cluster with only HDD - totally under water with out of sync db's and async pileups 21:20:10 so this thing was a MESS - and it's WORKING!!! 21:20:21 mattoliverau: yes, there is some new content I want to add too 21:20:29 we've dug outself out of the whole and watching things just sort of settle down and clean up 21:20:34 mattoliverau: states being one 21:20:46 clayg: that's really good news 21:20:48 async's have made their way into the new shard db's and everything looks good 21:21:11 still waiting for the final unlinks to happen on a couple of replicas - then we're moving on to more testing on some new/different datasets and different hardware configs 21:21:18 this stuff is AMAZING 21:21:22 total game changer for swift 21:21:49 mattoliverau & acoles & timburke definately get some mad props here 21:21:51 kota_: m_kazuhiro: have either of you had a chance to run this feature branch yet? 21:21:59 clayg: yes, definitely! 21:22:35 notmyname: not yet playing. just run unit tests in my local at the refactoring patch. 21:22:46 kota_: that's good 21:22:56 i'll make it maybe in this week or the next. 21:23:05 thanks 21:23:16 It would be great to start getting some review on https://review.openstack.org/565744 21:23:17 patch 565744 - swift (feature/deep-review) - Add support for sharding in ContainerBroker 21:23:21 clayg \o/ 21:23:28 at least in the review process, I should do something to check the behaivor and performance 21:23:36 I'm still hoping that we can have the patch chain land at the end of next week 21:23:52 (remember that may 17 is swift's birthday) 21:24:08 notmyname: I say if it's NOT landed we just go ahead and do it anyway - I trust Al 21:24:12 this stuff is too awesome not to have 21:24:14 :-) 21:24:21 clayg: do not trust anyone 21:24:28 I mean you can glance at it or whatever - it works exactly how mattoliverau told you it did 3 years ago ;) 21:24:34 well.. *mostly* :P 21:24:45 notmyname: not yet. but I will do. 21:25:13 Lol 21:25:23 mattoliverau: 4? :P 21:25:48 With alot of polish from acoles and timburke 21:25:48 I'm not super worried yet about not having a lot of visible reviews on the patch chain, if everything is actually mostly ok. but I'd like to see some by friday or monday 21:26:03 if any core wants to discuss patches with me in real time (voice/video) then I'll do my best to make that happen 21:26:33 seeing some reviews (positive and negative) will help us all get it landed sooner 21:27:14 clayg too early to math :p 21:27:27 I know for a fact we can still make improvements to container sharding. there's still a lot of opportunity to do more (eg automatic sharding is something we're leaving to do later) 21:28:20 so this comes down to (1) does it make users' lives better? (2) is it maintainable? (3) is it written so we can keep making it better? 21:29:56 is there anything else we need to talk about on this topic? 21:30:07 acoles: clayg: kota_: mattoliverau: you good? 21:30:22 Yup 21:30:30 yes 21:30:35 notmyname: yes 21:30:51 ok, let's look forward just a little bit... 21:31:13 i gotta bounce, I'll catch up on logs! 21:31:35 I'd like to tag a swift release after this feature branch lands 21:31:51 clayg: o/ 21:32:07 so if it lands next week, that means a new swift release will be the end of next week or, more likely, the beginning of the week of the 21st 21:32:15 that week is the week of the openstack summit in vancouver 21:32:41 ¥o/ 21:33:01 there's a lot of great stuff that's in swift that we'll be able to talk about there :-) 21:33:07 s3api and container sharding 21:33:23 That would be huge release 21:33:27 yep 21:33:48 there's also an onboarding sessions that a few people have signed up for. I'll be leading that (and I hope with a lot of help from kota_ and mattoliverau :-) 21:34:08 Yup I'll be there :) 21:34:38 oh, i didn't notice the onboarding session. I'll add it to my schedule 21:34:39 speaking of openstack events, please be sure to sign up for the PTG happening in september in Denver 21:34:52 kota_: it's immediately after the swift project update on thursday 21:35:24 (that means i didn't check the schedule in detail yet :P) 21:35:27 the PTG is $200 right now and the prices will go up at the end of next week 21:35:36 #link https://www.openstack.org/ptg 21:36:10 that's all I had for this week. feature/deep-review update. next swift release. sing up for the ptg 21:36:17 does anyone else have anything for the meeting this week? 21:36:18 Also to be a little annoying, ill be in Fiji next week for a family holiday, so will review as much as i can before i leave, and try among keep up with what ever internet i end up getting. 21:36:23 hmmm 21:36:39 tdasilva should be back next week, too 21:37:09 ah, RedHad summit in Boston this week. 21:37:24 nah, he's on vacation this week 21:37:34 but there is a red hat summit. but in san francisco 21:37:36 oh, really 21:37:42 Damn phone auto complete, * try to keep up.. 21:37:50 ok, not in Boston 21:38:19 i saw takashi's activities. he joined it. 21:38:47 thank you for your work on swift. 21:38:50 #endmeeting