Monday, 2018-12-03

*** psachin has joined #openstack-swift02:42
*** threestrands has joined #openstack-swift05:21
*** threestrands has quit IRC05:21
*** gkadam has joined #openstack-swift05:45
kota_hello06:37
kota_mattoliverau: :)06:37
*** rcernin has quit IRC06:57
*** pcaruana has joined #openstack-swift07:25
*** e0ne has joined #openstack-swift08:04
*** hseipp has joined #openstack-swift08:05
*** mahatic has quit IRC08:10
*** ccamacho has joined #openstack-swift08:45
*** cschwede has quit IRC11:25
*** SkyRocknRoll has joined #openstack-swift11:41
openstackgerritCyril Roelandt proposed openstack/swift master: Py3: Fix test_memcache  https://review.openstack.org/62155211:52
*** mpasserini has joined #openstack-swift12:31
openstackgerritCyril Roelandt proposed openstack/swift master: Py3: Fix two unbound variables  https://review.openstack.org/62156513:01
*** mpasserini has quit IRC13:05
*** mpasserini has joined #openstack-swift13:06
*** SkyRocknRoll has quit IRC13:09
*** ybunker has joined #openstack-swift13:12
*** mahatic has joined #openstack-swift13:25
*** ChanServ sets mode: +v mahatic13:25
*** cschwede has joined #openstack-swift13:43
*** ChanServ sets mode: +v cschwede13:43
*** two_tired has joined #openstack-swift14:02
*** mpasserini has quit IRC14:13
*** psachin has quit IRC14:24
*** mpasserini has joined #openstack-swift14:41
*** mpasserini has quit IRC15:08
*** mpasserini has joined #openstack-swift15:08
*** mpasserini has quit IRC15:16
*** mpasserini has joined #openstack-swift15:20
*** two_tired has quit IRC15:27
*** itlinux has quit IRC15:34
*** gyee has joined #openstack-swift16:04
*** mpasserini has quit IRC16:23
*** SkyRocknRoll has joined #openstack-swift16:25
*** itlinux has joined #openstack-swift16:33
*** itlinux has quit IRC16:33
*** itlinux has joined #openstack-swift16:33
*** ybunker has quit IRC16:43
*** ybunker has joined #openstack-swift16:44
*** e0ne has quit IRC16:48
*** ybunker has quit IRC16:49
*** itlinux_ has joined #openstack-swift16:50
*** itlinux has quit IRC16:54
*** itlinux_ has quit IRC16:55
*** itlinux has joined #openstack-swift16:56
*** ybunker has joined #openstack-swift17:19
*** zaitcev has joined #openstack-swift17:22
*** ChanServ sets mode: +v zaitcev17:22
*** Steap has joined #openstack-swift17:26
*** Steap has left #openstack-swift17:26
*** hseipp has quit IRC17:26
*** Steap has joined #openstack-swift17:27
SteapHey! Is the upstream CI broken?17:27
timburkeSteap: i was noticing that the in-process func tests all started failing, all the time, recently... that was going to be the first thing i look into this morning :-)17:34
Steaptimburke: ok :) Thanks a lot for working on this!17:42
timburkeoh, jeez... http://logs.openstack.org/77/621377/1/check/swift-tox-func/4060ce4/job-output.txt.gz#_2018-12-01_21_01_19_926639 can't be a good sign...17:42
notmynamegood morning17:46
timburkeso it looks like the switch from pass to fail was due to an upgrade of stestr from 2.1.0 to 2.2.0? seems most likely, anyway...17:59
timburkebut the BIG problem is that tests were bombing out exactly as they are now but still considered *passing*!17:59
timburkethat's maybe got something to do with the upgrade of PasteDeploy from 1.5.2 to 2.0.0?18:00
*** e0ne has joined #openstack-swift18:03
*** SkyRocknRoll has quit IRC18:29
*** e0ne has quit IRC19:04
*** e0ne has joined #openstack-swift19:17
openstackgerritTim Burke proposed openstack/swift master: DNM: debugging func test failures  https://review.openstack.org/62167819:21
openstackgerritBulat Gaifullin proposed openstack/swift master: Check that response is json in symlink proxy  https://review.openstack.org/62167919:28
*** e0ne has quit IRC19:51
*** e0ne has joined #openstack-swift19:52
*** ybunker has quit IRC19:59
*** e0ne has quit IRC20:02
*** e0ne has joined #openstack-swift20:03
*** e0ne has quit IRC20:10
*** e0ne has joined #openstack-swift20:17
*** e0ne has quit IRC20:21
timburkei think maybe some combination of https://bitbucket.org/cdent/pastedeploy/commits/23c60de073031bb57e06b3353aaeb7d5067f89de and https://review.openstack.org/#/c/573181/2/test/functional/__init__.py@541 is fouling us up? why did we add some dummy __file__ value??20:35
patchbotpatch 573181 - swift - Allow domain remap storage_domain to be configured... (MERGED) - 2 patch sets20:35
openstackgerritTim Burke proposed openstack/swift master: DNM: debugging func test failures  https://review.openstack.org/62167820:36
*** e0ne has joined #openstack-swift20:42
*** e0ne has quit IRC20:42
openstackgerritThiago da Silva proposed openstack/swift master: Support long-running multipart uploads  https://review.openstack.org/57581821:25
zaitcevI remember we used __file__ somewhere21:28
openstackgerritTim Burke proposed openstack/swift master: Fix in-process func tests to work with newer PasteDeploy  https://review.openstack.org/62167821:48
openstackgerritTim Burke proposed openstack/swift master: DNM: test with older PasteDeploy  https://review.openstack.org/62170821:48
zaitcevhuh, is that all21:48
zaitcevhmm21:50
zaitcevWell, even if we stuff this through the CI, does it guarantee that e.g. DLO work, is my question.21:50
zaitcevOuch, __file__ can also be a file-like.21:51
zaitcevokay, whatever21:51
timburkei think we're ok there... the swift-dsvm-functional jobs were passing the whole time21:52
*** rcernin has joined #openstack-swift22:18
*** pcaruana has quit IRC22:18
*** ChanServ changes topic to "OpenStack Swift object storage | Logs: http://eavesdrop.openstack.org/irclogs/%23openstack-swift/ | Meetings: https://wiki.openstack.org/wiki/Meetings/Swift | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Ideas: https://wiki.openstack.org/wiki/Swift/ideas"22:36
*** itlinux has quit IRC22:38
timburke:-( our lower-constraints job isn't testing what it ought to test23:02
timburke+23:02
timburkePasteDeploy should get down-rev'ed to 1.3.3, but if it *did*, we'd see errors like http://logs.openstack.org/08/621708/1/check/swift-tox-py27/b191aa4/job-output.txt.gz#_2018-12-03_22_28_36_297326 in our lower-constraints job23:04
openstackgerritTim Burke proposed openstack/swift master: DNM: test with older PasteDeploy  https://review.openstack.org/62170823:05
openstackgerritTim Burke proposed openstack/swift master: Fix unit tests on old PasteDeploy  https://review.openstack.org/62172323:05
*** irclogbot_2 has joined #openstack-swift23:10
notmynameFYI: https://bugs.launchpad.net/swift/+bug/180650023:14
openstackLaunchpad bug 1806500 in OpenStack Object Storage (swift) "sharder fails when drive is unmounted" [High,Confirmed]23:14
notmynametimburke and I found this and I'm working on a patch now23:14
mattoliveraumorning23:24
mattoliverauahh yeah, let's fix that. go notmyname go :)23:24
openstackgerritJohn Dickinson proposed openstack/swift master: fixed _check_node() in the container sharder  https://review.openstack.org/62172723:47
*** gkadam has quit IRC23:48
notmynameclayg: timburke: mattoliverau: there's the patch ^^. I left an inline comment about some questions I have about some of the test changes23:50
timburkenotmyname: the change in test_one_shard_cycle seems...weird to me23:51
notmynameyeah, exactly ;-)23:51
notmynameI left a comment. reload if you can't see it yet23:51
notmynamemostly I didn't like the test_one_cycle combined with the "assume everything is mounted" override to _check_node(). seems a bit unrealistic23:52
notmynameie it's fine to add a test to make sure we call a method correctly. but seems like there should be /something/ for how it should be used in the real shard cycle23:53
timburkeoh... is sharder.ring already some FakeRing thing? hmm...23:57

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