Thursday, 2020-01-02

*** d34dh0r53 has quit IRC00:15
*** d34dh0r53 has joined #openstack-swift00:18
*** mvkr has joined #openstack-swift01:23
*** ianychoi has joined #openstack-swift03:04
*** ianychoi has quit IRC03:11
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-swift05:33
*** hoonetorg has quit IRC05:44
*** hoonetorg has joined #openstack-swift05:56
*** diablo_rojo has joined #openstack-swift06:20
*** pcaruana has joined #openstack-swift07:34
openstackgerritCharles Hsu proposed openstack/python-swiftclient master: Support v3 application credentials auth.  https://review.opendev.org/69945708:19
*** jistr_ is now known as jistr08:29
*** rdejoux has joined #openstack-swift08:35
*** rpittau|afk is now known as rpittau08:41
*** diablo_rojo has quit IRC09:49
*** Fengli1 has joined #openstack-swift10:00
*** Fengli has quit IRC10:01
*** Fengli1 is now known as Fengli10:01
mahaticHappy new year swift team!10:04
mahaticmattoliverau, glad to know you're safe. Take care10:04
*** spsurya has joined #openstack-swift12:48
*** ianychoi has joined #openstack-swift13:15
*** theintern_ has joined #openstack-swift15:16
*** Fengli has quit IRC15:21
*** theintern_ has quit IRC15:43
*** theintern_ has joined #openstack-swift15:44
*** pcaruana has quit IRC16:04
*** theintern_ has quit IRC16:08
*** theintern_ has joined #openstack-swift16:23
*** rdejoux has quit IRC16:32
*** theintern_ has quit IRC16:36
*** pcaruana has joined #openstack-swift16:43
*** rpittau is now known as rpittau|afk16:51
clayghappy new years!17:07
*** zaitcev has joined #openstack-swift17:26
*** ChanServ sets mode: +v zaitcev17:26
*** evrardjp has quit IRC17:33
*** evrardjp has joined #openstack-swift17:33
*** gyee has joined #openstack-swift17:53
ormandjhappy new years :)18:41
claygtimburke: i'm going to fix that pep8 error on p 700818 real quick19:11
patchbothttps://review.opendev.org/#/c/700818/ - swift - Deprecate per-service auto_create_account_prefix - 1 patch set19:11
openstackgerritClay Gerrard proposed openstack/swift master: Deprecate per-service auto_create_account_prefix  https://review.opendev.org/70081819:12
*** pcaruana has quit IRC19:15
openstackgerritClay Gerrard proposed openstack/swift master: Use less responses from handoffs  https://review.opendev.org/70023919:20
timburkeclayg, i'm loving auto_create_account_prefix in constraints! seems like a good fit, particularly given its pervasiveness19:53
clayggood to hear!  When you suggested it I thought it was a great idea19:54
ormandjquick question (as if anything i mention is quick): right now, when dealing with a web display for containers, we look if there's less than X objects, if so, get the list, do pagination, display forward/backward/go to end/to to beginning buttons works great. for containers over 1000 objects, we just provide forward/backward/go to beginning - but no go to end since we displayy folders and a container20:13
ormandjHEAD shows total object count, so if we were try to use that to paginate, it would cause issues due to some of the objects being within a folder abstraction20:13
ormandjhow are people working around this aside from just getting a full object list every time20:14
*** camelCaser has quit IRC20:54
claygormandj: "working around this" ... for "containers over 1000 objects"?  like... the "go to end" option?  maybe `?reverse=true`20:55
*** camelCaser has joined #openstack-swift20:56
clayg"it seems strange that it's configurable at all" 👍20:57
clayg"this is only configurable for legacy compatibility, please pretend it's not here and don't change it"20:57
claygor just... "don't change it"20:57
ormandjclayg: right, giving the ability to seek to the end of the object listing for display where you display folders, so you can't count on the total object count being correct21:03
ormandjwell, in terms of objects to display that is, it's correct in the count of objects in the container21:04
claygyeah if I wanted to display the "-1" page I'd use the reverse query param (as opposed to if I just wanted the "next" page in which case I'd use a marker query)21:19
claygbut then for "backward" from the last page you'd have to do a reverse+marker query21:19
claygI'm curious if this kind of navigation is really all that useful?21:20
claygIf you're doing "prefix+delim" queries to display "subfolders" that should get you the first 10K top level folders21:21
claygmore than that it's not obvious how much they're going to want to look through them 1K at a time?  🤷‍♂️21:21
claygI have auto_shard = true and shard_container_threshold = 5021:27
claygwhy is my container with 1K objects not sharding?21:27
claygI'm running -sharder and -replicator in the background on my saio21:27
claygit's sayingcontainer-sharder-6041: Since Thu Jan  2 21:27:10 2020 visited - attempted:0 success:0 failure:0 skipped:1 completed21:27
claygthe container shows up in recon drops21:28
claygeven, the recon drop for node_index: 121:30
claygmaybe with auto turned on ... it still wants for me to tell the database to enable sharding?  and *then* it will automatically find and replicate shard ranges?21:31
claygpost as an admin I guess 🤔21:37
claygI HAVE SO MANY DBs NOW!21:38
claygit created all the shard range dbs, but got hung after cleaving only two of them 😠21:44
claygAudit failed for root /srv/node2/sdb2/containers/42/<...>.db (AUTH_test/100%25beef): missing range(s): obj0249-21:50
clayg👎21:50
ormandjclayg: thanks, will sit and think on that for a while. i'm not the ui guy, for good reason ;)21:51
claygit's true tho, none of the replicas have any shard ranges after   Name: .shards_AUTH_test/100%beef-a849a64896e6e27dfd4ed768ffa7bd2f-1578001061.04390-921:53
clayg    lower: 'obj0224', upper: 'obj0249'21:53
claygwhich is weird because there's 1K objXXX in the container21:53
claygmaybe i should have used the manage-shard-ranges enable instead of a POST as an admin (but that's how probetests do it!)21:56
clayggah, how does auto sharding work!  😠22:06
clayg@timburke I almost always end up messing up sharding until I turn of auto and do it like we do in the controller, do you run with auto_shard = true !?22:07
claygwtf, with auto still turned on it looks like some of by shard dbs started sharding on their own!?22:23
claygtimburke: I think https://gist.github.com/tipabu/c28fb327c0bf0c74ac418b08e0986395 looks GREAT22:29
claygif it's in gerrit I'd approve it - or I can respin with that added later22:29
mattoliveraumorning22:33
mattoliverauclayg: yeah, I think iin the current version of auto sharding, we wanted to still opt in on the container sharding. So you need to activavte it on the root container. Then that container (including shards) will start auto sharding.22:39
mattoliverauideally, when were ready we want to just not have to enable sharding on anything.22:39
claygmattoliverau: thanks!22:40
mattoliverauthey might also shrink22:41
clayghow long does a sharded shard stick around?22:58
claygI mean the old db, that was the shard, that cleaves itself into a sub-root - it's children update the parent root directly, no?22:59
clayghrm... yeah the old shard doesn't recieve any count updates - they go directly to the root23:05
claygI assumed it kind of had to stick around incase any async pendings try send it updates (and then it could redirect same as the root)23:05
claygbut at somepoint that probably starts to do more harm than good if it's ranges get out of date23:06
*** Fengli has joined #openstack-swift23:08
timburkeclayg, it's part of _audit_shard_container and based on reclaim_age: https://github.com/openstack/swift/blob/master/swift/container/sharder.py#L748-L75923:37
clayghell yeah it is, that's awesome23:37
timburkeer, https://github.com/openstack/swift/blob/2.23.0/swift/container/sharder.py#L747-L758 (in case we come back to these logs in a few months or whatever)23:38
timburkebut yeah, the sharded shard should still be getting some updates from the root DB; i'm pretty sure its redirects ought to stay pretty up-to-date23:40
timburkeoh... maybe not, actually... i wonder if https://github.com/openstack/swift/blob/2.23.0/swift/container/sharder.py#L749 should be `broker.merge_shard_ranges(shard_ranges)`, with an "s"...23:41
timburkemeh. this is why we have the updater notice if it gets a bunch of redirects and just go back to the root23:42

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!