| *** ianw has quit IRC | 00:02 | |
| *** ianw has joined #openstack-qa | 00:03 | |
| jogo | dtroyer: ping https://review.openstack.org/#/c/107380/ | 00:05 |
|---|---|---|
| jogo | dtroyer: I was hoping to get your feedback on that grenade patch as I think its needed and your the only other grenade core around this week | 00:06 |
| *** gfidente has quit IRC | 00:10 | |
| *** flwang_ has quit IRC | 00:13 | |
| *** dims_ has joined #openstack-qa | 00:18 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: WIP Ironic migration testing https://review.openstack.org/111859 | 00:25 |
| *** dimtruck is now known as zz_dimtruck | 00:25 | |
| openstackgerrit | badveli_vishnuus proposed a change to openstack/tempest: Add support for Service Group v2.0 API to Tempest clients https://review.openstack.org/113409 | 00:27 |
| *** marun has quit IRC | 00:46 | |
| *** marun has joined #openstack-qa | 00:47 | |
| *** mlavalle has quit IRC | 01:11 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Remove code according to TODO comment https://review.openstack.org/113414 | 01:12 |
| *** aysyd has quit IRC | 01:13 | |
| *** xianghuihui has quit IRC | 01:21 | |
| *** xianghui has joined #openstack-qa | 01:26 | |
| *** jyuso has joined #openstack-qa | 01:27 | |
| *** xianghui has quit IRC | 01:34 | |
| *** xianghuihui has joined #openstack-qa | 01:34 | |
| *** yamahata has joined #openstack-qa | 01:36 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Relax check for router interface in _verify_network_details https://review.openstack.org/113420 | 01:37 |
| *** suneelb has quit IRC | 01:37 | |
| *** marun has quit IRC | 01:38 | |
| *** marun has joined #openstack-qa | 01:40 | |
| *** suneelb has joined #openstack-qa | 01:42 | |
| *** nosnos has joined #openstack-qa | 01:43 | |
| *** nosnos has quit IRC | 01:45 | |
| *** nosnos has joined #openstack-qa | 01:45 | |
| *** nosnos has quit IRC | 01:46 | |
| *** nosnos has joined #openstack-qa | 01:47 | |
| *** dims_ has quit IRC | 01:50 | |
| *** suneelb has quit IRC | 01:50 | |
| *** weshay has quit IRC | 02:01 | |
| *** dims_ has joined #openstack-qa | 02:02 | |
| *** flwang_ has joined #openstack-qa | 02:14 | |
| *** flwang_ has quit IRC | 02:18 | |
| *** Poornima has joined #openstack-qa | 02:21 | |
| *** adalbas has quit IRC | 02:33 | |
| *** dims_ has quit IRC | 02:39 | |
| *** dims_ has joined #openstack-qa | 02:40 | |
| *** rbak has joined #openstack-qa | 02:42 | |
| *** dims_ has quit IRC | 02:44 | |
| *** pcrews has quit IRC | 02:49 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a config option to select compute xml api https://review.openstack.org/99737 | 02:50 |
| *** dpaterson has quit IRC | 02:55 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix errors on import with service tags https://review.openstack.org/111635 | 02:58 |
| *** dims_ has joined #openstack-qa | 03:00 | |
| *** dims_ has quit IRC | 03:01 | |
| *** dims_ has joined #openstack-qa | 03:02 | |
| *** Vaidy has joined #openstack-qa | 03:04 | |
| *** jgrimm has joined #openstack-qa | 03:05 | |
| *** dims_ has quit IRC | 03:06 | |
| *** ArxCruz has quit IRC | 03:12 | |
| *** jyuso has quit IRC | 03:12 | |
| *** jyuso has joined #openstack-qa | 03:12 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Fix check-sanity + save-state /w disabled services https://review.openstack.org/113405 | 03:17 |
| *** nosnos has quit IRC | 03:22 | |
| *** Poornima has quit IRC | 03:31 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Add console_output compute feature flag https://review.openstack.org/107203 | 03:31 |
| *** dims_ has joined #openstack-qa | 03:32 | |
| *** rwsu has quit IRC | 03:38 | |
| *** dims_ has quit IRC | 03:39 | |
| *** KimJ has quit IRC | 03:42 | |
| Puneet | andreaf: | 03:51 |
| Puneet | andreaf: Hi | 03:51 |
| *** chandankumar has joined #openstack-qa | 03:55 | |
| *** rfolco has quit IRC | 03:58 | |
| *** Poornima has joined #openstack-qa | 03:58 | |
| openstackgerrit | A change was merged to openstack/tempest: Add a new credential provider to use a list of creds https://review.openstack.org/107371 | 04:02 |
| openstackgerrit | A change was merged to openstack/tempest: Remove skipping flavor_access_add/remove related tests https://review.openstack.org/112718 | 04:13 |
| *** HenryG is now known as HenryG_afk | 04:14 | |
| *** flwang_ has joined #openstack-qa | 04:15 | |
| *** marun has quit IRC | 04:17 | |
| *** flwang_ has quit IRC | 04:19 | |
| *** chandankumar has quit IRC | 04:20 | |
| *** nosnos has joined #openstack-qa | 04:25 | |
| *** rbak has quit IRC | 04:26 | |
| *** rbak has joined #openstack-qa | 04:27 | |
| *** yfried has quit IRC | 04:29 | |
| *** Madan has joined #openstack-qa | 04:39 | |
| *** ahirshbe has joined #openstack-qa | 04:41 | |
| openstackgerrit | A change was merged to openstack-dev/grenade: Remove deprecated glance commands from javelin setup https://review.openstack.org/109724 | 04:53 |
| openstackgerrit | A change was merged to openstack-dev/grenade: move from screen dumping to using worlddump tool https://review.openstack.org/109123 | 04:53 |
| *** ArxCruz has joined #openstack-qa | 04:56 | |
| *** ildikov has quit IRC | 05:03 | |
| *** jamielen- has joined #openstack-qa | 05:07 | |
| *** Longgeek has joined #openstack-qa | 05:07 | |
| *** ianw has quit IRC | 05:08 | |
| *** ianw has joined #openstack-qa | 05:08 | |
| *** jamielennox has quit IRC | 05:08 | |
| *** enikanorov has quit IRC | 05:10 | |
| *** enikanorov has joined #openstack-qa | 05:11 | |
| *** gema has joined #openstack-qa | 05:12 | |
| *** jaypipes has joined #openstack-qa | 05:14 | |
| *** ChanServ sets mode: +o jaypipes | 05:14 | |
| *** gema_ has quit IRC | 05:15 | |
| *** jamielen- is now known as jamielennox | 05:15 | |
| *** chandankumar has joined #openstack-qa | 05:16 | |
| *** chandankumar has quit IRC | 05:17 | |
| *** chandankumar has joined #openstack-qa | 05:17 | |
| *** gema_ has joined #openstack-qa | 05:17 | |
| *** gema has quit IRC | 05:18 | |
| *** gema_ has quit IRC | 05:22 | |
| *** gema has joined #openstack-qa | 05:22 | |
| *** dims_ has joined #openstack-qa | 05:23 | |
| Puneet | oomichi: hi | 05:23 |
| Puneet | oomichi: Can you please review https://review.openstack.org/#/c/112780/ | 05:24 |
| *** gmann has quit IRC | 05:24 | |
| oomichi | Puneet: I see, will see it later :) | 05:24 |
| *** gema has quit IRC | 05:27 | |
| *** gema has joined #openstack-qa | 05:27 | |
| oomichi | Puneet: done | 05:28 |
| openstackgerrit | Takashi Sogabe proposed a change to openstack-dev/devstack: Fix specifying a wrong password in creating swift user https://review.openstack.org/113437 | 05:28 |
| *** dims_ has quit IRC | 05:29 | |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test case for DVR test router interface https://review.openstack.org/113212 | 05:29 |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Add API tests for Neutron DVR extension https://review.openstack.org/98235 | 05:29 |
| *** gema_ has joined #openstack-qa | 05:32 | |
| openstackgerrit | Ravikumar Venkatesan proposed a change to openstack/tempest: test_tokens_nocatalog to identity v3 test_tokens https://review.openstack.org/93172 | 05:33 |
| *** Vaidy has quit IRC | 05:34 | |
| *** gema has quit IRC | 05:35 | |
| *** k4n0 has joined #openstack-qa | 05:37 | |
| *** gema has joined #openstack-qa | 05:37 | |
| *** gema_ has quit IRC | 05:38 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: address bashate style violations https://review.openstack.org/101268 | 05:40 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Consult target when deciding to upgrade service https://review.openstack.org/112707 | 05:40 |
| *** ianw has quit IRC | 05:42 | |
| *** ianw has joined #openstack-qa | 05:42 | |
| *** ianw has quit IRC | 05:43 | |
| *** ianw has joined #openstack-qa | 05:43 | |
| *** ianw has quit IRC | 05:43 | |
| *** jamielen| has joined #openstack-qa | 05:44 | |
| *** ianw has joined #openstack-qa | 05:44 | |
| *** ianw has quit IRC | 05:44 | |
| *** jamielen| has quit IRC | 05:44 | |
| *** yfried has joined #openstack-qa | 05:44 | |
| *** ianw has joined #openstack-qa | 05:44 | |
| *** ianw has quit IRC | 05:45 | |
| *** ianw has joined #openstack-qa | 05:45 | |
| *** jamielen| has joined #openstack-qa | 05:45 | |
| *** jamielen| has quit IRC | 05:45 | |
| *** ianw has quit IRC | 05:46 | |
| *** jamielennox has quit IRC | 05:46 | |
| *** ianw has joined #openstack-qa | 05:46 | |
| *** jamielennox has joined #openstack-qa | 05:47 | |
| openstackgerrit | Preethi Dsilva proposed a change to openstack/tempest: Added test to check DVR interface add using subnet https://review.openstack.org/113440 | 05:47 |
| *** ianw has quit IRC | 05:50 | |
| *** ianw has joined #openstack-qa | 05:51 | |
| openstackgerrit | Puneet Arora proposed a change to openstack/tempest: Added neutron cli test case https://review.openstack.org/112780 | 05:55 |
| *** ianw has quit IRC | 06:01 | |
| *** jamielen- has joined #openstack-qa | 06:01 | |
| *** bdperkin has quit IRC | 06:02 | |
| *** jamielennox has quit IRC | 06:05 | |
| *** rdekel has joined #openstack-qa | 06:05 | |
| *** bdperkin has joined #openstack-qa | 06:07 | |
| *** evgenyf has joined #openstack-qa | 06:07 | |
| *** gmann has joined #openstack-qa | 06:09 | |
| *** ildikov has joined #openstack-qa | 06:14 | |
| *** ArxCruz has quit IRC | 06:15 | |
| *** flwang_ has joined #openstack-qa | 06:16 | |
| *** oomichi has quit IRC | 06:16 | |
| *** Longgeek has quit IRC | 06:19 | |
| *** flwang_ has quit IRC | 06:20 | |
| *** Longgeek has joined #openstack-qa | 06:21 | |
| *** ildikov has quit IRC | 06:27 | |
| *** vaidy-nick has joined #openstack-qa | 06:28 | |
| openstackgerrit | Puneet Arora proposed a change to openstack/tempest: Added neutron cli test case https://review.openstack.org/112780 | 06:33 |
| openstackgerrit | garyk proposed a change to openstack-dev/devstack: Update deprecated service_neutron_metadata_proxy variable https://review.openstack.org/113448 | 06:41 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Enable custom BUILD_TIMEOUT https://review.openstack.org/108028 | 06:43 |
| *** rdekel has quit IRC | 06:50 | |
| *** ildikov has joined #openstack-qa | 06:54 | |
| mkoderer | andreaf: hi, I was on vacation yesterday. Which reviews are still open? | 06:58 |
| *** SridharG has joined #openstack-qa | 07:12 | |
| *** markus_z has joined #openstack-qa | 07:13 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Migrate test_server_basic_ops to tempest client https://review.openstack.org/108902 | 07:21 |
| *** sballe has joined #openstack-qa | 07:27 | |
| *** salv-orlando has quit IRC | 07:27 | |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Migrate test_server_advanced_ops to tempest client https://review.openstack.org/108906 | 07:28 |
| openstackgerrit | shihanzhang proposed a change to openstack-dev/devstack: Support ipset for security group https://review.openstack.org/113453 | 07:30 |
| *** afazekas has joined #openstack-qa | 07:31 | |
| openstackgerrit | prabhu murthy proposed a change to openstack/tempest: Added test for DVR device owner check using port https://review.openstack.org/113455 | 07:41 |
| *** jlanoux has joined #openstack-qa | 07:46 | |
| yfried | andreaf: ping | 07:50 |
| *** flwang_ has joined #openstack-qa | 07:51 | |
| *** qba73 has joined #openstack-qa | 07:53 | |
| *** jpich has joined #openstack-qa | 07:53 | |
| *** Krast has joined #openstack-qa | 07:55 | |
| *** gszasz has joined #openstack-qa | 07:56 | |
| *** Krast has quit IRC | 08:02 | |
| *** gema has quit IRC | 08:04 | |
| *** gema has joined #openstack-qa | 08:04 | |
| *** dosaboy_ has joined #openstack-qa | 08:05 | |
| *** Krast has joined #openstack-qa | 08:06 | |
| *** Dafna has joined #openstack-qa | 08:07 | |
| *** gmann has quit IRC | 08:11 | |
| *** rdekel has joined #openstack-qa | 08:12 | |
| *** jlibosva has joined #openstack-qa | 08:14 | |
| *** qba73_ has joined #openstack-qa | 08:22 | |
| *** gfidente has joined #openstack-qa | 08:22 | |
| *** gfidente has quit IRC | 08:22 | |
| *** mkollaro has joined #openstack-qa | 08:23 | |
| *** qba73_ has quit IRC | 08:23 | |
| *** andreaf_ has joined #openstack-qa | 08:23 | |
| *** andreaf has quit IRC | 08:23 | |
| *** jamielen- is now known as jamielennox|away | 08:24 | |
| yfried | SridharG: ping | 08:24 |
| *** gfidente has joined #openstack-qa | 08:25 | |
| *** qba73 has quit IRC | 08:25 | |
| *** dims_ has joined #openstack-qa | 08:26 | |
| *** yfried is now known as yfried_afk | 08:27 | |
| *** dims_ has quit IRC | 08:31 | |
| *** gema has quit IRC | 08:33 | |
| *** gema has joined #openstack-qa | 08:33 | |
| *** dosaboy_ has quit IRC | 08:36 | |
| *** gema has quit IRC | 08:36 | |
| *** andreaf has joined #openstack-qa | 08:37 | |
| *** Krast has quit IRC | 08:40 | |
| *** andreaf_ has quit IRC | 08:40 | |
| *** Krast has joined #openstack-qa | 08:40 | |
| *** andreaf_ has joined #openstack-qa | 08:42 | |
| *** gema has joined #openstack-qa | 08:45 | |
| *** andreaf has quit IRC | 08:45 | |
| *** dosaboy_ has joined #openstack-qa | 08:46 | |
| *** dosaboy_ is now known as dosaboy-alt | 08:46 | |
| *** dosaboy-alt has quit IRC | 08:47 | |
| *** gmann has joined #openstack-qa | 08:54 | |
| openstackgerrit | Puneet Arora proposed a change to openstack/tempest: Added neutron cli test case https://review.openstack.org/112780 | 08:55 |
| *** mkollaro has quit IRC | 09:04 | |
| *** mkollaro1 has joined #openstack-qa | 09:04 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Fix "NotFound" error in _clear_stacks() https://review.openstack.org/113466 | 09:05 |
| openstackgerrit | nithya-ganesan proposed a change to openstack/tempest: Migrate test_snapshot_pattern to tempest client https://review.openstack.org/110426 | 09:06 |
| *** jyuso has quit IRC | 09:08 | |
| *** mkollaro has joined #openstack-qa | 09:11 | |
| *** mkollaro1 has quit IRC | 09:11 | |
| *** mkollaro has quit IRC | 09:14 | |
| *** mkollaro has joined #openstack-qa | 09:14 | |
| *** mkollaro has quit IRC | 09:14 | |
| *** mkollaro has joined #openstack-qa | 09:15 | |
| *** mkollaro has quit IRC | 09:18 | |
| *** niedbalski has quit IRC | 09:18 | |
| *** mkollaro has joined #openstack-qa | 09:19 | |
| *** dosaboy has quit IRC | 09:20 | |
| *** dosaboy has joined #openstack-qa | 09:23 | |
| *** niedbalski has joined #openstack-qa | 09:24 | |
| *** evgenyf has quit IRC | 09:27 | |
| *** evgenyf has joined #openstack-qa | 09:32 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Don't set tunnel_types on VLAN network type https://review.openstack.org/111780 | 09:32 |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/tempest: Improve IPV6 parity in Security Group testcases https://review.openstack.org/94130 | 09:37 |
| openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Configure mtu for ovs with the common protocols https://review.openstack.org/112523 | 09:39 |
| *** andreaf has joined #openstack-qa | 09:40 | |
| *** andreaf has quit IRC | 09:40 | |
| *** andreaf has joined #openstack-qa | 09:41 | |
| *** andreaf_ has quit IRC | 09:42 | |
| *** tosky has joined #openstack-qa | 09:44 | |
| *** andreaf has quit IRC | 09:46 | |
| *** zhiyan has quit IRC | 09:47 | |
| *** sweston has quit IRC | 09:50 | |
| *** zhiyan has joined #openstack-qa | 09:52 | |
| *** sweston has joined #openstack-qa | 09:54 | |
| *** gmann has quit IRC | 09:55 | |
| *** zhiyan has quit IRC | 09:57 | |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/tempest: Add IPv4/IPv6 Provider Network Scenario tests https://review.openstack.org/100143 | 09:58 |
| *** mkollaro has quit IRC | 09:59 | |
| *** zhiyan has joined #openstack-qa | 10:00 | |
| yfried_afk | SridharG: here? | 10:01 |
| *** andreaf has joined #openstack-qa | 10:04 | |
| *** vrovachev has joined #openstack-qa | 10:04 | |
| *** vrovachev has quit IRC | 10:06 | |
| *** ianw has joined #openstack-qa | 10:14 | |
| *** dims_ has joined #openstack-qa | 10:15 | |
| *** dims_ has quit IRC | 10:20 | |
| *** vaidy-nick has quit IRC | 10:26 | |
| *** jlanoux has quit IRC | 10:30 | |
| yfried_afk | andreaf: ping | 10:33 |
| *** andreaf_ has joined #openstack-qa | 10:35 | |
| yfried_afk | andreaf_: ping | 10:35 |
| *** yfried_afk is now known as yfried | 10:35 | |
| *** andreaf has quit IRC | 10:37 | |
| *** rfolco has joined #openstack-qa | 10:37 | |
| *** jhenner has joined #openstack-qa | 10:38 | |
| *** andreaf_ has quit IRC | 10:43 | |
| *** sballe has quit IRC | 10:46 | |
| *** Krast has quit IRC | 10:50 | |
| *** andreaf has joined #openstack-qa | 11:01 | |
| *** ZZelle has joined #openstack-qa | 11:02 | |
| ZZelle | yfried, ok | 11:02 |
| yfried | ZZelle: so please repeat you question. I'm not sure I understood it | 11:02 |
| ZZelle | yfried, i have to provide a client checking return codes but i don't understand where | 11:03 |
| *** andreaf_ has joined #openstack-qa | 11:04 | |
| yfried | ZZelle: that's because network_client still has to get on board of that bp | 11:05 |
| *** cdent has joined #openstack-qa | 11:06 | |
| yfried | ZZelle: https://github.com/openstack/tempest/blob/master/tempest/api_schema/response/compute/servers.py | 11:06 |
| yfried | ^ an examle of how nova's servers_client did this | 11:07 |
| yfried | ZZelle: code should go here | 11:08 |
| yfried | https://github.com/openstack/tempest/tree/master/tempest/api_schema/response | 11:08 |
| *** andreaf has quit IRC | 11:08 | |
| yfried | ZZelle: to my knowledge, it's tempest policy not to accept new tests that go against that bp | 11:09 |
| *** Longgeek has quit IRC | 11:10 | |
| *** Longgeek has joined #openstack-qa | 11:10 | |
| ZZelle | yfried, so why do you a +1 on https://review.openstack.org/#/c/103498/9/tempest/api/network/admin/test_provider_extension.py | 11:11 |
| ZZelle | ? | 11:11 |
| *** yamahata has quit IRC | 11:11 | |
| yfried | ZZelle: I -1'ed it | 11:11 |
| *** dims_ has joined #openstack-qa | 11:12 | |
| *** mkollaro has joined #openstack-qa | 11:14 | |
| *** Longgeek has quit IRC | 11:14 | |
| ZZelle | yfried, honestly with the poor review rate on tempest and the extra work i have to do, i am asking myself if i will not abandon my tempest changes | 11:14 |
| yfried | ZZelle: about review rate, please ping me on your patches I'll try to give them my attention. the review que is really big | 11:18 |
| yfried | ZZelle: I'm sorry about the extra work. please talk to the core members and see if they might be more relaxed regarding this bp | 11:19 |
| yfried | ZZelle: I'm just a lowly regular member. I try to help with the review as much as I can, according to my knowledge. | 11:19 |
| *** ArxCruz has joined #openstack-qa | 11:20 | |
| ZZelle | yfried, i am not complaining about you but globally, 2 reviews in a month for a new contributor in tempest, the core dev associated to the change never review it | 11:26 |
| ZZelle | yfried, should i feel motivated? | 11:26 |
| *** dims_ is now known as dims | 11:28 | |
| yfried | ZZelle: " the core dev associated to the change" ? you mean the person you tagged on your patch? | 11:28 |
| ZZelle | yfried, the one responsible for tempest parity AFAIK | 11:29 |
| *** Poornima has quit IRC | 11:31 | |
| yfried | ZZelle: tempest core is very small team. they have hard time prioritizing patches. AFAIK they rely on regular reviewers so they can focus on patches that are ready to be merged. if you think that your patch deserves special attention from a specific member, you should talk to them personally or join weekly qa-meetings on IRC were we have a section dedicated to patches waiting on review | 11:32 |
| *** jamielennox|away has quit IRC | 11:34 | |
| yfried | ZZelle: take a look here https://dague.net/2013/09/27/gerrit-queries-to-avoid-openstack-review-overload/ | 11:34 |
| *** Longgeek has joined #openstack-qa | 11:34 | |
| openstackgerrit | Denis M. proposed a change to openstack-dev/devstack: Add mgmt API for datastores https://review.openstack.org/110711 | 11:35 |
| ZZelle | yfried, i am already contributing to other projects so i know about overloading | 11:35 |
| ZZelle | yfried, but when you propose to someone (a core) to implement a change instead of him and he never review, what should i think? | 11:36 |
| *** weshay has joined #openstack-qa | 11:36 | |
| yfried | ZZelle: please see private message | 11:37 |
| *** jamielennox|away has joined #openstack-qa | 11:37 | |
| *** dims has quit IRC | 11:37 | |
| *** andreaf_ has quit IRC | 11:45 | |
| *** andreaf_ has joined #openstack-qa | 11:45 | |
| *** gmann has joined #openstack-qa | 11:46 | |
| *** flwang_ has quit IRC | 11:58 | |
| yfried | andreaf_: are you around? | 11:59 |
| *** salv-orlando has joined #openstack-qa | 12:10 | |
| *** adalbas has joined #openstack-qa | 12:16 | |
| *** dkranz has joined #openstack-qa | 12:24 | |
| *** SridharG has quit IRC | 12:25 | |
| *** aysyd has joined #openstack-qa | 12:28 | |
| *** Madan has quit IRC | 12:29 | |
| *** Dafna has quit IRC | 12:44 | |
| *** Madan has joined #openstack-qa | 12:45 | |
| *** dims has joined #openstack-qa | 12:48 | |
| *** dims has joined #openstack-qa | 12:49 | |
| ZZelle | dkranz, hi | 12:49 |
| *** dims is now known as Guest75191 | 12:50 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 12:50 |
| *** k4n0 has quit IRC | 12:53 | |
| yfried | dkranz: andreaf_: ping | 12:55 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add provider extension positive test cases https://review.openstack.org/103498 | 12:58 |
| dkranz | ZZelle: hello | 13:00 |
| dkranz | yfried: pong | 13:01 |
| *** HenryG_afk is now known as HenryG | 13:02 | |
| ZZelle | dkranz, is it a strong requirement to follow https://github.com/openstack/qa-specs/blob/master/specs/client-checks-success.rst? | 13:02 |
| *** gentlesaurus is now known as annegentle | 13:02 | |
| ZZelle | dkranz, i am implementing some tests about network services but no client checks has been already implemented for neutron | 13:03 |
| dkranz | ZZelle: yes, unless there is a good reason not to | 13:03 |
| dkranz | ZZelle: BUt, it should not be a requirement for new tests if its client has not been changed already | 13:04 |
| ZZelle | dkranz, great :) | 13:04 |
| ZZelle | dkranz, thanks | 13:05 |
| dkranz | ZZelle: on the other hand, it can't hurt to put client checks in even if some tests are still also checking | 13:05 |
| *** dustins has joined #openstack-qa | 13:05 | |
| dkranz | ZZelle: then less code would have to be changed when the whole client is fixed | 13:05 |
| ZZelle | dkranz, by whole client you mean neutron client or the client for each neutron resource? | 13:07 |
| *** SridharG has joined #openstack-qa | 13:10 | |
| *** dkranz has quit IRC | 13:11 | |
| *** nosnos has quit IRC | 13:13 | |
| *** Madan has quit IRC | 13:15 | |
| *** toure has joined #openstack-qa | 13:16 | |
| *** pcrews has joined #openstack-qa | 13:25 | |
| yfried | ZZelle: neutron client has only 1 client for all resources | 13:27 |
| *** Madan has joined #openstack-qa | 13:28 | |
| *** evgenyf has quit IRC | 13:29 | |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Change ordering of ceilometer service startup https://review.openstack.org/113522 | 13:32 |
| *** rbak has quit IRC | 13:36 | |
| *** jecarey has joined #openstack-qa | 13:36 | |
| *** dkranz has joined #openstack-qa | 13:37 | |
| *** reaper has joined #openstack-qa | 13:40 | |
| *** reaper has quit IRC | 13:41 | |
| *** mriedem has joined #openstack-qa | 13:42 | |
| mkoderer | dkranz: ping | 13:46 |
| dkranz | mkoderer: pong | 13:46 |
| mkoderer | dkranz: hi there.. i am currently working on the api_schema unification | 13:47 |
| mkoderer | dkranz: if you have some time for reviews: https://review.openstack.org/#/c/112566/, https://review.openstack.org/#/c/112570/, https://review.openstack.org/#/c/112783/ | 13:48 |
| dkranz | mkoderer: will do | 13:48 |
| mkoderer | dkranz: cool thx | 13:48 |
| *** marun has joined #openstack-qa | 13:51 | |
| *** evgenyf has joined #openstack-qa | 13:53 | |
| *** cbader has joined #openstack-qa | 13:56 | |
| *** flwang_ has joined #openstack-qa | 13:59 | |
| *** Madan has quit IRC | 14:01 | |
| *** flwang_ has quit IRC | 14:03 | |
| jlibosva | dtroyer: hi | 14:03 |
| *** gszasz has quit IRC | 14:05 | |
| *** afazekas has quit IRC | 14:05 | |
| *** gszasz has joined #openstack-qa | 14:05 | |
| *** dkranz has quit IRC | 14:05 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack-dev/devstack: Ironic: Setup/Configure iPXE https://review.openstack.org/99677 | 14:06 |
| *** rbak has joined #openstack-qa | 14:08 | |
| *** zz_dimtruck is now known as dimtruck | 14:10 | |
| *** cpallares has joined #openstack-qa | 14:11 | |
| *** cpallares has quit IRC | 14:12 | |
| *** gszasz has quit IRC | 14:14 | |
| *** gszasz has joined #openstack-qa | 14:14 | |
| *** cpallares has joined #openstack-qa | 14:15 | |
| *** afazekas has joined #openstack-qa | 14:20 | |
| *** dkranz has joined #openstack-qa | 14:21 | |
| dtroyer | jlibosva: hi | 14:21 |
| jlibosva | dtroyer: I think https://review.openstack.org/#/c/97317/12 change broke neutron grenade job | 14:22 |
| jlibosva | dtroyer: I'm trying to see where it fails but I cannot find anything useful. http://logs.openstack.org/00/108700/7/check/check-grenade-dsvm-neutron/ade2494/logs/grenade.sh.txt.gz | 14:22 |
| jlibosva | dtroyer: could point me where to look why javelin2 failed? | 14:22 |
| jlibosva | dtroyer: better link: http://logs.openstack.org/00/108700/7/check/check-grenade-dsvm-neutron/ade2494/logs/grenade.sh.txt.gz#_2014-08-12_12_14_26_202 | 14:23 |
| dtroyer | unfortunately, since javelin2 is python it is going to be a lot more opaque…I'll have a look | 14:23 |
| jlibosva | thanks | 14:24 |
| *** ildikov has quit IRC | 14:30 | |
| *** salv-orlando has quit IRC | 14:33 | |
| *** Dafna has joined #openstack-qa | 14:36 | |
| mriedem | afazekas: couple small things here https://review.openstack.org/#/c/112558/ | 14:37 |
| dtroyer | jlibosva: all I can tell is javelin2 exits with non-zero code but doesn't report why. comparing to a good run the output is essentially the same | 14:39 |
| dtroyer | I don't have time to dig into it right now, but I think sorting this is going to require making javelin2 more verbose | 14:39 |
| *** SridharG has quit IRC | 14:41 | |
| jlibosva | dtroyer: thanks. is javelin2 part of tempest or where can I find source code? | 14:44 |
| *** markmcclain has joined #openstack-qa | 14:44 | |
| mtreinish | jlibosva: yeah javelin2 is in the tempest tree | 14:45 |
| jlibosva | mtreinish: thanks | 14:45 |
| afazekas | mriedem: thx | 14:46 |
| mtreinish | dtroyer: https://review.openstack.org/#/c/107380/ should fix the javelin2 verbose issue | 14:46 |
| mtreinish | dtroyer: the issue is that the log file for javelin2 isn't being captured by devstack-gate | 14:47 |
| mtreinish | so either printing to the console, or capturing the new log file will fix that | 14:47 |
| mtreinish | jlibosva: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/cmd/javelin.py | 14:48 |
| *** mkollaro has quit IRC | 14:49 | |
| jlibosva | dtroyer: mtreinish thanks for the link to patch - the error from javelin can be seen here: http://logs.openstack.org/80/107380/12/check/check-grenade-dsvm-neutron/c6f64e0/logs/grenade.sh.txt.gz#_2014-08-11_11_37_33_331 | 14:49 |
| mtreinish | jlibosva: nice so it's hard coded for nova-net and that breaks the neutron job? | 14:52 |
| jlibosva | mtreinish: yes | 14:52 |
| jlibosva | I'm reporting bug onw | 14:52 |
| jlibosva | now | 14:52 |
| *** mkollaro has joined #openstack-qa | 14:53 | |
| *** Guest75191 is now known as dims | 14:54 | |
| *** gmann has quit IRC | 14:55 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Fix errors on import with service tags https://review.openstack.org/111635 | 14:58 |
| *** markmcclain1 has joined #openstack-qa | 15:00 | |
| *** markmcclain has quit IRC | 15:01 | |
| *** jlibosva has quit IRC | 15:01 | |
| *** dkranz has quit IRC | 15:08 | |
| *** afazekas has quit IRC | 15:08 | |
| *** gszasz has quit IRC | 15:10 | |
| mtreinish | adam_g: ^^^ you were looking at that yesterday right (the javelin part not my patch) | 15:15 |
| *** jlanoux has joined #openstack-qa | 15:16 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Remove code according to TODO comment https://review.openstack.org/113414 | 15:18 |
| *** markus_z has quit IRC | 15:20 | |
| *** mlavalle has joined #openstack-qa | 15:20 | |
| *** mlavalle has quit IRC | 15:20 | |
| Shrews | mtreinish: hi! | 15:20 |
| mtreinish | Shrews: hey | 15:21 |
| *** dkranz has joined #openstack-qa | 15:21 | |
| Shrews | mtreinish: so, re: 94439 and the sleep call... yeah, i could NOT get reliable results until i added that (which i hated doing). the sync should have been enough | 15:21 |
| *** afazekas has joined #openstack-qa | 15:21 | |
| Shrews | mtreinish: not sure where the issue was, but testing it again without the sleep on trusty with the latest libs... appears to be working. going to give it several more runs, and it no failures, resubmit without the sleep | 15:22 |
| Shrews | if* no failures | 15:22 |
| mtreinish | Shrews: ok cool, yeah that would be great | 15:23 |
| Shrews | thx for making me revisit it | 15:23 |
| mtreinish | then I don't think I'd have any issues with it :) | 15:23 |
| Shrews | mtreinish: awesome sauce | 15:23 |
| mtreinish | we can chalk it up to a kernel bug pre trusty :) | 15:23 |
| Shrews | or maybe libvirt... i saw some fixed issues in 0.9.11 (new min versions) about syncing | 15:23 |
| *** johnnysouriyapho has joined #openstack-qa | 15:24 | |
| mtreinish | ah, yeah that could be it | 15:24 |
| *** johnnysouriyapho has left #openstack-qa | 15:25 | |
| *** ahirshbe has quit IRC | 15:26 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for fixed ip shortage with n-net https://review.openstack.org/112558 | 15:26 |
| *** amotoki has joined #openstack-qa | 15:27 | |
| *** johnnysouriyapho has joined #openstack-qa | 15:27 | |
| johnnysouriyapho | Does anyone know of a IRC channel for general QA topics? | 15:28 |
| * johnnysouriyapho is Vashtor | 15:29 | |
| *** johnnysouriyapho has left #openstack-qa | 15:29 | |
| *** tosky has quit IRC | 15:31 | |
| *** mlavalle has joined #openstack-qa | 15:35 | |
| *** afazekas has quit IRC | 15:35 | |
| *** rwsu has joined #openstack-qa | 15:35 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/tempest: Add ironic instance rebuild test https://review.openstack.org/94439 | 15:37 |
| Shrews | mtreinish: ^^^^ there ya go :) | 15:39 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack-dev/devstack: Ironic: Setup/Configure iPXE https://review.openstack.org/99677 | 15:51 |
| *** mkollaro has quit IRC | 15:53 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for py26/centos6 node setup bug 1355907 https://review.openstack.org/113567 | 15:55 |
| *** flwang_ has joined #openstack-qa | 16:00 | |
| *** arnaud has joined #openstack-qa | 16:00 | |
| *** flwang_ has quit IRC | 16:04 | |
| *** jlanoux has quit IRC | 16:06 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for infra setup bug 1354829 https://review.openstack.org/113575 | 16:09 |
| mriedem | jogo: ^ | 16:09 |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for infra setup bug 1354829 https://review.openstack.org/113575 | 16:10 |
| *** yfried_ has joined #openstack-qa | 16:13 | |
| *** mpaolino has joined #openstack-qa | 16:14 | |
| *** yfried has quit IRC | 16:14 | |
| *** evgenyf has quit IRC | 16:15 | |
| *** yfried__ has joined #openstack-qa | 16:16 | |
| *** arnaud has quit IRC | 16:18 | |
| *** yfried_ has quit IRC | 16:20 | |
| *** yfried__ has quit IRC | 16:22 | |
| *** rdekel has quit IRC | 16:25 | |
| *** ildikov has joined #openstack-qa | 16:27 | |
| *** chandankumar has quit IRC | 16:36 | |
| *** suneelb has joined #openstack-qa | 16:37 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for test_postgresql_opportunistically bug 1355929 https://review.openstack.org/113581 | 16:37 |
| *** markmcclain1 has quit IRC | 16:42 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for fixed ip shortage with n-net https://review.openstack.org/112558 | 16:44 |
| mtreinish | andreaf_: hey, do you have any thoughts on?: https://review.openstack.org/#/c/111467/ | 16:44 |
| *** mpaolino has quit IRC | 16:45 | |
| *** dims has quit IRC | 16:46 | |
| *** dims has joined #openstack-qa | 16:47 | |
| *** dims is now known as Guest27104 | 16:47 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for infra setup bug 1354829 https://review.openstack.org/113575 | 16:50 |
| *** Guest27104 has quit IRC | 16:51 | |
| *** SridharG has joined #openstack-qa | 16:59 | |
| *** jpich has quit IRC | 17:00 | |
| *** k4n0 has joined #openstack-qa | 17:01 | |
| *** arnaud has joined #openstack-qa | 17:01 | |
| *** mestery has quit IRC | 17:02 | |
| *** mestery has joined #openstack-qa | 17:03 | |
| *** ianw has quit IRC | 17:08 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for py26/centos6 node setup bug 1355907 https://review.openstack.org/113567 | 17:09 |
| *** SridharG has quit IRC | 17:09 | |
| *** krtaylor has quit IRC | 17:15 | |
| *** andreaf_ has quit IRC | 17:17 | |
| *** andreaf_ has joined #openstack-qa | 17:17 | |
| *** andreaf has joined #openstack-qa | 17:20 | |
| *** andreaf_ has quit IRC | 17:22 | |
| andreaf | mtreinish: welcome back - the idea sound good - basically reply your results based on run-id and use any subunit tooling available on top of it | 17:25 |
| *** kmartin has quit IRC | 17:29 | |
| *** kmartin has joined #openstack-qa | 17:30 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Remove cleanup code according to TODO comment https://review.openstack.org/113414 | 17:31 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/tempest: Relax check for router interface in _verify_network_details https://review.openstack.org/113420 | 17:32 |
| mtreinish | andreaf: yeah, that's what I thought too. I'm just not happy about self approving all those subunit2sql patches without getting other eyes on it | 17:32 |
| mtreinish | so I'm going to be bugging you about every patch I push :) | 17:32 |
| mtreinish | just because you expressed an interest in the project | 17:32 |
| *** dwalleck has joined #openstack-qa | 17:36 | |
| *** yfried__ has joined #openstack-qa | 17:36 | |
| mtreinish | andreaf: I need to respin that patch to add a TODO comment about fixing the tags attr | 17:36 |
| mtreinish | but then I'll push it through | 17:36 |
| *** dwalleck has left #openstack-qa | 17:38 | |
| *** salv-orlando has joined #openstack-qa | 17:40 | |
| *** mriedem has quit IRC | 17:41 | |
| *** dims has joined #openstack-qa | 17:47 | |
| *** dims is now known as Guest61254 | 17:48 | |
| *** flwang_ has joined #openstack-qa | 17:48 | |
| *** flwang_ has quit IRC | 17:52 | |
| yfried__ | dkranz: ping | 17:53 |
| *** mriedem has joined #openstack-qa | 17:56 | |
| dkranz | yfried__: pong | 17:56 |
| *** k4n0 has quit IRC | 17:58 | |
| openstackgerrit | Simeon Monov proposed a change to openstack/tempest: VPN/Firewall integration test scenario https://review.openstack.org/106473 | 18:00 |
| yfried__ | dkranz: you and marun had an idea about enhancing tempest client to OO mode in the way it's being done with AttributeDict. right? | 18:00 |
| *** mestery has quit IRC | 18:00 | |
| dkranz | yfried__: Yes, but it was not well loved. | 18:00 |
| yfried__ | dkranz: I was playing with something like this https://review.openstack.org/#/c/102700/8/tempest/api/network/common.py,cm | 18:01 |
| *** sabeen3 has joined #openstack-qa | 18:01 | |
| yfried__ | dkranz: do you think I should go on with it, or drop it? | 18:01 |
| *** sabeen1 has quit IRC | 18:02 | |
| dkranz | yfried__: marun and I posted a similar https://review.openstack.org/#/c/106916/ | 18:02 |
| dkranz | yfried__: If anything like this happens it will be like this: | 18:02 |
| dkranz | yfried__: 1. Individual projects develop api/functional tests using such a client | 18:03 |
| dkranz | yfried__: 2. A bunch of tempest tests are removed. | 18:03 |
| dkranz | yfried__: 3. The remaining tempest tests are changed to use a similar client. | 18:03 |
| yfried__ | dkranz: we just decided to drop all outside clients | 18:04 |
| dkranz | yfried__: Note that OO was only one part of this. The other was to remove boilerplate code from tests that does response validation, serialize, deserialize, etc. | 18:04 |
| dkranz | yfried__: No, the new functional tests will be in projects, not in tempest. They will use whatever client they decide to use. | 18:05 |
| yfried__ | dkranz: "boilerplate code from tests that does response validation, serialize, deserialize"? could you explain? | 18:05 |
| dkranz | yfried__: Look at the patch I linked to above | 18:05 |
| dkranz | yfried__: That should make it clear | 18:05 |
| *** dwalleck has joined #openstack-qa | 18:05 | |
| yfried__ | dkranz: nope. no idea what's builerpalte code | 18:06 |
| *** jamielennox|away is now known as jamielennox | 18:06 | |
| dkranz | yfried__: Oh, sorry. It is English slang for code that must always be provided and could be abstracted to somewhere else or automatically generated. | 18:07 |
| yfried__ | dkranz: ??? | 18:07 |
| yfried__ | dkranz: sorry I don't understand | 18:07 |
| dkranz | yfried__: Like having every api call check the response instead of having the client do it in one place. | 18:08 |
| yfried__ | dkranz: that's an already existing bp | 18:08 |
| dkranz | yfried__: Yes, it was an example of what "boilerplate code" is. | 18:08 |
| yfried__ | dkranz: you talked about it with ZZelle earlier | 18:08 |
| yfried__ | dkranz: so what else? | 18:09 |
| dkranz | yfried__: No, I pinged back but he did not reply. | 18:09 |
| dkranz | yfried__: Look at the patch and see what was deleted from the test methods | 18:09 |
| yfried__ | dkranz: I'm only seeing the response validation and the dict/attribute change | 18:10 |
| yfried__ | dkranz: anyway, tnx. I'm gonna finish my patch even if it's just a personal excercise in metaprograming | 18:11 |
| dkranz | yfried__: enjoy | 18:12 |
| yfried__ | dkranz: what do you think of the decorator used here? | 18:12 |
| yfried__ | https://review.openstack.org/#/c/113124/2/tempest/services/network/resources.py,cm | 18:12 |
| *** cpallares has quit IRC | 18:15 | |
| dkranz | yfried__: I am not personally fond of decorators like this. I would just replace the "pass" with a call to a method that does the call and update. | 18:17 |
| dkranz | yfried__: IMO decorators are like C macros which if used carefully can save a lot of boilerplate code but can also just make code harder to read. | 18:18 |
| *** krtaylor has joined #openstack-qa | 18:19 | |
| *** marun has quit IRC | 18:21 | |
| *** marun has joined #openstack-qa | 18:21 | |
| *** markmcclain has joined #openstack-qa | 18:21 | |
| *** Longgeek has quit IRC | 18:32 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Fix check-sanity + save-state /w disabled services https://review.openstack.org/113405 | 18:32 |
| *** suneelb has quit IRC | 18:33 | |
| *** suneelb has joined #openstack-qa | 18:34 | |
| *** mkollaro has joined #openstack-qa | 18:36 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for infra setup bug 1354829 https://review.openstack.org/113575 | 18:43 |
| *** catohornet has joined #openstack-qa | 18:45 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for ceilometer py26 bug 1355983 https://review.openstack.org/113612 | 18:47 |
| *** cdent has quit IRC | 18:48 | |
| *** sdmonov has joined #openstack-qa | 18:50 | |
| *** ianw has joined #openstack-qa | 18:58 | |
| *** suneelb has quit IRC | 18:59 | |
| *** suneelb has joined #openstack-qa | 18:59 | |
| *** suneelb has quit IRC | 18:59 | |
| *** suneelb has joined #openstack-qa | 19:00 | |
| *** dimtruck is now known as zz_dimtruck | 19:00 | |
| *** rockyg has joined #openstack-qa | 19:01 | |
| *** zz_dimtruck is now known as dimtruck | 19:01 | |
| *** dimtruck is now known as zz_dimtruck | 19:11 | |
| *** gema_ has joined #openstack-qa | 19:12 | |
| *** gema has quit IRC | 19:15 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Fix MessageConfig https://review.openstack.org/113620 | 19:18 |
| jogo | mriedem: elastic-recheck fix ^ | 19:19 |
| jogo | mriedem: which is why it hasn't been commenting on patches | 19:19 |
| mriedem | jogo: ok | 19:22 |
| mriedem | was that a regression from the message catalog stuff? | 19:22 |
| mriedem | nvm, that's obvious | 19:22 |
| *** mestery has joined #openstack-qa | 19:27 | |
| *** dwalleck has quit IRC | 19:35 | |
| *** ArxCruz has quit IRC | 19:40 | |
| *** flwang_ has joined #openstack-qa | 19:49 | |
| *** marun has quit IRC | 19:50 | |
| *** marun has joined #openstack-qa | 19:50 | |
| openstackgerrit | Simeon Monov proposed a change to openstack/tempest: VPN/Firewall integration test scenario https://review.openstack.org/106473 | 19:51 |
| *** flwang_ has quit IRC | 19:53 | |
| *** dwalleck has joined #openstack-qa | 19:53 | |
| openstackgerrit | Lakshmi N Sampath proposed a change to openstack/tempest: Glance Metadata Definitions Catalog - Tempest tests https://review.openstack.org/113632 | 20:01 |
| *** amotoki has quit IRC | 20:02 | |
| *** zz_dimtruck is now known as dimtruck | 20:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-infra/elastic-recheck: Add query for Postgres startup failure https://review.openstack.org/113634 | 20:07 |
| *** dwalleck has quit IRC | 20:09 | |
| *** andreaf has quit IRC | 20:10 | |
| *** andreaf has joined #openstack-qa | 20:12 | |
| *** dwalleck has joined #openstack-qa | 20:12 | |
| *** dwalleck has quit IRC | 20:13 | |
| openstackgerrit | A change was merged to openstack/tempest: Add min_client_version decorator for CLI tests https://review.openstack.org/100031 | 20:15 |
| mtreinish | mriedem: ^^^ does that close the BP? | 20:16 |
| mriedem | mtreinish: yup | 20:17 |
| mtreinish | awesome | 20:17 |
| *** mlavalle has quit IRC | 20:18 | |
| *** gema has joined #openstack-qa | 20:18 | |
| *** gema_ has quit IRC | 20:19 | |
| *** aimon has joined #openstack-qa | 20:21 | |
| *** andreaf_ has joined #openstack-qa | 20:22 | |
| *** andreaf has quit IRC | 20:26 | |
| *** toure has quit IRC | 20:27 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Fix MessageConfig https://review.openstack.org/113620 | 20:34 |
| *** mlavalle has joined #openstack-qa | 20:36 | |
| *** Guest61254 has quit IRC | 20:41 | |
| andreaf_ | mtreinish: ping - I'm having problems with options with the same name in different groups, the config generation tool is not doing the right thing - do you know anything about this? | 20:41 |
| jgrimm | andreaf_, this problem? -> https://bugs.launchpad.net/tempest/+bug/1282887 | 20:45 |
| *** rfolco has quit IRC | 20:46 | |
| *** dims has joined #openstack-qa | 20:46 | |
| andreaf_ | jgrimm: yes exactly :) | 20:46 |
| *** dims is now known as Guest60092 | 20:46 | |
| andreaf_ | jgrimm: thanks for working on a fix | 20:48 |
| jgrimm | andreaf_, hopefully will get fix in soon by moving to new generator from oslo.config.. bug workaround in meantime can be done like this -> https://review.openstack.org/#/c/73503/ | 20:48 |
| andreaf_ | jgrimm: thanks - I'm changing the help message | 20:53 |
| mtreinish | andreaf_: yeah that's a bug with the config generator tool | 20:53 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 20:54 |
| andreaf_ | which is even more annoying because the conflict is on allow_tenant_isolation in the orchestration group which as far as I can see is not used anywhere | 20:55 |
| mtreinish | andreaf_: yeah just pull that out, there is no need for that option | 20:55 |
| mtreinish | I think there are probably a class of duplicate options like that for no reason | 20:55 |
| mtreinish | tenant isolation should be enabled everywhere or nowhere | 20:56 |
| mtreinish | not per service | 20:56 |
| andreaf_ | mtreinish: do you want it in the same change? | 20:56 |
| *** rfolco has joined #openstack-qa | 20:56 | |
| andreaf_ | https://review.openstack.org/#/c/107685/ | 20:56 |
| mtreinish | a separate change from what? BUt, no that's probably an independent change | 20:56 |
| mtreinish | oh, yeah that's definitely independent from the cred factory | 20:57 |
| openstackgerrit | Chris Dent proposed a change to openstack-dev/devstack: Change ordering of ceilometer service startup https://review.openstack.org/113522 | 20:57 |
| mtreinish | err, actually I could go either way, because you're compacting down all the options into the new auth group | 20:57 |
| mtreinish | I'll leave it up to you | 20:57 |
| jgrimm | mtreinish, speaking of https://bugs.launchpad.net/tempest/+bug/1282887 .. ok with suggestion to move to new oslo-config-generator as fix? if so, i'm interested in understanding your comment in generate_sample_tempest.py... i want to make sure i can reproduce issue/make sure i'm covering it | 20:58 |
| mtreinish | yeah switching to use the new oslo sample config generator is fine, we'll have to do that eventually anyway | 20:59 |
| mtreinish | I'm not sure if the bug is still there in the new version of the tool | 20:59 |
| jgrimm | mtreinish, cool. i verified it actually does fix the dup problem | 20:59 |
| mtreinish | jgrimm: before since we did all of the option registration at runtime the tool would fail to work correctly unless we manually registered all the ops | 21:00 |
| mtreinish | that may not be an issue anymore | 21:00 |
| jgrimm | yeah, i couldn't see any issue so i wanted to poke you for how to recreate | 21:01 |
| mtreinish | jgrimm: if you want to reproduce what that comment is about remove this file http://git.openstack.org/cgit/openstack/tempest/tree/tools/config/oslo.config.generator.rc | 21:01 |
| mtreinish | and run the tool | 21:01 |
| mtreinish | the old one | 21:01 |
| jgrimm | yeah, i did that .. couldn't see an issue.. verified that all options made it into the sample config | 21:02 |
| mtreinish | jgrimm: hmm, I get: http://paste.openstack.org/show/94029/ | 21:02 |
| jgrimm | mtreinish, oh.. lemme go look. | 21:03 |
| jgrimm | mtreinish, fwiw not getting that with calling new generator | 21:04 |
| *** suneelb has quit IRC | 21:04 | |
| mtreinish | and it's the same stack trace on py2 (just realized I forgot to switch the interpreter) http://paste.openstack.org/show/94031/ | 21:04 |
| mtreinish | jgrimm: yeah they probably fixed it for the new one | 21:04 |
| *** suneelb has joined #openstack-qa | 21:04 | |
| jgrimm | mtreinish, fwiw i have a wip review -> https://review.openstack.org/#/c/112111/ | 21:04 |
| mtreinish | because the recommended usage pattern changed to add options at runtime instead of import | 21:05 |
| *** suneelb has quit IRC | 21:05 | |
| *** sdmonov has quit IRC | 21:05 | |
| *** suneelb has joined #openstack-qa | 21:05 | |
| jgrimm | mtreinish, excellent, thanks for the history | 21:05 |
| *** suneelb has quit IRC | 21:06 | |
| *** suneelb has joined #openstack-qa | 21:06 | |
| mtreinish | jgrimm: well it was news to me too, I caught dhellmann's note on the ML :) | 21:08 |
| mtreinish | I would have just assumed we were still the black sheep for using oslo.config like this | 21:08 |
| jgrimm | heh | 21:09 |
| mtreinish | jgrimm: err, does the new generator actually require the alpha version of oslo.config? | 21:11 |
| *** aysyd has quit IRC | 21:13 | |
| *** weshay has quit IRC | 21:14 | |
| *** mlavalle has quit IRC | 21:17 | |
| *** aimon has quit IRC | 21:20 | |
| *** gfidente has quit IRC | 21:23 | |
| *** yfried__ has quit IRC | 21:25 | |
| *** dkranz has quit IRC | 21:25 | |
| *** Guest60092 is now known as dims_ | 21:27 | |
| *** marun has quit IRC | 21:33 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/devstack-gate: Collect javelin2 log https://review.openstack.org/113658 | 21:42 |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/grenade: Log javelin into in its own logfile. https://review.openstack.org/113660 | 21:43 |
| jogo | mtreinish: ^ javelin log change | 21:43 |
| mtreinish | jogo: cool, thanks | 21:43 |
| mtreinish | jogo: are you going to add an e-s indexer update after those 2 merge? | 21:44 |
| *** dims_ has quit IRC | 21:45 | |
| jogo | mtreinish: oh good idea, yes I will | 21:45 |
| jogo | once those land | 21:45 |
| mtreinish | ok cool | 21:45 |
| *** dims_ has joined #openstack-qa | 21:45 | |
| andreaf_ | mtreinish: re the scenario test migration, we need to start closing some of the initial changes, else it will become a rebase nightmare for everyone - do you have some time for reviews there? | 21:48 |
| mtreinish | andreaf_: sure I'll trade you | 21:49 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable E128 ignore E129 https://review.openstack.org/98909 | 21:49 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Enable H407,H305,H307,E122 ignore E123 https://review.openstack.org/98918 | 21:49 |
| mtreinish | andreaf_: ^^^ | 21:49 |
| mtreinish | :) | 21:49 |
| mtreinish | andreaf_: because that's been a rebase nightmare too | 21:50 |
| *** flwang_ has joined #openstack-qa | 21:50 | |
| *** sdmonov has joined #openstack-qa | 21:50 | |
| andreaf_ | mtreinish: thanks, I'll look at your changes | 21:51 |
| *** suneelb has quit IRC | 21:51 | |
| *** suneelb has joined #openstack-qa | 21:51 | |
| *** flwang_ has quit IRC | 21:54 | |
| *** cpallares has joined #openstack-qa | 21:55 | |
| *** rockyg has quit IRC | 21:59 | |
| *** sdmonov has quit IRC | 22:01 | |
| *** amotoki has joined #openstack-qa | 22:03 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/grenade: Log javelin into in its own logfile. https://review.openstack.org/113660 | 22:06 |
| *** amotoki has quit IRC | 22:08 | |
| openstackgerrit | Jon Grimm proposed a change to openstack/tempest: Move tempest to oslo-config-generator https://review.openstack.org/112111 | 22:09 |
| andreaf_ | mtreinish: where's H407 define? | 22:10 |
| andreaf_ | mtreinish: s/define/defined | 22:10 |
| *** ZZelle has left #openstack-qa | 22:10 | |
| andreaf_ | mtreinish: I just pulled the latest hacking and I don't see it | 22:10 |
| mtreinish | andreaf_: hmm, I can't find it either but it came up at some point otherwise I wouldn't have put it on the list | 22:14 |
| mtreinish | jogo: ^^^ ? | 22:14 |
| mtreinish | andreaf_: so I -1'd the first patch, but because you didn't inherit the proper classes | 22:15 |
| mtreinish | I think a matcher is fine | 22:15 |
| jogo | andreaf_: where do you see H407 being raised? | 22:16 |
| mtreinish | and then there was already a -1 on the 2nd patch in the series | 22:16 |
| mtreinish | jogo: it's on my patch https://review.openstack.org/#/c/98918/ it's not being raised but I put it on the list for some reason | 22:16 |
| mtreinish | I think that I got it when I first wrote 2 months ago | 22:17 |
| jogo | mtreinish: H407 is not a thing AFAIK | 22:17 |
| * mtreinish wonders why he can't type today | 22:17 | |
| jogo | 'git log -SH407' shows nothing in hacking | 22:18 |
| mtreinish | jogo: ok, it's more than likely that I imagined it when I was going through them all | 22:18 |
| mtreinish | there were a lot of failures after that upgrade... | 22:18 |
| jogo | mtreinish: hehe it happens to the best of us | 22:18 |
| mtreinish | andreaf_: do you want me to remove that from the commit message then? | 22:18 |
| mtreinish | because we still need to remove it from the ignore list | 22:18 |
| *** aimon has joined #openstack-qa | 22:19 | |
| *** jecarey has quit IRC | 22:22 | |
| *** dustins has quit IRC | 22:23 | |
| *** mkollaro has quit IRC | 22:31 | |
| *** rhsu has joined #openstack-qa | 22:32 | |
| *** dkranz has joined #openstack-qa | 22:33 | |
| *** weshay has joined #openstack-qa | 22:34 | |
| *** weshay is now known as weshay_afk | 22:35 | |
| *** jaypipes has quit IRC | 22:36 | |
| *** rbak has quit IRC | 23:04 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add REVIEWING.rst https://review.openstack.org/99451 | 23:06 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add a test documentation section to the docs https://review.openstack.org/113674 | 23:06 |
| mtreinish | dkranz: ^^^ sorry it took me forever to update that patch | 23:07 |
| *** dims_ has quit IRC | 23:30 | |
| *** dims_ has joined #openstack-qa | 23:30 | |
| *** dimtruck is now known as zz_dimtruck | 23:30 | |
| *** dims_ has quit IRC | 23:34 | |
| *** cpallares has quit IRC | 23:40 | |
| *** rfolco has quit IRC | 23:41 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Fix load balancer scenario if no tenant network https://review.openstack.org/113707 | 23:51 |
| *** flwang_ has joined #openstack-qa | 23:51 | |
| *** dims_ has joined #openstack-qa | 23:53 | |
| *** andreaf has joined #openstack-qa | 23:53 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Fix load balancer scenario if no tenant network https://review.openstack.org/113707 | 23:54 |
| *** rfolco has joined #openstack-qa | 23:54 | |
| *** flwang_ has quit IRC | 23:56 | |
| *** andreaf_ has quit IRC | 23:56 | |
| *** andreaf has quit IRC | 23:56 | |
| *** andreaf has joined #openstack-qa | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!