*** psachin has joined #openstack-swift | 02:42 | |
*** threestrands has joined #openstack-swift | 05:21 | |
*** threestrands has quit IRC | 05:21 | |
*** gkadam has joined #openstack-swift | 05:45 | |
kota_ | hello | 06:37 |
---|---|---|
kota_ | mattoliverau: :) | 06:37 |
*** rcernin has quit IRC | 06:57 | |
*** pcaruana has joined #openstack-swift | 07:25 | |
*** e0ne has joined #openstack-swift | 08:04 | |
*** hseipp has joined #openstack-swift | 08:05 | |
*** mahatic has quit IRC | 08:10 | |
*** ccamacho has joined #openstack-swift | 08:45 | |
*** cschwede has quit IRC | 11:25 | |
*** SkyRocknRoll has joined #openstack-swift | 11:41 | |
openstackgerrit | Cyril Roelandt proposed openstack/swift master: Py3: Fix test_memcache https://review.openstack.org/621552 | 11:52 |
*** mpasserini has joined #openstack-swift | 12:31 | |
openstackgerrit | Cyril Roelandt proposed openstack/swift master: Py3: Fix two unbound variables https://review.openstack.org/621565 | 13:01 |
*** mpasserini has quit IRC | 13:05 | |
*** mpasserini has joined #openstack-swift | 13:06 | |
*** SkyRocknRoll has quit IRC | 13:09 | |
*** ybunker has joined #openstack-swift | 13:12 | |
*** mahatic has joined #openstack-swift | 13:25 | |
*** ChanServ sets mode: +v mahatic | 13:25 | |
*** cschwede has joined #openstack-swift | 13:43 | |
*** ChanServ sets mode: +v cschwede | 13:43 | |
*** two_tired has joined #openstack-swift | 14:02 | |
*** mpasserini has quit IRC | 14:13 | |
*** psachin has quit IRC | 14:24 | |
*** mpasserini has joined #openstack-swift | 14:41 | |
*** mpasserini has quit IRC | 15:08 | |
*** mpasserini has joined #openstack-swift | 15:08 | |
*** mpasserini has quit IRC | 15:16 | |
*** mpasserini has joined #openstack-swift | 15:20 | |
*** two_tired has quit IRC | 15:27 | |
*** itlinux has quit IRC | 15:34 | |
*** gyee has joined #openstack-swift | 16:04 | |
*** mpasserini has quit IRC | 16:23 | |
*** SkyRocknRoll has joined #openstack-swift | 16:25 | |
*** itlinux has joined #openstack-swift | 16:33 | |
*** itlinux has quit IRC | 16:33 | |
*** itlinux has joined #openstack-swift | 16:33 | |
*** ybunker has quit IRC | 16:43 | |
*** ybunker has joined #openstack-swift | 16:44 | |
*** e0ne has quit IRC | 16:48 | |
*** ybunker has quit IRC | 16:49 | |
*** itlinux_ has joined #openstack-swift | 16:50 | |
*** itlinux has quit IRC | 16:54 | |
*** itlinux_ has quit IRC | 16:55 | |
*** itlinux has joined #openstack-swift | 16:56 | |
*** ybunker has joined #openstack-swift | 17:19 | |
*** zaitcev has joined #openstack-swift | 17:22 | |
*** ChanServ sets mode: +v zaitcev | 17:22 | |
*** Steap has joined #openstack-swift | 17:26 | |
*** Steap has left #openstack-swift | 17:26 | |
*** hseipp has quit IRC | 17:26 | |
*** Steap has joined #openstack-swift | 17:27 | |
Steap | Hey! Is the upstream CI broken? | 17:27 |
timburke | Steap: 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 |
Steap | timburke: ok :) Thanks a lot for working on this! | 17:42 |
timburke | oh, 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 |
notmyname | good morning | 17:46 |
timburke | so 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 |
timburke | but the BIG problem is that tests were bombing out exactly as they are now but still considered *passing*! | 17:59 |
timburke | that'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-swift | 18:03 | |
*** SkyRocknRoll has quit IRC | 18:29 | |
*** e0ne has quit IRC | 19:04 | |
*** e0ne has joined #openstack-swift | 19:17 | |
openstackgerrit | Tim Burke proposed openstack/swift master: DNM: debugging func test failures https://review.openstack.org/621678 | 19:21 |
openstackgerrit | Bulat Gaifullin proposed openstack/swift master: Check that response is json in symlink proxy https://review.openstack.org/621679 | 19:28 |
*** e0ne has quit IRC | 19:51 | |
*** e0ne has joined #openstack-swift | 19:52 | |
*** ybunker has quit IRC | 19:59 | |
*** e0ne has quit IRC | 20:02 | |
*** e0ne has joined #openstack-swift | 20:03 | |
*** e0ne has quit IRC | 20:10 | |
*** e0ne has joined #openstack-swift | 20:17 | |
*** e0ne has quit IRC | 20:21 | |
timburke | i 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 |
patchbot | patch 573181 - swift - Allow domain remap storage_domain to be configured... (MERGED) - 2 patch sets | 20:35 |
openstackgerrit | Tim Burke proposed openstack/swift master: DNM: debugging func test failures https://review.openstack.org/621678 | 20:36 |
*** e0ne has joined #openstack-swift | 20:42 | |
*** e0ne has quit IRC | 20:42 | |
openstackgerrit | Thiago da Silva proposed openstack/swift master: Support long-running multipart uploads https://review.openstack.org/575818 | 21:25 |
zaitcev | I remember we used __file__ somewhere | 21:28 |
openstackgerrit | Tim Burke proposed openstack/swift master: Fix in-process func tests to work with newer PasteDeploy https://review.openstack.org/621678 | 21:48 |
openstackgerrit | Tim Burke proposed openstack/swift master: DNM: test with older PasteDeploy https://review.openstack.org/621708 | 21:48 |
zaitcev | huh, is that all | 21:48 |
zaitcev | hmm | 21:50 |
zaitcev | Well, even if we stuff this through the CI, does it guarantee that e.g. DLO work, is my question. | 21:50 |
zaitcev | Ouch, __file__ can also be a file-like. | 21:51 |
zaitcev | okay, whatever | 21:51 |
timburke | i think we're ok there... the swift-dsvm-functional jobs were passing the whole time | 21:52 |
*** rcernin has joined #openstack-swift | 22:18 | |
*** pcaruana has quit IRC | 22: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 IRC | 22:38 | |
timburke | :-( our lower-constraints job isn't testing what it ought to test | 23:02 |
timburke | + | 23:02 |
timburke | PasteDeploy 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 job | 23:04 |
openstackgerrit | Tim Burke proposed openstack/swift master: DNM: test with older PasteDeploy https://review.openstack.org/621708 | 23:05 |
openstackgerrit | Tim Burke proposed openstack/swift master: Fix unit tests on old PasteDeploy https://review.openstack.org/621723 | 23:05 |
*** irclogbot_2 has joined #openstack-swift | 23:10 | |
notmyname | FYI: https://bugs.launchpad.net/swift/+bug/1806500 | 23:14 |
openstack | Launchpad bug 1806500 in OpenStack Object Storage (swift) "sharder fails when drive is unmounted" [High,Confirmed] | 23:14 |
notmyname | timburke and I found this and I'm working on a patch now | 23:14 |
mattoliverau | morning | 23:24 |
mattoliverau | ahh yeah, let's fix that. go notmyname go :) | 23:24 |
openstackgerrit | John Dickinson proposed openstack/swift master: fixed _check_node() in the container sharder https://review.openstack.org/621727 | 23:47 |
*** gkadam has quit IRC | 23:48 | |
notmyname | clayg: timburke: mattoliverau: there's the patch ^^. I left an inline comment about some questions I have about some of the test changes | 23:50 |
timburke | notmyname: the change in test_one_shard_cycle seems...weird to me | 23:51 |
notmyname | yeah, exactly ;-) | 23:51 |
notmyname | I left a comment. reload if you can't see it yet | 23:51 |
notmyname | mostly I didn't like the test_one_cycle combined with the "assume everything is mounted" override to _check_node(). seems a bit unrealistic | 23:52 |
notmyname | ie 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 cycle | 23:53 |
timburke | oh... 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!