| openstackrecheck | openstack/glance change: https://review.openstack.org/35178 failed tempest with an unrecognized error | 00:00 |
|---|---|---|
| *** fbo_away is now known as fbo | 00:02 | |
| *** jhenner has quit IRC | 00:07 | |
| *** mkollaro has quit IRC | 00:11 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/59110 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 00:11 |
| *** fbo is now known as fbo_away | 00:13 | |
| *** yfried has quit IRC | 00:15 | |
| *** weshay has joined #openstack-qa | 00:16 | |
| *** dims has joined #openstack-qa | 00:23 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/56692 failed tempest with an unrecognized error | 00:26 |
| *** matsuhashi has joined #openstack-qa | 00:29 | |
| *** hemna is now known as hemnafk | 00:30 | |
| *** dmorita has joined #openstack-qa | 00:39 | |
| *** bill_az has quit IRC | 00:47 | |
| *** matsuhashi has quit IRC | 00:50 | |
| *** matsuhashi has joined #openstack-qa | 00:50 | |
| *** matsuhas_ has joined #openstack-qa | 00:53 | |
| *** matsuhashi has quit IRC | 00:54 | |
| *** marun has joined #openstack-qa | 00:55 | |
| *** ken1ohmichi has joined #openstack-qa | 00:56 | |
| *** reaper has quit IRC | 01:00 | |
| *** marun has quit IRC | 01:02 | |
| *** bill_az has joined #openstack-qa | 01:22 | |
| *** nosnos has joined #openstack-qa | 01:26 | |
| openstackgerrit | A change was merged to openstack/tempest: Enable a uuid flavor https://review.openstack.org/56326 | 01:36 |
| openstackgerrit | A change was merged to openstack/tempest: Update openstack/common/lockutils https://review.openstack.org/56738 | 01:39 |
| openstackgerrit | A change was merged to openstack/tempest: Add test for Swift formpost middleware https://review.openstack.org/56196 | 01:39 |
| openstackgerrit | Kui Shi proposed a change to openstack-dev/devstack: Add ironic exercise in devstack https://review.openstack.org/59286 | 01:50 |
| *** rhsu has quit IRC | 01:56 | |
| *** mriedem has joined #openstack-qa | 02:05 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Remove kernel-headers apt-get call https://review.openstack.org/54386 | 02:08 |
| *** devlaps has quit IRC | 02:20 | |
| openstackgerrit | A change was merged to openstack/tempest: Support Neutron security groups in scenario testing https://review.openstack.org/50892 | 02:21 |
| *** bill_az has quit IRC | 02:23 | |
| *** masayukig has joined #openstack-qa | 02:30 | |
| *** matsuhas_ has quit IRC | 02:31 | |
| masayukig | away/ I'm in the hospital from Sun to Wed for an operation... | 02:31 |
| *** matsuhashi has joined #openstack-qa | 02:31 | |
| masayukig | Oops. typo... | 02:32 |
| *** masayukig has quit IRC | 02:39 | |
| *** masayuki_ has joined #openstack-qa | 02:39 | |
| *** ivanzhu has joined #openstack-qa | 02:44 | |
| *** masayuki_ has quit IRC | 02:44 | |
| *** dkranz has joined #openstack-qa | 02:54 | |
| *** nosnos has quit IRC | 02:58 | |
| *** nosnos has joined #openstack-qa | 02:59 | |
| *** dkranz has quit IRC | 03:04 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 03:23 |
| *** SushilKM has joined #openstack-qa | 03:39 | |
| *** matsuhashi has quit IRC | 03:42 | |
| *** dims has quit IRC | 03:43 | |
| openstackgerrit | wanghao proposed a change to openstack/tempest: add some test for force delete: https://review.openstack.org/55977 | 03:45 |
| *** liudong_ has joined #openstack-qa | 03:48 | |
| *** SushilKM has quit IRC | 03:48 | |
| openstackgerrit | Feng Ju proposed a change to openstack/tempest: Refactors network test in scenario to support advanced network test https://review.openstack.org/58718 | 03:48 |
| *** liudong_ has quit IRC | 03:49 | |
| openstackgerrit | ayoung proposed a change to openstack-dev/devstack: make mysql user be defaulted to other than root additional change to create the user https://review.openstack.org/40676 | 03:54 |
| *** SushilKM has joined #openstack-qa | 04:10 | |
| *** weshay has quit IRC | 04:17 | |
| *** SushilKM has quit IRC | 04:20 | |
| *** SushilKM has joined #openstack-qa | 04:20 | |
| *** matsuhashi has joined #openstack-qa | 04:25 | |
| *** raies has joined #openstack-qa | 04:45 | |
| *** mriedem has quit IRC | 04:53 | |
| *** DennyZhang has joined #openstack-qa | 04:57 | |
| *** matsuhashi has quit IRC | 05:00 | |
| *** SushilKM has quit IRC | 05:00 | |
| *** matsuhashi has joined #openstack-qa | 05:08 | |
| *** raies has quit IRC | 05:09 | |
| *** raies has joined #openstack-qa | 05:19 | |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Use fixed network for ssh when n-net is enabled https://review.openstack.org/59619 | 05:20 |
| *** matsuhashi has quit IRC | 05:27 | |
| *** DennyZhang has quit IRC | 05:31 | |
| *** jaypipes has quit IRC | 05:33 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the update extra route https://review.openstack.org/50880 | 05:36 |
| openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Check HTTP response headers of Swift middleware API in detail https://review.openstack.org/59623 | 05:37 |
| *** raies has quit IRC | 05:38 | |
| *** ivanzhu has quit IRC | 05:39 | |
| openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 05:45 |
| *** rdekel has joined #openstack-qa | 05:47 | |
| *** SergeyLukjanov has joined #openstack-qa | 05:48 | |
| *** raies has joined #openstack-qa | 05:54 | |
| *** jaypipes has joined #openstack-qa | 05:54 | |
| *** ChanServ sets mode: +o jaypipes | 05:54 | |
| *** matsuhashi has joined #openstack-qa | 05:59 | |
| *** ivanzhu has joined #openstack-qa | 06:05 | |
| *** prekarat has quit IRC | 06:08 | |
| *** DennyZhang has joined #openstack-qa | 06:08 | |
| *** yfried has joined #openstack-qa | 06:11 | |
| *** SergeyLukjanov has quit IRC | 06:27 | |
| *** fungi has quit IRC | 06:35 | |
| *** fungi has joined #openstack-qa | 06:36 | |
| *** ivanzhu has quit IRC | 06:37 | |
| *** SushilKM has joined #openstack-qa | 06:38 | |
| *** psharma has joined #openstack-qa | 06:41 | |
| openstackgerrit | wangchy proposed a change to openstack/tempest: Fix a minor error in method _get_alternative_flavor https://review.openstack.org/59635 | 06:57 |
| *** SushilKM has quit IRC | 06:57 | |
| *** jlibosva has joined #openstack-qa | 07:15 | |
| *** gszasz has joined #openstack-qa | 07:29 | |
| yfried | salv-orlando: ping | 07:35 |
| *** jhenner has joined #openstack-qa | 07:44 | |
| *** gszasz has quit IRC | 07:45 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: prot servers negative tests into v3 Part2 https://review.openstack.org/40638 | 07:45 |
| openstackgerrit | Abhijeet Malawade proposed a change to openstack/tempest: Add testcases for security groups and images https://review.openstack.org/47079 | 07:46 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_server_rescue into v3 part1 https://review.openstack.org/44220 | 07:51 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_server_rescue into v3 part2 https://review.openstack.org/44221 | 07:51 |
| *** liudong_ has joined #openstack-qa | 07:51 | |
| *** DennyZhang has quit IRC | 07:52 | |
| *** liudong_ has quit IRC | 07:52 | |
| *** fbo_away is now known as fbo | 07:52 | |
| openstackgerrit | A change was merged to openstack/tempest: Add two more tests for Swift staticweb middleware https://review.openstack.org/58735 | 08:00 |
| *** ivanzhu has joined #openstack-qa | 08:01 | |
| *** jhenner has quit IRC | 08:11 | |
| *** tkammer_ has joined #openstack-qa | 08:22 | |
| *** SergeyLukjanov has joined #openstack-qa | 08:23 | |
| *** dmorita has quit IRC | 08:24 | |
| *** che-arne has quit IRC | 08:25 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add internal testing for the stress test framework https://review.openstack.org/51122 | 08:32 |
| *** gszasz has joined #openstack-qa | 08:42 | |
| *** jorisroovers has joined #openstack-qa | 08:53 | |
| *** Madan has joined #openstack-qa | 08:56 | |
| *** jpich has joined #openstack-qa | 09:02 | |
| *** giulivo has joined #openstack-qa | 09:03 | |
| *** tkammer_ has quit IRC | 09:05 | |
| *** jroovers has joined #openstack-qa | 09:05 | |
| *** jorisroovers has quit IRC | 09:05 | |
| *** yassine has joined #openstack-qa | 09:10 | |
| *** tkammer has joined #openstack-qa | 09:17 | |
| *** ivanzhu has quit IRC | 09:21 | |
| *** jroovers has quit IRC | 09:32 | |
| *** jhenner has joined #openstack-qa | 09:37 | |
| openstackgerrit | wangchy proposed a change to openstack/tempest: Wait for 4 seconds so that back rotation function can complete. https://review.openstack.org/59656 | 09:37 |
| *** rossella_s has joined #openstack-qa | 09:40 | |
| *** che-arne has joined #openstack-qa | 09:42 | |
| *** matsuhashi has quit IRC | 09:51 | |
| *** matsuhashi has joined #openstack-qa | 09:54 | |
| *** raies has quit IRC | 09:57 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Update job names https://review.openstack.org/59573 | 10:04 |
| *** raies has joined #openstack-qa | 10:06 | |
| raies | hi anyone I need help on devstack installation using FWaaS service enabled | 10:07 |
| raies | I run stack.sh | 10:08 |
| raies | but I find following error in the last | 10:08 |
| raies | Error: Service n-api is not running | 10:08 |
| *** ken1ohmichi has quit IRC | 10:09 | |
| raies | anyone please give some input on above | 10:09 |
| *** openstackrecheck has quit IRC | 10:09 | |
| *** openstackrecheck has joined #openstack-qa | 10:10 | |
| openstackgerrit | A change was merged to openstack/tempest: Adds improvements to the Swift TempURL test https://review.openstack.org/57287 | 10:15 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/54117 failed tempest with an unrecognized error | 10:19 |
| *** marun has joined #openstack-qa | 10:20 | |
| *** raies_ has joined #openstack-qa | 10:22 | |
| *** raies has quit IRC | 10:22 | |
| *** fbo is now known as fbo_away | 10:28 | |
| raies_ | mkoderer: hello | 10:30 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/59650 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 10:30 |
| raies_ | I am trying to implement API test for FWAAS extensions | 10:31 |
| raies_ | please let me know is there any pre setup requirement to implement firewall as a service | 10:32 |
| raies_ | such as creation af network , subnet etc ?? | 10:32 |
| raies_ | till now wht I see here that I am able to create rule , policy and firewall without need of network, subnet, port etc | 10:33 |
| raies_ | mkodere : ^^ | 10:33 |
| raies_ | mkoderer: ^^ | 10:34 |
| *** prekarat has joined #openstack-qa | 10:46 | |
| *** markmcclain has joined #openstack-qa | 10:49 | |
| *** weshay has joined #openstack-qa | 10:58 | |
| *** markmcclain has quit IRC | 11:00 | |
| *** markmcclain has joined #openstack-qa | 11:00 | |
| openstackrecheck | openstack/python-cinderclient change: https://review.openstack.org/57404 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 11:02 |
| openstackgerrit | wangchy proposed a change to openstack/tempest: Wait for 4 seconds so that back rotation function can complete https://review.openstack.org/59656 | 11:05 |
| yfried | marun: ping | 11:09 |
| marun | yfried: pong | 11:09 |
| yfried | marun: can you take a look at the comment here - https://review.openstack.org/#/c/55101/20/tempest/scenario/test_cross_tenant_connectivity.py? | 11:10 |
| yfried | https://review.openstack.org/#/c/55101/20/tempest/scenario/test_cross_tenant_connectivity.py | 11:10 |
| yfried | marun: it's copied from your network_basic_ops, so maybe you know what's the story behind the HP copyright? | 11:11 |
| *** weshay has quit IRC | 11:11 | |
| marun | https://wiki.openstack.org/wiki/Documentation/Copyright | 11:11 |
| marun | I believe these rules apply equally to the code. | 11:12 |
| marun | The rule is - you can add to, but should not remove, copyright attribution. | 11:12 |
| yfried | marun: so you copied the HP line as well? | 11:13 |
| marun | If you copy a module I think the general rule is to include the original copyright declaration. If the module is new, then that isn't required, | 11:14 |
| marun | new -> created with new content | 11:14 |
| marun | the HP line was added after I created the module by someone modifying it | 11:14 |
| marun | I don't tend to add copyright lines unless I'm creating a new module. | 11:15 |
| *** weshay has joined #openstack-qa | 11:16 | |
| marun | Red Hat gets credit for the commits I do - the git history ensures that - so I don't think there is any value in a company having their name as a copyright holder in modiles. | 11:16 |
| marun | modules | 11:16 |
| *** weshay has quit IRC | 11:17 | |
| *** prekarat has quit IRC | 11:22 | |
| *** zhikunliu has joined #openstack-qa | 11:25 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/57574 failed tempest with an unrecognized error | 11:26 |
| yfried | marun: tnx | 11:26 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: cinder v2 api tests - part1 https://review.openstack.org/59113 | 11:27 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: cinder v2 api tests - part1 https://review.openstack.org/59113 | 11:31 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_aggregates and test_hosts into nova v3 part2 https://review.openstack.org/42869 | 11:36 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_aggregates and test_hosts into nova v3 part1 https://review.openstack.org/42868 | 11:36 |
| openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add internal testing for the stress test framework https://review.openstack.org/51122 | 11:49 |
| openstackgerrit | Saranya Pandian proposed a change to openstack/tempest: Adds paramiko logs to console output https://review.openstack.org/59690 | 11:52 |
| *** adalbas has joined #openstack-qa | 11:53 | |
| *** dims has joined #openstack-qa | 11:58 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_keypairs into nova v3 part1 https://review.openstack.org/57389 | 12:03 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_keypairs into nova v3 part2 https://review.openstack.org/57390 | 12:03 |
| *** tkammer has quit IRC | 12:04 | |
| openstackrecheck | openstack/python-cinderclient change: https://review.openstack.org/57404 failed tempest with an unrecognized error | 12:04 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/59675 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 12:05 |
| *** tkammer has joined #openstack-qa | 12:06 | |
| *** mkollaro has joined #openstack-qa | 12:06 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/42868 failed tempest because of: https://bugs.launchpad.net/bugs/1254872 | 12:13 |
| *** Madan has quit IRC | 12:17 | |
| openstackgerrit | Brent Eagles proposed a change to openstack/tempest: Add a control point for floating IP assignment https://review.openstack.org/59517 | 12:24 |
| *** yfried has quit IRC | 12:24 | |
| *** markmcclain has quit IRC | 12:30 | |
| *** markmcclain has joined #openstack-qa | 12:32 | |
| *** YorikSar has quit IRC | 12:33 | |
| *** YorikSar has joined #openstack-qa | 12:35 | |
| *** markmcclain has quit IRC | 12:36 | |
| *** markmcclain has joined #openstack-qa | 12:40 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/tempest: Delete stack after unsuccessful testing https://review.openstack.org/59341 | 12:41 |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer https://review.openstack.org/58697 | 12:45 |
| *** ivanzhu has joined #openstack-qa | 12:46 | |
| *** ivanzhu has left #openstack-qa | 12:46 | |
| *** adalbas has quit IRC | 12:52 | |
| *** gszasz has quit IRC | 12:55 | |
| *** sthaha has joined #openstack-qa | 12:56 | |
| *** sthaha has joined #openstack-qa | 12:56 | |
| *** fbo_away is now known as fbo | 12:56 | |
| *** yfried has joined #openstack-qa | 12:57 | |
| *** prekarat has joined #openstack-qa | 12:57 | |
| *** dkranz has joined #openstack-qa | 12:58 | |
| openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/48803 failed tempest with an unrecognized error | 12:59 |
| *** jhenner has quit IRC | 13:01 | |
| *** gszasz has joined #openstack-qa | 13:01 | |
| *** jhenner has joined #openstack-qa | 13:01 | |
| *** mkollaro has quit IRC | 13:02 | |
| *** mkollaro1 has joined #openstack-qa | 13:02 | |
| *** yfried has quit IRC | 13:02 | |
| *** yfried has joined #openstack-qa | 13:03 | |
| *** rossella_s has quit IRC | 13:04 | |
| *** adalbas has joined #openstack-qa | 13:09 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/57390 failed tempest with an unrecognized error | 13:18 |
| *** weshay has joined #openstack-qa | 13:18 | |
| *** dims has quit IRC | 13:21 | |
| *** rossella_s has joined #openstack-qa | 13:23 | |
| *** dims has joined #openstack-qa | 13:25 | |
| *** rossella_s has quit IRC | 13:26 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port some servers tests into nova v3 part2 https://review.openstack.org/58222 | 13:26 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port some server tests into nova v3 part1 https://review.openstack.org/58221 | 13:26 |
| *** rossella-s has joined #openstack-qa | 13:26 | |
| *** zul has quit IRC | 13:29 | |
| *** zul has joined #openstack-qa | 13:31 | |
| openstackgerrit | adalbas proposed a change to openstack/tempest: Negative tests separate file for floating_ips_actions https://review.openstack.org/59711 | 13:31 |
| maurosr | sdague: around? do you have some a couple of minutes to chat?? | 13:38 |
| sdague | yeh, what's up? | 13:39 |
| maurosr | sdague: so was looking what was wrong with my change and grenade http://logs.openstack.org/74/53874/11/check/check-grenade-devstack-vm/bd5050f/console.html and noticed that it was cloning grizzly as base | 13:39 |
| *** matsuhashi has quit IRC | 13:40 | |
| maurosr | sdague: looking into d-g, that change you made in my patch to gate forward may have caused it , take a look on https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate-wrap.sh#L513 | 13:40 |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/42407 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 13:41 |
| *** inkerra has joined #openstack-qa | 13:41 | |
| maurosr | sdague: so I was thinking how do we change that, maybe that should go inside the else that checks if we want to gate forward? | 13:42 |
| maurosr | I'm asking before submit something cause I'm not really sure how you plan to gate forwars | 13:42 |
| sdague | ok, let me look | 13:43 |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer https://review.openstack.org/58697 | 13:43 |
| *** rossella-s has quit IRC | 13:43 | |
| sdague | maurosr: so that link you provided about coming from grizzly | 13:44 |
| *** rossella_s has joined #openstack-qa | 13:44 | |
| sdague | that was done about a month ago, because we couldn't upgrade from havana | 13:44 |
| *** aguzikov1 has joined #openstack-qa | 13:44 | |
| sdague | we need the change that lets grenade upgrade from either havana or grizzly, then we can flip that var | 13:44 |
| mkoderer | dkranz: ping | 13:45 |
| sdague | dprince actually has a patch for that, it just fails at the moment | 13:45 |
| maurosr | sdague: ok, will prepare a patch, that will have to land before the grenade | 13:45 |
| maurosr | oh cool | 13:45 |
| sdague | maurosr: hold on, what do you think you need to patch before :) | 13:46 |
| sdague | because this is tricky as everything is self gating | 13:46 |
| maurosr | do we have some kind of staging area to do this kind of stuff in infra? cause in the best world, we would test that before goes to real gate | 13:46 |
| sdague | so I expect we're going to have a sequence that needs to land | 13:46 |
| sdague | it's all self gating | 13:46 |
| *** YorikSar has quit IRC | 13:46 | |
| sdague | so if it doesn't pass, it can't land | 13:46 |
| sdague | all devstack gate changes are gatting on grenade runs, for instance | 13:47 |
| sdague | so I think as I currently understand it - https://review.openstack.org/#/c/57744/ needs to respond to dtroyer's questions | 13:47 |
| sdague | then that lands | 13:48 |
| sdague | then we can land the config change (probably?) | 13:48 |
| sdague | sorry the devstack-gate change | 13:48 |
| dkranz | mkoderer: In a meeting. Will ping you after. | 13:48 |
| maurosr | damn, didn't see his questions | 13:48 |
| mkoderer | dkranz: ok | 13:49 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_quotas into v3 part1 https://review.openstack.org/58689 | 13:51 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_quotas into v3 part2 https://review.openstack.org/58690 | 13:51 |
| jog0 | sdague: what do you think of adding https://github.com/lyda/misspell-check to hacking? | 13:52 |
| maurosr | sdague: answered dtroyer_zz comments, not sure, but he will problably ask to remove the changes in the {BASE,TARGET}_RELEASE from d-g.. just guessing | 13:52 |
| jog0 | example output http://paste.openstack.org/show/54354 | 13:52 |
| jog0 | or maybe a seperate job | 13:53 |
| dkranz | jog0: I like it | 13:55 |
| openstackgerrit | Alexei Kornienko proposed a change to openstack-dev/devstack: Added ceilometer-agent-notification service https://review.openstack.org/59723 | 13:56 |
| sdague | jog0: so, I'm really not ok with -2ing a security patch because of a spelling error | 13:57 |
| sdague | which is what it would do if we put it in hacking | 13:58 |
| *** aguzikova has joined #openstack-qa | 13:58 | |
| jog0 | sdague: so you can just ammend the patch, and this uses a list of commonly misspelled words, and i have yet to see a real false positive | 13:58 |
| sdague | and, honestly, I'm not convinced that this would save anyone but lifeless review time | 13:59 |
| sdague | jog0: it's not the false positive problem | 13:59 |
| jog0 | sdague: it would save review bandwidth in nova (by a tiny tiny fraction) | 13:59 |
| sdague | it's the fact that if someone jumps the gate | 13:59 |
| sdague | because they need to land a patch for a critical reason | 14:00 |
| *** SergeyLukjanov has quit IRC | 14:00 | |
| sdague | they'll get bounced over a spelling error | 14:00 |
| sdague | which I don't agree with | 14:00 |
| *** aguzikova has quit IRC | 14:00 | |
| jog0 | sdague: ahh I see what your saying, we have that for hacking as is | 14:00 |
| *** aguzikova has joined #openstack-qa | 14:00 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/59554 failed tempest because of: https://bugs.launchpad.net/bugs/1210483 | 14:00 |
| sdague | jog0: yes, and there is arguably a case where we shouldn't be bouncing on commit messages for that same reason | 14:00 |
| *** mkollaro has joined #openstack-qa | 14:00 | |
| jog0 | I think the situation your raising is easily reconciled | 14:01 |
| jog0 | just run 'tox -epep8' locally and fix accordingly | 14:01 |
| *** jasondotstar has joined #openstack-qa | 14:01 | |
| sdague | jog0: I think there would be plenty of infrastructure to get this right | 14:01 |
| sdague | and honestly, it seems really low priority | 14:02 |
| *** mkollaro1 has quit IRC | 14:02 | |
| jog0 | sdague: how so? with regard to infara | 14:02 |
| sdague | like 1000x less important than live upgrade testing | 14:02 |
| *** psharma has quit IRC | 14:02 | |
| jog0 | the patch to do this would take 20 minutes | 14:02 |
| jog0 | and I would let others actually fix the bugs | 14:02 |
| *** inkerra__ has joined #openstack-qa | 14:02 | |
| sdague | jog0: honestly... I think it's an overreach, and I already think there are plenty of folks that think hacking is overreaching as is | 14:03 |
| *** inkerra__ has quit IRC | 14:03 | |
| *** inkerra_ has joined #openstack-qa | 14:03 | |
| *** inkerra_ has quit IRC | 14:04 | |
| *** aguzikova has quit IRC | 14:04 | |
| *** reaper has joined #openstack-qa | 14:04 | |
| *** aguzikov1 has quit IRC | 14:04 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer https://review.openstack.org/58697 | 14:04 |
| *** inkerra has quit IRC | 14:05 | |
| dkranz | sdague: Agree with your comment. But this would be of great value to individuals who want t o run it before submitting. | 14:05 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/56751 failed tempest with an unrecognized error | 14:05 |
| sdague | anyway, I'm a soft -1 on the idea. Especially as part of hacking. Because hacking doesn't currently dictate spelling. It seems like this needs to be a different job | 14:05 |
| dkranz | sdague: A useful tool, not part of the gate. | 14:05 |
| *** inkerra has joined #openstack-qa | 14:05 | |
| *** gszasz has quit IRC | 14:05 | |
| jog0 | dkranz: exactly, if this reduces review churneven a tiny percent its worth it | 14:05 |
| sdague | jog0: actually, I think this would generate more churn | 14:06 |
| dkranz | jog0: Yes, and I was the one that started the ml flame war about the spelling issue | 14:06 |
| sdague | like when the license thing was brought up | 14:06 |
| sdague | and it actually meant more review overhead because now everyone was confused | 14:06 |
| jog0 | sdague: let me define what I mean by churn: number of back and forts between reviewers and patch authors | 14:06 |
| *** SushilKM has joined #openstack-qa | 14:06 | |
| dkranz | sdague: How could it generate churn? It just improves the quality of what gets pushed in the first place. | 14:06 |
| sdague | jog0: yep, this would create more | 14:06 |
| sdague | dkranz: because that list is a small list of common misspellings | 14:07 |
| sdague | but then you are going to get reviewers fighting about other misspellings | 14:07 |
| dkranz | sdague: Better than nothing | 14:07 |
| dkranz | sdague: I would gladly run such a script before doing git review | 14:07 |
| sdague | and reviewers thinking it's ok to -1 patches on spellings that don't really matter | 14:07 |
| sdague | then it will be on gramar nits, and capitalization | 14:07 |
| sdague | and it will generate more review churn | 14:08 |
| sdague | there are secondary effects by adding something like this you've declared open season that deep english construction is -1able | 14:08 |
| *** inkerra has quit IRC | 14:08 | |
| *** inkerra has joined #openstack-qa | 14:09 | |
| jog0 | sdague: yeah the secondary effects could be bad I agree | 14:09 |
| dkranz | sdague: Well I would explicitly say it is not ok to do that, and provide this tool to help people with spelling | 14:09 |
| dkranz | But it's not a big deal. | 14:09 |
| dkranz | So never mind. | 14:09 |
| jog0 | dkranz sdague: I only brought it up because I have reviewed a bunch of typo fix patches recently | 14:09 |
| sdague | yeh, I would rather just punt on this. Because otherwise I'm going to need to get into lots of fights with lifeless again. :) | 14:09 |
| jog0 | sdague: on secondary issues I assume | 14:10 |
| *** mriedem has joined #openstack-qa | 14:10 | |
| sdague | jog0: yeh | 14:10 |
| dkranz | jog0: Why not just let people know that a spell checker for code exists and leave it at that? | 14:10 |
| jog0 | sdague: for now I just filed https://bugs.launchpad.net/oslo/+bug/1257295 | 14:10 |
| jog0 | dkranz: sounds like a good middle ground | 14:10 |
| *** markmcclain has quit IRC | 14:10 | |
| mriedem | ha | 14:11 |
| dkranz | mkoderer: ping | 14:11 |
| mriedem | me fail english? that's unpossible. | 14:11 |
| sdague | mriedem: you missed the fun part of that conversation, you need an irc proxy solution :) | 14:11 |
| dkranz | And we should apply a spell checker to the irc bot | 14:11 |
| sdague | because, let me say, I find review comments about capitalizing git commit messages *not useful* | 14:12 |
| dkranz | sdague: +1 | 14:12 |
| mriedem | wow | 14:12 |
| dkranz | sdague: The flame war I started about this was never resolved | 14:12 |
| mriedem | link to the review? | 14:12 |
| *** tkammer has quit IRC | 14:13 | |
| sdague | and I think that beating people up about this stuff too much actually just drives down participation rates by non native english speakers | 14:13 |
| sdague | mriedem: any review of mine that lifeless has commented on :) | 14:13 |
| *** tkammer has joined #openstack-qa | 14:13 | |
| sdague | there are probably a few up there now | 14:13 |
| dkranz | sdague: Yes, we should not beat them up. But letting them know about a spell checker can only help them IMO | 14:14 |
| mriedem | sdague: just do something like this now: tHiS pAtCh FiXeS sTuFf ReAl G00d | 14:14 |
| jog0 | mriedem: or camel case | 14:14 |
| sdague | is that like a paradox bomp? | 14:14 |
| dkranz | mkoderer: I have to go to the office. Will ping you later. | 14:14 |
| sdague | bomb | 14:14 |
| jog0 | mriedem: you forgot the period | 14:14 |
| mriedem | oops | 14:14 |
| *** inkerra_ has joined #openstack-qa | 14:14 | |
| sdague | o_O | 14:14 |
| *** inkerra_ has quit IRC | 14:15 | |
| *** krtaylor has quit IRC | 14:15 | |
| sdague | jog0: seriously, you are the man with trivial trolling threads? :) still working on those vim mode lines :) | 14:16 |
| jog0 | sdague: no I go tthose out of oslo | 14:17 |
| mriedem | there was a nice long thread about gitignore awhile back which i remembered because of this https://review.openstack.org/#/c/59072/ | 14:17 |
| jog0 | and am leaving it at that | 14:17 |
| mriedem | and jog0 was all over it | 14:17 |
| sdague | heh | 14:18 |
| *** gszasz has joined #openstack-qa | 14:18 | |
| *** dkranz has quit IRC | 14:18 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/58222 failed tempest with an unrecognized error | 14:19 |
| *** jhenner has quit IRC | 14:19 | |
| *** nosnos has quit IRC | 14:20 | |
| jog0 | maintaining 16 seperate files to do the same thing for infinite editors can get out of hand | 14:21 |
| sdague | agreed | 14:21 |
| jog0 | but thats how mordred bumps his stats up ;) | 14:21 |
| jog0 | so sending out an FYI email and leaving it at that | 14:22 |
| openstackgerrit | Alexei Kornienko proposed a change to openstack-dev/devstack: Added ceilometer-agent-notification service https://review.openstack.org/59723 | 14:23 |
| jog0 | anyway out for a bit o/ | 14:24 |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer's health monitor https://review.openstack.org/59729 | 14:26 |
| *** safchain has joined #openstack-qa | 14:26 | |
| *** reaper has quit IRC | 14:28 | |
| *** sthaha has quit IRC | 14:29 | |
| mordred | what did I do? | 14:31 |
| mordred | ooh. | 14:31 |
| mordred | so - mriedem, jog0 ... what if we took the opposite approach | 14:31 |
| mordred | and made .gitignore files something that o/r or o-i updated for us? | 14:32 |
| mordred | like, have two sections in the file, one with a comment heading of "Standard ignores" and one with a heading of "per-project ignores" | 14:32 |
| *** jhenner has joined #openstack-qa | 14:32 | |
| mriedem | and then you ignore eclipse files in the standard section for all projects | 14:33 |
| *** krtaylor has joined #openstack-qa | 14:33 | |
| *** salv-orlando has quit IRC | 14:33 | |
| mordred | we can put .tox .venv .project .*~ .*.swp .pyc *.egg-info in the standard section | 14:33 |
| mordred | yeah | 14:33 |
| mordred | and screw it - put all of the editors in there, who cares | 14:33 |
| mordred | but we do it once, in one place | 14:33 |
| mriedem | that would be fine with me | 14:33 |
| mordred | now - the most important question - do we do that in oslo-incubator or in openstack/requirements? | 14:34 |
| mordred | o/r syncs setup.py and submits things automatically, so it's almost already a "align your project meta crap" update | 14:34 |
| mordred | but having that file in that repo seems a little counter-intuitive | 14:34 |
| mriedem | o/r sounded more ready for that kind of thing | 14:34 |
| mriedem | agreed | 14:35 |
| mordred | screw it - let's put it there | 14:35 |
| mriedem | given the debates over auto-sync for o/i, might be good to get something going there, tip of the iceberg kind of thing | 14:35 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_server_metadata and test_server_personality into v3 part1 https://review.openstack.org/57590 | 14:35 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port test_server_metadata and test_server_personality into v3 part2 https://review.openstack.org/57591 | 14:35 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: cinder v2 api tests - part2 https://review.openstack.org/59733 | 14:35 |
| mordred | oh - like put it in o-i and then do auto-sync for the gitignore module | 14:36 |
| mriedem | yes | 14:36 |
| mordred | and make a whitelist file that lists which modules in o-i are good for auto-syncing? | 14:36 |
| mriedem | something like that | 14:36 |
| mordred | hrm | 14:36 |
| mordred | both and neither of these are both good and bad | 14:36 |
| mriedem | then if we ever want to start auto-syncing more o/i stuff, the tooling is in place | 14:36 |
| mriedem | like log or other utils | 14:36 |
| mordred | yeah. | 14:36 |
| mriedem | why do i feel like it's going to turn into the gay marriage debate? :) | 14:37 |
| *** sthaha has joined #openstack-qa | 14:37 | |
| *** sthaha has joined #openstack-qa | 14:37 | |
| mordred | because it is | 14:37 |
| mriedem | 'once we open the door....' | 14:37 |
| mordred | there are days I'm sad that we're so good at code copy syncing | 14:37 |
| mordred | on the other hand - I gotta say, the pure-library approach isn't exactly, you know, working for us | 14:37 |
| *** matel has joined #openstack-qa | 14:40 | |
| *** amotoki has quit IRC | 14:41 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: cinder v2 api tests - part2 https://review.openstack.org/59733 | 14:41 |
| *** sjcazzol_ has joined #openstack-qa | 14:41 | |
| sjcazzol_ | Someone, please could take a look to this review? https://review.openstack.org/#/c/57957 | 14:43 |
| *** sthaha has quit IRC | 14:44 | |
| *** SushilKM has quit IRC | 14:44 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59108 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 14:44 |
| sdague | mordred: so I missed that session, but I wonder if oslo changes actually ran devstack gate, if that would be helpful | 14:44 |
| sdague | it seems weird that we're actually going to make pecan more integrated into knowing it won't break openstack than oslo | 14:45 |
| *** zhikunliu has quit IRC | 14:47 | |
| mordred | sdague: oslo to me is a VERY weird duck - although I understand the arguments | 14:48 |
| mordred | it's the "we want to use this library before it's a library" thing | 14:48 |
| mordred | sdague: but to be fair, once they are libraries, we do add oslo.* to the gate | 14:48 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/56639 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 14:49 |
| sdague | mordred: maybe... I'm not convinced though | 14:52 |
| mordred | well - me either - BUT | 14:53 |
| mordred | the recent discussion of bumpig the major version in glanceclient may have me more convinced | 14:53 |
| mordred | because once our projects use an API, it's really hard to change to a version of a library that breaks that API | 14:53 |
| mordred | if it's a pip-installed depend | 14:53 |
| openstackgerrit | Shantanu Tushar proposed a change to openstack-infra/devstack-gate: Adds retry and timeout to git_fetch_at_ref Closes-Bug: 1252137 https://review.openstack.org/59742 | 14:54 |
| mordred | spekaing of - I think I've figured out what markwash needs to do for glanceclient - he's not going to be happy, but I think I have a path forward | 14:54 |
| sdague | yeh, I'm still not sure I feel like that holds for oslo, but there are subtleties | 14:54 |
| sdague | anyway, it's down my list of concerns right now | 14:54 |
| mordred | I think we need something akin to an overlapping release | 14:55 |
| mordred | so, like, he releases a 1.0 that ADDS all the new APIs he wants to make, but does not delete any of the old ones | 14:55 |
| mordred | then we start bumping the min-required version on all of the servers to >=1.0 | 14:56 |
| sdague | yep, which is deprecation :) | 14:56 |
| mordred | once that's done and released | 14:56 |
| mordred | then we can wait until that release is no longer supported - so 1 year | 14:56 |
| mordred | _then_ we can make a release that removes the deprecated API | 14:56 |
| sdague | yep | 14:56 |
| mordred | because there will be no supported openstack releases that need to consume it | 14:56 |
| *** sjcazzol_ has quit IRC | 14:57 | |
| *** krtaylor has quit IRC | 14:58 | |
| *** afazekas has joined #openstack-qa | 15:01 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59729 failed tempest with an unrecognized error | 15:04 |
| openstackgerrit | Shantanu Tushar proposed a change to openstack-infra/devstack-gate: Adds retry and timeout to git_fetch_at_ref https://review.openstack.org/59742 | 15:10 |
| *** rfolco has quit IRC | 15:12 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for extension: os-extended-volume of nova v2 api https://review.openstack.org/56364 | 15:14 |
| *** dkranz has joined #openstack-qa | 15:18 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/57591 failed tempest with an unrecognized error | 15:24 |
| *** rfolco has joined #openstack-qa | 15:33 | |
| dims | sdague, do you have a few mins to look at a theory about the large-ops failures? (bug 1254890) | 15:34 |
| *** krtaylor has joined #openstack-qa | 15:35 | |
| *** jlibosva has quit IRC | 15:36 | |
| sdague | dims: I will this afternoon, have to run to something else at the moment | 15:38 |
| dims | sdague, cool, will leave a few notes in the bug. thx | 15:39 |
| russellb | hey, you guys on board with https://blueprints.launchpad.net/nova/+spec/list-tested-apis ? | 15:39 |
| russellb | i'm fine with it if there's consensus on the direction | 15:40 |
| *** yfried has quit IRC | 15:43 | |
| openstackgerrit | Joe Hakim Rahme proposed a change to openstack/tempest: Makes the Swift middleware tests optional https://review.openstack.org/59759 | 15:47 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/59236 failed tempest because of: https://bugs.launchpad.net/bugs/1210483 | 15:53 |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: add tests for server detail and show with extensions https://review.openstack.org/56324 | 15:56 |
| *** yfried has joined #openstack-qa | 15:57 | |
| openstackrecheck | openstack/python-keystoneclient change: https://review.openstack.org/59735 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 15:59 |
| *** reaper has joined #openstack-qa | 16:01 | |
| *** dkranz has quit IRC | 16:02 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/56022 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 16:02 |
| *** SushilKM has joined #openstack-qa | 16:07 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/56364 failed tempest with an unrecognized error | 16:08 |
| *** mlavalle has joined #openstack-qa | 16:12 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/tempest: Ensure we check func() at least once after sleep() https://review.openstack.org/59769 | 16:13 |
| openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/59742 failed tempest with an unrecognized error | 16:14 |
| openstackgerrit | Yassine Lamgarchal proposed a change to openstack/tempest: Adds client and CRUD tests for Ceilometer alarms https://review.openstack.org/39237 | 16:15 |
| openstackgerrit | Yassine Lamgarchal proposed a change to openstack/tempest: Add client and tests for Ceilometer compute metering https://review.openstack.org/43481 | 16:15 |
| sdague | russellb: yes, +1 to that | 16:15 |
| sdague | sorry, I meant to respond to your previous ping on that... it fell off a queue | 16:15 |
| russellb | sdague: cool, thanks for the ack | 16:17 |
| *** dkranz has joined #openstack-qa | 16:17 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/57517 failed tempest with an unrecognized error | 16:24 |
| *** atiwari has joined #openstack-qa | 16:31 | |
| openstackgerrit | A change was merged to openstack/tempest: add some test for force delete: https://review.openstack.org/55977 | 16:36 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Refactor new options into a new config file. https://review.openstack.org/57744 | 16:36 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59312 failed tempest with an unrecognized error | 16:37 |
| *** marun has quit IRC | 16:37 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer's health monitor https://review.openstack.org/59729 | 16:37 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59759 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 16:42 |
| *** rdekel has quit IRC | 16:43 | |
| *** yassine has quit IRC | 16:49 | |
| *** mestery has quit IRC | 16:49 | |
| *** mestery has joined #openstack-qa | 16:50 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Rip out the coverage extension client from tempest https://review.openstack.org/59776 | 16:51 |
| *** SushilKM has quit IRC | 16:51 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/devstack-gate: Rip out the tempest coverage runs from devstack-gate https://review.openstack.org/59777 | 16:51 |
| *** DennyZhang has joined #openstack-qa | 16:52 | |
| *** marun has joined #openstack-qa | 16:57 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/56324 failed tempest with an unrecognized error | 16:58 |
| *** jlibosva has joined #openstack-qa | 17:03 | |
| *** gszasz has quit IRC | 17:04 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59769 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 17:05 |
| *** syerrapragada has quit IRC | 17:12 | |
| *** syerrapragada has joined #openstack-qa | 17:13 | |
| *** marun has quit IRC | 17:13 | |
| mkoderer | sdague: jfyi I will be on vacation next week until next year | 17:20 |
| mkoderer | sdague: and I will have very limited internet connection during that time ;) | 17:20 |
| *** tkammer has quit IRC | 17:22 | |
| *** krtaylor has quit IRC | 17:28 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: Increase DomU's memory https://review.openstack.org/59792 | 17:36 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: Increase DomU's memory https://review.openstack.org/59792 | 17:37 |
| openstackrecheck | openstack/swift change: https://review.openstack.org/59778 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 17:37 |
| *** rhsu has joined #openstack-qa | 17:38 | |
| *** fbo is now known as fbo_away | 17:38 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/59219 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 17:40 |
| *** SushilKM has joined #openstack-qa | 17:40 | |
| *** yfried has quit IRC | 17:42 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59729 failed tempest with an unrecognized error | 17:51 |
| *** dkranz has quit IRC | 17:55 | |
| *** mkollaro has quit IRC | 17:59 | |
| jeblair | mtreinish: what does "The coverage extension has been removed from nova" mean? | 18:00 |
| jeblair | mtreinish: i mean, i think i understand the plain english meaning there, but what other nuances are there? | 18:01 |
| jeblair | mtreinish: is it replaced with other functionality; no longer needed for some reason; or people just gave up on it; etc...? | 18:01 |
| jeblair | sdague, russellb: ^ maybe one of you can enlighten me? | 18:02 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59787 failed tempest because of: https://bugs.launchpad.net/bugs/1244255 and https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/1224001 | 18:02 |
| *** SushilKM has quit IRC | 18:02 | |
| *** markmcclain has joined #openstack-qa | 18:03 | |
| *** rossella_s has quit IRC | 18:03 | |
| *** yfried has joined #openstack-qa | 18:06 | |
| *** dkranz has joined #openstack-qa | 18:07 | |
| *** SushilKM has joined #openstack-qa | 18:08 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/59784 failed tempest with an unrecognized error | 18:09 |
| *** SushilKM__ has joined #openstack-qa | 18:12 | |
| *** krtaylor has joined #openstack-qa | 18:13 | |
| *** SushilKM has quit IRC | 18:13 | |
| mtreinish | jeblair: sry I was at lunch. The coverage extension in nova had some bugs so it never really worked well | 18:14 |
| mtreinish | I have a plan for a better architecture, but I don't think I'll get a chance to implement it anytime soon | 18:14 |
| jeblair | mtreinish: ok, thanks | 18:15 |
| mtreinish | in the short term there is a bp for an api listing to see what endpoints get hit or something like that | 18:15 |
| mtreinish | let me dig that up | 18:15 |
| mtreinish | jeblair: https://blueprints.launchpad.net/nova/+spec/list-tested-apis | 18:15 |
| mtreinish | that will a be an interm replacement for the coverage extension. It doesn't give the same level of detail but it should work more reliably | 18:16 |
| *** DennyZhang has quit IRC | 18:17 | |
| yfried | dkranz: sdague, can someone please take a look at this patch - https://review.openstack.org/#/c/55101/ ? | 18:18 |
| yfried | it's been stuck in review limbo for a month now. | 18:18 |
| *** devlaps has joined #openstack-qa | 18:20 | |
| *** SushilKM__ has quit IRC | 18:23 | |
| *** markmcclain has quit IRC | 18:26 | |
| *** SushilKM has joined #openstack-qa | 18:27 | |
| *** SergeyLukjanov has joined #openstack-qa | 18:32 | |
| *** openstackgerrit has quit IRC | 18:33 | |
| *** openstackgerrit has joined #openstack-qa | 18:33 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/elastic-recheck: add support for installing the web dashboard https://review.openstack.org/59289 | 18:41 |
| dkranz | yfried: Copy the copyright from tools/check_logs.py | 18:42 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: scenario cross_tenant_connectivity https://review.openstack.org/55101 | 18:42 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: scenario cross_tenant_connectivity https://review.openstack.org/55101 | 18:44 |
| yfried | dkranz: tnx. anything else on this patch? | 18:45 |
| openstackrecheck | openstack/swift change: https://review.openstack.org/59219 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 18:45 |
| dkranz | yfried: I am still looking at it. | 18:45 |
| *** safchain has quit IRC | 18:45 | |
| yfried | dkranz: can't there be a bot that enables you to ping/tag all core reviewers at once? | 18:46 |
| dkranz | yfried: There could be, but isn't. | 18:46 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/59797 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 18:47 |
| mtreinish | sdague, maurosr: have you seen this failure before? http://logs.openstack.org/61/59561/1/gate/gate-grenade-dsvm/8c1692b/console.html | 18:48 |
| maurosr | let me look.. | 18:49 |
| dkranz | yfried: I just got pulled so probably won't finish til your morning. | 18:50 |
| lifeless | sdague: jog0: I'm not convinced that automating spelling checking would be a win; some parts of OpenStack treat it as '0', some as '-1', none as '-2'; but automated checks are effectively -2 as they cause verification fails. | 18:51 |
| yfried | dkranz: np. It's not urgent but I am looking to push this because it's an important patch of us, and I think it's also important upstream. though considering the bug it revealed it will probably won't matter until it is fixed | 18:52 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: make ResultSet actually inherit from list https://review.openstack.org/59805 | 18:53 |
| *** dkranz has quit IRC | 18:53 | |
| sdague | lifeless: 100% agree | 18:56 |
| lifeless | sdague: jog0: Besides which, I don't catch sdague at spelling errors, only grammar :). | 18:58 |
| sdague | heh | 18:58 |
| maurosr | it failed in glance, before any upgrade.. | 19:00 |
| maurosr | looking for the reason | 19:00 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59729 failed tempest with an unrecognized error | 19:01 |
| *** che-arne has quit IRC | 19:05 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/51662 failed tempest with an unrecognized error | 19:10 |
| *** dkranz has joined #openstack-qa | 19:11 | |
| maurosr | mtreinish: can't realize what is wrong, the logs just stop during g-api on stack.sh.. | 19:12 |
| mtreinish | maurosr: yeah that's what I see too and the error log really isn't helpful: http://logs.openstack.org/61/59561/1/gate/gate-grenade-dsvm/8c1692b/logs/old/error.txt.gz | 19:12 |
| openstackrecheck | openstack/glance change: https://review.openstack.org/54252 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 19:14 |
| maurosr | and the devstack log suddenly stops http://logs.openstack.org/61/59561/1/gate/gate-grenade-dsvm/8c1692b/logs/old/devstacklog.txt.gz | 19:14 |
| mtreinish | maurosr: https://bugs.launchpad.net/devstack/+bug/1119428 and https://bugs.launchpad.net/devstack/+bug/1087131 | 19:15 |
| *** fbo_away is now known as fbo | 19:17 | |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59787 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1244255 | 19:21 |
| jog0 | lifeless: as you have already figured out by now, no one is arguing with you about enforcing spelling.[6~ | 19:22 |
| jog0 | wow patch 59787 just hit 4 bugs! | 19:23 |
| *** raies_ has quit IRC | 19:23 | |
| openstackgerrit | Kaitlin Farr proposed a change to openstack/tempest: Adds scenario tests for volume encryption https://review.openstack.org/58018 | 19:23 |
| lifeless | jog0: Good :) | 19:23 |
| mtreinish | jog0: I only count 3... | 19:24 |
| lifeless | jog0: I'm a huge fan of graceful context sensitive application of things that can't be automated. (Note that hacking isn't automated, it's punishment - there is a difference). | 19:24 |
| jog0 | mtreinish: wow I cannot count today | 19:24 |
| lifeless | jog0: pretty printing is automated. | 19:24 |
| mtreinish | although it seems to have done 3 twice | 19:24 |
| *** jlibosva has quit IRC | 19:25 | |
| jog0 | lifeless: I don't follow the difference between automation and punishment -- can't you automate punishment? | 19:25 |
| lifeless | jog0: you can, but thats sadism. | 19:26 |
| lifeless | jog0: consider golang - there are formatting rules every bit as particular as hacking; but there is a tool that actions them rather than than just complaining. | 19:26 |
| lifeless | S[6~o you run your code through it, and it's done. | 19:26 |
| jog0 | lifeless: ah I buy that, so thats step 2 there is autopep8 | 19:27 |
| jog0 | we need autohacking | 19:27 |
| *** dkranz has quit IRC | 19:28 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Add Savanna testing support https://review.openstack.org/57325 | 19:29 |
| lifeless | basically, things that don't need humans to do shouldn't use humans to do them | 19:35 |
| lifeless | things that we still need humans to do, should be done by humans. | 19:35 |
| sdague | jog0: we need unit testing in hacking first :) | 19:36 |
| sdague | and we need live upgrade testing before that | 19:36 |
| lifeless | hell yes | 19:36 |
| jog0 | sdague: I was joking mostly | 19:36 |
| sdague | sure... but I can still make fun :) | 19:37 |
| jog0 | so upgrade testing is funny now? | 19:37 |
| sdague | always | 19:40 |
| jog0 | ^_^ | 19:41 |
| sdague | what is more funny is me writing client side faceting so that we can do nested facets with ER | 19:41 |
| *** dkranz has joined #openstack-qa | 19:41 | |
| jog0 | oh god | 19:42 |
| openstackgerrit | adalbas proposed a change to openstack/tempest: Separate negative tests for list_floating_ips https://review.openstack.org/59815 | 19:50 |
| sdague | jog0: recursion for the win | 19:52 |
| sdague | jog0: so I built this new class called FacetSet | 19:53 |
| jog0 | sdague: why don't you push that upstream to elasticsearch ;) | 19:54 |
| sdague | and you can feed it an er ResultSet, plus a list of facets, and it will recursively build you the FacetSet hierarchy with ResultSet as leaves | 19:54 |
| sdague | jog0: I'm finishing the tests first :) | 19:54 |
| sdague | then, yes | 19:54 |
| sdague | https://review.openstack.org/#/c/59805/ is the lead up | 19:55 |
| jog0 | sdague: cool | 19:56 |
| jog0 | I am a little surprised someone else hasn't done this yet for es | 19:56 |
| openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 19:56 |
| sdague | yeh, well I asked on the es channel | 19:57 |
| openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 19:57 |
| sdague | they said multi level facets weren't even in their plans | 19:57 |
| jog0 | lol | 19:58 |
| jog0 | I'm always surprised when we hit new ground on things | 19:59 |
| jog0 | what was there reaction to your idea? | 19:59 |
| *** yfried has quit IRC | 19:59 | |
| *** yfried has joined #openstack-qa | 19:59 | |
| sdague | jog0: just that it wasn't in their plans | 20:01 |
| jog0 | I was hoping for something more like - your crazy or oh yeah neat | 20:02 |
| *** rockyg has joined #openstack-qa | 20:03 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove queries for bugs 1225024 and 1225664 https://review.openstack.org/59816 | 20:07 |
| *** salv-orlando has joined #openstack-qa | 20:17 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: implementation of FacetSet for client side nested facets https://review.openstack.org/59822 | 20:24 |
| sdague | jog0: ok, there is the change | 20:25 |
| *** SushilKM has quit IRC | 20:27 | |
| *** salv-orlando_ has joined #openstack-qa | 20:28 | |
| *** salv-orlando has quit IRC | 20:30 | |
| *** salv-orlando has joined #openstack-qa | 20:31 | |
| *** salv-orlando_ has quit IRC | 20:33 | |
| dkranz | mtreinish: The allow_tenant_reuse conf flag is never used. Is there a reason not to delete it? | 20:42 |
| *** rockyg has quit IRC | 20:42 | |
| *** jasondotstar has quit IRC | 20:45 | |
| mtreinish | dkranz: yeah, I left it around because I intended to look into making it useful at one point | 20:45 |
| mtreinish | but as it sits now it's probably safe to remove it | 20:45 |
| dkranz | mtreinish: ok, I'll do that. I am preparing an email about some isolation issues discussed in https://review.openstack.org/#/c/54948/ | 20:47 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Cleanup graphite graph https://review.openstack.org/59829 | 20:47 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Remove check jobs from graphite graph https://review.openstack.org/59830 | 20:47 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add more gate jobs to graphite graph https://review.openstack.org/59831 | 20:47 |
| jog0 | sdague: here are mine ^ | 20:47 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/56263 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 and https://bugs.launchpad.net/bugs/1210483 | 20:47 |
| mriedem | mtreinish: this seems almost too easy https://review.openstack.org/#/c/59635/ | 20:47 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: port check_success to facets https://review.openstack.org/59832 | 20:48 |
| openstackrecheck | openstack/swift change: https://review.openstack.org/58396 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 20:48 |
| jog0 | hmm the postgres job fails a lot more then mysql | 20:49 |
| jog0 | sdague: ^ | 20:49 |
| mriedem | jog0: seeing migration failures for postgres here too https://review.openstack.org/#/c/58384/ | 20:50 |
| mriedem | well, and mysql i guess, in differen't migrations | 20:50 |
| mriedem | and only ever on py27 | 20:50 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Remove unused allow_tenant_reuse flag https://review.openstack.org/59833 | 20:51 |
| *** salv-orlando has quit IRC | 20:51 | |
| jog0 | current numbers for gate (not check) are dsvm-full:0%, dsvm-neutron:0, dsvm-neutron-large-ops=2.94% dsvm-large-ops: 0% dsvm-postgres-full :8% | 20:51 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Fix a minor error in method _get_alternative_flavor https://review.openstack.org/59635 | 20:51 |
| mtreinish | mriedem: heh, yeah although I don't think that condition gets hit in the gate | 20:52 |
| mtreinish | which is probably why it's not caught | 20:52 |
| mriedem | mtreinish: that's what i was wondering | 20:52 |
| mriedem | trivial fix so i fixed his commit message to reference the bug | 20:52 |
| sdague | jog0: well that's just because those jobs reset yesterday | 20:53 |
| jog0 | sdague: I don't think so because those are only 5 hour moving averages | 20:54 |
| sdague | ok | 20:54 |
| jog0 | mriedem: just saw a migration fail in the gate | 20:54 |
| jog0 | https://jenkins02.openstack.org/job/gate-nova-python27/11635/console | 20:54 |
| sdague | ok, how many neutron jobs actually went to the gate in that time? | 20:55 |
| jog0 | sdague: excellent question | 20:55 |
| mriedem | jog0: looks the same as that check failure | 20:55 |
| sdague | I guess my question is it's easy to get 0% when your numerator is zero | 20:55 |
| mriedem | the migrations are always failing on py27 for whatever reason | 20:55 |
| jog0 | sdague: do you know how to answer that question? | 20:55 |
| jog0 | mriedem: migration 142 here | 20:56 |
| sdague | in graphite or er? | 20:56 |
| jog0 | sdague: yes | 20:57 |
| jog0 | ideally graphite though | 20:57 |
| jog0 | so we could overlay that information on the same graph | 20:57 |
| sdague | honestly, graphite doesn't make a ton of sense to me most of the time | 20:57 |
| jog0 | actually I think I can do it | 20:58 |
| mriedem | sdague: i thought i was the only one but didn't want to say that :) | 20:58 |
| sdague | my inclination is to take the data back in, do the analysis in numpy, and build something with matplotlib | 20:58 |
| jog0 | sdague: I can't argue with that | 20:58 |
| jog0 | sdague: this was a short path to get something | 20:59 |
| mtreinish | sdague: +1 I like numpy and matplotlib | 20:59 |
| jog0 | mriedem: message:"Failed to migrate to version" | 20:59 |
| jog0 | logstash that | 20:59 |
| mriedem | k | 21:00 |
| mtreinish | jog0: maybe check that it's the postgres job too? | 21:00 |
| mriedem | mtreinish: it's not just postgres | 21:00 |
| mtreinish | oh ok | 21:00 |
| jog0 | mtreinish: those are py27 uni ttest failures | 21:00 |
| mtreinish | nm then | 21:00 |
| mriedem | mysql fails and postgres fails in py27 | 21:00 |
| mriedem | gate-nova-python27 | 21:00 |
| sdague | mtreinish: jeblair made a good point, we should never put job names in our bugs | 21:00 |
| mtreinish | jog0: yeah but they run both on postgres and mysql | 21:01 |
| mtreinish | but I incorrectly assumed that it was postgres failures only | 21:01 |
| mtreinish | sdague: yeah that's a good thing to enforce | 21:01 |
| mtreinish | it's not really specific enough | 21:02 |
| jog0 | speaking of bugs, mtreinish lets open one for nova unit tests | 21:02 |
| jog0 | I am logstashing to see how far back it goes | 21:02 |
| mriedem | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiRmFpbGVkIHRvIG1pZ3JhdGUgdG8gdmVyc2lvblwiIEFORCBmaWxlbmFtZTpcImNvbnNvbGUuaHRtbFwiIiwiZmllbGRzIjpbXSwib2Zmc2V0IjowLCJ0aW1lZnJhbWUiOiJjdXN0b20iLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsiZnJvbSI6IjIwMTMtMTEtMDVUMjE6MDE6MjQrMDA6MDAiLCJ0byI6IjIwMTMtMTItMDNUMjE6MDE6MjQrMDA6MDAiLCJ1c2VyX2ludGVydmFsIjoiMCJ9LCJzdGFtcCI6MTM4NjEwNDUzMzA5M30= | 21:03 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59561 failed tempest with an unrecognized error | 21:03 |
| mriedem | looks like 11/20, not sure why i keep seeing 11/20 in logstash today? is it me? coincidence? | 21:03 |
| *** dkorn has joined #openstack-qa | 21:03 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/58018 failed tempest with an unrecognized error | 21:03 |
| *** rockyg has joined #openstack-qa | 21:03 | |
| jog0 | mriedem: 2 weeks of logs in logstash | 21:03 |
| jog0 | clarkb: ^ | 21:04 |
| jog0 | we have a bug to dump this into? | 21:05 |
| jog0 | hmm its a timeout | 21:06 |
| jog0 | odd | 21:06 |
| jog0 | I think they may be taking too long | 21:07 |
| mriedem | jog0: here is an old postgres one: https://bugs.launchpad.net/nova/+bug/1216851 | 21:09 |
| mriedem | that's the same error (different migration script) that we're seeing now for postgres | 21:09 |
| jog0 | so itsa timeout | 21:09 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/56059 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 21:09 |
| mtreinish | jog0: have you seen this before?: http://logs.openstack.org/61/59561/1/gate/gate-tempest-dsvm-postgres-full/bf42017/logs/screen-n-cpu.txt.gz?level=TRACE#_2013-12-03_20_24_49_272 | 21:10 |
| mriedem | jog0: mark it critical? | 21:10 |
| jog0 | we have something for that | 21:10 |
| jog0 | I Forgot where | 21:10 |
| jog0 | its on the ML | 21:10 |
| jog0 | see [openstack-dev][nova][libvirt] Gate bug 'libvirtError: Unable to read from monitor: Connection reset by peer' | 21:10 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59815 failed tempest because of: https://bugs.launchpad.net/bugs/1177134 and https://bugs.launchpad.net/bugs/1250168 | 21:11 |
| mtreinish | jog0: cool thanks | 21:12 |
| openstackrecheck | openstack/python-cinderclient change: https://review.openstack.org/59576 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 21:12 |
| jog0 | the libvirt thing is a biggger issue which libvirt is stacktracing ... freakout | 21:12 |
| *** weshay has quit IRC | 21:13 | |
| jog0 | fungi: http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiRmFpbGVkIHRvIG1pZ3JhdGUgdG8gdmVyc2lvblwiIiwiZmllbGRzIjpbXSwib2Zmc2V0IjowLCJ0aW1lZnJhbWUiOiJhbGwiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxMzg2MTA0NTA0MTkyfQ== | 21:13 |
| fungi | so what's the deal with a nova unit test timeout? | 21:13 |
| fungi | ahh | 21:13 |
| jog0 | its been happening for a long time now | 21:13 |
| jog0 | for both postgres and mysql | 21:13 |
| jog0 | although there heas been an uptic recently | 21:14 |
| fungi | oh, you said nova unit test | 21:14 |
| jog0 | mriedem: I will hijack bug 1216851 | 21:14 |
| mriedem | jog0: be my guest :) | 21:14 |
| fungi | ahh, right, the migrations in nova test suite | 21:15 |
| jog0 | yeah so I am wondering if its the boxes are too small | 21:16 |
| jog0 | but we have no insight into that | 21:16 |
| jog0 | which is why I pinged infra | 21:16 |
| fungi | too small in what dimension? | 21:16 |
| jog0 | fungi: well lets take an example http://logs.openstack.org/62/55362/7/gate/gate-nova-python27/7f5147f/console.html | 21:17 |
| jog0 | http://logs.openstack.org/62/55362/7/gate/gate-nova-python27/7f5147f/console.html#_2013-12-03_20_34_44_569 | 21:19 |
| jog0 | is the error being raised | 21:19 |
| fungi | what timeout is being enforced for that? a per-migration timeout? | 21:19 |
| jog0 | and there is a huge timegap at the beginning of the stacktrace http://logs.openstack.org/62/55362/7/gate/gate-nova-python27/7f5147f/console.html#_2013-12-03_20_34_44_547 | 21:19 |
| jog0 | fungi: not sure yet | 21:20 |
| jog0 | I was wondering if any way to rule out load on the unit test machines or get other logs | 21:20 |
| mtreinish | dkranz: thanks for posting the meeting log on that review. I had forgotten some of the details from the previous discussion and that we decided against what Chris proposed in his review. | 21:20 |
| dkranz | mtreinish: I remembered part of it because I really didn't like what you ended up with for the same reason Chris didn't like it. | 21:21 |
| fungi | jog0: possibly. i can jump onto that particular machine and just see if there are stale/cruft processes left hehind by a previous test somehow | 21:21 |
| dkranz | mtreinish: But I didn't have a solid alternative. | 21:21 |
| dkranz | mtreinish: The issue was the one that Sean raised about what happens when a new extension is added. | 21:21 |
| dkranz | mtreinish: This is all kind of a mess. | 21:21 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/59798 failed tempest because of: https://bugs.launchpad.net/bugs/1249065 | 21:22 |
| jog0 | fungi: so there has been a major uptic in these bugs | 21:22 |
| jog0 | starting on ... | 21:22 |
| jog0 | 12/2 | 21:22 |
| jog0 | only 47 cases in last week | 21:22 |
| jog0 | for some context | 21:22 |
| fungi | does logstash give us any wat to correlate by the worker where that job ran? | 21:23 |
| jog0 | so anything change on 12/2? | 21:23 |
| jog0 | fungi: only if clarkb put that in | 21:23 |
| jog0 | i don't see it | 21:23 |
| fungi | yeah, not spotting it | 21:23 |
| jog0 | fungi: does the log_url reveal that | 21:23 |
| fungi | nope | 21:24 |
| jog0 | oh waita lot of these are for a single review | 21:24 |
| mtreinish | dkranz: I don't think there is a right answer. No matter what it'll be a tradeoff. Personally, I think we should go for being the most explicit even if the UI suffers | 21:24 |
| sdague | jog0: I think that on 12/2 we finally got back to normal flow | 21:24 |
| jog0 | 12% for a single hange | 21:24 |
| jog0 | sdague: ah post thanksgiving | 21:24 |
| openstackrecheck | openstack/oslo.messaging change: https://review.openstack.org/59757 failed tempest with an unrecognized error | 21:25 |
| dkranz | mtreinish: I think that is what you proposed and chris objected. BUt you said you were adding it to the agenda again. | 21:25 |
| fungi | my gut says some enterprising nova dev finally got a change through yesterday to improve migrations in such a way that they now occasionally time out ;) | 21:25 |
| mtreinish | dkranz: because I forgot the previous discussion (vacation...) | 21:25 |
| mtreinish | I'll update the review again | 21:25 |
| jog0 | so there is no check vs gate for unit tests ... | 21:25 |
| dkranz | mtreinish: :) | 21:25 |
| dkranz | mtreinish: Too much sake? | 21:26 |
| mtreinish | dkranz: the other option is to use multistr instead of list which I think would be an even worse UI | 21:26 |
| mtreinish | dkranz: :) yeah a bit too much | 21:26 |
| fungi | precise34, where our example job bombed earlier, is currently running the same job and has load hovering around 2.5-3.5 on a quad-vcpu vm | 21:26 |
| dkranz | mtreinish: Yeah, I don't feel very strongly about this because they are all kind of ugly | 21:26 |
| fungi | and precise34 finished running that job and its load average is dropping like a stone (already below 0.5 | 21:28 |
| jog0 | fungi: any funny errors from postgres or mysql? | 21:30 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59729 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 21:32 |
| clarkb | fungi: jog0: you can look at the console log to see where the job ran | 21:33 |
| clarkb | but jenkins doesn;t output that info in theevent plugins so we don't get it directly | 21:33 |
| fungi | clarkb: right, just trying to figure out if there's a way for kibana to query for alist of the slaves where this error was witnessed | 21:34 |
| fungi | guess not | 21:34 |
| clarkb | fungi: you can do two queries, first get a list of all occurencse then check console logs of all of those occurences, you will want ot script that | 21:34 |
| fungi | right | 21:34 |
| jog0 | anyway updated 1216851, being I am in UTC+2 | 21:35 |
| jog0 | g'night | 21:35 |
| fungi | oh, right, forgot you were out there | 21:36 |
| fungi | plenty of "fatal" entries in the postgres log on that slave from when those migrations were running | 21:37 |
| fungi | things like 2013-12-03 20:24:31 UTC FATAL: database "openstack_baremetal_citest" does not exist | 21:37 |
| fungi | 2013-12-03 20:24:31 UTC FATAL: password authentication failed for user "openstack_cifail" | 21:37 |
| fungi | 2013-12-03 20:31:12 UTC FATAL: pg_hba.conf rejects connection for host "127.0.0.1", user "postgres", database "postgres", SSL off | 21:38 |
| fungi | since i'm unfamiliar with the internals of nova's migrations, i don't know whether this is normal | 21:38 |
| *** mkollaro has joined #openstack-qa | 21:47 | |
| jog0 | I would hope we aren't hitting fatal things | 21:47 |
| *** fbo is now known as fbo_away | 21:47 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/59126 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 21:47 |
| jog0 | sdague: got the graphite query for jobs | 21:48 |
| jog0 | hitcount( sum(stats.zuul.pipeline.gate.job.gate-tempest-dsvm-full.{SUCCESS,FAILURE}),"5hours") | 21:48 |
| sdague | ah cool | 21:48 |
| jog0 | http://graphite.openstack.org/graphlot/?width=586&height=308&_salt=1386107323.619&target=hitcount(%20sum(stats.zuul.pipeline.gate.job.gate-tempest-dsvm-full.%7BSUCCESS%2CFAILURE%7D)%2C%221hours%22) | 21:49 |
| jog0 | anyway o/ | 21:49 |
| *** rockyg has quit IRC | 21:51 | |
| openstackgerrit | A change was merged to openstack/tempest: update to hacking 0.8 https://review.openstack.org/56308 | 21:58 |
| *** safchain has joined #openstack-qa | 21:59 | |
| *** safchain has quit IRC | 21:59 | |
| openstackrecheck | openstack/python-neutronclient change: https://review.openstack.org/59836 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 | 21:59 |
| *** krtaylor has quit IRC | 22:05 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/59533 failed tempest with an unrecognized error | 22:07 |
| *** yfried has quit IRC | 22:08 | |
| jog0 | sdague mtreinish: can you look at https://review.openstack.org/#/c/58765/ its a possible fix for bug 1253896 | 22:09 |
| sdague | jog0: I'm happy with it | 22:10 |
| sdague | hmmm... deleting 11k unread tempest review emails over IMAP... pegs thunderbird | 22:10 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59635 failed tempest with an unrecognized error | 22:13 |
| mtreinish | jog0: I'm fine with it too +A | 22:13 |
| jog0 | thanks that was easy | 22:18 |
| jog0 | now we wait and see if that helps | 22:18 |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/57715 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 22:22 |
| *** salv-orlando has joined #openstack-qa | 22:33 | |
| *** mriedem has quit IRC | 22:39 | |
| openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 22:39 |
| openstackgerrit | Arnaud Legendre proposed a change to openstack-dev/devstack: Attempt to retrieve the vmdk descriptor data-pair https://review.openstack.org/58356 | 22:40 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59034 failed tempest with an unrecognized error | 22:41 |
| stevebaker | sdague: hey, could you take a look at my comment on https://review.openstack.org/#/c/59102/ ? | 22:41 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/58017 failed tempest with an unrecognized error | 22:50 |
| *** krtaylor has joined #openstack-qa | 22:50 | |
| *** dkranz has quit IRC | 22:53 | |
| openstackrecheck | openstack/heat change: https://review.openstack.org/59309 failed tempest with an unrecognized error | 22:57 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/57379 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 22:58 |
| *** jpich has quit IRC | 23:01 | |
| *** devlaps has quit IRC | 23:05 | |
| *** sc68cal has quit IRC | 23:17 | |
| sdague | stevebaker: sure... I guess my question is how are we vetting these in the gate? because they aren't running anywhere except that experimental job, which looks extra broken right now | 23:17 |
| *** sc68cal has joined #openstack-qa | 23:18 | |
| sdague | unless I'm missing something | 23:18 |
| sdague | https://review.openstack.org/#/c/59102/2/tempest/api/orchestration/stacks/test_server_cfn_init.py only has "slow" tests in it | 23:19 |
| stevebaker | sdague: they can't currently be vetted in gate. I've been blocked on the release of pip-1.4.2 which has been due any day for 3 months now (required to install diskimage-builder on devstack gate) | 23:21 |
| sdague | hmph | 23:22 |
| sdague | pip is Alex_Gaynor? | 23:22 |
| sdague | maybe I'll go haress him to break that log jam | 23:22 |
| stevebaker | sdague: but since they've never been gated on, but they are run outside of gating, it would be nice if they could continue to evolve. Fedora 20 is out in a week or so, which may mean dib isn't needed at all | 23:23 |
| sdague | oh, it's dstuft | 23:23 |
| sdague | stevebaker: do you guys have something able to report back in results on them? | 23:23 |
| openstackrecheck | openstack/cinder change: https://review.openstack.org/59698 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 23:24 |
| *** giulivo has quit IRC | 23:24 | |
| stevebaker | sdague: not yet, there is about to be a flurry of heat integration test activity | 23:24 |
| sdague | yeh, we've kept that stance about not integrating code we don't see executing somewhere for a reason, as it tends to break people a lot when we do | 23:25 |
| sdague | what's special about the f20 guest image? | 23:25 |
| stevebaker | it has heat-cfntools pre-installed, so no need for custom images | 23:25 |
| *** SergeyLukjanov has quit IRC | 23:26 | |
| stevebaker | its still an unknown if the slow tests run at all in gating, although it was never the intention that it would take this long | 23:27 |
| sdague | stevebaker: I've got dstuft active in -infra, can you jump in on that thread | 23:28 |
| sdague | this is to sort out pip story | 23:28 |
| sdague | we don't have a non pip path for dib? | 23:30 |
| sdague | I'd like to do this right if we can, because I think the longer we wait to get these jobs doing a thing, the harder it is going to be to get them there | 23:31 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/59561 failed tempest because of: https://bugs.launchpad.net/bugs/1177134 | 23:36 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/59110 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 23:38 |
| openstackrecheck | openstack/heat change: https://review.openstack.org/59111 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 23:43 |
| openstackrecheck | openstack/neutron change: https://review.openstack.org/59863 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 23:45 |
| *** atiwari has quit IRC | 23:49 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/59856 failed tempest because of: https://bugs.launchpad.net/bugs/1250168 | 23:50 |
| openstackgerrit | Paul Mathews proposed a change to openstack-dev/devstack: Implement matchmaker for ZeroMQ https://review.openstack.org/59875 | 23:50 |
| openstackgerrit | Mike Metral proposed a change to openstack-dev/devstack: Wait for docker to be up before connecting Closes-Bug: #1257536 https://review.openstack.org/59878 | 23:55 |
| *** openstackgerrit has quit IRC | 23:56 | |
| *** openstackgerrit has joined #openstack-qa | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!