| *** phuoc has joined #openstack-telemetry | 01:53 | |
| *** phuoc has joined #openstack-telemetry | 01:54 | |
| *** psachin has joined #openstack-telemetry | 02:42 | |
| *** threestrands has quit IRC | 02:57 | |
| *** rcernin has quit IRC | 04:18 | |
| *** rcernin has joined #openstack-telemetry | 04:18 | |
| *** hoonetorg has quit IRC | 04:31 | |
| *** hoonetorg has joined #openstack-telemetry | 05:57 | |
| *** hoonetorg has quit IRC | 06:14 | |
| *** hoonetorg has joined #openstack-telemetry | 06:31 | |
| *** shardy has joined #openstack-telemetry | 07:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/aodh master: Imported Translations from Zanata https://review.openstack.org/599301 | 07:09 |
|---|---|---|
| *** rcernin has quit IRC | 07:11 | |
| *** alexchadin has joined #openstack-telemetry | 07:24 | |
| *** witek has joined #openstack-telemetry | 07:28 | |
| *** piepmatz has joined #openstack-telemetry | 07:56 | |
| *** psachin has quit IRC | 07:57 | |
| *** cdent has joined #openstack-telemetry | 08:17 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer master: DNM! Replace deprecated snmp metrics https://review.openstack.org/585840 | 08:18 |
| *** pas-ha has joined #openstack-telemetry | 08:25 | |
| *** psachin has joined #openstack-telemetry | 08:41 | |
| *** r-mibu has joined #openstack-telemetry | 09:02 | |
| *** pcaruana has joined #openstack-telemetry | 09:09 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Cleanup tox.ini https://review.openstack.org/599335 | 09:55 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Factorize gabbi tests https://review.openstack.org/599336 | 09:55 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Remove directory setup complexity https://review.openstack.org/599337 | 09:55 |
| *** shardy has quit IRC | 10:03 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Use gabbi yaml file loader https://review.openstack.org/599344 | 10:26 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Factorize gabbi tests https://review.openstack.org/599336 | 10:32 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Remove directory setup complexity https://review.openstack.org/599337 | 10:32 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Raise error if gabbits directory is empty https://review.openstack.org/599345 | 10:32 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Use gabbi yaml file loader https://review.openstack.org/599344 | 10:34 |
| *** alexchadin has quit IRC | 10:37 | |
| *** alexchadin has joined #openstack-telemetry | 10:41 | |
| *** alexchadin has quit IRC | 10:52 | |
| *** mattia has joined #openstack-telemetry | 10:54 | |
| sileht | cdent, hi | 11:41 |
| sileht | cdent, did http://logs.openstack.org/44/599344/2/check/telemetry-dsvm-integration/bd2707f/job-output.txt.gz#_2018-09-03_11_07_25_155909 error ring a bell to you ? | 11:41 |
| * cdent looks | 11:42 | |
| sileht | cdent, looks like the constructed test name is unicode instead of str | 11:42 |
| sileht | cdent, we pass a str as base name here: https://github.com/openstack/telemetry-tempest-plugin/blob/master/telemetry_tempest_plugin/scenario/test_telemetry_integration.py#L102 | 11:42 |
| sileht | cdent, so I'm guessing this have something to do with name picked from the yaml file | 11:43 |
| cdent | jd_ and I talked about that a couple days ago and it didn't ring a bell and gabbi hasn't changed recently so we figured it was either somewhere in the telemetry tests themselves or in tempest | 11:43 |
| cdent | in that context is argument 1 mcs or name? If it is name, did I see that you recently changed the yaml reading to be more ut8 savvy? | 11:44 |
| sileht | cdent, it's name | 11:44 |
| sileht | cdent, mcs is not a str but the type class itself | 11:44 |
| sileht | cdent, we have the issue since a saome days and we didn't change anymore around telemetry | 11:45 |
| cdent | It's hard to be clear about what's going on in those tests because you've done quite a bit of entering gabbi a bit deeper than gabbi does itself | 11:45 |
| sileht | cdent, to add some comlexity, it work locally for me | 11:45 |
| cdent | oh great | 11:46 |
| cdent | not a different python version is it? | 11:46 |
| sileht | cdent, both use 2.7 | 11:46 |
| cdent | oh, I wonder if LOCALE is different somehow from machine to machine when reading the yaml | 11:46 |
| sileht | cdent, yeah that is my next testing step | 11:48 |
| cdent | sileht: If you figure out that there's a bug in gabbi, please report it. It wouldn't surprise me. | 11:49 |
| sileht | cdent, it's blocking our gate, so you will even have a fix for it :) | 11:50 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 12:13 |
| *** Bhujay has joined #openstack-telemetry | 12:14 | |
| piepmatz | sileht: cdent: I am having this str vs unicode problem for a couple of weeks in my telemetry setup. but I didn't figure out why it didn't hit you upstream | 12:39 |
| piepmatz | my dirty hack to work around this issue was casting https://github.com/cdent/gabbi/blob/master/gabbi/suitemaker.py#L78 to str. I'd guess the issue would be absent when running tempest and gabbi with python 3, but I didn't try | 12:43 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 12:43 |
| cdent | piepmatz: any guess about what's in the telemetry tests that's not in the gabbi tests, such that gabbi isn't being tickled by this? | 12:44 |
| cdent | Or do you think that gabbi would be showing it too, if it had the right locale setup? | 12:44 |
| *** noonedeadpunk has joined #openstack-telemetry | 12:49 | |
| *** pcaruana has quit IRC | 12:52 | |
| *** shardy has joined #openstack-telemetry | 12:55 | |
| piepmatz | cdent: no idea, sorry | 12:59 |
| cdent | no problem, just curious | 13:00 |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 13:09 |
| *** psachin has quit IRC | 13:34 | |
| *** Bhujay has quit IRC | 13:58 | |
| *** r-daneel has joined #openstack-telemetry | 14:09 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 14:10 |
| *** r-daneel has quit IRC | 14:28 | |
| *** r-daneel has joined #openstack-telemetry | 14:30 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 15:08 |
| *** r-mibu has quit IRC | 15:21 | |
| *** purplerbot has quit IRC | 16:14 | |
| *** purplerbot has joined #openstack-telemetry | 16:15 | |
| *** tonyb has quit IRC | 16:31 | |
| *** piepmatz has quit IRC | 16:31 | |
| *** dhellmann has joined #openstack-telemetry | 16:38 | |
| *** dhellmann has quit IRC | 16:39 | |
| *** dhellmann has joined #openstack-telemetry | 16:39 | |
| *** patrickeast has left #openstack-telemetry | 16:42 | |
| *** dhellmann has quit IRC | 16:44 | |
| *** dhellmann has joined #openstack-telemetry | 16:45 | |
| *** dhellmann has quit IRC | 16:54 | |
| *** dhellmann has joined #openstack-telemetry | 16:54 | |
| *** r-daneel has quit IRC | 17:13 | |
| *** r-daneel has joined #openstack-telemetry | 17:13 | |
| *** dave-mccowan has quit IRC | 17:47 | |
| *** vkmc has quit IRC | 19:10 | |
| *** csatari has quit IRC | 19:10 | |
| *** awestin1 has quit IRC | 19:10 | |
| *** AndyWojo has quit IRC | 19:10 | |
| *** vkmc has joined #openstack-telemetry | 19:14 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 19:58 |
| cdent | sileht: did the issue with gabbi clear up? | 20:02 |
| sileht | cdent, no | 20:02 |
| cdent | :( | 20:02 |
| sileht | cdent, I just confirmed that yaml.safe_load returns unicode instead of str in gate | 20:03 |
| cdent | I guess, given that that's a thing that can happen, we need (in gabbi itself) to de-unicode test names (in only py 2?) before calling TestBuilder | 20:04 |
| sileht | cdent, we could do that | 20:05 |
| sileht | cdent, I known that pyyaml return unicode if it find unicode char or a \xXXXX | 20:06 |
| sileht | but by default it's str | 20:06 |
| *** cdent has quit IRC | 20:28 | |
| *** cdent has joined #openstack-telemetry | 20:43 | |
| *** cdent has quit IRC | 20:43 | |
| *** tonyb has joined #openstack-telemetry | 20:51 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 21:16 |
| *** hoonetorg has quit IRC | 22:29 | |
| *** rcernin has joined #openstack-telemetry | 22:37 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/telemetry-tempest-plugin master: Reorganize scenario testing. https://review.openstack.org/599336 | 22:50 |
| *** threestrands has joined #openstack-telemetry | 23:03 | |
| *** threestrands has quit IRC | 23:03 | |
| *** threestrands has joined #openstack-telemetry | 23:04 | |
| *** threestrands has quit IRC | 23:05 | |
| *** threestrands has joined #openstack-telemetry | 23:05 | |
| *** threestrands has quit IRC | 23:06 | |
| *** threestrands has joined #openstack-telemetry | 23:07 | |
| *** threestrands has quit IRC | 23:08 | |
| *** r-daneel has quit IRC | 23:15 | |
| *** dave-mccowan has joined #openstack-telemetry | 23:17 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!