*** mahito has joined #openstack-qa | 00:00 | |
*** achanda has quit IRC | 00:00 | |
*** salv-orl_ has quit IRC | 00:06 | |
*** dmorita has joined #openstack-qa | 00:24 | |
*** yamamoto has joined #openstack-qa | 00:49 | |
*** jhesketh has quit IRC | 00:50 | |
*** jhesketh has joined #openstack-qa | 00:53 | |
*** achanda has joined #openstack-qa | 01:01 | |
*** achanda has quit IRC | 01:06 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for horizon integration test bug 1436903 https://review.openstack.org/168753 | 01:07 |
---|---|---|
openstack | bug 1436903 in OpenStack Dashboard (Horizon) "integration tests failing blocking gate" [Critical,Triaged] https://launchpad.net/bugs/1436903 - Assigned to David Lyle (david-lyle) | 01:07 |
*** salv-orlando has joined #openstack-qa | 01:07 | |
*** salv-orlando has quit IRC | 01:12 | |
*** Longgeek has joined #openstack-qa | 01:15 | |
*** yamamoto has quit IRC | 01:15 | |
openstackgerrit | Merged openstack/tempest: Handle fixed_network edge cases gracefully https://review.openstack.org/168353 | 01:18 |
lifeless | sdague: 2614 | 01:24 |
lifeless | bah | 01:24 |
lifeless | sdague: https://github.com/pypa/pip/issues/2612 | 01:24 |
openstackgerrit | Merged openstack/tempest: Remove hyphen from rand_name calls in identity tests https://review.openstack.org/166676 | 01:24 |
lifeless | sdague: if you have the time to put a reproducer together that would be helpful | 01:25 |
openstackgerrit | Ghanshyam Mann proposed openstack-dev/devstack: Add Nova v2 API option https://review.openstack.org/163718 | 01:27 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove hyphen from rand_name calls in network tests https://review.openstack.org/166669 | 01:28 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Add a rule for blocking "-" from rand_name call https://review.openstack.org/166679 | 01:28 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: Remove hyphen from rand_name calls in scenario tests https://review.openstack.org/166678 | 01:28 |
*** annashen has joined #openstack-qa | 01:37 | |
*** annashen has quit IRC | 01:41 | |
*** yamamoto has joined #openstack-qa | 01:46 | |
*** mhorban has quit IRC | 01:49 | |
*** mhorban has joined #openstack-qa | 01:49 | |
*** salv-orlando has joined #openstack-qa | 01:53 | |
*** salv-orlando has quit IRC | 01:58 | |
*** sputnik13 has quit IRC | 02:05 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/tempest: WIP: Add hacking rule for "GET /resources" https://review.openstack.org/168762 | 02:09 |
*** salv-orlando has joined #openstack-qa | 02:10 | |
*** salv-orlando has quit IRC | 02:12 | |
*** Longgeek has quit IRC | 02:12 | |
*** Longgeek has joined #openstack-qa | 02:16 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/qa-specs: Add consistent-service-method-names spec https://review.openstack.org/159310 | 02:25 |
*** Longgeek has quit IRC | 02:26 | |
*** Longgeek has joined #openstack-qa | 02:26 | |
*** mriedem1 has joined #openstack-qa | 02:37 | |
*** annashen has joined #openstack-qa | 02:38 | |
*** mriedem has quit IRC | 02:39 | |
openstackgerrit | Merged openstack/tempest: Update description of Enable block_migrate_cinder_iscsi https://review.openstack.org/165182 | 02:41 |
*** annashen has quit IRC | 02:42 | |
*** sputnik13 has joined #openstack-qa | 02:51 | |
*** david-ly_ is now known as david-lyle | 02:52 | |
*** chandankumar has joined #openstack-qa | 02:58 | |
*** mriedem1 has quit IRC | 02:58 | |
*** dims has quit IRC | 02:59 | |
*** dims has joined #openstack-qa | 02:59 | |
*** dims has quit IRC | 03:00 | |
*** haigang has joined #openstack-qa | 03:04 | |
*** Longgeek_ has joined #openstack-qa | 03:07 | |
*** Longgeek has quit IRC | 03:10 | |
*** sabeen has joined #openstack-qa | 03:11 | |
*** chandankumar has quit IRC | 03:13 | |
*** ajmiller has quit IRC | 03:13 | |
*** sabeen has quit IRC | 03:14 | |
openstackgerrit | Merged openstack/tempest: Add test for the various config cases of is_admin_available() https://review.openstack.org/164225 | 03:18 |
*** salv-orlando has joined #openstack-qa | 03:25 | |
*** salv-orlando has quit IRC | 03:30 | |
openstackgerrit | Yuki Nishiwaki proposed openstack-dev/devstack: Restart rabbitmq if rabbitmq-server isn’t running https://review.openstack.org/168705 | 03:33 |
*** Madan has joined #openstack-qa | 03:33 | |
*** dpaterson has quit IRC | 03:46 | |
*** sabeen1 has joined #openstack-qa | 03:58 | |
*** sabeen1 has quit IRC | 04:05 | |
openstackgerrit | Jamie Lennox proposed openstack-dev/devstack: Remove keystone public/admin_endpoint options https://review.openstack.org/168770 | 04:13 |
*** dims has joined #openstack-qa | 04:15 | |
*** woodster_ has quit IRC | 04:20 | |
*** dims has quit IRC | 04:21 | |
*** pal_ has joined #openstack-qa | 04:22 | |
*** achanda has joined #openstack-qa | 04:24 | |
openstackgerrit | Dean Troyer proposed openstack-dev/devstack: Add inc/rootwrap https://review.openstack.org/168773 | 04:25 |
*** jecarey has quit IRC | 04:27 | |
*** chandankumar has joined #openstack-qa | 04:27 | |
*** annashen has joined #openstack-qa | 04:39 | |
*** annashen has quit IRC | 04:44 | |
*** pal__ has joined #openstack-qa | 04:53 | |
*** pal_ has quit IRC | 04:56 | |
*** BharatK has joined #openstack-qa | 04:57 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/168568 failed check-heat-dsvm-functional-mysql in the gate queue with an unrecognized error | 05:09 |
*** pal__ has quit IRC | 05:10 | |
*** pal_ has joined #openstack-qa | 05:12 | |
*** k4n0 has joined #openstack-qa | 05:12 | |
*** mrodden has quit IRC | 05:17 | |
*** achanda has quit IRC | 05:22 | |
*** achanda_ has joined #openstack-qa | 05:22 | |
*** jlibosva has joined #openstack-qa | 05:25 | |
*** achanda has joined #openstack-qa | 05:29 | |
*** achanda_ has quit IRC | 05:30 | |
*** mrodden has joined #openstack-qa | 05:32 | |
*** pal__ has joined #openstack-qa | 05:32 | |
*** pal_ has quit IRC | 05:34 | |
*** woodster_ has joined #openstack-qa | 05:37 | |
*** pal_ has joined #openstack-qa | 06:02 | |
*** pal__ has quit IRC | 06:05 | |
*** neeti has joined #openstack-qa | 06:06 | |
*** e0ne has joined #openstack-qa | 06:19 | |
*** jamielennox is now known as jamielennox|away | 06:20 | |
*** e0ne has quit IRC | 06:23 | |
*** yfried has joined #openstack-qa | 06:26 | |
*** gamado has joined #openstack-qa | 06:27 | |
*** e0ne has joined #openstack-qa | 06:28 | |
*** e0ne is now known as e0ne_ | 06:35 | |
*** e0ne_ is now known as e0ne | 06:35 | |
*** itzikb has joined #openstack-qa | 06:36 | |
*** e0ne has quit IRC | 06:36 | |
*** Poornima has joined #openstack-qa | 06:37 | |
mkoderer | andreaf: ping | 06:38 |
openstackgerrit | James Polley proposed openstack-dev/devstack: Add a target for, and link to, minimal config docs https://review.openstack.org/168794 | 06:41 |
*** hichihara has quit IRC | 06:54 | |
*** salv-orlando has joined #openstack-qa | 06:54 | |
openstackgerrit | Bharat Kumar Kobagana proposed openstack-dev/devstack: Clone external plugins before overriding defaults https://review.openstack.org/168796 | 06:57 |
*** salv-orl_ has joined #openstack-qa | 06:58 | |
*** salv-orlando has quit IRC | 06:58 | |
*** afazekas has joined #openstack-qa | 07:02 | |
*** pal__ has joined #openstack-qa | 07:02 | |
*** pal_ has quit IRC | 07:05 | |
*** sergsh has joined #openstack-qa | 07:07 | |
*** achanda has quit IRC | 07:08 | |
*** ihrachyshka has joined #openstack-qa | 07:22 | |
*** ibiris_away is now known as ibiris | 07:26 | |
*** jlibosva has quit IRC | 07:27 | |
*** shardy has joined #openstack-qa | 07:32 | |
*** jordanP has joined #openstack-qa | 07:38 | |
*** amotoki has joined #openstack-qa | 07:38 | |
*** markus_z has joined #openstack-qa | 07:39 | |
*** woodster_ has quit IRC | 07:40 | |
*** annashen has joined #openstack-qa | 07:42 | |
*** salv-orl_ has quit IRC | 07:42 | |
*** chandankumar has quit IRC | 07:42 | |
*** chandankumar has joined #openstack-qa | 07:43 | |
*** jlanoux has joined #openstack-qa | 07:43 | |
*** liusheng has quit IRC | 07:44 | |
*** jyuso1 has joined #openstack-qa | 07:46 | |
*** annashen has quit IRC | 07:46 | |
*** cody-somerville has quit IRC | 07:47 | |
*** mpaolino has joined #openstack-qa | 07:52 | |
*** dims has joined #openstack-qa | 07:53 | |
*** dkusidlo has joined #openstack-qa | 07:53 | |
*** gfidente has joined #openstack-qa | 07:54 | |
*** gfidente has quit IRC | 07:54 | |
*** gfidente has joined #openstack-qa | 07:54 | |
*** pal_ has joined #openstack-qa | 07:55 | |
*** yamamoto has quit IRC | 07:56 | |
*** pal__ has quit IRC | 07:57 | |
*** dims has quit IRC | 07:58 | |
*** tkammer has joined #openstack-qa | 07:58 | |
*** gamado_ has joined #openstack-qa | 07:59 | |
*** gszasz has joined #openstack-qa | 08:00 | |
*** cody-somerville has joined #openstack-qa | 08:02 | |
*** gamado_ has quit IRC | 08:03 | |
*** gamado has quit IRC | 08:04 | |
*** vigneshvar has joined #openstack-qa | 08:05 | |
*** vigneshvar_ has joined #openstack-qa | 08:05 | |
*** SridharG has joined #openstack-qa | 08:06 | |
*** jlibosva has joined #openstack-qa | 08:09 | |
*** safchain has joined #openstack-qa | 08:12 | |
*** haigang has quit IRC | 08:18 | |
*** haigang has joined #openstack-qa | 08:18 | |
*** yamamoto has joined #openstack-qa | 08:19 | |
*** haigang has quit IRC | 08:19 | |
*** pal__ has joined #openstack-qa | 08:25 | |
*** yamamoto has quit IRC | 08:25 | |
*** yfried is now known as yfried|afk | 08:25 | |
andreaf | mkoderer: pong - what's up | 08:26 |
*** pal_ has quit IRC | 08:28 | |
* andreaf will be afk for ~30min | 08:29 | |
*** yfried|afk is now known as yfried | 08:31 | |
*** sergsh has quit IRC | 08:37 | |
*** ihrachyshka has quit IRC | 08:39 | |
*** salv-orlando has joined #openstack-qa | 08:43 | |
*** e0ne has joined #openstack-qa | 08:43 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: WIP - Add new multi ssh configuration options https://review.openstack.org/146535 | 08:46 |
*** salv-orlando has quit IRC | 08:47 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Define validation_resources function for ssh validation https://review.openstack.org/149229 | 08:48 |
*** sergsh has joined #openstack-qa | 08:49 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: WIP - Add create_test_server function https://review.openstack.org/151601 | 08:49 |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 08:50 |
*** e0ne is now known as e0ne_ | 08:51 | |
*** dkusidlo has quit IRC | 08:53 | |
*** Longgeek_ has quit IRC | 08:53 | |
*** e0ne_ is now known as e0ne | 08:54 | |
*** haigang has joined #openstack-qa | 08:55 | |
*** vigneshvar has quit IRC | 08:56 | |
*** vigneshvar_ has quit IRC | 08:56 | |
*** jlibosva has quit IRC | 08:58 | |
mkoderer | andreaf: I added a comment on https://review.openstack.org/#/c/168253/, I would really like to get it merged if this is ok for you | 08:59 |
*** gszasz has quit IRC | 09:02 | |
*** jlibosva has joined #openstack-qa | 09:02 | |
*** jlibosva has quit IRC | 09:06 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 09:07 | |
*** jlibosva has joined #openstack-qa | 09:09 | |
*** jlibosva has quit IRC | 09:10 | |
*** jlibosva has joined #openstack-qa | 09:11 | |
*** bkopilov has quit IRC | 09:18 | |
*** pal_ has joined #openstack-qa | 09:21 | |
*** ihrachyshka has joined #openstack-qa | 09:21 | |
*** tkammer has quit IRC | 09:22 | |
*** apevec has joined #openstack-qa | 09:23 | |
*** apevec has quit IRC | 09:23 | |
*** apevec has joined #openstack-qa | 09:23 | |
*** bkopilov has joined #openstack-qa | 09:23 | |
*** pal__ has quit IRC | 09:23 | |
*** ajo has joined #openstack-qa | 09:25 | |
*** mpaolino has quit IRC | 09:25 | |
*** mahito_ has joined #openstack-qa | 09:25 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Allow Javelin to create volumes without servers https://review.openstack.org/168830 | 09:25 |
*** mahito has quit IRC | 09:28 | |
*** tkammer has joined #openstack-qa | 09:28 | |
*** ajo_ has joined #openstack-qa | 09:30 | |
*** jlibosva has quit IRC | 09:33 | |
*** ajo has quit IRC | 09:33 | |
andreaf | mkoderer: sure I wanted to lift my -1 on Friday but I forgot, sorry - I'll do a review today and if ok I'll approve it | 09:34 |
mkoderer | andreaf: ok cool thx alot | 09:35 |
*** bkopilov has quit IRC | 09:37 | |
*** ajo_ is now known as ajo | 09:38 | |
*** bkopilov has joined #openstack-qa | 09:39 | |
*** dkusidlo has joined #openstack-qa | 09:39 | |
*** ajo has quit IRC | 09:40 | |
*** ajo has joined #openstack-qa | 09:40 | |
*** dkusidlo has quit IRC | 09:42 | |
*** annashen has joined #openstack-qa | 09:43 | |
*** dkusidlo has joined #openstack-qa | 09:45 | |
*** annashen has quit IRC | 09:48 | |
*** pal__ has joined #openstack-qa | 09:53 | |
*** dims has joined #openstack-qa | 09:54 | |
*** tosky has joined #openstack-qa | 09:54 | |
*** pal_ has quit IRC | 09:56 | |
*** vigneshvar has joined #openstack-qa | 09:58 | |
*** dratushnyy has quit IRC | 09:58 | |
*** Longgeek has joined #openstack-qa | 09:58 | |
*** vigneshvar_ has joined #openstack-qa | 09:59 | |
*** mahito_ has quit IRC | 10:04 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin https://review.openstack.org/165775 | 10:07 |
*** yamamoto has joined #openstack-qa | 10:12 | |
openstackgerrit | Alex proposed openstack-infra/subunit2sql: Extend database schema of run_metadata https://review.openstack.org/168840 | 10:17 |
*** dmorita has quit IRC | 10:17 | |
*** Longgeek has quit IRC | 10:17 | |
*** gamado has joined #openstack-qa | 10:18 | |
*** pal_ has joined #openstack-qa | 10:23 | |
*** salv-orlando has joined #openstack-qa | 10:23 | |
*** jlibosva has joined #openstack-qa | 10:26 | |
*** pal__ has quit IRC | 10:26 | |
*** gamado has quit IRC | 10:29 | |
*** dratushnyy has joined #openstack-qa | 10:32 | |
*** yfried is now known as yfried|afk | 10:32 | |
*** yfried|afk is now known as yfried | 10:34 | |
*** yamamoto has quit IRC | 10:35 | |
*** pc_m has joined #openstack-qa | 10:36 | |
*** bkopilov has quit IRC | 10:37 | |
*** haigang has quit IRC | 10:40 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 10:41 | |
*** jhenner has joined #openstack-qa | 10:41 | |
*** yamamoto has joined #openstack-qa | 10:42 | |
*** bkopilov has joined #openstack-qa | 10:47 | |
*** pal_ has quit IRC | 10:52 | |
*** mwagner_lap has quit IRC | 10:52 | |
*** yamamoto has quit IRC | 10:52 | |
*** pal_ has joined #openstack-qa | 10:52 | |
*** pal_ has quit IRC | 11:01 | |
*** vigneshvar_ has quit IRC | 11:04 | |
*** vigneshvar has quit IRC | 11:04 | |
*** vigneshvar has joined #openstack-qa | 11:05 | |
*** vigneshvar_ has joined #openstack-qa | 11:05 | |
*** sergsh has quit IRC | 11:11 | |
*** Longgeek has joined #openstack-qa | 11:15 | |
*** jokke_ has quit IRC | 11:15 | |
*** jaypipes has joined #openstack-qa | 11:17 | |
*** ChanServ sets mode: +o jaypipes | 11:17 | |
*** sergsh has joined #openstack-qa | 11:26 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin https://review.openstack.org/165775 | 11:26 |
*** markus_z has quit IRC | 11:27 | |
*** BharatK has quit IRC | 11:33 | |
*** rdekel has joined #openstack-qa | 11:37 | |
*** neeti has quit IRC | 11:39 | |
*** Longgeek has quit IRC | 11:40 | |
*** neeti has joined #openstack-qa | 11:42 | |
*** annashen has joined #openstack-qa | 11:45 | |
*** vigneshvar has quit IRC | 11:49 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add new multi ssh configuration options https://review.openstack.org/146535 | 11:49 |
*** annashen has quit IRC | 11:49 | |
*** vigneshvar_ has quit IRC | 11:50 | |
*** yamamoto has joined #openstack-qa | 11:50 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Define validation_resources function for ssh validation https://review.openstack.org/149229 | 11:51 |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Add create_test_server function https://review.openstack.org/151601 | 11:55 |
*** woodster_ has joined #openstack-qa | 12:00 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: Create shared ssh verification by extending RemoteClient https://review.openstack.org/153899 | 12:01 |
*** Poornima has quit IRC | 12:08 | |
*** cdent has joined #openstack-qa | 12:08 | |
*** tkammer has quit IRC | 12:11 | |
*** dims has quit IRC | 12:17 | |
*** dims has joined #openstack-qa | 12:18 | |
*** dkranz has joined #openstack-qa | 12:18 | |
*** krtaylor has quit IRC | 12:19 | |
*** yfried is now known as yfried|afk | 12:20 | |
*** tkammer has joined #openstack-qa | 12:23 | |
*** aysyd has joined #openstack-qa | 12:23 | |
*** salv-orlando has quit IRC | 12:27 | |
*** salv-orlando has joined #openstack-qa | 12:30 | |
*** sputnik13 has quit IRC | 12:30 | |
*** weshay has joined #openstack-qa | 12:30 | |
*** yfried|afk is now known as yfried | 12:31 | |
openstackgerrit | Richard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification https://review.openstack.org/168880 | 12:32 |
*** pal_ has joined #openstack-qa | 12:32 | |
*** rfolco has joined #openstack-qa | 12:32 | |
*** Longgeek has joined #openstack-qa | 12:36 | |
*** gamado has joined #openstack-qa | 12:41 | |
*** e0ne is now known as e0ne_ | 12:44 | |
*** tochi has quit IRC | 12:45 | |
*** e0ne_ is now known as e0ne | 12:51 | |
*** dkranz has quit IRC | 12:53 | |
*** mwagner_lap has joined #openstack-qa | 12:53 | |
*** dboik has quit IRC | 12:56 | |
*** Madan has quit IRC | 12:59 | |
*** krtaylor has joined #openstack-qa | 13:00 | |
*** k4n0 has quit IRC | 13:01 | |
*** marrusl has joined #openstack-qa | 13:03 | |
*** e0ne is now known as e0ne_ | 13:03 | |
*** dkranz has joined #openstack-qa | 13:04 | |
*** bknudson has quit IRC | 13:05 | |
*** jecarey has joined #openstack-qa | 13:09 | |
*** Longgeek has quit IRC | 13:10 | |
*** haigang has joined #openstack-qa | 13:12 | |
*** e0ne_ has quit IRC | 13:14 | |
*** neeti has quit IRC | 13:15 | |
*** dboik has joined #openstack-qa | 13:16 | |
*** e0ne has joined #openstack-qa | 13:16 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 13:21 | |
*** dustins has joined #openstack-qa | 13:24 | |
*** johnthetubaguy is now known as zz_johnthetubagu | 13:34 | |
*** apevec has quit IRC | 13:36 | |
*** rfolco has quit IRC | 13:40 | |
*** jlanoux has quit IRC | 13:40 | |
*** rfolco has joined #openstack-qa | 13:42 | |
*** mriedem has joined #openstack-qa | 13:42 | |
*** markus_z has joined #openstack-qa | 13:43 | |
*** e0ne is now known as e0ne_ | 13:43 | |
*** jgrimm is now known as zz_jgrimm | 13:45 | |
*** bknudson has joined #openstack-qa | 13:45 | |
*** e0ne_ is now known as e0ne | 13:46 | |
*** rfolco has quit IRC | 13:47 | |
*** annashen has joined #openstack-qa | 13:47 | |
*** rfolco has joined #openstack-qa | 13:48 | |
*** tkammer has quit IRC | 13:48 | |
*** apevec has joined #openstack-qa | 13:48 | |
*** chandankumar has quit IRC | 13:50 | |
openstackgerrit | Abhishek G M proposed openstack/tempest: Test Creeation of server in a shared_network https://review.openstack.org/168910 | 13:50 |
andreaf | mtreinish: hi - you around? | 13:50 |
*** rbak has quit IRC | 13:50 | |
*** annashen has quit IRC | 13:52 | |
mtreinish | andreaf: yep | 13:52 |
andreaf | mtreinish: regarding subuni2sql, I was thinking about how to get more details about the system under test in the DB | 13:53 |
andreaf | mtreinish: the problem I see is that there's no ID that identifies a specific combination of d-g flags, like multinode, mysql or postgres, neutron or n-net etc\ | 13:54 |
andreaf | mtreinish: so assuming we get that info into the subunit stream, we would need to store it for each run again and again | 13:55 |
mtreinish | andreaf: so we store whatever flags logstash would have in the run_metadata table | 13:56 |
*** superdan is now known as dansmith | 13:56 | |
*** sabeen1 has joined #openstack-qa | 13:56 | |
mtreinish | so if they're fields that apply to the whole run, like the db backend, that's probably where they make sense to go | 13:56 |
andreaf | mtreinish: yes but there's kind of a limited number or run types we have | 13:58 |
andreaf | mtreinish: so it would be interesting to be able know that a run uses the same combination of flags as another one, without the need of matching 10 or 15 different fields | 13:59 |
*** pal_ has quit IRC | 14:01 | |
mtreinish | andreaf: well that's kinda how we do it with infra. Only the job name is stored in the run_metadata table, which can be used to indicate which flags were used. | 14:01 |
*** yamamoto has quit IRC | 14:03 | |
*** yamamoto has joined #openstack-qa | 14:03 | |
*** dboik_ has joined #openstack-qa | 14:03 | |
openstackgerrit | Abhishek G M proposed openstack/tempest: Test Creeation of server in a shared_network https://review.openstack.org/168910 | 14:04 |
*** jyuso1 has quit IRC | 14:05 | |
andreaf | mtreinish: yes the job name helps, but that works in upstream only because the test job has full control over the deployment on the cloud as well, so the job name implies the cloud configuration - but when I use subunit2sql for my internal post CI tests I cannot rely on that assumption | 14:05 |
*** dboik has quit IRC | 14:06 | |
mtreinish | andreaf: so what are you looking for then? Because you could also just make a comma separate string as the value for the specific flags you wanted too. | 14:06 |
andreaf | mtreinish: well I need to know for instance all test runs that have the same target configuration in common - including software version, configuration options, deployment type etc | 14:09 |
*** yamamoto has quit IRC | 14:10 | |
andreaf | mtreinish: but I would also like to read the data in a different way, e.g. see the trends for tests across different versions but with the same deployment type - so having a comma separate list for that does not help - unless I store both individual items and the list | 14:10 |
*** dkranz has quit IRC | 14:11 | |
*** rbak has joined #openstack-qa | 14:12 | |
mtreinish | andreaf: oh, ok yeah I follow now. But, my only concern with adding columns for things like this is that they are very use case specific. Which is why I went arbitrary key value pair for that table, because every use case will have different needs | 14:13 |
andreaf | mtreinish: yes I was thinking more about a new table of target systems with its key-value metadata - the problem is that there's no ID | 14:15 |
*** rbak has quit IRC | 14:15 | |
andreaf | mtreinish: not at the moment at least | 14:16 |
mtreinish | andreaf: an id for what? | 14:16 |
mtreinish | andreaf: my other thought is that you can overload the key value for a particular run_id fk in the run_metadata table. (it's not like it has a unique constraint for a given fk) | 14:17 |
andreaf | mtreinish: target system, i.e. combination of deployment type and configuration | 14:17 |
openstackgerrit | Kirill Shileev proposed openstack/tempest: Check IPv6 mupltiprefix feature https://review.openstack.org/157816 | 14:17 |
*** mtanino has joined #openstack-qa | 14:18 | |
*** Alex______ has joined #openstack-qa | 14:19 | |
*** rbak has joined #openstack-qa | 14:19 | |
mtreinish | andreaf: oh, so you're thinking about adding a new table that would map your specific configuration data to an arbitrary id which can be associated to a given run? | 14:20 |
andreaf | mtreinish: yes | 14:21 |
*** tkammer has joined #openstack-qa | 14:21 | |
andreaf | mtreinish: but there's no such ID - getting one would be having some algorithm in d-g to create it in a repeatable way, so that when the subunit stream is injected if the ID already exists it can be used and the the rest of the metadata associated with it discareded | 14:23 |
mtreinish | andreaf: in that case why not just store that id in the run_metadata table then? You could write a small script to query your table against a set of params you pass it (from d-g or where ever) and return the id or create one and return that | 14:25 |
mtreinish | then use that as a field in the run_metadata table with the subunit2sql util | 14:25 |
*** jlanoux has joined #openstack-qa | 14:26 | |
*** dkranz has joined #openstack-qa | 14:27 | |
andreaf | mtreinish: so create a kind of hash of the metadata I'm interested into, add store it in the metadata table as an additional key/value, and search by that? | 14:28 |
*** ajo has quit IRC | 14:28 | |
*** ChuckC has quit IRC | 14:29 | |
mtreinish | andreaf: yeah, although it doesn't even need to be a hash, you could just use an int or uuid too which is the primary key in your new table. | 14:30 |
andreaf | mtreinish: oh I see what you mean - so get the ID by searching in the new table and add it into the run_metadata as a fk | 14:31 |
*** jlanoux_ has joined #openstack-qa | 14:31 | |
mtreinish | andreaf: yeah except it probably won't be a real fk because I don't think you could add that relationship to the value column in run_metadata | 14:32 |
andreaf | mtreinish: indeed | 14:33 |
*** achanda has joined #openstack-qa | 14:33 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin https://review.openstack.org/165775 | 14:34 |
*** jlanoux has quit IRC | 14:34 | |
*** jlibosva has quit IRC | 14:37 | |
*** cbader has quit IRC | 14:37 | |
*** asselin has quit IRC | 14:37 | |
*** cbader has joined #openstack-qa | 14:37 | |
*** asselin has joined #openstack-qa | 14:37 | |
andreaf | mtreinish: I could prepare a draft subunit2sql patch to show you what the table would look like - for me it would be interesting to this as long as it's supported by subunit2sql | 14:38 |
*** ilyashakhat has joined #openstack-qa | 14:38 | |
*** zz_dimtruck is now known as dimtruck | 14:38 | |
mtreinish | andreaf: sure you can push it up, but my concern will be that it will be too specific to your use case | 14:39 |
*** cbader has quit IRC | 14:39 | |
*** cbader has joined #openstack-qa | 14:39 | |
*** asselin has quit IRC | 14:40 | |
*** asselin has joined #openstack-qa | 14:41 | |
andreaf | mtreinish: my use case seems to be quite general to me, I agree that we should not have anything specific in there | 14:41 |
*** asselin has quit IRC | 14:41 | |
*** asselin_ has joined #openstack-qa | 14:41 | |
andreaf | mtreinish: I'll think a bit more about it a make a draft proposal, I think I can keep it generic enough and that it would then be beneficial for everyone | 14:42 |
*** asselin_ has quit IRC | 14:42 | |
*** patrickeast has joined #openstack-qa | 14:42 | |
*** cbader has quit IRC | 14:42 | |
mtreinish | andreaf: ok cool, I'll keep an eye out for it then | 14:43 |
*** cbader has joined #openstack-qa | 14:43 | |
*** asselin_ has joined #openstack-qa | 14:43 | |
*** asselin_ has quit IRC | 14:44 | |
*** achanda has quit IRC | 14:44 | |
*** asselin_ has joined #openstack-qa | 14:44 | |
*** zz_jgrimm is now known as jgrimm | 14:44 | |
*** cbader has quit IRC | 14:45 | |
*** asselin_ has quit IRC | 14:45 | |
*** cbader has joined #openstack-qa | 14:45 | |
*** asselin_ has joined #openstack-qa | 14:45 | |
mtreinish | andreaf: also unrelated to subunit2sql I was thinking a bit more about the non-locking providers. Is there a reason we would ever want to use it with the accounts.yaml file? | 14:46 |
*** cbader has quit IRC | 14:46 | |
mtreinish | the locking provider will work fine in the serial case (with a limited number of creds in the file) | 14:46 |
*** tsekiyama has joined #openstack-qa | 14:46 | |
*** asselin_ has quit IRC | 14:46 | |
*** asselin has joined #openstack-qa | 14:47 | |
*** cbader has joined #openstack-qa | 14:47 | |
*** asselin has quit IRC | 14:47 | |
andreaf | mtreinish: I see no reason for that, it was designed for the accounts in tempest.conf - so we might want to explicitly forbid that combination | 14:47 |
*** cbader has quit IRC | 14:47 | |
andreaf | non-locking provider and accounts.yaml | 14:47 |
*** cbader has joined #openstack-qa | 14:48 | |
*** asselin has joined #openstack-qa | 14:48 | |
mtreinish | andreaf: yeah that's what I was thinking when I was reviewing: https://review.openstack.org/168373 | 14:48 |
andreaf | mtreinish: and once we get rid of the configured accounts completely, we can also drop the non-locking account provider | 14:48 |
*** cbader has quit IRC | 14:48 | |
mtreinish | yep | 14:48 |
*** yfried has quit IRC | 14:48 | |
mtreinish | hogepodge: ^^^ since it's your patch in question any thoughts? | 14:49 |
*** asselin has quit IRC | 14:49 | |
*** cbader has joined #openstack-qa | 14:49 | |
*** asselin_ has joined #openstack-qa | 14:49 | |
*** rdekel has quit IRC | 14:49 | |
*** asselin_ has quit IRC | 14:50 | |
*** cbader has quit IRC | 14:50 | |
*** asselin_ has joined #openstack-qa | 14:50 | |
*** cbader has joined #openstack-qa | 14:50 | |
*** asselin_ has quit IRC | 14:51 | |
*** cbader has quit IRC | 14:51 | |
*** cbader has joined #openstack-qa | 14:51 | |
*** asselin_ has joined #openstack-qa | 14:51 | |
*** hemnafk is now known as hemna | 14:52 | |
*** cbader has quit IRC | 14:52 | |
*** asselin_ has quit IRC | 14:52 | |
*** cbader has joined #openstack-qa | 14:52 | |
*** asselin_ has joined #openstack-qa | 14:52 | |
*** haigang has quit IRC | 14:53 | |
hogepodge | mtreinish: andreaf: We have two issues, the short term bug that causes the crash, and the longer term strategy for handling accounts. | 14:54 |
hogepodge | mtreinish: andreaf: also, kind of related, this: https://etherpad.openstack.org/p/accounts-yaml-generation | 14:55 |
*** Longgeek has joined #openstack-qa | 14:55 | |
mtreinish | hogepodge: but we could fix both if we re-spun your patch to just remove the accounts.yaml path from the non-locking provider and removed the explicity option for enabling locking (and just made it the check for the accounts file) | 14:56 |
*** Guest96413 is now known as redrobot | 14:58 | |
hogepodge | mtreinish: what do you expect the amount of work would be for the latter? I think it's better to do the formally correct thing, but a short term patch until we get there would be nice. I understand that short term solutions have a nasty habit of becoming long term solutions though. | 14:59 |
mtreinish | hogepodge: heh, well that list cast didn't even fix that code path IIRC which was part of the problem | 14:59 |
mtreinish | hogepodge: it would be pretty easy I think to disallow that case | 14:59 |
hogepodge | It fixes part of the code path, but not the complete path | 15:00 |
*** rwsu has joined #openstack-qa | 15:01 | |
hogepodge | going into the mirantis meeting | 15:01 |
hogepodge | brb | 15:01 |
mtreinish | hogepodge: I think http://paste.openstack.org/show/197563/ is all that would be needed along with dropping the locking accounts option (and a unit test to verify that path probably) | 15:03 |
*** dboik_ has quit IRC | 15:04 | |
*** dboik has joined #openstack-qa | 15:04 | |
openstackgerrit | Merged openstack-dev/devstack: Move back isset to the functions-common https://review.openstack.org/158683 | 15:06 |
andreaf | mtreinish, hogepodge: the only problem I have with that approach is that it is less explicit - if I do want to use accounts.yaml, and thus set my testr to parallel - but for whatever reason the file is not there - the tests will still run, but there will be problems because of concurrency | 15:07 |
*** jlanoux_ has quit IRC | 15:07 | |
andreaf | mtreinish, hogepodge: so I was thinking about keeping the option, but ignore accounts.yaml in the non-locking provider, just always used configured credentials | 15:08 |
*** jlanoux has joined #openstack-qa | 15:08 | |
andreaf | mtreinish, hogepodge: but maybe less configuration options is better, easier to use :) | 15:09 |
*** catohornet has joined #openstack-qa | 15:10 | |
hogepodge | andreaf: mtreinish: I'm much less of an expert, so my feeling is to defer to your judgment. I do have a very short-term interest though, since testing will start soon. | 15:11 |
*** ChuckC has joined #openstack-qa | 15:12 | |
*** Longgeek has quit IRC | 15:13 | |
mtreinish | andreaf: well maybe, because don't we default the path to None? in which case you'd have to explicitly set it to use it | 15:13 |
mtreinish | hogepodge: well the implementation is simple either way, it's more of about end-user friendliness on which approach we go. Which you definitely can weigh in on | 15:14 |
*** chandankumar has joined #openstack-qa | 15:15 | |
*** afazekas has quit IRC | 15:16 | |
andreaf | mtreinish: no, we default to etc/accounts.yaml http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n38 | 15:16 |
mtreinish | andreaf: oh, maybe we don't want to do that then | 15:16 |
andreaf | mtreinish: but by default the file does not exists | 15:16 |
andreaf | mtreinish: which is almost like defaulting to None | 15:17 |
mtreinish | except that if you don't set the option to anything and that file was created it would still try to use it. (which seems less than desirable) | 15:18 |
andreaf | mtreinish: ok, right | 15:19 |
*** dkranz has quit IRC | 15:22 | |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: DO NOT MERGE- Test ssh validation in the gate https://review.openstack.org/165504 | 15:22 |
*** sputnik13 has joined #openstack-qa | 15:23 | |
*** itzikb has quit IRC | 15:27 | |
*** dboik_ has joined #openstack-qa | 15:30 | |
hogepodge | mtreinish: trying to access a set as a list is still a straightforward fix that removes an immediate crash, and fixing the larger behavior is desirable too, so do the first then do the second would be my preference, but I may be over-complicating things | 15:30 |
hogepodge | but regards to settings, I need to think. It's not clear in my mind yet, but it's early and I just started thinking about it in parallel with another meeting. ;-) | 15:31 |
mtreinish | hogepodge: heh ok | 15:32 |
*** e0ne is now known as e0ne_ | 15:32 | |
*** dboik has quit IRC | 15:32 | |
mtreinish | hogepodge: my issue with the list casting patch as it sits now is that we have no way to verify it fixes anything right now | 15:33 |
mtreinish | also IIRC the non-locking provider with the accounts file will just fail slightly later in that code path (which is why we switched your test setup to use the locking provider) | 15:34 |
hogepodge | andreaf mtreinish My preference is to be explicit in what behavior I want. If I ask for accounts.yaml, I should get it regardless of parallel or serial execution. If it's a bug to run in serial, an error describing the state should be thrown. | 15:34 |
*** eugene_bag has joined #openstack-qa | 15:34 | |
hogepodge | mtreinish: agree. Not sure exactly how to test it, but that's me being a bit fuzzy on how the system works. | 15:35 |
*** SridharG has left #openstack-qa | 15:35 | |
mtreinish | hogepodge: well the issue now is that accounts.yaml is usable between 2 providers right now the locking and non-locking (which doesn't work and your patch begins to try and fix) | 15:35 |
mtreinish | but I think we've realized that there isn't a reason to use accounts.yaml with the non-locking provider | 15:35 |
mtreinish | because the locking provider works fine when running serially (which is the use case for the non-locking provider) | 15:36 |
hogepodge | mtreinish: what are the use cases for locking vs non-locking? What's a summary of the fix for the non-locking case? | 15:37 |
mtreinish | hogepodge: I think we're in agreement on the config approach, which is just have a single accounts.yaml option (which defaults to None) and if specified it'll be used with locking provider | 15:37 |
*** e0ne_ is now known as e0ne | 15:38 | |
mtreinish | hogepodge: the non-locking provider is used like the original config one was (just using the same internal api as the locking and isolated creds) | 15:38 |
mtreinish | right now it supports accounts.yaml because it was simple to add, but there isn't any advantage to doing that | 15:39 |
mtreinish | that is the code path your patch is trying to fix | 15:39 |
mtreinish | as for the full fix for that path I'm not sure what it would entail I haven't ever tried to run in that combination, we would need to push up a test patch to verify it | 15:39 |
mtreinish | but instead of bothering with that I think it would be easier to disable that path and only allow the non-locking provider to be used with config options | 15:40 |
mtreinish | since there isn't a benefit to ever using it with the accounts.yaml file (even when running serially) | 15:40 |
mtreinish | the locking provider works fine serially and it would make the code and configuration around this a bit simpler | 15:42 |
*** dboik_ has quit IRC | 15:42 | |
*** dboik has joined #openstack-qa | 15:43 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 15:46 | |
*** tsekiyama has quit IRC | 15:49 | |
*** tsekiyama has joined #openstack-qa | 15:49 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for horizon integration test bug 1436903 https://review.openstack.org/168753 | 15:53 |
openstack | bug 1436903 in OpenStack Dashboard (Horizon) "integration tests failing blocking gate" [Critical,Triaged] https://launchpad.net/bugs/1436903 - Assigned to David Lyle (david-lyle) | 15:53 |
* andreaf afk for ~1h | 15:56 | |
*** markus_z has quit IRC | 15:57 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Only use accounts.yaml with locking provider https://review.openstack.org/168981 | 15:59 |
mtreinish | hogepodge, andreaf: ^^^ this is what I was talking about | 15:59 |
*** edmondsw has joined #openstack-qa | 16:02 | |
*** openstackrecheck has quit IRC | 16:02 | |
*** openstackrecheck has joined #openstack-qa | 16:02 | |
*** Alex______ has quit IRC | 16:03 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Adds unit test for Javelin https://review.openstack.org/165775 | 16:04 |
edmondsw | check-grenade-dsvm is failing for https://review.openstack.org/#/c/167370/ with a ceilometer error that seems to indicate using a different policy.json than what is in that ceilometer patch set... how do I figure out where the policy.json is coming from that is used in this test? | 16:04 |
*** annashen has joined #openstack-qa | 16:05 | |
edmondsw | seems to be using this, which doesn't match the patch set: http://logs.openstack.org/70/167370/5/check/check-grenade-dsvm/7767ba3/logs/etc/ceilometer/policy.json.txt.gz | 16:06 |
mtreinish | edmondsw: it's being caused by your patch changing the policy.json file and permissions. Grenade takes the previous stable release deploys it runs tests then upgrades to the current master (plus your proposed patch) without changing any of the config from the stable deployment (which includes the policy file) | 16:08 |
mtreinish | edmondsw: you'll have to change your patch to make it backwards compat and ensure it works using an older version of the policy file | 16:09 |
edmondsw | mtreinish: Thanks. | 16:10 |
mtreinish | edmondsw: the alternative would be to backport the fixes to fix the policy enforcement behavior in the stable branches so the upgrade path works. (but I don't think that would be a backportable change) | 16:11 |
edmondsw | mtreinish: ceilometer policy enforcement is a completely mess (non-existent in juno and previous, still incomplete in kilo), so being backward compatible may not really be possible | 16:13 |
*** yfried has joined #openstack-qa | 16:13 | |
*** riwinters has joined #openstack-qa | 16:13 | |
mtreinish | edmondsw: well there is a mechanism to manual upgrade config in grenade (using the upgrade_ceilometer file in this case) but you really should try to avoid using it at all costs. | 16:14 |
edmondsw | mtreinish: we'll do what we can and come back with questions about that if necessary... Thanks | 16:14 |
mtreinish | edmondsw: also before a patch adding a manual upgrade step can land in grenade there needs to be buy in from the ceilo team and something added to the release notes reflecting that all users will have to make the manual step when upgrading | 16:15 |
edmondsw | mtreinish: sure | 16:15 |
rahmu | mtreinish: hello. I started working on Javelin unit tests here https://review.openstack.org/#/c/165775/6. It's still a WIP.I will continue pushing code progressively as I write it, but any early feedback and comment I can get would help me a lot (before I replicate all this on all the resources Javelin deals with). | 16:16 |
openstackgerrit | Jordan Pittier proposed openstack-dev/devstack: Swift PyEClib build requires 'make' https://review.openstack.org/168990 | 16:17 |
mtreinish | rahmu: cool thanks, I saw that you started pushing that. I'll put it on my review list and try to take a prelim look at it today | 16:17 |
rahmu | that would be great thanks :) | 16:18 |
*** ajmiller has joined #openstack-qa | 16:28 | |
hogepodge | mtreinish: it lgtm at first glance. Let me run it through some paces. | 16:28 |
*** jlibosva has joined #openstack-qa | 16:28 | |
riwinters | Hi all, I uploaded a patch this morning that failed miserably - looks like import of data_utils from tempest.common.utils | 16:28 |
dtroyer | sdague, jogo: https://review.openstack.org/#/c/168581/ teaches Grenade how to deal with the coming neutron rework in DevStack, and https://review.openstack.org/168591 gets rid of the turns-out-to-be-trouble-in-gerrit symlink | 16:28 |
riwinters | should I be using tempest_lib data utils ? | 16:28 |
*** dkranz has joined #openstack-qa | 16:28 | |
mtreinish | riwinters: yes, we moved the data_utils module over to tempest_lib and removed the local copy of it from tempest soon after it was in a released version | 16:30 |
riwinters | Ah ok - thanks | 16:30 |
*** e0ne has quit IRC | 16:30 | |
mtreinish | riwinters: fyi: https://review.openstack.org/#/c/158012/ | 16:31 |
openstackgerrit | Joseph Lanoux proposed openstack/tempest: DO NOT MERGE- Test ssh validation in the gate https://review.openstack.org/165504 | 16:31 |
sdague | dtroyer: +A | 16:31 |
*** Nic has joined #openstack-qa | 16:32 | |
*** aysyd has quit IRC | 16:32 | |
*** jlibosva has quit IRC | 16:33 | |
*** marun has joined #openstack-qa | 16:33 | |
mtreinish | sdague: so on: https://review.openstack.org/#/c/166993/ I think the project config change landed that was assuming this was in d-g (it didn't have a depends on in the commit msg) | 16:35 |
mtreinish | the project-config patch was: https://review.openstack.org/#/c/166964/ | 16:35 |
*** dratushnyy has quit IRC | 16:35 | |
*** aysyd has joined #openstack-qa | 16:35 | |
sdague | mtreinish: ok, can you just fix project-config to use pass through? | 16:36 |
mtreinish | probably, it shouldnt be too hard | 16:36 |
*** sergsh has quit IRC | 16:41 | |
*** jlvillal has left #openstack-qa | 16:41 | |
*** yamahata has joined #openstack-qa | 16:44 | |
*** Nic has quit IRC | 16:46 | |
*** jaypipes has quit IRC | 16:49 | |
openstackgerrit | Richard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification https://review.openstack.org/168880 | 16:57 |
*** erlon has joined #openstack-qa | 16:59 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack-dev/devstack: WIP: Trying Nova with oslo.db trunk https://review.openstack.org/169004 | 16:59 |
*** dkranz has quit IRC | 17:01 | |
*** adalbas has joined #openstack-qa | 17:04 | |
*** bknudson has quit IRC | 17:04 | |
*** safchain has quit IRC | 17:11 | |
*** dkusidlo has quit IRC | 17:11 | |
*** Nic has joined #openstack-qa | 17:12 | |
*** pal_ has joined #openstack-qa | 17:13 | |
*** dkranz has joined #openstack-qa | 17:16 | |
*** yamahata has quit IRC | 17:18 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Only use accounts.yaml with locking provider https://review.openstack.org/168981 | 17:19 |
*** eugene_bag1 has joined #openstack-qa | 17:19 | |
*** eugene_bag1 has quit IRC | 17:19 | |
*** jgrimm is now known as zz_jgrimm | 17:22 | |
*** eugene_bag has quit IRC | 17:23 | |
*** achanda has joined #openstack-qa | 17:24 | |
*** melwitt has joined #openstack-qa | 17:25 | |
*** harlowja_away is now known as harlowja | 17:25 | |
*** chandankumar has quit IRC | 17:28 | |
*** dkranz has quit IRC | 17:28 | |
*** e0ne has joined #openstack-qa | 17:28 | |
*** mriedem has quit IRC | 17:34 | |
*** dboik has quit IRC | 17:35 | |
*** e0ne is now known as e0ne_ | 17:35 | |
*** dboik has joined #openstack-qa | 17:35 | |
*** amotoki has quit IRC | 17:36 | |
*** yamahata has joined #openstack-qa | 17:38 | |
*** e0ne_ is now known as e0ne | 17:39 | |
openstackgerrit | Merged openstack-dev/grenade: Add new DevStack Neutron requirements https://review.openstack.org/168581 | 17:39 |
openstackgerrit | Merged openstack-dev/devstack: Remove symlink for lib/neutron compat https://review.openstack.org/168591 | 17:40 |
openstackgerrit | Clark Boylan proposed openstack-dev/devstack: Update libvirt cpu map before starting nova https://review.openstack.org/168407 | 17:40 |
*** soumit has joined #openstack-qa | 17:41 | |
*** gamado has quit IRC | 17:41 | |
*** dkranz has joined #openstack-qa | 17:41 | |
clarkb | dansmith: jogo sdague ^ that updates two things, first only affects libvirt if we are using libvirt and second attempts to use the core2duo cpu as a base since I Think the scenario tests may have failed due to the pentiumpro + lm not be "real" enough | 17:42 |
*** jaypipes has joined #openstack-qa | 17:42 | |
*** ChanServ sets mode: +o jaypipes | 17:42 | |
clarkb | I think I am doing the feature disabling correctly, hopefully that is valid in cpu_map.xml | 17:42 |
clarkb | kashyap: ^ | 17:42 |
kashyap | clarkb: Looking | 17:42 |
dansmith | clarkb: what about the scenario tests care about the cpu model? | 17:42 |
dansmith | clarkb: that seems odd and potentially broken | 17:43 |
kashyap | Hmm, that's likely -- "not being real". /me will experiment locally a bit in a few. | 17:43 |
clarkb | dansmith: I think its the compiled binaries for eg ssh expecting more instructions to exist on the 64bit build of cirros | 17:43 |
clarkb | dansmith: because ssh seems to not be working properly | 17:44 |
dansmith | clarkb: ah | 17:44 |
clarkb | https://jenkins03.openstack.org/job/check-tempest-dsvm-aiopcpu-full/10/console is the job run | 17:44 |
*** dpaterson has joined #openstack-qa | 17:45 | |
hogepodge | mtreinish: can you verify my commit message on this patch for accuracy? https://review.openstack.org/#/c/169021/ | 17:45 |
*** annashen has quit IRC | 17:46 | |
*** catohornet has quit IRC | 17:46 | |
clarkb | but if this works then I can figure out a better way to deal with xml in devstack and make this cpu_map update a bit more optional | 17:47 |
*** ihrachyshka has quit IRC | 17:47 | |
*** annashen has joined #openstack-qa | 17:47 | |
*** Sukhdev has joined #openstack-qa | 17:48 | |
*** e0ne is now known as e0ne_ | 17:49 | |
clarkb | jogo: https://jenkins03.openstack.org/job/check-tempest-dsvm-aiopcpu-full/10/console just failed on the ssh thing :/ | 17:49 |
clarkb | jogo: heisenbug | 17:49 |
clarkb | kashyap: dansmith https://jenkins05.openstack.org/job/check-tempest-dsvm-neutron-aiopcpu-full/23/ is a better one to follow, the earlier job failed due to a known issue we haven't been able to track down | 17:49 |
kashyap | clarkb: Thanks, I was trying to follow the other one. | 17:50 |
*** mriedem has joined #openstack-qa | 17:51 | |
openstackgerrit | Andrey Pavlov proposed openstack-dev/devstack: install euca2ools for tempest preparation https://review.openstack.org/169027 | 17:52 |
*** chandankumar has joined #openstack-qa | 17:53 | |
*** e0ne_ has quit IRC | 17:54 | |
*** jordanP has quit IRC | 17:54 | |
*** pal_ has quit IRC | 17:55 | |
*** dustins_ has joined #openstack-qa | 17:56 | |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: add back python-guestfs https://review.openstack.org/169029 | 17:57 |
*** dustins_ has quit IRC | 17:57 | |
dansmith | clarkb: after this merges are you going to work on making the devstacks run in parallel? | 17:58 |
clarkb | dansmith: ya I think that is going to have to be the next improvement | 17:58 |
dansmith | okay cool | 17:58 |
*** dustins has quit IRC | 17:59 | |
sdague | dtroyer: https://review.openstack.org/169029 - a few too many things removed in doing the nova venv | 17:59 |
*** dustins has joined #openstack-qa | 18:01 | |
dtroyer | ah, that was more of a 'see if we can get away without it' that I forgot about. we'll beed to add it to the wheel build list | 18:01 |
kashyap | clarkb: I'm off for the night, will check the status tomm. | 18:04 |
clarkb | kashyap: thanks | 18:04 |
*** greghaynes has quit IRC | 18:07 | |
openstackgerrit | Richard Winters proposed openstack/tempest: Adding multi-node test case for ML2 driver verification https://review.openstack.org/168880 | 18:08 |
*** soumit has quit IRC | 18:08 | |
*** soumit has joined #openstack-qa | 18:09 | |
*** jlanoux has quit IRC | 18:09 | |
*** bknudson has joined #openstack-qa | 18:10 | |
*** chandankumar has quit IRC | 18:10 | |
*** bknudson has left #openstack-qa | 18:12 | |
*** bknudson has joined #openstack-qa | 18:14 | |
mtreinish | hogepodge: well, it's a subset of tests that require ssh. It's anything that actually honors the run_ssh config flag | 18:19 |
mtreinish | there are other tests which use ssh but don't take into account that flag | 18:19 |
mtreinish | for example, most of the scenario tests | 18:19 |
hogepodge | mtreinish: can I have you leave a comment that clarifies that so I can update the commit message | 18:22 |
hogepodge | ? | 18:22 |
mtreinish | hogepodge: sure, one sec | 18:22 |
hogepodge | mtreinish: thanks | 18:23 |
*** johnthetubaguy is now known as zz_johnthetubagu | 18:23 | |
clarkb | hrm nova seems unhappy with my latest cpu update will dig in once logs come back | 18:23 |
mtreinish | hogepodge: ok, left a comment | 18:27 |
*** eugene_bag has joined #openstack-qa | 18:27 | |
*** eugene_bag1 has joined #openstack-qa | 18:30 | |
*** eugene_bag has quit IRC | 18:33 | |
*** e0ne has joined #openstack-qa | 18:34 | |
*** achanda has quit IRC | 18:38 | |
*** pal_ has joined #openstack-qa | 18:40 | |
*** tosky has quit IRC | 18:40 | |
*** dustins has quit IRC | 18:42 | |
*** chandankumar has joined #openstack-qa | 18:43 | |
*** catohornet has joined #openstack-qa | 18:45 | |
*** dustins has joined #openstack-qa | 18:48 | |
*** soumit has quit IRC | 18:48 | |
*** Nic has quit IRC | 18:51 | |
*** pal_ has quit IRC | 18:51 | |
openstackgerrit | Clark Boylan proposed openstack-dev/devstack: Update libvirt cpu map before starting nova https://review.openstack.org/168407 | 18:55 |
*** gfidente has quit IRC | 18:55 | |
*** Rockyg has joined #openstack-qa | 18:57 | |
*** achanda has joined #openstack-qa | 18:57 | |
*** achanda has quit IRC | 18:57 | |
*** achanda has joined #openstack-qa | 18:58 | |
*** nikil22 has joined #openstack-qa | 19:07 | |
nikil22 | hi is there a way to restart all openstack service or at-least neutron related service in devstack through a script. Because i use to kill all screen and rejoin.sh or manually move to particular screen ctrl+c up arror and start again. But i need to make a script to restart all service . Is it possible without starting a screen? | 19:09 |
openstackrecheck | openstack/nova change: https://review.openstack.org/152997 failed gate-grenade-dsvm-ironic-sideways in the gate queue with an unrecognized error | 19:09 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Embed network resoruces and credentials in TestResources https://review.openstack.org/167830 | 19:10 |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add network support to the accounts providers https://review.openstack.org/167988 | 19:10 |
mtreinish | andreaf: ^^^ my patch still won't work because I need to init a client to get the network list, but I think that's the last piece | 19:12 |
mtreinish | andreaf: also I had to rebase everything on https://review.openstack.org/168981 | 19:13 |
*** Nic has joined #openstack-qa | 19:15 | |
*** Sukhdev has quit IRC | 19:20 | |
*** annashen has quit IRC | 19:22 | |
openstackrecheck | Console logs not available after 13:27s for gate-glance-docs 119174,71,0a8d820 | 19:27 |
*** cdent has quit IRC | 19:30 | |
*** adalbas has quit IRC | 19:31 | |
*** annashen has joined #openstack-qa | 19:34 | |
*** adalbas has joined #openstack-qa | 19:45 | |
andreaf | mtreinish: ok great, I'll start with reviewing https://review.openstack.org/168981 | 19:50 |
lifeless | sdague: ping (re https://github.com/pypa/pip/issues/2612) | 19:58 |
*** jecarey has quit IRC | 20:24 | |
openstackgerrit | Merged openstack-dev/devstack: add back python-guestfs https://review.openstack.org/169029 | 20:26 |
*** afazekas has joined #openstack-qa | 20:27 | |
*** melwitt has quit IRC | 20:33 | |
*** melwitt_ has joined #openstack-qa | 20:33 | |
*** afazekas has quit IRC | 20:34 | |
*** artom has joined #openstack-qa | 20:36 | |
*** afazekas has joined #openstack-qa | 20:37 | |
*** mwagner_lap has quit IRC | 20:40 | |
*** afazekas has quit IRC | 20:44 | |
*** e0ne has quit IRC | 20:46 | |
*** Sukhdev has joined #openstack-qa | 20:46 | |
*** Rockyg has quit IRC | 20:48 | |
*** jecarey has joined #openstack-qa | 20:50 | |
*** rfolco has quit IRC | 20:54 | |
*** adalbas has quit IRC | 20:56 | |
*** yamamoto has joined #openstack-qa | 20:58 | |
*** amotoki has joined #openstack-qa | 20:59 | |
*** melwitt_ has quit IRC | 21:01 | |
*** melwitt has joined #openstack-qa | 21:01 | |
openstackgerrit | Merged openstack/tempest: Merge availability_zone response schema into one file https://review.openstack.org/162816 | 21:01 |
*** melwitt has quit IRC | 21:02 | |
mtreinish | dkranz: if you're around any thoughts on: https://review.openstack.org/#/c/168981/ ? | 21:02 |
dkranz | mtreinish: I saw that go by and it made sense to me. I'll review it in a few minutes | 21:03 |
dkranz | mtreinish: I suppose the README has to be updated as well, right? | 21:04 |
mtreinish | is there something in the readme about that? | 21:04 |
*** afazekas has joined #openstack-qa | 21:06 | |
*** melwitt has joined #openstack-qa | 21:09 | |
*** chandankumar has quit IRC | 21:11 | |
*** dustins has quit IRC | 21:11 | |
dkranz | mtreinish: oh, sorry. It was in doc/source/configuration | 21:13 |
*** riwinters has quit IRC | 21:17 | |
*** dpaterson has quit IRC | 21:18 | |
*** boris-42 has quit IRC | 21:18 | |
*** afazekas has quit IRC | 21:18 | |
openstackgerrit | David Kranz proposed openstack/tempest: Prevent failures when running as non-admin https://review.openstack.org/166955 | 21:18 |
mriedem | mtreinish: have you seen this one? http://logs.openstack.org/46/168646/1/check/check-grenade-dsvm-ironic-sideways/59aadb5/console.html#_2015-03-30_19_31_10_311 | 21:21 |
mtreinish | mriedem: https://bugs.launchpad.net/grenade/+bug/1438415 | 21:21 |
openstack | Launchpad bug 1438415 in grenade "Some network tests failing with IndexError" [Undecided,New] | 21:21 |
mtreinish | mriedem: yep just a few mins ago | 21:21 |
mtreinish | although I'm wondering if we should just skip lbaas on ironic | 21:22 |
mtreinish | I mean who's going to do that combo | 21:22 |
mtreinish | adam_g: ^^^ | 21:22 |
adam_g | mtreinish, hey, i just noticed the same and looking into it now | 21:22 |
mriedem | hmm, what changed recently? | 21:22 |
mtreinish | mriedem: we dropped the devstack network name as the default value for compute.fixed_network_name | 21:23 |
mtreinish | mriedem: in http://git.openstack.org/cgit/openstack/tempest/commit/?id=03feae04e670e0f4f7a950628ce0c9ec3025fba1 | 21:24 |
mriedem | mtreinish: why isn't this trace in tempest.txt.gz? http://logs.openstack.org/46/168646/1/check/check-grenade-dsvm-ironic-sideways/59aadb5/console.html#_2015-03-30_19_31_10_311 | 21:27 |
*** ibiris is now known as ibiris_away | 21:28 | |
openstackgerrit | Adam Gandelman proposed openstack-dev/devstack: Set fixed_network_name in Tempest config https://review.openstack.org/169095 | 21:28 |
mtreinish | mriedem: either it's happening before the logging fixture is started, or the fixtures are misconfigured and stealing the traceback | 21:29 |
*** eugene_bag1 has quit IRC | 21:29 | |
adam_g | https://review.openstack.org/169095 and its juno cousin should fix it | 21:29 |
*** boris-42 has joined #openstack-qa | 21:30 | |
mtreinish | adam_g: a corresponding tempest fix would be to catch that and print out a useful InvalidConfiguration exception. (if you're so inclined :) ) | 21:30 |
*** cbader has quit IRC | 21:30 | |
adam_g | mtreinish, sure | 21:30 |
mtreinish | adam_g: also yoy might need it on icehouse | 21:31 |
adam_g | mtreinish, why for? tempest doesn't run against ironic icehouse anywhere | 21:31 |
mtreinish | adam_g: well a couple tests get skipped if it's not set, not a big deal either way | 21:32 |
mtreinish | I don't really care | 21:32 |
*** dkranz has quit IRC | 21:34 | |
*** yamahata has quit IRC | 21:34 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for ironic sideways bug 1438415 https://review.openstack.org/169099 | 21:35 |
openstack | bug 1438415 in devstack "Some network tests failing with IndexError" [Undecided,In progress] https://launchpad.net/bugs/1438415 | 21:35 |
openstackgerrit | Adam Gandelman proposed openstack/tempest: Print an error message and fail if network could not be found by name https://review.openstack.org/169100 | 21:36 |
*** jamielennox|away is now known as jamielennox | 21:36 | |
*** ajmiller_ has joined #openstack-qa | 21:45 | |
*** mriedem is now known as mriedem_away | 21:45 | |
*** jamielennox is now known as jamielennox|away | 21:46 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for ironic sideways bug 1438415 https://review.openstack.org/169099 | 21:48 |
openstack | bug 1438415 in devstack "Some network tests failing with IndexError" [Undecided,In progress] https://launchpad.net/bugs/1438415 | 21:48 |
*** ajmiller has quit IRC | 21:48 | |
mtreinish | mriedem_away: ^^^ | 21:48 |
*** jamielennox|away is now known as jamielennox | 21:53 | |
mriedem_away | <3 | 21:54 |
*** aysyd has quit IRC | 21:55 | |
andreaf | mtreinish: I will wait for https://review.openstack.org/#/c/168981/ to merge, then I'll submit a new version of https://review.openstack.org/#/c/167830/13 that addresses masayukig's comment | 21:56 |
openstackgerrit | Aaron Rosen proposed openstack-infra/devstack-gate: Allow SERVICE_TIMEOUT to be configured https://review.openstack.org/169103 | 21:56 |
*** annashen has quit IRC | 21:57 | |
mtreinish | andreaf: oh, I didn't even notice it had a -1 | 21:57 |
*** arosen has joined #openstack-qa | 22:00 | |
arosen | Hi I'm curious if you guys see a lot of CI failures like this: | 22:00 |
arosen | 2015-03-30 21:23:31.683 | [ERROR] /opt/stack/new/devstack/lib/ceilometer:351 ceilometer-api did not start | 22:00 |
arosen | i see the same thing for cinder-api | 22:00 |
arosen | and sometimes glance. | 22:00 |
arosen | though if i look at the ceilometer-api logs It seems like it's running: http://logs2.aaronorosen.com/11/169011/1/check/dsvm-tempest-full-congress-nodepool/b779c78/logs/screen-ceilometer-api.txt.gz#_2015-03-30_21_23_37_360 | 22:00 |
clarkb | sdague: dtroyer https://review.openstack.org/#/c/168407/7/extras.d/99-cpu_map.sh is working now and the aiopcpu tests pass with that change in place. So question for you all is how do I make that more devstacky? I don't think we need to update the cpu_map.xml by default and suggestions on a better way to parse XML? though the current method seems to work finally | 22:02 |
dtroyer | clarkb: sdague and I talked about that a bit last week. One conclusion was to use a real XML parser to make those changes rather than attempt to do it in shell. although the options there need to be considered, ie, I don't want to wind up with lxml as a DevStack prereq. | 22:04 |
clarkb | dtroyer: python's stdlib xml module should be sufficient | 22:05 |
andreaf | mtreinish, masayukig, sdague: regarding ssh-auth-bp, the WIP has been lifted: https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/ssh-auth-strategy,n,z | 22:05 |
clarkb | dtroyer: I could write a tiny python script to do it and embed that in devstack | 22:05 |
dtroyer | clarkb: I think that is what sdague had in mind. we'll still want to call it from a script like you have though to do the phase checking | 22:06 |
dtroyer | clarkb: does that need to be re-run on every stack.sh run? | 22:06 |
dtroyer | or is it a one-time setup thing? | 22:06 |
clarkb | dtroyer: its a one time setup thing, which sdague also mentioned I needed, idempotency which is doable with python script | 22:07 |
clarkb | dtroyer: basically if gate64 cpu model exists then return else add it | 22:07 |
dtroyer | clarkb: ok, I'll leave a couple of other comments on some details | 22:09 |
clarkb | dtroyer: awesome, thank you | 22:09 |
*** mhorban has quit IRC | 22:10 | |
openstackgerrit | Merged openstack/tempest: Only use accounts.yaml with locking provider https://review.openstack.org/168981 | 22:10 |
*** amotoki has quit IRC | 22:10 | |
mtreinish | andreaf: ^^^ | 22:11 |
*** openstackrecheck has quit IRC | 22:14 | |
*** openstackrecheck has joined #openstack-qa | 22:14 | |
*** dboik_ has joined #openstack-qa | 22:15 | |
*** yamamoto has quit IRC | 22:15 | |
*** dboik has quit IRC | 22:18 | |
*** sabeen1 has quit IRC | 22:18 | |
*** dboik_ has quit IRC | 22:19 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Initial class creds creation in test base class https://review.openstack.org/153681 | 22:21 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Embed network resoruces and credentials in TestResources https://review.openstack.org/167830 | 22:21 |
nikil22 | @clarkb is there any available script to restart all service in devstack , without using screen? | 22:21 |
*** mhorban has joined #openstack-qa | 22:22 | |
andreaf | mtreinish: ^^^ | 22:22 |
clarkb | nikil22: unstack then stack? | 22:26 |
nikil22 | clarkb: If I do that all bridge settings will go | 22:27 |
*** pc_m has quit IRC | 22:34 | |
openstackgerrit | Joe Hakim Rahme proposed openstack/tempest: Allows Javelin to specify Swift role https://review.openstack.org/169108 | 22:37 |
openstackgerrit | Adam Gandelman proposed openstack/tempest: Print an error message and fail if network could not be found by name https://review.openstack.org/169100 | 22:42 |
*** annashen has joined #openstack-qa | 22:42 | |
*** dboik_ has joined #openstack-qa | 22:43 | |
*** harlowja has quit IRC | 22:45 | |
*** harlowja_ has joined #openstack-qa | 22:45 | |
*** Sukhdev has quit IRC | 22:46 | |
*** marrusl has quit IRC | 22:48 | |
*** dimtruck is now known as zz_dimtruck | 22:58 | |
*** annashen has quit IRC | 23:00 | |
*** marrusl has joined #openstack-qa | 23:01 | |
*** rbak has quit IRC | 23:02 | |
mtreinish | dkranz, andreaf: btw we need: https://review.openstack.org/169042 to make the non-admin jobs do what we expect | 23:04 |
mtreinish | right now they'll run with admin enabled | 23:04 |
openstackgerrit | Ian Wienand proposed openstack-dev/devstack: [WIP] Testing pip version of pyOpenSSL https://review.openstack.org/169112 | 23:06 |
*** patrickeast_ has joined #openstack-qa | 23:08 | |
*** patrickeast has quit IRC | 23:08 | |
*** patrickeast_ is now known as patrickeast | 23:08 | |
*** melwitt has quit IRC | 23:17 | |
*** dratushnyy has joined #openstack-qa | 23:19 | |
*** hemna is now known as hemnafk | 23:20 | |
arosen | mtreinish: if you have a sec do you have any idea why i'm getting this? http://paste.openstack.org/show/197632/ | 23:26 |
mtreinish | arosen: oh, because we changed the default there to None | 23:27 |
mtreinish | facepalm that's an obvious bug how did that get through | 23:27 |
arosen | mtreinish: default where? | 23:27 |
arosen | mtreinish: I might need to make a change to the congress tempest files | 23:28 |
mtreinish | arosen: a default in the tempest config file for auth.test_accounts_file: https://review.openstack.org/#/c/168981/ | 23:28 |
mtreinish | arosen: it's an obvious bug in the code, in the is_admin_available function we never take into account that CONF.auth.test_accounts_file could be None | 23:29 |
*** mattoliverau has quit IRC | 23:29 | |
mtreinish | so it's trying to do an isfile(None) which causes that error | 23:29 |
arosen | mtreinish: gotcha | 23:29 |
*** mattoliverau has joined #openstack-qa | 23:29 | |
mtreinish | arosen: it's an easy patch if you want the honors | 23:30 |
arosen | sure | 23:30 |
mtreinish | arosen: qq though you're not using tenant isolation in the congress stuff? | 23:30 |
arosen | mtreinish: we will soon though we aren't because a badly written tempest test we have that i haven't rewritten. | 23:31 |
arosen | we need to rewrite our congress tempest tests to do so | 23:31 |
mtreinish | ah, ok. Well you caught a legit bug because of that | 23:31 |
arosen | if we don't use tenant isolation doesn't it run in parallel? | 23:31 |
mtreinish | arosen: you need tenant isolation to run in parallel (or use an accounts.yaml file) | 23:32 |
mtreinish | arosen: http://docs.openstack.org/developer/tempest/configuration.html#auth-credentials | 23:32 |
arosen | mtreinish: right so we want to run in serial right now for the congress tests. | 23:32 |
arosen | but i'll be fixing that soon so we don't need to (just need to update the tests). | 23:33 |
openstackgerrit | Aaron Rosen proposed openstack/tempest: test_accounts_file can be None https://review.openstack.org/169115 | 23:34 |
arosen | mtreinish: ^ | 23:34 |
mtreinish | arosen: reviewed, I think we just need that in one more place | 23:36 |
openstackgerrit | Aaron Rosen proposed openstack/tempest: test_accounts_file can be None https://review.openstack.org/169115 | 23:37 |
arosen | mtreinish: ah good catch. Fixed ^ | 23:38 |
*** dims has quit IRC | 23:38 | |
mtreinish | hogepodge: ^^^ sry I think I broke things again... | 23:39 |
*** tochi has joined #openstack-qa | 23:45 | |
openstackgerrit | Merged openstack/tempest: Add API tests for Neutron DVR extension https://review.openstack.org/98235 | 23:49 |
*** hichihara has joined #openstack-qa | 23:50 | |
*** weshay has quit IRC | 23:58 | |
*** mahito has joined #openstack-qa | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!