Monday, 2015-08-24

*** Kennan2 is now known as Kennan00:02
openstackgerritRyan Moats proposed openstack/neutron: L3 DVR developer document  https://review.openstack.org/18291700:02
Fdaisukearmax: Hi,00:03
*** salv-orlando has quit IRC00:03
*** bharath has left #openstack-neutron00:04
*** anshul has quit IRC00:04
Fdaisukearmax: I commented and uploaded new patch set to https://review.openstack.org/#/c/207328/00:06
FdaisukeWould you please have a look at this?00:06
*** dsanders has quit IRC00:13
*** agireud has joined #openstack-neutron00:15
*** agireud_ has joined #openstack-neutron00:19
*** bharathm has joined #openstack-neutron00:19
*** agireud has quit IRC00:21
*** agireud_ is now known as agireud00:21
*** dannywil_ has joined #openstack-neutron00:21
*** shadower49 has quit IRC00:23
*** tiswanso has joined #openstack-neutron00:23
*** shadower49 has joined #openstack-neutron00:23
*** dannywilson has quit IRC00:24
openstackgerritRyan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting DVR  https://review.openstack.org/21565500:26
openstackgerritMiguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports  https://review.openstack.org/20095200:27
*** britthouser has joined #openstack-neutron00:28
*** banix has quit IRC00:28
openstackgerritKenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS  https://review.openstack.org/14841600:30
*** tiswanso has quit IRC00:36
*** tiswanso has joined #openstack-neutron00:37
*** dsanders has joined #openstack-neutron00:37
*** julim has quit IRC00:39
*** julim has joined #openstack-neutron00:40
*** RA_ has joined #openstack-neutron00:42
*** tiswanso has quit IRC00:44
*** rushil has joined #openstack-neutron00:44
*** bharathm has quit IRC00:44
*** tflynn_ has quit IRC00:44
*** amotoki has joined #openstack-neutron00:47
*** rotbeard has quit IRC00:50
*** amotoki has quit IRC00:50
*** rotbeard has joined #openstack-neutron00:50
*** tiswanso has joined #openstack-neutron00:51
*** tflynn has joined #openstack-neutron00:53
*** amotoki has joined #openstack-neutron00:56
*** tiswanso has quit IRC00:59
*** agireud has quit IRC00:59
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378101:01
*** zhenguo has joined #openstack-neutron01:01
*** salv-orlando has joined #openstack-neutron01:03
*** banix has joined #openstack-neutron01:03
*** yanxb__ has quit IRC01:04
*** yanxb has joined #openstack-neutron01:04
*** salv-orlando has quit IRC01:05
*** rossella_s has quit IRC01:05
*** rossella_s has joined #openstack-neutron01:06
*** tflynn has quit IRC01:06
openstackgerritYujiAzama proposed openstack/neutron-specs: [WIP] Add common classifier resource  https://review.openstack.org/19046301:08
*** agireud has joined #openstack-neutron01:09
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity  https://review.openstack.org/21381901:09
*** salv-orlando has joined #openstack-neutron01:11
*** tiswanso has joined #openstack-neutron01:13
openstackgerritAkihiro Motoki proposed openstack/neutron: Decomposition phase2 of NEC plugin  https://review.openstack.org/21607101:13
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378101:13
*** nplanel has joined #openstack-neutron01:16
*** nlahouti has quit IRC01:16
*** salv-orlando has quit IRC01:21
*** nplanel has quit IRC01:21
*** pavel_bondar has joined #openstack-neutron01:25
*** achanda has joined #openstack-neutron01:26
*** tiswanso has quit IRC01:29
*** shwetaap has quit IRC01:29
*** tiswanso has joined #openstack-neutron01:30
*** diegows has quit IRC01:30
openstackgerritKenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS  https://review.openstack.org/14841601:31
*** shivrao has quit IRC01:32
*** erkules_ has joined #openstack-neutron01:33
*** stanzgy has joined #openstack-neutron01:34
*** agireud has quit IRC01:35
*** erkules has quit IRC01:35
*** iwamoto has joined #openstack-neutron01:37
*** agireud has joined #openstack-neutron01:37
*** yalie has joined #openstack-neutron01:38
*** yalie has left #openstack-neutron01:38
*** piyanai has quit IRC01:44
*** bharathm has joined #openstack-neutron01:44
*** bharathm has quit IRC01:49
*** zhhuabj_ has quit IRC01:49
*** jbell8 has quit IRC01:52
*** agireud has quit IRC01:56
*** hichihara has joined #openstack-neutron02:00
*** Kennan has quit IRC02:02
*** Kennan has joined #openstack-neutron02:02
*** agireud has joined #openstack-neutron02:04
*** liudong has joined #openstack-neutron02:05
*** jingbo has joined #openstack-neutron02:06
*** changzhi has joined #openstack-neutron02:07
*** yamamoto has joined #openstack-neutron02:07
jingboHi, I have a problem while I am debuging q-agt.02:07
jingboDEBUG 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:4602:07
*** agireud has quit IRC02:08
jingboIt stopped at this step while I am using pydev to debug the q-agt.02:08
jingboAny suggestion?02:08
*** reedip has quit IRC02:09
*** tiswanso has quit IRC02:12
*** tiswanso has joined #openstack-neutron02:13
*** shwetaap has joined #openstack-neutron02:17
*** yamamoto has quit IRC02:18
*** jamielennox is now known as jamielennox|away02:20
*** tiswanso has quit IRC02:24
*** tiswanso has joined #openstack-neutron02:25
*** sthillma has joined #openstack-neutron02:30
*** jamielennox|away is now known as jamielennox02:32
*** britthouser has quit IRC02:34
*** echirivella has quit IRC02:38
openstackgerritRyan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting DVR  https://review.openstack.org/21565502:41
*** dsanders has quit IRC02:42
*** nplanel has joined #openstack-neutron02:45
*** shivrao has joined #openstack-neutron02:47
*** bharathm has joined #openstack-neutron02:47
*** salv-orlando has joined #openstack-neutron02:48
*** shivrao has quit IRC02:51
*** nplanel has quit IRC02:52
*** bharathm has quit IRC02:52
*** hakimo has joined #openstack-neutron02:52
*** hakimo_ has quit IRC02:55
*** zhhuabj has joined #openstack-neutron02:56
*** rushil has quit IRC02:57
*** jbell8 has joined #openstack-neutron02:58
*** gberginc has joined #openstack-neutron03:00
*** salv-orlando has quit IRC03:01
*** dsanders has joined #openstack-neutron03:03
*** yamamoto has joined #openstack-neutron03:04
*** Aish has joined #openstack-neutron03:06
*** y_sawai_ has quit IRC03:06
AishHi, 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
AishRemoving these two imports, my tests run fine..03:09
*** tflynn has joined #openstack-neutron03:11
AishAny idea why this haapens?03:11
*** tiswanso has quit IRC03:11
*** nplanel has joined #openstack-neutron03:14
*** Aish has left #openstack-neutron03:23
*** outworlder has quit IRC03:24
*** dsanders has quit IRC03:26
*** VikasC has joined #openstack-neutron03:28
*** vivek-ebay has joined #openstack-neutron03:29
*** julim has quit IRC03:30
*** openstackgerrit has quit IRC03:31
*** openstackgerrit has joined #openstack-neutron03:32
*** iwamoto has quit IRC03:36
*** saksham has joined #openstack-neutron03:37
*** sdake has quit IRC03:38
*** stanzgy__ has joined #openstack-neutron03:45
*** stanzgy has quit IRC03:47
*** lykinsbd has joined #openstack-neutron03:49
*** jingbo has quit IRC03:55
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources  https://review.openstack.org/21358603:59
*** hanzhang has joined #openstack-neutron04:03
hanzhangping ajo04:03
*** mkolesni_ has joined #openstack-neutron04:06
*** coolsvap|away is now known as coolsvap04:06
openstackgerritMerged openstack/neutron: Add support for unaddressed port  https://review.openstack.org/20080904:09
*** vivek-ebay has quit IRC04:09
*** stanzgy__ has quit IRC04:11
*** gberginc has quit IRC04:13
*** saksham_ has joined #openstack-neutron04:15
*** miyagishi_t has joined #openstack-neutron04:16
*** regXboi has quit IRC04:17
*** vivek-ebay has joined #openstack-neutron04:17
*** saksham has quit IRC04:18
*** saksham_ is now known as saksham04:18
*** shivrao has joined #openstack-neutron04:19
*** dsanders has joined #openstack-neutron04:23
*** salv-orlando has joined #openstack-neutron04:24
*** BhavyaM has joined #openstack-neutron04:24
*** mohankumar has joined #openstack-neutron04:28
*** stanzgy has joined #openstack-neutron04:28
*** ircuser-1 has joined #openstack-neutron04:32
*** salv-orlando has quit IRC04:34
*** mohankumar has quit IRC04:34
*** hoangcx has joined #openstack-neutron04:36
*** bharathm has joined #openstack-neutron04:36
*** hoangcx has quit IRC04:37
*** hoangcx has joined #openstack-neutron04:38
*** ajmiller has quit IRC04:38
*** fawadkhaliq has joined #openstack-neutron04:38
*** salv-orlando has joined #openstack-neutron04:39
*** Ramanjaneya has joined #openstack-neutron04:40
*** hoangcx has quit IRC04:40
*** bharathm has quit IRC04:40
*** changzhi has quit IRC04:41
*** hoangcx has joined #openstack-neutron04:41
*** hoangcx has quit IRC04:42
*** Aish has joined #openstack-neutron04:42
*** hoangcx has joined #openstack-neutron04:42
*** hoangcx has quit IRC04:43
*** nplanel_ has joined #openstack-neutron04:43
openstackgerritGal Sagie proposed openstack/neutron-specs: Create resource with fixed id spec  https://review.openstack.org/21507904:44
*** salv-orlando has quit IRC04:45
*** BhavyaM has quit IRC04:49
openstackgerritHirofumi Ichihara proposed openstack/neutron: Enable to update external network subnet's gateway-ip  https://review.openstack.org/19493904:50
*** fedexo has quit IRC04:57
*** tflynn has quit IRC04:57
*** jckasper has quit IRC04:58
*** tflynn has joined #openstack-neutron04:58
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB  https://review.openstack.org/18336904:59
*** rossella_s has quit IRC05:06
*** rossella_s has joined #openstack-neutron05:06
*** iwamoto has joined #openstack-neutron05:08
*** Aish has left #openstack-neutron05:09
*** kiran-r has joined #openstack-neutron05:10
*** salv-orlando has joined #openstack-neutron05:10
*** shaohe_feng has joined #openstack-neutron05:13
*** shausy has joined #openstack-neutron05:17
*** mohankumar has joined #openstack-neutron05:19
*** numan has joined #openstack-neutron05:21
*** anilvenkata has joined #openstack-neutron05:23
*** vivek-ebay has quit IRC05:24
*** shaohe_feng has quit IRC05:24
*** shaohe_feng has joined #openstack-neutron05:25
*** lazy_prince has joined #openstack-neutron05:25
*** dsanders has quit IRC05:26
*** numan has quit IRC05:26
*** lajos-katona has joined #openstack-neutron05:27
*** gberginc has joined #openstack-neutron05:28
*** tflynn_ has joined #openstack-neutron05:29
*** gardenshed has joined #openstack-neutron05:31
*** tflynn has quit IRC05:32
*** changzhi has joined #openstack-neutron05:33
*** srilatha has joined #openstack-neutron05:35
*** numan has joined #openstack-neutron05:38
*** mohankumar has quit IRC05:38
*** gardenshed has quit IRC05:39
*** salv-orlando has quit IRC05:45
*** jamielennox is now known as jamielennox|away05:47
*** yonglihe has quit IRC05:48
*** bharathm has joined #openstack-neutron05:50
*** shwetaap has quit IRC05:51
*** SridharG has joined #openstack-neutron05:52
*** mickeys has joined #openstack-neutron05:52
*** shivrao has quit IRC05:54
*** yonglihe has joined #openstack-neutron05:54
*** srilatha has quit IRC05:54
*** srilatha has joined #openstack-neutron05:55
*** bharath has joined #openstack-neutron05:56
*** yfried|afk has quit IRC05:56
*** shaohe_feng1 has joined #openstack-neutron05:58
*** srilatha has quit IRC05:59
*** shaohe_feng has quit IRC06:00
*** shausy has quit IRC06:04
*** Murali has joined #openstack-neutron06:04
*** neeti has joined #openstack-neutron06:05
*** abregman has joined #openstack-neutron06:05
bharathL3Natagent  self.router_info is not getting populated. Which is causing FW breakage. Can someone help me out06:08
*** shausy has joined #openstack-neutron06:08
*** mickeys has quit IRC06:11
*** shaohe_feng has joined #openstack-neutron06:12
*** irenab has quit IRC06:13
*** shaohe_feng1 has quit IRC06:13
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler  https://review.openstack.org/20443606:15
*** ameybhide has joined #openstack-neutron06:17
*** ddepaoli has joined #openstack-neutron06:18
*** jbell8 has quit IRC06:20
*** saksham has quit IRC06:20
*** diga has joined #openstack-neutron06:22
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler  https://review.openstack.org/20443606:23
*** shaohe_feng1 has joined #openstack-neutron06:23
*** _kiran_ has joined #openstack-neutron06:25
*** shausy has quit IRC06:25
*** shaohe_feng has quit IRC06:25
*** shausy has joined #openstack-neutron06:26
openstackgerritLi Ma proposed openstack/neutron: Validate local_ip for linuxbridge-agent  https://review.openstack.org/21144306:27
*** kiran-r has quit IRC06:27
*** trinaths has joined #openstack-neutron06:28
*** slaweq has joined #openstack-neutron06:28
*** itzikb has joined #openstack-neutron06:30
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources  https://review.openstack.org/21358606:31
*** armax has quit IRC06:31
*** killer_prince has joined #openstack-neutron06:34
*** trinaths has left #openstack-neutron06:35
*** neeti has quit IRC06:36
*** lazy_prince has quit IRC06:37
*** ildikov has quit IRC06:38
*** yalie has joined #openstack-neutron06:44
*** anshul has joined #openstack-neutron06:44
*** ihrachyshka has joined #openstack-neutron06:46
openstackgerritEric Brown proposed openstack/neutron: Use min and max on IntOpt option types  https://review.openstack.org/21611906:47
*** achanda has quit IRC06:48
*** irenab has joined #openstack-neutron06:48
openstackgerritLi Ma proposed openstack/neutron-vpnaas: Add test_cisco_ipsec.py module for py34 support  https://review.openstack.org/21551506:49
openstackgerritLi Ma proposed openstack/neutron-vpnaas: Add cisco_csr_rest_client.py module for py34 support  https://review.openstack.org/21552006:50
*** moshele has joined #openstack-neutron06:50
*** abregman_ has joined #openstack-neutron06:52
*** abregman has quit IRC06:55
*** nmagnezi has joined #openstack-neutron06:56
*** bharathm has quit IRC06:57
*** itzikb has quit IRC06:58
*** itzikb_ has joined #openstack-neutron06:58
*** lukasa has joined #openstack-neutron06:59
*** jpena has joined #openstack-neutron07:03
*** evgenyf has joined #openstack-neutron07:05
*** haigang has joined #openstack-neutron07:05
*** tflynn_ has quit IRC07:06
*** haigang has quit IRC07:06
*** changzhi has quit IRC07:06
*** I has joined #openstack-neutron07:06
*** I is now known as Guest6305707:07
*** banix has quit IRC07:07
*** Guest63057 has quit IRC07:07
*** haigang has joined #openstack-neutron07:07
*** y_sawai has joined #openstack-neutron07:07
*** devvesa has joined #openstack-neutron07:07
openstackgerritChristian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide  https://review.openstack.org/21612507:08
*** itzikb has joined #openstack-neutron07:08
*** dannywil_ has quit IRC07:08
*** RA_ has quit IRC07:09
*** itzikb_ has quit IRC07:09
*** neeti has joined #openstack-neutron07:09
*** haigang has quit IRC07:09
*** rubasov has joined #openstack-neutron07:10
*** haigang has joined #openstack-neutron07:10
*** ronis has joined #openstack-neutron07:11
*** ronis_ has joined #openstack-neutron07:12
openstackgerritChristian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide  https://review.openstack.org/21612507:14
*** ronis has quit IRC07:15
*** ildikov has joined #openstack-neutron07:17
*** _kiran_ has quit IRC07:17
*** slaweq has quit IRC07:18
*** slaweq has joined #openstack-neutron07:18
*** hichihara has quit IRC07:20
*** saksham has joined #openstack-neutron07:20
*** luqas has joined #openstack-neutron07:20
bharathfacing below error in neutron.07:20
bharathInvalidRequestError: 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-neutron07:20
bharathcan someone help me out07:21
*** apuimedo has joined #openstack-neutron07:21
*** john-davidge has joined #openstack-neutron07:21
apuimedodiga: Both tfukushima and I posted some reviews on your patch07:23
*** gardenshed has joined #openstack-neutron07:24
*** mohankumar has joined #openstack-neutron07:25
*** yamamoto has quit IRC07:26
*** drankis has joined #openstack-neutron07:26
*** ushkalim has joined #openstack-neutron07:26
*** VikasC_ has joined #openstack-neutron07:27
*** y_sawai has quit IRC07:28
*** y_sawai has joined #openstack-neutron07:28
*** gardenshed has quit IRC07:28
*** yonglihe has quit IRC07:29
*** VikasC has quit IRC07:30
*** gowri has joined #openstack-neutron07:32
*** sudipto has joined #openstack-neutron07:33
*** neeti has quit IRC07:33
*** sgordon has quit IRC07:33
*** sgordon has joined #openstack-neutron07:34
*** jlibosva has joined #openstack-neutron07:34
*** jschwarz has joined #openstack-neutron07:35
*** mohankumar has quit IRC07:35
matttihrachyshka: hey, got a sec?07:36
ihrachyshkamattt, yeap.07:36
*** yfried|afk has joined #openstack-neutron07:37
*** jlibosva1 has joined #openstack-neutron07:37
matttihrachyshka: 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
ihrachyshkamattt, hm, I think I am dragged into a meeting, so can be unresponsive next 30 mins07:38
matttihrachyshka: no worries, catch you later07:38
*** neeti has joined #openstack-neutron07:38
*** gongysh has joined #openstack-neutron07:38
*** fzdarsky has joined #openstack-neutron07:39
ihrachyshkamattt, 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-neutron07:40
matttihrachyshka: right now using automation i cannot determine if i have to run online/offline migrations07:41
matttihrachyshka: so i always have to shut down neutron to presumably run offline migrations07:41
*** nplanel has quit IRC07:41
*** Zhang has joined #openstack-neutron07:41
matttihrachyshka: if that spat out the branch i could determine if either are not on head and then upgrade accordingly07:41
ihrachyshkamattt, 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 contract07:41
*** yfried|afk has quit IRC07:41
ihrachyshkamattt, why don't you just unconditionally run upgrade expand@head, then shutdown and proceed with upgrade contract@head?07:42
ihrachyshkaif there is nothing to run, the first step will be no-op07:42
matttihrachyshka: well what i want to avoid doing is shutting down neutron-server if i don't have to07:42
matttihrachyshka: 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 migrations07:43
ihrachyshkaroger. so what you really is interested is to determine whether contract db head is the alembic chain contract head07:43
matttihrachyshka: exactly, that would be super helpful for us07:44
matttihrachyshka: 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 great07:44
*** salv-orlando has joined #openstack-neutron07:44
*** jlibosva has quit IRC07:44
miyagishi_tfujitsu07:45
miyagishi_ti made a mistake.07:47
*** alex_klimov has joined #openstack-neutron07:47
*** nyechiel has joined #openstack-neutron07:47
*** yamamoto has joined #openstack-neutron07:49
*** salv-orlando has quit IRC07:50
*** rotbeard has quit IRC07:50
*** doude has joined #openstack-neutron07:50
*** doude has quit IRC07:50
*** doude has joined #openstack-neutron07:50
*** rotbeard has joined #openstack-neutron07:51
*** numan has quit IRC07:52
digaapuimedo: I saw your comments07:52
digaapuimedo: just 1 question07:52
*** shihanzhang has joined #openstack-neutron07:52
*** fawadkhaliq has quit IRC07:53
*** alex_klimov has quit IRC07:53
digacan I use oslo_concurrency processutils instead of subprocess07:53
*** jlibosva1 has quit IRC07:54
*** hanzhang has quit IRC07:55
*** nmagnezi has quit IRC07:56
*** alex_klimov has joined #openstack-neutron08:00
*** lajos-katona1 has joined #openstack-neutron08:01
*** lajos-katona has quit IRC08:02
*** matrohon has joined #openstack-neutron08:04
*** haigang has quit IRC08:06
*** y_sawai has quit IRC08:06
*** fawadkhaliq has joined #openstack-neutron08:07
*** l6unchpad has joined #openstack-neutron08:08
*** changzhi has joined #openstack-neutron08:09
*** scheuran has joined #openstack-neutron08:09
gongyshamotoki: ping08:10
*** abregman_ is now known as abregman08:10
openstackgerritJohn Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2).  https://review.openstack.org/18965508:10
gongyshamotoki: https://review.openstack.org/#/c/214981/08:10
*** nmagnezi has joined #openstack-neutron08:11
gongyshamotoki: I have replied to your comments, what do u think of it?08:11
*** jpena is now known as jpena|away08:11
*** Zhang has quit IRC08:12
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow implementation  https://review.openstack.org/15394608:12
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Switch the default to "native" of_interface  https://review.openstack.org/20632908:14
*** scheuran has quit IRC08:15
*** jlibosva has joined #openstack-neutron08:15
*** jistr has joined #openstack-neutron08:17
ihrachyshkamattt, 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-neutron08:20
*** yanxb__ has joined #openstack-neutron08:22
*** yanxb has quit IRC08:22
*** yanxb_ has quit IRC08:22
*** luqas has quit IRC08:22
openstackgerrityalei wang proposed openstack/neutron: Add verification for device_owner when missing the port_security attr  https://review.openstack.org/21614308:23
*** yanxb_ has joined #openstack-neutron08:23
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_ovs_lib: Update a comment  https://review.openstack.org/21614408:23
matttihrachyshka: 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 effort08:24
*** ygbo has joined #openstack-neutron08:24
*** julien_ZTE has joined #openstack-neutron08:25
ihrachyshkamattt, 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 IRC08:26
*** zhhuabj has joined #openstack-neutron08:26
*** shausy has quit IRC08:26
matttihrachyshka: RFE?  what is that ...08:26
*** shausy has joined #openstack-neutron08:26
ihrachyshkamattt, request for enhancement: http://docs.openstack.org/developer/neutron/policies/blueprints.html#neutron-request-for-feature-enhancements08:27
matttihrachyshka: ok certainly, let me draft up something today08:28
*** zhhuabj has quit IRC08:28
*** zhhuabj has joined #openstack-neutron08:28
ihrachyshkamattt, thanks.08:28
openstackgerrityalei wang proposed openstack/neutron: Add verification for device_owner when missing the port_security attr  https://review.openstack.org/21614308:29
*** zhhuabj has quit IRC08:29
*** zhhuabj has joined #openstack-neutron08:30
openstackgerritOleg Bondarev proposed openstack/neutron: Move db agent schedulers test to a more appropriate place  https://review.openstack.org/21413108:30
*** zhhuabj has quit IRC08:30
*** zhhuabj has joined #openstack-neutron08:30
*** zhhuabj has quit IRC08:31
*** zhhuabj has joined #openstack-neutron08:31
*** yfried is now known as yfried|afk08:32
*** sthillma has quit IRC08:34
*** zhhuabj has quit IRC08:34
*** zhhuabj has joined #openstack-neutron08:34
*** yfried|afk is now known as yfried08:35
*** sthillma has joined #openstack-neutron08:36
*** yog_ has joined #openstack-neutron08:36
apuimedodiga: what's the question?08:38
digaapuimedo: can I use oslo_concurrency processutils instead of subprocess08:39
*** zhhuabj has quit IRC08:39
*** zhhuabj has joined #openstack-neutron08:39
*** zhhuabj has quit IRC08:41
*** VikasC has joined #openstack-neutron08:41
*** zhhuabj has joined #openstack-neutron08:41
apuimedodiga: it's okay for me. gsagie: irenab: yamamoto: what do you think?08:42
*** SourabhP has joined #openstack-neutron08:42
irenabapuimedo: diga :+108:42
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_ovs_neutron_agent: Fix test_cleanup_stale_flows_iter_0  https://review.openstack.org/21615208:42
*** VikasC_ has quit IRC08:42
yamamotoapuimedo: diga: +108:43
*** jistr has quit IRC08:45
digathanks apuimedo, irenab08:45
digayamamoto: thanks08:45
digawill use that module08:46
*** neophy has joined #openstack-neutron08:46
yamamotoakamyshnikova: ping08:47
*** rotbeard has quit IRC08:47
irenabdiga: apuimedo : I think we will use oslo.config for kuryr configuration08:47
*** jistr has joined #openstack-neutron08:47
*** slaweq has quit IRC08:48
apuimedoirenab: great!08:48
apuimedoThe more oslo the better08:48
*** shadower49 is now known as shadower08:48
* apuimedo likes small codebases08:48
*** SourabhP_ has joined #openstack-neutron08:48
digairenab: +108:48
irenabapuimedo: :-)08:49
*** Zhang has joined #openstack-neutron08:49
*** jpena|away is now known as jpena08:49
*** Zhang has quit IRC08:51
ihrachyshkaajo, what blocks us from merging https://review.openstack.org/#/c/214249/ ?08:51
*** SourabhP has quit IRC08:51
*** SourabhP_ is now known as SourabhP08:51
*** fawadkhaliq has quit IRC08:52
*** iwamoto has quit IRC08:53
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: [DevStack] Fix StrongSwan setup on Fedora  https://review.openstack.org/21571008:53
*** Marga_ has joined #openstack-neutron08:53
*** Marga_ has quit IRC08:53
*** Marga_ has joined #openstack-neutron08:54
ihrachyshkaajo, btw it's cool that everyone is deep into dscp discussion but I think we have at least upgrade story to solve first08:54
*** ivar-lazzaro has joined #openstack-neutron08:56
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442408:57
*** changzhi has quit IRC08:57
*** sthillma has quit IRC09:02
*** julien_ZTE has quit IRC09:06
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442409:07
*** ivar-lazzaro has quit IRC09:07
*** ivar-lazzaro has joined #openstack-neutron09:07
*** y_sawai has joined #openstack-neutron09:07
*** jistr has quit IRC09:08
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports  https://review.openstack.org/21030109:08
ajoihrachyshka, we won't need that until we switch any object version :)09:09
ajoihrachyshka, but I have solved it on my mind :D I can share the idea if you want,09:09
ajoihrachyshka, no manual config, intervention or rolling stuff, all automated and09:09
ajo*VERY SIMPLE* :-)09:09
ihrachyshkaajo, once you add a new rule type, you change the version09:09
ihrachyshkabecause it will get into policy[rules]09:09
ajohm09:10
ajotrue :)09:10
ihrachyshkaso I suggest you to come up with the documented plan09:10
ihrachyshkadevref change maybe09:10
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports  https://review.openstack.org/21030109:10
ajoihrachyshka, that sounds good09:11
*** l6unchpad has quit IRC09:11
*** ivar-lazzaro has quit IRC09:12
*** y_sawai has quit IRC09:12
*** Marga__ has joined #openstack-neutron09:12
*** Marga_ has quit IRC09:12
ihrachyshkaakamyshnikova, congrats on getting graceful ovs restart in!09:14
ihrachyshkaajo, thanks09:14
*** l6unchpad has joined #openstack-neutron09:17
*** abregman is now known as abregman|lunch09:17
openstackgerritgongysh proposed openstack/neutron: Add empty policy rule to get_rule_type action  https://review.openstack.org/21548409:17
*** aix has joined #openstack-neutron09:20
ajoakamyshnikova++09:21
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports  https://review.openstack.org/21030409:23
*** claudiub has joined #openstack-neutron09:23
*** moshele has quit IRC09:26
*** numan has quit IRC09:27
*** nplanel_ has quit IRC09:28
*** y_sawai has joined #openstack-neutron09:31
*** SourabhP has quit IRC09:33
*** fawadkhaliq has joined #openstack-neutron09:33
*** mohankumar has joined #openstack-neutron09:34
*** lajos-katona1 has quit IRC09:34
*** BobBall has joined #openstack-neutron09:35
matttihrachyshka: created https://bugs.launchpad.net/neutron/+bug/1488021, hopefully i captured the request correctly :)09:36
openstackLaunchpad bug 1488021 in neutron "Display alembic branch in neutron-db-manage current" [Undecided,New]09:36
*** l6unchpad has quit IRC09:37
*** l6unchpad has joined #openstack-neutron09:38
*** moshele has joined #openstack-neutron09:39
ihrachyshkamattt, thanks a lot! we'll take it from there.09:40
raornhi, everyone!09:43
openstackgerritgongysh proposed openstack/neutron: Add empty policy rule to get_rule_type action  https://review.openstack.org/21548409:43
matttihrachyshka: thanks a lot for your help09:44
*** jianyong has quit IRC09:45
*** jianyong has joined #openstack-neutron09:45
*** mohankumar has quit IRC09:46
*** yalie has quit IRC09:47
*** jianyong has quit IRC09:48
*** numan has joined #openstack-neutron09:49
*** luqas has joined #openstack-neutron09:50
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21508609:50
*** mohankumar has joined #openstack-neutron09:51
*** lajos-katona has joined #openstack-neutron09:52
*** yanxb_ has quit IRC09:53
openstackgerritFawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II  https://review.openstack.org/21119609:54
*** yanxb_ has joined #openstack-neutron09:54
*** Marga__ has quit IRC09:56
*** yamamoto has quit IRC09:57
*** yanxb_ has quit IRC09:58
*** Marga_ has joined #openstack-neutron09:59
*** lori|away is now known as lori10:02
ajodougwig, amotoki , kevinbenton: ready for +A ;) https://review.openstack.org/#/c/214249/10:03
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Make NeutronDbObjectDuplicateEntry exception more verbose  https://review.openstack.org/21460810:03
ajoihrachyshka, kevinbenton, obondarev_  ^10:03
ajofixed nits and unittest10:03
ajoI have removed the inner_exception and object_class storage in self, as it was a nit... if a caller needs it, can be added later10:03
amotokiajo: looking10:04
openstackgerritMerged openstack/neutron: Graceful OVS restart for DVR  https://review.openstack.org/21559610:04
openstackgerritFawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II  https://review.openstack.org/21119610:05
amotokiajo: one question. q-qos is always enabled when this devstack plugin is enabled. is it intended?10:07
ajoamotoki, yes, that's the intention10:07
amotokiajo:  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
ajoamotoki, so all qos related API tests will be passed on every commit10:07
*** liangbo has joined #openstack-neutron10:08
ajoihrachyshka ^10:08
ajoamotoki, I guess, that we could use that plugin for any in-tree services for neutron10:08
ajoand generally we'd like to enable those by default for testing, I guess10:08
ihrachyshkaajo, we should expand it yes. amotoki, yes, we'll need to move the enablement of the plugin to localrc10:09
ihrachyshkabut for now it's good enough to avoid infra changes10:09
*** abregman|lunch is now known as abregman10:09
*** Kennan has quit IRC10:09
amotokiajo: for testing, it is good. when we run devstack in local env, we would like to enable/disable services10:09
*** l6unchpad has quit IRC10:09
*** Kennan has joined #openstack-neutron10:10
ajoamotoki: ah, ok, I get what you mean10:10
amotokiajo: but ihrachyshka's point is also important.10:10
ajoyes, may be we can go with infra changes early M10:11
ajoI think that's the plan10:11
ihrachyshkaajo, I plan to work on enabling it for tempest and overall utilizing gate_hook there10:11
ihrachyshkain start of M10:11
ajoihrachyshka++10:11
amotokiajo: we have disable_service, so it may be okay. I am okay with the current version and agree with the plan.10:11
amotokiajo: ihrachyshka: thanks10:12
ihrachyshkaI don't want to mess with infra these days :) and it's feature freeze approaching, not the best time to break stuff10:12
amotokiihrachyshka: totally agree10:12
ajoihrachyshka, 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
ajoright?10:13
ajoand will be in fact enabling q-qos10:14
openstackgerritRossella Sblendido proposed openstack/neutron: OVS agent don't hard code tunnel bridge name  https://review.openstack.org/21559010:14
amotokiajo: right10:14
ajoI guess I will add that to the qos devref10:14
amotokiajo: ihrachyshka: /fyi/: https://review.openstack.org/#/c/213253/ is introducing a new serivce name "neutron-vpnaas"10:14
amotokiI am not sure which is better: q-qos, neutron-qos and either looks good.10:15
*** l6unchpad has joined #openstack-neutron10:15
*** aix has quit IRC10:15
*** rotbeard has joined #openstack-neutron10:15
ajoah, finally we're going to shake off quantum ;)10:16
*** Fdaisuke has quit IRC10:17
openstackgerritNguyen Phuong An proposed openstack/neutron-fwaas: [WIP] Support logging configuration feature for FWaaS  https://review.openstack.org/20448410:18
ihrachyshkaamotoki, oh. maybe neutron-qos would be better, yes.10:19
ihrachyshkabut we will need to solve rename for other services anyway. not sure.10:19
openstackgerritKevin Benton proposed openstack/neutron: Make models_v2 explicitly import rbac_db_models  https://review.openstack.org/21617710:20
*** sudipto has quit IRC10:21
*** ccard has joined #openstack-neutron10:22
*** bharath has quit IRC10:22
openstackgerritNguyen Phuong An proposed openstack/neutron: [WIP] Logging API for security group and firewall rule  https://review.openstack.org/20448110:22
*** diga has quit IRC10:23
*** gowri1 has joined #openstack-neutron10:25
ajoamotoki10:25
ajocan I ask you a question about RPC and AgentNotifiers in ML2?10:26
ajohttps://review.openstack.org/#/c/215608/1/neutron/plugins/ml2/rpc.py10:26
ajohere I see we have bumps on the comment versions (I guess because of mixin classes..)10:26
ajobut on "target" which remains in 1.010:26
*** sdake has joined #openstack-neutron10:26
ajois correct what I'm doing there, or shall I bump target?10:26
*** gowri has quit IRC10:26
amotokiajo: looking10:27
ajoif 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
ajomay be that's the reason why we don't bump ?10:27
ajoor I'm just missing something10:27
ajothanks amotoki  :)10:27
amotokiajo: it is a client side of RPC, so we need to keep the version of Target.10:27
ajoah10:28
openstackgerritNeil Jerram proposed openstack/neutron: DHCP agent: allow using gateway IPs instead of uniquely allocated  https://review.openstack.org/20607810:28
amotokiajo: otherwise, methods added in the initial version will have a newer version.10:28
amotokiajo: in this case, we need to specify version=1.2 in network_update explicitly. looking for an example10:28
ajoahhh amotoki , I was about to ask that10:29
ajoif it made sense to put an explicit version per call10:29
amotokiajo: http://docs.openstack.org/developer/neutron/devref/rpc_api.html10:29
ajoamotoki, I think also port_delete (or network_delete) was also added during this cycle without explicit version in the call10:29
amotokiajo: I think so too10:30
ajoamotoki: 1 sec, let me track down when did that happen, so we can add the version to the other one too10:30
*** Marga_ has quit IRC10:31
amotokiajo: "the other one" is "port_delete"?10:31
ajokevinbenton: https://github.com/openstack/neutron/commit/f87a74bfa83eeb859dfd047719622b54cdb5f68b10:31
ajothis was pre-L right?10:31
*** ekarlso has quit IRC10:32
*** ekarlso has joined #openstack-neutron10:32
ajomay be we're safe :)10:32
ajoyeah, port_delete was kilo: https://github.com/openstack/neutron/blob/stable/kilo/neutron/plugins/ml2/rpc.py#L22410:33
*** yfried is now known as yfried|afk10:35
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: [OpenSwan] Enable usage of the MTU value of an IPSec connection  https://review.openstack.org/20653910:36
*** bharath has joined #openstack-neutron10:38
amotokideleting a method in a server side (= callback side) is not a good way because it blocks live upgrade....10:38
*** sudipto has joined #openstack-neutron10:39
*** l6unchpad has quit IRC10:41
*** l6unchpad has joined #openstack-neutron10:43
ajoamotoki: last question10:45
ajoas per OVS, it seems to me: https://review.openstack.org/#/c/215608/1/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py10:45
ajoI may need to bump to 1.4 version (now it's 1.3)10:45
ajoand target that from the cttxt prepare on Api side10:46
amotokiajo: right. we need to bump the version to 1.4.10:46
ajothanks for confirmation10:46
amotokiajo: it is tricky. I just remmeber I thought we need to break down agent side callback into callbacks per feature.10:47
ajoamotoki, yes, it seems like that's what we pay for having all endpoints in "self"10:47
*** yamamoto has joined #openstack-neutron10:47
*** SridharG has left #openstack-neutron10:48
ajoamotoki, I can tackle that for M10:48
*** aix has joined #openstack-neutron10:48
ajodo you remember if we have a bug for it? otherwise I will fill one10:48
*** dguitarbite has quit IRC10:49
*** stanzgy has quit IRC10:49
*** yamamoto has quit IRC10:50
*** yamamoto has joined #openstack-neutron10:50
*** jamielennox|away has quit IRC10:50
*** yamamoto_ has joined #openstack-neutron10:50
amotokiajo: I think this is the counterpart of https://blueprints.launchpad.net/neutron/+spec/rpc-docs-and-namespaces10:51
amotokiand https://bugs.launchpad.net/neutron/+bug/1112634 is related.10:51
openstackLaunchpad 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 IRC10:52
*** jistr has joined #openstack-neutron10:52
amotokiI haven't checked how was the abandoned patch.10:52
amotokiit is just about l2-pop. We can take the bug.10:53
*** mwagner_lap has quit IRC10:53
*** jianyong has joined #openstack-neutron10:54
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing trunk ports  https://review.openstack.org/21030110:54
ajoamotoki, every time I try to write "amotoki" in my browser it tries to change it to "atomic" :)10:54
*** gongysh has quit IRC10:54
*** yamamoto has quit IRC10:54
amotokiajo: :-)10:55
*** adam_g has quit IRC10:55
*** rackertom has quit IRC10:55
*** lbragstad has quit IRC10:55
*** yfried|afk is now known as yfried10:55
*** ctracey has quit IRC10:55
*** wendar_ has joined #openstack-neutron10:56
*** zacksh has quit IRC10:56
*** wendar has quit IRC10:56
*** jroll has quit IRC10:56
openstackgerritMerged openstack/neutron: manual add/remove router for dvr_snat agent  https://review.openstack.org/12202410:57
*** rex_lee has quit IRC10:57
*** zacksh has joined #openstack-neutron10:58
*** ctracey has joined #openstack-neutron10:58
*** xgerman has quit IRC10:58
*** pc_m has joined #openstack-neutron10:59
*** rackertom has joined #openstack-neutron10:59
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports  https://review.openstack.org/21030411:00
*** xgerman has joined #openstack-neutron11:00
openstackgerritLajos Katona proposed openstack/python-neutronclient: Support CLI changes for managing ports as subports  https://review.openstack.org/21030411:00
*** evgenyf has quit IRC11:01
*** adam_g has joined #openstack-neutron11:01
*** adam_g has quit IRC11:01
*** adam_g has joined #openstack-neutron11:01
*** lbragstad has joined #openstack-neutron11:02
*** jroll has joined #openstack-neutron11:02
*** amotoki has quit IRC11:02
*** yamamoto_ has quit IRC11:03
*** jamielennox|away has joined #openstack-neutron11:04
*** rex_lee has joined #openstack-neutron11:04
*** jamielennox|away is now known as jamielennox11:04
*** rubasov has quit IRC11:10
openstackgerritRossella Sblendido proposed openstack/neutron: Add updated_attrs in port_update  https://review.openstack.org/15522311:11
*** sudipto has quit IRC11:14
openstackgerritJakub Libosvar proposed openstack/neutron: [WIP] Add QoS fullstack test  https://review.openstack.org/21568811:15
*** l6unchpad has quit IRC11:18
*** l6unchpad has joined #openstack-neutron11:19
*** yamamoto has joined #openstack-neutron11:21
*** liangbo has joined #openstack-neutron11:22
*** l6unchpad has quit IRC11:23
*** shaohe_feng1 has quit IRC11:23
*** lindycoder has joined #openstack-neutron11:29
*** ferest has joined #openstack-neutron11:29
*** rubasov has joined #openstack-neutron11:30
*** evgenyf has joined #openstack-neutron11:31
*** salv-orlando has joined #openstack-neutron11:32
*** ferest has quit IRC11:35
*** anilvenkata has left #openstack-neutron11:36
*** thorst has joined #openstack-neutron11:36
*** nmagnezi has quit IRC11:37
*** l6unchpad has joined #openstack-neutron11:37
openstackgerritMerged openstack/neutron: Added initial devstack plugin  https://review.openstack.org/21424911:37
openstackgerritMerged openstack/neutron: Move tests for non pluggable ipam backend  https://review.openstack.org/21150711:38
*** BobBall_ has joined #openstack-neutron11:39
*** BobBall_ is now known as Guest2482411:40
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants  https://review.openstack.org/21578211:40
*** BobBall has quit IRC11:43
*** Marga_ has joined #openstack-neutron11:43
*** sudipto has joined #openstack-neutron11:47
ihrachyshkaajo, ^^ ok, now qos api tests are really enabled in gate11:47
jschwarzihrachyshka, really?11:53
jschwarzXD11:53
ihrachyshkajschwarz, well, not tempest still11:55
jschwarzihrachyshka, but qos api11:55
jschwarzXD11:55
*** salv-orlando has quit IRC11:55
*** yanxb_ has joined #openstack-neutron12:00
*** dguitarbite has joined #openstack-neutron12:01
*** Guest24824 has quit IRC12:01
*** pradk has quit IRC12:01
*** BobBall has joined #openstack-neutron12:01
*** dguitarbite has quit IRC12:01
*** dguitarbite has joined #openstack-neutron12:01
*** BobBall is now known as Guest3725212:01
*** numan has quit IRC12:02
*** jpena is now known as jpena|lunch12:04
*** Guest37252 has quit IRC12:06
*** moha_hunt has joined #openstack-neutron12:07
*** shoutm has joined #openstack-neutron12:07
ajoihrachyshka++12:08
ajoyeheheyyy12:08
*** gampel1 has joined #openstack-neutron12:09
*** markvoelker_ has quit IRC12:10
*** BobBall__ has joined #openstack-neutron12:12
*** numan has joined #openstack-neutron12:13
openstackgerritOleg Bondarev proposed openstack/neutron: Move db agent schedulers test to a more appropriate place  https://review.openstack.org/21413112:14
*** piyanai has joined #openstack-neutron12:14
*** nyechiel has quit IRC12:14
*** sudipto has quit IRC12:16
*** liangbo has quit IRC12:17
*** gampel1 has left #openstack-neutron12:17
*** markvoelker has joined #openstack-neutron12:17
*** lyanchih has joined #openstack-neutron12:17
*** y_sawai has quit IRC12:17
*** y_sawai has joined #openstack-neutron12:19
*** y_sawai has quit IRC12:19
*** markvoelker has quit IRC12:19
*** markvoelker has joined #openstack-neutron12:20
*** piyanai has quit IRC12:20
*** markvoelker has quit IRC12:20
*** diegows has joined #openstack-neutron12:20
*** diegows has quit IRC12:21
*** VikasC has quit IRC12:22
*** julim has joined #openstack-neutron12:22
*** markvoelker has joined #openstack-neutron12:25
*** nmagnezi has joined #openstack-neutron12:25
*** numan has quit IRC12:26
*** piyanai has joined #openstack-neutron12:27
*** itzikb has quit IRC12:29
*** itzikb has joined #openstack-neutron12:29
*** edmondsw has joined #openstack-neutron12:31
*** jistr has quit IRC12:31
*** jistr has joined #openstack-neutron12:33
*** nmagnezi has quit IRC12:33
*** yfried is now known as yfried|afk12:34
*** regXboi has joined #openstack-neutron12:35
*** mwagner_lap has joined #openstack-neutron12:37
*** gberginc has quit IRC12:37
*** zengine has joined #openstack-neutron12:38
*** nmagnezi has joined #openstack-neutron12:41
*** numan has joined #openstack-neutron12:41
*** yfried|afk is now known as yfried12:44
*** Marga_ has quit IRC12:46
*** sdake_ has joined #openstack-neutron12:47
openstackgerritshihanzhang proposed openstack/neutron: Update port's status DOWN if its binding info has changed  https://review.openstack.org/21546712:49
*** sdake__ has joined #openstack-neutron12:50
*** sdake has quit IRC12:50
*** sdake_ has quit IRC12:53
*** rotbeard has quit IRC12:55
*** rotbeard has joined #openstack-neutron12:56
*** nyechiel has joined #openstack-neutron12:56
*** tiswanso has joined #openstack-neutron12:57
*** amuller has joined #openstack-neutron12:58
*** sdake__ is now known as sdake12:59
*** tiswanso has quit IRC13:00
*** baoli has joined #openstack-neutron13:00
*** banix has joined #openstack-neutron13:01
*** tiswanso has joined #openstack-neutron13:01
*** bharath has quit IRC13:01
*** jpena|lunch is now known as jpena13:02
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Splitting out models from database class  https://review.openstack.org/21624813:05
*** annegentle has joined #openstack-neutron13:05
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Splitting out models from database class  https://review.openstack.org/21624813:07
*** annegentle has quit IRC13:08
*** vikram has joined #openstack-neutron13:08
*** annegentle has joined #openstack-neutron13:08
openstackgerritenikanorov proposed openstack/neutron: Add periodic agents health check.  https://review.openstack.org/18163313:11
*** britthouser has joined #openstack-neutron13:11
*** dims has joined #openstack-neutron13:12
*** clev-away is now known as clev13:13
*** clev is now known as clev-away13:13
*** mkolesni_ has quit IRC13:14
*** nate_gone is now known as njohnston13:14
*** kbringard has joined #openstack-neutron13:14
*** britthou_ has joined #openstack-neutron13:16
*** miyagishi_t has quit IRC13:16
*** ildikov has quit IRC13:18
*** ijw_ has quit IRC13:18
*** britthouser has quit IRC13:18
*** banix has quit IRC13:19
*** ijw has joined #openstack-neutron13:19
*** ildikov has joined #openstack-neutron13:20
*** Samos123 has quit IRC13:22
*** dsneddon has quit IRC13:22
*** clev-away has quit IRC13:22
*** Marga_ has joined #openstack-neutron13:22
*** jroll has quit IRC13:23
*** dims has quit IRC13:23
*** jamielennox has quit IRC13:26
*** Samos123 has joined #openstack-neutron13:27
*** irenab has quit IRC13:27
*** tiswanso has quit IRC13:27
*** tiswanso has joined #openstack-neutron13:28
*** bapalm_ has joined #openstack-neutron13:28
*** shausy has quit IRC13:29
*** salv-orlando has joined #openstack-neutron13:29
*** signed8bit has joined #openstack-neutron13:29
*** zzzeek has joined #openstack-neutron13:30
*** BobBall__ is now known as BobBall13:30
*** liangbo has joined #openstack-neutron13:30
*** salv-orl_ has joined #openstack-neutron13:31
*** salv-orlando has quit IRC13:31
*** shausy has joined #openstack-neutron13:31
*** links has joined #openstack-neutron13:31
*** signed8bit has quit IRC13:34
*** dsneddon has joined #openstack-neutron13:34
openstackgerritBence Romsics proposed openstack/neutron: Trunk port: DB models and service plugin  https://review.openstack.org/21031013:34
openstackgerritBence Romsics proposed openstack/neutron: Trunk port: API extension  https://review.openstack.org/21030913:34
openstackgerritBence Romsics proposed openstack/neutron: Trunk port: ML2 extension driver for subports  https://review.openstack.org/21031313:34
*** yamamoto has quit IRC13:34
*** prad_ has joined #openstack-neutron13:34
*** prad_ is now known as pradk13:35
*** lyanchih has quit IRC13:35
*** signed8bit has joined #openstack-neutron13:37
*** clev has joined #openstack-neutron13:37
*** jroll has joined #openstack-neutron13:37
openstackgerritFawad Khaliq proposed openstack/neutron: PLUMgrid plugin decomposition part II  https://review.openstack.org/21119613:40
*** piyanai has quit IRC13:41
*** rkukura has joined #openstack-neutron13:41
*** crose has joined #openstack-neutron13:42
*** rkukura has quit IRC13:42
*** shihanzhang has quit IRC13:42
*** jamielennox|away has joined #openstack-neutron13:43
*** shihanzhang has joined #openstack-neutron13:43
*** jamielennox|away is now known as jamielennox13:43
*** neeti has quit IRC13:43
*** shaohe_feng has joined #openstack-neutron13:45
*** bharath has joined #openstack-neutron13:46
*** shwetaap has joined #openstack-neutron13:47
openstackgerritRyan Moats proposed openstack/neutron: Rationalize neutron logs to help in troubleshooting router issues  https://review.openstack.org/21565513:48
*** yamamoto has joined #openstack-neutron13:50
*** rushil has joined #openstack-neutron13:52
*** dims_ has joined #openstack-neutron13:52
*** gowri1 has quit IRC13:53
*** baoli_ has joined #openstack-neutron13:53
*** jckasper has joined #openstack-neutron13:53
openstackgerritRyan Moats proposed openstack/neutron: Remove redundant logging statements from RootWrapDaemonHelper  https://review.openstack.org/21626013:54
*** baoli has quit IRC13:57
openstackgerritLi Ma proposed openstack/neutron: Validate local_ip for linuxbridge-agent  https://review.openstack.org/21144313:59
*** boris-42 has joined #openstack-neutron14:00
*** amotoki has joined #openstack-neutron14:00
jschwarzamuller, do you know where I might find ZZelle?14:00
*** rushil has quit IRC14:01
*** rushil has joined #openstack-neutron14:02
*** lukasa has quit IRC14:04
eezhovapc_m, hello!14:04
*** tiswanso has quit IRC14:05
pc_meezhova: hi14:06
eezhovapc_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-neutron14:07
*** neiljerram has joined #openstack-neutron14:07
*** fawadkhaliq has quit IRC14:07
*** frickler has quit IRC14:07
pc_meezhova: ouch. So an API difference that is conflicting.14:08
eezhovapc_m, in the current situation everything will be working fine as long as we have no more that one connection per vpn service14:08
*** jamielennox has quit IRC14:08
eezhovapc_m, exactly14:08
eezhovapc_m, so I don't really know what should be done in a situation like that14:09
pc_meezhova: I can only think of hacky ways to deal with that situation.14:09
*** jamielennox|away has joined #openstack-neutron14:10
pc_meezhova: 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 jamielennox14:11
*** irenab has joined #openstack-neutron14:11
eezhovapc_m, we can add another option "vpn_service_mtu" but that gonna involve complicated to validation14:12
pc_meezhova: 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-neutron14:13
*** jprovazn has joined #openstack-neutron14:13
pc_meezhova: 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
eezhovapc_m, but isn't it a limitation? I mean some options, other than mtu , can differ14:14
pc_meezhova: 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-neutron14:15
*** zhenguo has quit IRC14:17
*** lyanchih has joined #openstack-neutron14:17
eezhovapc_m, I'll have a look at how they do it. meanwhile your last suggestion sounds like the best plan for now14:17
*** shausy has quit IRC14:18
pc_meezhova: Which one? Adding validators and requiring all connections use same MTU?14:19
*** sdake has quit IRC14:19
*** frickler_ is now known as frickler14:19
gsagiemestery, 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 IRC14:20
eezhovapc_m, ignoring service mtu for OpenSwan and ignoring connection mtu for StrongSwan14:20
russellbgsagie: yeah, double check to see what was actually implemented first though14:20
russellbgsagie: but in general, consistency between APIs ++14:20
kevinbentongsagie: hold off on making changes until we can chat about it in the drivers meeting tomorrow14:20
pc_meezhova: Not sure that is better.14:20
johnthetubaguygsagie: 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.html14:21
gsagiekevinbenton: ok14:21
kevinbentongsagie: just so it's not changing before people get a chance to read the email14:21
*** markvoelker has quit IRC14:21
*** nplanel has quit IRC14:22
pc_meezhova: 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
gsagiejohnthetubaguy: ok, thanks14:22
*** markvoelker has joined #openstack-neutron14:22
pc_meezhova: Requiring that all connections use the same MTU for a service is clearly less effort.14:22
johnthetubaguygsagie: ah, this is the one: http://specs.openstack.org/openstack/api-wg/guidelines/tags.html14:23
eezhovapc_m, I guess you are right. adding validators and requiring all connections use same mtu is also easier14:23
*** agireud has joined #openstack-neutron14:23
eezhovapc_m, I'll try it out and see whether it works out14:24
*** rushil has quit IRC14:24
pc_meezhova: Yeah, should, and should be easy to query the database for that case.14:24
*** rushil has joined #openstack-neutron14:24
*** shoutm has quit IRC14:24
eezhovapc_m, thank you for the advice!14:24
pc_meezhova: sure np.14:24
openstackgerritMiguel Angel Ajo proposed openstack/neutron: [WIP] Process update_network in the openvswitch agent  https://review.openstack.org/21560814:24
gsagiejohnthetubaguy : thanks for the link14:24
*** yog_ has quit IRC14:25
*** sdake has joined #openstack-neutron14:25
*** rushil has quit IRC14:27
*** kbringard has quit IRC14:28
*** sdake_ has quit IRC14:28
*** Murali has quit IRC14:30
*** shoutm has joined #openstack-neutron14:30
*** crose has quit IRC14:31
*** kbringard has joined #openstack-neutron14:32
*** ajmiller has joined #openstack-neutron14:33
*** jckasper has quit IRC14:36
*** yamahata has joined #openstack-neutron14:36
*** alejandrito has joined #openstack-neutron14:37
*** liangbo has quit IRC14:38
*** lajos-katona has quit IRC14:39
*** liangbo has joined #openstack-neutron14:40
*** bapalm_ has quit IRC14:41
*** dmartls1 has joined #openstack-neutron14:42
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron base resources  https://review.openstack.org/21358614:43
*** yamahata has quit IRC14:43
*** irenab has quit IRC14:44
*** irenab has joined #openstack-neutron14:45
*** bapalm- has joined #openstack-neutron14:45
*** bharath has quit IRC14:46
*** zhenguo has joined #openstack-neutron14:46
*** vivek-ebay has joined #openstack-neutron14:48
*** ddepaoli has quit IRC14:52
*** annegentle has quit IRC14:52
*** achanda has joined #openstack-neutron14:52
pc_msalv-orl_: hi14:58
*** shaohe_feng has quit IRC14:59
*** numan has quit IRC14:59
*** shoutm has quit IRC15:00
*** mlavalle has joined #openstack-neutron15:00
ihrachyshkasalv-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
ihrachyshkamaybe a new command for that? so that you can compare 'current' output with e.g. 'heads' to see whether there is anything not applied yet15:01
*** jlibosva has quit IRC15:02
*** numan has joined #openstack-neutron15:02
*** achanda has quit IRC15:02
*** pbrooko has joined #openstack-neutron15:03
*** emagana has joined #openstack-neutron15:04
*** vivek-ebay has quit IRC15:04
*** rushil has joined #openstack-neutron15:05
*** yamamoto has quit IRC15:07
*** diga has joined #openstack-neutron15:08
pc_mdougwig: ping15:08
*** jaypipes has joined #openstack-neutron15:09
*** slaweq has joined #openstack-neutron15:09
*** Aish has joined #openstack-neutron15:10
regXboiamuller: ping?15:10
*** thorst has quit IRC15:11
regXboiamuller: is there *anywhere* in the patch where we log attached ports of a router for troubleshooting purposes?15:11
*** anshul has quit IRC15:11
*** thorst has joined #openstack-neutron15:11
*** lyanchih has quit IRC15:12
*** lyanchih has joined #openstack-neutron15:12
*** itzikb has quit IRC15:15
*** piyanai has joined #openstack-neutron15:15
*** crose has joined #openstack-neutron15:15
kevinbentonhaleyb: ping15:15
*** amotoki has quit IRC15:16
*** abregman has quit IRC15:17
*** ronis_ has quit IRC15:17
*** jbell8 has joined #openstack-neutron15:17
haleybkevinbenton: pong15:18
*** signed8bit has quit IRC15:21
*** srilatha has joined #openstack-neutron15:23
kevinbentonhaleyb: which are the possible interface types in DVR that floating IP traffic can come from?15:23
*** ildikov has quit IRC15:23
kevinbentonhaleyb: device_owner field15:24
openstackgerritChristian Berendt proposed openstack/neutron: Update the URLs to the Cloud Admin Guide  https://review.openstack.org/21612515:24
*** mohankumar has quit IRC15:25
kevinbentonhaleyb: e.g. does floating IP traffic ever come from these? https://github.com/openstack/neutron/blob/master/neutron/common/constants.py#L3815:25
*** zengine has quit IRC15:25
*** annegentle has joined #openstack-neutron15:25
regXboikevinbenton: why are you asking?15:26
*** Ramanjaneya has quit IRC15:26
*** zengine has joined #openstack-neutron15:26
kevinbentonregXboi: these things depend on ARP spoofing being allowed15:26
regXboikevinbenton: so that looks to be used in the router gateway port - let me confirm15:27
kevinbentonregXboi: 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 enabled15:27
regXboiah15:28
kevinbentonregXboi: so i need to add some flag to pass to the L2 agents for this case15:29
*** vikram has quit IRC15:29
kevinbentonregXboi: to tell it to skip port security for those15:29
*** killer_prince has quit IRC15:30
*** TrevorV has joined #openstack-neutron15:30
regXboikevinbenton: 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 mangling15:31
regXboiso FIP traffic will flow through that port15:31
amullerregXboi: internal_network_added?15:31
*** slaweq has quit IRC15:31
*** zhenguo has quit IRC15:31
regXboiamuller: 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 that15:32
regXboiamuller: so I was looking to see if the logs are incomplete or I'm just blind15:32
salv-orl_ihrachyshka: yes. we already expose history to that aim. But heads is more immediate15:33
*** Sukhdev has joined #openstack-neutron15:33
*** moshele has quit IRC15:34
ihrachyshkasalv-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 IRC15:35
regXboiamuller: not yet - that's part of what my patch adds :)15:36
ihrachyshkasalv-orl_, will you want to proceed with it? or do you want me to take over it?15:36
*** zengine has quit IRC15:36
*** dims__ has joined #openstack-neutron15:36
regXboican we get it merged ?15:36
*** vivek-ebay has joined #openstack-neutron15:36
*** britthou_ has quit IRC15:36
amullerregXboi: I thought we were talking about your patch =D15:37
pc_msalv-orl_: ping15:38
regXboiamuller: no - I was talking in the absence of it -15:38
*** david-ly_ is now known as david-lyle15:38
*** jistr has joined #openstack-neutron15:38
*** vivek-eb_ has joined #openstack-neutron15:38
regXboiamuller: trying to solve a problem with a tempest patch without it, and it's .... well .... painful ....15:38
amullerregXboi: just approved your patch15:39
amullerso hopefully next time it won't be as painful :)15:39
*** dims_ has quit IRC15:39
*** vivek-ebay has quit IRC15:40
openstackgerritPengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db  https://review.openstack.org/21356315:40
regXboiamuller: mahalo, moi loa15:40
salv-orl_ihrachyshka: I can proceed with this change. But you know I'm old and old people are slower15:41
salv-orl_hi pc_m15:41
pc_msalv-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_msalv-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 reviewers15:42
*** SourabhP has joined #openstack-neutron15:42
*** rstarr has quit IRC15:43
*** rstarr has joined #openstack-neutron15: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 complete15:43
*** devvesa has quit IRC15:44
*** SourabhP_ has joined #openstack-neutron15:44
pc_msalv-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
ihrachyshkasalv-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 worries15:45
pc_msalv-orl_: IOW, add the extension when doing the part that modifies APIs and uses the new API?15:45
*** Swami has joined #openstack-neutron15:45
ihrachyshkasalv-orl_, looking forward :)15:45
salv-orl_pc_m: I see; I did not consider you had a prerequiste change in the DB model15:45
*** diga has quit IRC15: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-neutron15:46
pc_msalv-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 IRC15:47
*** SourabhP has quit IRC15:47
*** SourabhP_ is now known as SourabhP15:47
pc_mIt would also switch from using the subnet id in service to the endpoint group's subnet based endpoints.15:47
*** evgenyf has quit IRC15:47
pc_msalv-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_msalv-orl_: not quite...15:48
pc_msalv-orl_: The endpoint groups can be pushed independently. They would only be used, when trying to implement the multiple local subnets.15:49
pc_mendpoints are a prerequisite for the multiple local subnets.15:49
*** dannywilson has joined #openstack-neutron15: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 merge15:50
pc_mexactly. Just populating a table that is unused.15:50
openstackgerritRossella Sblendido proposed openstack/neutron: Move logic to handle OVS status out of rpc_loop  https://review.openstack.org/19916515: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 groups15:50
*** dannywilson has quit IRC15:50
salv-orl_pc_m:  whereas it's actually the opposite15:50
*** dannywil_ has joined #openstack-neutron15:51
pc_msalv-orl_: correct15:51
*** Aish has quit IRC15: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_v315:51
salv-orl_pc_m: now about the extensions labels.15:52
pc_mThought 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-neutron15: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 group15:52
salv-orl_but they would still use the "old" API until you go through the deprecation cycle15: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 IRC15:53
*** sridhar_ram has joined #openstack-neutron15:54
*** sridhar_ram has quit IRC15:54
pc_msalv-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_msalv-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 IRC15:55
pc_msalv-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-neutron15: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 use15:57
salv-orl_load balancing went through a similar thing.15:58
salv-orl_I hereby summon dougwig15:58
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444115:58
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444615:58
salv-orl_dougwig: is it true that lbaasv1 is still in its deprecation cycle but not yet killed?15:58
*** matrohon has quit IRC15:58
pc_msalv-orl_: so, place the new API under a new extension?15:58
*** irenab has joined #openstack-neutron15:59
salv-orl_pc_m: this is what you're doiing afaict15:59
*** rstarr has quit IRC15:59
*** rstarr has joined #openstack-neutron15:59
salv-orl_the other question is what to do with the old one.15:59
openstackgerritIhar Hrachyshka proposed openstack/neutron: [neutron-db-manage] Introduce contract and expand commands  https://review.openstack.org/21631315:59
pc_msalv-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-neutron16:00
salv-orl_pc_m: adding endpoint groups is totally fine. Changing the VPNService resource is the tricky bit, in my opinion16: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" thing16:01
openstackgerritRossella Sblendido proposed openstack/neutron: OVS agent add functional tests of OVS status  https://review.openstack.org/20544216:01
*** fawadkhaliq has joined #openstack-neutron16:01
pc_msalv-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 broken16:02
salv-orl_pc_m: that sounds like a backward compatibility strategy, yes16:02
pc_mAt 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-neutron16:03
salv-orl_so subnet_id and endpoint_group_id become mutually exclusive?16:03
*** insequent has joined #openstack-neutron16:03
pc_mFor 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-neutron16:03
pc_msalv-orl_: not sure about mutex. I was thinking more of a "live migration" sort of.16:05
*** ushkalim has quit IRC16:05
pc_msalv-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_id16:06
pc_myes16:07
salv-orl_and a db migration will ensure continuitiy for all existing VPN services16:07
pc_myes16:07
salv-orl_so that you have "backward compatibility" (with quotes)16:07
pc_mthinking 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 API16:07
*** links has quit IRC16:08
*** yamamoto has joined #openstack-neutron16:08
*** sudipto has joined #openstack-neutron16:08
pc_mthen, 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 work16:08
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444616:09
*** gberginc has joined #openstack-neutron16:09
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445516:09
pc_msalv-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-neutron16:09
salv-orl_for instance the API client will expect that same subnet_id in a GET response16:09
salv-orl_pc_m: everything is a tradeoff16:09
salv-orl_either you manage the complexity in software, or the user takes the hit for handling the backward incompatible change.16:10
BobBallIs 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 IRC16:10
*** vivek-ebay has joined #openstack-neutron16:10
salv-orl_Personally I would always spare the user from the pain of dealing with anything, but there are pro and cons16:10
amullerBobBall: That's a no16:10
salv-orl_for insttance, it might be that users really don't care16:10
salv-orl_so we16:10
salv-orl_would be talking about nothing16:10
BobBallamuller: That's a shame :/ But thanks for confirming.  I won't go looking for him then!16:11
pc_msalv-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-neutron16:12
mesterypc_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 it16: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_mmestery: discussion how to handle migration to new API.16:12
*** jckasper has joined #openstack-neutron16:12
*** dims__ has quit IRC16:12
*** yamamoto has quit IRC16:12
*** jistr has quit IRC16:13
*** dims has joined #openstack-neutron16:13
pc_mdiscussing...16:13
*** vivek-eb_ has quit IRC16: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 anyway16:14
mesterysalv-orl_: +1000.516:14
*** mitchjameson has joined #openstack-neutron16:14
pc_mmestery: So what should we do in this case?16:14
pc_msalv-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 deployed16:15
*** SourabhP has quit IRC16:15
salv-orl_if people complain then we'll study a backward compatibility strategy16:15
pc_msalv-orl_: sounds good to me. Is that a different mailing list than openstack dev one?16:16
mesterysalv-orl_ pc_m : Yes, we need that email sent16:16
mesterypc_m: It is16:16
pc_mmestery: Do you want me to send that, or you?16:16
*** rushil has quit IRC16:17
mesterypc_m: Can you send this out? I'll back you up should someone push back16:17
pc_mmestery: Sure. Just tell me the list16:17
pc_msalv-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 IRC16:18
mesterypc_m: openstack-operators16:18
pc_mmestery: thanks.16:19
*** _cjones_ has joined #openstack-neutron16:19
salv-orl_pc_m: please fill me in - I emptied my brain over the weekend as I washed it in litres of wines16:19
mesterylol16:20
salv-orl_is the new module for the endpoint groups stuff16:20
*** rushil has joined #openstack-neutron16:20
pc_mmestery: 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 IRC16:20
pc_msalv-orl_: Yes, this one https://review.openstack.org/#/c/212692/2/neutron_vpnaas/extensions/vpnaas_v3.py16:21
*** l6unchpad has quit IRC16:21
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: [WIP] Add MTU support for FedoraStrongSwan driver  https://review.openstack.org/21632416:21
*** moha_hunt has quit IRC16:22
pc_msalv-orl_: I was a little confused about the comment in vpn_db.py module, on not having two classes.16:22
*** liangbo has quit IRC16:23
pc_msalv-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 team16:23
* salv-orl_ noone of team studied at Hogwarths apparently16:24
salv-orl_pc_m: now let's assume we adopt the strategy you just outlined16:24
pc_msalv-orl_: I'm still pretty fuzzy on how to handle the the two versions.16:24
pc_msalv-orl_: OK16:24
salv-orl_in that case I think we would not need two versions at all, but an incremental change16:24
*** sudipto has quit IRC16:25
pc_msalv-orl_: You're talking add the dbase and API to existing extension module?16:25
salv-orl_pc_m:  ext modules16: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 back16:26
pc_msalv-orl_: OK. Need to reset to make sure we're talking of same proposal.16:26
*** ronis_ has joined #openstack-neutron16:26
* pc_m we talked about several things.16:26
pc_mmestery: 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
mesterythanks pc_m!16:27
openstackgerritenikanorov proposed openstack/neutron: Add periodic agents health check.  https://review.openstack.org/18163316:28
pc_mmestery: I updated the endpoint group dev ref to pick the proposal desired, and have some commits out for this (as discussed).16:28
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532916:28
*** geekinutah has quit IRC16:28
mesterythanks pc_m16:28
*** sudipto has joined #openstack-neutron16:29
pc_mmestery: 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-neutron16:30
*** gberginc has quit IRC16:31
*** jbell8 has quit IRC16:31
*** cdl has left #openstack-neutron16:31
*** vivek-ebay has quit IRC16:32
*** ildikov has joined #openstack-neutron16:33
*** jckasper has quit IRC16:33
*** sdake_ has joined #openstack-neutron16: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 requirements16:34
salv-orl_the endpoint_groups extension will introduce the new endpoint resources and do the bw incompatible change in the VPNService resource16:34
*** stanzgy has joined #openstack-neutron16:34
salv-orl_but that's pretty much it.16:34
salv-orl_you will still have a single DB class16:34
*** clev is now known as clev-away16: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_msalv-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 IRC16: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 ext16:36
*** pbrooko has quit IRC16:36
salv-orl_but if you don't care about that, just add it to vpnaas16:36
salv-orl_since you'd still be using the same db class, in this case doing the extra extension module is trivial16:37
*** sdake has quit IRC16:37
pc_msalv-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 IRC16:39
*** lyanchih has quit IRC16:39
*** alex_klimov has quit IRC16:40
pc_msalv-orl_: sort fo getting a bit lost... probably because I don't understand all the terminology. Sorry.16:40
pc_msalv-orl_: When you say adding endpoint_groups as a new extension which has vpnaas as a requirements, what do you mean?16:40
openstackgerritIhar Hrachyshka proposed openstack/neutron: [neutron-db-manage] Introduce contract and expand commands  https://review.openstack.org/21631316:40
pc_mIn 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-neutron16:42
pc_msalv-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 IRC16:43
pc_mThat's what I have in the PS now, only with vpnaas_v3.py as the module name.16:43
*** rushil has quit IRC16:44
*** wendar_ is now known as wendar16: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-neutron16:45
*** dims__ has joined #openstack-neutron16:45
*** mohankumar has joined #openstack-neutron16:46
*** ameybhide has quit IRC16:47
armaxpc_m: regarding API extension handling there’s some review https://review.openstack.org/#/c/203878/ that’s still waiting for you16:47
armaxpc_m:  do you think that what you’re doing has some interceptions with the work I did with service providers?16:48
pc_marmax: Will look at comment.16:49
*** john-davidge has quit IRC16:49
*** nyechiel has quit IRC16: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-neutron16: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 it16:51
*** vivek-ebay has joined #openstack-neutron16:51
salv-orl_otherwise add the new stuff to vpnaas.py and don't bother about anything16:51
*** liangbo has joined #openstack-neutron16:51
*** jbell8 has joined #openstack-neutron16:52
pc_msalv-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: yes16:53
pc_msalv-orl_: Would it have its own "base class" only for that extension?16:53
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532916:53
*** piyanai has quit IRC16:54
openstackgerritMiguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports  https://review.openstack.org/20095216:55
pc_msalv-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-neutron16:56
*** jwy has joined #openstack-neutron16:56
*** Aish has joined #openstack-neutron16:56
salv-orl_pc_m: correct16:57
pc_msalv-orl_: OK, so I did that wrong. Thanks.16:58
*** greystart has joined #openstack-neutron16:58
*** minwang2 has joined #openstack-neutron16:58
*** lindycoder has quit IRC16:59
*** diegows has joined #openstack-neutron17:00
pc_msalv-orl_: So don't need to subclass from the Vpnaas class.17:00
*** piyanai has joined #openstack-neutron17:00
salv-orl_pc_m: not unless you want to push an alternative vpnaas api.17:02
salv-orl_there is only one caveat17:02
salv-orl_if you do two extension modules you need to ensure that vpnaas is loaded before the module for endpoint groups17:02
pc_msalv-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 done17:02
*** sdake has joined #openstack-neutron17:02
*** mitchjameson has quit IRC17:03
salv-orl_pc_m: sorry, what might be needed when doing multiple local subnet?17:03
*** insequent has quit IRC17:03
*** nyechiel has quit IRC17:04
pc_msalv-orl_: That's when we introduce use of the new API to existing commands.17:04
*** shivrao has joined #openstack-neutron17:05
pc_msalv-orl_: Or could they be updated under the vpnaas.py module?17:05
*** rossella_s has quit IRC17:05
*** sdake_ has quit IRC17:06
*** rossella_s has joined #openstack-neutron17:06
pc_msalv-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 matter17:06
*** fitoduarte has joined #openstack-neutron17:07
*** garyk has quit IRC17:07
*** lindycoder has joined #openstack-neutron17:07
*** britthouser has joined #openstack-neutron17:07
salv-orl_users cannot really benefit from knowing that there are endpoints group because the MUST use it17:07
*** Swami has quit IRC17:07
salv-orl_it's not like they have a choice17:07
pc_msalv-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 IRC17:09
*** jckasper has joined #openstack-neutron17:09
salv-orl_pc_m: yes, but if you choose to do a new extension, subclassing is not really needed17:10
salv-orl_because the new extension should describe vpnaas endpoint groups only17:10
pc_msalv-orl_: by new, I meant additional extension.17:10
salv-orl_pc_m: sure17:10
pc_msalv-orl_: gotcha, but have to make sure things load right, correct?17:10
salv-orl_pc_m: yes, still looking that up17:11
pc_msalv-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 case17:11
*** Swami has joined #openstack-neutron17:12
*** Aish has quit IRC17:12
*** insequent has joined #openstack-neutron17:12
*** nmagnezi has quit IRC17:13
*** Marga_ has quit IRC17:13
pc_msalv-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: sure17:14
salv-orl_that's the simplest thing to do17:14
salv-orl_and possibly even the most correct one in your case17:14
pc_msalv-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
openstackgerritBrian Haley proposed openstack/neutron: Defer freeing of conntrack zone ids until allocation fails  https://review.openstack.org/21579117: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_msalv-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 method17:16
*** jckasper has quit IRC17:17
salv-orl_and that method must return ['vpnaas']17:17
salv-orl_that will ensure vpnaas is always loaded first17: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_msalv-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 do17:18
pc_msalv-orl_: That was my thought... add to existing for now.17:18
pc_msalv-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_msalv-orl_: I just need to see how VPN reviewers want to digest the patches.17:19
*** madhu_ak has joined #openstack-neutron17: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 IRC17:20
pc_msalv-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-neutron17: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 API17:22
openstackgerritKevin Benton proposed openstack/neutron: Flag port sec as disabled on router GW ports  https://review.openstack.org/21549117:23
*** abhiraut has joined #openstack-neutron17:23
*** sudipto has joined #openstack-neutron17:23
pc_msalv-orl_: Right.17:24
pc_mmestery: 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
mesterypc_m: I'd like to see it so the jobs aren't broken :)17:25
armaxmestery: +117:25
pc_mmestery: armax: dougwig: My preference is probably to do two patches, one for each feature.17:25
armax:)17:25
armaxmakes sense17:26
*** emagana has quit IRC17:26
mestery+1.517:26
pc_mmestery: armax: me too :) I think the two patches will do that.17:26
*** lilynn has quit IRC17:26
pc_mmestery: 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-neutron17:28
*** jckasper has joined #openstack-neutron17:28
*** diegows has quit IRC17:29
*** abhiraut has quit IRC17:29
*** nlahouti has joined #openstack-neutron17:30
openstackgerritIhar Hrachyshka proposed openstack/neutron: devref: added guidelines on how to maintain sub-projects  https://review.openstack.org/21634817:32
*** emagana has joined #openstack-neutron17:32
*** lazy_prince has quit IRC17:32
ihrachyshkamestery, I've pushed a devref page with some guidelines for subprojects. please take a look and say what you think ^^17:32
pc_msalv-orl_: Thank so much for all the guidance here!17:32
*** achanda has joined #openstack-neutron17:33
*** Sukhdev has quit IRC17:33
openstackgerritMatthew Treinish proposed openstack/neutron: Stop logging STDOUT and STDERR on every shell out  https://review.openstack.org/21635017:34
openstackgerritMerged openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/20442217:35
*** piyanai has quit IRC17:36
*** emagana has quit IRC17:37
*** abhiraut has joined #openstack-neutron17:38
*** garyk has joined #openstack-neutron17:38
*** sudipto has quit IRC17:38
*** baoli_ has quit IRC17:40
*** SridharG has joined #openstack-neutron17:40
*** emagana has joined #openstack-neutron17:40
SridharGhaleyb: can you pls take a look at https://review.openstack.org/#/c/205101/ thanks.17:40
*** amuller is now known as amuller_afk17:41
*** gberginc has joined #openstack-neutron17:43
*** bharathm has joined #openstack-neutron17:43
*** sudipto has joined #openstack-neutron17:44
*** gsagie_ has quit IRC17:44
*** baoli has joined #openstack-neutron17:44
*** clev-away is now known as clev17:46
*** srilatha has quit IRC17:48
*** srilatha has joined #openstack-neutron17:48
pc_mmestery: Do you know if dougwig is around?17:49
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants  https://review.openstack.org/21578217:50
salv-orl_pc_m: that was my pleasure17:50
*** vikram has quit IRC17:50
*** Sukhdev has joined #openstack-neutron17:52
*** jprovazn has quit IRC17:55
*** ihrachyshka has quit IRC17:56
pc_mmestery: 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-neutron17:58
*** rm_work is now known as rm_work|away18:00
*** sbalukoff has joined #openstack-neutron18:00
openstackgerritMerged openstack/python-neutronclient: Remove --shared option from firewall-create  https://review.openstack.org/19465218:00
*** piyanai has joined #openstack-neutron18:02
*** Sukhdev has quit IRC18:03
*** mohankumar has quit IRC18:04
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442418:05
*** baoli has quit IRC18:06
*** saksham has joined #openstack-neutron18:11
*** neophy has quit IRC18:12
*** SridharG has left #openstack-neutron18:14
*** sdake_ has joined #openstack-neutron18:14
*** zengine has joined #openstack-neutron18:14
openstackgerritShraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent  https://review.openstack.org/21283618:15
*** agireud has quit IRC18:15
*** baoli has joined #openstack-neutron18:16
*** piyanai has quit IRC18:16
*** sdake has quit IRC18:17
*** sdake has joined #openstack-neutron18:18
*** bapalm- is now known as bapalm_18:19
*** ivar-lazzaro has quit IRC18:20
*** baoli has quit IRC18:20
*** ivar-lazzaro has joined #openstack-neutron18:20
*** jckasper has quit IRC18:21
*** amuller_afk is now known as amuller18:21
*** sdake_ has quit IRC18:21
*** arosen has joined #openstack-neutron18:22
*** rushil has quit IRC18:24
*** ivar-lazzaro has quit IRC18:25
*** baoli has joined #openstack-neutron18:25
*** piyanai has joined #openstack-neutron18:28
*** anilvenkata has joined #openstack-neutron18:28
*** sthillma has joined #openstack-neutron18:28
*** zhhuabj has quit IRC18:29
*** zhhuabj has joined #openstack-neutron18:30
*** garyk has quit IRC18:30
*** liangbo has quit IRC18:30
*** abhiraut has quit IRC18:31
*** mickeys has quit IRC18:31
*** piyanai has quit IRC18:31
pc_mmestery: ping18:32
*** nlahouti has quit IRC18:33
*** neophy has joined #openstack-neutron18:33
*** piyanai has joined #openstack-neutron18:34
*** agireud has joined #openstack-neutron18:34
*** piyanai has quit IRC18:35
*** absubram has quit IRC18:36
*** carl_baldwin has joined #openstack-neutron18:37
*** jckasper has joined #openstack-neutron18:40
*** sthillma_ has joined #openstack-neutron18:40
*** sthillma has quit IRC18:40
*** sthillma_ is now known as sthillma18:40
*** stanzgy has quit IRC18:41
*** stanzgy has joined #openstack-neutron18:41
regXboiamuller: ping - how comfortable would you be with the idea of setting oslo.concurrency log level to something above DEBUG during tempest runs?18:41
amullerregXboi: sounds like a horrible idea =D18:41
amullerregXboi: what problem are you facing?18:41
regXboiamuller: 20% of the q-svc log is from oslo.concurrency and I'm wondering if we need it18:42
regXboiamuller: well, ~18% to be more precise18:42
amullerregXboi: too much logging then? why set it to higher than DEBUG then?18:42
regXboihigher meaning to *not* show DEBUG level statement18:43
*** Marga_ has joined #openstack-neutron18:44
*** Marga_ has quit IRC18:45
*** Marga_ has joined #openstack-neutron18:46
*** bharathm has quit IRC18:48
*** rm_work|away is now known as rm_work18:48
*** bharathm has joined #openstack-neutron18:48
*** VimalTewari has joined #openstack-neutron18:50
*** piyanai has joined #openstack-neutron18:51
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532918:53
*** minwang2 has quit IRC18:54
amullerregXboi: generally I'm OK with no showing those lock grabbed messages18:54
amullernot*18:54
*** numan has joined #openstack-neutron18:55
*** fawadkhaliq has quit IRC18:55
regXboiamuller: 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
regXboior devstack-gate - not sure yet18:55
regXboidougwig (as infra lieutenant): FYI ^^^^^^^18:56
*** numan has quit IRC18:56
*** numan has joined #openstack-neutron18:57
*** sthillma_ has joined #openstack-neutron18:58
*** jgriffith has quit IRC18:59
*** lori is now known as lori|away18:59
*** anilvenkata has left #openstack-neutron19:00
clarkbit shouldnt19:00
*** sthillma has quit IRC19:00
*** sthillma_ is now known as sthillma19:00
clarkblog config is all in neutron ir devstack19:00
regXboiclarkb: there is no way to inject specific config into devstack-gate?19:01
*** drankis has quit IRC19:01
*** jgriffith has joined #openstack-neutron19:01
*** jgriffith is now known as Guest719319:02
clarkbthere is but we shouldnt19:02
clarkbdevstack gate should not set defaults for you19:02
regXboiclarkb: ok, then I'll go look at devstack-gate19:02
clarkbno please dont19:02
regXboiwait19:02
regXboiI think I'm confused19:02
regXboiI don't want to set the default - I want it to be injected as part of a particular job's config19:02
clarkbdevstack-gate exists to configure jobs it does not exist to set neutron/devstack defaults19:02
clarkbif you set it in devstack-gate it becomes dfacto default19:03
clarkbbecause all the testing is then done in that setup19:03
*** Guest7193 has quit IRC19:04
regXboiclarkb: let me explain the situation19:04
regXboiclarkb: right now 18% of the logs of neutron server during a gate/check pipeline run come from oslo_concurrency.lockutils19:05
regXboiclarkb: I'd like to be able to set the log level for that module to WARN so that the DEBUG statements don't appear19:05
clarkbyes so do that in either neutron or devstaxk19:06
clarkbdo it in neutron if it makes sense for all neutron deployments, do it in devstack if it makes sense for dev test19:06
regXboiI'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 only19:07
clarkbif its going to be set in devstack gate as a default it likely should just go in devstack at least19:08
* regXboi thinks about this19:09
clarkbits 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 issue19:09
* regXboi still is considering19:09
*** numan is now known as numans19:10
regXboiclarkb: ok, having looked at devstack, I can grok that it should go there19:11
*** annegent_ has joined #openstack-neutron19:12
amullerregXboi: it'd be interesting to hear oslo concurrency devs19:12
regXboiamuller: in what way?19:13
amullerregXboi: maybe oslo concurrency devs are interested in that DEBUG output but the different applications (nova, neutron etc) aren't19:13
amullerregXboi: so worth hearing their opinion on that stement19:13
amullerstatement19:13
regXboiah19:13
regXboido we have an oslo concurrency folks here?19:13
amullerregXboi: I'd try #openstack-oslo19:14
regXboiamuller: already there19:15
*** annegentle has quit IRC19:16
*** jaypipes has quit IRC19:18
*** matrohon has joined #openstack-neutron19:20
regXboiamuller: 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 WARN19:20
amullerregXboi: I think you should send an email to openstack-dev with the problem and your suggestion19:21
regXboitag it neutron/oslo/infra ?19:21
amullerregXboi: sure19:21
regXboiack19:21
amullerregXboi: I'd also present the possibility of neutron raising oslo concurrency logging to info/warn globally for all files19:21
*** sudipto has quit IRC19:22
regXboias opposed to the dual patch approach?19:22
*** njohnston is now known as nate_gone19:23
amullerregXboi: as opposed to the idea of changing log levels per module via neutron.conf19:23
regXboiack19:23
*** neiljerram_bb has joined #openstack-neutron19:24
dims__amuller: regXboi: nova sets log levels for a bunch of things, https://github.com/openstack/nova/blob/master/nova/config.py#L4719:26
regXboiamuller, dims__: I think we could run with that as a start19:27
regXboiamuller: and then if we need to change it, overlay it with config settings19:27
regXboiamuller: so two patches19:27
regXboiamuller: the one for now and the other as "if we find we need it"19:28
*** tidwellr1 has joined #openstack-neutron19:29
regXboibecause there are other modules that we could catch with that first patch19:29
*** jwy has quit IRC19:30
*** nlahouti has joined #openstack-neutron19:30
pc_marmax: ping19:31
armaxpc_m: pong19:32
pc_marmax: I approved https://review.openstack.org/#/c/203878/. Sorry for the delay. I missed the response.19:32
armaxpc_m: no worries19:32
pc_mI don' t see dougwig around. Can you approve https://review.openstack.org/#/c/213253/ ?19:32
* armax looks19:33
pc_marmax: 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-neutron19:33
*** jgriffit- has joined #openstack-neutron19:34
*** srilatha has quit IRC19:34
*** srilatha has joined #openstack-neutron19:34
*** crose has quit IRC19:35
*** srilatha has quit IRC19:39
*** kbringard has quit IRC19:40
*** minwang2 has joined #openstack-neutron19:42
openstackgerritAssaf Muller proposed openstack/neutron: Add tunneling support to full stack tests  https://review.openstack.org/19078619:43
*** VimalTewari has quit IRC19:45
*** numans has quit IRC19:46
*** shivrao has quit IRC19:47
*** ZZelle has joined #openstack-neutron19:49
*** ZZelle is now known as ZZll-off19:49
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants  https://review.openstack.org/21578219:51
*** ronis_ has quit IRC19:51
*** abregman has joined #openstack-neutron19:52
*** jwy has joined #openstack-neutron19:52
*** yfried has joined #openstack-neutron19:52
*** gberginc has quit IRC19:53
*** piyanai has quit IRC19:53
*** shivrao has joined #openstack-neutron19:57
*** sdake_ has joined #openstack-neutron19:58
*** crose has joined #openstack-neutron19:58
*** crose has quit IRC19:58
*** j^2 has left #openstack-neutron19:59
openstackgerritAssaf Muller proposed openstack/neutron: Add l2pop support to full stack tests  https://review.openstack.org/19083219:59
*** sdake has quit IRC20:01
*** kutija has joined #openstack-neutron20:03
*** Marga_ has quit IRC20:03
*** kbringard has joined #openstack-neutron20:03
openstackgerritRyan Moats proposed openstack/neutron: WiP: set imported module default logging levels  https://review.openstack.org/21640420:04
*** Marga_ has joined #openstack-neutron20:05
*** annegent_ has quit IRC20:07
*** achanda has quit IRC20:07
*** annegentle has joined #openstack-neutron20:07
openstackgerritSalvatore Orlando proposed openstack/neutron: Remove _extract_roles method from neutron.policy  https://review.openstack.org/21640520:07
*** piyanai has joined #openstack-neutron20:11
*** greystart has quit IRC20:11
openstackgerritSalvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement  https://review.openstack.org/21641020:13
*** dmartls1 has quit IRC20:13
pc_marmax: Thanks for the review approval!20:14
openstackgerritSalvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement  https://review.openstack.org/21641020:14
*** kbringard has quit IRC20:15
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor AGENT_TYPE_* constants  https://review.openstack.org/21578220:16
*** jckasper has quit IRC20:16
*** jckasper has joined #openstack-neutron20:17
*** baoli has quit IRC20:17
openstackgerritMerged openstack/neutron: Move db agent schedulers test to a more appropriate place  https://review.openstack.org/21413120:18
*** fzdarsky has quit IRC20:18
*** kbringard has joined #openstack-neutron20:19
*** mickeys has joined #openstack-neutron20:20
*** absubram has joined #openstack-neutron20:21
*** Sukhdev has joined #openstack-neutron20:25
*** vivek-ebay has quit IRC20:25
openstackgerritCedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants  https://review.openstack.org/21480620:26
*** slaweq has quit IRC20:28
openstackgerritCedric Brandily proposed openstack/neutron: Remove out-of-tree vendor AGENT_TYPE_* constants  https://review.openstack.org/21641720:30
*** kbringard1 has joined #openstack-neutron20:31
*** TrevorV has quit IRC20:32
*** kbringard has quit IRC20:32
*** jerrygb has joined #openstack-neutron20:33
openstackgerritMerged openstack/neutron: Rationalize neutron logs to help in troubleshooting router issues  https://review.openstack.org/21565520:34
*** sdake_ is now known as sdake20:34
*** jerrygb is now known as test0rz20:34
*** test0rz is now known as asd112z20:34
*** krtaylor has quit IRC20:35
*** yfried has quit IRC20:36
ZZll-offkevinbenton, hi20:38
kevinbentonZZll-off: yo20:40
*** neiljerram_bb has quit IRC20:40
*** achanda has joined #openstack-neutron20:40
*** achanda has quit IRC20:41
*** achanda has joined #openstack-neutron20:41
ZZll-offkevinbenton, any idea why there are 2 configuration options defining the integration bridge?20:42
*** daneyon has joined #openstack-neutron20:42
openstackgerritRyan Moats proposed openstack/neutron: l3_db, l3_dvr_db: separate _process_floating_{ips, interfaces}()  https://review.openstack.org/12469620:42
amullerZZll-off: two?20:42
ZZll-offkevinbenton, https://github.com/openstack/neutron/blob/master/neutron/agent/linux/interface.py#L34-L36 (default section)20:42
openstackgerritAssaf Muller proposed openstack/neutron: Uniquely identify tunnel interfaces  https://review.openstack.org/19377020:43
kevinbentonZZll-off: i suspect one is to use for things that plug into OVS20:43
kevinbentonZZll-off: and the other is used for the OVS agent20:43
kevinbentonZZll-off: but that's just a quick guess20:44
ZZll-offkevinbenton, 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-neutron20:44
kevinbentonZZll-off: yeah, the first one you sent is for things like DHCP agent and l3 agent20:44
ZZll-offkevinbenton, because the option helps do not help to understand the difference20:44
*** sdake has quit IRC20:44
kevinbentonZZll-off: that need to plug into OVS20:44
kevinbentonZZll-off: and the second is for the OVS agent to know where to expect them20:45
ZZll-offkevinbenton, by why do we 2 options?20:45
ZZll-offkevinbenton, by why do we need 2 options?20:45
ZZll-offkevinbenton, (except option registration trouble)20:45
kevinbentonZZll-off: we probably don't20:46
kevinbentonZZll-off: i can't think of a reason you would want that20:47
*** daneyon has quit IRC20:47
*** daneyon has joined #openstack-neutron20:47
*** stanzgy has quit IRC20:47
*** krtaylor has joined #openstack-neutron20:48
*** vivek-ebay has joined #openstack-neutron20:48
*** srilatha has joined #openstack-neutron20:48
ZZll-offkevinbenton, me neither, i could be a rare opportunity to reduce option number O_o20:50
*** shivrao has quit IRC20:51
*** jwy has quit IRC20:53
kevinbentonZZll-off: yeah20:54
*** tflynn has joined #openstack-neutron20:54
kevinbentonZZll-off: does oslo have an option to deprecate in a consolidating fashion like that?20:54
ZZll-offkevinbenton, let me find an example20:54
*** matrohon has quit IRC20:55
*** daneyon_ has joined #openstack-neutron20:55
*** daneyon has quit IRC20:58
*** shivrao has joined #openstack-neutron20:58
ZZll-offkevinbenton, https://github.com/openstack/neutron/blob/8546d5ec3db3bd589dd810e2e8ed097b29068116/neutron/agent/dhcp/config.py#L50-L5820:58
*** zengine has quit IRC20:58
ZZll-offkevinbenton, 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 IRC21:00
*** rushil has joined #openstack-neutron21:01
*** jbell8 has quit IRC21:01
*** rkukura has quit IRC21:01
*** fitoduarte has quit IRC21:02
*** vivek-ebay has quit IRC21:02
*** fitoduarte has joined #openstack-neutron21:02
*** emagana has joined #openstack-neutron21:03
*** thorst has quit IRC21:03
*** geekinutah has quit IRC21:04
*** amuller has quit IRC21:04
*** piyanai has quit IRC21:06
*** neiljerram_bb has joined #openstack-neutron21:07
neiljerram_bbcarl_baldwin : hi, are you around?21:07
carl_baldwinneiljerram_bb: hi21:07
*** pc_m has quit IRC21:08
neiljerram_bbcarl_baldwin : hi, good evening ; or perhaps still good day for you?21:08
*** tflynn has quit IRC21:09
*** moshele has joined #openstack-neutron21:09
carl_baldwinneiljerram_bb: good day/evening to you too.21:09
neiljerram_bbcarl_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/148664921:10
openstackLaunchpad 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
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: Endpoint groups: Implements API and database  https://review.openstack.org/21269221:10
*** lilynn has joined #openstack-neutron21:11
*** jbell8 has joined #openstack-neutron21:12
neiljerram_bbcarl_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_bbcarl_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-neutron21:18
*** jwy has joined #openstack-neutron21:18
carl_baldwinneiljerram_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-neutron21:20
*** moshele has quit IRC21:20
*** absubram has quit IRC21:23
*** neiljerram_bb has quit IRC21:23
*** neiljerram_bb has joined #openstack-neutron21:23
openstackgerritAdolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR  https://review.openstack.org/19689321:26
*** mwagner_lap has quit IRC21:27
*** abregman has quit IRC21:29
*** neiljerram_home has joined #openstack-neutron21:33
*** lilynn has quit IRC21:33
*** lilynn has joined #openstack-neutron21:33
*** diegows has joined #openstack-neutron21:33
neiljerram_homecarl_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 IRC21:34
carl_baldwinneiljerram_home: Not sure what you might’ve missed.  Eavesdrop should be able to fill you in.21:34
*** piyanai has joined #openstack-neutron21:35
neiljerram_homecarl_baldwin: Ah yes, I see what you said about picking up the L3 model conversation.21:35
openstackgerritAdolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR  https://review.openstack.org/19689321:37
neiljerram_homecarl_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
mlavallecarl_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.py21:38
mlavalle?21:39
*** abhiraut has joined #openstack-neutron21:39
carl_baldwinmlavalle: Yes.21:39
openstackgerritAdolfo Duarte proposed openstack/neutron: L3 Agent support for routers with HA and DVR  https://review.openstack.org/19689321:40
mlavallecarl_baldwin: cool, thanks!21:40
carl_baldwinmlavalle: Sorry for the churn.21:40
mlavallecarl_baldwin: i'll fix the nits you found in that implementation21:40
mlavallecarl_baldwin: not a problem. that is what open source development is all about...... healthy debate21:41
mlavallecarl_baldwin: i've learned a lot, so thanks to you and kevinbenton21:41
carl_baldwinmlavalle: I’ll get the egg wiped off my face soon and move on.  :)21:42
*** Piet has quit IRC21:42
*** yfried has joined #openstack-neutron21:44
*** madhu_ak_ has joined #openstack-neutron21:46
*** madhu_ak has quit IRC21:50
*** lindycoder has quit IRC21:50
*** tflynn has joined #openstack-neutron21:51
*** emagana has quit IRC21:55
*** agireud has joined #openstack-neutron21:56
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442421:57
*** neiljerram_home has quit IRC22:00
*** rushil has quit IRC22:00
*** madhu_ak_ is now known as madhu_ak22:01
*** vivek-ebay has joined #openstack-neutron22:02
*** emagana has joined #openstack-neutron22:03
*** pradk has quit IRC22:04
*** ujjain has quit IRC22:04
*** rushil has joined #openstack-neutron22:05
*** piyanai has quit IRC22:06
*** ujjain has joined #openstack-neutron22:07
*** ujjain has quit IRC22:07
*** ujjain has joined #openstack-neutron22:07
*** vivek-ebay has quit IRC22:07
*** kbringard1 has quit IRC22:10
*** tflynn has quit IRC22:10
*** apuimedo_ has joined #openstack-neutron22:13
openstackgerritRyan Moats proposed openstack/neutron: Set imported module default logging levels  https://review.openstack.org/21640422:16
*** agireud has quit IRC22:19
*** azbiswas has joined #openstack-neutron22:22
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444122:24
*** Sukhdev has quit IRC22:25
*** asd112z has quit IRC22:25
openstackgerritMerged openstack/neutron: Remove redundant logging statements from RootWrapDaemonHelper  https://review.openstack.org/21626022:26
openstackgerritMiguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports  https://review.openstack.org/20095222:27
*** regXboi has quit IRC22:27
openstackgerritMerged openstack/neutron: Template for ModelMigrationTest for external repos  https://review.openstack.org/21319422:27
*** sdake has joined #openstack-neutron22:29
openstackgerritMerged openstack/neutron: Make NeutronDbObjectDuplicateEntry exception more verbose  https://review.openstack.org/21460822:29
*** shoutm has joined #openstack-neutron22:32
*** sbalukoff has quit IRC22:34
*** azbiswas has quit IRC22:37
*** jckasper has quit IRC22:37
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532922:37
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444622:37
*** emagana has quit IRC22:39
*** thumpba has quit IRC22:42
*** nplanel has joined #openstack-neutron22:43
*** nplanel has quit IRC22:44
*** nplanel has joined #openstack-neutron22:44
*** shwetaap has quit IRC22:45
*** diegows has quit IRC22:45
*** insequent has quit IRC22:46
*** sridhar_ram has quit IRC22:50
openstackgerritSalvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement  https://review.openstack.org/21641022:50
*** dannywilson has joined #openstack-neutron22:50
*** apuimedo_ has quit IRC22:51
*** mwagner_lap has joined #openstack-neutron22:51
openstackgerritSalvatore Orlando proposed openstack/neutron: Devref for authorization policies enforcement  https://review.openstack.org/21641022:52
*** garyk has joined #openstack-neutron22:52
abhirautping HenryG22:53
*** dannywil_ has quit IRC22:53
*** emagana has joined #openstack-neutron22:53
*** mlavalle has quit IRC22:55
*** edmondsw has quit IRC22:57
*** jgriffit- has quit IRC23:00
*** arosen has quit IRC23:02
*** jckasper has joined #openstack-neutron23:02
*** tongli has joined #openstack-neutron23:02
*** Marga_ has quit IRC23:02
*** Marga_ has joined #openstack-neutron23:03
*** jckasper has quit IRC23:03
*** Aish has joined #openstack-neutron23:05
*** Marga_ has quit IRC23:08
*** thorst has joined #openstack-neutron23:08
*** fitoduarte has quit IRC23:10
*** zzzeek has quit IRC23:10
*** neiljerram_bb has quit IRC23:11
*** claudiub has quit IRC23:12
*** luqas__ has joined #openstack-neutron23:15
*** dims_ has joined #openstack-neutron23:16
*** luqas has quit IRC23:17
*** dims__ has quit IRC23:18
armaxobondarev_, rossella_s: https://review.openstack.org/#/c/214131 broke neutron-vpnaas23:19
*** annegentle has quit IRC23:19
*** vivek-ebay has joined #openstack-neutron23:19
armaxnot sure if there are other probjects affected23:19
*** dims__ has joined #openstack-neutron23:19
*** dims_ has quit IRC23:21
*** vivek-ebay has quit IRC23:23
openstackgerritArmando Migliaccio proposed openstack/neutron-vpnaas: Fix stale module import  https://review.openstack.org/21646023:24
openstackgerritArmando Migliaccio proposed openstack/neutron-vpnaas: Fix stale module import  https://review.openstack.org/21646023:24
armaxmestery: ^^^^^^^23:24
armaxdougwig: ^^^^23:25
abhirautping akamyshnikova23:30
*** achanda_ has joined #openstack-neutron23:31
*** dannywilson has quit IRC23:33
*** achanda has quit IRC23:34
*** abhiraut has quit IRC23:34
*** dannywilson has joined #openstack-neutron23:34
*** EmilienM has quit IRC23:34
*** Fdaisuke has joined #openstack-neutron23:35
*** NightKhaos has quit IRC23:35
*** saksham has quit IRC23:35
*** q3k has quit IRC23:35
*** abhiraut has joined #openstack-neutron23:35
*** achanda_ has quit IRC23:35
*** vivek-ebay has joined #openstack-neutron23:36
*** carl_baldwin has quit IRC23:36
*** q3k has joined #openstack-neutron23:36
*** EmilienM has joined #openstack-neutron23:36
*** dsneddon is now known as dsneddon_away23:36
*** NightKhaos has joined #openstack-neutron23:42
*** morganfainberg is now known as morgan23:42
Fdaisukearmax: Hi,23:44
Fdaisukearmax: I commented and uploaded new patch set to https://review.openstack.org/#/c/207328/23:45
FdaisukeWould you please have a look at this?23:45
*** garyk has quit IRC23:46
*** ajmiller has quit IRC23:50
*** sdake_ has joined #openstack-neutron23:51
*** tidwellr1 has quit IRC23:53
*** emagana has quit IRC23:54
*** tongli has quit IRC23:54
*** sdake has quit IRC23:55
*** emagana has joined #openstack-neutron23:56
*** apuimedo has quit IRC23:57
*** stefanb has quit IRC23:58
*** stefanb has joined #openstack-neutron23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!