| *** sthaha has joined #openstack-qa | 00:09 | |
| *** nati_ueno has joined #openstack-qa | 00:12 | |
| *** nati_ueno has quit IRC | 00:17 | |
| *** nati_uen_ has joined #openstack-qa | 00:22 | |
| *** matsuhashi has joined #openstack-qa | 00:26 | |
| *** nati_ue__ has joined #openstack-qa | 00:27 | |
| *** dmorita has joined #openstack-qa | 00:28 | |
| *** nati_uen_ has quit IRC | 00:30 | |
| *** nati_ue__ has quit IRC | 00:32 | |
| *** matsuhashi has quit IRC | 00:33 | |
| *** matsuhashi has joined #openstack-qa | 00:34 | |
| *** nati_uen_ has joined #openstack-qa | 00:35 | |
| *** nati_uen_ has quit IRC | 00:39 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Ironic and baremetal use installed diskimage-builder https://review.openstack.org/92257 | 00:48 |
|---|---|---|
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: A service to install diskimage-builder https://review.openstack.org/92256 | 00:48 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Install heat-cfntools and heat-templates repos https://review.openstack.org/92258 | 00:48 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Build a custom fedora image for tempest heat-slow https://review.openstack.org/92260 | 00:48 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Utility functions for building test images https://review.openstack.org/95616 | 00:48 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 00:48 |
| *** nati_uen_ has joined #openstack-qa | 00:52 | |
| *** nati_ue__ has joined #openstack-qa | 00:54 | |
| *** nati_uen_ has quit IRC | 00:57 | |
| *** nati_ueno has joined #openstack-qa | 00:57 | |
| *** nati_ue__ has quit IRC | 00:59 | |
| *** nati_ueno has quit IRC | 00:59 | |
| *** nati_ueno has joined #openstack-qa | 01:00 | |
| *** nati_ueno has quit IRC | 01:04 | |
| openstackgerrit | A change was merged to openstack/qa-specs: Remove run_tests.sh https://review.openstack.org/94889 | 01:09 |
| *** nati_ueno has joined #openstack-qa | 01:09 | |
| *** tiantian has joined #openstack-qa | 01:09 | |
| openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/80857 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1303802 | 01:22 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/90137 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 01:28 |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Make test_drivers.py use driver name from conf https://review.openstack.org/92996 | 01:32 |
| *** matsuhashi has quit IRC | 01:33 | |
| *** nati_ueno has quit IRC | 01:33 | |
| *** matsuhashi has joined #openstack-qa | 01:34 | |
| *** suneelb has joined #openstack-qa | 01:36 | |
| *** yamahata has joined #openstack-qa | 01:37 | |
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Autoscaling trigger scale-down with template update https://review.openstack.org/44967 | 01:37 |
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Refactor not-found check for scenario cleanup https://review.openstack.org/95624 | 01:37 |
| *** oomichi has joined #openstack-qa | 01:40 | |
| *** nosnos has joined #openstack-qa | 01:44 | |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 01:55 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: Build a custom fedora image for tempest heat-slow https://review.openstack.org/92260 | 01:55 |
| *** nati_ueno has joined #openstack-qa | 02:00 | |
| *** nati_ueno has quit IRC | 02:10 | |
| *** nati_ueno has joined #openstack-qa | 02:19 | |
| *** nati_ueno has quit IRC | 02:25 | |
| openstackrecheck | openstack/nova change: https://review.openstack.org/86213 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1263417 https://bugs.launchpad.net/bugs/1286818 | 02:30 |
| *** nati_ueno has joined #openstack-qa | 02:33 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/95629 | 02:38 |
| *** suneelb has quit IRC | 02:46 | |
| openstackrecheck | Console logs not available after 13:30s for check-tempest-dsvm-neutron-heat-slow 95425,2,e3a9878 | 02:53 |
| *** GMann has quit IRC | 03:02 | |
| *** GMann_ has joined #openstack-qa | 03:02 | |
| *** coolsvap|afk is now known as coolsvap | 03:06 | |
| *** dims has quit IRC | 03:16 | |
| openstackrecheck | Console logs not available after 13:25s for gate-python-novaclient-python26 95436,2,03ebd4a | 03:21 |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/93446 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1226412 | 03:22 |
| *** nosnos has quit IRC | 03:25 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-dev/devstack: Remove Fedora/libvirt workaround https://review.openstack.org/95635 | 03:59 |
| openstackgerrit | Steve Baker proposed a change to openstack-infra/devstack-gate: Enable dib service by default https://review.openstack.org/95636 | 04:01 |
| openstackgerrit | Steve Baker proposed a change to openstack-dev/devstack: WIP enable building test image https://review.openstack.org/95617 | 04:03 |
| *** psharma has joined #openstack-qa | 04:12 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/qa-specs: Add qa spec for bp/trove-api-test-coverage-tempest https://review.openstack.org/95639 | 04:17 |
| *** Madan has joined #openstack-qa | 04:20 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/qa-specs: Add qa spec for bp/trove-api-test-coverage-tempest https://review.openstack.org/95639 | 04:25 |
| SlickNik | GMann_: around? | 04:27 |
| openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Verify "get a server" API response attributes https://review.openstack.org/80175 | 04:28 |
| *** nati_ueno has quit IRC | 04:31 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/qa-specs: Add qa spec for bp/trove-api-test-coverage-tempest https://review.openstack.org/95639 | 04:31 |
| *** nati_ueno has joined #openstack-qa | 04:32 | |
| *** TravT has joined #openstack-qa | 04:33 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/qa-specs: Add qa spec for bp/trove-api-test-coverage-tempest https://review.openstack.org/95639 | 04:35 |
| *** suneelb has joined #openstack-qa | 04:43 | |
| *** suneelb has quit IRC | 04:45 | |
| *** nosnos has joined #openstack-qa | 04:46 | |
| *** sthaha has quit IRC | 04:46 | |
| *** sthaha has joined #openstack-qa | 04:46 | |
| *** ashishsnigam has joined #openstack-qa | 04:51 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/qa-specs: Add qa spec for bp/trove-api-test-coverage-tempest https://review.openstack.org/95639 | 04:53 |
| *** coolsvap is now known as coolsvap|afk | 04:54 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: validate server action 'console_output' attributes https://review.openstack.org/84651 | 05:01 |
| *** Madan has quit IRC | 05:05 | |
| *** vkdrao has joined #openstack-qa | 05:12 | |
| *** Madan has joined #openstack-qa | 05:21 | |
| stevebaker | Could I please get some reviews on this tempest change? https://review.openstack.org/#/c/88137/13 it is wedging progress on my building images in devstack changes | 05:25 |
| *** psharma has quit IRC | 05:27 | |
| *** psharma has joined #openstack-qa | 05:29 | |
| *** TravT has quit IRC | 05:33 | |
| *** ashishsnigam has quit IRC | 05:36 | |
| *** SridharG has joined #openstack-qa | 05:37 | |
| openstackgerrit | Pranav Salunke proposed a change to openstack/tempest: Renames Variables, fixs code to follow PEP8. https://review.openstack.org/95358 | 05:48 |
| *** yfried__ has joined #openstack-qa | 05:49 | |
| openstackgerrit | Pranav Salunke proposed a change to openstack/tempest: Renames Variables, fixs code to follow PEP8 https://review.openstack.org/95358 | 05:49 |
| *** coolsvap|afk is now known as coolsvap | 05:50 | |
| *** tkammer has joined #openstack-qa | 05:51 | |
| openstackrecheck | openstack/keystone change: https://review.openstack.org/90137 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566 | 05:57 |
| *** yfried__ has quit IRC | 06:01 | |
| *** yfried__ has joined #openstack-qa | 06:01 | |
| *** yfried__ has quit IRC | 06:02 | |
| *** yfried__ has joined #openstack-qa | 06:03 | |
| *** bkopilov has joined #openstack-qa | 06:04 | |
| *** yfried__ has quit IRC | 06:04 | |
| *** yfried__ has joined #openstack-qa | 06:04 | |
| *** yfried__ has quit IRC | 06:04 | |
| *** yfried has joined #openstack-qa | 06:05 | |
| *** tiantian has quit IRC | 06:06 | |
| *** tiantian has joined #openstack-qa | 06:09 | |
| *** yfried has quit IRC | 06:11 | |
| openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Add Ironic GET validate-node API test https://review.openstack.org/90282 | 06:11 |
| bkopilov | Hi | 06:21 |
| bkopilov | i am getting errors related to tempest.api.compute.images.test_image_metadata.ImagesMetadataTestXML.create_test_server_group (from nosetests) | 06:22 |
| bkopilov | dmorita, Hi | 06:23 |
| *** evgenyf has joined #openstack-qa | 06:34 | |
| *** ashishsnigam has joined #openstack-qa | 06:36 | |
| openstackgerrit | Marc Koderer proposed a change to openstack/qa-specs: Add qa spec for bp/autogen-negative-tests https://review.openstack.org/95658 | 06:55 |
| *** boris-42 has quit IRC | 07:03 | |
| *** afazekas has joined #openstack-qa | 07:08 | |
| *** boris-42 has joined #openstack-qa | 07:11 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack/tempest: Use old dict generator syntax in baremetal/test_ports https://review.openstack.org/95132 | 07:15 |
| *** salv-orlando has joined #openstack-qa | 07:17 | |
| *** nati_ueno has quit IRC | 07:18 | |
| *** nati_ueno has joined #openstack-qa | 07:19 | |
| *** markmcclain has joined #openstack-qa | 07:23 | |
| *** nati_ueno has quit IRC | 07:23 | |
| *** Albert_13 has joined #openstack-qa | 07:24 | |
| openstackgerrit | Ravikumar Venkatesan proposed a change to openstack/tempest: Test script for CRUD operations on regions https://review.openstack.org/93347 | 07:27 |
| *** gszasz has joined #openstack-qa | 07:30 | |
| *** yrabl has joined #openstack-qa | 07:31 | |
| *** jlibosva has joined #openstack-qa | 07:31 | |
| *** markmcclain has quit IRC | 07:35 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/95629 | 07:44 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Sharing codes for cinder v1 and v2 tests https://review.openstack.org/95629 | 07:45 |
| *** amotoki has joined #openstack-qa | 07:49 | |
| *** nati_ueno has joined #openstack-qa | 07:50 | |
| *** nati_ueno has quit IRC | 07:54 | |
| *** safchain has joined #openstack-qa | 07:54 | |
| *** jordanP has joined #openstack-qa | 07:56 | |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/tempest: Test IPV6 Subnet creation with Gateway-IP as LLA https://review.openstack.org/93483 | 08:01 |
| *** jpich has joined #openstack-qa | 08:02 | |
| *** ashishsnigam has quit IRC | 08:07 | |
| *** ashishsnigam has joined #openstack-qa | 08:07 | |
| *** cdent has joined #openstack-qa | 08:09 | |
| *** Ju has joined #openstack-qa | 08:19 | |
| *** qba73 has joined #openstack-qa | 08:27 | |
| *** coolsvap is now known as coolsvap|afk | 08:34 | |
| *** tosky has joined #openstack-qa | 08:35 | |
| *** jamielennox is now known as jamielennox|away | 08:40 | |
| *** coolsvap|afk is now known as coolsvap | 08:45 | |
| *** nosnos has quit IRC | 08:48 | |
| *** nosnos has joined #openstack-qa | 08:48 | |
| *** nati_ueno has joined #openstack-qa | 08:50 | |
| *** nosnos has quit IRC | 08:53 | |
| *** dmorita has quit IRC | 08:54 | |
| *** nati_ueno has quit IRC | 08:55 | |
| openstackgerrit | Andrey Pavlov proposed a change to openstack/tempest: remove skip attributes for fixed bug https://review.openstack.org/94571 | 08:57 |
| *** andreaf has joined #openstack-qa | 08:59 | |
| *** GMann has joined #openstack-qa | 09:01 | |
| *** GMann_ has quit IRC | 09:03 | |
| *** amotoki has quit IRC | 09:22 | |
| *** evgenyf has quit IRC | 09:23 | |
| *** rdekel has joined #openstack-qa | 09:24 | |
| *** k4n0 has joined #openstack-qa | 09:25 | |
| *** ashishsnigam has quit IRC | 09:26 | |
| *** rdekel has quit IRC | 09:33 | |
| *** nosnos has joined #openstack-qa | 09:35 | |
| *** salv-orlando_ has joined #openstack-qa | 09:35 | |
| *** salv-orlando has quit IRC | 09:37 | |
| *** salv-orlando_ is now known as salv-orlando | 09:37 | |
| openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Implement Ceph backend for Glance / Cinder / Nova https://review.openstack.org/65113 | 09:40 |
| *** Madan has quit IRC | 09:41 | |
| *** Madan has joined #openstack-qa | 09:46 | |
| *** evgenyf has joined #openstack-qa | 09:49 | |
| *** nati_ueno has joined #openstack-qa | 09:51 | |
| *** nati_ueno has quit IRC | 09:55 | |
| *** matsuhashi has quit IRC | 09:58 | |
| *** mkollaro has joined #openstack-qa | 09:59 | |
| *** matsuhashi has joined #openstack-qa | 10:00 | |
| *** coolsvap is now known as coolsvap|afk | 10:00 | |
| *** matsuhashi has quit IRC | 10:07 | |
| *** nosnos has quit IRC | 10:07 | |
| *** matsuhashi has joined #openstack-qa | 10:07 | |
| *** matsuhashi has quit IRC | 10:12 | |
| *** mkollaro has quit IRC | 10:14 | |
| openstackgerrit | Haiyang DING proposed a change to openstack/tempest: Add veritification to name property in cinder volume. https://review.openstack.org/95714 | 10:27 |
| *** salv-orlando has quit IRC | 10:29 | |
| *** vkdrao has quit IRC | 10:29 | |
| *** yfried has joined #openstack-qa | 10:37 | |
| *** yfried has quit IRC | 10:42 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Adds test for list/get volume attachments V2 APIs https://review.openstack.org/87191 | 10:47 |
| *** SridharG has quit IRC | 10:51 | |
| *** nati_ueno has joined #openstack-qa | 10:52 | |
| *** nati_ueno has quit IRC | 10:56 | |
| *** qba73 has quit IRC | 11:04 | |
| *** sthaha has quit IRC | 11:19 | |
| *** mkollaro has joined #openstack-qa | 11:21 | |
| *** dims has joined #openstack-qa | 11:28 | |
| *** safchain has quit IRC | 11:31 | |
| *** sthaha has joined #openstack-qa | 11:32 | |
| *** gszasz has quit IRC | 11:32 | |
| *** gszasz has joined #openstack-qa | 11:33 | |
| *** tiantian has quit IRC | 11:34 | |
| *** safchain has joined #openstack-qa | 11:36 | |
| *** yamahata has quit IRC | 11:36 | |
| *** gszasz has quit IRC | 11:37 | |
| *** gszasz has joined #openstack-qa | 11:37 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check create/get/delete V2 server_group attributes https://review.openstack.org/95064 | 11:42 |
| *** salv-orlando has joined #openstack-qa | 11:49 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify V2 list server_group API attributes https://review.openstack.org/95065 | 11:51 |
| *** gszasz has quit IRC | 11:53 | |
| *** nati_uen_ has joined #openstack-qa | 11:55 | |
| *** gszasz has joined #openstack-qa | 11:57 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify V2 list server_group API attributes https://review.openstack.org/95065 | 11:57 |
| *** yfried has joined #openstack-qa | 11:57 | |
| *** weshay has joined #openstack-qa | 11:58 | |
| *** nati_uen_ has quit IRC | 11:59 | |
| *** SridharG has joined #openstack-qa | 12:03 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Modify horizon log copy for Fedora https://review.openstack.org/93251 | 12:03 |
| *** jhenner has quit IRC | 12:05 | |
| *** jhenner has joined #openstack-qa | 12:05 | |
| *** mkollaro has quit IRC | 12:05 | |
| *** mkollaro1 has joined #openstack-qa | 12:05 | |
| *** afazekas has quit IRC | 12:07 | |
| *** salv-orlando has quit IRC | 12:07 | |
| *** mwagner_lap has quit IRC | 12:08 | |
| *** dims has quit IRC | 12:20 | |
| *** afazekas has joined #openstack-qa | 12:21 | |
| NithyaG | afazekas: ping | 12:23 |
| NithyaG | afazekas: Could you please review https://review.openstack.org/#/c/81834/ ? | 12:24 |
| NithyaG | andreaf: Could you also have a look at it? https://review.openstack.org/#/c/81834/ | 12:24 |
| andreaf | NithyaG: I will review it today | 12:24 |
| *** salv-orlando has joined #openstack-qa | 12:27 | |
| *** gszasz has quit IRC | 12:28 | |
| SridharG | sdague: Good morning | 12:29 |
| *** yfried_ has joined #openstack-qa | 12:29 | |
| sdague | SridharG: morning | 12:29 |
| SridharG | sdague: Can you please review https://review.openstack.org/#/c/93400/1 | 12:29 |
| *** yfried_ has quit IRC | 12:30 | |
| *** yfried_ has joined #openstack-qa | 12:30 | |
| *** afazekas is now known as _afazekas_mtg | 12:31 | |
| sdague | SridharG: so my concern is you are creating an arbitrary ipv6 topology which is not supported by the underlying networking, so while you've set db values, it doesn't mean that it works. | 12:32 |
| *** aysyd has joined #openstack-qa | 12:33 | |
| _afazekas_mtg | NithyaG: I will review it later today | 12:33 |
| *** yfried has quit IRC | 12:33 | |
| SridharG | sdague: as you know, the actual functionality would be verified using the scenario tests and I'm working on it along with Sean Collins. | 12:34 |
| sdague | well, thats not entirely true. Things like setting up networks should be verified in the API tests | 12:34 |
| SridharG | sdague: this is more of an API test to verify that Neutron Subnet API allows creation of Subnets with LLA as the Gateway-IP. | 12:34 |
| sdague | SridharG: but it doesn't actually verify that such things are created. When creating resources via the API tests, those resources really should be tested or testable that they work | 12:36 |
| SridharG | sdague: I agree with your statement that some of the API tests also verify the functionality and not just the public interface. Here the patch was more to fix the hardcoding for an existing tests. | 12:39 |
| *** k4n0 has quit IRC | 12:40 | |
| *** GMann has quit IRC | 12:40 | |
| openstackgerrit | Ashish Nigam proposed a change to openstack/tempest: Add basic read only tests for trove cli https://review.openstack.org/94599 | 12:41 |
| *** yamahata has joined #openstack-qa | 12:42 | |
| *** yfried_ has quit IRC | 12:42 | |
| *** dims has joined #openstack-qa | 12:45 | |
| *** yamahata has quit IRC | 12:45 | |
| SridharG | sdague: the test case creates a network, followed by subnet with the required parameters and verifies if the subnet is created with the params that were passed. IMHO, If we have to verify that Subnet is truly created we will have to spawn a VM, attach it to the network and check if the VM receives the appropriate details - but this is more like a Scenario tests. Sorry to bother you, I'm kind of new to tempest tests so any pointers in this regard | 12:45 |
| SridharG | would be very much appreciated. | 12:45 |
| *** yamahata has joined #openstack-qa | 12:46 | |
| *** eezhova19 has quit IRC | 12:47 | |
| sdague | SridharG: the expectation is that API tests can and will actually verify the resources exist. Not all our API tests do that, but that's considered a bug. | 12:48 |
| sdague | API tests that don't do that are just unit tests for the wsgi/db layer | 12:49 |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds v2 keystone service test to service v3 https://review.openstack.org/90370 | 12:49 |
| sdague | and really don't need to be in tempest at all | 12:49 |
| *** ekhugen has joined #openstack-qa | 12:49 | |
| sdague | scenario tests were really meant as more complicated flows between a set of services | 12:50 |
| *** ekhugen has quit IRC | 12:51 | |
| *** salv-orlando has quit IRC | 12:51 | |
| *** ekhugen has joined #openstack-qa | 12:52 | |
| SridharG | sdague: ok thanks for the explanation. So the existing test itself was kind of unit test and had the hardcoding which was bad. | 12:52 |
| SridharG | sdague: I will be working on scenario tests for IPv6 functionality. Thanks for the pointers. | 12:52 |
| *** jgrimm has quit IRC | 12:53 | |
| sdague | SridharG: thanks | 12:54 |
| *** nati_ueno has joined #openstack-qa | 12:55 | |
| andreaf | sdague: ping | 12:56 |
| *** gszasz has joined #openstack-qa | 12:59 | |
| *** mkollaro has joined #openstack-qa | 12:59 | |
| *** nati_ueno has quit IRC | 13:00 | |
| *** mkollaro1 has quit IRC | 13:02 | |
| *** jhenner has quit IRC | 13:02 | |
| *** yfried_ has joined #openstack-qa | 13:02 | |
| sdague | andreaf: pong | 13:03 |
| *** yfried_ has quit IRC | 13:03 | |
| *** jhenner has joined #openstack-qa | 13:03 | |
| *** yfried__ has joined #openstack-qa | 13:03 | |
| *** _afazekas_mtg has quit IRC | 13:04 | |
| *** jhenner has quit IRC | 13:04 | |
| andreaf | sdague: this heat patch seems to make the cfn_init heat test more stable (at least in the last few runs) and it adds console log for troubleshooting - I think it's a good one towards getting heat voting - https://review.openstack.org/#/c/88137 | 13:04 |
| andreaf | sdague: if you have time for review | 13:04 |
| sdague | andreaf: lgtm | 13:05 |
| sdague | approved | 13:05 |
| sdague | thanks for flagging it | 13:05 |
| andreaf | sdague: np, thank you | 13:06 |
| *** yfried__ is now known as yfried | 13:12 | |
| *** psharma has quit IRC | 13:12 | |
| *** jhenner has joined #openstack-qa | 13:13 | |
| *** oomichi has quit IRC | 13:14 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Deduplicate status call checks https://review.openstack.org/93035 | 13:15 |
| *** _afazekas_mtg has joined #openstack-qa | 13:16 | |
| *** _afazekas_mtg is now known as afazekas | 13:19 | |
| sdague | mtreinish: can you get generate_config to run in a non devstack env? | 13:19 |
| sdague | because I can't, and this is really irking me | 13:20 |
| sdague | http://paste.openstack.org/show/81692/ | 13:20 |
| mtreinish | sdague: I wasn't aware that it didn't run in a non devstack env | 13:21 |
| mtreinish | because it's part of the tox job | 13:21 |
| mtreinish | pep8 | 13:21 |
| mtreinish | sdague: oh you're running the wrong script | 13:21 |
| sdague | well, I can't seem to run it manually | 13:21 |
| mtreinish | well actually missing the env varible | 13:21 |
| mtreinish | just run the script in tools | 13:21 |
| sdague | this is the script in tools | 13:21 |
| mtreinish | ./tools/generate_sample.sh | 13:22 |
| mtreinish | not ./tools/config/generate_sample.sh | 13:22 |
| sdague | check_uptodate.sh: tempest.conf.sample is not up to date. | 13:22 |
| sdague | check_uptodate.sh: Please run /home/sdague/code/openstack/tempest/tools/config/generate_sample.sh. | 13:22 |
| sdague | ERROR: InvocationError: '/home/sdague/code/openstack/tempest/tools/config/check_uptodate.sh' | 13:22 |
| *** jaypipes has joined #openstack-qa | 13:22 | |
| *** ChanServ sets mode: +o jaypipes | 13:22 | |
| sdague | well, the tool tells you to run this | 13:22 |
| mtreinish | if you want to run that other script you need to specify MODULEPATH=tempest.common.generate_sample_tempest | 13:23 |
| sdague | ok, so if that's the case, we should not make the error say "run this script" which can't work :) | 13:23 |
| mtreinish | it's because the check script comes from oslo | 13:23 |
| mtreinish | we didn't have that problem before | 13:23 |
| sdague | ok, it's still terrible | 13:23 |
| sdague | "error, this doesn't work, do this thing" and that thing being wrong, is about the worst thing we could be doing | 13:24 |
| *** bkopilov has quit IRC | 13:24 | |
| *** mriedem has joined #openstack-qa | 13:24 | |
| *** malini_afk is now known as malini | 13:24 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: set all build_intervals to 1s https://review.openstack.org/95339 | 13:25 |
| mtreinish | sdague: I don't disagree, I'll add a patch to oslo to make the error message configurable | 13:27 |
| *** Madan has quit IRC | 13:30 | |
| *** GMann has joined #openstack-qa | 13:39 | |
| *** jchai has joined #openstack-qa | 13:40 | |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: make config generator more robust https://review.openstack.org/95770 | 13:42 |
| sdague | that's basically all changes to the wrapper | 13:42 |
| sdague | mtreinish: it seems like the right fix on the config generator is actually to make it read a dot file at the top of the project | 13:42 |
| sdague | instead of needing to put wrapper scripts everywhere | 13:43 |
| mtreinish | sdague: oh, I pushed out https://review.openstack.org/#/c/95772/ | 13:45 |
| mtreinish | but reading a config file make a bit more sense | 13:45 |
| mtreinish | instead of doing everything with env variables | 13:45 |
| sdague | yeh | 13:45 |
| sdague | also, the config generator should really be in olso.config instead of needing to be synced | 13:46 |
| *** Obulpathi has joined #openstack-qa | 13:46 | |
| *** Obulpathi has quit IRC | 13:46 | |
| openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Fix sudo_cmd_call in tempest.common.commands https://review.openstack.org/94577 | 13:47 |
| *** Obulpathi has joined #openstack-qa | 13:47 | |
| *** jgrimm has joined #openstack-qa | 13:47 | |
| mtreinish | sdague: heh, well that's not my decision | 13:47 |
| sdague | I already poked dhellmann about that one :) | 13:48 |
| *** SridharG has quit IRC | 13:49 | |
| *** SridharGaddam has joined #openstack-qa | 13:49 | |
| *** nati_ueno has joined #openstack-qa | 13:56 | |
| *** GMann_ has joined #openstack-qa | 13:59 | |
| *** SridharGaddam has quit IRC | 13:59 | |
| *** annegentle has joined #openstack-qa | 13:59 | |
| *** SridharG has joined #openstack-qa | 14:00 | |
| *** nati_ueno has quit IRC | 14:00 | |
| *** beekneemech is now known as bnemec | 14:01 | |
| *** GMann has quit IRC | 14:01 | |
| *** vrovachev has joined #openstack-qa | 14:02 | |
| afazekas | sdague: This https://bugs.launchpad.net/cirros/+bug/1312199 is not cirros specific, it affects all Linux image running with soft qemu (it is working with kvm) | 14:02 |
| openstackgerrit | Peter Stachowski proposed a change to openstack/tempest: Enhanced Trove (database) flavor API tests https://review.openstack.org/89396 | 14:05 |
| afazekas | 1. nova needs to pass 'no_timer_check=1' kernel parameter for all UAC image when the LIBVIRT_TYPE=qemu , probably all non uec image needs to be altered (grub) to use this option (devstack) | 14:05 |
| *** rbak has joined #openstack-qa | 14:07 | |
| afazekas | 1. nova needs to pass 'no_timer_check=1' kernel parameter for all UEC image when the LIBVIRT_TYPE=qemu 2. probably all non-uec image needs to be altered (grub) to use this option (devstack) , enabling nested guest support also could help | 14:07 |
| *** yfried has quit IRC | 14:09 | |
| *** cdent_ has joined #openstack-qa | 14:11 | |
| *** cdent has quit IRC | 14:14 | |
| *** cdent_ is now known as cdent | 14:14 | |
| *** corvus is now known as jeblair | 14:16 | |
| openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 14:18 |
| afazekas | sdague, mtreinish: I accidentally approved this change https://review.openstack.org/#/c/81834/ (New UI), If you do not link the code I can -2 if for while | 14:18 |
| *** GMann_ has quit IRC | 14:19 | |
| *** SridharG has quit IRC | 14:20 | |
| *** tkammer has quit IRC | 14:21 | |
| andreaf | afazekas: even though I'm totally ok with that change I think it would be fair to have another core reviewing it | 14:23 |
| sdague | afazekas: if you un +A it it wont merge | 14:23 |
| afazekas | sdague: ok, thx. as I remember long time ago someone had -2 it to prevent the merge | 14:26 |
| *** sileht has quit IRC | 14:28 | |
| *** SridharG has joined #openstack-qa | 14:28 | |
| openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 14:28 |
| *** sileht has joined #openstack-qa | 14:29 | |
| *** atiwari has joined #openstack-qa | 14:35 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for Neutron SSH EOFError bug 1323658 https://review.openstack.org/95782 | 14:35 |
| mriedem | sdague: jogo: clarkb: mtreinish: query for a neutron ssh bug that looks similar to the one we used to see all the time ^ | 14:36 |
| *** ekhugen_ has joined #openstack-qa | 14:36 | |
| openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 14:37 |
| mtreinish | mriedem: do you have a log link for a sample failure? | 14:37 |
| mriedem | http://logs.openstack.org/99/94599/4/check/check-tempest-dsvm-neutron-full/7a450db/ | 14:37 |
| *** sileht has quit IRC | 14:37 | |
| mriedem | mtreinish: the IP in that one is 172.24.4.110 | 14:38 |
| mriedem | i see it allocated in the neutron server log but nothing really else in the other logs | 14:38 |
| *** ekhugen__ has joined #openstack-qa | 14:38 | |
| *** TravT has joined #openstack-qa | 14:39 | |
| *** sileht has joined #openstack-qa | 14:39 | |
| openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 14:41 |
| mtreinish | mriedem: yeah, it's more of the logs being useless I think... | 14:41 |
| mriedem | mtreinish: this does show up in the tempest debug logs: "EOF in transport thread" | 14:41 |
| mtreinish | I'm just afraid that query is too broad like what we had before | 14:42 |
| mriedem | mtreinish: ideas on tightening it up? | 14:42 |
| mtreinish | no idea, that's why I asked for the log link :) | 14:42 |
| mtreinish | it doesn't help that the only useful info we get is from the test failure traceback | 14:43 |
| mriedem | mtreinish: yeah, there is a big network dump when it fails into the console log, but i'm not sure what to pull out of that | 14:43 |
| mriedem | was asking for help in the neutron channel this morning but everyone must be on hot dog hangovers | 14:43 |
| mtreinish | did you see this log: http://logs.openstack.org/99/94599/4/check/check-tempest-dsvm-neutron-full/7a450db/logs/screen-q-meta.txt.gz talk about useful... | 14:47 |
| mriedem | very | 14:47 |
| mriedem | maybe we should be sending paramiko's logs to a separate log that we can index? | 14:48 |
| mriedem | i guess they just go to tempest.txt now so nvm | 14:48 |
| mriedem | and they are debug level logs so we can't fingerprint on them | 14:48 |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 14:48 |
| *** sileht has quit IRC | 14:49 | |
| mtreinish | hmm, I didn't realize it successfully auth'd against the server before it failed | 14:50 |
| mtreinish | oh, nm different test running at the same time | 14:50 |
| *** sileht has joined #openstack-qa | 14:51 | |
| mriedem | yeah apparently ubuntu automated testing sees the same thing with multiple VMs running on the host at the same time https://bugs.launchpad.net/utah/+bug/1158224 | 14:52 |
| mriedem | so probably just a latent paramiko bug | 14:52 |
| mtreinish | mriedem: did you look at this?: http://logs.openstack.org/99/94599/4/check/check-tempest-dsvm-neutron-full/7a450db/logs/tempest.txt.gz?level=TRACE | 14:53 |
| mtreinish | seems like a lot of tracebacks | 14:53 |
| *** mkollaro has quit IRC | 14:53 | |
| mriedem | yeah was poking through there | 14:53 |
| mtreinish | mriedem: oh ok, I guess it could be a paramiko parallel thing | 14:54 |
| *** nati_ueno has joined #openstack-qa | 14:57 | |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 15:00 |
| *** mkollaro has joined #openstack-qa | 15:01 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/tempest: Modify BaremetalBasicOptsPXESSH.validate_driver_info() https://review.openstack.org/95789 | 15:01 |
| *** SridharG has quit IRC | 15:01 | |
| *** nati_ueno has quit IRC | 15:01 | |
| *** KurtMartin is now known as kmartin | 15:01 | |
| *** malini is now known as malini_afk | 15:02 | |
| *** mkollaro has quit IRC | 15:02 | |
| *** mkollaro1 has joined #openstack-qa | 15:02 | |
| *** malini_afk is now known as malini | 15:03 | |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 15:04 |
| *** morganfainberg_Z is now known as morganfainberg | 15:04 | |
| mriedem | mtreinish: i remember you saying that we stored stack traces as a single entry for logstash queries so you can query on multiple pieces of a traceback, | 15:04 |
| mriedem | mtreinish: but getting errors trying to query on this: message:"_check_public_network_connectivity" AND message:"EOFError" AND tags:console | 15:05 |
| mriedem | mtreinish: for this for example: http://logs.openstack.org/99/94599/4/check/check-tempest-dsvm-neutron-full/7a450db/logs/tempest.txt.gz?level=TRACE#_2014-05-27_13_33_32_051 | 15:05 |
| mtreinish | mriedem: try dropping the tags:console does that hit the tempest log? | 15:06 |
| mtreinish | it doesn't work for the console logs I think | 15:06 |
| mriedem | hmm, 0 hits when dropping tags:console | 15:06 |
| mriedem | mtreinish: is tempest.txt indexed? | 15:08 |
| mtreinish | dunno, doesn't look like it | 15:09 |
| mtreinish | clarkb, sdague: ^^^ ? | 15:09 |
| mtreinish | just doing a search for EOFError doesn't show any of the full tracebacks | 15:10 |
| NithyaG | sdague: could you also review https://review.openstack.org/#/c/81834/ along with afazekas and andreaf please? | 15:10 |
| mtreinish | but they're all from the console log | 15:10 |
| mriedem | mtreinish: not seeing tempest here http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/logstash/jenkins-log-client.yaml | 15:11 |
| mriedem | so it's not indexed i guess | 15:11 |
| mriedem | mtreinish: i can push up a patch for that | 15:11 |
| mtreinish | mriedem: go for it | 15:11 |
| openstackgerrit | afazekas proposed a change to openstack/qa-specs: bp/net-debug WIP https://review.openstack.org/95794 | 15:12 |
| *** jchai is now known as jchai_afk | 15:13 | |
| *** jchai_afk is now known as jchai | 15:14 | |
| *** bkopilov has joined #openstack-qa | 15:18 | |
| *** afazekas has quit IRC | 15:18 | |
| *** torandu has quit IRC | 15:21 | |
| *** vchernomor has joined #openstack-qa | 15:23 | |
| *** sthaha has quit IRC | 15:27 | |
| *** mrodden has quit IRC | 15:29 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: Update policy for Neutron after upgrade https://review.openstack.org/95801 | 15:29 |
| *** yrabl has quit IRC | 15:29 | |
| *** mlavalle has joined #openstack-qa | 15:31 | |
| mriedem | mtreinish: the tempest log is super chatty with the requests from the rest_client, can we drop those to debug level? http://logs.openstack.org/99/94599/4/check/check-tempest-dsvm-neutron-full/7a450db/logs/tempest.txt.gz?level=INFO | 15:32 |
| mtreinish | mriedem: that was sdague's thing. I'll defer to him on which level those should be at... | 15:32 |
| mriedem | orly | 15:32 |
| mtreinish | I remember an argument about it before, but that might have been about os-log-analyzer being bold for info | 15:33 |
| mtreinish | it was a while ago | 15:33 |
| mriedem | i seem to remember sdague being on a logs crusade :) | 15:33 |
| *** torandu has joined #openstack-qa | 15:35 | |
| *** NithyaG is now known as NithyaG_afk | 15:35 | |
| *** jgrimm has quit IRC | 15:39 | |
| *** mrodden has joined #openstack-qa | 15:41 | |
| *** jgrimm has joined #openstack-qa | 15:52 | |
| *** jhenner1 has joined #openstack-qa | 16:00 | |
| *** jhenner has quit IRC | 16:01 | |
| *** jlibosva has quit IRC | 16:02 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Log requests at debug level rather than info level https://review.openstack.org/95806 | 16:02 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Make unable to ping log messages appropriate level https://review.openstack.org/95815 | 16:05 |
| mtreinish | mriedem: ^^^ that should fix the excess tracebacks in the tempest log | 16:05 |
| *** jhenner1 has quit IRC | 16:09 | |
| *** asselin has joined #openstack-qa | 16:09 | |
| mriedem | mtreinish: LOG.WARN == LOG.warn? | 16:12 |
| *** aimon has quit IRC | 16:12 | |
| *** gszasz has quit IRC | 16:12 | |
| *** nati_ueno has joined #openstack-qa | 16:14 | |
| mtreinish | I thought it was caps, is it not? | 16:14 |
| mriedem | i've never seen LOG.WARN used | 16:14 |
| *** gszasz has joined #openstack-qa | 16:15 | |
| mtreinish | mriedem: yeah I think you're right. I was confusing it with the config option: http://paste.openstack.org/show/81721/ | 16:16 |
| mriedem | paste.o.o doesn't work, but i'll believe you :) | 16:16 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Make unable to ping log messages appropriate level https://review.openstack.org/95815 | 16:17 |
| mtreinish | mriedem: why doesn't paste.o.o work? | 16:17 |
| *** amotoki has joined #openstack-qa | 16:17 | |
| mriedem | shrug | 16:18 |
| mriedem | get a 500 when trying to view someone else's paste | 16:18 |
| *** gszasz has quit IRC | 16:18 | |
| *** catohornet has joined #openstack-qa | 16:19 | |
| openstackgerrit | Dirk Mueller proposed a change to openstack/tempest: Remove dependency on keyring https://review.openstack.org/95822 | 16:23 |
| openstackrecheck | Console logs not available after 13:29s for check-grenade-dsvm-neutron 91554,6,74dd3b1 | 16:24 |
| *** jhenner has joined #openstack-qa | 16:27 | |
| *** leseb has joined #openstack-qa | 16:31 | |
| *** Albert_13 has quit IRC | 16:34 | |
| *** mwagner_lap has joined #openstack-qa | 16:35 | |
| *** jordanP has quit IRC | 16:35 | |
| sdague | mriedem: back from bike ride, reading scrollback | 16:41 |
| sdague | the issue with tempest at debug level is that it's an extra MB compressed data per run | 16:42 |
| sdague | so like 25GB a week | 16:42 |
| sdague | that being said, it might be interesting to put debug into a circular buffer and dump it for a failed test | 16:42 |
| *** safchain has quit IRC | 16:42 | |
| *** evgenyf has quit IRC | 16:44 | |
| *** evgenyf has joined #openstack-qa | 16:44 | |
| mriedem | sdague: how is the size/compression different for debug and info? | 16:44 |
| sdague | mriedem: the old level of debug logging we had on the rest client was about equal to the rest of the logs in tempest | 16:45 |
| sdague | we were running about 2.2MB compressed | 16:45 |
| sdague | and I got it down to ~ 1 | 16:45 |
| mtreinish | sdague: level https://review.openstack.org/95806 | 16:45 |
| mtreinish | wow what happened with my paste buffer there? where did level come from? | 16:46 |
| mtreinish | but the link is at least right | 16:46 |
| *** ashishsnigam has joined #openstack-qa | 16:46 | |
| mriedem | can we do the same thing with debug logs in tempest as we did in nova? | 16:46 |
| sdague | mriedem: so if info isn't the right level for API requests in tempest, what do you think is | 16:46 |
| sdague | because I'd actually argue that that is the unit of work in tempest | 16:47 |
| mriedem | i wasn't thinking about it from that perspective | 16:47 |
| mriedem | my alternative thought was to add something to openstack-infra/config to allow configuring the log level that we index for logstash | 16:48 |
| mriedem | so for tempest it'd be WARNING and higher | 16:48 |
| mriedem | default to INFO as we have now | 16:48 |
| sdague | ok, are the tempest logs causing a logstash issue? | 16:48 |
| mriedem | sdague: tempest.txt isn't indexed | 16:48 |
| sdague | I guess I missed the beginning of this | 16:48 |
| sdague | ok | 16:49 |
| sdague | and is the concern that it can't be? | 16:49 |
| sdague | at current size | 16:49 |
| mriedem | but clarkb was worried about the size of it due to INFO | 16:49 |
| mriedem | right | 16:49 |
| *** yfried has joined #openstack-qa | 16:49 | |
| mriedem | starts here: https://review.openstack.org/#/c/95782/ | 16:49 |
| mriedem | i can get a better fingerprint if tempest log is indexed, so then i have https://review.openstack.org/#/c/95796/ | 16:49 |
| mriedem | and led to https://review.openstack.org/#/c/95806/ | 16:49 |
| *** yfried has quit IRC | 16:50 | |
| sdague | so what's the % increase in bytes and lines per run to add this in? | 16:50 |
| *** yfried has joined #openstack-qa | 16:50 | |
| sdague | because I'm surprised we'd think it's an issue | 16:50 |
| *** leseb has quit IRC | 16:50 | |
| *** yfried has quit IRC | 16:51 | |
| mriedem | i looked at one check-tempest-dsvm-neutron-full run and there were 17,880 lines for INFO tempest.common.rest_client | 16:51 |
| mriedem | i'm not sure what the size is | 16:51 |
| *** yfried has joined #openstack-qa | 16:51 | |
| mriedem | i can compare when https://review.openstack.org/#/c/95806/ is done | 16:51 |
| openstackrecheck | Console logs not available after 13:23s for gate-sahara-docs 73989,8,f24a150 | 16:52 |
| *** nati_ueno has quit IRC | 16:53 | |
| *** kgriffs is now known as kgriffs|afk | 16:54 | |
| sdague | so there are 130k lines in logstash in tempest full run from other logs | 16:54 |
| sdague | so that would be about a 15% line increase, though those lines are short, so probably 8% byte increase | 16:55 |
| *** harlowja_away is now known as harlowja_ | 16:55 | |
| *** yamahata has quit IRC | 16:55 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Consolidate _check_public_network_connectivity into NetworkScenarioTest https://review.openstack.org/95831 | 16:56 |
| sdague | so I'd like so see if clarkb thinks we can just take it as is | 16:56 |
| *** nati_ueno has joined #openstack-qa | 16:56 | |
| sdague | because I actually think seeing the client and server requests in the logstash by req id would be really useful | 16:56 |
| mriedem | fine with me | 16:57 |
| mriedem | i just want to fingerprint this mfer :) | 16:58 |
| sdague | :) | 16:58 |
| mriedem | but hey, at least i found a bunch of duplicate code to cleanup along the way | 16:58 |
| andreaf | mtreinish: ping | 16:58 |
| *** aimon has joined #openstack-qa | 16:58 | |
| *** ashishsnigam has quit IRC | 16:59 | |
| *** jlibosva has joined #openstack-qa | 17:01 | |
| *** kgriffs|afk is now known as kgriffs | 17:03 | |
| bnemec | Could somebody unabandon https://review.openstack.org/#/c/61037 so I can continue working on it (I'm not the owner of the original change)? | 17:04 |
| *** mmaglana has joined #openstack-qa | 17:04 | |
| mtreinish | andreaf: pong | 17:05 |
| mtreinish | bnemec: restored | 17:05 |
| bnemec | mtreinish: Thanks! | 17:05 |
| openstackrecheck | Console logs not available after 13:27s for gate-trove-tox-checkdeletions 92701,4,14381a0 | 17:06 |
| sdague | ok, this I think will speed up jobs a bit - https://review.openstack.org/#/c/95339/ would be nice to get in. I'll do a future refactoring to extract some of the config | 17:06 |
| mtreinish | sdague: you really want to hammer the api servers :) | 17:07 |
| andreaf | mtreinish: re the ssh auth, thanks for your review | 17:07 |
| mtreinish | andreaf: oh, sure np. It had a +2 which was my cue | 17:07 |
| andreaf | mtreinish: one big question mark here is really where do we want to write the logic | 17:08 |
| sdague | mtreinish: a 1s poll for status by 2 or 4 requests simultaneously should not be considered hammering :) | 17:08 |
| *** dims has quit IRC | 17:08 | |
| mtreinish | andreaf: I think it belongs in tempest.common. That's where we normally put this kind of stuff | 17:08 |
| sdague | our normal build throughput ends up generating 10s of requests per sec | 17:08 |
| andreaf | mtreinish: right so we want to try and enforce the same approach to ssh checks for all tests? | 17:09 |
| mtreinish | andreaf: maybe eventually, but there is definite value in it for most things | 17:09 |
| andreaf | mtreinish: ok | 17:10 |
| *** yfried has quit IRC | 17:10 | |
| mtreinish | like volume attach verification | 17:10 |
| andreaf | mtreinish: ok | 17:10 |
| andreaf | mtreinish: a 2nd thing, I created https://etherpad.openstack.org/p/juno-qa-summary | 17:11 |
| *** yfried has joined #openstack-qa | 17:11 | |
| mtreinish | yeah I saw your ping over the weekend | 17:11 |
| mtreinish | I'll try to add onto it soon | 17:12 |
| andreaf | mtreinish: it does not have a lot of content, but if you think it's useful, we could suggest everyone who was at the summit to contribute | 17:12 |
| mtreinish | andreaf: yeah summarizing things in one place is probably useful | 17:12 |
| andreaf | mtreinish: so I would link this into https://wiki.openstack.org/wiki/Summit/Juno/Etherpads | 17:12 |
| *** dims has joined #openstack-qa | 17:13 | |
| yfried | mlavalle: ping | 17:13 |
| mtreinish | yeah why not, make a new subheading under qa for it and link to it | 17:13 |
| andreaf | mtreinish: ok | 17:14 |
| *** evgenyf has quit IRC | 17:14 | |
| *** andreaf has quit IRC | 17:20 | |
| openstackgerrit | A change was merged to openstack/tempest: Make test_server_cfn_init a scenario test https://review.openstack.org/88137 | 17:22 |
| ylobankov | Hello, guys. Does anyone know IRC nickname of Ken'ichi Ohmichi? I have not found nickname here https://wiki.openstack.org/wiki/People | 17:23 |
| *** amotoki has quit IRC | 17:23 | |
| mtreinish | ylobankov: it's oomichi, but he's offline now being in UTC+9 | 17:25 |
| ylobankov | mtreinish, thanks a lot! | 17:26 |
| openstackrecheck | Console logs not available after 13:29s for check-tempest-dsvm-neutron-full 93035,4,db79527 | 17:30 |
| openstackgerrit | Pranav Salunke proposed a change to openstack/tempest: Rename variables, fixs code to follow PEP8 https://review.openstack.org/95358 | 17:31 |
| *** nati_ueno has quit IRC | 17:31 | |
| sdague | mtreinish: got a moment for a tempest conf issue | 17:38 |
| mtreinish | sure | 17:38 |
| mtreinish | sdague: did I break something? | 17:39 |
| sdague | nope | 17:39 |
| sdague | well... I don't know | 17:39 |
| sdague | I'm trying to understand the way we die on having a null config | 17:40 |
| sdague | because ideally I'd like to not depend on it in javelin | 17:40 |
| sdague | so when I created a zero config | 17:40 |
| sdague | I got a totally whacky message | 17:40 |
| mtreinish | an empty file or a nonexistent file? | 17:41 |
| sdague | empty file | 17:41 |
| sdague | for now, I can live with empty file | 17:41 |
| mtreinish | it should work with defaults, until it tries to actually do something for one of the required opts that has like an empty string default | 17:41 |
| sdague | http://paste.openstack.org/show/81739/ | 17:42 |
| mtreinish | I don't think we have a catch for the empty case | 17:42 |
| mtreinish | ahh, that stupid default | 17:42 |
| *** nati_ueno has joined #openstack-qa | 17:42 | |
| mtreinish | sdague: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n139 | 17:42 |
| mtreinish | we just need to change that to be something sane | 17:43 |
| mtreinish | I've been meaning to do that | 17:43 |
| mtreinish | because we get a bunch of emails about that from new users | 17:43 |
| sdague | mtreinish: what kind of default is it even? | 17:44 |
| sdague | because I found no reference to setting it anywhere | 17:44 |
| sdague | which is really the question | 17:44 |
| mtreinish | sdague: I was going to have no default | 17:45 |
| mtreinish | and set it to required | 17:45 |
| mtreinish | in the help | 17:45 |
| sdague | ok, which is fine, I'm just curious how the current code works at all | 17:45 |
| sdague | like what's supposed to be setting things | 17:45 |
| sdague | is that pulling from env? | 17:47 |
| sdague | or is it just completely broken | 17:47 |
| mtreinish | I think that might have been the intent when it was first added as an option. But I'm pretty sure that doesn't work | 17:47 |
| sdague | ok so if I nuke it, it's all good? | 17:48 |
| mtreinish | sdague: I've got the patch pending right now | 17:48 |
| mtreinish | 30 sec and I'll push it out | 17:48 |
| *** jlibosva has quit IRC | 17:49 | |
| sdague | ok | 17:49 |
| sdague | also, is there a way to say a thing is required to have a value in oslo land? | 17:50 |
| mtreinish | dunno that's a good question | 17:50 |
| sdague | dhellmann: ... ? | 17:51 |
| mtreinish | sdague: http://docs.openstack.org/developer/oslo.config/cfg.html#special-handling-instructions | 17:52 |
| mtreinish | but no example :) | 17:52 |
| openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron-full-icehouse 94577,2,bf08cc8 | 17:52 |
| *** davlaps has joined #openstack-qa | 17:53 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove default from image_ref options https://review.openstack.org/95843 | 17:53 |
| mtreinish | sdague: ^^^ | 17:53 |
| mtreinish | I'll figure out the required bit and add it in the next rev | 17:53 |
| mriedem | sdague: mtreinish: http://git.openstack.org/cgit/openstack/oslo.config/tree/oslo/config/cfg.py#n594 | 17:53 |
| mriedem | required=True | 17:53 |
| mriedem | yields http://git.openstack.org/cgit/openstack/oslo.config/tree/oslo/config/cfg.py#n387 if not supplied | 17:53 |
| mtreinish | mriedem: yep just found it by digging into the oslo tree | 17:54 |
| mtreinish | I guess I should push a doc fix too | 17:55 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Consolidate _check_public_network_connectivity into NetworkScenarioTest https://review.openstack.org/95831 | 17:56 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/tempest: Consolidate _check_tenant_network_connectivity in NetworkScenarioTest https://review.openstack.org/95844 | 17:56 |
| openstackrecheck | openstack/nova change: https://review.openstack.org/93827 failed gate-grenade-dsvm in the gate queue with an unrecognized error | 17:59 |
| mtreinish | sdague: although should I set them as required, because they're only needed if service_available.nova == True | 18:00 |
| mtreinish | or at least it should be that way | 18:00 |
| *** jamielennox|away is now known as jamielennox | 18:05 | |
| *** ylobankov has quit IRC | 18:09 | |
| *** malini is now known as malini_afk | 18:12 | |
| *** nati_ueno has quit IRC | 18:13 | |
| *** nati_ueno has joined #openstack-qa | 18:14 | |
| *** malini_afk is now known as malini | 18:17 | |
| sdague | mtreinish: that's another issue I'm finding with javelin, the python stack traces aren't coming out when things die | 18:19 |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/95849 | 18:19 |
| sdague | instead I get a CRITICAL logging issue | 18:19 |
| sdague | and a bad exit | 18:19 |
| sdague | http://paste.openstack.org/show/81768/ | 18:19 |
| sdague | thoughts on why that is? | 18:19 |
| *** jpich has quit IRC | 18:20 | |
| *** krtaylor has quit IRC | 18:22 | |
| mtreinish | sdague: it's something with the logger config. I saw it as a puzzle to figure out what failed when I was working working on verify_config | 18:22 |
| mtreinish | I haven't had a chance to really dig into it though | 18:22 |
| mtreinish | sdague: but, I have faith that you can figure out what failed just from the message 'id' | 18:23 |
| openstackgerrit | Zu Qiang proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy https://review.openstack.org/95849 | 18:24 |
| sdague | :P | 18:25 |
| sdague | apparently you only get the traceback on a critical if verbose=True in oslo logs | 18:27 |
| sdague | that's kind of terrible | 18:27 |
| mtreinish | well, it is more verbose with a traceback... | 18:28 |
| mlavalle | yfried: pong | 18:29 |
| openstackrecheck | Console logs not available after 15:53s for check-grenade-dsvm-neutron 95522,2,45224bd | 18:30 |
| yfried | mlavalle: hi, how was the summit session? I saw there was some extra activity on the etherpad. What decision did you reach? How do we proceed? | 18:30 |
| mlavalle | yfried: yesterday I submitted this spec for review, where I outline the process going forward during Juno: https://review.openstack.org/#/c/95600/ | 18:32 |
| mlavalle | yfried: over the next few days I will be adding the tutorial pages to the tempest docs and writing detailed specification for the scenarios we will implement this cycle | 18:33 |
| mlavalle | yfried: you are more than welcome to pitch in | 18:34 |
| yfried | how does this work? should i +1 the spec? | 18:36 |
| yfried | mlavalle: what about owners for existing scenarios? | 18:37 |
| yfried | mlavalle: in charge of maintaining the test and enhancing it's doc | 18:37 |
| yfried | its | 18:37 |
| mlavalle | yfried: that's a good point. Feel free to -1 my spec and make that comment. I will add it | 18:37 |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-dev/devstack: Make curl following redirects for downloading pip https://review.openstack.org/95854 | 18:38 |
| mlavalle | yfried: in fact, feel free to -1 the spec with all the comments you might have | 18:38 |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-dev/devstack: Make curl following redirects for pip downloading https://review.openstack.org/95854 | 18:39 |
| anteaya | mtreinish: any dates selected for the qa mid-cycle meetup? | 18:42 |
| openstackrecheck | Console logs not available after 13:26s for check-grenade-dsvm 89905,6,d5eb4c1 | 18:44 |
| yfried | mlavalle: I suggest we set some strict guidelines for documentation, as these tests are unique and not always trivial. should this be a different spec or could it be added to current one? | 18:44 |
| mlavalle | yfried: I think we can add to this one | 18:44 |
| mlavalle | yfried: the documentation guidelines can be part of what we are going to add to tempet docs | 18:45 |
| yfried | mlavalle: agreed | 18:45 |
| mlavalle | yfried: so as far as the spec, it can be part of the addition to the docs | 18:45 |
| yfried | mlavalle: also for logging. test should strive to fail with as much information as possible, since there's usually great significance to to where the scenario failed, and what was achieved prior to crash | 18:47 |
| mlavalle | yfried: you bet! | 18:47 |
| yfried | mlavalle: however, it's hard for me to define these guidelines off the top of my head. I'm open the suggestions. maybe this *should* be a sub-spec | 18:48 |
| mlavalle | yfried: most likely a part of the addition to the docs. So they can be reused in the future | 18:49 |
| yfried | mlavalle: I'm sorry, I didn't understand you last comment ^. could you please explain? | 18:51 |
| mlavalle | yfried: I think the guidelines for logging should be part of the docs as well | 18:52 |
| *** mkollaro1 has quit IRC | 18:55 | |
| openstackrecheck | Console logs not available after 13:26s for check-grenade-dsvm-neutron 95816,1,8eed07b | 18:57 |
| mlavalle | yfried: I will email you a presentation that igawa gave in Atlanta about scenario tests. He agreed that we can use it as a starting point for the addition to the docs. Do you want it in ppt or pdf format? | 18:58 |
| yfried | mlavalle: yeah, but I wonder if these guidelines should be discussed in the spec (or a different one) or simply as part of the doc review? | 18:58 |
| *** nati_ueno has quit IRC | 18:58 | |
| mlavalle | yfried: oh, I agree, let's discuss it in the spec | 18:59 |
| *** nati_ueno has joined #openstack-qa | 18:59 | |
| yfried | mlavalle: tnx. doesn't really matter, though I'm using fedora so pdf is better, I think | 18:59 |
| mlavalle | yfried: yeah, I suspected, Red Hat guy :-) | 18:59 |
| yfried | mlavalle: same spec? or maybe once this is approved, it's specifics should be discussed, as to not block the general idea from moving on | 19:00 |
| yfried | mlavalle: what's "juno-1" july? | 19:01 |
| openstackgerrit | Ivan Kolodyazhny proposed a change to openstack-dev/devstack: Make curl following redirects for pip downloading https://review.openstack.org/95854 | 19:01 |
| mlavalle | yfried: June 12th | 19:02 |
| anteaya | dtroyer: ping | 19:02 |
| anteaya | dtroyer: we are about to talk about devstack docs in the infra meeting | 19:02 |
| anteaya | in -meeting | 19:03 |
| yfried | mlavalle: why not attach this the existing bp for network scenario, rather than creating a new one? | 19:06 |
| *** rockyg has joined #openstack-qa | 19:07 | |
| mlavalle | yfried: that's probably a good idea as well….. Write all your comments in the current spec and I will create a new one with them. The spec has to have the same name as the bp | 19:08 |
| *** mkollaro has joined #openstack-qa | 19:13 | |
| openstackrecheck | Console logs not available after 13:25s for check-grenade-dsvm-neutron 95823,1,b2f5625 | 19:21 |
| openstackrecheck | openstack/python-keystoneclient change: https://review.openstack.org/91578 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262566 | 19:22 |
| *** nati_ueno has quit IRC | 19:33 | |
| *** malini is now known as malini_afk | 19:34 | |
| openstackgerrit | Ben Nemec proposed a change to openstack/tempest: Start instances using fixed network when possible https://review.openstack.org/61037 | 19:34 |
| *** mkollaro has quit IRC | 19:36 | |
| *** yfried has quit IRC | 19:37 | |
| *** dustins has joined #openstack-qa | 19:38 | |
| openstackrecheck | Console logs not available after 13:23s for check-grenade-dsvm-neutron 70630,9,677c017 | 19:44 |
| *** catohornet has quit IRC | 19:46 | |
| *** andreaf has joined #openstack-qa | 19:46 | |
| *** nati_ueno has joined #openstack-qa | 19:50 | |
| *** dims has quit IRC | 19:54 | |
| *** nati_ueno has quit IRC | 20:00 | |
| *** markmcclain has joined #openstack-qa | 20:00 | |
| *** leseb has joined #openstack-qa | 20:01 | |
| leseb | sdague: hey, it would be nice if you could revert your -2 on this patch https://review.openstack.org/#/c/65113/, I talked with Dean in Atlanta and I reworked the patch according to what he suggested. Thanks in advance. | 20:03 |
| sdague | leseb: done | 20:03 |
| leseb | sdague: wow thanks a lot for the quick reply :) | 20:04 |
| sdague | no prob, thanks for respinning. I'll try to get a real review on it later this week | 20:04 |
| sdague | but at least I 0ed out my review for now | 20:04 |
| leseb | sdague: it would be awesome, thanks for taking the time :) | 20:05 |
| *** catohornet has joined #openstack-qa | 20:05 | |
| openstackrecheck | Console logs not available after 13:29s for check-grenade-dsvm 95300,2,cbd08f6 | 20:06 |
| *** krtaylor has joined #openstack-qa | 20:07 | |
| *** nati_ueno has joined #openstack-qa | 20:12 | |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 20:17 |
| openstackrecheck | Console logs not available after 13:32s for gate-grenade-dsvm 95469,1,495053c | 20:19 |
| mestery | mriedem: Regarding https://bugs.launchpad.net/neutron/+bug/1323658, how can we verify if this may be a paramiko problem, per your last comment? Any ideas? | 20:20 |
| mriedem | mestery: not sure | 20:20 |
| mestery | mriedem: OK, was just curious. :) | 20:21 |
| mriedem | mestery: haven't checked what versions we're running with compared to what's available either | 20:21 |
| mestery | mriedem: Neither have I, in fact. | 20:21 |
| *** boris-42 has quit IRC | 20:21 | |
| mriedem | mestery: there is this though: https://review.openstack.org/#/c/81132/ | 20:21 |
| * mestery looks. | 20:21 | |
| *** boris-42 has joined #openstack-qa | 20:22 | |
| mriedem | mestery: | 20:22 |
| mriedem | http://logs.openstack.org/09/89409/8/check/check-grenade-dsvm-neutron/a579837/logs/pip-freeze.txt.gz | 20:22 |
| mriedem | 1.14.0 | 20:22 |
| mriedem | so latest | 20:22 |
| mestery | mriedem: OK, worth a shot at least, thanks! | 20:22 |
| *** afazekas has joined #openstack-qa | 20:25 | |
| sdague | mriedem: fwiw: DEBUG paramiko.transport [-] EOF in transport thread | 20:25 |
| sdague | is normal behavior | 20:26 |
| sdague | it logs a message when you kill the socket | 20:26 |
| sdague | for reasons of complete confusion | 20:26 |
| *** dims has joined #openstack-qa | 20:26 | |
| sdague | dtroyer: if you can get to the ceph patch before you head out on vacation, that would be cool. Then I can have a look later this week | 20:29 |
| sdague | maybe when heading down to grok ipv6 things with sc68cal on friday | 20:29 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack-infra/devstack-gate: Allow to configure git base URL https://review.openstack.org/95901 | 20:30 |
| *** mlavalle has quit IRC | 20:32 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack-infra/devstack-gate: Allow to configure git base URL https://review.openstack.org/95901 | 20:39 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack-infra/devstack-gate: Make the master branch configurable https://review.openstack.org/95904 | 20:45 |
| *** mmaglana has quit IRC | 20:56 | |
| openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-neutron 91578,15,4df2bdb | 21:00 |
| *** leseb has quit IRC | 21:01 | |
| mtreinish | sdague: https://review.openstack.org/#/c/93035/ look at the jenkins results from rev 4. It has import errors but only on 2 of the 5 grenade jobs, everything else is fine. Any ideas what could be up there? | 21:03 |
| sdague | mtreinish: I think this waas hp cloud region being terrible? | 21:04 |
| sdague | nope, it's on az2 not b | 21:05 |
| mtreinish | sdague: oh, way to rain on my parade. I was expecting something exotic to track down :( | 21:05 |
| sdague | oh, so this is the thing that someone broke set -o errexit | 21:06 |
| sdague | it actually died during tempest old | 21:06 |
| sdague | but tried to run tempest new | 21:06 |
| sdague | which it shouldn't have | 21:06 |
| mtreinish | oh, hmm thats interesting | 21:06 |
| sdague | it's actually a devstack gate bug | 21:06 |
| sdague | I started noticing a couple of weeks ago | 21:07 |
| *** nati_ueno has quit IRC | 21:08 | |
| mtreinish | sdague: is there a bug number? | 21:09 |
| sdague | nope | 21:09 |
| mtreinish | then I'll just recheck no bug it... | 21:10 |
| sdague | so it was a real fail | 21:10 |
| sdague | just not the import failure | 21:11 |
| sdague | it failed for reals | 21:11 |
| *** nati_ueno has joined #openstack-qa | 21:11 | |
| mtreinish | that was a hint for you to open a bug... | 21:11 |
| sdague | but that's not why it failed | 21:11 |
| sdague | the import error wouldn't be the recheck bug | 21:11 |
| mtreinish | sdague: oh I misread there is a tempest old failure | 21:12 |
| *** aysyd has quit IRC | 21:12 | |
| *** dims has quit IRC | 21:13 | |
| mtreinish | 500 from xml create sec group | 21:13 |
| mtreinish | I see that now | 21:13 |
| openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-full 94014,5,4dd3c61 | 21:13 |
| mtreinish | ooh: http://logs.openstack.org/35/93035/4/check/check-grenade-dsvm/1aef614/logs/old/screen-n-api.txt.gz?level=TRACE#_2014-05-27_13_38_18_086 | 21:13 |
| sdague | yep, that's a real fail | 21:14 |
| cdent | sdague (if you're the right guy to ask): i've got a simple patch for devstack, cool to just commit it for review, or is better to make a bug and then refer to that in the patch? | 21:15 |
| *** dustins has quit IRC | 21:15 | |
| *** rockyg has quit IRC | 21:15 | |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: chown stack/data/nova/networks to $STACK_USER https://review.openstack.org/95914 | 21:19 |
| * cdent goes ahead, is nearly bedtimes | 21:20 | |
| * cdent waves | 21:20 | |
| *** cdent has quit IRC | 21:20 | |
| *** markmcclain has quit IRC | 21:21 | |
| mtreinish | sdague: the fix was already in progress for that: https://review.openstack.org/#/c/92488/ | 21:25 |
| mtreinish | I'll push out an e-r query to get it off the list | 21:25 |
| sdague | mtreinish: cool | 21:25 |
| *** catohornet has quit IRC | 21:26 | |
| *** Obulpathi has quit IRC | 21:26 | |
| openstackrecheck | Console logs not available after 13:28s for check-tempest-dsvm-neutron-heat-slow 94294,4,bee5cc9 | 21:27 |
| *** Obulpathi has joined #openstack-qa | 21:27 | |
| *** Obulpathi has quit IRC | 21:32 | |
| *** mlavalle has joined #openstack-qa | 21:32 | |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/tempest: Add compute notifications tests for ceilometer https://review.openstack.org/64136 | 21:33 |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Add query for bug 1308715 https://review.openstack.org/95918 | 21:33 |
| mtreinish | sdague: ^^^ | 21:33 |
| -openstackstatus- NOTICE: Zuul is offline due to an operational issue; ETA 2200 UTC. | 21:33 | |
| *** ChanServ changes topic to "Zuul is offline due to an operational issue; ETA 2200 UTC." | 21:33 | |
| *** nati_ueno has quit IRC | 21:37 | |
| *** davidlenwell_ has joined #openstack-qa | 21:40 | |
| openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-full 94663,4,54eee11 | 21:41 |
| *** nati_ueno has joined #openstack-qa | 21:43 | |
| *** davidlenwell_ has quit IRC | 21:44 | |
| *** davidlenwell has quit IRC | 21:45 | |
| *** dims has joined #openstack-qa | 21:45 | |
| *** mriedem has quit IRC | 21:46 | |
| *** davidlenwell has joined #openstack-qa | 21:47 | |
| *** mwagner_lap has quit IRC | 21:52 | |
| openstackrecheck | Console logs not available after 14:04s for check-grenade-dsvm-neutron 95840,2,18e0272 | 21:55 |
| *** ekhugen has quit IRC | 21:56 | |
| *** ekhugen__ has quit IRC | 21:56 | |
| *** ekhugen_ has quit IRC | 21:56 | |
| *** harlowja_ is now known as harlowja_away | 21:57 | |
| *** jchai has quit IRC | 21:57 | |
| *** harlowja_away is now known as harlowja_ | 21:57 | |
| openstackgerrit | Julien Vey proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/95778 | 22:03 |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/grenade: don't corrupt grenade exit value https://review.openstack.org/95934 | 22:04 |
| sdague | mtreinish: ^^ | 22:04 |
| sdague | I think that's where we went wrong | 22:04 |
| *** gabriel-bezerra has joined #openstack-qa | 22:05 | |
| *** nati_ueno has quit IRC | 22:05 | |
| gabriel-bezerra | Hi guys. Is there a way in DevStack's scripts to get the Apache or Ubuntu version that it is running on? | 22:06 |
| *** yamahata has joined #openstack-qa | 22:07 | |
| gabriel-bezerra | I'd like to check whether devstack is running on Ubuntu with Apache 2.2 or Apache 2.4, so I can configure the scripts with the right names. | 22:07 |
| openstackrecheck | Console logs not available after 13:26s for check-tempest-dsvm-neutron-full 90370,7,730378f | 22:08 |
| *** ChanServ changes topic to "All Things OpenStack QA" | 22:10 | |
| -openstackstatus- NOTICE: Zuul is started and processing changes that were in the queue when it was stopped. Changes uploaded or approved since then will need to be re-approved or rechecked. | 22:10 | |
| *** mmaglana has joined #openstack-qa | 22:14 | |
| *** lbragstad has joined #openstack-qa | 22:16 | |
| lbragstad | quick question on api validation and adding test cases to tempest. In keystone, we're looking to add a validation layer that will validate user request on resource operations. Our end-goal is to do this on all Keystone resources and have it enforce the existing identity api spec (currently, some resource have their own validation, we want to use one implementation consistently in Keystone). | 22:19 |
| lbragstad | So when adding test cases to tempest, would anyone prefer them added in a certain way? Since we will be adding tests for all the keystone resources including different permutations of invalid requests. | 22:20 |
| lbragstad | morganfainberg: ^ | 22:20 |
| mtreinish | lbragstad: how is the api spec being defined? Like with jsonschema? | 22:21 |
| mtreinish | lbragstad: before for the invalid requests using the negative test autogeneration is prefered | 22:21 |
| mtreinish | the other, passing test cases are fair game | 22:22 |
| lbragstad | mtreinish: yes, the validator would be using jsonschema | 22:22 |
| openstackrecheck | Console logs not available after 13:48s for check-tempest-dsvm-postgres-full 94915,7,171b7ec | 22:22 |
| mtreinish | ok yeah then take a look at what we do for nova with autogeneration here: http://git.openstack.org/cgit/openstack/tempest/tree/etc/schemas/compute/flavors/flavors_list.json and http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/flavors/test_flavors_negative.py | 22:24 |
| lbragstad | mtreinish: perfect, thanks | 22:25 |
| morganfainberg | mtreinish, woot thanks! | 22:25 |
| mtreinish | lbragstad, morganfainberg: I'd talk to mkoderer and oomichi about jsonschema in tempest, because they're the ones who have pushing that | 22:26 |
| morganfainberg | mtreinish, ack, thanks again | 22:26 |
| lbragstad | mtreinish: yeah, I saw that they were the ones doing a lot of the work with the nova v3 api | 22:26 |
| mtreinish | for example the nova_v3 tests also have jsonschema response format validation built in | 22:26 |
| *** jgrimm has quit IRC | 22:27 | |
| *** nati_ueno has joined #openstack-qa | 22:32 | |
| *** kgriffs is now known as kgriffs|afk | 22:35 | |
| openstackrecheck | Console logs not available after 13:27s for check-tempest-dsvm-postgres-full 95862,1,fa9b3ca | 22:37 |
| *** Sukhdev has joined #openstack-qa | 22:38 | |
| *** nati_uen_ has joined #openstack-qa | 22:42 | |
| *** nati_ueno has quit IRC | 22:42 | |
| openstackrecheck | Console logs not available after 13:42s for check-tempest-dsvm-neutron-full 95844,1,f311854 | 22:51 |
| *** atiwari has quit IRC | 22:54 | |
| *** andreaf has quit IRC | 22:59 | |
| *** rbak has quit IRC | 23:04 | |
| *** mwagner_lap has joined #openstack-qa | 23:08 | |
| *** yamahata has quit IRC | 23:16 | |
| *** davlaps has quit IRC | 23:33 | |
| *** TravT has quit IRC | 23:35 | |
| *** fbo has quit IRC | 23:44 | |
| *** tosky has quit IRC | 23:51 | |
| *** aimon has quit IRC | 23:51 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!