Friday, 2014-12-05

*** bknudson has joined #openstack-qa00:01
*** tonyb has joined #openstack-qa00:01
*** rbak has quit IRC00:04
*** tosky has quit IRC00:06
*** marun has quit IRC00:07
*** Masahiro has joined #openstack-qa00:10
openstackgerritMerged openstack/tempest-lib: Switch to elapsed time in subunit-trace summary  https://review.openstack.org/12941800:11
*** shashankhegde has quit IRC00:14
*** Masahiro has quit IRC00:14
*** heyongli has joined #openstack-qa00:15
*** shashankhegde has joined #openstack-qa00:17
*** dimtruck is now known as zz_dimtruck00:17
openstackgerritTony Breeds proposed openstack-dev/devstack: Create service definition for nova-serialproxy  https://review.openstack.org/13895600:17
*** openstackgerrit has quit IRC00:18
anteayaheyongli: so did you get enough information from sdague and jogo to move onto your next step with where your tests should go?00:19
*** openstackgerrit has joined #openstack-qa00:19
openstackgerritTony Breeds proposed openstack-dev/devstack: Create service definition for nova-serialproxy  https://review.openstack.org/13895600:19
heyonglianteaya, not yet, still need more discuss and find a better way to do the testing.00:19
anteayaokay00:21
*** andreykurilin_ has quit IRC00:21
*** sabeen1 has quit IRC00:21
heyonglisdague, concern the encoded information about pci in the temptest is unproperly, it probably indicate the testing of pci is not only a API testing. it's a 'functional" test use existing API.  the scenerio, might be a choice?00:23
openstackgerritGhanshyam Mann proposed openstack-infra/devstack-gate: Add conf to replace nova V2 endpoint with V2.1 API  https://review.openstack.org/13894900:24
gmannsdague: oomichi: Thanks. update the d-g patch. https://review.openstack.org/#/c/138949/00:26
*** marun has joined #openstack-qa00:28
oomichigmann: lgtm, thanks!00:28
*** mlavalle has quit IRC00:35
*** mlavalle has joined #openstack-qa00:35
*** dmorita has joined #openstack-qa00:36
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: try running with -o nounset  https://review.openstack.org/13928700:38
openstackgerritMerged openstack-dev/devstack: Rework the old logo  https://review.openstack.org/13622000:38
openstackrecheckopenstack/keystone change: https://review.openstack.org/138355 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1362528 https://bugs.launchpad.net/bugs/132365800:39
*** ranger81 has quit IRC00:43
*** ranger81 has joined #openstack-qa00:43
*** mlavalle has quit IRC00:45
*** salv-orlando has quit IRC00:47
*** salv-orlando has joined #openstack-qa00:48
*** marun has quit IRC00:52
*** suneelb has quit IRC00:52
*** bkopilov has joined #openstack-qa00:53
oomichichmouel: hi00:55
*** boris-42 has quit IRC00:57
*** topshare has joined #openstack-qa00:59
*** dims_ has quit IRC01:00
*** adalbas has quit IRC01:05
*** Masahiro has joined #openstack-qa01:06
*** yamahata has joined #openstack-qa01:08
*** melwitt has quit IRC01:11
*** dims has joined #openstack-qa01:31
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: try running with -o nounset  https://review.openstack.org/13928701:36
*** andreykurilin has quit IRC01:40
sdagueheyongli: hey, on for a few, but need to run off to do family things shortly. So I think jogo wants to get a spec going. There was some confusion over the approach here.01:41
*** andreykurilin has joined #openstack-qa01:41
sdagueso I think we really want a set of tests in the nova tree that effectively live on a fake libvirt which can present a number of different pci configurations and we can test that things work to the fake libvirt level.01:42
sdaguethen a tool which verifies that that mocking layer is correct, which we can spot check.01:43
sdagueon real hardware01:43
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: try running with -o nounset  https://review.openstack.org/13928701:45
heyonglisdague,do you mean a fakelivrit provide funcitonal, then run the nova on the fakelibvirt or unit testing on this libvirt?01:46
*** salv-orlando has quit IRC01:47
sdaguerunning nova on a fakelibvirt01:48
*** salv-orlando has joined #openstack-qa01:48
sdagueso we can test from the wsgi layer down through to the mock01:48
sdaguethen we know all the nova code gets us to the right input / output with libvirt01:49
sdaguethe thing of it is, based on my experience testing end to end from tempest is going to be very limitting in what you can expose for issues in code like this, and will be very hard to debug01:49
*** salv-orlando has quit IRC01:49
heyonglisdague, it's great idea. we can check the xml it genrated to ensure all functional.01:50
sdagueyep01:50
sdagueexactly01:50
sdaguethen have another tool that can test "bottom half"01:50
sdaguetake that xml we generate and actually try to launch libvirt guests with it01:50
sdagueto make sure our mock is valid01:50
sdaguebecause someone has to watch the watcher01:50
sdagueanyway, I need to drop off. I think jogo was going to try to get a spec rolling just so we can make sure everyone is on the same page.01:51
*** nosnos has joined #openstack-qa01:52
heyonglisdague, boot this xml in tempest? this also need tempest know the pci infomations.01:52
sdagueno, just manually01:52
heyonglisdague, get it.01:52
sdagueso it would just be "does this libvirt xml boot a guest"01:52
sdaguejust connect to libvirt via python, push our mock xml in, and make sure libvirt doesn't die horribly01:53
sdagueand the guest comes up01:53
sdaguethat should let us have actually a bunch of samples of that interface from a bunch of different pieces of hardware that we can simulate in tree01:54
sdagueok, got to run, have a good evening01:54
heyonglisdague, good evening, see you01:54
*** bnemec has quit IRC02:06
*** Masahiro has quit IRC02:22
*** Masahiro has joined #openstack-qa02:24
*** ranger81 has quit IRC02:31
*** yamahata has quit IRC02:35
*** yamamoto_ has joined #openstack-qa02:38
*** zz_dimtruck is now known as dimtruck02:39
*** ranger81 has joined #openstack-qa02:43
*** nosnos has quit IRC02:48
*** salv-orlando has joined #openstack-qa02:50
*** shashankhegde has quit IRC02:51
*** harlowja_ is now known as harlowja_away02:54
*** salv-orlando has quit IRC02:55
*** topshare has quit IRC02:55
*** achanda has joined #openstack-qa03:01
*** topshare has joined #openstack-qa03:02
*** wuhg has joined #openstack-qa03:06
*** achanda has quit IRC03:11
*** om has joined #openstack-qa03:17
*** patrickeast has quit IRC03:24
*** dims has quit IRC03:28
*** Masahiro has quit IRC03:38
*** Masahiro has joined #openstack-qa03:44
*** Masahiro has quit IRC03:49
*** salv-orlando has joined #openstack-qa03:51
openstackgerritJeremy Stanley proposed openstack/tempest-lib: Workflow documentation is now in infra-manual  https://review.openstack.org/13939603:52
openstackgerritJeremy Stanley proposed openstack-dev/devstack: Workflow documentation is now in infra-manual  https://review.openstack.org/13940803:53
openstackgerritJeremy Stanley proposed openstack-dev/hacking: Workflow documentation is now in infra-manual  https://review.openstack.org/13940903:53
openstackgerritJeremy Stanley proposed openstack-infra/elastic-recheck: Workflow documentation is now in infra-manual  https://review.openstack.org/13941503:53
*** salv-orlando has quit IRC03:56
*** pcrews has quit IRC03:58
*** Masahiro has joined #openstack-qa03:59
*** shashankhegde has joined #openstack-qa04:07
*** dpaterson has quit IRC04:13
*** weshay has quit IRC04:14
*** dims has joined #openstack-qa04:29
*** topshare has quit IRC04:30
*** topshare has joined #openstack-qa04:30
openstackgerritSabari proposed openstack-dev/devstack: VMware: add default image metadata for hw_vif_model  https://review.openstack.org/13951504:33
*** dims has quit IRC04:34
*** topshare has quit IRC04:37
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Remove uneeded version string from setup.cfg  https://review.openstack.org/13951604:40
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Remove unneeded version string from setup.cfg  https://review.openstack.org/13951604:41
*** topshare has joined #openstack-qa04:42
*** jecarey has joined #openstack-qa04:45
openstackgerritMerged openstack-infra/subunit2sql: Remove unneeded version string from setup.cfg  https://review.openstack.org/13951604:47
*** chandankumar has joined #openstack-qa05:08
*** chandankumar has quit IRC05:09
*** heyongli has quit IRC05:10
*** heyongli has joined #openstack-qa05:10
*** Masahiro has quit IRC05:12
*** shashankhegde has quit IRC05:15
*** om has quit IRC05:22
*** chandankumar has joined #openstack-qa05:24
*** om has joined #openstack-qa05:27
*** Masahiro has joined #openstack-qa05:27
*** dimtruck is now known as zz_dimtruck05:38
*** yamamot__ has joined #openstack-qa05:40
*** salv-orlando has joined #openstack-qa05:40
*** jecarey has quit IRC05:40
*** yamamo___ has joined #openstack-qa05:43
*** yamamoto_ has quit IRC05:43
*** ranger81 has joined #openstack-qa05:44
*** salv-orlando has quit IRC05:45
*** yamamot__ has quit IRC05:46
*** ranger81 has quit IRC05:48
*** Longgeek has joined #openstack-qa05:52
*** ranger81 has joined #openstack-qa06:00
*** bnemec has joined #openstack-qa06:01
*** SridharG has joined #openstack-qa06:02
*** rushiagr is now known as rushiagr_away06:10
*** topshare has quit IRC06:19
*** topshare has joined #openstack-qa06:23
*** Longgeek has quit IRC06:30
*** Longgeek has joined #openstack-qa06:32
*** Masahiro has quit IRC06:35
*** Masahiro has joined #openstack-qa06:39
*** ajo has joined #openstack-qa06:47
*** rushiagr_away is now known as rushiagr06:49
*** openstackgerrit has quit IRC06:49
*** openstackgerrit has joined #openstack-qa06:49
*** amotoki is now known as amotoki__away06:55
*** ranger81 has quit IRC07:00
*** ranger81 has joined #openstack-qa07:00
*** topshare has quit IRC07:04
*** enikanorov_ has joined #openstack-qa07:04
*** enikanorov__ has quit IRC07:07
*** topshare has joined #openstack-qa07:08
*** yfried_ has quit IRC07:10
*** shashankhegde has joined #openstack-qa07:13
*** topshare has quit IRC07:20
*** topshare has joined #openstack-qa07:26
*** ranger81 has quit IRC07:28
*** topshare has quit IRC07:29
*** salv-orlando has joined #openstack-qa07:29
*** shashankhegde has quit IRC07:31
*** salv-orlando has quit IRC07:34
*** dkusidlo has joined #openstack-qa07:44
*** SridharG has quit IRC07:45
openstackgerritMerged openstack-dev/devstack: Fix NOVA_API_VERSION option  https://review.openstack.org/13925807:47
*** SridharG has joined #openstack-qa07:48
*** SridharG has quit IRC07:50
heyonglisdague, jogo ,  about the fake libvirt. we might have 2 choice, fake a libvirt driver of nova. second, fake a libvirt API which current nova libvirt driver use of.07:56
heyonglisdague, jogo, i prefer the second approch, because fake a libvirt driver of nova can not testing the pci code in current nova driver.07:57
heyongliand, i would like to know,  if the test case can not be upstream, does our CI could go on and online?   even the fake libvirt does testing most of nova code. there still some thing else can not cove by the fake driver: the hypervisor and the real hw.08:00
heyonglisdague, jogo : and, i would like to know,  if the test case can not be upstream, does our CI could go on and online?   even the fake libvirt does testing most of nova code. there still some thing else can not cove by the fake driver: the hypervisor and the real hw08:00
heyonglisdague, jogo : i also concern the sriov,  should use real-hw, the fake libvirt can not ensure the configuration of the instance interface is worked as expect.08:02
heyonglisdague, jogo , i register a fake libvirt blueprints(no spec yet): https://blueprints.launchpad.net/nova/+spec/fake-libvirt-hw-testing-driver08:06
*** tiantian has quit IRC08:11
*** tiantian has joined #openstack-qa08:11
*** bkopilov has quit IRC08:20
*** pkoniszewski has joined #openstack-qa08:21
*** bkopilov has joined #openstack-qa08:22
*** gfidente has joined #openstack-qa08:23
*** om has quit IRC08:28
*** Masahiro has quit IRC08:36
*** markus_z has joined #openstack-qa08:40
*** om has joined #openstack-qa08:40
*** tkammer has joined #openstack-qa08:40
*** yamamo___ has quit IRC08:41
*** oomichi has quit IRC08:41
*** luqas has joined #openstack-qa08:42
*** shashankhegde has joined #openstack-qa08:42
openstackgerritAbhishek G M proposed openstack/tempest: Floating IP test enhancements  https://review.openstack.org/13200008:42
openstackgerritAbhishek G M proposed openstack/tempest: Router test enhancements  https://review.openstack.org/13957108:42
*** anshul has joined #openstack-qa08:45
*** mpaolino has joined #openstack-qa08:46
*** Masahiro has joined #openstack-qa08:47
*** vigneshvar has joined #openstack-qa08:48
*** dkusidlo has quit IRC08:50
*** shashankhegde has quit IRC08:51
*** anshul has quit IRC08:53
*** safchain has joined #openstack-qa08:55
*** jpich has joined #openstack-qa08:55
*** anshul has joined #openstack-qa08:56
*** jordanP has joined #openstack-qa09:00
*** salv-orlando has joined #openstack-qa09:03
*** salv-orlando has quit IRC09:04
*** salv-orlando has joined #openstack-qa09:05
*** yamamoto_ has joined #openstack-qa09:09
*** salv-orl_ has joined #openstack-qa09:16
*** dkusidlo has joined #openstack-qa09:19
openstackgerritMerged openstack-dev/hacking: Workflow documentation is now in infra-manual  https://review.openstack.org/13940909:20
*** salv-orlando has quit IRC09:20
*** e0ne has joined #openstack-qa09:22
openstackgerrityolanda.robla proposed openstack-infra/devstack-gate: Add is_debian and uses_debs functions  https://review.openstack.org/13076809:22
*** luqas has left #openstack-qa09:22
*** luqas has joined #openstack-qa09:23
*** yamamoto_ has quit IRC09:23
*** dkusidlo has quit IRC09:31
*** dmorita has quit IRC09:32
*** andreykurilin_ has joined #openstack-qa09:32
*** SridharG has joined #openstack-qa09:33
*** yamamoto has joined #openstack-qa09:35
openstackgerritAndrea Frittoli  proposed openstack/qa-specs: add bp:test-accounts-continued.rst  https://review.openstack.org/13957809:35
*** luqas has quit IRC09:37
*** luqas has joined #openstack-qa09:37
*** vigneshvar has quit IRC09:38
*** abehl has joined #openstack-qa09:38
*** luqas has quit IRC09:39
*** luqas has joined #openstack-qa09:40
*** jlibosva has joined #openstack-qa09:40
*** anshul has quit IRC09:41
*** SridharG has quit IRC09:45
*** luqas has quit IRC09:46
*** dkusidlo has joined #openstack-qa09:46
*** e0ne has quit IRC09:47
*** luqas has joined #openstack-qa09:49
*** yamamoto has quit IRC09:50
*** luqas has quit IRC09:50
*** mpaolino has quit IRC09:50
*** luqas has joined #openstack-qa09:51
*** vigneshvar has joined #openstack-qa09:51
*** luqas has quit IRC09:53
*** luqas has joined #openstack-qa09:53
*** andreykurilin_ has quit IRC09:54
*** andreykurilin_ has joined #openstack-qa09:54
*** luqas has quit IRC09:55
*** luqas has joined #openstack-qa09:55
*** luqas__ has joined #openstack-qa09:58
*** luqas has quit IRC09:58
*** nikesh_vedams_ has joined #openstack-qa09:59
*** luqas__ has quit IRC10:00
*** luqas__ has joined #openstack-qa10:00
nikesh_vedams_in 3 node juno devstack,controller,block and compute,do we have to manually run "nova network-create"10:02
*** chandankumar has quit IRC10:02
nikesh_vedams_when i am creating a vm,its not getting ip10:02
*** luqas__ has quit IRC10:03
*** luqas__ has joined #openstack-qa10:03
*** luqas__ has quit IRC10:04
*** luqas__ has joined #openstack-qa10:05
*** luqas__ has quit IRC10:07
*** luqas__ has joined #openstack-qa10:08
*** gszasz has joined #openstack-qa10:08
*** chandankumar has joined #openstack-qa10:09
*** luqas__ has quit IRC10:09
*** luqas has joined #openstack-qa10:10
*** abehl has quit IRC10:11
*** luqas__ has joined #openstack-qa10:12
*** luqas has quit IRC10:12
*** luqas__ has quit IRC10:13
*** luqas___ has joined #openstack-qa10:13
*** luqas___ has quit IRC10:15
*** luqas__ has joined #openstack-qa10:15
*** aix has joined #openstack-qa10:16
*** e0ne has joined #openstack-qa10:16
*** luqas__ has quit IRC10:17
*** luqas__ has joined #openstack-qa10:17
*** luqas__ has quit IRC10:19
*** luqas has joined #openstack-qa10:20
*** andreykurilin_ has quit IRC10:20
*** andreykurilin_ has joined #openstack-qa10:21
*** amotoki__away is now known as amotoki10:22
*** luqas has quit IRC10:23
*** luqas has joined #openstack-qa10:23
*** abehl has joined #openstack-qa10:25
*** luqas has quit IRC10:26
*** luqas has joined #openstack-qa10:26
*** luqas has quit IRC10:28
*** cdent has joined #openstack-qa10:29
*** luqas has joined #openstack-qa10:29
*** vigneshvar has quit IRC10:31
*** luqas has quit IRC10:31
*** luqas has joined #openstack-qa10:31
*** Masahiro has quit IRC10:33
*** luqas__ has joined #openstack-qa10:35
*** luqas has quit IRC10:35
*** luqas__ has quit IRC10:36
*** luqas__ has joined #openstack-qa10:37
*** vigneshvar has joined #openstack-qa10:44
*** grafuls has joined #openstack-qa10:47
*** SridharG has joined #openstack-qa10:47
*** yamamoto has joined #openstack-qa10:50
*** tkammer has quit IRC10:51
*** mpaolino has joined #openstack-qa10:53
*** tosky has joined #openstack-qa10:53
*** yamamoto has quit IRC10:56
*** tiantian has quit IRC10:59
*** salv-orlando has joined #openstack-qa11:13
*** salv-orl_ has quit IRC11:17
*** jhenner has joined #openstack-qa11:29
*** vigneshvar has quit IRC11:32
*** Masahiro has joined #openstack-qa11:34
*** aysyd has joined #openstack-qa11:35
*** dkusidlo has quit IRC11:35
*** dkusidlo has joined #openstack-qa11:38
*** Masahiro has quit IRC11:39
*** kashyap has left #openstack-qa11:42
*** vigneshvar has joined #openstack-qa11:46
*** luqas has joined #openstack-qa11:47
*** luqas__ has quit IRC11:47
*** dkusidlo has quit IRC11:48
*** luqas has quit IRC11:49
*** luqas has joined #openstack-qa11:49
*** luqas has quit IRC11:51
*** luqas has joined #openstack-qa11:52
*** luqas has quit IRC11:53
*** luqas has joined #openstack-qa11:54
*** amotoki is now known as amotoki__away11:54
*** luqas has quit IRC11:56
*** luqas__ has joined #openstack-qa11:56
*** luqas__ has quit IRC11:58
*** luqas has joined #openstack-qa11:59
*** luqas has quit IRC12:01
*** luqas__ has joined #openstack-qa12:01
*** yfried_ has joined #openstack-qa12:01
*** jpich has quit IRC12:01
*** jpich has joined #openstack-qa12:02
*** luqas___ has joined #openstack-qa12:03
*** luqas__ has quit IRC12:03
*** markus_z has quit IRC12:04
*** dims has joined #openstack-qa12:04
*** luqas__ has joined #openstack-qa12:07
*** luqas___ has quit IRC12:07
*** luqas__ has quit IRC12:08
*** andreykurilin_ has quit IRC12:09
*** luqas has joined #openstack-qa12:09
sdagueheyongli: definitely was thinking the second, this would be a mock of the libvirt, we'd still be using the normal libvirt driver code12:10
*** luqas has quit IRC12:10
*** luqas has joined #openstack-qa12:11
*** luqas__ has joined #openstack-qa12:13
*** luqas has quit IRC12:13
*** luqas__ has quit IRC12:15
*** luqas has joined #openstack-qa12:16
nikesh_vedams_in my 3 node devstack setup,when i am creating a instance,its not getting IP12:16
nikesh_vedams_i am able to get instance12:17
nikesh_vedams_but without ip12:17
nikesh_vedams_i have juno devstack12:18
*** SridharG has quit IRC12:18
*** luqas__ has joined #openstack-qa12:19
*** SridharG has joined #openstack-qa12:19
nikesh_vedams_nova.conf on compute node12:19
nikesh_vedams_http://paste.openstack.org/show/145401/12:19
*** luqas has quit IRC12:20
*** luqas__ has quit IRC12:20
nikesh_vedams_do i have to manually run "nova netwotk-create" on controller node in case of 3 node juno devstack12:20
*** luqas__ has joined #openstack-qa12:20
*** luqas__ has quit IRC12:22
*** luqas__ has joined #openstack-qa12:23
openstackrecheckopenstack/sahara change: https://review.openstack.org/139387 failed gate-sahara-pep8 in the gate queue with an unrecognized error12:23
*** luqas__ has quit IRC12:24
*** luqas__ has joined #openstack-qa12:25
*** luqas__ has quit IRC12:26
*** luqas has joined #openstack-qa12:27
*** luqas has quit IRC12:29
*** luqas has joined #openstack-qa12:30
openstackgerritRajkumar proposed openstack/tempest: Add Test to Create Port with no security groups  https://review.openstack.org/11590812:31
*** luqas has quit IRC12:32
*** luqas has joined #openstack-qa12:33
*** luqas has quit IRC12:35
*** luqas has joined #openstack-qa12:35
*** cbader has quit IRC12:36
*** cbader has joined #openstack-qa12:36
*** luqas__ has joined #openstack-qa12:38
*** luqas has quit IRC12:39
openstackrecheckopenstack/sahara change: https://review.openstack.org/139209 failed gate-sahara-pep8 in the gate queue with an unrecognized error12:39
*** luqas___ has joined #openstack-qa12:41
*** luqas__ has quit IRC12:42
*** markus_z has joined #openstack-qa12:46
*** markus_z has quit IRC12:46
*** luqas___ has quit IRC12:46
*** luqas has joined #openstack-qa12:47
*** mwagner_lap has quit IRC12:48
*** luqas has quit IRC12:48
*** weshay has joined #openstack-qa12:48
*** luqas has joined #openstack-qa12:49
*** luqas has quit IRC12:50
*** luqas has joined #openstack-qa12:51
*** markus_z has joined #openstack-qa12:51
openstackrecheckopenstack/sahara change: https://review.openstack.org/138211 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/137049612:52
*** Longgeek has quit IRC12:52
*** luqas has quit IRC12:52
*** luqas has joined #openstack-qa12:53
*** tochi_ has quit IRC12:53
*** luqas has quit IRC12:54
*** luqas has joined #openstack-qa12:55
*** luqas has quit IRC12:56
*** luqas has joined #openstack-qa12:57
*** luqas has quit IRC13:00
*** luqas has joined #openstack-qa13:00
*** luqas has quit IRC13:01
*** luqas has joined #openstack-qa13:02
*** luqas has quit IRC13:04
*** luqas has joined #openstack-qa13:04
*** chandankumar has quit IRC13:05
*** luqas has quit IRC13:05
*** luqas has joined #openstack-qa13:06
*** dkusidlo has joined #openstack-qa13:06
*** luqas has quit IRC13:07
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: code coloring test with gerrit  https://review.openstack.org/13961913:08
*** luqas has joined #openstack-qa13:08
*** Masahiro has joined #openstack-qa13:08
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: code coloring test with gerrit  https://review.openstack.org/13961913:09
*** luqas has quit IRC13:10
*** luqas__ has joined #openstack-qa13:10
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: code coloring test with gerrit  https://review.openstack.org/13961913:10
openstackgerritSean Dague proposed openstack-dev/devstack: WIP: code coloring test with gerrit  https://review.openstack.org/13961913:10
*** yamamoto has joined #openstack-qa13:10
*** luqas__ has quit IRC13:11
*** mriedem has joined #openstack-qa13:12
*** luqas has joined #openstack-qa13:12
*** Masahiro has quit IRC13:13
*** luqas has quit IRC13:13
openstackgerritRob Cresswell proposed openstack-dev/devstack: Create initial network when using Cisco N1k plugin  https://review.openstack.org/13962013:14
*** luqas has joined #openstack-qa13:14
*** luqas has quit IRC13:15
*** luqas has joined #openstack-qa13:16
*** Longgeek has joined #openstack-qa13:17
*** nikesh_vedams_ has quit IRC13:17
*** luqas has quit IRC13:18
*** luqas has joined #openstack-qa13:18
*** luqas has quit IRC13:21
*** luqas__ has joined #openstack-qa13:21
*** om has quit IRC13:21
*** e0ne has quit IRC13:22
*** luqas__ has quit IRC13:22
*** luqas__ has joined #openstack-qa13:22
mriedemshameless review plug https://review.openstack.org/#/c/135656/13:23
*** andreaf has quit IRC13:23
*** dkusidlo has quit IRC13:25
*** mwagner_lap has joined #openstack-qa13:26
*** e0ne has joined #openstack-qa13:28
*** yfried_ has quit IRC13:28
*** dims has quit IRC13:29
*** dims has joined #openstack-qa13:29
openstackgerritSean Dague proposed openstack-dev/devstack: remove nova baremetal driver support  https://review.openstack.org/13962513:31
sdague+213:32
*** ylobankov has joined #openstack-qa13:33
*** abehl has quit IRC13:35
openstackgerritMerged openstack-dev/devstack: VMware: add default image metadata for hw_vif_model  https://review.openstack.org/13951513:40
*** dpaterson has joined #openstack-qa13:53
*** toure has joined #openstack-qa13:53
*** SridharG has quit IRC13:54
*** yamamoto has quit IRC13:56
*** tiantian has joined #openstack-qa13:59
*** vigneshvar has quit IRC14:03
*** dims has quit IRC14:05
*** tiantian has quit IRC14:06
*** dims has joined #openstack-qa14:06
*** andreaf has joined #openstack-qa14:07
*** yamamoto has joined #openstack-qa14:13
*** SridharG has joined #openstack-qa14:20
*** dkusidlo has joined #openstack-qa14:23
*** e0ne has quit IRC14:28
sdaguemtreinish: how would you feel about me running the abandon policy script from nova on tempest repo?14:28
*** dustins has joined #openstack-qa14:29
*** dkusidlo_ has joined #openstack-qa14:29
jogoheyongli: can you put your thoughts/concerns ideas about the possible approaches to testing in a spec for that BP14:30
jogoheyongli: that way we have a central place to discuss this14:30
jogoheyongli: part of the reason for planning this out, is we are expecting more HW configuration and features will need testing in the future14:31
jogoheyongli: if we need an entire new fake libvirt driver for HW simulation testing, it may not be worth going down that path14:32
*** dkusidlo has quit IRC14:32
*** dkusidlo_ is now known as dkusidlo14:32
jogoheyongli: a doc with the pros/cons of each testing method would help sort this out14:32
jogoheyongli: we are trying to answer the question 'what is the best way to test this'14:39
*** dustins has quit IRC14:40
*** e0ne has joined #openstack-qa14:41
*** gszasz has quit IRC14:42
*** dustins has joined #openstack-qa14:42
*** cdent has quit IRC14:42
*** jecarey has joined #openstack-qa14:43
*** jogo is now known as flashgordon14:48
mtreinishsdague: I think I'd be fine with that, but let me take a look at the script first14:49
mtreinishdo you have a link?14:49
sdagueyeh, how about I upload a tempest copy and you can look there14:50
*** luqas__ has quit IRC14:50
openstackgerritSean Dague proposed openstack/tempest: abandon reviews script  https://review.openstack.org/13964514:50
*** luqas__ has joined #openstack-qa14:51
sdagueI put a DRYRUN in there to see how much work there was to do14:51
*** dansmith is now known as superdan14:52
*** luqas__ has quit IRC14:52
openstackgerritMikhail S Medvedev proposed openstack-dev/devstack: Add PRECACHE_IMAGE_URL parameter and default value  https://review.openstack.org/13116714:53
*** ajo has quit IRC14:53
*** luqas__ has joined #openstack-qa14:53
*** mpaolino has quit IRC14:53
*** luqas__ has quit IRC14:54
*** luqas__ has joined #openstack-qa14:55
sdaguedtroyer: I see you looked active yesterday14:55
sdagueI had a question about devstack vs gerrit when you wander around14:55
*** dkusidlo has quit IRC14:56
*** luqas__ has quit IRC14:56
*** luqas__ has joined #openstack-qa14:57
sdagueflashgordon: any chance you want to fix up this patch of yours - https://review.openstack.org/#/c/89470/14:57
*** rbak has joined #openstack-qa14:57
*** Masahiro has joined #openstack-qa14:57
mtreinishsdague: it looks like 133 hits on tempest with the script...14:59
*** luqas__ has quit IRC14:59
*** luqas__ has joined #openstack-qa14:59
openstackgerritMerged openstack-infra/elastic-recheck: Workflow documentation is now in infra-manual  https://review.openstack.org/13941515:00
flashgordonsdague: not today no15:00
*** luqas__ has quit IRC15:00
flashgordonat least not yet15:00
sdaguemtreinish: yeh, sounds about right15:01
*** luqas__ has joined #openstack-qa15:01
*** Masahiro has quit IRC15:01
sdaguemtreinish: so... how do you feel about it?15:02
mtreinishsdague: considering there are only 226 open reviews in tempest right now thats a pretty big chunk15:02
sdagueyeh, but I think those are all boned15:02
sdaguethey are blocked for one reason or another15:02
mtreinishsdague: I think I'm fine with it, it'll at least wake people up if the review is sitting there15:03
sdaguewhich, is a different issue15:03
sdagueok, will run now15:03
*** luqas__ has quit IRC15:03
*** luqas__ has joined #openstack-qa15:03
sdagueit will also massively clean up my reviews view15:04
sdaguepart of this is that it also leaves instructions about what to do next15:04
*** cdent has joined #openstack-qa15:04
*** openstackrecheck has quit IRC15:06
*** luqas__ has quit IRC15:06
*** luqas___ has joined #openstack-qa15:06
*** openstackrecheck has joined #openstack-qa15:06
sdagueok, done15:06
mtreinishsdague: ok, thanks15:07
*** dims is now known as dimsum__15:07
*** luqas__ has joined #openstack-qa15:09
*** luqas___ has quit IRC15:09
*** pkoniszewski has quit IRC15:09
*** markmcclain has joined #openstack-qa15:10
*** luqas has joined #openstack-qa15:11
*** luqas__ has quit IRC15:12
*** bnemec is now known as beekneemech15:13
*** yamahata has joined #openstack-qa15:15
*** mpaolino has joined #openstack-qa15:16
*** andreykurilin_ has joined #openstack-qa15:20
*** esp has joined #openstack-qa15:21
*** dims_ has joined #openstack-qa15:24
*** dimsum__ has quit IRC15:28
*** zz_dimtruck is now known as dimtruck15:30
*** rushiagr is now known as rushiagr_away15:35
*** ajo has joined #openstack-qa15:36
*** jlibosva has quit IRC15:37
openstackrecheckopenstack/nova change: https://review.openstack.org/136477 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error15:38
*** mestery has quit IRC15:42
dkranzmtreinish: were the heat slow jobs completely eliminated?15:44
dkranzmtreinish: The last two nightly tempest all runs are failing on one of the heat slow tests but I don't see a heat slow test in our check jobs anymore15:45
mtreinishdkranz: yes we removed that job from everywhere, there was a ML thread about it15:45
mtreinishit was only running like 5 tests anyway15:46
dkranzmtreinish: yes, but I diagnosed a previous issue by comparing results of the nightly vs the check jobs15:46
dkranzmtreinish: anyway, it was the right thing so I will file a heat bug15:47
*** jhenner has quit IRC15:47
dkranzshardy: Do the heat slow jobs have an equivalent in heat functional tests now?15:48
shardydkranz: Yes I think so, but stevebaker can confirm they've all now been migrated15:51
dkranzshardy: ok. NeutronResourcesTestJSON has failed the last two days, but was working before that15:52
dkranzshardy: I am filing a heat bug to record that fact15:52
*** pcrews has joined #openstack-qa15:52
shardydkranz: interesting, yes please do15:52
dkranzshardy: might not really be a heat issue of course15:53
openstackgerritSean Dague proposed openstack-dev/grenade: remove sourcing of nova baremetal driver  https://review.openstack.org/13966615:53
*** yamahata has quit IRC15:57
*** e0ne has quit IRC15:59
*** e0ne has joined #openstack-qa16:01
*** rushiagr_away is now known as rushiagr16:07
*** yamamoto has quit IRC16:08
*** luqas has quit IRC16:15
*** luqas__ has joined #openstack-qa16:15
*** luqas__ has quit IRC16:16
*** luqas__ has joined #openstack-qa16:16
*** sabeen has joined #openstack-qa16:17
*** luqas__ has quit IRC16:19
*** luqas__ has joined #openstack-qa16:19
*** luqas___ has joined #openstack-qa16:21
*** luqas__ has quit IRC16:21
*** mestery has joined #openstack-qa16:21
*** luqas___ has quit IRC16:23
*** luqas has joined #openstack-qa16:24
*** luqas has quit IRC16:26
*** luqas has joined #openstack-qa16:26
*** yamamoto has joined #openstack-qa16:28
*** mestery has quit IRC16:28
*** mestery has joined #openstack-qa16:28
*** luqas__ has joined #openstack-qa16:29
*** luqas has quit IRC16:29
openstackgerritMauro Rodrigues proposed openstack/tempest-lib: Fix migration script to use original SHA1 from tempest's commits  https://review.openstack.org/13967916:32
*** luqas has joined #openstack-qa16:32
*** luqas__ has quit IRC16:32
mtreinishmaurosr: ^^^ awesome thanks16:33
maurosrmtreinish: sorry the time it took, I forgot it, just remembered couple of days ago16:33
*** luqas has quit IRC16:33
*** luqas has joined #openstack-qa16:34
mtreinishno worries, there wasn't much of a rush on it anyway. No one has done a migration in a while16:34
mtreinishbut at least now they won't be blocked by that anymore16:34
maurosrmtreinish: also there is an alternative that I barelly need to change the original script (there is a way to redirect the original ids to new ones mapping) if you preffer16:34
maurosrI can submit that as well16:34
mtreinishmaurosr: heh, it doesn't really matter. :) I think this way might be better because it's simpler16:35
maurosrand faster hehe16:35
mtreinishthe reason it was like that before was the original script actually merged the history into tempest-lib16:35
*** luqas has quit IRC16:35
*** luqas has joined #openstack-qa16:36
mtreinishwhen we couldn't do that with infra after the repo was created, I just switched the merge to write the log into the commit msg16:36
*** nfedotov has left #openstack-qa16:37
*** luqas__ has joined #openstack-qa16:37
*** luqas has quit IRC16:37
*** luqas__ has quit IRC16:40
*** luqas__ has joined #openstack-qa16:40
openstackgerritOpenStack Proposal Bot proposed openstack/tempest-lib: Updated from global requirements  https://review.openstack.org/12764916:40
*** jpich has quit IRC16:41
*** markus_z has quit IRC16:41
*** luqas__ has quit IRC16:42
*** luqas__ has joined #openstack-qa16:43
*** Masahiro has joined #openstack-qa16:46
*** luqas has joined #openstack-qa16:46
*** luqas__ has quit IRC16:46
*** dims_ is now known as dimsum__16:47
*** luqas__ has joined #openstack-qa16:49
*** luqas has quit IRC16:50
*** Masahiro has quit IRC16:50
*** luqas__ has quit IRC16:51
*** luqas__ has joined #openstack-qa16:52
*** luqas__ has quit IRC16:53
*** luqas__ has joined #openstack-qa16:53
*** mwagner_lap has quit IRC16:54
*** luqas__ has quit IRC16:55
*** luqas has joined #openstack-qa16:56
*** luqas has quit IRC16:57
*** luqas has joined #openstack-qa16:57
*** mlavalle has joined #openstack-qa16:59
*** toure has quit IRC16:59
*** luqas__ has joined #openstack-qa17:00
*** luqas has quit IRC17:00
*** safchain has quit IRC17:02
*** luqas has joined #openstack-qa17:03
*** sfineberg has joined #openstack-qa17:03
*** SridharG has left #openstack-qa17:03
*** luqas__ has quit IRC17:03
*** luqas has quit IRC17:04
*** cdent has quit IRC17:04
*** luqas has joined #openstack-qa17:05
*** luqas has quit IRC17:07
*** luqas__ has joined #openstack-qa17:07
*** luqas___ has joined #openstack-qa17:09
*** luqas__ has quit IRC17:10
*** marun has joined #openstack-qa17:10
*** luqas___ has quit IRC17:11
*** luqas___ has joined #openstack-qa17:12
*** luqas___ has quit IRC17:13
*** luqas__ has joined #openstack-qa17:13
*** vigneshvar has joined #openstack-qa17:14
*** luqas__ has quit IRC17:15
*** luqas__ has joined #openstack-qa17:16
*** luqas__ has quit IRC17:17
*** luqas__ has joined #openstack-qa17:18
*** shashankhegde has joined #openstack-qa17:18
andreafmtreinish: I posted the spec for test-accounts part2 https://review.openstack.org/#/c/139578/17:18
*** luqas__ has quit IRC17:22
dtroyersdague: what's up17:22
*** andreykurilin_ has quit IRC17:24
*** grafuls is now known as grafuls|gone17:25
*** david-lyle has joined #openstack-qa17:29
*** jordanP has quit IRC17:30
*** ranger81 has joined #openstack-qa17:30
*** abehl has joined #openstack-qa17:32
*** grafuls|gone has quit IRC17:33
*** mestery has quit IRC17:35
*** shashankhegde has quit IRC17:36
*** beekneemech has quit IRC17:37
*** mpaolino has quit IRC17:40
*** ajo has quit IRC17:44
*** yamamoto has quit IRC17:49
*** krtaylor has quit IRC17:56
*** boris-42 has joined #openstack-qa17:57
*** ajo has joined #openstack-qa18:00
*** rwsu has joined #openstack-qa18:02
*** shashankhegde has joined #openstack-qa18:03
*** wuhg has quit IRC18:03
*** suneelb has joined #openstack-qa18:03
*** patrickeast has joined #openstack-qa18:05
*** melwitt has joined #openstack-qa18:06
*** achanda has joined #openstack-qa18:08
*** mestery has joined #openstack-qa18:09
*** krtaylor has joined #openstack-qa18:13
*** mwagner_lap has joined #openstack-qa18:14
*** achanda has quit IRC18:17
dkranzmarun: Have a minute?18:17
marundkranz: sure18:17
*** achanda has joined #openstack-qa18:18
dkranzmarun: Now that there is agreement to do functional testing for neutron in the neutron project, we need to get more specific.18:18
dkranzmarun: we agreed not to move stuff out of tempest until it is in place in neutron18:18
marun'functional api testing' you mean (as per my recent email)18:18
dkranzmarun: yes18:18
marunok, let's get more specific18:18
dkranzmarun: but that leaves the question of new tempest patches that add neutron tests18:18
marundkranz: right18:19
dkranzmarun: I added you as reviewer for a few of these18:19
marundkranz: I'll take a look.18:19
dkranzmarun: but am trying to understand the criteria so we can publicize it for reviewers and contributors18:19
marundkranz: armax and I have been working hard on getting the api job running so that new test additions could land in-tree.18:19
marundkranz: what criteria do you mean?18:19
dkranzmarun: great.18:20
dkranzmarun:  I mean how one decides if a functional test involving neutron should go in tempest or in neutron18:20
marundkranz: is it just about 'neutron-only'?18:20
dkranzmarun: mostly18:20
dkranzmarun: I guess we could say that any neutron-only test should not go in tempest18:21
dkranzmarun: but I'm not sure how the tempest team feels about that18:21
marundkranz: That's my thinking.18:21
dkranzmtreinish: ^^^ any thought?18:21
marundkranz: For some reason I thought there was general consensus on that point.18:21
dkranzmarun: In theory yes, but the fact that we keep getting neutron patches in tempest indicates not every one is aware18:22
*** achanda has quit IRC18:22
marundkranz: And if not, it would be good to figure out what we have to do to make people comfortable with doing Neutron-only test development in th eNeutron tree.18:22
dkranzmarun: I guess first we need for it to be possible. Any ETA on that?18:23
marundkranz: I'm working on getting the check job running today.18:23
dkranzmarun: and can you point me to an example of one of these tests ?18:23
dkranzmarun: I guess we are talking about the kind that runs in a real stack?18:24
marundkranz: for now, yeah.18:24
dkranzmarun: so any test that was coded for tempest should translate to something that can run there18:24
marundkranz: though we're working on a framework that should allow full-stack neutron without devstack or tempest configuration being required.18:24
dkranzmarun: that's fine and should not really impact the test code I think18:25
marundkranz: there are some work items required on the tempest/tempest-lib side, though.18:25
marundkranz: currently the in-tree tests use tempest for tenant isolation and for the rest client18:25
dkranzmarun: are they enumerated some where?18:26
marundkranz: that's the high-level right there18:26
dkranzmarun: ah, do you mean the base rest client or the neutron-specific one?18:26
marundkranz: the neutron-specific one18:26
dkranzmarun: ok. I started doing some stuff with the clients-return-one-value thing on identity but perhaps we should give neutron more priority18:27
marundkranz: We'll need to migrate everything the in-tree tests need to tempest-lib and then force tempest-lib to gate on neutron api job18:27
marundkranz: without that, we can't gate on the api job, since it could be broken at any point by changes to tempest18:27
dkranzmarun: Do you need tenant_isolation or could you use the new test-accounts feature in tempest?18:28
marundkranz: can you explain the new feature?18:28
dkranzmarun: that allows you to specify some users and run each of them in parallel without creating a new tenant for each test class18:28
dkranzmarun: the disadvantage over tenant isolation is that leaked resources on one test failure could infect a subsequent test class18:29
marundkranz: has tempest switched to the new feature?  and if so, have leaked resources proven a problem?18:30
dkranzmarun: I am not sure whether there is any one currently working on  moving the rest client into tempest lib18:30
marundkranz: the other option is that we can copy the existing client verbatim pending its relocation18:30
marundkranz: oslo incubator-style18:30
dkranzmarun: no, tempest has not switched. There are still a few glitches like each user should have its own network. THere is a new blueprint to clean that up.18:31
*** harlowja_away is now known as harlowja_18:31
marundkranz: the other possibility is that we rely a test-managed neutron service that we can manage test isolation on more explicitly18:31
dkranzmarun: That seems like a reasonable idea once the stupid returning multiple values that are ignored thing is fixed18:31
dhellmannmtreinish: can you help out with reviews on https://review.openstack.org/#/c/139631/ to unblock oslo, please?18:32
dkranzmarun: this is probably worth a ml discussion18:32
dkranzmarun: I'm not sure of the details of what you mean by a test-managed neutron service18:33
marundkranz: a test fixture would manage a neutron daemon that could be targeted for testing18:33
dkranzmarun: but you still need client calls18:33
marundkranz: that's fine18:34
flashgordonadam_g: https://review.openstack.org/#/c/134734/2 what about the commit message needs updating?18:34
marundkranz: I realize, though, that I haven't thought through how Keystone fits into this.18:34
dkranzmarun: ok, so I will start on cleaning up the neutron rest client and test code to prepare for copying or moving18:34
marundkranz: I guess we'll have to stick with devstack/tempest configured api testing for now.18:34
*** Masahiro has joined #openstack-qa18:35
marundkranz: ok, great.18:35
adam_gflashgordon, the patch no longer makes grenade 'deal with both names for a minute', it just switches entirely from /apts/ to /debs/, using the symlinks that are in devstack now18:35
marundkranz: This being a cross-project effort, what do you think about filing a spec in that new repo?18:36
marundkranz: or how would you like to otherwise track this effort?18:36
flashgordonadam_g: want to just fix it for mordred18:37
flashgordonadam_g: I don't think he will mind18:37
adam_gflashgordon, sure18:37
*** e0ne has quit IRC18:37
*** Masahiro has quit IRC18:39
openstackgerritAdam Gandelman proposed openstack-dev/grenade: Rename apts to debs  https://review.openstack.org/13473418:39
adam_gflashgordon, ^18:40
flashgordonadam_g: thanks18:42
flashgordonadam_g: waiting on mordred to update a commit message didn't seem like it would be productive18:42
adam_gflashgordon, :)18:42
*** tosky has quit IRC18:43
*** nfedotov has joined #openstack-qa18:48
*** yamamoto has joined #openstack-qa18:49
*** yamamoto has quit IRC18:54
*** suneelb has quit IRC19:00
*** suneelb has joined #openstack-qa19:00
mtreinishdhellmann: sure I'll take a look19:00
dhellmannmtreinish: thanks19:00
mtreinishandreaf: ok cool, I'll review it in a bit19:00
*** suneelb has quit IRC19:00
*** suneelb has joined #openstack-qa19:01
*** suneelb has quit IRC19:01
*** suneelb has joined #openstack-qa19:01
mtreinishdhellmann: +219:02
dhellmannmtreinish: thanks!19:03
mtreinishthose backports have added quite a few bugs to a bunch of different stable jobs19:03
mordredflashgordon, adam_g thanks!!!19:03
*** nfedotov has quit IRC19:03
dhellmanndtroyer: do you have +2 on stable branches for devstack? https://review.openstack.org/#/c/139631/ is blocking a few of the oslo libs19:03
sdaguedtroyer: so... new ui on gerrit does code coloring client side19:05
sdaguewhich means that if we add #!/bin/bash to all the devstack lib files19:05
sdagueit will detect it as bash correctly19:05
*** nfedotov has joined #openstack-qa19:05
sdaguedtroyer: https://review.openstack.org/#/c/139619/1/lib/nova,cm19:06
sdaguewhich would make me extraordinarily happy19:06
sdaguebecause I find reviewing code without colors difficult19:06
sdaguedhellmann: I'm going to just fast approve 139631 as it's blocking you19:07
sdagueadam_g: https://review.openstack.org/#/c/139666/ and https://review.openstack.org/#/c/139625/ could use your +119:08
sdagueI think it's completely safe purge of nova bm19:08
sdaguebut you would probably know best19:09
*** ajo has quit IRC19:09
*** gfidente has quit IRC19:10
dtroyerdhellmann: +2 but sdague beat me to it…19:10
sdaguedtroyer: so... gerrit19:10
dtroyersdague: I think I can live with shebang lines easier than adding .sh extensions to everything.19:10
sdagueok19:10
dtroyerI'm thinking if there is a way to re-inforce that the lib/* files are not stand-alone scripts?19:10
sdaguealso, did you have an opinion on the set -o nounset email I sent19:11
sdaguewell they won't have +x on them19:11
dtroyerI replied to that email earlier (I thought I hit send)…19:11
openstackrecheckopenstack/nova change: https://review.openstack.org/137322 failed gate-nova-python27 in the gate queue with an unrecognized error19:11
sdagueoh, I haven't read email yet19:11
dtroyersdague: tl;dr, I'm OK, slight tweak to the function, and a thought about doing it in increments19:12
sdagueyeh, though the way the include works, it's honestly one of those things that you kind of just plough through the common path19:13
sdaguefix all the devstack tests that hit different trees19:13
sdaguethen fast merge patches19:13
sdaguebecause it doesn't trigger until it touches a var, it's hard to get to all the branches19:13
dtroyerwhen is K1?  maybe this should wait until after the pre-milestone rush?19:14
*** om has joined #openstack-qa19:18
*** kirshil has quit IRC19:20
adam_gsdague, +1s, should be fine in master19:23
openstackgerritJoe Gordon proposed openstack-dev/devstack: Fix permissions for keystonemiddleware signing_dir  https://review.openstack.org/13971419:24
flashgordon^ should fix all those keystone middleware warnings19:25
flashgordonmorganfainberg: ^19:25
sdagueflashgordon: would you mind extracting that to a common function?19:27
sdagueit seems like it's the same 4 lines repeated over and over19:27
flashgordonsdague: no I don't mind, -1 the patch and once I know the code works I'll extract it19:27
sdaguedone19:28
openstackgerritSean Dague proposed openstack-dev/devstack: add shebang lines to all lib files  https://review.openstack.org/13971519:28
sdaguedtroyer: ^^^19:28
flashgordonsdague: as part of fixing that I noticed it was super duper repetitive and silly19:29
sdagueyep :)19:29
*** ranger81 has quit IRC19:36
*** nfedotov1 has joined #openstack-qa19:39
*** nfedotov has quit IRC19:40
sdaguedtroyer: you want to +2 this grenade change - https://review.openstack.org/#/c/139666/ - then I can rerun the devstack change which guts baremetal and it should pass19:40
*** nfedotov1 has quit IRC19:41
dtroyersdague: grenade +A19:43
*** gordc has joined #openstack-qa19:43
*** suneelb has quit IRC19:43
*** dustins has quit IRC19:43
*** tkammer has joined #openstack-qa19:43
*** suneelb has joined #openstack-qa19:43
gordcsdague: re: http://lists.openstack.org/pipermail/openstack-dev/2014-December/052121.html do you have a link handy of what the workaround for ceilo bug was?19:45
*** om has quit IRC19:45
gordclink or what project the change was made too (i can dig for it there)19:45
dtroyerflashgordon: in https://review.openstack.org/#/c/139714/ is using install a feasable alternative?19:46
*** nfedotov has joined #openstack-qa19:46
flashgordoninstall?19:47
*** ranger81 has joined #openstack-qa19:47
mtreinishdkranz: on https://review.openstack.org/#/c/134062/ I would be inclined to just block that because it's a test for FWaaS. I don't think the neutron advanced services stuff really isn't reliable enough yet19:47
mtreinishwe've blocked all the other tests for this stuff for the most part19:48
flashgordondtroyer: something like function install_foo19:48
dtroyerflashgordon: /usr/bin/install, the old BSD command19:48
*** ajo has joined #openstack-qa19:48
dkranzmtreinish: Well, I gave it a -1 because I wasn't sure it belonged in tempest.19:49
dkranzmtreinish: But it looked ok as a test.19:49
dtroyerIIRC it should reduce those 4 lines to one or two at the most and is a standard command19:49
dkranzmtreinish: How are reviewers in tempest supposed to know that a test was submitted for a "released" feature we now think is not ready?19:50
mtreinishdkranz: well the fact that neutron is trying to remove them from their tree is a pretty good indicator I feel19:50
dhellmanndtroyer, sdague : thanks!19:50
mtreinishdkranz: it's come up on the meetings before too19:51
mtreinishdkranz: as a last resort just ask salv-orlando :)19:51
dkranzmtreinish: ok, I must have missed that. Anyway I gave it a -1 :)19:51
dkranzmtreinish: Are you ok with just saying "no new neutron api tests in tempest"? I think that is where Maru thinks we are going.19:53
flashgordondtroyer: hmm that may work19:53
flashgordondtroyer: I still think step one is make a common function19:53
flashgordonunless that common function can be install19:53
dkranzmtreinish: we had a discussion about that in this channel earlier today19:53
mtreinishdkranz: I'd rather take it on a case by case basis honestly19:54
mtreinishI'm uncomfortable making that as a blanket statement, but I think pushback is useful19:54
dkranzmtreinish: I was pointing out to Maru that we have no criteria to do it on a case by case19:54
dtroyerflashgordon: I'd prefer to avoid trivial functions unless they add some other value19:54
flashgordondtroyer: want to amend my patch?19:55
dkranzmtreinish: anyway, marun is very close to having a neutron job running so that neutron functional api tests can be submitted there19:55
dtroyerflashgordon: sure, after I clear another thing or two first19:56
*** abehl has quit IRC19:56
mtreinishdkranz: ok thats fine, but I think instead of just blocking everything, why not just ask in the review why does this have to be in tempest19:56
mtreinishand if there isn't a good answer then block it19:56
dkranzmtreinish: that's fine. But I think this is really caused by people not knowing about the policy change that was agreed in Paris. And until how there was no where else to put such tests.19:57
dkranzmtreinish: that should change soon19:57
dkranzmtreinish: until now19:57
dkranzmtreinish: anyway, I was asking for input from marun in the review, not blocking everything19:58
flashgordondtroyer: thanks19:58
dkranzmtreinish: Like I did not block the firewall test which you wanted me too for a different reason :)19:59
mtreinishdkranz: heh, yeah asking a neutron dev to weigh in on the proper place for a test works19:59
mtreinishdkranz: heh, well that's something else entirely :)20:00
*** e0ne has joined #openstack-qa20:01
flashgordonmtreinish: tempest question --20:10
mtreinishuh oh20:10
flashgordonin most cases tempest creates instances without networks associated to the tenant right?20:10
flashgordonsee https://review.openstack.org/#/c/139712/20:11
flashgordonbecause we don't always need to gain access to the tenant for all tests20:11
mtreinishflashgordon: well it depends, with neutron it creates per tenant networks20:11
mtreinishwith nova-network it's basically assuming flat networking20:11
flashgordonmtreinish: by default?20:11
flashgordonmtreinish: for neutron20:12
mtreinishflashgordon: if you're running in parallel (ie tenant isolation is enabled)20:12
mtreinishso it's default in the gate, but not default in the config file20:12
flashgordonhmm20:12
flashgordonwhere is the code that sets that up20:12
flashgordonbecause the test  test_create_with_existing_server_name20:12
flashgordonis triggering No network configured20:13
mtreinishflashgordon: some test classes override this though20:13
flashgordonamong others20:13
mtreinishflashgordon: you'd have to look at the code for a set_network_resources call in the inheritance tree for the test class20:13
dkranzmtreinish: with regard to firewall, it is obviously not widely known that fwaas is in disrepute. gmann gave a +2 to https://review.openstack.org/#/c/126884/20:14
mtreinishdkranz: yeah, we probably should revisit it again20:14
mtreinishI'll circle back with mestery and salv-orlando about what we should do with regard to testing the advanced services20:15
mtreinishand put it on the meeting agenda20:15
dkranzmtreinish: thanks.20:15
dkranzmtreinish: Especially because how we test them is not the same as the issue of saying they are "not stable enough to use"20:15
mesterymtreinish: Testing of advanced services will be taking a slight twist once we spin them off next week into their own repositories.20:16
mesterymtreinish: But I suspect the reference wasn't about that?20:16
mtreinishmestery: sort of I was saying it had been the policy to not add tempest tests for them because of stability reasons20:17
flashgordonmtreinish: hmm what happens if force_tenant_isolation = False?20:17
mesterymtreinish: Yes, that's still the case. Hoping the split lets each services team focus on hardening their existing tests. :)20:17
mesterymtreinish: Lets see20:17
mtreinishflashgordon: force_tenant isolation is unrelated, it's just a shortcut to avoid code duplication20:17
mtreinishflashgordon: it's false everywhere except for things like quota tests where tenant isolation is needed20:18
mtreinishmestery: heh, well we can hope :)20:19
dkranzmestery: to be clear, are you saying the issue is just with insufficient tests, or that you believe the service to be not-ready-to-use?20:19
mesterydkranz: Certainly it's likely there are insufficient tests, the latter isn't necessarily the case.20:20
dkranzmestery: ok, good20:20
dkranzmestery: not that there are insufficient tests :)20:20
mesterydkranz: lol20:21
mestery:)20:21
mesterydkranz: We'll miss you at the Neutron meetup next week, it was nice having you at hte last one in Minnesota :)20:21
dkranzmestery: I would have liked to come but I was too jammed up just after Paris20:22
dkranzmestery: I spoke with Maru this morning and said I could call in if that seemed useful at some point.20:22
mesterydkranz: Yeah, no worries. We're really going to be cramming code next week, should be lots of fun!20:22
mesterydkranz: I'm going to do my best to enable remote folks.20:22
flashgordonhmm tempest is creating the network20:23
dkranzmestery: ok, cool. I thought you guys would be skiing though.20:23
mesterydkranz: Some folks may partake in a bit of that I'm sure ;)20:23
*** Masahiro has joined #openstack-qa20:23
openstackgerritMerged openstack-dev/grenade: remove sourcing of nova baremetal driver  https://review.openstack.org/13966620:24
*** Masahiro has quit IRC20:27
*** rushiagr is now known as rushiagr_away20:31
flashgordonmestery: neutron doesn't log tenant id or anything along with req-ids20:31
flashgordonthe logs are really hard to sort out without that20:32
salv-orlandomtreinish: I think our plan is still the same we agreed 3 months ago. We don't want to test them anymore in the integrated gate20:32
flashgordonmestery:20:32
flashgordonreq-c0b4dfb9-8af3-40eb-b0dd-7b576cfd1d55 AggregatesAdminTestJSON-917687995 AggregatesAdminTestJSON-39439841420:32
flashgordonvs [req-94a39f87-e470-4032-82af-9a6b429b60fa None]20:32
salv-orlando(reading scrollback in the meanwhile)20:32
mesteryflashgordon: Sorry about that, you could file a bug for that.20:33
*** ranger81 has quit IRC20:33
salv-orlandoflashgordon: I did file a bug already once. Can't remember its id or keyword20:33
salv-orlandojust file another one20:33
mesterysalv-orlando: Thanks :)20:34
* flashgordon files another bug20:34
openstackgerritAndrea Frittoli  proposed openstack/qa-specs: add bp:test-accounts-continued  https://review.openstack.org/13957820:36
ekhugen_altandreaf do you think the latest resource cleanup spec will be ready to be worked on soon?  I was holding off starting patches for it until this was approved https://review.openstack.org/#/c/115353/20:38
flashgordonmestery salv-orlando: https://bugs.launchpad.net/neutron/+bug/139978820:39
flashgordonmestery: trying to figure out why nova is logging 'No network configured' https://review.openstack.org/#/c/139712/20:39
*** ajo has quit IRC20:39
andreafekhugen_alt: I was off on holidays for a while, that's why it didn't progress - I'll try to get it approved beginning of next week20:40
flashgordonand without logging tenant info I don20:40
mesteryflashgordon: Thanks for the bug reference20:40
flashgordondon't think I can figure it out20:40
mesteryflashgordon: I think that would help you for sure20:41
ekhugen_altandreaf okay, thanks, I'll start working on it next week then20:41
openstackgerritAndrea Frittoli  proposed openstack/qa-specs: add bp:test-accounts-continued  https://review.openstack.org/13957820:42
andreafmtreinish, dkranz: new patchset on https://review.openstack.org/#/c/139578, addressing (hopefully) your comments20:43
andreafekhugen_alt: thanks20:45
*** yamamoto has joined #openstack-qa20:52
flashgordonmtreinish: http://logs.openstack.org/12/139712/1/check/check-tempest-dsvm-neutron-full/3348b00/logs/tempest.txt.gz#_2014-12-05_19_19_44_39020:53
flashgordonthat looks funny20:53
*** andreaf has quit IRC20:56
*** tkammer has quit IRC20:56
*** yamamoto has quit IRC20:57
flashgordonmtreinish: Clearing network: None, subnet: None, router: None21:00
flashgordonthat sounds bad no?21:00
openstackgerritJoe Gordon proposed openstack/tempest: Log the credentials used to clear networks  https://review.openstack.org/13973421:00
flashgordonmtreinish: any idea why that could happen?21:02
*** sabeen3 has joined #openstack-qa21:03
*** sabeen has quit IRC21:04
mtreinishflashgordon: I'll take a look after I migrate home21:06
mtreinishbut I think that's expected21:06
mtreinishit probably means that set_network_resources is being used there21:06
*** andreaf has joined #openstack-qa21:07
flashgordonahh21:07
flashgordonadded some logging to help21:07
openstackgerritEmily Hugenbruch proposed openstack-dev/devstack: Adding tempest install to devstack  https://review.openstack.org/13676721:08
openstackgerritEmily Hugenbruch proposed openstack-dev/devstack: Adding tempest install to devstack  https://review.openstack.org/13676721:09
openstackgerritEmily Hugenbruch proposed openstack-dev/devstack: Adding tempest install to devstack  https://review.openstack.org/13676721:10
*** weshay has quit IRC21:12
dpatersonmtreinish: I see your comment on the CLI spec I submitted and will checkout your patch and have updates for next qa meeting.21:12
dpatersonsdague: I think that construction the entire --help as a top down design for CLI is reasonable and will update spec for next week's qa meeting as well.21:13
dpatersonThanks for feedback.21:13
sdaguedpaterson: awesome, thanks21:14
sdagueI very much look forward to it21:14
dpatersonhave a nice weekend.21:14
sdagueI think it would be great to have a tempest cli21:14
sdagueyou too!21:14
*** nfedotov has quit IRC21:21
*** e0ne has quit IRC21:26
*** sabeen3 has quit IRC21:31
*** sabeen has joined #openstack-qa21:31
*** ajo has joined #openstack-qa21:31
*** ajo has quit IRC21:32
*** ranger81 has joined #openstack-qa21:34
mtreinishdpaterson: ok cool, thanks21:36
*** ranger81 has quit IRC21:36
*** ranger81 has joined #openstack-qa21:36
mtreinishflashgordon: yeah that log message is what happens when set_network_resources() is called in the setUpClass() to not create network resources for the test class21:37
*** boris-42 has quit IRC21:37
mtreinishflashgordon: which test are you hitting this on? I'll dig up where that's getting set21:38
sdagueneutron-full job get a lot less reliable this week?21:40
flashgordonmtreinish: don't know, the logs aren't helpful21:40
*** mriedem has quit IRC21:41
*** Longgeek has quit IRC21:42
sdaguedtroyer: https://review.openstack.org/#/c/139625/ https://review.openstack.org/#/c/139715/ might be nice to get in for the weekend. I'm about to drop off though as family calls.21:43
*** jecarey has quit IRC21:48
*** ajo has joined #openstack-qa21:54
openstackrecheckopenstack/requirements change: https://review.openstack.org/138813 failed because of: gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/132365821:59
*** sabeen has quit IRC22:07
*** Masahiro has joined #openstack-qa22:12
*** Masahiro has quit IRC22:17
*** dpaterson has quit IRC22:20
dtroyersdague: I'll have a look at those tests after a bit…think they're false results?22:23
*** shashankhegde has quit IRC22:27
*** andreaf has quit IRC22:29
sdaguethey are refreshing22:29
*** shashankhegde has joined #openstack-qa22:30
sdaguedtroyer: they have passed now22:30
*** aysyd has quit IRC22:40
mtreinishflashgordon: wait, which test are you seeing the failures on? That's what I was asking22:40
mtreinishThat log message is pretty generic I agree, it should be cleaned up. It'll also be printed a bunch of places too22:41
*** cbader has quit IRC22:45
*** gordc has quit IRC22:46
*** andreykurilin_ has joined #openstack-qa22:52
*** ranger81 has quit IRC22:54
*** ajo has quit IRC23:01
*** ranger81 has joined #openstack-qa23:08
*** ranger81 has quit IRC23:14
*** vigneshvar has quit IRC23:18
*** boris-42 has joined #openstack-qa23:21
*** rwsu has quit IRC23:24
*** mlavalle has quit IRC23:27
*** andreykurilin_ has quit IRC23:28
flashgordonI am seeing the 'Network not found' nova error23:32
flashgordonon test_create_with_existing_server_name23:33
flashgordonthat is the top level thing I am trying to figure out23:33
mtreinishflashgordon: ok, yeah the network creation bit is being disabled for that test here: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/base.py#n3923:35
mtreinishif you need to add network creation for that test class it's not difficult (just 1 line)23:36
mtreinishflashgordon: just add this line if you need networks created for the test class: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/servers/test_server_rescue.py#n3223:38
mtreinishalthough I'm not sure why it would start failing now, its kind of a binary thing either you need a network or you don't23:38
flashgordonmtreinish: oh its not failing23:45
flashgordonjust wondering if that is an expected behavior that is all23:45
flashgordonso the issue is a network isn't created and nova is throwing a warning about that23:45
mtreinishflashgordon: it's just an optimization to limit the load on neutron23:45
flashgordonbut no network is not an issue the operator really should care about in logs23:45
mtreinishwhen we had tempest configured to make a network for each new tenant back in the day things fell apart from the load23:46
mtreinishso we switched it to just do it for things that needed networking23:46
flashgordonso the test is working fine and so is nova, the logs are just a little alarmist23:46
flashgordonmtreinish: it may be time to try  breaking neutron again23:51
mtreinishflashgordon: If you want to push a patch which drops all the non-neutron (it would probably interfere with neutron specific test cases) test usages of set_network_resources() we can give a try23:53
mtreinishbut we'll have to circle back around with salv-orlando before we merge it, because he did a lot of the initial stuff around this23:54
*** yamamoto has joined #openstack-qa23:54
flashgordonmtreinish: well we can do that too, but I was thinking the opposite23:55
flashgordonmtreinish: create a network for more tests. you said it was an optimization to limit the load on neutron23:55
flashgordonwhat happens if we raise the load23:55
mtreinishturn off all networking? (removing the calls will create all the resources by default, it's a little counter intuitive)23:55
mtreinishflashgordon: yeah, that's what I was proposing23:56
*** yamamoto has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!