Thursday, 2019-05-09

*** zaitcev has joined #openstack-swift00:11
*** ChanServ sets mode: +v zaitcev00:11
*** gyee has quit IRC00:38
openstackgerritTim Burke proposed openstack/swift master: Get functional/tests.py running under py3  https://review.opendev.org/64252000:44
openstackgerritTim Burke proposed openstack/swift master: Add py37 func test job  https://review.opendev.org/64585600:44
openstackgerritTim Burke proposed openstack/swift master: py3: Cover account/container func tests  https://review.opendev.org/64538800:44
openstackgerritTim Burke proposed openstack/swift master: py3: Bring functional/test_object.py under test; add func-ec-py37 job  https://review.opendev.org/64589500:44
openstackgerritTim Burke proposed openstack/swift master: py3: port staticweb and domain_remap func tests  https://review.opendev.org/64792300:44
openstackgerritTim Burke proposed openstack/swift master: py3: port dlo func tests  https://review.opendev.org/64292000:44
openstackgerritTim Burke proposed openstack/swift master: py3: finish porting proxy/test_server.py  https://review.opendev.org/65770000:47
openstackgerritTim Burke proposed openstack/swift master: py3: port proxy/test_mem_server.py  https://review.opendev.org/65770500:47
*** NM has joined #openstack-swift01:02
*** NM has quit IRC01:26
*** zaitcev_ has joined #openstack-swift01:31
*** ChanServ sets mode: +v zaitcev_01:31
*** baojg has joined #openstack-swift01:32
*** zaitcev has quit IRC01:34
*** zaitcev_ has quit IRC02:00
*** zaitcev_ has joined #openstack-swift02:11
*** ChanServ sets mode: +v zaitcev_02:11
*** zaitcev__ has joined #openstack-swift02:13
*** ChanServ sets mode: +v zaitcev__02:13
*** zaitcev_ has quit IRC02:16
*** itlinux has joined #openstack-swift02:28
*** itlinux has quit IRC02:38
*** zaitcev__ has quit IRC03:24
*** psachin has joined #openstack-swift03:34
*** zaitcev__ has joined #openstack-swift03:36
*** ChanServ sets mode: +v zaitcev__03:36
*** zaitcev__ has quit IRC03:36
*** e0ne has joined #openstack-swift03:47
*** e0ne has quit IRC04:46
*** samueldmq has quit IRC05:19
*** tesseract has joined #openstack-swift07:09
*** rdejoux has joined #openstack-swift07:16
*** [diablo] has quit IRC07:42
*** dr_gogeta86 has joined #openstack-swift07:58
*** [diablo] has joined #openstack-swift07:59
*** mikecmpbll has joined #openstack-swift08:07
*** e0ne has joined #openstack-swift08:14
*** e0ne has quit IRC08:17
*** tkajinam has quit IRC08:30
*** gmann_pto has joined #openstack-swift08:59
*** tdasilva has quit IRC09:23
*** e0ne has joined #openstack-swift09:38
*** e0ne has quit IRC09:40
*** e0ne has joined #openstack-swift09:56
*** e0ne has quit IRC09:59
*** mugsie has quit IRC10:25
*** mugsie has joined #openstack-swift10:35
*** mugsie has quit IRC10:35
*** mugsie has joined #openstack-swift10:36
*** mugsie has quit IRC10:38
*** mugsie has joined #openstack-swift10:39
*** tesseract has quit IRC10:40
*** tesseract has joined #openstack-swift10:41
*** tesseract has quit IRC10:45
*** tesseract has joined #openstack-swift10:45
*** NM has joined #openstack-swift11:39
*** NM has quit IRC11:42
*** gmann_pto has quit IRC11:50
*** zigo has quit IRC11:59
*** baojg has quit IRC12:08
*** zigo has joined #openstack-swift12:24
*** NM has joined #openstack-swift12:27
*** tdasilva has joined #openstack-swift12:52
*** ChanServ sets mode: +v tdasilva12:52
*** NM has quit IRC13:06
*** tdasilva has quit IRC13:12
*** NM has joined #openstack-swift13:14
*** samueldmq has joined #openstack-swift13:16
*** baojg has joined #openstack-swift13:20
*** tesseract has quit IRC13:35
*** tesseract has joined #openstack-swift13:35
*** altlogbot_2 has quit IRC13:43
*** altlogbot_0 has joined #openstack-swift13:45
*** psachin has quit IRC13:51
*** tdasilva has joined #openstack-swift13:58
*** ChanServ sets mode: +v tdasilva13:58
*** itlinux has joined #openstack-swift14:17
*** ccamacho has joined #openstack-swift14:21
*** itlinux has quit IRC14:23
*** tesseract has quit IRC14:24
*** tdasilva has quit IRC14:25
*** mvkr has quit IRC14:37
*** BjoernT has joined #openstack-swift14:39
*** BjoernT has quit IRC14:40
*** tesseract has joined #openstack-swift14:50
*** tesseract has quit IRC14:51
*** tesseract has joined #openstack-swift14:51
*** BjoernT has joined #openstack-swift14:54
*** tesseract has quit IRC15:01
*** tesseract has joined #openstack-swift15:03
*** NM has quit IRC15:04
*** tesseract has quit IRC15:11
*** tesseract has joined #openstack-swift15:16
*** tesseract has quit IRC15:18
*** tesseract has joined #openstack-swift15:20
alecuyerhey kota_, I have tried to run the ansible playbooks starting from clay's work (https://github.com/swiftstack/vagrant-swift-all-in-one/pull/80)15:33
alecuyerThe first problem I see is that the object-rpcmanager fails to start the golang binary, because it cannot find the leveldb and snappy libs, which are in /usr/local/lib15:34
alecuyerThis can be worked around using LD_LIBRARY_PATH or adding /usr/local/lib to a new file in /etc/ld.so.conf.d/ (until we figure out how to set rpath with go modules)15:35
*** baojg has quit IRC15:37
alecuyerAfter that, I got level=fatal msg="write udp 127.0.0.1:37037->127.0.0.1:8125: write: connection refused" socket=/srv/node1/sdb1/losf/rpc.socket15:37
alecuyerwhich happens because the go module picked the wrong statsd commit, (it tries to check if there's a statsd server listening and exits if it doesn't find one)15:38
alecuyerSo, I still have some reading to do on go modules15:39
*** mvkr has joined #openstack-swift15:42
*** hugokuo has joined #openstack-swift15:50
alecuyerok so that second problem is because rpc.go has the import as "github.com/alecuyer/statsd" when it should be "github.com/alecuyer/statsd/v2". Then it all works15:50
*** timburke has joined #openstack-swift15:51
*** ChanServ sets mode: +v timburke15:51
kota_alecuyer: sounds nice. could you make a commit for gerrit review?15:52
alecuyerkota_: yes, running a few more tests in the VM and I'll do that15:54
kota_alecuyer: great, thanks!15:54
alecuyerlet's see if it works ;)15:54
*** gyee has joined #openstack-swift15:54
kota_:D15:54
*** mvkr has quit IRC15:59
*** tesseract has quit IRC16:01
alecuyerhmm I don't think we even try to start swift-object-rpcmanager ? (I only find "swift-init main start")16:03
timburke:D p 657705 gets us to 83% unit test coverage!16:05
patchbothttps://review.opendev.org/#/c/657705/ - swift - py3: port proxy/test_mem_server.py - 3 patch sets16:05
alecuyerouch nevermind, I can't read..16:10
*** BjoernT has quit IRC16:26
openstackgerritAlexandre Lécuyer proposed openstack/swift feature/losf: WIP: setup losf policy for in-process functional testing  https://review.opendev.org/65707516:34
openstackgerritAlexandre Lécuyer proposed openstack/swift feature/losf: Change losf functest to run on saio setup  https://review.opendev.org/65715316:34
openstackgerritAlexandre Lécuyer proposed openstack/swift feature/losf: Use statsd v2, set LD_LIBRARY_PATH for index server  https://review.opendev.org/65815216:34
*** irclogbot_3 has quit IRC16:46
*** irclogbot_2 has joined #openstack-swift16:47
*** NM has joined #openstack-swift16:57
*** tdasilva has joined #openstack-swift17:15
*** ChanServ sets mode: +v tdasilva17:16
*** mikecmpbll has quit IRC17:32
*** rdejoux has quit IRC17:44
*** BjoernT has joined #openstack-swift17:55
*** NM has quit IRC18:01
*** NM has joined #openstack-swift18:02
openstackgerritTim Burke proposed openstack/swift master: We don't have to keep the retrieved token anymore  https://review.opendev.org/60916818:21
*** BjoernT has quit IRC18:34
*** NM has quit IRC18:39
*** BjoernT has joined #openstack-swift18:44
clayg@timburke what do you think about pulling the MERGE_ITEMS interface and object-expirer-async-delete-row-handling into orthogonal patches with the s3api bulk delete feature dependent on them?  Then we could discuss merging the base primitives maybe ahead of the next bit?19:10
timburkeclayg, that was exactly my thought as well. having a tool (even if it starts out just for operators!) that async-deletes a container or some prefix within a container seems valuable separate from the s3api change19:14
timburke*and* it side-steps the "but what if you delete the now-empty container" problem because *it doesn't need to update the container right away* -- let the expirer's delete take care of getting listings cleaned up19:15
claygthat would be sick!19:15
claygoh right, yeah 👍19:16
claygbut maybe still use merge items to bulk insert the deletes?19:16
timburkei think i feel justified in still having a new UPDATE verb for that... since the idea would be that we some day want it as a client-facing thing...19:17
claygis update merge items?  i thought... *not* client thing - just like... *InternalClient* ???19:21
timburkei did realize that there are some more funny things going on with storage policies though... like, what happens if you get a split-brain in the expirer queue container? the reconciler will never be able to sort those rows out because there are no actual objects to move...19:21
claygso the reconciler will recocile tombstones - presumably it could reconcile pending delete rows as well.. eventually19:22
timburkeyeah, for a start. and since InternalClient is a type of proxy, i'd still want UPDATE19:22
timburkeat some point, i'd want this (async delete) to be more than an operator feature19:22
*** mathiasb has joined #openstack-swift19:22
claygbut that's not what'd we'd be doing right away right - in this change we're suggesting the expirer just cleans up the bulk-inserted rows?19:22
*** mikecmpbll has joined #openstack-swift19:23
claygyeah that works, UPDATE as internal client only verb to facilitate stuffing rows into expirer queue - sounds like a great operator feature!!!19:23
timburkeyeah, so, three patches: 1, add ability to bulk-update rows from an InternalClient. this should probably be a new UPDATE verb, as i don't think IC knows about replication interfaces19:24
timburke2, operator feature to take a container and a (possibly empty) prefix, spin up an IC, list everything in that prefix and drop it in an expiring queue. as part of it, expirer learns how to delete without an X-Delete-At19:25
timburke3, s3api patch19:26
timburkewhich still may or may not end up landing19:26
claygyeah, part 3 is optional 🤣19:26
claygbut the first two sound like nothing but sexy operator hawtness19:27
claygyou could maybe squash 1 & 2 - depending on volume19:27
timburkeoh, and 2b, be able to mark a container as getting reaped so it'll 410 via proxy-server and get cleaned up in the background. definitely needs more thought/design though19:28
timburke2 puts us a long way toward being able to do that though. just gotta figure out how to make it flip to *actually* getting deleted...19:29
timburkei seem to remember us getting into trouble with sharding, deleting containers, and having them lose all their metadata... so i think it'd need to be a sysmeta flag or something...19:30
openstackgerritPete Zaitcev proposed openstack/swift master: py3: slo  https://review.opendev.org/65050719:33
claygyeah I think problem of 2b is really more like a pre-req for 3, like a 3a, and 3b is something about accounting, maybe the deleted=2 flag19:34
*** zaitcev has joined #openstack-swift19:35
*** ChanServ sets mode: +v zaitcev19:35
*** BjoernT has quit IRC19:45
timburkeclayg, nah, 2b and 3 are separate -- the s3api takes a list of files to ax, not marking the whole bucket deleted (or even some prefix)19:47
clayg@timburke sure, but it also marks them as "already deleted" in the container - which is really where all the trouble starts 🤷‍♂️19:51
timburkei'm thinking 2b doesn't need to do that though19:52
timburkeaccount reaper sure doesn't do it19:53
claygah... ok... I see your strategy I think then - as long as comes separate/after 2 I'm happy 😁19:54
claygcause like you could say "empty prefix async delete" and still just have the container there until it finishes processing (and if you add a *new* object you can't reap the container, and if you overwrite a name that hasn't been reaped yet it will get gobbled)19:55
claygbut being able to lock our writes while reaping (in the special case) would be a cool next step (do I have it right?)19:56
*** e0ne has joined #openstack-swift20:09
*** e0ne has quit IRC20:10
*** NM has joined #openstack-swift20:25
*** zaitcev_ has joined #openstack-swift20:53
*** ChanServ sets mode: +v zaitcev_20:53
*** zaitcev has quit IRC20:56
alecuyerkota_:  the functests have passed! but "retry limit" in swift-dvsm-functional, I'll take a look tomorrow21:10
*** dasp has quit IRC21:52
*** zaitcev_ has quit IRC21:54
*** zaitcev_ has joined #openstack-swift22:06
*** ChanServ sets mode: +v zaitcev_22:06
*** NM has quit IRC22:09
*** NM has joined #openstack-swift22:16
*** NM has quit IRC22:16
*** nottrobin has quit IRC22:35
*** nottrobin has joined #openstack-swift22:36
mattoliveraumorning22:52
*** tkajinam has joined #openstack-swift23:03
*** samueldmq has quit IRC23:31
*** zaitcev_ has quit IRC23:51

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