*** ykarel|away is now known as ykarel | 03:34 | |
*** ysandeep|out is now known as ysandeep | 04:32 | |
*** ykarel_ is now known as ykarel | 07:47 | |
*** ysandeep is now known as ysandeep|lunch | 08:16 | |
*** ysandeep|lunch is now known as ysandeep | 08:37 | |
*** bauzas_ is now known as bauzas | 08:57 | |
*** rlandy is now known as rlandy|ruck | 10:32 | |
*** jcapitao is now known as jcapitao_lunch | 11:08 | |
arxcruz|ruck | fungi: hey, around? We are having failure in our molecule test, inherit from mol | 11:53 |
---|---|---|
arxcruz|ruck | fungi: the initial bug was https://bugs.launchpad.net/tripleo/+bug/1945682 | 11:53 |
arxcruz|ruck | you reverted the patch, but two weeks ago, you merged the patch again | 11:54 |
fungi | arxcruz|ruck: yes, after adding a test for the triggering condition observed and then fixing the bug with it which caused that problem via other patches https://review.opendev.org/812004 and https://review.opendev.org/812160 | 12:03 |
fungi | arxcruz|ruck: are you sure it's the same failure? have an example build result i can look at? | 12:04 |
arxcruz|ruck | fungi: https://logserver.rdoproject.org/56/36156/2/check/mol-get_hash-centos-7/27e6ccb/job-output.txt | 12:04 |
arxcruz|ruck | fungi: latest failure ^ | 12:04 |
arxcruz|ruck | fungi: this is centos-7 in python2 though | 12:04 |
arxcruz|ruck | centos8 jobs are working fine | 12:04 |
fungi | arxcruz|ruck: looks like a new, different error this time: "TypeError: initial_value must be unicode or None, not str" | 12:06 |
fungi | prior error was a pyyaml parsing issue | 12:07 |
arxcruz|ruck | fungi: hmmm do you have any idea? I'm trying to understad where this TypeError comes from, but i'm not being able to track the error | 12:07 |
arxcruz|ruck | just say it's in the main() of the module | 12:07 |
fungi | arxcruz|ruck: the backtrace says line 330, looks like the AttributeError exception handler added in https://review.opendev.org/806621 is insufficient for centos-7, i wonder if rh "backported" read_string() to the 2.7 on centos-7 | 12:13 |
fungi | my 2.7.18 doesn't have a ConfigParser.ConfigParser.read_string() method, fwiw | 12:15 |
fungi | so the call on that line _should_ be raising AttributeError | 12:16 |
fungi | File "/usr/lib/python2.7/site-packages/backports/configparser/__init__.py", line 793, in read_string | 12:17 |
fungi | the "backports" there makes me suspicious | 12:18 |
fungi | that function should not exist at all on 2.7 | 12:18 |
fungi | we can probably work around it by adding TypeError to the except on the next line | 12:19 |
rlandy|ruck | bhagyashris|rover: ^^ | 12:20 |
rlandy|ruck | fungi: thanks for the pointers | 12:20 |
bhagyashris|rover | rlandy|ruck, ack | 12:21 |
fungi | rlandy|ruck: bhagyashris|rover: i'm trying to reproduce it now with https://review.opendev.org/813620 | 12:28 |
fungi | hopefully that breaks, and then i can use it to test potential fixes | 12:28 |
rlandy|ruck | yep - only centos7 thing | 12:29 |
rlandy|ruck | we keep one test going on c7 for the last train support release there | 12:29 |
bhagyashris|rover | fungi, ack thanks | 12:29 |
*** jcapitao_lunch is now known as jcapitao | 12:30 | |
*** thelounge94 is now known as redrobot | 13:02 | |
*** redrobot is now known as thelounge94 | 13:04 | |
*** thelounge94 is now known as redrobot | 13:04 | |
*** atarakt is now known as nhicher | 13:07 | |
fungi | rlandy|ruck: bhagyashris|rover: i can't reproduce the failure with that, it gets past the sibling install task and into running stestr before it hits something unrelated which won't work on centos 7: https://zuul.opendev.org/t/zuul/build/2b2c15bea12a45e8b832d8e56dae79bd | 13:08 |
fungi | are your jobs maybe installing that configparser backport from somewhere? | 13:08 |
fungi | or could it be preinstalled on your job nodes as a dependency of some other package? | 13:20 |
rlandy|ruck | fungi: sorry - think bhagyashris|rover is going end_of_day - shortly ... will look more in a bit | 13:58 |
fungi | no worries, i just don't want to leave you broken any longer than necessary | 13:59 |
bhagyashris|rover | rlandy|ruck, i will check tomorrow sorry for inconvenience .... | 14:00 |
rlandy|ruck | bhagyashris|rover: np - feel better | 14:03 |
*** ykarel is now known as ykarel|away | 14:48 | |
*** ysandeep is now known as ysandeep|out | 16:18 | |
*** stevebak[m] is now known as stevebaker[m] | 18:11 | |
*** rlandy|ruck is now known as rlandy|ruck|bbl | 22:24 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!