*** Kennan2 is now known as Kennan | 00:02 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: L3 DVR developer document https://review.openstack.org/182917 | 00:02 |
---|---|---|
Fdaisuke | armax: Hi, | 00:03 |
*** salv-orlando has quit IRC | 00:03 | |
*** bharath has left #openstack-neutron | 00:04 | |
*** anshul has quit IRC | 00:04 | |
Fdaisuke | armax: I commented and uploaded new patch set to https://review.openstack.org/#/c/207328/ | 00:06 |
Fdaisuke | Would you please have a look at this? | 00:06 |
*** dsanders has quit IRC | 00:13 | |
*** agireud has joined #openstack-neutron | 00:15 | |
*** agireud_ has joined #openstack-neutron | 00:19 | |
*** bharathm has joined #openstack-neutron | 00:19 | |
*** agireud has quit IRC | 00:21 | |
*** agireud_ is now known as agireud | 00:21 | |
*** dannywil_ has joined #openstack-neutron | 00:21 | |
*** shadower49 has quit IRC | 00:23 | |
*** tiswanso has joined #openstack-neutron | 00:23 | |
*** shadower49 has joined #openstack-neutron | 00:23 | |
*** dannywilson has quit IRC | 00:24 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting DVR https://review.openstack.org/215655 | 00:26 |
openstackgerrit | Miguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports https://review.openstack.org/200952 | 00:27 |
*** britthouser has joined #openstack-neutron | 00:28 | |
*** banix has quit IRC | 00:28 | |
openstackgerrit | Kenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS https://review.openstack.org/148416 | 00:30 |
*** tiswanso has quit IRC | 00:36 | |
*** tiswanso has joined #openstack-neutron | 00:37 | |
*** dsanders has joined #openstack-neutron | 00:37 | |
*** julim has quit IRC | 00:39 | |
*** julim has joined #openstack-neutron | 00:40 | |
*** RA_ has joined #openstack-neutron | 00:42 | |
*** tiswanso has quit IRC | 00:44 | |
*** rushil has joined #openstack-neutron | 00:44 | |
*** bharathm has quit IRC | 00:44 | |
*** tflynn_ has quit IRC | 00:44 | |
*** amotoki has joined #openstack-neutron | 00:47 | |
*** rotbeard has quit IRC | 00:50 | |
*** amotoki has quit IRC | 00:50 | |
*** rotbeard has joined #openstack-neutron | 00:50 | |
*** tiswanso has joined #openstack-neutron | 00:51 | |
*** tflynn has joined #openstack-neutron | 00:53 | |
*** amotoki has joined #openstack-neutron | 00:56 | |
*** tiswanso has quit IRC | 00:59 | |
*** agireud has quit IRC | 00:59 | |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally https://review.openstack.org/213781 | 01:01 |
*** zhenguo has joined #openstack-neutron | 01:01 | |
*** salv-orlando has joined #openstack-neutron | 01:03 | |
*** banix has joined #openstack-neutron | 01:03 | |
*** yanxb__ has quit IRC | 01:04 | |
*** yanxb has joined #openstack-neutron | 01:04 | |
*** salv-orlando has quit IRC | 01:05 | |
*** rossella_s has quit IRC | 01:05 | |
*** rossella_s has joined #openstack-neutron | 01:06 | |
*** tflynn has quit IRC | 01:06 | |
openstackgerrit | YujiAzama proposed openstack/neutron-specs: [WIP] Add common classifier resource https://review.openstack.org/190463 | 01:08 |
*** agireud has joined #openstack-neutron | 01:09 | |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity https://review.openstack.org/213819 | 01:09 |
*** salv-orlando has joined #openstack-neutron | 01:11 | |
*** tiswanso has joined #openstack-neutron | 01:13 | |
openstackgerrit | Akihiro Motoki proposed openstack/neutron: Decomposition phase2 of NEC plugin https://review.openstack.org/216071 | 01:13 |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally https://review.openstack.org/213781 | 01:13 |
*** nplanel has joined #openstack-neutron | 01:16 | |
*** nlahouti has quit IRC | 01:16 | |
*** salv-orlando has quit IRC | 01:21 | |
*** nplanel has quit IRC | 01:21 | |
*** pavel_bondar has joined #openstack-neutron | 01:25 | |
*** achanda has joined #openstack-neutron | 01:26 | |
*** tiswanso has quit IRC | 01:29 | |
*** shwetaap has quit IRC | 01:29 | |
*** tiswanso has joined #openstack-neutron | 01:30 | |
*** diegows has quit IRC | 01:30 | |
openstackgerrit | Kenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS https://review.openstack.org/148416 | 01:31 |
*** shivrao has quit IRC | 01:32 | |
*** erkules_ has joined #openstack-neutron | 01:33 | |
*** stanzgy has joined #openstack-neutron | 01:34 | |
*** agireud has quit IRC | 01:35 | |
*** erkules has quit IRC | 01:35 | |
*** iwamoto has joined #openstack-neutron | 01:37 | |
*** agireud has joined #openstack-neutron | 01:37 | |
*** yalie has joined #openstack-neutron | 01:38 | |
*** yalie has left #openstack-neutron | 01:38 | |
*** piyanai has quit IRC | 01:44 | |
*** bharathm has joined #openstack-neutron | 01:44 | |
*** bharathm has quit IRC | 01:49 | |
*** zhhuabj_ has quit IRC | 01:49 | |
*** jbell8 has quit IRC | 01:52 | |
*** agireud has quit IRC | 01:56 | |
*** hichihara has joined #openstack-neutron | 02:00 | |
*** Kennan has quit IRC | 02:02 | |
*** Kennan has joined #openstack-neutron | 02:02 | |
*** agireud has joined #openstack-neutron | 02:04 | |
*** liudong has joined #openstack-neutron | 02:05 | |
*** jingbo has joined #openstack-neutron | 02:06 | |
*** changzhi has joined #openstack-neutron | 02:07 | |
*** yamamoto has joined #openstack-neutron | 02:07 | |
jingbo | Hi, I have a problem while I am debuging q-agt. | 02:07 |
jingbo | DEBUG neutron.agent.linux.utils [-] Running command: ['sudo', '/usr/local/bin/neutron-rootwrap', '/etc/neutron/rootwrap.conf', 'ovs-vsctl', '--timeout=10', '--', '--may-exist', 'add-br', 'br-int'] create_process /opt/stack/neutron/neutron/agent/linux/utils.py:46 | 02:07 |
*** agireud has quit IRC | 02:08 | |
jingbo | It stopped at this step while I am using pydev to debug the q-agt. | 02:08 |
jingbo | Any suggestion? | 02:08 |
*** reedip has quit IRC | 02:09 | |
*** tiswanso has quit IRC | 02:12 | |
*** tiswanso has joined #openstack-neutron | 02:13 | |
*** shwetaap has joined #openstack-neutron | 02:17 | |
*** yamamoto has quit IRC | 02:18 | |
*** jamielennox is now known as jamielennox|away | 02:20 | |
*** tiswanso has quit IRC | 02:24 | |
*** tiswanso has joined #openstack-neutron | 02:25 | |
*** sthillma has joined #openstack-neutron | 02:30 | |
*** jamielennox|away is now known as jamielennox | 02:32 | |
*** britthouser has quit IRC | 02:34 | |
*** echirivella has quit IRC | 02:38 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting DVR https://review.openstack.org/215655 | 02:41 |
*** dsanders has quit IRC | 02:42 | |
*** nplanel has joined #openstack-neutron | 02:45 | |
*** shivrao has joined #openstack-neutron | 02:47 | |
*** bharathm has joined #openstack-neutron | 02:47 | |
*** salv-orlando has joined #openstack-neutron | 02:48 | |
*** shivrao has quit IRC | 02:51 | |
*** nplanel has quit IRC | 02:52 | |
*** bharathm has quit IRC | 02:52 | |
*** hakimo has joined #openstack-neutron | 02:52 | |
*** hakimo_ has quit IRC | 02:55 | |
*** zhhuabj has joined #openstack-neutron | 02:56 | |
*** rushil has quit IRC | 02:57 | |
*** jbell8 has joined #openstack-neutron | 02:58 | |
*** gberginc has joined #openstack-neutron | 03:00 | |
*** salv-orlando has quit IRC | 03:01 | |
*** dsanders has joined #openstack-neutron | 03:03 | |
*** yamamoto has joined #openstack-neutron | 03:04 | |
*** Aish has joined #openstack-neutron | 03:06 | |
*** y_sawai_ has quit IRC | 03:06 | |
Aish | Hi, I am writing rally scenario tests for vpnaas. When I try to import “from neutron.agent.linux import ip_lib , from neutron.agent.linux.utils import utils as linux_utils” these files, It errors out with arguments already parsed: cannot register CLI option.. | 03:08 |
Aish | Removing these two imports, my tests run fine.. | 03:09 |
*** tflynn has joined #openstack-neutron | 03:11 | |
Aish | Any idea why this haapens? | 03:11 |
*** tiswanso has quit IRC | 03:11 | |
*** nplanel has joined #openstack-neutron | 03:14 | |
*** Aish has left #openstack-neutron | 03:23 | |
*** outworlder has quit IRC | 03:24 | |
*** dsanders has quit IRC | 03:26 | |
*** VikasC has joined #openstack-neutron | 03:28 | |
*** vivek-ebay has joined #openstack-neutron | 03:29 | |
*** julim has quit IRC | 03:30 | |
*** openstackgerrit has quit IRC | 03:31 | |
*** openstackgerrit has joined #openstack-neutron | 03:32 | |
*** iwamoto has quit IRC | 03:36 | |
*** saksham has joined #openstack-neutron | 03:37 | |
*** sdake has quit IRC | 03:38 | |
*** stanzgy__ has joined #openstack-neutron | 03:45 | |
*** stanzgy has quit IRC | 03:47 | |
*** lykinsbd has joined #openstack-neutron | 03:49 | |
*** jingbo has quit IRC | 03:55 | |
openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources https://review.openstack.org/213586 | 03:59 |
*** hanzhang has joined #openstack-neutron | 04:03 | |
hanzhang | ping ajo | 04:03 |
*** mkolesni_ has joined #openstack-neutron | 04:06 | |
*** coolsvap|away is now known as coolsvap | 04:06 | |
openstackgerrit | Merged openstack/neutron: Add support for unaddressed port https://review.openstack.org/200809 | 04:09 |
*** vivek-ebay has quit IRC | 04:09 | |
*** stanzgy__ has quit IRC | 04:11 | |
*** gberginc has quit IRC | 04:13 | |
*** saksham_ has joined #openstack-neutron | 04:15 | |
*** miyagishi_t has joined #openstack-neutron | 04:16 | |
*** regXboi has quit IRC | 04:17 | |
*** vivek-ebay has joined #openstack-neutron | 04:17 | |
*** saksham has quit IRC | 04:18 | |
*** saksham_ is now known as saksham | 04:18 | |
*** shivrao has joined #openstack-neutron | 04:19 | |
*** dsanders has joined #openstack-neutron | 04:23 | |
*** salv-orlando has joined #openstack-neutron | 04:24 | |
*** BhavyaM has joined #openstack-neutron | 04:24 | |
*** mohankumar has joined #openstack-neutron | 04:28 | |
*** stanzgy has joined #openstack-neutron | 04:28 | |
*** ircuser-1 has joined #openstack-neutron | 04:32 | |
*** salv-orlando has quit IRC | 04:34 | |
*** mohankumar has quit IRC | 04:34 | |
*** hoangcx has joined #openstack-neutron | 04:36 | |
*** bharathm has joined #openstack-neutron | 04:36 | |
*** hoangcx has quit IRC | 04:37 | |
*** hoangcx has joined #openstack-neutron | 04:38 | |
*** ajmiller has quit IRC | 04:38 | |
*** fawadkhaliq has joined #openstack-neutron | 04:38 | |
*** salv-orlando has joined #openstack-neutron | 04:39 | |
*** Ramanjaneya has joined #openstack-neutron | 04:40 | |
*** hoangcx has quit IRC | 04:40 | |
*** bharathm has quit IRC | 04:40 | |
*** changzhi has quit IRC | 04:41 | |
*** hoangcx has joined #openstack-neutron | 04:41 | |
*** hoangcx has quit IRC | 04:42 | |
*** Aish has joined #openstack-neutron | 04:42 | |
*** hoangcx has joined #openstack-neutron | 04:42 | |
*** hoangcx has quit IRC | 04:43 | |
*** nplanel_ has joined #openstack-neutron | 04:43 | |
openstackgerrit | Gal Sagie proposed openstack/neutron-specs: Create resource with fixed id spec https://review.openstack.org/215079 | 04:44 |
*** salv-orlando has quit IRC | 04:45 | |
*** BhavyaM has quit IRC | 04:49 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Enable to update external network subnet's gateway-ip https://review.openstack.org/194939 | 04:50 |
*** fedexo has quit IRC | 04:57 | |
*** tflynn has quit IRC | 04:57 | |
*** jckasper has quit IRC | 04:58 | |
*** tflynn has joined #openstack-neutron | 04:58 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB https://review.openstack.org/183369 | 04:59 |
*** rossella_s has quit IRC | 05:06 | |
*** rossella_s has joined #openstack-neutron | 05:06 | |
*** iwamoto has joined #openstack-neutron | 05:08 | |
*** Aish has left #openstack-neutron | 05:09 | |
*** kiran-r has joined #openstack-neutron | 05:10 | |
*** salv-orlando has joined #openstack-neutron | 05:10 | |
*** shaohe_feng has joined #openstack-neutron | 05:13 | |
*** shausy has joined #openstack-neutron | 05:17 | |
*** mohankumar has joined #openstack-neutron | 05:19 | |
*** numan has joined #openstack-neutron | 05:21 | |
*** anilvenkata has joined #openstack-neutron | 05:23 | |
*** vivek-ebay has quit IRC | 05:24 | |
*** shaohe_feng has quit IRC | 05:24 | |
*** shaohe_feng has joined #openstack-neutron | 05:25 | |
*** lazy_prince has joined #openstack-neutron | 05:25 | |
*** dsanders has quit IRC | 05:26 | |
*** numan has quit IRC | 05:26 | |
*** lajos-katona has joined #openstack-neutron | 05:27 | |
*** gberginc has joined #openstack-neutron | 05:28 | |
*** tflynn_ has joined #openstack-neutron | 05:29 | |
*** gardenshed has joined #openstack-neutron | 05:31 | |
*** tflynn has quit IRC | 05:32 | |
*** changzhi has joined #openstack-neutron | 05:33 | |
*** srilatha has joined #openstack-neutron | 05:35 | |
*** numan has joined #openstack-neutron | 05:38 | |
*** mohankumar has quit IRC | 05:38 | |
*** gardenshed has quit IRC | 05:39 | |
*** salv-orlando has quit IRC | 05:45 | |
*** jamielennox is now known as jamielennox|away | 05:47 | |
*** yonglihe has quit IRC | 05:48 | |
*** bharathm has joined #openstack-neutron | 05:50 | |
*** shwetaap has quit IRC | 05:51 | |
*** SridharG has joined #openstack-neutron | 05:52 | |
*** mickeys has joined #openstack-neutron | 05:52 | |
*** shivrao has quit IRC | 05:54 | |
*** yonglihe has joined #openstack-neutron | 05:54 | |
*** srilatha has quit IRC | 05:54 | |
*** srilatha has joined #openstack-neutron | 05:55 | |
*** bharath has joined #openstack-neutron | 05:56 | |
*** yfried|afk has quit IRC | 05:56 | |
*** shaohe_feng1 has joined #openstack-neutron | 05:58 | |
*** srilatha has quit IRC | 05:59 | |
*** shaohe_feng has quit IRC | 06:00 | |
*** shausy has quit IRC | 06:04 | |
*** Murali has joined #openstack-neutron | 06:04 | |
*** neeti has joined #openstack-neutron | 06:05 | |
*** abregman has joined #openstack-neutron | 06:05 | |
bharath | L3Natagent self.router_info is not getting populated. Which is causing FW breakage. Can someone help me out | 06:08 |
*** shausy has joined #openstack-neutron | 06:08 | |
*** mickeys has quit IRC | 06:11 | |
*** shaohe_feng has joined #openstack-neutron | 06:12 | |
*** irenab has quit IRC | 06:13 | |
*** shaohe_feng1 has quit IRC | 06:13 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler https://review.openstack.org/204436 | 06:15 |
*** ameybhide has joined #openstack-neutron | 06:17 | |
*** ddepaoli has joined #openstack-neutron | 06:18 | |
*** jbell8 has quit IRC | 06:20 | |
*** saksham has quit IRC | 06:20 | |
*** diga has joined #openstack-neutron | 06:22 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler https://review.openstack.org/204436 | 06:23 |
*** shaohe_feng1 has joined #openstack-neutron | 06:23 | |
*** _kiran_ has joined #openstack-neutron | 06:25 | |
*** shausy has quit IRC | 06:25 | |
*** shaohe_feng has quit IRC | 06:25 | |
*** shausy has joined #openstack-neutron | 06:26 | |
openstackgerrit | Li Ma proposed openstack/neutron: Validate local_ip for linuxbridge-agent https://review.openstack.org/211443 | 06:27 |
*** kiran-r has quit IRC | 06:27 | |
*** trinaths has joined #openstack-neutron | 06:28 | |
*** slaweq has joined #openstack-neutron | 06:28 | |
*** itzikb has joined #openstack-neutron | 06:30 | |
openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources https://review.openstack.org/213586 | 06:31 |
*** armax has quit IRC | 06:31 | |
*** killer_prince has joined #openstack-neutron | 06:34 | |
*** trinaths has left #openstack-neutron | 06:35 | |
*** neeti has quit IRC | 06:36 | |
*** lazy_prince has quit IRC | 06:37 | |
*** ildikov has quit IRC | 06:38 | |
*** yalie has joined #openstack-neutron | 06:44 | |
*** anshul has joined #openstack-neutron | 06:44 | |
*** ihrachyshka has joined #openstack-neutron | 06:46 | |
openstackgerrit | Eric Brown proposed openstack/neutron: Use min and max on IntOpt option types https://review.openstack.org/216119 | 06:47 |
*** achanda has quit IRC | 06:48 | |
*** irenab has joined #openstack-neutron | 06:48 | |
openstackgerrit | Li Ma proposed openstack/neutron-vpnaas: Add test_cisco_ipsec.py module for py34 support https://review.openstack.org/215515 | 06:49 |
openstackgerrit | Li Ma proposed openstack/neutron-vpnaas: Add cisco_csr_rest_client.py module for py34 support https://review.openstack.org/215520 | 06:50 |
*** moshele has joined #openstack-neutron | 06:50 | |
*** abregman_ has joined #openstack-neutron | 06:52 | |
*** abregman has quit IRC | 06:55 | |
*** nmagnezi has joined #openstack-neutron | 06:56 | |
*** bharathm has quit IRC | 06:57 | |
*** itzikb has quit IRC | 06:58 | |
*** itzikb_ has joined #openstack-neutron | 06:58 | |
*** lukasa has joined #openstack-neutron | 06:59 | |
*** jpena has joined #openstack-neutron | 07:03 | |
*** evgenyf has joined #openstack-neutron | 07:05 | |
*** haigang has joined #openstack-neutron | 07:05 | |
*** tflynn_ has quit IRC | 07:06 | |
*** haigang has quit IRC | 07:06 | |
*** changzhi has quit IRC | 07:06 | |
*** I has joined #openstack-neutron | 07:06 | |
*** I is now known as Guest63057 | 07:07 | |
*** banix has quit IRC | 07:07 | |
*** Guest63057 has quit IRC | 07:07 | |
*** haigang has joined #openstack-neutron | 07:07 | |
*** y_sawai has joined #openstack-neutron | 07:07 | |
*** devvesa has joined #openstack-neutron | 07:07 | |
openstackgerrit | Christian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide https://review.openstack.org/216125 | 07:08 |
*** itzikb has joined #openstack-neutron | 07:08 | |
*** dannywil_ has quit IRC | 07:08 | |
*** RA_ has quit IRC | 07:09 | |
*** itzikb_ has quit IRC | 07:09 | |
*** neeti has joined #openstack-neutron | 07:09 | |
*** haigang has quit IRC | 07:09 | |
*** rubasov has joined #openstack-neutron | 07:10 | |
*** haigang has joined #openstack-neutron | 07:10 | |
*** ronis has joined #openstack-neutron | 07:11 | |
*** ronis_ has joined #openstack-neutron | 07:12 | |
openstackgerrit | Christian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide https://review.openstack.org/216125 | 07:14 |
*** ronis has quit IRC | 07:15 | |
*** ildikov has joined #openstack-neutron | 07:17 | |
*** _kiran_ has quit IRC | 07:17 | |
*** slaweq has quit IRC | 07:18 | |
*** slaweq has joined #openstack-neutron | 07:18 | |
*** hichihara has quit IRC | 07:20 | |
*** saksham has joined #openstack-neutron | 07:20 | |
*** luqas has joined #openstack-neutron | 07:20 | |
bharath | facing below error in neutron. | 07:20 |
bharath | InvalidRequestError: When initializing mapper Mapper|Network|networks, expression 'NetworkRBAC' failed to locate a name ("name 'NetworkRBAC' is not defined"). If this is a class name, consider adding this relationship() to the <class 'neutron.db.models_v2.Network'> class after both dependent classes have been defined. | 07:20 |
*** saksham has left #openstack-neutron | 07:20 | |
bharath | can someone help me out | 07:21 |
*** apuimedo has joined #openstack-neutron | 07:21 | |
*** john-davidge has joined #openstack-neutron | 07:21 | |
apuimedo | diga: Both tfukushima and I posted some reviews on your patch | 07:23 |
*** gardenshed has joined #openstack-neutron | 07:24 | |
*** mohankumar has joined #openstack-neutron | 07:25 | |
*** yamamoto has quit IRC | 07:26 | |
*** drankis has joined #openstack-neutron | 07:26 | |
*** ushkalim has joined #openstack-neutron | 07:26 | |
*** VikasC_ has joined #openstack-neutron | 07:27 | |
*** y_sawai has quit IRC | 07:28 | |
*** y_sawai has joined #openstack-neutron | 07:28 | |
*** gardenshed has quit IRC | 07:28 | |
*** yonglihe has quit IRC | 07:29 | |
*** VikasC has quit IRC | 07:30 | |
*** gowri has joined #openstack-neutron | 07:32 | |
*** sudipto has joined #openstack-neutron | 07:33 | |
*** neeti has quit IRC | 07:33 | |
*** sgordon has quit IRC | 07:33 | |
*** sgordon has joined #openstack-neutron | 07:34 | |
*** jlibosva has joined #openstack-neutron | 07:34 | |
*** jschwarz has joined #openstack-neutron | 07:35 | |
*** mohankumar has quit IRC | 07:35 | |
mattt | ihrachyshka: hey, got a sec? | 07:36 |
ihrachyshka | mattt, yeap. | 07:36 |
*** yfried|afk has joined #openstack-neutron | 07:37 | |
*** jlibosva1 has joined #openstack-neutron | 07:37 | |
mattt | ihrachyshka: since you're working on neutron-db-manage, would it be possible to modify 'current' so that it lists the branch in the output? | 07:38 |
ihrachyshka | mattt, hm, I think I am dragged into a meeting, so can be unresponsive next 30 mins | 07:38 |
mattt | ihrachyshka: no worries, catch you later | 07:38 |
*** neeti has joined #openstack-neutron | 07:38 | |
*** gongysh has joined #openstack-neutron | 07:38 | |
*** fzdarsky has joined #openstack-neutron | 07:39 | |
ihrachyshka | mattt, the way it's implemented now, it just passes the command to 'alembic current', so the output is defined there. of course we could reimplement the command ourselves to accomodate it, but why do you need it? | 07:39 |
*** yfried has joined #openstack-neutron | 07:40 | |
mattt | ihrachyshka: right now using automation i cannot determine if i have to run online/offline migrations | 07:41 |
mattt | ihrachyshka: so i always have to shut down neutron to presumably run offline migrations | 07:41 |
*** nplanel has quit IRC | 07:41 | |
*** Zhang has joined #openstack-neutron | 07:41 | |
mattt | ihrachyshka: if that spat out the branch i could determine if either are not on head and then upgrade accordingly | 07:41 |
ihrachyshka | mattt, btw there are patches in flight that will remove kilo from 'HEADS' file and will also rename liberty_expand into just expand and liberty_contract into just contract | 07:41 |
*** yfried|afk has quit IRC | 07:41 | |
ihrachyshka | mattt, why don't you just unconditionally run upgrade expand@head, then shutdown and proceed with upgrade contract@head? | 07:42 |
ihrachyshka | if there is nothing to run, the first step will be no-op | 07:42 |
mattt | ihrachyshka: well what i want to avoid doing is shutting down neutron-server if i don't have to | 07:42 |
mattt | ihrachyshka: if i can grep for contract and see if i'm alraedy on head, i can avoid shutting down neutron-server and running the contract migrations | 07:43 |
ihrachyshka | roger. so what you really is interested is to determine whether contract db head is the alembic chain contract head | 07:43 |
mattt | ihrachyshka: exactly, that would be super helpful for us | 07:44 |
mattt | ihrachyshka: then using automation we can determine if we need to run migrations or not, the expand ones are a non-issue but obviously shutting down services unnecessarily isn't great | 07:44 |
*** salv-orlando has joined #openstack-neutron | 07:44 | |
*** jlibosva has quit IRC | 07:44 | |
miyagishi_t | fujitsu | 07:45 |
miyagishi_t | i made a mistake. | 07:47 |
*** alex_klimov has joined #openstack-neutron | 07:47 | |
*** nyechiel has joined #openstack-neutron | 07:47 | |
*** yamamoto has joined #openstack-neutron | 07:49 | |
*** salv-orlando has quit IRC | 07:50 | |
*** rotbeard has quit IRC | 07:50 | |
*** doude has joined #openstack-neutron | 07:50 | |
*** doude has quit IRC | 07:50 | |
*** doude has joined #openstack-neutron | 07:50 | |
*** rotbeard has joined #openstack-neutron | 07:51 | |
*** numan has quit IRC | 07:52 | |
diga | apuimedo: I saw your comments | 07:52 |
diga | apuimedo: just 1 question | 07:52 |
*** shihanzhang has joined #openstack-neutron | 07:52 | |
*** fawadkhaliq has quit IRC | 07:53 | |
*** alex_klimov has quit IRC | 07:53 | |
diga | can I use oslo_concurrency processutils instead of subprocess | 07:53 |
*** jlibosva1 has quit IRC | 07:54 | |
*** hanzhang has quit IRC | 07:55 | |
*** nmagnezi has quit IRC | 07:56 | |
*** alex_klimov has joined #openstack-neutron | 08:00 | |
*** lajos-katona1 has joined #openstack-neutron | 08:01 | |
*** lajos-katona has quit IRC | 08:02 | |
*** matrohon has joined #openstack-neutron | 08:04 | |
*** haigang has quit IRC | 08:06 | |
*** y_sawai has quit IRC | 08:06 | |
*** fawadkhaliq has joined #openstack-neutron | 08:07 | |
*** l6unchpad has joined #openstack-neutron | 08:08 | |
*** changzhi has joined #openstack-neutron | 08:09 | |
*** scheuran has joined #openstack-neutron | 08:09 | |
gongysh | amotoki: ping | 08:10 |
*** abregman_ is now known as abregman | 08:10 | |
openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2). https://review.openstack.org/189655 | 08:10 |
gongysh | amotoki: https://review.openstack.org/#/c/214981/ | 08:10 |
*** nmagnezi has joined #openstack-neutron | 08:11 | |
gongysh | amotoki: I have replied to your comments, what do u think of it? | 08:11 |
*** jpena is now known as jpena|away | 08:11 | |
*** Zhang has quit IRC | 08:12 | |
openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow implementation https://review.openstack.org/153946 | 08:12 |
openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Switch the default to "native" of_interface https://review.openstack.org/206329 | 08:14 |
*** scheuran has quit IRC | 08:15 | |
*** jlibosva has joined #openstack-neutron | 08:15 | |
*** jistr has joined #openstack-neutron | 08:17 | |
ihrachyshka | mattt, why would you run migrations at all if you don't need new features (they won't be enabled before you restart the service)? | 08:19 |
*** numan has joined #openstack-neutron | 08:20 | |
*** yanxb__ has joined #openstack-neutron | 08:22 | |
*** yanxb has quit IRC | 08:22 | |
*** yanxb_ has quit IRC | 08:22 | |
*** luqas has quit IRC | 08:22 | |
openstackgerrit | yalei wang proposed openstack/neutron: Add verification for device_owner when missing the port_security attr https://review.openstack.org/216143 | 08:23 |
*** yanxb_ has joined #openstack-neutron | 08:23 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: test_ovs_lib: Update a comment https://review.openstack.org/216144 | 08:23 |
mattt | ihrachyshka: it's easier for us to apply migrations when we upgrade, we deploy from head so keeping track of which migration may actually be used or not is too much effort | 08:24 |
*** ygbo has joined #openstack-neutron | 08:24 | |
*** julien_ZTE has joined #openstack-neutron | 08:25 | |
ihrachyshka | mattt, got it. I think we won't be able to get to it in L, can you report a RFE bug? | 08:25 |
*** zhhuabj has quit IRC | 08:26 | |
*** zhhuabj has joined #openstack-neutron | 08:26 | |
*** shausy has quit IRC | 08:26 | |
mattt | ihrachyshka: RFE? what is that ... | 08:26 |
*** shausy has joined #openstack-neutron | 08:26 | |
ihrachyshka | mattt, request for enhancement: http://docs.openstack.org/developer/neutron/policies/blueprints.html#neutron-request-for-feature-enhancements | 08:27 |
mattt | ihrachyshka: ok certainly, let me draft up something today | 08:28 |
*** zhhuabj has quit IRC | 08:28 | |
*** zhhuabj has joined #openstack-neutron | 08:28 | |
ihrachyshka | mattt, thanks. | 08:28 |
openstackgerrit | yalei wang proposed openstack/neutron: Add verification for device_owner when missing the port_security attr https://review.openstack.org/216143 | 08:29 |
*** zhhuabj has quit IRC | 08:29 | |
*** zhhuabj has joined #openstack-neutron | 08:30 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Move db agent schedulers test to a more appropriate place https://review.openstack.org/214131 | 08:30 |
*** zhhuabj has quit IRC | 08:30 | |
*** zhhuabj has joined #openstack-neutron | 08:30 | |
*** zhhuabj has quit IRC | 08:31 | |
*** zhhuabj has joined #openstack-neutron | 08:31 | |
*** yfried is now known as yfried|afk | 08:32 | |
*** sthillma has quit IRC | 08:34 | |
*** zhhuabj has quit IRC | 08:34 | |
*** zhhuabj has joined #openstack-neutron | 08:34 | |
*** yfried|afk is now known as yfried | 08:35 | |
*** sthillma has joined #openstack-neutron | 08:36 | |
*** yog_ has joined #openstack-neutron | 08:36 | |
apuimedo | diga: what's the question? | 08:38 |
diga | apuimedo: can I use oslo_concurrency processutils instead of subprocess | 08:39 |
*** zhhuabj has quit IRC | 08:39 | |
*** zhhuabj has joined #openstack-neutron | 08:39 | |
*** zhhuabj has quit IRC | 08:41 | |
*** VikasC has joined #openstack-neutron | 08:41 | |
*** zhhuabj has joined #openstack-neutron | 08:41 | |
apuimedo | diga: it's okay for me. gsagie: irenab: yamamoto: what do you think? | 08:42 |
*** SourabhP has joined #openstack-neutron | 08:42 | |
irenab | apuimedo: diga :+1 | 08:42 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: test_ovs_neutron_agent: Fix test_cleanup_stale_flows_iter_0 https://review.openstack.org/216152 | 08:42 |
*** VikasC_ has quit IRC | 08:42 | |
yamamoto | apuimedo: diga: +1 | 08:43 |
*** jistr has quit IRC | 08:45 | |
diga | thanks apuimedo, irenab | 08:45 |
diga | yamamoto: thanks | 08:45 |
diga | will use that module | 08:46 |
*** neophy has joined #openstack-neutron | 08:46 | |
yamamoto | akamyshnikova: ping | 08:47 |
*** rotbeard has quit IRC | 08:47 | |
irenab | diga: apuimedo : I think we will use oslo.config for kuryr configuration | 08:47 |
*** jistr has joined #openstack-neutron | 08:47 | |
*** slaweq has quit IRC | 08:48 | |
apuimedo | irenab: great! | 08:48 |
apuimedo | The more oslo the better | 08:48 |
*** shadower49 is now known as shadower | 08:48 | |
* apuimedo likes small codebases | 08:48 | |
*** SourabhP_ has joined #openstack-neutron | 08:48 | |
diga | irenab: +1 | 08:48 |
irenab | apuimedo: :-) | 08:49 |
*** Zhang has joined #openstack-neutron | 08:49 | |
*** jpena|away is now known as jpena | 08:49 | |
*** Zhang has quit IRC | 08:51 | |
ihrachyshka | ajo, what blocks us from merging https://review.openstack.org/#/c/214249/ ? | 08:51 |
*** SourabhP has quit IRC | 08:51 | |
*** SourabhP_ is now known as SourabhP | 08:51 | |
*** fawadkhaliq has quit IRC | 08:52 | |
*** iwamoto has quit IRC | 08:53 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [DevStack] Fix StrongSwan setup on Fedora https://review.openstack.org/215710 | 08:53 |
*** Marga_ has joined #openstack-neutron | 08:53 | |
*** Marga_ has quit IRC | 08:53 | |
*** Marga_ has joined #openstack-neutron | 08:54 | |
ihrachyshka | ajo, btw it's cool that everyone is deep into dscp discussion but I think we have at least upgrade story to solve first | 08:54 |
*** ivar-lazzaro has joined #openstack-neutron | 08:56 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 08:57 |
*** changzhi has quit IRC | 08:57 | |
*** sthillma has quit IRC | 09:02 | |
*** julien_ZTE has quit IRC | 09:06 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 09:07 |
*** ivar-lazzaro has quit IRC | 09:07 | |
*** ivar-lazzaro has joined #openstack-neutron | 09:07 | |
*** y_sawai has joined #openstack-neutron | 09:07 | |
*** jistr has quit IRC | 09:08 | |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports https://review.openstack.org/210301 | 09:08 |
ajo | ihrachyshka, we won't need that until we switch any object version :) | 09:09 |
ajo | ihrachyshka, but I have solved it on my mind :D I can share the idea if you want, | 09:09 |
ajo | ihrachyshka, no manual config, intervention or rolling stuff, all automated and | 09:09 |
ajo | *VERY SIMPLE* :-) | 09:09 |
ihrachyshka | ajo, once you add a new rule type, you change the version | 09:09 |
ihrachyshka | because it will get into policy[rules] | 09:09 |
ajo | hm | 09:10 |
ajo | true :) | 09:10 |
ihrachyshka | so I suggest you to come up with the documented plan | 09:10 |
ihrachyshka | devref change maybe | 09:10 |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports https://review.openstack.org/210301 | 09:10 |
ajo | ihrachyshka, that sounds good | 09:11 |
*** l6unchpad has quit IRC | 09:11 | |
*** ivar-lazzaro has quit IRC | 09:12 | |
*** y_sawai has quit IRC | 09:12 | |
*** Marga__ has joined #openstack-neutron | 09:12 | |
*** Marga_ has quit IRC | 09:12 | |
ihrachyshka | akamyshnikova, congrats on getting graceful ovs restart in! | 09:14 |
ihrachyshka | ajo, thanks | 09:14 |
*** l6unchpad has joined #openstack-neutron | 09:17 | |
*** abregman is now known as abregman|lunch | 09:17 | |
openstackgerrit | gongysh proposed openstack/neutron: Add empty policy rule to get_rule_type action https://review.openstack.org/215484 | 09:17 |
*** aix has joined #openstack-neutron | 09:20 | |
ajo | akamyshnikova++ | 09:21 |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports https://review.openstack.org/210304 | 09:23 |
*** claudiub has joined #openstack-neutron | 09:23 | |
*** moshele has quit IRC | 09:26 | |
*** numan has quit IRC | 09:27 | |
*** nplanel_ has quit IRC | 09:28 | |
*** y_sawai has joined #openstack-neutron | 09:31 | |
*** SourabhP has quit IRC | 09:33 | |
*** fawadkhaliq has joined #openstack-neutron | 09:33 | |
*** mohankumar has joined #openstack-neutron | 09:34 | |
*** lajos-katona1 has quit IRC | 09:34 | |
*** BobBall has joined #openstack-neutron | 09:35 | |
mattt | ihrachyshka: created https://bugs.launchpad.net/neutron/+bug/1488021, hopefully i captured the request correctly :) | 09:36 |
openstack | Launchpad bug 1488021 in neutron "Display alembic branch in neutron-db-manage current" [Undecided,New] | 09:36 |
*** l6unchpad has quit IRC | 09:37 | |
*** l6unchpad has joined #openstack-neutron | 09:38 | |
*** moshele has joined #openstack-neutron | 09:39 | |
ihrachyshka | mattt, thanks a lot! we'll take it from there. | 09:40 |
raorn | hi, everyone! | 09:43 |
openstackgerrit | gongysh proposed openstack/neutron: Add empty policy rule to get_rule_type action https://review.openstack.org/215484 | 09:43 |
mattt | ihrachyshka: thanks a lot for your help | 09:44 |
*** jianyong has quit IRC | 09:45 | |
*** jianyong has joined #openstack-neutron | 09:45 | |
*** mohankumar has quit IRC | 09:46 | |
*** yalie has quit IRC | 09:47 | |
*** jianyong has quit IRC | 09:48 | |
*** numan has joined #openstack-neutron | 09:49 | |
*** luqas has joined #openstack-neutron | 09:50 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/215086 | 09:50 |
*** mohankumar has joined #openstack-neutron | 09:51 | |
*** lajos-katona has joined #openstack-neutron | 09:52 | |
*** yanxb_ has quit IRC | 09:53 | |
openstackgerrit | Fawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II https://review.openstack.org/211196 | 09:54 |
*** yanxb_ has joined #openstack-neutron | 09:54 | |
*** Marga__ has quit IRC | 09:56 | |
*** yamamoto has quit IRC | 09:57 | |
*** yanxb_ has quit IRC | 09:58 | |
*** Marga_ has joined #openstack-neutron | 09:59 | |
*** lori|away is now known as lori | 10:02 | |
ajo | dougwig, amotoki , kevinbenton: ready for +A ;) https://review.openstack.org/#/c/214249/ | 10:03 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Make NeutronDbObjectDuplicateEntry exception more verbose https://review.openstack.org/214608 | 10:03 |
ajo | ihrachyshka, kevinbenton, obondarev_ ^ | 10:03 |
ajo | fixed nits and unittest | 10:03 |
ajo | I have removed the inner_exception and object_class storage in self, as it was a nit... if a caller needs it, can be added later | 10:03 |
amotoki | ajo: looking | 10:04 |
openstackgerrit | Merged openstack/neutron: Graceful OVS restart for DVR https://review.openstack.org/215596 | 10:04 |
openstackgerrit | Fawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II https://review.openstack.org/211196 | 10:05 |
amotoki | ajo: one question. q-qos is always enabled when this devstack plugin is enabled. is it intended? | 10:07 |
ajo | amotoki, yes, that's the intention | 10:07 |
amotoki | ajo: At now only q-qos is supported, so it is okay, but if we have more services are supported by this, we need to declare q-qos explicitly in local.conf. | 10:07 |
ajo | amotoki, so all qos related API tests will be passed on every commit | 10:07 |
*** liangbo has joined #openstack-neutron | 10:08 | |
ajo | ihrachyshka ^ | 10:08 |
ajo | amotoki, I guess, that we could use that plugin for any in-tree services for neutron | 10:08 |
ajo | and generally we'd like to enable those by default for testing, I guess | 10:08 |
ihrachyshka | ajo, we should expand it yes. amotoki, yes, we'll need to move the enablement of the plugin to localrc | 10:09 |
ihrachyshka | but for now it's good enough to avoid infra changes | 10:09 |
*** abregman|lunch is now known as abregman | 10:09 | |
*** Kennan has quit IRC | 10:09 | |
amotoki | ajo: for testing, it is good. when we run devstack in local env, we would like to enable/disable services | 10:09 |
*** l6unchpad has quit IRC | 10:09 | |
*** Kennan has joined #openstack-neutron | 10:10 | |
ajo | amotoki: ah, ok, I get what you mean | 10:10 |
amotoki | ajo: but ihrachyshka's point is also important. | 10:10 |
ajo | yes, may be we can go with infra changes early M | 10:11 |
ajo | I think that's the plan | 10:11 |
ihrachyshka | ajo, I plan to work on enabling it for tempest and overall utilizing gate_hook there | 10:11 |
ihrachyshka | in start of M | 10:11 |
ajo | ihrachyshka++ | 10:11 |
amotoki | ajo: we have disable_service, so it may be okay. I am okay with the current version and agree with the plan. | 10:11 |
amotoki | ajo: ihrachyshka: thanks | 10:12 |
ihrachyshka | I don't want to mess with infra these days :) and it's feature freeze approaching, not the best time to break stuff | 10:12 |
amotoki | ihrachyshka: totally agree | 10:12 |
ajo | ihrachyshka, so after this, if anybody wants to test it locally, he just needs to do: "enable_plugin neutron git://git.openstack.org/openstack/neutron" | 10:13 |
ajo | right? | 10:13 |
ajo | and will be in fact enabling q-qos | 10:14 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent don't hard code tunnel bridge name https://review.openstack.org/215590 | 10:14 |
amotoki | ajo: right | 10:14 |
ajo | I guess I will add that to the qos devref | 10:14 |
amotoki | ajo: ihrachyshka: /fyi/: https://review.openstack.org/#/c/213253/ is introducing a new serivce name "neutron-vpnaas" | 10:14 |
amotoki | I am not sure which is better: q-qos, neutron-qos and either looks good. | 10:15 |
*** l6unchpad has joined #openstack-neutron | 10:15 | |
*** aix has quit IRC | 10:15 | |
*** rotbeard has joined #openstack-neutron | 10:15 | |
ajo | ah, finally we're going to shake off quantum ;) | 10:16 |
*** Fdaisuke has quit IRC | 10:17 | |
openstackgerrit | Nguyen Phuong An proposed openstack/neutron-fwaas: [WIP] Support logging configuration feature for FWaaS https://review.openstack.org/204484 | 10:18 |
ihrachyshka | amotoki, oh. maybe neutron-qos would be better, yes. | 10:19 |
ihrachyshka | but we will need to solve rename for other services anyway. not sure. | 10:19 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Make models_v2 explicitly import rbac_db_models https://review.openstack.org/216177 | 10:20 |
*** sudipto has quit IRC | 10:21 | |
*** ccard has joined #openstack-neutron | 10:22 | |
*** bharath has quit IRC | 10:22 | |
openstackgerrit | Nguyen Phuong An proposed openstack/neutron: [WIP] Logging API for security group and firewall rule https://review.openstack.org/204481 | 10:22 |
*** diga has quit IRC | 10:23 | |
*** gowri1 has joined #openstack-neutron | 10:25 | |
ajo | amotoki | 10:25 |
ajo | can I ask you a question about RPC and AgentNotifiers in ML2? | 10:26 |
ajo | https://review.openstack.org/#/c/215608/1/neutron/plugins/ml2/rpc.py | 10:26 |
ajo | here I see we have bumps on the comment versions (I guess because of mixin classes..) | 10:26 |
ajo | but on "target" which remains in 1.0 | 10:26 |
*** sdake has joined #openstack-neutron | 10:26 | |
ajo | is correct what I'm doing there, or shall I bump target? | 10:26 |
*** gowri has quit IRC | 10:26 | |
amotoki | ajo: looking | 10:27 |
ajo | if I do bump target, won't it fail when we upgrade server, then agents, and server tries to notify agent on network_update, port_delete, port_update which were there first ? | 10:27 |
ajo | may be that's the reason why we don't bump ? | 10:27 |
ajo | or I'm just missing something | 10:27 |
ajo | thanks amotoki :) | 10:27 |
amotoki | ajo: it is a client side of RPC, so we need to keep the version of Target. | 10:27 |
ajo | ah | 10:28 |
openstackgerrit | Neil Jerram proposed openstack/neutron: DHCP agent: allow using gateway IPs instead of uniquely allocated https://review.openstack.org/206078 | 10:28 |
amotoki | ajo: otherwise, methods added in the initial version will have a newer version. | 10:28 |
amotoki | ajo: in this case, we need to specify version=1.2 in network_update explicitly. looking for an example | 10:28 |
ajo | ahhh amotoki , I was about to ask that | 10:29 |
ajo | if it made sense to put an explicit version per call | 10:29 |
amotoki | ajo: http://docs.openstack.org/developer/neutron/devref/rpc_api.html | 10:29 |
ajo | amotoki, I think also port_delete (or network_delete) was also added during this cycle without explicit version in the call | 10:29 |
amotoki | ajo: I think so too | 10:30 |
ajo | amotoki: 1 sec, let me track down when did that happen, so we can add the version to the other one too | 10:30 |
*** Marga_ has quit IRC | 10:31 | |
amotoki | ajo: "the other one" is "port_delete"? | 10:31 |
ajo | kevinbenton: https://github.com/openstack/neutron/commit/f87a74bfa83eeb859dfd047719622b54cdb5f68b | 10:31 |
ajo | this was pre-L right? | 10:31 |
*** ekarlso has quit IRC | 10:32 | |
*** ekarlso has joined #openstack-neutron | 10:32 | |
ajo | may be we're safe :) | 10:32 |
ajo | yeah, port_delete was kilo: https://github.com/openstack/neutron/blob/stable/kilo/neutron/plugins/ml2/rpc.py#L224 | 10:33 |
*** yfried is now known as yfried|afk | 10:35 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [OpenSwan] Enable usage of the MTU value of an IPSec connection https://review.openstack.org/206539 | 10:36 |
*** bharath has joined #openstack-neutron | 10:38 | |
amotoki | deleting a method in a server side (= callback side) is not a good way because it blocks live upgrade.... | 10:38 |
*** sudipto has joined #openstack-neutron | 10:39 | |
*** l6unchpad has quit IRC | 10:41 | |
*** l6unchpad has joined #openstack-neutron | 10:43 | |
ajo | amotoki: last question | 10:45 |
ajo | as per OVS, it seems to me: https://review.openstack.org/#/c/215608/1/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py | 10:45 |
ajo | I may need to bump to 1.4 version (now it's 1.3) | 10:45 |
ajo | and target that from the cttxt prepare on Api side | 10:46 |
amotoki | ajo: right. we need to bump the version to 1.4. | 10:46 |
ajo | thanks for confirmation | 10:46 |
amotoki | ajo: it is tricky. I just remmeber I thought we need to break down agent side callback into callbacks per feature. | 10:47 |
ajo | amotoki, yes, it seems like that's what we pay for having all endpoints in "self" | 10:47 |
*** yamamoto has joined #openstack-neutron | 10:47 | |
*** SridharG has left #openstack-neutron | 10:48 | |
ajo | amotoki, I can tackle that for M | 10:48 |
*** aix has joined #openstack-neutron | 10:48 | |
ajo | do you remember if we have a bug for it? otherwise I will fill one | 10:48 |
*** dguitarbite has quit IRC | 10:49 | |
*** stanzgy has quit IRC | 10:49 | |
*** yamamoto has quit IRC | 10:50 | |
*** yamamoto has joined #openstack-neutron | 10:50 | |
*** jamielennox|away has quit IRC | 10:50 | |
*** yamamoto_ has joined #openstack-neutron | 10:50 | |
amotoki | ajo: I think this is the counterpart of https://blueprints.launchpad.net/neutron/+spec/rpc-docs-and-namespaces | 10:51 |
amotoki | and https://bugs.launchpad.net/neutron/+bug/1112634 is related. | 10:51 |
openstack | Launchpad bug 1112634 in neutron "allow RPC version evolves independently for each of features" [Medium,In progress] - Assigned to YAMAMOTO Takashi (yamamoto) | 10:51 |
*** liangbo has quit IRC | 10:52 | |
*** jistr has joined #openstack-neutron | 10:52 | |
amotoki | I haven't checked how was the abandoned patch. | 10:52 |
amotoki | it is just about l2-pop. We can take the bug. | 10:53 |
*** mwagner_lap has quit IRC | 10:53 | |
*** jianyong has joined #openstack-neutron | 10:54 | |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports https://review.openstack.org/210301 | 10:54 |
ajo | amotoki, every time I try to write "amotoki" in my browser it tries to change it to "atomic" :) | 10:54 |
*** gongysh has quit IRC | 10:54 | |
*** yamamoto has quit IRC | 10:54 | |
amotoki | ajo: :-) | 10:55 |
*** adam_g has quit IRC | 10:55 | |
*** rackertom has quit IRC | 10:55 | |
*** lbragstad has quit IRC | 10:55 | |
*** yfried|afk is now known as yfried | 10:55 | |
*** ctracey has quit IRC | 10:55 | |
*** wendar_ has joined #openstack-neutron | 10:56 | |
*** zacksh has quit IRC | 10:56 | |
*** wendar has quit IRC | 10:56 | |
*** jroll has quit IRC | 10:56 | |
openstackgerrit | Merged openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 10:57 |
*** rex_lee has quit IRC | 10:57 | |
*** zacksh has joined #openstack-neutron | 10:58 | |
*** ctracey has joined #openstack-neutron | 10:58 | |
*** xgerman has quit IRC | 10:58 | |
*** pc_m has joined #openstack-neutron | 10:59 | |
*** rackertom has joined #openstack-neutron | 10:59 | |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports https://review.openstack.org/210304 | 11:00 |
*** xgerman has joined #openstack-neutron | 11:00 | |
openstackgerrit | Lajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports https://review.openstack.org/210304 | 11:00 |
*** evgenyf has quit IRC | 11:01 | |
*** adam_g has joined #openstack-neutron | 11:01 | |
*** adam_g has quit IRC | 11:01 | |
*** adam_g has joined #openstack-neutron | 11:01 | |
*** lbragstad has joined #openstack-neutron | 11:02 | |
*** jroll has joined #openstack-neutron | 11:02 | |
*** amotoki has quit IRC | 11:02 | |
*** yamamoto_ has quit IRC | 11:03 | |
*** jamielennox|away has joined #openstack-neutron | 11:04 | |
*** rex_lee has joined #openstack-neutron | 11:04 | |
*** jamielennox|away is now known as jamielennox | 11:04 | |
*** rubasov has quit IRC | 11:10 | |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add updated_attrs in port_update https://review.openstack.org/155223 | 11:11 |
*** sudipto has quit IRC | 11:14 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: [WIP] Add QoS fullstack test https://review.openstack.org/215688 | 11:15 |
*** l6unchpad has quit IRC | 11:18 | |
*** l6unchpad has joined #openstack-neutron | 11:19 | |
*** yamamoto has joined #openstack-neutron | 11:21 | |
*** liangbo has joined #openstack-neutron | 11:22 | |
*** l6unchpad has quit IRC | 11:23 | |
*** shaohe_feng1 has quit IRC | 11:23 | |
*** lindycoder has joined #openstack-neutron | 11:29 | |
*** ferest has joined #openstack-neutron | 11:29 | |
*** rubasov has joined #openstack-neutron | 11:30 | |
*** evgenyf has joined #openstack-neutron | 11:31 | |
*** salv-orlando has joined #openstack-neutron | 11:32 | |
*** ferest has quit IRC | 11:35 | |
*** anilvenkata has left #openstack-neutron | 11:36 | |
*** thorst has joined #openstack-neutron | 11:36 | |
*** nmagnezi has quit IRC | 11:37 | |
*** l6unchpad has joined #openstack-neutron | 11:37 | |
openstackgerrit | Merged openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 11:37 |
openstackgerrit | Merged openstack/neutron: Move tests for non pluggable ipam backend https://review.openstack.org/211507 | 11:38 |
*** BobBall_ has joined #openstack-neutron | 11:39 | |
*** BobBall_ is now known as Guest24824 | 11:40 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants https://review.openstack.org/215782 | 11:40 |
*** BobBall has quit IRC | 11:43 | |
*** Marga_ has joined #openstack-neutron | 11:43 | |
*** sudipto has joined #openstack-neutron | 11:47 | |
ihrachyshka | ajo, ^^ ok, now qos api tests are really enabled in gate | 11:47 |
jschwarz | ihrachyshka, really? | 11:53 |
jschwarz | XD | 11:53 |
ihrachyshka | jschwarz, well, not tempest still | 11:55 |
jschwarz | ihrachyshka, but qos api | 11:55 |
jschwarz | XD | 11:55 |
*** salv-orlando has quit IRC | 11:55 | |
*** yanxb_ has joined #openstack-neutron | 12:00 | |
*** dguitarbite has joined #openstack-neutron | 12:01 | |
*** Guest24824 has quit IRC | 12:01 | |
*** pradk has quit IRC | 12:01 | |
*** BobBall has joined #openstack-neutron | 12:01 | |
*** dguitarbite has quit IRC | 12:01 | |
*** dguitarbite has joined #openstack-neutron | 12:01 | |
*** BobBall is now known as Guest37252 | 12:01 | |
*** numan has quit IRC | 12:02 | |
*** jpena is now known as jpena|lunch | 12:04 | |
*** Guest37252 has quit IRC | 12:06 | |
*** moha_hunt has joined #openstack-neutron | 12:07 | |
*** shoutm has joined #openstack-neutron | 12:07 | |
ajo | ihrachyshka++ | 12:08 |
ajo | yeheheyyy | 12:08 |
*** gampel1 has joined #openstack-neutron | 12:09 | |
*** markvoelker_ has quit IRC | 12:10 | |
*** BobBall__ has joined #openstack-neutron | 12:12 | |
*** numan has joined #openstack-neutron | 12:13 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Move db agent schedulers test to a more appropriate place https://review.openstack.org/214131 | 12:14 |
*** piyanai has joined #openstack-neutron | 12:14 | |
*** nyechiel has quit IRC | 12:14 | |
*** sudipto has quit IRC | 12:16 | |
*** liangbo has quit IRC | 12:17 | |
*** gampel1 has left #openstack-neutron | 12:17 | |
*** markvoelker has joined #openstack-neutron | 12:17 | |
*** lyanchih has joined #openstack-neutron | 12:17 | |
*** y_sawai has quit IRC | 12:17 | |
*** y_sawai has joined #openstack-neutron | 12:19 | |
*** y_sawai has quit IRC | 12:19 | |
*** markvoelker has quit IRC | 12:19 | |
*** markvoelker has joined #openstack-neutron | 12:20 | |
*** piyanai has quit IRC | 12:20 | |
*** markvoelker has quit IRC | 12:20 | |
*** diegows has joined #openstack-neutron | 12:20 | |
*** diegows has quit IRC | 12:21 | |
*** VikasC has quit IRC | 12:22 | |
*** julim has joined #openstack-neutron | 12:22 | |
*** markvoelker has joined #openstack-neutron | 12:25 | |
*** nmagnezi has joined #openstack-neutron | 12:25 | |
*** numan has quit IRC | 12:26 | |
*** piyanai has joined #openstack-neutron | 12:27 | |
*** itzikb has quit IRC | 12:29 | |
*** itzikb has joined #openstack-neutron | 12:29 | |
*** edmondsw has joined #openstack-neutron | 12:31 | |
*** jistr has quit IRC | 12:31 | |
*** jistr has joined #openstack-neutron | 12:33 | |
*** nmagnezi has quit IRC | 12:33 | |
*** yfried is now known as yfried|afk | 12:34 | |
*** regXboi has joined #openstack-neutron | 12:35 | |
*** mwagner_lap has joined #openstack-neutron | 12:37 | |
*** gberginc has quit IRC | 12:37 | |
*** zengine has joined #openstack-neutron | 12:38 | |
*** nmagnezi has joined #openstack-neutron | 12:41 | |
*** numan has joined #openstack-neutron | 12:41 | |
*** yfried|afk is now known as yfried | 12:44 | |
*** Marga_ has quit IRC | 12:46 | |
*** sdake_ has joined #openstack-neutron | 12:47 | |
openstackgerrit | shihanzhang proposed openstack/neutron: Update port's status DOWN if its binding info has changed https://review.openstack.org/215467 | 12:49 |
*** sdake__ has joined #openstack-neutron | 12:50 | |
*** sdake has quit IRC | 12:50 | |
*** sdake_ has quit IRC | 12:53 | |
*** rotbeard has quit IRC | 12:55 | |
*** rotbeard has joined #openstack-neutron | 12:56 | |
*** nyechiel has joined #openstack-neutron | 12:56 | |
*** tiswanso has joined #openstack-neutron | 12:57 | |
*** amuller has joined #openstack-neutron | 12:58 | |
*** sdake__ is now known as sdake | 12:59 | |
*** tiswanso has quit IRC | 13:00 | |
*** baoli has joined #openstack-neutron | 13:00 | |
*** banix has joined #openstack-neutron | 13:01 | |
*** tiswanso has joined #openstack-neutron | 13:01 | |
*** bharath has quit IRC | 13:01 | |
*** jpena|lunch is now known as jpena | 13:02 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Splitting out models from database class https://review.openstack.org/216248 | 13:05 |
*** annegentle has joined #openstack-neutron | 13:05 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Splitting out models from database class https://review.openstack.org/216248 | 13:07 |
*** annegentle has quit IRC | 13:08 | |
*** vikram has joined #openstack-neutron | 13:08 | |
*** annegentle has joined #openstack-neutron | 13:08 | |
openstackgerrit | enikanorov proposed openstack/neutron: Add periodic agents health check. https://review.openstack.org/181633 | 13:11 |
*** britthouser has joined #openstack-neutron | 13:11 | |
*** dims has joined #openstack-neutron | 13:12 | |
*** clev-away is now known as clev | 13:13 | |
*** clev is now known as clev-away | 13:13 | |
*** mkolesni_ has quit IRC | 13:14 | |
*** nate_gone is now known as njohnston | 13:14 | |
*** kbringard has joined #openstack-neutron | 13:14 | |
*** britthou_ has joined #openstack-neutron | 13:16 | |
*** miyagishi_t has quit IRC | 13:16 | |
*** ildikov has quit IRC | 13:18 | |
*** ijw_ has quit IRC | 13:18 | |
*** britthouser has quit IRC | 13:18 | |
*** banix has quit IRC | 13:19 | |
*** ijw has joined #openstack-neutron | 13:19 | |
*** ildikov has joined #openstack-neutron | 13:20 | |
*** Samos123 has quit IRC | 13:22 | |
*** dsneddon has quit IRC | 13:22 | |
*** clev-away has quit IRC | 13:22 | |
*** Marga_ has joined #openstack-neutron | 13:22 | |
*** jroll has quit IRC | 13:23 | |
*** dims has quit IRC | 13:23 | |
*** jamielennox has quit IRC | 13:26 | |
*** Samos123 has joined #openstack-neutron | 13:27 | |
*** irenab has quit IRC | 13:27 | |
*** tiswanso has quit IRC | 13:27 | |
*** tiswanso has joined #openstack-neutron | 13:28 | |
*** bapalm_ has joined #openstack-neutron | 13:28 | |
*** shausy has quit IRC | 13:29 | |
*** salv-orlando has joined #openstack-neutron | 13:29 | |
*** signed8bit has joined #openstack-neutron | 13:29 | |
*** zzzeek has joined #openstack-neutron | 13:30 | |
*** BobBall__ is now known as BobBall | 13:30 | |
*** liangbo has joined #openstack-neutron | 13:30 | |
*** salv-orl_ has joined #openstack-neutron | 13:31 | |
*** salv-orlando has quit IRC | 13:31 | |
*** shausy has joined #openstack-neutron | 13:31 | |
*** links has joined #openstack-neutron | 13:31 | |
*** signed8bit has quit IRC | 13:34 | |
*** dsneddon has joined #openstack-neutron | 13:34 | |
openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: DB models and service plugin https://review.openstack.org/210310 | 13:34 |
openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: API extension https://review.openstack.org/210309 | 13:34 |
openstackgerrit | Bence Romsics proposed openstack/neutron: Trunk port: ML2 extension driver for subports https://review.openstack.org/210313 | 13:34 |
*** yamamoto has quit IRC | 13:34 | |
*** prad_ has joined #openstack-neutron | 13:34 | |
*** prad_ is now known as pradk | 13:35 | |
*** lyanchih has quit IRC | 13:35 | |
*** signed8bit has joined #openstack-neutron | 13:37 | |
*** clev has joined #openstack-neutron | 13:37 | |
*** jroll has joined #openstack-neutron | 13:37 | |
openstackgerrit | Fawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II https://review.openstack.org/211196 | 13:40 |
*** piyanai has quit IRC | 13:41 | |
*** rkukura has joined #openstack-neutron | 13:41 | |
*** crose has joined #openstack-neutron | 13:42 | |
*** rkukura has quit IRC | 13:42 | |
*** shihanzhang has quit IRC | 13:42 | |
*** jamielennox|away has joined #openstack-neutron | 13:43 | |
*** shihanzhang has joined #openstack-neutron | 13:43 | |
*** jamielennox|away is now known as jamielennox | 13:43 | |
*** neeti has quit IRC | 13:43 | |
*** shaohe_feng has joined #openstack-neutron | 13:45 | |
*** bharath has joined #openstack-neutron | 13:46 | |
*** shwetaap has joined #openstack-neutron | 13:47 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting router issues https://review.openstack.org/215655 | 13:48 |
*** yamamoto has joined #openstack-neutron | 13:50 | |
*** rushil has joined #openstack-neutron | 13:52 | |
*** dims_ has joined #openstack-neutron | 13:52 | |
*** gowri1 has quit IRC | 13:53 | |
*** baoli_ has joined #openstack-neutron | 13:53 | |
*** jckasper has joined #openstack-neutron | 13:53 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Remove redundant logging statements from RootWrapDaemonHelper https://review.openstack.org/216260 | 13:54 |
*** baoli has quit IRC | 13:57 | |
openstackgerrit | Li Ma proposed openstack/neutron: Validate local_ip for linuxbridge-agent https://review.openstack.org/211443 | 13:59 |
*** boris-42 has joined #openstack-neutron | 14:00 | |
*** amotoki has joined #openstack-neutron | 14:00 | |
jschwarz | amuller, do you know where I might find ZZelle? | 14:00 |
*** rushil has quit IRC | 14:01 | |
*** rushil has joined #openstack-neutron | 14:02 | |
*** lukasa has quit IRC | 14:04 | |
eezhova | pc_m, hello! | 14:04 |
*** tiswanso has quit IRC | 14:05 | |
pc_m | eezhova: hi | 14:06 |
eezhova | pc_m, There is another peculiarity with strongswan config that I found. It isn't possible to set mtu per ipsec connection, only for vpn service: http://paste.openstack.org/show/426098/ | 14:06 |
*** frickler_ has joined #openstack-neutron | 14:07 | |
*** neiljerram has joined #openstack-neutron | 14:07 | |
*** fawadkhaliq has quit IRC | 14:07 | |
*** frickler has quit IRC | 14:07 | |
pc_m | eezhova: ouch. So an API difference that is conflicting. | 14:08 |
eezhova | pc_m, in the current situation everything will be working fine as long as we have no more that one connection per vpn service | 14:08 |
*** jamielennox has quit IRC | 14:08 | |
eezhova | pc_m, exactly | 14:08 |
eezhova | pc_m, so I don't really know what should be done in a situation like that | 14:09 |
pc_m | eezhova: I can only think of hacky ways to deal with that situation. | 14:09 |
*** jamielennox|away has joined #openstack-neutron | 14:10 | |
pc_m | eezhova: In addition, it adds IPsec specific setting to the service API, which we really don't want to do, if we want to support other VPN types. | 14:10 |
*** jamielennox|away is now known as jamielennox | 14:11 | |
*** irenab has joined #openstack-neutron | 14:11 | |
eezhova | pc_m, we can add another option "vpn_service_mtu" but that gonna involve complicated to validation | 14:12 |
pc_m | eezhova: hackish change would be to have a validator on the connection that ensures that the connections on one service are the same. | 14:12 |
*** nplanel has joined #openstack-neutron | 14:13 | |
*** jprovazn has joined #openstack-neutron | 14:13 | |
pc_m | eezhova: yes, very awkward, ignoring service set MTU for Openswan (or applying to all connections), and ignoring connection set MTU for strongswan (or using validation to reject). | 14:14 |
eezhova | pc_m, but isn't it a limitation? I mean some options, other than mtu , can differ | 14:14 |
pc_m | eezhova: Ideally there should be validators for each driver. For example, the Cisco driver has validation for things it doesn't support. | 14:15 |
*** sdake_ has joined #openstack-neutron | 14:15 | |
*** zhenguo has quit IRC | 14:17 | |
*** lyanchih has joined #openstack-neutron | 14:17 | |
eezhova | pc_m, I'll have a look at how they do it. meanwhile your last suggestion sounds like the best plan for now | 14:17 |
*** shausy has quit IRC | 14:18 | |
pc_m | eezhova: Which one? Adding validators and requiring all connections use same MTU? | 14:19 |
*** sdake has quit IRC | 14:19 | |
*** frickler_ is now known as frickler | 14:19 | |
gsagie | mestery, russellb, kevinbenton: i think i will align the spec of adding metadata to resources with the nova tags spec http://specs.openstack.org/openstack/nova-specs/specs/kilo/approved/tag-instances.html, thoughts? | 14:19 |
*** yfried has quit IRC | 14:20 | |
eezhova | pc_m, ignoring service mtu for OpenSwan and ignoring connection mtu for StrongSwan | 14:20 |
russellb | gsagie: yeah, double check to see what was actually implemented first though | 14:20 |
russellb | gsagie: but in general, consistency between APIs ++ | 14:20 |
kevinbenton | gsagie: hold off on making changes until we can chat about it in the drivers meeting tomorrow | 14:20 |
pc_m | eezhova: Not sure that is better. | 14:20 |
johnthetubaguy | gsagie: I think there is an API WG doc on tags, we haven't got that spec merged yet, there is an updated version here: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/tag-instances.html | 14:21 |
gsagie | kevinbenton: ok | 14:21 |
kevinbenton | gsagie: just so it's not changing before people get a chance to read the email | 14:21 |
*** markvoelker has quit IRC | 14:21 | |
*** nplanel has quit IRC | 14:22 | |
pc_m | eezhova: It has the implication of changing the service API (with something that is ambiguous now - which is used?), and has ripple down changes to the driver. | 14:22 |
gsagie | johnthetubaguy: ok, thanks | 14:22 |
*** markvoelker has joined #openstack-neutron | 14:22 | |
pc_m | eezhova: Requiring that all connections use the same MTU for a service is clearly less effort. | 14:22 |
johnthetubaguy | gsagie: ah, this is the one: http://specs.openstack.org/openstack/api-wg/guidelines/tags.html | 14:23 |
eezhova | pc_m, I guess you are right. adding validators and requiring all connections use same mtu is also easier | 14:23 |
*** agireud has joined #openstack-neutron | 14:23 | |
eezhova | pc_m, I'll try it out and see whether it works out | 14:24 |
*** rushil has quit IRC | 14:24 | |
pc_m | eezhova: Yeah, should, and should be easy to query the database for that case. | 14:24 |
*** rushil has joined #openstack-neutron | 14:24 | |
*** shoutm has quit IRC | 14:24 | |
eezhova | pc_m, thank you for the advice! | 14:24 |
pc_m | eezhova: sure np. | 14:24 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: [WIP] Process update_network in the openvswitch agent https://review.openstack.org/215608 | 14:24 |
gsagie | johnthetubaguy : thanks for the link | 14:24 |
*** yog_ has quit IRC | 14:25 | |
*** sdake has joined #openstack-neutron | 14:25 | |
*** rushil has quit IRC | 14:27 | |
*** kbringard has quit IRC | 14:28 | |
*** sdake_ has quit IRC | 14:28 | |
*** Murali has quit IRC | 14:30 | |
*** shoutm has joined #openstack-neutron | 14:30 | |
*** crose has quit IRC | 14:31 | |
*** kbringard has joined #openstack-neutron | 14:32 | |
*** ajmiller has joined #openstack-neutron | 14:33 | |
*** jckasper has quit IRC | 14:36 | |
*** yamahata has joined #openstack-neutron | 14:36 | |
*** alejandrito has joined #openstack-neutron | 14:37 | |
*** liangbo has quit IRC | 14:38 | |
*** lajos-katona has quit IRC | 14:39 | |
*** liangbo has joined #openstack-neutron | 14:40 | |
*** bapalm_ has quit IRC | 14:41 | |
*** dmartls1 has joined #openstack-neutron | 14:42 | |
openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources https://review.openstack.org/213586 | 14:43 |
*** yamahata has quit IRC | 14:43 | |
*** irenab has quit IRC | 14:44 | |
*** irenab has joined #openstack-neutron | 14:45 | |
*** bapalm- has joined #openstack-neutron | 14:45 | |
*** bharath has quit IRC | 14:46 | |
*** zhenguo has joined #openstack-neutron | 14:46 | |
*** vivek-ebay has joined #openstack-neutron | 14:48 | |
*** ddepaoli has quit IRC | 14:52 | |
*** annegentle has quit IRC | 14:52 | |
*** achanda has joined #openstack-neutron | 14:52 | |
pc_m | salv-orl_: hi | 14:58 |
*** shaohe_feng has quit IRC | 14:59 | |
*** numan has quit IRC | 14:59 | |
*** shoutm has quit IRC | 15:00 | |
*** mlavalle has joined #openstack-neutron | 15:00 | |
ihrachyshka | salv-orl_, re 'current' patch: do you think it's still reasonable to have a command to get the list of heads for an alembic tree? | 15:00 |
ihrachyshka | maybe a new command for that? so that you can compare 'current' output with e.g. 'heads' to see whether there is anything not applied yet | 15:01 |
*** jlibosva has quit IRC | 15:02 | |
*** numan has joined #openstack-neutron | 15:02 | |
*** achanda has quit IRC | 15:02 | |
*** pbrooko has joined #openstack-neutron | 15:03 | |
*** emagana has joined #openstack-neutron | 15:04 | |
*** vivek-ebay has quit IRC | 15:04 | |
*** rushil has joined #openstack-neutron | 15:05 | |
*** yamamoto has quit IRC | 15:07 | |
*** diga has joined #openstack-neutron | 15:08 | |
pc_m | dougwig: ping | 15:08 |
*** jaypipes has joined #openstack-neutron | 15:09 | |
*** slaweq has joined #openstack-neutron | 15:09 | |
*** Aish has joined #openstack-neutron | 15:10 | |
regXboi | amuller: ping? | 15:10 |
*** thorst has quit IRC | 15:11 | |
regXboi | amuller: is there *anywhere* in the patch where we log attached ports of a router for troubleshooting purposes? | 15:11 |
*** anshul has quit IRC | 15:11 | |
*** thorst has joined #openstack-neutron | 15:11 | |
*** lyanchih has quit IRC | 15:12 | |
*** lyanchih has joined #openstack-neutron | 15:12 | |
*** itzikb has quit IRC | 15:15 | |
*** piyanai has joined #openstack-neutron | 15:15 | |
*** crose has joined #openstack-neutron | 15:15 | |
kevinbenton | haleyb: ping | 15:15 |
*** amotoki has quit IRC | 15:16 | |
*** abregman has quit IRC | 15:17 | |
*** ronis_ has quit IRC | 15:17 | |
*** jbell8 has joined #openstack-neutron | 15:17 | |
haleyb | kevinbenton: pong | 15:18 |
*** signed8bit has quit IRC | 15:21 | |
*** srilatha has joined #openstack-neutron | 15:23 | |
kevinbenton | haleyb: which are the possible interface types in DVR that floating IP traffic can come from? | 15:23 |
*** ildikov has quit IRC | 15:23 | |
kevinbenton | haleyb: device_owner field | 15:24 |
openstackgerrit | Christian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide https://review.openstack.org/216125 | 15:24 |
*** mohankumar has quit IRC | 15:25 | |
kevinbenton | haleyb: e.g. does floating IP traffic ever come from these? https://github.com/openstack/neutron/blob/master/neutron/common/constants.py#L38 | 15:25 |
*** zengine has quit IRC | 15:25 | |
*** annegentle has joined #openstack-neutron | 15:25 | |
regXboi | kevinbenton: why are you asking? | 15:26 |
*** Ramanjaneya has quit IRC | 15:26 | |
*** zengine has joined #openstack-neutron | 15:26 | |
kevinbenton | regXboi: these things depend on ARP spoofing being allowed | 15:26 |
regXboi | kevinbenton: so that looks to be used in the router gateway port - let me confirm | 15:27 |
kevinbenton | regXboi: so we have this issue where if the port security extension is disabled (or doesn't exist in Juno), ARP spoofing protection can't be enabled | 15:27 |
regXboi | ah | 15:28 |
kevinbenton | regXboi: so i need to add some flag to pass to the L2 agents for this case | 15:29 |
*** vikram has quit IRC | 15:29 | |
kevinbenton | regXboi: to tell it to skip port security for those | 15:29 |
*** killer_prince has quit IRC | 15:30 | |
*** TrevorV has joined #openstack-neutron | 15:30 | |
regXboi | kevinbenton: so I can confirm that ports with that device_owner act as the router gateway ports for DVR and map to the qr- interface that is attached to br-int and placed into the dist router namespace for iptables mangling | 15:31 |
regXboi | so FIP traffic will flow through that port | 15:31 |
amuller | regXboi: internal_network_added? | 15:31 |
*** slaweq has quit IRC | 15:31 | |
*** zhenguo has quit IRC | 15:31 | |
regXboi | amuller: let me look - I had the DVR job fail this morning on a router getting a second port from the same subnet and the logs don't reflect that | 15:32 |
regXboi | amuller: so I was looking to see if the logs are incomplete or I'm just blind | 15:32 |
salv-orl_ | ihrachyshka: yes. we already expose history to that aim. But heads is more immediate | 15:33 |
*** Sukhdev has joined #openstack-neutron | 15:33 | |
*** moshele has quit IRC | 15:34 | |
ihrachyshka | salv-orl_, so I thought we would rework your patch to introduce the heads command as a new one instead of rewriting what 'current' does. | 15:35 |
*** jistr has quit IRC | 15:35 | |
regXboi | amuller: not yet - that's part of what my patch adds :) | 15:36 |
ihrachyshka | salv-orl_, will you want to proceed with it? or do you want me to take over it? | 15:36 |
*** zengine has quit IRC | 15:36 | |
*** dims__ has joined #openstack-neutron | 15:36 | |
regXboi | can we get it merged ? | 15:36 |
*** vivek-ebay has joined #openstack-neutron | 15:36 | |
*** britthou_ has quit IRC | 15:36 | |
amuller | regXboi: I thought we were talking about your patch =D | 15:37 |
pc_m | salv-orl_: ping | 15:38 |
regXboi | amuller: no - I was talking in the absence of it - | 15:38 |
*** david-ly_ is now known as david-lyle | 15:38 | |
*** jistr has joined #openstack-neutron | 15:38 | |
*** vivek-eb_ has joined #openstack-neutron | 15:38 | |
regXboi | amuller: trying to solve a problem with a tempest patch without it, and it's .... well .... painful .... | 15:38 |
amuller | regXboi: just approved your patch | 15:39 |
amuller | so hopefully next time it won't be as painful :) | 15:39 |
*** dims_ has quit IRC | 15:39 | |
*** vivek-ebay has quit IRC | 15:40 | |
openstackgerrit | Pengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db https://review.openstack.org/213563 | 15:40 |
regXboi | amuller: mahalo, moi loa | 15:40 |
salv-orl_ | ihrachyshka: I can proceed with this change. But you know I'm old and old people are slower | 15:41 |
salv-orl_ | hi pc_m | 15:41 |
pc_m | salv-orl_: On the endpoint group work. Had a few questions... Should I do all the endpoint-group CRUD part as one commit, and then a separate commit for the multiple local subnets part? | 15:41 |
pc_m | salv-orl_: Also, should I do the endpoint groups (which is adding a new API and tables only) as part of the existing API (use same extension module, no new class, etc.)? | 15:41 |
salv-orl_ | pc_m: I think that would be fine. Whether having one or more commut for the endpoint group CRUD then it's a decision that pertains more the load on vpnaas reviewers | 15:42 |
*** SourabhP has joined #openstack-neutron | 15:42 | |
*** rstarr has quit IRC | 15:43 | |
*** rstarr has joined #openstack-neutron | 15:43 | |
salv-orl_ | pc_m: I think the change I reviewed proposed a new extension module. This is the best way to let clients know when the endpoint group feature is available. The other side of the question is however how hard it will be for you and the vpnaas team to maintain the "old" API until the deprecation cycle is complete | 15:43 |
*** devvesa has quit IRC | 15:44 | |
*** SourabhP_ has joined #openstack-neutron | 15:44 | |
pc_m | salv-orl_: yes, it did. I was wondering, because the new API and tables would not be "used" until the multiple local subnet change, should I defer the new extension module? | 15:44 |
ihrachyshka | salv-orl_, that joke became boring. you can do it. ;) and/or I will get back to you when and if I have spare time, and we'll see whether you made any progress, and if not, I will take over. | 15:44 |
salv-orl_ | ihrachyshka: I can fine plenty of new jokes no worries | 15:45 |
pc_m | salv-orl_: IOW, add the extension when doing the part that modifies APIs and uses the new API? | 15:45 |
*** Swami has joined #openstack-neutron | 15:45 | |
ihrachyshka | salv-orl_, looking forward :) | 15:45 |
salv-orl_ | pc_m: I see; I did not consider you had a prerequiste change in the DB model | 15:45 |
*** diga has quit IRC | 15:46 | |
salv-orl_ | in that case it might make sense to first do all the changes needed in the database models and then push the extension. | 15:46 |
*** cdl has joined #openstack-neutron | 15:46 | |
pc_m | salv-orl_: So, two pieces... one adds the endpoint groups API and tables, the other has the connection API use the new table and no longer use the old attributes. | 15:46 |
*** numan has quit IRC | 15:47 | |
*** SourabhP has quit IRC | 15:47 | |
*** SourabhP_ is now known as SourabhP | 15:47 | |
pc_m | It would also switch from using the subnet id in service to the endpoint group's subnet based endpoints. | 15:47 |
*** evgenyf has quit IRC | 15:47 | |
pc_m | salv-orl_: could essentially upstream it, but no one would be using it. | 15:47 |
salv-orl_ | pc_m: help me understand... are you in a sort of chicken and egg problem? That if you push the change for the multiple local subnets then the old API won't work anymore, but if you first push the new API then you are not able to use it as it requires multiple local subnets support? | 15:48 |
pc_m | salv-orl_: not quite... | 15:48 |
pc_m | salv-orl_: The endpoint groups can be pushed independently. They would only be used, when trying to implement the multiple local subnets. | 15:49 |
pc_m | endpoints are a prerequisite for the multiple local subnets. | 15:49 |
*** dannywilson has joined #openstack-neutron | 15:49 | |
salv-orl_ | pc_m: so they would be there and work, with the only issue that they would not serve however any purpose until multiple loc subnets merge | 15:50 |
pc_m | exactly. Just populating a table that is unused. | 15:50 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: Move logic to handle OVS status out of rpc_loop https://review.openstack.org/199165 | 15:50 |
salv-orl_ | and from your last sentence I gather that what I got wrong was probably thinking that multiple local subnets were a preprequiste for endpoing groups | 15:50 |
*** dannywilson has quit IRC | 15:50 | |
salv-orl_ | pc_m: whereas it's actually the opposite | 15:50 |
*** dannywil_ has joined #openstack-neutron | 15:51 | |
pc_m | salv-orl_: correct | 15:51 |
*** Aish has quit IRC | 15:51 | |
salv-orl_ | and in this case your strategy lgtm. I don't think it's a big deal to have an unused piece of API - especially if that API is clearly labelled as vpnaas_v3 | 15:51 |
salv-orl_ | pc_m: now about the extensions labels. | 15:52 |
pc_m | Thought is for mulitple local subnets, could migrate subnet id from service to endpoint group, and peer_cidrs from connection table to endpoint group and change attributes on the commands. | 15:52 |
*** geekinutah has joined #openstack-neutron | 15:52 | |
salv-orl_ | I had the impression from previous chats we had that after the multiple subnets change an API client could still use the old API, and the subnet_id requesty would be translated into a "trivlal" endpoint group | 15:52 |
salv-orl_ | but they would still use the "old" API until you go through the deprecation cycle | 15:53 |
salv-orl_ | now it looks like to me you ensure "business continuity" with a db migration but once you switch the old API gets killed? | 15:53 |
*** alejandrito has quit IRC | 15:53 | |
*** sridhar_ram has joined #openstack-neutron | 15:54 | |
*** sridhar_ram has quit IRC | 15:54 | |
pc_m | salv-orl_: Yeah, I think so. I think it would be work, to try to take old API requests and morph them to new. | 15:54 |
pc_m | salv-orl_: May be possible. Haven't thought it through. | 15:55 |
salv-orl_ | pc_m: I am just trying to understand the strategy. In my opinion using a new extension name would still help users. If the previous API calls start failing with 404s they know have to use the new API. | 15:55 |
*** irenab has quit IRC | 15:55 | |
pc_m | salv-orl_: And I'm trying to figure out the right strategy :) | 15:56 |
salv-orl_ | otherwise they would get weird failures on the API endpoints they were calling before (because they lack endpoint groups for their VPN service) | 15:56 |
*** lazy_prince has joined #openstack-neutron | 15:56 | |
salv-orl_ | regarding the transition strategy my recommendation, for what is worth, is to always look for a path for a smooth transition unless the current API is so bad that without such a drastic change no one is going to use | 15:57 |
salv-orl_ | load balancing went through a similar thing. | 15:58 |
salv-orl_ | I hereby summon dougwig | 15:58 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 15:58 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 15:58 |
salv-orl_ | dougwig: is it true that lbaasv1 is still in its deprecation cycle but not yet killed? | 15:58 |
*** matrohon has quit IRC | 15:58 | |
pc_m | salv-orl_: so, place the new API under a new extension? | 15:58 |
*** irenab has joined #openstack-neutron | 15:59 | |
salv-orl_ | pc_m: this is what you're doiing afaict | 15:59 |
*** rstarr has quit IRC | 15:59 | |
*** rstarr has joined #openstack-neutron | 15:59 | |
salv-orl_ | the other question is what to do with the old one. | 15:59 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: [neutron-db-manage] Introduce contract and expand commands https://review.openstack.org/216313 | 15:59 |
pc_m | salv-orl_: Yeah, that's what I started with, but was wondering if I should just add endpoints to the existing extension. | 15:59 |
*** armax has joined #openstack-neutron | 16:00 | |
salv-orl_ | pc_m: adding endpoint groups is totally fine. Changing the VPNService resource is the tricky bit, in my opinion | 16:00 |
salv-orl_ | there's nothing to prevent you from doing that, but users which are not up-to-date with API clients will necessarily experience failures, as you're breaking backward compatibility. | 16:01 |
salv-orl_ | This is not a "no, no, no" thing | 16:01 |
openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent add functional tests of OVS status https://review.openstack.org/205442 | 16:01 |
*** fawadkhaliq has joined #openstack-neutron | 16:01 | |
pc_m | salv-orl_: I guess we could mark subnet as optional, and then, if provided, copy it also into new endpoint group. | 16:02 |
salv-orl_ | but still you need to have a good reason for doing so - as I'm assuming there are active users out there, and some of them might be slightly pissed off by seeing their applications broken | 16:02 |
salv-orl_ | pc_m: that sounds like a backward compatibility strategy, yes | 16:02 |
pc_m | At conn create, we have to lookup the subnet id, find the endpoint group and use that ID in the connection. | 16:02 |
*** mickeys has joined #openstack-neutron | 16:03 | |
salv-orl_ | so subnet_id and endpoint_group_id become mutually exclusive? | 16:03 |
*** insequent has joined #openstack-neutron | 16:03 | |
pc_m | For peer-cidrs, could create subnet group and use it too. | 16:03 |
salv-orl_ | and if you specify a subnet_id an endpoint group is implicitly created? | 16:03 |
*** garyk has joined #openstack-neutron | 16:03 | |
pc_m | salv-orl_: not sure about mutex. I was thinking more of a "live migration" sort of. | 16:05 |
*** ushkalim has quit IRC | 16:05 | |
pc_m | salv-orl_: Concerned if any messy parts, like how to delete endpoint groups created from subnet entries. | 16:05 |
salv-orl_ | pc_m: ok, I maybe misread your sentence here "I guess we could mark subnet as optional, and then, if provided, copy it also into new endpoint group" | 16:05 |
salv-orl_ | pc_m: just to make sure I fully understand - there is going to be a definite point in time in which subnet_id will disappear from the VPNService model, and will be replaced by endpointgroup_id | 16:06 |
pc_m | yes | 16:07 |
salv-orl_ | and a db migration will ensure continuitiy for all existing VPN services | 16:07 |
pc_m | yes | 16:07 |
salv-orl_ | so that you have "backward compatibility" (with quotes) | 16:07 |
pc_m | thinking if they enter in a subnet id in service API, could create an endpoint group with that subnet. | 16:07 |
salv-orl_ | then API users managing their VPN services *must(* update their clients to handle the new and improved API | 16:07 |
*** links has quit IRC | 16:08 | |
*** yamamoto has joined #openstack-neutron | 16:08 | |
*** sudipto has joined #openstack-neutron | 16:08 | |
pc_m | then, if they also specify peer-cidrs in the connection, we could create another group reference it, look up the subnet from the service and find what endpoint group it is in and use that ID. | 16:08 |
salv-orl_ | if you still allow to create VPNServices with a subnet_id it is not difficult to generate an implicitly endpoint group out of it, but there are caveats that require extra work | 16:08 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 16:09 |
*** gberginc has joined #openstack-neutron | 16:09 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 16:09 |
pc_m | salv-orl_: yeah, that was what I was thinking, to allow old API for a while. Would that make sense or cause more issues? | 16:09 |
*** Aish has joined #openstack-neutron | 16:09 | |
salv-orl_ | for instance the API client will expect that same subnet_id in a GET response | 16:09 |
salv-orl_ | pc_m: everything is a tradeoff | 16:09 |
salv-orl_ | either you manage the complexity in software, or the user takes the hit for handling the backward incompatible change. | 16:10 |
BobBall | Is Maru Newby still involved? Gerrit doesn't show commits for a while. Asking as he's a SME for the old XenAPI integration and we have some questions :) | 16:10 |
*** jschwarz has quit IRC | 16:10 | |
*** vivek-ebay has joined #openstack-neutron | 16:10 | |
salv-orl_ | Personally I would always spare the user from the pain of dealing with anything, but there are pro and cons | 16:10 |
amuller | BobBall: That's a no | 16:10 |
salv-orl_ | for insttance, it might be that users really don't care | 16:10 |
salv-orl_ | so we | 16:10 |
salv-orl_ | would be talking about nothing | 16:10 |
BobBall | amuller: That's a shame :/ But thanks for confirming. I won't go looking for him then! | 16:11 |
pc_m | salv-orl_: That's another good point. mestery did an email and it doesn't sound like many operators using VPN. | 16:11 |
*** thumpba has joined #openstack-neutron | 16:12 | |
mestery | pc_m salv-orl_: Right, like many things, not many using it, the ones who are want it kept around, but no one wnats to put people on it | 16:12 |
salv-orl_ | pc_m: I see. The reason is if the only think we're breaking are 3/4 operators who haven't even deployed this stuff in production, then why bother? | 16:12 |
pc_m | mestery: discussion how to handle migration to new API. | 16:12 |
*** jckasper has joined #openstack-neutron | 16:12 | |
*** dims__ has quit IRC | 16:12 | |
*** yamamoto has quit IRC | 16:12 | |
*** jistr has quit IRC | 16:13 | |
*** dims has joined #openstack-neutron | 16:13 | |
pc_m | discussing... | 16:13 |
*** vivek-eb_ has quit IRC | 16:13 | |
*** dims is now known as dims__ | 16:13 | |
salv-orl_ | mestery, pc_m: my point is - if you have users you must care about them as if they were your own children (well not really like that, maybe more pets) If you haven't we should not be spending time thinking about providing backward compatibility if noone is going to enjoy it anyway | 16:14 |
mestery | salv-orl_: +1000.5 | 16:14 |
*** mitchjameson has joined #openstack-neutron | 16:14 | |
pc_m | mestery: So what should we do in this case? | 16:14 |
pc_m | salv-orl_: Agreed totally. | 16:14 |
salv-orl_ | pc_m, mestery: why not send a mail on the operators list warning of an impeding backward incompatible change in the vpnaas API? | 16:15 |
salv-orl_ | if nobody barks in like - 7 days - then go ahead with your plan of guaranteeing continuity only for services already deployed | 16:15 |
*** SourabhP has quit IRC | 16:15 | |
salv-orl_ | if people complain then we'll study a backward compatibility strategy | 16:15 |
pc_m | salv-orl_: sounds good to me. Is that a different mailing list than openstack dev one? | 16:16 |
mestery | salv-orl_ pc_m : Yes, we need that email sent | 16:16 |
mestery | pc_m: It is | 16:16 |
pc_m | mestery: Do you want me to send that, or you? | 16:16 |
*** rushil has quit IRC | 16:17 | |
mestery | pc_m: Can you send this out? I'll back you up should someone push back | 16:17 |
pc_m | mestery: Sure. Just tell me the list | 16:17 |
pc_m | salv-orl_: back to the extension stuff... so I have a new module. Should it be introducing a new class, as I did so far? | 16:18 |
*** ygbo has quit IRC | 16:18 | |
mestery | pc_m: openstack-operators | 16:18 |
pc_m | mestery: thanks. | 16:19 |
*** _cjones_ has joined #openstack-neutron | 16:19 | |
salv-orl_ | pc_m: please fill me in - I emptied my brain over the weekend as I washed it in litres of wines | 16:19 |
mestery | lol | 16:20 |
salv-orl_ | is the new module for the endpoint groups stuff | 16:20 |
*** rushil has joined #openstack-neutron | 16:20 | |
pc_m | mestery: I have PS out with create, and another with delete/list/show APIs done for endpoint groups. As reviewer, do you want to see several small change sets chained together, or one larger commit with all of endpoint group work (and another for multiple local subnets, for example)? | 16:20 |
*** Aish has quit IRC | 16:20 | |
pc_m | salv-orl_: Yes, this one https://review.openstack.org/#/c/212692/2/neutron_vpnaas/extensions/vpnaas_v3.py | 16:21 |
*** l6unchpad has quit IRC | 16:21 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [WIP] Add MTU support for FedoraStrongSwan driver https://review.openstack.org/216324 | 16:21 |
*** moha_hunt has quit IRC | 16:22 | |
pc_m | salv-orl_: I was a little confused about the comment in vpn_db.py module, on not having two classes. | 16:22 |
*** liangbo has quit IRC | 16:23 | |
pc_m | salv-orl_: Also, I set this up so that vpnaas_v3.py is a superset of the vpnaas.py. Is that correct? | 16:23 |
salv-orl_ | pc_m: indeed with that kind of change you will end up either needing to classes or doing some magic, which is generally frowned upon by our core team | 16:23 |
* salv-orl_ noone of team studied at Hogwarths apparently | 16:24 | |
salv-orl_ | pc_m: now let's assume we adopt the strategy you just outlined | 16:24 |
pc_m | salv-orl_: I'm still pretty fuzzy on how to handle the the two versions. | 16:24 |
pc_m | salv-orl_: OK | 16:24 |
salv-orl_ | in that case I think we would not need two versions at all, but an incremental change | 16:24 |
*** sudipto has quit IRC | 16:25 | |
pc_m | salv-orl_: You're talking add the dbase and API to existing extension module? | 16:25 |
salv-orl_ | pc_m: ext modules | 16:25 |
* pc_m want to make sure we're talking about same thing. | 16:25 | |
salv-orl_ | pc_m: I gotta go afk for 10 minutes. I'll ping you when I'll be back | 16:26 |
pc_m | salv-orl_: OK. Need to reset to make sure we're talking of same proposal. | 16:26 |
*** ronis_ has joined #openstack-neutron | 16:26 | |
* pc_m we talked about several things. | 16:26 | |
pc_m | mestery: FYI, I created a commit to split models from database (per salv-orl_ suggestion), and have pushed that out as https://review.openstack.org/#/c/216248/ | 16:27 |
mestery | thanks pc_m! | 16:27 |
openstackgerrit | enikanorov proposed openstack/neutron: Add periodic agents health check. https://review.openstack.org/181633 | 16:28 |
pc_m | mestery: I updated the endpoint group dev ref to pick the proposal desired, and have some commits out for this (as discussed). | 16:28 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 16:28 |
*** geekinutah has quit IRC | 16:28 | |
mestery | thanks pc_m | 16:28 |
*** sudipto has joined #openstack-neutron | 16:29 | |
pc_m | mestery: sure. Looking to see how you, and other cores want to review the endpoint group stuff. One commit or chained patch sets. Preference? | 16:29 |
*** vikram has joined #openstack-neutron | 16:30 | |
*** gberginc has quit IRC | 16:31 | |
*** jbell8 has quit IRC | 16:31 | |
*** cdl has left #openstack-neutron | 16:31 | |
*** vivek-ebay has quit IRC | 16:32 | |
*** ildikov has joined #openstack-neutron | 16:33 | |
*** jckasper has quit IRC | 16:33 | |
*** sdake_ has joined #openstack-neutron | 16:34 | |
salv-orl_ | pc_m: I'm back. For extension modules, I think that if we accept the bw incompatible change there's no reason to declare a vpnaas_v3 API with a new extension that supersedes vpnaas_v2. In that case I would ass endpoint_groups as a new extension which has vpnaas as a requirements | 16:34 |
salv-orl_ | the endpoint_groups extension will introduce the new endpoint resources and do the bw incompatible change in the VPNService resource | 16:34 |
*** stanzgy has joined #openstack-neutron | 16:34 | |
salv-orl_ | but that's pretty much it. | 16:34 |
salv-orl_ | you will still have a single DB class | 16:34 |
*** clev is now known as clev-away | 16:34 | |
salv-orl_ | pc_m: If that works for you the next step is to discuss how to transition (single patch, patch train, feature branch, etc) | 16:35 |
pc_m | salv-orl_: Gotcha. Would I just add it to the existing vpnaas.py file and use the existing base class. | 16:36 |
pc_m | ? | 16:36 |
*** jpena has quit IRC | 16:36 | |
salv-orl_ | maybe still a new module ;) because you want to let the user know that you support this new feature, don't you? | 16:36 |
salv-orl_ | and the only way you have to do that is to declare a new ext | 16:36 |
*** pbrooko has quit IRC | 16:36 | |
salv-orl_ | but if you don't care about that, just add it to vpnaas | 16:36 |
salv-orl_ | since you'd still be using the same db class, in this case doing the extra extension module is trivial | 16:37 |
*** sdake has quit IRC | 16:37 | |
pc_m | salv-orl_: So I'd use the existing module (maybe rename to vpn_endpoint_groups.py?) and what happens with the class, do the same as currently, new subclass? | 16:38 |
* pc_m I'm still not sure how it all fits together. | 16:39 | |
salv-orl_ | pc_m: by existing you mean the module in the patch under review or the one in trunk? | 16:39 |
*** sbalukoff has quit IRC | 16:39 | |
*** lyanchih has quit IRC | 16:39 | |
*** alex_klimov has quit IRC | 16:40 | |
pc_m | salv-orl_: sort fo getting a bit lost... probably because I don't understand all the terminology. Sorry. | 16:40 |
pc_m | salv-orl_: When you say adding endpoint_groups as a new extension which has vpnaas as a requirements, what do you mean? | 16:40 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: [neutron-db-manage] Introduce contract and expand commands https://review.openstack.org/216313 | 16:40 |
pc_m | In PS, I have a new module with the API, a base class, and in the plugin I refer to vpnaas_v3, right now. | 16:41 |
*** absubram has joined #openstack-neutron | 16:42 | |
pc_m | salv-orl_: If I put the API in a new module, is it the module name (less .py) that is used in the plugin as an alias? Is that how it "sees" the new API? | 16:43 |
*** dims__ has quit IRC | 16:43 | |
pc_m | That's what I have in the PS now, only with vpnaas_v3.py as the module name. | 16:43 |
*** rushil has quit IRC | 16:44 | |
*** wendar_ is now known as wendar | 16:45 | |
* pc_m feels bad I don't have a good grounding in how all the API stuff works.. | 16:45 | |
*** rushil has joined #openstack-neutron | 16:45 | |
*** dims__ has joined #openstack-neutron | 16:45 | |
*** mohankumar has joined #openstack-neutron | 16:46 | |
*** ameybhide has quit IRC | 16:47 | |
armax | pc_m: regarding API extension handling there’s some review https://review.openstack.org/#/c/203878/ that’s still waiting for you | 16:47 |
armax | pc_m: do you think that what you’re doing has some interceptions with the work I did with service providers? | 16:48 |
pc_m | armax: Will look at comment. | 16:49 |
*** john-davidge has quit IRC | 16:49 | |
*** nyechiel has quit IRC | 16:50 | |
salv-orl_ | pc_m: your last stament is correct, and you would have to add the alias fot the endpoint group extension to the vpnaas plugin. If you "expand" the vpnaas extension that won't be needed. Again, the question is: is there any benefit to users in letting them explicitly know that there is a new feature called endpoint groups? | 16:50 |
*** geekinutah has joined #openstack-neutron | 16:50 | |
salv-orl_ | if the answer if yes, and the reason is not "because it would be cool" or "because it's nice to", then do it | 16:51 |
*** vivek-ebay has joined #openstack-neutron | 16:51 | |
salv-orl_ | otherwise add the new stuff to vpnaas.py and don't bother about anything | 16:51 |
*** liangbo has joined #openstack-neutron | 16:51 | |
*** jbell8 has joined #openstack-neutron | 16:52 | |
pc_m | salv-orl_: gotcha. Now, if it is a new module, is the alias in the plugin what makes that new extension visible? | 16:52 |
salv-orl_ | pc_m: yes | 16:53 |
pc_m | salv-orl_: Would it have its own "base class" only for that extension? | 16:53 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 16:53 |
*** piyanai has quit IRC | 16:54 | |
openstackgerrit | Miguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports https://review.openstack.org/200952 | 16:55 |
pc_m | salv-orl_: In my PS, I had class Vpnaas_v3, which was a superset of the methods for the existing extension and the new extension. Should that have only the APIs for the new extension? | 16:55 |
*** nyechiel has joined #openstack-neutron | 16:56 | |
*** jwy has joined #openstack-neutron | 16:56 | |
*** Aish has joined #openstack-neutron | 16:56 | |
salv-orl_ | pc_m: correct | 16:57 |
pc_m | salv-orl_: OK, so I did that wrong. Thanks. | 16:58 |
*** greystart has joined #openstack-neutron | 16:58 | |
*** minwang2 has joined #openstack-neutron | 16:58 | |
*** lindycoder has quit IRC | 16:59 | |
*** diegows has joined #openstack-neutron | 17:00 | |
pc_m | salv-orl_: So don't need to subclass from the Vpnaas class. | 17:00 |
*** piyanai has joined #openstack-neutron | 17:00 | |
salv-orl_ | pc_m: not unless you want to push an alternative vpnaas api. | 17:02 |
salv-orl_ | there is only one caveat | 17:02 |
salv-orl_ | if you do two extension modules you need to ensure that vpnaas is loaded before the module for endpoint groups | 17:02 |
pc_m | salv-orl_: So may be needed, when do multiple local subnet? | 17:02 |
salv-orl_ | there is a way to tell that to the ext manager, gimme 2 minutes to find how it's done | 17:02 |
*** sdake has joined #openstack-neutron | 17:02 | |
*** mitchjameson has quit IRC | 17:03 | |
salv-orl_ | pc_m: sorry, what might be needed when doing multiple local subnet? | 17:03 |
*** insequent has quit IRC | 17:03 | |
*** nyechiel has quit IRC | 17:04 | |
pc_m | salv-orl_: That's when we introduce use of the new API to existing commands. | 17:04 |
*** shivrao has joined #openstack-neutron | 17:05 | |
pc_m | salv-orl_: Or could they be updated under the vpnaas.py module? | 17:05 |
*** rossella_s has quit IRC | 17:05 | |
*** sdake_ has quit IRC | 17:06 | |
*** rossella_s has joined #openstack-neutron | 17:06 | |
pc_m | salv-orl_: I'm not sure whether it should be a separate extension. Seems like it logically should - a new API in a new module, but I'm not sure of a user advantage. Suggestions? | 17:06 |
salv-orl_ | pc_m: I do not have a strong or even a weak opinion here. It seems to me that at the end of the day since you're getting rid of the old way of doings things it does not really matter | 17:06 |
*** fitoduarte has joined #openstack-neutron | 17:07 | |
*** garyk has quit IRC | 17:07 | |
*** lindycoder has joined #openstack-neutron | 17:07 | |
*** britthouser has joined #openstack-neutron | 17:07 | |
salv-orl_ | users cannot really benefit from knowing that there are endpoints group because the MUST use it | 17:07 |
*** Swami has quit IRC | 17:07 | |
salv-orl_ | it's not like they have a choice | 17:07 |
pc_m | salv-orl_: To summarize, can place under existing extension, or as a new extension module (and use subclass and only alias to the new module)? | 17:08 |
*** sudipto has quit IRC | 17:09 | |
*** jckasper has joined #openstack-neutron | 17:09 | |
salv-orl_ | pc_m: yes, but if you choose to do a new extension, subclassing is not really needed | 17:10 |
salv-orl_ | because the new extension should describe vpnaas endpoint groups only | 17:10 |
pc_m | salv-orl_: by new, I meant additional extension. | 17:10 |
salv-orl_ | pc_m: sure | 17:10 |
pc_m | salv-orl_: gotcha, but have to make sure things load right, correct? | 17:10 |
salv-orl_ | pc_m: yes, still looking that up | 17:11 |
pc_m | salv-orl_: unless do a subclass and remove the vpnaas alias, if I understand. | 17:11 |
salv-orl_ | pc_m: that's an unnecessary alternative in this case | 17:11 |
*** Swami has joined #openstack-neutron | 17:12 | |
*** Aish has quit IRC | 17:12 | |
*** insequent has joined #openstack-neutron | 17:12 | |
*** nmagnezi has quit IRC | 17:13 | |
*** Marga_ has quit IRC | 17:13 | |
pc_m | salv-orl_: seems like the simplest approach is to add to vpnaas.py, and it would be an unused API. | 17:13 |
salv-orl_ | pc_m: sure | 17:14 |
salv-orl_ | that's the simplest thing to do | 17:14 |
salv-orl_ | and possibly even the most correct one in your case | 17:14 |
pc_m | salv-orl_: Now this endpoint groups API could be used by other VPN types (BGP, MPLS, L2,...). Does that fit into consideration at all? | 17:14 |
openstackgerrit | Brian Haley proposed openstack/neutron: Defer freeing of conntrack zone ids until allocation fails https://review.openstack.org/215791 | 17:15 |
salv-orl_ | pc_m: it ultimately depends on how these other VPN types are going to be managed by API users. I have no idea of that unfortunately. | 17:15 |
pc_m | salv-orl_: me neither. | 17:15 |
salv-orl_ | pc_m: just in case you opt for putting endpoint_groups in a new module, the extension module must have a get_required_extensions method | 17:16 |
*** jckasper has quit IRC | 17:17 | |
salv-orl_ | and that method must return ['vpnaas'] | 17:17 |
salv-orl_ | that will ensure vpnaas is always loaded first | 17:17 |
salv-orl_ | but as I said this is really a minor point in your strategy now. | 17:17 |
salv-orl_ | perhaps more of a distraction, because you can even go ahead and do everything in the current extension module, | 17:17 |
pc_m | salv-orl_: thank you though. It helps complete picture. | 17:17 |
salv-orl_ | and maybe move endpoint groups in another module later on, should you realize it's the best thing to do | 17:18 |
pc_m | salv-orl_: That was my thought... add to existing for now. | 17:18 |
pc_m | salv-orl_: So let me go in that direction, send email to operators about backward incompatible change (with migration though), and see how it works out. | 17:19 |
pc_m | salv-orl_: I just need to see how VPN reviewers want to digest the patches. | 17:19 |
*** madhu_ak has joined #openstack-neutron | 17:20 | |
salv-orl_ | pc_m: the plane sounds good to me. Do you have voting VPN jobs at the moment? | 17:20 |
salv-orl_ | like API or scenario jobs? | 17:20 |
*** aix has quit IRC | 17:20 | |
pc_m | salv-orl_: UT, and functional (including scenario) jobs voting right now. API is under neutron, but is in the process of being moved over to VPN repo by another developer. | 17:21 |
*** gsagie_ has joined #openstack-neutron | 17:21 | |
salv-orl_ | pc_m: ok then you are surely aware that if you do multiple patches then the jobs might be broken for a while, and that you'll need also to change the code for the API job at least to reflect the new API | 17:22 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Flag port sec as disabled on router GW ports https://review.openstack.org/215491 | 17:23 |
*** abhiraut has joined #openstack-neutron | 17:23 | |
*** sudipto has joined #openstack-neutron | 17:23 | |
pc_m | salv-orl_: Right. | 17:24 |
pc_m | mestery: armax: dougwig: For the VPN endpoint groups and multiple local subnet feature, how would you like to see the patches? One patch for all, one for endpoint groups and one for multiple local subnets, chain of patches for each of the two parts, or something else? Note salv-orl_ point above. | 17:25 |
mestery | pc_m: I'd like to see it so the jobs aren't broken :) | 17:25 |
armax | mestery: +1 | 17:25 |
pc_m | mestery: armax: dougwig: My preference is probably to do two patches, one for each feature. | 17:25 |
armax | :) | 17:25 |
armax | makes sense | 17:26 |
*** emagana has quit IRC | 17:26 | |
mestery | +1.5 | 17:26 |
pc_m | mestery: armax: me too :) I think the two patches will do that. | 17:26 |
*** lilynn has quit IRC | 17:26 | |
pc_m | mestery: armax: salv-orl_: I'll do two patches, one for endpoint groups, and one for multiple local subnets. The former should be pretty independent. Will also add the new API under the existing extension, unless you guys have heartburn with that choice. OK? | 17:28 |
*** sridhar_ram has joined #openstack-neutron | 17:28 | |
*** jckasper has joined #openstack-neutron | 17:28 | |
*** diegows has quit IRC | 17:29 | |
*** abhiraut has quit IRC | 17:29 | |
*** nlahouti has joined #openstack-neutron | 17:30 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: devref: added guidelines on how to maintain sub-projects https://review.openstack.org/216348 | 17:32 |
*** emagana has joined #openstack-neutron | 17:32 | |
*** lazy_prince has quit IRC | 17:32 | |
ihrachyshka | mestery, I've pushed a devref page with some guidelines for subprojects. please take a look and say what you think ^^ | 17:32 |
pc_m | salv-orl_: Thank so much for all the guidance here! | 17:32 |
*** achanda has joined #openstack-neutron | 17:33 | |
*** Sukhdev has quit IRC | 17:33 | |
openstackgerrit | Matthew Treinish proposed openstack/neutron: Stop logging STDOUT and STDERR on every shell out https://review.openstack.org/216350 | 17:34 |
openstackgerrit | Merged openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/204422 | 17:35 |
*** piyanai has quit IRC | 17:36 | |
*** emagana has quit IRC | 17:37 | |
*** abhiraut has joined #openstack-neutron | 17:38 | |
*** garyk has joined #openstack-neutron | 17:38 | |
*** sudipto has quit IRC | 17:38 | |
*** baoli_ has quit IRC | 17:40 | |
*** SridharG has joined #openstack-neutron | 17:40 | |
*** emagana has joined #openstack-neutron | 17:40 | |
SridharG | haleyb: can you pls take a look at https://review.openstack.org/#/c/205101/ thanks. | 17:40 |
*** amuller is now known as amuller_afk | 17:41 | |
*** gberginc has joined #openstack-neutron | 17:43 | |
*** bharathm has joined #openstack-neutron | 17:43 | |
*** sudipto has joined #openstack-neutron | 17:44 | |
*** gsagie_ has quit IRC | 17:44 | |
*** baoli has joined #openstack-neutron | 17:44 | |
*** clev-away is now known as clev | 17:46 | |
*** srilatha has quit IRC | 17:48 | |
*** srilatha has joined #openstack-neutron | 17:48 | |
pc_m | mestery: Do you know if dougwig is around? | 17:49 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants https://review.openstack.org/215782 | 17:50 |
salv-orl_ | pc_m: that was my pleasure | 17:50 |
*** vikram has quit IRC | 17:50 | |
*** Sukhdev has joined #openstack-neutron | 17:52 | |
*** jprovazn has quit IRC | 17:55 | |
*** ihrachyshka has quit IRC | 17:56 | |
pc_m | mestery: Can you approve https://review.openstack.org/#/c/213253/. Doug wanted me to do a dummy neutron change to verify the test, and I did, under https://review.openstack.org/#/c/215208/ and it's fine. Need approval, so this can upstream. | 17:57 |
*** ivar-lazzaro has joined #openstack-neutron | 17:58 | |
*** rm_work is now known as rm_work|away | 18:00 | |
*** sbalukoff has joined #openstack-neutron | 18:00 | |
openstackgerrit | Merged openstack/python-neutronclient: Remove --shared option from firewall-create https://review.openstack.org/194652 | 18:00 |
*** piyanai has joined #openstack-neutron | 18:02 | |
*** Sukhdev has quit IRC | 18:03 | |
*** mohankumar has quit IRC | 18:04 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 18:05 |
*** baoli has quit IRC | 18:06 | |
*** saksham has joined #openstack-neutron | 18:11 | |
*** neophy has quit IRC | 18:12 | |
*** SridharG has left #openstack-neutron | 18:14 | |
*** sdake_ has joined #openstack-neutron | 18:14 | |
*** zengine has joined #openstack-neutron | 18:14 | |
openstackgerrit | Shraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent https://review.openstack.org/212836 | 18:15 |
*** agireud has quit IRC | 18:15 | |
*** baoli has joined #openstack-neutron | 18:16 | |
*** piyanai has quit IRC | 18:16 | |
*** sdake has quit IRC | 18:17 | |
*** sdake has joined #openstack-neutron | 18:18 | |
*** bapalm- is now known as bapalm_ | 18:19 | |
*** ivar-lazzaro has quit IRC | 18:20 | |
*** baoli has quit IRC | 18:20 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:20 | |
*** jckasper has quit IRC | 18:21 | |
*** amuller_afk is now known as amuller | 18:21 | |
*** sdake_ has quit IRC | 18:21 | |
*** arosen has joined #openstack-neutron | 18:22 | |
*** rushil has quit IRC | 18:24 | |
*** ivar-lazzaro has quit IRC | 18:25 | |
*** baoli has joined #openstack-neutron | 18:25 | |
*** piyanai has joined #openstack-neutron | 18:28 | |
*** anilvenkata has joined #openstack-neutron | 18:28 | |
*** sthillma has joined #openstack-neutron | 18:28 | |
*** zhhuabj has quit IRC | 18:29 | |
*** zhhuabj has joined #openstack-neutron | 18:30 | |
*** garyk has quit IRC | 18:30 | |
*** liangbo has quit IRC | 18:30 | |
*** abhiraut has quit IRC | 18:31 | |
*** mickeys has quit IRC | 18:31 | |
*** piyanai has quit IRC | 18:31 | |
pc_m | mestery: ping | 18:32 |
*** nlahouti has quit IRC | 18:33 | |
*** neophy has joined #openstack-neutron | 18:33 | |
*** piyanai has joined #openstack-neutron | 18:34 | |
*** agireud has joined #openstack-neutron | 18:34 | |
*** piyanai has quit IRC | 18:35 | |
*** absubram has quit IRC | 18:36 | |
*** carl_baldwin has joined #openstack-neutron | 18:37 | |
*** jckasper has joined #openstack-neutron | 18:40 | |
*** sthillma_ has joined #openstack-neutron | 18:40 | |
*** sthillma has quit IRC | 18:40 | |
*** sthillma_ is now known as sthillma | 18:40 | |
*** stanzgy has quit IRC | 18:41 | |
*** stanzgy has joined #openstack-neutron | 18:41 | |
regXboi | amuller: ping - how comfortable would you be with the idea of setting oslo.concurrency log level to something above DEBUG during tempest runs? | 18:41 |
amuller | regXboi: sounds like a horrible idea =D | 18:41 |
amuller | regXboi: what problem are you facing? | 18:41 |
regXboi | amuller: 20% of the q-svc log is from oslo.concurrency and I'm wondering if we need it | 18:42 |
regXboi | amuller: well, ~18% to be more precise | 18:42 |
amuller | regXboi: too much logging then? why set it to higher than DEBUG then? | 18:42 |
regXboi | higher meaning to *not* show DEBUG level statement | 18:43 |
*** Marga_ has joined #openstack-neutron | 18:44 | |
*** Marga_ has quit IRC | 18:45 | |
*** Marga_ has joined #openstack-neutron | 18:46 | |
*** bharathm has quit IRC | 18:48 | |
*** rm_work|away is now known as rm_work | 18:48 | |
*** bharathm has joined #openstack-neutron | 18:48 | |
*** VimalTewari has joined #openstack-neutron | 18:50 | |
*** piyanai has joined #openstack-neutron | 18:51 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 18:53 |
*** minwang2 has quit IRC | 18:54 | |
amuller | regXboi: generally I'm OK with no showing those lock grabbed messages | 18:54 |
amuller | not* | 18:54 |
*** numan has joined #openstack-neutron | 18:55 | |
*** fawadkhaliq has quit IRC | 18:55 | |
regXboi | amuller: yeah - I'm thinking to get them to not show up in the gate runs will take a patch to the infra project structure *somewhere* | 18:55 |
regXboi | or devstack-gate - not sure yet | 18:55 |
regXboi | dougwig (as infra lieutenant): FYI ^^^^^^^ | 18:56 |
*** numan has quit IRC | 18:56 | |
*** numan has joined #openstack-neutron | 18:57 | |
*** sthillma_ has joined #openstack-neutron | 18:58 | |
*** jgriffith has quit IRC | 18:59 | |
*** lori is now known as lori|away | 18:59 | |
*** anilvenkata has left #openstack-neutron | 19:00 | |
clarkb | it shouldnt | 19:00 |
*** sthillma has quit IRC | 19:00 | |
*** sthillma_ is now known as sthillma | 19:00 | |
clarkb | log config is all in neutron ir devstack | 19:00 |
regXboi | clarkb: there is no way to inject specific config into devstack-gate? | 19:01 |
*** drankis has quit IRC | 19:01 | |
*** jgriffith has joined #openstack-neutron | 19:01 | |
*** jgriffith is now known as Guest7193 | 19:02 | |
clarkb | there is but we shouldnt | 19:02 |
clarkb | devstack gate should not set defaults for you | 19:02 |
regXboi | clarkb: ok, then I'll go look at devstack-gate | 19:02 |
clarkb | no please dont | 19:02 |
regXboi | wait | 19:02 |
regXboi | I think I'm confused | 19:02 |
regXboi | I don't want to set the default - I want it to be injected as part of a particular job's config | 19:02 |
clarkb | devstack-gate exists to configure jobs it does not exist to set neutron/devstack defaults | 19:02 |
clarkb | if you set it in devstack-gate it becomes dfacto default | 19:03 |
clarkb | because all the testing is then done in that setup | 19:03 |
*** Guest7193 has quit IRC | 19:04 | |
regXboi | clarkb: let me explain the situation | 19:04 |
regXboi | clarkb: right now 18% of the logs of neutron server during a gate/check pipeline run come from oslo_concurrency.lockutils | 19:05 |
regXboi | clarkb: I'd like to be able to set the log level for that module to WARN so that the DEBUG statements don't appear | 19:05 |
clarkb | yes so do that in either neutron or devstaxk | 19:06 |
clarkb | do it in neutron if it makes sense for all neutron deployments, do it in devstack if it makes sense for dev test | 19:06 |
regXboi | I'm not sure it makes sense in either - I'm not hubris enough to say it should apply in either of those cases - I'm talking about what is produced in the check/gate pipelines only | 19:07 |
clarkb | if its going to be set in devstack gate as a default it likely should just go in devstack at least | 19:08 |
* regXboi thinks about this | 19:09 | |
clarkb | its also possible lockutils should change but again dont set what should be defaults in devstaxk gate. we can and should push those closer to the issue | 19:09 |
* regXboi still is considering | 19:09 | |
*** numan is now known as numans | 19:10 | |
regXboi | clarkb: ok, having looked at devstack, I can grok that it should go there | 19:11 |
*** annegent_ has joined #openstack-neutron | 19:12 | |
amuller | regXboi: it'd be interesting to hear oslo concurrency devs | 19:12 |
regXboi | amuller: in what way? | 19:13 |
amuller | regXboi: maybe oslo concurrency devs are interested in that DEBUG output but the different applications (nova, neutron etc) aren't | 19:13 |
amuller | regXboi: so worth hearing their opinion on that stement | 19:13 |
amuller | statement | 19:13 |
regXboi | ah | 19:13 |
regXboi | do we have an oslo concurrency folks here? | 19:13 |
amuller | regXboi: I'd try #openstack-oslo | 19:14 |
regXboi | amuller: already there | 19:15 |
*** annegentle has quit IRC | 19:16 | |
*** jaypipes has quit IRC | 19:18 | |
*** matrohon has joined #openstack-neutron | 19:20 | |
regXboi | amuller: in the absence of an answer from the oslo folks, I'm thinking the answer is to add a patch to set logging levels of specific modules via neutron.conf and then a patch to devstack setting oslo_concurrency.lockutils to something like INFO or WARN | 19:20 |
amuller | regXboi: I think you should send an email to openstack-dev with the problem and your suggestion | 19:21 |
regXboi | tag it neutron/oslo/infra ? | 19:21 |
amuller | regXboi: sure | 19:21 |
regXboi | ack | 19:21 |
amuller | regXboi: I'd also present the possibility of neutron raising oslo concurrency logging to info/warn globally for all files | 19:21 |
*** sudipto has quit IRC | 19:22 | |
regXboi | as opposed to the dual patch approach? | 19:22 |
*** njohnston is now known as nate_gone | 19:23 | |
amuller | regXboi: as opposed to the idea of changing log levels per module via neutron.conf | 19:23 |
regXboi | ack | 19:23 |
*** neiljerram_bb has joined #openstack-neutron | 19:24 | |
dims__ | amuller: regXboi: nova sets log levels for a bunch of things, https://github.com/openstack/nova/blob/master/nova/config.py#L47 | 19:26 |
regXboi | amuller, dims__: I think we could run with that as a start | 19:27 |
regXboi | amuller: and then if we need to change it, overlay it with config settings | 19:27 |
regXboi | amuller: so two patches | 19:27 |
regXboi | amuller: the one for now and the other as "if we find we need it" | 19:28 |
*** tidwellr1 has joined #openstack-neutron | 19:29 | |
regXboi | because there are other modules that we could catch with that first patch | 19:29 |
*** jwy has quit IRC | 19:30 | |
*** nlahouti has joined #openstack-neutron | 19:30 | |
pc_m | armax: ping | 19:31 |
armax | pc_m: pong | 19:32 |
pc_m | armax: I approved https://review.openstack.org/#/c/203878/. Sorry for the delay. I missed the response. | 19:32 |
armax | pc_m: no worries | 19:32 |
pc_m | I don' t see dougwig around. Can you approve https://review.openstack.org/#/c/213253/ ? | 19:32 |
* armax looks | 19:33 | |
pc_m | armax: This renames the devstack plugin for VPN. It was tested via a dummy neutron commit (https://review.openstack.org/#/c/215208/) | 19:33 |
*** slaweq has joined #openstack-neutron | 19:33 | |
*** jgriffit- has joined #openstack-neutron | 19:34 | |
*** srilatha has quit IRC | 19:34 | |
*** srilatha has joined #openstack-neutron | 19:34 | |
*** crose has quit IRC | 19:35 | |
*** srilatha has quit IRC | 19:39 | |
*** kbringard has quit IRC | 19:40 | |
*** minwang2 has joined #openstack-neutron | 19:42 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add tunneling support to full stack tests https://review.openstack.org/190786 | 19:43 |
*** VimalTewari has quit IRC | 19:45 | |
*** numans has quit IRC | 19:46 | |
*** shivrao has quit IRC | 19:47 | |
*** ZZelle has joined #openstack-neutron | 19:49 | |
*** ZZelle is now known as ZZll-off | 19:49 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants https://review.openstack.org/215782 | 19:51 |
*** ronis_ has quit IRC | 19:51 | |
*** abregman has joined #openstack-neutron | 19:52 | |
*** jwy has joined #openstack-neutron | 19:52 | |
*** yfried has joined #openstack-neutron | 19:52 | |
*** gberginc has quit IRC | 19:53 | |
*** piyanai has quit IRC | 19:53 | |
*** shivrao has joined #openstack-neutron | 19:57 | |
*** sdake_ has joined #openstack-neutron | 19:58 | |
*** crose has joined #openstack-neutron | 19:58 | |
*** crose has quit IRC | 19:58 | |
*** j^2 has left #openstack-neutron | 19:59 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Add l2pop support to full stack tests https://review.openstack.org/190832 | 19:59 |
*** sdake has quit IRC | 20:01 | |
*** kutija has joined #openstack-neutron | 20:03 | |
*** Marga_ has quit IRC | 20:03 | |
*** kbringard has joined #openstack-neutron | 20:03 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: WiP: set imported module default logging levels https://review.openstack.org/216404 | 20:04 |
*** Marga_ has joined #openstack-neutron | 20:05 | |
*** annegent_ has quit IRC | 20:07 | |
*** achanda has quit IRC | 20:07 | |
*** annegentle has joined #openstack-neutron | 20:07 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Remove _extract_roles method from neutron.policy https://review.openstack.org/216405 | 20:07 |
*** piyanai has joined #openstack-neutron | 20:11 | |
*** greystart has quit IRC | 20:11 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement https://review.openstack.org/216410 | 20:13 |
*** dmartls1 has quit IRC | 20:13 | |
pc_m | armax: Thanks for the review approval! | 20:14 |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement https://review.openstack.org/216410 | 20:14 |
*** kbringard has quit IRC | 20:15 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants https://review.openstack.org/215782 | 20:16 |
*** jckasper has quit IRC | 20:16 | |
*** jckasper has joined #openstack-neutron | 20:17 | |
*** baoli has quit IRC | 20:17 | |
openstackgerrit | Merged openstack/neutron: Move db agent schedulers test to a more appropriate place https://review.openstack.org/214131 | 20:18 |
*** fzdarsky has quit IRC | 20:18 | |
*** kbringard has joined #openstack-neutron | 20:19 | |
*** mickeys has joined #openstack-neutron | 20:20 | |
*** absubram has joined #openstack-neutron | 20:21 | |
*** Sukhdev has joined #openstack-neutron | 20:25 | |
*** vivek-ebay has quit IRC | 20:25 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants https://review.openstack.org/214806 | 20:26 |
*** slaweq has quit IRC | 20:28 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove out-of-tree vendor AGENT_TYPE_* constants https://review.openstack.org/216417 | 20:30 |
*** kbringard1 has joined #openstack-neutron | 20:31 | |
*** TrevorV has quit IRC | 20:32 | |
*** kbringard has quit IRC | 20:32 | |
*** jerrygb has joined #openstack-neutron | 20:33 | |
openstackgerrit | Merged openstack/neutron: Rationalize neutron logs to help in troubleshooting router issues https://review.openstack.org/215655 | 20:34 |
*** sdake_ is now known as sdake | 20:34 | |
*** jerrygb is now known as test0rz | 20:34 | |
*** test0rz is now known as asd112z | 20:34 | |
*** krtaylor has quit IRC | 20:35 | |
*** yfried has quit IRC | 20:36 | |
ZZll-off | kevinbenton, hi | 20:38 |
kevinbenton | ZZll-off: yo | 20:40 |
*** neiljerram_bb has quit IRC | 20:40 | |
*** achanda has joined #openstack-neutron | 20:40 | |
*** achanda has quit IRC | 20:41 | |
*** achanda has joined #openstack-neutron | 20:41 | |
ZZll-off | kevinbenton, any idea why there are 2 configuration options defining the integration bridge? | 20:42 |
*** daneyon has joined #openstack-neutron | 20:42 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: l3_db, l3_dvr_db: separate _process_floating_{ips, interfaces}() https://review.openstack.org/124696 | 20:42 |
amuller | ZZll-off: two? | 20:42 |
ZZll-off | kevinbenton, https://github.com/openstack/neutron/blob/master/neutron/agent/linux/interface.py#L34-L36 (default section) | 20:42 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Uniquely identify tunnel interfaces https://review.openstack.org/193770 | 20:43 |
kevinbenton | ZZll-off: i suspect one is to use for things that plug into OVS | 20:43 |
kevinbenton | ZZll-off: and the other is used for the OVS agent | 20:43 |
kevinbenton | ZZll-off: but that's just a quick guess | 20:44 |
ZZll-off | kevinbenton, https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py#L29-L30 (OVS section) | 20:44 |
*** rkukura has joined #openstack-neutron | 20:44 | |
kevinbenton | ZZll-off: yeah, the first one you sent is for things like DHCP agent and l3 agent | 20:44 |
ZZll-off | kevinbenton, because the option helps do not help to understand the difference | 20:44 |
*** sdake has quit IRC | 20:44 | |
kevinbenton | ZZll-off: that need to plug into OVS | 20:44 |
kevinbenton | ZZll-off: and the second is for the OVS agent to know where to expect them | 20:45 |
ZZll-off | kevinbenton, by why do we 2 options? | 20:45 |
ZZll-off | kevinbenton, by why do we need 2 options? | 20:45 |
ZZll-off | kevinbenton, (except option registration trouble) | 20:45 |
kevinbenton | ZZll-off: we probably don't | 20:46 |
kevinbenton | ZZll-off: i can't think of a reason you would want that | 20:47 |
*** daneyon has quit IRC | 20:47 | |
*** daneyon has joined #openstack-neutron | 20:47 | |
*** stanzgy has quit IRC | 20:47 | |
*** krtaylor has joined #openstack-neutron | 20:48 | |
*** vivek-ebay has joined #openstack-neutron | 20:48 | |
*** srilatha has joined #openstack-neutron | 20:48 | |
ZZll-off | kevinbenton, me neither, i could be a rare opportunity to reduce option number O_o | 20:50 |
*** shivrao has quit IRC | 20:51 | |
*** jwy has quit IRC | 20:53 | |
kevinbenton | ZZll-off: yeah | 20:54 |
*** tflynn has joined #openstack-neutron | 20:54 | |
kevinbenton | ZZll-off: does oslo have an option to deprecate in a consolidating fashion like that? | 20:54 |
ZZll-off | kevinbenton, let me find an example | 20:54 |
*** matrohon has quit IRC | 20:55 | |
*** daneyon_ has joined #openstack-neutron | 20:55 | |
*** daneyon has quit IRC | 20:58 | |
*** shivrao has joined #openstack-neutron | 20:58 | |
ZZll-off | kevinbenton, https://github.com/openstack/neutron/blob/8546d5ec3db3bd589dd810e2e8ed097b29068116/neutron/agent/dhcp/config.py#L50-L58 | 20:58 |
*** zengine has quit IRC | 20:58 | |
ZZll-off | kevinbenton, there is 2 ways: set an option as deprecated (deprecated_for_removal=True) or define an option with its deprecated version (deprecated_name=...) | 20:59 |
*** emagana has quit IRC | 21:00 | |
*** rushil has joined #openstack-neutron | 21:01 | |
*** jbell8 has quit IRC | 21:01 | |
*** rkukura has quit IRC | 21:01 | |
*** fitoduarte has quit IRC | 21:02 | |
*** vivek-ebay has quit IRC | 21:02 | |
*** fitoduarte has joined #openstack-neutron | 21:02 | |
*** emagana has joined #openstack-neutron | 21:03 | |
*** thorst has quit IRC | 21:03 | |
*** geekinutah has quit IRC | 21:04 | |
*** amuller has quit IRC | 21:04 | |
*** piyanai has quit IRC | 21:06 | |
*** neiljerram_bb has joined #openstack-neutron | 21:07 | |
neiljerram_bb | carl_baldwin : hi, are you around? | 21:07 |
carl_baldwin | neiljerram_bb: hi | 21:07 |
*** pc_m has quit IRC | 21:08 | |
neiljerram_bb | carl_baldwin : hi, good evening ; or perhaps still good day for you? | 21:08 |
*** tflynn has quit IRC | 21:09 | |
*** moshele has joined #openstack-neutron | 21:09 | |
carl_baldwin | neiljerram_bb: good day/evening to you too. | 21:09 |
neiljerram_bb | carl_baldwin : I wanted to ask if you'd mind taking a look at an RFE bug that I raised, in advance of tomorrow's drivers meeting. It's at https://bugs.launchpad.net/neutron/+bug/1486649 | 21:10 |
openstack | Launchpad bug 1486649 in neutron "Enhance DHCP agent and IP library for networking-calico interface driver" [Undecided,In progress] - Assigned to Neil Jerram (neil-jerram) | 21:10 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: Endpoint groups: Implements API and database https://review.openstack.org/212692 | 21:10 |
*** lilynn has joined #openstack-neutron | 21:11 | |
*** jbell8 has joined #openstack-neutron | 21:12 | |
neiljerram_bb | carl_baldwin : Kyle wrote, at http://lists.openstack.org/pipermail/openstack-dev/2015-August/072675.html, that he'd be keen to hear your view on it. | 21:12 |
neiljerram_bb | carl_baldwin : Separately, BTW, I'm very keen to help, however I can, as and when L3 modelling discussions continue. I know that's probably not a priority right now, but please just ask if I can do anything to help there. | 21:17 |
*** s3wong has joined #openstack-neutron | 21:18 | |
*** jwy has joined #openstack-neutron | 21:18 | |
carl_baldwin | neiljerram_bb: I’d really like to pick up that conversation. I feel I owe the community a summary of what we have discussed. | 21:20 |
*** Piet has joined #openstack-neutron | 21:20 | |
*** moshele has quit IRC | 21:20 | |
*** absubram has quit IRC | 21:23 | |
*** neiljerram_bb has quit IRC | 21:23 | |
*** neiljerram_bb has joined #openstack-neutron | 21:23 | |
openstackgerrit | Adolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/196893 | 21:26 |
*** mwagner_lap has quit IRC | 21:27 | |
*** abregman has quit IRC | 21:29 | |
*** neiljerram_home has joined #openstack-neutron | 21:33 | |
*** lilynn has quit IRC | 21:33 | |
*** lilynn has joined #openstack-neutron | 21:33 | |
*** diegows has joined #openstack-neutron | 21:33 | |
neiljerram_home | carl_baldwin: Hi again, I'm sorry, my phone dropped its IRC connection for a while, so I might have missed some of your messages... | 21:33 |
*** agireud has quit IRC | 21:34 | |
carl_baldwin | neiljerram_home: Not sure what you might’ve missed. Eavesdrop should be able to fill you in. | 21:34 |
*** piyanai has joined #openstack-neutron | 21:35 | |
neiljerram_home | carl_baldwin: Ah yes, I see what you said about picking up the L3 model conversation. | 21:35 |
openstackgerrit | Adolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/196893 | 21:37 |
neiljerram_home | carl_baldwin: As I said, I'm keen to help, but not sure how best to. Obviously we will be able to do a lot - other things permitting - in Tokyo; but perhaps there are other mechanisms or opportunities before then? | 21:38 |
mlavalle | carl_baldwin: hi... so, baed on your latest comment, you would be ok with the _validate_dns_name shown here: https://review.openstack.org/#/c/200952/20/neutron/extensions/dns.py | 21:38 |
mlavalle | ? | 21:39 |
*** abhiraut has joined #openstack-neutron | 21:39 | |
carl_baldwin | mlavalle: Yes. | 21:39 |
openstackgerrit | Adolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/196893 | 21:40 |
mlavalle | carl_baldwin: cool, thanks! | 21:40 |
carl_baldwin | mlavalle: Sorry for the churn. | 21:40 |
mlavalle | carl_baldwin: i'll fix the nits you found in that implementation | 21:40 |
mlavalle | carl_baldwin: not a problem. that is what open source development is all about...... healthy debate | 21:41 |
mlavalle | carl_baldwin: i've learned a lot, so thanks to you and kevinbenton | 21:41 |
carl_baldwin | mlavalle: I’ll get the egg wiped off my face soon and move on. :) | 21:42 |
*** Piet has quit IRC | 21:42 | |
*** yfried has joined #openstack-neutron | 21:44 | |
*** madhu_ak_ has joined #openstack-neutron | 21:46 | |
*** madhu_ak has quit IRC | 21:50 | |
*** lindycoder has quit IRC | 21:50 | |
*** tflynn has joined #openstack-neutron | 21:51 | |
*** emagana has quit IRC | 21:55 | |
*** agireud has joined #openstack-neutron | 21:56 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 21:57 |
*** neiljerram_home has quit IRC | 22:00 | |
*** rushil has quit IRC | 22:00 | |
*** madhu_ak_ is now known as madhu_ak | 22:01 | |
*** vivek-ebay has joined #openstack-neutron | 22:02 | |
*** emagana has joined #openstack-neutron | 22:03 | |
*** pradk has quit IRC | 22:04 | |
*** ujjain has quit IRC | 22:04 | |
*** rushil has joined #openstack-neutron | 22:05 | |
*** piyanai has quit IRC | 22:06 | |
*** ujjain has joined #openstack-neutron | 22:07 | |
*** ujjain has quit IRC | 22:07 | |
*** ujjain has joined #openstack-neutron | 22:07 | |
*** vivek-ebay has quit IRC | 22:07 | |
*** kbringard1 has quit IRC | 22:10 | |
*** tflynn has quit IRC | 22:10 | |
*** apuimedo_ has joined #openstack-neutron | 22:13 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Set imported module default logging levels https://review.openstack.org/216404 | 22:16 |
*** agireud has quit IRC | 22:19 | |
*** azbiswas has joined #openstack-neutron | 22:22 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 22:24 |
*** Sukhdev has quit IRC | 22:25 | |
*** asd112z has quit IRC | 22:25 | |
openstackgerrit | Merged openstack/neutron: Remove redundant logging statements from RootWrapDaemonHelper https://review.openstack.org/216260 | 22:26 |
openstackgerrit | Miguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports https://review.openstack.org/200952 | 22:27 |
*** regXboi has quit IRC | 22:27 | |
openstackgerrit | Merged openstack/neutron: Template for ModelMigrationTest for external repos https://review.openstack.org/213194 | 22:27 |
*** sdake has joined #openstack-neutron | 22:29 | |
openstackgerrit | Merged openstack/neutron: Make NeutronDbObjectDuplicateEntry exception more verbose https://review.openstack.org/214608 | 22:29 |
*** shoutm has joined #openstack-neutron | 22:32 | |
*** sbalukoff has quit IRC | 22:34 | |
*** azbiswas has quit IRC | 22:37 | |
*** jckasper has quit IRC | 22:37 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 22:37 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 22:37 |
*** emagana has quit IRC | 22:39 | |
*** thumpba has quit IRC | 22:42 | |
*** nplanel has joined #openstack-neutron | 22:43 | |
*** nplanel has quit IRC | 22:44 | |
*** nplanel has joined #openstack-neutron | 22:44 | |
*** shwetaap has quit IRC | 22:45 | |
*** diegows has quit IRC | 22:45 | |
*** insequent has quit IRC | 22:46 | |
*** sridhar_ram has quit IRC | 22:50 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement https://review.openstack.org/216410 | 22:50 |
*** dannywilson has joined #openstack-neutron | 22:50 | |
*** apuimedo_ has quit IRC | 22:51 | |
*** mwagner_lap has joined #openstack-neutron | 22:51 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement https://review.openstack.org/216410 | 22:52 |
*** garyk has joined #openstack-neutron | 22:52 | |
abhiraut | ping HenryG | 22:53 |
*** dannywil_ has quit IRC | 22:53 | |
*** emagana has joined #openstack-neutron | 22:53 | |
*** mlavalle has quit IRC | 22:55 | |
*** edmondsw has quit IRC | 22:57 | |
*** jgriffit- has quit IRC | 23:00 | |
*** arosen has quit IRC | 23:02 | |
*** jckasper has joined #openstack-neutron | 23:02 | |
*** tongli has joined #openstack-neutron | 23:02 | |
*** Marga_ has quit IRC | 23:02 | |
*** Marga_ has joined #openstack-neutron | 23:03 | |
*** jckasper has quit IRC | 23:03 | |
*** Aish has joined #openstack-neutron | 23:05 | |
*** Marga_ has quit IRC | 23:08 | |
*** thorst has joined #openstack-neutron | 23:08 | |
*** fitoduarte has quit IRC | 23:10 | |
*** zzzeek has quit IRC | 23:10 | |
*** neiljerram_bb has quit IRC | 23:11 | |
*** claudiub has quit IRC | 23:12 | |
*** luqas__ has joined #openstack-neutron | 23:15 | |
*** dims_ has joined #openstack-neutron | 23:16 | |
*** luqas has quit IRC | 23:17 | |
*** dims__ has quit IRC | 23:18 | |
armax | obondarev_, rossella_s: https://review.openstack.org/#/c/214131 broke neutron-vpnaas | 23:19 |
*** annegentle has quit IRC | 23:19 | |
*** vivek-ebay has joined #openstack-neutron | 23:19 | |
armax | not sure if there are other probjects affected | 23:19 |
*** dims__ has joined #openstack-neutron | 23:19 | |
*** dims_ has quit IRC | 23:21 | |
*** vivek-ebay has quit IRC | 23:23 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-vpnaas: Fix stale module import https://review.openstack.org/216460 | 23:24 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-vpnaas: Fix stale module import https://review.openstack.org/216460 | 23:24 |
armax | mestery: ^^^^^^^ | 23:24 |
armax | dougwig: ^^^^ | 23:25 |
abhiraut | ping akamyshnikova | 23:30 |
*** achanda_ has joined #openstack-neutron | 23:31 | |
*** dannywilson has quit IRC | 23:33 | |
*** achanda has quit IRC | 23:34 | |
*** abhiraut has quit IRC | 23:34 | |
*** dannywilson has joined #openstack-neutron | 23:34 | |
*** EmilienM has quit IRC | 23:34 | |
*** Fdaisuke has joined #openstack-neutron | 23:35 | |
*** NightKhaos has quit IRC | 23:35 | |
*** saksham has quit IRC | 23:35 | |
*** q3k has quit IRC | 23:35 | |
*** abhiraut has joined #openstack-neutron | 23:35 | |
*** achanda_ has quit IRC | 23:35 | |
*** vivek-ebay has joined #openstack-neutron | 23:36 | |
*** carl_baldwin has quit IRC | 23:36 | |
*** q3k has joined #openstack-neutron | 23:36 | |
*** EmilienM has joined #openstack-neutron | 23:36 | |
*** dsneddon is now known as dsneddon_away | 23:36 | |
*** NightKhaos has joined #openstack-neutron | 23:42 | |
*** morganfainberg is now known as morgan | 23:42 | |
Fdaisuke | armax: Hi, | 23:44 |
Fdaisuke | armax: I commented and uploaded new patch set to https://review.openstack.org/#/c/207328/ | 23:45 |
Fdaisuke | Would you please have a look at this? | 23:45 |
*** garyk has quit IRC | 23:46 | |
*** ajmiller has quit IRC | 23:50 | |
*** sdake_ has joined #openstack-neutron | 23:51 | |
*** tidwellr1 has quit IRC | 23:53 | |
*** emagana has quit IRC | 23:54 | |
*** tongli has quit IRC | 23:54 | |
*** sdake has quit IRC | 23:55 | |
*** emagana has joined #openstack-neutron | 23:56 | |
*** apuimedo has quit IRC | 23:57 | |
*** stefanb has quit IRC | 23:58 | |
*** stefanb has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!