| *** rbak has joined #openstack-qa | 00:03 | |
| *** nicholasgoracke has joined #openstack-qa | 00:06 | |
| *** rbrndt has quit IRC | 00:07 | |
| *** agireud has quit IRC | 00:07 | |
| *** weshay_xchat has quit IRC | 00:08 | |
| devananda | ianw: the bug is that, no matter what cleaning steps are defined, none of them will get run by any driver | 00:08 |
|---|---|---|
| devananda | ianw: there is only one defined upstream, and it is skipped in most drivers right now, because it takes too long | 00:08 |
| devananda | ianw: in fixing the bug, we uncovered a test that actually would run that step, so we need to skip it there too | 00:09 |
| devananda | ianw: we will _also_ need to do something else so that we can ensure cleaning is tested. skipping the only step means we aren't testing it right now | 00:09 |
| devananda | ianw: also, thanks for the review | 00:09 |
| *** nicholasgoracke has quit IRC | 00:10 | |
| ianw | devananda: right, pretty much what i thought. is it the lack of nested-kvm that makes it so slow? or just that it's doing military grade shreds that's slow at the best of times? | 00:10 |
| devananda | ianw: both :) | 00:11 |
| *** zhangjn has joined #openstack-qa | 00:13 | |
| *** agireud has joined #openstack-qa | 00:13 | |
| *** sileht has quit IRC | 00:15 | |
| *** woodster_ has quit IRC | 00:16 | |
| *** agireud has quit IRC | 00:18 | |
| gmann | mtreinish: done, just 1 fix in hacking.rst | 00:18 |
| *** jasonsb has joined #openstack-qa | 00:18 | |
| *** akshai has quit IRC | 00:19 | |
| mtreinish | gmann: thanks, I'll respin right now | 00:20 |
| openstackgerrit | Matthew Treinish proposed openstack/tempest: Use the idempotent id and tooling from tempest-lib https://review.openstack.org/253272 | 00:21 |
| *** agireud has joined #openstack-qa | 00:21 | |
| mtreinish | gmann: ^^^ | 00:21 |
| *** sileht has joined #openstack-qa | 00:27 | |
| *** Swami has quit IRC | 00:27 | |
| *** zhangjn has quit IRC | 00:28 | |
| *** devananda has quit IRC | 00:28 | |
| *** amotoki has joined #openstack-qa | 00:28 | |
| openstackgerrit | Ian Wienand proposed openstack-dev/devstack: Only use lsb_release for distro tags https://review.openstack.org/117104 | 00:29 |
| *** yamamoto has quit IRC | 00:36 | |
| *** agireud has quit IRC | 00:42 | |
| gmann | mtreinish: regarding https://review.openstack.org/#/c/247278/ | 00:42 |
| gmann | mtreinish: will not project-config will take stable branch devstack for stable branch jobs on master and on devstack stable branch do not set microversion on tempest | 00:44 |
| gmann | mtreinish: or missing something ? | 00:44 |
| *** amotoki has quit IRC | 00:44 | |
| mtreinish | gmann: that works until the mitaka release | 00:44 |
| mtreinish | gmann: and we want to enable the microversion tests on stable too | 00:44 |
| gmann | mtreinish: but we want to enable on stable branch from mitaka onward | 00:45 |
| gmann | mtreinish: i mean for kilo, liberty we do not want to run microversion tests | 00:46 |
| gmann | mtreinish: and from mitaka we will run all applicable microversions tests, even that is stable branch in N | 00:48 |
| mtreinish | gmann: why not? tempest is branchless so we should start running it on liberty and kilo | 00:48 |
| mtreinish | gmann: nova 2.2 was available in liberty right, why wouldn't you test it there too? | 00:49 |
| gmann | mtreinish: actually on kilo it would not work as devstack makes tests to run on /v2.0 (not on /v2.1 which is added in liberty) | 00:50 |
| mtreinish | ok, then we can ignore kilo, because that's just annoying | 00:50 |
| gmann | mtreinish: yea for liberty it will work but as discussed earlier sdague mentioned we can skip running on stable branches | 00:50 |
| gmann | mtreinish: ok, for liberty we can run | 00:51 |
| mtreinish | gmann: ok, but even if we ignore liberty you're only making life diffult in a few months | 00:51 |
| mtreinish | s/diffult/difficult | 00:51 |
| gmann | mtreinish: you mean when mitaka becomes stable? | 00:52 |
| mtreinish | because when we release mitaka and start working on N things won't behave as expected | 00:52 |
| mtreinish | yes | 00:52 |
| gmann | mtreinish: it should work right? for mitaka stable branch tests will allow to run all applicable versions tests | 00:52 |
| mtreinish | gmann: it works until you add a test for microversion 2.30 which is added in N and doesn't exist in mitaka | 00:53 |
| mtreinish | then mitaka tries to run the test and the microversion doesnt exist | 00:53 |
| mtreinish | which is why you should encode the default max microversion in devstack, because only devstack knows which version of the code is deployed (and therefore which microversions are available) | 00:54 |
| *** devananda has joined #openstack-qa | 00:54 | |
| openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Make date window for job pages longer for periodic jobs https://review.openstack.org/253214 | 00:54 |
| gmann | mtreinish: ah yea, we need to cap version on mitaka on release like you mentioned as for extensions list | 00:54 |
| mtreinish | yep | 00:54 |
| gmann | mtreinish: actually making default on devstack would not work for v2 jobs (i replied on patch) | 00:55 |
| mtreinish | gmann: right, which is why it's a default that you can override | 00:55 |
| mtreinish | you update the job definitions for the v2 jobs to not set a microversion (or set 2.0) | 00:55 |
| mtreinish | you went about things backwards | 00:55 |
| gmann | mtreinish: i see | 00:56 |
| * gmann thinking | 00:56 | |
| mtreinish | or you add logic to devstack when it does a 2.0 only deploy to not set the microversion options in tempest, because devstack presumably knows about that too | 00:57 |
| gmann | mtreinish: yea that is nice, so we can avoid any project-config changes right | 00:58 |
| gmann | mtreinish: but we need to update devstack each time on release right | 00:58 |
| mtreinish | gmann: you would have too anyway | 00:58 |
| *** hichihara has joined #openstack-qa | 00:59 | |
| mtreinish | it's the same thing with the extension lists today | 00:59 |
| * mtreinish checks if he remembered to do that on liberty | 00:59 | |
| gmann | mtreinish: yea, and how we coordinate that on release, its something we need to mention somewhere or on failure we update ? | 01:00 |
| mtreinish | gmann: I'd put it on: https://wiki.openstack.org/wiki/QA/releases#Things_to_remember_when_creating_a_stable_branch | 01:00 |
| mtreinish | gmann: although the failure case is a good fallback | 01:00 |
| mtreinish | it'd be better to be proactive about it, but when a failure is hit it'll remind us quickly :) | 01:01 |
| *** zhangjn has joined #openstack-qa | 01:01 | |
| gmann | mtreinish: yea, thats nice. | 01:01 |
| gmann | mtreinish: i will update patches and wiki too. with that we can have 2.2 mivrocersion tests running and check its all fine on all jobs | 01:02 |
| gmann | mtreinish: thanks :) | 01:02 |
| mtreinish | gmann: sure, np | 01:02 |
| *** eil397 has quit IRC | 01:02 | |
| * mtreinish is excited to have microversion support finally :) | 01:02 | |
| gmann | me too :) | 01:03 |
| *** agireud has joined #openstack-qa | 01:12 | |
| *** agireud has quit IRC | 01:14 | |
| *** eil397 has joined #openstack-qa | 01:14 | |
| *** pvaneck has quit IRC | 01:17 | |
| *** yamamoto has joined #openstack-qa | 01:21 | |
| openstackgerrit | John Warren proposed openstack/tempest: Split out Neutron metering label rules client https://review.openstack.org/253133 | 01:23 |
| gmann | mtreinish: sdague another idea to avoid devstack change on each release is to discover max supported version from Nova and set on tempest ? | 01:24 |
| *** jecarey has joined #openstack-qa | 01:25 | |
| *** mriedem has joined #openstack-qa | 01:29 | |
| *** jecarey has quit IRC | 01:29 | |
| *** agireud has joined #openstack-qa | 01:30 | |
| *** mtanino has quit IRC | 01:32 | |
| *** agireud has quit IRC | 01:34 | |
| openstackgerrit | Ramy Asselin proposed openstack-infra/elastic-recheck: Make Elastic Recheck Watch more reusable https://review.openstack.org/251515 | 01:42 |
| openstackgerrit | Masayuki Igawa proposed openstack-infra/subunit2sql: Add docstrings of test_ids https://review.openstack.org/252223 | 01:42 |
| mtreinish | gmann: discovery potentially masks errors, so we shouldn't do it automatically | 01:45 |
| mtreinish | that's the whole reason we added config options for this to tempest | 01:46 |
| gmann | mtreinish: yea on tempest side i agree but on devstack we can do? :) | 01:48 |
| *** agireud has joined #openstack-qa | 01:48 | |
| mtreinish | it defeats the purpose to do it in devstack... | 01:48 |
| mtreinish | gmann: for microversions it's very easy to push a patch there is only one max version available for a single service | 01:48 |
| mtreinish | extensions list is much harder | 01:48 |
| gmann | mtreinish: yea, only thinking about sdague concern not to make devstack keep changes but yea discovery might be end up with wrong configuration etc | 01:51 |
| *** TC01 has joined #openstack-qa | 01:51 | |
| gmann | mtreinish: lets do without discovery and if we feel it is overhead of changing on each release then we can think those later | 01:52 |
| gmann | thanks | 01:52 |
| *** agireud has quit IRC | 01:53 | |
| openstackgerrit | Ramy Asselin proposed openstack-infra/elastic-recheck: Make Elastic Recheck Watch more reusable https://review.openstack.org/251515 | 01:57 |
| *** eil397 has quit IRC | 01:58 | |
| *** salv-orlando has joined #openstack-qa | 02:02 | |
| *** agireud has joined #openstack-qa | 02:03 | |
| *** salv-orlando has quit IRC | 02:07 | |
| *** agireud has quit IRC | 02:07 | |
| *** amotoki has joined #openstack-qa | 02:13 | |
| *** sabeen has joined #openstack-qa | 02:16 | |
| *** markvoelker has quit IRC | 02:40 | |
| *** mateusz_ has joined #openstack-qa | 02:43 | |
| *** mateusz_ has quit IRC | 02:44 | |
| *** mateusz_ has joined #openstack-qa | 02:44 | |
| *** mateusz_ has quit IRC | 02:45 | |
| *** mateusz_ has joined #openstack-qa | 02:45 | |
| *** mateusz_ has quit IRC | 02:46 | |
| *** mateusz_ has joined #openstack-qa | 02:46 | |
| *** mateusz_ has quit IRC | 02:47 | |
| *** mateusz_ has joined #openstack-qa | 02:47 | |
| *** mateusz__ has joined #openstack-qa | 02:47 | |
| *** mateusz_ has quit IRC | 02:48 | |
| *** Poornima has joined #openstack-qa | 02:48 | |
| *** mateusz__ has quit IRC | 02:48 | |
| openstackgerrit | Ghanshyam Mann proposed openstack-dev/devstack: Add compute microversions configuration on tempest https://review.openstack.org/247246 | 02:53 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add tests for Nova microversion v2.2 https://review.openstack.org/244996 | 02:54 |
| *** akshai has joined #openstack-qa | 02:59 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add tests to check migrations and models match https://review.openstack.org/253301 | 02:59 |
| openstackgerrit | Masayuki Igawa proposed openstack-infra/subunit2sql: Make graph.py just a module https://review.openstack.org/253302 | 03:00 |
| *** akshai has quit IRC | 03:03 | |
| *** akshai has joined #openstack-qa | 03:04 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Enable db api tests on all backends https://review.openstack.org/253304 | 03:13 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Remove group regex from testr.conf https://review.openstack.org/253305 | 03:13 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add tests to check migrations and models match https://review.openstack.org/253301 | 03:20 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Remove group regex from testr.conf https://review.openstack.org/253305 | 03:20 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Enable db api tests on all backends https://review.openstack.org/253304 | 03:20 |
| *** hitalia has quit IRC | 03:23 | |
| *** yamahata has quit IRC | 03:23 | |
| openstackgerrit | xing-yang proposed openstack/tempest: Add tempest tests for CG https://review.openstack.org/252213 | 03:25 |
| *** tochi has quit IRC | 03:28 | |
| *** hichihara has quit IRC | 03:36 | |
| *** moravec1 has joined #openstack-qa | 03:39 | |
| *** flwang1 has quit IRC | 03:46 | |
| *** jecarey has joined #openstack-qa | 03:50 | |
| *** e0ne_ has joined #openstack-qa | 03:52 | |
| *** zhangjn has quit IRC | 03:56 | |
| *** e0ne_ has quit IRC | 04:00 | |
| *** zhangjn has joined #openstack-qa | 04:05 | |
| *** tochi has joined #openstack-qa | 04:06 | |
| *** stevemar has quit IRC | 04:09 | |
| *** e0ne has joined #openstack-qa | 04:12 | |
| *** stevemar_znc has joined #openstack-qa | 04:12 | |
| *** akshai has quit IRC | 04:16 | |
| *** akshai has joined #openstack-qa | 04:16 | |
| *** e0ne has quit IRC | 04:16 | |
| *** zhangjn has quit IRC | 04:17 | |
| oomichi | mtreinish: ok, +A | 04:17 |
| *** zhangjn has joined #openstack-qa | 04:17 | |
| openstackgerrit | Ghanshyam Mann proposed openstack-dev/devstack: Add compute microversions configuration on tempest https://review.openstack.org/247246 | 04:17 |
| *** zhangjn has quit IRC | 04:18 | |
| oomichi | mtreinish: can you take a look at https://review.openstack.org/#/c/252739/ instead ? | 04:19 |
| openstackgerrit | Merged openstack-infra/subunit2sql: Add docstrings of test_ids https://review.openstack.org/252223 | 04:19 |
| oomichi | mtreinish: necessary for bumping tempest-lib version on Tempest | 04:19 |
| *** jecarey has quit IRC | 04:22 | |
| openstackgerrit | Merged openstack-infra/subunit2sql: Make graph.py just a module https://review.openstack.org/253302 | 04:24 |
| *** dpaterson has quit IRC | 04:26 | |
| *** akshai has quit IRC | 04:26 | |
| *** mriedem has quit IRC | 04:27 | |
| *** agireud has joined #openstack-qa | 04:51 | |
| *** salv-orlando has joined #openstack-qa | 04:51 | |
| *** salv-orlando has quit IRC | 04:56 | |
| openstackgerrit | Jacky_lei_zhang proposed openstack/tempest: Make volume_types_client use **kwargs https://review.openstack.org/250228 | 05:00 |
| *** rfolco has quit IRC | 05:02 | |
| *** rbak has quit IRC | 05:04 | |
| *** mwagner_lap has joined #openstack-qa | 05:05 | |
| *** markvoelker has joined #openstack-qa | 05:05 | |
| *** amotoki_ has joined #openstack-qa | 05:09 | |
| *** markvoelker has quit IRC | 05:09 | |
| *** markvoelker has joined #openstack-qa | 05:10 | |
| *** amotoki_ has quit IRC | 05:12 | |
| *** amotoki has quit IRC | 05:13 | |
| oomichi | mtreinish: thanks, I was surprized your review on your timezone | 05:15 |
| oomichi | mtreinish: good night :) | 05:15 |
| *** amotoki has joined #openstack-qa | 05:17 | |
| *** amotoki has quit IRC | 05:17 | |
| *** bkopilov has quit IRC | 05:17 | |
| openstackgerrit | Merged openstack-dev/devstack: Update lib/ironic to always skip cleaning https://review.openstack.org/253089 | 05:22 |
| *** agireud has quit IRC | 05:23 | |
| *** sabeen3 has joined #openstack-qa | 05:25 | |
| *** agireud has joined #openstack-qa | 05:26 | |
| *** sabeen has quit IRC | 05:28 | |
| *** bkopilov has joined #openstack-qa | 05:31 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Make create_image use **kwargs https://review.openstack.org/251310 | 05:35 |
| *** agireud has quit IRC | 05:35 | |
| *** ekhugen has quit IRC | 05:42 | |
| *** ekhugen has joined #openstack-qa | 05:42 | |
| *** timello_ has quit IRC | 05:43 | |
| *** timello has joined #openstack-qa | 05:45 | |
| *** markvoelker_ has joined #openstack-qa | 05:47 | |
| *** armax has quit IRC | 05:48 | |
| *** markvoelker has quit IRC | 05:50 | |
| *** sabeen3 has quit IRC | 05:51 | |
| *** markvoelker_ has quit IRC | 05:51 | |
| *** salv-orlando has joined #openstack-qa | 05:52 | |
| *** sabeen1 has joined #openstack-qa | 05:52 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Make create_floating_ip use **kwargs https://review.openstack.org/252778 | 05:53 |
| *** hoangcx has joined #openstack-qa | 05:53 | |
| *** hoangcx has left #openstack-qa | 05:53 | |
| *** salv-orl_ has joined #openstack-qa | 05:56 | |
| *** salv-orlando has quit IRC | 05:56 | |
| *** salv-orl_ has quit IRC | 05:56 | |
| *** salv-orlando has joined #openstack-qa | 05:57 | |
| openstackgerrit | Takashi NATSUME proposed openstack/tempest: Use container/disk format settings in glance v1 test https://review.openstack.org/218775 | 06:01 |
| *** salv-orlando has quit IRC | 06:10 | |
| *** pc_m has quit IRC | 06:10 | |
| *** pc_m has joined #openstack-qa | 06:11 | |
| *** rcernin has joined #openstack-qa | 06:12 | |
| *** sabeen1 has quit IRC | 06:18 | |
| *** jdandrea has quit IRC | 06:19 | |
| *** jdandrea has joined #openstack-qa | 06:22 | |
| *** amotoki has joined #openstack-qa | 06:30 | |
| *** jlk has quit IRC | 06:41 | |
| *** jlk has joined #openstack-qa | 06:41 | |
| *** oomichi has quit IRC | 06:46 | |
| *** agireud has joined #openstack-qa | 06:50 | |
| *** agireud has quit IRC | 06:54 | |
| *** rcernin has quit IRC | 06:59 | |
| *** agireud has joined #openstack-qa | 07:04 | |
| *** rcernin has joined #openstack-qa | 07:07 | |
| *** tochi has quit IRC | 07:08 | |
| *** tochi has joined #openstack-qa | 07:09 | |
| *** agireud has quit IRC | 07:09 | |
| *** neeti has joined #openstack-qa | 07:12 | |
| *** tochi has quit IRC | 07:31 | |
| *** safchain has joined #openstack-qa | 07:41 | |
| *** salv-orlando has joined #openstack-qa | 07:44 | |
| *** grafuls has joined #openstack-qa | 07:45 | |
| *** yamahata has joined #openstack-qa | 07:54 | |
| *** rossella_s has joined #openstack-qa | 08:04 | |
| *** yamahata has quit IRC | 08:13 | |
| *** Poornima has quit IRC | 08:16 | |
| *** maht has quit IRC | 08:22 | |
| *** Poornima has joined #openstack-qa | 08:26 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Handle StrOpt config option of microversion https://review.openstack.org/253380 | 08:30 |
| *** Poornima has left #openstack-qa | 08:30 | |
| *** maht has joined #openstack-qa | 08:31 | |
| openstackgerrit | Piyush Pathak proposed openstack/tempest: Make add_router_to_l3_agent use **kwargs https://review.openstack.org/252839 | 08:31 |
| *** ifarkas has joined #openstack-qa | 08:32 | |
| *** neeti has quit IRC | 08:34 | |
| *** neeti has joined #openstack-qa | 08:34 | |
| *** dmellado has joined #openstack-qa | 08:35 | |
| *** Poornima has joined #openstack-qa | 08:35 | |
| *** _nadya_ has joined #openstack-qa | 08:37 | |
| *** jlanoux has joined #openstack-qa | 08:40 | |
| *** vgridnev has quit IRC | 08:40 | |
| *** salv-orlando has quit IRC | 08:42 | |
| *** tochi has joined #openstack-qa | 08:44 | |
| openstackgerrit | Merged openstack/tempest: Rename private methods of glance_http https://review.openstack.org/251330 | 08:44 |
| openstackgerrit | Piyush Pathak proposed openstack/tempest: Make update_extra_routes use **kwargs & doc string https://review.openstack.org/252862 | 08:44 |
| openstackgerrit | Maho Koshiya proposed openstack/tempest: Add OS-INHERIT of keystone v3 in api tempest https://review.openstack.org/250795 | 08:47 |
| *** salv-orlando has joined #openstack-qa | 08:48 | |
| openstackgerrit | Merged openstack/tempest: Remove unused json_request() https://review.openstack.org/251332 | 08:53 |
| *** oomichi has joined #openstack-qa | 08:55 | |
| openstackgerrit | Ghanshyam Mann proposed openstack-dev/devstack: Add compute microversions configuration on tempest https://review.openstack.org/247246 | 08:55 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add compute client base class for microversion support https://review.openstack.org/242346 | 09:00 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add logic to select the request microversion https://review.openstack.org/244439 | 09:00 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add support of schema versioning for microversion https://review.openstack.org/244974 | 09:00 |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add tests for Nova microversion v2.2 https://review.openstack.org/244996 | 09:01 |
| *** tochi has quit IRC | 09:03 | |
| *** dims_ has quit IRC | 09:05 | |
| *** shardy has joined #openstack-qa | 09:05 | |
| *** gfidente has joined #openstack-qa | 09:08 | |
| *** gfidente has quit IRC | 09:08 | |
| *** gfidente has joined #openstack-qa | 09:08 | |
| *** dims has joined #openstack-qa | 09:11 | |
| *** shardy has quit IRC | 09:17 | |
| *** dims has quit IRC | 09:23 | |
| *** jordanP has joined #openstack-qa | 09:24 | |
| *** shardy has joined #openstack-qa | 09:25 | |
| *** hop|2 has joined #openstack-qa | 09:27 | |
| -openstackstatus- NOTICE: Tox tests are broken at the moment. From openstack-infra we are working to fix them. Please don't approve changes until we notify that tox tests work again. | 09:31 | |
| openstackgerrit | Masayuki Igawa proposed openstack/os-testr: Fix coverage section in tox.ini https://review.openstack.org/253407 | 09:32 |
| *** dims has joined #openstack-qa | 09:34 | |
| *** vgridnev has joined #openstack-qa | 09:34 | |
| *** shardy has quit IRC | 09:34 | |
| *** dtantsur|afk is now known as dtantsur | 09:35 | |
| openstackgerrit | Piyush Pathak proposed openstack/tempest: Make update_extra_routes use **kwargs & doc string https://review.openstack.org/252862 | 09:35 |
| openstackgerrit | Masayuki Igawa proposed openstack/os-testr: [WIP] Add unit test for colorizer https://review.openstack.org/253409 | 09:36 |
| *** shardy has joined #openstack-qa | 09:37 | |
| *** shardy has quit IRC | 09:43 | |
| *** shardy has joined #openstack-qa | 09:46 | |
| openstackgerrit | Daniel Mellado proposed openstack/tempest: Split out keystone tenant client https://review.openstack.org/248170 | 09:49 |
| *** ansiwen has quit IRC | 09:52 | |
| *** salv-orlando has quit IRC | 09:52 | |
| *** slunkad has joined #openstack-qa | 09:53 | |
| openstackgerrit | Masayuki Igawa proposed openstack/os-testr: [WIP] Add unit test for colorizer https://review.openstack.org/253409 | 09:55 |
| *** kevinbenton has quit IRC | 10:01 | |
| *** yfried has quit IRC | 10:03 | |
| *** kevinbenton has joined #openstack-qa | 10:04 | |
| *** apevec has joined #openstack-qa | 10:06 | |
| *** apevec has quit IRC | 10:06 | |
| *** apevec has joined #openstack-qa | 10:06 | |
| *** salv-orlando has joined #openstack-qa | 10:10 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make v1 list_images use **kwargs & doc string update https://review.openstack.org/252291 | 10:15 |
| openstackgerrit | Piyush Pathak proposed openstack/tempest: Make v3 update_user_password use **kwargs https://review.openstack.org/253419 | 10:15 |
| *** slunkad has quit IRC | 10:24 | |
| *** slunkad has joined #openstack-qa | 10:25 | |
| *** ansiwen has joined #openstack-qa | 10:27 | |
| *** neeti has quit IRC | 10:29 | |
| *** agireud has joined #openstack-qa | 10:29 | |
| *** kbaikov has quit IRC | 10:30 | |
| *** openstackgerrit has quit IRC | 10:32 | |
| *** neeti has joined #openstack-qa | 10:32 | |
| *** openstackgerrit has joined #openstack-qa | 10:32 | |
| *** agireud has quit IRC | 10:34 | |
| *** kbaikov has joined #openstack-qa | 10:37 | |
| *** Poornima has quit IRC | 10:38 | |
| *** dzamboni has joined #openstack-qa | 10:39 | |
| *** gszasz has joined #openstack-qa | 10:42 | |
| openstackgerrit | Merged openstack/tempest: Remove unused agents's schema https://review.openstack.org/252799 | 10:51 |
| *** neeti has quit IRC | 10:51 | |
| *** tosky has joined #openstack-qa | 10:52 | |
| *** vgridnev has quit IRC | 10:52 | |
| *** vgridnev has joined #openstack-qa | 10:54 | |
| openstackgerrit | Merged openstack/tempest: Use the idempotent id and tooling from tempest-lib https://review.openstack.org/253272 | 10:54 |
| lewo | oomichi, jordanP [232432] Thanks for your reviews. I've pushsed a new patch set according to your remarks. If you could have a look when you have time...;) | 11:02 |
| *** salv-orlando has quit IRC | 11:07 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 11:12 |
| *** yamamoto has quit IRC | 11:14 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 11:14 |
| *** aysyd has joined #openstack-qa | 11:17 | |
| *** salv-orlando has joined #openstack-qa | 11:18 | |
| *** piyush_ has joined #openstack-qa | 11:20 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 11:32 |
| *** salv-orlando has quit IRC | 11:36 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 11:40 |
| *** erlon has joined #openstack-qa | 11:44 | |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 11:45 |
| openstackgerrit | Merged openstack/tempest: Rename image_client to images_client https://review.openstack.org/251317 | 11:53 |
| ylobankov | gmann: ping | 11:55 |
| *** yamamoto has joined #openstack-qa | 12:04 | |
| ylobankov | gmann, oomichi, jordanP, andreaf: hi. could you please take a look https://review.openstack.org/#/c/252432/ and https://review.openstack.org/#/c/252467/ | 12:07 |
| *** yamamoto has quit IRC | 12:08 | |
| dmellado | I'll take a look too ylobankov ;) | 12:09 |
| ylobankov | dmellado: Thanks :) | 12:09 |
| *** dtantsur is now known as dtantsur|brb | 12:09 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Add tests for Nova microversion v2.2 https://review.openstack.org/244996 | 12:16 |
| *** gmann_ has joined #openstack-qa | 12:19 | |
| gmann_ | mtreinish: sdague andreaf oomichi - please review microversion testing set - https://review.openstack.org/#/q/status:open++branch:master+topic:bp/api-microversions-testing-support,n,z | 12:19 |
| gmann_ | now it will pass on gate | 12:19 |
| *** gmann_ has quit IRC | 12:25 | |
| *** _nadya_ has quit IRC | 12:46 | |
| *** dtantsur|brb is now known as dtantsur | 12:46 | |
| *** cdent has joined #openstack-qa | 12:50 | |
| sdague | gmann: is there a devstack-gate or project config change up there somewhere? | 12:51 |
| *** salv-orlando has joined #openstack-qa | 12:54 | |
| *** weshay_xchat has joined #openstack-qa | 12:55 | |
| *** salv-orlando has quit IRC | 12:58 | |
| openstackgerrit | Merged openstack/tempest: Switch run_tests.sh to use subunit-trace https://review.openstack.org/253121 | 13:05 |
| *** cdent has quit IRC | 13:09 | |
| *** cdent has joined #openstack-qa | 13:09 | |
| *** piyush_ has quit IRC | 13:12 | |
| *** yamamoto has joined #openstack-qa | 13:22 | |
| *** slunkad has quit IRC | 13:26 | |
| *** yamamoto has quit IRC | 13:26 | |
| *** rfolco has joined #openstack-qa | 13:28 | |
| openstackgerrit | Ankit Agrawal proposed openstack/tempest: Add test for glance's tasks API https://review.openstack.org/216665 | 13:31 |
| *** regXboi has joined #openstack-qa | 13:35 | |
| *** david-lyle has quit IRC | 13:43 | |
| *** sayali has joined #openstack-qa | 13:47 | |
| *** markvoelker has joined #openstack-qa | 13:47 | |
| *** cdent has quit IRC | 13:57 | |
| *** mriedem has joined #openstack-qa | 13:59 | |
| *** mhorban_ has joined #openstack-qa | 14:00 | |
| *** alaski is now known as lascii | 14:04 | |
| mriedem | sdague: mtreinish: looks like the ebs test is regressed in the last 24 hours | 14:06 |
| mriedem | http://logs.openstack.org/59/252659/1/check/gate-tempest-dsvm-full-ceph/31dc302/console.html#_2015-12-03_20_09_28_874 | 14:07 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message:%5C%22Details:%20%7Bu'message':%20u'Invalid%20volume:%20Volume%20still%20has%201%20dependent%20snapshots.',%20u'code':%20400%7D%5C%22%20AND%20tags:%5C%22console%5C%22 | 14:07 |
| mriedem | only in the ceph jobs | 14:07 |
| mriedem | jbernard: ^ | 14:07 |
| mriedem | failing on all branches | 14:09 |
| *** cdent has joined #openstack-qa | 14:10 | |
| mriedem | tracking here https://bugs.launchpad.net/cinder/+bug/1520296 | 14:11 |
| openstack | Launchpad bug 1520296 in Cinder "tempest test failed test_create_ebs_image_and_check_boot" [High,Confirmed] | 14:11 |
| *** dustins has joined #openstack-qa | 14:12 | |
| *** dkranz has joined #openstack-qa | 14:12 | |
| *** salv-orlando has joined #openstack-qa | 14:16 | |
| *** markvoelker has quit IRC | 14:21 | |
| *** TuanNguyen has joined #openstack-qa | 14:26 | |
| mriedem | sdague: this looks like the problem https://review.openstack.org/#/c/251421/ | 14:27 |
| *** cdent has quit IRC | 14:27 | |
| *** cdent has joined #openstack-qa | 14:28 | |
| jbernard | mriedem: i didn't realize the gate was using the new plugin | 14:39 |
| openstackgerrit | Martin Hickey proposed openstack-dev/devstack: Neutron VPNaaS: Use generated configuration files if available https://review.openstack.org/253530 | 14:43 |
| *** akshai has joined #openstack-qa | 14:44 | |
| samueldmq | hi, I have a change that fits in http://docs.openstack.org/developer/tempest/HACKING.html#bug-fix-on-core-project-needing-tempest-changes | 14:44 |
| samueldmq | but it also affects stable branches | 14:44 |
| samueldmq | should I propose the same patch against those branches ? | 14:45 |
| samueldmq | see https://review.openstack.org/#/c/253270/ | 14:45 |
| openstackgerrit | Joseph Lanoux proposed openstack/tempest: Update configuration options ssh-auth-strategy https://review.openstack.org/253444 | 14:48 |
| *** artom has joined #openstack-qa | 14:52 | |
| *** dansmith is now known as superdan | 14:53 | |
| *** rbak has joined #openstack-qa | 14:54 | |
| *** alex_xu is now known as alexus | 14:56 | |
| sdague | jbernard: you looking into that one? | 14:56 |
| jbernard | sdague: im looking, i thought the plugin was a -nv job for observation initially | 14:56 |
| jbernard | sdague: im trying to correct my project-config understanding | 14:57 |
| jbernard | sdague: it would explain why it suddenly change behaviour | 14:57 |
| *** salv-orlando has quit IRC | 14:57 | |
| *** zz_dimtruck is now known as dimtruck | 14:57 | |
| sdague | jbernard: hmm... yeh, that shouldn't impact this | 14:59 |
| sdague | mriedem: this isn't failing all the jobs | 15:00 |
| sdague | I also don't think the plugin is being used in the main job runs | 15:01 |
| *** tnguyen has quit IRC | 15:05 | |
| *** mfisher_ora has joined #openstack-qa | 15:06 | |
| *** patrickeast has quit IRC | 15:07 | |
| *** patrickeast has joined #openstack-qa | 15:08 | |
| *** shardy has quit IRC | 15:08 | |
| openstackgerrit | Alexander Gubanov proposed openstack/tempest: WIP: Add tests for API microversion with instance tags https://review.openstack.org/233176 | 15:10 |
| *** salv-orlando has joined #openstack-qa | 15:12 | |
| *** ir2ivps6 has quit IRC | 15:12 | |
| sdague | dtroyer: https://review.openstack.org/#/c/252372/ | 15:16 |
| sdague | trying to help the bleeding a bit | 15:17 |
| *** jecarey has joined #openstack-qa | 15:23 | |
| *** yamahata has joined #openstack-qa | 15:24 | |
| *** jecarey has quit IRC | 15:24 | |
| *** jecarey has joined #openstack-qa | 15:25 | |
| *** rbrndt has joined #openstack-qa | 15:26 | |
| *** sayali has quit IRC | 15:26 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: the timeout query isn't helping anything, it's just masking failures https://review.openstack.org/252985 | 15:27 |
| *** nadya has joined #openstack-qa | 15:29 | |
| *** mriedem has quit IRC | 15:30 | |
| *** nadya is now known as Guest26736 | 15:30 | |
| *** Guest26736 is now known as _nadya_ | 15:31 | |
| *** mtanino has joined #openstack-qa | 15:31 | |
| *** tnguyen has joined #openstack-qa | 15:35 | |
| *** openstackrecheck has joined #openstack-qa | 15:37 | |
| *** vgridnev has quit IRC | 15:39 | |
| *** vgridnev has joined #openstack-qa | 15:40 | |
| *** lucasagomes is now known as lucas-hungry | 15:41 | |
| *** salv-orlando has quit IRC | 15:44 | |
| *** dustins has quit IRC | 15:45 | |
| *** markvoelker has joined #openstack-qa | 15:45 | |
| *** shardy has joined #openstack-qa | 15:46 | |
| *** cdent has quit IRC | 15:57 | |
| *** pglass has joined #openstack-qa | 15:58 | |
| *** cdent has joined #openstack-qa | 15:58 | |
| kashyap | sdague: Got a moment to see if this ER query makes sense? - https://review.openstack.org/#/c/253134/ | 15:59 |
| sdague | kashyap: yeh, though have we figured out why that is? | 16:00 |
| sdague | because I was looking through cinder logs and didn't see anything obvious | 16:01 |
| kashyap | sdague: Have to investigate, ETOOMANYBUGS :-( | 16:01 |
| kashyap | sdague: What's worse - I can't see it on multiple runs | 16:01 |
| sdague | oh, it's only in multinode.... | 16:01 |
| *** sabeen1 has joined #openstack-qa | 16:02 | |
| sdague | it's also only on ovh | 16:02 |
| *** grafuls has quit IRC | 16:05 | |
| *** hop|2 has quit IRC | 16:10 | |
| kashyap | sdague: "ovh"? (Sorry for truncated attention, had to attend something at the door) | 16:15 |
| kashyap | Yes, it's only multi-node | 16:15 |
| kashyap | Ah, you're being badgered on -nova too, take your time. | 16:15 |
| *** armax has joined #openstack-qa | 16:18 | |
| *** vgridnev has quit IRC | 16:18 | |
| *** lucas-hungry is now known as lucasagomes | 16:19 | |
| sdague | ovh is one of our providers | 16:20 |
| sdague | kashyap: anyway, yes, this is a good query, +A | 16:20 |
| kashyap | I see | 16:20 |
| kashyap | Thanks for reviewing. | 16:21 |
| sdague | kashyap: I suspect this is going to be a bug we're going to need to look at quite a bit soon, so as much detail as you can add to it would be great | 16:22 |
| kashyap | sdague: Certainly. I've downloaded the logs locally to investigate, and try to artificially reproduce | 16:22 |
| sdague | yeh, I expect it's a timing issue | 16:22 |
| sdague | it should be all in the nova side though, that mount is made by nova-compute | 16:23 |
| sdague | I kind of wonder if the mount is hanging | 16:23 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add a query for libvirt error when 'TestBootVolumePattern' fails https://review.openstack.org/253134 | 16:23 |
| kashyap | afazekas: ^ You also mentioned here you've seen it a couple of times. | 16:23 |
| kashyap | sdague: Have you seen before at all such kind of mounts hanging in this context? | 16:24 |
| sdague | I've seen them in other places | 16:24 |
| afazekas | kashyap, 'Normally' this issue can happen when you do not have the required kernel module | 16:24 |
| *** kbaikov has quit IRC | 16:25 | |
| kashyap | afazekas: iscsi_target_mod? | 16:25 |
| afazekas | something like that | 16:25 |
| kashyap | And, 'target_core_mod | 16:25 |
| kashyap | ' | 16:25 |
| afazekas | it aslo can happen if you have some firewall rule.. | 16:26 |
| kashyap | afazekas: The Arch wiki says, they're in the Kernel and automatically loaded already | 16:26 |
| *** dimtruck is now known as zz_dimtruck | 16:26 | |
| kashyap | Probably not true for Ubuntu Kernels | 16:26 |
| *** jecarey has quit IRC | 16:26 | |
| afazekas | f23 also does not have everything for iscsi | 16:26 |
| afazekas | you need to install kernel-modules as well | 16:27 |
| *** jecarey has joined #openstack-qa | 16:27 | |
| *** jecarey has quit IRC | 16:27 | |
| afazekas | Lot of other issue can cause this kind of issue.. | 16:27 |
| *** ifarkas has quit IRC | 16:28 | |
| kashyap | afazekas: Yeah, seems like one of those cases which involves multiple moving components | 16:31 |
| *** cdent has quit IRC | 16:33 | |
| *** openstackrecheck has quit IRC | 16:37 | |
| *** zz_dimtruck is now known as dimtruck | 16:40 | |
| *** _nadya_ has quit IRC | 16:42 | |
| *** dramalho has joined #openstack-qa | 16:43 | |
| *** cdent has joined #openstack-qa | 16:43 | |
| *** dpaterson has joined #openstack-qa | 16:43 | |
| *** rfolco has quit IRC | 16:48 | |
| *** david-lyle has joined #openstack-qa | 16:49 | |
| *** rcernin has quit IRC | 16:52 | |
| -openstackstatus- NOTICE: The earlier JJB bug which disrupted tox-based job configurations has been reverted and applied; jobs seem to be running successfully for the past two hours. | 16:54 | |
| *** dtantsur is now known as dtantsur|afk | 16:54 | |
| *** rfolco has joined #openstack-qa | 16:55 | |
| *** jlanoux has quit IRC | 16:59 | |
| jbernard | sdague: do you know the site that shows all the failures for a paticular job in the last n minutes? | 17:00 |
| jbernard | sdague: i might be thinking of logstash | 17:01 |
| sdague | jbernard: yeh, logstash | 17:01 |
| jbernard | sdague: awesome, thanks | 17:01 |
| *** salv-orlando has joined #openstack-qa | 17:01 | |
| *** dpaterson has quit IRC | 17:03 | |
| *** vgridnev has joined #openstack-qa | 17:08 | |
| sdague | mtreinish / dtroyer - https://review.openstack.org/#/c/252944/ it's time for that one I think | 17:08 |
| sdague | we promissed m-1 | 17:08 |
| *** david-lyle has quit IRC | 17:13 | |
| *** dramalho has quit IRC | 17:27 | |
| *** yamamoto has joined #openstack-qa | 17:29 | |
| *** dramalho has joined #openstack-qa | 17:30 | |
| *** nic has joined #openstack-qa | 17:32 | |
| *** yamamoto has quit IRC | 17:34 | |
| *** asselin__ has quit IRC | 17:37 | |
| *** jordanP has quit IRC | 17:40 | |
| *** asselin_ has joined #openstack-qa | 17:41 | |
| openstackgerrit | Merged openstack/tempest: Add script to use tempest-lib files in tempest https://review.openstack.org/249560 | 17:42 |
| *** yamahata has quit IRC | 17:45 | |
| *** stevemar_znc is now known as stevemar | 17:47 | |
| *** Swami has joined #openstack-qa | 17:50 | |
| *** dpaterson has joined #openstack-qa | 17:53 | |
| openstackgerrit | Ankit Agrawal proposed openstack/tempest: Add test for glance's tasks API https://review.openstack.org/216665 | 17:55 |
| *** lezbar has quit IRC | 18:01 | |
| *** lezbar has joined #openstack-qa | 18:01 | |
| *** dustins has joined #openstack-qa | 18:05 | |
| *** maht has quit IRC | 18:06 | |
| *** lezbar has quit IRC | 18:06 | |
| *** diegows has joined #openstack-qa | 18:07 | |
| *** dzamboni has quit IRC | 18:07 | |
| *** mylu has joined #openstack-qa | 18:09 | |
| *** yamahata has joined #openstack-qa | 18:10 | |
| *** safchain has quit IRC | 18:12 | |
| *** erlon has quit IRC | 18:16 | |
| openstackgerrit | xing-yang proposed openstack/tempest: Add tempest tests for CG https://review.openstack.org/252213 | 18:31 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Enable db api tests on all backends https://review.openstack.org/253304 | 18:36 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add tests to check migrations and models match https://review.openstack.org/253301 | 18:36 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Remove group regex from testr.conf https://review.openstack.org/253305 | 18:36 |
| *** sabeen1 has quit IRC | 18:36 | |
| *** tosky has quit IRC | 18:37 | |
| mtreinish | HenryG: I've got a question, how do you read these failure messages: http://logs.openstack.org/01/253301/2/check/gate-subunit2sql-python27/65835c1/console.html#_2015-12-04_03_25_13_808 | 18:38 |
| mtreinish | I'm working on adding model sync tests to subunit2sql and clearly the models aren't synced right, but I'm not sure what the output is indicating | 18:38 |
| *** eil397 has joined #openstack-qa | 18:39 | |
| mtreinish | auggy: I've got a regex question. I'm trying to do a re.match() (I don't have a choice on that) that matches mysql, postgresql, or sqlite anywhere in a string case insensitive | 18:43 |
| mtreinish | if it was re.search() it would be as simple as "(?i)(mysql|postgresql|sqlite)" but the tool uses re.match | 18:43 |
| mtreinish | any ideas? | 18:43 |
| auggy | that's weird that that doesn't work | 18:43 |
| auggy | maybe try ((mysql)|(postgresql)|(sqlite)) ? | 18:44 |
| auggy | mtreinish: i'm not as well versed in match vs search but maybe try to group them more explicitly?? not sure | 18:45 |
| mtreinish | auggy: : that doesn't do it: http://paste.openstack.org/show/480917/ | 18:46 |
| mtreinish | auggy: iirc match matches from the start of a string, while search matches for anywhere in the string | 18:46 |
| *** rossella_s has quit IRC | 18:47 | |
| auggy | ohhhh | 18:47 |
| auggy | so you're looking for those terms embedded in function names and also as arguments within parens? | 18:47 |
| mtreinish | yeah | 18:47 |
| mtreinish | auggy: I'm trying to give testr a scheduler hint to put tests using the same db backend on the same worker | 18:49 |
| mtreinish | it will do regex.match(test_id).group(0) and use that as a group id | 18:49 |
| auggy | mtreinish: this is kind of nasty but i got it to match - regex = re.compile("(?i)(mysql|postgresql|sqlite)") | 18:51 |
| auggy | whoops | 18:51 |
| auggy | "(.*?)(mysql|postgresql|sqlite)(.*) | 18:51 |
| auggy | paste vs middle button paste ;D | 18:51 |
| mtreinish | auggy: cool that matches! | 18:51 |
| auggy | yeah but the . is kind of nasty | 18:52 |
| mtreinish | auggy: oh, but group(0) returns the full string | 18:52 |
| auggy | oh no | 18:52 |
| mtreinish | so that won't work because testr will group them individually | 18:52 |
| auggy | group(2) i think would be the right one | 18:52 |
| *** flwang1 has joined #openstack-qa | 18:52 | |
| auggy | or group(1) | 18:52 |
| auggy | group(2) returns it | 18:52 |
| mtreinish | auggy: yeah, but I'm not writing the code that does the match :( | 18:52 |
| mtreinish | heh, it kinda sucks | 18:52 |
| auggy | ah | 18:52 |
| auggy | oh well for that we can use exclusions | 18:53 |
| auggy | look ahead et al | 18:53 |
| auggy | lemme see if i can get that to work | 18:53 |
| mtreinish | auggy: https://github.com/mtreinish/testrepository/blob/master/testrepository/testcommand.py#L572-L581 is what I have to work with :( | 18:53 |
| mtreinish | although I was the one who originally contributed that to testr, so it's still my fault I guess :) | 18:54 |
| auggy | mtreinish: it seems to be ignoring lookahead syntax | 18:55 |
| auggy | i'll see if match uses that | 18:56 |
| auggy | unless i'm doing it wrong cuz i'm new to python ;) | 18:56 |
| auggy | hahaha man i'm having a rough day, i mixed up lookahead and lookbehind syntax | 18:56 |
| mtreinish | haha | 18:57 |
| mtreinish | you've just spent too much time doing python lately | 18:57 |
| auggy | hrm nope the lookahead still isn't working | 18:58 |
| mtreinish | heh, yeah I've been banging my head against this for about an hour | 18:59 |
| mtreinish | but, I also suck at regex :) | 18:59 |
| auggy | omg i know why | 18:59 |
| auggy | i named the new expression regex2 | 18:59 |
| auggy | and i've been testing against regex1 | 19:00 |
| auggy | man i'm special | 19:00 |
| auggy | mtreinish: "(?=<.*?\()(mysql|postgresql|sqlite)(?=.*)" <-- almost | 19:02 |
| auggy | the lookbehind isn't working yet | 19:02 |
| auggy | i think it's too general | 19:02 |
| mtreinish | wow, I never would have come up with that :) | 19:03 |
| mtreinish | auggy: hmm, that doesn't seem to match | 19:04 |
| auggy | yeh the first part is what i'm trying to fix | 19:04 |
| auggy | i think it's too general | 19:04 |
| auggy | this part: (?=<.*?\() | 19:04 |
| *** woodster_ has joined #openstack-qa | 19:05 | |
| *** david-ly_ has joined #openstack-qa | 19:06 | |
| *** cdent has quit IRC | 19:10 | |
| auggy | because it's supposed to be ?<= lol | 19:12 |
| *** mylu has quit IRC | 19:13 | |
| auggy | mtreinish: and apparantly python lookbehind assertions must be of a fixed width length | 19:15 |
| auggy | this is what i ended up with: (?<=\w+?\()(mysql|postgresql|sqlite)(?=.*) | 19:16 |
| auggy | and the \w isn't right because you have _ and . in your strings | 19:16 |
| *** mylu_ has joined #openstack-qa | 19:17 | |
| mtreinish | auggy: hmm, when I run re.compile() on that it complains | 19:17 |
| auggy | yeah lookbehind assertions have to be fixed width | 19:18 |
| mtreinish | auggy: yep: http://paste.openstack.org/show/480922/ | 19:18 |
| auggy | so what i understand from that if your match can be of a variable length it won't work | 19:18 |
| auggy | which is so weird | 19:18 |
| auggy | it gives you nothing to work with with match() basically | 19:18 |
| auggy | unless you can guarantee the lookbehind result is the same length | 19:18 |
| auggy | but the lookahead can vary :/ | 19:19 |
| auggy | that makes no sense | 19:19 |
| * mtreinish is confused :) | 19:19 | |
| *** pglass has quit IRC | 19:19 | |
| auggy | maybe someone who knows more about python knows more about that restriction and how to get around it | 19:19 |
| auggy | the only thing i can think of is if you can manipulate the string somehow | 19:20 |
| auggy | like break it up and then reassemble it | 19:20 |
| auggy | if you're stuck using match | 19:20 |
| auggy | split on the () maybe | 19:20 |
| mtreinish | unfotunately I don't have that option, because the test runner is running match over the list of test_ids it will run | 19:21 |
| mtreinish | I can push the patch to testr to switch it to a search() | 19:21 |
| clarkb | mtreinish: auggy the scrollback is large but what are you trying to achieve? | 19:21 |
| mtreinish | I don't think that'll be controversial | 19:21 |
| mtreinish | clarkb: I'm trying to set a testr group regex on the db backend for the subunit2sql tests | 19:21 |
| mtreinish | clarkb: testr does an re.match().group(0) to get the group id | 19:22 |
| mtreinish | clarkb: http://paste.openstack.org/show/480917/ was my early failed attempts | 19:22 |
| auggy | i was trying to use lookbehind and lookahead | 19:22 |
| auggy | but we ran into an issue where python requires lookbehind to be a fixed width or something? | 19:23 |
| auggy | that's so weird | 19:23 |
| *** mylu_ has quit IRC | 19:23 | |
| clarkb | what do the strings that you are matching look like? | 19:23 |
| clarkb | (I would be amazed if you need lookbehind | 19:23 |
| mtreinish | clarkb: there are 2: | 19:23 |
| mtreinish | subunit2sql.tests.db.test_api.TestDatabaseAPI.test_get_test_runs_dicts_with_no_start_time(mysql) | 19:23 |
| *** mylu has joined #openstack-qa | 19:23 | |
| *** gfidente has quit IRC | 19:23 | |
| mtreinish | subunit2sql.tests.migrations.test_migrations.TestWalkMigrations.test_postgresql_opportunistically | 19:23 |
| mtreinish | well 2 forms, it's either testscenarios or explicit in the test name | 19:24 |
| clarkb | .*\((\w)\) | 19:24 |
| auggy | but he needs to capture group(0) is the thing | 19:24 |
| clarkb | right isn't that going to capture the \w as group 0? | 19:25 |
| *** mylu has quit IRC | 19:25 | |
| *** dkranz has quit IRC | 19:25 | |
| mtreinish | clarkb: also doesn't that only match the testscenarios case? | 19:25 |
| *** mylu has joined #openstack-qa | 19:26 | |
| clarkb | mtreinish: ah yes need a ? to make the missing (mysql) match | 19:26 |
| *** dramalho has quit IRC | 19:26 | |
| clarkb | so looks like this .*(:?\((\w)\))? | 19:26 |
| *** dkranz has joined #openstack-qa | 19:27 | |
| clarkb | I think :? is don't make this a capture group | 19:27 |
| mtreinish | clarkb: hmm it returns the full string: http://paste.openstack.org/show/480926/ | 19:28 |
| melwitt | mtreinish: do you happen to know if floating_ip is really required for the _check_network_internal_connectivity in test_network_basic_ops? https://github.com/openstack/tempest/blob/4662ac0/tempest/scenario/test_network_basic_ops.py#L304 | 19:28 |
| melwitt | mtreinish: I'm trying to cover a gap in cells testing where floating ips are not supported, so we would just want to check if internal network connectivity is working, at a minimum | 19:29 |
| auggy | mtreinish: clarkb: ?: seems to be being ignored in match | 19:30 |
| mtreinish | melwitt: tbh, I'm not sure. Those neutron scenarios are uneccesarily complex imo | 19:30 |
| auggy | i tried replacing my ?>= with it and it didn't work | 19:30 |
| melwitt | yeah, ?: is for the non capturinggroup | 19:30 |
| clarkb | ah I got them flipped around | 19:30 |
| mtreinish | melwitt: you might be able to modify the test to not need floating ips, but at the same time floating ips might be part of what's being verified there | 19:31 |
| *** dustins_ has joined #openstack-qa | 19:31 | |
| melwitt | mtreinish: heh. yeah, I wasn't sure. I'll consult someone in neutron, I didn't realize that's where it originated | 19:31 |
| auggy | clarkb: mtreinish: so this is the expression i had with lookaheads and lookbehinds: (?<=\.+?\()(mysql|postgresql|sqlite)(?=.+) | 19:32 |
| *** dustins has quit IRC | 19:33 | |
| mtreinish | melwitt: there an abuse of what I consider "scenario" tests which I've always viewed as being an end user workflow | 19:33 |
| mtreinish | I've never known an end user who wants to do any of the things in those neutron tests :) | 19:33 |
| melwitt | mtreinish: I suppose you could stretch it by saying "user wants to ssh into the instance" and all the testing in between is needed to prove that | 19:34 |
| auggy | oh i mispasted something, it's (?<=.+?\()(mysql|postgresql|sqlite)(?=.+) | 19:35 |
| auggy | this is what i get for group(0) using ?: instead of ?<= 'subunit2sql.tests.db.test_api.TestDatabaseAPI.test_get_test_runs_dicts_with_no_start_time(mysql' | 19:36 |
| *** flwang1 has quit IRC | 19:36 | |
| mtreinish | melwitt: well, except we have other tests that do the ssh thing and don't require all of that weirdness. But, meh they exist, so it's a thing now | 19:36 |
| clarkb | mtreinish: auggy what should group(0) be in the second example above? | 19:36 |
| clarkb | is it supposed to be empty string/none? | 19:36 |
| melwitt | mtreinish: oh? maybe I should find those. I searched the repo for "login" to find connectivity tests and test_network_basic_ops is all I found | 19:37 |
| auggy | mysql | 19:37 |
| clarkb | auggy: where do you get mysql from in that case? | 19:37 |
| *** cdent has joined #openstack-qa | 19:37 | |
| *** mylu has quit IRC | 19:37 | |
| clarkb | the first example has (mysql) so thats mysql | 19:37 |
| clarkb | second example is postgres | 19:37 |
| mtreinish | clarkb: it should be either mysql, postgresql, or sqlite | 19:37 |
| *** mylu has joined #openstack-qa | 19:38 | |
| clarkb | mtreinish: auggy and thats true for both examples? | 19:38 |
| mtreinish | clarkb: I'm trying to tell testr to group tests with same backend on the same worker | 19:38 |
| auggy | i just verified that my expression should match but just breaking it out into groups and removing the special cases | 19:38 |
| auggy | subunit2sql.tests.db.test_api.TestDatabaseAPI.test_get_test_runs_dicts_with_no_start_time( matches group(1) | 19:38 |
| clarkb | I see | 19:38 |
| auggy | so the expressions are accurate in my expression, but match() seems to be ignoring ?: | 19:38 |
| auggy | clarkb: that's why i used lookaheds/behinds | 19:39 |
| auggy | but i haven't done much regex with python sooo | 19:39 |
| *** david-ly_ is now known as david-lyle | 19:39 | |
| clarkb | would .*(mysql|postgres|sqlite)?.* work? | 19:39 |
| auggy | clarkb: wouldn't that return the whole string for group(0)? | 19:39 |
| mtreinish | clarkb: it didn't when I tried it | 19:39 |
| mtreinish | auggy: yeah, iirc when I tried it thats what I got | 19:39 |
| clarkb | oh so in that case | 19:40 |
| auggy | from what i can tell with match group(0) is the whole match | 19:40 |
| clarkb | I would suggest testr group a feature for a named group | 19:40 |
| mtreinish | clarkb, auggy: https://github.com/testing-cabal/testrepository/pull/19 | 19:40 |
| mtreinish | I have a feeling it won't be that simple | 19:41 |
| auggy | mtreinish: that's probably a wiser move but yeah | 19:41 |
| auggy | yeah there might be some things that dependend on match | 19:41 |
| auggy | why are regexes in python so broken? | 19:41 |
| auggy | that's just so weird | 19:41 |
| clarkb | mtreinish: I think having a flag for --test-group-group foo then naming your capture group foo is the most flexible | 19:41 |
| clarkb | auggy: how ar ethey broken? | 19:41 |
| clarkb | lookbehind is CFG iirc | 19:41 |
| clarkb | or maybe just pushdown | 19:42 |
| clarkb | its counting | 19:42 |
| auggy | it was more of a rhetorical question out of frustration, not trying to start a language debate | 19:42 |
| mtreinish | clarkb: well the scheduler hints are a config thing, but yeah doing it as a seperate opt might be better | 19:42 |
| auggy | it just works differently in python than other places i'm used ot using it | 19:42 |
| mtreinish | we'll see what lifeless says :) | 19:42 |
| auggy | mtreinish: i gotta hop back onto the stuff i was working on! feel free to bug me with other regex questions if it's at all helpful for you :) | 19:43 |
| mtreinish | melwitt: most of the scenario tests that boot a server ssh into them | 19:43 |
| mtreinish | auggy: it is, I suck at regex :) | 19:43 |
| mtreinish | I don't think I've ever constructed one by myself that did it 100% correct | 19:44 |
| mtreinish | I always miss something | 19:44 |
| *** mylu has quit IRC | 19:44 | |
| auggy | whatever "correct" means | 19:44 |
| auggy | yeah they are tricky | 19:44 |
| mtreinish | hah, true | 19:44 |
| auggy | i've just done a lot of perl | 19:44 |
| auggy | ;D | 19:44 |
| *** markvoelker has quit IRC | 19:44 | |
| clarkb | mtreinish: the nice thing about having an optional name is it can be at any arbitrary location | 19:45 |
| clarkb | you don't have to do magic to overload what group(0) is | 19:45 |
| melwitt | I did this to capture the mysql or postgres or sqlite. I don't know if it fits all the cases you need m = re.search(r'[^(]*(mysql|postgres|sqlite)', s) | 19:46 |
| mtreinish | melwitt: well except testr uses re.match :) | 19:46 |
| mtreinish | that's the source of all the trouble | 19:46 |
| mtreinish | melwitt: https://github.com/mtreinish/testrepository/blob/master/testrepository/testcommand.py#L572-L581 | 19:46 |
| melwitt | sorry, I don't have the rest of the context | 19:46 |
| mtreinish | no worries, I've banged my head against this for long enough, it's not really important | 19:47 |
| openstackgerrit | Merged openstack-dev/devstack: create apt_get_update to try to work around broken mirrors https://review.openstack.org/252372 | 19:47 |
| mtreinish | there are only like 50 tests, so it's the diff between like 1.5 mins and 2.5 mins of run time :) | 19:47 |
| melwitt | mtreinish: okay. I'll look at the other scenario tests, hopefully it's already covered in the cells job. with all the exclusions, I was concerned it wasn't covered | 19:47 |
| mtreinish | melwitt: for example: https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py ssh's into the guests to verify the same volume is used | 19:48 |
| mtreinish | (it writes a timestamp on the fs) | 19:49 |
| mtreinish | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_minimum_basic.py logs in to check that the cinder volume is attached | 19:49 |
| *** rbrndt has quit IRC | 19:49 | |
| melwitt | mtreinish: thanks. I have a vague memory of running the equivalent of our tempest job locally on a patch I was working on, wherein I broke network connectivity not passing network_info_cache or something. and all the tests passed. so now I'm rummaging around to see which tests do the ssh | 19:50 |
| mtreinish | melwitt: there are also api tests that do ssh, but we turn that off in the gate (except for one nv job on tempest) | 19:51 |
| mtreinish | mriedem: http://git.openstack.org/cgit/openstack/tempest/tag/?h=8 sorry I forgot about that before | 19:52 |
| *** mylu has joined #openstack-qa | 19:52 | |
| *** mylu has quit IRC | 19:54 | |
| *** regXboi has quit IRC | 19:55 | |
| *** mylu has joined #openstack-qa | 19:56 | |
| melwitt | mtreinish: it appears not possible to do what you want with re.match https://docs.python.org/2/library/re.html#search-vs-match | 19:56 |
| mtreinish | melwitt: :( that's what I was afraid of | 19:56 |
| mtreinish | melwitt: I guess https://github.com/testing-cabal/testrepository/pull/19 or something like it is the answer | 19:57 |
| *** sabeen has joined #openstack-qa | 19:59 | |
| melwitt | mtreinish: yeah, I think so. it looks like the group_regex is intended to work only on "test ids" | 20:00 |
| mtreinish | melwitt: yeah that callback function gets run on each test_id that testr is going to run | 20:02 |
| *** dimtruck is now known as zz_dimtruck | 20:02 | |
| melwitt | mtreinish: and test id is in the beginning of the string and the rest of the test name after it I guess? | 20:02 |
| mtreinish | melwitt: the whole string is the test_id | 20:03 |
| mtreinish | the full path, test_name, attrs, testscenarios, etc | 20:03 |
| melwitt | ohh. okay | 20:03 |
| mtreinish | it makes doing the filtering and scheduling easier, but it makes result tracking more difficult | 20:06 |
| *** sabeen has quit IRC | 20:09 | |
| *** dkranz has quit IRC | 20:10 | |
| *** lucasagomes is now known as lucas-afk | 20:10 | |
| *** dkranz has joined #openstack-qa | 20:12 | |
| mtreinish | timothyb89, anteaya, david-lyle, regXboi, jhesketh: I think it's time we announce dashboard on the ML. (now that all the db migrations are done) This is my draft: https://etherpad.openstack.org/p/1Ibg78gTIn | 20:17 |
| mtreinish | I'd appreciate any feedback on it before I push it out | 20:17 |
| *** dtroyer has quit IRC | 20:19 | |
| timothyb89 | mtreinish: looks good to me, are there any last-minute things we want to take care of code-wise first? | 20:20 |
| mtreinish | timothyb89: the only thing I was thinking of was: https://review.openstack.org/#/c/253214/ | 20:20 |
| timothyb89 | ah, I'll take a look | 20:21 |
| mtreinish | but it is iterative, there will always be something else to do :) | 20:21 |
| timothyb89 | for sure | 20:21 |
| *** sabeen has joined #openstack-qa | 20:23 | |
| *** dtroyer has joined #openstack-qa | 20:28 | |
| *** jbernard has quit IRC | 20:32 | |
| david-lyle | mtreinish: LGTM | 20:34 |
| mtreinish | david-lyle: ok 2 +2s away it goes :) | 20:35 |
| *** rbrndt has joined #openstack-qa | 20:40 | |
| mtreinish | timothyb89: I'm looking at the openstack health todo list, the breadcumbs thing. Is that the thing you implemented? | 20:43 |
| timothyb89 | mtreinish: I guess so, though I don't know if that's necessarily what I implemented :) | 20:44 |
| mtreinish | TBH, I'm not sure what it means, but you called the commit a breadcrumb menu or something like that :) | 20:44 |
| timothyb89 | I'd say its an equivalent feature, hopefully it satisfies whoever wrote that | 20:45 |
| lifeless | mtreinish: I'll look on monday but having a bug and or examples of what you're struggling would be great | 20:47 |
| *** pglass has joined #openstack-qa | 20:48 | |
| mtreinish | lifeless: ok, I'll add an example to the PR | 20:48 |
| *** jasonsb has quit IRC | 20:49 | |
| *** diegows has quit IRC | 20:51 | |
| *** tnguyen has quit IRC | 20:51 | |
| mtreinish | timothyb89: so on: https://review.openstack.org/#/c/231287/ I'm kinda stuck | 20:52 |
| mtreinish | the console log is showing an error about the date format, but I couldn't figure out where the call is coming from | 20:52 |
| pc_m | Hi! In a newish repo, I'm trying to get the docs job to run. Anyone know why this is failing https://review.openstack.org/#/c/253660/ ? | 20:52 |
| mtreinish | timothyb89: well ignoring the test add screw up :) | 20:53 |
| *** mylu has quit IRC | 20:53 | |
| timothyb89 | mtreinish: hmm, odd... I need to run an errand, but I'll take a look in ~20 minutes | 20:54 |
| mtreinish | timothyb89: ok, cool thanks | 20:55 |
| *** maht has joined #openstack-qa | 20:59 | |
| *** ajmiller_ has joined #openstack-qa | 21:00 | |
| *** pvaneck has joined #openstack-qa | 21:03 | |
| *** diegows has joined #openstack-qa | 21:03 | |
| *** ajmiller has quit IRC | 21:03 | |
| *** weshay_xchat is now known as weshay | 21:05 | |
| *** diegows has quit IRC | 21:05 | |
| *** akshai has quit IRC | 21:12 | |
| *** cp16net has quit IRC | 21:14 | |
| *** cp16net has joined #openstack-qa | 21:16 | |
| *** pglass has quit IRC | 21:18 | |
| *** zz_dimtruck is now known as dimtruck | 21:24 | |
| *** jasonsb has joined #openstack-qa | 21:26 | |
| *** dustins_ has quit IRC | 21:27 | |
| *** boris-42_ has joined #openstack-qa | 21:28 | |
| *** gszasz has quit IRC | 21:31 | |
| *** rfolco has quit IRC | 21:31 | |
| *** TravT has joined #openstack-qa | 21:33 | |
| *** sabeen has quit IRC | 21:34 | |
| *** afazekas has quit IRC | 21:39 | |
| *** afazekas has joined #openstack-qa | 21:40 | |
| openstackgerrit | John Warren proposed openstack/tempest: Split out Neutron metering label rules client https://review.openstack.org/253133 | 21:40 |
| *** TuanNguyen has quit IRC | 21:42 | |
| timothyb89 | mtreinish: I think the source of the bug is line #66 here https://review.openstack.org/#/c/231287/15/app/js/controllers/test.js | 21:54 |
| timothyb89 | mtreinish: 'var startDate = ...' redefines the var in that scope, which overwrites the value passed into the controller | 21:56 |
| timothyb89 | mtreinish: renaming the 2nd startDate should fix that issue, at least | 21:56 |
| *** mwagner_lap has quit IRC | 21:59 | |
| timothyb89 | mtreinish: then on line #69 it should be testName rather than test_name ... after those fixes the page seems to be loading correctly | 22:02 |
| mtreinish | timothyb89: ok cool | 22:04 |
| * mtreinish gives it a try | 22:04 | |
| timothyb89 | mtreinish: that said, I do find it a bit suspicious that "Success and Failures" always shows 1 pass and 0 fails | 22:07 |
| *** mylu has joined #openstack-qa | 22:08 | |
| mtreinish | timothyb89: yeah looking at how it was refactored when I wasn't working on it there isn't any aggregation going on | 22:08 |
| mtreinish | so success and fails will alway be 1 or zero | 22:08 |
| mtreinish | and it'll be a point for each time the test is run | 22:08 |
| mtreinish | timothyb89: http://i.imgur.com/9ifFLom.png | 22:10 |
| timothyb89 | hmm, makes sense I suppose | 22:10 |
| mtreinish | well it makes sense why it's doing it. But it's a bad graph | 22:10 |
| mtreinish | should probably be a stacked bar graph or something for each day or something | 22:11 |
| *** mfisher_ora has quit IRC | 22:11 | |
| timothyb89 | that would make more sense | 22:11 |
| mtreinish | and for the run_time graph I really would like to see if we can use the same formatting as subunit2sql-graph run_tim | 22:11 |
| mtreinish | e | 22:11 |
| mtreinish | where it shows the average and std dev | 22:12 |
| mtreinish | but those are the next steps before I think it's ready to land (well that and a table of something) | 22:12 |
| mtreinish | at least now it's working :) | 22:12 |
| timothyb89 | is stdev and average recorded with subunit2sql? | 22:13 |
| mtreinish | timothyb89: no, I leveraged pandas to do the calculations in subunit2sql-graph | 22:17 |
| mtreinish | timothyb89: https://github.com/openstack-infra/subunit2sql/blob/master/subunit2sql/analysis/run_time.py#L56-L57 | 22:18 |
| timothyb89 | mtreinish: oh, okay, makes sense | 22:18 |
| *** mylu has quit IRC | 22:18 | |
| *** mylu has joined #openstack-qa | 22:19 | |
| mtreinish | timothyb89: I'm sure d3 has something similar, but I wonder if it makes sense to do that on the rest api side | 22:19 |
| timothyb89 | mtreinish: d3 can do stdev/mean/etc, if there's not an excessive amount of data to deal with that should be plenty fast | 22:21 |
| mtreinish | ok, I'l look into playing with that then | 22:21 |
| mtreinish | lets see how long it takes me to figure out the js syntax to do it :) | 22:21 |
| timothyb89 | should be something like d3.mean(timeEntries, function(d) { return d.y; }) | 22:23 |
| timothyb89 | though there doesn't seem to be a function for rolling means | 22:25 |
| *** david-ly_ has joined #openstack-qa | 22:29 | |
| openstackgerrit | Matthew Treinish proposed openstack/openstack-health: Add test detail page https://review.openstack.org/231287 | 22:29 |
| *** lascii is now known as alaski | 22:29 | |
| *** mylu has quit IRC | 22:30 | |
| timothyb89 | mtreinish: that said, doing some extra processing on the server might not be a bad idea, for lots tests we're looking at > 500kb of data to download, sometimes up to ~1MB | 22:30 |
| *** mylu has joined #openstack-qa | 22:30 | |
| *** david-ly_ is now known as david-lyle_ | 22:31 | |
| *** david-lyle has quit IRC | 22:32 | |
| *** mylu has quit IRC | 22:35 | |
| *** dimtruck is now known as zz_dimtruck | 22:36 | |
| EmilienM | mtreinish: I would love having puppet projects on http://status.openstack.org/openstack-health/#/ | 22:37 |
| EmilienM | does it makes sense to you? | 22:37 |
| *** dpaterson has quit IRC | 22:38 | |
| mtreinish | EmilienM: sure, the filters that collect subunit streams are: http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/files/logstash/jenkins-log-client.yaml#n14 | 22:38 |
| mtreinish | I take it the puppet jobs don't match any of those? | 22:38 |
| EmilienM | mtreinish: ok, will add some modules and see how it behaves | 22:38 |
| EmilienM | mtreinish: it does not | 22:39 |
| EmilienM | mtreinish: we don't use subunit | 22:39 |
| mtreinish | EmilienM: well the jobs that run tempest do | 22:40 |
| mtreinish | the beaker tests probably dont (although it wouldn't be too hard to do in ruby I don't think) | 22:40 |
| EmilienM | mtreinish: we have 2 jobs that run tempest | 22:40 |
| EmilienM | puppet-openstack-integration | 22:40 |
| mtreinish | EmilienM: I'd recommend pushing a patch that changes L17 (or all the job selection regex) to be ".*(tempest|grenade).*" | 22:42 |
| mtreinish | hopefully that won't be too generic and pick up things we don't want | 22:42 |
| mtreinish | EmilienM: and as long as the puppet jobs that run tempest put the subunit stream in the same place with the same filename it'll just pick it up automatically | 22:43 |
| EmilienM | let's see http://logs.openstack.org/20/253620/1/check/gate-puppet-openstack-integration-scenario001-dsvm-centos7/ddbea3d/ | 22:43 |
| EmilienM | I think we're missing something | 22:44 |
| mtreinish | hmm, yeah you need to save the subunit stream to the logs dir | 22:44 |
| EmilienM | maybe I'm not copying it | 22:44 |
| mtreinish | EmilienM: post test run from the tempest dir do 'testr last --subunit > log_dir/testrepository.subunit' | 22:44 |
| EmilienM | mtreinish: this is how I run tempest: https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L96 | 22:45 |
| mtreinish | like http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n629 | 22:45 |
| mtreinish | EmilienM: yeah in the cleanup phase when you collect all the artifacts you want to do 'pushd /tmp/openstack/tempest; testr last --subunit > $LOGDIR/testrepository.subunit; popd' or something like that | 22:46 |
| EmilienM | ok I see | 22:47 |
| EmilienM | mtreinish: I might also need to patch https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh | 22:47 |
| mtreinish | EmilienM: I would just do it right from that script | 22:48 |
| mtreinish | EmilienM: basically just copy this function: http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n611 | 22:48 |
| mtreinish | although you can skip the subunit2html thing (unless you want to do that) and you probably will have to call testr from inside the tox venv, so .tox/all/bin/testr instead of testr | 22:49 |
| *** eil397 has quit IRC | 22:50 | |
| mtreinish | EmilienM: oh, I jsut realized you don't say tempest in the job name too | 22:51 |
| mtreinish | EmilienM: can we rename that to include tempest, so 'gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7'? | 22:52 |
| EmilienM | mtreinish: something like https://review.openstack.org/253728? | 22:52 |
| EmilienM | mtreinish: yes of course we can | 22:52 |
| mtreinish | EmilienM: yeah that'll save the subunit stream | 22:53 |
| mtreinish | and then do you need to modify copy_puppet_logs.sh to pick it up? | 22:53 |
| EmilienM | yes | 22:53 |
| EmilienM | and I rename jobs | 22:53 |
| EmilienM | and life is beautiful | 22:53 |
| mtreinish | EmilienM: well I need to fix the selection regex in the gearman client so that it'll pick up the puppet jobs post rename, but that's simple :) | 22:54 |
| *** salv-orl_ has joined #openstack-qa | 22:54 | |
| EmilienM | cool | 22:54 |
| EmilienM | thanks for your help mtreinish ! | 22:54 |
| *** jasonsb has quit IRC | 22:55 | |
| *** salv-orlando has quit IRC | 22:57 | |
| mtreinish | EmilienM: https://review.openstack.org/#/c/253729/ | 22:58 |
| EmilienM | https://review.openstack.org/253728 and https://review.openstack.org/253730 | 22:59 |
| *** mylu has joined #openstack-qa | 22:59 | |
| mtreinish | EmilienM: reviewed | 23:01 |
| EmilienM | mtreinish: ok | 23:03 |
| *** shardy has quit IRC | 23:07 | |
| *** dims has quit IRC | 23:10 | |
| *** cdent has quit IRC | 23:14 | |
| *** pglass has joined #openstack-qa | 23:18 | |
| *** aix has quit IRC | 23:19 | |
| *** pglass has quit IRC | 23:22 | |
| *** pglass has joined #openstack-qa | 23:22 | |
| *** pglass has quit IRC | 23:27 | |
| *** weshay has quit IRC | 23:33 | |
| *** openstack has joined #openstack-qa | 23:38 | |
| *** aysyd has quit IRC | 23:38 | |
| *** ajmiller_ has quit IRC | 23:40 | |
| *** dims has joined #openstack-qa | 23:47 | |
| *** nicholasgoracke has joined #openstack-qa | 23:52 | |
| *** TravT has left #openstack-qa | 23:53 | |
| *** rbrndt has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
