zaitcev | mattoliverau: I dropped _path from https://review.openstack.org/570320 | 00:27 |
---|---|---|
patchbot | patch 570320 - swift - py3: port the container sharder - 14 patch sets | 00:27 |
*** gyee has quit IRC | 00:54 | |
*** fyx has joined #openstack-swift | 00:59 | |
*** coreycb has joined #openstack-swift | 00:59 | |
clayg | timburke: the test_locked_container_dbs has a 15s timeout - which might be related to the default 10s node timeout :'( | 01:05 |
clayg | this could be more complicated than first assumed | 01:06 |
*** mikecmpbll has quit IRC | 01:23 | |
*** thyrst has joined #openstack-swift | 01:25 | |
timburke | oh... hey... zaitcev: look familiar? http://logs.openstack.org/17/636217/2/experimental/swift-tox-py27-centos-7/4fd5053/job-output.txt.gz#_2019-02-12_00_12_39_858728 | 02:06 |
zaitcev | timburke, a sec | 02:06 |
timburke | i guess there's something about centos in particular that doesn't like the heartbeating? | 02:06 |
zaitcev | timburke: yes. There should be 6 of them. | 02:06 |
zaitcev | FAILED (failures=6) | 02:07 |
timburke | i wonder if it's tied to eventlet version... | 02:07 |
mattoliverau | zaitcev: cool, ta. I'll take a look | 02:26 |
zaitcev | woa, I'm amazed things actually work | 03:28 |
zaitcev | I found why a ton of tests created /var/tmp/tmp | 03:29 |
zaitcev | If we were running tests in parallel, it could be a source of phantom problems when two of them created "container.db" there. | 03:30 |
openstackgerrit | Pete Zaitcev proposed openstack/swift master: Leave less garbage in /var/tmp https://review.openstack.org/636262 | 03:50 |
kota_ | I'm back to online after short holidays. I'll catch up losf feature branch and recent upstreams. | 04:14 |
kota_ | notmyname: added a few items to https://etherpad.openstack.org/p/m06METHBW7 | 04:14 |
notmyname | kota_: thanks. unfortunately, I've already sent my response without your suggestions. but thank you for taking the time to look | 04:51 |
*** early has quit IRC | 05:14 | |
*** early has joined #openstack-swift | 05:22 | |
*** early has quit IRC | 06:05 | |
*** early has joined #openstack-swift | 06:13 | |
*** e0ne has joined #openstack-swift | 06:24 | |
timburke | zaitcev: hahahaha... i laugh because it feels better than crying... | 06:26 |
*** e0ne has quit IRC | 06:35 | |
*** ccamacho has quit IRC | 06:49 | |
*** e0ne has joined #openstack-swift | 07:12 | |
*** e0ne has quit IRC | 07:16 | |
*** e0ne has joined #openstack-swift | 07:24 | |
*** e0ne has quit IRC | 07:32 | |
*** e0ne has joined #openstack-swift | 07:42 | |
*** ccamacho has joined #openstack-swift | 07:58 | |
*** ccamacho has quit IRC | 07:59 | |
*** ccamacho has joined #openstack-swift | 07:59 | |
*** gkadam has joined #openstack-swift | 08:08 | |
*** hseipp has joined #openstack-swift | 08:10 | |
*** gkadam_ has joined #openstack-swift | 08:15 | |
*** gkadam has quit IRC | 08:17 | |
*** tkajinam has quit IRC | 08:29 | |
kota_ | notmyname: i see, sorry for my late response. | 08:29 |
*** e0ne has quit IRC | 08:35 | |
*** e0ne has joined #openstack-swift | 08:37 | |
*** e0ne has quit IRC | 08:38 | |
openstackgerrit | Merged openstack/swift master: encryption: Stop being cutesy with os.path.join() https://review.openstack.org/635380 | 08:42 |
*** e0ne has joined #openstack-swift | 08:46 | |
*** e0ne has quit IRC | 08:48 | |
*** e0ne has joined #openstack-swift | 08:54 | |
*** e0ne has quit IRC | 08:56 | |
*** mikecmpbll has joined #openstack-swift | 09:00 | |
*** jistr_ is now known as jistr | 09:09 | |
*** e0ne has joined #openstack-swift | 09:25 | |
*** mikecmpb_ has joined #openstack-swift | 11:21 | |
*** mikecmpbll has quit IRC | 11:23 | |
*** mvkr has quit IRC | 11:44 | |
*** coreycb has quit IRC | 12:36 | |
*** coreycb has joined #openstack-swift | 12:37 | |
*** mvkr has joined #openstack-swift | 12:46 | |
*** mvkr has quit IRC | 13:08 | |
*** openstackgerrit has quit IRC | 13:37 | |
*** mvkr has joined #openstack-swift | 13:40 | |
*** e0ne has joined #openstack-swift | 13:56 | |
*** e0ne has quit IRC | 14:44 | |
*** gkadam_ has quit IRC | 14:54 | |
*** gkadam has joined #openstack-swift | 14:54 | |
*** ianychoi has quit IRC | 15:07 | |
*** ianychoi has joined #openstack-swift | 15:07 | |
*** e0ne has joined #openstack-swift | 15:09 | |
*** hseipp has quit IRC | 15:12 | |
*** openstackgerrit has joined #openstack-swift | 15:27 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Increase node_timeout in gate https://review.openstack.org/636217 | 15:28 |
*** zaitcev has quit IRC | 15:31 | |
*** e0ne has quit IRC | 15:33 | |
*** hseipp has joined #openstack-swift | 15:37 | |
*** e0ne has joined #openstack-swift | 15:41 | |
*** gkadam has quit IRC | 16:09 | |
*** gkadam has joined #openstack-swift | 16:14 | |
*** ccamacho has quit IRC | 16:23 | |
*** ccamacho has joined #openstack-swift | 16:24 | |
*** hseipp has quit IRC | 16:29 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Increase node_timeout in gate https://review.openstack.org/636217 | 16:41 |
*** mvkr has quit IRC | 16:44 | |
*** ccamacho has quit IRC | 16:53 | |
notmyname | good morning | 17:11 |
*** mvkr has joined #openstack-swift | 17:16 | |
*** gkadam has quit IRC | 17:19 | |
*** zaitcev has joined #openstack-swift | 17:19 | |
*** ChanServ sets mode: +v zaitcev | 17:19 | |
*** e0ne has quit IRC | 17:27 | |
*** spsurya has quit IRC | 17:40 | |
*** e0ne has joined #openstack-swift | 17:43 | |
*** gyee has joined #openstack-swift | 17:47 | |
*** mikecmpb_ has quit IRC | 18:05 | |
clayg | git so I finally got the probe-tests passing with the 20s node timeout and I get a "unittest" popped because the in process container server's are taking > 10s to make a new sqlite db | 18:25 |
clayg | *greater than 10s* - we don't have a bunch of replication going on in the background... there's not 10K other clients hammering those same disks. 10s is LONG TIME. Are we thinking about this the wrong the way? | 18:26 |
clayg | idk, I guess we don't really want tests to be latency sensitive, I don't see any harm in identifying tests that ARE latency sensitive and trying to make them less so... and we're not going to stop doing integration testing - so "less latency sensitive" might be a reasonable endeavor... 🤷♂️ | 18:28 |
*** mikecmpbll has joined #openstack-swift | 18:36 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Leave less garbage in /var/tmp https://review.openstack.org/636262 | 18:44 |
*** nguyenhai_ has joined #openstack-swift | 19:01 | |
*** supamatt has quit IRC | 19:04 | |
*** nguyenhai has quit IRC | 19:04 | |
*** dr_gogeta86 has quit IRC | 19:04 | |
*** dr_gogeta86 has joined #openstack-swift | 19:05 | |
*** zaitcev has quit IRC | 19:22 | |
*** zaitcev has joined #openstack-swift | 19:23 | |
*** ChanServ sets mode: +v zaitcev | 19:23 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Fix decryption for broken objects https://review.openstack.org/633671 | 20:12 |
*** e0ne has quit IRC | 20:25 | |
*** e0ne has joined #openstack-swift | 20:48 | |
*** e0ne has quit IRC | 21:28 | |
timburke | clayg: i wonder how many of acoles's (surely good) ideas from https://review.openstack.org/#/c/181407/ have already landed now... | 21:37 |
patchbot | patch 181407 - swift - EC: Avoid conflicts when ssync'ing fragments - 12 patch sets | 21:37 |
zaitcev | Oh boy | 21:56 |
zaitcev | liberasurecode fails building on s390x | 21:56 |
zaitcev | I thought we fixed that a while ago. | 21:56 |
timburke | uh-oh. got a build log? | 21:56 |
zaitcev | it's truncated for some reason, so no | 21:57 |
zaitcev | I'll reproduce it and have a look... | 21:57 |
zaitcev | False alarm, x86_64 failed too. It's just the notification message focused on s390x for some reason. And the reason appears to be "BUILDSTDERR: cc1: all warnings being treated as errors" | 21:59 |
zaitcev | BUILDSTDERR: erasurecode_helpers.c: In function 'get_chksum': | 22:00 |
zaitcev | BUILDSTDERR: erasurecode_helpers.c:500:25: error: taking address of packed member of 'struct fragment_metadata' may result in an unaligned pointer value [-Werror=address-of-packed-member] | 22:00 |
zaitcev | BUILDSTDERR: 500 | return (uint32_t *) header->meta.chksum; | 22:00 |
zaitcev | I place the blame squarely onto Kevin and Tushar. | 22:00 |
zaitcev | Just what were they thinking! | 22:00 |
zaitcev | It's not like all this unaligned bullshit is harmless on x86 either, it's just the CPU fixes it up in hardware. | 22:01 |
timburke | :-( yeah, the struct alignment bit me when i was playing around with https://github.com/tipabu/erasurecode/, too... | 22:05 |
timburke | i'd love to overhaul the EC fragment headers -- both on-disk and in-memory. but i'm not familiar enough with how to go about maintaining backwards-compat for C libs... nor do i really know the degree to which we *need to*. like, clearly we need to still be able to read old data off disk -- but do i need to make sure that we're a drop-in replacement for everybody that may have linked against the old version? it'd probably be nice... bu | 22:10 |
timburke | t how many consumers of libec are there, really? | 22:10 |
zaitcev | One can rename all functions or use the link-time magic. | 22:11 |
zaitcev | But it's a lot of effort for an unclear benefit. | 22:11 |
*** mrjk_ has joined #openstack-swift | 22:52 | |
*** tkajinam has joined #openstack-swift | 23:16 | |
openstackgerrit | John Dickinson proposed openstack/swift stable/queens: changelog for queens backports https://review.openstack.org/636468 | 23:18 |
openstackgerrit | John Dickinson proposed openstack/swift stable/rocky: rocky stable backports https://review.openstack.org/636469 | 23:19 |
openstackgerrit | John Dickinson proposed openstack/swift stable/pike: pike stable backports https://review.openstack.org/636470 | 23:19 |
openstackgerrit | John Dickinson proposed openstack/swift stable/ocata: ocata stable backports https://review.openstack.org/636471 | 23:20 |
notmyname | bah! timburke just reminded me I need to reno-ify those :-( | 23:36 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!