21:00:12 #startmeeting swift 21:00:12 Meeting started Wed Aug 7 21:00:12 2019 UTC and is due to finish in 60 minutes. The chair is timburke. Information about MeetBot at http://wiki.debian.org/MeetBot. 21:00:13 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 21:00:15 The meeting name has been set to 'swift' 21:00:17 who's here for the swift meeting? 21:00:19 o/ 21:00:29 clayg, so excited! you beat the meeting start! 21:00:45 o/ 21:00:46 lol 21:00:46 😁 21:01:04 o/ 21:01:28 o/ 21:01:29 o/ 21:01:38 agenda's mostly a continuation of a lot of outstanding work 21:01:44 #link https://wiki.openstack.org/wiki/Meetings/Swift 21:01:53 #topic shanghai 21:02:01 just a quick reminder that prices for the shanghai summit increase next week -- if you're going, you might want to buy now 21:02:09 (i'm not sure off-hand what the price will increase *to*, i just know it will increase) 21:02:13 #link https://www.openstack.org/summit/shanghai-2019/ 21:02:59 thanks everybody for decalring whether you'll be there or not on the etherpad -- it should definitely help the foundation plan for attendance 21:03:08 and also timburke should respond to the TC how *Swift* PTG will be going? 21:03:28 yup -- on my list for this week :-) 21:03:36 great 21:03:52 (attendance will probably be a little light, but then, we were expecting that) 21:04:00 on to updates! 21:04:06 #topic py3 21:04:11 i've been working on finishing up the py3 func tests recently 21:04:18 there were a couple patches small enough that i didn't feel bad +A'ing myself 21:04:22 #link https://review.opendev.org/#/c/674878/ 21:04:25 #link https://review.opendev.org/#/c/674703/ 21:04:34 aside from that, the main advance was in starting on the s3 tests 21:04:38 #link https://review.opendev.org/#/c/674716/ 21:04:56 and i saw that mattoliverau's been doing some review, so thanks for that! 21:05:30 Nps 21:06:05 i'm torn between wanting probe tests (because they'll exercise more of the system) or func tests (so we can start reducing the total number of gate jobs running) 21:06:16 but we're getting there on both fronts! 21:06:51 any questions on py3? 21:07:25 #topic lots of small files 21:07:56 i'm guessing there hasn't been too much going on upstream here, with alecuyer still on vacation 21:08:05 timburke: true 21:08:15 timburke: yes 21:08:18 nothing except I bought the PTG ticket. 21:08:27 but we've been playing around with it at SwiftStack! we can get it to run, at least :-) 21:08:55 +1 21:09:29 is there anything else we should talk about for it? what are the next steps that we want to take? 21:09:31 wow, that's something! somebody else make it running \o/ 21:09:52 rledisez, always good when we can get past "works on my machine" :-) 21:10:07 or "works in my public cloud" or w/e 21:10:18 :) 21:10:32 but definately progress - it's gunna be a hoot 21:11:27 hopefully i'll be able to give it more attention Real Soon Now 21:12:02 rledisez, as we find issues, where's a good place for us to collect them? 21:12:21 gerrit? with fixes? 21:12:53 clayg: +1 21:12:54 obviously if we don't know *how* to fix it a failing test and a ping in IRC would be a good start... probably NOT lp bugs 21:13:04 clayg, that's my usual way of dealing with issues, but sometimes people want something more trackable ;-P 21:13:16 but we could certainly do a etherpad or trello tracker if activity ramps up to the point that it's needed? 21:13:23 i was trying to remember if we had a trello board set up or anything 21:13:34 trello might be good place 21:13:43 and definitively a ping, because we are some commits ahead for some bugfix (I think it's on alecuyer todolist to upstream them asap) 21:13:47 https://trello.com/b/xhNxrcLX/losf 21:13:52 #link: https://trello.com/b/xhNxrcLX/losf 21:13:56 thanks kota_ 21:13:59 ^ oh, BOOM 21:14:44 love it. i think there's one thing i oughta write up (or as clayg suggested, just go ahead and fix) 21:15:01 #topic sharding 21:15:14 i know we've got mattoliverau's patches 21:15:18 I haven't done much extra here 21:15:50 i put up one or two others -- not exactly to do with autosharding, but certainly to do with having a completed feature 21:15:51 I've been meaning to do some more testing of the existing patches, but got caught up with work :( 21:16:01 I saw! thanks 21:16:29 #link https://review.opendev.org/#/c/675014/ 21:16:35 One tracking updated and some py3 fixes right? 21:16:46 in particular, to stop hammering the root container quite so much 21:17:08 yeah thank'll be a good thing :) 21:17:15 *that'll 21:17:21 oh, right -- the other thing wasn't a patch but a bug 21:17:23 #link https://bugs.launchpad.net/swift/+bug/1839355 21:17:24 Launchpad bug 1839355 in OpenStack Object Storage (swift) "container-sharder should keep cleaving when there are no rows" [Undecided,New] 21:17:54 oh, interesting 21:18:29 though i'm not entirely sure on the right resolution there. working through mostly-empty DBs quickly would be good, but i wonder if we could save the hassle of creating (and replicating) all those shard dbs 21:18:44 somethign to think about, anyway 21:19:04 +1 21:19:18 #topic symlinks and versioning 21:19:31 the realization for me was that it can take a long time to shard an empty container just because of the cardinality of shard ranges (the handoff knows about ranges because of replication, but it still takes many cycles to say "yup, noting in that range, nothing in that one either; good enough for this cycle - see you next time!" ZZZzzzzz 21:20:50 yeah, that was about the point at which i wrote up the bug, shortly after doing the math to figure out how long it'll take to clear it 21:21:05 hardlinks to SLOs won't always have the slo_etag and symlink_bytes bits correct in the container listings -> https://review.opendev.org/#/c/633094/24/swift/common/middleware/symlink.py 21:21:38 not sure there's much we can do about that, though :-/ 21:22:05 would love to have some feedback on how we put that in the docs without sounding stupid... "fixing it" seems not worth it considering the cruft and time it would take to essentially just reupload the manifest for them 21:22:33 sometimes trying to make things better just makes things more complicated 🤷‍♂️ 21:22:34 on the plus side, neither will the SLO it's pointing at... 21:22:47 timburke: that's not a plus by any measure 21:23:23 it's just bad and now the badness will accretes 21:23:30 true enough. but we're *mostly* not making things actively worse? 21:24:25 I mean I think that paragraph that explains how hardlinks worse *does* make it actively worse - like the sphere of context where the problem forced users to have to think about it is bigger now 21:24:40 ... but on the whole I think the change is still a net gain - obviously I want hardlinks! 21:24:40 i'm still hopeful that the total number of SLOs written *with* the metadata will be orders of magnitude larger than the total number written *without*, given enough time 21:24:50 just rubber hit the road and this is best we've come up with yet 21:24:57 ... suggestions welcome 😁 21:26:06 i'm rebasing symlink_versions now and mostly hoping i'm done with hardlinks (sans the amazing doco suggestions I'm going to get following this meeting) 21:26:28 I am also finding it annoying that you have to dig out the manifest etag to make a hardlink 21:26:56 clayg, is there anything the rest of us should be weighing in on? are you looking for consensus on what the listing should look like, say? 21:27:14 I'd like to either add support python-swiftclient's `stat` command to do query-params (?multipart-manifest=get) OR make *LO's always return the an X-Manifest-Etag 21:27:23 ... or both? preferences? opinions? 21:27:59 I would like as many people as possible to read the docs on the hardlink patch so they understand the feature 21:27:59 x-manifest-etag's not a bad idea... we've already got it loaded in our head anyway... 21:28:07 seems like the api change is better than the client change?? 21:28:42 if they want to play with `swift upload links -H 'content-length: 0' --object-name link.version -H 'x-symlink-target: test/big.version' -H 'x-symlink-target-etag: b89c395acda3886b823803b7dccb4765' - but just reading the docs and asking questions about the parts that seem weird would be SUPER helpful - maybe we missed something obvious 21:29:33 but even better - it adds documentation to the review - so later after we merge it and we're like "why didn't we do XYZ" we wrote that down somewhere 21:29:33 client change might be nice, too -- i could see some utility to being able to download the SLO manifest from the cli more easily 21:30:55 timburke: well there's also symlink=get - so yeah broad support in the cli for query params would be pretty sick 21:31:05 timburke: agree, was just thinking in terms of one versus the other, but both sound even better :) 21:31:06 but it sounds like everyone likes x-manifest-etag too - so that's probably even cheaper 21:31:47 clayg, do you want to add that as part of the hardlinks patch, or should i propose that separately? 21:32:20 or you could do it. maybe you'd prefer i spend my time reviewing ;-) 21:32:23 no strong preference on my end - probably sufficiently orthogonal to warrent a seperate patch if for no other reason to keep patch count down 21:32:35 👍 21:32:52 well, I'm not going to do anything until I finish the rebase on symlink_versions and I'm out of time today... 21:33:13 i'll see about knocking that out real quick 21:33:16 so if while looking at hardlinks you were to "accidently" purpose a dependent patch that added x-manifest-etag I could +2 it in the morning ;) 21:33:40 anything else for symlinks and versioning? 21:34:36 not at this time - other non swiftstack cores - please take 10m to read the docs and throw some comments on there 21:34:42 happy to take questions 21:34:48 appreciate the feedback 21:35:06 will do 21:35:17 might take a time Friday on my time. 21:35:42 thanks kota_ 21:35:44 #topic open discussion 21:35:53 anything else we ought to discuss? 21:37:27 fwiw i've been playing with different ways to get a feel for what's going on in swift, and i feel like i've come up with a decent dashboard, at least to prep for this meeting 21:37:30 #link https://review.opendev.org/#/dashboard/?title=The+Week+That+Was&foreach=-age:1week&Landed+(Server)=is:merged+project:openstack/swift&Landed+(Client)=is:merged+project:openstack/python-swiftclient&Active+(Server)=is:open+project:openstack/swift&Active+(Client)=is:open+project:openstack/python-swiftclient&Abandoned+(Server)=is:abandoned+project:openstack/swift&Abandoned+(Client)=is:abandoned+project:openstack/python-swiftclient 21:38:24 looks good 21:38:58 i might need to propose fewer patches though -- i'm not sure i like seeing my name that much :-/ 21:39:24 timburke: don't stop being you, your awesome! 21:40:55 all right, seems like there isn't much more to talk about right now -- let's call it a little early and let kota_ and mattoliverau get breakfast ;-) 21:41:04 And just to jump back a bit. I'll start reading the hardlink docs today. I have some long meetings this morning... I'll need something to read :P 21:41:18 \o/ breakfast time! 21:41:21 thanks mattoliverau! 21:41:24 thank you all for coming, and thank you for working on swift! 21:41:32 #endmeeting