| *** mkollaro has quit IRC | 00:02 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52450 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 00:07 |
|---|---|---|
| *** matsuhashi has joined #openstack-qa | 00:21 | |
| *** sthaha has joined #openstack-qa | 00:43 | |
| *** nosnos has joined #openstack-qa | 01:08 | |
| *** jamielennox is now known as jamielennox|away | 01:08 | |
| rockyg | sdague: Hope you don't mind -- I just added a blueprint to capture Salvatore Orlando's suggestion to rationalize the Tempest config variables/extenstions | 01:19 |
| *** SergeyLukjanov has quit IRC | 01:20 | |
| rockyg | mtreinish: and I just made you the drafter of the blueprint, putting the link to your review in the whiteboard: https://blueprints.launchpad.net/tempest/+spec/rationalize-tempest-config-variables | 01:28 |
| *** HenryG has joined #openstack-qa | 01:32 | |
| *** sdague has quit IRC | 01:36 | |
| *** SergeyLukjanov has joined #openstack-qa | 01:37 | |
| *** ivanzhu has joined #openstack-qa | 01:39 | |
| *** mriedem has quit IRC | 01:56 | |
| *** SergeyLukjanov has quit IRC | 01:59 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add BaseV2ComputeTest as the base class of nova v2 api tests https://review.openstack.org/52562 | 02:17 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add shelve/unshelve test of nova APIs https://review.openstack.org/49892 | 02:28 |
| openstackgerrit | wanghao proposed a change to openstack/tempest: add positive tests for volume https://review.openstack.org/50813 | 02:29 |
| openstackgerrit | Yaguang Tang proposed a change to openstack-dev/devstack: Update to use LibvirtGenericVIFDriver vif driver https://review.openstack.org/49875 | 02:31 |
| *** mlavalle has quit IRC | 02:37 | |
| *** prekarat has joined #openstack-qa | 02:37 | |
| *** prekarat has left #openstack-qa | 02:37 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the update extra route https://review.openstack.org/50880 | 02:40 |
| *** mriedem has joined #openstack-qa | 02:41 | |
| *** mriedem has quit IRC | 02:42 | |
| openstackgerrit | Sun Jing proposed a change to openstack-dev/devstack: Install Ironic Client by devstack https://review.openstack.org/52565 | 02:57 |
| *** Madan has joined #openstack-qa | 03:02 | |
| *** matsuhashi has quit IRC | 03:04 | |
| *** matsuhashi has joined #openstack-qa | 03:05 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/49660 failed tempest with an unrecognized error | 03:06 |
| *** matsuhashi has quit IRC | 03:09 | |
| *** ivanzhu has quit IRC | 03:29 | |
| *** masayukig has quit IRC | 03:34 | |
| *** matsuhashi has joined #openstack-qa | 03:39 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add BaseV2ComputeTest as the base class of nova v2 api tests https://review.openstack.org/52562 | 03:44 |
| *** matsuhashi has quit IRC | 03:49 | |
| *** matsuhashi has joined #openstack-qa | 03:49 | |
| *** matsuhas_ has joined #openstack-qa | 03:52 | |
| *** matsuhashi has quit IRC | 03:53 | |
| *** yfried_mob has joined #openstack-qa | 03:54 | |
| *** yfried_mob has quit IRC | 03:58 | |
| openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 04:06 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52287 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 and https://bugs.launchpad.net/bugs/1235437 and | 04:07 |
| *** sthaha has quit IRC | 04:33 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fix false negative, when HEAT_CREATE_TEST_IMAGE is unset https://review.openstack.org/51336 | 04:43 |
| openstackgerrit | A change was merged to openstack-dev/devstack: refine rabbit cleanup https://review.openstack.org/51631 | 04:43 |
| *** yfried has joined #openstack-qa | 04:47 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Nicira plugin: do not die if NVP gateway IP is missing https://review.openstack.org/47414 | 04:47 |
| *** yfried_mob has joined #openstack-qa | 04:52 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: baremetal: Allow BM_SECOND_MAC to be unset https://review.openstack.org/51209 | 04:54 |
| *** yfried|mob has joined #openstack-qa | 04:56 | |
| *** yfried_mob has quit IRC | 04:59 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: remove useless step in cleanup_rpc_backend https://review.openstack.org/51519 | 05:01 |
| *** sthaha has joined #openstack-qa | 05:04 | |
| *** sthaha has quit IRC | 05:04 | |
| *** sthaha has joined #openstack-qa | 05:04 | |
| *** SergeyLukjanov has joined #openstack-qa | 05:10 | |
| *** yfried|mob has quit IRC | 05:16 | |
| *** yfried_mob has joined #openstack-qa | 05:16 | |
| openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Changed the exception name https://review.openstack.org/51835 | 05:17 |
| *** shilpa_ has joined #openstack-qa | 05:23 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Forced isolation for the nova quota test https://review.openstack.org/52387 | 05:24 |
| *** afazekas has joined #openstack-qa | 05:24 | |
| *** giulivo has quit IRC | 05:29 | |
| openstackgerrit | Zhu Zhu proposed a change to openstack/tempest: Test for flavor-access-list with private/public flavor https://review.openstack.org/51188 | 05:44 |
| *** bdperkin has quit IRC | 05:45 | |
| *** bdperkin has joined #openstack-qa | 05:46 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Don't kill ceilometer prematurally in devstackgate https://review.openstack.org/51623 | 05:46 |
| *** dims has joined #openstack-qa | 05:51 | |
| *** prekarat has joined #openstack-qa | 05:58 | |
| openstackrecheck | openstack/python-novaclient change: https://review.openstack.org/52255 failed tempest with an unrecognized error | 06:00 |
| afazekas | mkoderer: https://review.openstack.org/#/c/52429/ | 06:00 |
| Anju | afazekas: so I need to rebase my patch gain | 06:01 |
| Anju | afazekas: so I need to rebase my patch again | 06:01 |
| Anju | afazekas: this is also in hold https://review.openstack.org/#/c/52429/ so / | 06:02 |
| Anju | afazekas: ^^^ | 06:03 |
| afazekas | Anju: Not easy to made a change in every file. | 06:03 |
| Anju | afazekas: yesterday I rebased my patch gain .. | 06:03 |
| *** yfried_mob has joined #openstack-qa | 06:03 | |
| Anju | and this file also contain a Duplicate exception | 06:04 |
| Anju | afazekas: https://review.openstack.org/#/c/51835/ mine is waiting :) | 06:05 |
| mkoderer | afazekas: LGTM +A | 06:08 |
| mkoderer | afazekas: in order to get https://review.openstack.org/#/c/51835/6 merged | 06:12 |
| mkoderer | should I -2 your https://review.openstack.org/#/c/52429/? | 06:12 |
| mkoderer | I don't see a good way to get it merged since it's a real moving target | 06:12 |
| afazekas | mkoderer: just two core reviewer needs to +2 after it is re-based | 06:15 |
| *** yfried_mob has quit IRC | 06:15 | |
| mkoderer | afazekas: yep so we need to find a good merge window | 06:15 |
| mkoderer | afazekas: maybe this weekend would be good | 06:16 |
| afazekas | mkoderer: you can review some other possible conflicting change than ask Anju to re-base the 51835, then we we wait 40 minute after to jenkis says +1 and +2 +2 +A :) | 06:18 |
| Anju | yes afazekas I can :) | 06:19 |
| Anju | afazekas: https://review.openstack.org/#/c/48578/16/tempest/api/image/v1/test_images_negative.py | 06:20 |
| *** dims has quit IRC | 06:21 | |
| mkoderer | afazekas: zuul isn't busy currently. I'll wait for 52429 and Anju please rebase it after it's merged | 06:23 |
| Anju | thanks afazekas , mkoderer | 06:23 |
| *** gszasz has joined #openstack-qa | 06:33 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 06:34 | |
| *** _SergeyLukjanov has quit IRC | 06:35 | |
| *** SergeyLukjanov has joined #openstack-qa | 06:38 | |
| openstackgerrit | A change was merged to openstack/tempest: server status remained in unexpected state https://review.openstack.org/52429 | 06:43 |
| *** KapilSaxena has joined #openstack-qa | 06:43 | |
| mkoderer | afazekas: anju will rebase now and then we can review and vote | 06:44 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IKE policies tests https://review.openstack.org/47348 | 06:45 |
| *** Dharmit has joined #openstack-qa | 06:49 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 06:51 |
| *** raies has joined #openstack-qa | 06:53 | |
| *** coolsvap has joined #openstack-qa | 06:54 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec Site Connection API tests https://review.openstack.org/48220 | 06:55 |
| *** yassine has joined #openstack-qa | 06:56 | |
| openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Changed the exception name https://review.openstack.org/51835 | 07:01 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52287 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 07:09 |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding testcase for dhcp agent scheduler https://review.openstack.org/48632 | 07:09 |
| *** Madan has quit IRC | 07:12 | |
| *** Madan has joined #openstack-qa | 07:15 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/52577 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 and https://bugs.launchpad.net/bugs/1232748 and | 07:17 |
| mkoderer | Anju: pls ping me when jenkins gave the +1 | 07:25 |
| Anju | ok mkoderer :) | 07:26 |
| *** SergeyLukjanov_ has joined #openstack-qa | 07:26 | |
| *** SergeyLukjanov has quit IRC | 07:28 | |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/tempest: Adds initial ceilometerclient testing code https://review.openstack.org/49700 | 07:32 |
| *** SergeyLukjanov has joined #openstack-qa | 07:32 | |
| *** SergeyLukjanov_ has quit IRC | 07:36 | |
| Anju | mkoderer: ping | 07:45 |
| *** yfried_mob has joined #openstack-qa | 07:46 | |
| *** yfried|mob has joined #openstack-qa | 07:46 | |
| *** SergeyLukjanov has quit IRC | 07:47 | |
| mkoderer | afazekas: cyeoh: please review https://review.openstack.org/#/c/51835/ | 07:48 |
| mkoderer | we need to merge it before anything else gets merged | 07:48 |
| cyeoh | mkoderer: done | 07:50 |
| mkoderer | cyeoh: thx | 07:52 |
| *** yfried|mob has quit IRC | 07:52 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: use "" instead of "None" in xml file https://review.openstack.org/52584 | 07:53 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for set_metadata in aggregate https://review.openstack.org/52585 | 07:53 |
| *** yfried_mob has joined #openstack-qa | 07:53 | |
| cyeoh | np, I'd already +2'd it earlier today :-) | 07:53 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Add configdrive option as ssh authentication strategy https://review.openstack.org/52586 | 07:53 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Enable run_ssh and add floatingip + keypair auth https://review.openstack.org/50337 | 07:53 |
| *** yfried|mob has joined #openstack-qa | 07:53 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Add configdrive option as ssh authentication strategy https://review.openstack.org/52586 | 07:55 |
| *** thomasbiege1 has joined #openstack-qa | 07:55 | |
| *** yfried_mob has quit IRC | 07:57 | |
| *** safchain has joined #openstack-qa | 07:58 | |
| *** yfried|mob has quit IRC | 07:59 | |
| *** yfried_mob has joined #openstack-qa | 07:59 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: use "" instead of "None" in xml file https://review.openstack.org/52584 | 08:03 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/49700 failed tempest with an unrecognized error | 08:07 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for set_metadata in aggregate https://review.openstack.org/52585 | 08:07 |
| *** KapilSaxena has quit IRC | 08:08 | |
| *** KapilSaxena has joined #openstack-qa | 08:08 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/52580 failed tempest with an unrecognized error | 08:09 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for set_metadata in aggregate https://review.openstack.org/52585 | 08:09 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52583 failed tempest with an unrecognized error | 08:10 |
| *** sourabh has joined #openstack-qa | 08:11 | |
| *** yfried_mob has quit IRC | 08:13 | |
| *** yfried_mob has joined #openstack-qa | 08:13 | |
| *** yfried|mob has joined #openstack-qa | 08:13 | |
| *** sourabh has quit IRC | 08:13 | |
| *** yfried_mob has joined #openstack-qa | 08:16 | |
| *** yfried_mob has quit IRC | 08:16 | |
| openstackgerrit | Sun Jing proposed a change to openstack-dev/devstack: Install Ironic Client by devstack https://review.openstack.org/52565 | 08:17 |
| openstackgerrit | A change was merged to openstack/tempest: Changed the exception name https://review.openstack.org/51835 | 08:25 |
| openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Inject "-tempest-" string to rand_name https://review.openstack.org/46872 | 08:26 |
| *** prekarat has quit IRC | 08:30 | |
| *** prekarat has joined #openstack-qa | 08:33 | |
| *** russellb has quit IRC | 08:35 | |
| *** ivanzhu has joined #openstack-qa | 08:35 | |
| *** russellb has joined #openstack-qa | 08:36 | |
| *** giulivo has joined #openstack-qa | 08:38 | |
| *** ivanzhu has quit IRC | 08:40 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52584 failed tempest with an unrecognized error | 08:42 |
| *** yfried has quit IRC | 08:43 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/52585 failed tempest with an unrecognized error | 08:46 |
| *** yfried_mob has joined #openstack-qa | 09:14 | |
| *** yfried_mob has quit IRC | 09:23 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: use "" instead of "None" in xml file https://review.openstack.org/52584 | 09:25 |
| *** matsuhas_ has quit IRC | 09:27 | |
| *** matsuhashi has joined #openstack-qa | 09:27 | |
| *** matsuhashi has quit IRC | 09:28 | |
| *** matsuhashi has joined #openstack-qa | 09:28 | |
| *** thomasbiege2 has joined #openstack-qa | 09:30 | |
| *** thomasbiege1 has quit IRC | 09:34 | |
| *** thomasbiege2 has quit IRC | 09:35 | |
| *** prekarat has quit IRC | 09:41 | |
| *** prekarat has joined #openstack-qa | 09:56 | |
| *** yfried has joined #openstack-qa | 10:09 | |
| *** yfried_mob has joined #openstack-qa | 10:12 | |
| *** yfried_mob has quit IRC | 10:25 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52597 failed tempest with an unrecognized error | 10:28 |
| *** safchain has quit IRC | 10:29 | |
| *** mkollaro has joined #openstack-qa | 10:47 | |
| afazekas | mkoderer, giulivo: https://review.openstack.org/#/c/50337/ | 10:49 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Use lower case true/false in the sample config file https://review.openstack.org/52395 | 10:53 |
| *** yfried_mob has joined #openstack-qa | 11:09 | |
| *** matsuhashi has quit IRC | 11:17 | |
| *** matsuhashi has joined #openstack-qa | 11:17 | |
| *** KapilSaxena has quit IRC | 11:19 | |
| *** jhenner has joined #openstack-qa | 11:21 | |
| *** sdague has joined #openstack-qa | 11:21 | |
| *** matsuhashi has quit IRC | 11:22 | |
| *** nosnos has quit IRC | 11:34 | |
| *** coolsvap has quit IRC | 11:35 | |
| *** nosnos has joined #openstack-qa | 11:35 | |
| *** yfried|mob has joined #openstack-qa | 11:35 | |
| *** yfried_mob has quit IRC | 11:36 | |
| *** nosnos has quit IRC | 11:40 | |
| *** yfried|mob has quit IRC | 11:40 | |
| *** fbo_away is now known as fbo | 11:47 | |
| *** safchain has joined #openstack-qa | 11:48 | |
| *** safchain has quit IRC | 11:48 | |
| *** safchain has joined #openstack-qa | 11:50 | |
| *** prekarat has quit IRC | 11:50 | |
| *** jlibosva has joined #openstack-qa | 11:50 | |
| raies | giulivo: hi | 11:57 |
| raies | giulivo: these needs approval please check them https://review.openstack.org/#/c/47348/ and https://review.openstack.org/#/c/48632/ | 12:00 |
| *** weshay has joined #openstack-qa | 12:02 | |
| *** jlibosva has quit IRC | 12:02 | |
| *** shilpa_ has quit IRC | 12:19 | |
| *** mriedem has joined #openstack-qa | 12:20 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/47143 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 and https://bugs.launchpad.net/bugs/1239856 and | 12:25 |
| *** adalbas has joined #openstack-qa | 12:32 | |
| *** dkranz has quit IRC | 12:32 | |
| *** gszasz has quit IRC | 12:33 | |
| *** yfried_mob has joined #openstack-qa | 12:34 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 12:41 |
| *** woodspa has quit IRC | 12:59 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Fix typos and thinkos in docs https://review.openstack.org/52514 | 12:59 |
| *** yfried has quit IRC | 13:01 | |
| openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 13:03 |
| *** shilpa_ has joined #openstack-qa | 13:09 | |
| *** yfried_mob has quit IRC | 13:10 | |
| sileht | jeblair, clarkb , hi can you re-approved https://review.openstack.org/#/c/48046/, it already have two +2, but it have lost its '+1 Approved' due to a unrelated gate failure | 13:10 |
| sileht | sdague, perhaps can you do it ? | 13:11 |
| sileht | so, thanks in advance if you see my msg later | 13:12 |
| mriedem | is anyone noticing tempest runs taking longer than usual in the last couple of days? | 13:14 |
| mriedem | tempest.api.compute.servers.test_servers_negative took 26 minutes for me with libvirt | 13:14 |
| mriedem | on an m1.large flavor VM | 13:14 |
| mriedem | mtreinish: sdague: ^? | 13:15 |
| mriedem | i blame this: https://review.openstack.org/#/c/50374/ | 13:17 |
| *** shilpa_ has quit IRC | 13:18 | |
| mriedem | afazekas: looks like i've been in test_servers_negative.py lately and were on the above review - it looks like rather than make anything faster, it's blocking on server delete/create between runs which shouldn't be necessary unless you require a clean DB for the tests | 13:18 |
| mriedem | s/i've/you've/ | 13:19 |
| *** jaypipes has joined #openstack-qa | 13:24 | |
| *** ChanServ sets mode: +o jaypipes | 13:24 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/47557 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 and https://bugs.launchpad.net/bugs/1239856 and | 13:29 |
| *** yfried_mob has joined #openstack-qa | 13:31 | |
| *** thomasbiege has joined #openstack-qa | 13:34 | |
| afazekas | mriedem: https://review.openstack.org/#/c/52429/ | 13:34 |
| mriedem | afazekas: ha, you're too fast :) https://bugs.launchpad.net/tempest/+bug/1241588 | 13:36 |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds "list hosts" test case - Cinder https://review.openstack.org/48933 | 13:37 |
| afazekas | mriedem: https://bugs.launchpad.net/tempest/+bug/1240175 | 13:38 |
| mriedem | afazekas: ok, unfortunately my latest tempest run was before your patch merged, so i'll have to see if times are better in my next run | 13:40 |
| *** yfried_mob has quit IRC | 13:44 | |
| *** yfried|mob has joined #openstack-qa | 13:45 | |
| afazekas | may would be better to rebuild the server on tierDown for better time accounting | 13:45 |
| sdague | mriedem: that single test to 26 minus | 13:46 |
| sdague | mriedem: that single test took 26 minutes? | 13:47 |
| sdague | we should figure out why, because it's supposed to default to a noop | 13:47 |
| sdague | and only trigger if the server is broken | 13:48 |
| *** sthaha has quit IRC | 13:48 | |
| afazekas | 2*2 times 400 sec not needed wait +1 not needed server create+delete | 13:48 |
| *** dims has joined #openstack-qa | 13:48 | |
| mriedem | sdague: since 10/15, test_servers_negative has taken nearly 1 hour 50 minutes, about 25 minutes per json/xml run | 13:48 |
| mriedem | sorry, 25 minutes for 2 tests in each json/xml, so 25*4 | 13:49 |
| sdague | mriedem: interesting, I do see 400 secs for those in the gate | 13:49 |
| sdague | ok, afazekas so it's fixed/ | 13:50 |
| sdague | ? | 13:50 |
| sdague | timings go back to sane with your patch upstream | 13:50 |
| mriedem | sdague: i'm going to bring afazekas's patch down and run just those tests to verify it myself too | 13:51 |
| sdague | yeh, upstream looks good with his patch | 13:51 |
| mriedem | btw, any idea if neutron in the gate is running the lbaas tests? | 13:51 |
| mriedem | there is one that i see hang from time to time (about once every 2 weeks i'm seeing on average now) | 13:51 |
| sdague | mriedem: that being said, 25 minutes on those tests is 4x what the gate runs in | 13:52 |
| sdague | mriedem: I don't know, you could check devstack gate | 13:52 |
| afazekas | mriedem: what is your build_timeout value ? | 13:53 |
| afazekas | in the tempest.conf | 13:53 |
| mriedem | afazekas: for compute it's 1500 | 13:54 |
| mriedem | 25 minutes | 13:54 |
| mriedem | afazekas: there is some history there... | 13:54 |
| mriedem | this is on ppc64 and we're using the powervm driver, which has some tests that would fail with timeouts (snapshot tests i think, definitely related to images) | 13:55 |
| afazekas | I am thinking about to decrease the timeout to 196 sec on the gate | 13:55 |
| mriedem | because of a performance problem in the powervm driver in nova | 13:55 |
| sdague | mriedem: can you guys fix that with a cirros image rebuild? | 13:55 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/52372 failed tempest with an unrecognized error | 13:55 |
| sdague | it seems silly to be spinning big images all the time for all the tests | 13:55 |
| mriedem | sdague: if i remember correctly there was a problem getting the cirros image to boot on ppc64 | 13:55 |
| mriedem | so we're using a relatively small rhel image | 13:55 |
| mriedem | it's like 450 MB | 13:56 |
| sdague | yeh, cirros is really small though | 13:56 |
| mriedem | sdague: if i can get a cirros image that works on ppc64, i will definitely try that | 13:56 |
| afazekas | sdague: In most cases we could use very small image (which is even not able to boot), with 0.5 Mib Disk, and 4 MiB ram ... | 13:56 |
| mriedem | i tried using micro flavors and all kinds of crap but that wasn't working | 13:56 |
| sdague | I asked the kvm on power folks to look at rebuilding it | 13:56 |
| sdague | so if they end up with a working cirros, I'll make sure they loop around to you | 13:57 |
| mriedem | sdague: good idea, if they get a cirros image for ppc64 i would use it too, i'll ask krtaylor | 13:57 |
| mriedem | sdague: great, thakns | 13:57 |
| krtaylor | mriedem, pong | 13:58 |
| mriedem | sdague: we still have an issue with the powervm driver, but a smaller image would definitely help. the powervm driver does a bunch of gzip/gunzip of the image before/after ftp to/from the IVM. we had our performance guy profile it and that was taking up all the time. | 13:58 |
| * krtaylor reads scrollback | 13:58 | |
| mriedem | krtaylor: i heard you guys are working on rebuilding a cirros image on power? | 13:58 |
| afazekas | cirros is buildroot image with 'minimal' modifications, IMHO you could use buildroot to create a very small image | 13:58 |
| krtaylor | mriedem, working on is probably too strong, asking about is closer | 13:58 |
| sdague | mriedem: it's not possible to take a straight glance pull? | 13:59 |
| sdague | or it's because it has to be pushed over to the ivm | 13:59 |
| mriedem | sdague: correct | 13:59 |
| sdague | right, the joys of type 1 hypervisors | 13:59 |
| mriedem | sdague: so what mrodden and i were talking about doing was changing the powervm driver to not gzip/gunzip and see if network was faster than i/o, in our case it should be | 14:00 |
| sdague | wel, the cirros size should still help a lot | 14:00 |
| mriedem | problem was when i hacked the powervm driver to do that, it puked...so it's a WIP | 14:00 |
| *** salv-orlando has joined #openstack-qa | 14:00 | |
| mriedem | sdague: that will definitely be a blocker for powervm CI for nova | 14:00 |
| sdague | yeh, actually, when people looked at the glance numbers for gzipping before sending, it turns out that only helped if you were on < Gb network | 14:01 |
| sdague | so we defaulted that off in havana | 14:01 |
| sdague | I think the gzip before the wire is all premature optimization | 14:01 |
| afazekas | what about lzop ? | 14:01 |
| afazekas | The lzop compression is much faster, and relatively good | 14:02 |
| sdague | afazekas: honestly, at this point I'd just assume you have the bits, and only worry about compression if you really need it later | 14:02 |
| krtaylor | mriedem, I believe that we have a very small image for power, I am tracking that down today, hopefully | 14:02 |
| mriedem | sdague: interesting, i can use that against dperaza when he argues with me about the need for it | 14:03 |
| krtaylor | mriedem, we'll need it for our kvm on power CI anyway, I'll let you know what I find | 14:03 |
| sdague | heh | 14:03 |
| mriedem | krtaylor: awesome, thanks | 14:04 |
| sdague | yeh, the HP folks did a ton of analysis on it for the glance case, and NSA and CERN jumped in on that thread and said "yes, please change this, we hacked around it" | 14:04 |
| sdague | which was convincing enough for me | 14:04 |
| mriedem | sdague: good, glad to know what i was working on what the right thing | 14:04 |
| mriedem | *was | 14:04 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 14:04 |
| mriedem | now i just need to get the damn driver to work with it | 14:05 |
| sdague | yeh, each case might be different, but it's definitely naive to believe adding compression is always a win | 14:05 |
| mriedem | sdague: but you've convinced me to open a bug to track it in community anyway | 14:05 |
| *** ken1ohmichi has joined #openstack-qa | 14:05 | |
| sdague | cool | 14:05 |
| afazekas | sdague: Can you review this change https://review.openstack.org/#/c/50337/ ? | 14:08 |
| * afazekas config drive option in the next patchset | 14:09 | |
| *** Madan has quit IRC | 14:12 | |
| *** thomasbiege has quit IRC | 14:14 | |
| *** Madan has joined #openstack-qa | 14:16 | |
| *** gszasz has joined #openstack-qa | 14:16 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/51038 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 14:19 |
| *** SergeyLukjanov has joined #openstack-qa | 14:23 | |
| sdague | afazekas: so, can we keep run_ssh as a deprecated option? I'm a little concerned if people had that working, their test env would change, and they wouldn't know why | 14:26 |
| sdague | also, why are we defaulting to floating instead of fixed? | 14:26 |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Decrease tempest BUILD_TIMEOUT https://review.openstack.org/52639 | 14:26 |
| afazekas | sdague: because AFAIK the fixed is not working with the devstack-tempest neutron gate setup | 14:28 |
| *** yfried|mob has joined #openstack-qa | 14:29 | |
| mtreinish | afazekas: where did 196 sec come from? | 14:30 |
| afazekas | 256 is too big :) | 14:30 |
| afazekas | it is almost 3 times more then the expected minimum | 14:31 |
| sdague | afazekas: any idea why fixed doesn't work with neutron? | 14:32 |
| sdague | fixed seems like it should be the right answer | 14:32 |
| afazekas | you cannot ssh the 'private' /'fixed' network in the current setup | 14:32 |
| sdague | floating is actually more moving parts, because it's another nat | 14:32 |
| sdague | so I'd say then we need to us fixed in the nova network case, set it to floating in devstack-gate for neutron, and track that with a bug | 14:33 |
| afazekas | sdague: yes, now we have relatively low number of recheck issue , So I would vote for the more moving part | 14:33 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Create-stack-user script should have execute permissions https://review.openstack.org/52641 | 14:34 |
| afazekas | sdague: aour scenrio tests are using the floating IP for ssh, without a recheck issue | 14:34 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 14:35 | |
| sdague | afazekas: these are defaults for all environments though, where floating ips might be on quotas that get hit | 14:35 |
| *** _SergeyLukjanov has quit IRC | 14:35 | |
| sdague | so default fixed really seems the right answer, then adjust to floating for the neutron case in the gate | 14:35 |
| maurosr | sdague: are you with some minutes to chat about grenade? I cleared my questions with adalberto this morning, so I think we can start to chat about what you suggested me to work with | 14:36 |
| sdague | yeh, sure | 14:36 |
| maurosr | sdague: so, as I understood, grenade takes a base release (grizzly at this moment), run *only* devstack's exercises, creates the javelin project, setup it, put everything down, then checkout master, devstack on it, and run exercises on it, then tempest.. | 14:39 |
| sdague | yep | 14:39 |
| *** SergeyLukjanov has joined #openstack-qa | 14:40 | |
| sdague | so right now we're doing grizzly -> master | 14:40 |
| maurosr | sdague: so what we want to do? just add a layer to run tempest on the base branch when we're doing a backport? | 14:40 |
| sdague | actually.... lets etherpad some things | 14:40 |
| sdague | https://etherpad.openstack.org/p/grenade-todos-for-havana-release | 14:41 |
| maurosr | ok, | 14:41 |
| mtreinish | maurosr: it doesn't actually run devstack during the upgrade. Those upgrade scripts are self contained. (Although it's very similar) | 14:41 |
| *** shilpa_ has joined #openstack-qa | 14:41 | |
| maurosr | mtreinish: yup, adalbas told me that, part of that is cause devstack would destroy everything, am I right? | 14:41 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: python-libguestfs should only be installed for libvirt/KVM. https://review.openstack.org/52642 | 14:42 |
| mtreinish | maurosr: yeah | 14:43 |
| *** rdekel has joined #openstack-qa | 14:44 | |
| *** salv-orlando has quit IRC | 14:46 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest because of: https://bugs.launchpad.net/bugs/1230407 | 14:49 |
| *** prekarat has joined #openstack-qa | 14:50 | |
| sdague | maurosr: ok, I think that's mostly a list in the etherpad | 14:53 |
| sdague | dtroyer: you about? curious if you had any complains with doing stable/havana on devstack / grenade today so that maurosr can run with getting all the jobs in place | 14:54 |
| maurosr | sdague: question, when you say grenade tree, you mean grenade repository.. when you say config tree, you're talking about...?? | 14:55 |
| sdague | maurosr: https://github.com/openstack-infra/config | 14:55 |
| sdague | where all the test jobs are defined | 14:55 |
| maurosr | right | 14:55 |
| sdague | there may also need to be devstack-gate changes to make it all work | 14:56 |
| sdague | we've only ever had 1 grenade job, so I expect as we parametrize it, more changes in devstack-gate might be needed | 14:56 |
| mtreinish | sdague: I think devstack gate has the grenade targets parameterized already but I'm not sure | 15:00 |
| maurosr | sdague: so we will keep with one job when testing changes against master right? and two to ensure the backport doesn't break the upgrade right? (g->h, h->master) | 15:01 |
| sdague | mtreinish: it does, but for the stable jobs, there is going to be some funny matrix logic to sort out last -> next | 15:01 |
| sdague | maurosr: right, exactly | 15:01 |
| maurosr | cool.. I think I got it | 15:02 |
| maurosr | sdague: I'm going lunch, will start to reproduce the infra tests on the machine I set after it | 15:03 |
| maurosr | until dtroyer decides about the tag | 15:03 |
| sdague | sounds great | 15:04 |
| sdague | yeh, I'm going to try to get those tags set, I think we should be ok with them | 15:04 |
| *** yfried|mob has quit IRC | 15:04 | |
| sdague | I think the last 3 patches dtroyer wanted merged went in yesterday, and the ceilo fix went in, so everything else can backport if need be | 15:04 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/51399 failed tempest with an unrecognized error | 15:05 |
| *** mkollaro has quit IRC | 15:05 | |
| *** rdekel has quit IRC | 15:07 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: keystone : Add coverage of v2 client trust scoping https://review.openstack.org/51560 | 15:08 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: Add initial keystoneclient v3 auth tests https://review.openstack.org/51558 | 15:08 |
| openstackgerrit | Steven Hardy proposed a change to openstack/tempest: Add initial keystoneclient trusts tests https://review.openstack.org/51559 | 15:08 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 15:09 | |
| *** _SergeyLukjanov has quit IRC | 15:10 | |
| mriedem | sdague: you were +2 on this before mtreinish added the bug link in the commit message, otherwise it's unchanged: https://review.openstack.org/#/c/52211/ | 15:12 |
| *** SergeyLukjanov has joined #openstack-qa | 15:13 | |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Enable run_ssh and add floatingip + keypair auth https://review.openstack.org/50337 | 15:13 |
| sdague | mriedem: lgtm | 15:14 |
| mriedem | sdague: thanks | 15:15 |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/52642 failed tempest with an unrecognized error | 15:17 |
| ken1ohmichi | hi | 15:19 |
| ken1ohmichi | I have a question. | 15:19 |
| mtreinish | ken1ohmichi: hi | 15:20 |
| mtreinish | isn't it really late for you? | 15:20 |
| ken1ohmichi | If finding a bug of other project like nova, can we add a test case with skip_because deco to Tempest? | 15:20 |
| ken1ohmichi | mtreinish: yes, a little late night:-) | 15:20 |
| ken1ohmichi | or should we wait for fixing a bug? | 15:21 |
| mtreinish | ken1ohmichi: so we only add the decorators if the bug isn't going to be fixed anytime soon (or it's required to get the the bug fixed) | 15:21 |
| mtreinish | we generally prefer to just fix the bug first | 15:21 |
| ken1ohmichi | mtreinish: Thanks! can you give us an advice on https://review.openstack.org/#/c/49231/12/tempest/api/compute/servers/test_attach_interfaces_negative.py ? | 15:22 |
| mtreinish | ken1ohmichi: oh especially not there, new test cases aren't allowed to be added with skips | 15:23 |
| mtreinish | because they're never run through the gate | 15:23 |
| mtreinish | so there is no guarentee it works | 15:23 |
| ken1ohmichi | mtreinish: I see, now many negative tests are queued in gerrit, and some issues are found.. | 15:24 |
| ken1ohmichi | mtreinish: I will review these tests from the viewpoint you gave me, thanks! | 15:25 |
| mtreinish | ken1ohmichi: sure np, it's good to actually talk to you. I figured that wouldn't happen until summit because of the tz difference | 15:26 |
| mtreinish | are you going to be there? | 15:26 |
| *** mkollaro has joined #openstack-qa | 15:26 | |
| ken1ohmichi | mtreinish: now 24:26, will sleep zzz.. | 15:27 |
| *** ken1ohmichi has quit IRC | 15:28 | |
| openstackgerrit | A change was merged to openstack/tempest: Inject "-tempest-" string to rand_name https://review.openstack.org/46872 | 15:29 |
| rahmu | afazekas: ping | 15:43 |
| *** thomasbiege has joined #openstack-qa | 15:50 | |
| afazekas | rahmu: pong | 15:51 |
| * afazekas is leaving in ~5 minutes | 15:52 | |
| rahmu | afazekas: I saw your review here: https://review.openstack.org/#/c/50637/ | 15:52 |
| rahmu | should I send another patch or update this one? | 15:52 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/50337 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 15:52 |
| afazekas | rahmu: you should send a patch for devstack first | 15:52 |
| afazekas | rahmu: than update the change with a default True | 15:53 |
| rahmu | in gerrit how can I mark that the tempest patch depends on the devstack one? | 15:53 |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Add the doc build tools https://review.openstack.org/52226 | 15:53 |
| afazekas | You can't | 15:53 |
| rahmu | afazekas: okay good to know. | 15:53 |
| rahmu | thanks | 15:54 |
| afazekas | rahmu: thank you | 15:54 |
| *** dkranz has joined #openstack-qa | 15:55 | |
| *** Madan has quit IRC | 15:57 | |
| openstackgerrit | A change was merged to openstack/tempest: Add a skip for meter-label cli tests https://review.openstack.org/52211 | 15:57 |
| *** SergeyLukjanov has quit IRC | 16:01 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/51560 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 16:02 |
| *** krtaylor has quit IRC | 16:08 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/52642 failed tempest with an unrecognized error | 16:12 |
| *** SergeyLukjanov has joined #openstack-qa | 16:16 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/52595 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 and https://bugs.launchpad.net/bugs/1225664 and | 16:18 |
| *** dansmith is now known as Steely_Dan | 16:24 | |
| *** fbo is now known as fbo_away | 16:24 | |
| *** bill_az_ has joined #openstack-qa | 16:30 | |
| *** mriedem has quit IRC | 16:31 | |
| *** yassine has quit IRC | 16:37 | |
| *** dkorn has joined #openstack-qa | 16:42 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/49157 failed tempest with an unrecognized error | 16:47 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Create-stack-user script should have execute permissions https://review.openstack.org/52641 | 16:52 |
| *** thomasbiege has quit IRC | 16:52 | |
| *** thomasbiege has joined #openstack-qa | 16:52 | |
| *** krtaylor has joined #openstack-qa | 16:55 | |
| *** Dharmit has quit IRC | 16:56 | |
| *** mlavalle has joined #openstack-qa | 16:56 | |
| *** mlavalle_ has joined #openstack-qa | 16:58 | |
| *** mlavalle has quit IRC | 17:01 | |
| *** mlavalle_ is now known as mlavalle | 17:01 | |
| *** shilpa_ has quit IRC | 17:25 | |
| *** dkorn has quit IRC | 17:32 | |
| afazekas | Please review (1 line, 4 days old) https://review.openstack.org/#/c/51528/1 | 17:37 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 17:47 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 17:47 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 17:57 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 17:57 | |
| *** bdperkin has quit IRC | 18:04 | |
| *** bdperkin has joined #openstack-qa | 18:04 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: API tests for neutron router gateway https://review.openstack.org/47554 | 18:08 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/51753 failed tempest with an unrecognized error | 18:10 |
| *** giulivo has quit IRC | 18:17 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Make e-r compatible with old/new logstash schemas. https://review.openstack.org/52521 | 18:30 |
| *** openstackrecheck has quit IRC | 18:32 | |
| *** openstackrecheck has joined #openstack-qa | 18:32 | |
| *** VinodGupta has joined #openstack-qa | 18:33 | |
| *** prekarat has quit IRC | 18:37 | |
| *** rockyg has quit IRC | 18:58 | |
| *** rockyg has joined #openstack-qa | 18:58 | |
| *** prekarat has joined #openstack-qa | 18:59 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: python-libguestfs should only be installed for libvirt/KVM. https://review.openstack.org/52642 | 19:03 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/52101 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 19:08 |
| *** VinodGupta has quit IRC | 19:08 | |
| morganfainberg | sdague, so i've been digging around in libvirt, there appears to be sanity checks to explicitly disable migration to the same host. short of using some container type setup to run multiple libvirtd (with explciit host uuids being different) i don;t think it's going to be possible | 19:20 |
| morganfainberg | sdague, to migrate to the same host that is. | 19:21 |
| sdague | ok, no prob, thanks for checking though | 19:21 |
| *** mkollaro has quit IRC | 19:25 | |
| *** prekarat has quit IRC | 19:26 | |
| *** gszasz has quit IRC | 19:27 | |
| morganfainberg | sdague, yeah np. | 19:30 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 19:53 |
| *** krtaylor has quit IRC | 20:01 | |
| sdague | jog0 / mtreinish / clarkb: https://review.openstack.org/#/c/52698/ - thoughts? That's probably as far as that patch is getting today, but I wanted to get thoughts out there, so I can get these objects easy enough to refactor some of the other code to take advantage of them | 20:06 |
| dkranz | sdague: Got a minute to discuss rollout of log error crushing? | 20:10 |
| sdague | sure | 20:11 |
| dkranz | sdague: I was basically out this week until today | 20:11 |
| sdague | yep, no worries | 20:11 |
| clarkb | sdague: I did a quick review on 52698 | 20:11 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/52695 failed tempest with an unrecognized error | 20:11 |
| dkranz | sdague: So I had a modified version of elastic recheck that reports to me new log errors that are not whitelisted | 20:11 |
| sdague | cool | 20:12 |
| dkranz | sdague: I was going to let that go for about two weeks and file bugs as I add to the whitelist | 20:12 |
| dkranz | sdague: They are all flaky at this point so I was going to add them to elastic recheck | 20:12 |
| sdague | sounds good, realize elastic recheck is getting upgrade on monday, so we very well might ended up with zeroed data at that point | 20:12 |
| dkranz | sdague: That't ok. | 20:12 |
| dkranz | sdague: The two issues are when do we tutn on the gating, and how do we communicate what we are doing. | 20:13 |
| dkranz | sdague: And the fact that when I run my script today I get ERROR:gerrit.GerritWatcher:Exception on ssh event stream: | 20:13 |
| sdague | so I think we start by this running a report output at the end of the run | 20:13 |
| dkranz | sdague: This was working fine before | 20:14 |
| sdague | dkranz: interesting.... | 20:14 |
| dkranz | sdague: It already reports the errors at the end of the tempest run | 20:14 |
| sdague | ok, great | 20:14 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 20:14 | |
| dkranz | sdague: That is what my script looks for when it was working | 20:14 |
| sdague | so honestly, I think this is messaging that we do post summit, because between now and then, everything's just going to get lost in people ramping up | 20:14 |
| sdague | and we can talk about it there, as it's only 2 weeks away | 20:15 |
| *** _SergeyLukjanov has quit IRC | 20:15 | |
| dkranz | sdague: That's fine with me. | 20:15 |
| dkranz | sdague: two weeks was the minimumm time anyway | 20:15 |
| sdague | yep | 20:15 |
| dkranz | sdague: But I am now blocked by this error. Nothing changed on my side | 20:15 |
| dkranz | sdague: I can connect to gerrit fine manually | 20:15 |
| dkranz | sdague: I can take this to infra if you don't have any idea of what could be wrong | 20:16 |
| sdague | interesting.... clarkb thoughts? | 20:16 |
| sdague | yeh, I think it needs infra expertise | 20:16 |
| clarkb | what error? | 20:16 |
| dkranz | http://paste.openstack.org/show/48737/ | 20:16 |
| dkranz | clarkb: This same code worked a week ago | 20:17 |
| dkranz | clarkb: The first line is just a "print" of the arguments I gave to the gerrit stream constructor | 20:17 |
| dkranz | clarkb: The code is the same as in elastic recheck as of a week ago | 20:18 |
| clarkb | dkranz: that may be the paramiko needs to be imported after daemonization problem | 20:19 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: python-libguestfs should only be installed for libvirt/KVM. https://review.openstack.org/52642 | 20:19 |
| dkranz | sdague: Is that something that was fixed in elastic recheck recently? | 20:19 |
| mtreinish | clarkb: I fixed that well over a week ago though (before it was on infra I think) | 20:20 |
| dkranz | clarkb: I can try changing that but it is odd that it was working perfectly until I tried today | 20:20 |
| clarkb | dkranz: its a weird bug, basically paramiko can't deal with the chdiring and stuff that happens during daemonization | 20:21 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 20:21 |
| clarkb | if you run without daemonization its fine | 20:21 |
| sdague | clarkb: ok, I updated for your issue | 20:21 |
| sdague | and that raises exactly the question I had about ES result stability :) | 20:21 |
| clarkb | sdague: oh I thought you were asking about the API | 20:22 |
| sdague | as that's going to mean I'll need to ensure this code is good for monday, and I've done the refactor onto it | 20:22 |
| clarkb | API doesn't change, just the schema | 20:22 |
| sdague | no schema stability | 20:22 |
| sdague | because elastic-recheck is currently totally gross in how it pokes around in data | 20:22 |
| clarkb | it is all logstash's fault | 20:22 |
| sdague | but this should provide us an isolation layer | 20:22 |
| dkranz | clarkb, sdague : I will have to figure out why elastic recheck seems to be working but the same code fails for me. | 20:28 |
| clarkb | sdague: and really the only bits that hurt are things under the root level with @ that are dropping the @ | 20:28 |
| clarkb | sdague: at least for queries searching for foo:"bar" will return @fields.foo:"bar" <- I guess that doesn't help when dealing with the results | 20:29 |
| sdague | yeh, not so much, but it at least means another thing doesn't have to change | 20:30 |
| clarkb | sdague: so my change for forward and backward compat removes @fields and searches for @message:"foo" or message:"foo" | 20:33 |
| *** SergeyLukjanov has joined #openstack-qa | 20:38 | |
| sdague | clarkb: ok, got it. so e-r itself actually interprets a lot less of the schema then I remembered | 20:46 |
| sdague | it's the reporting tools that touch it more | 20:46 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: first iteration on ResultSet and Hit objects https://review.openstack.org/52698 | 20:47 |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 20:51 | |
| *** _SergeyLukjanov has quit IRC | 20:52 | |
| *** mlavalle has quit IRC | 20:52 | |
| *** jaypipes has quit IRC | 20:57 | |
| *** mtreinish has quit IRC | 20:58 | |
| *** mlavalle has joined #openstack-qa | 21:00 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/42965 failed tempest because of: https://bugs.launchpad.net/bugs/1226337 | 21:02 |
| *** SergeyLukjanov has joined #openstack-qa | 21:02 | |
| *** mtreinish has joined #openstack-qa | 21:04 | |
| *** rockygrober has joined #openstack-qa | 21:04 | |
| *** rockyg has quit IRC | 21:07 | |
| *** safchain has quit IRC | 21:10 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Enable Ceilometer alarm services https://review.openstack.org/48046 | 21:43 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/52101 failed tempest with an unrecognized error | 21:53 |
| jog0 | yeah | 21:56 |
| *** mlavalle has quit IRC | 22:02 | |
| jog0 | woops, wrong window. TGIF | 22:03 |
| *** krtaylor has joined #openstack-qa | 22:14 | |
| *** rockyg has joined #openstack-qa | 22:21 | |
| *** dosaboy has quit IRC | 22:30 | |
| *** dosaboy has joined #openstack-qa | 22:37 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 22:41 | |
| *** _SergeyLukjanov is now known as SergeyLukjanov | 22:41 | |
| *** openstackgerrit has quit IRC | 22:46 | |
| *** openstackgerrit has joined #openstack-qa | 22:46 | |
| *** adalbas has quit IRC | 22:50 | |
| *** SergeyLukjanov is now known as _SergeyLukjanov | 22:59 | |
| *** _SergeyLukjanov has quit IRC | 22:59 | |
| *** SergeyLukjanov has joined #openstack-qa | 23:01 | |
| *** mkollaro has joined #openstack-qa | 23:12 | |
| *** rockyg has quit IRC | 23:14 | |
| *** rockygrober has quit IRC | 23:14 | |
| *** weshay has quit IRC | 23:26 | |
| *** rockyg has joined #openstack-qa | 23:28 | |
| *** rockygrober has joined #openstack-qa | 23:28 | |
| *** rockyg has quit IRC | 23:40 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!