| *** yamamoto has joined #openstack-neutron | 00:01 | |
| *** shwetaap has quit IRC | 00:02 | |
| *** markvoelker has quit IRC | 00:03 | |
| *** Marga_ has joined #openstack-neutron | 00:03 | |
| *** Marga_ has quit IRC | 00:05 | |
| *** Marga_ has joined #openstack-neutron | 00:05 | |
| *** yamamoto has quit IRC | 00:05 | |
| *** Marga_ has quit IRC | 00:10 | |
| *** banix has joined #openstack-neutron | 00:15 | |
| *** yamamoto has joined #openstack-neutron | 00:22 | |
| *** emagana has joined #openstack-neutron | 00:23 | |
| *** dboik has joined #openstack-neutron | 00:25 | |
| *** jckasper has quit IRC | 00:25 | |
| *** shwetaap has joined #openstack-neutron | 00:26 | |
| *** dboik_ has joined #openstack-neutron | 00:26 | |
| *** emagana has quit IRC | 00:29 | |
| *** emagana has joined #openstack-neutron | 00:29 | |
| *** ijw has joined #openstack-neutron | 00:29 | |
| *** dboik has quit IRC | 00:30 | |
| *** sridhar_ram has joined #openstack-neutron | 00:34 | |
| *** emagana has quit IRC | 00:35 | |
| *** ijw has quit IRC | 00:35 | |
| *** shaohe_feng has joined #openstack-neutron | 00:45 | |
| *** nati_uen_ has quit IRC | 00:45 | |
| *** nati_ueno has joined #openstack-neutron | 00:46 | |
| *** yamamoto has quit IRC | 00:50 | |
| *** tiswanso has joined #openstack-neutron | 00:56 | |
| *** achanda has joined #openstack-neutron | 00:59 | |
| *** tiswanso_ has joined #openstack-neutron | 01:00 | |
| *** markvoelker has joined #openstack-neutron | 01:00 | |
| *** xianghui has quit IRC | 01:02 | |
| *** tiswanso has quit IRC | 01:02 | |
| *** baoli has joined #openstack-neutron | 01:03 | |
| *** xianghui has joined #openstack-neutron | 01:04 | |
| *** markvoelker has quit IRC | 01:04 | |
| *** Marga_ has joined #openstack-neutron | 01:06 | |
| *** Marga_ has quit IRC | 01:11 | |
| *** apuimedo has quit IRC | 01:11 | |
| *** baoli has quit IRC | 01:14 | |
| *** baoli has joined #openstack-neutron | 01:14 | |
| *** diegows has quit IRC | 01:15 | |
| *** emagana has joined #openstack-neutron | 01:18 | |
| *** YorikSar has quit IRC | 01:21 | |
| *** emagana has quit IRC | 01:23 | |
| *** oda-g has joined #openstack-neutron | 01:23 | |
| *** shaohe_feng has quit IRC | 01:24 | |
| *** baoli has quit IRC | 01:27 | |
| *** ijw has joined #openstack-neutron | 01:29 | |
| *** achanda has quit IRC | 01:33 | |
| *** ijw has quit IRC | 01:35 | |
| *** achanda has joined #openstack-neutron | 01:38 | |
| *** yamamoto_ has joined #openstack-neutron | 01:39 | |
| *** erkules_ has joined #openstack-neutron | 01:42 | |
| *** erkules has quit IRC | 01:44 | |
| *** dimsum__ has quit IRC | 01:46 | |
| *** nati_ueno has quit IRC | 01:51 | |
| *** nati_ueno has joined #openstack-neutron | 01:54 | |
| *** julim_ has quit IRC | 01:55 | |
| *** jbrendel has quit IRC | 01:57 | |
| *** markvoelker has joined #openstack-neutron | 02:00 | |
| *** archers has joined #openstack-neutron | 02:01 | |
| *** baoli has joined #openstack-neutron | 02:04 | |
| *** markvoelker has quit IRC | 02:05 | |
| *** Marga_ has joined #openstack-neutron | 02:06 | |
| *** archers has quit IRC | 02:08 | |
| *** baoli has quit IRC | 02:09 | |
| *** Marga_ has quit IRC | 02:11 | |
| *** emagana has joined #openstack-neutron | 02:12 | |
| *** tomoe_ has joined #openstack-neutron | 02:13 | |
| *** amotoki has joined #openstack-neutron | 02:13 | |
| *** emagana has quit IRC | 02:17 | |
| *** ijw has joined #openstack-neutron | 02:29 | |
| *** Marga_ has joined #openstack-neutron | 02:33 | |
| *** Marga_ has quit IRC | 02:34 | |
| *** Marga_ has joined #openstack-neutron | 02:35 | |
| *** ijw has quit IRC | 02:35 | |
| *** baoli has joined #openstack-neutron | 02:37 | |
| *** kiran has joined #openstack-neutron | 02:40 | |
| *** baoli has quit IRC | 02:42 | |
| *** dimsum__ has joined #openstack-neutron | 02:46 | |
| *** dimsum__ has quit IRC | 02:52 | |
| *** banix has quit IRC | 02:53 | |
| *** banix has joined #openstack-neutron | 02:53 | |
| *** emagana has joined #openstack-neutron | 02:54 | |
| *** kiran has quit IRC | 02:56 | |
| *** zhhuabj has joined #openstack-neutron | 02:56 | |
| *** emagana has quit IRC | 02:59 | |
| *** chlong has joined #openstack-neutron | 03:00 | |
| *** markvoelker has joined #openstack-neutron | 03:01 | |
| *** markvoelker has quit IRC | 03:06 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron-specs: OVS agent: Use python binding instead of ovs-ofctl command https://review.openstack.org/168767 | 03:08 |
|---|---|---|
| *** shwetaap has quit IRC | 03:09 | |
| *** shwetaap has joined #openstack-neutron | 03:19 | |
| *** banix has quit IRC | 03:19 | |
| *** leenheer has joined #openstack-neutron | 03:20 | |
| *** tiswanso_ has quit IRC | 03:20 | |
| *** tomoe_ has quit IRC | 03:21 | |
| *** vhosakot has joined #openstack-neutron | 03:23 | |
| *** emagana has joined #openstack-neutron | 03:23 | |
| *** vhosakot has left #openstack-neutron | 03:24 | |
| *** pritesh has joined #openstack-neutron | 03:24 | |
| *** pritesh has quit IRC | 03:26 | |
| *** pritesh has joined #openstack-neutron | 03:27 | |
| *** coolsvap|afk is now known as coolsvap | 03:29 | |
| *** ijw has joined #openstack-neutron | 03:29 | |
| *** emagana has quit IRC | 03:32 | |
| *** tomoe_ has joined #openstack-neutron | 03:32 | |
| *** changbl has joined #openstack-neutron | 03:32 | |
| *** shaohe_feng has joined #openstack-neutron | 03:32 | |
| *** achanda has quit IRC | 03:33 | |
| *** ijw has quit IRC | 03:35 | |
| *** chandankumar has joined #openstack-neutron | 03:40 | |
| *** archers has joined #openstack-neutron | 03:45 | |
| *** archers has quit IRC | 03:47 | |
| *** markvoelker has joined #openstack-neutron | 04:02 | |
| *** markvoelker has quit IRC | 04:07 | |
| *** ajmiller has quit IRC | 04:08 | |
| *** pritesh has quit IRC | 04:08 | |
| *** nati_ueno has quit IRC | 04:09 | |
| *** tflynn has joined #openstack-neutron | 04:10 | |
| *** shaohe_feng has quit IRC | 04:14 | |
| *** achanda has joined #openstack-neutron | 04:24 | |
| *** leenheer has quit IRC | 04:28 | |
| *** ijw has joined #openstack-neutron | 04:29 | |
| *** chandankumar has quit IRC | 04:31 | |
| *** shaohe_feng has joined #openstack-neutron | 04:33 | |
| *** watanabe_isao has joined #openstack-neutron | 04:34 | |
| *** ijw has quit IRC | 04:35 | |
| *** anish_ has joined #openstack-neutron | 04:37 | |
| *** changbl has quit IRC | 04:40 | |
| *** imcsk8 has quit IRC | 04:41 | |
| *** imcsk8 has joined #openstack-neutron | 04:41 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: DBDeadlock caused by blocked call in dvr_deletens_if_no_port https://review.openstack.org/170690 | 04:43 |
| *** Marga_ has quit IRC | 04:46 | |
| *** chandankumar has joined #openstack-neutron | 04:52 | |
| *** nati_ueno has joined #openstack-neutron | 04:55 | |
| *** nati_ueno has quit IRC | 04:56 | |
| *** nati_ueno has joined #openstack-neutron | 04:56 | |
| *** nati_ueno has quit IRC | 04:57 | |
| *** itzikb has joined #openstack-neutron | 04:59 | |
| openstackgerrit | watanabe isao proposed openstack/neutron: Make sure OVS restarts when Exception occurred https://review.openstack.org/169975 | 05:00 |
| *** daneyon has quit IRC | 05:01 | |
| *** kiran has joined #openstack-neutron | 05:02 | |
| *** daneyon has joined #openstack-neutron | 05:02 | |
| *** markvoelker has joined #openstack-neutron | 05:03 | |
| *** anshul has joined #openstack-neutron | 05:04 | |
| *** changbl has joined #openstack-neutron | 05:05 | |
| *** trinaths has joined #openstack-neutron | 05:05 | |
| *** kiran has quit IRC | 05:07 | |
| *** kiran has joined #openstack-neutron | 05:08 | |
| *** kiran is now known as kiran-r | 05:08 | |
| *** anilvenkata has joined #openstack-neutron | 05:14 | |
| *** Longgeek has joined #openstack-neutron | 05:14 | |
| *** suresh12 has joined #openstack-neutron | 05:15 | |
| *** vthapar has joined #openstack-neutron | 05:15 | |
| *** irenab has quit IRC | 05:25 | |
| *** anshul has quit IRC | 05:26 | |
| *** anshul has joined #openstack-neutron | 05:26 | |
| *** nati_ueno has joined #openstack-neutron | 05:28 | |
| *** ijw has joined #openstack-neutron | 05:29 | |
| *** anshul has quit IRC | 05:32 | |
| *** nati_ueno has quit IRC | 05:33 | |
| *** coolsvap is now known as coolsvap|afk | 05:33 | |
| *** changbl has quit IRC | 05:34 | |
| *** yatin has joined #openstack-neutron | 05:35 | |
| *** ijw_ has joined #openstack-neutron | 05:35 | |
| *** ijw has quit IRC | 05:35 | |
| *** neeti has joined #openstack-neutron | 05:37 | |
| *** tflynn_ has joined #openstack-neutron | 05:38 | |
| *** ijw_ has quit IRC | 05:40 | |
| *** tflynn has quit IRC | 05:41 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Add OVSDB connection as a parameter to the transaction https://review.openstack.org/170771 | 05:41 |
| *** tflynn_ has quit IRC | 05:44 | |
| *** coolsvap has joined #openstack-neutron | 05:47 | |
| *** Longgeek has quit IRC | 05:47 | |
| openstackgerrit | Hirofumi Ichihara proposed openstack/neutron-specs: Add enable_new_agents to agent conf in network node https://review.openstack.org/170774 | 05:49 |
| *** anshul has joined #openstack-neutron | 05:50 | |
| *** sudipto has joined #openstack-neutron | 05:50 | |
| *** seizadi has joined #openstack-neutron | 05:58 | |
| *** irenab has joined #openstack-neutron | 05:59 | |
| *** shaohe_feng has quit IRC | 05:59 | |
| *** anshul has quit IRC | 06:01 | |
| *** emagana has joined #openstack-neutron | 06:06 | |
| *** seizadi has quit IRC | 06:06 | |
| *** anshul has joined #openstack-neutron | 06:06 | |
| *** sudipto has quit IRC | 06:07 | |
| *** yamamoto_ has quit IRC | 06:07 | |
| *** seizadi has joined #openstack-neutron | 06:07 | |
| *** yamamoto_ has joined #openstack-neutron | 06:07 | |
| *** emagana has quit IRC | 06:11 | |
| *** anshul has quit IRC | 06:12 | |
| *** seizadi has quit IRC | 06:13 | |
| *** anshul has joined #openstack-neutron | 06:14 | |
| openstackgerrit | Hirofumi Ichihara proposed openstack/neutron-specs: Add enable_new_agents to agent conf in network node https://review.openstack.org/170774 | 06:17 |
| *** suresh12 has quit IRC | 06:22 | |
| *** apuimedo has joined #openstack-neutron | 06:23 | |
| *** akamyshnikova_ has joined #openstack-neutron | 06:23 | |
| *** Longgeek has joined #openstack-neutron | 06:24 | |
| *** anshul has quit IRC | 06:24 | |
| *** coolsvap has quit IRC | 06:25 | |
| *** ijw has joined #openstack-neutron | 06:29 | |
| *** numan has joined #openstack-neutron | 06:33 | |
| *** shwetaap has quit IRC | 06:34 | |
| *** ijw has quit IRC | 06:35 | |
| *** coolsvap has joined #openstack-neutron | 06:38 | |
| *** anshul has joined #openstack-neutron | 06:48 | |
| *** jamielennox|away is now known as jamielennox | 06:48 | |
| *** anshul has quit IRC | 06:52 | |
| *** atuvenie has joined #openstack-neutron | 06:55 | |
| *** markvoelker has quit IRC | 07:04 | |
| *** anshul has joined #openstack-neutron | 07:05 | |
| *** anshul has quit IRC | 07:12 | |
| *** anshul has joined #openstack-neutron | 07:12 | |
| *** YorikSar has joined #openstack-neutron | 07:13 | |
| *** chandankumar has quit IRC | 07:15 | |
| *** anshul has quit IRC | 07:17 | |
| *** yfried is now known as yfried|afk | 07:26 | |
| *** chandankumar has joined #openstack-neutron | 07:26 | |
| *** SridharG has joined #openstack-neutron | 07:27 | |
| *** jschwarz_ has joined #openstack-neutron | 07:27 | |
| *** jschwarz_ is now known as jschwarz | 07:28 | |
| *** achanda has quit IRC | 07:28 | |
| *** atuvenie has quit IRC | 07:29 | |
| *** ijw has joined #openstack-neutron | 07:29 | |
| *** yfried|afk is now known as yfried | 07:32 | |
| *** nplanel_ has quit IRC | 07:33 | |
| *** tomoe_ has quit IRC | 07:35 | |
| *** anshul has joined #openstack-neutron | 07:38 | |
| *** achanda has joined #openstack-neutron | 07:39 | |
| *** anshul has quit IRC | 07:41 | |
| *** anshul has joined #openstack-neutron | 07:41 | |
| *** yfried is now known as yfried|afk | 07:45 | |
| openstackgerrit | Mike Kolesnik proposed openstack/neutron: Update port bindings for master router https://review.openstack.org/141114 | 07:47 |
| *** yog__ has joined #openstack-neutron | 07:48 | |
| *** Miouge has joined #openstack-neutron | 07:48 | |
| *** anshul has quit IRC | 07:48 | |
| *** yfried|afk is now known as yfried | 07:50 | |
| *** nfedotov has joined #openstack-neutron | 07:50 | |
| *** ronis has joined #openstack-neutron | 07:53 | |
| *** emagana has joined #openstack-neutron | 07:54 | |
| *** nmagnezi has joined #openstack-neutron | 07:55 | |
| *** nati_ueno has joined #openstack-neutron | 07:56 | |
| *** atuvenie has joined #openstack-neutron | 07:58 | |
| *** emagana has quit IRC | 07:59 | |
| *** nati_ueno has quit IRC | 08:01 | |
| *** yeungp has quit IRC | 08:02 | |
| *** markvoelker has joined #openstack-neutron | 08:03 | |
| *** anshul has joined #openstack-neutron | 08:05 | |
| *** markvoelker has quit IRC | 08:08 | |
| *** yfried is now known as yfried|afk | 08:09 | |
| *** anshul has quit IRC | 08:13 | |
| *** ajo has joined #openstack-neutron | 08:17 | |
| *** Murali has joined #openstack-neutron | 08:17 | |
| *** SridharG1 has joined #openstack-neutron | 08:21 | |
| *** pradipta has joined #openstack-neutron | 08:22 | |
| *** pal has joined #openstack-neutron | 08:23 | |
| *** anshul has joined #openstack-neutron | 08:26 | |
| *** achanda has quit IRC | 08:27 | |
| *** chandankumar_ has joined #openstack-neutron | 08:27 | |
| *** psd has joined #openstack-neutron | 08:29 | |
| *** yfried|afk is now known as yfried | 08:30 | |
| *** Miouge has quit IRC | 08:31 | |
| *** chandankumar has quit IRC | 08:31 | |
| psd | I need some help using the openstack neutron python api for making a benchmark for Rally. Is the right place for some doubts? | 08:31 |
| *** ijw has quit IRC | 08:32 | |
| jschwarz | psd, probably. you better ask, else you'd never know :) | 08:32 |
| *** tomoe_ has joined #openstack-neutron | 08:36 | |
| openstackgerrit | Romil Gupta proposed openstack/neutron-fwaas: Updated Protocol named constants https://review.openstack.org/168709 | 08:36 |
| psd | very well! I was trying to add security group scenario for Neutron. I am essentially trying to call the various security group functions. but I am not sure what all is to passed for the body parameter in those functions. It'd be great if someone could explain it to me or point me towards some docs for the same. | 08:37 |
| psd | For eg. create_security_group_rule | 08:37 |
| psd | https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L714 | 08:37 |
| *** watanabe_isao has quit IRC | 08:39 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron: Update firewall to make it test 'fwaas insertion mode' https://review.openstack.org/165859 | 08:40 |
| *** tomoe_ has quit IRC | 08:40 | |
| *** gsagie has joined #openstack-neutron | 08:41 | |
| *** aepifanov has joined #openstack-neutron | 08:42 | |
| *** apuimedo has quit IRC | 08:42 | |
| *** roeyc has joined #openstack-neutron | 08:42 | |
| *** yfried has quit IRC | 08:48 | |
| *** neeti has quit IRC | 08:49 | |
| *** emagana has joined #openstack-neutron | 08:49 | |
| *** emagana has quit IRC | 08:53 | |
| *** hichihara has quit IRC | 08:54 | |
| *** chlong_ has joined #openstack-neutron | 08:54 | |
| *** coolsvap has quit IRC | 08:55 | |
| *** neeti has joined #openstack-neutron | 08:57 | |
| *** coolsvap has joined #openstack-neutron | 08:57 | |
| *** YorikSar has quit IRC | 08:57 | |
| *** chlong has quit IRC | 08:57 | |
| *** tomoe_ has joined #openstack-neutron | 08:57 | |
| *** tomoe_ has quit IRC | 08:58 | |
| *** yatin has quit IRC | 09:00 | |
| *** anshul has quit IRC | 09:01 | |
| *** yatin has joined #openstack-neutron | 09:01 | |
| *** ijw has joined #openstack-neutron | 09:02 | |
| *** markvoelker has joined #openstack-neutron | 09:04 | |
| *** ijw has quit IRC | 09:08 | |
| *** markvoelker has quit IRC | 09:08 | |
| *** anshul has joined #openstack-neutron | 09:13 | |
| *** anshul has quit IRC | 09:14 | |
| *** anshul has joined #openstack-neutron | 09:15 | |
| *** Miouge has joined #openstack-neutron | 09:17 | |
| *** YorikSar has joined #openstack-neutron | 09:21 | |
| *** roeyc has quit IRC | 09:22 | |
| *** pradipta has quit IRC | 09:24 | |
| *** veena has joined #openstack-neutron | 09:26 | |
| *** pradipta has joined #openstack-neutron | 09:27 | |
| *** anshul has quit IRC | 09:28 | |
| *** ijw has joined #openstack-neutron | 09:29 | |
| veena | What are the configurations to be done for neutron to use SRIOV functionality? I have installed devstack stable icehouse version. I see that in icehouse SRIOV for neutron was implemented and nova changes were not complete. | 09:29 |
| trinaths | veena: Use juno for full support | 09:30 |
| *** coolsvap has quit IRC | 09:30 | |
| *** jeroenvdl has joined #openstack-neutron | 09:31 | |
| veena | trinaths: I'm supposed to use Icehouse and bring up SRIOV funtionality in Icehouse itself | 09:31 |
| trinaths | veena: you may need to cherry-pick necessary patches to nova since, as said "I see that in icehouse SRIOV for neutron was implemented and nova changes were not complete." | 09:32 |
| veena | trinaths: Sure, will do that. But I thought of at least testing neutron port-create. Is it not possible to test only neutron withuot nova? | 09:33 |
| *** ijw has quit IRC | 09:35 | |
| trinaths | veena: you follow this link, https://wiki.openstack.org/wiki/SR-IOV-Passthrough-For-Networking | 09:35 |
| veena | trinaths: Thanks for the link. Will follow it | 09:39 |
| *** anshul has joined #openstack-neutron | 09:40 | |
| *** veena has quit IRC | 09:40 | |
| *** veena has joined #openstack-neutron | 09:41 | |
| veena | trinaths: Thanks for the link. Will follow it | 09:41 |
| *** emagana has joined #openstack-neutron | 09:43 | |
| *** Longgeek has quit IRC | 09:43 | |
| *** Longgeek has joined #openstack-neutron | 09:43 | |
| *** coolsvap has joined #openstack-neutron | 09:45 | |
| veena | trinaths: are there any changes need to be done in neutron to get the functionality | 09:45 |
| *** yuanying has quit IRC | 09:46 | |
| *** anshul has quit IRC | 09:47 | |
| *** emagana has quit IRC | 09:47 | |
| *** yamamoto_ has quit IRC | 09:50 | |
| *** chandankumar_ has quit IRC | 09:53 | |
| *** akamyshnikova_ has quit IRC | 09:53 | |
| *** chandankumar has joined #openstack-neutron | 09:53 | |
| *** coolsvap has quit IRC | 09:55 | |
| *** yuanying has joined #openstack-neutron | 09:55 | |
| *** dimsum__ has joined #openstack-neutron | 09:56 | |
| *** rook__ has joined #openstack-neutron | 09:59 | |
| *** anshul has joined #openstack-neutron | 10:00 | |
| vthapar | question about dhcp-agent: does it run one dnsmasq instance per tenant-network? | 10:01 |
| *** moshele has joined #openstack-neutron | 10:01 | |
| enikanorov | vthapar: one dhcp agent runs one dhcp instance for each tenant network that it hosts | 10:04 |
| enikanorov | however network may be hosted by several dhcp agents, so several dnsmasq instances may be serving one tenant network from several controllers | 10:04 |
| *** markvoelker has joined #openstack-neutron | 10:04 | |
| *** anshul has quit IRC | 10:05 | |
| *** coolsvap has joined #openstack-neutron | 10:07 | |
| vthapar | enikanorov: thanks. other question is when dhcp-agent makes an entry to dnsmasq config it restars dnsmasq, right? if yes, I believe only the affected dnsmasq on that particular agent will be restarted, right? | 10:08 |
| *** markvoelker has quit IRC | 10:09 | |
| enikanorov | vthapar: correct. incase of multiple dnsmasqs for single tenant network all instances restarted because the changes are applied to all of them | 10:09 |
| enikanorov | *all instances for that tenant network | 10:10 |
| vthapar | enikanorov: thank you, much appreciated. | 10:10 |
| *** yatin has quit IRC | 10:11 | |
| *** pc_m has joined #openstack-neutron | 10:13 | |
| *** yatin has joined #openstack-neutron | 10:14 | |
| *** rook__ has quit IRC | 10:17 | |
| *** moha_hunt has joined #openstack-neutron | 10:18 | |
| *** anshul has joined #openstack-neutron | 10:19 | |
| *** tomoe_ has joined #openstack-neutron | 10:19 | |
| *** moshele has quit IRC | 10:21 | |
| jeroenvdl | question about subnets: The external subnet I created, the ports have status DOWN. chaning it gives me "Cannot update read-only attribute status" | 10:23 |
| *** moshele has joined #openstack-neutron | 10:23 | |
| *** moshele has quit IRC | 10:24 | |
| jeroenvdl | even my router between my external subnet and internal subnet the port for the externel is DOWN | 10:24 |
| *** yamamoto has joined #openstack-neutron | 10:26 | |
| *** yatin has quit IRC | 10:29 | |
| *** anshul has quit IRC | 10:29 | |
| *** ijw has joined #openstack-neutron | 10:29 | |
| *** trinaths has quit IRC | 10:33 | |
| *** ZZelle_ has joined #openstack-neutron | 10:34 | |
| *** ijw has quit IRC | 10:35 | |
| *** ronis has quit IRC | 10:35 | |
| *** dboik_ has quit IRC | 10:37 | |
| *** chlong_ has quit IRC | 10:41 | |
| *** yatin has joined #openstack-neutron | 10:44 | |
| *** anilvenkata has quit IRC | 10:45 | |
| *** nati_ueno has joined #openstack-neutron | 10:47 | |
| *** anshul has joined #openstack-neutron | 10:47 | |
| *** psd has quit IRC | 10:49 | |
| *** yuanying has quit IRC | 10:49 | |
| *** yatin has quit IRC | 10:53 | |
| *** yatin has joined #openstack-neutron | 10:56 | |
| *** anshul has quit IRC | 10:57 | |
| *** chlong_ has joined #openstack-neutron | 10:58 | |
| *** luqas has joined #openstack-neutron | 10:59 | |
| *** anshul has joined #openstack-neutron | 10:59 | |
| *** yatin has quit IRC | 11:03 | |
| *** luqas has quit IRC | 11:03 | |
| *** yamamoto has quit IRC | 11:04 | |
| *** chlong_ has quit IRC | 11:05 | |
| *** markvoelker has joined #openstack-neutron | 11:05 | |
| *** tomoe_ has quit IRC | 11:06 | |
| *** markvoelker has quit IRC | 11:10 | |
| *** atuvenie has quit IRC | 11:11 | |
| *** anshul has quit IRC | 11:13 | |
| *** yamamoto has joined #openstack-neutron | 11:14 | |
| *** diegows has joined #openstack-neutron | 11:17 | |
| *** Longgeek has quit IRC | 11:20 | |
| *** chlong has joined #openstack-neutron | 11:24 | |
| *** anshul has joined #openstack-neutron | 11:26 | |
| *** veena has quit IRC | 11:28 | |
| *** ijw has joined #openstack-neutron | 11:29 | |
| *** veena has joined #openstack-neutron | 11:29 | |
| *** pradipta has quit IRC | 11:31 | |
| *** emagana has joined #openstack-neutron | 11:31 | |
| *** Miouge has quit IRC | 11:32 | |
| *** nmagnezi has quit IRC | 11:32 | |
| *** yatin has joined #openstack-neutron | 11:33 | |
| *** anshul has quit IRC | 11:35 | |
| *** ijw has quit IRC | 11:35 | |
| *** emagana has quit IRC | 11:35 | |
| *** pal has quit IRC | 11:36 | |
| *** nmagnezi has joined #openstack-neutron | 11:36 | |
| *** dave-mccowan has joined #openstack-neutron | 11:38 | |
| *** Longgeek has joined #openstack-neutron | 11:39 | |
| -openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 10:56 and 11:37 utc) will need to be rechecked or have new approval votes set | 11:39 | |
| *** anshul has joined #openstack-neutron | 11:48 | |
| *** jamielennox is now known as jamielennox|away | 11:49 | |
| *** shaohe_feng has joined #openstack-neutron | 11:49 | |
| *** diegows has quit IRC | 11:53 | |
| *** ronis has joined #openstack-neutron | 11:53 | |
| *** anilvenkata has joined #openstack-neutron | 11:56 | |
| *** xuhanp has joined #openstack-neutron | 11:58 | |
| *** xuhanp_ has joined #openstack-neutron | 11:58 | |
| *** raildo has joined #openstack-neutron | 12:03 | |
| jschwarz | ZZelle, ping | 12:03 |
| ZZelle_ | jschwarz, pong | 12:04 |
| jschwarz | ZZelle_, re: https://review.openstack.org/#/c/123000/41/neutron/tests/fullstack/fullstack_fixtures.py | 12:04 |
| jschwarz | I didn't understand your second comment there :) | 12:05 |
| *** baoli has joined #openstack-neutron | 12:05 | |
| *** gsagie_ has joined #openstack-neutron | 12:05 | |
| *** markvoelker has joined #openstack-neutron | 12:06 | |
| ZZelle_ | jschwarz, you define _get_br_ex_name and get_br_int/phy_name methods | 12:06 |
| *** anshul has quit IRC | 12:06 | |
| ZZelle_ | jschwarz, gor coherency, they should all be private or publuc | 12:06 |
| jschwarz | ZZelle_, so either all of them with '_' or without? | 12:07 |
| jschwarz | gottcha | 12:07 |
| ZZelle_ | s/publuc/public/ | 12:07 |
| ZZelle_ | jschwarz, yes | 12:07 |
| jschwarz | ZZelle, done :) | 12:08 |
| *** atuvenie has joined #openstack-neutron | 12:09 | |
| *** rkukura has joined #openstack-neutron | 12:09 | |
| *** markvoelker has quit IRC | 12:10 | |
| *** psd has joined #openstack-neutron | 12:10 | |
| *** chandankumar has quit IRC | 12:11 | |
| *** busterswt has joined #openstack-neutron | 12:13 | |
| ajo | ping sc68cal :) | 12:15 |
| ajo | I sent a reply on the QoS topic, | 12:15 |
| ajo | you got me writing an email to the list to organize a weekly meeting, I planned to lead it as you told me it's not your primary focus anymore, but I wanted to confirm if you're ok with it :) | 12:16 |
| sc68cal | ajo: pong | 12:16 |
| ajo | hi :) | 12:16 |
| sc68cal | ajo: I'd be happy to co-organize it | 12:17 |
| ajo | I'm otherwise happy to let you lead, or we can co-lead the thing :D | 12:17 |
| ajo | ahh, very cool sc68cal , that should make it esier :D | 12:17 |
| sc68cal | success has many fathers/mothers, so let's do it together :) | 12:17 |
| ajo | easier, sorry | 12:17 |
| ajo | sc68cal++ | 12:17 |
| ajo | :) | 12:17 |
| *** yamamoto has quit IRC | 12:17 | |
| ajo | Ok, I'll write the email about the meeting on behalf of both, let me know if you know of more interested people so we can put them on the initial email CC | 12:18 |
| *** anshul has joined #openstack-neutron | 12:18 | |
| ajo | (for now my list is on the last email CC) | 12:18 |
| ajo | sc68cal ^ | 12:18 |
| sc68cal | ajo: sure. I think we should include irena b | 12:20 |
| sc68cal | let me fetch the e-mail addy | 12:20 |
| sc68cal | ajo: https://review.openstack.org/#/c/139859/ | 12:20 |
| ajo | sc68cal, right, I think it's in the CC if I didn't do it wrong | 12:20 |
| sc68cal | ajo: ok | 12:20 |
| sc68cal | irenab@mellanox.com | 12:20 |
| ajo | that one is old :) | 12:20 |
| sc68cal | :) | 12:20 |
| ajo | irenab.dev##NOSPAM##@gmail.com (nospam thing for the logs...) | 12:21 |
| irenab | hi guys | 12:21 |
| ajo | irenab, is that email right irenab ? :D | 12:21 |
| ajo | I'm repeating myself :D | 12:21 |
| irenab | correct | 12:21 |
| *** anish_ has quit IRC | 12:22 | |
| ajo | ok, I'm going to get ahold of some lunch, and then I'll finish the email writing :) | 12:22 |
| ajo | irenab, check my last email, and let me no if you remember about more people with QoS interest to loop in into the meeting thing | 12:22 |
| irenab | great, bon appetite | 12:22 |
| *** bradjones_ has joined #openstack-neutron | 12:22 | |
| sc68cal | ajo: thanks - enjoy lunch | 12:22 |
| irenab | ajo: when did you sent it? | 12:22 |
| ajo | irenab: 14:14, "Regarding Neutron's QoS Capability" | 12:23 |
| *** ajo is now known as ajo|lunch | 12:23 | |
| *** bradjones has quit IRC | 12:24 | |
| *** bradjones_ is now known as bradjones | 12:24 | |
| *** bradjones has joined #openstack-neutron | 12:24 | |
| irenab | ajo: will check and let you know | 12:24 |
| *** markvoelker has joined #openstack-neutron | 12:24 | |
| *** coolsvap has quit IRC | 12:25 | |
| jschwarz | ZZelle_, also let me know what you think about my last comment here: https://review.openstack.org/#/c/123000/41/neutron/tests/fullstack/base.py | 12:25 |
| *** emagana has joined #openstack-neutron | 12:25 | |
| *** dimsum__ has quit IRC | 12:25 | |
| *** psd has quit IRC | 12:26 | |
| *** shaohe_feng has quit IRC | 12:27 | |
| *** chandankumar has joined #openstack-neutron | 12:28 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: get_rand_name should return the same length string https://review.openstack.org/170179 | 12:29 |
| *** ijw has joined #openstack-neutron | 12:29 | |
| *** emagana has quit IRC | 12:30 | |
| *** rotbeard has joined #openstack-neutron | 12:34 | |
| *** ijw has quit IRC | 12:35 | |
| *** pal has joined #openstack-neutron | 12:36 | |
| *** flaviof_ has joined #openstack-neutron | 12:43 | |
| *** banix has joined #openstack-neutron | 12:45 | |
| *** jckasper has joined #openstack-neutron | 12:49 | |
| *** veena has quit IRC | 12:49 | |
| *** annegentle has joined #openstack-neutron | 12:50 | |
| *** dimsum__ has joined #openstack-neutron | 12:51 | |
| *** shaohe_feng has joined #openstack-neutron | 12:51 | |
| *** annegentle has quit IRC | 12:52 | |
| *** annegentle has joined #openstack-neutron | 12:52 | |
| *** thomasem has joined #openstack-neutron | 12:54 | |
| *** psd has joined #openstack-neutron | 12:54 | |
| *** numan has quit IRC | 12:54 | |
| *** claudiub has joined #openstack-neutron | 12:57 | |
| *** dboik has joined #openstack-neutron | 12:57 | |
| *** yatin has quit IRC | 12:58 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack test https://review.openstack.org/123000 | 12:58 |
| *** amotoki has quit IRC | 12:58 | |
| *** wverdugo5001 has left #openstack-neutron | 12:59 | |
| *** kbringard has joined #openstack-neutron | 12:59 | |
| openstackgerrit | Evgeny Bagdasaryan proposed openstack/neutron: Test attaching router to IPv6 subnets https://review.openstack.org/169817 | 13:00 |
| *** dboik_ has joined #openstack-neutron | 13:00 | |
| jschwarz | anilvenkata, ping | 13:02 |
| *** dboik has quit IRC | 13:02 | |
| *** akamyshnikova_ has joined #openstack-neutron | 13:03 | |
| *** thomasem has quit IRC | 13:03 | |
| *** dimsum__ is now known as dims | 13:03 | |
| *** busterswt has quit IRC | 13:03 | |
| *** thomasem has joined #openstack-neutron | 13:03 | |
| *** britthou_ has joined #openstack-neutron | 13:06 | |
| *** tomoe_ has joined #openstack-neutron | 13:07 | |
| *** julim has joined #openstack-neutron | 13:09 | |
| *** britthouser has quit IRC | 13:09 | |
| *** irenab has quit IRC | 13:09 | |
| *** julim has quit IRC | 13:11 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack test https://review.openstack.org/123000 | 13:11 |
| *** rdo has quit IRC | 13:13 | |
| *** rotbeard has quit IRC | 13:13 | |
| *** rkukura has quit IRC | 13:14 | |
| *** dboik_ has quit IRC | 13:14 | |
| *** rdo has joined #openstack-neutron | 13:15 | |
| *** cference has joined #openstack-neutron | 13:17 | |
| *** tiswanso has joined #openstack-neutron | 13:18 | |
| mestery | ajo|lunch: When you return, can you eyeball this ipset review: https://review.openstack.org/#/c/170328/ | 13:18 |
| *** ajo|lunch is now known as ajo | 13:18 | |
| ajo | mestery, sure ;) | 13:18 |
| mestery | ajo: Cool, thanks! | 13:18 |
| mestery | ajo: haleyb was looking to you to get some eyes on it before we merge it, it fixes a gate issue found last thursday | 13:18 |
| ajo | thanks haleyb , hanzhang ^ | 13:19 |
| *** neeti has quit IRC | 13:19 | |
| ajo | hanzhang can you look at it too? | 13:19 |
| haleyb | med_: thanks, i was just getting around to that ping | 13:19 |
| mestery | haleyb: I've got your back ;) | 13:20 |
| *** emagana has joined #openstack-neutron | 13:20 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Add OVSDB connection as a parameter to the transaction https://review.openstack.org/170771 | 13:20 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: allow OVSDB connection schema to be configurable https://review.openstack.org/170842 | 13:20 |
| haleyb | ajo: i'm curious waht you think of the --resize 50 i also added, just in case it's not necessary | 13:21 |
| ajo | haleyb, ack, I'm reading the patch, I will also check the ipset documentation | 13:21 |
| haleyb | the man page didn't mention it locally, but i think it was Han (?) that asked about it | 13:22 |
| *** annegentle has quit IRC | 13:22 | |
| ajo | ah good hanzhang was already looking at it, | 13:23 |
| ajo | haleyb, so you had to look into the source code? :) | 13:23 |
| *** emagana has quit IRC | 13:24 | |
| haleyb | ajo: i did look at the kernel :) but the online man page had the newer arguments | 13:24 |
| *** ferest has joined #openstack-neutron | 13:25 | |
| *** zhenq has joined #openstack-neutron | 13:25 | |
| *** Murali has quit IRC | 13:27 | |
| *** ferest has quit IRC | 13:27 | |
| *** ijw has joined #openstack-neutron | 13:29 | |
| *** rpothier has joined #openstack-neutron | 13:29 | |
| *** johnbelamaric has joined #openstack-neutron | 13:30 | |
| jschwarz | So I'm getting rabbitmq timeouts on the functional tests upstream (specifically for a patch I'm running, not affecting everyone) and I'm trying to figure out what it is. | 13:31 |
| jschwarz | http://logs.openstack.org/00/123000/44/check/check-neutron-dsvm-functional/bebd01e/logs/neutron-l3-agent--2015-04-06--115101.txt.gz | 13:31 |
| jschwarz | Specifically at 11:52:02.756 (exactly a minute after a rabbitmq connection was made) it times out... | 13:32 |
| jschwarz | any ideas? | 13:32 |
| *** dboik has joined #openstack-neutron | 13:33 | |
| *** Longgeek has quit IRC | 13:33 | |
| *** dboik has quit IRC | 13:33 | |
| *** dboik has joined #openstack-neutron | 13:34 | |
| *** ijw has quit IRC | 13:35 | |
| *** rdo has quit IRC | 13:35 | |
| *** rdo has joined #openstack-neutron | 13:37 | |
| *** britthou_ has quit IRC | 13:37 | |
| openstackgerrit | Merged openstack/neutron: Prepare for unit test reorg https://review.openstack.org/170329 | 13:37 |
| *** psd has quit IRC | 13:40 | |
| HenryG | jschwarz: timeout? What are these errors? http://logs.openstack.org/00/123000/44/check/check-neutron-dsvm-functional/bebd01e/logs/neutron-l3-agent--2015-04-06--115101.txt.gz#_2015-04-06_11_52_02_866 | 13:41 |
| *** banix has quit IRC | 13:42 | |
| openstackgerrit | Evgeny Bagdasaryan proposed openstack/neutron: Test attaching router to IPv6 subnets https://review.openstack.org/169817 | 13:43 |
| *** chandankumar has quit IRC | 13:46 | |
| *** annegentle has joined #openstack-neutron | 13:52 | |
| *** bradjones has quit IRC | 13:53 | |
| *** anshul has quit IRC | 13:53 | |
| *** raildo has quit IRC | 13:53 | |
| *** xuhanp_ has quit IRC | 13:53 | |
| *** tiswanso has quit IRC | 13:53 | |
| *** markvoelker has quit IRC | 13:53 | |
| *** sc68cal has quit IRC | 13:53 | |
| *** x58 has quit IRC | 13:53 | |
| *** sdague has quit IRC | 13:53 | |
| *** rohara has quit IRC | 13:53 | |
| *** jamiec has quit IRC | 13:53 | |
| *** briancline has quit IRC | 13:53 | |
| *** _cerberus_ has quit IRC | 13:53 | |
| *** shaohe_feng has quit IRC | 13:53 | |
| *** itzikb has quit IRC | 13:53 | |
| *** xianghui has quit IRC | 13:53 | |
| *** yamahata has quit IRC | 13:53 | |
| *** david-lyle has quit IRC | 13:53 | |
| *** gilliard has quit IRC | 13:53 | |
| *** pcarver_ has quit IRC | 13:53 | |
| *** isq has quit IRC | 13:53 | |
| *** skoude has quit IRC | 13:53 | |
| *** puck has quit IRC | 13:53 | |
| *** fouxm has quit IRC | 13:53 | |
| *** raginbajin has quit IRC | 13:53 | |
| *** sgordon has quit IRC | 13:53 | |
| *** TrevorV has quit IRC | 13:53 | |
| *** Apsu has quit IRC | 13:53 | |
| *** mkoderer has quit IRC | 13:53 | |
| *** michatotol has quit IRC | 13:53 | |
| *** kbringard has quit IRC | 13:53 | |
| *** yog__ has quit IRC | 13:53 | |
| *** ZZelle has quit IRC | 13:53 | |
| *** krtaylor has quit IRC | 13:53 | |
| *** mitz has quit IRC | 13:53 | |
| *** hanzhang has quit IRC | 13:53 | |
| *** jdaggett has quit IRC | 13:53 | |
| *** mordred has quit IRC | 13:53 | |
| *** rstarr has quit IRC | 13:53 | |
| *** jmeridth has quit IRC | 13:53 | |
| *** adam_g_out has quit IRC | 13:53 | |
| *** comstud has quit IRC | 13:53 | |
| *** lbragstad has quit IRC | 13:53 | |
| *** gus has quit IRC | 13:53 | |
| *** zacksh has quit IRC | 13:53 | |
| *** ebagdasa has quit IRC | 13:53 | |
| *** ekarlso has quit IRC | 13:53 | |
| *** mikal has quit IRC | 13:53 | |
| *** coolsvap|afk has quit IRC | 13:53 | |
| *** Trozz has quit IRC | 13:53 | |
| *** mihero has quit IRC | 13:53 | |
| *** jroll has quit IRC | 13:53 | |
| *** mattt has quit IRC | 13:53 | |
| *** aignatov has quit IRC | 13:53 | |
| *** johnbelamaric has quit IRC | 13:53 | |
| *** rpothier has quit IRC | 13:53 | |
| *** dims has quit IRC | 13:53 | |
| *** chem has quit IRC | 13:53 | |
| *** lascii has quit IRC | 13:53 | |
| *** ianw has quit IRC | 13:53 | |
| *** strictlyb has quit IRC | 13:53 | |
| *** stefanb has quit IRC | 13:53 | |
| *** SpamapS has quit IRC | 13:53 | |
| *** inara` has quit IRC | 13:53 | |
| *** nplanel has quit IRC | 13:53 | |
| *** phschwartz has quit IRC | 13:53 | |
| *** xmltok has quit IRC | 13:53 | |
| *** q3k has quit IRC | 13:53 | |
| *** larsks has quit IRC | 13:53 | |
| *** imcsk8 has quit IRC | 13:53 | |
| *** enikanorov has quit IRC | 13:53 | |
| *** harlowja_away has quit IRC | 13:53 | |
| *** aleksandr_null has quit IRC | 13:53 | |
| *** kickinz1|afk has quit IRC | 13:53 | |
| *** skraynev has quit IRC | 13:53 | |
| *** pck has quit IRC | 13:53 | |
| *** hitalia has quit IRC | 13:53 | |
| *** mikehn has quit IRC | 13:53 | |
| *** frickler has quit IRC | 13:53 | |
| *** ondergetekende has quit IRC | 13:53 | |
| *** mdavidson has quit IRC | 13:53 | |
| *** nati_ueno has quit IRC | 13:53 | |
| *** raildo has joined #openstack-neutron | 13:54 | |
| *** britthouser has joined #openstack-neutron | 13:55 | |
| *** kbringard has joined #openstack-neutron | 13:55 | |
| *** yog__ has joined #openstack-neutron | 13:55 | |
| *** ZZelle has joined #openstack-neutron | 13:55 | |
| *** krtaylor has joined #openstack-neutron | 13:55 | |
| *** mitz has joined #openstack-neutron | 13:55 | |
| *** hanzhang has joined #openstack-neutron | 13:55 | |
| *** jdaggett has joined #openstack-neutron | 13:55 | |
| *** mordred has joined #openstack-neutron | 13:55 | |
| *** rstarr has joined #openstack-neutron | 13:55 | |
| *** jmeridth has joined #openstack-neutron | 13:55 | |
| *** adam_g_out has joined #openstack-neutron | 13:55 | |
| *** lbragstad has joined #openstack-neutron | 13:55 | |
| *** comstud has joined #openstack-neutron | 13:55 | |
| *** gus has joined #openstack-neutron | 13:55 | |
| *** zacksh has joined #openstack-neutron | 13:55 | |
| *** mattt has joined #openstack-neutron | 13:55 | |
| *** ebagdasa has joined #openstack-neutron | 13:55 | |
| *** ekarlso has joined #openstack-neutron | 13:55 | |
| *** mikal has joined #openstack-neutron | 13:55 | |
| *** coolsvap|afk has joined #openstack-neutron | 13:55 | |
| *** Trozz has joined #openstack-neutron | 13:55 | |
| *** mihero has joined #openstack-neutron | 13:55 | |
| *** jroll has joined #openstack-neutron | 13:55 | |
| *** aignatov has joined #openstack-neutron | 13:55 | |
| *** suresh12 has joined #openstack-neutron | 13:55 | |
| *** johnbelamaric has joined #openstack-neutron | 13:56 | |
| *** rpothier has joined #openstack-neutron | 13:56 | |
| *** dims has joined #openstack-neutron | 13:56 | |
| *** chem has joined #openstack-neutron | 13:56 | |
| *** lascii has joined #openstack-neutron | 13:56 | |
| *** ianw has joined #openstack-neutron | 13:56 | |
| *** strictlyb has joined #openstack-neutron | 13:56 | |
| *** stefanb has joined #openstack-neutron | 13:56 | |
| *** SpamapS has joined #openstack-neutron | 13:56 | |
| *** inara` has joined #openstack-neutron | 13:56 | |
| *** nplanel has joined #openstack-neutron | 13:56 | |
| *** phschwartz has joined #openstack-neutron | 13:56 | |
| *** xmltok has joined #openstack-neutron | 13:56 | |
| *** q3k has joined #openstack-neutron | 13:56 | |
| *** larsks has joined #openstack-neutron | 13:56 | |
| ajo | haleyb, mestery, I had some pedantic nits, and I need a few more minutes to look at the resize | 13:57 |
| *** tiswanso has joined #openstack-neutron | 13:58 | |
| *** markvoelker has joined #openstack-neutron | 13:58 | |
| *** sc68cal has joined #openstack-neutron | 13:58 | |
| *** x58 has joined #openstack-neutron | 13:58 | |
| *** sdague has joined #openstack-neutron | 13:58 | |
| *** rohara has joined #openstack-neutron | 13:58 | |
| *** jamiec has joined #openstack-neutron | 13:58 | |
| *** briancline has joined #openstack-neutron | 13:58 | |
| *** _cerberus_ has joined #openstack-neutron | 13:58 | |
| *** itzikb has joined #openstack-neutron | 13:58 | |
| *** xianghui has joined #openstack-neutron | 13:58 | |
| *** yamahata has joined #openstack-neutron | 13:58 | |
| *** david-lyle has joined #openstack-neutron | 13:58 | |
| *** gilliard has joined #openstack-neutron | 13:58 | |
| *** pcarver_ has joined #openstack-neutron | 13:58 | |
| *** isq has joined #openstack-neutron | 13:58 | |
| *** skoude has joined #openstack-neutron | 13:58 | |
| *** puck has joined #openstack-neutron | 13:58 | |
| *** fouxm has joined #openstack-neutron | 13:58 | |
| *** raginbajin has joined #openstack-neutron | 13:58 | |
| *** sgordon has joined #openstack-neutron | 13:58 | |
| *** TrevorV has joined #openstack-neutron | 13:58 | |
| *** Apsu has joined #openstack-neutron | 13:58 | |
| *** mkoderer has joined #openstack-neutron | 13:58 | |
| *** michatotol has joined #openstack-neutron | 13:58 | |
| *** bradjones has joined #openstack-neutron | 13:58 | |
| *** xuhanp_ has joined #openstack-neutron | 13:58 | |
| *** imcsk8 has joined #openstack-neutron | 13:58 | |
| *** enikanorov has joined #openstack-neutron | 13:58 | |
| *** harlowja_away has joined #openstack-neutron | 13:58 | |
| *** aleksandr_null has joined #openstack-neutron | 13:58 | |
| *** kickinz1|afk has joined #openstack-neutron | 13:58 | |
| *** skraynev has joined #openstack-neutron | 13:58 | |
| *** pck has joined #openstack-neutron | 13:58 | |
| *** hitalia has joined #openstack-neutron | 13:58 | |
| *** mikehn has joined #openstack-neutron | 13:58 | |
| *** frickler has joined #openstack-neutron | 13:58 | |
| *** ondergetekende has joined #openstack-neutron | 13:58 | |
| *** mdavidson has joined #openstack-neutron | 13:58 | |
| *** shaohe_feng has joined #openstack-neutron | 13:58 | |
| mestery | ajo: Ack! Overall, it looks pretty nice and it was super awesome of haleyb to jump on that one last week. Thanks to both of you! | 13:58 |
| ajo | haleyb, thank you for working on the fix | 13:58 |
| ajo | yes, mestery , that's exactly what I was thinking | 13:59 |
| ajo | a fast & good job :) | 13:59 |
| mestery | ++ | 13:59 |
| *** shaohe_feng has quit IRC | 13:59 | |
| *** shaohe_feng has joined #openstack-neutron | 13:59 | |
| *** busterswt has joined #openstack-neutron | 13:59 | |
| haleyb | ajo: i'll take a look at your comments and get a new patch out, thanks for reviewing so fast! | 13:59 |
| ajo | I need to confirm the --resize part, but it looks like haleyb did the homework :) | 14:00 |
| *** lascii is now known as alaski | 14:00 | |
| *** psd_ has joined #openstack-neutron | 14:00 | |
| *** tiswanso has quit IRC | 14:00 | |
| *** markvoelker has quit IRC | 14:00 | |
| *** sc68cal has quit IRC | 14:00 | |
| *** x58 has quit IRC | 14:00 | |
| *** sdague has quit IRC | 14:00 | |
| *** rohara has quit IRC | 14:00 | |
| *** jamiec has quit IRC | 14:00 | |
| *** briancline has quit IRC | 14:00 | |
| *** _cerberus_ has quit IRC | 14:00 | |
| *** tomoe_ has quit IRC | 14:00 | |
| *** shwetaap has joined #openstack-neutron | 14:00 | |
| *** tiswanso has joined #openstack-neutron | 14:01 | |
| *** markvoelker has joined #openstack-neutron | 14:01 | |
| *** sc68cal has joined #openstack-neutron | 14:01 | |
| *** x58 has joined #openstack-neutron | 14:01 | |
| *** sdague has joined #openstack-neutron | 14:01 | |
| *** rohara has joined #openstack-neutron | 14:01 | |
| *** jamiec has joined #openstack-neutron | 14:01 | |
| *** briancline has joined #openstack-neutron | 14:01 | |
| *** _cerberus_ has joined #openstack-neutron | 14:01 | |
| *** luqas has joined #openstack-neutron | 14:01 | |
| *** anilvenkata has quit IRC | 14:02 | |
| jschwarz | HenryG, those are intentional - I'm killing (kill -9) the process and removing the DB from under the process feet :) | 14:02 |
| *** Marga_ has joined #openstack-neutron | 14:02 | |
| *** anshul has joined #openstack-neutron | 14:03 | |
| *** annegentle has quit IRC | 14:05 | |
| *** luqas has quit IRC | 14:05 | |
| openstackgerrit | Evgeny Bagdasaryan proposed openstack/neutron: Test attaching router to IPv6 subnets https://review.openstack.org/169817 | 14:06 |
| *** Marga__ has joined #openstack-neutron | 14:06 | |
| *** banix has joined #openstack-neutron | 14:07 | |
| *** ajmiller has joined #openstack-neutron | 14:07 | |
| *** anshul has quit IRC | 14:08 | |
| *** Marga_ has quit IRC | 14:09 | |
| *** itzikb has quit IRC | 14:10 | |
| *** neeti has joined #openstack-neutron | 14:11 | |
| ajo | haleyb, I suspect the resize value is ignored in kernel: http://lxr.free-electrons.com/source/net/netfilter/ipset/ip_set_hash_gen.h#L533 | 14:13 |
| ajo | it sais "double the hash size" regardless of any value | 14:13 |
| ajo | but : | 14:13 |
| ajo | 50% seems a reasonable value if they implement that anytime or if I'm getting it wrong | 14:14 |
| ajo | which could definitely be | 14:14 |
| ajo | it's a more expensive operation, but I guess that will succeed regardless of the admin config for the hashsize, etc... | 14:14 |
| *** emagana has joined #openstack-neutron | 14:18 | |
| *** emagana has quit IRC | 14:18 | |
| *** emagana has joined #openstack-neutron | 14:19 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack test https://review.openstack.org/123000 | 14:19 |
| *** rook has quit IRC | 14:21 | |
| *** rushil has joined #openstack-neutron | 14:21 | |
| *** nmagnezi has quit IRC | 14:23 | |
| *** amotoki has joined #openstack-neutron | 14:23 | |
| *** amotoki has quit IRC | 14:23 | |
| *** gsagie_ has quit IRC | 14:24 | |
| *** ijw has joined #openstack-neutron | 14:29 | |
| *** xuhanp has quit IRC | 14:32 | |
| *** xuhanp_ has quit IRC | 14:32 | |
| *** neeti has quit IRC | 14:32 | |
| *** leenheer has joined #openstack-neutron | 14:32 | |
| *** ijw has quit IRC | 14:35 | |
| *** achanda has joined #openstack-neutron | 14:35 | |
| *** prad has joined #openstack-neutron | 14:36 | |
| *** rook has joined #openstack-neutron | 14:37 | |
| haleyb | ajo: so should i be passing --resize? i didn't understand your comment. The command needs the argument to succeed, but the kernel does seem to just double up-to 4096 hashsize | 14:37 |
| *** pal has quit IRC | 14:37 | |
| ajo | haleyb, may be I would use 100% instead of 50% to match the current kernel behavior? | 14:41 |
| *** johnbelamaric has quit IRC | 14:41 | |
| *** johnbelamaric has joined #openstack-neutron | 14:42 | |
| ajo | haleyb, this way, if they start actually making use of the parameter value in the future, the behavior won't change? | 14:42 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Implement IPAM Driver loader https://review.openstack.org/147479 | 14:42 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM https://review.openstack.org/153236 | 14:42 |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 14:42 |
| *** johnbelamaric has quit IRC | 14:42 | |
| *** kiran-r has quit IRC | 14:42 | |
| *** ivar-lazzaro has joined #openstack-neutron | 14:42 | |
| *** pavel_bondar has joined #openstack-neutron | 14:42 | |
| *** tomoe_ has joined #openstack-neutron | 14:43 | |
| ajo | haleyb, so that's what you actually saw while testing? the kernel will double up the size? | 14:43 |
| haleyb | ajo: or i just don't use it? it was more a pre-emptive change | 14:43 |
| ajo | wow, I'm happy that I understood the thing since the C code was kite abstracted :) | 14:43 |
| *** achanda has quit IRC | 14:43 | |
| haleyb | ajo: no, it was just a suggestion by a reviewer, the failure was always due to maxelem being too small | 14:43 |
| ajo | hmm, haleyb will it resize regardless of you providing the parameter or not? | 14:43 |
| haleyb | the kernel code is terrible | 14:43 |
| ajo | ahhh | 14:43 |
| ajo | haleyb lol X) | 14:43 |
| *** ZZelle has quit IRC | 14:43 | |
| ajo | haleyb, I've seen worse C code :D | 14:44 |
| openstackgerrit | Aman Kumar proposed openstack/neutron: ovs agent resync may miss port remove event https://review.openstack.org/165023 | 14:44 |
| *** ZZelle has joined #openstack-neutron | 14:44 | |
| ajo | haleyb, kernel code seems like "good" C code to me (compared to other references) | 14:44 |
| *** tomoe_ has quit IRC | 14:44 | |
| ajo | if it will double I would skip the parameter then | 14:44 |
| ajo | (if it does double regardless of the parameter) | 14:45 |
| ajo | I guess, in the future then they will either remove the parameter, or make 100% increase size the default. | 14:45 |
| *** shaohe_feng has quit IRC | 14:45 | |
| ajo | too bad they don't resize on maxelem size hit... :/ | 14:45 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack test https://review.openstack.org/123000 | 14:45 |
| haleyb | ajo: you need the arg or you get "ipset v6.20.1: Missing mandatory argument of option `resize'" | 14:46 |
| ajo | haleyb, I mean taking out the whole "resize" parameter | 14:47 |
| ajo | if you checked that it will resize anyway, | 14:47 |
| ajo | and that matches what we saw into the kernel code | 14:47 |
| ajo | then we don't need to add complexity to our code | 14:47 |
| *** prad_ has joined #openstack-neutron | 14:47 | |
| *** prad has quit IRC | 14:48 | |
| ajo | haleyb, do what I say make sense?, | 14:48 |
| haleyb | ajo: i have not checked it resized automatically | 14:48 |
| ajo | haleyb, what limit is hit when the "resize" happens? | 14:48 |
| ajo | ah, ok | 14:49 |
| ajo | then my code interpretation could be wrong | 14:49 |
| openstackgerrit | Terry Wilson proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/167913 | 14:49 |
| ajo | ok, let me re-read the manual and the code, then may be it's safer to leave it in place, I misunderstood you | 14:49 |
| *** annegentle has joined #openstack-neutron | 14:49 | |
| haleyb | i will just drop the argument since the only failure we know happens is the max elements is hit - the hashsize is just typically set to some value relative to that | 14:49 |
| *** wverdugo5001 has joined #openstack-neutron | 14:50 | |
| *** wverdugo5001 has left #openstack-neutron | 14:50 | |
| haleyb | too bad there isn't a min/max elements and it autosizes that (where min is a smaller default) | 14:50 |
| *** prad_ is now known as prad | 14:50 | |
| *** julim has joined #openstack-neutron | 14:52 | |
| *** gsagie has quit IRC | 14:52 | |
| ajo | haleyb, I guess with ipv6 history could be different? more bytes for elements than the hashes?. :) but yet, the parameter is tunable in your patch, so... | 14:53 |
| *** leenheer has quit IRC | 14:53 | |
| haleyb | ajo: well allowing 128K elements with only a 1K hash seems like a really bad idea (~128 per bucket) | 14:55 |
| *** jorgem has joined #openstack-neutron | 14:55 | |
| ajo | yes | 14:55 |
| ajo | and it has a default limit of 8 per bucket | 14:56 |
| ajo | you can be unlucky very fast... | 14:56 |
| *** signed8bit has joined #openstack-neutron | 14:56 | |
| *** jaypipes has joined #openstack-neutron | 14:56 | |
| *** ociuhandu has joined #openstack-neutron | 14:56 | |
| haleyb | ajo: so remove the --resize, for now? | 14:57 |
| ajo | haleyb, I'm looking into the cmdline tool (when I find it) sources | 14:57 |
| ajo | but kernel points to that, yes | 14:57 |
| ajo | I'd like to fully understand what they are doing, and try to guess what the'd do in the future, but... | 14:58 |
| haleyb | IPSET_ATTR_RESIZE is the attribute, and only passed at creation | 14:58 |
| ajo | anyway, | 14:58 |
| ajo | what we said, you can completely tune it by your parameters | 14:58 |
| prometheanfire | mestery: question about working on something that may have been missed in a blueprint for kilo | 14:58 |
| haleyb | except when there is this collision and it fails adding | 14:58 |
| prometheanfire | https://blueprints.launchpad.net/neutron/+spec/mtu-selection-and-advertisement | 14:58 |
| prometheanfire | here's the bug https://bugs.launchpad.net/neutron/+bug/1348788 | 14:59 |
| openstack | Launchpad bug 1348788 in neutron "network_device_mtu is not applied to VMs, only to agents" [Low,Confirmed] | 14:59 |
| haleyb | but we haven't seen the collision | 14:59 |
| *** chlong has quit IRC | 15:00 | |
| *** jobewan has joined #openstack-neutron | 15:01 | |
| ajo | haleyb, true, the IPSET_ATTR_RESIZE is only set, but never checked anywhere in the code | 15:01 |
| *** pal has joined #openstack-neutron | 15:02 | |
| ajo | and the code resizes to double | 15:02 |
| ajo | we can get the same removing the parameter (as the equivalent default is --resize 100) | 15:02 |
| *** johnbelamaric has joined #openstack-neutron | 15:02 | |
| *** johnbelamaric has quit IRC | 15:03 | |
| *** johnbelamaric has joined #openstack-neutron | 15:03 | |
| *** rkukura has joined #openstack-neutron | 15:03 | |
| *** suresh12 has quit IRC | 15:04 | |
| *** packet has joined #openstack-neutron | 15:07 | |
| *** zzzeek has joined #openstack-neutron | 15:07 | |
| *** TrevorV_ has joined #openstack-neutron | 15:08 | |
| jschwarz | anilvenkata, ping | 15:10 |
| jschwarz | prometheanfire, that launchpad bug seems odd since Ian Wells (the reporter of that bug) was the one who drove the mtu-selection-and-advertisement blueprint | 15:11 |
| jschwarz | prometheanfire, so probably it's just not resolved | 15:11 |
| prometheanfire | jschwarz: ya, I think I'll email (if I can find it) | 15:12 |
| *** TrevorV_ has quit IRC | 15:13 | |
| jschwarz | prometheanfire, why email when you can ping ijw? | 15:13 |
| *** moha_hunt has quit IRC | 15:13 | |
| jschwarz | (ijw isn't on now but he's around) | 15:13 |
| prometheanfire | ah | 15:13 |
| prometheanfire | ok | 15:13 |
| prometheanfire | didn't see him on :P | 15:13 |
| *** xgerman has joined #openstack-neutron | 15:14 | |
| *** dannywilson has joined #openstack-neutron | 15:14 | |
| jschwarz | (ijw isn't on now but he's around) | 15:14 |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Re-use context session in ML2 DB get_port_binding_host https://review.openstack.org/170690 | 15:14 |
| *** dsneddon has joined #openstack-neutron | 15:16 | |
| *** dannywilson has quit IRC | 15:16 | |
| *** dannywilson has joined #openstack-neutron | 15:17 | |
| prometheanfire | jschwarz: my question is kinda policy one too, should plugins be included with blueprints like that | 15:19 |
| jschwarz | prometheanfire, afaik it was implemented for OVS (and not LB), but I might be wrong | 15:20 |
| jschwarz | if that's what you asked | 15:20 |
| prometheanfire | more general policy | 15:20 |
| prometheanfire | this just seems like it was missed | 15:20 |
| *** packet has quit IRC | 15:20 | |
| jschwarz | what was missed? | 15:21 |
| prometheanfire | LB | 15:21 |
| *** jorgem has quit IRC | 15:21 | |
| mestery | prometheanfire jschwarz: Someone modified the DHCP agent to pass the MTU if it's set, which shudl get it to the VMs | 15:21 |
| prometheanfire | progress, but not the bug :P | 15:22 |
| *** suresh12 has joined #openstack-neutron | 15:22 | |
| * prometheanfire has a possible patch for LB | 15:23 | |
| sc68cal | it *must* be Easter - linuxbridge has risen from the dead | 15:25 |
| prometheanfire | ? | 15:25 |
| prometheanfire | is it being depricated? | 15:26 |
| sc68cal | no, it was just neglected | 15:26 |
| prometheanfire | ya, ovs was the hotness for a while | 15:26 |
| * prometheanfire likes vxlan/linuxbridge | 15:26 | |
| *** amuller has joined #openstack-neutron | 15:26 | |
| prometheanfire | or just LB over about anything else :P | 15:27 |
| sc68cal | prometheanfire: if you have a chance, please review https://review.openstack.org/168423 | 15:27 |
| sc68cal | we're trying to get a devstack configuration for LB that works, so we can resume testing it | 15:27 |
| prometheanfire | ah, neat | 15:27 |
| sc68cal | and possibly introduce it as a "minimal" neutron for devstack consumption | 15:27 |
| *** nfedotov has quit IRC | 15:28 | |
| *** bobmel_ has joined #openstack-neutron | 15:29 | |
| jschwarz | amuller, so I found what causes my fullstack tests to fail in said patch | 15:29 |
| *** ijw has joined #openstack-neutron | 15:29 | |
| jschwarz | amuller, apperantly adding 'import time;time.sleep(10)' after I spawn neutron-server but before any other agent, fixes it. | 15:30 |
| jschwarz | yay us | 15:30 |
| amuller | that sucks | 15:30 |
| prometheanfire | yay race | 15:30 |
| jschwarz | yeah. | 15:30 |
| prometheanfire | the eth0 thing is gonna suck | 15:30 |
| jschwarz | amuller, so I tried adding like a 'check that neutron-server is up before adding new agents' but that doesn't work | 15:30 |
| *** bobmel has quit IRC | 15:31 | |
| amuller | jschwarz: what issue are you talking about specifically? last time we talked you fixed the bridge naming thing | 15:31 |
| prometheanfire | eno1 is local here | 15:31 |
| jschwarz | amuller, specifically report_state doesn't reach the neutron-server. l3-agent for example times out after a minute. sec, I'll log | 15:31 |
| openstackgerrit | Sean M. Collins proposed openstack/neutron: DO NOT MERGE: Test OneConvergence IPv6 API operations https://review.openstack.org/168173 | 15:31 |
| jschwarz | amuller, http://logs.openstack.org/00/123000/46/check/check-neutron-dsvm-functional/c41a921/logs/neutron-l3-agent--2015-04-06--132138.txt.gz#_2015-04-06_13_22_39_786 | 15:32 |
| jschwarz | amuller, you'll see that the message is logged exactly one minute after the connection to the AMQP server was initiated | 15:32 |
| jschwarz | amuller, also that code can only be reached by an 'except RabbitTimeout' block | 15:32 |
| amuller | jschwarz: what happens if you start a L3 agent and after 5 seconds a Neutron server. Does the agent eventually start reporting its state correctly? | 15:33 |
| *** aranjan has joined #openstack-neutron | 15:33 | |
| jschwarz | amuller, gah, I didn't add a 'wait_until_true' in the code that checks it... :< sec | 15:34 |
| amuller | jschwarz: does the L3 agent report state time out and then the 2nd one succeeds? | 15:34 |
| jschwarz | amuller, I can try locally... give me a few | 15:35 |
| jschwarz | (btw, happy holidays :) ) | 15:35 |
| *** ijw has quit IRC | 15:35 | |
| *** absubram has quit IRC | 15:35 | |
| amuller | jschwarz: Thanks, you too | 15:35 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack test https://review.openstack.org/123000 | 15:36 |
| jschwarz | amuller, I can't try that locally since the l3 agent depends on neutron.conf, which is created for neutron-server... it will result in quite a rewrite just for this check | 15:36 |
| amuller | jschwarz: you can try it manually in devstack | 15:36 |
| amuller | outside of the full stack framework | 15:36 |
| jschwarz | yes I could | 15:36 |
| *** gsagie_ has joined #openstack-neutron | 15:36 | |
| jschwarz | sec | 15:37 |
| *** rwsu has joined #openstack-neutron | 15:37 | |
| busterswt | ijw Hi Ian. Questions about your MTU BP if you're around. https://blueprints.launchpad.net/neutron/+spec/mtu-selection-and-advertisement | 15:38 |
| *** carl_baldwin has joined #openstack-neutron | 15:38 | |
| jschwarz | amuller, it's stacking and gating, so i'll ping you whenever the first returns | 15:39 |
| *** nplanel_ has joined #openstack-neutron | 15:39 | |
| *** carl_baldwin has quit IRC | 15:40 | |
| *** Miouge has joined #openstack-neutron | 15:41 | |
| *** suresh12 has quit IRC | 15:41 | |
| *** carl_baldwin has joined #openstack-neutron | 15:41 | |
| *** pal has quit IRC | 15:42 | |
| *** pal has joined #openstack-neutron | 15:43 | |
| jschwarz | amuller, so manually starting neutron-server 10 seconds after the l3 agent works | 15:49 |
| jschwarz | :( | 15:49 |
| amuller | jschwarz: so what's the difference between doing it manually and the full stack tests? | 15:49 |
| jschwarz | amuller, configuration? but I've looked into that most of the day and already fixed meaningful dependencies | 15:50 |
| *** vishwana_ has quit IRC | 15:50 | |
| jschwarz | also starting the agents 10 seconds after the server works, so the configuration is good | 15:50 |
| jschwarz | maybe there's a retry rabbitmq option somewhere i've missed? | 15:50 |
| *** vishwanathj has joined #openstack-neutron | 15:50 | |
| *** yamamoto has joined #openstack-neutron | 15:50 | |
| *** yog__ has quit IRC | 15:51 | |
| jschwarz | amuller, it's been buffling me the whole day now :< | 15:52 |
| *** flaviof_ has quit IRC | 15:52 | |
| *** flaviof_ has joined #openstack-neutron | 15:53 | |
| *** dencaval has joined #openstack-neutron | 15:53 | |
| *** ZZelle_ has quit IRC | 15:54 | |
| *** seizadi has joined #openstack-neutron | 15:54 | |
| *** vishwanathj has quit IRC | 15:55 | |
| jschwarz | amuller, afraid I have to go... mail me if you can think of something? :) | 15:55 |
| amuller | jschwarz: Sorry I doubt I'll be able to assist you this week =/ | 15:55 |
| *** nlahouti has joined #openstack-neutron | 15:56 | |
| *** _cjones_ has joined #openstack-neutron | 15:56 | |
| *** emagana has quit IRC | 15:56 | |
| *** flaviof_ has quit IRC | 15:58 | |
| jschwarz | amuller, ack | 16:00 |
| *** jschwarz has quit IRC | 16:00 | |
| *** vthapar has quit IRC | 16:00 | |
| *** emagana has joined #openstack-neutron | 16:00 | |
| *** flaviof_ has joined #openstack-neutron | 16:00 | |
| *** _cjones_ has quit IRC | 16:01 | |
| *** annegentle has quit IRC | 16:03 | |
| *** SridharG1 has left #openstack-neutron | 16:03 | |
| *** flaviof_ has quit IRC | 16:04 | |
| *** flaviof_ has joined #openstack-neutron | 16:04 | |
| *** busterswt has quit IRC | 16:06 | |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: [WIP]: Add handling of SIGHUP - multiprocess mode https://review.openstack.org/161732 | 16:06 |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: Sync service from oslo-incubator https://review.openstack.org/167974 | 16:06 |
| openstackgerrit | Elena Ezhova proposed openstack/neutron: [WIP] Add SIGHUP support: single process mode https://review.openstack.org/170905 | 16:06 |
| *** flaviof_ is now known as flaviof | 16:06 | |
| *** marun has joined #openstack-neutron | 16:09 | |
| *** gsagie_ has quit IRC | 16:10 | |
| *** pritesh has joined #openstack-neutron | 16:10 | |
| *** adam_g_out is now known as adam_g | 16:12 | |
| *** armax has joined #openstack-neutron | 16:12 | |
| *** johnbelamaric has quit IRC | 16:13 | |
| *** atuvenie has quit IRC | 16:14 | |
| mestery | ZZelle: Can you rebase this one quickly and I'll move it back into the gate? Tahnsk! https://review.openstack.org/#/c/165115/ | 16:14 |
| mestery | emagana: Any updates on this bug for Kilo? How is it progressing? https://bugs.launchpad.net/neutron/+bug/1438329 | 16:15 |
| openstack | Launchpad bug 1438329 in neutron "Example configuration files lack changes for Kilo" [High,In progress] - Assigned to Edgar Magana (emagana) | 16:15 |
| *** kiran-r has joined #openstack-neutron | 16:15 | |
| emagana | mestery: Will be sending a patch later today ;-) | 16:17 |
| mestery | emagana: Ack, thank you! | 16:17 |
| *** kiran_ has joined #openstack-neutron | 16:18 | |
| *** crose has joined #openstack-neutron | 16:18 | |
| *** pritesh_ has joined #openstack-neutron | 16:20 | |
| *** kiran_ has quit IRC | 16:21 | |
| *** kiran_ has joined #openstack-neutron | 16:21 | |
| *** kiran-r has quit IRC | 16:22 | |
| *** kiran_ is now known as kiran-r | 16:22 | |
| *** mlavalle has joined #openstack-neutron | 16:23 | |
| *** pritesh has quit IRC | 16:23 | |
| *** alexpilotti has joined #openstack-neutron | 16:23 | |
| *** vishwanathj has joined #openstack-neutron | 16:26 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS Remove dependency on Neutron for unit test https://review.openstack.org/169008 | 16:27 |
| *** pritesh has joined #openstack-neutron | 16:28 | |
| *** vivek-ebay has joined #openstack-neutron | 16:28 | |
| *** ijw has joined #openstack-neutron | 16:29 | |
| *** tiswanso has quit IRC | 16:30 | |
| *** pritesh_ has quit IRC | 16:30 | |
| *** ivar-lazzaro has quit IRC | 16:30 | |
| *** tiswanso has joined #openstack-neutron | 16:30 | |
| *** kiran_ has joined #openstack-neutron | 16:30 | |
| *** aranjan has quit IRC | 16:31 | |
| *** sridhar_ram has joined #openstack-neutron | 16:31 | |
| *** baoli has quit IRC | 16:31 | |
| *** dboik_ has joined #openstack-neutron | 16:31 | |
| *** dboik_ has quit IRC | 16:31 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Allow metadata proxy running with nobody user/group https://review.openstack.org/165115 | 16:31 |
| *** dboik_ has joined #openstack-neutron | 16:32 | |
| *** pal has quit IRC | 16:32 | |
| *** ZZelle_ has joined #openstack-neutron | 16:33 | |
| *** seizadi has quit IRC | 16:33 | |
| *** seizadi has joined #openstack-neutron | 16:33 | |
| *** kiran-r has quit IRC | 16:34 | |
| *** dboik has quit IRC | 16:35 | |
| *** baoli has joined #openstack-neutron | 16:35 | |
| *** gsagie_ has joined #openstack-neutron | 16:35 | |
| openstackgerrit | pritesh proposed openstack/neutron: Moving VLAN Transparency support from core to extension https://review.openstack.org/169569 | 16:35 |
| *** ijw has quit IRC | 16:35 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Skip example retargetable functional test https://review.openstack.org/170915 | 16:36 |
| marun | HenryG, mestery, armax, kevinbenton: We should merge this skip asap to avoid accidental failure ^ | 16:36 |
| mestery | marun: +2 from me, thanks for the pointer | 16:37 |
| marun | mestery: thanks! | 16:37 |
| HenryG | marun: done | 16:39 |
| *** busterswt has joined #openstack-neutron | 16:39 | |
| marun | HenryG: thanks! | 16:39 |
| *** johnbelamaric has joined #openstack-neutron | 16:41 | |
| HenryG | marun: I assume some re-organization will allow these targets to co-run better, or independently? | 16:42 |
| marun | HenryG: that's the plan, the related bug documents the failure and I'll work with John to resolve | 16:42 |
| *** johnbelamaric has quit IRC | 16:42 | |
| *** johnbelamaric1 has joined #openstack-neutron | 16:42 | |
| *** vishwanathj has quit IRC | 16:42 | |
| *** sridhar_ram1 has joined #openstack-neutron | 16:42 | |
| marun | HenryG: another case of global use of cfg.CONF biting us | 16:43 |
| *** sridhar_ram has quit IRC | 16:43 | |
| armax | marun: it looks like I got here too late | 16:44 |
| *** killer_prince has joined #openstack-neutron | 16:45 | |
| *** killer_prince is now known as lazy_prince | 16:45 | |
| marun | armax: hopefully you'll be around for a rebased reorg patch :) | 16:45 |
| armax | marun: yeah, I just needs lots lots of coffee | 16:46 |
| marun | for that one, definiely | 16:46 |
| marun | definitely | 16:46 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS Remove dependency on Neutron for unit test https://review.openstack.org/169008 | 16:46 |
| *** erkules_ is now known as erkules | 16:47 | |
| *** erkules has joined #openstack-neutron | 16:47 | |
| *** smcquaid has joined #openstack-neutron | 16:47 | |
| *** vishwanathj has joined #openstack-neutron | 16:47 | |
| *** chandankumar has joined #openstack-neutron | 16:47 | |
| *** leenheer has joined #openstack-neutron | 16:51 | |
| *** smcquaid has quit IRC | 16:51 | |
| *** vishwanathj has quit IRC | 16:52 | |
| *** vishwanathj has joined #openstack-neutron | 16:53 | |
| *** akamyshnikova_ has quit IRC | 16:53 | |
| *** vishwana_ has joined #openstack-neutron | 16:54 | |
| *** suresh12 has joined #openstack-neutron | 16:54 | |
| *** suresh12 has quit IRC | 16:54 | |
| *** suresh12 has joined #openstack-neutron | 16:54 | |
| *** annegentle has joined #openstack-neutron | 16:55 | |
| *** vishwanathj has quit IRC | 16:57 | |
| *** yamamoto has quit IRC | 16:57 | |
| *** Aish has joined #openstack-neutron | 16:59 | |
| *** vishwana_ has quit IRC | 16:59 | |
| *** vishwanathj has joined #openstack-neutron | 17:00 | |
| *** pal has joined #openstack-neutron | 17:02 | |
| *** seizadi has quit IRC | 17:02 | |
| *** seizadi has joined #openstack-neutron | 17:02 | |
| openstackgerrit | Brian Haley proposed openstack/neutron: Add ipset element and hashsize tunables https://review.openstack.org/170328 | 17:04 |
| *** vishwanathj has quit IRC | 17:05 | |
| marun | pc_m, dougwig: hi | 17:05 |
| dougwig | marun: morning | 17:05 |
| pc_m | marun: hi | 17:05 |
| marun | have you seen this? https://review.openstack.org/#/c/158811/ | 17:05 |
| marun | It's close to being ready, I have an update I'm just running through local testing | 17:06 |
| pc_m | marun: no | 17:06 |
| dougwig | marun: i was going to run it against the *aas unit tests later today. need it sooner? | 17:06 |
| marun | It restructures the unit test tree in a way that is likely to break *aas, assuming the dependency on neutron unit test infrastructure remains | 17:06 |
| marun | dougwig: no, just wanted to make sure I wasn't springing it on you | 17:07 |
| marun | dougwig: there's a restructure script that you might be able to start from to automate the changes | 17:07 |
| dougwig | marun: ok, i'll take a look. | 17:07 |
| *** Marga__ has quit IRC | 17:07 | |
| marun | dougwig: it's not actually part of the commit, because this is intended to be on-shot | 17:08 |
| marun | dougwig: but I submitted it by accident so it's on gerrit | 17:08 |
| marun | https://review.openstack.org/#/c/170629/ | 17:08 |
| dougwig | ahh, sweet. | 17:08 |
| *** johnbelamaric1 has quit IRC | 17:08 | |
| marun | this might also be a great time to start vendoring, though that might be more work | 17:08 |
| openstackgerrit | Carl Baldwin proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool https://review.openstack.org/165910 | 17:09 |
| *** baoli has quit IRC | 17:09 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize unit test tree https://review.openstack.org/158811 | 17:09 |
| marun | armax, HenryG, mestery, kevinbenton: assuming my local success can be replicated in the gate, this should be ready ^ | 17:10 |
| armax | marun: looking | 17:10 |
| *** aepifanov has quit IRC | 17:10 | |
| pc_m | marun: So we need to hurry up and update the *aaS repos too? | 17:10 |
| marun | the docs and check script are probably worth reviewing. the renames, I'm not so sure. | 17:10 |
| mestery | marun: nice and thanks! | 17:11 |
| *** Miouge has quit IRC | 17:11 | |
| marun | pc_m: yes, I hope to merge this today so the sooner you fix the less you'll be broken | 17:11 |
| *** Marga_ has joined #openstack-neutron | 17:12 | |
| marun | ah crap, ws :( | 17:12 |
| marun | will respin | 17:12 |
| *** ijw has joined #openstack-neutron | 17:12 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize unit test tree https://review.openstack.org/158811 | 17:12 |
| *** baoli has joined #openstack-neutron | 17:13 | |
| pc_m | marun: I've been working on UT cleanup in VPN and have one that is ready for review (once I get the rebase resolved). Would be nice to get it in, before breakage. | 17:13 |
| marun | pc_m: We can try to make that happen | 17:14 |
| *** baoli has quit IRC | 17:14 | |
| *** vishwanathj has joined #openstack-neutron | 17:14 | |
| *** tiswanso has quit IRC | 17:14 | |
| *** achanda has joined #openstack-neutron | 17:15 | |
| pc_m | marun: I'm a bit confused as to what is going on with mine and could use some advice. The commit was dependent on another that was upstreamed and merged (165226). I rebased to origin/master and pushed, but it says merge failed. | 17:17 |
| *** YorikSar has quit IRC | 17:17 | |
| pc_m | marun: https://review.openstack.org/#/c/169008/2 | 17:17 |
| *** banix has quit IRC | 17:17 | |
| *** tiswanso_ has joined #openstack-neutron | 17:17 | |
| pc_m | marun: Not sure what I'm missing... | 17:17 |
| marun | pc_m: I'm not seeing 'merge failed'? | 17:18 |
| pc_m | marun: duh. It was on Cisco CI... | 17:18 |
| mestery | marun: Reviewed | 17:18 |
| marun | ah, ok :) | 17:18 |
| pc_m | Thought I was loosing my mind. | 17:18 |
| *** smcquaid has joined #openstack-neutron | 17:18 | |
| pc_m | marun: I need to ping them about that. See it occasionally. | 17:18 |
| *** baoli_ has joined #openstack-neutron | 17:19 | |
| openstackgerrit | Carl Baldwin proposed openstack/python-neutronclient: Allow passing None for subnetpool https://review.openstack.org/168499 | 17:19 |
| *** Marga_ has quit IRC | 17:19 | |
| *** smcquaid has quit IRC | 17:20 | |
| marun | mestery: thanks! | 17:20 |
| *** jorgem has joined #openstack-neutron | 17:21 | |
| *** vishwanathj has quit IRC | 17:21 | |
| *** rms_13 has joined #openstack-neutron | 17:22 | |
| *** vishwanathj has joined #openstack-neutron | 17:22 | |
| *** yamahata has quit IRC | 17:22 | |
| *** vhosakot has joined #openstack-neutron | 17:24 | |
| *** vishwanathj has quit IRC | 17:26 | |
| *** kiran_ has quit IRC | 17:27 | |
| *** johnbelamaric has joined #openstack-neutron | 17:28 | |
| *** sballe has joined #openstack-neutron | 17:28 | |
| armax | marun: reviewing too | 17:29 |
| armax | marun: one comment I have is about the use of bash | 17:29 |
| *** rm_work is now known as rm_work|away | 17:29 | |
| armax | marun: not that I care particularly about pure shell | 17:29 |
| marun | armax: arrays in pure shell sucks | 17:29 |
| HenryG | armax: no, I was going to comment that | 17:30 |
| armax | marun: I know | 17:30 |
| marun | armax: If it was something for runtime, I'd see the concern. Otherwise, there's no value in restricting ourselves | 17:30 |
| * HenryG deletes draft comment on use of bash | 17:30 | |
| marun | I like dougwig's idea of adding a comment in the header line so we don't have to keep updating misc checks | 17:31 |
| armax | marun: agreed, my point is that we can get yamamoto to rewrite your file in pure shell | 17:31 |
| marun | armax: and then who's going to maintain it? | 17:31 |
| armax | :) | 17:31 |
| *** gsagie_ has quit IRC | 17:31 | |
| * marun is so easily trolled :/ | 17:31 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:31 | |
| *** pritesh_ has joined #openstack-neutron | 17:31 | |
| armax | ok, let me step back | 17:31 |
| *** jorgem has quit IRC | 17:31 | |
| armax | I’d be okay with having bash files | 17:32 |
| HenryG | I think we should allow the use of bash. Why do some not like it? | 17:32 |
| marun | HenryG: it has more overhead | 17:32 |
| armax | but having them called in tox -epep8 goes against the check_opinionated_shell | 17:32 |
| *** leenheer has left #openstack-neutron | 17:32 | |
| armax | so I think that keep bumping check_opinionated_shell’s EXPECTED is not really viable | 17:33 |
| marun | HenryG: so for things that we are going to invoke at runtime, sh is preferable if at all possible | 17:33 |
| marun | check_opinionated_shell? | 17:33 |
| armax | the function’s whose EXPECTED value you modified | 17:33 |
| HenryG | marun: https://review.openstack.org/#/c/158811/7/tools/misc-sanity-checks.sh | 17:33 |
| *** kothari has joined #openstack-neutron | 17:33 | |
| marun | armax: I'd be for simply removing it | 17:33 |
| marun | I think it's a pointless waste of our time | 17:34 |
| armax | I agree that writing in shell is a bit of a pain | 17:34 |
| armax | marun: it took me a while to write the pylint check in pure shell | 17:34 |
| marun | I certainly didn't agree to that kind of arbitrary restriction | 17:34 |
| *** julim has quit IRC | 17:34 | |
| marun | who did, exactly? | 17:34 |
| * marun goes to check | 17:34 | |
| dougwig | the objection was not against bash, but against using it when you didn't need it, because some users are on netbsd. | 17:34 |
| *** pritesh has quit IRC | 17:34 | |
| armax | yamamoto | 17:34 |
| dougwig | note that i'm all for bash, since it's 2015 and all. | 17:35 |
| marun | dougwig: users vs developers :/ | 17:35 |
| armax | and I think also john swartz liked it | 17:35 |
| *** michatotol has quit IRC | 17:35 | |
| dougwig | marun: devs. i think some of our japan contributors dev on netbsd. | 17:35 |
| marun | dougwig: anything that is developer-only shouldn't be subject to this kind of arbitrary restriction | 17:35 |
| armax | all I am saying…is that bumping expected for a command that goes in tox -epep8 is not really the right thing to do | 17:35 |
| armax | we either kill shell in favor of bash | 17:35 |
| dougwig | here was the thing marun was referring to, though perhaps it needs to be broken into its own review: https://review.openstack.org/#/c/163062/ | 17:35 |
| marun | dougwig: wait, netbsd doesn't have bash? | 17:35 |
| HenryG | Is it not possible to install bash on netbsd? | 17:35 |
| armax | or we keep using pure shell for commands that get invoked in tox -epep8 | 17:35 |
| marun | dougwig: I'd vote for that | 17:35 |
| marun | dougwig: although, I'd also be in favor of dropping the arbitrary restriction | 17:36 |
| marun | maybe I'll go ahead and propose that | 17:36 |
| marun | I'd rather netbsd folks move into the present than be held back by their intransigence | 17:36 |
| *** pritesh_ has quit IRC | 17:37 | |
| armax | marun: I am okay either way | 17:37 |
| *** julim has joined #openstack-neutron | 17:37 | |
| *** leenheer has joined #openstack-neutron | 17:37 | |
| marun | armax: I don't understand even a little bit how restricting outselves to pure shell makes sense | 17:37 |
| marun | I mean, they don't use devstack? | 17:37 |
| armax | marun: my only point being: bumping the variable is equivalent as getting rid of the check altogether | 17:37 |
| *** baoli_ has quit IRC | 17:38 | |
| *** baoli has joined #openstack-neutron | 17:38 | |
| armax | marun: not necessarily if your dev env != from test | 17:38 |
| marun | armax: which is stupid, but whatever | 17:38 |
| armax | marun: well, I certainly don’t! | 17:38 |
| armax | marun: perhaps we can just file a bug associted to the change https://review.openstack.org/#/c/163062/ | 17:39 |
| armax | titled: ‘pure shell kills productivity’ and have yamamoto or whomever is interest comment ettc | 17:39 |
| *** catohornet has joined #openstack-neutron | 17:39 | |
| armax | marun: that said, I am satifisifed that at least we acklowledged the issue | 17:40 |
| *** sripriya has joined #openstack-neutron | 17:41 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 17:41 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 17:41 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseLinuxTestCase by BaseSudoTestCase https://review.openstack.org/161913 | 17:41 |
| *** jorgem has joined #openstack-neutron | 17:42 | |
| *** wpward has joined #openstack-neutron | 17:43 | |
| *** sthillma has joined #openstack-neutron | 17:44 | |
| *** tomoe_ has joined #openstack-neutron | 17:45 | |
| *** atuvenie has joined #openstack-neutron | 17:46 | |
| *** yamahata has joined #openstack-neutron | 17:47 | |
| *** tomoe_ has quit IRC | 17:49 | |
| *** YorikSar has joined #openstack-neutron | 17:49 | |
| *** Marga_ has joined #openstack-neutron | 17:50 | |
| *** Aish has quit IRC | 17:50 | |
| *** banix has joined #openstack-neutron | 17:51 | |
| *** Aish has joined #openstack-neutron | 17:51 | |
| *** atuvenie has quit IRC | 17:55 | |
| *** baoli has quit IRC | 17:56 | |
| *** ijw has quit IRC | 17:57 | |
| *** leenheer has left #openstack-neutron | 17:57 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Remove check for bash usage https://review.openstack.org/170939 | 17:58 |
| *** sripriya has quit IRC | 17:59 | |
| *** dboik_ has quit IRC | 17:59 | |
| *** baoli_ has joined #openstack-neutron | 18:00 | |
| *** dboik has joined #openstack-neutron | 18:00 | |
| *** jorgem1 has joined #openstack-neutron | 18:01 | |
| *** jorgem1 has joined #openstack-neutron | 18:01 | |
| *** jorgem1 has quit IRC | 18:03 | |
| *** jorgem has quit IRC | 18:03 | |
| marun | dougwig, armax ^^ I've submitted a patch proposal and added to tomorrow's agenda | 18:03 |
| armax | marun: looking | 18:03 |
| openstackgerrit | Maru Newby proposed openstack/neutron: Remove check for bash usage https://review.openstack.org/170939 | 18:03 |
| *** luqas has joined #openstack-neutron | 18:04 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Remove check for bash usage https://review.openstack.org/170939 | 18:04 |
| *** Marga_ has quit IRC | 18:04 | |
| * marun needs a grammar checker already | 18:04 | |
| armax | I was going to be your grammarf checker :) | 18:04 |
| armax | but you were faster | 18:04 |
| *** Marga_ has joined #openstack-neutron | 18:04 | |
| *** vishwanathj has joined #openstack-neutron | 18:06 | |
| *** dave-mccowan has quit IRC | 18:07 | |
| *** luqas has quit IRC | 18:08 | |
| dougwig | Did you mean to workflow that? | 18:10 |
| marun | mestery: regarding unit test reorg, how would you like to see the landing coordinated? | 18:10 |
| marun | workflow? | 18:10 |
| *** atuvenie has joined #openstack-neutron | 18:10 | |
| marun | oh, yeah | 18:10 |
| mestery | marun: However it happens, it may be best to land it ASAP, I think we may cut the RC branch tomorrow. | 18:10 |
| *** yamahata has quit IRC | 18:10 | |
| marun | dougwig: don't want it merging before we talk about it tomorrow | 18:10 |
| dougwig | armax: the bash wip | 18:10 |
| dougwig | I got a +a email. :) | 18:11 |
| marun | dougwig: maybe yamamoto has a good reason I'm not seeing for restricting | 18:11 |
| mestery | I can talk to ttx and maybe cut the RC branch Wednesday | 18:11 |
| marun | mestery: ok | 18:11 |
| *** harlowja_away is now known as harlowja | 18:11 | |
| dougwig | Right, I think it got mis-merged. | 18:11 |
| marun | mestery: armax suggested tracking down those with pending reviews so they know about the potential conflict | 18:11 |
| dougwig | Pecking on my phone. | 18:11 |
| *** johnbelamaric has quit IRC | 18:11 | |
| armax | dougwig: what did I do? | 18:11 |
| armax | I am still full from yesterday’s meal | 18:12 |
| mestery | marun: armax is smart like that :)( | 18:12 |
| armax | and can’t think straight | 18:12 |
| dougwig | Merged the bash wip | 18:12 |
| dougwig | marun: can we test aas units first? | 18:12 |
| mestery | marun: I'm mostly concerned about 1) the RC, and second 2) ensuring we get this merged into the RC so backports are easier | 18:12 |
| *** vishwanathj has quit IRC | 18:12 | |
| mestery | 3) aas unit tests. /cc dougwig | 18:12 |
| mestery | marun: Make sense? | 18:12 |
| *** vishwanathj has joined #openstack-neutron | 18:13 | |
| marun | dougwig: I think so, yeah | 18:14 |
| marun | dougwig: hopefully skip-if will mean your test changes don't require tempest getting involved. we're dumb enough to use setup.cfg entrypoints to discover test-only classes so we're not so lucky | 18:15 |
| *** rangansw has joined #openstack-neutron | 18:15 | |
| *** Marga_ has quit IRC | 18:16 | |
| *** pal has quit IRC | 18:16 | |
| marun | in fact, I'm going to file a bug on that bit of terrible while it's on my mind | 18:16 |
| *** Marga_ has joined #openstack-neutron | 18:16 | |
| *** vishwana_ has joined #openstack-neutron | 18:16 | |
| *** jeroenvdl has quit IRC | 18:16 | |
| *** vishwanathj has quit IRC | 18:17 | |
| *** crose has quit IRC | 18:18 | |
| *** vishwana_ has quit IRC | 18:21 | |
| *** sbfox has joined #openstack-neutron | 18:21 | |
| *** jorgem has joined #openstack-neutron | 18:21 | |
| *** ebagdasa has quit IRC | 18:22 | |
| armax | HenryG: ping | 18:22 |
| HenryG | armax: pong | 18:22 |
| *** bobmel has joined #openstack-neutron | 18:22 | |
| marun | dougwig: how fast do you figure you can get vpnaas unit tests running? | 18:22 |
| *** signed8bit is now known as signed8bit_ZZZzz | 18:22 | |
| armax | can you get dane on irc, I was looking at his patch today and I had a comment that would involve improving a change that kevinbenton merged in the kilo timeframe | 18:22 |
| marun | dougwig: also, who's on point for fwaas? | 18:22 |
| *** dboik has quit IRC | 18:23 | |
| *** ebagdasa has joined #openstack-neutron | 18:23 | |
| marun | dougwig: the check queue for the change is eta ~30m, would like to merge before anything conflicting lands though maybe that's a longshot | 18:23 |
| pc_m | marun: dougwig: Are you doing the vpnaas changes? (I was going to) | 18:23 |
| dougwig | I'm boarding a plane, with no empty seats, so I won't be able to touch it for a few hours. | 18:23 |
| marun | pc_m: oh, I mean lbaas, sorry | 18:23 |
| armax | HenryG: I’d also need to thank him for preventing me to resign | 18:23 |
| pc_m | marun: Sumit is FWaaS point | 18:23 |
| kevinbenton | armax: which change? | 18:23 |
| pc_m | marun: OK. had me confused. | 18:23 |
| armax | kevinbenton: the one you and I were talking about: https://review.openstack.org/#/c/170690/ | 18:24 |
| HenryG | armax: he is currently offline, i'll get him here when he shows up | 18:24 |
| armax | HenryG: ok, thanks | 18:24 |
| dougwig | I was going to start with Lbaas and then fix any others that needed it. If someone can test before me, great. | 18:24 |
| *** rm_work|away is now known as rm_work | 18:24 | |
| kevinbenton | armax: no, i mean which of my patches? | 18:24 |
| marun | dougwig: so we can't merge until you land and have time to work:/ | 18:24 |
| pc_m | marun: I'm got a bit of work I'm in the middle of, but then will start on VPN changes...later today or first thing tomorrow (East coast time) | 18:24 |
| armax | kevinbenton: there’s the hash in my comment, looking... | 18:25 |
| armax | kevinbenton: abc16ebf | 18:25 |
| marun | mestery: it looks like we're going to be held up on *aas testing today and likely won't be able to merge | 18:25 |
| armax | kevinbenton: https://review.openstack.org/#/c/124478/ | 18:25 |
| *** bobmel_ has quit IRC | 18:26 | |
| mestery | marun: OK. I would push the RC release out then, beause I tihnk we need this as a part of the RC. Is there a bug for this work so I can target it to the RC? | 18:26 |
| mestery | marun: Ack, it's a BP, I see it now. | 18:26 |
| armax | mestery: I think it’s a blueprint | 18:26 |
| mestery | marun armax: Any chacne we can get a bug? ttx will want to know why the release is blocked, I can file one quick if you guys want | 18:26 |
| *** dboik has joined #openstack-neutron | 18:26 | |
| marun | https://blueprints.launchpad.net/neutron/+spec/reorganize-unit-test-tree | 18:27 |
| armax | I am okay with demoting this to a bug | 18:27 |
| marun | yeah, I don't think it matters what it gets tracked aas | 18:27 |
| marun | ok, I'll file | 18:28 |
| *** rm_work is now known as rm_work|away | 18:28 | |
| mestery | marun: Thanks! That will make it easier to explain to ttx why we're stalling, if I added a BP this late he'd freak out :) | 18:28 |
| mestery | marun: And I agree, lets get this in pre-RC, so no worries there | 18:28 |
| marun | mestery: it's all procedural anyway. we've been delaying this until close to rc to minimize conflict | 18:28 |
| mestery | marun: Exactly! | 18:28 |
| mestery | marun: And thanks again for tracking this and doing all hte hardwork | 18:29 |
| *** sudipto has joined #openstack-neutron | 18:29 | |
| *** vivek-ebay has quit IRC | 18:29 | |
| dougwig | marun: ping blogan | 18:30 |
| marun | blogan: hi! | 18:30 |
| marun | dougwig: safe flight! | 18:30 |
| *** johnbelamaric has joined #openstack-neutron | 18:32 | |
| marun | mestery: https://bugs.launchpad.net/neutron/+bug/1440834 | 18:32 |
| openstack | Launchpad bug 1440834 in neutron "Unit test tree does not match the structure of the code tree" [Critical,In progress] | 18:32 |
| openstackgerrit | Brian Haley proposed openstack/neutron: Add ipset element and hashsize tunables https://review.openstack.org/170328 | 18:32 |
| mestery | marun: Tahnks sir! | 18:32 |
| *** rm_work|away is now known as rm_work | 18:34 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize unit test tree https://review.openstack.org/158811 | 18:34 |
| *** suresh12 has quit IRC | 18:34 | |
| kevinbenton | armax: i don't think there is a point | 18:35 |
| *** ijw has joined #openstack-neutron | 18:35 | |
| kevinbenton | armax: the only thing that calls in that direction is an RPC call to retrieve details | 18:35 |
| armax | ok | 18:36 |
| kevinbenton | armax: the context associated with that won't have any session editing other stuff | 18:36 |
| *** sridhar_ram1 has quit IRC | 18:36 | |
| kevinbenton | armax: i will leave a comment | 18:36 |
| armax | kevinbenton: ok | 18:36 |
| *** dane_leblanc has joined #openstack-neutron | 18:37 | |
| armax | kevinbenton: so it would be only cleaner, rather than necessary, is that your point? | 18:37 |
| kevinbenton | armax: yeah, i'm really uncertain where this fix is helping though | 18:38 |
| mestery | armax kevinbenton: What's the status of this one? https://review.openstack.org/#/c/170690/ | 18:39 |
| mestery | armax kevinbenton: Last I saw, armax was waiting for kevinbenton to comment on it. | 18:40 |
| *** s3wong has joined #openstack-neutron | 18:40 | |
| armax | mestery: that’s the one we’re talking about now | 18:40 |
| kevinbenton | mestery: i'm processing it now | 18:40 |
| mestery | armax kevinbenton: Ack, sorry to interrupt, carry on :P | 18:40 |
| armax | kevinbenton: are you punching cards? | 18:40 |
| armax | :) | 18:40 |
| * armax funny | 18:41 | |
| dane_leblanc | armax, kevinbenton: HenryG and I are working on another possible solution | 18:41 |
| armax | dane_leblanc: hi! | 18:41 |
| dane_leblanc | armax: Hi! | 18:41 |
| amuller | marun: heya Maru, are you aware of any advancement in multi node Tempest? | 18:42 |
| armax | dane_leblanc: is that going to be based on 170690 or is it different? | 18:42 |
| marun | amuller: apparently it's on | 18:42 |
| marun | amuller: sdague posted recently that it's running in the nova check queue | 18:42 |
| *** vishwanathj has joined #openstack-neutron | 18:42 | |
| amuller | hmm I missed that | 18:42 |
| marun | aiopcpu? | 18:42 |
| marun | armax: ^ | 18:42 |
| dane_leblanc | armax: Separate patch | 18:42 |
| marun | ? | 18:43 |
| amuller | are there any Neutron jobs that run multinode or is it just that nova one? | 18:43 |
| armax | amuller: swami and I are working on it | 18:43 |
| *** johnbelamaric has quit IRC | 18:43 | |
| armax | amuller: we got to patches pending | 18:43 |
| armax | amuller: one for d-g and the other for project config | 18:43 |
| armax | amuller: swami is the owner of both | 18:43 |
| *** dboik has quit IRC | 18:43 | |
| armax | they flip the existing experimental neutron aiopcpu to be dvr enabled | 18:44 |
| *** johnbelamaric has joined #openstack-neutron | 18:44 | |
| *** crose has joined #openstack-neutron | 18:44 | |
| armax | dane_leblanc: ok | 18:44 |
| armax | dane_leblanc: so would the newer supersedes the former? | 18:44 |
| amuller | armax: what tests would run against that job? | 18:45 |
| amuller | I mean where do I put a test that would run on that job | 18:45 |
| armax | that would be tempest | 18:45 |
| dane_leblanc | armax: Yes, hopefully, but what we're doing might not be 100% right, need some input to see best way to fix it. | 18:45 |
| armax | amuller: I don’t think the fullstack work can allow multi-node deployment yet | 18:46 |
| amuller | armax: if I had a test that requires multi node and would fail on AIO, is there a way to flag such a test? | 18:46 |
| amuller | in Tempest | 18:46 |
| amuller | armax: no it doesn't | 18:46 |
| armax | and it’s my understanding that is going to be the infrastructure that the api job will rely on | 18:46 |
| amuller | armax: I'm heavily involved in the full stack work I'm well aware of the status there | 18:46 |
| armax | amuller: ok, so disregard my last comment | 18:46 |
| armax | dane_leblanc: ok, tag us as soon as it’s up | 18:47 |
| armax | amuller: typically these are driven by config variables | 18:47 |
| armax | for instance live migration tests require multi-node | 18:47 |
| amuller | armax: I'll look at those then, thanks | 18:47 |
| armax | those are driven by a flag that skips the test when the feature is off | 18:47 |
| dane_leblanc | armax: Will do. | 18:47 |
| armax | amuller: there are a few example like these | 18:47 |
| armax | in the tempest code | 18:47 |
| amuller | armax: hopefully we can have a few basic L3 HA tests then in Tempest :) | 18:47 |
| armax | amuller: that would be neat | 18:48 |
| kevinbenton | armax, dane_leblanc: left a comment on the patch | 18:48 |
| pc_m | marun: Have a question on functional tests. Have a minute? | 18:48 |
| kevinbenton | need to step way for 10-15 mins | 18:49 |
| armax | kevinbenton: thanks kevinbenton | 18:49 |
| *** dboik has joined #openstack-neutron | 18:50 | |
| marun | pc_m: shoot | 18:50 |
| *** Aish has quit IRC | 18:50 | |
| pc_m | marun: In configure_for_funct_testing.sh it does two things... one is sources neutron-legacy, which install sudoers for /usr/local/bin/neutron-rootwrap, and _install_rootwrap_sudoers, which seems to supt sudoers for .tox/dsvm-functional/.../neutron-rootwrap | 18:50 |
| marun | armax: what is actually implied by multi-node? | 18:51 |
| armax | marun: what do you mean? | 18:51 |
| marun | armax: I can't imagine it being more than a single controller+compute and another copmute | 18:51 |
| pc_m | marun: In my VPN funtional test, it is failing, saying that /usr/local/bin/neutron-rootwrap does not exist. | 18:51 |
| marun | it's not arbitrary, in other words | 18:51 |
| armax | marun: yes, that’s what it is | 18:51 |
| marun | amuller: so I don't know that l3 ha is going to benefit from the current state of things | 18:51 |
| marun | amuller: since it's multi-compute, not multi-controller | 18:52 |
| amuller | marun: I see it's AIO + 1 other compute | 18:52 |
| pc_m | marun: Wondering... who installs neutron-rootwrap into /usr/local/bin? Should we be running that or the one in .tox? | 18:52 |
| *** kutija has quit IRC | 18:52 | |
| armax | marun, amuller I don’t think we could piggyback on the same infrastructure | 18:52 |
| marun | pc_m: devstack installs it to /usr/local/bin | 18:52 |
| armax | marun: I mean same job | 18:53 |
| *** wpward has left #openstack-neutron | 18:53 | |
| * amuller sighs | 18:53 | |
| armax | marun: the extra node can really run whatever you’d want | 18:53 |
| marun | pc_m: the neutron functional setup instead installs a sudo rule that allows targeting the rootwrap in the tox env's bin | 18:53 |
| pc_m | marun: But we're not stacking with the latest. | 18:53 |
| marun | pc_m: not sure what you mean | 18:53 |
| armax | marun: for instance for DVR it runs an L3 agent in ‘dvr’ mode | 18:53 |
| armax | marun: plust n-cpu and l2-agt | 18:53 |
| marun | armax: so you can potentially run multiple l3's? | 18:53 |
| armax | marun: technically speaking it’s possible | 18:54 |
| amuller | armax: are you planning on DVR + vlans or tunnels + l2pop? | 18:54 |
| marun | armax: and that's configured in the job def? (link?) | 18:54 |
| amuller | armax: Because we don't test tunnels and l2pop anywhere, that alone would be a huge win | 18:54 |
| armax | amuller: when it’s up... | 18:54 |
| pc_m | marun: I see the configure_for_func_testing.sh making a sudo entry for .tox and the neutron-legacy source doing the same for /usr/local/bin version of script. | 18:54 |
| armax | amuller: and I stress, when it’s up… | 18:54 |
| armax | amuller: it’s going to be DVR with l2pop with vxlan | 18:55 |
| amuller | armax: that same deployment should be good for L3 HA as well then | 18:55 |
| pc_m | marun: But I don't see where neutron-rootwrapper is placed into /user/local/bin (or .tox/ for that matter). | 18:55 |
| *** signed8bit_ZZZzz is now known as signed8bit | 18:55 | |
| armax | amuller: it’s the underlying config driven by the Q_DVR_MODE available in devstack | 18:55 |
| armax | amuller: at one point vivek was going to work on tweaking devstack to allow the configuration of vlan instead of vxlan | 18:56 |
| *** Marga_ has quit IRC | 18:56 | |
| armax | but I fear that effort has stalled | 18:56 |
| *** atuvenie has quit IRC | 18:56 | |
| dane_leblanc | kevinbenton: Can you help me understand your concern re. the removal of the subtransaction, and the parent transaction getting into an error state? | 18:56 |
| pc_m | marun: On my local machine the FT works, but it fails when run in check queue, where it is looking for /usr/local/bin/neutron-rootwrap | 18:56 |
| marun | pc_m: installing neutron system-wide puts all commands (including rootwrap) in /usr/local/bin | 18:56 |
| marun | pc_m: installing in tox env puts it in .tox/[env name]/bin | 18:56 |
| marun | pc_m: we configure the tox env to use the right rootwrap via env vars | 18:57 |
| marun | https://github.com/openstack/neutron/blob/master/tox.ini#L43 | 18:57 |
| *** ijw has quit IRC | 18:57 | |
| marun | pc_m: that it's failing in the gate suggests that tox.ini is not setting the env correctly | 18:57 |
| pc_m | marun: So my confusion is that it is looking in /usr/local/bin for the script. | 18:58 |
| kevinbenton | dane_leblanc: i remember under certain conditions an exception being thrown by a query would stop any more things from being done inside of the transaction | 18:58 |
| pc_m | marun: Ah, so the tox.ini has /usr/local/bin | 18:58 |
| kevinbenton | dane_leblanc: so to be safe, just keep the subtransaction there | 18:58 |
| marun | pc_m: right | 18:58 |
| pc_m | So I need to use {envbindir} | 18:59 |
| *** ijw has joined #openstack-neutron | 18:59 | |
| dane_leblanc | kevinbenton: Ah, yes, I see now. | 18:59 |
| *** jorgem has quit IRC | 18:59 | |
| dane_leblanc | kevinbenton: Thanks! | 18:59 |
| pc_m | marun: Thanks! I'll correct it! | 18:59 |
| marun | pc_m: np | 18:59 |
| kevinbenton | dane_leblanc: but in the broader context, i don't think you should edit those functions because they aren't related to the issue from what i can see | 19:00 |
| kevinbenton | dane_leblanc: this late in the cycle, i would prefer the fix was just isolated to the problem get_port_binding_host function | 19:01 |
| dane_leblanc | kevinbenton: Makes sense. I'll revert the other changes. | 19:02 |
| kevinbenton | dane_leblanc: then we can refactor the rest to use context passing after L opens | 19:02 |
| HenryG | kevinbenton: dane_leblanc: ++ | 19:02 |
| *** baoli_ has quit IRC | 19:02 | |
| *** jorgem has joined #openstack-neutron | 19:03 | |
| *** sridhar_ram has joined #openstack-neutron | 19:04 | |
| *** ijw_ has joined #openstack-neutron | 19:04 | |
| pc_m | marun: I see in the tox.ini you gave, there was also a OS_ROOTWRAP_DAEMON_CMD specified. Is that used, and if so, under what conditions? | 19:04 |
| marun | pc_m: rootwrap daemon has a significant impact on repeated rootwrap invocations | 19:04 |
| marun | pc_m: it's controlled by a configuration variable that I believe is enabled by default in neutron as of kilo | 19:05 |
| *** dboik has quit IRC | 19:06 | |
| marun | pc_m: I lied | 19:06 |
| marun | https://github.com/openstack/neutron/blob/master/neutron/agent/common/config.py#L38 | 19:06 |
| marun | pc_m: the configuration var needs to be set to enable ti | 19:06 |
| *** ijw has quit IRC | 19:07 | |
| pc_m | marun: OK, for VPN, where they have neutron-rootwrap running neutron-vpn-netns-wrapper (from what I can tell). Sounds like I can just use OS_ROOTWRAP_CMD. | 19:09 |
| marun | pc_m: I think you might want to use the daemon as well | 19:09 |
| marun | pc_m: but that's just me | 19:09 |
| marun | pc_m: maybe too late to decide for kilo | 19:09 |
| marun | pc_m: they are both required | 19:09 |
| marun | pc_m: the daemon is only good for blocking calls, not long-running daemons | 19:10 |
| pc_m | marun: There's two cases, I guesss. One is the starting of the VPN process, which is long running. The other are commands for checking VPN link status. | 19:10 |
| marun | pc_m: since the ability to communicate with spawned processes is minimal | 19:10 |
| marun | pc_m: as I said, probably not something to worry about this close to rc | 19:10 |
| marun | it will work regardless | 19:10 |
| marun | it's an optimization | 19:10 |
| pc_m | marun: So even for the VPN process runs? | 19:10 |
| * pc_m just trying to understand the mechanism | 19:11 | |
| pc_m | marun: Or is there a way, on a per call basis to use one vs the other? | 19:11 |
| marun | pc_m: vpn process -> managed by a process manager | 19:11 |
| marun | pc_m: I think by default the daemon would be used | 19:12 |
| marun | pc_m: but otherwiseguy is the best guy to ask | 19:12 |
| marun | pc_m: most client code doesn't have to are | 19:12 |
| marun | care | 19:12 |
| marun | pc_m: https://github.com/openstack/neutron/blob/master/neutron/agent/linux/utils.py | 19:13 |
| *** chandankumar has quit IRC | 19:13 | |
| otherwiseguy | daemon, if enabled, will be used for anything that uses execute(). Anything using create_process() will just use regular rootwrap. | 19:13 |
| marun | otherwiseguy: so things like processmanager presumably use create_process? | 19:14 |
| otherwiseguy | marun: yes. | 19:14 |
| marun | otherwiseguy: ok, thanks for clarification | 19:14 |
| *** johnbelamaric has quit IRC | 19:15 | |
| otherwiseguy | marun: actually, processmanager uses ip netns exec. | 19:15 |
| otherwiseguy | which uses execute. | 19:15 |
| otherwiseguy | hmm. | 19:15 |
| marun | yikes | 19:15 |
| marun | that would seem to be problematic | 19:15 |
| marun | though I guess we haven't hit any problem yet? | 19:15 |
| marun | (not sure how that's possible) | 19:15 |
| otherwiseguy | marun: yeah, I'd have to stare at it a bit... | 19:17 |
| marun | otherwiseguy: please do :) | 19:17 |
| pc_m | marun: VPN has a | 19:17 |
| pc_m | marun: process manager, but it is not a ProcessManager :) | 19:18 |
| *** nuritv has joined #openstack-neutron | 19:18 | |
| blogan | marun: sorry was in a meeting | 19:18 |
| blogan | marun: do i need to test lbaas unit tests with a change? | 19:18 |
| *** boris-42 has quit IRC | 19:18 | |
| marun | blogan: yes | 19:18 |
| marun | https://review.openstack.org/#/c/158811/ | 19:18 |
| *** uschreiber_ has joined #openstack-neutron | 19:18 | |
| marun | blogan: guaranteed to break you | 19:18 |
| pc_m | marun: Just an object used to manage things. Everything called, goes through execute() it looks like | 19:18 |
| blogan | well if its guaranteed to break, ill check that guarantee out | 19:18 |
| marun | blogan: the restructure script in this abandoned change may be helpful in automating the rewriting of imports: https://review.openstack.org/#/c/170629/ | 19:19 |
| marun | blogan: it's a one-shot deal so that change being on gerrit was an accident, but it may prove useful after all. | 19:19 |
| blogan | ah okay you're just moving test classes/modules around? | 19:19 |
| marun | blogan: that's correct | 19:20 |
| marun | blogan: strictly a rename | 19:20 |
| blogan | marun: ah okay, ill get this started then | 19:21 |
| marun | blogan: awesome, thanks :) | 19:21 |
| nuritv | Hi all, i'm looking for reviewers to look at this bug fix - https://review.openstack.org/#/c/162648/. it's a small fix after several fixes. core reviwers would be really appreciated :) | 19:21 |
| blogan | marun: np | 19:21 |
| *** jorgem has quit IRC | 19:22 | |
| *** atuvenie has joined #openstack-neutron | 19:22 | |
| *** shwetaap has quit IRC | 19:22 | |
| marun | nuritv: we're closing in on rc in the next couple of days | 19:23 |
| *** ijw_ has quit IRC | 19:23 | |
| *** jorgem has joined #openstack-neutron | 19:23 | |
| marun | nuritv: the fact that the bug in question isn't targeted to rc means it won't be getting attention until after the release is cut and we open for contributions for liberty | 19:23 |
| *** uschreiber_ has quit IRC | 19:23 | |
| nuritv | marun: is that mean no more fixes for Kilo? | 19:24 |
| marun | nuritv: no fixes that haven't been triaged for kilo, no | 19:24 |
| marun | nuritv: if for some reason you think the fix should be in kilo, you should raise it with mestery | 19:25 |
| marun | mestery: https://bugs.launchpad.net/neutron/+bug/1427641 | 19:25 |
| openstack | Launchpad bug 1427641 in neutron "ml2 extension mechanism do not notify agents on update port" [Undecided,In progress] - Assigned to Nurit Vilosny (nuritv) | 19:25 |
| nuritv | marun: thanks! it is just that the code right now is broken, and the fix is very very minor | 19:26 |
| marun | nuritv: minor or no, we have lots of competition for gate time right now | 19:26 |
| nuritv | marun: i understand. i hope maybe mestrey would decide to push it anyway. thanks for explaining the situation to me. | 19:27 |
| marun | nuritv: np, best of luck :) | 19:27 |
| *** leenheer has joined #openstack-neutron | 19:29 | |
| *** yfauser has joined #openstack-neutron | 19:29 | |
| *** Marga_ has joined #openstack-neutron | 19:29 | |
| *** yfauser has left #openstack-neutron | 19:29 | |
| *** vivek-ebay has joined #openstack-neutron | 19:30 | |
| *** leenheer has quit IRC | 19:30 | |
| *** sudipto has quit IRC | 19:31 | |
| *** leenheer has joined #openstack-neutron | 19:31 | |
| openstackgerrit | Brian Haley proposed openstack/neutron: Add ipset element and hashsize tunables https://review.openstack.org/170328 | 19:33 |
| *** jorgem has quit IRC | 19:34 | |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks https://review.openstack.org/168115 | 19:34 |
| *** vivek-ebay has quit IRC | 19:34 | |
| *** suresh12 has joined #openstack-neutron | 19:35 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Synchronize auto-addr updates for subnet create https://review.openstack.org/170968 | 19:35 |
| *** sbfox has quit IRC | 19:35 | |
| *** sbfox has joined #openstack-neutron | 19:38 | |
| *** ajmiller has quit IRC | 19:39 | |
| *** suresh12 has quit IRC | 19:40 | |
| *** johnbelamaric has joined #openstack-neutron | 19:41 | |
| *** nlahouti has quit IRC | 19:43 | |
| dane_leblanc | armax, kevinbenton: I submitted a new patch (working with HenryG on this) for the DB deadlock: https://review.openstack.org/170968. Could use some guidance, I think it's the right area for the fix, but the fix can probably be done better or more efficiently. | 19:44 |
| otherwiseguy | marun: the thing I worry about w/ daemon mode with long-running stuff is that it uses Popen.communicate() so it is storing all of the stdout/err for returning. Still not completely sure about how well it handles interleaving multiple simultaneous execute() calls either, but the fact that it hasn't completely fallen over must mean that it handles it fairly well, anyway. YorikSar should know since he wrote it, though. :) | 19:44 |
| marun | otherwiseguy: so, we pray :) | 19:45 |
| *** dboik has joined #openstack-neutron | 19:47 | |
| *** dboik has quit IRC | 19:47 | |
| *** dboik has joined #openstack-neutron | 19:47 | |
| otherwiseguy | marun: although old-style utils/execute() also uses communicate(), so that never caused a memory issue... | 19:47 |
| * YorikSar digs out of his lair | 19:48 | |
| *** Marga_ has quit IRC | 19:48 | |
| *** alexpilotti has quit IRC | 19:48 | |
| YorikSar | otherwiseguy, marun: Daemon mode shouldn't be used for long-running processes that you might want to manage somehow or communicate with. | 19:49 |
| *** kbringard1 has joined #openstack-neutron | 19:49 | |
| otherwiseguy | YorikSar: Was just noticing that we are using daemon mode for ProcessMonitor which spawns long-running processes. So it's being used with l3agent/dhcpagent... | 19:49 |
| *** moha_hunt has joined #openstack-neutron | 19:50 | |
| YorikSar | otherwiseguy: You're right, daemon will wait for process to complete and buffer all stdout/err. | 19:50 |
| otherwiseguy | YorikSar: but the old pre-daemon mode utils.execute() also uses Popen.communicate() so that itself hasn't changed I guess. | 19:51 |
| YorikSar | otherwiseguy: It shouldn't work... | 19:51 |
| *** johnbelamaric has quit IRC | 19:51 | |
| *** baoli has joined #openstack-neutron | 19:51 | |
| otherwiseguy | but still seems crazy. | 19:51 |
| YorikSar | otherwiseguy: Yes, for long-running processes utils.execute shouldn't have been used... It's for one-off calls, not background process spawning. | 19:51 |
| *** baoli has quit IRC | 19:52 | |
| YorikSar | Which commands are executed this way | 19:52 |
| *** dencaval has quit IRC | 19:52 | |
| YorikSar | ? | 19:52 |
| *** shwetaap has joined #openstack-neutron | 19:52 | |
| *** baoli has joined #openstack-neutron | 19:52 | |
| *** nuritv has quit IRC | 19:52 | |
| YorikSar | otherwiseguy: btw, about "interleaving multiple simultaneous execute() calls", it's just a simple multithreaded server. One thread per RPC request. | 19:53 |
| otherwiseguy | YorikSar: neutron/agent/l3/agent.py has a ProcessMonitor for instance. | 19:54 |
| otherwiseguy | and ProcessMonitor.enable() calls IpWrapper.netns.execute() which calls utils.execute() | 19:54 |
| *** kbringard has quit IRC | 19:55 | |
| otherwiseguy | YorikSar: so it sounds like basically "well, the code's always been a bit silly but hasn't managed to crash yet presumably because the processes being launched don't write to stdout? | 19:56 |
| otherwiseguy | Having an IpWrapper.netns.create_process() might be a good idea. | 19:57 |
| *** ijw has joined #openstack-neutron | 19:57 | |
| YorikSar | otherwiseguy: It looks like noone actually calls enable there... | 19:59 |
| YorikSar | otherwiseguy: And process monitor is used only in agent.linux.ra with only register/unregister methods | 20:01 |
| *** johnbelamaric has joined #openstack-neutron | 20:01 | |
| *** annegentle has quit IRC | 20:01 | |
| YorikSar | otherwiseguy: Oh, right. Process _manager_ uses execute, process _monitor_ doesn't execute anything. | 20:02 |
| otherwiseguy | YorikSar: see spawn_tate_change_monitor() in ha_router.py. | 20:02 |
| marun | otherwiseguy: would you please file a bug to track this issue? | 20:03 |
| marun | otherwiseguy: I have a feeling it'll need to be fixed in l and backported | 20:03 |
| *** alexpilotti has joined #openstack-neutron | 20:04 | |
| YorikSar | otherwiseguy: Oh, ok... Is HA router covered in gate? | 20:04 |
| *** cference has quit IRC | 20:05 | |
| anteaya | mestery: great etherpad colour | 20:05 |
| otherwiseguy | YorikSar: Think so. And neautron/agent/linux/dhcp.py uses ProcessManager and it definitely is tested. | 20:05 |
| marun | YorikSar: Maybe? amuller would know | 20:05 |
| anteaya | dougwig: I can't believe you are doing this to yourself again, what are you thinking? https://review.openstack.org/#/c/154736/7/specs/liberty/neutron-lib.rst | 20:06 |
| mestery | anteaya: lol | 20:06 |
| anteaya | it is | 20:06 |
| mestery | anteaya: We'll make it happen | 20:06 |
| anteaya | I get so frustrated we can't pick our colours with a hex value | 20:06 |
| otherwiseguy | marun: I'm less worried about it after realizing that we've always called Popen.communicate() for these things. | 20:07 |
| anteaya | make what happen, great colours or driving dougwig crazy? | 20:07 |
| mestery | anteaya: both ;) | 20:07 |
| anteaya | I'm all for great colours | 20:07 |
| mestery | anteaya: neutron-lib split | 20:07 |
| YorikSar | otherwiseguy: It runs keepalived-state-change which daemonizes itself | 20:07 |
| otherwiseguy | And the thread per RPC request interleaving not being an issue. | 20:07 |
| anteaya | mestery: yeah that is what I was looking at | 20:07 |
| YorikSar | otherwiseguy: So it is one-off | 20:07 |
| mestery | anteaya: :) | 20:07 |
| YorikSar | otherwiseguy: I mean it finishes fast. | 20:07 |
| *** aepifanov has joined #openstack-neutron | 20:08 | |
| YorikSar | otherwiseguy: And I have no idea how it communicates with calling process... | 20:08 |
| anteaya | dougwig: apparently you have grown some attitude since last round: This is a concrete, and opinionated, | 20:08 |
| YorikSar | otherwiseguy: Oh, it looks like it uses some HTTP... | 20:08 |
| anteaya | dougwig: that may be your saving grace | 20:09 |
| *** jorgem has joined #openstack-neutron | 20:09 | |
| otherwiseguy | YorikSar: and metadata proxy has spawn_monitored_metadata_proxy() which uses a ProcessManager. But basically none of them communicate with the process via the stdin/out, just make sure they are respawned if they exit, etc. | 20:09 |
| YorikSar | otherwiseguy: over UNIX socket, omg | 20:09 |
| YorikSar | otherwiseguy: Do they all just start some daemons? | 20:10 |
| otherwiseguy | YorikSar: Yeah. | 20:10 |
| otherwiseguy | As far as I can tell. | 20:10 |
| YorikSar | otherwiseguy: So how does process manager manage them then? | 20:10 |
| *** SumitNaiksatam has joined #openstack-neutron | 20:11 | |
| otherwiseguy | disable() calls kill() active() checks /proc to see it is still there, reload() sends a HUP. Doesn't look like there is really much actual management going on. | 20:12 |
| YorikSar | otherwiseguy: Oh, they're tied by pid files... | 20:12 |
| *** dboik has quit IRC | 20:12 | |
| YorikSar | and some UNIX sockets. | 20:12 |
| *** atuvenie has quit IRC | 20:12 | |
| YorikSar | looks like lots of overcomplications... | 20:13 |
| openstackgerrit | pritesh proposed openstack/neutron: Moving VLAN Transparency support from core to extension https://review.openstack.org/169569 | 20:13 |
| otherwiseguy | And ProcessMonitor manages ProcessManagers. | 20:13 |
| YorikSar | otherwiseguy: Anyway, it seems we have no problems with utils.execute here :) | 20:14 |
| otherwiseguy | Basically checking in an eventlet thread every so often that something is active() andif not, respawning. | 20:14 |
| YorikSar | (although why, oh why didn't one just use multiprocessing here?..) | 20:14 |
| otherwiseguy | YorikSar: yeah. I just was originally worried because of the communicate() and memory growing forever...didn't realize at first that we'd been using communicate() all along and the daemon mode didn't cause a difference there. :p | 20:15 |
| *** nuritv has joined #openstack-neutron | 20:15 | |
| *** alexpilotti has quit IRC | 20:16 | |
| *** ociuhandu has quit IRC | 20:16 | |
| YorikSar | otherwiseguy: btw, I had plans on adding some long-running processes support to daemon mode, but it was too long ago... | 20:16 |
| otherwiseguy | YorikSar: Thanks for digging yourself out of your lair. :) | 20:17 |
| YorikSar | otherwiseguy: E.g. pass stdin/out/err fds back to caller and then just notify it if process exits. | 20:17 |
| YorikSar | otherwiseguy: Oh, it's always nice to look at some Python code now and then :) | 20:18 |
| *** emagana has quit IRC | 20:19 | |
| YorikSar | otherwiseguy: We're now in deep research phase, so all I see is either docs or bash :) | 20:19 |
| otherwiseguy | YorikSar: Yeah. Time passes and we move on. :) Since most of the benefit in daemon mode is eliminating overhead of starting up, long-running processes were definitely a lower priority for it. :) | 20:19 |
| otherwiseguy | YorikSar: I'm so sorry. ;) | 20:19 |
| *** Marga_ has joined #openstack-neutron | 20:19 | |
| otherwiseguy | That reminds me that I've had some documentation tasks that I've been putting off for a while... | 20:20 |
| *** baoli has quit IRC | 20:21 | |
| *** raildo is now known as raildo|away | 20:23 | |
| *** vivek-ebay has joined #openstack-neutron | 20:24 | |
| *** Marga_ has quit IRC | 20:24 | |
| *** ajmiller has joined #openstack-neutron | 20:24 | |
| *** ijw_ has joined #openstack-neutron | 20:24 | |
| *** emagana has joined #openstack-neutron | 20:24 | |
| *** dboik has joined #openstack-neutron | 20:26 | |
| *** ijw has quit IRC | 20:27 | |
| *** dboik has quit IRC | 20:27 | |
| *** dboik has joined #openstack-neutron | 20:27 | |
| *** krtaylor has quit IRC | 20:28 | |
| *** dkehn has quit IRC | 20:29 | |
| *** dkehn has joined #openstack-neutron | 20:30 | |
| *** ijw_ has quit IRC | 20:30 | |
| *** ijw has joined #openstack-neutron | 20:31 | |
| *** krtaylor has joined #openstack-neutron | 20:33 | |
| *** suresh12 has joined #openstack-neutron | 20:36 | |
| *** jorgem has quit IRC | 20:37 | |
| *** jorgem has joined #openstack-neutron | 20:39 | |
| *** yamahata has joined #openstack-neutron | 20:40 | |
| *** nlahouti has joined #openstack-neutron | 20:40 | |
| openstackgerrit | Merged openstack/neutron: Skip example retargetable functional test https://review.openstack.org/170915 | 20:41 |
| *** suresh12 has quit IRC | 20:41 | |
| *** smcquaid has joined #openstack-neutron | 20:44 | |
| *** kothari has quit IRC | 20:44 | |
| *** smcquaid has quit IRC | 20:44 | |
| *** nuritv has quit IRC | 20:45 | |
| *** shwetaap has quit IRC | 20:45 | |
| *** stevemcquaid has joined #openstack-neutron | 20:46 | |
| *** dave-mccowan has joined #openstack-neutron | 20:46 | |
| *** stevemcquaid has left #openstack-neutron | 20:47 | |
| *** leenheer has left #openstack-neutron | 20:49 | |
| *** banix has quit IRC | 20:55 | |
| *** salv-orlando has joined #openstack-neutron | 20:56 | |
| *** Marga_ has joined #openstack-neutron | 20:58 | |
| *** moha_hunt has quit IRC | 20:59 | |
| *** jorgem has quit IRC | 21:01 | |
| *** shwetaap has joined #openstack-neutron | 21:02 | |
| *** ZZelle_ has quit IRC | 21:02 | |
| *** hichihara has joined #openstack-neutron | 21:03 | |
| *** shwetaap has quit IRC | 21:03 | |
| amuller | YorikSar: otherwiseguy: Back from AFK land | 21:05 |
| amuller | YorikSar: otherwiseguy: Are there any action items from that discussion? | 21:05 |
| *** luqas has joined #openstack-neutron | 21:06 | |
| *** busterswt has quit IRC | 21:08 | |
| *** ZZelle_ has joined #openstack-neutron | 21:09 | |
| *** suresh12 has joined #openstack-neutron | 21:10 | |
| *** marun has quit IRC | 21:10 | |
| *** emagana has quit IRC | 21:10 | |
| *** luqas has quit IRC | 21:11 | |
| *** dane_leblanc has quit IRC | 21:11 | |
| openstackgerrit | Merged openstack/neutron: Allow metadata proxy running with nobody user/group https://review.openstack.org/165115 | 21:12 |
| *** marun has joined #openstack-neutron | 21:12 | |
| *** emagana has joined #openstack-neutron | 21:13 | |
| *** xgerman has quit IRC | 21:13 | |
| *** ajmiller_ has joined #openstack-neutron | 21:14 | |
| *** Bsony has joined #openstack-neutron | 21:14 | |
| *** julim has quit IRC | 21:14 | |
| *** ajmiller has quit IRC | 21:17 | |
| *** dane_leblanc has joined #openstack-neutron | 21:18 | |
| *** ajmiller__ has joined #openstack-neutron | 21:19 | |
| ZZelle_ | mestery, ^^, Yippee! | 21:19 |
| mestery | ZZelle_: Yay! :) | 21:20 |
| blogan | marun: https://review.openstack.org/#/c/170983/ | 21:20 |
| marun | blogan: lgtm | 21:21 |
| marun | blogan: what do you think of centralizing the neutron imports as a follow-on? | 21:21 |
| marun | blogan: i.e. a single module responsible for importing anything test-related from neutron | 21:22 |
| marun | blogan: so that management and vendoring/replacement have a single point of coordination | 21:22 |
| blogan | almost like a neutron tests lib? | 21:22 |
| blogan | except still just neutron | 21:22 |
| marun | blogan: well, I don't think that is warranted tbh | 21:22 |
| *** ajmiller_ has quit IRC | 21:22 | |
| marun | blogan: the in-process requirement of runtime code doesn't apply to tests | 21:23 |
| marun | blogan: so vendoring seems a more reasonable option to me | 21:23 |
| blogan | marun: so how would a typical import of a base test class in neutron look like from a neutron_lbaas module? | 21:23 |
| blogan | just so i understand correctly | 21:24 |
| marun | blogan: i think we should manage test infrastructure oslo incubator-style in the near term | 21:24 |
| marun | blogan: *aas can copy code into the tree as required | 21:24 |
| blogan | yeah that would solve the issues i have with the way we're doing it now | 21:24 |
| marun | blogan: the overhead of stabilizing non-production code doesn't seem justifiable to me, at least not yet. | 21:24 |
| blogan | marun: valid point, copy pasta is a much more efficient use of time, especially if its just a script that does it for maintainers | 21:25 |
| *** ronis has quit IRC | 21:25 | |
| marun | blogan: it would still require us to isolate the stuff to be copied | 21:26 |
| blogan | marun: which goes back to the centralizing right? | 21:26 |
| marun | blogan: the anti-pattern I've seen in doing this restructure is tests importing other test modules | 21:26 |
| marun | blogan: test modules should be test-only, and shouldn't depend on anything else | 21:27 |
| prometheanfire | ijw: you made this right? https://blueprints.launchpad.net/neutron/+spec/mtu-selection-and-advertisement | 21:27 |
| marun | blogan: (sorry, anything test_*.py) | 21:27 |
| marun | blogan: so we'll need to factor out common stuff - constants, fixture, etc | 21:27 |
| prometheanfire | ijw: had a question on if https://bugs.launchpad.net/neutron/+bug/1348788 should be included with it | 21:27 |
| openstack | Launchpad bug 1348788 in neutron "network_device_mtu is not applied to VMs, only to agents" [Low,Confirmed] | 21:27 |
| prometheanfire | ijw: rushed into kilo more or less | 21:27 |
| blogan | marun: well can't they import a base test module? | 21:28 |
| marun | blogan: and put it in one place for reuse both in-tree and out | 21:28 |
| marun | blogan: they should not import anything with tests in it | 21:28 |
| marun | blogan: base.py shouldn't have any tests | 21:28 |
| marun | test_*.py should denote modules that contain tests | 21:28 |
| blogan | marun: oh okay thats what you meant, then yes i totally agree | 21:28 |
| *** Bsony has quit IRC | 21:28 | |
| blogan | marun: though i've been guilty of such shortcuts im sure :( | 21:28 |
| marun | blogan: and we can round up the mixins and bury them | 21:29 |
| marun | :) | 21:29 |
| *** rpothier has quit IRC | 21:29 | |
| *** tiswanso_ has quit IRC | 21:29 | |
| marun | blogan: hey, it worked in the small. | 21:29 |
| marun | blogan: it just hasn't proven to scale. | 21:29 |
| blogan | marun: oh the mixin seductresses | 21:29 |
| *** xgerman has joined #openstack-neutron | 21:29 | |
| blogan | marun: yeah used in moderation, they can be useful | 21:30 |
| marun | blogan: I think something like testscenarios is preferable, so that there is a clear indication of what tests will be run at the point of their definition | 21:30 |
| marun | blogan: at least within a given project | 21:30 |
| blogan | marun: what do you mean by testscenarios? | 21:31 |
| marun | blogan: https://github.com/openstack/neutron/blob/master/neutron/tests/functional/scheduler/test_dhcp_agent_scheduler.py | 21:32 |
| marun | blogan: it's generative testing, basically | 21:32 |
| blogan | marun: ah haven't seen this yet | 21:32 |
| *** vhosakot has quit IRC | 21:32 | |
| marun | blogan: a 'scenarios' attribute can define sets of parameters to execute the class's tests with | 21:32 |
| blogan | marun: is this like data driven tests? | 21:33 |
| marun | blogan: yes | 21:33 |
| marun | blogan: and it doesn't have to be static, either | 21:33 |
| blogan | marun: ah okay, great! | 21:33 |
| *** dboik has quit IRC | 21:33 | |
| marun | blogan: a subclass could override by setting scenarios to something else, for example | 21:33 |
| blogan | marun: yep, i like it | 21:33 |
| marun | blogan: also, the scenarios attribute could be generated by a function | 21:33 |
| *** sripriya has joined #openstack-neutron | 21:33 | |
| *** dboik has joined #openstack-neutron | 21:34 | |
| *** johnbelamaric has quit IRC | 21:34 | |
| marun | blogan: it can be abused, same as mixins, though. We'd have to be careful not to allow arbitrary use - a given test class should have scenarios defined once and only once in the project. No subclassing | 21:34 |
| marun | blogan: subclassing could be allowed in other repos, though. | 21:34 |
| *** pritesh has joined #openstack-neutron | 21:35 | |
| *** annegentle has joined #openstack-neutron | 21:35 | |
| blogan | marun: yeah, id say lbaas repo should implement the same and revisit the need for subclassing if enough benefits warrant it | 21:36 |
| dougwig | back. sweet, blogan is on it. | 21:36 |
| blogan | dougwig: im on it like a parking spot i shouldn't park in | 21:37 |
| blogan | how the hell did that change get a merge conflict | 21:38 |
| *** crose has quit IRC | 21:40 | |
| *** jckasper has quit IRC | 21:40 | |
| *** pc_m has quit IRC | 21:43 | |
| *** baoli has joined #openstack-neutron | 21:43 | |
| *** melwitt has joined #openstack-neutron | 21:43 | |
| *** annegentle has quit IRC | 21:46 | |
| *** pritesh has quit IRC | 21:46 | |
| *** ivar-lazzaro has quit IRC | 21:47 | |
| *** ivar-lazzaro has joined #openstack-neutron | 21:48 | |
| *** dboik has quit IRC | 21:49 | |
| *** dboik has joined #openstack-neutron | 21:49 | |
| *** dims has quit IRC | 21:52 | |
| *** dboik has quit IRC | 21:53 | |
| *** dboik has joined #openstack-neutron | 21:54 | |
| *** dims has joined #openstack-neutron | 21:54 | |
| *** dims has quit IRC | 21:54 | |
| *** dims has joined #openstack-neutron | 21:55 | |
| *** baoli has quit IRC | 21:55 | |
| *** dsneddon has quit IRC | 22:03 | |
| *** dsneddon has joined #openstack-neutron | 22:03 | |
| *** kbringard1 has quit IRC | 22:05 | |
| *** jblunt has joined #openstack-neutron | 22:08 | |
| *** achanda has quit IRC | 22:08 | |
| *** aepifanov has quit IRC | 22:08 | |
| *** sridhar_ram1 has joined #openstack-neutron | 22:08 | |
| *** sridhar_ram has quit IRC | 22:09 | |
| *** achanda has joined #openstack-neutron | 22:14 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Mechanism to allow using bash in shell scripts https://review.openstack.org/170992 | 22:14 |
| dougwig | marun: i prefer your removal change, but there's the alternate, just in case ^^ | 22:14 |
| *** jblunt has quit IRC | 22:17 | |
| HenryG | marun: How do I run an individual testscenario unit test case from the command line, not using tox? I am trying unit2 and nose2 but not having any luck. | 22:20 |
| *** dboik has quit IRC | 22:21 | |
| marun | dougwig: I'd settle for that, maybe add a Related-Bug link? | 22:21 |
| marun | HenryG: that's a good question | 22:21 |
| marun | HenryG: Given that filtering happens post-discovery, and scenarios are generated by discovery, I'd try running the class and noting the form of the generated test name | 22:21 |
| marun | HenryG: then it should be possible to filter | 22:22 |
| marun | HenryG: looks like test_name(scenario name) should work | 22:22 |
| marun | HenryG: e.g. neutron.tests.functional.agent.test_ovs_lib.OVSBridgeTestCase.test_add_tunnel_port(vsctl) | 22:22 |
| HenryG | marun: yeah, I tried that. No luck. | 22:25 |
| marun | HenryG: you can hack around by limiting the scenario list to the test you want to run, but there is probably a way to get it working. | 22:25 |
| marun | lifeless: surely this is possible? ^ | 22:26 |
| *** ociuhandu has joined #openstack-neutron | 22:29 | |
| mtreinish | HenryG: using testr a regex filter should work fine, because the scenario is added to the test_id | 22:30 |
| mtreinish | at least that was my experience the last time I used it | 22:30 |
| *** _cjones_ has joined #openstack-neutron | 22:30 | |
| mtreinish | HenryG: I have no idea how nose2 does filtering and for unittest.run I think you would have to pass it the full path to run, I'm not sure how that interacts with load_tests | 22:33 |
| openstackgerrit | Merged openstack/neutron: Fix docstring for l3_dvr_db.dvr_vmarp_table_update https://review.openstack.org/170158 | 22:33 |
| *** rushil has quit IRC | 22:33 | |
| *** Bsony has joined #openstack-neutron | 22:34 | |
| mtreinish | HenryG: you might want to try the wrapper I've been working on to replace pretty_tox.sh too: https://pypi.python.org/pypi/os-testr | 22:35 |
| lifeless | marun: HenryG: python -m testtools discover . --load-list <(echo testid) | 22:35 |
| lifeless | marun: HenryG: or testr run testid | 22:36 |
| lifeless | marun: HenryG: (which will take care of that plumbing for you) | 22:36 |
| mtreinish | lifeless: oh, so you have to use discover to ensure load_tests is run. That makes sense | 22:36 |
| lifeless | marun: HenryG: there is a bug open requesting a nicer UI for command line test filtering | 22:36 |
| lifeless | mtreinish: yup | 22:37 |
| *** __afazekas has quit IRC | 22:39 | |
| *** Bsony has quit IRC | 22:39 | |
| ijw | prometheanfire: ping | 22:41 |
| *** banix has joined #openstack-neutron | 22:42 | |
| prometheanfire | ijw: pong | 22:42 |
| *** ijw has quit IRC | 22:42 | |
| *** ijw has joined #openstack-neutron | 22:43 | |
| *** annegentle has joined #openstack-neutron | 22:44 | |
| *** nlahouti has quit IRC | 22:44 | |
| *** sbfox has quit IRC | 22:52 | |
| *** nlahouti has joined #openstack-neutron | 22:52 | |
| *** markvoelker has quit IRC | 22:52 | |
| *** claudiub has quit IRC | 22:55 | |
| ZZelle_ | marun, hi | 22:58 |
| *** devlaps has joined #openstack-neutron | 22:59 | |
| *** igordcard_ has joined #openstack-neutron | 22:59 | |
| *** annegentle has quit IRC | 22:59 | |
| marun | ZZelle_: hi | 23:00 |
| *** annegent_ has joined #openstack-neutron | 23:00 | |
| *** ivar-lazzaro has quit IRC | 23:00 | |
| ZZelle_ | marun, if you have some time, could you have a look to the discussion at the end of https://review.openstack.org/#/c/143412/22/neutron/tests/common/helpers.py ? | 23:01 |
| ZZelle_ | marun, as we fail to find an elegant solution to handle ip pool/subnet/cidr | 23:02 |
| *** sridhar_ram1 has quit IRC | 23:03 | |
| marun | ZZelle_: why are we defining the ips instead of letting neutron define them for us? | 23:04 |
| marun | ZZelle_: or this isn't for neutron-managed ports? | 23:04 |
| *** sridhar_ram1 has joined #openstack-neutron | 23:05 | |
| ZZelle_ | marun, it's not neutron-managed ports | 23:05 |
| marun | ZZelle_: are these ips being configured in a namespace? | 23:06 |
| ZZelle_ | marun, yes | 23:07 |
| *** jamielennox|away is now known as jamielennox | 23:07 | |
| marun | ZZelle_: maybe we need a subnet fixture that can manage allocation? | 23:07 |
| marun | ZZelle_: I'm not sure the cidr range should matter if it's internal to a namespace | 23:07 |
| * marun is good at making things unnecessarily complicated | 23:08 | |
| ZZelle_ | marun, a really old PS proposed to manage allocation with a set of allocated ips ... it seems overkill | 23:09 |
| marun | ZZelle_: I think a bit of abstraction wouldn't be bad | 23:09 |
| marun | ZZelle_: tests shouldn't care about specific ips, they just need to be able to configure resources in the same subnet or different subnets, and an ip on a given subnet should only be used once | 23:10 |
| *** chlong has joined #openstack-neutron | 23:10 | |
| marun | ZZelle_: so s = MySubnet(); cidr1 = s.allocate_ip() etc | 23:10 |
| *** baoli has joined #openstack-neutron | 23:11 | |
| marun | ZZelle_: the same pattern might make sense for subnet management | 23:11 |
| * marun digresses | 23:11 | |
| *** annegent_ has quit IRC | 23:13 | |
| ZZelle_ | marun, we could allow mysubnet1 = MySubnet(no_overloapping_with=mysubnet0) | 23:13 |
| *** _cjones_ has quit IRC | 23:13 | |
| marun | ZZelle_: that could work | 23:14 |
| * ZZelle_ wondering if subnet is the right term as subnet embeds also routes/dns | 23:14 | |
| marun | ZZelle_: IpPool? | 23:15 |
| marun | ZZelle_: I dunno, we're overloading left right and center | 23:15 |
| *** jaypipes has quit IRC | 23:16 | |
| marun | ZZelle_: maybe IpCidr.allocate_ip()? | 23:17 |
| marun | ZZelle_: My main concern is that it's easy for test writers | 23:17 |
| *** signed8bit is now known as signed8bit_ZZZzz | 23:17 | |
| ZZelle_ | marun, i should perhaps look a "new" IPAM abstract class names ... it should be easy if it just allocates ip | 23:19 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add simple ARP spoofing protection https://review.openstack.org/171003 | 23:21 |
| *** jobewan has quit IRC | 23:21 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize unit test tree https://review.openstack.org/158811 | 23:23 |
| openstackgerrit | Maru Newby proposed openstack/neutron: Merge open source plugin test code modules https://review.openstack.org/171004 | 23:23 |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize plugin test modules https://review.openstack.org/171005 | 23:23 |
| *** _cjones_ has joined #openstack-neutron | 23:24 | |
| *** yuanying has joined #openstack-neutron | 23:25 | |
| *** seizadi has quit IRC | 23:26 | |
| *** markvoelker has joined #openstack-neutron | 23:27 | |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize unit test tree https://review.openstack.org/158811 | 23:28 |
| kevinbenton | dane_leblanc, HenryG: hey guys, still around? | 23:28 |
| openstackgerrit | Maru Newby proposed openstack/neutron: Merge open source plugin test code modules https://review.openstack.org/171004 | 23:28 |
| openstackgerrit | Maru Newby proposed openstack/neutron: Reorganize plugin test modules https://review.openstack.org/171005 | 23:28 |
| dane_leblanc | kevinbenton: Yes | 23:29 |
| *** banix has quit IRC | 23:29 | |
| HenryG | kevinbenton: yes | 23:29 |
| kevinbenton | dane_leblanc: why this other approach using the lock? | 23:29 |
| kevinbenton | dane_leblanc: was passing the context not fixing it? | 23:29 |
| dane_leblanc | kevinbenton: Passing the context seemed to solve the issue, but there's no good explanation, as far as I know, why this works. | 23:30 |
| dane_leblanc | kevinbenton: There were some review comments saying that this fix isn't addressing the real root cause, since it doesn't line up with the deadlock timeouts. | 23:31 |
| *** ZZelle_ has quit IRC | 23:35 | |
| *** annegent_ has joined #openstack-neutron | 23:36 | |
| *** carl_baldwin has quit IRC | 23:37 | |
| kevinbenton | dane_leblanc: your comment indicated that it was correct though, right? the dhcp request yielded while holding a lock | 23:37 |
| *** banix has joined #openstack-neutron | 23:37 | |
| kevinbenton | dane_leblanc: my concern was just around the other stuff for retrieving device details | 23:37 |
| *** Aish has joined #openstack-neutron | 23:38 | |
| *** vishwanathj has quit IRC | 23:39 | |
| dane_leblanc | kevinbenton: I can revert the other changes outside of the immediate need for the fix... but which approach do we want to go with? With the latest approach, it's clearer that it's addressing the root cause. With the earlier approach, there's always going to be some doubt that maybe we're just getting lucky by changing the timing in a subtle way. | 23:40 |
| kevinbenton | dane_leblanc: give me a few minutes here, checking something | 23:40 |
| HenryG | lifeless: sorry, I was pulled away for a bit | 23:45 |
| HenryG | lifeless: I ran the following ... | 23:45 |
| HenryG | testr run neutron.tests.unit.test_l3_schedulers.TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents | 23:46 |
| lifeless | HenryG: yes, and ? | 23:46 |
| HenryG | lifeless: It gives me "Ran 9 (+9) tests in 0.745s", which is good, there are 9 scenarios | 23:47 |
| *** Marga_ has quit IRC | 23:47 | |
| HenryG | lifeless: Now I am trying to run just one. For example: | 23:47 |
| HenryG | "neutron.tests.unit.test_l3_schedulers.TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents (legacy)" | 23:47 |
| openstackgerrit | pritesh proposed openstack/neutron: Moving VLAN Transparency support from core to extension https://review.openstack.org/169569 | 23:47 |
| lifeless | testr run 'TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents(legacy)' | 23:48 |
| HenryG | lifeless: tried that, nothing runs | 23:48 |
| lifeless | if that runs nothing, its the () | 23:48 |
| lifeless | testr run 'TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents.legacy' | 23:48 |
| *** annegent_ has quit IRC | 23:48 | |
| HenryG | lifeless: ha! | 23:48 |
| lifeless | because its a regex | 23:48 |
| HenryG | lifeless: that runs 4 tests. ?? | 23:48 |
| * HenryG begins to understand (hopefully) | 23:49 | |
| lifeless | testr last --subunit | subunit-ls | 23:49 |
| lifeless | will show you everything that was reported | 23:50 |
| HenryG | lifeless: yup, 4 tests starting with legacy | 23:50 |
| lifeless | oh, you have multiple scenarios with that prefixing the name? | 23:50 |
| lifeless | testr run 'TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents\(legacy\)' | 23:50 |
| lifeless | or possibly, depending on your shell and hell - | 23:51 |
| lifeless | testr run 'TestGetL3AgentsWithAgentModeFilter.test_get_l3_agents\\(legacy\\)' | 23:51 |
| *** achanda has quit IRC | 23:51 | |
| HenryG | lifeless: awesome, thanks! | 23:52 |
| HenryG | lifeless: now to figure out how to run a single scenario test in my debugger ... | 23:52 |
| *** suresh12 has quit IRC | 23:53 | |
| *** suresh12 has joined #openstack-neutron | 23:53 | |
| *** markvoelker has quit IRC | 23:53 | |
| *** baoli has quit IRC | 23:53 | |
| *** emagana has quit IRC | 23:54 | |
| *** emagana has joined #openstack-neutron | 23:55 | |
| *** SumitNaiksatam has quit IRC | 23:56 | |
| *** suresh12 has quit IRC | 23:57 | |
| *** puck has quit IRC | 23:58 | |
| *** dannywilson has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!