| *** atiwari has quit IRC | 00:03 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for nova bug 1315201 https://review.openstack.org/91707 | 00:07 |
|---|---|---|
| *** kgriffs is now known as kgriffs|afk | 00:10 | |
| *** mriedem has quit IRC | 00:17 | |
| *** mmaglana has joined #openstack-qa | 00:22 | |
| *** TravT has quit IRC | 00:24 | |
| *** matsuhashi has joined #openstack-qa | 00:29 | |
| *** mestery has joined #openstack-qa | 00:37 | |
| *** Aimon has quit IRC | 00:44 | |
| *** jamielennox is now known as jamielennox|away | 00:53 | |
| *** dmorita has joined #openstack-qa | 00:55 | |
| *** mrodden has quit IRC | 01:00 | |
| openstackgerrit | Jun Wu proposed a change to openstack-dev/devstack: Better "exec 3>&1" comment https://review.openstack.org/90709 | 01:09 |
| *** yamahata has quit IRC | 01:10 | |
| *** rbak has joined #openstack-qa | 01:23 | |
| *** rbak has quit IRC | 01:24 | |
| *** Aimon has joined #openstack-qa | 01:27 | |
| *** Aimon has quit IRC | 01:28 | |
| *** jamielennox|away is now known as jamielennox | 01:28 | |
| *** rbak has joined #openstack-qa | 01:30 | |
| *** mriedem has joined #openstack-qa | 01:31 | |
| *** Aimon has joined #openstack-qa | 01:31 | |
| *** Aimon has joined #openstack-qa | 01:32 | |
| *** rbak has quit IRC | 01:34 | |
| *** nosnos has joined #openstack-qa | 01:36 | |
| *** nosnos has quit IRC | 01:44 | |
| *** nosnos has joined #openstack-qa | 01:44 | |
| *** weshay has quit IRC | 01:49 | |
| *** nati_uen_ has joined #openstack-qa | 01:55 | |
| *** torandu has quit IRC | 01:56 | |
| *** nati_uen_ has quit IRC | 01:56 | |
| *** torandu has joined #openstack-qa | 01:58 | |
| *** nati_ueno has quit IRC | 01:59 | |
| *** annegentle has joined #openstack-qa | 02:00 | |
| *** catohornet has quit IRC | 02:01 | |
| *** yamahata has joined #openstack-qa | 02:04 | |
| *** openstackgerrit has quit IRC | 02:04 | |
| *** openstackgerrit has joined #openstack-qa | 02:05 | |
| *** mrodden has joined #openstack-qa | 02:14 | |
| *** coolsvap|afk is now known as coolsvap | 02:19 | |
| *** mriedem has quit IRC | 02:28 | |
| openstackgerrit | David Paterson proposed a change to openstack/qa-specs: Add spec for bp:post-run-cleanup https://review.openstack.org/91725 | 02:31 |
| *** nati_ueno has joined #openstack-qa | 02:33 | |
| *** rbak has joined #openstack-qa | 02:36 | |
| *** rbak has quit IRC | 02:36 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/91575 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 02:40 |
| *** harlowja is now known as harlowja_away | 02:50 | |
| *** nosnos has quit IRC | 02:51 | |
| *** nati_ueno has quit IRC | 02:55 | |
| *** jerryz has joined #openstack-qa | 03:02 | |
| *** Abhi_ has joined #openstack-qa | 03:04 | |
| *** jerryz has quit IRC | 03:11 | |
| *** Ch00k has joined #openstack-qa | 03:18 | |
| *** matsuhashi has quit IRC | 03:27 | |
| *** mmaglana has quit IRC | 03:35 | |
| *** mmaglana has joined #openstack-qa | 03:36 | |
| *** marun has joined #openstack-qa | 03:37 | |
| *** mmaglana has quit IRC | 03:40 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/84037 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 03:50 |
| *** nati_ueno has joined #openstack-qa | 03:53 | |
| *** Ch00k has quit IRC | 04:01 | |
| *** nosnos has joined #openstack-qa | 04:05 | |
| *** coolsvap is now known as coolsvap|afk | 04:28 | |
| *** yfried has quit IRC | 04:29 | |
| *** coolsvap|afk is now known as coolsvap | 04:30 | |
| *** Ch00k has joined #openstack-qa | 04:31 | |
| *** chandan_kumar has joined #openstack-qa | 04:36 | |
| *** nati_ueno has quit IRC | 04:43 | |
| *** Ch00k has quit IRC | 04:44 | |
| *** Ch00k has joined #openstack-qa | 04:44 | |
| *** Ch00k has quit IRC | 04:45 | |
| *** yfried has joined #openstack-qa | 04:53 | |
| *** Madan has joined #openstack-qa | 04:55 | |
| *** matsuhashi has joined #openstack-qa | 05:03 | |
| *** malini is now known as malini_afk | 05:03 | |
| *** nati_ueno has joined #openstack-qa | 05:04 | |
| *** nati_ueno has quit IRC | 05:04 | |
| *** Anju has quit IRC | 05:07 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Fix logging for keystone under mod_wsgi https://review.openstack.org/91735 | 05:09 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Fix logging for keystone under mod_wsgi https://review.openstack.org/91735 | 05:12 |
| *** Anju has joined #openstack-qa | 05:33 | |
| *** lbragstad has joined #openstack-qa | 05:47 | |
| *** chandan_kumar has quit IRC | 05:47 | |
| openstackgerrit | Lance Bragstad proposed a change to openstack/tempest: Default domain ID to valid UUID format https://review.openstack.org/90773 | 05:51 |
| *** yfried has quit IRC | 05:53 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added deletion part in some test https://review.openstack.org/91747 | 05:56 |
| openstackgerrit | Ravikumar Venkatesan proposed a change to openstack/tempest: Test cases for database limits https://review.openstack.org/85320 | 05:57 |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Cleanup WSGI config for keystone on stop https://review.openstack.org/91748 | 05:58 |
| *** chandan_kumar has joined #openstack-qa | 06:01 | |
| *** morganfainberg is now known as morganfainberg_Z | 06:03 | |
| *** Madan has quit IRC | 06:04 | |
| *** sseverson has quit IRC | 06:08 | |
| *** lbragstad is now known as lbragstad_ | 06:09 | |
| *** Madan has joined #openstack-qa | 06:12 | |
| *** rwsu is now known as rwsu-away | 06:14 | |
| *** psharma has joined #openstack-qa | 06:14 | |
| openstackgerrit | anju Tiwari proposed a change to openstack/tempest: Added deletion part in some test https://review.openstack.org/91747 | 06:15 |
| *** thomasbiege has joined #openstack-qa | 06:27 | |
| *** thomasbiege has quit IRC | 06:28 | |
| *** chandan_kumar has quit IRC | 06:51 | |
| *** beisner has quit IRC | 06:55 | |
| *** beisner has joined #openstack-qa | 06:56 | |
| *** openstackgerrit has quit IRC | 06:57 | |
| *** chandan_kumar has joined #openstack-qa | 07:05 | |
| *** waterkinfe has joined #openstack-qa | 07:10 | |
| *** sthaha has quit IRC | 07:11 | |
| *** jamielennox is now known as jamielennox|away | 07:14 | |
| *** Madan has quit IRC | 07:18 | |
| openstackrecheck | openstack/glance change: https://review.openstack.org/90273 failed gate-glance-python26 in the gate queue with an unrecognized error | 07:20 |
| *** Madan has joined #openstack-qa | 07:20 | |
| *** qba73 has joined #openstack-qa | 07:20 | |
| *** giulivo has joined #openstack-qa | 07:26 | |
| openstackrecheck | openstack-infra/devstack-gate change: https://review.openstack.org/87677 failed because of: gate-grenade-dsvm-partial-ncpu: https://bugs.launchpad.net/bugs/1254890 | 07:28 |
| *** sthaha has joined #openstack-qa | 07:28 | |
| *** nati_ueno has joined #openstack-qa | 07:55 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/91318 failed because of: gate-grenade-dsvm-partial-ncpu: https://bugs.launchpad.net/bugs/1262566 | 07:55 |
| *** jordanP has joined #openstack-qa | 07:56 | |
| *** jpich has joined #openstack-qa | 07:57 | |
| *** waterkinfe has quit IRC | 08:02 | |
| *** stillbert is now known as mikal | 08:03 | |
| *** enikanorov has joined #openstack-qa | 08:17 | |
| *** qba73_ has joined #openstack-qa | 08:17 | |
| *** qba73_ has quit IRC | 08:17 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/87387 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1254890, gate-swift-python26: unrecognized error, gate-swift-unittests-func: unrecognized error | 08:19 |
| *** amotoki has quit IRC | 08:20 | |
| *** cyeoh has quit IRC | 08:21 | |
| *** qba73 has quit IRC | 08:21 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/86469 failed gate-swift-python26, gate-tempest-dsvm-neutron, gate-swift-unittests-func in the gate queue with an unrecognized error | 08:22 |
| *** Ju_ has joined #openstack-qa | 08:25 | |
| *** KurtMartin has joined #openstack-qa | 08:25 | |
| *** jordan_ has joined #openstack-qa | 08:25 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/91580 failed because of: gate-horizon-pep8: unrecognized error, gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1312199 https://bugs.launchpad.net/bugs/1254872 https://bugs.launchpad.net/bugs/1273292 | 08:26 |
| *** cyeoh has joined #openstack-qa | 08:26 | |
| *** beekneemech has joined #openstack-qa | 08:27 | |
| *** dhellmann_ has joined #openstack-qa | 08:33 | |
| *** Ju_ has quit IRC | 08:33 | |
| *** parstac_pete_ has joined #openstack-qa | 08:33 | |
| openstackrecheck | openstack/swift change: https://review.openstack.org/87265 failed gate-swift-python26, gate-swift-unittests-func in the gate queue with an unrecognized error | 08:34 |
| *** Madan has quit IRC | 08:34 | |
| *** psharma has quit IRC | 08:34 | |
| *** mrodden has quit IRC | 08:34 | |
| *** jhenner has quit IRC | 08:34 | |
| *** bnemec has quit IRC | 08:34 | |
| *** hemna__ has quit IRC | 08:34 | |
| *** asselin__ has quit IRC | 08:34 | |
| *** dhellmann has quit IRC | 08:34 | |
| *** peterstac has quit IRC | 08:34 | |
| *** dhellmann_ is now known as dhellmann | 08:34 | |
| *** jordanP has quit IRC | 08:34 | |
| *** Ju has quit IRC | 08:34 | |
| *** ianw has quit IRC | 08:34 | |
| *** enikanorov_ has quit IRC | 08:34 | |
| *** kmartin has quit IRC | 08:34 | |
| *** ianw has joined #openstack-qa | 08:35 | |
| *** parstac_pete_ is now known as peterstac | 08:36 | |
| *** amotoki has joined #openstack-qa | 08:36 | |
| *** dmorita has quit IRC | 08:38 | |
| *** chandan_kumar has quit IRC | 08:40 | |
| *** hemna__ has joined #openstack-qa | 08:42 | |
| *** Madan has joined #openstack-qa | 08:45 | |
| *** jhenner has joined #openstack-qa | 08:46 | |
| *** mrodden has joined #openstack-qa | 08:46 | |
| *** chandan_kumar has joined #openstack-qa | 08:53 | |
| *** psharma has joined #openstack-qa | 08:53 | |
| *** asselin__ has joined #openstack-qa | 08:53 | |
| *** gszasz has joined #openstack-qa | 08:55 | |
| *** rwsu-away has quit IRC | 08:56 | |
| *** nati_ueno has quit IRC | 08:56 | |
| openstackrecheck | openstack/ceilometer change: https://review.openstack.org/91049 failed gate-ceilometer-python26 in the gate queue with an unrecognized error | 09:00 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/84211 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 09:00 |
| *** matsuhashi has quit IRC | 09:04 | |
| *** matsuhashi has joined #openstack-qa | 09:05 | |
| *** yamahata has quit IRC | 09:30 | |
| *** Ju has joined #openstack-qa | 09:33 | |
| *** tosky has joined #openstack-qa | 09:35 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/90115 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1253896 https://bugs.launchpad.net/bugs/1273292 | 09:38 |
| *** safchain has joined #openstack-qa | 09:48 | |
| *** nati_ueno has joined #openstack-qa | 09:57 | |
| *** nati_ueno has quit IRC | 10:01 | |
| *** nati_ueno has joined #openstack-qa | 10:02 | |
| *** nati_uen_ has joined #openstack-qa | 10:04 | |
| *** yfried has joined #openstack-qa | 10:07 | |
| *** nati_ueno has quit IRC | 10:07 | |
| *** nati_uen_ has quit IRC | 10:09 | |
| *** asselin__ has quit IRC | 10:15 | |
| *** safchain has quit IRC | 10:20 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-docs 82051,3,2c5bb98 | 10:21 |
| openstackrecheck | openstack/sahara change: https://review.openstack.org/91067 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1314313 | 10:21 |
| *** giulivo has quit IRC | 10:24 | |
| *** giulivo has joined #openstack-qa | 10:27 | |
| *** matsuhashi has quit IRC | 10:35 | |
| *** aysyd has joined #openstack-qa | 10:35 | |
| *** matsuhashi has joined #openstack-qa | 10:37 | |
| *** aysyd has quit IRC | 10:41 | |
| *** nati_ueno has joined #openstack-qa | 10:53 | |
| *** nati_ueno has quit IRC | 10:58 | |
| *** qba73 has joined #openstack-qa | 10:59 | |
| *** coolsvap is now known as coolsvap|afk | 11:05 | |
| *** mmaglana has joined #openstack-qa | 11:08 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/87377 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1224518 | 11:09 |
| *** matsuhashi has quit IRC | 11:22 | |
| *** Madan has quit IRC | 11:28 | |
| *** psharma has quit IRC | 11:29 | |
| *** Abhi_ has quit IRC | 11:33 | |
| *** nosnos has quit IRC | 11:37 | |
| *** matsuhashi has joined #openstack-qa | 11:48 | |
| *** nosnos has joined #openstack-qa | 11:53 | |
| *** nati_ueno has joined #openstack-qa | 11:54 | |
| *** nati_ueno has quit IRC | 11:59 | |
| *** safchain has joined #openstack-qa | 12:15 | |
| *** coolsvap|afk is now known as coolsvap | 12:27 | |
| *** dkranz has joined #openstack-qa | 12:29 | |
| *** weshay has joined #openstack-qa | 12:29 | |
| sdague | dkranz: your habit of not +2ing the change caused interesting history reading (something's wrong with the old approved column after update)- https://review.openstack.org/#/c/78548/ | 12:33 |
| sdague | can I convince you to start putting +2 on changes when you +A as well :) | 12:33 |
| dkranz | sdague: I won't take much convincing. I simply did not know that was the protocol. | 12:33 |
| sdague | yeh, it is | 12:33 |
| sdague | as far as I know I think you are the only one not doing it :) | 12:33 |
| dkranz | sdague: No problem. I was completely oblivious to this issue. | 12:34 |
| sdague | it doesn't exactly hurt anything, except your own review stats I think | 12:34 |
| dkranz | sdague: OH................. | 12:34 |
| dkranz | sdague: I looked at that a few times and wondered why I was so far below. Is this why? | 12:35 |
| sdague | I don't know, could be | 12:35 |
| dkranz | sdague: ok, that makes my life easier :) | 12:35 |
| sdague | btw - https://review.openstack.org/#/c/91770/1 | 12:36 |
| sdague | I pushed a revert on the change in question. I was reading the test for a bug, and found that it didn't actually test anything that's not already tested | 12:37 |
| dkranz | sdague: I mentioned the strange missing +A to fungi and he said it only involved old reviews | 12:37 |
| sdague | yep | 12:37 |
| dkranz | sdague: I'll review ^^^ now | 12:37 |
| sdague | well the database schema changed, I'm wondering if we can bring over those comments | 12:38 |
| sdague | that might also explain why stackalytics has gone all crazy pants on review count | 12:38 |
| *** nosnos has quit IRC | 12:39 | |
| dkranz | sdague: fungi said he thought it was risky to try and do that | 12:41 |
| sdague | gotcha | 12:42 |
| dkranz | sdague: About the personality test there was a stated intent that checking with ssh was going to be done. If those checks were in there would you have wanted to revert this test? | 12:42 |
| sdague | if those checks were in there, then this would be a different test | 12:42 |
| sdague | and I'd be fine with it | 12:43 |
| sdague | but as is, it's duplicative | 12:43 |
| sdague | and costly | 12:43 |
| sdague | even in the best case this is probably a 40s test | 12:43 |
| dkranz | sdague: Yes. A personality check could be part of an existing snapshot or scenario test. | 12:43 |
| sdague | yep | 12:44 |
| *** nati_ueno has joined #openstack-qa | 12:44 | |
| sdague | or, it could be it's own thing. I don't really care, except that in the current format it's not helpful | 12:44 |
| *** nati_ueno has quit IRC | 12:48 | |
| fungi | fwiw i suggested that updating all the approve votes to workflow +1 ran the risk of screwing up timestamps on the votes, but apparently that happened anyway because of the upgrade script | 12:54 |
| sdague | heh | 12:57 |
| *** matsuhashi has quit IRC | 13:04 | |
| *** matsuhashi has joined #openstack-qa | 13:04 | |
| *** markmcclain has joined #openstack-qa | 13:05 | |
| *** matsuhas_ has joined #openstack-qa | 13:08 | |
| *** matsuhashi has quit IRC | 13:08 | |
| *** lbragstad_ is now known as lbragstad | 13:15 | |
| *** matsuhas_ has quit IRC | 13:18 | |
| *** mriedem has joined #openstack-qa | 13:21 | |
| *** dansmith is now known as superdan | 13:23 | |
| openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/91412 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 13:24 |
| *** qba73 has quit IRC | 13:27 | |
| *** jgrimm has joined #openstack-qa | 13:31 | |
| *** Obulpathi has joined #openstack-qa | 13:36 | |
| *** Obulpathi has quit IRC | 13:36 | |
| *** Obulpathi has joined #openstack-qa | 13:37 | |
| *** sthaha has quit IRC | 13:44 | |
| *** nati_ueno has joined #openstack-qa | 13:45 | |
| *** nati_ueno has quit IRC | 13:50 | |
| *** krtaylor has quit IRC | 13:56 | |
| *** coolsvap is now known as coolsvap|afk | 13:56 | |
| *** yfried has quit IRC | 13:56 | |
| *** sthaha has joined #openstack-qa | 13:56 | |
| *** sthaha has joined #openstack-qa | 13:56 | |
| *** jaypipes has joined #openstack-qa | 14:01 | |
| *** ChanServ sets mode: +o jaypipes | 14:01 | |
| *** openstackgerrit has joined #openstack-qa | 14:09 | |
| *** gszasz has quit IRC | 14:10 | |
| mriedem | our categorization rates are pretty bad right now: http://status.openstack.org/elastic-recheck/data/uncategorized.html | 14:12 |
| mriedem | 67% | 14:12 |
| *** thomasbiege has joined #openstack-qa | 14:12 | |
| *** markmcclain has quit IRC | 14:13 | |
| *** mlavalle has joined #openstack-qa | 14:13 | |
| sdague | mriedem: yep, agreed | 14:18 |
| mriedem | writing a query for one now | 14:18 |
| mriedem | and looking at the proposed tempest fix | 14:18 |
| *** comstud is now known as bearhands | 14:18 | |
| *** rbak has joined #openstack-qa | 14:20 | |
| mriedem | sdague: https://review.openstack.org/#/c/91528/ | 14:20 |
| *** bill_az has joined #openstack-qa | 14:21 | |
| sdague | mriedem: sorting by dates is problematic regardless | 14:23 |
| sdague | given that these are all async calls | 14:23 |
| sdague | even the sleep would have a race | 14:23 |
| sdague | just a smaller one | 14:23 |
| sdague | my suggestion is we use a different, non date, param for sorting | 14:24 |
| sdague | which I put in the review | 14:24 |
| *** yfried has joined #openstack-qa | 14:24 | |
| mriedem | sounds good to me | 14:26 |
| mriedem | i'm working on fingerprinting it | 14:26 |
| mriedem | which you can only do on the failing test name | 14:26 |
| *** salv-orlando has joined #openstack-qa | 14:27 | |
| *** salv-orlando has quit IRC | 14:27 | |
| *** salv-orlando has joined #openstack-qa | 14:28 | |
| *** markmcclain has joined #openstack-qa | 14:28 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for Tempest volume test bug 1313970 https://review.openstack.org/91800 | 14:31 |
| mriedem | sdague: ^ | 14:31 |
| *** bill_az_ has joined #openstack-qa | 14:32 | |
| *** bill_az has quit IRC | 14:32 | |
| sdague | approved | 14:36 |
| *** krtaylor has joined #openstack-qa | 14:37 | |
| *** markmcclain has quit IRC | 14:38 | |
| *** dims is now known as swip | 14:39 | |
| *** jaypipes is now known as leakypipes | 14:41 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for Tempest volume test bug 1313970 https://review.openstack.org/91800 | 14:42 |
| *** markmcclain has joined #openstack-qa | 14:43 | |
| *** openstackrecheck has quit IRC | 14:52 | |
| *** openstackrecheck has joined #openstack-qa | 14:52 | |
| *** kgriffs|afk is now known as kgriffs | 14:58 | |
| *** nati_ueno has joined #openstack-qa | 15:02 | |
| *** che-arne has joined #openstack-qa | 15:03 | |
| dkranz | mtreinish: ping | 15:08 |
| dkranz | mtreinish: I am doing the little refactor of your verify script and saw that there is no unit test for the -u option. I noticed because it does not work. | 15:11 |
| dkranz | mtreinish: Because when you moved the script it was pushed down one level and the conf file location does not work. | 15:12 |
| dkranz | mtreinish: What was your intent? All the other scripts assume you are in the tempest dir by default. | 15:12 |
| *** thomasbiege has quit IRC | 15:17 | |
| *** marun has quit IRC | 15:30 | |
| *** marun has joined #openstack-qa | 15:31 | |
| *** nati_ueno has quit IRC | 15:35 | |
| dkranz | mtreinish: Never mind. I'll just fix the code to work from where it is now. | 15:35 |
| *** markmcclain has quit IRC | 15:36 | |
| *** atiwari has joined #openstack-qa | 15:36 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/88428 failed gate-horizon-selenium in the gate queue with an unrecognized error | 15:37 |
| *** sseverson has joined #openstack-qa | 15:40 | |
| *** kgriffs is now known as kgriffs|afk | 15:41 | |
| *** safchain has quit IRC | 15:49 | |
| *** yamahata has joined #openstack-qa | 15:51 | |
| *** yfried has quit IRC | 15:53 | |
| *** chandan_kumar has quit IRC | 15:55 | |
| *** kgriffs|afk is now known as kgriffs | 15:58 | |
| *** Aimon has quit IRC | 16:00 | |
| *** mkollaro has joined #openstack-qa | 16:02 | |
| *** catohornet has joined #openstack-qa | 16:09 | |
| *** mrodden has quit IRC | 16:13 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/83661 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 16:15 |
| *** asselin has joined #openstack-qa | 16:19 | |
| *** mlavalle has quit IRC | 16:20 | |
| *** hemna__ is now known as hemna | 16:21 | |
| *** mrodden has joined #openstack-qa | 16:25 | |
| *** jpich has quit IRC | 16:28 | |
| *** jogo is now known as flashgordon | 16:29 | |
| *** yamahata has quit IRC | 16:31 | |
| *** yamahata has joined #openstack-qa | 16:31 | |
| *** Obulpathi has quit IRC | 16:39 | |
| *** mlavalle has joined #openstack-qa | 16:54 | |
| *** giulivo has quit IRC | 16:55 | |
| *** giulivo has joined #openstack-qa | 16:58 | |
| *** Aimon has joined #openstack-qa | 17:00 | |
| openstackgerrit | Miguel Lavalle proposed a change to openstack/tempest: Neutron Allowed Address Pair API test https://review.openstack.org/66259 | 17:05 |
| *** harlowja_away is now known as harlowja | 17:06 | |
| openstackgerrit | A change was merged to openstack/tempest: Stress runner/driver prints stats after SIGINT https://review.openstack.org/91059 | 17:10 |
| *** sthaha has quit IRC | 17:10 | |
| *** dims has joined #openstack-qa | 17:13 | |
| *** dims has quit IRC | 17:13 | |
| *** mmaglana has quit IRC | 17:14 | |
| *** mmaglana has joined #openstack-qa | 17:14 | |
| *** swip has quit IRC | 17:15 | |
| *** dims has joined #openstack-qa | 17:16 | |
| *** markmcclain has joined #openstack-qa | 17:18 | |
| *** mmaglana has quit IRC | 17:18 | |
| *** dims has quit IRC | 17:20 | |
| *** dims has joined #openstack-qa | 17:20 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix stress runner not stopping on first fail https://review.openstack.org/91060 | 17:21 |
| *** rbak has quit IRC | 17:22 | |
| *** mlavalle has quit IRC | 17:26 | |
| *** KurtMartin has quit IRC | 17:28 | |
| *** jordan_ has quit IRC | 17:30 | |
| asselin | sdague, and others: can I get a review for https://review.openstack.org/#/c/90449/ ? | 17:32 |
| *** mmaglana has joined #openstack-qa | 17:32 | |
| asselin | psedlak, saw you did some changes to stress runnder/driver. Perhaps you'd like to review this as well? ^^^ | 17:33 |
| *** mkollaro has quit IRC | 17:35 | |
| *** dims has quit IRC | 17:36 | |
| *** dims has joined #openstack-qa | 17:36 | |
| *** morganfainberg_Z is now known as morganfainberg | 17:37 | |
| *** Obulpathi has joined #openstack-qa | 17:42 | |
| *** TravT has joined #openstack-qa | 17:48 | |
| mtreinish | dkranz: sorry I haven't really been online this week | 17:54 |
| mtreinish | the unit tests for -u were next on my list | 17:54 |
| mtreinish | I just assumed it worked the same after I moved it | 17:54 |
| dkranz | mtreinish: No problem. I fixed it. | 17:55 |
| mtreinish | if you've got a fix for it already that's great | 17:55 |
| mtreinish | ok awesome | 17:55 |
| dkranz | mtreinish: I just wanted to make sure you were ok with my approach. | 17:55 |
| dkranz | mtreinish: Which is to abstract the print/set-config parts into a function that you pass into a more generic discover_and_verify function | 17:56 |
| dkranz | mtreinish: It was also much easier to make printing messages and updating conf be one of the other and not both. | 17:56 |
| dkranz | mtreinish: one or the other | 17:56 |
| dkranz | mtreinish: I you are specifying -u I don't see why you need to print out what you did. You can diff the files after if you care. | 17:57 |
| dkranz | mtreinish: But this could be changed to with a bit more work. | 17:57 |
| *** kmartin has joined #openstack-qa | 17:58 | |
| mtreinish | well my view was it's a verification tool first and generating a new config was secondary functionality | 17:58 |
| mtreinish | there was some discussion about that in the spec review with oomichi | 17:58 |
| mtreinish | but, yeah I'm fine with refactoring it to make things a bit more generic | 17:59 |
| mtreinish | it'll probably help the tool in general | 17:59 |
| dkranz | mtreinish: Yeah, I am not sure we can have a tool that does everything. But if we make the discovery-and-return-detected-changes modular we can easily build other tools. | 18:02 |
| dkranz | mtreinish: I would actually be just as happy to remove the -u option from this tool. | 18:02 |
| dkranz | mtreinish: But I won't if you want to keep it. | 18:03 |
| dkranz | mtreinish: They are really different use cases IMO. | 18:03 |
| mtreinish | I think there is some value in leaving it there for now | 18:03 |
| openstackgerrit | Rob Pothier proposed a change to openstack/tempest: Add tempest test for heat resource AWS::EC2::EIP https://review.openstack.org/89564 | 18:03 |
| dkranz | mtreinish: ok, so be it. | 18:03 |
| mtreinish | maybe not long term if we have a better solution for config generation in the future | 18:03 |
| dkranz | mtreinish: fair | 18:03 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1315468 https://review.openstack.org/91842 | 18:07 |
| *** tosky has quit IRC | 18:10 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1315468 https://review.openstack.org/91842 | 18:11 |
| *** rbak has joined #openstack-qa | 18:16 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Generalize fingerprint 1315468 to cover all pypy jobs https://review.openstack.org/91845 | 18:16 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Fedora mysql log dir is /var/log/mariadb https://review.openstack.org/91318 | 18:20 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Generalize fingerprint 1315468 to cover all pypy jobs https://review.openstack.org/91845 | 18:21 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/devstack-gate: Set a default value for DEVSTACK_GATE_USE_APACHE https://review.openstack.org/91850 | 18:21 |
| *** openstackrecheck has quit IRC | 18:21 | |
| *** openstackrecheck has joined #openstack-qa | 18:22 | |
| openstackrecheck | openstack/horizon change: https://review.openstack.org/90115 failed gate-horizon-selenium in the gate queue with an unrecognized error | 18:27 |
| mtreinish | flashgordon, sdague: have you seen this before: http://logs.openstack.org/03/90603/2/gate/gate-grenade-dsvm-partial-ncpu/fa44f67/logs/old/screen-n-cpu.txt.gz?level=TRACE | 18:32 |
| mtreinish | did rabbit die or a service crash? | 18:32 |
| openstackgerrit | Edgar Magana proposed a change to openstack-dev/devstack: Add driver parameter for PLUMgrid configuration file https://review.openstack.org/91856 | 18:33 |
| flashgordon | mtreinish: nova network isn't running | 18:34 |
| flashgordon | not sure why | 18:34 |
| flashgordon | and I did see that before | 18:34 |
| flashgordon | but never got to the bottom of it | 18:34 |
| flashgordon | mtreinish: or rather there are no nova-network logs collected. and not sure why | 18:36 |
| mtreinish | flashgordon: well if it failed to start would devstack fail? | 18:37 |
| *** Obulpathi has quit IRC | 18:37 | |
| flashgordon | mriedem: not sure | 18:37 |
| sdague | if it was a persistent issue | 18:37 |
| *** Obulpathi has joined #openstack-qa | 18:38 | |
| mriedem | this isn't the mriedem you're looking for | 18:38 |
| sdague | honestly could it be a race between n-net and conductor? | 18:38 |
| mtreinish | mriedem: is it ever? | 18:38 |
| flashgordon | mriedem: woops | 18:39 |
| flashgordon | mtreinish: lol | 18:39 |
| sdague | we should capture those status files | 18:39 |
| flashgordon | sdague: so what confuses me is the lack of nova-network logs | 18:39 |
| mtreinish | mriedem: I still think for casual nick fridays which should just swap nicks :) | 18:40 |
| sdague | flashgordon: so that happens some times with screen as best as I can tell | 18:40 |
| sdague | we never got to the bottom of it | 18:40 |
| flashgordon | sdague: maybe stop using screen in gate? | 18:40 |
| sdague | we don't in main gate | 18:40 |
| sdague | we still use it in grenade | 18:41 |
| mriedem | mtreinish: i'm already confused | 18:41 |
| sdague | because of the stop calls | 18:41 |
| flashgordon | sdague: ahhh right guh | 18:41 |
| sdague | that we should be able to fix | 18:41 |
| sdague | but it hasn't acted up in a while | 18:41 |
| flashgordon | sdague: yeah, ++ to less screen in grenade | 18:41 |
| flashgordon | mtreinish mriedem: so I am looking at two different bugs, one from each of you | 18:42 |
| mtreinish | flashgordon: is there already a bug number to recheck against? or should I open one with a new e-r query? | 18:42 |
| * flashgordon lets sdague fix grenade and goes back to looking at mriedem's bug | 18:42 | |
| flashgordon | mtreinish: I didn't file a bug, so go for it | 18:42 |
| mtreinish | flashgordon: heh, yeah that's not going to be confusing at all... | 18:42 |
| sdague | flashgordon: ummmmm | 18:42 |
| mtreinish | flashgordon: +1 sdague will fix grenade | 18:43 |
| sdague | see, I'm within punching distance of mtreinish right now... | 18:43 |
| mriedem | mtreinish: this is the bug i reported yesterday for the grenade timeout issue https://bugs.launchpad.net/nova/+bug/1315201 | 18:44 |
| mriedem | not for the virtual interface issue | 18:44 |
| mtreinish | mriedem: hmm ok, well my fail matches that e-r query | 18:45 |
| mtreinish | I was going to push an e-r uery with those messaging timeout back traces | 18:46 |
| mriedem | mtreinish: i have one | 18:46 |
| mriedem | https://review.openstack.org/#/c/91707/ | 18:46 |
| mriedem | mtreinish: but it's uber generic, so was going to refactor to use find_caller in the timeouts | 18:46 |
| mriedem | so the timeout is scoped to the class.test that fails | 18:46 |
| flashgordon | so looking at mriedem issue, about timeout: http://logs.openstack.org/61/89961/2/check/check-tempest-dsvm-postgres-full/f74a94f/logs/dstat.txt.gz | 18:46 |
| flashgordon | we spike to a load of 15 | 18:47 |
| flashgordon | is that bad? | 18:47 |
| mtreinish | mriedem: no I was talking about adding a query for: http://logs.openstack.org/03/90603/2/gate/gate-grenade-dsvm-partial-ncpu/fa44f67/logs/old/screen-n-cpu.txt.gz?level=TRACE#_2014-05-01_13_20_21_418 | 18:47 |
| mtreinish | mriedem: although the fail will match 91707 | 18:48 |
| mtreinish | flashgordon: nah, we've had higher loads before I think :) | 18:48 |
| flashgordon | mtreinish: kk, I am just trying to figure what possible things could casue a VM to boot but nova not to know about it | 18:50 |
| mtreinish | flashgordon: I was just kidding that could be part of the problem | 18:51 |
| flashgordon | mtreinish: o_O | 18:51 |
| mtreinish | the high load was why we dropped the concurrency because it caused too many weird issues | 18:51 |
| flashgordon | mtreinish: right, but I don't remember what we considered 'high' | 18:52 |
| mtreinish | I think it was less than 15 | 18:52 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/91137 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 18:52 |
| * flashgordon noticed ceilometer is one of the most CPU intensive processes | 18:52 | |
| mtreinish | it's not like we've got 16 cpu guests in the gate | 18:52 |
| *** mriedem has quit IRC | 18:53 | |
| flashgordon | sdague: do you remember? ^ | 18:53 |
| *** mrodden is now known as tylerdurden | 18:53 | |
| sdague | load of 15 is not great | 18:54 |
| sdague | but we have 8 cpus | 18:54 |
| sdague | usually | 18:54 |
| mtreinish | I thought a bunch of the nodes were 4 cpu | 18:54 |
| sdague | most are 8 now | 18:54 |
| mtreinish | flashgordon: it was an 8 cpu guest | 18:56 |
| sdague | so 15 on an 8 isn't really a problem | 18:56 |
| sdague | it's only really a problem if we are running with no idle % for a long window of time | 18:57 |
| *** tylerdurden is now known as mrodden | 18:58 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: Add in a concurrency aware subunit filter https://review.openstack.org/76674 | 18:59 |
| *** mriedem has joined #openstack-qa | 19:12 | |
| mriedem | my system locked up for the last 20 minutes, what did i miss? ceilometer is killing the node? that's not news. | 19:12 |
| mtreinish | mriedem: are you still using windows? | 19:13 |
| mriedem | yeah... | 19:13 |
| mriedem | i think it's vmware player that kills me, syncing the disk | 19:13 |
| mriedem | anywho...don't ask questions.... | 19:14 |
| mriedem | mtreinish: sdague: i remember a thread about ceilometer in the gate and how they basically couldn't run tests | 19:14 |
| mriedem | did anything ever come out of that? something about the packages they needed to work weren't in the distro that was used? | 19:14 |
| mtreinish | mriedem: yeah I started that one, a cli test took >2min to run | 19:14 |
| mtreinish | because it was too slow | 19:14 |
| mtreinish | on the api side | 19:14 |
| sdague | ceilometer sql backend is 'sub optimal' to the point of not being useful | 19:15 |
| mriedem | we had a team hitting scaling issues internally and they were doing auditing with ceilometer, their controller was completely maxed out | 19:15 |
| mriedem | they had to kill that just to operate | 19:15 |
| sdague | what backend? | 19:15 |
| mriedem | nosql db2 | 19:15 |
| mriedem | with CADF turned on | 19:15 |
| mriedem | so between ceilometer and db2 and the various services on a single controller, qpid was timing out all over the place | 19:16 |
| sdague | so that's like all kinds of unknowns | 19:16 |
| mriedem | yeah | 19:16 |
| mriedem | granted it was a rather large control node i think, it couldn't keep up which didn't surprise me | 19:17 |
| mtreinish | mriedem: doing that just seems like a bad idea to me... | 19:17 |
| mriedem | well i gave them a list of things to try/disable, one of which was cadf with ceilometer, they did that along with adjusting some other kernel settings for performance and they started getting better | 19:17 |
| mriedem | but still telling them to maybe move qpid server to it's own node, and db2 | 19:18 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Kill spinner process when stack.sh exits https://review.openstack.org/90534 | 19:20 |
| openstackgerrit | A change was merged to openstack/tempest: Revert "add server personality files test" https://review.openstack.org/91770 | 19:20 |
| *** bill_az_ has quit IRC | 19:23 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Set default for DEVSTACK_GATE_USE_APACHE https://review.openstack.org/91646 | 19:23 |
| sdague | yeh, well it would be worth to at least understand what's consuming the resources | 19:24 |
| sdague | even the relatively dumb dstat output would help | 19:24 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: Add in a concurrency aware subunit filter https://review.openstack.org/76674 | 19:36 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/83565 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1254890 | 19:44 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: Add in a concurrency aware subunit filter https://review.openstack.org/76674 | 19:53 |
| sdague | lifeless: you haven't wandered on yet today have you? | 19:58 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1315095 https://review.openstack.org/91886 | 20:10 |
| openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Fix incorrect comparisions https://review.openstack.org/91887 | 20:10 |
| *** rbak has quit IRC | 20:13 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/91670 failed because of: gate-nova-python27: https://bugs.launchpad.net/bugs/1291620 | 20:15 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1315095 https://review.openstack.org/91886 | 20:25 |
| *** dims has quit IRC | 20:27 | |
| *** harlowja is now known as harlowja_away | 20:33 | |
| *** harlowja_away is now known as harlowja | 20:37 | |
| *** marun has quit IRC | 20:45 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Refactor _find_caller into a public test finder utility https://review.openstack.org/91896 | 20:47 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Use find_test_caller to put test name in timeout exception details https://review.openstack.org/91897 | 20:47 |
| mriedem | sdague: flashgordon: ^ | 20:47 |
| mriedem | what we talked about earlier | 20:47 |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Fix a few bugs when running with -u https://review.openstack.org/91899 | 20:49 |
| dkranz | mtreinish: So I pushed some bug fixes before the refactor. | 20:50 |
| openstackgerrit | Sergey Lukjanov proposed a change to openstack-dev/devstack: Run all-in-one instead of sahara-api https://review.openstack.org/91900 | 20:54 |
| *** Obulpathi has quit IRC | 21:00 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Fix incorrect comparisions https://review.openstack.org/91887 | 21:04 |
| *** weshay has quit IRC | 21:19 | |
| *** weshay has joined #openstack-qa | 21:20 | |
| *** Sukhdev has joined #openstack-qa | 21:24 | |
| *** weshay has quit IRC | 21:24 | |
| *** dims has joined #openstack-qa | 21:24 | |
| *** mmaglana has quit IRC | 21:27 | |
| *** weshay has joined #openstack-qa | 21:27 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Insert required iptables rules for Ironic https://review.openstack.org/87377 | 21:28 |
| *** annegentle has quit IRC | 21:29 | |
| *** Aimon has quit IRC | 21:29 | |
| mriedem | sdague: unit tests in tempest makes you so happy | 21:33 |
| sdague | yes, well especially with code I wrote that I was too lazy to do tests for | 21:34 |
| mriedem | \o/ | 21:34 |
| lifeless | sdague: hi, just passing through | 21:43 |
| sdague | lifeless: ok, quick passing question | 21:43 |
| sdague | how do I pull 'reason' attachment out of the subunit stream for skips | 21:44 |
| sdague | finally respinning - https://review.openstack.org/#/c/76674/ | 21:44 |
| sdague | but the skip output changes was a side effect I had not anticipated | 21:44 |
| *** weshay has quit IRC | 21:45 | |
| *** mriedem has quit IRC | 21:48 | |
| mtreinish | mrodden: does #noqa (or whatever it is) not work on custom hacking checks? | 21:48 |
| mtreinish | mrodden: sorry meant mriedem who I missed by 30 sec... | 21:49 |
| openstackgerrit | A change was merged to openstack/tempest: Add unit tests for verifying extensions list https://review.openstack.org/90603 | 21:49 |
| sdague | mtreinish: it has to be coded for every check | 21:49 |
| sdague | it's not a general thing | 21:49 |
| sdague | though jogo finally agreed to change that | 21:50 |
| mtreinish | sdague: I know, but for https://review.openstack.org/#/c/91896/1 we could unit test the setUpclass case despite T105 with noqa | 21:50 |
| mtreinish | that's a perfect use case for it | 21:50 |
| *** dims has quit IRC | 21:51 | |
| lifeless | sdague: put the answer in the review | 21:56 |
| -openstackstatus- NOTICE: Zuul is being restarted with some dependency upgrades and configuration changes; ETA 2215 | 21:59 | |
| *** marun has joined #openstack-qa | 22:01 | |
| sdague | lifeless: thanks | 22:09 |
| mtreinish | flashgordon, sdague: is there a trick to getting noqa to work on a project specific check. I tried adding the noqa param to the check method and also decorating it with @core.flake8ext. | 22:18 |
| sdague | it's super manual | 22:19 |
| flashgordon | mtreinish: there is a trick actaully | 22:19 |
| sdague | honestly, I haven't looked in detail at that code path | 22:19 |
| mtreinish | sdague: so regex check for noqa and pass if it's found | 22:19 |
| flashgordon | mtreinish: can you share your patch | 22:19 |
| mtreinish | flashgordon: it's already in tree http://git.openstack.org/cgit/openstack/tempest/tree/tempest/hacking/checks.py#n58 | 22:20 |
| flashgordon | mtreinish: I0ddefdb84854ae61d2f15f20d6e8019f8c34f30a | 22:20 |
| flashgordon | ahh so physical line checks don | 22:20 |
| flashgordon | don't support the noqa arg | 22:20 |
| mtreinish | flashgordon: ahh ok that explains it | 22:21 |
| flashgordon | mtreinish: try making it a logical line | 22:21 |
| mtreinish | because I was looking at that commit to figure it out | 22:21 |
| flashgordon | add then it should work | 22:21 |
| *** Aimon has joined #openstack-qa | 22:21 | |
| flashgordon | mtreinish: no_setupclass_for_unit_tests should be logical line anyway IMHO. you may need ot update the regex a bit | 22:22 |
| *** mmaglana has joined #openstack-qa | 22:24 | |
| *** ianw has quit IRC | 22:26 | |
| *** ianw has joined #openstack-qa | 22:27 | |
| mtreinish | flashgordon: no go: http://paste.openstack.org/show/78540/ | 22:27 |
| *** kgriffs is now known as kgriffs|afk | 22:28 | |
| flashgordon | mtreinish: ohh right, that is for hacking 0.9 (pep8 1.5x | 22:36 |
| flashgordon | sorry | 22:36 |
| mtreinish | flashgordon: ok np I just manually check for no qa in the line then :) | 22:36 |
| flashgordon | for phiysical lines, you can do 'pep8.noqa(physical_line) | 22:37 |
| flashgordon | import pep8 ... | 22:37 |
| mtreinish | flashgordon: ok, I'll give that a try | 22:37 |
| flashgordon | mtreinish: sorry about the wildgoose chase earlier | 22:40 |
| *** leakypipes has quit IRC | 22:49 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Refactor _find_caller into a public test finder utility https://review.openstack.org/91896 | 22:53 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add noqa support to no_setupclass_for_unit_tests https://review.openstack.org/91920 | 22:53 |
| mtreinish | flashgordon: ^^^ | 22:55 |
| flashgordon | mtreinish: test coverage? | 22:58 |
| flashgordon | does tempest have local hacking check test coverage already? | 22:59 |
| mtreinish | testing on the hacking rules? We don't have that now I can add it I guess | 23:00 |
| mtreinish | does nova do that? | 23:00 |
| flashgordon | mtreinish: yup | 23:01 |
| *** HenryG has quit IRC | 23:01 | |
| flashgordon | mtreinish: nova/tests/test_hacking.py | 23:02 |
| mtreinish | flashgordon: well unit tests are a relatively new thing for tempest | 23:03 |
| mtreinish | I'll add tests and repush | 23:03 |
| *** hemna is now known as hemnafk | 23:06 | |
| flashgordon | mtreinish: oh right almost forgot haha | 23:08 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/83565 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1314313, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254890 | 23:09 |
| *** mmaglana_ has joined #openstack-qa | 23:10 | |
| *** mmaglana has quit IRC | 23:10 | |
| *** mmaglana_ has quit IRC | 23:11 | |
| *** rupsky has joined #openstack-qa | 23:12 | |
| *** rupsky has left #openstack-qa | 23:12 | |
| *** HenryG has joined #openstack-qa | 23:14 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add noqa support to no_setupclass_for_unit_tests https://review.openstack.org/91920 | 23:14 |
| mtreinish | flashgordon: ^^^ unit tests and all | 23:15 |
| mtreinish | now time to add them for the other rules | 23:15 |
| flashgordon | mtreinish: I take it you ran those tests | 23:16 |
| flashgordon | the assertFalse and assertIsNone just seem odd together, but make sense | 23:17 |
| mtreinish | flashgordon: yeah the nova tests used both, but I wanted to specifically check that it was a blank return call for noqa | 23:17 |
| mtreinish | I guess I could leave a comment on that | 23:18 |
| flashgordon | if you want, just wanted to double check | 23:19 |
| mtreinish | no it's ok, if I get a -1 I guess I'll respin it | 23:19 |
| *** mrodden has quit IRC | 23:22 | |
| *** Sukhdev has quit IRC | 23:24 | |
| -openstackstatus- NOTICE: paste.openstack.org is going down for a short database upgrade | 23:27 | |
| *** dims has joined #openstack-qa | 23:32 | |
| *** dims has quit IRC | 23:44 | |
| *** flashgordon is now known as jogo | 23:49 | |
| *** dims has joined #openstack-qa | 23:55 | |
| *** dims has joined #openstack-qa | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!