| *** david-lyle has quit IRC | 00:03 | |
| -openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:05 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use a conntrack zone per port in OVS https://review.openstack.org/207464 | 00:06 |
|---|---|---|
| *** bharath has quit IRC | 00:07 | |
| *** pritesh has quit IRC | 00:08 | |
| *** sthillma_ has joined #openstack-neutron | 00:09 | |
| *** madhu_ak has quit IRC | 00:09 | |
| *** sthillma has quit IRC | 00:11 | |
| *** sthillma_ is now known as sthillma | 00:11 | |
| *** minwang2 has quit IRC | 00:14 | |
| *** hichihara has joined #openstack-neutron | 00:14 | |
| *** sthillma_ has joined #openstack-neutron | 00:16 | |
| *** sthillma has quit IRC | 00:17 | |
| *** sthillma_ is now known as sthillma | 00:17 | |
| *** azbiswas has quit IRC | 00:21 | |
| *** tiswanso has joined #openstack-neutron | 00:22 | |
| *** shadower has quit IRC | 00:23 | |
| *** shadower has joined #openstack-neutron | 00:23 | |
| *** oomichi has joined #openstack-neutron | 00:23 | |
| *** zhhuabj has joined #openstack-neutron | 00:23 | |
| *** piyanai has quit IRC | 00:24 | |
| *** enikanorov2 has quit IRC | 00:27 | |
| *** dane_leblanc has quit IRC | 00:27 | |
| *** rawl has joined #openstack-neutron | 00:29 | |
| *** tiswanso has quit IRC | 00:33 | |
| *** claudiub has quit IRC | 00:34 | |
| *** achanda_ has joined #openstack-neutron | 00:35 | |
| *** ijw_ has joined #openstack-neutron | 00:35 | |
| *** thorst has joined #openstack-neutron | 00:36 | |
| *** achanda has quit IRC | 00:38 | |
| *** gildub has quit IRC | 00:38 | |
| *** ijw has quit IRC | 00:39 | |
| *** achanda_ has quit IRC | 00:39 | |
| *** thumpba has quit IRC | 00:42 | |
| *** abhiraut has quit IRC | 00:44 | |
| *** abhiraut has joined #openstack-neutron | 00:46 | |
| *** tiswanso has joined #openstack-neutron | 00:48 | |
| *** yalie has joined #openstack-neutron | 00:48 | |
| Sam-I-Am | kevinbenton: ping | 00:49 |
| kevinbenton | Sam-I-Am: pong | 00:49 |
| *** Aish has quit IRC | 00:49 | |
| Sam-I-Am | looking at a mailing list post about vxlan_udp_port in the linux bridge agent... is that even honored? | 00:49 |
| Sam-I-Am | doesnt seem to be | 00:49 |
| kevinbenton | no, i had him file a bug | 00:49 |
| kevinbenton | something for the linux bridge catch-up | 00:50 |
| Sam-I-Am | ok, thanks for confirming my suspicion looking through the code | 00:50 |
| kevinbenton | no prob | 00:50 |
| *** abhiraut has quit IRC | 00:51 | |
| Sam-I-Am | ha, now i get the email | 00:51 |
| Sam-I-Am | stupid internet | 00:51 |
| *** tiswanso has quit IRC | 00:51 | |
| *** bharath has joined #openstack-neutron | 00:51 | |
| *** tiswanso has joined #openstack-neutron | 00:52 | |
| *** david-lyle has joined #openstack-neutron | 00:54 | |
| *** ijw_ has quit IRC | 00:54 | |
| *** ijw_ has joined #openstack-neutron | 00:55 | |
| *** bharath has quit IRC | 00:56 | |
| *** thumpba has joined #openstack-neutron | 00:56 | |
| *** gildub has joined #openstack-neutron | 00:58 | |
| *** thumpba has quit IRC | 00:59 | |
| *** _cjones_ has quit IRC | 01:00 | |
| *** tflynn_ has joined #openstack-neutron | 01:01 | |
| *** sthillma has quit IRC | 01:02 | |
| *** tflynn has quit IRC | 01:05 | |
| alejandrito | HI, one question, why is that when i create a "neutron meter-label-rule-create" the CIDR parameter matches the TUNNELED ip, and not the floating ip one ? | 01:05 |
| *** tflynn_ has quit IRC | 01:05 | |
| *** gangil has quit IRC | 01:07 | |
| *** thumpba has joined #openstack-neutron | 01:08 | |
| *** jwy has quit IRC | 01:12 | |
| *** baoli has joined #openstack-neutron | 01:12 | |
| *** sridhar_ram has quit IRC | 01:12 | |
| openstackgerrit | yalei wang proposed openstack/neutron: Replace internal calls of create_{network, subnet, port} https://review.openstack.org/129773 | 01:13 |
| *** thumpba_ has joined #openstack-neutron | 01:13 | |
| *** thumpba has quit IRC | 01:14 | |
| *** thumpba_ has quit IRC | 01:15 | |
| *** thumpba has joined #openstack-neutron | 01:15 | |
| *** thorst has quit IRC | 01:16 | |
| *** zhenguo has joined #openstack-neutron | 01:16 | |
| *** Marga__ has joined #openstack-neutron | 01:19 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Improve callback registry devref documentation and usability https://review.openstack.org/211798 | 01:19 |
| *** Marga_ has quit IRC | 01:20 | |
| *** shivrao has quit IRC | 01:20 | |
| *** agireud has joined #openstack-neutron | 01:24 | |
| *** britthouser has quit IRC | 01:30 | |
| *** lyanchih has joined #openstack-neutron | 01:31 | |
| *** erkules has joined #openstack-neutron | 01:32 | |
| *** vivek-ebay has joined #openstack-neutron | 01:34 | |
| *** erkules_ has quit IRC | 01:35 | |
| *** breitz1 has joined #openstack-neutron | 01:38 | |
| *** breitz has quit IRC | 01:38 | |
| *** britthouser has joined #openstack-neutron | 01:39 | |
| *** zhhuabj has quit IRC | 01:41 | |
| *** ijw_ has quit IRC | 01:48 | |
| *** baoli has quit IRC | 01:49 | |
| *** ijw has joined #openstack-neutron | 01:49 | |
| *** cdl has joined #openstack-neutron | 01:49 | |
| *** piyanai has joined #openstack-neutron | 01:51 | |
| *** jwy has joined #openstack-neutron | 01:52 | |
| *** erkules has quit IRC | 01:52 | |
| *** erkules has joined #openstack-neutron | 01:54 | |
| *** alejandrito has quit IRC | 01:55 | |
| *** Aish has joined #openstack-neutron | 01:59 | |
| *** rushil has joined #openstack-neutron | 02:03 | |
| *** thumpba has quit IRC | 02:04 | |
| *** patrickeast has quit IRC | 02:04 | |
| *** baoli has joined #openstack-neutron | 02:04 | |
| *** thumpba has joined #openstack-neutron | 02:05 | |
| *** thumpba has quit IRC | 02:05 | |
| *** ajmiller has joined #openstack-neutron | 02:06 | |
| *** yamamoto has joined #openstack-neutron | 02:09 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow implementation https://review.openstack.org/153946 | 02:10 |
| *** iwamoto has joined #openstack-neutron | 02:12 | |
| *** y_sawai has joined #openstack-neutron | 02:14 | |
| *** y_sawai has quit IRC | 02:14 | |
| *** yamahata has quit IRC | 02:15 | |
| *** rushil has quit IRC | 02:19 | |
| *** arif-ali has quit IRC | 02:21 | |
| *** tflynn has joined #openstack-neutron | 02:22 | |
| *** yamamoto has quit IRC | 02:23 | |
| *** tflynn_ has joined #openstack-neutron | 02:24 | |
| *** arif-ali has joined #openstack-neutron | 02:26 | |
| *** tflynn has quit IRC | 02:27 | |
| *** Marga__ has quit IRC | 02:30 | |
| *** Marga_ has joined #openstack-neutron | 02:30 | |
| *** vivek-ebay has quit IRC | 02:33 | |
| *** david-lyle has quit IRC | 02:33 | |
| *** agireud has quit IRC | 02:42 | |
| *** shwetaap has joined #openstack-neutron | 02:44 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 02:44 |
| *** bharath has joined #openstack-neutron | 02:44 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-vpnaas: Remove default service provider from conf file https://review.openstack.org/211758 | 02:45 |
| *** wendar has quit IRC | 02:46 | |
| *** zacksh has quit IRC | 02:46 | |
| *** wendar has joined #openstack-neutron | 02:46 | |
| *** zacksh has joined #openstack-neutron | 02:47 | |
| *** bharath has quit IRC | 02:49 | |
| *** britthouser has quit IRC | 02:52 | |
| *** hakimo has joined #openstack-neutron | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** cdl has left #openstack-neutron | 02:54 | |
| *** patrickeast has joined #openstack-neutron | 02:55 | |
| *** jwy has quit IRC | 02:56 | |
| *** saksham has quit IRC | 02:56 | |
| openstackgerrit | Rui Zang proposed openstack/neutron: Add UUID check of tenant ID for quota show and update. Currently tenant ID validation (to Keyston or other) is missing in Neutron and some other projects. For quota-update, random string is being accepted and written to the tenant-id column in DB. This pa https://review.openstack.org/211441 | 02:57 |
| *** agireud has joined #openstack-neutron | 03:00 | |
| *** yamamoto has joined #openstack-neutron | 03:03 | |
| *** banix has quit IRC | 03:08 | |
| *** rm_work|away is now known as rm_work | 03:10 | |
| *** agireud has quit IRC | 03:10 | |
| *** patrickeast has quit IRC | 03:15 | |
| *** tflynn_ has quit IRC | 03:15 | |
| *** tflynn has joined #openstack-neutron | 03:19 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-vpnaas: Remove default service provider from conf file https://review.openstack.org/211758 | 03:19 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 03:21 |
| *** fawadkhaliq has joined #openstack-neutron | 03:22 | |
| *** baoli has quit IRC | 03:23 | |
| *** tflynn has quit IRC | 03:24 | |
| *** humble__ has joined #openstack-neutron | 03:24 | |
| *** dan has quit IRC | 03:26 | |
| *** adreznec has quit IRC | 03:26 | |
| *** dan has joined #openstack-neutron | 03:26 | |
| *** achanda has joined #openstack-neutron | 03:27 | |
| *** adreznec has joined #openstack-neutron | 03:27 | |
| *** shivrao has joined #openstack-neutron | 03:28 | |
| *** baoli has joined #openstack-neutron | 03:28 | |
| *** moha_hunt has quit IRC | 03:28 | |
| *** moha_hunt has joined #openstack-neutron | 03:30 | |
| *** david-lyle has joined #openstack-neutron | 03:30 | |
| *** nlahouti has joined #openstack-neutron | 03:34 | |
| *** sdake_ has joined #openstack-neutron | 03:35 | |
| *** rawl has quit IRC | 03:36 | |
| *** sdake has quit IRC | 03:39 | |
| *** fedexo has joined #openstack-neutron | 03:40 | |
| *** nlahouti has quit IRC | 03:41 | |
| *** nlahouti has joined #openstack-neutron | 03:41 | |
| *** azbiswas has joined #openstack-neutron | 03:41 | |
| *** vivek-ebay has joined #openstack-neutron | 03:41 | |
| *** piyanai has quit IRC | 03:44 | |
| *** tsg has joined #openstack-neutron | 03:44 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Improve callback registry devref documentation and usability https://review.openstack.org/211798 | 03:44 |
| *** sdake_ is now known as sdake | 03:46 | |
| *** bharath has joined #openstack-neutron | 03:47 | |
| *** bharath_ has joined #openstack-neutron | 03:51 | |
| *** tiswanso has quit IRC | 03:54 | |
| *** baoli has quit IRC | 03:55 | |
| *** bharath_ has quit IRC | 03:56 | |
| *** fedexo has quit IRC | 04:00 | |
| *** ijw has quit IRC | 04:00 | |
| *** vivek-ebay has quit IRC | 04:04 | |
| *** pai15 has joined #openstack-neutron | 04:06 | |
| *** vivek-ebay has joined #openstack-neutron | 04:06 | |
| *** vikram has joined #openstack-neutron | 04:07 | |
| *** dsanders1 has quit IRC | 04:16 | |
| *** Sukhdev_ has joined #openstack-neutron | 04:19 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 04:23 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 04:23 |
| *** morgan_404 is now known as morgan_410 | 04:26 | |
| *** morgan_410 is now known as morgan_404 | 04:26 | |
| *** bharath has quit IRC | 04:28 | |
| *** pai15 has quit IRC | 04:30 | |
| *** sthillma has joined #openstack-neutron | 04:36 | |
| *** smallbig has joined #openstack-neutron | 04:37 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Add Kuryr to sub_projects.rst https://review.openstack.org/209361 | 04:39 |
| *** sthillma_ has joined #openstack-neutron | 04:39 | |
| *** minwang2 has joined #openstack-neutron | 04:41 | |
| *** sthillma has quit IRC | 04:42 | |
| *** sthillma_ is now known as sthillma | 04:42 | |
| *** links has joined #openstack-neutron | 04:42 | |
| *** Aish has quit IRC | 04:42 | |
| *** bharath has joined #openstack-neutron | 04:43 | |
| *** abregman has joined #openstack-neutron | 04:45 | |
| *** shwetaap has quit IRC | 04:50 | |
| *** moha_hunt has quit IRC | 04:51 | |
| *** ekuris has joined #openstack-neutron | 04:52 | |
| *** enikanorov2 has joined #openstack-neutron | 05:04 | |
| *** lyanchih has quit IRC | 05:07 | |
| *** ircuser-1 has joined #openstack-neutron | 05:07 | |
| *** vikram_ has joined #openstack-neutron | 05:09 | |
| *** itzikb_ has joined #openstack-neutron | 05:09 | |
| *** vikram has quit IRC | 05:09 | |
| *** bharath has quit IRC | 05:10 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 05:10 |
| *** sthillma has quit IRC | 05:12 | |
| *** lajos-katona has joined #openstack-neutron | 05:16 | |
| openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports https://review.openstack.org/210301 | 05:21 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Switch the default to "native" of_interface https://review.openstack.org/206329 | 05:25 |
| *** tsg has quit IRC | 05:27 | |
| *** jianyong has quit IRC | 05:27 | |
| openstackgerrit | Rui Zang proposed openstack/neutron: Add UUID check of tenant ID for quota show and update. Currently tenant ID validation (to Keyston or other) is missing in Neutron and some other projects. For quota-update, random string is being accepted and written to the tenant-id column in DB. This pa https://review.openstack.org/211441 | 05:27 |
| *** vikram_ has quit IRC | 05:33 | |
| *** fawadkhaliq has quit IRC | 05:38 | |
| *** minwang2 has quit IRC | 05:38 | |
| *** vivek-ebay has quit IRC | 05:40 | |
| *** bharath has joined #openstack-neutron | 05:40 | |
| *** jianyong has joined #openstack-neutron | 05:40 | |
| *** bharath_ has joined #openstack-neutron | 05:42 | |
| *** Aish has joined #openstack-neutron | 05:43 | |
| *** jianyong has joined #openstack-neutron | 05:43 | |
| *** rook has quit IRC | 05:44 | |
| *** bharath has quit IRC | 05:45 | |
| *** jianyong has quit IRC | 05:46 | |
| *** ihrachyshka has joined #openstack-neutron | 05:47 | |
| *** humble__ has quit IRC | 05:48 | |
| *** diga_ has joined #openstack-neutron | 05:50 | |
| *** dsanders has joined #openstack-neutron | 05:51 | |
| *** dsanders has quit IRC | 05:51 | |
| *** gberginc has joined #openstack-neutron | 05:53 | |
| *** Murali_ has joined #openstack-neutron | 05:53 | |
| *** neophy has joined #openstack-neutron | 05:54 | |
| *** lyanchih has joined #openstack-neutron | 05:55 | |
| *** Ramanjaneya has joined #openstack-neutron | 05:56 | |
| *** mkolesni has joined #openstack-neutron | 05:56 | |
| *** yonglihe has joined #openstack-neutron | 05:56 | |
| *** vthapar has joined #openstack-neutron | 05:56 | |
| *** yonglihe has quit IRC | 05:56 | |
| *** yonglihe has joined #openstack-neutron | 05:57 | |
| *** Sukhdev_ has quit IRC | 05:58 | |
| openstackgerrit | Merged openstack/neutron: Clean up test_dvr_router_rem_fips_on_restarted_agent https://review.openstack.org/211700 | 05:58 |
| *** fawadkhaliq has joined #openstack-neutron | 05:59 | |
| *** josecastroleon has joined #openstack-neutron | 06:00 | |
| openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports https://review.openstack.org/210301 | 06:02 |
| ihrachyshka | ajo, hi! :) | 06:03 |
| openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports https://review.openstack.org/210304 | 06:03 |
| ihrachyshka | ajo, have you seen sc68cal dropped the fwaas revert from the merge queue? | 06:03 |
| ihrachyshka | ajo, I am a bit pissed off that we changed the direction with no backup plan, so I requested revert for devstack-gate: https://review.openstack.org/211853 | 06:04 |
| *** trinaths has joined #openstack-neutron | 06:05 | |
| ihrachyshka | ajo, one thing that we can do now is take some qos patches and make them Depends-On the fwaas revert and recheck several times to see whether it fixes the issue | 06:05 |
| *** vikram has joined #openstack-neutron | 06:07 | |
| *** hareeshp has joined #openstack-neutron | 06:07 | |
| *** yfried has joined #openstack-neutron | 06:07 | |
| tarun | ajo: Hi regarding mo files generation for neutron. You told 'emagana' could help me in this. Could you please share some details of him as i did not see anyone online with this name | 06:10 |
| ihrachyshka | ajo, ok, I updated 3 patches with Depends-On and we'll do some rechecks on them | 06:11 |
| *** zhenguo has quit IRC | 06:13 | |
| ihrachyshka | ajo, but let's not update all of them, it may not help. | 06:13 |
| *** irenab has quit IRC | 06:14 | |
| *** rook has joined #openstack-neutron | 06:14 | |
| *** Aish has quit IRC | 06:15 | |
| *** rook has quit IRC | 06:19 | |
| *** hareeshp has quit IRC | 06:19 | |
| *** vikram has quit IRC | 06:20 | |
| *** shivrao has quit IRC | 06:20 | |
| *** neophy has quit IRC | 06:21 | |
| *** lyanchih has quit IRC | 06:21 | |
| *** lyanchih has joined #openstack-neutron | 06:22 | |
| *** trinaths has quit IRC | 06:25 | |
| *** jprovazn has joined #openstack-neutron | 06:25 | |
| *** jianyong has joined #openstack-neutron | 06:26 | |
| tarun | Hi i want to understand how the mo files are generated in neutron code for localization as these were not a part of repository | 06:27 |
| *** gampel has joined #openstack-neutron | 06:28 | |
| *** nlahouti has quit IRC | 06:28 | |
| *** blackmonk has joined #openstack-neutron | 06:30 | |
| *** moshele has joined #openstack-neutron | 06:30 | |
| *** azbiswas has quit IRC | 06:31 | |
| *** miyagishi_t has joined #openstack-neutron | 06:32 | |
| *** vikram has joined #openstack-neutron | 06:32 | |
| *** azbiswas has joined #openstack-neutron | 06:32 | |
| *** azbiswas has quit IRC | 06:35 | |
| *** azbiswas has joined #openstack-neutron | 06:35 | |
| *** sdake_ has joined #openstack-neutron | 06:35 | |
| *** vivek-ebay has joined #openstack-neutron | 06:36 | |
| *** ihrachyshka has quit IRC | 06:39 | |
| *** sdake has quit IRC | 06:39 | |
| *** scheuran has joined #openstack-neutron | 06:40 | |
| *** vivek-ebay has quit IRC | 06:41 | |
| *** irenab has joined #openstack-neutron | 06:43 | |
| *** trinaths has joined #openstack-neutron | 06:43 | |
| *** azbiswas has quit IRC | 06:44 | |
| *** mohankumar has joined #openstack-neutron | 06:45 | |
| *** jlibosva has joined #openstack-neutron | 06:45 | |
| *** aranjan has joined #openstack-neutron | 06:48 | |
| *** rdo has quit IRC | 06:49 | |
| *** rdo has joined #openstack-neutron | 06:51 | |
| gsagie_ | irenab: https://etherpad.openstack.org/p/Kuryr_vif_binding_unbinding | 06:52 |
| gsagie_ | apuimedo : ^^ | 06:52 |
| irenab | gsagie_: thanks | 06:52 |
| *** SridharG has joined #openstack-neutron | 06:53 | |
| *** bharath_ has quit IRC | 06:57 | |
| *** yonglihe has quit IRC | 06:57 | |
| itzikb_ | good morning | 06:58 |
| *** kawa2014 has joined #openstack-neutron | 06:58 | |
| *** ildikov has quit IRC | 07:00 | |
| *** bharath has joined #openstack-neutron | 07:02 | |
| *** abehl has joined #openstack-neutron | 07:02 | |
| *** alex_klimov has joined #openstack-neutron | 07:03 | |
| *** alex_klimov has quit IRC | 07:03 | |
| *** devvesa has joined #openstack-neutron | 07:04 | |
| *** alex_klimov has joined #openstack-neutron | 07:04 | |
| *** xek has quit IRC | 07:04 | |
| *** xek has joined #openstack-neutron | 07:05 | |
| *** ronis has joined #openstack-neutron | 07:07 | |
| *** ronis_ has joined #openstack-neutron | 07:09 | |
| *** diga_ has quit IRC | 07:09 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce entities and model https://review.openstack.org/201621 | 07:10 |
| *** BhavyaM has joined #openstack-neutron | 07:11 | |
| *** fawadkhaliq has quit IRC | 07:12 | |
| *** fawadk has joined #openstack-neutron | 07:13 | |
| *** ronis has quit IRC | 07:13 | |
| *** cdl has joined #openstack-neutron | 07:14 | |
| *** ig0r_ has joined #openstack-neutron | 07:14 | |
| *** cdl has quit IRC | 07:16 | |
| *** ddepaoli has joined #openstack-neutron | 07:18 | |
| *** Ramanjaneya has quit IRC | 07:19 | |
| *** yamamoto has quit IRC | 07:20 | |
| *** jschwarz has joined #openstack-neutron | 07:20 | |
| *** Ramanjaneya has joined #openstack-neutron | 07:22 | |
| *** yamahata has joined #openstack-neutron | 07:24 | |
| jschwarz | ajo, SYN | 07:27 |
| *** achanda has quit IRC | 07:31 | |
| *** afazekas has joined #openstack-neutron | 07:32 | |
| *** _ has joined #openstack-neutron | 07:32 | |
| *** moshele has quit IRC | 07:32 | |
| *** _ is now known as Guest23 | 07:32 | |
| *** Guest23 has quit IRC | 07:33 | |
| *** Marga__ has joined #openstack-neutron | 07:33 | |
| *** moshele has joined #openstack-neutron | 07:33 | |
| *** liudong has joined #openstack-neutron | 07:34 | |
| scheuran | mestery, can I put the macvtap ml2-driver/agent feature extension on the neutron drivers meeting agenda for next week? Meaning that it gets confirmed and I get a direction if a spec is required. | 07:34 |
| scheuran | mestery, https://bugs.launchpad.net/neutron/+bug/1480979 | 07:35 |
| openstack | Launchpad bug 1480979 in neutron "Adding macvtap ml2 driver and agent" [Undecided,In progress] - Assigned to Andreas Scheuring (andreas-scheuring) | 07:35 |
| scheuran | mestery, I know that it won't make liberty, but this helps me getting early feedback and getting prepared for the next release... | 07:35 |
| *** achanda has joined #openstack-neutron | 07:36 | |
| *** zhenguo has joined #openstack-neutron | 07:36 | |
| *** nyechiel has joined #openstack-neutron | 07:37 | |
| *** Marga_ has quit IRC | 07:37 | |
| *** mohankumar has quit IRC | 07:38 | |
| *** rossella_s has joined #openstack-neutron | 07:38 | |
| *** rdo has quit IRC | 07:38 | |
| *** ig0r__ has joined #openstack-neutron | 07:38 | |
| *** ig0r_ has quit IRC | 07:41 | |
| *** lori|zzz is now known as lori | 07:43 | |
| *** jlanoux has joined #openstack-neutron | 07:45 | |
| *** jlibosva has quit IRC | 07:45 | |
| *** azbiswas has joined #openstack-neutron | 07:45 | |
| *** rdo has joined #openstack-neutron | 07:45 | |
| *** achanda has quit IRC | 07:46 | |
| *** azbiswas has quit IRC | 07:49 | |
| *** boris-42 has quit IRC | 07:50 | |
| openstackgerrit | vikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce entities and model https://review.openstack.org/201621 | 07:51 |
| *** yamamoto has joined #openstack-neutron | 07:51 | |
| *** blackmonk has quit IRC | 07:55 | |
| *** ajmiller has quit IRC | 07:56 | |
| *** ushkalim has joined #openstack-neutron | 07:58 | |
| *** gildub has quit IRC | 08:02 | |
| *** claudiub has joined #openstack-neutron | 08:03 | |
| *** sdake_ has quit IRC | 08:04 | |
| *** yog_ has joined #openstack-neutron | 08:05 | |
| *** yamahata has quit IRC | 08:08 | |
| *** aranjan has quit IRC | 08:08 | |
| *** ildikov has joined #openstack-neutron | 08:08 | |
| ajo | jschwarz, ACK | 08:08 |
| ajo | well | 08:08 |
| jschwarz | ajo, you mean 'SYN,ACK' | 08:09 |
| ajo | I may say: jschwarz , SYN + ACK | 08:09 |
| jschwarz | but OK | 08:09 |
| jschwarz | XD | 08:09 |
| ajo | to be right | 08:09 |
| ajo | lol | 08:09 |
| jschwarz | ajo, so where are we on the mergeback? | 08:09 |
| jschwarz | problems with fwaas, I assume? | 08:09 |
| ajo | jschwarz: blocked by gate | 08:09 |
| *** gberginc has quit IRC | 08:09 | |
| *** dguerri` is now known as dguerri | 08:09 | |
| jschwarz | gah. | 08:09 |
| ajo | I will discuss with ihar, that we may build the diff and provide it for review | 08:09 |
| ajo | even if gate is givin a hard time to neutron ATM | 08:09 |
| jschwarz | ajo, I think that's wise | 08:09 |
| jschwarz | it might not merge but we can start the review process at least | 08:10 |
| ajo | it's not qos fault, just the mysql driver change and something in fwaas | 08:10 |
| jschwarz | aye, I saw the discussion | 08:10 |
| *** lyanchih has quit IRC | 08:10 | |
| jschwarz | ajo, still with mestery in a convention or something next week... we want to do as much as possible this week | 08:10 |
| *** jlibosva has joined #openstack-neutron | 08:12 | |
| *** ajmiller has joined #openstack-neutron | 08:12 | |
| *** ig0r__ has quit IRC | 08:13 | |
| *** diga_ has joined #openstack-neutron | 08:14 | |
| ajo | jschwarz, yes, if we get reviews started, and the patches won't change, that's a good thing | 08:14 |
| ajo | we should convince ihar when he's back | 08:14 |
| jschwarz | ajo, we might have to think-ahead with what we have stuck at the gate but... | 08:14 |
| jschwarz | ajo, ping me when he's back and I'll help you ;-) | 08:14 |
| ajo | jschwarz, I'm not sure exactly, ask jlibosva in priv ;) | 08:15 |
| jschwarz | ajo, should be around 1ish, no? | 08:15 |
| ajo | jschwarz, absolutely no idea | 08:15 |
| jschwarz | jlibosva, ? | 08:15 |
| jlibosva | o/ | 08:16 |
| jlibosva | jschwarz: ajo I don't know the context :) | 08:16 |
| jschwarz | jlibosva, asking when ihar will be in the office so we can conspire against him and do evil things with qos :) | 08:16 |
| jlibosva | jschwarz: at about noon | 08:17 |
| jschwarz | GMT+2? | 08:17 |
| *** akorzeni has joined #openstack-neutron | 08:17 | |
| jschwarz | so 2 hours from now | 08:17 |
| jlibosva | y | 08:17 |
| jschwarz | :) | 08:17 |
| jlibosva | yep | 08:17 |
| *** ig0r_ has joined #openstack-neutron | 08:17 | |
| jschwarz | ajo, & | 08:17 |
| jschwarz | ^ | 08:17 |
| salv-orlando | did ihar move to the us? | 08:17 |
| *** cdl has joined #openstack-neutron | 08:17 | |
| *** Marga__ has quit IRC | 08:17 | |
| jlibosva | salv-orlando: nope :) | 08:18 |
| jschwarz | salv-orlando, he has some classes in the early mornings, it seems | 08:18 |
| salv-orlando | I see, good for him | 08:18 |
| *** Marga_ has joined #openstack-neutron | 08:18 | |
| jlibosva | but he's usually online in the morning before he does hit business | 08:18 |
| jlibosva | s/hit/his/ | 08:18 |
| *** ig0r_ has quit IRC | 08:19 | |
| jschwarz | jlibosva, funnier if you leave it 'hit'. | 08:19 |
| jlibosva | I haven't had coffee yet so I'm writing random words | 08:19 |
| *** lyanchih has joined #openstack-neutron | 08:20 | |
| salv-orlando | jlibosva: hit business is usually a highly remunerative one | 08:21 |
| *** jpena has joined #openstack-neutron | 08:21 | |
| *** jistr has joined #openstack-neutron | 08:21 | |
| * jlibosva looks for "remunerative" in dictionary | 08:21 | |
| *** vthapar has quit IRC | 08:21 | |
| * jschwarz also looks for that word | 08:21 | |
| jschwarz | aaahh | 08:22 |
| jschwarz | XD | 08:22 |
| salv-orlando | it's one of those words english inherited from latin ;) It sounds posh in english but is common in italian | 08:22 |
| ajo | salv-orlando: lol | 08:23 |
| ajo | I completely understood it :) | 08:24 |
| ajo | http://www.wordreference.com/es/en/translation.asp?spen=remunerar | 08:24 |
| *** tsg has joined #openstack-neutron | 08:24 | |
| ajo | http://www.wordreference.com/iten/remunerazione | 08:25 |
| ajo | :-) | 08:25 |
| ajo | salv-orlando: well, english keeps polluting spanish, italian and all other languages, I'm ok with polluting it a little bit :P | 08:26 |
| *** tsg has quit IRC | 08:30 | |
| *** roeyc has joined #openstack-neutron | 08:30 | |
| *** yog_ has quit IRC | 08:30 | |
| *** yamamoto has quit IRC | 08:31 | |
| *** miyagishi_t has quit IRC | 08:31 | |
| *** yamamoto has joined #openstack-neutron | 08:32 | |
| *** tfukushima has joined #openstack-neutron | 08:34 | |
| *** yalie has left #openstack-neutron | 08:34 | |
| salv-orlando | ajo: well spanish not so much. now go back yo your computadora | 08:35 |
| salv-orlando | or ordenador if your computer is a boy ;) | 08:36 |
| *** sudipto has joined #openstack-neutron | 08:40 | |
| *** pavel_bondar has joined #openstack-neutron | 08:43 | |
| *** yog_ has joined #openstack-neutron | 08:44 | |
| *** hichihara has quit IRC | 08:53 | |
| *** trinaths has quit IRC | 09:00 | |
| ajo | salv-orlando: lol :) | 09:01 |
| *** yalie has joined #openstack-neutron | 09:02 | |
| yalie | hi yamamoto? | 09:02 |
| yamamoto | yalie: hi | 09:02 |
| yalie | I just saw your comments https://review.openstack.org/#/c/129773/25/neutron/db/l3_db.py | 09:02 |
| yalie | thanks | 09:03 |
| yalie | do you mean that we don't need specify the 'mac' ? | 09:03 |
| yalie | related to some other bugs? | 09:04 |
| yamamoto | yalie: my comment is not specific to 'mac' | 09:04 |
| yamamoto | the change fills default values for unspecified values, right? | 09:04 |
| *** trinaths has joined #openstack-neutron | 09:05 | |
| *** yfried is now known as yfried|afk | 09:05 | |
| yalie | yes | 09:05 |
| yalie | I see | 09:05 |
| *** ihrachyshka has joined #openstack-neutron | 09:05 | |
| ihrachyshka | ajo, jschwarz: searched for me? | 09:05 |
| yalie | ok, I will check it , thanks | 09:05 |
| ajo | hi ihrachyshka , jschwarz and I had a plan | 09:06 |
| yamamoto | yalie: np | 09:06 |
| ajo | since the gate issue is not a feature/qos thing | 09:06 |
| *** apuimedo has joined #openstack-neutron | 09:06 | |
| ihrachyshka | ajo, does it include 'going out, have some fun'? | 09:06 |
| ajo | and the last patches are clear | 09:06 |
| ajo | ihrachyshka: :-) I wish it was that :D | 09:06 |
| ajo | ihrachyshka: what do you think about providing the diff | 09:07 |
| ajo | and the email with merge/back request | 09:07 |
| ajo | given that those last patches are not changing | 09:07 |
| ajo | so we get people already reviewing it, | 09:07 |
| ajo | and I will provide the install instructions, and the video | 09:07 |
| ajo | (working on the video ATM) | 09:08 |
| ajo | jschwarz ^ anything to say? :) | 09:09 |
| *** iwamoto has quit IRC | 09:09 | |
| ihrachyshka | ajo, I think we should get them in by reverting devstack-gate, then rechecking | 09:10 |
| ihrachyshka | it should be doable | 09:10 |
| ihrachyshka | as I see, fwaas revert does not help | 09:10 |
| ajo | ihrachyshka: I didn't see any fwaas failure with the fwaas revert | 09:10 |
| ajo | did you? | 09:10 |
| ihrachyshka | in depends-on patches | 09:10 |
| ajo | may be I missed it | 09:10 |
| ihrachyshka | I made some to depend on it | 09:10 |
| ajo | yes, no Fwaas failures on those | 09:11 |
| ihrachyshka | http://logs.openstack.org/37/211537/2/check/gate-neutron-dsvm-api/6b5668d/ | 09:11 |
| ajo | hhmmm | 09:11 |
| ajo | let me check | 09:11 |
| ihrachyshka | that's in https://review.openstack.org/#/c/211537/ | 09:11 |
| openstackgerrit | vikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce dragent scheduler model https://review.openstack.org/207607 | 09:11 |
| ajo | :-( | 09:11 |
| ajo | ihrachyshka, ok, I missed that one | 09:12 |
| ihrachyshka | I will now chase for infra | 09:12 |
| ihrachyshka | ajo, oh, that one actually has wrong tag! | 09:12 |
| ihrachyshka | jlibosva, bad boy! :D | 09:13 |
| jlibosva | not my fault! | 09:13 |
| ihrachyshka | ajo, test failure? https://review.openstack.org/#/c/211520/ | 09:13 |
| jlibosva | I'm just a brainless copy&paster | 09:13 |
| ihrachyshka | hm, so it seems that other patches succeeded | 09:14 |
| ihrachyshka | then it's good news! | 09:14 |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 09:14 |
| *** trinaths has quit IRC | 09:15 | |
| ihrachyshka | ajo, have you rechecked all? | 09:16 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Rename function '_update_port_down' https://review.openstack.org/211920 | 09:17 |
| ajo | ihrachyshka, yes, not that one with the wrong tag, | 09:17 |
| ihrachyshka | ajo, ok, I'll make others critical to depends-on | 09:17 |
| ajo | I think I did, let me "recheck" my work ;) | 09:17 |
| *** park_heijlong has joined #openstack-neutron | 09:18 | |
| *** john-davidge has joined #openstack-neutron | 09:18 | |
| ihrachyshka | all good | 09:19 |
| ihrachyshka | ajo, fix the topic: https://review.openstack.org/211603 | 09:20 |
| ajo | ihrachyshka: done | 09:20 |
| ajo | ihrachyshka: so the FwAAS revert so far seems to be fixing it, right? | 09:20 |
| *** sayali has quit IRC | 09:21 | |
| ajo | let's see how next rechecks behave | 09:21 |
| ajo | I think it's better to fix the specific sin, than opening the gate for more sins then | 09:21 |
| *** john-davidge_ has joined #openstack-neutron | 09:21 | |
| ihrachyshka | ajo, meh. https://review.openstack.org/#/c/211603/ fails in fwaas | 09:21 |
| ihrachyshka | please verify me | 09:21 |
| * ajo checks | 09:21 | |
| ajo | tearDownClass (neutron.tests.api.test_fwaas_extensions.FWaaSExtensionTestJSON) | 09:22 |
| ajo | :( | 09:22 |
| ajo | let's check the tag | 09:22 |
| ajo | depends-on is good | 09:22 |
| ajo | yikes | 09:23 |
| ihrachyshka | sigh | 09:23 |
| ihrachyshka | ok, then devstack-gate revert | 09:23 |
| *** john-davidge has quit IRC | 09:23 | |
| *** john-davidge_ is now known as john-davidge | 09:23 | |
| ihrachyshka | ajo, the revert https://review.openstack.org/#/c/211853/ | 09:24 |
| *** yfried|afk is now known as yfried | 09:25 | |
| ajo | ihrachyshka: yep, I commented | 09:26 |
| ihrachyshka | thanks | 09:26 |
| ajo | ihrachyshka, shall we remove the depends-on for now, and play lottery? | 09:26 |
| ihrachyshka | yeah, I think we should | 09:26 |
| ajo | ok, | 09:26 |
| ihrachyshka | can you handle it while I chat with infra? | 09:26 |
| * ajo buys some tickets | 09:26 | |
| ajo | ihrachyshka: sure | 09:26 |
| ihrachyshka | thanks | 09:26 |
| *** park_heijlong has quit IRC | 09:28 | |
| ihrachyshka | ajo, I'll report a bug for that failure | 09:28 |
| *** kawa2014 has quit IRC | 09:28 | |
| *** kawa2014 has joined #openstack-neutron | 09:28 | |
| ihrachyshka | oh there is one already: https://bugs.launchpad.net/neutron/+bug/1483875 | 09:29 |
| openstack | Launchpad bug 1483875 in neutron "FWaaS - firewall deleted unexpectedly by agent" [Undecided,New] | 09:29 |
| *** roeyc has quit IRC | 09:31 | |
| *** sayali has joined #openstack-neutron | 09:33 | |
| *** yfried is now known as yfried|afk | 09:35 | |
| ajo | ihrachyshka, checking we're on the same page, we mean cleaning all the depends-on tags | 09:38 |
| ajo | not updating to the gate revert of the mysql workaround, right? | 09:38 |
| ihrachyshka | ajo, yes. no. | 09:38 |
| ihrachyshka | ajo, we'll play lottery until devstack-gate is merged. | 09:38 |
| ajo | ack | 09:38 |
| sc68cal | I'm awake | 09:38 |
| *** vivek-ebay has joined #openstack-neutron | 09:38 | |
| ihrachyshka | sc68cal, cool. so fwaas revert does not help | 09:38 |
| ihrachyshka | sc68cal, we go devstack-gate revert way for now | 09:39 |
| ihrachyshka | sc68cal, the bug you reported will be used to track the fixes in fwaas itself | 09:39 |
| sc68cal | ihrachyshka: understood. | 09:39 |
| *** yfried|afk is now known as yfried | 09:39 | |
| ihrachyshka | sc68cal, it will be hard to get attention from infra cores now since most are asleep, so we'll play lottery for qos in the meantime | 09:39 |
| sc68cal | ihrachyshka: I saw from your one comment the failures related to bug #1483875 - higher percentage in master branch and single digit percentage in feature/qos? | 09:41 |
| openstack | bug 1483875 in neutron "FWaaS - firewall deleted unexpectedly by agent" [Undecided,New] https://launchpad.net/bugs/1483875 | 09:41 |
| ihrachyshka | sc68cal, well, it may be a glitch, I don't know (it was ajo's comment) | 09:41 |
| ihrachyshka | I don't know anything in qos that could affect it | 09:41 |
| sc68cal | it could be just there was more chrun in qos than master yesterday | 09:41 |
| sc68cal | *churn | 09:41 |
| ihrachyshka | maybe a bit more work on the agent side that makes it respond with ACTIVE status later?.. | 09:42 |
| *** mohankumar has joined #openstack-neutron | 09:42 | |
| ajo | sc68cal, ihrachyshka , yes, it's totally related to the amount of checks you run for each branch in gate | 09:42 |
| *** Kennan2 is now known as Kennan | 09:42 | |
| ihrachyshka | ack :) we were wild to push it :) | 09:42 |
| ajo | those numbers may need to be correlated to the amount of gate checks per branch... | 09:42 |
| ajo | yep | 09:42 |
| ajo | so, for 7d it's 20% master, 6% feature/qos | 09:42 |
| *** vivek-ebay has quit IRC | 09:42 | |
| ajo | which could diverge for shorter days as we were trying to push things at qos :) | 09:43 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 09:43 |
| sc68cal | sorry - I was trying to look for silver bullets to unblock you guys. I apologies for waffling later in the day but I just looked more and more and became less and less certain of my initial move | 09:44 |
| ihrachyshka | sc68cal, that was actually a good one in the end since it does not fix the issue | 09:44 |
| ihrachyshka | sc68cal, we could have devstack-gate revert up earlier probably though | 09:44 |
| jschwarz | ihrachyshka, hello :) | 09:45 |
| ihrachyshka | as it always goes... we should first unblock the gate, then dig. | 09:45 |
| sc68cal | ihrachyshka: ack - based on traffic in -infra I was hoping to keep this one "in the family" as it were | 09:45 |
| ihrachyshka | jschwarz, hey man | 09:45 |
| ihrachyshka | sc68cal, yeah, it was kinda wild there | 09:45 |
| sc68cal | ihrachyshka: instead of going to them off the bat | 09:45 |
| sc68cal | do we want to put something up on the ML that we think maybe the d-g revert is the way to go ? | 09:46 |
| sc68cal | for those playing at home | 09:46 |
| jschwarz | ihrachyshka, so why we don't... just... uh... disable the fwaas test that's failing, and let them worry about it later? | 09:46 |
| ihrachyshka | sc68cal, that's a good idea. me/you? | 09:47 |
| sc68cal | jschwarz: I thought about it - I think we need to break out the xfail decorator - run the test, but don't fail the job if it fails | 09:47 |
| sc68cal | so we keep getting telemetry | 09:47 |
| *** bharath has quit IRC | 09:47 | |
| jschwarz | sc68cal, sounds good to me | 09:48 |
| sc68cal | http://pytest.org/latest/skipping.html | 09:48 |
| *** azbiswas has joined #openstack-neutron | 09:48 | |
| ihrachyshka | jschwarz, not sure. let's see how quickly infra responds. | 09:48 |
| jschwarz | ihrachyshka, ajo, in the mean time please +2+A this: https://review.openstack.org/#/c/211565/ | 09:48 |
| ihrachyshka | sc68cal, xfail is interesting. | 09:48 |
| jschwarz | votes were lost in the commit message changes you made | 09:48 |
| *** bharath has joined #openstack-neutron | 09:49 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: DVR: fix router rescheduling on server side https://review.openstack.org/199514 | 09:49 |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: DVR: fix router rescheduling on agent side https://review.openstack.org/211939 | 09:49 |
| jschwarz | ihrachyshka, anyway what do you say about the 'push diff review and ML mail now so we can get reviews for branch merge'? | 09:49 |
| *** yamamoto has quit IRC | 09:50 | |
| *** Marga__ has joined #openstack-neutron | 09:50 | |
| *** yamamoto has joined #openstack-neutron | 09:51 | |
| *** Marga_ has quit IRC | 09:52 | |
| *** azbiswas has quit IRC | 09:52 | |
| *** bharath has quit IRC | 09:53 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Add test that checks external tables are not changed https://review.openstack.org/210449 | 09:53 |
| ihrachyshka | jschwarz, I hate it and hope that we won't need it. but let's keep it as fallback plan. | 09:54 |
| *** erkules has quit IRC | 09:54 | |
| *** erkules has joined #openstack-neutron | 09:54 | |
| *** gampel1 has joined #openstack-neutron | 09:56 | |
| ajo | ihrachyshka, jschwarz , in zuul we trust | 09:56 |
| ajo | ihrachyshka: what's the status of this? https://review.openstack.org/#/c/208273/ | 09:56 |
| ajo | another refactor not for now? | 09:57 |
| ajo | I would abandon those not-for-now refactors, so the list of patches is clean | 09:57 |
| ajo | it's more a zen thing, so people looking at the branch won't find a mess | 09:57 |
| *** gampel has quit IRC | 09:57 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 09:58 |
| *** moshele has quit IRC | 09:58 | |
| ajo | ihrachyshka: shall we remove +A from https://review.openstack.org/#/c/210483/ until jenkins check passes again? | 09:58 |
| ajo | ihrachyshka: and here: https://review.openstack.org/#/c/211565/ | 09:59 |
| ajo | well, those can only fail because of fwaas, our stuff it's already been tested | 10:01 |
| ajo | so +A makes sense probably | 10:01 |
| *** claudiub has quit IRC | 10:03 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Remove cache of registered ports in the OVS agent https://review.openstack.org/211192 | 10:04 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: check_changed_vlans doesn't need registered_ports as param https://review.openstack.org/211191 | 10:04 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Don't sync all devices when some fail https://review.openstack.org/202098 | 10:04 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent set max number of attempts to sync failed devices https://review.openstack.org/211586 | 10:04 |
| ihrachyshka | ajo, yes, that was my reasoning | 10:05 |
| ihrachyshka | that's why +A | 10:05 |
| ihrachyshka | ajo, as for refactor, it will go in master. | 10:05 |
| ajo | ihrachyshka: we need an external +2 https://review.openstack.org/#/c/211520/ | 10:05 |
| ihrachyshka | ajo, I don't think we should drop it now though | 10:05 |
| *** pavel_bondar has quit IRC | 10:05 | |
| ajo | ihrachyshka, it's just a cleanup thing, they are stored in gerrit anyway | 10:05 |
| ihrachyshka | ajo, it's not just a cleanup, it's API rework. why do you want to drop it? | 10:06 |
| ihrachyshka | let the author do it | 10:06 |
| ajo | ihrachyshka, sure, I won't do it without moshele here | 10:06 |
| ihrachyshka | zen is overrated | 10:06 |
| ajo | here->agreeing ;) | 10:06 |
| *** fzdarsky has joined #openstack-neutron | 10:06 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Python 3: compare response.body to bytes in namespace_proxy test https://review.openstack.org/206991 | 10:07 |
| ajo | rossella_s ping | 10:07 |
| ajo | can I ask you to look on a tiny patch on feature/qos ? :) | 10:08 |
| ajo | https://review.openstack.org/#/c/211520/ | 10:08 |
| rossella_s | ajo: yes of course | 10:08 |
| ajo | we need a +2 to move it forward, I've manually tested it, and it also has functional test | 10:08 |
| * rossella_s looking at it | 10:08 | |
| ajo | rossella_s, it's on the backend, setting "egress" limits on the interface, (ovs knows it as ingress, because ovs considers ingress from the switch point of view) | 10:09 |
| ajo | rossella_s++ thanks | 10:09 |
| *** ig0r_ has joined #openstack-neutron | 10:10 | |
| rossella_s | ajo thanks for the background info...feel free to ping me when you need some review, I need to defeat my laziness somehow :p | 10:10 |
| ajo | rossella_s: same thing for me... feel free to ping me too | 10:11 |
| ajo | I'm quite tied lately with the QoS thing, but starting to look to other patches again | 10:11 |
| rossella_s | I was trying to have a look at the qos patches from time to time but you people are working too hard can't keep your pace | 10:11 |
| ajo | rossella_s: I made a pause in the "too hard", ihrachyshka it's been our hardest QoS'er ;) | 10:12 |
| ajo | it's -> has | 10:12 |
| ajo | pause=PTO | 10:12 |
| rossella_s | humans need some rest from time to time | 10:13 |
| openstackgerrit | Sean M. Collins proposed openstack/neutron: Mark FwaaS tests as xfail https://review.openstack.org/211947 | 10:15 |
| ajo | :-) | 10:15 |
| sc68cal | Don't know if I did it right - docs for it are a little vague but we'll see | 10:15 |
| ajo | sc68cal: expect fail will fail if it doesn't fail? :) | 10:15 |
| ajo | ihrachyshka ^ | 10:16 |
| jschwarz | rossella_s, that's the point - ihrachyshka is inhuman :) | 10:16 |
| ajo | this other approach may work without opening the gate to other races | 10:16 |
| ihrachyshka | jschwarz, as long as I pass Turing test, you never know | 10:16 |
| ajo | ihrachyshka: lol | 10:16 |
| *** fzdarsky has quit IRC | 10:16 | |
| rossella_s | jschwarz, ihrachyshka ajo lol! | 10:17 |
| sc68cal | ajo: something like that | 10:17 |
| *** sudipto_ has joined #openstack-neutron | 10:18 | |
| *** claudiub has joined #openstack-neutron | 10:19 | |
| *** diga_ has quit IRC | 10:19 | |
| *** jkraj has joined #openstack-neutron | 10:19 | |
| ihrachyshka | sc68cal, reading https://docs.python.org/2/library/unittest.html, I don't see expectedFailure applied to classes there but test case methods | 10:20 |
| sc68cal | ihrachyshka: true, but the paragraph above says: "Classes can be skipped just like methods" | 10:21 |
| sc68cal | ihrachyshka: I don't know if they just mean for skip, or if that applies to expected failure as well | 10:21 |
| *** sudipto has quit IRC | 10:21 | |
| ihrachyshka | oh ok let's see then :) | 10:21 |
| * jschwarz notes that decorators can work on classes as well | 10:21 | |
| *** vikram has quit IRC | 10:22 | |
| sc68cal | ihrachyshka: the tricky part is that for some reason the tearDownClass method is failing due to the race, so i'd have to override and super() to decorate a method for it | 10:22 |
| sc68cal | hence trying to decorate the whole class | 10:22 |
| ihrachyshka | sc68cal, I think it would not fail if not the first test | 10:23 |
| ihrachyshka | sc68cal, it basically fails to cleanup the firewall that the first test failed to cleanup | 10:23 |
| sc68cal | ihrachyshka: yeah - test_firewall_insertion_mode_add_remove_router | 10:23 |
| ihrachyshka | so I expect that we could apply that to the test case only | 10:23 |
| sc68cal | ihrachyshka: I can respin and we can try it. | 10:26 |
| ihrachyshka | sc68cal, I am for it | 10:26 |
| openstackgerrit | Sean M. Collins proposed openstack/neutron: Mark a FwaaS tests as expectedFailure https://review.openstack.org/211947 | 10:28 |
| * sc68cal goes to make coffee | 10:29 | |
| *** yuanying has quit IRC | 10:30 | |
| *** yuanying has joined #openstack-neutron | 10:30 | |
| *** vikram has joined #openstack-neutron | 10:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/208705 | 10:32 |
| rossella_s | ajo sorry for the delay, I just wanted to understand the code a bit...so I +2 and +A | 10:33 |
| *** itzikb_ has quit IRC | 10:33 | |
| rossella_s | ajo hope you wanted the +A too .. | 10:34 |
| ihrachyshka | rossella_s, show me someone who does not want +A | 10:36 |
| *** abehl has quit IRC | 10:36 | |
| rossella_s | ihrachyshka good point...anyway since I am not so much involved in your work, I thought better ask :) | 10:37 |
| ihrachyshka | rossella_s, we strive to flush remaining patches in the tree and request merge-back to master today :) | 10:37 |
| ihrachyshka | that said, we planned to do it on Mon initially, so every day it's today | 10:38 |
| rossella_s | I know | 10:38 |
| *** vikram has quit IRC | 10:38 | |
| rossella_s | I have been following that | 10:38 |
| rossella_s | if you need help just ping | 10:38 |
| rossella_s | ajo ihrachyshka btw not sure if this can be useful but I stepped into this blog post looking for some more info regarding queues https://n40lab.wordpress.com/2013/05/04/openvswitch-setting-a-bandwidth-limit/ for that guy it seems that ingress is not limited by default | 10:40 |
| *** fawadk has quit IRC | 10:41 | |
| *** gampel has joined #openstack-neutron | 10:42 | |
| *** gampel1 has quit IRC | 10:44 | |
| *** gampel has quit IRC | 10:46 | |
| *** ig0r_ has quit IRC | 10:46 | |
| jschwarz | ihrachyshka, ajo, https://review.openstack.org/#/c/211565/ failed at the gate again... | 10:46 |
| ihrachyshka | jschwarz, we'll recheck. note that devstack-gate revert is in the gate already | 10:47 |
| ihrachyshka | so we should be fixed quickly | 10:47 |
| *** moshele has joined #openstack-neutron | 10:48 | |
| jschwarz | working quickly, you are | 10:48 |
| jschwarz | ihrachyshka, also, the dvr job failed again because of timeouts I think | 10:49 |
| jschwarz | that's twice in a row now. | 10:49 |
| *** abehl has joined #openstack-neutron | 10:49 | |
| ihrachyshka | jschwarz, they don't play alone | 10:49 |
| *** bharath has joined #openstack-neutron | 10:49 | |
| jschwarz | ihrachyshka, who do they play with? | 10:49 |
| ihrachyshka | jschwarz, us, I guess ;) | 10:50 |
| ajo | rossella_s thanks, looking at the blog post | 10:50 |
| jschwarz | we do nothing with dvr :< | 10:50 |
| *** pc_m has joined #openstack-neutron | 10:50 | |
| ajo | rossella_s, ok, it matches what I saw | 10:51 |
| ihrachyshka | jschwarz, but dvr does something with you! | 10:51 |
| ajo | with queue + qos + linux-htb, we limit the ingress from VM point of view :) | 10:51 |
| ajo | where we were expecting to limit egress :) | 10:51 |
| ajo | rossella_s, I think that to limit egress via a queue we also need to use OF rules to direct traffic to the queue | 10:51 |
| *** vikram has joined #openstack-neutron | 10:51 | |
| *** bharath has quit IRC | 10:54 | |
| rossella_s | ajo now I get it...I thought in the blog the test was downloading a file located on the vm but you are probably right, the test is starting a download from the VM | 10:54 |
| rossella_s | ajo glad it matches what you saw then | 10:55 |
| *** alexpilotti has joined #openstack-neutron | 10:55 | |
| ajo | rossella_s, you had to see my face when trying to test it manually | 10:55 |
| ajo | as it as changing the ingress speed, while I was forcing egress, it was funny | 10:55 |
| ajo | it was TCP | 10:56 |
| ajo | so 10kbps ingress matched to 1Mbps egress more or less to tcp | 10:56 |
| rossella_s | ajo I can imagine, you are brave | 10:56 |
| ajo | and 20kbps, to 2.2Mbps more or less.. | 10:56 |
| ajo | so initially I thought it was a magnitude issue | 10:56 |
| ajo | :D | 10:56 |
| ajo | rossella_s, things got wierd, when I configured 50kbps and I got 10Mbps... (funny TCP) | 10:56 |
| ajo | and... then I realized it was limiting ingress, which btw, it was supposed to be a very complicated thing to do :) (well, it's for non TCP protocols actually) | 10:57 |
| *** yfried is now known as yfried|afk | 11:00 | |
| openstackgerrit | Marga Millet proposed openstack/neutron: Allow host route injection of metadata server IP https://review.openstack.org/211963 | 11:00 |
| *** rossella_s has quit IRC | 11:00 | |
| *** gberginc has joined #openstack-neutron | 11:00 | |
| openstackgerrit | sean mooney proposed openstack/neutron: Adds configurable agent type https://review.openstack.org/197210 | 11:01 |
| openstackgerrit | sean mooney proposed openstack/neutron: Add config option to specify ovs datapath. https://review.openstack.org/197209 | 11:01 |
| *** mwagner_lap has quit IRC | 11:02 | |
| *** zhenguo has quit IRC | 11:03 | |
| *** kawa2014 has quit IRC | 11:03 | |
| *** lyanchih has quit IRC | 11:04 | |
| *** mohankumar has quit IRC | 11:05 | |
| *** oomichi has quit IRC | 11:05 | |
| *** yfried|afk is now known as yfried | 11:08 | |
| *** roeyc has joined #openstack-neutron | 11:08 | |
| *** yamamoto has quit IRC | 11:08 | |
| *** Ramanjaneya has quit IRC | 11:09 | |
| *** trinaths has joined #openstack-neutron | 11:09 | |
| sc68cal | scheuran: ping - sorry about yesterday | 11:11 |
| *** jpena is now known as jpena|lunch | 11:15 | |
| *** BhavyaM has quit IRC | 11:18 | |
| sc68cal | kevinbenton: Sam-I-Am: I saw a related patch, to the vxlan_udp_port - https://review.openstack.org/208762 | 11:21 |
| sc68cal | kevinbenton: Sam-I-Am: oops, that's just a devstack patch. sorry thought it was neutron :( | 11:23 |
| ihrachyshka | sc68cal, UnexpectedSuccess ? :) | 11:26 |
| ihrachyshka | sc68cal, the xfail patch just failed with ^^ | 11:26 |
| openstackgerrit | shihanzhang proposed openstack/neutron: Destroy ipset when the corresponding rule is removed https://review.openstack.org/190991 | 11:27 |
| sc68cal | ihrachyshka: lol wat | 11:28 |
| ihrachyshka | sc68cal, apparently it fails on success | 11:28 |
| ihrachyshka | so it's a sort of a way to introduce negative cases ;) | 11:28 |
| sc68cal | oh for cryin out loud | 11:28 |
| sc68cal | so it says unexpected success but there's a stacktrace where it couldn't find the firewall | 11:29 |
| ihrachyshka | sc68cal, nope | 11:31 |
| ihrachyshka | sc68cal, there is no traceback | 11:31 |
| ihrachyshka | sc68cal, the way test works is it creates a FW, then requests a delete, then spins waiting for the agent to report back with firewall_deleted() and the FW to go away from the db | 11:32 |
| ihrachyshka | then it gets NotFound and claims success | 11:32 |
| ihrachyshka | so it walked thru the case, got success and failed because it expected failure | 11:32 |
| ihrachyshka | as easy as that | 11:32 |
| *** baoli has joined #openstack-neutron | 11:33 | |
| * john-davidge thinks unittest needs a @NotSureWhatToExpect decorator | 11:34 | |
| john-davidge | sc68cal: Would it be worth skipping the test for now? | 11:35 |
| *** baoli has quit IRC | 11:35 | |
| *** baoli has joined #openstack-neutron | 11:35 | |
| *** gampel has joined #openstack-neutron | 11:36 | |
| *** thorst has joined #openstack-neutron | 11:37 | |
| sc68cal | ihrachyshka: ah. I see now. The test *has* to fail with that decorator | 11:37 |
| ihrachyshka | yeap | 11:37 |
| sc68cal | ihrachyshka: dang it - why can't I get a decorator to RUN THE TEST BUT NOT FAIL THE JOB IF IT DOESN'T FAIL | 11:38 |
| ihrachyshka | so an easy way to flip from success to negative case | 11:38 |
| * sc68cal slams head against desk | 11:38 | |
| *** julim has joined #openstack-neutron | 11:38 | |
| sc68cal | ok, i'll push a patch to just skip it for now - I'll play around with xfail on separate track | 11:39 |
| ihrachyshka | sc68cal, note they say we should import unittest2 in the email thread | 11:40 |
| *** sudipto_ has quit IRC | 11:40 | |
| sc68cal | yeah saw it | 11:40 |
| *** Marga__ has quit IRC | 11:40 | |
| *** Marga_ has joined #openstack-neutron | 11:41 | |
| *** matrohon has joined #openstack-neutron | 11:41 | |
| * sc68cal keeps mashing buttons | 11:42 | |
| openstackgerrit | Sean M. Collins proposed openstack/neutron: Skip FwaaS test that is failing due to race condition https://review.openstack.org/211979 | 11:42 |
| *** Ramanjaneya has joined #openstack-neutron | 11:43 | |
| *** mohankumar has joined #openstack-neutron | 11:45 | |
| ihrachyshka | devstack-gate revert merged | 11:48 |
| ihrachyshka | ajo, jschwarz, jlibosva ^^ | 11:48 |
| ihrachyshka | moshele, ^^ | 11:48 |
| jschwarz | whooooooooooooooooooooooooo | 11:48 |
| jschwarz | whooooooooooooooooooooooooooooooooooooo | 11:48 |
| jschwarz | whooo | 11:48 |
| jschwarz | :) | 11:48 |
| ihrachyshka | let's recheck where applies and see what breaks us now | 11:48 |
| ihrachyshka | ajo, please manage the recheck effort :) | 11:48 |
| jschwarz | ihrachyshka, i rechecked https://review.openstack.org/#/c/211565/ a few minutes back so I'll ping it every now and then on the zuul and see | 11:49 |
| *** trinaths has quit IRC | 11:49 | |
| ihrachyshka | jschwarz, well, you should have waited a bit | 11:49 |
| jschwarz | I know :( sorry | 11:49 |
| ihrachyshka | jschwarz, because now it could catch the old devstack-gate | 11:49 |
| ihrachyshka | we'll see | 11:49 |
| ihrachyshka | not a great deal :) | 11:50 |
| ajo | ok, we'll see | 11:50 |
| jschwarz | ihrachyshka, there's a 1% chance it won't ;-) | 11:50 |
| ajo | I'm handling the rechecks, please not collide | 11:50 |
| ajo | ;) | 11:50 |
| jschwarz | (or that it will not fail) | 11:50 |
| dmellado | ihrachyshka: just out of curiosity, how much should you usually wait for the former devstack-gate not to be used anymore? | 11:50 |
| ihrachyshka | AJO IS ON RECHECKS, STEP BACK OR HE WILL HIT YOU | 11:50 |
| ajo | ihrachyshka: lol | 11:50 |
| ihrachyshka | dmellado, hm? I hope it's immediate | 11:51 |
| ihrachyshka | dmellado, I know that zuul/jenkins changes (f.e. project-config) do not apply immediately | 11:51 |
| *** dtynan has quit IRC | 11:51 | |
| ihrachyshka | but devstack-gate should be different. hopefully. | 11:51 |
| dmellado | ihrachyshka: thanks for the clarification | 11:52 |
| ihrachyshka | dmellado, jenkins changes are special, they are not really gated against real openstack code | 11:52 |
| dmellado | I see | 11:52 |
| *** britthouser has joined #openstack-neutron | 11:53 | |
| ajo | ihrachyshka: can you check: https://review.openstack.org/#/c/208543/ this one passed check, and has +A, but I don't see it in gate | 11:54 |
| *** britthou_ has joined #openstack-neutron | 11:54 | |
| ajo | may be it's waiting for the depended 211565 | 11:54 |
| scheuran | sc68cal, a little late, but - pong | 11:55 |
| *** dtynan has joined #openstack-neutron | 11:56 | |
| jschwarz | ajo, it is indeed :) | 11:56 |
| *** yfried has quit IRC | 11:56 | |
| *** zhhuabj has joined #openstack-neutron | 11:56 | |
| ajo | ok, ihrachyshka, jschwarz no recheck, all patches were already in gate or check queue | 11:57 |
| ajo | in fact, at the top of the gate, we have 3 in a row | 11:57 |
| jschwarz | ajo. I'm checking with 211565 to see if it does need a recheck (triggered before the devstack revert merged) | 11:57 |
| *** britthouser has quit IRC | 11:57 | |
| ajo | jschwarz, don't do it yet | 11:57 |
| ajo | jschwarz, if it passes, the recheck will make it being recheck | 11:57 |
| jschwarz | ajo, yep. that's why I'm looking at the log | 11:58 |
| jschwarz | ajo, if it fails on the fwaas I'll let you know | 11:58 |
| sc68cal | scheuran: My hope was to keep the linux bridge agent class similar to the OVS agent class, where they inherit the RPC mixins | 11:58 |
| ajo | jschwarz: ack :) | 11:58 |
| sc68cal | scheuran: your patches sort of reverse the change I did in the base patch, where there was an RPC class that inherited the mixins, then an agent class | 11:59 |
| ajo | jschwarz, ihrachyshka : https://review.openstack.org/#/c/211520/ being merged in a second ;) | 11:59 |
| scheuran | sc68cal, yes, right | 11:59 |
| sc68cal | scheuran: there's probably still a lot of goop in the agent class that needs to be extracted into the bridge manager class, but I don't think the RPC stuff should go into the managers | 12:00 |
| scheuran | sc68cal, that'S something that needs to be discussed | 12:00 |
| jschwarz | whoohoo | 12:00 |
| jschwarz | ajo, 211565 started api tests | 12:00 |
| * ajo takes out popcorn | 12:00 | |
| jschwarz | ajo, https://jenkins07.openstack.org/job/gate-neutron-dsvm-api/910/console | 12:00 |
| scheuran | sc68cal, were you able to read my reply on your comment? | 12:01 |
| jschwarz | what was the failing function? | 12:01 |
| scheuran | sc68cal, I tried to explain why it would make sense from my point of view | 12:01 |
| sc68cal | scheuran: yes, I saw, and I do agree, current structure would have lots of delegator methods | 12:02 |
| *** vikram has quit IRC | 12:02 | |
| sc68cal | scheuran: but for now that might be what is required until we can tease out what port_update really entails that is macvtap vs. bridge specific | 12:02 |
| sc68cal | not to mention, OVS specific, going the other way | 12:03 |
| jschwarz | ajo, it passed | 12:03 |
| jschwarz | ihrachyshka, ^ | 12:03 |
| ajo | yeyy | 12:03 |
| sc68cal | scheuran: I'm not sure how reasonable this is or not, but ideally we'd boil out all the common stuff into the linuxbridge agent class, then the bridge manager and macvtap manager just handle bridge and macvtap specific VIF stuff | 12:04 |
| ajo | jschwarz: it sensed the mysql was changed, even if it wasn't for itself ;) | 12:04 |
| jschwarz | XD | 12:04 |
| scheuran | sc68cal, yes. I mean the question is, what is the scope of this refactoring? Is it only to get macvtap integrated - fine | 12:04 |
| ihrachyshka | jschwarz, I guess it depends on when it actually triggers the job | 12:04 |
| ihrachyshka | not just the moment of the queueing | 12:05 |
| scheuran | sc68cal, but if we aim for a larger scope, there might not be that much common code anymore | 12:05 |
| jschwarz | erm | 12:05 |
| jschwarz | ihrachyshka, perhaps abregman can inlighten us ;-) | 12:05 |
| *** nyechiel has quit IRC | 12:05 | |
| *** roeyc has quit IRC | 12:06 | |
| sc68cal | scheuran: At this point, try and A) make LB Agent become more like the OVS agent structure, and B) macvtap driver | 12:06 |
| scheuran | sc68cal, sorry, missinterpreded your statement. I totally agree moving all lb and macvtap common code up to the networkmanager class | 12:06 |
| *** nyechiel has joined #openstack-neutron | 12:06 | |
| ihrachyshka | jschwarz, not sure he is into zuul | 12:06 |
| scheuran | sc68cal, if that's the goal, you're absolutely right. The RPC stuff should stay in the agent itself | 12:07 |
| sc68cal | scheuran: cool. I know it might put a little extra work on you to work within the boundaries, but let's give it a try. If it doesn't work we can revisit | 12:08 |
| *** tellesnobrega_ has joined #openstack-neutron | 12:08 | |
| *** yamamoto has joined #openstack-neutron | 12:09 | |
| scheuran | sc68cal, so I could make a parallel patch set beside my last | 12:10 |
| scheuran | sc68cal, what do you think about the configuration stuff? | 12:10 |
| sc68cal | scheuran: yeah - parallel patch sounds fine to me | 12:10 |
| scheuran | sc68cal, macvtap will have at least 2 new configuration options | 12:10 |
| scheuran | sc68cal, and some of lb might not be required | 12:10 |
| sc68cal | scheuran: config stuff looks good, OVS agent has something similar | 12:11 |
| scheuran | sc68cal, I moved "up" all the stuff as well in the last patchset | 12:11 |
| sc68cal | scheuran: although it's not a class method | 12:11 |
| scheuran | sc68cal, ok, let me have a look at ovs | 12:11 |
| sc68cal | scheuran: I'll fetch line # | 12:11 |
| *** yog_ has quit IRC | 12:12 | |
| *** dave-mccowan has quit IRC | 12:12 | |
| sc68cal | scheuran: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1679 | 12:12 |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: Fix DVR interface delete by port when gateway is set https://review.openstack.org/173001 | 12:12 |
| *** roeyc has joined #openstack-neutron | 12:13 | |
| *** itzikb has joined #openstack-neutron | 12:13 | |
| abregman | jschwarz, ihrachyshka: sorry. not familiar yet enough with upstream stuff =| | 12:13 |
| scheuran | sc68cal, ah ok. So I would use the same name, but move it up to the lb manager instead.. | 12:13 |
| *** norman has joined #openstack-neutron | 12:13 | |
| sc68cal | scheuran: that method is a module level function, so if you want - you could do same in the linux bridge | 12:13 |
| sc68cal | instead of making it a class method of the network managers | 12:14 |
| *** zhhuabj has quit IRC | 12:14 | |
| *** cbouch has joined #openstack-neutron | 12:14 | |
| *** abehl has quit IRC | 12:15 | |
| *** vikram has joined #openstack-neutron | 12:15 | |
| *** tellesnobrega_ has quit IRC | 12:16 | |
| *** anshul has joined #openstack-neutron | 12:17 | |
| scheuran | sc68cal, but that would mean, one method calld get_configurations_lb and one get_configurations_macvtap, right? | 12:17 |
| sc68cal | scheuran: hopefully not | 12:20 |
| openstackgerrit | Andreas Scheuring proposed openstack/neutron: [WIP] Isolated agent loop from linuxbridge code https://review.openstack.org/211637 | 12:20 |
| *** rotbeard has joined #openstack-neutron | 12:20 | |
| scheuran | sc68cal, ok, I could check first the interface-manager configuration and then have a if-else | 12:21 |
| sc68cal | scheuran: ok, I'm willing to wait and see | 12:22 |
| *** pavel_bondar has joined #openstack-neutron | 12:22 | |
| norman | all , any run into the issue that after creating a Fwaas rule with the TCP deny, and then ping is also denied . | 12:23 |
| *** yog_ has joined #openstack-neutron | 12:24 | |
| sc68cal | norman: search for a bug on launchpad? | 12:24 |
| *** yamamoto has quit IRC | 12:25 | |
| scheuran | sc68cal, another question. Is making the lb agent more ovs-like one of your goals for M? Or is it just playing around? | 12:25 |
| norman | sc68cal, i searched today, but did not find | 12:26 |
| sc68cal | scheuran: I think one of the goals for M is to have a modular l2 agent, this WIP is relate | 12:26 |
| norman | sc68cal, I will try it again | 12:26 |
| sc68cal | norman: ok, then file a bug and dump as much information about it as you can | 12:26 |
| sc68cal | norman: and add the tag "fwaas" | 12:27 |
| sc68cal | scheuran: *related | 12:27 |
| *** jpena|lunch is now known as jpena | 12:27 | |
| sc68cal | scheuran: I see in https://github.com/scheuran/networking-macvtap/blob/networking-macvtap-poc/networking_macvtap/plugins/ml2/drivers/macvtap/agent/macvtap_neutron_agent.py - the macvtap agent won't support security groups? | 12:27 |
| scheuran | sc68cal, yes, it's like with sriov agent | 12:27 |
| scheuran | sc68cal, you have to use the noopdriver | 12:28 |
| sc68cal | scheuran: kind of an important feature to omit | 12:28 |
| scheuran | that's because it's a "direct" connection - but built into linux kernel | 12:28 |
| scheuran | sc68cal, yes either or omit or just use the noop driver | 12:29 |
| *** amuller has joined #openstack-neutron | 12:30 | |
| *** bapalm has joined #openstack-neutron | 12:31 | |
| *** erlarese has joined #openstack-neutron | 12:31 | |
| *** shausy has joined #openstack-neutron | 12:32 | |
| *** rossella_s has joined #openstack-neutron | 12:34 | |
| *** zhhuabj has joined #openstack-neutron | 12:35 | |
| sc68cal | i have to refresh my memory on how ml2 handles API extensions and mech drivers that don't support api extensions - like the SG API | 12:35 |
| rossella_s | ajo sorry I had to leave earlier but I read all you wrote! egress/ingress is confusing...point of views :p | 12:38 |
| *** itzikb has quit IRC | 12:38 | |
| ajo | rossella_s, yes... :) even my commit message was a bit confusing now that I reread, but ok :) | 12:38 |
| *** bharath has joined #openstack-neutron | 12:38 | |
| ajo | rossella_s: it depends at which side of the wire you look at | 12:39 |
| moshele | ihrachyshka: hi | 12:39 |
| moshele | ihrachyshka: do we have a gate problem ? | 12:39 |
| ajo | moshele, not anymore for now | 12:39 |
| ihrachyshka | moshele, huh, you missed everything! we had and now it's fixed | 12:39 |
| *** zhipeng has joined #openstack-neutron | 12:40 | |
| ihrachyshka | so we are merging the patches and claim done | 12:40 |
| ihrachyshka | moshele, we had it kinda broken yesterday | 12:40 |
| ihrachyshka | and we recovered an hour ago | 12:40 |
| moshele | ihrachyshka: this still failed https://review.openstack.org/#/c/210483/ | 12:40 |
| rossella_s | ajo yep...you have to remember every time what's the reference point...anyway we can handle that, just two direction is fine :) | 12:40 |
| moshele | ihrachyshka: I need to rebase ? | 12:41 |
| ihrachyshka | moshele, apparently started before fix was in | 12:41 |
| ihrachyshka | moshele, no rebase needed | 12:41 |
| ajo | moshele, no need :) | 12:41 |
| ihrachyshka | moshele, the fix was in devstack-gate | 12:41 |
| *** edmondsw has joined #openstack-neutron | 12:41 | |
| *** nyechiel_ has joined #openstack-neutron | 12:42 | |
| *** nyechiel has quit IRC | 12:42 | |
| jlibosva | ajo: moshele ihrachyshka it's caused by different issue | 12:42 |
| ajo | what caused ihrachyshka ? | 12:42 |
| ajo | it's , not "is", ... my joke was broken | 12:43 |
| ajo | :) | 12:43 |
| *** bharath has quit IRC | 12:43 | |
| *** alexpilotti has quit IRC | 12:44 | |
| ihrachyshka | jlibosva, what is 'it'? | 12:45 |
| *** alexpilotti has joined #openstack-neutron | 12:45 | |
| norman | sc68cal, I read the admin guide, it said that "Fwaas will always add 'deny all' as the lowest one in the policy" , is it true? | 12:46 |
| jlibosva | 'it' is https://bugs.launchpad.net/neutron/+bug/1478847 | 12:47 |
| openstack | Launchpad bug 1478847 in neutron "test_udp_output_port fails with " AssertionError: RuntimeError not raised."" [Undecided,Confirmed] - Assigned to Jakub Libosvar (libosvar) | 12:47 |
| norman | sc68cal, if so , then do you think that I don't need to add the 'Deny' rules, I only need/have to add the 'Allow' rule when apply the firewall/policy to a router? | 12:48 |
| amuller | jlibosva: what is the issue then? :) | 12:49 |
| jlibosva | amuller: I *think* there *might* be a bug :) | 12:50 |
| jlibosva | or race! | 12:50 |
| jlibosva | amuller: I really don't know yet :) | 12:50 |
| amuller | jlibosva: Yay testing | 12:50 |
| *** busterswt has joined #openstack-neutron | 12:50 | |
| HenryG | ihrachyshka: sorry I missed how the gate got fixed? | 12:50 |
| ihrachyshka | HenryG, devstack-gate cleanup reverted: https://review.openstack.org/211853 | 12:51 |
| openstackgerrit | Marga Millet proposed openstack/neutron: Allow host route injection of metadata server IP https://review.openstack.org/211963 | 12:51 |
| *** yog_ has quit IRC | 12:51 | |
| *** yamamoto has joined #openstack-neutron | 12:51 | |
| *** azbiswas has joined #openstack-neutron | 12:52 | |
| *** gberginc has quit IRC | 12:53 | |
| *** irenab has quit IRC | 12:53 | |
| *** yamamoto has quit IRC | 12:54 | |
| ajo | HenryG, ihrachyshka , I believe a good strategy could be, disabling the specific test which is causing trouble, | 12:56 |
| ajo | and then revert the revert ? | 12:57 |
| ajo | so we don't let any other races come in because of the mysql driver switch | 12:57 |
| *** merooney has joined #openstack-neutron | 12:57 | |
| ihrachyshka | ajo, I guess yes. then we are able to test on a patch that reverts it however we want | 12:57 |
| ihrachyshka | good idea | 12:57 |
| HenryG | ajo: ihrachyshka: agree. I thought we had switch the mysql driver a long time ago | 12:58 |
| HenryG | too many knobs and switches in the gate :( | 12:58 |
| *** azbiswas has quit IRC | 12:58 | |
| *** mohankumar has quit IRC | 12:58 | |
| ihrachyshka | HenryG, not for default jobs. we switched but there were issues so an unstable job with new driver was introduced, but default stayed on the old on | 12:58 |
| *** josecastroleon has quit IRC | 12:59 | |
| amuller | ihrachyshka: HenryG: Can we merge this https://review.openstack.org/#/c/211979/ then revert the revert in devstack? | 13:00 |
| *** Aish has joined #openstack-neutron | 13:00 | |
| *** Aish has quit IRC | 13:00 | |
| amuller | ihrachyshka: HenryG: Also I would recommend we do a partial revert-revert by first enabling multiple API workers | 13:00 |
| amuller | I don't know why those are tied | 13:00 |
| amuller | api_workers and the mysql driver | 13:00 |
| ihrachyshka | agreed on partial revert | 13:00 |
| ihrachyshka | amuller, also note that the original patch we revert does not clean up all the UNSTABLE stuff from the repo | 13:01 |
| HenryG | amuller: agreed | 13:01 |
| ihrachyshka | amuller, +A the skip | 13:02 |
| ihrachyshka | amuller, but please don't unrevert now | 13:03 |
| ihrachyshka | amuller, we don't have the skip in feature branches | 13:03 |
| *** josecastroleon has joined #openstack-neutron | 13:04 | |
| sc68cal | norman: not sure -depends on also what security groups you have applied to a VM | 13:04 |
| jschwarz | ihrachyshka, ajo, 211565 passed the gate! :) | 13:04 |
| *** zhhuabj has quit IRC | 13:04 | |
| sc68cal | norman: but yes if the policy is to deny all by default then you'll need to add a rule that allows it in, in addition to a security group that allows it | 13:04 |
| sc68cal | norman: for icmp | 13:05 |
| *** roeyc has quit IRC | 13:05 | |
| *** sudipto_ has joined #openstack-neutron | 13:05 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add configurable options for HA networks https://review.openstack.org/212017 | 13:06 |
| jschwarz | amuller, ^ | 13:06 |
| amuller | jschwarz: yay | 13:06 |
| *** jecarey has joined #openstack-neutron | 13:07 | |
| *** shwetaap has joined #openstack-neutron | 13:08 | |
| norman | sc68cal, my security group for the VM is allow all icmp/tcp for ingress/engress, I don't quite understand the relationship between firewall and security group | 13:09 |
| sc68cal | norman: fwaas filters at the router - security group filters at the VM port | 13:09 |
| sc68cal | norman: so, unless I am mistaken you need to have harmony betwee your security groups and firewall policies, to allow traffic in from external | 13:10 |
| *** lyanchih has joined #openstack-neutron | 13:10 | |
| norman | sc68cal, yes, I know that , I just curious about the why fwaas depends on the security group policy | 13:10 |
| *** srilatha has joined #openstack-neutron | 13:10 | |
| sc68cal | norman: because if you have a rule in fwaas that allows traffic in, but no SG rule that allows it in it'll get dropped at the hypervisor | 13:10 |
| *** ferest has joined #openstack-neutron | 13:11 | |
| *** roeyc has joined #openstack-neutron | 13:11 | |
| *** srilatha has quit IRC | 13:12 | |
| *** britthou_ has quit IRC | 13:14 | |
| *** nyechiel_ has quit IRC | 13:14 | |
| norman | sc68cal, Ok, then for my case, I all all traffic in the SG, and I want to just deny some IP/port as destination, then when I add it as the rule, it deny all, is it reasonable? | 13:14 |
| *** ferest has quit IRC | 13:14 | |
| norman | sc68cal, '.. I allow all traffic ...." | 13:14 |
| *** akamyshnikova_ has quit IRC | 13:14 | |
| *** ilyashakhat has quit IRC | 13:14 | |
| *** sbelous has quit IRC | 13:15 | |
| *** skraynev has quit IRC | 13:15 | |
| *** obondarev has quit IRC | 13:15 | |
| *** eezhova has quit IRC | 13:15 | |
| sc68cal | norman: that's one way to do it, I think | 13:15 |
| sc68cal | if I understand you correctly | 13:16 |
| *** apuimedo has quit IRC | 13:16 | |
| *** kbringard has joined #openstack-neutron | 13:16 | |
| *** zhhuabj has joined #openstack-neutron | 13:17 | |
| norman | sc68cal, actually , I add the 'TCP deny 192.168.1.2/32' to the firewall with allowing all traffic in the SG, but with the testing result , the firewall deny All, so I have to add ALL the other allow rules explicitly. | 13:17 |
| *** cbouch_ has joined #openstack-neutron | 13:18 | |
| *** mwagner_lap has joined #openstack-neutron | 13:19 | |
| ajo | vikram, ihrachyshka , jlibosva , jschwarz | 13:19 |
| ajo | $ neutron qos-bandwidth-limit-rule-update 3b1d1444-8807-4c4a-acaf-13bf96cdb6bf bw-limiter --max_kbps 3000 --max_burst 0 | 13:19 |
| ajo | Invalid input for operation: 'name' is not supported for filtering. | 13:19 |
| ajo | I'm getting this now on latest neutron-client | 13:19 |
| ajo | am I doing something wrong? | 13:19 |
| jlibosva | ajo: neutron --debug | 13:19 |
| ajo | jlibosva, jschwarz , vikram : http://ur1.ca/nf2k2 | 13:20 |
| sc68cal | norman: 192.168.1.2/32 as the source ip or destination | 13:20 |
| jlibosva | ajo: so, it didn't find policy with id 3b1d1444-8807-4c4a-acaf-13bf96cdb6bf | 13:21 |
| *** cbouch has quit IRC | 13:21 | |
| jlibosva | ajo: and we don't support filters yet I think | 13:21 |
| amuller | jlibosva: didn't nmagnezi fix that in his patch? | 13:21 |
| jlibosva | ajo: like http://10.211.55.103:9696/v2.0/qos/policies/ac3ca837-6981-4e4f-9f57-6b2b1730444f/bandwidth_limit_rules.json?fields=id&name=3b1d1444-8807-4c4a-acaf-13bf96cdb6bf | 13:21 |
| ajo | jlibosva, we do, it was working before | 13:21 |
| *** irenab has joined #openstack-neutron | 13:21 | |
| jlibosva | then ignore me | 13:21 |
| * jlibosva hides | 13:21 | |
| ajo | :D | 13:21 |
| norman | sc68cal, destination | 13:22 |
| amuller | ajo: jlibosva: https://review.openstack.org/#/c/208201/ | 13:22 |
| ajo | hmm, it's not finding the rule in the policy hmm | 13:22 |
| *** ilyashakhat has joined #openstack-neutron | 13:23 | |
| *** eezhova has joined #openstack-neutron | 13:23 | |
| *** akamyshnikova has joined #openstack-neutron | 13:23 | |
| *** skraynev has joined #openstack-neutron | 13:23 | |
| ajo | amuller, jlibosva , ignoreme | 13:23 |
| ajo | I was using wrong rule id... | 13:23 |
| * ajo hides... | 13:23 | |
| *** lyanchih has quit IRC | 13:23 | |
| ajo | but, that error is not very consistent on neutronclient | 13:23 |
| *** sbelous has joined #openstack-neutron | 13:24 | |
| *** edmondsw has quit IRC | 13:24 | |
| vikram | ajo:I cannot find "bw-limiter" in the review link | 13:24 |
| ajo | still failing with right id.. | 13:24 |
| *** roeyc has quit IRC | 13:24 | |
| ajo | vikram: you're trying to filter a bw rule by name | 13:25 |
| ajo | rules don't have names | 13:25 |
| ajo | have ids ;) | 13:25 |
| ajo | GET http://10.211.55.103:9696/v2.0/qos/policies/ac3ca837-6981-4e4f-9f57-6b2b1730444f/bandwidth_limit_rules.json?fields=id&name=8a9e30ea-fb3e-413c-b725-7b83b7b6c2e6 | 13:25 |
| *** ujjain has quit IRC | 13:25 | |
| ajo | or trying to translate "name to uuid" i guess.. | 13:25 |
| *** yamamoto has joined #openstack-neutron | 13:25 | |
| *** mohankumar has joined #openstack-neutron | 13:26 | |
| vikram | check the policy and rule for their positional value | 13:26 |
| ihrachyshka | ajo, so is there an issue? | 13:26 |
| ihrachyshka | yeah, the order | 13:26 |
| jlibosva | ajo: yeah, well that's what I said - it didn't find the id :) | 13:26 |
| vikram | ajo: check the order once | 13:26 |
| jlibosva | but I thought the error would be different | 13:27 |
| ihrachyshka | hm, why does it send name=id?.. | 13:27 |
| vikram | ajo: I think it should be "neutron qos-bandwidth-limit-rule-update bw-limiter 3b1d1444-8807-4c4a-acaf-13bf96cdb6bf --max_kbps 3000 --max_burst 0" | 13:27 |
| ihrachyshka | it should not do it that way. it should ask for id maybe (but then, why doesn't it show the rule instead of filtering?) | 13:28 |
| ihrachyshka | it seems like a bug in the client, or in your CLI | 13:28 |
| ihrachyshka | server is of no fault here | 13:28 |
| *** roeyc has joined #openstack-neutron | 13:28 | |
| *** zhhuabj has quit IRC | 13:31 | |
| *** tiswanso has joined #openstack-neutron | 13:32 | |
| ihrachyshka | ajo, ^^ | 13:32 |
| *** lindycoder has joined #openstack-neutron | 13:33 | |
| *** norman has quit IRC | 13:33 | |
| *** zhhuabj has joined #openstack-neutron | 13:33 | |
| *** lyanchih has joined #openstack-neutron | 13:33 | |
| ajo | correct | 13:35 |
| ajo | it's a CLI thing | 13:36 |
| *** zzzeek has joined #openstack-neutron | 13:36 | |
| ajo | vikram, client says the opposite on help | 13:36 |
| ajo | says, first rule id, then policy | 13:36 |
| ajo | ok, we need to polish that on the client | 13:36 |
| vikram | you mean the later worked | 13:36 |
| *** breitz1 has quit IRC | 13:36 | |
| jschwarz | ajo, looking | 13:36 |
| ajo | $ neutron qos-bandwidth-limit-rule-update bw-limiter 8a9e30ea-fb3e-413c-b725-7b83b7b6c2e6 --max_kbps 3000 --max_burst 0 | 13:37 |
| ajo | Unable to find policy with name '8a9e30ea-fb3e-413c-b725-7b83b7b6c2e6' | 13:37 |
| *** crose has joined #openstack-neutron | 13:37 | |
| ajo | vikram, nope ^ | 13:37 |
| ajo | jschwarz, thanks :) | 13:37 |
| jschwarz | ajo, policy is first | 13:37 |
| jschwarz | or was it second? | 13:37 |
| jschwarz | lol, i'm not helpful at all. | 13:37 |
| jschwarz | sec | 13:37 |
| ajo | jschwarz help says second | 13:37 |
| ajo | but neither works | 13:37 |
| vikram | :( | 13:37 |
| ajo | it used to work, but I was trying my "test instructions" | 13:37 |
| ajo | and.. | 13:37 |
| ajo | after updating neutronclient stumped into this | 13:37 |
| vikram | ajo: "Unable to find policy with name '8a9e30ea-fb3e-413c-b725-7b83b7b6c2e6'" says policy should be the second one | 13:38 |
| jschwarz | ajo, the name thing might have been caused by https://review.openstack.org/#/c/208201/ | 13:38 |
| *** kawa2014 has joined #openstack-neutron | 13:38 | |
| jschwarz | ajo, actually it's not... neutronclient is supposed to know to use the correct field | 13:39 |
| ajo | jschwarz: nope | 13:39 |
| ajo | jschwarz: it's neutronclient trying to query rules by name, and rules don't have names :D | 13:39 |
| jschwarz | ajo, ah! rules don't have names so that's why it shouldn't try the name at all. | 13:39 |
| jschwarz | XD | 13:39 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Support for independent alembic branches in sub-projects https://review.openstack.org/198524 | 13:40 |
| jschwarz | ajo, 'neutron qos-policy-list'? | 13:40 |
| ajo | yes, that works | 13:41 |
| jschwarz | ajo, can you pastebin the output please? | 13:41 |
| ajo | http://ur1.ca/nf2p8 | 13:41 |
| jschwarz | erm that's weird | 13:42 |
| jschwarz | let me try this on my setup.. i'll get back to you | 13:42 |
| *** jaypipes has joined #openstack-neutron | 13:42 | |
| *** dane_leblanc has joined #openstack-neutron | 13:43 | |
| ajo | thanks jschwarz | 13:44 |
| *** kawa2014 has quit IRC | 13:44 | |
| *** pradk has joined #openstack-neutron | 13:44 | |
| *** pradk has quit IRC | 13:44 | |
| jschwarz | ajo, reproduced locally, will look for a solution now | 13:46 |
| *** mkolesni has quit IRC | 13:46 | |
| ajo | jschwarz++ | 13:46 |
| ajo | I was looking but I didn't find what's the reason yet, also, I'm not feeling well, I believe my lunch was cursed ;) | 13:47 |
| *** johnbelamaric has joined #openstack-neutron | 13:47 | |
| jschwarz | ajo, btw it's max_burst_kbps | 13:48 |
| jschwarz | not max_burst | 13:48 |
| ajo | :) | 13:48 |
| *** ekuris has quit IRC | 13:49 | |
| jschwarz | ajo, pastebin ' | 13:49 |
| jschwarz | ajo, pastebin 'neutron qos-bandwidth-limit-rule-list bw-limiter' for me please | 13:49 |
| jschwarz | I suspect your rule id is wrong | 13:49 |
| ajo | http://ur1.ca/nf2rp | 13:50 |
| ajo | a weird history about tenants? | 13:50 |
| jschwarz | errr that's a policy error | 13:50 |
| ajo | http://ur1.ca/nf2rx | 13:50 |
| jschwarz | policy.json, that is | 13:50 |
| ajo | ok, but that's a different issue | 13:50 |
| ajo | there you have the rule id | 13:50 |
| ajo | In my original paste rule id was wrong | 13:51 |
| ajo | I tried the good one with same result | 13:51 |
| ajo | let me rerun & fpaste | 13:51 |
| *** links has quit IRC | 13:51 | |
| ajo | jschwarz: http://ur1.ca/nf2s6 | 13:52 |
| *** edmondsw has joined #openstack-neutron | 13:52 | |
| jschwarz | ajo, right so your policy.json is not updated | 13:52 |
| jschwarz | see like 13 where it gives that error | 13:53 |
| *** breitz has joined #openstack-neutron | 13:53 | |
| ajo | hmmm | 13:53 |
| ajo | let me update policy | 13:53 |
| jschwarz | if it would have been correct (or if the tenant_id would have been sent?) then that would have returned an OK response and wouldn't have gone to line 15 | 13:53 |
| *** Piet has quit IRC | 13:54 | |
| jschwarz | might be that the tenant_id isn't sent? | 13:54 |
| *** yapeng has joined #openstack-neutron | 13:54 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: [neutron-db-manage] check_migration: validate labels https://review.openstack.org/206746 | 13:56 |
| *** annegentle has joined #openstack-neutron | 13:56 | |
| *** zhhuabj has quit IRC | 13:56 | |
| *** zhhuabj has joined #openstack-neutron | 13:57 | |
| *** jistr is now known as jistr|mtg | 13:57 | |
| *** kawa2014 has joined #openstack-neutron | 13:58 | |
| jschwarz | ajo, any luck? | 13:58 |
| ihrachyshka | jlibosva, jschwarz, ajo, moshele, gampel, gsagie, irenab, vikram, vichoward, Ramanjaneya: meeting time /join #openstack-meeting-3 | 14:00 |
| *** sridhar_ram has joined #openstack-neutron | 14:00 | |
| ajo | jschwarz: same result | 14:00 |
| jschwarz | ajo, perhaps your keystonerc needs updating :S | 14:00 |
| *** sdake_ has joined #openstack-neutron | 14:00 | |
| ajo | hmm, it could be I'm trying to do it as a tenant | 14:00 |
| ajo | and not as admin | 14:01 |
| jschwarz | ajo, either way that problem sounds like a setup problem and not a neutron problem (other than the name-filter-thingy which I'm looking into now) | 14:01 |
| jschwarz | ajo, try switching to admin | 14:01 |
| *** dave-mccowan has joined #openstack-neutron | 14:01 | |
| *** rossella_ has joined #openstack-neutron | 14:01 | |
| *** Aish has joined #openstack-neutron | 14:01 | |
| *** Aish has quit IRC | 14:01 | |
| ajo | jschwarz: it works as admin | 14:01 |
| jschwarz | ajo, so there. | 14:02 |
| ajo | probably we need to fix the policies | 14:02 |
| jschwarz | ajo, is that what we want? | 14:02 |
| ajo | btw, we shall not allow policy creation or updates from tenants at this point | 14:02 |
| jschwarz | lets take this to the meeting and then everyone can pitch in | 14:02 |
| ajo | and it did allow me to create policy :) | 14:02 |
| ajo | yep | 14:02 |
| *** rossella_s has quit IRC | 14:02 | |
| *** zhhuabj has quit IRC | 14:02 | |
| *** ajmiller has quit IRC | 14:03 | |
| *** zhhuabj has joined #openstack-neutron | 14:03 | |
| *** ushkalim has quit IRC | 14:03 | |
| *** panni has joined #openstack-neutron | 14:03 | |
| *** nyechiel_ has joined #openstack-neutron | 14:04 | |
| *** absubram has joined #openstack-neutron | 14:04 | |
| *** keico has joined #openstack-neutron | 14:05 | |
| *** regXboi has joined #openstack-neutron | 14:06 | |
| *** lajos-katona has quit IRC | 14:06 | |
| *** mohankumar has quit IRC | 14:09 | |
| *** keico_ has joined #openstack-neutron | 14:09 | |
| *** Ramanjaneya has quit IRC | 14:11 | |
| *** rushil has joined #openstack-neutron | 14:12 | |
| *** sudipto_ has quit IRC | 14:12 | |
| *** abregman is now known as abregman|brb | 14:12 | |
| *** vikram_ has joined #openstack-neutron | 14:12 | |
| *** panni has quit IRC | 14:12 | |
| regXboi | armax: ping | 14:13 |
| *** yamahata has joined #openstack-neutron | 14:14 | |
| *** thumpba has joined #openstack-neutron | 14:15 | |
| *** keico has left #openstack-neutron | 14:15 | |
| *** zhipeng has quit IRC | 14:15 | |
| *** vikram_ has quit IRC | 14:16 | |
| *** jbreader has joined #openstack-neutron | 14:17 | |
| *** shausy has quit IRC | 14:17 | |
| *** banix has joined #openstack-neutron | 14:17 | |
| *** shausy has joined #openstack-neutron | 14:17 | |
| *** jbreader has quit IRC | 14:18 | |
| amuller | pc_m: morning | 14:18 |
| pc_m | amuller: hi | 14:18 |
| amuller | pc_m: can I get an update on unborking the neutron-client gate? | 14:19 |
| pc_m | amuller: Well... it needs to specify the new VPN devstack plugin. To do that, both of the commits that I proposed, need to be approved by infra and neutron. | 14:20 |
| pc_m | amuller: I think infra is still deciding on how to resolve the bigger issue of stable/kilo breakage. | 14:21 |
| *** britthouser has joined #openstack-neutron | 14:21 | |
| amuller | pc_m: how come we didn't immediately revert the breaking change then with stable gates come up with a plan? | 14:21 |
| amuller | this is incredibly costly | 14:21 |
| pc_m | amuller: https://review.openstack.org/#/c/209887/ and https://review.openstack.org/#/c/210021/1 | 14:22 |
| *** lyanchih has quit IRC | 14:23 | |
| pc_m | amuller: There is an etherpad on this. It would require 4 reverts, as doing one for stable/kilo would break other commits. | 14:23 |
| amuller | pc_m: can you link the etherpad? | 14:23 |
| pc_m | amuller: I think infra was opposed to reverting (and later unreverting) the patches. | 14:23 |
| *** helenxu has joined #openstack-neutron | 14:24 | |
| regXboi | obondarev: ping? | 14:24 |
| pc_m | amuller: In the interim, I've been studying the issue and have formed some proposals on the etherpad. I'm working locally to implement option C, in case it is selected by infra (I think it is best). | 14:24 |
| keico_ | Hi everyone. I want to know if it's possible to set up a GRE tunnel from an Openstack tenant VM (linux based) to a bare-metal machine running outside the Openstack cloud. The two machines are networking connected to the public network and can ping each other. | 14:24 |
| pc_m | amuller: We could move forward with the two neutronclient commits, provided infra wants to allow the change to project-config. | 14:25 |
| pc_m | amuller: https://etherpad.openstack.org/p/vpn-test-changes | 14:25 |
| pc_m | amuller: So I think option C is the right thing to do... but there is likely some lack of confidence in hearing my opinion at this point. :( | 14:26 |
| ihrachyshka | amuller, you may want to talk to dougwig also since there was a discussion between anteaya and Doug on how we fix gate and re: some plan that neutron should come up with | 14:26 |
| *** bharath has joined #openstack-neutron | 14:27 | |
| pc_m | ihrachyshka: Some of that is on the etherpad. Like I said, I've proposed 3 solutions on the pad, and think people should review them for accuracy. | 14:27 |
| pc_m | think people -> encourage people | 14:27 |
| regXboi | carl_baldwin: ping? | 14:28 |
| *** carl_baldwin has joined #openstack-neutron | 14:28 | |
| *** cristicalin has joined #openstack-neutron | 14:30 | |
| ajo | ping irenab :) | 14:30 |
| irenab | ajo: there is an option to set qos requirements for nova flavor in the following way: nova-manage flavor set_key --name m1.small  --key quota:vif_inbound_average --value 10240 | 14:30 |
| ajo | yes | 14:31 |
| irenab | ajo: I think we should consider if we can integrate this with neutron-qos | 14:31 |
| jschwarz | pc_m, thanks a lot for working on this, looks like there's quite a bit of thought on how to go about fixing it | 14:31 |
| ajo | irenab, we will have to look at that, yes | 14:32 |
| ajo | irenab, but more like: | 14:32 |
| jschwarz | pc_m, did you consider, for the mean time, commenting out the failing vpn-functional tests in the neutronclient repo so it won't fail? | 14:32 |
| *** bharath has quit IRC | 14:32 | |
| pc_m | jschwarz: yw. Been digging into it for days (out of guild for making a mistake). | 14:32 |
| ajo | nova-manage flavor set_key --name m1.small --key qos:neutron_qos_policy <policy-name-or-id> | 14:32 |
| ajo | or similar | 14:32 |
| irenab | ajo: great, maybe for next week, lets include it in agenda of the qos-meeting, I will start etherpad to put ideas there | 14:32 |
| ajo | otherwise we make nova responsible to create a profile, etc... | 14:32 |
| jschwarz | pc_m, no worries, they happen. lets focus on making it right now ;-) | 14:32 |
| *** SridharG has left #openstack-neutron | 14:32 | |
| *** ildikov has quit IRC | 14:33 | |
| ajo | %s/profile/policy/g | 14:33 |
| *** yamahata has quit IRC | 14:33 | |
| irenab | ajo: do you know who support the current nova qos? | 14:33 |
| pc_m | jschwarz: no I did not. Could comment out tests, move job to non-voting, or add the plugin enable. | 14:33 |
| ajo | irenab: yes, that sounds good, but nova spec process is closed for this cycle already | 14:33 |
| ajo | irenab, it will reopen for M, | 14:33 |
| ajo | I guess we can push a spec and get some support to do it by M | 14:33 |
| jschwarz | pc_m, sounds like the last 2 depend on -infra, which we'd like to avoid usually | 14:34 |
| ajo | irenab: I also believe we should work on making the nova scheduler aware of our policies, to avoid over-comitting when we put guarantees | 14:34 |
| jschwarz | commenting out the tests is local to -neutron and once the fix is up we can have something that reverts the fix and check that it works in the mean time. what do you think? | 14:34 |
| pc_m | jschwarz: yeah, though we need to add the plugin eventually, so last one is really the end goal | 14:34 |
| pc_m | jschwarz: you could. | 14:35 |
| *** Murali_ has quit IRC | 14:35 | |
| jschwarz | pc_m, I agree that eventually the plugin should be added | 14:35 |
| irenab | ajo: generally agree, but this pieces are ortogonal. One is per flavor qos requirements, second is qos-aware nova scheduler | 14:36 |
| jschwarz | pc_m, but until then we can not block some patches :) | 14:36 |
| jschwarz | pc_m, I'll work on a commenting patch and add you as a reviewer | 14:36 |
| pc_m | jschwarz: I changed 209887 to no longer be WIP. Once (if?) that upstreams, then 210021 can be rechecked and go through. | 14:36 |
| pc_m | jschwarz: Sure. thanks. Need to keep things going. | 14:37 |
| ajo | irenab: correct | 14:37 |
| *** jistr|mtg is now known as jistr | 14:38 | |
| regXboi | armax: ping? | 14:38 |
| *** ajmiller has joined #openstack-neutron | 14:38 | |
| *** bharath has joined #openstack-neutron | 14:39 | |
| *** banix has quit IRC | 14:39 | |
| *** thedodd has joined #openstack-neutron | 14:39 | |
| irenab | ajo: cool, lets start discussion next week | 14:40 |
| *** roeyc has quit IRC | 14:40 | |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Disable failing vpn tests https://review.openstack.org/212049 | 14:41 |
| jschwarz | pc_m, ihrachyshka, amuller, ^ | 14:41 |
| *** annegentle has quit IRC | 14:41 | |
| ihrachyshka | jschwarz, good move | 14:41 |
| jschwarz | ihrachyshka, which cores can +2 patches on neutronclient? | 14:42 |
| jschwarz | same ones as neutron>? | 14:42 |
| amuller | jschwarz: there's probably a skip decorator you can use instead of comments? | 14:42 |
| amuller | with a link to a bug | 14:42 |
| jschwarz | amuller, probably. will use that instead | 14:42 |
| amuller | jschwarz: yeah it's same cores | 14:42 |
| jschwarz | do we have a launchpad bug for this? pc_m | 14:42 |
| *** roeyc has joined #openstack-neutron | 14:43 | |
| amuller | jschwarz: please check real quick if we have some sort of nice decorator for this in client repo and ihar and I can ninja merge it | 14:43 |
| pc_m | jschwarz: not sure if there is a bug created. | 14:43 |
| jschwarz | amuller, looking | 14:43 |
| amuller | pc_m: I'll file one | 14:44 |
| *** afazekas has quit IRC | 14:44 | |
| jschwarz | amuller, link me once you do. found "@unittest.skip(...)" | 14:44 |
| amuller | jschwarz: pc_m: https://bugs.launchpad.net/neutron/+bug/1484148 | 14:45 |
| openstack | Launchpad bug 1484148 in neutron "stable/kilo and neutronclient gates broken following VPNaaS infra changes" [Critical,Confirmed] - Assigned to Paul Michali (pcm) | 14:45 |
| *** Piet has joined #openstack-neutron | 14:45 | |
| *** abregman|brb is now known as abregman | 14:46 | |
| pc_m | amuller: I think there is one for stable/kilo breakage. | 14:46 |
| jschwarz | pc_m, amuller, https://bugs.launchpad.net/neutron/+bug/1473194 ? | 14:47 |
| openstack | Launchpad bug 1473194 in neutron "Grenade tests fail for *aaS migrations" [Undecided,In progress] - Assigned to Paul Michali (pcm) | 14:47 |
| jschwarz | or is that not related? | 14:48 |
| dougwig | amuller: the reverts were blocked by qa. They are some fixes out, and I'm also in favor of your client skips. pc_m , please hold off on the etherpad stuff for now. | 14:48 |
| pc_m | jschwarz: That's different. | 14:48 |
| amuller | dougwig: do we have a short term plan to unbork stable/kilo? | 14:48 |
| amuller | dougwig: now that client is covered | 14:49 |
| jschwarz | ack | 14:49 |
| *** iymht has joined #openstack-neutron | 14:49 | |
| dougwig | amuller: sort of. Sec, landing. | 14:49 |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Disable failing vpn tests https://review.openstack.org/212049 | 14:50 |
| jschwarz | amuller, ihrachyshka, checked that it skips locally so should do the trick: ^ | 14:50 |
| amuller | +2'd | 14:51 |
| *** aranjan has joined #openstack-neutron | 14:51 | |
| *** dannywilson has joined #openstack-neutron | 14:51 | |
| dougwig | I have two fixes for stable. Let me unboard and will upload. | 14:52 |
| amuller | dougwig++ | 14:52 |
| *** merooney has quit IRC | 14:52 | |
| *** jprovazn has quit IRC | 14:55 | |
| *** roeyc has quit IRC | 14:55 | |
| *** minwang2 has joined #openstack-neutron | 14:55 | |
| *** aranjan has quit IRC | 14:57 | |
| *** roeyc has joined #openstack-neutron | 14:57 | |
| ihrachyshka | ajo, moshele we have a problem with sr-iov patch: http://logs.openstack.org/83/210483/11/check/gate-neutron-dsvm-functional/c4ecf1c/testr_results.html.gz | 14:58 |
| openstackgerrit | Merged openstack/neutron: Add Kuryr to sub_projects.rst https://review.openstack.org/209361 | 14:58 |
| *** merooney has joined #openstack-neutron | 14:59 | |
| *** cbouch_ has quit IRC | 14:59 | |
| moshele | ihrachyshka: but the failure is in ovs | 14:59 |
| *** cbouch_ has joined #openstack-neutron | 14:59 | |
| *** patrickeast has joined #openstack-neutron | 15:00 | |
| moshele | ihrachyshka: maybe we should rebase ? | 15:00 |
| jschwarz | gah ihrachyshka | 15:00 |
| jschwarz | it works in unittest-not-2 | 15:00 |
| jschwarz | why do we need unittest2? | 15:01 |
| *** rkukura has joined #openstack-neutron | 15:01 | |
| ihrachyshka | jschwarz, I don't know, ask qa people :) | 15:01 |
| *** piyanai has joined #openstack-neutron | 15:01 | |
| ihrachyshka | moshele, I want ajo to check it | 15:01 |
| jschwarz | right... i'll resubmit it | 15:01 |
| ihrachyshka | moshele, maybe rebase applies, but I don't know | 15:01 |
| *** iymht has quit IRC | 15:02 | |
| moshele | ihrachyshka: the gate doesn't test SR-IOV so the code I change is not even running | 15:02 |
| ihrachyshka | moshele, I know but we need to pass gate ;) | 15:03 |
| *** alexpilotti has quit IRC | 15:03 | |
| *** banix has joined #openstack-neutron | 15:03 | |
| ihrachyshka | moshele, ok, I'll take a look now myself | 15:03 |
| pc_m | amuller: 1484148 should probably just talk about neutron-client. | 15:03 |
| amuller | pc_m: why is that? | 15:04 |
| openstackgerrit | John Davidge proposed openstack/neutron: Fix update_subnet for prefix delegation https://review.openstack.org/209978 | 15:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: Service plugin for trunk ports https://review.openstack.org/210311 | 15:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: DB model for trunk ports https://review.openstack.org/210310 | 15:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: API extension https://review.openstack.org/210309 | 15:04 |
| openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: ML2 extension driver for subports https://review.openstack.org/210313 | 15:05 |
| openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: DB model for subports https://review.openstack.org/210312 | 15:05 |
| *** sridhar_ram has quit IRC | 15:05 | |
| pc_m | amuller: becasue I think there is already a bug about the gate. | 15:05 |
| amuller | pc_m: ok, can you link it? | 15:05 |
| pc_m | stable/kilo | 15:05 |
| *** helenxu_ has joined #openstack-neutron | 15:05 | |
| *** helenxu has quit IRC | 15:05 | |
| pc_m | amuller: https://bugs.launchpad.net/neutron/+bug/1483266 | 15:05 |
| openstack | Launchpad bug 1483266 in neutron "q-svc fails to start in kilo due to "ImportError: No module named neutron_vpnaas.services.vpn.service_drivers.ipsec"" [High,Confirmed] - Assigned to Doug Wiegley (dougwig) | 15:05 |
| *** puck has quit IRC | 15:05 | |
| *** rm_work has quit IRC | 15:05 | |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Disable failing vpn tests https://review.openstack.org/212049 | 15:06 |
| jschwarz | amuller, ihrachyshka, unittest2 ^ | 15:06 |
| amuller | pc_m: thanks | 15:06 |
| *** jckasper has quit IRC | 15:06 | |
| *** ronis_ has quit IRC | 15:06 | |
| *** gus has quit IRC | 15:06 | |
| *** abregman has quit IRC | 15:08 | |
| *** gus has joined #openstack-neutron | 15:08 | |
| *** dave-mccowan has quit IRC | 15:09 | |
| *** rm_work has joined #openstack-neutron | 15:10 | |
| *** rm_work has quit IRC | 15:10 | |
| *** rm_work has joined #openstack-neutron | 15:10 | |
| *** puck has joined #openstack-neutron | 15:11 | |
| *** lyanchih has joined #openstack-neutron | 15:12 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Improve callback registry devref documentation and usability https://review.openstack.org/211798 | 15:12 |
| *** mlavalle has joined #openstack-neutron | 15:13 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent add functional tests of OVS status https://review.openstack.org/205442 | 15:14 |
| jschwarz | ajo, SYN | 15:20 |
| jschwarz | $ neutron qos-bandwidth-limit-rule-update ad644278-d38d-4c17-8da1-5cf8eed2730f bw-limiter --max_kbps 3000 --max_burst 0 | 15:20 |
| jschwarz | Unable to find bandwidth_limit_rule with id 'ad644278-d38d-4c17-8da1-5cf8eed2730f' | 15:20 |
| jschwarz | (fixed) | 15:20 |
| *** pc_m is now known as pc_m_AWAY | 15:22 | |
| *** dguerri is now known as dguerri` | 15:23 | |
| *** rushil has quit IRC | 15:23 | |
| *** ddepaoli has quit IRC | 15:23 | |
| scheuran | sc68cal, I need an opinion regarding configuration parameters | 15:24 |
| scheuran | sc68cal, I tried to refactor it in the ovs way | 15:25 |
| sc68cal | scheuran: oh? | 15:25 |
| scheuran | sc68cal, first creating a agent_config map containing all parameters, an then extract it and pass it to the Agent | 15:25 |
| scheuran | sc68cal, meaning Agent constructor takes about 10-20 args... | 15:26 |
| scheuran | sc68cal, but a couple of them are only specific to lb, others to other potential users | 15:26 |
| *** shausy has quit IRC | 15:27 | |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2). https://review.openstack.org/198277 | 15:27 |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2). https://review.openstack.org/189655 | 15:27 |
| jschwarz | ihrachyshka, irenab, moshele, ajo, ^ - that's the neutronclient change re: qos-policy-show we discussed the other day, and the bug ajo found :) | 15:27 |
| jschwarz | ihrachyshka, it's also based on the disabled checks so it's bound to go through jenkins :) | 15:27 |
| *** tiswanso has quit IRC | 15:27 | |
| sc68cal | scheuran: pass it in as kwargs like https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L1736 | 15:27 |
| *** merooney has quit IRC | 15:28 | |
| scheuran | sc68cal, this line is extracting it | 15:28 |
| *** annegentle has joined #openstack-neutron | 15:28 | |
| scheuran | sc68cal, see https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L130 | 15:29 |
| sc68cal | scheuran: yeah, I know OVS agent has a big constructor | 15:29 |
| sc68cal | scheuran: my thought is give it a try, if it's too ugly we'll think of something else | 15:29 |
| sc68cal | OVS' is what, 17 args or so? | 15:29 |
| scheuran | sc68cal, doing it this way is really ugly. What I tried first is passing it the kwargs directly, without extracting it | 15:30 |
| *** tiswanso has joined #openstack-neutron | 15:30 | |
| scheuran | in fact it's a dictionary | 15:30 |
| sc68cal | scheuran: OK - that'll work for the time being, we can always take common ones out and make them positional args | 15:30 |
| *** gsagie__ has joined #openstack-neutron | 15:31 | |
| *** phil_h has joined #openstack-neutron | 15:31 | |
| *** thumpba has quit IRC | 15:31 | |
| scheuran | sc68cal, yes, mixing both would be an alternative | 15:31 |
| sc68cal | scheuran: either way sounds like you're on the right track - proceed ! :) | 15:32 |
| *** thumpba has joined #openstack-neutron | 15:33 | |
| *** tsg has joined #openstack-neutron | 15:33 | |
| scheuran | sc68cal, we'll see.... I think I can push up something tomorrow. | 15:33 |
| *** keico_ has quit IRC | 15:34 | |
| irenab | jschwarz: reviewed | 15:35 |
| jschwarz | irenab, saw. uploading a new rev in a minute | 15:35 |
| jschwarz | irenab, re: gotchas - there were quite a few to count and remember :< | 15:36 |
| irenab | jschwarz: :-) | 15:36 |
| *** zhenguo has joined #openstack-neutron | 15:36 | |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2). https://review.openstack.org/198277 | 15:37 |
| *** salv-orl_ has joined #openstack-neutron | 15:37 | |
| mlavalle | mestery: I am trying to add to your gerrit query this patchset: https://review.openstack.org/#/c/200952/. It implements https://blueprints.launchpad.net/neutron/+spec/internal-dns-resolution, which has high priority for Liberty-3. I am not getting it to show in the gerrit query What am I doing wrong? | 15:37 |
| *** lyanchih has quit IRC | 15:37 | |
| *** agireud has joined #openstack-neutron | 15:38 | |
| *** lyanchih has joined #openstack-neutron | 15:39 | |
| *** salv-orlando has quit IRC | 15:40 | |
| *** cbouch__ has joined #openstack-neutron | 15:40 | |
| *** jlibosva has quit IRC | 15:42 | |
| *** zhenguo has quit IRC | 15:42 | |
| *** scheuran has quit IRC | 15:43 | |
| *** cbouch_ has quit IRC | 15:43 | |
| *** jschwarz has quit IRC | 15:44 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Don't ever import providers when not using the relevant plugin https://review.openstack.org/212078 | 15:45 |
| *** josecastroleon has quit IRC | 15:46 | |
| *** josecastroleon has joined #openstack-neutron | 15:47 | |
| *** moshele has quit IRC | 15:48 | |
| *** banix has quit IRC | 15:48 | |
| *** bapalm has quit IRC | 15:49 | |
| *** bapalm has joined #openstack-neutron | 15:49 | |
| Sam-I-Am | amuller: ping | 15:49 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Final decomposition of the ML2 NCS driver https://review.openstack.org/212081 | 15:50 |
| *** piyanai has quit IRC | 15:53 | |
| *** piyanai has joined #openstack-neutron | 15:54 | |
| *** slaweq has joined #openstack-neutron | 15:55 | |
| *** nlahouti has joined #openstack-neutron | 15:56 | |
| *** thumpba has quit IRC | 15:57 | |
| *** jckasper has joined #openstack-neutron | 15:57 | |
| *** thumpba has joined #openstack-neutron | 15:57 | |
| *** geekinutah has joined #openstack-neutron | 15:57 | |
| *** thumpba has quit IRC | 15:58 | |
| *** thumpba has joined #openstack-neutron | 15:58 | |
| *** vivek-ebay has joined #openstack-neutron | 16:01 | |
| *** anshul has quit IRC | 16:03 | |
| *** piyanai has quit IRC | 16:03 | |
| *** akorzeni has quit IRC | 16:04 | |
| *** merooney has joined #openstack-neutron | 16:04 | |
| *** iymht has joined #openstack-neutron | 16:04 | |
| *** vivek-eb_ has joined #openstack-neutron | 16:05 | |
| *** vivek-ebay has quit IRC | 16:05 | |
| openstackgerrit | Merged openstack/neutron: Replace 'import json' with oslo_serialization https://review.openstack.org/211756 | 16:06 |
| *** jlanoux has quit IRC | 16:06 | |
| openstackgerrit | Merged openstack/neutron: Skip FwaaS test that is failing due to race condition https://review.openstack.org/211979 | 16:06 |
| *** sudipto_ has joined #openstack-neutron | 16:06 | |
| *** piyanai has joined #openstack-neutron | 16:08 | |
| openstackgerrit | Sergey Belous proposed openstack/neutron: Adds base in-tree functional testing of the dhcp agent (OVS) https://review.openstack.org/136834 | 16:10 |
| *** iymht has quit IRC | 16:11 | |
| *** gsagie__ has quit IRC | 16:11 | |
| *** jkraj has quit IRC | 16:11 | |
| *** alex_klimov has quit IRC | 16:11 | |
| *** devvesa has quit IRC | 16:11 | |
| dougwig | sc68cal: do we have a bug filed to re-enabled that skipped test? ^^ | 16:12 |
| sc68cal | no, i should do that | 16:12 |
| *** jistr has quit IRC | 16:13 | |
| *** sdake_ is now known as sdake | 16:13 | |
| *** ivar-lazzaro has joined #openstack-neutron | 16:14 | |
| *** ivar-lazzaro has quit IRC | 16:14 | |
| openstackgerrit | John Davidge proposed openstack/neutron: L3 agent changes and reference implementation for IPv6 PD https://review.openstack.org/185977 | 16:15 |
| *** anshul has joined #openstack-neutron | 16:15 | |
| *** ivar-lazzaro has joined #openstack-neutron | 16:15 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Don't ever import providers when not using the relevant plugin https://review.openstack.org/212078 | 16:17 |
| *** roeyc has quit IRC | 16:17 | |
| *** roeyc has joined #openstack-neutron | 16:18 | |
| *** iymht has joined #openstack-neutron | 16:19 | |
| *** vikram has quit IRC | 16:20 | |
| *** rotbeard has quit IRC | 16:20 | |
| *** anshul has quit IRC | 16:21 | |
| *** zhhuabj has quit IRC | 16:22 | |
| *** josecastroleon has quit IRC | 16:23 | |
| *** bharath has quit IRC | 16:25 | |
| *** bharath has joined #openstack-neutron | 16:25 | |
| *** _cjones_ has joined #openstack-neutron | 16:26 | |
| *** josecastroleon has joined #openstack-neutron | 16:26 | |
| *** mickeys has joined #openstack-neutron | 16:27 | |
| *** helenxu_ has quit IRC | 16:27 | |
| *** zhhuabj has joined #openstack-neutron | 16:27 | |
| ihrachyshka | mestery, last feature/qos patch in the check queue, hopefully we'll get it in in ~2h unless we are hit by some other random failure | 16:28 |
| *** slaweq has quit IRC | 16:28 | |
| *** iymht has quit IRC | 16:28 | |
| *** kawa2014 has quit IRC | 16:29 | |
| *** aranjan has joined #openstack-neutron | 16:29 | |
| dougwig | ihrachyshka: that sounds like a challenge. | 16:30 |
| *** erlarese has quit IRC | 16:30 | |
| ihrachyshka | dougwig, getting thru without one? :) | 16:30 |
| *** Marga_ has quit IRC | 16:30 | |
| dougwig | ihrachyshka: or tempting me to change the commit message 50 times. | 16:30 |
| HenryG | armax: ping | 16:31 |
| ihrachyshka | dougwig, it was already hit by one before, that's why it lags a bit with the merge. Hopefully, a bomb does not get twice in the same spot. | 16:31 |
| armax | HenryG: pong | 16:31 |
| *** Marga_ has joined #openstack-neutron | 16:31 | |
| HenryG | armax: What happened to our wonderful plan to switch to pymysql? | 16:31 |
| ihrachyshka | dougwig, heh only if for multiple patches. zuul is smart enough to run a single job per patch | 16:31 |
| armax | it crashed and burned | 16:31 |
| ihrachyshka | HenryG, armax I guess we want unstable job, now with *aas? | 16:32 |
| HenryG | armax: nah, it seems to have hit a road bump | 16:32 |
| *** vikram has joined #openstack-neutron | 16:33 | |
| ihrachyshka | HenryG, mestery: should we push https://review.openstack.org/#/c/212049/ ? | 16:33 |
| dougwig | ihrachyshka: aas won't be in the neutron jobs much longer, they'll have their own. | 16:33 |
| *** sthillma has joined #openstack-neutron | 16:34 | |
| *** ronis_ has joined #openstack-neutron | 16:34 | |
| *** zhhuabj has quit IRC | 16:35 | |
| HenryG | ihrachyshka: on skipping the vpn tests, please check with dougwig and pc_m | 16:35 |
| *** pc_m_AWAY is now known as pc_m | 16:35 | |
| ihrachyshka | dougwig, how long is your much longer? :) | 16:36 |
| HenryG | ihrachyshka: I thought they had some plans to revert stuff and try again differently | 16:36 |
| *** anshul has joined #openstack-neutron | 16:36 | |
| ihrachyshka | HenryG, I thought it was supported by pc_m but let's hear from *aas people indeed | 16:36 |
| armax | ihrachyshka: isn’t just fwaas being affected? | 16:36 |
| armax | ihrachyshka: or all of *aas projects? | 16:37 |
| *** dave-mccowan has joined #openstack-neutron | 16:37 | |
| *** tiswanso has quit IRC | 16:37 | |
| ihrachyshka | armax, as per API tests, yes. | 16:37 |
| *** johnbelamaric has quit IRC | 16:37 | |
| dougwig | lbaas isn't in any neutron job anymore, fyi. | 16:37 |
| dougwig | vpn is moving. fwaas is not in progress yet. | 16:37 |
| *** vivek-eb_ has quit IRC | 16:38 | |
| *** tiswanso has joined #openstack-neutron | 16:38 | |
| armax | ihrachyshka: but lbaas API tests are tested on enutron changes | 16:39 |
| *** Piet has quit IRC | 16:39 | |
| armax | and didn’t notice any instability | 16:39 |
| *** piyanai has quit IRC | 16:39 | |
| ihrachyshka | yeah, probably lbaas has less racy service plugin | 16:40 |
| armax | ihrachyshka: ic | 16:40 |
| *** zhhuabj has joined #openstack-neutron | 16:40 | |
| *** diga_ has joined #openstack-neutron | 16:40 | |
| *** salv-orl_ has quit IRC | 16:40 | |
| *** salv-orlando has joined #openstack-neutron | 16:41 | |
| mestery | ihrachyshka: \o/ regarding QoS patches! A merge commit back may happen today, or did you want to wait until tomorrow? | 16:41 |
| armax | ihrachyshka: so you guys are thinking of introducing an unstable job for the *aas projects? | 16:41 |
| ihrachyshka | mestery, if I will not sleep till when it's in, I'll send it today. I already tried the merge to see what shows up, and it seems it's a single conflict in migration HEADs only | 16:42 |
| *** pradk has joined #openstack-neutron | 16:42 | |
| *** Marga_ has quit IRC | 16:42 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: DO NOT MERGE - fake test of not enabled plugin/sp https://review.openstack.org/212097 | 16:42 |
| *** sudipto_ has quit IRC | 16:42 | |
| mestery | ihrachyshka: Cool! | 16:42 |
| *** Marga_ has joined #openstack-neutron | 16:43 | |
| ihrachyshka | armax, well, if it's only fwaas, we may just go with testing a patch that re-enables the disabled fwaas test periodically to see whether we make progress there. | 16:43 |
| ihrachyshka | and once we don't see the failures, merge it | 16:43 |
| armax | ihrachyshka: but we reverted the d-g patch | 16:44 |
| *** kawa2014 has joined #openstack-neutron | 16:44 | |
| ihrachyshka | armax, after the fwaas test is disabled, we can re-revert. I hope we can. | 16:44 |
| ihrachyshka | though let's do it after feature/qos merge ;) | 16:44 |
| *** slaweq has joined #openstack-neutron | 16:44 | |
| *** matrohon has quit IRC | 16:45 | |
| *** diga_ has quit IRC | 16:46 | |
| *** minwang2 has quit IRC | 16:47 | |
| *** cristicalin has quit IRC | 16:48 | |
| *** slaweq has quit IRC | 16:49 | |
| armax | ihrachyshka: I am slightly baffled by the failure mode we experienced in fwaas | 16:50 |
| *** slaweq has joined #openstack-neutron | 16:50 | |
| armax | ihrachyshka: did anyone attempt to triage it at all? | 16:50 |
| armax | ihrachyshka: kevinbenton’s changes should have made the whole thing pretty bullet-proof | 16:50 |
| *** shivrao has joined #openstack-neutron | 16:50 | |
| armax | ihrachyshka: I am really baffled | 16:51 |
| kevinbenton | armax: what changes? | 16:51 |
| *** rossella_ has quit IRC | 16:52 | |
| ihrachyshka | armax, it's not API deadlock or smth | 16:52 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 16:52 |
| armax | kevinbenton: the changes that added the retry decorated | 16:52 |
| *** hogepodge has joined #openstack-neutron | 16:52 | |
| armax | decorator | 16:52 |
| *** shivrao has quit IRC | 16:52 | |
| ihrachyshka | armax, I think I had a scenario of failure and possible culprit in my mind, I'll check whether I dumped it into the bug | 16:52 |
| *** bapalm has quit IRC | 16:52 | |
| dougwig | ihrachyshka: i'd also be in favor of the "fixing the skip" being "move fwaas tests to their own job, non-voting". | 16:52 |
| ihrachyshka | dougwig, well, most fwaas tests pass, and you know no one cares about non-voting jobs | 16:54 |
| *** salv-orlando has quit IRC | 16:54 | |
| *** piyanai has joined #openstack-neutron | 16:54 | |
| *** salv-orlando has joined #openstack-neutron | 16:54 | |
| dougwig | ihrachyshka: i'm not saying it should stay non-voting. the lbaas jobs that neutron runs started as non-voting, e.g. | 16:54 |
| *** rushil has joined #openstack-neutron | 16:56 | |
| *** azbiswas has joined #openstack-neutron | 16:57 | |
| *** ivar-lazzaro has quit IRC | 16:58 | |
| *** sthillma has quit IRC | 16:58 | |
| *** aranjan_ has joined #openstack-neutron | 16:59 | |
| *** ivar-lazzaro has joined #openstack-neutron | 16:59 | |
| *** sthillma has joined #openstack-neutron | 16:59 | |
| *** erlarese has joined #openstack-neutron | 17:01 | |
| ihrachyshka | dougwig, armax: I dumped my brain in https://bugs.launchpad.net/neutron/+bug/1483875 | 17:01 |
| openstack | Launchpad bug 1483875 in neutron "FWaaS - firewall deleted unexpectedly by agent" [Undecided,New] | 17:01 |
| * armax looks | 17:01 | |
| *** Piet has joined #openstack-neutron | 17:02 | |
| *** aranjan has quit IRC | 17:02 | |
| *** rushil has quit IRC | 17:02 | |
| *** josecastroleon has quit IRC | 17:03 | |
| *** jpena has quit IRC | 17:03 | |
| *** tiswanso has quit IRC | 17:03 | |
| * ihrachyshka hopes he makes sense | 17:03 | |
| *** tiswanso has joined #openstack-neutron | 17:04 | |
| ihrachyshka | I need to run now, please drop an email or reply in the bug if you have thoughts | 17:04 |
| *** shwetaap has quit IRC | 17:04 | |
| *** shwetaap has joined #openstack-neutron | 17:05 | |
| *** dane_leblanc_ has joined #openstack-neutron | 17:05 | |
| *** piyanai has quit IRC | 17:05 | |
| *** josecastroleon has joined #openstack-neutron | 17:05 | |
| *** erlarese has quit IRC | 17:05 | |
| *** erlarese has joined #openstack-neutron | 17:06 | |
| xgerman | ihrachyshka thanks for the analysis | 17:06 |
| *** piyanai has joined #openstack-neutron | 17:06 | |
| *** kawa2014 has quit IRC | 17:07 | |
| *** lyanchih has quit IRC | 17:07 | |
| *** jschwarz has joined #openstack-neutron | 17:07 | |
| jschwarz | ihrachyshka, I see my 2 patches got merged! :) | 17:07 |
| *** dane_leblanc has quit IRC | 17:07 | |
| *** vivek-ebay has joined #openstack-neutron | 17:08 | |
| *** xek has quit IRC | 17:08 | |
| *** ihrachyshka has quit IRC | 17:09 | |
| *** jschwarz has quit IRC | 17:09 | |
| *** tsg has quit IRC | 17:10 | |
| *** tsg has joined #openstack-neutron | 17:10 | |
| *** isq_ has quit IRC | 17:12 | |
| pc_m | dougwig: When you get a chance, let me know what the plans are for resolution of failures, which reviews I need to look at, and if you need me to do anything. | 17:12 |
| *** Marga_ has quit IRC | 17:12 | |
| *** xek has joined #openstack-neutron | 17:13 | |
| *** achanda has joined #openstack-neutron | 17:13 | |
| *** isq_ has joined #openstack-neutron | 17:13 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Improve callback registry devref documentation and usability https://review.openstack.org/211798 | 17:15 |
| *** aranjan_ has quit IRC | 17:16 | |
| *** kawa2014 has joined #openstack-neutron | 17:16 | |
| *** _cjones_ has quit IRC | 17:16 | |
| *** drjones has joined #openstack-neutron | 17:16 | |
| *** piyanai has quit IRC | 17:16 | |
| *** patrickeast has quit IRC | 17:16 | |
| *** kawa2014 has quit IRC | 17:17 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: Avoid DetachedInstanceError after session rollback/expunge https://review.openstack.org/212110 | 17:17 |
| *** obondarev has joined #openstack-neutron | 17:17 | |
| *** dsneddon has joined #openstack-neutron | 17:18 | |
| *** yamamoto has quit IRC | 17:20 | |
| *** aranjan has joined #openstack-neutron | 17:21 | |
| amuller | Sam-I-Am hey you pinged me earlier | 17:23 |
| amuller | Sam-I-Am: What's up? | 17:23 |
| Sam-I-Am | amuller: i updated the l3ha docs in the networking guide, tagged you in the patch | 17:24 |
| *** minwang2 has joined #openstack-neutron | 17:25 | |
| *** roeyc has quit IRC | 17:25 | |
| amuller | Sam-I-Am: sounds good | 17:25 |
| Sam-I-Am | trying to make it mo betta | 17:25 |
| amuller | Sam-I-Am: I'll try to get to it as soon as I can | 17:25 |
| Sam-I-Am | plus re-testing it on kilo | 17:25 |
| Sam-I-Am | sure | 17:25 |
| *** shwetaap has quit IRC | 17:27 | |
| *** britthou_ has joined #openstack-neutron | 17:27 | |
| *** saksham has joined #openstack-neutron | 17:27 | |
| *** vishwanathj has joined #openstack-neutron | 17:27 | |
| *** phil_h has quit IRC | 17:28 | |
| *** dave-mcc_ has joined #openstack-neutron | 17:28 | |
| *** dane_leblanc__ has joined #openstack-neutron | 17:28 | |
| *** shwetaap has joined #openstack-neutron | 17:28 | |
| *** vikram has quit IRC | 17:29 | |
| *** britthouser has quit IRC | 17:30 | |
| *** dane_leblanc_ has quit IRC | 17:30 | |
| *** vivek-eb_ has joined #openstack-neutron | 17:30 | |
| *** dave-mccowan has quit IRC | 17:31 | |
| *** patrickeast has joined #openstack-neutron | 17:31 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Fix docs job https://review.openstack.org/212119 | 17:31 |
| *** piyanai has joined #openstack-neutron | 17:31 | |
| klindgren | Is anyone on Kilo noticing an uptick in the number of timeouts from nova -> neutron? Since upgrading to kilo we are averaging ~10-20 timeouts a day, where in the neutron log I cannot find the wsgi layer logging that it received a connection | 17:31 |
| *** rpothier has joined #openstack-neutron | 17:32 | |
| klindgren | Typically see: RequestTimeout: Request to https://<neutronurl>:9696/v2.0/ports.json?fixed_ips=ip_address%3D10.22.191.19 timed out (HTTP 408) | 17:32 |
| *** nyechiel_ has quit IRC | 17:33 | |
| *** banix has joined #openstack-neutron | 17:34 | |
| *** vivek-ebay has quit IRC | 17:34 | |
| *** dane_leblanc__ has quit IRC | 17:36 | |
| *** rpothier has quit IRC | 17:36 | |
| *** tflynn has joined #openstack-neutron | 17:36 | |
| *** dane_leblanc__ has joined #openstack-neutron | 17:37 | |
| *** dave-mccowan has joined #openstack-neutron | 17:38 | |
| *** shwetaap1 has joined #openstack-neutron | 17:38 | |
| *** shwetaap has quit IRC | 17:39 | |
| *** tflynn_ has joined #openstack-neutron | 17:40 | |
| *** dave-mcc_ has quit IRC | 17:40 | |
| *** rpothier has joined #openstack-neutron | 17:41 | |
| *** gsagie__ has joined #openstack-neutron | 17:41 | |
| *** mickeys has quit IRC | 17:41 | |
| *** josecastroleon has quit IRC | 17:42 | |
| *** vikram has joined #openstack-neutron | 17:42 | |
| *** tflynn has quit IRC | 17:43 | |
| *** josecastroleon has joined #openstack-neutron | 17:43 | |
| *** yamamoto has joined #openstack-neutron | 17:43 | |
| *** annegentle has quit IRC | 17:44 | |
| *** abregman has joined #openstack-neutron | 17:45 | |
| *** moha_hunt has joined #openstack-neutron | 17:45 | |
| openstackgerrit | James E. Blair proposed openstack/neutron: Treat sphinx warnings as errors https://review.openstack.org/212122 | 17:45 |
| *** johnbelamaric has joined #openstack-neutron | 17:46 | |
| *** padkrish has joined #openstack-neutron | 17:46 | |
| *** crc32 has joined #openstack-neutron | 17:49 | |
| *** mickeys has joined #openstack-neutron | 17:50 | |
| *** shivrao has joined #openstack-neutron | 17:51 | |
| *** phil_h has joined #openstack-neutron | 17:53 | |
| openstackgerrit | Merged openstack/python-neutronclient: Disable failing vpn tests https://review.openstack.org/212049 | 17:53 |
| *** abregman is now known as abregman|afk | 17:54 | |
| *** absubram has quit IRC | 17:55 | |
| *** ihrachyshka has joined #openstack-neutron | 17:55 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron: WIP - Placeholder for changes based on empty sp sections https://review.openstack.org/211760 | 17:56 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Update fullstack multinode simulation image https://review.openstack.org/212124 | 17:56 |
| *** shashank_hegde has joined #openstack-neutron | 17:56 | |
| *** absubram has joined #openstack-neutron | 17:57 | |
| *** madhu_ak has joined #openstack-neutron | 17:57 | |
| *** piyanai has quit IRC | 17:58 | |
| *** ildikov has joined #openstack-neutron | 17:59 | |
| *** salv-orl_ has joined #openstack-neutron | 17:59 | |
| *** sridhar_ram has joined #openstack-neutron | 18:01 | |
| *** Marga_ has joined #openstack-neutron | 18:01 | |
| *** salv-orlando has quit IRC | 18:01 | |
| *** banix has quit IRC | 18:01 | |
| *** tidwellr1 has joined #openstack-neutron | 18:02 | |
| *** dannywilson has quit IRC | 18:04 | |
| *** jagter has quit IRC | 18:05 | |
| *** ijw has joined #openstack-neutron | 18:05 | |
| openstackgerrit | Merged openstack/neutron: Revert "Revert "Add VIF_DELETED notification event to Nova"" https://review.openstack.org/187871 | 18:05 |
| *** jagter has joined #openstack-neutron | 18:06 | |
| *** piyanai has joined #openstack-neutron | 18:06 | |
| *** abregman|afk has quit IRC | 18:07 | |
| *** anshul has quit IRC | 18:08 | |
| *** ijw_ has joined #openstack-neutron | 18:08 | |
| *** ijw has quit IRC | 18:11 | |
| madhu_ak | pc_m: saw your comments, will take a look into that | 18:11 |
| *** ijw has joined #openstack-neutron | 18:11 | |
| pc_m | madhu_ak: ok | 18:12 |
| ihrachyshka | amuller, fyi I tried to make the gate job execute tox -edocs before: http://git.openstack.org/cgit/openstack-infra/project-config/commit/jenkins/scripts/run-docs.sh?id=0ced503c2792b2f50559f66abd92d81e1571b77e | 18:12 |
| ihrachyshka | amuller, but it seems it was reverted | 18:12 |
| madhu_ak | pc_m you were suggesting to have two patches in vpnaas repo with job names ( new job names refers to the proposed patch in infra project, old one doesnt want to be disturbed) | 18:13 |
| madhu_ak | my assumption is correct pc_m ? | 18:13 |
| *** ijw_ has quit IRC | 18:14 | |
| *** bapalm_ has joined #openstack-neutron | 18:14 | |
| pc_m | madhu_ak: Suggesting that the hook scripts support both the new and old naming, so that existing tests are not broken. Then, after new names are added to project-config, the old names can be removed from neutron-vpnaas. | 18:15 |
| *** piyanai has quit IRC | 18:15 | |
| ihrachyshka | amuller, I guess the comment in the script suggests that our docs job should use the same as gate does, not vice versa | 18:15 |
| madhu_ak | pc_m got it. thanks for the note | 18:15 |
| pc_m | madhu_ak: You may want to consider handling these two separate efforts (adding API test and refactoring the names). | 18:15 |
| *** ijw has quit IRC | 18:16 | |
| *** piyanai has joined #openstack-neutron | 18:16 | |
| pc_m | madhu_ak: It's more commits, but isolates the changes (and may be easier to gain acceptance). | 18:16 |
| madhu_ak | pc_m in that case, the job-template may be duplicated and see if that works, if that is so, then we can incorporate the changes? | 18:17 |
| amuller | ihrachyshka: https://review.openstack.org/212122 | 18:18 |
| amuller | this is the solution provided by infra | 18:18 |
| *** lori is now known as lori|away | 18:18 | |
| *** ijw has joined #openstack-neutron | 18:18 | |
| *** johnbelamaric has quit IRC | 18:19 | |
| *** josecastroleon has quit IRC | 18:19 | |
| ihrachyshka | amuller, I think it won't magically tox -edocs behave as per testing doc they refer | 18:20 |
| ihrachyshka | amuller, since we have direct sphinx-build call there | 18:20 |
| ihrachyshka | so I would expect all pbr machinery avoided now for tox -edocs | 18:21 |
| *** josecastroleon has joined #openstack-neutron | 18:21 | |
| amuller | ihrachyshka: if that patch doesn't fail the docs job I'll reexamine this | 18:21 |
| pc_m | madhu_ak: With it split as two efforts, the new job-template for API could be added, along with support in hook scripts to test that job, and then a separate effort could remove the duplication in all jobs (using both names in the interim). | 18:21 |
| ihrachyshka | ack. pretty sure it won't but let's get some popcorn | 18:22 |
| *** ijw has quit IRC | 18:22 | |
| ihrachyshka | huh, it actually failed | 18:22 |
| *** banix has joined #openstack-neutron | 18:22 | |
| *** ijw has joined #openstack-neutron | 18:23 | |
| madhu_ak | gotcha pc_m thanks ! | 18:23 |
| *** bapalm has joined #openstack-neutron | 18:23 | |
| pc_m | madhu_ak: yw | 18:23 |
| *** bapalm has quit IRC | 18:24 | |
| amuller | ihrachyshka: are you comfortable with rebasing that patch on my patch that fixes the docs job then merging both? | 18:24 |
| amuller | ihrachyshka: or am I missing something? | 18:24 |
| ihrachyshka | amuller, I guess I need to shut up now since apparently I don't get how it works :) but indeed it makes it fail both in gate and locally, so we are good to go | 18:26 |
| openstackgerrit | Brian Haley proposed openstack/neutron: Distributed router can not add routes https://review.openstack.org/144205 | 18:26 |
| *** lilynn2 has joined #openstack-neutron | 18:26 | |
| amuller | ihrachyshka: ok please approve this one | 18:27 |
| amuller | https://review.openstack.org/#/c/212119/ | 18:27 |
| *** tfukushima has quit IRC | 18:27 | |
| *** piyanai has quit IRC | 18:27 | |
| amuller | ihrachyshka: and I'll rebase the job fix on it | 18:27 |
| *** tfukushima has joined #openstack-neutron | 18:28 | |
| ihrachyshka | amuller, done | 18:28 |
| *** tfukushima has quit IRC | 18:28 | |
| amuller | ihrachyshka: I'll ping you in a sec to approve the other one | 18:28 |
| *** vikram has quit IRC | 18:29 | |
| ihrachyshka | amuller, ack. btw have you seen: https://review.openstack.org/#/c/211642/ ? | 18:30 |
| amuller | ihrachyshka: yes but it was not trivial so it went in to the 'check it later' bin :) | 18:31 |
| *** yapeng has quit IRC | 18:32 | |
| *** sbalukoff has joined #openstack-neutron | 18:32 | |
| *** yamamoto has quit IRC | 18:33 | |
| *** jwy has joined #openstack-neutron | 18:33 | |
| *** arcimboldo has joined #openstack-neutron | 18:35 | |
| *** gberginc has joined #openstack-neutron | 18:35 | |
| arcimboldo | hi all, trying to set the gateway of a router in Kilo I get the following error: http://paste.openstack.org/show/412758/ | 18:35 |
| *** absubram has quit IRC | 18:35 | |
| arcimboldo | it looks pretty weird | 18:36 |
| *** ijw_ has joined #openstack-neutron | 18:37 | |
| *** Sukhdev_ has joined #openstack-neutron | 18:37 | |
| *** piyanai has joined #openstack-neutron | 18:38 | |
| *** absubram has joined #openstack-neutron | 18:38 | |
| *** ijw has quit IRC | 18:40 | |
| *** vikram has joined #openstack-neutron | 18:41 | |
| *** enikanorov2 has quit IRC | 18:42 | |
| *** gsagie__ has quit IRC | 18:45 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron-fwaas: Do not make delete_db_firewall conditional https://review.openstack.org/212143 | 18:45 |
| *** padkrish has quit IRC | 18:46 | |
| *** salv-orl_ has quit IRC | 18:46 | |
| *** padkrish has joined #openstack-neutron | 18:46 | |
| *** nlahouti has quit IRC | 18:47 | |
| armax | ihrachyshka: ping | 18:48 |
| *** absubram has quit IRC | 18:48 | |
| *** slaweq has quit IRC | 18:49 | |
| *** salv-orlando has joined #openstack-neutron | 18:49 | |
| ihrachyshka | armax, pong | 18:50 |
| armax | ihrachyshka: quick question: | 18:50 |
| ihrachyshka | oh I see some patch above | 18:50 |
| armax | ihrachyshka: yes | 18:50 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Unskip this test to lure out the bug https://review.openstack.org/212145 | 18:50 |
| *** padkrish has quit IRC | 18:50 | |
| armax | but now that we reverted to stable pymysql I wonder how we can lure out the bug | 18:51 |
| armax | shall we re-re-revert? | 18:51 |
| armax | since sc68cal patch went in | 18:51 |
| armax | this could in theory be reverted: https://review.openstack.org/#/c/211853/ no? | 18:52 |
| ihrachyshka | armax, yes, we should re-re-revert it, but please do it after qos merge-back | 18:52 |
| ihrachyshka | can we wait several days till that? | 18:53 |
| *** jpena has joined #openstack-neutron | 18:53 | |
| armax | but isn’t the culprit test disabled? | 18:53 |
| *** baoli has quit IRC | 18:53 | |
| armax | what’s blocking qos merge-back? | 18:53 |
| *** baoli has joined #openstack-neutron | 18:54 | |
| *** boris-42 has joined #openstack-neutron | 18:54 | |
| *** absubram has joined #openstack-neutron | 18:54 | |
| ihrachyshka | armax, well, I am just a bit cautious since we were postponing the merge several times due to some fail or another. | 18:54 |
| ihrachyshka | armax, I don't see that it will break us, just a bit cautious. so I won't have particular objections if you go and re-re-revert now. | 18:55 |
| armax | ihrachyshka: but I’d rather flush this stuff out as early as we can | 18:55 |
| *** arcimboldo has quit IRC | 18:55 | |
| armax | ihrachyshka: pushing back is only going to make things worse | 18:55 |
| openstackgerrit | vikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce drgent https://review.openstack.org/207625 | 18:55 |
| armax | if the culrpit test is skipped I don’t see what else might affect qos | 18:55 |
| *** enikanorov2 has joined #openstack-neutron | 18:55 | |
| armax | if qos itself is at fault | 18:55 |
| armax | in which case, we might as well know this sooner rather than later | 18:56 |
| armax | L3 is approach fast | 18:56 |
| *** anshul has joined #openstack-neutron | 18:56 | |
| armax | that infra patch was really meant to merge long time ago | 18:56 |
| ihrachyshka | armax, :) well ok, I guess I should not keep progress there just in case. | 18:56 |
| armax | and if it did we would have had a lot more time to debug and resolve these issues | 18:56 |
| armax | ihrachyshka: I agree we need to be cautious to avoid breaking everyone | 18:57 |
| armax | ihrachyshka: but only if we have hard data or a good hunch that things can go south | 18:57 |
| *** ijw has joined #openstack-neutron | 18:57 | |
| ihrachyshka | armax, wait... but is the fwaas test disabled in feature/qos? I guess no since we haven't merged back. | 18:57 |
| ihrachyshka | master->qos | 18:58 |
| armax | ihrachyshka: if we’re confident the only affected project is fwaas + and neutron on the api job | 18:58 |
| armax | and we resolved that by skipping the test temporarily | 18:58 |
| ihrachyshka | armax, yeah, but it's skipped in master only. | 18:58 |
| *** moha_hunt has quit IRC | 18:58 | |
| armax | let’s reistate the source of instability and lure the bug out with patch https://review.openstack.org/#/c/212145/ | 18:58 |
| ihrachyshka | we obviously can merge master->qos to unblock you, but I really hoped we just get rid of the branch next days. | 18:58 |
| armax | so that we know better what’s going on | 18:58 |
| *** dannywilson has joined #openstack-neutron | 18:59 | |
| armax | ihrachyshka: agreed…but how long does it take to merge the two? | 18:59 |
| armax | ihrachyshka: they must not be that much different at this point are tehy? | 18:59 |
| *** ijw_ has quit IRC | 19:00 | |
| armax | ihrachyshka: your call in the end, I am only making sure the sql driver switch and api workers flag doesn’t fall through the cracks | 19:01 |
| armax | as it did last cycle | 19:01 |
| armax | we’re so close that it would be a shame see this slip yet again | 19:01 |
| ihrachyshka | armax, ok, let's do it this way: I'll request master->qos merge, and then back: qos->master, so we'll have both branches synced and with the test disabled. I will give you both links, and then you can depends-on the re-re-revert patch on the one that synces master into qos | 19:01 |
| *** vikram has quit IRC | 19:01 | |
| *** nlahouti has joined #openstack-neutron | 19:01 | |
| *** padkrish has joined #openstack-neutron | 19:01 | |
| armax | ihrachyshka: ok | 19:02 |
| ihrachyshka | then re-re-revert will wait until feature/qos is unbroken. but also note we have pecan people who will get broken too. | 19:02 |
| *** sthillma has quit IRC | 19:02 | |
| * ihrachyshka notes feature branches are extremely expensive when it comes to gate fixes | 19:03 | |
| *** nlahouti has quit IRC | 19:03 | |
| armax | ihrachyshka: all I am saying is that I’d rather merge these features knowning that the ‘unstable’ configuration doesn’t get any more unstable than it already is | 19:03 |
| ihrachyshka | armax, the way qos is implemented, it's hard to find a place where it could really affect anything | 19:04 |
| *** padkrish has quit IRC | 19:04 | |
| ihrachyshka | armax, since 99% of the changes are not enabled by default and isolated | 19:04 |
| *** padkrish has joined #openstack-neutron | 19:04 | |
| ihrachyshka | not sure about pecan | 19:04 |
| *** cristicalin has joined #openstack-neutron | 19:04 | |
| armax | ihrachyshka: ok then, so you’re convinced me that we should really re-re-revert asap! | 19:05 |
| ihrachyshka | huh :) | 19:05 |
| *** cristicalin has quit IRC | 19:07 | |
| *** ijw has quit IRC | 19:07 | |
| *** Sukhdev__ has joined #openstack-neutron | 19:07 | |
| *** ijw has joined #openstack-neutron | 19:07 | |
| *** tidwellr1 has quit IRC | 19:09 | |
| *** nlahouti has joined #openstack-neutron | 19:09 | |
| armax | ihrachyshka: all I am saying is that qos should not affect anything as you claim, what can affect qos has been idenfitied… | 19:10 |
| armax | ihrachyshka: so I don’t see why we should wait | 19:10 |
| armax | to re-enable the new sql driver with api_workers>0 | 19:11 |
| *** aranjan has quit IRC | 19:11 | |
| *** Sukhdev_ has quit IRC | 19:11 | |
| ihrachyshka | armax, go forward with re-re-revert, we'll cope. also, our last patch is about to merge, and hopefully we won't have more patches in the branches since there will be nits only ;) | 19:11 |
| ihrachyshka | in the branch* | 19:11 |
| armax | ihrachyshka: well, the re-re-revert has been posted here: https://review.openstack.org/#/c/199672/ | 19:11 |
| *** phil_h has quit IRC | 19:11 | |
| armax | ihrachyshka: I can’t do anything to make it merge fast | 19:12 |
| armax | ihrachyshka: and I won’t get in your way if you want to hold on and wait for the dust to settle | 19:12 |
| *** Sukhdev_ has joined #openstack-neutron | 19:13 | |
| armax | ihrachyshka: my suggestion would be to have this enabled ahead of the merge back and deal with the potential fallout in the process, rather than after the fact | 19:13 |
| armax | ihrachyshka: as that would give us more time to troubleshoot | 19:13 |
| *** Sukhdev__ has quit IRC | 19:14 | |
| *** abhiraut has joined #openstack-neutron | 19:14 | |
| ihrachyshka | armax, ok, +1 | 19:14 |
| ihrachyshka | all I have for you there; ) | 19:14 |
| armax | ihrachyshka: I appreciate the support! | 19:15 |
| *** mplex has left #openstack-neutron | 19:15 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Unskip this test to lure out the bug https://review.openstack.org/212145 | 19:15 |
| *** jpena is now known as jpena|away | 19:15 | |
| armax | ihrachyshka: ^^ | 19:16 |
| *** shashank_hegde has quit IRC | 19:16 | |
| armax | ihrachyshka: I added it to the list of depends on here, we’ll see how this goes | 19:16 |
| ihrachyshka | cool. I will review the fwaas one tomorrow | 19:17 |
| ihrachyshka | not obvious from the first sight :) | 19:17 |
| armax | ihrachyshka: it’s not fixing anything right now | 19:17 |
| armax | ihrachyshka: but it’s making it more sane and easier to debug | 19:17 |
| armax | once I collect more data, I might be able to see better what’s going on... | 19:17 |
| ihrachyshka | ack | 19:18 |
| armax | but the problem is obvious | 19:18 |
| armax | delete and update interleave | 19:18 |
| ihrachyshka | yeap. apparently status report | 19:18 |
| armax | and things goes astray | 19:18 |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: NSX: Move DB models as part of core vendor decomposition https://review.openstack.org/211370 | 19:18 |
| armax | becasue there’s no critical region between the two | 19:18 |
| *** nlahouti has quit IRC | 19:18 | |
| ihrachyshka | armax, you can't have one in neutron-server apart from db-constrainted, right? | 19:19 |
| armax | right | 19:19 |
| armax | but this is pretty obvious here: | 19:19 |
| armax | https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/services/firewall/fwaas_plugin.py#L269 | 19:19 |
| *** suresh12 has joined #openstack-neutron | 19:19 | |
| armax | during update we check that the firewall is not in pending status right at the very beginning | 19:20 |
| armax | but here: | 19:20 |
| armax | https://github.com/openstack/neutron-fwaas/blob/master/neutron_fwaas/services/firewall/fwaas_plugin.py#L289 | 19:20 |
| *** padkrish has quit IRC | 19:20 | |
| ihrachyshka | ouch. | 19:20 |
| armax | the status might as well have gone to pending_delete | 19:20 |
| armax | and we’re screwed | 19:20 |
| ihrachyshka | autonested_transaction needed? | 19:20 |
| armax | I need to mull over this a bit | 19:20 |
| armax | I am not entirely sure what’s going on yet | 19:21 |
| ihrachyshka | woot, the last qos patch is in | 19:21 |
| * ihrachyshka -> merging mode | 19:21 | |
| armax | but I can clearly see here that states can get messed up easily | 19:21 |
| amuller | for the past 2 days git fetch and push and have been *really* slow for me | 19:21 |
| amuller | has anyone else run in to this? | 19:22 |
| *** abhiraut has quit IRC | 19:22 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Treat sphinx warnings as errors https://review.openstack.org/212122 | 19:22 |
| armax | amuller: not for me | 19:22 |
| ihrachyshka | amuller, really? like how? | 19:22 |
| ihrachyshka | it's several seconds here, as usual | 19:22 |
| amuller | ihrachyshka: every time I git fetch or git review it takes over 2 minutes | 19:22 |
| amuller | just from the last 2-3 days | 19:22 |
| ihrachyshka | meh, definitely local | 19:22 |
| amuller | :( | 19:22 |
| kevinbenton | amuller: which origin are you using? | 19:22 |
| ihrachyshka | Thou Shalt Not Move to East Coast | 19:22 |
| *** abhiraut has joined #openstack-neutron | 19:23 | |
| amuller | kevinbenton: http://paste.openstack.org/show/412766/ | 19:23 |
| amuller | using origin when fetching | 19:24 |
| kevinbenton | see if you get the same issue if you switch it to github.com/openstack/neutron | 19:25 |
| kevinbenton | amuller: ^^ | 19:25 |
| amuller | ihrachyshka: this should be approvable now https://review.openstack.org/#/c/212122/ | 19:25 |
| amuller | kevinbenton: thanks I'll try | 19:25 |
| amuller | kevinbenton: so fetch is instant now | 19:28 |
| amuller | kevinbenton: but I still have to configure .gitreview to point to review.openstack.org right, I can't get around that | 19:29 |
| amuller | or I can just use git push | 19:29 |
| *** padkrish has joined #openstack-neutron | 19:29 | |
| amuller | kevinbenton: anyway thanks :) | 19:29 |
| *** nlahouti has joined #openstack-neutron | 19:30 | |
| kevinbenton | amuller: yeah, you can just push dirrectly to the review branch | 19:30 |
| kevinbenton | amuller: is review.openstack.org slow as well? | 19:30 |
| amuller | kevinbenton: nope =D https access via my browser is just fine | 19:30 |
| kevinbenton | amuller: i mean like when you fetch from open reviews | 19:31 |
| amuller | it is yes | 19:31 |
| kevinbenton | strange | 19:33 |
| *** nlahouti has quit IRC | 19:33 | |
| kevinbenton | amuller: did you pay for your premium git subscription to Linus this month? | 19:33 |
| *** yamamoto has joined #openstack-neutron | 19:33 | |
| amuller | kevinbenton: Yes but not to Comcast :( | 19:34 |
| amuller | sc68cal: I know you're no longer there but I still blame you btw | 19:34 |
| sc68cal | amuller: don't let pesky facts get in the way | 19:34 |
| * sc68cal does his palpatine voice | 19:34 | |
| amuller | facts... pfft, you can prove anything with facts. | 19:34 |
| sc68cal | give into your hate | 19:34 |
| salv-orlando | amuller: do not rule out comcast throttling git traffic | 19:34 |
| *** abhiraut has quit IRC | 19:34 | |
| amuller | salv-orlando: heaven forbid I infringe on poor starving artists using the git protocol | 19:35 |
| *** piyanai has quit IRC | 19:35 | |
| sc68cal | speaking of which, I need to call them up and cancel extreme 150 internet. My wifi setup only pushes like 70mbps in my house | 19:35 |
| sc68cal | now that it's costing me real money | 19:35 |
| *** piyanai has joined #openstack-neutron | 19:35 | |
| *** piyanai has quit IRC | 19:36 | |
| sc68cal | only my desktop is wired into the switch and gets the full 150 - wifi stinks | 19:36 |
| *** tidwellr1 has joined #openstack-neutron | 19:36 | |
| amuller | sc68cal: wifi is for animals | 19:36 |
| amuller | sc68cal: scratch that my cats are too good for wifi | 19:36 |
| sc68cal | lol | 19:36 |
| *** mickeys has quit IRC | 19:36 | |
| kevinbenton | sc68cal: wifi AC! | 19:37 |
| kevinbenton | sc68cal: access points are pricey but awesome. one of my friends has one and can get over 100mbps | 19:38 |
| *** yamamoto has quit IRC | 19:38 | |
| sc68cal | the problem is I have a wifi AP - but it uses powerline ethernet to go from the AP to my equipment rack | 19:38 |
| sc68cal | where the gateway is | 19:39 |
| *** annegentle has joined #openstack-neutron | 19:40 | |
| *** suresh12 has quit IRC | 19:41 | |
| amuller | anyone wanna approve this? https://review.openstack.org/#/c/212124/ - updates the fullstack multinode simulation image | 19:42 |
| *** piyanai has joined #openstack-neutron | 19:43 | |
| *** sc68cal has quit IRC | 19:44 | |
| mestery | amuller: I love that "fullstack-doc-love" patch, nice images :) | 19:46 |
| *** suresh12 has joined #openstack-neutron | 19:46 | |
| amuller | mestery: google draw | 19:46 |
| mestery | ha! | 19:46 |
| *** sc68cal has joined #openstack-neutron | 19:46 | |
| ihrachyshka | mestery, hm, how do I send a merge patch into master? I follow http://docs.openstack.org/infra/manual/drivers.html#merge-feature-branch-into-master but get it in feature/qos anyway. | 19:48 |
| *** john-davidge has quit IRC | 19:49 | |
| ihrachyshka | that's what it generated https://review.openstack.org/212166 | 19:49 |
| ihrachyshka | and -R master does not help either (it says ! [remote rejected] HEAD -> refs/publish/master/bug/1468990 (no new changes)) so apparently it tried the same branch | 19:50 |
| *** thurloat has quit IRC | 19:50 | |
| *** swat30 has quit IRC | 19:50 | |
| ihrachyshka | oh I think it's .gitreview changes in the branch... | 19:51 |
| *** shashank_hegde has joined #openstack-neutron | 19:51 | |
| *** anshul has quit IRC | 19:52 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Merge remote-tracking branch 'origin/feature/qos' into merge-branch https://review.openstack.org/212170 | 19:52 |
| *** gberginc has quit IRC | 19:52 | |
| *** padkrish has quit IRC | 19:52 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Merge remote-tracking branch 'origin/feature/qos' into merge-branch https://review.openstack.org/212170 | 19:53 |
| *** vivek-eb_ has quit IRC | 19:53 | |
| *** padkrish has joined #openstack-neutron | 19:53 | |
| ihrachyshka | mestery, ^^ \o/ | 19:53 |
| mestery | ihrachyshka: WOOOOOOOOOOOOOTTTTTTTTT!!!! | 19:54 |
| amuller | wow | 19:54 |
| mestery | I think my excited yell scared some folks ihrachyshka ;) | 19:54 |
| amuller | no I was 'wowing' the existence of that patch :) | 19:55 |
| amuller | ihrachyshka: you must feel great :) | 19:55 |
| mestery | :) | 19:55 |
| mestery | lol | 19:55 |
| mestery | ihrachyshka amuller: It's pretty awesome to see us fold a feature branch back in. | 19:55 |
| mestery | ihrachyshka: Incredible work sir, well done. | 19:55 |
| *** tflynn_ has quit IRC | 19:56 | |
| ihrachyshka | amuller, not yet man, not yet. let's see how it gets thru the gate and reviewers ;) | 19:56 |
| amuller | ihrachyshka: were there CI jobs that did not run on feature/qos, or that were configured differently than they are against the master branch? Is there any reason why this patch will fail CI? | 19:56 |
| ihrachyshka | amuller, in the qos branch, we had qos service and ml2 qos extension driver on. API tests seem to check for the former, so should be no-op for now. | 19:57 |
| ihrachyshka | apart from that, it should be the same. but we'll definitely see! :) | 19:57 |
| *** padkrish has quit IRC | 19:57 | |
| amuller | ihrachyshka: what's the plan to enable the api tests? we need to first merge then send a patch to devstack to enable the qos plugin? | 19:58 |
| kevinbenton | haleyb: ¡YO! | 19:58 |
| ihrachyshka | amuller, mestery, ajo, http://lists.openstack.org/pipermail/openstack-dev/2015-August/071914.html | 19:58 |
| *** tflynn has joined #openstack-neutron | 19:58 | |
| *** johnbelamaric has joined #openstack-neutron | 19:58 | |
| ihrachyshka | amuller, I think we should enable it after merge to avoid the testing gap | 19:58 |
| *** ijw_ has joined #openstack-neutron | 19:58 | |
| amuller | yeah | 19:58 |
| amuller | ok | 19:58 |
| ihrachyshka | I am confident it should not introduce instability | 19:59 |
| ihrachyshka | at least we haven't experienced any lately | 19:59 |
| *** lazy_prince has joined #openstack-neutron | 19:59 | |
| haleyb | kevinbenton: pong. i was still trying to test if the conntrack code ever gets a qvb device in that code :) | 20:01 |
| *** padkrish has joined #openstack-neutron | 20:01 | |
| *** ijw has quit IRC | 20:02 | |
| *** TrevorV has joined #openstack-neutron | 20:03 | |
| kevinbenton | haleyb: ack | 20:05 |
| * haleyb is assuming that was the YO | 20:06 | |
| *** piyanai has quit IRC | 20:06 | |
| *** piyanai has joined #openstack-neutron | 20:07 | |
| openstackgerrit | Ryan Moats proposed openstack/neutron: Preserve DVR FIP rule priority over Agent restarts https://review.openstack.org/193711 | 20:07 |
| kevinbenton | haleyb: http://paste.openstack.org/show/412772/ | 20:08 |
| kevinbenton | haleyb: right in those first rules | 20:08 |
| kevinbenton | haleyb: the other option is to use the port id to store zones | 20:10 |
| kevinbenton | haleyb: and then when populating the initial zone map i need to try to lookup the port ids either by asking the server or ovs | 20:11 |
| *** ivar-lazzaro has quit IRC | 20:11 | |
| haleyb | kevinbenton: i put some debug in the conntrack code, and it's using the port uuid as the device name, for example fcd99706-45..., but never with qvb or tap on the front | 20:12 |
| amuller | ihrachyshka: it's going to be hard to give comments because there's no diff | 20:12 |
| amuller | maybe that's a good thing =D | 20:13 |
| haleyb | kevinbenton: just meaning the check can be moved to the ovs code and not affect anything from what i can see | 20:13 |
| ihrachyshka | amuller, well, that's one of the prices. wanna have an etherpad for that? :) | 20:13 |
| kevinbenton | haleyb: what do you mean? | 20:14 |
| *** merooney has quit IRC | 20:14 | |
| kevinbenton | haleyb: right now i'm storing zones based on interface names | 20:14 |
| amuller | ihrachyshka: I think so because commenting on the commit message or the patch itself will not scale | 20:14 |
| *** piyanai has quit IRC | 20:14 | |
| ihrachyshka | amuller, ack, I'm on it. 5 mins. | 20:14 |
| kevinbenton | haleyb: and the conntrack code needs to lookup the zone for a given port id | 20:14 |
| amuller | ihrachyshka: and you can link to the etherpad from the top of the commit message | 20:14 |
| haleyb | kevinbenton: the name the conntrack manager calls self.get_device_zone() with is a uuid from what i can see, not a qvb or tap | 20:15 |
| *** achanda has quit IRC | 20:15 | |
| kevinbenton | haleyb: right, it doesn't know what to look for | 20:15 |
| kevinbenton | haleyb: thats why get_device_zone has to convert it to a name | 20:16 |
| kevinbenton | haleyb: zone map is based on device names | 20:16 |
| kevinbenton | haleyb: and the conntrack manager doesn't know names | 20:16 |
| Sam-I-Am | amuller: thanks for the comments on the patch. i thought l2pop was only broken with linuxbridge. | 20:17 |
| *** johnbelamaric has quit IRC | 20:17 | |
| amuller | Sam-I-Am: l2pop itself should work but not with HA routers | 20:17 |
| haleyb | kevinbenton: but that code seems to normalize everything it can into tap* from qvb* - what if the uuid passed does start with qvb? it will change it | 20:17 |
| amuller | Sam-I-Am: we fixed that interaction a weekish ago on master | 20:17 |
| amuller | haven't backported to kilo yet | 20:17 |
| *** piyanai has joined #openstack-neutron | 20:17 | |
| *** ronis_ has quit IRC | 20:17 | |
| kevinbenton | haleyb: yes, because the qvb and tap followed by the same UUID need to be in the same zone | 20:17 |
| amuller | Sam-I-Am: the bad interaction is because l2pop assumes that a port is bound to a single host but with HA routers that's not the case | 20:17 |
| *** phil_h has joined #openstack-neutron | 20:17 | |
| kevinbenton | haleyb: because they belong to the same VM port | 20:17 |
| Sam-I-Am | amuller: yeah, i noted the patches. just always assumed it was only linuxbridge :) | 20:18 |
| *** mitchjameson has joined #openstack-neutron | 20:18 | |
| Sam-I-Am | because... linuxbridge lol | 20:18 |
| *** padkrish has quit IRC | 20:18 | |
| amuller | Sam-I-Am: this time linuxbridge is not at fault | 20:18 |
| *** azbiswas has quit IRC | 20:18 | |
| Sam-I-Am | amuller: haha, amazing | 20:18 |
| *** padkrish has joined #openstack-neutron | 20:18 | |
| Sam-I-Am | i'll pull the l2pop stuff and address all the other comments | 20:18 |
| Sam-I-Am | looks juicy :) | 20:19 |
| haleyb | kevinbenton: yes, understood. but you will only get a device starting with qvb when running the OVS firewall driver, correct ? the conntrack code won't ever pass one | 20:19 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Merge remote-tracking branch 'origin/feature/qos' into merge-branch https://review.openstack.org/212170 | 20:21 |
| ihrachyshka | amuller, added the link https://review.openstack.org/#/c/212170/ | 20:21 |
| kevinbenton | haleyb: yeah, it looks like it would need to call get_device_name as well | 20:21 |
| ihrachyshka | ok, I will leave now and allow you folks to search for major flaws that will block the merge ;) | 20:22 |
| kevinbenton | haleyb: because it's just the full port UUID | 20:22 |
| kevinbenton | haleyb: get_device_zone is either going to have to know how to generate the device names, or the zone map will need to be indexed by something else | 20:23 |
| haleyb | kevinbenton: so you were trying to have the tap, qvb, and whatver ipconntrack use map to the same value? | 20:25 |
| kevinbenton | haleyb: yes | 20:25 |
| kevinbenton | haleyb: the port UUID what i really need | 20:25 |
| kevinbenton | haleyb: but unfortunately on startup all i get is the interface names | 20:25 |
| *** tflynn has quit IRC | 20:25 | |
| kevinbenton | haleyb: so maybe i should just add the logic to figure out the UUID for th map | 20:25 |
| kevinbenton | the* | 20:26 |
| madhu_ak | pc_m, https://review.openstack.org/#/c/211767/ | 20:26 |
| madhu_ak | another patch is on the way | 20:26 |
| kevinbenton | haleyb: probably an RPC call | 20:26 |
| *** tflynn has joined #openstack-neutron | 20:26 | |
| pc_m | madhu_ak: looking... | 20:26 |
| kevinbenton | haleyb: either that or i always assume a 3-char prefix and strip off the first 3 on the firewall side | 20:27 |
| haleyb | kevinbenton: yeah, you could do that, because otherwise you have to shorten everything by stripping-off known values, and use a smaller id space, like the first 8 chars of the uuid | 20:27 |
| kevinbenton | haleyb: yeah, 8 chars is okay | 20:27 |
| haleyb | kevinbenton: yes, the firewall code could always pass the whole uuid too to the zone mapping code | 20:27 |
| kevinbenton | haleyb: because if we have a collision we will already have an issue since that means two ports map to the same interface name | 20:27 |
| haleyb | kevinbenton: basically pass port['device'] | 20:28 |
| *** padkrish has quit IRC | 20:28 | |
| kevinbenton | haleyb: yep, that would work | 20:28 |
| kevinbenton | haleyb: so the only kinda messy assumption is on startup reading the interface names | 20:28 |
| *** ihrachyshka has quit IRC | 20:28 | |
| *** padkrish has joined #openstack-neutron | 20:29 | |
| *** abhiraut has joined #openstack-neutron | 20:29 | |
| kevinbenton | haleyb: and chopping off what we think is a prefix | 20:29 |
| *** piyanai has quit IRC | 20:29 | |
| haleyb | kevinbenton: yes, that's error prone since in reality you could have a uuid starting with qvb | 20:29 |
| *** ijw has joined #openstack-neutron | 20:29 | |
| *** piyanai has joined #openstack-neutron | 20:29 | |
| haleyb | but if you know it's a device you can just strip the first three chars, if not the last ? | 20:30 |
| *** apuimedo has joined #openstack-neutron | 20:30 | |
| kevinbenton | haleyb: right, i would just strip the first 3 | 20:30 |
| *** padkrish_ has joined #openstack-neutron | 20:30 | |
| *** padkrish has quit IRC | 20:30 | |
| kevinbenton | haleyb: but i think right now we are just lucky we happen to have all 3 char prefixes | 20:30 |
| kevinbenton | haleyb: do you think it's safe to codify that? | 20:31 |
| *** ijw_ has quit IRC | 20:32 | |
| *** piyanai has quit IRC | 20:32 | |
| pc_m | madhu_ak: Is the intent to do both API addition and refactoring changes? | 20:33 |
| haleyb | kevinbenton: you mean force a 3-char max prefix? i think so given the precendence we do already | 20:33 |
| haleyb | and we actually use 11 chars today not 8, but the max is 15 hence the +3 +NULL | 20:34 |
| madhu_ak | pc_m yep with the new job template, we can test all the three jobs including api and run them as experimental | 20:34 |
| pc_m | I see, using a new job name. | 20:35 |
| *** mwagner_lap has quit IRC | 20:35 | |
| madhu_ak | yeah, so it should just not disturb the current jobs upstream in vpnrepo | 20:35 |
| madhu_ak | pc_m, please let me know if I am missing something | 20:36 |
| pc_m | Looks fine, only suggestions on making commit message clearer. | 20:38 |
| *** Marga_ has quit IRC | 20:39 | |
| madhu_ak | pc_m sure, would like to see them | 20:39 |
| *** Marga_ has joined #openstack-neutron | 20:39 | |
| *** thumpba has quit IRC | 20:40 | |
| madhu_ak | pc_m addressed them | 20:41 |
| *** sthillma has joined #openstack-neutron | 20:41 | |
| *** nlahouti has joined #openstack-neutron | 20:42 | |
| *** vivek-ebay has joined #openstack-neutron | 20:42 | |
| pc_m | Looks good to me madhu_ak | 20:43 |
| *** tiswanso has quit IRC | 20:44 | |
| kevinbenton | haleyb: yeah | 20:44 |
| kevinbenton | haleyb: so i'll just assume all devs have a 3-char prefix | 20:45 |
| madhu_ak | thanks pc_m | 20:45 |
| kevinbenton | haleyb: and then key the dict on the 11 chars | 20:45 |
| kevinbenton | haleyb: sound good? | 20:45 |
| *** jpena|away has quit IRC | 20:45 | |
| *** carl_baldwin has quit IRC | 20:45 | |
| klindgren | carl_baldwin, ping | 20:45 |
| haleyb | kevinbenton: what about the call from the conntrack code? it just passes 11 ? | 20:45 |
| klindgren | *sigh* missed him by \-\ much | 20:45 |
| kevinbenton | haleyb: i'll just keep that logic in get_device_zone, have it take a port UUID and then it handles stripping it down | 20:46 |
| *** pc_m has quit IRC | 20:46 | |
| kevinbenton | haleyb: just to keep it easy for the caller | 20:46 |
| kevinbenton | haleyb: then each caller just passes the UUID | 20:46 |
| kevinbenton | haleyb: and the prefix stripping logic just lives in the initial zone map | 20:47 |
| *** TrevorV has quit IRC | 20:47 | |
| haleyb | kevinbenton: ah, ok, and not to be a pain, but using the constants like LINUX_DEV_LEN and TAP_DEVICE_PREFIX (or some new one set to 3) would be best so it's obvious | 20:48 |
| openstackgerrit | Merged openstack/neutron: Fix docs job https://review.openstack.org/212119 | 20:49 |
| kevinbenton | haleyb: ack | 20:49 |
| HenryG | haleyb: kevinbenton: can you look at https://review.openstack.org/205101 ? Infra would *really* like to see it merged. | 20:50 |
| HenryG | It quenches a lot of log noise. | 20:50 |
| haleyb | HenryG: looking | 20:50 |
| openstackgerrit | Ryan Moats proposed openstack/neutron: Preserve DVR FIP rule priority over Agent restarts https://review.openstack.org/193711 | 20:52 |
| *** thurloat has joined #openstack-neutron | 20:54 | |
| *** swat30 has joined #openstack-neutron | 20:54 | |
| kevinbenton | HenryG: ack | 20:55 |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: NSX: Move DB models as part of core vendor decomposition https://review.openstack.org/211370 | 20:55 |
| openstackgerrit | Merged openstack/neutron: Update fullstack multinode simulation image https://review.openstack.org/212124 | 20:57 |
| *** krtaylor has quit IRC | 20:58 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Use a conntrack zone per port in OVS https://review.openstack.org/207464 | 20:58 |
| *** abhiraut has quit IRC | 20:59 | |
| *** abhiraut has joined #openstack-neutron | 21:00 | |
| kevinbenton | HenryG: there is a bit of logic there i don't understand | 21:01 |
| *** achanda has joined #openstack-neutron | 21:01 | |
| kevinbenton | HenryG: now it only calls _release_lease if the dhcp_port_exists. why does the port existing have anything to do with the lease? | 21:02 |
| *** mickeys has joined #openstack-neutron | 21:04 | |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-vpnaas: New jobs for VPNaaS tests https://review.openstack.org/211768 | 21:07 |
| mlavalle | mestery: ping | 21:07 |
| mestery | mlavalle: pong | 21:07 |
| *** jckasper has quit IRC | 21:07 | |
| HenryG | kevinbenton: Good question | 21:08 |
| mlavalle | mestery: I am trying to add https://review.openstack.org/#/c/200952/ to your gerrit query screen | 21:08 |
| *** cbouch__ has quit IRC | 21:08 | |
| mlavalle | mestery: I reviewed the query, and it seems it should capture bp/internal-dns-resolution, wich is the topic I set for that patchset | 21:09 |
| mestery | mlavalle: I believe the custom gerrit query will fitler out patchs that you own :) | 21:09 |
| *** ijw has quit IRC | 21:09 | |
| mestery | mlavalle: IF you want, feel free to submit a followup which adds a section for patches you own | 21:09 |
| mestery | I think that's the issue there | 21:09 |
| mestery | I see your patch in my query mlavalle :) | 21:10 |
| mlavalle | mestery: ahhh! I should have suspected it :-) | 21:10 |
| *** ijw has joined #openstack-neutron | 21:10 | |
| mlavalle | mestery: ok... perfect you aware of that patchset | 21:10 |
| mlavalle | thanks! | 21:10 |
| mlavalle | mestery: it just needs the last push | 21:11 |
| *** alex_klimov has joined #openstack-neutron | 21:11 | |
| *** krtaylor has joined #openstack-neutron | 21:11 | |
| mestery | awesome thanks! | 21:11 |
| madhu_ak | pc_m pushed the latest patch: https://review.openstack.org/#/c/211768/ | 21:12 |
| *** Sukhdev_ has quit IRC | 21:13 | |
| *** yamamoto has joined #openstack-neutron | 21:13 | |
| *** thorst has quit IRC | 21:16 | |
| *** yamamoto has quit IRC | 21:19 | |
| *** crose has quit IRC | 21:21 | |
| *** erlarese has quit IRC | 21:22 | |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-vpnaas: New jobs for VPNaaS tests https://review.openstack.org/211768 | 21:23 |
| *** rpothier has quit IRC | 21:25 | |
| regXboi | so... who can help me with questions about POSTs to os-floating-ips? | 21:29 |
| regXboi | (yes, I know that is a nova URI) | 21:29 |
| regXboi | I'm looking at https://bugs.launchpad.net/neutron/+bug/1404743 and that's I'm seeing signatures related to os-floating-ips that look... well... odd | 21:31 |
| openstack | Launchpad bug 1404743 in neutron "sporadic test failures due to VMs not getting a DHCP lease" [Critical,Confirmed] - Assigned to Ryan Moats (rmoats) | 21:31 |
| *** rkukura has quit IRC | 21:32 | |
| *** gangil has joined #openstack-neutron | 21:35 | |
| *** johnbelamaric has joined #openstack-neutron | 21:36 | |
| openstackgerrit | Berezovsky Irena proposed openstack/neutron-specs: Update spec to reflect current QoS API https://review.openstack.org/199112 | 21:36 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-vpnaas: New jobs for VPNaaS tests https://review.openstack.org/211768 | 21:40 |
| *** mwagner_lap has joined #openstack-neutron | 21:41 | |
| *** abhiraut has quit IRC | 21:41 | |
| *** fawadkhaliq has joined #openstack-neutron | 21:43 | |
| *** zhhuabj has quit IRC | 21:47 | |
| *** abhiraut has joined #openstack-neutron | 21:47 | |
| *** fawadkhaliq has quit IRC | 21:50 | |
| *** tflynn has quit IRC | 21:50 | |
| *** merooney has joined #openstack-neutron | 21:52 | |
| *** annegentle has quit IRC | 21:54 | |
| *** yamamoto has joined #openstack-neutron | 21:57 | |
| *** mitchjameson has quit IRC | 21:57 | |
| *** agireud has quit IRC | 21:57 | |
| *** tsg has quit IRC | 21:58 | |
| *** padkrish_ has quit IRC | 22:01 | |
| *** padkrish has joined #openstack-neutron | 22:01 | |
| *** dsneddon_ has joined #openstack-neutron | 22:03 | |
| *** kbringard has quit IRC | 22:03 | |
| *** carl_baldwin has joined #openstack-neutron | 22:03 | |
| *** achippa_ has joined #openstack-neutron | 22:04 | |
| *** regXboi has quit IRC | 22:05 | |
| *** shwetaap1 has quit IRC | 22:05 | |
| *** dsneddon has quit IRC | 22:06 | |
| *** edmondsw has quit IRC | 22:06 | |
| *** zhhuabj has joined #openstack-neutron | 22:07 | |
| *** carl_baldwin has quit IRC | 22:09 | |
| *** roeyc has joined #openstack-neutron | 22:11 | |
| *** lindycoder has quit IRC | 22:12 | |
| *** zhhuabj has quit IRC | 22:15 | |
| *** jecarey has quit IRC | 22:18 | |
| *** zhhuabj has joined #openstack-neutron | 22:18 | |
| *** pradk has quit IRC | 22:18 | |
| *** tflynn has joined #openstack-neutron | 22:19 | |
| *** padkrish has quit IRC | 22:21 | |
| *** achippa_ has quit IRC | 22:24 | |
| *** achippa has joined #openstack-neutron | 22:24 | |
| *** padkrish_ has joined #openstack-neutron | 22:25 | |
| *** zhhuabj has quit IRC | 22:26 | |
| *** achippa has quit IRC | 22:29 | |
| *** krtaylor has quit IRC | 22:30 | |
| *** killer_prince has joined #openstack-neutron | 22:33 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Remove Cisco Meta and N1KV monolithic plugins https://review.openstack.org/212209 | 22:36 |
| *** roeyc has quit IRC | 22:36 | |
| *** lazy_prince has quit IRC | 22:36 | |
| *** dguerri` is now known as dguerri | 22:39 | |
| *** achanda has quit IRC | 22:40 | |
| *** phil_h has quit IRC | 22:40 | |
| *** mitchjameson has joined #openstack-neutron | 22:41 | |
| *** mitchjameson has quit IRC | 22:41 | |
| *** killer_prince is now known as lazy_prince | 22:41 | |
| *** rawl has joined #openstack-neutron | 22:41 | |
| abhiraut | ping HenryG | 22:41 |
| *** ivar-lazzaro has joined #openstack-neutron | 22:42 | |
| *** rawl has quit IRC | 22:42 | |
| *** dguerri is now known as dguerri` | 22:43 | |
| *** annegentle has joined #openstack-neutron | 22:44 | |
| *** devlaps has joined #openstack-neutron | 22:44 | |
| *** annegentle has quit IRC | 22:45 | |
| *** dane_leblanc__ has quit IRC | 22:45 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: NSX: Move DB models as part of core vendor decomposition https://review.openstack.org/211370 | 22:45 |
| *** annegentle has joined #openstack-neutron | 22:46 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: The unnecessary value "sgids" was deleted https://review.openstack.org/209552 | 22:47 |
| *** rawl has joined #openstack-neutron | 22:47 | |
| *** thurloat is now known as thurloat_isgone | 22:48 | |
| *** zhhuabj has joined #openstack-neutron | 22:50 | |
| *** johnbelamaric has quit IRC | 22:52 | |
| *** johnbelamaric has joined #openstack-neutron | 22:52 | |
| *** johnbelamaric has quit IRC | 22:52 | |
| *** annegentle has quit IRC | 22:52 | |
| *** merooney has quit IRC | 22:53 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: External DNS driver reference implementation https://review.openstack.org/212213 | 22:54 |
| *** yamamoto has quit IRC | 22:54 | |
| openstackgerrit | fumihiko kakuma proposed openstack/neutron: Python 3: specify a bytes to an argument for a format type 's' of struct.pack() https://review.openstack.org/210746 | 22:55 |
| *** moshele has joined #openstack-neutron | 22:56 | |
| *** busterswt has quit IRC | 22:56 | |
| *** zhhuabj has quit IRC | 22:57 | |
| *** mlavalle has quit IRC | 22:58 | |
| *** gsagie_ has quit IRC | 23:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/208705 | 23:09 |
| *** vivek-ebay has quit IRC | 23:09 | |
| *** alex_klimov has quit IRC | 23:11 | |
| *** zhhuabj has joined #openstack-neutron | 23:11 | |
| *** hichihara has joined #openstack-neutron | 23:11 | |
| *** moshele has quit IRC | 23:11 | |
| *** ijw_ has joined #openstack-neutron | 23:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/204422 | 23:13 |
| *** markvoelker has quit IRC | 23:14 | |
| *** ijw has quit IRC | 23:15 | |
| *** lazy_prince has quit IRC | 23:15 | |
| *** claudiub has quit IRC | 23:16 | |
| *** achanda has joined #openstack-neutron | 23:16 | |
| *** gangil has quit IRC | 23:17 | |
| *** agireud has joined #openstack-neutron | 23:17 | |
| *** carl_baldwin has joined #openstack-neutron | 23:18 | |
| *** vivek-ebay has joined #openstack-neutron | 23:18 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: The unnecessary value "sgids" was deleted https://review.openstack.org/209552 | 23:19 |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: judge ICMP port code in range [0,255] https://review.openstack.org/212220 | 23:19 |
| *** rawl has quit IRC | 23:23 | |
| *** suresh12 has quit IRC | 23:23 | |
| *** chlong has quit IRC | 23:25 | |
| *** carl_baldwin has quit IRC | 23:27 | |
| *** sridhar_ram1 has joined #openstack-neutron | 23:31 | |
| *** sridhar_ram has quit IRC | 23:32 | |
| *** salv-orl_ has joined #openstack-neutron | 23:35 | |
| *** banix has quit IRC | 23:36 | |
| *** mickeys has quit IRC | 23:37 | |
| *** salv-orlando has quit IRC | 23:37 | |
| *** sridhar_ram1 has quit IRC | 23:38 | |
| *** abhiraut has quit IRC | 23:39 | |
| *** ijw_ has quit IRC | 23:44 | |
| *** geekinutah has quit IRC | 23:44 | |
| *** ijw has joined #openstack-neutron | 23:44 | |
| *** abhiraut has joined #openstack-neutron | 23:46 | |
| *** oomichi has joined #openstack-neutron | 23:51 | |
| *** Marga_ has quit IRC | 23:52 | |
| *** baoli has quit IRC | 23:52 | |
| *** yamamoto has joined #openstack-neutron | 23:55 | |
| *** suresh12 has joined #openstack-neutron | 23:56 | |
| *** baoli has joined #openstack-neutron | 23:56 | |
| *** veena has joined #openstack-neutron | 23:57 | |
| *** minwang2 has quit IRC | 23:58 | |
| *** crc32 has quit IRC | 23:58 | |
| *** achanda has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!