| *** markvoelker has quit IRC | 00:01 | |
| *** asselin_ has joined #openstack-qa | 00:02 | |
| *** kzaitsev_mb has quit IRC | 00:02 | |
| *** yangyapeng has quit IRC | 00:05 | |
| *** asselin_ has quit IRC | 00:07 | |
| *** rcernin has quit IRC | 00:13 | |
| *** zeih has joined #openstack-qa | 00:21 | |
| *** zeih has quit IRC | 00:25 | |
| *** kzaitsev_mb has joined #openstack-qa | 00:29 | |
| *** hoangcx has joined #openstack-qa | 00:30 | |
| *** hoangcx has left #openstack-qa | 00:31 | |
| *** kzaitsev_mb has quit IRC | 00:55 | |
| *** mriedem has quit IRC | 01:04 | |
| *** dpaterson has joined #openstack-qa | 01:10 | |
| *** yangyapeng has joined #openstack-qa | 01:11 | |
| *** hoangcx_ has joined #openstack-qa | 01:27 | |
| *** hoangcx_ has left #openstack-qa | 01:29 | |
| *** hoangcx has joined #openstack-qa | 01:33 | |
| gmann | oomichi: about - https://review.openstack.org/#/c/327971/5 | 01:33 |
|---|---|---|
| gmann | oomichi: i think we should mention v2 service clients in release notes | 01:34 |
| *** david-lyle has quit IRC | 01:34 | |
| gmann | oomichi: if you think it should and you want to do as follow up i can +A | 01:34 |
| *** hoangcx_ has joined #openstack-qa | 01:36 | |
| *** EinstCrazy has joined #openstack-qa | 01:37 | |
| *** hoangcx__ has joined #openstack-qa | 01:38 | |
| *** hoangcx__ has left #openstack-qa | 01:38 | |
| *** hoangcx has quit IRC | 01:39 | |
| *** hoangcx_ has quit IRC | 01:41 | |
| *** timothyb89 has quit IRC | 01:41 | |
| *** timothyb89 has joined #openstack-qa | 01:42 | |
| *** kzaitsev_mb has joined #openstack-qa | 01:51 | |
| *** hoangcx has joined #openstack-qa | 01:55 | |
| *** hoangcx has left #openstack-qa | 01:56 | |
| *** kzaitsev_mb has quit IRC | 01:56 | |
| *** markvoelker has joined #openstack-qa | 01:58 | |
| *** EinstCrazy has quit IRC | 02:01 | |
| *** markvoelker has quit IRC | 02:02 | |
| *** EinstCrazy has joined #openstack-qa | 02:04 | |
| *** yangyape_ has joined #openstack-qa | 02:04 | |
| *** yangyapeng has quit IRC | 02:05 | |
| *** yamamoto_ has joined #openstack-qa | 02:05 | |
| *** zz_dimtruck is now known as dimtruck | 02:06 | |
| *** dpaterson has quit IRC | 02:10 | |
| *** Poornima has joined #openstack-qa | 02:11 | |
| *** yangyapeng has joined #openstack-qa | 02:12 | |
| *** r-mibu has quit IRC | 02:12 | |
| *** hoangcx_ has joined #openstack-qa | 02:15 | |
| *** hoangcx_ has left #openstack-qa | 02:15 | |
| *** bkopilov has quit IRC | 02:15 | |
| *** yangyape_ has quit IRC | 02:16 | |
| *** r-mibu has joined #openstack-qa | 02:17 | |
| *** zeih has joined #openstack-qa | 02:21 | |
| *** yangyape_ has joined #openstack-qa | 02:26 | |
| *** zeih has quit IRC | 02:26 | |
| *** yangyap__ has joined #openstack-qa | 02:28 | |
| *** yangyape_ has quit IRC | 02:28 | |
| *** yangyapeng has quit IRC | 02:29 | |
| *** yamamoto_ has quit IRC | 02:33 | |
| *** hieulq has quit IRC | 02:33 | |
| *** yamamoto_ has joined #openstack-qa | 02:37 | |
| *** hieulq has joined #openstack-qa | 02:37 | |
| *** yangyapeng has joined #openstack-qa | 02:45 | |
| *** hieulq has quit IRC | 02:46 | |
| *** yamamoto_ has quit IRC | 02:46 | |
| *** yangyap__ has quit IRC | 02:47 | |
| *** yamamoto_ has joined #openstack-qa | 02:48 | |
| *** hieulq has joined #openstack-qa | 02:50 | |
| *** yamamoto_ has quit IRC | 02:50 | |
| openstackgerrit | Merged openstack/tempest: Add unit test coverage for account generator https://review.openstack.org/327182 | 02:51 |
| *** kzaitsev_mb has joined #openstack-qa | 02:53 | |
| *** kzaitsev_mb has quit IRC | 02:59 | |
| *** artom has quit IRC | 03:03 | |
| *** ramishra has joined #openstack-qa | 03:06 | |
| *** artom has joined #openstack-qa | 03:08 | |
| *** artom has quit IRC | 03:08 | |
| *** artom has joined #openstack-qa | 03:08 | |
| *** gouthamr has quit IRC | 03:12 | |
| *** zackf has joined #openstack-qa | 03:17 | |
| *** yangyape_ has joined #openstack-qa | 03:28 | |
| *** yangyapeng has quit IRC | 03:28 | |
| openstackrecheck | Console logs not available after 13:23s for gate-nova-pep8 326326,5,deacd7f | 03:30 |
| *** hoangcx has joined #openstack-qa | 03:32 | |
| *** hoangcx has left #openstack-qa | 03:32 | |
| *** yamamoto_ has joined #openstack-qa | 03:34 | |
| *** yamamoto_ has quit IRC | 03:36 | |
| *** EinstCrazy has quit IRC | 03:36 | |
| *** EinstCrazy has joined #openstack-qa | 03:37 | |
| *** bkopilov has joined #openstack-qa | 03:40 | |
| *** artom has quit IRC | 03:50 | |
| *** artom has joined #openstack-qa | 03:50 | |
| *** tochi has quit IRC | 03:51 | |
| *** tochi has joined #openstack-qa | 03:54 | |
| *** kzaitsev_mb has joined #openstack-qa | 03:55 | |
| *** EinstCrazy has quit IRC | 03:57 | |
| *** EinstCrazy has joined #openstack-qa | 03:57 | |
| *** markvoelker has joined #openstack-qa | 03:59 | |
| *** kzaitsev_mb has quit IRC | 04:00 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/tempest: Make --concurrency not required in account_generator cmd https://review.openstack.org/328827 | 04:01 |
| *** markvoelker has quit IRC | 04:03 | |
| gmann | andreaf: oomichi masayukig mtreinish : will be simple one to look - https://review.openstack.org/328827 | 04:03 |
| *** dimtruck is now known as zz_dimtruck | 04:06 | |
| masayukig | gmann: +2 | 04:09 |
| gmann | masayukig: Thanks | 04:10 |
| *** zz_dimtruck is now known as dimtruck | 04:16 | |
| *** artom has quit IRC | 04:17 | |
| *** artom has joined #openstack-qa | 04:18 | |
| *** zeih has joined #openstack-qa | 04:22 | |
| *** dimtruck is now known as zz_dimtruck | 04:26 | |
| *** zeih has quit IRC | 04:27 | |
| *** M00nr41n has left #openstack-qa | 04:29 | |
| *** yamamoto_ has joined #openstack-qa | 04:33 | |
| openstackgerrit | Benny Kopilov proposed openstack/tempest: Fix nova admin naming https://review.openstack.org/328832 | 04:33 |
| *** artom has quit IRC | 04:35 | |
| *** artom has joined #openstack-qa | 04:40 | |
| *** artom has quit IRC | 04:40 | |
| *** artom has joined #openstack-qa | 04:41 | |
| *** Alex_Stef has joined #openstack-qa | 04:47 | |
| *** pgadiya has joined #openstack-qa | 04:50 | |
| *** asselin_ has joined #openstack-qa | 04:51 | |
| *** kzaitsev_mb has joined #openstack-qa | 04:55 | |
| *** kzaitsev_mb has quit IRC | 05:00 | |
| *** zeih has joined #openstack-qa | 05:02 | |
| openstackgerrit | Jianghua Wang proposed openstack/tempest: tempest: use host instead of hypervisor for aggregate test https://review.openstack.org/328836 | 05:05 |
| *** zeih has quit IRC | 05:07 | |
| *** artom has quit IRC | 05:09 | |
| *** artom has joined #openstack-qa | 05:10 | |
| *** nadya has joined #openstack-qa | 05:16 | |
| *** gvrangan has joined #openstack-qa | 05:18 | |
| *** salv-orlando has joined #openstack-qa | 05:21 | |
| *** salv-orl_ has quit IRC | 05:21 | |
| openstackrecheck | Console logs not available after 13:24s for gate-cinder-pep8 328052,8,94034a4 | 05:22 |
| *** artom has quit IRC | 05:24 | |
| *** itzikb has joined #openstack-qa | 05:24 | |
| *** artom has joined #openstack-qa | 05:25 | |
| *** dave-mccowan has quit IRC | 05:27 | |
| *** salv-orl_ has joined #openstack-qa | 05:29 | |
| *** salv-orlando has quit IRC | 05:31 | |
| openstackgerrit | Masayuki Igawa proposed openstack/stackviz: Remove duplicated imports from test-requirements.txt https://review.openstack.org/328846 | 05:48 |
| *** yangyapeng has joined #openstack-qa | 05:51 | |
| *** jianghuaw has joined #openstack-qa | 05:52 | |
| *** yangyape_ has quit IRC | 05:55 | |
| *** kzaitsev_mb has joined #openstack-qa | 05:57 | |
| jianghuaw | hi @zhufl. The Citrix XenServer CI has been broken. Per my understanding, it should be caused this change https://review.openstack.org/#/c/316672. | 05:58 |
| jianghuaw | I have a quick fix on it: https://review.openstack.org/#/c/328836 | 05:59 |
| jianghuaw | Could you help to review it? | 05:59 |
| *** markvoelker has joined #openstack-qa | 05:59 | |
| *** cody-somerville_ has quit IRC | 06:01 | |
| *** kzaitsev_mb has quit IRC | 06:02 | |
| jianghuaw | @gmann and @masayukig: Also could you help to check this fix: https://review.openstack.org/#/c/328836 | 06:02 |
| jianghuaw | thanks very much. It has broken our CI for three days. We hope that can be fixed ASAP. | 06:03 |
| *** markvoelker has quit IRC | 06:04 | |
| *** artom has quit IRC | 06:04 | |
| *** artom has joined #openstack-qa | 06:05 | |
| *** gcheresh_ has joined #openstack-qa | 06:10 | |
| *** rcernin has joined #openstack-qa | 06:14 | |
| *** sabeen has quit IRC | 06:18 | |
| *** rcernin has quit IRC | 06:18 | |
| *** rcernin has joined #openstack-qa | 06:19 | |
| *** zeih has joined #openstack-qa | 06:22 | |
| *** nadya has quit IRC | 06:22 | |
| masayukig | jianghuaw: thanks for your patch! lgtm Hypervisor/hosts is complicated/puzzling, though.. | 06:23 |
| *** yolanda has joined #openstack-qa | 06:24 | |
| *** zhufl has quit IRC | 06:24 | |
| jianghuaw | masayukig: Thanks very much. | 06:25 |
| *** zeih has quit IRC | 06:27 | |
| openstackrecheck | Console logs not available after 13:24s for gate-grenade-dsvm-neutron-multinode 328832,1,fbaf1fd | 06:27 |
| jianghuaw | masayukig: BTW, could we make this XenServer CI to vote on tempest test. We see this CI ran failed on the previous patch but it was ignored due to not voting. | 06:27 |
| *** zeih has joined #openstack-qa | 06:30 | |
| openstackgerrit | Benny Kopilov proposed openstack/tempest: Fix nova admin naming https://review.openstack.org/328832 | 06:31 |
| masayukig | jianghuaw: I think it depends on the stability of your XenSerrver CI. We really need to recheck as you did before. We don't want to add more rechecks. | 06:31 |
| *** gcheresh_ has quit IRC | 06:33 | |
| *** yfried has joined #openstack-qa | 06:35 | |
| *** zhufl has joined #openstack-qa | 06:39 | |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Cinder volume type access tests https://review.openstack.org/319798 | 06:39 |
| *** sridharg has joined #openstack-qa | 06:39 | |
| *** edannon has joined #openstack-qa | 06:41 | |
| jianghuaw | masayukig: I think XenServer CI is stable now. Please check ci-watch.tintri.com, it's a even more stable than jenkins (when not broken by a bad merge). | 06:47 |
| jianghuaw | We also have a exclusion list which we can isolate some unstable cases very easily. So I don't think we need much rechecks. | 06:48 |
| *** jpena|off is now known as jpena | 06:51 | |
| *** k4n0 has joined #openstack-qa | 06:52 | |
| *** e0ne has joined #openstack-qa | 06:53 | |
| *** EinstCrazy has quit IRC | 06:53 | |
| *** anshul has joined #openstack-qa | 06:54 | |
| *** anshul is now known as Guest13720 | 06:54 | |
| masayukig | jianghuaw: How about bringing it to ML if you want to do so? | 06:55 |
| *** EinstCrazy has joined #openstack-qa | 06:56 | |
| *** amotoki has joined #openstack-qa | 07:00 | |
| *** tesseract has joined #openstack-qa | 07:00 | |
| jianghuaw | masayukig: sure. thanks. | 07:01 |
| *** agireud has quit IRC | 07:04 | |
| *** salv-orl_ has quit IRC | 07:07 | |
| *** agireud has joined #openstack-qa | 07:07 | |
| *** salv-orlando has joined #openstack-qa | 07:08 | |
| *** yolanda has quit IRC | 07:15 | |
| *** nadya has joined #openstack-qa | 07:16 | |
| *** kbaikov has joined #openstack-qa | 07:21 | |
| *** yolanda has joined #openstack-qa | 07:22 | |
| openstackrecheck | Console logs not available after 13:29s for gate-grenade-dsvm-multinode 322943,2,0cbb654 | 07:22 |
| *** yangyapeng has quit IRC | 07:22 | |
| *** ihrachys has joined #openstack-qa | 07:23 | |
| *** Poornima has quit IRC | 07:23 | |
| *** grafuls has joined #openstack-qa | 07:24 | |
| *** yangyapeng has joined #openstack-qa | 07:26 | |
| *** jlanoux has joined #openstack-qa | 07:30 | |
| *** Alex_Stef has quit IRC | 07:30 | |
| *** yolanda has quit IRC | 07:31 | |
| *** yolanda has joined #openstack-qa | 07:32 | |
| *** e0ne has quit IRC | 07:37 | |
| *** Poornima has joined #openstack-qa | 07:39 | |
| openstackrecheck | Console logs not available after 13:30s for gate-cinder-pep8 305633,6,3f285b6 | 07:40 |
| *** artom has quit IRC | 07:42 | |
| *** artom has joined #openstack-qa | 07:42 | |
| *** dtantsur|afk is now known as dtantsur | 07:43 | |
| *** nadya has quit IRC | 07:46 | |
| *** yolanda has quit IRC | 07:48 | |
| gmann | jianghuaw: ahh yea, i did not notice the CI failure in that patch, sorry about that. you are right about type 1 hypervisor_name. | 07:51 |
| gmann | jianghuaw: waiting for CI results then m +A | 07:51 |
| openstackrecheck | Console logs not available after 13:47s for gate-grenade-dsvm-neutron-multinode 328827,1,db292eb | 07:54 |
| *** e0ne has joined #openstack-qa | 07:55 | |
| *** kzaitsev_mb has joined #openstack-qa | 07:58 | |
| *** e0ne has quit IRC | 07:59 | |
| *** BobBall has joined #openstack-qa | 08:00 | |
| *** markvoelker has joined #openstack-qa | 08:00 | |
| *** ifarkas has joined #openstack-qa | 08:03 | |
| *** kzaitsev_mb has quit IRC | 08:03 | |
| *** markvoelker has quit IRC | 08:05 | |
| jianghuaw | gmann: Thanks. | 08:06 |
| openstackrecheck | Console logs not available after 13:24s for gate-grenade-dsvm-multinode 328131,3,f95546f | 08:08 |
| openstackgerrit | Bob Ball proposed openstack-dev/devstack: Fix brick cinderclient override https://review.openstack.org/328892 | 08:10 |
| BobBall | We have several internal process that parse stackrc. I'm trying to reduce these dependencies, but in the short term could someone look at approving the above fix? It's blocking some of our internal build processes so prompt reviews would be hugely appreciated :) | 08:11 |
| *** shardy has joined #openstack-qa | 08:18 | |
| *** shardy has quit IRC | 08:21 | |
| *** salv-orlando has quit IRC | 08:23 | |
| *** salv-orlando has joined #openstack-qa | 08:24 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-postgres-full 319798,8,c06ff7a | 08:24 |
| *** yangyape_ has joined #openstack-qa | 08:24 | |
| *** yangyapeng has quit IRC | 08:24 | |
| *** yolanda has joined #openstack-qa | 08:29 | |
| *** gvrangan has quit IRC | 08:30 | |
| *** yfried_ has joined #openstack-qa | 08:34 | |
| *** Serlex has joined #openstack-qa | 08:34 | |
| *** yfried has quit IRC | 08:34 | |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Cinder volume type access tests https://review.openstack.org/319798 | 08:34 |
| *** apevec has joined #openstack-qa | 08:34 | |
| *** apevec has quit IRC | 08:34 | |
| *** apevec has joined #openstack-qa | 08:34 | |
| *** gcheresh has joined #openstack-qa | 08:36 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 08:41 |
| *** apevec has left #openstack-qa | 08:44 | |
| jianghuaw | gmann: Thanks very much for the +A. | 08:47 |
| *** yfried has joined #openstack-qa | 08:51 | |
| *** Alex_Stef has joined #openstack-qa | 08:52 | |
| *** yfried_ has quit IRC | 08:55 | |
| zhufl | hello team, would you please do a review at https://review.openstack.org/#/c/294425/ ? Thanks:) | 08:56 |
| *** kzaitsev_mb has joined #openstack-qa | 08:59 | |
| *** artom has quit IRC | 08:59 | |
| *** cdent has joined #openstack-qa | 08:59 | |
| *** artom has joined #openstack-qa | 09:01 | |
| *** artom has quit IRC | 09:01 | |
| *** artom has joined #openstack-qa | 09:02 | |
| *** ishant|away is now known as ishant | 09:04 | |
| *** kzaitsev_mb has quit IRC | 09:04 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/tempest: Use is_valid_ipv4 from oslo.utils https://review.openstack.org/328914 | 09:05 |
| openstackgerrit | Benny Kopilov proposed openstack/tempest: Fix nova admin naming https://review.openstack.org/328832 | 09:14 |
| *** gszasz has joined #openstack-qa | 09:15 | |
| *** nadya has joined #openstack-qa | 09:17 | |
| openstackrecheck | Console logs not available after 13:26s for gate-grenade-dsvm-multinode 328827,1,18aff1a | 09:18 |
| *** drptbl has joined #openstack-qa | 09:20 | |
| *** e0ne has joined #openstack-qa | 09:22 | |
| *** mvk_ has quit IRC | 09:24 | |
| *** pcaruana has joined #openstack-qa | 09:27 | |
| *** amotoki has quit IRC | 09:31 | |
| openstackrecheck | Console logs not available after 13:27s for gate-cinder-python27-db 328887,1,b828325 | 09:31 |
| *** rossella_s has joined #openstack-qa | 09:34 | |
| *** Guest13720 has quit IRC | 09:38 | |
| *** tosky has joined #openstack-qa | 09:42 | |
| *** drptbl has quit IRC | 09:46 | |
| *** drptbl has joined #openstack-qa | 09:50 | |
| *** mvk_ has joined #openstack-qa | 09:51 | |
| andreaf | gmann, afazekas, masayukig, mkoderer, mtreinish, oomichi: what do you think about this? I'm trying to make the changes to client manager classes as painless as possible for plugins - but I'd like your input on it https://review.openstack.org/#/c/326683/ | 09:52 |
| *** Guest13720 has joined #openstack-qa | 09:53 | |
| openstackrecheck | Console logs not available after 13:26s for gate-grenade-dsvm-multinode 328836,1,e7e6d5f | 09:57 |
| *** artom has quit IRC | 09:58 | |
| *** drptbl has quit IRC | 09:58 | |
| *** artom has joined #openstack-qa | 09:59 | |
| *** kzaitsev_mb has joined #openstack-qa | 10:00 | |
| *** nadya has quit IRC | 10:02 | |
| *** kzaitsev_mb has quit IRC | 10:05 | |
| *** artom has quit IRC | 10:08 | |
| *** artom has joined #openstack-qa | 10:09 | |
| *** tfreger has joined #openstack-qa | 10:10 | |
| *** tfreger has quit IRC | 10:14 | |
| *** yamamoto_ has quit IRC | 10:19 | |
| *** amotoki has joined #openstack-qa | 10:19 | |
| *** armax has joined #openstack-qa | 10:28 | |
| openstackgerrit | Itzik Brown proposed openstack/tempest: Moving common.waiters to tempest.lib https://review.openstack.org/322492 | 10:30 |
| *** armax has quit IRC | 10:33 | |
| *** armax has joined #openstack-qa | 10:42 | |
| *** artom has quit IRC | 10:43 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 10:44 |
| openstackrecheck | Console logs not available after 13:22s for gate-cinder-pep8 305633,7,25ef09d | 10:46 |
| *** artom has joined #openstack-qa | 10:48 | |
| *** artom has quit IRC | 10:49 | |
| *** EinstCrazy has quit IRC | 10:49 | |
| *** artom has joined #openstack-qa | 10:49 | |
| *** pilgrimstack has joined #openstack-qa | 10:50 | |
| *** armax has quit IRC | 10:50 | |
| *** armax has joined #openstack-qa | 10:50 | |
| *** Guest13720 has quit IRC | 10:54 | |
| *** armax has quit IRC | 10:55 | |
| *** alyson_ has joined #openstack-qa | 10:56 | |
| *** kzaitsev_mb has joined #openstack-qa | 11:01 | |
| *** Guest13720 has joined #openstack-qa | 11:03 | |
| *** Guest13720 has quit IRC | 11:04 | |
| *** anshul has joined #openstack-qa | 11:05 | |
| *** anshul is now known as Guest59138 | 11:05 | |
| *** kzaitsev_mb has quit IRC | 11:06 | |
| *** alyson_ has left #openstack-qa | 11:06 | |
| *** salv-orlando has quit IRC | 11:07 | |
| *** armax has joined #openstack-qa | 11:08 | |
| *** salv-orlando has joined #openstack-qa | 11:08 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Factor up (most) CONF value in clients.Manager https://review.openstack.org/326667 | 11:18 |
| *** pilgrimstack has quit IRC | 11:20 | |
| openstackrecheck | Console logs not available after 13:27s for gate-tempest-dsvm-neutron-full 305079,19,bbb9d2a | 11:25 |
| *** salv-orl_ has joined #openstack-qa | 11:28 | |
| *** artom has quit IRC | 11:31 | |
| *** nadya has joined #openstack-qa | 11:31 | |
| *** salv-orlando has quit IRC | 11:32 | |
| *** artom has joined #openstack-qa | 11:32 | |
| *** yamamoto has joined #openstack-qa | 11:33 | |
| *** k4n0 has quit IRC | 11:33 | |
| *** bkopilov has quit IRC | 11:39 | |
| *** nadya has quit IRC | 11:40 | |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Backup and restore scenario https://review.openstack.org/328964 | 11:43 |
| *** yamamoto has quit IRC | 11:44 | |
| *** jpena is now known as jpena|lunch | 11:44 | |
| *** pilgrimstack has joined #openstack-qa | 11:47 | |
| *** amotoki has quit IRC | 11:49 | |
| *** sdague has joined #openstack-qa | 11:49 | |
| *** pilgrimstack has quit IRC | 11:50 | |
| *** fesp has joined #openstack-qa | 11:52 | |
| *** yamamoto has joined #openstack-qa | 11:53 | |
| openstackrecheck | Console logs not available after 13:47s for gate-grenade-dsvm-multinode 327694,2,e932c64 | 11:55 |
| *** weshay has joined #openstack-qa | 11:56 | |
| *** aysyd has joined #openstack-qa | 11:56 | |
| *** moravec1 has quit IRC | 11:59 | |
| *** nadya has joined #openstack-qa | 12:01 | |
| *** raildo-afk is now known as raildo | 12:01 | |
| *** artom has quit IRC | 12:01 | |
| *** markvoelker has joined #openstack-qa | 12:01 | |
| *** moravec has joined #openstack-qa | 12:02 | |
| *** mvk_ has quit IRC | 12:03 | |
| *** moravec has quit IRC | 12:05 | |
| *** yangyape_ has quit IRC | 12:05 | |
| *** markvoelker has quit IRC | 12:06 | |
| *** pauloewerton has joined #openstack-qa | 12:07 | |
| *** fesp has quit IRC | 12:08 | |
| openstackrecheck | Console logs not available after 13:27s for gate-grenade-dsvm-multinode 326970,8,9d21187 | 12:08 |
| *** pilgrimstack has joined #openstack-qa | 12:09 | |
| *** flaper87 has quit IRC | 12:09 | |
| *** mtanino has joined #openstack-qa | 12:11 | |
| *** yamamoto has quit IRC | 12:13 | |
| *** flaper87 has joined #openstack-qa | 12:15 | |
| *** flaper87 has quit IRC | 12:15 | |
| *** flaper87 has joined #openstack-qa | 12:15 | |
| *** e0ne has quit IRC | 12:15 | |
| *** e0ne has joined #openstack-qa | 12:15 | |
| *** sdague has quit IRC | 12:16 | |
| *** sdague has joined #openstack-qa | 12:16 | |
| *** amotoki has joined #openstack-qa | 12:16 | |
| *** Guest59138 has quit IRC | 12:20 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-full 328931,1,f6399b2 | 12:22 |
| *** dkranz has joined #openstack-qa | 12:23 | |
| *** markvoelker has joined #openstack-qa | 12:25 | |
| *** ramishra has quit IRC | 12:27 | |
| *** ramishra has joined #openstack-qa | 12:29 | |
| *** pgadiya has quit IRC | 12:30 | |
| *** scottda has joined #openstack-qa | 12:32 | |
| *** pilgrimstack has quit IRC | 12:32 | |
| *** Guest59138 has joined #openstack-qa | 12:32 | |
| *** hoangcx has joined #openstack-qa | 12:34 | |
| *** pilgrimstack has joined #openstack-qa | 12:34 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-full 326667,12,056f574 | 12:35 |
| *** amotoki_ has joined #openstack-qa | 12:38 | |
| *** kashyap has left #openstack-qa | 12:39 | |
| *** tochi has quit IRC | 12:39 | |
| *** amotoki has quit IRC | 12:41 | |
| *** yfried has quit IRC | 12:43 | |
| *** jpena|lunch is now known as jpena | 12:44 | |
| openstackgerrit | Itzik Brown proposed openstack/tempest: Moving common.waiters to tempest.lib https://review.openstack.org/322492 | 12:45 |
| *** yamamoto has joined #openstack-qa | 12:46 | |
| *** yamamoto has quit IRC | 12:47 | |
| *** amotoki_ has quit IRC | 12:48 | |
| *** e0ne has quit IRC | 12:48 | |
| openstackrecheck | Console logs not available after 13:25s for gate-grenade-dsvm-neutron-multinode 326683,11,67684ca | 12:48 |
| *** yolanda has quit IRC | 12:49 | |
| *** mvk has joined #openstack-qa | 12:49 | |
| *** e0ne has joined #openstack-qa | 12:49 | |
| *** EinstCrazy has joined #openstack-qa | 12:51 | |
| *** hoangcx has quit IRC | 12:51 | |
| *** yolanda has joined #openstack-qa | 12:54 | |
| *** EinstCrazy has quit IRC | 12:55 | |
| *** pilgrimstack has quit IRC | 12:56 | |
| *** yamamoto has joined #openstack-qa | 12:56 | |
| *** yamamoto has quit IRC | 12:57 | |
| *** dkranz has quit IRC | 13:00 | |
| *** armax has quit IRC | 13:00 | |
| *** armax_ has joined #openstack-qa | 13:00 | |
| *** dpaterson has joined #openstack-qa | 13:00 | |
| *** Alex_Stef has quit IRC | 13:01 | |
| openstackrecheck | Console logs not available after 13:25s for gate-tempest-dsvm-full-devstack-plugin-ceph 322492,14,e67598c | 13:02 |
| *** kzaitsev_mb has joined #openstack-qa | 13:02 | |
| *** dave-mccowan has joined #openstack-qa | 13:03 | |
| *** armax has joined #openstack-qa | 13:04 | |
| *** armax_ has quit IRC | 13:05 | |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Cinder get-capabilities tests https://review.openstack.org/323737 | 13:08 |
| *** zeih has quit IRC | 13:09 | |
| *** armax_ has joined #openstack-qa | 13:10 | |
| *** zeih has joined #openstack-qa | 13:10 | |
| *** armax has quit IRC | 13:11 | |
| *** armax_ is now known as armax | 13:11 | |
| *** yamamoto has joined #openstack-qa | 13:13 | |
| *** moravec has joined #openstack-qa | 13:14 | |
| *** armax has quit IRC | 13:14 | |
| *** zeih has quit IRC | 13:15 | |
| *** edmondsw has joined #openstack-qa | 13:15 | |
| *** gouthamr has joined #openstack-qa | 13:15 | |
| openstackrecheck | Console logs not available after 13:24s for gate-grenade-dsvm-multinode 327012,3,f08bd76 | 13:15 |
| *** Alex_Stef has joined #openstack-qa | 13:16 | |
| *** zeih has joined #openstack-qa | 13:16 | |
| *** yamamoto has quit IRC | 13:16 | |
| *** dustins has joined #openstack-qa | 13:19 | |
| openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Fixing test_add_list_remove_router_on_l3_agent when >= two L3 agents https://review.openstack.org/322118 | 13:20 |
| *** zeih has quit IRC | 13:21 | |
| *** sshnaidm has quit IRC | 13:21 | |
| *** sshnaidm has joined #openstack-qa | 13:21 | |
| *** sshnaidm has quit IRC | 13:21 | |
| *** sshnaidm has joined #openstack-qa | 13:22 | |
| openstackgerrit | Liron Kuchlani proposed openstack/tempest: Backup and restore scenario https://review.openstack.org/328964 | 13:23 |
| *** moravec1 has joined #openstack-qa | 13:26 | |
| *** moravec has quit IRC | 13:27 | |
| *** yamamoto has joined #openstack-qa | 13:28 | |
| openstackrecheck | Console logs not available after 13:26s for gate-nova-python34-db 328966,1,86406be | 13:29 |
| *** bkopilov has joined #openstack-qa | 13:29 | |
| *** mriedem has joined #openstack-qa | 13:31 | |
| *** yamamoto has quit IRC | 13:31 | |
| *** yamamoto has joined #openstack-qa | 13:32 | |
| *** mvk has quit IRC | 13:32 | |
| *** zeih has joined #openstack-qa | 13:37 | |
| *** Alex_Stef has quit IRC | 13:38 | |
| openstackgerrit | Merged openstack-dev/devstack: Set volume service min and max microversions https://review.openstack.org/316144 | 13:41 |
| openstackrecheck | Console logs not available after 13:26s for gate-tempest-dsvm-full-liberty 328964,1,b5e6243 | 13:42 |
| *** rodrigods has quit IRC | 13:43 | |
| *** rodrigods has joined #openstack-qa | 13:43 | |
| *** tosky has quit IRC | 13:48 | |
| *** yamamoto has quit IRC | 13:48 | |
| *** dtantsur is now known as dtantsur|brb | 13:48 | |
| *** Alex_Stef has joined #openstack-qa | 13:51 | |
| *** tosky has joined #openstack-qa | 13:51 | |
| *** yamamoto has joined #openstack-qa | 13:54 | |
| *** Poornima has quit IRC | 13:54 | |
| *** aernhart has quit IRC | 13:56 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Factor up (most) CONF value in clients.Manager https://review.openstack.org/326667 | 13:56 |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Prepare the Manager class for tempest.lib https://review.openstack.org/326683 | 13:56 |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-pep8 293104,56,ceebffa | 13:56 |
| *** mvk has joined #openstack-qa | 13:56 | |
| *** Liuqing has joined #openstack-qa | 13:58 | |
| *** mriedem1 has joined #openstack-qa | 13:59 | |
| *** rbak has joined #openstack-qa | 14:01 | |
| *** mriedem has quit IRC | 14:02 | |
| *** jecarey has joined #openstack-qa | 14:02 | |
| *** woodster_ has joined #openstack-qa | 14:02 | |
| *** e0ne has quit IRC | 14:04 | |
| *** e0ne has joined #openstack-qa | 14:05 | |
| *** mriedem1 is now known as mriedem | 14:06 | |
| *** pglass has joined #openstack-qa | 14:08 | |
| *** ishant is now known as ishant|away | 14:08 | |
| *** krtaylor has quit IRC | 14:09 | |
| *** lutostag has joined #openstack-qa | 14:09 | |
| openstackrecheck | Console logs not available after 13:43s for gate-grenade-dsvm-multinode 322492,15,a773fd5 | 14:10 |
| *** zz_dimtruck is now known as dimtruck | 14:14 | |
| *** dimtruck is now known as zz_dimtruck | 14:15 | |
| *** ajmiller has joined #openstack-qa | 14:15 | |
| *** kzaitsev_mb has quit IRC | 14:20 | |
| *** sridharg has quit IRC | 14:21 | |
| *** nadya has quit IRC | 14:23 | |
| *** yolanda has quit IRC | 14:23 | |
| *** pcrews_ has quit IRC | 14:23 | |
| openstackrecheck | Console logs not available after 13:28s for gate-grenade-dsvm-neutron-multinode 315190,3,c03a767 | 14:23 |
| *** yamamoto has quit IRC | 14:24 | |
| *** pcrews has joined #openstack-qa | 14:25 | |
| *** yamamoto has joined #openstack-qa | 14:25 | |
| *** yolanda has joined #openstack-qa | 14:27 | |
| andreaf | mtreinish, masayukig: it looks like one day is missing here: http://status.openstack.org/openstack-health/#/job/periodic-tempest-dsvm-neutron-full-test-accounts-master | 14:27 |
| *** kzaitsev_mb has joined #openstack-qa | 14:27 | |
| *** yamamoto has quit IRC | 14:28 | |
| *** zeih has quit IRC | 14:30 | |
| *** Gen has joined #openstack-qa | 14:31 | |
| *** yamamoto has joined #openstack-qa | 14:32 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:32 | |
| *** yamamoto has quit IRC | 14:35 | |
| *** dkranz has joined #openstack-qa | 14:36 | |
| *** cody-somerville has joined #openstack-qa | 14:37 | |
| *** zz_dimtruck is now known as dimtruck | 14:37 | |
| *** EinstCrazy has joined #openstack-qa | 14:39 | |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/328713 failed gate-grenade-dsvm-multinode in the gate queue with an unrecognized error | 14:39 |
| *** corvus is now known as jeblair | 14:39 | |
| *** fnaval has quit IRC | 14:40 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Cleanup exceptions in tempest https://review.openstack.org/329032 | 14:41 |
| *** sabeen has joined #openstack-qa | 14:41 | |
| *** mfisher_ora has joined #openstack-qa | 14:44 | |
| mtreinish | andreaf: if a day is missing that means either collection stopped working or there was a failure before devstack | 14:46 |
| mtreinish | so there isn't anything in the database | 14:46 |
| andreaf | mtreinish: right I didn't really check tbh - I just thought I'd let you know - when I checked yesterday O-H was telling me that data was 1 day old, and it may be that collection stopped, but I can't remember on which page it was telling me that | 14:47 |
| *** porrua has joined #openstack-qa | 14:48 | |
| mtreinish | andreaf: that check happens on all pages, (it just does SELECT run_at from runs order by run_at limit 1 and checks the date) | 14:48 |
| andreaf | mtreinish: I'm keeping an eye on test-account periodic jobs as I'd like to move the test accounts testing to gate when possible | 14:48 |
| *** nadya has joined #openstack-qa | 14:49 | |
| andreaf | mtreinish: so if I saw that it wasn't specific to my filter, it means that probably we lost a day of data | 14:49 |
| *** yamamoto has joined #openstack-qa | 14:49 | |
| *** jlanoux has quit IRC | 14:50 | |
| *** amotoki has joined #openstack-qa | 14:51 | |
| *** gvrangan has joined #openstack-qa | 14:51 | |
| *** gszasz has quit IRC | 14:52 | |
| *** yamamoto has quit IRC | 14:53 | |
| *** artom has joined #openstack-qa | 14:53 | |
| *** gszasz has joined #openstack-qa | 14:53 | |
| *** fnaval has joined #openstack-qa | 14:53 | |
| *** mriedem1 has joined #openstack-qa | 14:53 | |
| *** artom has quit IRC | 14:53 | |
| *** nadya has quit IRC | 14:54 | |
| *** artom has joined #openstack-qa | 14:54 | |
| mtreinish | andreaf: oh actually it checks on gate date in 24 hrs, so if nothing ran in the gate in 24hrs that would trigger it: https://github.com/openstack/openstack-health/blob/master/app/js/directives/fresh-check.js | 14:54 |
| mtreinish | it's been a few months since I added that, couldn't remember the details | 14:55 |
| *** kzaitsev_mb has quit IRC | 14:55 | |
| *** mriedem has quit IRC | 14:55 | |
| mtreinish | I'm not sure it was a collection issue, looking at the graph on the home page shows runs all the time | 14:55 |
| mtreinish | andreaf: we'd have to ask in -infra to see if there are any errors in the worker logs | 14:55 |
| *** yamamoto has joined #openstack-qa | 14:56 | |
| *** suyash has joined #openstack-qa | 14:57 | |
| *** Alex_Stef has quit IRC | 14:58 | |
| andreaf | mtreinish: interesting - there is a folder for Jun 12th, but most of the data there was created on Jun 5th http://logs.openstack.org/periodic/periodic-tempest-dsvm-neutron-full-test-accounts-master/b9fe4bf/ | 14:58 |
| *** anteaya has joined #openstack-qa | 14:58 | |
| mtreinish | andreaf: hmm, I guess something modified the directory on the 12th but the run was from the 5th | 14:59 |
| andreaf | mtreinish: meh, many folders have the Jun 12th date | 14:59 |
| *** gcheresh has quit IRC | 15:00 | |
| *** Liuqing has quit IRC | 15:00 | |
| andreaf | mtreinish: not sure how to find out which folder is the right one without access to Jenkins anymore | 15:00 |
| mtreinish | andreaf: found it: http://logs.openstack.org/periodic/periodic-tempest-dsvm-neutron-full-test-accounts-master/652707b/ | 15:02 |
| mtreinish | it ran on the zuul launcher stuff that jeblair is working on | 15:03 |
| mtreinish | so that explains why subunit2sql doesn't have it | 15:03 |
| mtreinish | because that collection path depends on the jenkins zmq plugin | 15:03 |
| jeblair | what's up? | 15:03 |
| jeblair | mtreinish: zuul launcher emits zmq events | 15:03 |
| mtreinish | jeblair: sure, but does the logstash gearman client listen on that? | 15:03 |
| jeblair | mtreinish: yep | 15:04 |
| *** yamamoto has quit IRC | 15:05 | |
| mtreinish | well there goes my guess for why http://logs.openstack.org/periodic/periodic-tempest-dsvm-neutron-full-test-accounts-master/652707b/ results aren't in subunit2sql | 15:05 |
| jeblair | mtreinish: zmq is notoriously bad at dealing with connection errors though | 15:05 |
| jeblair | like, it hides them | 15:05 |
| jeblair | so we should see if it needs to be kicked or something | 15:05 |
| *** edannon has quit IRC | 15:06 | |
| *** gvrangan has quit IRC | 15:06 | |
| *** grafuls has quit IRC | 15:06 | |
| *** yamamoto has joined #openstack-qa | 15:07 | |
| *** yamamoto has quit IRC | 15:10 | |
| *** david-lyle has joined #openstack-qa | 15:11 | |
| *** yamamoto has joined #openstack-qa | 15:12 | |
| *** itzikb has quit IRC | 15:13 | |
| *** ajmiller_ has joined #openstack-qa | 15:13 | |
| *** ajmiller has quit IRC | 15:16 | |
| *** yamamoto has quit IRC | 15:17 | |
| *** krtaylor has joined #openstack-qa | 15:17 | |
| *** vhosakot has joined #openstack-qa | 15:17 | |
| *** Swami has joined #openstack-qa | 15:17 | |
| *** Swami_ has joined #openstack-qa | 15:18 | |
| *** Swami_ has quit IRC | 15:18 | |
| *** Swami has quit IRC | 15:18 | |
| *** Swami has joined #openstack-qa | 15:18 | |
| *** jlanoux has joined #openstack-qa | 15:20 | |
| *** yangyapeng has joined #openstack-qa | 15:21 | |
| *** dtantsur|brb is now known as dtantsur | 15:21 | |
| *** artom has quit IRC | 15:21 | |
| *** artom has joined #openstack-qa | 15:22 | |
| *** mriedem1 is now known as mriedem | 15:22 | |
| *** edtubill has joined #openstack-qa | 15:22 | |
| openstackrecheck | Console logs not available after 13:41s for gate-nova-python27-db 327188,2,e7eba43 | 15:23 |
| *** kzaitsev_mb has joined #openstack-qa | 15:26 | |
| *** weshay is now known as weshay_lunch_brb | 15:26 | |
| *** mtanino has quit IRC | 15:28 | |
| *** agireud has quit IRC | 15:36 | |
| *** ddieterly has joined #openstack-qa | 15:37 | |
| *** agireud has joined #openstack-qa | 15:38 | |
| *** tesseract has quit IRC | 15:39 | |
| openstackgerrit | Andrea Frittoli proposed openstack/tempest: Handle service available parameters https://review.openstack.org/329063 | 15:40 |
| openstackrecheck | Console logs not available after 13:26s for gate-nova-tox-db-functional 293104,57,37f1fa2 | 15:41 |
| openstackrecheck | openstack/tempest change: https://review.openstack.org/327971 failed gate-grenade-dsvm-neutron-multinode in the gate queue with an unrecognized error | 15:41 |
| *** dkranz has quit IRC | 15:42 | |
| *** Guest59138 has quit IRC | 15:44 | |
| openstackgerrit | Ashley Handy proposed openstack/tempest: Add Test Coverage for Temp Url https://review.openstack.org/319456 | 15:44 |
| *** weshay_lunch_brb is now known as weshay | 15:45 | |
| *** vhosakot has quit IRC | 15:49 | |
| *** vhosakot has joined #openstack-qa | 15:49 | |
| *** nadya has joined #openstack-qa | 15:50 | |
| andreaf | mtreinish: not sure if you had a chance to look at https://review.openstack.org/#/q/topic:bp/client-manager-refactor+status:+open+is:mergeable+label:+Workflow%253D0 yet | 15:51 |
| andreaf | mtreinish: I'm trying to be nice to plugins, and keep an unstable interface stable for them | 15:51 |
| mtreinish | andreaf: I haven't had a chance yet, I'll put it on my list for today | 15:52 |
| andreaf | mtreinish: and I'm defining an interface that takes a dict as in input, which kind of replaces all the service specific config settings, to make the manager object independent from config, and not having 200 parameters | 15:52 |
| *** Apoorva has joined #openstack-qa | 15:52 | |
| andreaf | mtreinish: thanks, I'd like your input on that approach | 15:52 |
| *** grafuls has joined #openstack-qa | 15:52 | |
| openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 15:54 |
| *** Swami has quit IRC | 15:55 | |
| *** nadya has quit IRC | 15:55 | |
| *** Swami has joined #openstack-qa | 15:56 | |
| *** rcernin has quit IRC | 15:59 | |
| openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 16:03 |
| *** EinstCrazy has quit IRC | 16:04 | |
| openstackgerrit | Hardik Italia proposed openstack/tempest: DNM: ssh into active instance before and after live migration https://review.openstack.org/286855 | 16:07 |
| *** gszasz has quit IRC | 16:09 | |
| openstackrecheck | Console logs not available after 13:27s for gate-grenade-dsvm-multinode 329032,1,aec0e62 | 16:09 |
| openstackgerrit | Scott DAngelo proposed openstack/tempest: Add tempest tests for volume retype with migration https://review.openstack.org/195443 | 16:10 |
| *** suyash has quit IRC | 16:11 | |
| *** mtanino has joined #openstack-qa | 16:13 | |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for os-brick privsep sudo bug 1592043 https://review.openstack.org/329084 | 16:16 |
| openstack | bug 1592043 in os-brick "os-brick 1.4.0 increases volume setup failure rates" [Undecided,Confirmed] https://launchpad.net/bugs/1592043 | 16:16 |
| mriedem | sdague: ^ | 16:16 |
| *** Swami has quit IRC | 16:17 | |
| *** Gen has quit IRC | 16:18 | |
| *** fnaval_ has joined #openstack-qa | 16:21 | |
| openstackgerrit | Hardik Italia proposed openstack/tempest: Added negative test for DVR routers https://review.openstack.org/183726 | 16:23 |
| *** fnaval has quit IRC | 16:24 | |
| mriedem | mtreinish: dims: any idea if we have many libraries running grenade on changes? | 16:25 |
| *** salv-orl_ has quit IRC | 16:25 | |
| *** salv-orlando has joined #openstack-qa | 16:25 | |
| mtreinish | mriedem: I wouldn't expect any to really be running grenade | 16:26 |
| *** ddieterly is now known as ddieterly[away] | 16:27 | |
| *** suyash has joined #openstack-qa | 16:29 | |
| *** e0ne has quit IRC | 16:29 | |
| *** rcernin has joined #openstack-qa | 16:30 | |
| *** fnaval_ has quit IRC | 16:31 | |
| *** ddieterly[away] is now known as ddieterly | 16:32 | |
| *** fnaval has joined #openstack-qa | 16:32 | |
| *** dwalleck has joined #openstack-qa | 16:33 | |
| scottda | mtreinish: You had commented in https://review.openstack.org/#/c/195443 that we need a CI to run this (and I assume every Tempest) patch. Do I need to setup my own 3rd party CI for that? | 16:34 |
| *** Swami has joined #openstack-qa | 16:34 | |
| scottda | mriedem: You had mentioned a community CI, but I cannot seem to find any info on where that exists. | 16:34 |
| *** rcernin has quit IRC | 16:35 | |
| scottda | oops. sorry mriedem . ^^^ that should be mtreinish | 16:35 |
| *** rcernin has joined #openstack-qa | 16:36 | |
| mtreinish | scottda: it just has to be a CI system reporting on tempest, third party is fine (although no one has submitted a test case that only ran in 3rd party ci before) | 16:36 |
| mtreinish | scottda: tempest is mostly self verifying, especially the test code, so we need to see it actually running before we merge it | 16:36 |
| scottda | mtreinish: sure, that makes sense. But is there a community resource to test tempest patches? | 16:37 |
| scottda | I cannot find where such a CI exists, or how to get a patch to run on it. | 16:37 |
| mtreinish | scottda: it's the same ci as that would run on cinder or any other openstack project | 16:38 |
| openstackrecheck | Console logs not available after 13:24s for gate-keystone-dsvm-functional 303502,15,1904c01 | 16:38 |
| *** pcaruana has quit IRC | 16:38 | |
| mtreinish | there's nothing particularly special about tempest in that regard, you just have to listen for tempest changes and have results reporting back | 16:38 |
| scottda | mtreinish: OK. But it sounds like *someone* must have a CI for this setup. Is there any communty resource? Or must I find a Cinder storage vendor who is willing to test this on their CI? | 16:39 |
| mtreinish | like I think there is an emc ci that occasionally is reporting on tempest changes (it doesn't seem to run too reliably though) | 16:39 |
| mtreinish | scottda: well the community resource is all of us. Last time we talked about that patch I was told it didn't work on lvm, which is the cinder default backend (and what we use by default in devstack) | 16:40 |
| scottda | mtreinish: OK, I guess that makes sense. I'll look for a sponsor to test this patch. But it still leaves me wondering what folks do to validate tempest patches if they've no 3rd party CI to work with.... | 16:40 |
| mtreinish | which is a bigger issue, but I'll table that discussion for now | 16:40 |
| scottda | mtreinish: I need a devstack-gate patch as a dependency for that patch, but it works fine with LVM and devstack. | 16:40 |
| mtreinish | we also run ceph in the gate if that api works on ceph you could look at enabling it on the ceph jobs | 16:40 |
| *** grafuls has quit IRC | 16:43 | |
| *** gvrangan has joined #openstack-qa | 16:44 | |
| *** Alex_Stef has joined #openstack-qa | 16:46 | |
| mtreinish | scottda: ok, cool yeah if it works fine with lvm, lets get it working in the gate and when the test starts running just ping me again and I'll drop the -2 | 16:50 |
| *** tosky has quit IRC | 16:51 | |
| *** clu_ has joined #openstack-qa | 16:52 | |
| *** early has quit IRC | 16:54 | |
| *** nadya has joined #openstack-qa | 16:57 | |
| *** dkranz has joined #openstack-qa | 17:01 | |
| *** rossella_s has quit IRC | 17:01 | |
| *** nadya has quit IRC | 17:02 | |
| *** artom_ has joined #openstack-qa | 17:03 | |
| *** jlanoux has quit IRC | 17:03 | |
| *** Serlex has quit IRC | 17:04 | |
| *** artom has quit IRC | 17:05 | |
| *** ihrachys has quit IRC | 17:06 | |
| *** ifarkas has quit IRC | 17:09 | |
| *** yamahata has joined #openstack-qa | 17:15 | |
| *** ddieterly is now known as ddieterly[away] | 17:20 | |
| *** Gen has joined #openstack-qa | 17:23 | |
| *** artom_ has quit IRC | 17:24 | |
| *** artom has joined #openstack-qa | 17:25 | |
| openstackrecheck | Console logs not available after 13:35s for gate-tempest-dsvm-neutron-full-liberty 329063,1,fd7374f | 17:27 |
| *** yamahata has quit IRC | 17:28 | |
| *** salv-orl_ has joined #openstack-qa | 17:29 | |
| *** salv-orl_ has quit IRC | 17:32 | |
| *** salv-orlando has quit IRC | 17:32 | |
| *** salv-orlando has joined #openstack-qa | 17:32 | |
| jroll | couple e-r reviews if someone has a moment: https://review.openstack.org/#/c/320994/ unbreaks CI and https://review.openstack.org/#/c/326739/ adds a query for a bug affecting ironic | 17:35 |
| *** vhosakot has quit IRC | 17:36 | |
| openstackgerrit | Jim Rollenhagen proposed openstack-infra/elastic-recheck: Add e-r query for 1590139 https://review.openstack.org/326739 | 17:37 |
| * jroll rebases the latter on the former | 17:37 | |
| *** pcaruana has joined #openstack-qa | 17:39 | |
| openstackrecheck | Console logs not available after 13:24s for gate-tempest-dsvm-neutron-dvr 326683,12,0511a75 | 17:41 |
| *** vhosakot has joined #openstack-qa | 17:41 | |
| *** sambetts is now known as sambetts|afk | 17:42 | |
| mtreinish | jroll: reviewed both | 17:43 |
| jroll | thanks | 17:44 |
| *** fnaval has quit IRC | 17:45 | |
| *** fnaval has joined #openstack-qa | 17:45 | |
| *** mtanino has quit IRC | 17:47 | |
| *** erlon has joined #openstack-qa | 17:49 | |
| *** early has joined #openstack-qa | 17:52 | |
| *** jpena is now known as jpena|off | 17:53 | |
| openstackrecheck | Console logs not available after 13:34s for gate-tempest-dsvm-postgres-full 319456,6,d37b457 | 17:54 |
| *** yamahata has joined #openstack-qa | 17:54 | |
| *** dpaterson has quit IRC | 17:57 | |
| *** csomerville has joined #openstack-qa | 18:01 | |
| *** yfried has joined #openstack-qa | 18:02 | |
| *** Apoorva_ has joined #openstack-qa | 18:04 | |
| *** cody-somerville has quit IRC | 18:04 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack-infra/devstack-gate: Removing 'horizon' from Ironic feature matrix https://review.openstack.org/329132 | 18:05 |
| *** mtanino has joined #openstack-qa | 18:05 | |
| *** artom has quit IRC | 18:06 | |
| *** Apoorva has quit IRC | 18:07 | |
| *** artom has joined #openstack-qa | 18:07 | |
| openstackrecheck | Console logs not available after 13:33s for gate-nova-tox-db-functional 323250,4,6d84c8b | 18:08 |
| *** dimtruck is now known as zz_dimtruck | 18:08 | |
| openstackgerrit | Rob Crittenden proposed openstack-dev/devstack: Make wait_for_service more robust by checking HTTP response https://review.openstack.org/321249 | 18:10 |
| *** dtantsur is now known as dtantsur|afk | 18:11 | |
| *** jecarey_ has joined #openstack-qa | 18:15 | |
| *** jecarey has quit IRC | 18:18 | |
| *** aernhart has joined #openstack-qa | 18:19 | |
| *** pvaneck has joined #openstack-qa | 18:20 | |
| *** ddieterly[away] has quit IRC | 18:20 | |
| *** aernhart has quit IRC | 18:20 | |
| *** aernhart has joined #openstack-qa | 18:21 | |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-python27-db 269672,21,6a5f23a | 18:21 |
| openstackgerrit | Merged openstack/tempest: Use is_valid_ipv4 from oslo.utils https://review.openstack.org/327543 | 18:23 |
| *** csomerville has quit IRC | 18:25 | |
| *** csomerville has joined #openstack-qa | 18:25 | |
| openstackgerrit | Mark T. Voelker proposed openstack/tempest: Add simple test for Neutron GET / https://review.openstack.org/311747 | 18:31 |
| *** csomerville has quit IRC | 18:32 | |
| openstackrecheck | Console logs not available after 13:25s for gate-requirements-docs 329069,1,24a462b | 18:35 |
| *** tqtran has joined #openstack-qa | 18:35 | |
| *** zz_dimtruck is now known as dimtruck | 18:38 | |
| openstackgerrit | Merged openstack/openstack-health: Remove failover logic outside of flask context https://review.openstack.org/324958 | 18:39 |
| *** gvrangan has quit IRC | 18:42 | |
| *** csomerville has joined #openstack-qa | 18:43 | |
| *** pkoniszewski has quit IRC | 18:45 | |
| *** jesusaur has quit IRC | 18:45 | |
| *** jesusaur has joined #openstack-qa | 18:46 | |
| *** pkoniszewski has joined #openstack-qa | 18:48 | |
| openstackrecheck | Console logs not available after 13:27s for gate-tempest-pep8 320495,19,e3f554d | 18:48 |
| *** aernhart has quit IRC | 18:50 | |
| *** aernhart has joined #openstack-qa | 18:50 | |
| *** mvk has quit IRC | 18:51 | |
| *** ddieterly has joined #openstack-qa | 18:54 | |
| *** nadya has joined #openstack-qa | 18:56 | |
| *** csomerville has quit IRC | 19:00 | |
| *** jianghuaw has quit IRC | 19:00 | |
| *** jecarey_ has quit IRC | 19:01 | |
| openstackrecheck | Console logs not available after 13:31s for gate-tempest-dsvm-neutron-full 328836,1,0ade81f | 19:02 |
| *** salv-orlando has quit IRC | 19:04 | |
| *** salv-orlando has joined #openstack-qa | 19:05 | |
| *** jecarey_ has joined #openstack-qa | 19:06 | |
| *** jecarey_ has quit IRC | 19:07 | |
| *** jecarey_ has joined #openstack-qa | 19:07 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Move get_image_meta_from_headers from images_client https://review.openstack.org/328509 | 19:08 |
| *** e0ne has joined #openstack-qa | 19:11 | |
| *** tqtran has quit IRC | 19:11 | |
| *** jianghuaw has joined #openstack-qa | 19:12 | |
| *** csomerville has joined #openstack-qa | 19:12 | |
| *** nadya has quit IRC | 19:14 | |
| oomichi | bkopilov: hi, can you check a comment on https://review.openstack.org/#/c/328832/ ? | 19:15 |
| oomichi | kbaikov: I put a new comment on that now | 19:15 |
| openstackrecheck | Console logs not available after 13:34s for gate-tempest-dsvm-layer4 286855,8,c17fff4 | 19:15 |
| *** gvrangan has joined #openstack-qa | 19:26 | |
| *** csomerville has quit IRC | 19:29 | |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-docs 327761,2,6899468 | 19:29 |
| *** salv-orlando has quit IRC | 19:31 | |
| *** salv-orlando has joined #openstack-qa | 19:31 | |
| *** St3F_A13x has joined #openstack-qa | 19:33 | |
| *** Alex_Stef has quit IRC | 19:34 | |
| *** ihrachys has joined #openstack-qa | 19:35 | |
| *** artom has quit IRC | 19:41 | |
| *** gvrangan has quit IRC | 19:42 | |
| *** artom has joined #openstack-qa | 19:42 | |
| openstackrecheck | Console logs not available after 13:24s for gate-nova-docs 319439,7,9bd1dd2 | 19:42 |
| *** e0ne has quit IRC | 19:48 | |
| *** St3F_A13x has quit IRC | 19:49 | |
| timothyb89 | oomichi: if you have a moment, could you look at https://review.openstack.org/327947 and https://review.openstack.org/327945 ? | 19:56 |
| openstackrecheck | Console logs not available after 13:35s for gate-tempest-dsvm-postgres-full 329122,2,d800e07 | 19:56 |
| *** cody-somerville has joined #openstack-qa | 20:00 | |
| *** St3F_A13x has joined #openstack-qa | 20:02 | |
| *** mvk has joined #openstack-qa | 20:03 | |
| *** cdent has quit IRC | 20:06 | |
| timothyb89 | mtreinish: looks like subunit collections is working on https://review.openstack.org/322449 :) | 20:08 |
| timothyb89 | mtreinish: stackviz mostly "just works" as well: https://stackviz.timothyb89.org/s/15fb7acf-2fc5-48ab-9293-fe83d82c4588/#/karma.subunit | 20:08 |
| mtreinish | timothyb89: cool, +2 now | 20:09 |
| timothyb89 | mtreinish: thanks! | 20:09 |
| mtreinish | heh, that's a pretty boring timeline though | 20:10 |
| timothyb89 | mtreinish: yeah, its hard to tell, but its actually somewhat broken | 20:10 |
| openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: Add query for os-brick privsep sudo bug 1592043 https://review.openstack.org/329084 | 20:10 |
| openstack | bug 1592043 in os-brick "os-brick 1.4.0 increases volume setup failure rates" [Undecided,Confirmed] https://launchpad.net/bugs/1592043 | 20:10 |
| openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: Fix e-r with fixtures 3.0.0 https://review.openstack.org/329170 | 20:10 |
| mriedem | mtreinish: for that auto-allocated network test, i'm not currently doing a network delete cleanup in there - do i need to? | 20:10 |
| openstackrecheck | Console logs not available after 14:06s for gate-swift-docs 328203,2,e13bf83 | 20:10 |
| sdague | mriedem: I had to fix ER unit tests | 20:10 |
| timothyb89 | mtreinish: since they're all async functions the rects are largely overlapping ... will need to be more intelligent about positioning | 20:10 |
| sdague | because fixtures | 20:10 |
| sdague | then your stuff should be able to land | 20:11 |
| mtreinish | mriedem: nova/neutron isn't cleaning up the created network on tenant delete is it? | 20:11 |
| mtreinish | mriedem: if not then we probably should add an addCleanup there so we don't end up leaving a network without a tenant | 20:11 |
| sdague | mriedem / mtreinish - https://review.openstack.org/#/c/329170/1 - quick review? | 20:11 |
| mriedem | mtreinish: nothing ever cleans anything up on tenant delete | 20:12 |
| mriedem | age old openstack bug | 20:12 |
| mriedem | gah, it's going to get weird | 20:12 |
| mtreinish | mriedem: heh, I know | 20:12 |
| mtreinish | mriedem: the other thing I was thinking about is how does this test work in the preprovisioned credentials case? | 20:13 |
| mtreinish | for dynamic creds it's easy, we get a fresh tenant and tell it don't make any networks | 20:13 |
| mriedem | i can't delete the network until the subnet(s) are deleted, and i can't delete those until the fixed ips are deleted and i can't delete those until the ports are deleted and those get deleted when the servers are deleted | 20:13 |
| mriedem | mtreinish: yeah that was my other question, who do i enforce this test to only run for tenant isolation? | 20:13 |
| mtreinish | but for preprov we can't really enforce the absence of a network easily | 20:13 |
| mriedem | s/who/how/ | 20:14 |
| mtreinish | mriedem: if you move it to admin you can set force_tenant_isolation (or something like that) as a class attr to force | 20:14 |
| mriedem | there used to be the force_tenant_isolation flag | 20:14 |
| openstackgerrit | Merged openstack/stackviz: Remove duplicated imports from test-requirements.txt https://review.openstack.org/328846 | 20:14 |
| mtreinish | mriedem: you can look at the quota tests for an example | 20:14 |
| mtreinish | mriedem: heh, welcome to the normal neutron teardown case | 20:14 |
| *** Alex_Stef has joined #openstack-qa | 20:14 | |
| mriedem | i don't suppose there is a helper for that | 20:15 |
| mtreinish | mriedem: https://github.com/openstack/tempest/blob/master/tempest/common/dynamic_creds.py#L361-L390 | 20:15 |
| mtreinish | but that can't really be used because you're not relying on dynamic creds to create the networks for you | 20:15 |
| mriedem | nope | 20:15 |
| mtreinish | there might be something in the neutron tests, but I doubt it's easily reusable given the state of a lot of those tests | 20:16 |
| *** St3F_A13x has quit IRC | 20:16 | |
| mriedem | looks like force_tenant_isolation doesn't need to be an admin test | 20:18 |
| mriedem | just specify it and it forces the dynamic creds provider | 20:18 |
| mriedem | in get_credentials_provider | 20:18 |
| mtreinish | mriedem: but tenant_isolation forces admin creds, you can't use that mechanism unless you have admin | 20:19 |
| mriedem | https://github.com/openstack/tempest/blob/master/tempest/common/credentials_factory.py#L75 | 20:19 |
| mtreinish | https://github.com/openstack/tempest/blob/master/tempest/common/dynamic_creds.py#L50 | 20:20 |
| mriedem | why does it require admin? | 20:20 |
| mtreinish | mriedem: you need admin to create a user and a tenant in keystone | 20:21 |
| mriedem | psh, only by default :) | 20:21 |
| mtreinish | heh, you're cloud gives that to everyone? | 20:22 |
| mriedem | hey, self service portal really means self service portal | 20:22 |
| mriedem | need some quota? help yourself. | 20:22 |
| openstackrecheck | Console logs not available after 13:24s for gate-os-brick-docs 328297,4,83c12fc | 20:24 |
| sdague | it's hippie cloud! | 20:24 |
| *** porrua has quit IRC | 20:25 | |
| *** gvrangan has joined #openstack-qa | 20:26 | |
| openstackgerrit | Patrick East proposed openstack-dev/devstack: Setup the Cinder image-volume cache by default https://review.openstack.org/312305 | 20:29 |
| *** asselin has joined #openstack-qa | 20:31 | |
| *** tqtran has joined #openstack-qa | 20:34 | |
| openstackgerrit | Scott DAngelo proposed openstack-infra/devstack-gate: Allow Cinder multi-backend tests https://review.openstack.org/329178 | 20:35 |
| openstackrecheck | Console logs not available after 13:26s for gate-requirements-tox-validate 329101,1,06bbc64 | 20:37 |
| *** jecarey_ has quit IRC | 20:43 | |
| *** mfisher_ora has quit IRC | 20:45 | |
| *** samueldmq has joined #openstack-qa | 20:46 | |
| *** ihrachys has quit IRC | 20:47 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Fix e-r with fixtures 3.0.0 https://review.openstack.org/329170 | 20:47 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for os-brick privsep sudo bug 1592043 https://review.openstack.org/329084 | 20:48 |
| openstack | bug 1592043 in os-brick "os-brick 1.4.0 increases volume setup failure rates" [Undecided,Confirmed] https://launchpad.net/bugs/1592043 | 20:48 |
| *** vgridnev has joined #openstack-qa | 20:48 | |
| *** edtubill has quit IRC | 20:48 | |
| *** ihrachys has joined #openstack-qa | 20:50 | |
| *** ddieterly is now known as ddieterly[away] | 20:50 | |
| openstackrecheck | Console logs not available after 13:27s for gate-grenade-dsvm-multinode 329077,2,5594d4e | 20:50 |
| *** vgridnev has quit IRC | 20:55 | |
| *** openstackrecheck has quit IRC | 20:56 | |
| *** openstackrecheck has joined #openstack-qa | 20:56 | |
| *** pauloewerton has quit IRC | 20:57 | |
| *** gouthamr has quit IRC | 20:59 | |
| *** hichihara has joined #openstack-qa | 21:01 | |
| oomichi | timothyb89: thanks, +1 | 21:01 |
| timothyb89 | oomichi: thank you! | 21:01 |
| *** raildo is now known as raildo-afk | 21:02 | |
| *** dkranz has quit IRC | 21:03 | |
| mriedem | mtreinish: neutronclient has a purge command :) | 21:04 |
| mtreinish | mriedem: heh, that doesn't help in tempest too much though. If you want to mirror that functionality in a helper function in a common place in tempest I'd totally be +2 on that | 21:05 |
| mriedem | it doesn't remove subnets, so i'm not sure how it removes networks | 21:06 |
| mriedem | my guess is it probably doens't work | 21:06 |
| *** moravec1 has quit IRC | 21:07 | |
| mriedem | kevinbenton: hey i have to add some network resource cleanup stuff to my auto-allocated-topology test, i'm trying to test that out manually in devstack. i ran auto-allocated-topology-show to create the resources and then created a server from the network. i then deleted the ports for the given device_id (server) that i created. i tried deleting the subnets though and that failed with a conflict. | 21:08 |
| mriedem | kevinbenton: what else needs to be deleted before the subnets? | 21:08 |
| mriedem | oh, disassociate the router and subnets first? | 21:09 |
| *** aysyd has quit IRC | 21:11 | |
| *** edtubill has joined #openstack-qa | 21:11 | |
| kevinbenton | mriedem: yep | 21:11 |
| kevinbenton | mriedem: detach the router from the subnet | 21:11 |
| kevinbenton | mriedem: then you can delete the network (it will auto remove subnet if empty) | 21:12 |
| *** ihrachys has quit IRC | 21:14 | |
| *** ihrachys has joined #openstack-qa | 21:14 | |
| mriedem | it actually looks like the auto_allocated_router is associated with the subnets on the 'public' network in devstack | 21:15 |
| mriedem | not the auto_allocated_network | 21:15 |
| openstackgerrit | Scott DAngelo proposed openstack-infra/devstack-gate: Allow Cinder multi-backend tests https://review.openstack.org/329178 | 21:16 |
| *** artom has quit IRC | 21:18 | |
| *** ddieterly[away] is now known as ddieterly | 21:18 | |
| openstackgerrit | Mark T. Voelker proposed openstack/tempest: Add simple test for Neutron GET / https://review.openstack.org/311747 | 21:19 |
| mriedem | nvm, got confused with external_gateway_info in the router | 21:21 |
| mriedem | so disassociate the auto-allocated-router from the auto-allocated-subnets, then delete the auto-allocated-router | 21:22 |
| *** dustins has quit IRC | 21:22 | |
| mriedem | then delete ports for the network i guess? then delete the subnets for the network, then the network | 21:23 |
| *** artom has joined #openstack-qa | 21:24 | |
| mriedem | ok, that order worked for me | 21:24 |
| mriedem | woot | 21:24 |
| *** vgridnev has joined #openstack-qa | 21:27 | |
| mriedem | mtreinish: i'm guessing any compute api tests that extend BaseV2ComputeAdminTest should live in tempest.api.compute.admin? i wasn't sure because of tempest.api.compute.test_live_block_migration_negative | 21:28 |
| mriedem | just wondering if i need to move this module when i make the test class extend BaseV2ComputeAdminTest | 21:28 |
| *** gouthamr has joined #openstack-qa | 21:28 | |
| mtreinish | mriedem: yeah, it should live in tempest.api.compute.admin, that makes it much easier for users to figure out where the divide is | 21:33 |
| *** edtubill has quit IRC | 21:33 | |
| mtreinish | that live_migrate test is a bug (it might predate the existence of an admin dir too, it's been around for a while) | 21:33 |
| mriedem | alright | 21:34 |
| mriedem | so besides needing to extend the admin test and setting force_tenant_isolation=True, i can still continue to use the non-admin tenant clients for everything else right? | 21:35 |
| mriedem | i don't actually need the admin user to test this api | 21:35 |
| *** cody-somerville has quit IRC | 21:35 | |
| mtreinish | mriedem: right, nothing else should change | 21:36 |
| mriedem | and the tenant user should own the ports/subnets/router/network so i shouldn't need admin clients to delete those network resources | 21:36 |
| mtreinish | yep | 21:36 |
| mriedem | ok, cool | 21:36 |
| *** cody-somerville has joined #openstack-qa | 21:36 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Make endpoints_client use **kwargs https://review.openstack.org/329196 | 21:38 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Define v2 endpoints_client as libarry https://review.openstack.org/329197 | 21:38 |
| *** edtubill has joined #openstack-qa | 21:40 | |
| kevinbenton | mriedem: sorry, didn't notice your replies. you don't even have to delete the subnet explicitly if you don't want | 21:42 |
| mriedem | kevinbenton: i'll just remove the subnets explicitly, don't want to leave anything to chance :) | 21:42 |
| kevinbenton | mriedem: ok | 21:42 |
| *** edtubill has quit IRC | 21:43 | |
| openstackrecheck | Console logs not available after 13:30s for gate-tempest-dsvm-full 328509,2,d3fb13c | 21:43 |
| *** csomerville has joined #openstack-qa | 21:43 | |
| *** cody-somerville has quit IRC | 21:44 | |
| *** rcernin has quit IRC | 21:48 | |
| *** edtubill has joined #openstack-qa | 21:51 | |
| *** hichihara has quit IRC | 21:57 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Move get_image_meta_from_headers from images_client https://review.openstack.org/328509 | 21:59 |
| *** ihrachys has quit IRC | 22:00 | |
| *** edtubill has quit IRC | 22:00 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Move get_image_meta_from_headers from images_client https://review.openstack.org/328509 | 22:04 |
| *** moravec has joined #openstack-qa | 22:05 | |
| *** ihrachys has joined #openstack-qa | 22:05 | |
| *** edmondsw has quit IRC | 22:07 | |
| openstackrecheck | Console logs not available after 13:26s for gate-nova-python34-db 329149,2,5145118 | 22:07 |
| openstackgerrit | Matt Riedemann proposed openstack/tempest: Add compute API tests for 'get-me-a-network' https://review.openstack.org/327191 | 22:08 |
| mriedem | mtreinish: hopefully that does the trick | 22:08 |
| mriedem | i wasn't sure if i should throw that network resource cleanup in a self.addCleanup stack, figured it doesn't matter since it's the last thing the test does | 22:10 |
| *** ddieterly has quit IRC | 22:10 | |
| *** Alex_Stef has quit IRC | 22:14 | |
| *** edtubill has joined #openstack-qa | 22:14 | |
| mtreinish | mriedem: it doesn't really matter from a functional standpoint. Normally if we put it in the test we're asserting that's part of what we're testing though | 22:14 |
| mtreinish | but I don't think that's really enforced | 22:14 |
| mriedem | the test would still fail if the teardown fails though right | 22:14 |
| mriedem | if someone has a style issue with it i can change it | 22:14 |
| mtreinish | mriedem: the real advantage with addCleanup though, is it ensure that it runs even if the test fails | 22:15 |
| mriedem | yeah i know | 22:15 |
| mtreinish | mriedem: tearDown will run even if the test fails | 22:15 |
| mriedem | but since it's the last thing | 22:15 |
| mriedem | ... | 22:15 |
| mriedem | see what i mean? | 22:15 |
| mtreinish | mriedem: if it's in the test method and something fails before it, then it doesn't run | 22:15 |
| mriedem | yeah i know | 22:16 |
| mriedem | but i also don't have it before then | 22:16 |
| mriedem | the network and router that is | 22:16 |
| mriedem | i guess i could move the network/router query/assertions into the cleanup method itself | 22:16 |
| mtreinish | mriedem: sure, so put an if on it to just be a no op if they're not defined | 22:16 |
| *** amotoki has quit IRC | 22:17 | |
| mriedem | an if on what? | 22:17 |
| *** ihrachys_ has joined #openstack-qa | 22:18 | |
| mtreinish | the delete function | 22:18 |
| mriedem | i guess i'm not really following | 22:18 |
| mriedem | if you mean, | 22:19 |
| mriedem | do addCleanup at the beginning of the test | 22:19 |
| mriedem | and in the cleanup method, look for the network and router to cleanup, noop if not found, | 22:19 |
| mriedem | then i can see that | 22:19 |
| mriedem | although it is nice to validate that the actual network we expected to be created was created | 22:19 |
| *** edtubill has quit IRC | 22:20 | |
| *** ihrachys has quit IRC | 22:20 | |
| *** edtubill has joined #openstack-qa | 22:20 | |
| mtreinish | mriedem: like this: http://paste.openstack.org/show/515843/ | 22:22 |
| mtreinish | or something like that | 22:22 |
| mtreinish | although I don't know if that really works | 22:23 |
| mriedem | it doesn't | 22:23 |
| *** edtubill has quit IRC | 22:23 | |
| mriedem | those would have to be class var | 22:23 |
| mriedem | if self.network | 22:23 |
| mtreinish | you could do that | 22:23 |
| mriedem | and then set self.network later | 22:23 |
| mriedem | could | 22:23 |
| mriedem | class vars kind of suck | 22:23 |
| mtreinish | they do | 22:23 |
| mtreinish | I'm just trying to come up with a way to make sure we don't leak on failure | 22:23 |
| *** edtubill has joined #openstack-qa | 22:24 | |
| *** ajmiller_ has quit IRC | 22:24 | |
| mriedem | really the only thing going on that can fail is in that create_test_server call | 22:24 |
| mriedem | besides the assertions i mean | 22:24 |
| mtreinish | mriedem: you could make the delete method an inner function, then avoid the class var too | 22:24 |
| *** ajmiller has joined #openstack-qa | 22:24 | |
| *** aernhart has quit IRC | 22:25 | |
| mtreinish | mriedem: it's the assertions that would be the issue | 22:25 |
| *** ihrachys_ has quit IRC | 22:25 | |
| *** edtubill has quit IRC | 22:25 | |
| mriedem | but it's also the assertions that give me the resources i'm looking for to delete | 22:25 |
| mtreinish | I think, I'd have to test it though | 22:25 |
| mriedem | i could also just build the assertions into the list_networks and list_routers calls, by filtering on name | 22:26 |
| mriedem | and asserting that i get one back | 22:26 |
| mriedem | i couldn't assert those in the cleanup method if it's called because creating the server failed, and there wasn't a network or router because we didn't get that far | 22:26 |
| *** ihrachys has joined #openstack-qa | 22:27 | |
| mriedem | idk | 22:27 |
| mriedem | i'm pretty sure my test as is is perfect and i don't need to change anything :) | 22:27 |
| mriedem | and i just need to go home | 22:27 |
| mtreinish | mriedem: heh, this is the trouble with testing things that happen in the background | 22:28 |
| mtreinish | mriedem: sure, I'll think on it tonight too | 22:28 |
| mriedem | alright, ttyl then - thanks for the help | 22:29 |
| *** mriedem is now known as mriedem_away | 22:29 | |
| *** edtubill has joined #openstack-qa | 22:30 | |
| *** aernhart has joined #openstack-qa | 22:31 | |
| *** Gen has quit IRC | 22:32 | |
| *** ihrachys has quit IRC | 22:33 | |
| *** akshai has joined #openstack-qa | 22:36 | |
| openstackgerrit | Scott DAngelo proposed openstack-dev/devstack: Add logic for volume.backend_names https://review.openstack.org/329210 | 22:36 |
| *** dimtruck is now known as zz_dimtruck | 22:37 | |
| *** ihrachys has joined #openstack-qa | 22:39 | |
| *** edtubill has quit IRC | 22:41 | |
| *** pglass has quit IRC | 22:44 | |
| *** jecarey has joined #openstack-qa | 22:45 | |
| *** akshai has quit IRC | 22:45 | |
| *** dave-mccowan has quit IRC | 22:47 | |
| *** ihrachys has quit IRC | 22:48 | |
| *** jecarey has quit IRC | 22:49 | |
| *** artom has quit IRC | 22:50 | |
| *** artom has joined #openstack-qa | 22:51 | |
| *** salv-orlando has quit IRC | 22:51 | |
| *** ihrachys has joined #openstack-qa | 22:52 | |
| *** dave-mccowan has joined #openstack-qa | 22:52 | |
| *** salv-orlando has joined #openstack-qa | 22:52 | |
| *** rbak has quit IRC | 23:01 | |
| *** ihrachys has quit IRC | 23:01 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Clarify "data" arg in create/update_image() https://review.openstack.org/329218 | 23:01 |
| *** ihrachys has joined #openstack-qa | 23:01 | |
| jianghuaw | hi, the jenkins(gate pipe) failed with the following two tests but if we check into the detail console log. We can see both tests finished successfully. Anyone know why? Thanks. | 23:03 |
| jianghuaw | https://review.openstack.org/#/c/328836/gate-tempest-dsvm-neutron-full FAILURE in 52m 12s | 23:03 |
| jianghuaw | https://review.openstack.org/#/c/328836/ | 23:03 |
| jianghuaw | gate-tempest-dsvm-neutron-full FAILURE in 52m 12s | 23:03 |
| jianghuaw | gate-tempest-pep8 FAILURE in 3m 46s | 23:03 |
| *** edtubill has joined #openstack-qa | 23:04 | |
| openstackrecheck | Console logs not available after 13:43s for gate-tempest-dsvm-full-devstack-plugin-ceph 329196,1,6b69694 | 23:04 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Make endpoints_client use **kwargs https://review.openstack.org/329196 | 23:05 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Define v2 endpoints_client as libarry https://review.openstack.org/329197 | 23:05 |
| *** artom has quit IRC | 23:07 | |
| *** edtubill has quit IRC | 23:09 | |
| *** edtubill has joined #openstack-qa | 23:10 | |
| *** yamahata has quit IRC | 23:11 | |
| *** akshai has joined #openstack-qa | 23:13 | |
| *** artom has joined #openstack-qa | 23:13 | |
| mtreinish | jeblair: ^^^ it looks jianghuaw hit some issues with the new zuul stuff not reporting correctly | 23:16 |
| jeblair | mtreinish, jianghuaw: yeah, we broke and fixed that this morning -- it should be good with a recheck now | 23:17 |
| *** ihrachys has quit IRC | 23:17 | |
| jianghuaw | mtreinish and jeblair: Thanks very much. I've given a recheck. | 23:18 |
| *** artom has quit IRC | 23:21 | |
| *** ihrachys has joined #openstack-qa | 23:22 | |
| *** akshai_ has joined #openstack-qa | 23:24 | |
| *** akshai has quit IRC | 23:24 | |
| *** salv-orl_ has joined #openstack-qa | 23:28 | |
| *** Swami has quit IRC | 23:31 | |
| *** salv-orlando has quit IRC | 23:31 | |
| *** edtubill has quit IRC | 23:32 | |
| *** ihrachys has quit IRC | 23:33 | |
| *** ihrachys has joined #openstack-qa | 23:33 | |
| *** gouthamr_ has joined #openstack-qa | 23:38 | |
| *** sdague has quit IRC | 23:38 | |
| *** tochi has joined #openstack-qa | 23:40 | |
| *** gouthamr has quit IRC | 23:42 | |
| *** akshai_ has quit IRC | 23:42 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Define v2 endpoints_client as libarry https://review.openstack.org/329197 | 23:46 |
| *** amotoki has joined #openstack-qa | 23:46 | |
| *** ihrachys has quit IRC | 23:47 | |
| *** ihrachys has joined #openstack-qa | 23:47 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove unnecessary setUp() https://review.openstack.org/329226 | 23:48 |
| *** amotoki has quit IRC | 23:50 | |
| *** yamahata has joined #openstack-qa | 23:51 | |
| openstackgerrit | Merged openstack-dev/devstack: Updated from generate-devstack-plugins-list https://review.openstack.org/328705 | 23:51 |
| *** weshay has quit IRC | 23:54 | |
| *** ihrachys has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!