Thursday, 2015-08-20

*** dave-mccowan has quit IRC00:00
*** shaohe_feng has joined #openstack-neutron00:01
*** debedb has joined #openstack-neutron00:02
HenryGsure, understood00:03
openstackgerritMerged openstack/neutron: DVR: fix router rescheduling on agent side  https://review.openstack.org/21193900:03
*** rushil has quit IRC00:03
HenryGwe could submit a patch for rally to chew on00:04
abhirautwe could reuse my dummy patch again to add a new dependency00:06
*** debedb has quit IRC00:06
*** clev is now known as clev-away00:07
*** mickeys has quit IRC00:09
*** mickeys has joined #openstack-neutron00:09
*** banix has quit IRC00:10
*** srilatha has quit IRC00:10
*** srilatha has joined #openstack-neutron00:11
*** achanda has quit IRC00:11
*** dave-mccowan has joined #openstack-neutron00:14
*** padkrish has quit IRC00:14
*** padkrish has joined #openstack-neutron00:15
*** srilatha has quit IRC00:15
openstackgerritShraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent  https://review.openstack.org/21283600:16
*** Aish has joined #openstack-neutron00:16
*** Aish has left #openstack-neutron00:16
*** madhu_ak has quit IRC00:16
*** moha_hunt has quit IRC00:16
*** shaohe_feng has quit IRC00:17
openstackgerritShraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent  https://review.openstack.org/21283600:17
*** shoutm has quit IRC00:18
*** padkrish has quit IRC00:19
*** armax has quit IRC00:20
*** nlahouti has quit IRC00:20
*** moha_hunt has joined #openstack-neutron00:20
*** shoutm has joined #openstack-neutron00:21
*** nlahouti has joined #openstack-neutron00:21
openstackgerritRitesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers.  https://review.openstack.org/18815700:22
*** armax has joined #openstack-neutron00:22
*** padkrish has joined #openstack-neutron00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-neutron00:23
*** minwang2 has quit IRC00:23
*** armax has quit IRC00:24
*** padkrish has quit IRC00:27
*** dannywil_ has quit IRC00:28
*** salv-orl_ has quit IRC00:29
*** garyk1 has quit IRC00:30
*** dims has joined #openstack-neutron00:30
*** padkrish has joined #openstack-neutron00:30
*** tiny-hands has joined #openstack-neutron00:31
*** azbiswas has quit IRC00:32
*** redbeard has joined #openstack-neutron00:32
*** azbiswas has joined #openstack-neutron00:32
*** padkrish has quit IRC00:34
openstackgerritPengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255]  https://review.openstack.org/21222000:35
*** azbiswas has quit IRC00:36
*** dave-mccowan has quit IRC00:37
*** padkrish has joined #openstack-neutron00:37
*** thorst has joined #openstack-neutron00:38
*** piyanai has quit IRC00:38
*** hoangcx has joined #openstack-neutron00:42
*** reedip has joined #openstack-neutron00:42
*** aranjan has quit IRC00:43
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process  https://review.openstack.org/21470100:48
*** tiswanso has joined #openstack-neutron00:48
*** shivrao has quit IRC00:48
*** achanda has joined #openstack-neutron00:48
*** hoangcx has quit IRC00:49
*** hoangcx has joined #openstack-neutron00:50
*** reedip_ has joined #openstack-neutron00:50
*** tiswanso has quit IRC00:50
*** reedip has left #openstack-neutron00:51
*** tiswanso has joined #openstack-neutron00:51
*** shivrao has joined #openstack-neutron00:56
*** zhhuabj has quit IRC00:57
*** vishwanathj has quit IRC00:58
*** yalie has joined #openstack-neutron00:58
*** dave-mccowan has joined #openstack-neutron00:59
*** shivrao_ has joined #openstack-neutron00:59
*** Piet has quit IRC01:00
*** liudong has joined #openstack-neutron01:00
*** shivrao has quit IRC01:00
*** shivrao_ is now known as shivrao01:00
*** shaohe_feng has joined #openstack-neutron01:02
*** yanxb_ has joined #openstack-neutron01:02
*** yanxb__ has quit IRC01:04
abhirautping: HenryG .. if you could validate this one https://review.openstack.org/#/c/214413/01:05
*** baoli has joined #openstack-neutron01:07
hoangcxsc68cal: ping at this channel01:07
HenryGabhiraut: you have good copying skillz ;)01:07
*** zhhuabj has joined #openstack-neutron01:08
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan  https://review.openstack.org/21267301:08
*** liangbo has joined #openstack-neutron01:08
*** nlahouti has quit IRC01:08
*** tfukushima has quit IRC01:08
*** tfukushima has joined #openstack-neutron01:09
openstackgerrityalei wang proposed openstack/neutron: Add missing tenant_id validation in RESOURCE_ATTRIBUTE_MAP  https://review.openstack.org/21450601:09
sc68calhoangcx: pong01:09
hoangcxsc68cal: Logging API for SG and FW rules for reviewing here.01:10
*** thorst has quit IRC01:10
abhirautaw well there wasnt much to do in it anyway :)01:10
hoangcxsc68cal: RFE #link https://bugs.launchpad.net/neutron/+bug/146836601:10
openstackLaunchpad bug 1468366 in neutron "RFE - Logging API for security group and firewall rules" [Undecided,Confirmed] - Assigned to Yushiro FURUKAWA (y-furukawa-2)01:10
hoangcxsc68cal: Specification #link https://review.openstack.org/#/c/20350901:10
*** regXboi has quit IRC01:11
hoangcxsc68cal: Source codes #link https://review.openstack.org/#/c/204481/01:11
hoangcxand here #link https://review.openstack.org/#/c/204484/01:11
abhirautHenryG: I shouldnt be screwing that up at least ;) thanks01:11
*** baoli has quit IRC01:12
*** yamamoto has joined #openstack-neutron01:12
*** nlahouti has joined #openstack-neutron01:15
*** lazy_prince has joined #openstack-neutron01:15
*** haigang has joined #openstack-neutron01:16
sc68calhoangcx: understood - however I don't know if that'll make FFE01:16
*** yalie1 has joined #openstack-neutron01:17
hoangcxsc68cal: That information from weekly meeting in "Announcements / Reminders" section #link https://wiki.openstack.org/wiki/Network/Meetings01:18
*** yalie1 has quit IRC01:19
*** yalie1 has joined #openstack-neutron01:19
*** yalie has quit IRC01:19
hoangcxsc68cal: There are current targeted for L-3 #link https://launchpad.net/neutron/+milestone/liberty-301:19
hoangcxsc68cal: would love to moving on Logging API to the Milestone because it can be reviewed and handled by FWaaS core team as we discussed in last FWaaS weekly meeting01:21
*** sdake_ has joined #openstack-neutron01:21
*** agireud has joined #openstack-neutron01:22
*** yalie1 has quit IRC01:23
*** jwy has quit IRC01:24
*** yalie has joined #openstack-neutron01:25
*** sdake has quit IRC01:25
*** dims has quit IRC01:25
*** abhiraut has quit IRC01:27
*** amuller has joined #openstack-neutron01:32
*** sthillma has quit IRC01:33
*** padkrish has quit IRC01:33
*** s3wong has quit IRC01:35
*** stanzgy has joined #openstack-neutron01:35
*** shivrao has quit IRC01:36
*** liangbo has quit IRC01:37
*** killer_prince has joined #openstack-neutron01:38
*** erkules_ has joined #openstack-neutron01:38
*** yamamoto has quit IRC01:40
*** gangil has quit IRC01:40
*** erkules has quit IRC01:41
*** lazy_prince has quit IRC01:42
*** britthouser has joined #openstack-neutron01:45
*** mickeys has quit IRC01:45
*** lyanchih has joined #openstack-neutron01:46
*** britthou_ has joined #openstack-neutron01:46
*** britthouser has quit IRC01:50
*** boris-42 has quit IRC01:50
*** liangbo has joined #openstack-neutron01:50
*** piyanai has joined #openstack-neutron01:51
*** britthou_ has quit IRC01:52
*** shashank_hegde has quit IRC01:52
*** blogan_ has quit IRC01:54
*** britthouser has joined #openstack-neutron01:54
*** outworlder has joined #openstack-neutron01:54
*** britthou_ has joined #openstack-neutron01:55
*** jwy has joined #openstack-neutron01:55
*** agireud has quit IRC01:58
*** SourabhP has quit IRC01:59
*** britthouser has quit IRC01:59
*** ajmiller has joined #openstack-neutron01:59
*** saksham has quit IRC02:00
*** agireud has joined #openstack-neutron02:00
*** neiljerram_bb has quit IRC02:01
*** ajmiller_ has joined #openstack-neutron02:02
*** _cjones_ has quit IRC02:02
*** shwetaap has joined #openstack-neutron02:02
*** killer_prince has quit IRC02:03
*** zhhuabj has quit IRC02:03
openstackgerritKevin Benton proposed openstack/neutron: Add object status transition barrier mechanism  https://review.openstack.org/18167402:05
*** ajmiller has quit IRC02:05
*** banix has joined #openstack-neutron02:06
*** rynn has joined #openstack-neutron02:07
*** baoli has joined #openstack-neutron02:08
*** sridhar_ram1 has quit IRC02:10
*** baoli has quit IRC02:12
*** yalie1 has joined #openstack-neutron02:14
*** zhhuabj has joined #openstack-neutron02:14
*** baojg has joined #openstack-neutron02:16
*** yalie has quit IRC02:16
openstackgerritMerged openstack/neutron: Fix tenant access to qos policies  https://review.openstack.org/21403002:17
*** dims has joined #openstack-neutron02:21
*** lyanchih has quit IRC02:21
*** vivek-ebay has quit IRC02:22
*** vivek-ebay has joined #openstack-neutron02:28
*** vivek-ebay has quit IRC02:28
*** tiny-hands has quit IRC02:29
*** sthillma has joined #openstack-neutron02:29
*** dims has quit IRC02:29
*** dims has joined #openstack-neutron02:30
*** shoutm_ has joined #openstack-neutron02:30
*** shoutm has quit IRC02:31
*** sthillma_ has joined #openstack-neutron02:33
*** dims has quit IRC02:34
*** tiny-hands has joined #openstack-neutron02:35
*** sthillma has quit IRC02:35
*** sthillma_ is now known as sthillma02:35
*** agireud has quit IRC02:35
*** yamamoto has joined #openstack-neutron02:40
*** agireud has joined #openstack-neutron02:43
*** salv-orlando has joined #openstack-neutron02:44
*** markvoelker has quit IRC02:46
*** ivar-laz_ has joined #openstack-neutron02:46
*** sbalukoff has quit IRC02:46
openstackgerritShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly  https://review.openstack.org/21493002:47
*** iwamoto has joined #openstack-neutron02:48
*** ivar-lazzaro has quit IRC02:49
openstackgerritShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly  https://review.openstack.org/21493002:49
*** ivar-laz_ has quit IRC02:50
*** britthou_ has quit IRC02:51
*** salv-orlando has quit IRC02:51
*** hakimo has joined #openstack-neutron02:52
*** yamahata has quit IRC02:53
*** agireud has quit IRC02:53
openstackgerritZhi Chang proposed openstack/neutron: Notify specify host in DVR create/update port  https://review.openstack.org/21493302:53
*** piyanai has quit IRC02:54
*** hakimo_ has quit IRC02:54
*** jwy has quit IRC02:55
*** salv-orlando has joined #openstack-neutron02:55
openstackgerritZhi Chang proposed openstack/neutron: Notify specify host in DVR create/update port  https://review.openstack.org/21493302:55
*** lyanchih has joined #openstack-neutron03:01
*** salv-orlando has quit IRC03:02
openstackgerritHenry Gessau proposed openstack/neutron: Do not query reservations table when counting resources  https://review.openstack.org/21460203:02
openstackgerritHenry Gessau proposed openstack/neutron: Fix _ensure_default_security_group logic  https://review.openstack.org/20338403:02
*** gongysh has joined #openstack-neutron03:04
*** shoutm has joined #openstack-neutron03:06
*** y_sawai has joined #openstack-neutron03:06
*** gildub has quit IRC03:07
*** shoutm_ has quit IRC03:07
*** hoangcx has quit IRC03:08
*** hoangcx has joined #openstack-neutron03:09
*** shashank_hegde has joined #openstack-neutron03:12
*** tidwellr has joined #openstack-neutron03:13
*** jwy has joined #openstack-neutron03:14
*** armax has joined #openstack-neutron03:15
*** amuller has quit IRC03:16
HenryGkevinbenton: ping03:17
HenryGkevinbenton: can you throw an eye on https://review.openstack.org/211963 ?03:17
*** y_sawai has quit IRC03:18
*** y_sawai has joined #openstack-neutron03:19
*** agireud has joined #openstack-neutron03:20
*** shivrao has joined #openstack-neutron03:21
kevinbentonHenryG: looking03:23
*** baoli has joined #openstack-neutron03:23
openstackgerritZhi Chang proposed openstack/neutron: Notify specific host in DVR create/update port  https://review.openstack.org/21493303:24
Fdaisukearmax: Hi,03:24
Fdaisukearmax: I commented to https://review.openstack.org/#/c/207328/03:25
FdaisukeWould you please have a look at this?03:26
*** saksham has joined #openstack-neutron03:28
*** baoli has quit IRC03:28
*** saksham_ has joined #openstack-neutron03:30
*** piyanai has joined #openstack-neutron03:32
*** vikram has joined #openstack-neutron03:33
*** saksham has quit IRC03:33
*** saksham_ is now known as saksham03:33
*** y_sawai has quit IRC03:35
*** rushil has joined #openstack-neutron03:36
*** Kennan2 is now known as Kennan03:36
*** jwy has left #openstack-neutron03:37
*** vivek-ebay has joined #openstack-neutron03:37
*** diga has joined #openstack-neutron03:38
*** jianyong has quit IRC03:39
*** saksham has left #openstack-neutron03:39
openstackgerritgongysh proposed openstack/python-neutronclient: Add document for entry point in setup.cfg  https://review.openstack.org/21494503:40
*** tiny-hands has quit IRC03:42
*** vivek-eb_ has joined #openstack-neutron03:43
*** vivek-ebay has quit IRC03:43
*** rushil_ has joined #openstack-neutron03:44
*** rushil_ has quit IRC03:46
*** markvoelker has joined #openstack-neutron03:46
*** gildub has joined #openstack-neutron03:46
*** baojg has quit IRC03:47
*** rushil has quit IRC03:48
*** y_sawai has joined #openstack-neutron03:48
*** banix has quit IRC03:49
*** lpetrut has joined #openstack-neutron03:51
*** markvoelker has quit IRC03:51
*** baojg has joined #openstack-neutron03:54
*** changzhi has quit IRC03:54
*** pal has joined #openstack-neutron03:55
*** klindgren has joined #openstack-neutron03:55
*** azbiswas has joined #openstack-neutron03:56
*** y_sawai has quit IRC03:56
*** piyanai has quit IRC03:58
*** shoutm_ has joined #openstack-neutron04:00
*** shoutm has quit IRC04:01
openstackgerritgongysh proposed openstack/python-neutronclient: Add document for entry point in setup.cfg  https://review.openstack.org/21494504:02
*** mkolesni_ has joined #openstack-neutron04:03
*** tiswanso has quit IRC04:05
*** achanda has quit IRC04:05
*** y_sawai has joined #openstack-neutron04:06
openstackgerritVadivel Poonathan proposed openstack/neutron: Adding Ale Omniswitch to sub_projects document  https://review.openstack.org/21400104:08
*** irenab has quit IRC04:09
openstackgerritVadivel Poonathan proposed openstack/neutron: Adding Ale Omniswitch to sub_projects document  https://review.openstack.org/21400104:10
*** gberginc has quit IRC04:10
*** irenab has joined #openstack-neutron04:11
openstackgerritKenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS  https://review.openstack.org/14841604:16
*** minwang2 has joined #openstack-neutron04:17
*** dave-mccowan has quit IRC04:22
*** lyanchih has quit IRC04:22
*** zhhuabj has quit IRC04:24
*** baoli has joined #openstack-neutron04:24
*** jianyong has joined #openstack-neutron04:24
*** jianyong has quit IRC04:24
*** zhhuabj has joined #openstack-neutron04:24
*** diga has quit IRC04:25
*** pal has quit IRC04:27
*** minwang2 has quit IRC04:28
*** baoli has quit IRC04:29
*** lpetrut has quit IRC04:31
*** fawadkhaliq has joined #openstack-neutron04:32
*** salv-orlando has joined #openstack-neutron04:32
*** agireud has quit IRC04:33
*** shivrao_ has joined #openstack-neutron04:34
*** shivrao has quit IRC04:34
*** shivrao_ is now known as shivrao04:34
*** agireud has joined #openstack-neutron04:35
*** vivek-ebay has joined #openstack-neutron04:35
*** salv-orlando has quit IRC04:35
*** shaohe_feng has quit IRC04:37
*** baojg has quit IRC04:37
*** vivek-eb_ has quit IRC04:38
*** shivrao_ has joined #openstack-neutron04:38
*** baojg has joined #openstack-neutron04:38
*** shivrao has quit IRC04:39
*** shivrao_ is now known as shivrao04:39
*** vikram has quit IRC04:41
*** lyanchih has joined #openstack-neutron04:42
openstackgerritMerged openstack/neutron: Final decomposition of ML2 Cisco UCSM driver  https://review.openstack.org/21366804:42
*** Sukhdev has joined #openstack-neutron04:43
*** jianyong has joined #openstack-neutron04:45
*** jianyong has quit IRC04:45
*** salv-orlando has joined #openstack-neutron04:47
*** Sukhdev has quit IRC04:47
*** srilatha has joined #openstack-neutron04:48
*** jianyong has joined #openstack-neutron04:49
*** jianyong has quit IRC04:49
*** shashank_hegde has quit IRC04:50
*** azbiswas has quit IRC04:51
*** salv-orlando has quit IRC04:51
*** azbiswas has joined #openstack-neutron04:52
*** vikram has joined #openstack-neutron04:54
*** azbiswas has quit IRC04:56
vikramajo: ping04:58
*** Sukhdev has joined #openstack-neutron04:58
*** ajmiller_ has quit IRC05:02
ajohi mr vikram , morning05:05
*** jianyong has joined #openstack-neutron05:05
ajokevinbenton ping: https://review.openstack.org/#/c/182920/05:05
ajoyamamoto  ^ akamyshnikova05:06
ajoakamyshnikova, I guess you're reaching exhaustion trying to get that done, it's usually a PITA to merge big patches05:07
*** sdake has joined #openstack-neutron05:07
ajothere's always something to nitpick about, or that reviewers didn't see before05:07
*** achanda has joined #openstack-neutron05:07
*** shwetaap has quit IRC05:08
ajoakamyshnikova, do you think it could take too long / too much to address the current comments?05:08
*** afazekas has joined #openstack-neutron05:08
*** shwetaap has joined #openstack-neutron05:08
ajoI guess the complicated part is that testing br-tun needs manual testing05:08
*** shwetaap has quit IRC05:09
*** y_sawai has quit IRC05:10
*** sdake_ has quit IRC05:11
vikramajo: Good Morning05:13
vikramajo: I have few more ideas about classifier05:13
ajovikram: and good afternoon for you :)05:13
ajohi, what did you think about? :)05:13
vikramajo: mentioned over the review links..05:13
vikramcan you please check once05:13
vikram:)05:13
ajoah, 1 sec vikram , I'm going over all my open reviews05:14
vikramajo: thanks..05:14
vikramhope you will like it05:14
vikram:)05:14
*** haigang has quit IRC05:14
ajovikram: which line?05:14
vikramhttps://review.openstack.org/#/c/190463/05:14
ajoyeah, yeah, there I am05:15
vikram1505:15
ajobut I'm not locating your comment05:15
ajoaahh05:15
ajoup ther05:15
ajothere05:15
ajovikram: I think that's quite firewall specific (permit/deny?)05:15
*** afazekas has quit IRC05:15
ajovikram, just a first impression not very meditated05:16
vikramajo: I was thinking that just specify classifer won't convey any meaning..05:16
ajovikram, if it's a common resources to many other services, it should not be a separate service05:16
vikramit could be permit / deny05:16
ajootherwise the inter dependencies are going to be a mess05:16
vikrami agree...05:16
vikramI was thinking how about grouping classifier's and the application can just use the group05:17
yamamotoajo: do you think merging it as-is saves efforts?  follow-ups need testing as well...05:17
ajovikram: that's interesting, what do you mean by grouping classifier?05:17
*** liangbo has quit IRC05:18
ajoyamamoto: it provides some emotional relief I guess, I changed my vote to let them do it05:18
vikrami mean let's say i need 3 classifiers... C1, C2, C305:18
ajoyamamoto: I understand your point05:18
vikramthen one API can group them and application can use group instead of C1, C2 and C305:18
*** emagana has joined #openstack-neutron05:18
*** kiran-r has joined #openstack-neutron05:18
ajoyamamoto: btw, a clean history is more important if we were thinking of backports, but I don't believe it's backportable, (I didn't try to see if codebases are compatible, but 400Ls make me guess it won't be)05:19
vikramajo: this will give more flexibility and convenience to the user/application05:19
ajovikram, I see05:20
ajovikram, grouping can become complicated05:20
ajoyou could want to do05:20
ajo(C1 and C2) or C305:20
ajo;)05:20
ajovikram, I'd leave that to a 2nd step :)05:20
yamamotoajo: i guess this patch made me insensitive of that kind of emotion...  https://review.openstack.org/#/c/153946/05:20
ajoin a future RFE05:20
vikramajo: ok05:20
ajovikram: I believe it's a good idea, but may be we should do things in smaller steps :), and think grouping wisely :D05:21
vikramajo: we can atleast mention that as a potential work in the spec05:21
*** irenab has quit IRC05:21
ajoyamamoto: reading05:21
ajoyamamoto: oouch05:21
iwamotohello05:21
ajo128 patchsets05:21
vikramajo: regarding the action.. I was thinking if the user want to deny a flow then how to do? then probably it has to define the action of it's own..05:22
*** rynn has quit IRC05:22
vikramajo: which could be also repetitive.05:22
ajoyamamoto: big patches are a PITA05:22
*** azbiswas has joined #openstack-neutron05:22
*** gsagie has joined #openstack-neutron05:23
ajoyamamoto: how's that going, is it ready?05:24
ajoyamamoto: btw, I wanted to chat with you one day about some sort of ToR switch control for the openvswitch ml2...05:25
ajoI still don't know how to model it, or do it, having ryu on board I'm sure it will help05:25
ajoyamamoto: because, basically we will need it for NFV, since they're needing to grab "whole physical cards" for some specific apps05:26
ajowhere we loose the ability to tag via SR-IOV or ovs.. and it can be done via an OF compatible ToR switch05:26
*** Sukhdev has quit IRC05:26
*** srilatha has quit IRC05:26
*** srilatha has joined #openstack-neutron05:27
ajoI guess we may need an OF controller to handle the switches05:27
*** neophy has joined #openstack-neutron05:31
yamamotoajo: the big patch of mine is almost always kept ready.  (thank you iwamoto)05:31
ajoyamamoto: I'm reading it05:31
*** srilatha has quit IRC05:31
ajoyamamoto: looking good05:31
ajoyamamoto: I guess it intersects with akamyshnikova patch a bit...05:31
yamamotoi suppose ovs-agent restart patch will go first and it requires major update though.05:32
yamamotosure.05:32
*** irenab has joined #openstack-neutron05:32
ajoyamamoto: any testing plans for this? I mean05:32
ajowhat are we going to do in the gate? :)05:32
ajomay be we can make it available this cycle05:33
ajoand enable by default starting next05:33
ajoto make sure it's heavily tested on gate05:33
yamamotoa plan is to have a separate job with it enabled05:33
ajoyamamoto, I guess we need both tested, but05:33
ajoonce this is proven to work well05:34
ajoI'd get rid of the ovs-vsctl version,05:34
ajoor there's something we can't do through ryu?05:34
yamamotoyes it's the plan05:34
iwamotoovs-vsctl will not go away with that patch05:34
ajoI guess, the only issue with that is new features need to be supported in ryu first, but I know you'd be fast then ;)05:34
iwamotoonly ovs-ofctl05:34
ajosorry05:34
ajoovs-ofctl, I was meaning that05:34
ajo:)05:34
*** mlavalle has quit IRC05:35
ajowe already have the native ovsdb driver...05:35
ajofor ovs-vsctl05:35
ajomay be we should plan that too...05:35
iwamotocurrent status -> https://review.openstack.org/#/c/206329/05:35
iwamotobut, yes, I should add some jenkins jobs05:36
ajoiwamoto: niiice, all green ;)05:36
yamamotoajo: look at "Plan:" https://github.com/openstack/neutron-specs/blob/master/specs/liberty/ovs-ofctl-to-python.rst#proposed-change05:36
ajogood job iwamoto , yamamoto !05:36
*** vivek-ebay has quit IRC05:37
ajoyamamoto: I'd say, instead of experimental, a separate job running all the time05:37
ajoso we can make sure it's stable05:37
yamamotoi agree05:37
*** azbiswas has quit IRC05:37
*** Marga_ has joined #openstack-neutron05:38
*** azbiswas has joined #openstack-neutron05:38
*** amotoki has quit IRC05:39
*** azbiswas_ has joined #openstack-neutron05:39
*** dasp has quit IRC05:40
*** jreeves has quit IRC05:41
*** tflynn has joined #openstack-neutron05:41
ajoyamamoto, iwamoto , can I ask you, what do you think about the ToR switch control needs I was talking about before?05:41
*** jprovazn has joined #openstack-neutron05:41
*** nate_gone has quit IRC05:42
*** mfranc213 has quit IRC05:42
ajo(NFV grabbing a whole physical card requirement)05:42
yamamotoajo: i'm not familiar with such a hardware05:42
*** azbiswas has quit IRC05:42
*** vichoward has quit IRC05:42
ajoyamamoto: NFV  you mean?05:42
yamamotoToR switch05:42
*** aveiga has quit IRC05:42
ajoyamamoto: I mean top of rack switch05:42
*** neophy has quit IRC05:43
ajoany top of rack switch openflow capable05:43
yamamotois it an ordinary OF switch?05:43
ajoI was meaning that, not a branch :)05:43
ajobranch->brand05:43
*** tflynn_ has joined #openstack-neutron05:43
*** shausy has joined #openstack-neutron05:44
ajoyamamoto, I guess, we'd need some agent/s to act as controllers of the switches05:44
yamamotoprobably.05:44
ajoand modeling the network to handle cases where an ext-net is then connected to a switch port..05:44
yamamotoin your model leafs are handled by ovs-agent?05:44
ajoyamamoto: I have no model yet05:45
*** tflynn has quit IRC05:45
openstackgerrityalei wang proposed openstack/neutron: Add support for unaddressed port  https://review.openstack.org/20080905:45
ajoyamamoto: just a lot of NFV crying out loud because they want to grab whole physical cards in some cases05:45
ajoyamamoto: I guess I need to sit down, and think,05:46
iwamotoit sounds a bit like vlan-aware-vms and the ongoing ironic+neutron tenant isolation effort05:46
*** dasp has joined #openstack-neutron05:46
*** aveiga has joined #openstack-neutron05:46
*** mfranc213 has joined #openstack-neutron05:46
yamamotoi thought NFV is something virtual.  i'm not sure why it want to grab hardware.  for performance reasons?05:46
*** agireud has quit IRC05:47
ajoyamamoto: yeah, I had the same question for them a few times05:47
*** yog_ has joined #openstack-neutron05:47
*** nate_gone has joined #openstack-neutron05:47
*** nate_gone is now known as njohnston05:47
ajoyamamoto: basically it seems that if they grab a single VF from the SR-IOV NIC PF (physical port)...05:47
*** zhhuabj has quit IRC05:48
ajothey get limited to only 1 queue of the hardware05:48
*** vichoward has joined #openstack-neutron05:48
ajowhile if they grab the whole card they can use several queues at the same time05:48
ajoalso05:48
yamamotodoesn't it depend on nic?05:48
ajothe cards have some limitations when handling promiscous unicast05:48
*** jreeves has joined #openstack-neutron05:48
ajoyamamoto: totally, yes...05:49
ajobut they seem to be standarizing VNFs for SR-IOV cards like intel or mellanox, or stuff like that05:49
ajothe VNF has to have all the drivers...05:49
ajo"drivers"05:49
*** bharath_ has joined #openstack-neutron05:49
ajosometimes they go via DPDK to the card05:49
ajoso as long DPDK supports the nic, they are happy05:50
*** SridharG has joined #openstack-neutron05:50
yamamotoi have to admit i can't distinguish NFV and VNF05:50
ajoNFV is the general concept of virtualizing network functions05:50
ajoand VNF is an specific nework function...05:50
ajolike an IMS, DHCP server, DNS, SIP server,05:51
ajoIDSs, ...05:51
ajothey seem to have many more than I could imagine...05:52
*** vernhart is now known as vern05:52
*** shoutm_ has quit IRC05:52
ajoyamamoto, iwamoto : I saw one telco which added a mechanism driver of their own to ml205:53
ajoand in combination with floodlight,05:53
ajothey were controlling the top of rack switches, to do vlan tagging /untagging05:53
yamamotois it opensource?05:53
*** vthapar has joined #openstack-neutron05:53
ajoyamamoto: I asked them to opensource it05:54
*** y_sawai has joined #openstack-neutron05:54
*** anilvenkata has joined #openstack-neutron05:54
yamamotoi've heard cisco does security group on non-leaf switches05:54
ajoyamamoto, what do you mean by non-leaf?05:54
*** numan has joined #openstack-neutron05:55
ajoleaf = top of rack?05:55
yamamotoleaf == hypervisor switch05:55
openstackgerritgongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line  https://review.openstack.org/21496105:55
ajoah, ok05:55
ajoyamamoto: yes, that could be an advantage of controlling the top of rack05:55
*** trinaths has joined #openstack-neutron05:55
ajowe could offload some work if we wanted05:56
ajobut, then the switch has to be powerful enough :)05:56
*** itzikb has joined #openstack-neutron05:56
ajomay be the current design distributes the work better, but loads the hypervisor05:56
*** aranjan has joined #openstack-neutron05:56
*** pal has joined #openstack-neutron05:57
ajoand definitely, we should not force openstack users to have openflow switches,05:57
openstackgerritgongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line  https://review.openstack.org/21496105:57
ajoping gongysh  :)05:57
openstackgerritKevin Benton proposed openstack/neutron: l2pop: check port mac in pre-commit to stop change  https://review.openstack.org/21447405:57
ajogongysh: I saw you took a qos bug, thanks, will you work on it soon? :) we need to clear it out before the end of the cycle05:58
openstackgerritMerged openstack/neutron-fwaas: Reduce contention between update and delete  https://review.openstack.org/21281105:58
ajomerges++05:58
*** kiran-r has quit IRC05:59
*** armax has quit IRC06:00
*** zhhuabj has joined #openstack-neutron06:00
*** y_sawai has quit IRC06:00
gsagieHello, i am trying to understand something, i see that in db_base_plugin_v2 , create_network can recieve 'id' from the caller (if not it generates one), but from the CLI when i try to create network with specific id i get this:"Attribute 'id' not allowed in POST"  where is this blocked? and why?06:02
gsagieis this just for internal networks being created in the DB layer?06:02
*** lukasa has joined #openstack-neutron06:03
*** azbiswas_ has quit IRC06:04
iwamotogsagie: neutron/api/v2/attributes.py RESOURCE_ATTRIBUTE_MAP ?06:04
*** azbiswas has joined #openstack-neutron06:05
gsagieiwamoto: thanks, does anyone know why creating network with specific 'id' is blocked from the API level but allowed in the DB layer? i am interested to know why its blocked :)06:05
*** salv-orlando has joined #openstack-neutron06:05
*** outworlder has quit IRC06:05
*** y_sawai has joined #openstack-neutron06:05
*** shoutm has joined #openstack-neutron06:07
irenabajo: with PF per VNF, how PF is supposed to be managed?06:08
ajoirenab, they're working with the nova guys and the scheduler to handle that06:09
*** moha_hunt has quit IRC06:09
fawadkhaliqgsagie: don't know the reason why Neutron initial design took this approach but normally for POST, server decides the location of the resource and returns the address (uuid) and let the server manage consistent way of doing addresses.06:09
*** m0h4_hun7 has joined #openstack-neutron06:09
ajoirenab: they will need to allocate only PFs with no used VFs06:09
* iwamoto scared by a lot of NFV acronyms06:09
*** azbiswas has quit IRC06:09
*** amotoki has joined #openstack-neutron06:09
ajoiwamoto: yes, hard to get... I know :/06:09
irenabajo: I understand this, but once you give PF for VNF, host does not have control over it06:10
ajoiwamoto: in SR-IOV:   PF = physical port of a card,   VF= virtual network card (with pci mem map) that goes to a PF06:10
irenabI think you will have SR-IOV disabled to support PF=> VNF06:10
gsagiefawadkhaliq: yeah, but its already allowed in the DB layer, so you could call create_network with specific 'id' internally but you cant do it from the API, but just wondering i do want to propose allowing adding metadata for networks06:10
ajoiwamoto: so a card can be composed of n PF's (so it has n physical ports) , and each PF has y VFs ( y virtual network cards a VM/instance can grab)06:10
fawadkhaliqgsagie: but preferred way should if location of resource is unknown, let the server decides the URL06:11
ajoirenab: that's the thing06:11
fawadkhaliqs/decides/decide/06:11
ajoirenab: host does not have any control, so you could only control segmentation if you can control the top of rack switch06:11
ajoirenab: you say the switch: tag this port as VLANX, ... or send this port to tunnel Y06:11
ajoirenab: that's what they were doing in their mechanism driver, via floodlight, and floodlight talked to their switches06:12
yamamotogsagie: it's this way so that plugin can provide id.  eg taken from its backend.06:12
ajoirenab: but their thing is just a POC, they're looking for upstream support on that06:12
gsagieyamamoto: yeah, i am wondering why not to allow it in the API level aswell06:12
ajoirenab: I will try to get their code open so we can all read and think06:12
*** reedip_ has quit IRC06:13
*** yanxb has joined #openstack-neutron06:15
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process  https://review.openstack.org/21470106:15
gsagiewhy not to allow an orchestration system that wants to pick its own network id's to be able to do it from the API level06:15
*** salv-orlando has quit IRC06:16
irenabajo: management of segmentations via ToR makes sense in such case06:16
ajoirenab: otherwise you make all the VNFs vendors to accept your "segmentation parameters" somehow06:17
yamamotogsagie: no idea.  it would simplify kuryr a bit.06:17
ajoI guess for VLANs that could be ok, but other types of segmentations could get trickier06:17
*** aranjan has quit IRC06:18
gsagieyamamoto: yes, Kuryr might need an additional metadata as well, but it can also help multi cloud solutions06:18
*** neophy has joined #openstack-neutron06:18
irenabajo: agree06:19
gsagieajo: thats what hierarchical port binding is for i believe06:20
openstackgerritShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly  https://review.openstack.org/21493006:20
gsagieyou dont need to change the mechanism driver for OVS, just add the additional binding for the ToR06:20
*** njohnston has quit IRC06:20
ajogsagie: yeah, we could leverage that, afaik hierarchical port binding is only supported by cisco :)06:21
ajogsagie: I need to dive into hierarchical port binding a bit btw...06:21
*** afazekas has joined #openstack-neutron06:21
vikramajo: I registered a spec for that log back:)06:21
*** shashank_hegde has joined #openstack-neutron06:21
ajobut that was an impression I had too06:21
vikram*long06:21
ajovikram++06:21
ajo:)06:21
gsagieajo: it doesnt matter, its the fact that you can do it in ML2, all you need is first bind OVS plugin then just communicate with the ToR the mapping of the segmentations06:21
*** vichoward has quit IRC06:21
*** dasp has quit IRC06:21
*** nyechiel_ has joined #openstack-neutron06:22
vikramajo: involve me as well ++06:22
*** baojg has quit IRC06:22
gsagieso you just need (as far as i understand) add another mechanism driver that can communicate with the ToR06:22
*** jreeves has quit IRC06:22
*** mfranc213 has quit IRC06:22
ajogsagie: I will re-read hierarchical port binding again, I need to refresh my knowledge06:22
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process  https://review.openstack.org/21470106:22
gsagieajo: np, let me know if you need any help06:22
*** aveiga has quit IRC06:23
ajogsagie: I won't tackle it now, not enough bandwidth06:23
ajobut may be something I will look in next cycle06:23
gsagieok, thats an area i am also looking at right now06:23
ajogsagie, for OVN, right? :)06:23
ajoor dragonflow?06:23
*** pal has quit IRC06:23
*** tidwellr has quit IRC06:23
gsagiebecause thats getting to be a very common deployment these days06:23
*** baojg has joined #openstack-neutron06:24
gsagieajo: Dragonflow, OVN since its not a mechanism driver can support these kind of deployments using VTEP schema only, cant leverage hierarchical port binding06:24
* ajo watches: hierarchical port binding presentation: https://www.youtube.com/watch?v=Pk5bN5Fs_Vk06:24
*** Murali has joined #openstack-neutron06:26
openstackgerritvenkata anil proposed openstack/neutron-vpnaas: Support VPNaaS with L3 HA  https://review.openstack.org/20063606:27
openstackgerritLiang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents  https://review.openstack.org/21496906:27
*** pal has joined #openstack-neutron06:27
*** gangil has joined #openstack-neutron06:27
*** dasp has joined #openstack-neutron06:28
*** nate_gone has joined #openstack-neutron06:28
*** aveiga has joined #openstack-neutron06:28
*** nate_gone is now known as njohnston06:28
*** vichoward has joined #openstack-neutron06:29
*** jreeves has joined #openstack-neutron06:29
*** vikram has quit IRC06:30
*** mfranc213 has joined #openstack-neutron06:31
ajogsagie: now that I see, it's a similar thing, but it's not the same06:31
ajothey dynamically allocate vlans on the ToR switch to handle a tenant net traffic06:31
ajoand then interconnect on the upper layer of switches06:32
ajohere06:32
*** salv-orlando has joined #openstack-neutron06:32
ajothey just want to make sure a physical port is connected to a specific network, that's all06:32
*** shaohe_feng has joined #openstack-neutron06:32
ajoI'm probably still missing details of this hierarchical port binding thing06:33
* ajo keeps watching video06:33
openstackgerritMohankumar proposed openstack/python-neutronclient: Adding registration interface for non_admin_status_resources  https://review.openstack.org/21419306:34
*** vikram has joined #openstack-neutron06:35
*** stanzgy has quit IRC06:35
*** mohankumar has joined #openstack-neutron06:35
*** jreeves has quit IRC06:35
ajogsagie: It's very similar, I need to dig my head into it deeper...06:35
openstackgerritTu Hong Jun proposed openstack/neutron: Changed filter field to router_id  https://review.openstack.org/21497406:36
*** aveiga has quit IRC06:36
*** stanzgy has joined #openstack-neutron06:36
*** dasp has quit IRC06:36
*** haigang has joined #openstack-neutron06:37
*** mfranc213 has quit IRC06:37
*** vichoward has quit IRC06:37
*** njohnston has quit IRC06:37
*** vikram has quit IRC06:37
*** afazekas has quit IRC06:37
*** liangbo has joined #openstack-neutron06:38
*** nlahouti has joined #openstack-neutron06:39
*** nlahouti1 has joined #openstack-neutron06:41
gsagieajo: what are you trying to do? what is the use case? from what i know these deployments usualy connect the host to the ToR with VLAN and do the encapsulation to VXLAN in the ToR (so you save encapsulation peformance)06:41
*** reedip has joined #openstack-neutron06:41
*** jreeves has joined #openstack-neutron06:42
*** aveiga has joined #openstack-neutron06:42
*** mfranc213 has joined #openstack-neutron06:42
*** dasp has joined #openstack-neutron06:42
ajogsagie: VNF grabs a whole card (PF)06:42
*** nate_gone has joined #openstack-neutron06:43
*** nate_gone is now known as njohnston06:43
ajogsagie: we need to tag/tunnel untag/untunnel the specific port of the switch to still provide segmentation to that VNF interface06:43
*** vichoward has joined #openstack-neutron06:43
*** vikram has joined #openstack-neutron06:43
ajogsagie: we can only do it via the top of rack switch, or with VNF cooperation..06:43
*** nlahouti has quit IRC06:44
ajoa hierarchy supporting hierarchical port binding, probably could do it easily, I guess, I need to look into that,06:44
*** ddepaoli has joined #openstack-neutron06:44
ajobut for other types of network arrangements,  if you want to do this, you couldn't06:45
*** fawadkhaliq has quit IRC06:45
*** y_sawai has quit IRC06:45
*** nlahouti1 has quit IRC06:45
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/21447006:45
gsagieajo : so you tackling the mixed setups with SR-IOV if i understand correctly06:46
gsagie(which i am doing as well :))06:46
*** y_sawai has joined #openstack-neutron06:47
*** vikram has quit IRC06:48
*** moshele has joined #openstack-neutron06:48
ajogsagie: yep06:48
*** vikram has joined #openstack-neutron06:48
ajoSR-IOV -> grab a whole PF06:48
ajonova is working on it already06:48
ajoon our side, is just adding another type of interface06:49
ajobut then, the traffic would be unsegmented unless your plugin is able to do that06:49
ajo:)06:49
*** nlahouti has joined #openstack-neutron06:49
*** nlahouti has quit IRC06:49
*** gangil has quit IRC06:50
gsagieajo: why cant you use VLANS?06:50
ajogsagie: because the VM grabs the PF, so host loses control over setting a vlan to the PF06:51
ajogsagie: the VM grabs the whole PCI map for the card port... even if you set it before handing it down to the VM, it will be reset by the VM driver initialization...06:52
ajoand also, only nova could do that,06:52
ajoonce the pci address space is mapped to the VM, we (neutron) cannot touch it anymore06:52
*** scheuran has joined #openstack-neutron06:52
*** anshul has joined #openstack-neutron06:53
*** jbell8 has quit IRC06:53
*** vikram is now known as vikschw06:55
*** vikschw is now known as vikram06:55
*** jprovazn has quit IRC06:56
openstackgerritgongysh proposed openstack/python-neutronclient: Clear the extension requirement  https://review.openstack.org/21498106:56
*** y_sawai has quit IRC06:57
*** vikram is now known as vikschw06:57
*** vikschw is now known as vikram_choudhary06:58
*** Fdaisuke has quit IRC06:58
*** Fdaisuke has joined #openstack-neutron06:59
*** vikram_choudhary is now known as vikschw06:59
*** vikschw has quit IRC07:00
*** achanda has quit IRC07:00
*** ronis_ has joined #openstack-neutron07:01
openstackgerritgongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line  https://review.openstack.org/21496107:01
*** outworlder has joined #openstack-neutron07:01
*** klindgren has quit IRC07:03
*** ronis__ has joined #openstack-neutron07:03
*** ronis_ has quit IRC07:06
*** y_sawai has joined #openstack-neutron07:07
openstackgerritgongysh proposed openstack/python-neutronclient: Clear the extension requirement  https://review.openstack.org/21498107:07
*** vikram_choudhary has joined #openstack-neutron07:07
*** m0h4_hun7 has quit IRC07:08
*** fawadkhaliq has joined #openstack-neutron07:09
*** vikram_choudhary is now known as vikram07:10
*** vikram has left #openstack-neutron07:10
*** outworlder has quit IRC07:10
*** vikram_choudhary has joined #openstack-neutron07:15
*** tflynn_ has quit IRC07:17
*** vikram_choudhary has quit IRC07:18
*** claudiub has joined #openstack-neutron07:19
*** kawa2014 has joined #openstack-neutron07:19
*** yamamoto has quit IRC07:19
openstackgerritOpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/20442207:20
*** vikram_choudhary has joined #openstack-neutron07:22
*** Murali_ has joined #openstack-neutron07:23
*** vikram_choudhary is now known as Vikram07:23
*** Vikram is now known as vikram07:23
*** jschwarz has joined #openstack-neutron07:23
openstackgerritLiang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents  https://review.openstack.org/21496907:25
*** Murali has quit IRC07:25
*** Murali_ is now known as Murali07:25
*** SourabhP has joined #openstack-neutron07:26
*** gardenshed has joined #openstack-neutron07:28
*** SourabhP_ has joined #openstack-neutron07:29
*** Murali has quit IRC07:30
*** Murali_ has joined #openstack-neutron07:30
*** Murali_ is now known as Murali07:30
*** neophy has quit IRC07:31
*** SourabhP has quit IRC07:32
*** SourabhP_ is now known as SourabhP07:32
openstackgerritOleg Bondarev proposed openstack/neutron: Fix _ensure_default_security_group logic  https://review.openstack.org/20338407:32
*** pal has quit IRC07:32
openstackgerritMoshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy  https://review.openstack.org/21421207:33
jschwarzajo, will rebase asap :)07:33
ajojschwarz: I'm re-reviewing to make sure I don't come with more nitpicking later07:34
*** jlibosva has joined #openstack-neutron07:34
jschwarzajo, ack. so I'll wait until your go-ahead so I don't do double the job :)07:34
ajoack jschwarz :)07:35
*** nyechiel_ has quit IRC07:36
*** nyechiel_ has joined #openstack-neutron07:36
*** tfukushima has quit IRC07:38
ajojschwarz: 1st patch looks good now07:40
ajore-reading 2nd07:40
*** salv-orlando has quit IRC07:40
*** yamamoto has joined #openstack-neutron07:40
*** mohankumar has quit IRC07:41
*** sthillma has quit IRC07:42
*** ushkalim has joined #openstack-neutron07:43
jschwarzajo, ack07:43
jschwarzbtw it would seem that the gate is clogged again07:43
*** SourabhP has quit IRC07:44
jschwarzajo, rebased locally, waiting for your go-ahead for submission07:46
*** jpena has joined #openstack-neutron07:48
ajojschwarz: ok, reviewed07:48
ajojschwarz, go go :)07:48
jschwarzajo, done07:48
openstackgerritJohn Schwarz proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2).  https://review.openstack.org/19827707:48
openstackgerritJohn Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2).  https://review.openstack.org/18965507:48
ajothanks jschwarz07:48
jschwarzyou're most welcome ajo07:48
*** markvoelker has joined #openstack-neutron07:48
*** fedexo has quit IRC07:48
openstackgerritLiang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents  https://review.openstack.org/21496907:49
Fdaisukeexit07:52
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart  https://review.openstack.org/18292007:52
*** Fdaisuke has quit IRC07:52
*** markvoelker has quit IRC07:53
*** salv-orlando has joined #openstack-neutron07:55
ajojschwarz: https://review.openstack.org/#/c/189655/30/neutronclient/v2_0/client.py07:56
ajocan you check a comment07:56
ajoit's a nitpick btw, may be I'm wrong, probably needed for dict extraction?07:56
*** gberginc has joined #openstack-neutron07:57
jschwarzajo, aye we do07:57
jschwarzsince the API returns the resource as 'policies' and not 'qos_policies'07:57
jlibosvaajo: o/07:58
ajoyep, jschwarz, you're right07:58
ajojschwarz++07:58
*** ihrachyshka has joined #openstack-neutron07:58
jschwarzXD07:58
ihrachyshkaajo, jlibosva, remember that glitch in alembic when our functional test failed to detect difference in foreign key constraint in terms of cascade delete? well it's fixed in 0.8.1: http://alembic.readthedocs.org/en/latest/changelog.html#change-0fbe5d57da5d0fc69adb44615b46a83f07:59
*** y_sawai has quit IRC07:59
ihrachyshkazzzeek is freaking quick07:59
openstackgerritgongysh proposed openstack/python-neutronclient: Clear the extension requirement  https://review.openstack.org/21498107:59
ihrachyshkaI suspect we may be broken now in gate?..07:59
ihrachyshkalet me check :)07:59
jlibosvagreat!07:59
jlibosvayeah, I wanted to ask :D07:59
ajoihrachyshka: I was just suspecting that07:59
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart  https://review.openstack.org/18292008:00
ihrachyshkaah well, it's not released. good.08:00
ajoprobably lots of bad FKs around models/migrations08:00
*** yamahata has joined #openstack-neutron08:00
ajoihrachyshka, what's the plan, checking locally by getting it ?08:00
ihrachyshkaI guess it would be great to get the version from git and check it08:00
jlibosvaI'll do that08:00
*** tfukushima has joined #openstack-neutron08:00
ihrachyshkajlibosva, tnx!08:00
ajojlibosva++08:00
ajoone day I'm going to spend 1h setting up a karma bot here08:00
ajoif freenode allows that08:01
jschwarzihrachyshka, can you check the comment I made here: https://review.openstack.org/#/c/210830/5/neutron/tests/fullstack/resources/environment.py08:01
jschwarzand tell me if it makes sense? :)08:01
jschwarzajo, why wouldn't it allow a karma bot?08:02
jschwarzthey already have bots such as openstackgerrit ...08:02
*** diga has joined #openstack-neutron08:02
ihrachyshkajschwarz, agreed and I wanted to make similar point yesterday when I saw smth like l2_agent=False in test_connectivity08:02
ihrachyshkale*08:02
ihrachyshkal3*08:02
jschwarzihrachyshka, :)08:02
jschwarzI'll add it to the patch then08:03
ihrachyshkaajo, it should be infra wide bot ;)08:03
ihrachyshkajschwarz, cool08:03
*** shivrao has quit IRC08:04
openstackgerritJohn Schwarz proposed openstack/neutron: Add EnvironmentDescription, pass it down  https://review.openstack.org/21083008:05
jschwarzihrachyshka, so ^ is the next patch up the fullstack chain... hopefully you'll find the time to review it and +2 it ;-)08:05
ajoihrachyshka: let's test something locally, then let's suggest it to infra08:05
ajoihrachyshka: so they can see "the fun of it" ;)08:06
ajoakamyshnikova: https://review.openstack.org/#/c/182920/56/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py  <-- question08:06
*** lori|away is now known as lori08:06
*** outworlder has joined #openstack-neutron08:06
*** iwamoto has quit IRC08:06
ajowhat compatibility reasons?08:06
ajomay be some cooperating ovs-daemons install canary rules to check when we restarted?, but those would be gone by not having the latest cookie08:07
ihrachyshkaajo, makes sense, just note that it should probably be based on the bot code they already have08:10
ajoihrachyshka, what's the bot they use?08:10
*** sridhar_ram has joined #openstack-neutron08:11
ajoI think they have several08:12
ajomeetbot, openstackstatus...08:12
ajo-openstackstatus CTCP- VERSION Python irc.bot (8.5.4)08:13
*** yalie1 has quit IRC08:13
*** DWfuturetec has joined #openstack-neutron08:15
*** outworlder has quit IRC08:15
jschwarzajo can write a bot with python code on his free time ;)08:15
*** sridhar_ram has quit IRC08:15
*** devvesa has joined #openstack-neutron08:16
ajojschwarz: I'd rather start by using an existing one :)08:16
ajojschwarz: I had time when I was young to write my own IRC bot in C....08:16
jschwarzpss08:17
ajoIf i was able to read such code now, I would certainly puke :D08:17
jschwarzthat ought to have taken time08:17
*** yalie has joined #openstack-neutron08:17
jschwarzI once wrote a tetris in C++ and OpenGL08:17
akamyshnikovaajo, see enikanorov respond in  https://review.openstack.org/#/c/182920/56/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py08:17
ajojschwarz: lots of learnings :)08:17
*** afazekas has joined #openstack-neutron08:18
ajoakamyshnikova: thanks :)08:18
ajoakamyshnikova, I guess it makes sense, I just trust the approach quite much I didn't thought that was necessary08:18
ajoI'm fine with it08:19
matttis there a straightforward way of knowing, after upgrading neutron, if there are pending db migrations to apply?08:19
ajoihrachyshka ^08:19
jschwarzmoshele, reviewed https://review.openstack.org/#/c/214212/ - sorry for the -1 :(08:20
moshelejschwarz: you are a human pep8 :)08:21
ihrachyshkamattt, compare neutron-db-manage current output with neutron-db-manage branches08:21
jschwarzmoshele, like you wouldn't believe ;-)08:21
moshelejschwarz: ok fixing08:21
jschwarzty08:22
*** lyanchih has quit IRC08:22
matttihrachyshka: not seeing anything for 'branches', https://gist.github.com/mattt416/89abd52677381ff4e93d08:23
*** lyanchih has joined #openstack-neutron08:23
*** agireud has joined #openstack-neutron08:23
*** fawadkhaliq has quit IRC08:23
*** fawadkhaliq has joined #openstack-neutron08:26
*** gildub has quit IRC08:26
openstackgerritMoshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy  https://review.openstack.org/21421208:27
*** yalie has quit IRC08:28
*** jistr has joined #openstack-neutron08:28
matttihrachyshka: i'd love to be able to determine if i have outstanding expand/contract migrations, so i can either press ahead and do expand migrations w/ neutron running and then shut down neutron and do contract ones08:29
ihrachyshkamattt, I am looking into why it may not work08:29
*** shoutm has quit IRC08:29
matttihrachyshka: thank you, let me know if i can provide you any information, i'm using neutron from commit da1ea9a45595c8170e908fc5533a0dace997ed3308:30
*** l6unchpad has joined #openstack-neutron08:30
*** l6unchpad has left #openstack-neutron08:31
*** l6unchpad has joined #openstack-neutron08:31
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos  https://review.openstack.org/21319408:31
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py  https://review.openstack.org/21501208:31
ihrachyshkamattt, I think I am either wrong about what branches is expected to do, or there is a bug08:32
ihrachyshkamattt, for the time being, you can compare 'current' output with neutron/db/migration/alembic_migrations/versions/HEADS contents (but yeah, it sucks. it also seams that 'current' does not output 'kilo' head)08:32
openstackgerritJakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted  https://review.openstack.org/21421508:32
*** agireud has quit IRC08:33
openstackgerritJakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted  https://review.openstack.org/21421508:33
ihrachyshkamattt, ok, fo the issue of no kilo head in 'current' output, we have a patch: https://review.openstack.org/#/c/204551/08:34
*** yalie has joined #openstack-neutron08:34
ihrachyshkamattt, it failed in gate, I pushed it now.08:35
*** sayali has quit IRC08:35
ihrachyshkamattt, still looking at why branches does not do what I thought it does08:35
*** SourabhP has joined #openstack-neutron08:35
*** sayali has joined #openstack-neutron08:37
*** pal has joined #openstack-neutron08:37
*** sambetts has quit IRC08:38
openstackgerritMerged openstack/python-neutronclient: Add document for entry point in setup.cfg  https://review.openstack.org/21494508:38
matttihrachyshka: ok, thank you ... a side question, if i 'upgrade heads', do i actually have to stamp or does that happen as part of the upgrade?08:38
ihrachyshkamattt, it should stamp. jlibosva ^^ correct?08:39
ihrachyshkamattt, btw to upgrade just expand, do upgrade liberty_expand@head08:39
*** sambetts has joined #openstack-neutron08:39
matttihrachyshka: ah!  cool, thank you08:39
ihrachyshkamattt, some docs: http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html#applying-database-migration-rules08:41
*** shashank_hegde has quit IRC08:41
*** aix has joined #openstack-neutron08:41
jlibosvaI confirm08:41
ihrachyshkamattt, it's in my todo for L to implement expand and contract commands to avoid liberty/head notions08:42
*** scheuran has quit IRC08:44
*** shoutm has joined #openstack-neutron08:44
jlibosvamattt: you can verify by looking into alembic_versions table in neutron database08:44
jlibosvathere is also a command for neutron-db-manage which I forgot08:44
*** DWfuturetec has quit IRC08:45
ihrachyshkajlibosva, it's current, but it is broken a bit now08:45
jlibosvayes! that one08:45
*** apuimedo has joined #openstack-neutron08:46
*** vikram has quit IRC08:46
*** rossella_s has joined #openstack-neutron08:52
*** shoutm has quit IRC08:54
*** shoutm has joined #openstack-neutron08:55
*** jpena_ has joined #openstack-neutron08:55
jlibosvaihrachyshka: ajo about what was said yesterday in qos mtg - should we mark qos bugs with bp/<topic> ? Wouldn't be better to use bug/<number> and prioritize bug to be in L-# ?08:55
jlibosvaif I understood that correctly08:55
*** jpena has quit IRC08:56
ihrachyshkajlibosva, well, it depends on whether it's critical for l08:57
ihrachyshkajlibosva, if it is, I think it's better to solve in scope of bp08:57
salv-orlandoihrachyshka is right as usual. The bottom line imho is that you should ask yourself whether this break or not some fundamental QoS API workflow08:59
salv-orlandoif not -> bp scope, if yes -> bug and prioritize08:59
*** nyechiel_ has quit IRC08:59
ihrachyshkaHenryG, I try to understand what is the intent of 'branches' command in db cli. seems like it will always show us nothing (since we don't allow branch splits)09:00
ihrachyshkasalv-orlando, I think it's vice versa: if it breaks, then bp scope, otherwise a bug?09:01
jlibosvagot it :)09:01
*** vikram has joined #openstack-neutron09:01
salv-orlandoihrachyshka: I am seeing the world upside down maybe, but why would you not prioritize fixing something broken, moving the patch in the bp scope?09:02
*** jamie_h has joined #openstack-neutron09:02
ihrachyshkasalv-orlando, I will do it. I think we are actually on common ground, though I read your 'if not -> bp scope, if yes -> bug and prioritize' as you suggesting to fix major issues as bugs09:03
*** yamamoto has quit IRC09:04
*** 7GHAAZ22K has joined #openstack-neutron09:04
ihrachyshkaI would expect that bp has a higher prio than a bug, since it shows up in reviewer dashboard etc. But I may be wrong. Anyway, seems like in the end I don't mind how we marked a patch as long as it's merged in time09:05
ihrachyshkaakamyshnikova, hi09:05
salv-orlandoihrachyshka: I see... that's the reviewer perspective then ;)09:05
*** SourabhP has quit IRC09:05
salv-orlandowhen I was one of those +2 people, it would put high/critical bugs on top of my review list09:05
ihrachyshkaakamyshnikova, in db cli, we have 'branches' command. but it does not do anything because we don't use branch points09:05
*** jpena_ is now known as jpena09:06
salv-orlandoihrachyshka: followed by essential/high blueprints09:06
ihrachyshkasalv-orlando, aye, I guess it's my reviewer glitch, sorry09:06
salv-orlandoand then everything else - ie: the stuff I ended up not reviewing at all09:06
*** nyechiel_ has joined #openstack-neutron09:08
*** fawadkhaliq has quit IRC09:08
*** sbalukoff has joined #openstack-neutron09:09
*** apuimedo is now known as apuimedo|away09:10
*** shaohe_feng has quit IRC09:10
*** l6unchpad has quit IRC09:11
*** l6unchpad has joined #openstack-neutron09:12
*** john-davidge has joined #openstack-neutron09:13
*** shoutm has quit IRC09:15
*** john-davidge_ has joined #openstack-neutron09:15
*** john-davidge has quit IRC09:17
*** john-davidge_ is now known as john-davidge09:17
*** sdake_ has joined #openstack-neutron09:18
ihrachyshkasalv-orlando, hm, quick question re https://review.openstack.org/#/c/204551/09:18
ihrachyshkasalv-orlando, now that I started to dig thru branches/current code, it occurs to me that we actually may redefine the command meaning here.09:18
ihrachyshkasalv-orlando, alembic current shows the db state, while alembic heads show the state of migration tree, no?09:19
*** jistr has quit IRC09:19
akamyshnikovaihrachyshka, seems it became useless now09:20
akamyshnikovaihrachyshka, where is it used?09:20
salv-orlandoakamyshnikova: yes the patch might have become useless if the rest of the migration CLI evolved in the meanwhile09:20
ihrachyshkaakamyshnikova, yeah, I think we should kill the cli and instead make another validation that will walk thru revisions and make sure we have no branch points.09:20
*** jistr has joined #openstack-neutron09:20
ihrachyshkasalv-orlando, I think akamyshnikova referred to branches cli, not the patch :)09:21
salv-orlandobut maybe I'm not following09:21
salv-orlandoihrachyshka: ah I see09:21
salv-orlandothe 'heads' command is still useufl09:21
*** sdake has quit IRC09:21
salv-orlandobecause it tells you the head for each branch09:21
salv-orlandoand ihrachyshka you're right - current is supposed to show the state in the DB09:22
*** bharath_ has quit IRC09:22
ihrachyshkasalv-orlando, it is, I mostly wonder whether we actually break current09:22
ihrachyshkameh.09:22
salv-orlandoso perhaps renaming current -> heads is a terrible idea09:22
ihrachyshkaaye, I was a bad reviewer who just assumed it's what it looked like to me instead of checking myself. sigh.09:23
ihrachyshkalet's block it in gate now and think how to proceed.09:23
salv-orlandoihrachyshka: I think there is a chance current might be broken in alembic as it's not ready to deal with multiple heads? what do you reckon akamyshnikova?09:23
ihrachyshkasalv-orlando, yeah, I see 'current' does not show 'kilo' in output09:24
*** y_sawai has joined #openstack-neutron09:24
salv-orlandoihrachyshka: because the db table alembic_version does not contain that unless you have a continuos deployment that was first upgraded to kilo and then to head09:24
salv-orlandoif you start a devstack deployment post-kilo you never see a "kilo' revision, so you totally ignore the kilo branch09:25
salv-orlandowhich is fine.09:25
salv-orlandoIt's just that I find weird that it's not easy to associate a head id to its branch09:25
*** neophy has joined #openstack-neutron09:25
ihrachyshkasalv-orlando, every revision has branch_label09:26
ihrachyshkathat is inherited from any other revision in the branch that has it set09:26
openstackgerritOleg Bondarev proposed openstack/neutron: Fix _ensure_default_security_group logic  https://review.openstack.org/20338409:27
vikramihrachyshka: ping09:27
vikramihrachyshka: what changes are required for "https://bugs.launchpad.net/neutron/+bug/1485537"09:27
openstackLaunchpad bug 1485537 in neutron "adopt oslo_config.fixture.Config" [Undecided,New] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)09:27
ihrachyshkasalv-orlando, if I upgrade to kilo, then to all heads, I still miss kilo09:27
ihrachyshkavikram, I think it will belong to neutron/tests/base.py mostly. we have config_parse and friends there that may need modification to use the fixture09:28
*** mohankumar has joined #openstack-neutron09:28
vikramihrachyshka: ok09:29
ihrachyshkacurrently we just reuse cfg.CONF there, but we need to use the fixture that is based on the cfg.CONF09:29
salv-orlandoihrachyshka: let me check a bit the branch history09:30
akamyshnikovasalv-orlando, we need to check this with Mike, as it seems expected to work09:31
ihrachyshkasalv-orlando, I guess it's alembic that should have put kilo in the table?09:31
salv-orlandoihrachyshka: so both liberty/contract and liberty/expand branch off kilo. So when you run the first migration for any one of these branches I think alembic overwrite kilo in the table alembic_revision09:31
ihrachyshkait drops it once we apply a revision that depends_on kilo09:31
salv-orlandobecause as a matter of fact you have no element of your schema that belongs anymore to the kilo branch...09:32
salv-orlandoat least that's my interpretation09:32
ihrachyshkasalv-orlando, in fact, there is no kilo branch09:32
*** yamahata has quit IRC09:32
ihrachyshkawell, there is one, but it's not even labeled09:32
*** britthouser has joined #openstack-neutron09:32
salv-orlandoif you added a migration in this kilo branch then probably alembic_revision would list it once you upgraded the schema09:33
salv-orlandobut I'm speculating09:33
salv-orlandoare we ever going to add anything to that kilo branch? Can we consider kilo a branchpoint, and the kilo branch dead with kilo revision id itself?09:33
*** britthou_ has joined #openstack-neutron09:33
ihrachyshkaI guess you are right. but it's weird that script.get_heads() return kilo too09:34
ihrachyshkasalv-orlando, you mean, we switch expand/contract roots from depends_on to down_revision?09:35
salv-orlandono not really09:35
salv-orlandoI was just saying whether kilo for us is an active branch or just a branch representing the sequence of reevision until the expand/contract branchpoint09:36
salv-orlandoihrachyshka: I do not think we consider it an active branch, do we?09:36
*** jpena has quit IRC09:36
ihrachyshkasalv-orlando, we don't09:36
ihrachyshkasalv-orlando, so once I switch roots to down_revision = 'kilo', both current and script.get_heads() that we put into HEADS file now do not contain kilo09:37
salv-orlandoihrachyshka: but we have no way to tell that alembic. Because if I wanted I could go ahead and add a new migration in that branch09:37
*** openkarma has joined #openstack-neutron09:37
*** britthouser has quit IRC09:37
openstackgerritJohn Davidge proposed openstack/neutron: L3 agent changes and reference implementation for IPv6 PD  https://review.openstack.org/18597709:37
ihrachyshkasalv-orlando, you could do it, yes. in fact, it could be a way to backport db migrations09:37
ihrachyshkathough it would require conditionals in master09:38
*** openkarma has quit IRC09:38
ihrachyshkaif we are afraid of branching, we may just forbid branch points expect for specific release revisions09:38
ihrachyshka*except09:38
jlibosvamoshele: I'm looking at https://review.openstack.org/#/c/214212/5/neutron/tests/api/test_qos.py09:39
salv-orlandoihrachyshka: so by using down_revision = kilo for the root branch you can actually tell alembic that the head you're branching off is not anymore an active branch?09:39
ihrachyshkaI actually now think that down_revision = 'kilo' is the way to go.09:39
jlibosvamoshele: have you experienced inconsistency between what is returned by create_qos_bw_limit_rule and list_bw_limit_rules?09:39
ihrachyshkasalv-orlando, well, not really. alembic will still allow to branch off kilo09:39
jlibosvamoshele: I did in fullstack job and I'm surprised the api test works09:39
ihrachyshkasalv-orlando, but we can add a validation to check_migration to forbid it09:39
moshelejlibosva: why?09:40
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994309:40
jlibosvamoshele: I saw list_bw_limit_rules returns 'type' in the response while create doesn't09:40
jlibosvamoshele: ohh09:40
jlibosvamoshele: sorry, I mixed that09:40
jlibosvamoshele: disregard, sorry for bothering09:41
moshelejlibosva: np09:41
* jlibosva hides09:41
*** gberginc has quit IRC09:41
*** y_sawai has quit IRC09:42
salv-orlandoihrachyshka: yes that makes sense09:42
*** vikram has quit IRC09:42
*** openkarma has joined #openstack-neutron09:44
*** openkarma has quit IRC09:44
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py  https://review.openstack.org/21501209:44
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos  https://review.openstack.org/21319409:49
*** jpena has joined #openstack-neutron09:49
*** markvoelker has joined #openstack-neutron09:49
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos  https://review.openstack.org/21319409:50
*** sudipto has joined #openstack-neutron09:50
*** aix has quit IRC09:51
*** gardenshed has quit IRC09:51
*** vikram has joined #openstack-neutron09:54
*** markvoelker has quit IRC09:54
*** mickeys has joined #openstack-neutron09:56
*** dims has joined #openstack-neutron09:56
*** yog_ has quit IRC09:57
*** fawadkhaliq has joined #openstack-neutron09:58
openstackgerritIhar Hrachyshka proposed openstack/neutron: neutron-db-manage: sync HEADS file with 'current' output  https://review.openstack.org/21504809:59
ihrachyshkasalv-orlando, akamyshnikova, HenryG  ^^ take a look09:59
*** DWfuturetec has joined #openstack-neutron09:59
ihrachyshkahm, it though opens a question of what we do when mitaka_expand comes in. if we down_revision, it will get liberty_expand branch label due to inheritance10:00
ihrachyshkaoh, and I think kilo may actually now get liberty_expand label. hm.10:01
ihrachyshkayeah, with the patch 'upgrade liberty_expand@kilo' is a valid command. meh.10:02
*** Kennan2 has joined #openstack-neutron10:03
*** Kennan has quit IRC10:04
salv-orlandoihrachyshka: an loosely related question. Why did you ended up having release specific branches, and not just "expand" and "contract" branches?10:04
salv-orlando0"10:05
ihrachyshkasalv-orlando, honestly, mostly because it is as in zzzeek's spec.10:05
ihrachyshkawe may actually want it to be expand/contract, yeah10:05
salv-orlandoihrachyshka: I'm ok with it, it's just that we need to handle branching at each release10:06
*** tiny-hands has joined #openstack-neutron10:06
salv-orlandoit complicates mgmt but gives us leeway to backport migrations10:06
*** gampel has joined #openstack-neutron10:06
*** tiny-hands has quit IRC10:08
*** gardenshed has joined #openstack-neutron10:08
*** amotoki has quit IRC10:08
salv-orlandoihrachyshka: now a related question. with your patch "upgrade liberty_expand@kilo" is valid, but what are its semantics?10:08
*** jprovazn has joined #openstack-neutron10:09
ihrachyshkasalv-orlando, I don't like it now that I think about it, I will probably WIP-1 it. but alembic wise, it means 'upgrade to the kilo revision that belongs to liberty_expand branch. now that it's down_revision, kilo belongs to both expand and contract bracnhes.10:10
*** vikram has quit IRC10:10
salv-orlandoihrachyshka: and it most cases that would not work because it would be a downgrade. If you think about it, it makes sense10:10
salv-orlandoliberty_{expand|contract} start with kilo10:11
salv-orlandomitaka_expand starts with liberty and so on10:11
ihrachyshkasalv-orlando, but then if you do upgrade liberty_expand@head on mitaka, if mitaka root is using down_revision and not depends_on, it will upgrade till mitaka10:12
ihrachyshkawhile someone may expect it to upgrade till last revision in liberty only10:12
*** fawadkhaliq has quit IRC10:13
*** nyechiel_ has quit IRC10:13
*** bharath_ has joined #openstack-neutron10:13
*** sdake_ is now known as sdake10:15
*** lyanchih has quit IRC10:16
salv-orlandoihrachyshka: yes unfortunately this is true. But this also means that we're actually not terminating the X branch but contuining it10:18
salv-orlandoand therefore the mitaka_expand branch revisions, will belong also to the liberty_expand one. And that's not correct10:18
salv-orlandosounds like down_revision does not work either10:18
ihrachyshkaI guess renaming labels not to include release name would avoid confusion.10:19
salv-orlandoihrachyshka: do you mean the branch labels or the revision ids?10:20
*** britthou_ has quit IRC10:20
ihrachyshkabranch labels. they would be just expand and contract, so you would do 'upgrade expand@head' instead of liberty_expand@head, so it would not confuse you if you get mitaka expansion scripts too10:21
*** vikram has joined #openstack-neutron10:22
salv-orlandoihrachyshka: correct. This will limit the ability to backport migrations though. If we were planning to provide that capability - regardless of stable backport rules - it might be a problem10:23
openstackgerritOleg Bondarev proposed openstack/neutron: DVR: make sure snat portion is always scheduled when needed  https://review.openstack.org/21505510:23
ihrachyshkasalv-orlando, you mean, for downstream consumers in case they want a backport?10:23
ihrachyshkait would still require modifications both in stable release (the backport) and the next one (modifying the script that you've backported to not apply if we see the changes are already there)10:24
*** 7GHAAZ22K has quit IRC10:25
ihrachyshkawith single chain, it's pretty the same: you break the migration chain and modify two scripts: one that you move to the top of stable branch, to point to stable head as down_revision, and then the next script after the script you move: to point to backported script ancestor10:25
*** mickeys has quit IRC10:26
salv-orlandoihrachyshka: sure. I was suggesting that with distinct branches might be easier because if trunk is X you have a branch {X-1}_expand with a {X-1} release identifier (the one with the depends on for the X_expand branch10:28
salv-orlandoand you can add backported revision to X-1_expand10:28
salv-orlandoihrachyshka: but then upgrading to X will be a funny activity10:29
salv-orlandothat will probably end up in tears10:29
ihrachyshkasalv-orlando, then in X, you should kill the migration rule from the X_expand chain10:29
salv-orlandoso yeah the backport thing does not really work and is always a non non10:29
salv-orlandoso I agree we can just drop the release names in branch labels10:30
salv-orlandoand simplify things10:30
ihrachyshkasalv-orlando, well, unless you consider master chasing (downstreams usually don't) it mostly works, except that you need to notify users that they need to upgrade after a minor update10:30
ihrachyshkawhich is PITA but may be a better option from plenty of bad ones10:31
*** DWfuturetec has quit IRC10:31
*** vthapar has quit IRC10:31
salv-orlandoyes, but as you too wrote, then you need to kill the backported migrations first when upgrading to a new major release10:31
salv-orlandoand we have no way to automate this at the moment10:32
*** baojg has quit IRC10:33
*** baojg has joined #openstack-neutron10:34
*** gberginc has joined #openstack-neutron10:34
openstackgerritneethi shashidhar proposed openstack/neutron-specs: CLI implementation for listing and deleting tunnel endpoints  https://review.openstack.org/21505810:35
*** vikram has quit IRC10:37
*** baojg has quit IRC10:38
*** fawadkhaliq has joined #openstack-neutron10:38
*** vikram_choudhary has joined #openstack-neutron10:38
*** lpetrut has joined #openstack-neutron10:38
*** hoangcx has quit IRC10:41
*** hoangcx has joined #openstack-neutron10:41
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan  https://review.openstack.org/21267310:43
HenryGsalv-orlando: ping10:43
salv-orlandogood morning HenryG10:43
salv-orlandoour lord of the database and saviour of the gate10:44
HenryGsalv-orlando: that can't be me10:44
HenryGsalv-orlando: Sorry for barging in on your reservations patch yesterday10:45
*** cbouch has joined #openstack-neutron10:45
HenryGsalv-orlando: any thoughts on it?10:46
*** stanzgy__ has joined #openstack-neutron10:47
*** gardenshed has quit IRC10:48
*** DWfuturetec has joined #openstack-neutron10:50
*** stanzgy has quit IRC10:50
*** mohankumar has quit IRC10:51
*** lazy_prince has joined #openstack-neutron10:52
salv-orlandoHenryG: that issue was already knownn. It's not very worrying because:10:52
salv-orlando1) ihrachyshka pushed a patch for it which is not yet approved10:52
*** shaohe_feng has joined #openstack-neutron10:53
salv-orlando2) I am trying to get rid of the lock altogether10:53
digahi tfukushima10:54
digahi irenab10:54
tfukushimaHi diga,10:54
irenabhi10:54
salv-orlandoHenryG: if this occurs in master branch with rally then I guess you can merge my patch as it is now10:54
salv-orlandothe fix is good anyway10:54
ihrachyshkaHenryG, https://review.openstack.org/#/c/214133/10:55
digaveth pair is not the interface but by the way I understood now how to do that10:55
*** sudipto has quit IRC10:55
*** sudipto has joined #openstack-neutron10:56
digawe need to create port using ovs on host & that should get added to veth10:56
digacorrect me if I am wrong10:56
HenryGsalv-orlando: I think ihrachyshka's patch is safer10:57
HenryGAt least until you get rid of the locking10:57
salv-orlandoHenryG: ok, let's merge it then.10:57
salv-orlandoHenryG: I am still finishing up the patch. It might take the rest of the day10:57
*** hoangcx has quit IRC10:58
salv-orlandobecause I am dealing with the most annoying of the sqlalchemy errors: "This Session's transaction has been rolled back by a nested rollback() call"10:58
HenryGouch10:58
openstackgerritvikram.choudhary proposed openstack/neutron: BGP Dynamic Routing: introduce entities and model  https://review.openstack.org/20162110:58
tfukushimadiga, I only know the OVS datapath but we can bind a "link" to the datapath and the link is turned into a "port" of the datapath.10:58
digatfukushima, can you share docs related to this, let me go through it11:00
irenabdiga: here comes the difference between different vif types binding. For OVS/OVN it will ad port on ovs bridge11:01
*** mwagner_lap has quit IRC11:01
HenryGsalv-orlando: fyi, I approved the "less red in rally" fix, https://review.openstack.org/20338411:01
irenabdiga: for Midonet it will do what tfukushima said11:01
salv-orlandoHenryG: Yes that seems to work for the default sec group issue11:02
digairenab: okay11:02
salv-orlandothe reservation work will still have red, but not failures.11:02
HenryGsalv-orlando: what do you think about making the rally job voting soon?11:02
salv-orlandorally as a fairly high contention model, so deadlock will happen. But the decorator will sort out the situation.11:02
salv-orlandoHenryG: it's a dual-edged sword11:02
salv-orlandobecause on the one hand every developer will be forced to look at it.11:03
digairenab: I need to read about this ovs datapath11:03
openstackgerritMerged openstack/neutron: Minor typo fix  https://review.openstack.org/21301911:03
salv-orlandobut on the other hand expect an increase in time the gate is wedged11:03
salv-orlandoI think most, if not all, core reviewers look at it.11:04
*** yalie has quit IRC11:04
*** alex_klimov has joined #openstack-neutron11:04
salv-orlandoThe issues we had happened because the pymysql and workers switch merged asynchronously with the quota reservation patch11:04
salv-orlandoon devstack-gate - which does not run the neutron rally test11:04
*** apuimedo|away is now known as apuimedo11:04
apuimedodiga: https://github.com/stackforge/nova-docker/blob/master/novadocker/virt/docker/vifs.py11:06
apuimedolook at this handling11:06
akamyshnikovaihrachyshka, about your letter, can you send me what difference was discovered as http://pastebin.test.redhat.com/306865 does not opened for me.11:06
apuimedothis is a superset of what the executables need to do11:06
apuimedonote that for ovs it does: linux_net.create_ovs_vifg_port11:07
digasure apuimedo11:07
ihrachyshkaakamyshnikova, ^^ oh right, it's an internal thing. jlibosva, can you help akamyshnikova with seeing what the new alembic caught?11:07
apuimedothat is practically the only thing that goes into the ovs executable11:07
apuimedoand for midonet it is going to be the mm-ctl --bind-port...11:07
digaHere wht  I have implemented according to comments received from irenab11:07
HenryGsalv-orlando: just feels to me that currently rally is the only job that decently catches DB issues11:08
digaapuimedo: you can take a look at this - https://review.openstack.org/#/c/214002/11:08
jlibosvaakamyshnikova: there you go :) http://paste.openstack.org/show/422619/11:08
apuimedothanks diga. I'll take a look11:08
digasure apuimedo11:09
HenryGsalv-orlando: so I guess it will be my job to always look at it11:09
jlibosvaakamyshnikova: I pasted the same to bug I opened: https://bugs.launchpad.net/neutron/+bug/148693611:09
openstackLaunchpad bug 1486936 in neutron "Migration scripts and db models are not in sync" [Undecided,New]11:09
*** stanzgy has joined #openstack-neutron11:10
cbouchcore reviewers:  I have a couple cherry-picks pending approval for kilo.  Please help. They are https://review.openstack.org/#/c/214784/ and https://review.openstack.org/#/c/213327/11:10
*** ushkalim has quit IRC11:11
*** ushkalim has joined #openstack-neutron11:11
*** britthouser has joined #openstack-neutron11:12
openstackgerritIhar Hrachyshka proposed openstack/neutron: Dropped release name from migration branch labels  https://review.openstack.org/21507611:12
*** yalie has joined #openstack-neutron11:12
*** stanzgy__ has quit IRC11:13
akamyshnikovajlibosva, thanks!11:13
openstackgerritGal Sagie proposed openstack/neutron-specs: Create resource with fixed id spec  https://review.openstack.org/21507911:13
salv-orlandoHenryG: yours because you have a duty as a core reviewer, but it should be everyone's. due to its nature I am bit wary of making the rally job voring.11:14
salv-orlando*voting11:14
*** yamamoto_ has joined #openstack-neutron11:14
*** jamie_h has quit IRC11:14
salv-orlandoHenryG: but still you could define a lighter version of the job by trimming down tasks, and have that job vote11:15
*** vikram_choudhary has quit IRC11:15
*** dims has quit IRC11:15
*** dims has joined #openstack-neutron11:16
*** aix has joined #openstack-neutron11:16
*** shaohe_feng has quit IRC11:17
HenryGihrachyshka: Are you going make some alembic branch labels change?11:20
*** dims has quit IRC11:20
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py  https://review.openstack.org/21501211:20
ihrachyshkaHenryG, we discussed it with salv-orlando today, you may catch up in irc logs. currently I try to see whether it will be easy now that we have other projects that use the labels11:22
ihrachyshkathe rationale is in the commit message11:22
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21508611:22
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/21508711:22
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/21508911:22
HenryGihrachyshka: yes, I read the irc backlog11:22
HenryGihrachyshka: which commit msg?11:23
ihrachyshkaHenryG, https://review.openstack.org/#/c/215076/11:23
ihrachyshkaHenryG, also see the parent that switches to down_revision to make current output in sync with alembic_version in db11:24
ihrachyshkameh, sorry... alembic heads vs alembic_version11:25
openstackgerritOpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/20442211:25
*** baoli has joined #openstack-neutron11:25
HenryGihrachyshka: thanks, reading11:26
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Common l2_extension test code moved to the right directory  https://review.openstack.org/21461011:26
*** vikram_choudhary has joined #openstack-neutron11:27
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Common l2_extension test code moved to the right directory  https://review.openstack.org/21461011:28
*** pc_m has joined #openstack-neutron11:28
HenryGihrachyshka: I guess we'll have to see if we can undo the external projects that have initialized their alembic branches11:29
*** baoli has quit IRC11:30
*** DWfuturetec has quit IRC11:31
jlibosvaihrachyshka: HenryG can you confirm please that neutron-db-manage --service <adv_service> upgrade head is not needed once there is entrypoint in system (from adv service egg file)? And it's executed automatically without --service11:31
*** scheuran has joined #openstack-neutron11:32
ihrachyshkaHenryG, or temporarily support their schema11:33
ihrachyshkajlibosva, it should11:33
jlibosvaihrachyshka: thanks11:33
*** pal has quit IRC11:33
*** dims has joined #openstack-neutron11:33
*** markvoelker has joined #openstack-neutron11:35
*** Murali has quit IRC11:35
*** pal has joined #openstack-neutron11:39
*** markvoelker has quit IRC11:40
openstackgerritIhar Hrachyshka proposed openstack/neutron: Dropped release name from migration branch labels  https://review.openstack.org/21507611:40
ihrachyshkaHenryG, I think it should work for all subprojects ^^11:40
*** Marga_ has quit IRC11:42
*** Marga_ has joined #openstack-neutron11:42
HenryGihrachyshka: cool hack11:42
*** thorst has joined #openstack-neutron11:43
*** l6unchpad has quit IRC11:43
ihrachyshkaHenryG, I hate how I manage the feature that it needs those hacks. seems like I didn't think about it for long enough.11:43
ihrachyshkaanother can of worms that I left to solve for later is splitting HEADS file11:43
*** yamamoto_ has quit IRC11:44
ajoihrachyshka, kevinbenton : https://review.openstack.org/#/c/214608/1/neutron/objects/base.py11:44
HenryGihrachyshka: I am equally to blame for not catching the issues11:44
*** l6unchpad has joined #openstack-neutron11:45
*** salv-orlando has quit IRC11:45
*** afazekas has quit IRC11:46
HenryGihrachyshka: splitting HEADS can now be simplified, no? contract/HEAD, expand/HEAD ?11:46
ihrachyshkaHenryG, yes, that's the idea. but since, again, projects caught HEADS, we need compatibility11:46
ihrachyshkafor a cycle11:47
*** dansmith has quit IRC11:47
*** dansmith has joined #openstack-neutron11:48
*** dansmith is now known as Guest9532811:48
apuimedotfukushima: https://review.openstack.org/#/c/208566/11:49
apuimedoare you going to make a new version of it with the imports fixed so we can get this merged?11:49
HenryGihrachyshka: at least for the project I manage no one is trunk chasing so I can revert everything before L is released11:49
openstackgerritMoshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy  https://review.openstack.org/21421211:53
*** britthouser has quit IRC11:55
HenryGakamyshnikova: hi11:55
*** vikram_choudhary is now known as vikram11:55
ihrachyshkaHenryG, you think no one does, yes. But I believe I should suffer thru the pain to be more wise in the future :)11:55
akamyshnikovaHenryG, hi11:56
*** emagana has quit IRC11:56
HenryGihrachyshka: :)11:56
HenryGakamyshnikova: bug #148607711:56
openstackbug 1486077 in neutron "VPNAAS: dsvm-functional doesn't have installed database backends" [Undecided,Invalid] https://launchpad.net/bugs/148607711:56
akamyshnikovaHenryG, I forgot to include deps in tox.ini :(11:57
akamyshnikovaHenryG, so mysal and postgres are installed11:57
HenryGakamyshnikova: aha! no worries11:58
*** gardenshed has joined #openstack-neutron11:59
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Make NeutronObjectDuplicateEntry exception more verbose  https://review.openstack.org/21460811:59
HenryGakamyshnikova: but functional-sswan is still not working?12:01
akamyshnikovaHenryG, no the same, it has separate section in tox.ini12:02
*** mohankumar has joined #openstack-neutron12:02
akamyshnikovaHenryG, I was going to update it, but I need to update depend change12:02
*** lpetrut has quit IRC12:02
HenryGakamyshnikova: I will leave a comment on how to make common deps for functional12:03
*** gardenshed has quit IRC12:03
akamyshnikovapc_m, https://review.openstack.org/213174 - should I push new patch removing tenant_id in cisco_csr_identifier_map table? Is it really unneeded?12:04
akamyshnikovaHenryG, great, thanks!12:04
akamyshnikovapc_m, or we need some other people opinion on that?12:05
pc_makamyshnikova: I don't think it is needed for that table (it maps OS IDs to CSR IDs and is supposed to be globally unique). I'd suggest committing a patch to remove and we can make sure that CSR team reviews it (as they are doing active support of the drivers).12:07
*** b3nt_pin is now known as beagles12:07
pc_makamyshnikova: That's Tom Holtzen and Yanping Qu.12:08
akamyshnikovapc_m, OK, thanks!12:09
*** piyanai has joined #openstack-neutron12:10
*** jpena is now known as jpena|lunch12:10
*** azbiswas has joined #openstack-neutron12:11
*** salv-orlando has joined #openstack-neutron12:11
salv-orlandoihrachyshka: why would you split it? I was thinking we might just add branch labels to it12:12
*** bharath_ has quit IRC12:13
ihrachyshkasalv-orlando, HEADS file? mostly to avoid half of git conflicts due to merge of a new script12:13
salv-orlandoihrachyshka: I see that's a good point inded.12:13
ihrachyshkayeah, it's painful to rebase, so if we can help to avoid it in some cases...12:14
*** trinaths has quit IRC12:14
salv-orlandoihrachyshka: rebasing by itself is not painful. It takes <1 minute. the problem are the patches going into merge conflict and needing a respin12:15
*** gildub has joined #openstack-neutron12:15
salv-orlandobut with 90% of migrations being expand ones perhaps we'd still have those conflicts even after splitting HEADS12:15
ihrachyshkayes, that's what I meant. and vote reset.12:15
ihrachyshkasalv-orlando, well ok, not half :)12:16
*** azbiswas has quit IRC12:16
ihrachyshkajschwarz, can you check the silly comment at https://review.openstack.org/#/c/210830/ ?12:17
*** lpetrut has joined #openstack-neutron12:17
jschwarzihrachyshka, sure, in a few minutes12:17
*** sdake has quit IRC12:17
*** haigang has quit IRC12:18
pc_mDoes anyone know how networking-* projects handle CLI support? Do they have their own client code, use neutronclient, or something else?12:21
*** edmondsw has joined #openstack-neutron12:23
*** pal has quit IRC12:23
*** vikram has quit IRC12:23
*** fawadkhaliq has quit IRC12:24
*** gberginc has quit IRC12:24
*** stanzgy has quit IRC12:25
*** rpothier has quit IRC12:27
*** mkolesni_ has quit IRC12:31
*** emagana has joined #openstack-neutron12:32
*** banix has joined #openstack-neutron12:32
jschwarzihrachyshka, gah, missed that comment12:33
jschwarzit doesn't apply, will resubmit12:33
openstackgerritJohn Schwarz proposed openstack/neutron: Add EnvironmentDescription, pass it down  https://review.openstack.org/21083012:34
*** britthouser has joined #openstack-neutron12:36
*** signed8bit has joined #openstack-neutron12:36
*** mwagner_lap has joined #openstack-neutron12:37
*** vikram has joined #openstack-neutron12:38
*** lpetrut has quit IRC12:39
*** markvoelker has joined #openstack-neutron12:39
*** gongysh has quit IRC12:39
*** banix has quit IRC12:40
*** neophy has quit IRC12:41
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table  https://review.openstack.org/21317412:42
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994312:42
*** diga has quit IRC12:44
*** yamamoto has joined #openstack-neutron12:44
*** chlong has joined #openstack-neutron12:46
HenryGsalv-orlando: can I trouble you for an IPAM code question?12:46
salv-orlandosure12:46
HenryGsalv-orlando: https://review.openstack.org/#/c/212688/4/neutron/ipam/drivers/neutrondb_ipam/driver.py12:47
*** jpena|lunch is now known as jpena12:47
*** garyk has joined #openstack-neutron12:49
HenryGsalv-orlando: I guess my question is, is there a better way to get self._cidr to create_allocation_pools() than passing it through a bunch of classmethods?12:49
*** yamamoto has quit IRC12:49
*** annegentle has joined #openstack-neutron12:50
openstackgerritOleg Bondarev proposed openstack/neutron: Validate router admin_state_up on upgrade to distributed  https://review.openstack.org/21409712:51
*** yamamoto has joined #openstack-neutron12:51
*** yamamoto has quit IRC12:51
*** yamamoto has joined #openstack-neutron12:52
*** yamamoto has quit IRC12:52
*** yamamoto has joined #openstack-neutron12:52
*** dave-mccowan has joined #openstack-neutron12:52
*** jistr is now known as jistr|mtg12:53
*** vthapar has joined #openstack-neutron12:53
salv-orlandoHenryG: moving those method to instance methods might not be thread safe. So you'd need also to ensure that every request get a different instance of the driver12:53
*** aix has quit IRC12:54
*** yalie has quit IRC12:54
*** tkatarki has joined #openstack-neutron12:55
*** signed8bit is now known as signed8bit_ZZZzz12:55
*** shausy has quit IRC12:56
*** shausy has joined #openstack-neutron12:56
*** emagana has quit IRC12:57
*** liangbo has quit IRC12:57
HenryGsalv-orlando: If there is a good reason for the classmethods, that's fine. I had not considered the threads aspect.12:57
*** shausy has quit IRC12:58
ihrachyshkaajo, ok, so in the end we merged a qos patch that broke tests. nice12:58
salv-orlandoHenryG: the reason mostly is not that classmehtod are important. You can use instance methods but then there will be other changes needed to avoid races12:59
*** pc_m has quit IRC12:59
*** lpetrut has joined #openstack-neutron12:59
*** pc_m has joined #openstack-neutron12:59
*** aix has joined #openstack-neutron13:00
irenabpc_m: hey13:00
*** chlong has quit IRC13:00
*** gongysh has joined #openstack-neutron13:00
*** signed8bit_ZZZzz is now known as signed8bit13:00
ajoihrachyshka, really?13:01
*** gongysh has quit IRC13:01
ihrachyshkaajo, well http://logs.openstack.org/49/214249/3/gate/gate-neutron-dsvm-api/6d486ff/testr_results.html.gz13:01
ajoihrachyshka, which one?, I saw your api-enabling one passing.13:01
ihrachyshkaajo, that's the devstack plugin13:01
ihrachyshkaI guess the policy.json patch you pushed13:02
*** amuller has joined #openstack-neutron13:02
ajook, let me fix that13:02
ajoI will depend on your qos-enable first13:02
ajoand then we can put it in front via rebase13:02
ajoihrachyshka: sounds like a plan?13:02
ihrachyshkaajo, yeap. please rebase the plugin on top of a fix13:03
ihrachyshkathen we'll retain votes there13:03
ajoack13:03
pc_mirenab: hi13:03
ihrachyshkaajo, tnx13:03
*** chlong has joined #openstack-neutron13:03
ajoihrachyshka, you're welcome, sorry for the breakage..13:04
*** tiny-hands has joined #openstack-neutron13:05
tfukushimaapuimedo: I'm fixing the issues and adding unit tests. Although I'm experiencing some conflicts, I'll push them soon.13:06
irenabpc_m: I have few questions regarding neutron-vpnaas. Would you be the right person to ask?13:06
apuimedotfukushima: thanks!13:06
*** amuller is now known as amuller_afk13:06
pc_mirenab: yes13:06
irenabpc_m: I am looking into adding support for VPNaaS support for midonet which has its own L3 implementation without L3agents involved.13:08
irenabpc_m: It looks like current implementation assumes L3 (vpn) agent is always  deployed. Am I right?13:09
*** gardenshed has joined #openstack-neutron13:09
pc_mirenab: not necessarily. The reference implementation defines a VPN agent, which is a subclass of the L3 agent (so when VPN is enabled, you have L3).13:12
irenabpc_m: I am looking at CSRDriver, but it looks like it also assumes agent is deployed13:12
pc_mirenab: L3 agent?13:12
*** kbringard has joined #openstack-neutron13:12
irenabpc_m: vpn agent13:12
jschwarzajo, -1'd your agents patch13:13
jschwarzajo, it breaks fullstack13:13
ajojschwarz, lol, then jenkins -1'd it I hope :)13:13
jschwarzajo, fullstack is non-voting13:13
ajoaah, well done then, I will check the fullstack log13:13
jschwarz:)13:13
ajoajo--13:13
ajo!13:13
ajojschwarz13:14
ajoquestion about tox -eapi13:14
jschwarzyes13:14
ajojschwarz: first time it started runnig, I CTRL+C'd13:14
ajonow I try to start it... and it get's stuck at start13:14
jschwarzajo, erm. what does neutron-server say?13:15
ajojschwarz: seems quiet13:15
jschwarzerm13:15
jschwarzrestart httpd?13:15
ajohmmm13:15
ajothere's something wrong13:15
ajothat tab in screen is locked..13:15
ajo:D13:15
pc_mirenab: maybe there is some confusion with terms.13:16
ajojschwarz: you nailed it13:16
ajo;)13:16
irenabpc_m: there is an RPC channel between server driver and device driver. device driver calls Csr API. Any idea why its not done at server side?13:16
jschwarzajo, :)13:16
ajoneutron server was stuck somehow13:16
*** pc_m_ has joined #openstack-neutron13:16
pc_m_irenab: Lost my connection for a min.13:16
pc_m_irenab: For VPN, there is a plugin for the service. A service driver is specified by the plugin, and one or more device drivers are loaded as well.13:17
*** vikram has quit IRC13:17
pc_m_irenab: The drivers are paired, so for example the CSR service and device driver will work together to handle VPN requests.13:17
irenabpc_m_: so for CSR, device driver is server side component?13:18
*** krtaylor has quit IRC13:18
pc_m_irenab: The service driver is on server side. Called by plugin.  Talks to device driver via RPC.13:19
ihrachyshkaajo, should we finally get it in: https://review.openstack.org/#/c/182920/ ?13:19
ihrachyshkawhat's your take?13:19
* ihrachyshka has not read the patch with due dilligence to do it himself13:20
*** pc_m has quit IRC13:20
ajoihrachyshka: IMHO yes, I'm glad to +A, but I'd like aarosen to come back and agree13:20
ajo(or disagree) :)13:21
irenabpc_m: if I understand correctly, device drivers are loaded by vpn agent. Am I right? vpn agent is subclass of L3 agent.13:21
pc_m_irenab: Yes, for reference drivers.13:21
*** lindycoder has joined #openstack-neutron13:21
*** liangbo has joined #openstack-neutron13:21
ihrachyshkaajo, meh. well ok.13:22
openstackgerritJakub Libosvar proposed openstack/neutron: Sync FK constraints in db models with migration scripts  https://review.openstack.org/21512913:22
ajoihrachyshka; I believe the patch is good, fixes the issue, they reduced some unnecesary complexity about vlan id recovery...13:22
ajoihrachyshka: how about backport? :P :-)13:23
ajoit's just a tiny patch (400L) :)13:23
openstackgerritsean mooney proposed openstack/neutron: Adds configurable agent type  https://review.openstack.org/19721013:23
ihrachyshkaajo, nah, let's wait if you think aaron can stay disagreed13:23
openstackgerritsean mooney proposed openstack/neutron: Add config option to specify ovs datapath.  https://review.openstack.org/19720913:23
pc_m_irenab: IIRC, CSR does not use the L3 agent (the service is not enabled).13:23
ajo(I'm joking)13:23
ihrachyshkaajo, it's good for your health you do!13:23
pc_m_irenab: Trying to recall how the device drivers are loaded in that case.13:23
*** fawadkhaliq has joined #openstack-neutron13:24
irenabpc_m: So if I am looking for guidence to add new vpnaas driver which just need to propagate requests for vpnservice/connection to the Controller, I need to add service driver (server component), and device driver is not required, right?  Or its still needed? I am not sure need the overhead of RPC13:25
*** neophy has joined #openstack-neutron13:26
*** fawadkhaliq has quit IRC13:28
irenabpc_m: If you can share details about CSR integraton model, this can be very helpful. I think my case is similar.13:28
pc_m_irenab: In the reference drivers, there is one service driver, but may be multiple device drivers, if >1 host.13:28
*** vikram has joined #openstack-neutron13:29
pc_m_irenab: So I don't know the answer to your question. Maybe if your controller can handle talking to different hosts to do the underlying work, then I guess maybe not.13:29
*** jamielennox is now known as jamielennox|away13:30
pc_m_irenab: You can look at the Yvatta drivers. I think they use a controller, but am not too sure of how it is all set up. Sridhar R (not on right now) would know.13:30
*** krtaylor has joined #openstack-neutron13:31
ajojschwarz: question, neutron.tests.api.test_qos.QosBandwidthLimitRuleTestJSON.test_rule_association_nonshared_policy   shouldn't be doing both things as admin ? (policy creation, and rule creation?)13:31
irenabpc_m_: More general question, do you see any potential problem if there is no device _driver at all?13:32
* jschwarz is looking13:32
ajoi see is_admin=False13:32
jschwarzno13:32
ajojschwarz, ok, new policies disallow rule creation and policy creation for non-admins13:33
jschwarzajo, if create_bandwidth_limit_rule is done as admin, then it should see the policy even if the policy is not shared (because the user asking is admin and it ignores 'shared')13:33
*** nyechiel_ has joined #openstack-neutron13:33
irenabpc_m_: and thanks for the pointer, I will check Yvattta case13:33
pc_m_irenab: Not sure how you handle multiple nodes.13:33
Sam-I-Amanyone here working on the ebtables anti-spoofing stuff?13:33
ajojschwarz: I guess some tests wouldn't apply in this new context, but I'm checking now how is it possible it's creating a policy as non-admin13:33
ajothat fails here locally13:34
jschwarzajo, so... I guess 'shared=False' means it's supposed to be accessible only to admins13:34
Sam-I-Amsc68cal: ping13:34
ajojschwarz: yep13:34
pc_m_irenab: service driver resides with plugin on Controller, where as agent(s)/device_drivers(s) are on network node(s).13:34
jschwarzajo, self.create_qos_policy is creating with an admin user13:34
irenabpc_m_: by nodes, you mean the ones with Routers ?13:34
ajoahh13:34
pc_m_irenab: hosts13:34
ajook, that explains the thing13:34
ajoso, create is alwais done as admin13:34
ajoalways13:34
jschwarzajo, no13:34
ajook, we cannot create rules as non admins now13:35
jschwarzajo, specifically self.create_qos_policy13:35
ajoor policies13:35
jschwarzajo, and self.create_qos_bandwidth_limit_rules13:35
jschwarzajo, have a look at neutron/tests/api/base.py line 44513:35
ajook, that's not clear when reading the tests13:35
ajoack13:35
*** liangbo has quit IRC13:35
*** dims_ has joined #openstack-neutron13:35
irenabpc_m_: in my case Router is a distributed entity, so its up to Controller to handle it.13:35
*** liangbo has joined #openstack-neutron13:36
pc_m_irenab: So if multiple Network nodes, each with routers, there would be agent/device driver on each to handle VPN.13:36
*** amuller_afk is now known as amuller13:36
*** prad_ has joined #openstack-neutron13:36
jschwarzajo, the API client we're using atm is pretty implicit13:36
*** liangbo has quit IRC13:36
*** clev-away is now known as clev13:36
jschwarzI was thinking of revamping it but that's a lot of work13:36
*** zzzeek has joined #openstack-neutron13:36
irenabpc_m_: got it, I think I should look into integration with server side component only. Thanks a lot for your help.13:37
ajojschwarz: but bandwidth limit rule create ... seems to be admin too13:37
ajoisn't it?13:37
pc_m_irenab: So I guess there is an L3 agent running. I guess with CSR it talks with with their L3Router plugin, instead of the q-l3 service (I know for sure there is no q-l3 servcie running, when using CSR)13:38
jschwarzajo, yes13:38
jschwarzajo, is that a problem?13:38
ajojschwarz, the policy check says it's not13:38
jschwarzajo, link?13:38
ajohttp://fpaste.org/257148/77923144/13:38
ajolook for is_admin13:38
pc_m_irenab: yw. Afraid I don't know much about how to connect with a separate controller... hopefully some useful info though.13:38
*** dims has quit IRC13:39
*** tiswanso has joined #openstack-neutron13:40
ajojschwarz: but create policy works...13:40
jschwarzajo, hold on, was multi tasking13:40
jschwarzajo, looking now13:40
jschwarzajo, I need some context - what are you trying to do?13:40
ajojschwarz: we introduced the final policies.json13:41
ajoin this patch:13:41
ajohttps://review.openstack.org/#/c/214030/13:41
ajobut now,13:41
* jschwarz when did that happen? :O13:41
ajohttps://review.openstack.org/#/c/214249/ api tests fail here13:41
jschwarzOK now I understand13:41
amullerjschwarz: how's the local_ip fix coming along?13:42
jschwarzamuller, it's not... will do it shortly13:42
jschwarzsorry :<13:42
ihrachyshkaI told you guys we should keep the order13:42
jschwarzihrachyshka, everything is in order13:42
ihrachyshka:)13:42
jschwarzI was out of touch for a bit ;-)13:42
ajojschwarz:  he's telling me ;)13:42
jschwarzah13:43
ihrachyshkayeah, I brag a bit about us missing opportunity to get api tests on today ;)13:43
jschwarzajo, so according to the code it IS creating it as admin13:43
*** gildub has quit IRC13:44
ajojschwarz, but policy says it's not13:44
ajoand test_rule_create works13:44
ajono exception thrown...13:44
* ajo doesn't understand13:44
*** dims_ has quit IRC13:44
ajojschwarz, no worries13:44
ajoI will debug what's going on13:44
ajojschwarz: go back to the local_ip thing ;)13:44
jschwarzajo, the gate is clogged anyway ;-(13:45
ajojschwarz: I can reproduce locally.13:45
jschwarzajo, where is the exception thrown from on the server's side?13:45
ajojschwarz: yep13:46
*** dims has joined #openstack-neutron13:46
openstackgerritgongysh proposed openstack/python-neutronclient: Clear the extension requirement  https://review.openstack.org/21498113:46
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513613:46
ajoI wonder if creating the policy with tenant_id=.... in that specific test breaks the admin client13:46
*** julim has joined #openstack-neutron13:46
ajois the only difference to other tests13:46
jschwarzajo, that might be13:46
*** signed8bit is now known as signed8bit_ZZZzz13:46
jschwarzohhhhhhhhhh13:46
jschwarzajo, look13:46
* ajo smiles13:46
*** prad_ is now known as pradk13:47
jschwarzajo, it's running "self.client.create_bandwidth_limit_rule"13:47
jschwarzajo, note the "self.client.XXX"13:47
ajoahhh13:47
ajocrap :D13:47
jschwarzit's not using the self.admin_client13:47
ajook, that's it13:47
jschwarzso it's in user mode13:47
jschwarzit SHOULD be throwing 403 :)13:47
ajook, I will remove that test it doesn't make sense anymore with the default policies :)13:47
jschwarzglad to be of service13:47
jschwarzonward, to local_ip!13:47
ajoI will revamp into two: tenant can't create policies, tenant can't create rules..13:47
*** fawadkhaliq has joined #openstack-neutron13:48
jschwarzajo, make sure to add me as a reviewer to that patch that has the change and I'll have a look13:48
*** pal has joined #openstack-neutron13:48
ajojschwarz, about idempotent_id's13:49
*** signed8bit_ZZZzz is now known as signed8bit13:49
*** rpothier has joined #openstack-neutron13:49
ajoshould I create random ones each time I write a new test?13:49
jschwarzajo, yes13:49
ajoack13:49
*** nyechiel_ has quit IRC13:49
*** signed8bit is now known as signed8bit_ZZZzz13:50
openstackgerritMarga Millet proposed openstack/neutron: Support dhcp metadata service for all networks  https://review.openstack.org/21196313:51
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process  https://review.openstack.org/21470113:51
*** neophy has quit IRC13:52
jlibosvalifeless: hi - can you please revisit https://review.openstack.org/#/c/180587/6 ? You asked for Doug's input and it's been a while since his reply13:53
*** piyanai has quit IRC13:53
*** neiljerram has quit IRC13:53
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513613:54
openstackgerritHenry Gessau proposed openstack/neutron: Do not query reservations table when counting resources  https://review.openstack.org/21460213:56
*** pal has quit IRC13:56
*** njohnston is now known as nate_gone13:56
*** regXboi has joined #openstack-neutron13:56
openstackgerritgongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line  https://review.openstack.org/21496113:56
*** mohankumar has quit IRC13:56
*** obondarev_ has joined #openstack-neutron13:56
*** tfukushima has quit IRC13:57
openstackgerritgongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line  https://review.openstack.org/21496113:57
*** pal has joined #openstack-neutron13:57
*** sbelous has quit IRC13:58
*** sbelous has joined #openstack-neutron13:58
*** nate_gone is now known as njohnston13:58
*** obondarev has quit IRC13:59
*** eezhova has quit IRC13:59
*** skraynev has quit IRC13:59
ihrachyshkaajo, I see you avoid +2'ing qos patches. I wonder wy13:59
ihrachyshka*why13:59
*** eezhova has joined #openstack-neutron13:59
*** skraynev has joined #openstack-neutron14:00
ajoihrachyshka: db is not in my area of expertise,14:00
ajook, I have some expertise14:00
ajobut not supossedly in neutron :)14:00
*** Guest95328 is now known as dansmith14:00
ihrachyshkaajo, well then I wonder if you are not capable of reviewing qos stuff then who is14:00
ajoit's in a fine line, I guess I'm an expert in the qos service, I know quite a bit about DBs, but for example , I didn't know that we recreated models from sqlalchemy models during testing...14:01
ajoI guess I should raise the point with mestery, it felt right to me14:01
ajobut I didn't want to step on the bad side ;)14:01
ihrachyshkaajo, I guess few people knew it. well, you decide, but I just wondered.14:01
openstackgerritJakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted  https://review.openstack.org/21421514:01
ihrachyshkait's all about trust man! everyone trusts you know qos! :)14:02
jlibosvaakamyshnikova: Thanks for testing on postgres! I didn't do that :)14:03
*** aix has quit IRC14:04
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin  https://review.openstack.org/21424914:04
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes  https://review.openstack.org/21513814:04
*** lukasa has quit IRC14:04
ajoihrachyshka, there it goes ^14:04
ajojschwarz   ^14:05
jschwarzajo, ack14:05
*** signed8bit_ZZZzz has quit IRC14:06
akamyshnikovajlibosva, np :)14:07
*** afazekas has joined #openstack-neutron14:07
*** anilvenkata has quit IRC14:08
*** jistr|mtg is now known as jistr14:08
openstackgerritJohn Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling  https://review.openstack.org/21514214:09
jschwarzamuller, ^14:09
*** aix has joined #openstack-neutron14:09
amullerjschwarz: are we aiming for a PS1 merge?14:09
amullerjschwarz: aww, no bug14:09
jschwarzamuller, my code is always perfect on first try ;-)14:10
amullerjschwarz: better luck next time =D14:10
jschwarzamuller, bug?14:10
amullerjschwarz: yeah, need a bug report14:10
jschwarzamuller, peh.14:10
jschwarzamuller, this is a commit-message change14:10
amullerjschwarz: still counts14:10
jschwarzamuller, your will shall be done, my sire14:11
ihrachyshkaajo++14:12
*** ajmiller has joined #openstack-neutron14:13
*** lyanchih has joined #openstack-neutron14:13
openstackgerritJohn Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling  https://review.openstack.org/21514214:14
jschwarzamuller, ^14:14
*** signed8bit has joined #openstack-neutron14:14
*** kiran-r has joined #openstack-neutron14:14
*** kiran-r has quit IRC14:15
*** garyk has quit IRC14:15
*** tkatarki has quit IRC14:16
*** yamamoto has quit IRC14:17
*** yamamoto has joined #openstack-neutron14:19
*** yamamoto has quit IRC14:19
*** yamamoto has joined #openstack-neutron14:19
amullerjschwarz: thanks I'll review after meetings14:20
jschwarzhopefully14:20
*** dmartls1 has joined #openstack-neutron14:20
ihrachyshkaajo, some nits14:21
regXboiSwami: ping14:23
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes  https://review.openstack.org/21513814:24
*** drankis has joined #openstack-neutron14:25
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin  https://review.openstack.org/21424914:25
*** dsneddon is now known as dsneddon_afk14:26
*** tkatarki has joined #openstack-neutron14:26
*** pal_ has joined #openstack-neutron14:27
*** pal has quit IRC14:27
*** vishwanathj has joined #openstack-neutron14:27
*** sayali has quit IRC14:27
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table  https://review.openstack.org/21317414:27
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994314:27
*** tidwellr has joined #openstack-neutron14:28
jlibosvaajo: does https://review.openstack.org/#/c/215138/ fix the recent qos api failures we saw on the gate? I'm not sure if there was a bug reported14:28
jschwarzajo, so shouldn't https://review.openstack.org/#/c/214249/ fail at the gate?14:29
jschwarzajo, because the api test will fail?14:30
*** banix has joined #openstack-neutron14:30
*** fawadkhaliq has quit IRC14:33
*** shwetaap has joined #openstack-neutron14:36
amullerjschwarz: can you respin https://review.openstack.org/#/c/215142/ and I'll +2?14:36
jschwarzamuller, I will in a few14:36
jschwarzamuller, in the middle of some heavy-weight DVR debugging ;-)14:36
amullerjschwarz: ok talk to you in a month then14:37
*** lpetrut has quit IRC14:37
jschwarzamuller, lol14:37
jschwarzamuller, actually in 23 minutes14:37
*** lpetrut has joined #openstack-neutron14:38
openstackgerritVlad Gridin proposed openstack/neutron: This changeset removes ML2 Nuage driver vendor code, currently present in neutron core to vendor repo.  https://review.openstack.org/21515514:39
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: [OpenSwan] Enable usage of the MTU value of an IPSec connection  https://review.openstack.org/20653914:40
clarkb14:42
clarkbwhoops14:42
*** signed8bit has quit IRC14:43
*** signed8bit has joined #openstack-neutron14:44
openstackgerritvikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce dragent scheduler model  https://review.openstack.org/20760714:47
openstackgerritJohn Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling  https://review.openstack.org/21514214:48
jschwarzamuller, jlibosva, ^14:48
*** pal_ has quit IRC14:48
*** vikram has quit IRC14:48
*** piyanai has joined #openstack-neutron14:49
*** john-davidge has quit IRC14:50
*** john-davidge has joined #openstack-neutron14:51
amulleryamamoto: heya14:51
amulleryamamoto: wanna check out https://review.openstack.org/#/c/215142/ ?14:51
jlibosvajschwarz: yet another questions :)14:51
*** neiljerram has joined #openstack-neutron14:52
jschwarzjlibosva, yet another question? :<14:52
yamamotoamuller: sure14:52
jschwarzjlibosva, meh you're really making me sad :<14:52
*** minwang2 has joined #openstack-neutron14:53
jlibosvajschwarz: it's a question(s)!14:53
*** SridharG has left #openstack-neutron14:54
jschwarzjlibosva, replied14:54
jschwarzgood questions :)14:54
*** signed8bit has quit IRC14:55
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513614:55
*** signed8bit has joined #openstack-neutron14:55
*** mlavalle has joined #openstack-neutron14:55
*** garyk has joined #openstack-neutron14:56
garykihrachyshka: around?14:57
ihrachyshkagaryk, almost running away, but shoot14:57
garykwhat was the conclusion for the tagging?14:57
garyki have a few more backports and then i can do the tag14:57
*** annegentle has quit IRC14:58
*** cbouch has quit IRC14:58
*** annegentle has joined #openstack-neutron14:58
openstackgerritMichael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters  https://review.openstack.org/21330314:59
openstackgerritMichael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension  https://review.openstack.org/20384214:59
ihrachyshkagaryk, I don't think we had time to discuss it the previous meeting14:59
ihrachyshkagaryk, I believe we need to follow up with mestery though he is pretty unavailable these week14:59
*** itzikb has quit IRC15:01
*** ddepaoli has quit IRC15:01
ihrachyshkagaryk, ok, I need to run now. I'll be avail in an hour15:01
*** alex_klimov has quit IRC15:01
garykihrachyshka: ok, np. guess it is not that urgent - lets talk tomorrow - have a good evening15:02
*** vikram_choudhary has joined #openstack-neutron15:02
openstackgerritMichael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters  https://review.openstack.org/21330315:03
openstackgerritMichael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension  https://review.openstack.org/20384215:03
openstackgerritPengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db  https://review.openstack.org/21356315:04
*** shoutm has joined #openstack-neutron15:05
*** fawadkhaliq has joined #openstack-neutron15:06
gridinvHenryG: wrt https://review.openstack.org/215155, entry point should be removed from setup.cfg and move to vendor repo?15:06
*** l6unchpad has quit IRC15:07
HenryGgridinv: Yep, you got it15:07
gridinvHenryG: thanks15:07
*** signed8bit has quit IRC15:08
*** alejandrito has joined #openstack-neutron15:08
*** pal has joined #openstack-neutron15:09
openstackgerritJohn Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling  https://review.openstack.org/21514215:10
amulleryamamoto: https://review.openstack.org/#/c/215142/15:10
*** ihrachyshka has quit IRC15:11
yamamotoamuller: lgtm15:13
*** ronis__ has quit IRC15:13
openstackgerritVlad Gridin proposed openstack/neutron: This changeset removes ML2 Nuage driver vendor code, currently present in neutron core to vendor repo.  https://review.openstack.org/21515515:13
amulleryamamoto: thanks15:14
*** jbell8 has joined #openstack-neutron15:16
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513615:16
amullerjschwarz: https://review.openstack.org/#/c/215055/15:16
*** anshul has quit IRC15:17
*** gardenshed has quit IRC15:17
*** vthapar has quit IRC15:17
*** kfox1111 has quit IRC15:19
*** shoutm has quit IRC15:20
*** dannywilson has joined #openstack-neutron15:21
*** shaohe_feng has joined #openstack-neutron15:21
*** jistr is now known as jistr|mtg15:22
*** pal has quit IRC15:24
*** gberginc has joined #openstack-neutron15:24
*** pal has joined #openstack-neutron15:24
openstackgerritPengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db  https://review.openstack.org/21356315:25
HenryGgridinv: ping15:25
*** moshele has quit IRC15:28
*** scheuran has quit IRC15:28
*** Swami has joined #openstack-neutron15:31
*** piyanai has quit IRC15:33
*** piyanai has joined #openstack-neutron15:34
*** pal has quit IRC15:35
*** r-daneel has joined #openstack-neutron15:36
*** TrevorV has joined #openstack-neutron15:37
*** TrevorV has quit IRC15:41
*** TrevorV has joined #openstack-neutron15:43
*** armax has joined #openstack-neutron15:44
*** mickeys has joined #openstack-neutron15:46
openstackgerritMerged openstack/neutron: Add logging to debug oslo.messaging failure  https://review.openstack.org/21336015:48
*** kbringard has quit IRC15:48
openstackgerritPengtao Huang proposed openstack/neutron: ignore the repeated value of service_plugins  https://review.openstack.org/21259715:49
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444115:49
*** vikram_choudhary has quit IRC15:49
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444615:49
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445515:49
*** mwagner_lap has quit IRC15:50
*** crose has joined #openstack-neutron15:51
*** salv-orlando has quit IRC15:52
*** vikram_ has joined #openstack-neutron15:52
*** neophy has joined #openstack-neutron15:53
*** hogepodge has quit IRC15:53
*** mkolesni_ has joined #openstack-neutron15:54
openstackgerritZhi Chang proposed openstack/neutron: Notify specific agent when update floatingip  https://review.openstack.org/21513615:55
*** devvesa has quit IRC15:56
*** liudong has quit IRC15:56
*** armax has quit IRC15:56
*** redbeard has quit IRC15:56
*** AlexeyElagin has joined #openstack-neutron15:57
*** lazy_prince has quit IRC15:59
*** salv-orlando has joined #openstack-neutron15:59
*** tflynn has joined #openstack-neutron16:01
openstackgerritPaul Michali proposed openstack/neutron: TEST COMMIT - DO NOT REVIEW/APPROVE  https://review.openstack.org/21520816:01
*** ihrachyshka has joined #openstack-neutron16:02
*** azbiswas has joined #openstack-neutron16:02
*** SourabhP has joined #openstack-neutron16:02
*** arajagopal has joined #openstack-neutron16:05
*** DWfuturetec has joined #openstack-neutron16:05
*** SourabhP_ has joined #openstack-neutron16:05
*** sdake has joined #openstack-neutron16:06
openstackgerritJakub Libosvar proposed openstack/neutron: fullstack: use migration scripts to create db schema  https://review.openstack.org/21521316:06
*** SourabhP has quit IRC16:08
*** SourabhP_ is now known as SourabhP16:08
*** jproulx has joined #openstack-neutron16:08
*** yamahata has joined #openstack-neutron16:09
*** piyanai has quit IRC16:09
*** jistr|mtg is now known as jistr16:10
*** crose has quit IRC16:10
*** kbringard has joined #openstack-neutron16:11
openstackgerritZhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process  https://review.openstack.org/21470116:12
jproulxHi All having trouble with production DB migration (juno->kilo) & of course didn't happen in test with same DB content16:12
jproulxBasically this part of a table creation is failing:16:12
jproulxALTER TABLE nsxv_internal_networks ADD FOREIGN KEY(network_id) REFERENCES networks (id) ON DELETE CASCADE;16:12
jproulxseems like trying to reference a nonexistant foreign key but the 'networks' table definitely has an 'id' field, help?16:12
*** tflynn_ has joined #openstack-neutron16:12
*** ihrachyshka has quit IRC16:13
*** _cjones_ has joined #openstack-neutron16:13
*** piyanai has joined #openstack-neutron16:13
*** shaohe_feng has quit IRC16:13
*** SourabhP has quit IRC16:15
*** tflynn has quit IRC16:15
*** paulose has joined #openstack-neutron16:15
*** minwang2 has quit IRC16:16
*** piyanai has quit IRC16:18
*** krtaylor has quit IRC16:18
*** dsneddon_afk is now known as dsneddon16:19
*** moshele has joined #openstack-neutron16:20
*** kiran-r has joined #openstack-neutron16:20
*** Marga_ has quit IRC16:22
*** hogepodge has joined #openstack-neutron16:23
*** gberginc has quit IRC16:23
*** ihrachyshka has joined #openstack-neutron16:25
*** kawa2014 has quit IRC16:25
*** rossella_s has quit IRC16:25
*** kbringard has quit IRC16:26
*** rossella_s has joined #openstack-neutron16:26
*** lpetrut has quit IRC16:26
*** crose has joined #openstack-neutron16:26
*** kawa2014 has joined #openstack-neutron16:26
*** vivek-ebay has joined #openstack-neutron16:27
*** mwagner_lap has joined #openstack-neutron16:28
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes  https://review.openstack.org/21513816:29
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin  https://review.openstack.org/21424916:29
*** kbringard has joined #openstack-neutron16:29
ajojschwarz ^ :)16:29
*** Marga_ has joined #openstack-neutron16:29
*** klindgren has joined #openstack-neutron16:29
ajoun-nitted ;)16:29
*** piyanai has joined #openstack-neutron16:29
*** nlahouti has joined #openstack-neutron16:30
*** sdake_ has joined #openstack-neutron16:30
*** salv-orlando has quit IRC16:30
*** _kiran_ has joined #openstack-neutron16:30
*** kiran-r has quit IRC16:31
*** pradk has quit IRC16:31
*** gampel has quit IRC16:32
*** jckasper has quit IRC16:32
jschwarzajo, it wasn't a nit though16:32
jschwarzit was a needed test ;-)16:32
*** matrohon has quit IRC16:32
*** cdl has joined #openstack-neutron16:32
*** jckasper has joined #openstack-neutron16:33
*** vikram_ has quit IRC16:33
*** sdake has quit IRC16:33
*** julim has quit IRC16:34
*** piyanai has quit IRC16:34
*** klindgren_ has joined #openstack-neutron16:34
*** ivar-lazzaro has joined #openstack-neutron16:34
*** sbelous_ has joined #openstack-neutron16:34
*** cdl has left #openstack-neutron16:34
*** sdake_ is now known as sdake16:34
*** numan has quit IRC16:35
*** SridharGaddam has quit IRC16:35
*** klindgren has quit IRC16:35
*** SridharG has joined #openstack-neutron16:36
*** numan has joined #openstack-neutron16:36
*** piyanai has joined #openstack-neutron16:37
*** shwetaap has quit IRC16:37
*** fawadk has joined #openstack-neutron16:37
*** gangil has joined #openstack-neutron16:37
*** fawadkhaliq has quit IRC16:37
*** jckasper has quit IRC16:38
*** diga has joined #openstack-neutron16:39
haleybneiljerram: i'm not avoiding you, i'm like and old mainframe and have a bad time-slicing algorithm16:39
*** numan has quit IRC16:39
*** vikram_ has joined #openstack-neutron16:41
*** armax has joined #openstack-neutron16:43
*** drankis has quit IRC16:43
*** ivar-laz_ has joined #openstack-neutron16:45
*** clev is now known as clev-away16:45
*** gberginc has joined #openstack-neutron16:45
openstackgerritSergey Belous proposed openstack/neutron: Adds base in-tree functional testing of the dhcp agent (OVS)  https://review.openstack.org/13683416:45
*** DWfuturetec has quit IRC16:46
*** jistr has quit IRC16:46
*** pradk has joined #openstack-neutron16:46
*** ivar-lazzaro has quit IRC16:46
openstackgerritMerged openstack/neutron: Add a fullstack fake VM, basic connectivity test  https://review.openstack.org/19119716:47
sbelous_amuller, ajo, hi, guys, can you take a look at this patch: https://review.openstack.org/#/c/136834/ please? I updated functional tests of dhcp-agent and solved the problem with overriding system resolv.conf16:48
ajosbelous_!!, nice :)16:48
ajoon mtg, but I will look as soon as we finish16:48
*** gangil has quit IRC16:49
*** piyanai has quit IRC16:49
sbelous_ajo, thanks :)16:49
*** tflynn_ has quit IRC16:49
*** tflynn has joined #openstack-neutron16:50
*** Kennan2 has quit IRC16:50
ajosbelous_: niiice ;) it looks good, so it will take the netns resolv.conf if existing16:51
*** kawa2014 has quit IRC16:51
*** kawa2014 has joined #openstack-neutron16:51
sbelous_ajo, yep. I think, that it's good way16:52
*** park_ has quit IRC16:52
sbelous_but it's requires strict filters settings16:53
*** minwang2 has joined #openstack-neutron16:53
*** shwetaap has joined #openstack-neutron16:53
*** piyanai has joined #openstack-neutron16:54
*** SourabhP has joined #openstack-neutron16:55
*** kbringard has quit IRC16:56
*** piyanai has quit IRC16:56
digaapuimedo: irenab : when do we have container networking meeting ?16:57
pauloseis vxlan in openvswitch implemented in userspace16:57
*** armax has quit IRC16:57
*** gangil has joined #openstack-neutron16:58
*** piyanai has joined #openstack-neutron16:59
sc68calpaulose: https://github.com/openvswitch/ovs#what-is-open-vswitch16:59
*** tkatarki has quit IRC17:00
sc68calpaulose: last paragraph discusses userspace vs kernel module17:00
jlibosvaajo: ihrachyshka https://review.openstack.org/#/c/215138/3/neutron/tests/api/test_qos.py - sorry I'm dumb, does it mean regular tenant cannot create qos bw rules?17:00
*** kawa2014 has quit IRC17:02
*** armax has joined #openstack-neutron17:02
*** afazekas has quit IRC17:02
*** sbelous_ has quit IRC17:03
*** mohankumar has joined #openstack-neutron17:03
*** mkolesni_ has quit IRC17:03
*** sbelous_ has joined #openstack-neutron17:03
*** padkrish has joined #openstack-neutron17:04
ihrachyshkajlibosva, by default, I think it should not be allowed to create neither policies nor rules in them17:04
ihrachyshkajust attach, if it's shared17:04
*** sbelous_ has quit IRC17:05
*** achanda has joined #openstack-neutron17:05
*** jschwarz has quit IRC17:05
*** Kennan has joined #openstack-neutron17:05
*** diga has quit IRC17:06
ihrachyshkaamuller, https://review.openstack.org/#/c/211166/2/neutron/api/rpc/handlers/l3_rpc.py line 162: is there space for a race there? let's say we got active, it's none, so we set a random one, but the latter hits db after master was elected.17:06
jlibosvaihrachyshka: aha, thanks for explanation17:06
*** park_ has joined #openstack-neutron17:06
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21508617:07
*** SourabhP has quit IRC17:08
*** abhiraut has joined #openstack-neutron17:09
*** jlibosva has quit IRC17:10
*** kbringard has joined #openstack-neutron17:10
*** DWfuturetec has joined #openstack-neutron17:12
paulosesc68cal since there is standard port for vxlan won't that be listening in userspace?17:12
*** piyanai has quit IRC17:13
*** piyanai has joined #openstack-neutron17:13
sc68calpaulose: I am not very familiar with the userspace implementation - this is a question that is probably meant for #openvswitch or ovs-discuss17:14
*** Kennan has quit IRC17:14
*** park_ has quit IRC17:14
*** absubram has joined #openstack-neutron17:14
*** jpena has quit IRC17:15
*** lyanchih has quit IRC17:16
*** ivar-la__ has joined #openstack-neutron17:16
paulosesc68cal i dont have any experience using vxlan with neutron,i believe in GRE implementation there wasn't any user space17:17
*** s3wong has joined #openstack-neutron17:17
*** crose has quit IRC17:17
*** ivar-laz_ has quit IRC17:17
sc68calpaulose: I believe that most distros install the openvswitch kernel module17:18
*** ronis__ has joined #openstack-neutron17:19
*** Kennan has joined #openstack-neutron17:20
*** park_ has joined #openstack-neutron17:20
*** julim has joined #openstack-neutron17:20
*** aix has quit IRC17:22
*** tkatarki has joined #openstack-neutron17:22
*** neophy has quit IRC17:22
*** Kennan has quit IRC17:24
*** park_ has quit IRC17:25
*** madhu_ak has joined #openstack-neutron17:26
*** vikram_ has quit IRC17:26
*** signed8bit has joined #openstack-neutron17:26
*** achanda has quit IRC17:27
*** amuller is now known as amuller_afk17:27
*** tfukushima has joined #openstack-neutron17:31
*** ihrachyshka has quit IRC17:31
paulosesc68cal do you have a running neutron setup with vxlan network_type?17:31
*** piyanai has quit IRC17:32
sc68calpaulose: why do you ask?17:33
*** yamahata has quit IRC17:34
*** smccully has joined #openstack-neutron17:34
*** neophy has joined #openstack-neutron17:36
*** yamamoto has quit IRC17:36
*** park_ has joined #openstack-neutron17:37
paulosesc68cal never mind i got the answer from #openvswitch17:39
*** jproulx has left #openstack-neutron17:39
*** piyanai has joined #openstack-neutron17:39
*** john-davidge has quit IRC17:40
*** gardenshed has joined #openstack-neutron17:40
*** Kennan has joined #openstack-neutron17:40
*** saksham has joined #openstack-neutron17:40
*** piyanai has quit IRC17:41
*** salv-orlando has joined #openstack-neutron17:42
*** DWfuturetec has quit IRC17:42
*** emagana has joined #openstack-neutron17:43
*** shivrao has joined #openstack-neutron17:43
*** salv-orlando has quit IRC17:46
*** krtaylor has joined #openstack-neutron17:47
*** tidwellr has left #openstack-neutron17:47
*** clev-away is now known as clev17:48
*** DWfuturetec has joined #openstack-neutron17:48
*** sridhar_ram has joined #openstack-neutron17:48
*** pradk has quit IRC17:48
*** bharath has joined #openstack-neutron17:48
*** SumitNaiksatam has joined #openstack-neutron17:48
*** prad_ has joined #openstack-neutron17:49
*** _kiran_ has quit IRC17:50
*** padkrish has quit IRC17:51
*** breitz has quit IRC17:52
*** breitz has joined #openstack-neutron17:52
*** tflynn has quit IRC17:52
*** nlahouti has quit IRC17:54
*** achanda has joined #openstack-neutron17:55
*** sthillma has joined #openstack-neutron17:56
*** signed8bit is now known as signed8bit_ZZZzz17:56
*** pc_m_ has quit IRC17:57
*** jwy has joined #openstack-neutron17:57
*** pc_m has joined #openstack-neutron17:57
*** pc_m has quit IRC17:58
*** pc_m has joined #openstack-neutron17:59
openstackgerritCedric Brandily proposed openstack/neutron: Remove VIF_TYPES constant  https://review.openstack.org/21390518:00
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor VIF_TYPE_* constants  https://review.openstack.org/21480518:00
openstackgerritCedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants  https://review.openstack.org/21480618:00
*** piyanai has joined #openstack-neutron18:00
*** gardenshed has quit IRC18:01
*** gberginc has quit IRC18:01
openstackgerritJames Arendt proposed openstack/python-neutronclient: WIP: Add Neutron flavor framework CLI  https://review.openstack.org/21477618:02
*** arajagopal has quit IRC18:03
*** DWfuturetec has quit IRC18:03
*** prad_ is now known as pradk18:04
*** padkrish has joined #openstack-neutron18:04
*** arajagopal has joined #openstack-neutron18:04
*** nlahouti has joined #openstack-neutron18:05
*** signed8bit_ZZZzz is now known as signed8bit18:06
*** tiswanso has quit IRC18:06
*** pc_m has quit IRC18:10
*** SourabhP has joined #openstack-neutron18:10
*** yfried|afk has joined #openstack-neutron18:11
*** pc_m has joined #openstack-neutron18:11
*** bapalm has quit IRC18:11
*** tiswanso has joined #openstack-neutron18:11
*** minwang2 has quit IRC18:12
*** bapalm has joined #openstack-neutron18:14
*** mohankumar has quit IRC18:14
*** padkrish has quit IRC18:16
*** padkrish has joined #openstack-neutron18:17
*** armax has quit IRC18:18
*** moshele has quit IRC18:21
*** padkrish has quit IRC18:21
openstackgerritMerged openstack/neutron: Fix _ensure_default_security_group logic  https://review.openstack.org/20338418:21
*** padkrish has joined #openstack-neutron18:21
openstackgerritMerged openstack/neutron: Unskip firewall test  https://review.openstack.org/21214518:21
*** tkatarki has quit IRC18:22
*** madhu_ak has quit IRC18:22
*** tkatarki has joined #openstack-neutron18:22
*** sthillma_ has joined #openstack-neutron18:23
*** madhu_ak has joined #openstack-neutron18:24
*** achanda has quit IRC18:24
*** sthillma has quit IRC18:25
*** arosen has joined #openstack-neutron18:25
*** sthillma has joined #openstack-neutron18:26
*** mickeys has quit IRC18:26
*** pc_m has quit IRC18:27
*** fawadk has quit IRC18:27
*** signed8bit is now known as signed8bit_ZZZzz18:27
*** pc_m has joined #openstack-neutron18:27
*** signed8bit_ZZZzz is now known as signed8bit18:28
*** sthillma_ has quit IRC18:28
*** lindycoder has quit IRC18:28
*** tkatarki has quit IRC18:29
*** cbouch has joined #openstack-neutron18:31
*** tiswanso has quit IRC18:32
*** kiran-r has joined #openstack-neutron18:33
*** tkatarki has joined #openstack-neutron18:33
*** srilatha has joined #openstack-neutron18:33
*** tidwellr has joined #openstack-neutron18:34
*** pc_m_ has joined #openstack-neutron18:34
*** Sukhdev has joined #openstack-neutron18:34
*** dave-mcc_ has joined #openstack-neutron18:36
*** yamamoto has joined #openstack-neutron18:37
*** lindycoder has joined #openstack-neutron18:38
*** pc_m has quit IRC18:38
*** Samos123 has quit IRC18:38
*** dave-mccowan has quit IRC18:39
*** signed8bit is now known as signed8bit_ZZZzz18:39
*** madhu_ak has quit IRC18:40
*** Samos123 has joined #openstack-neutron18:41
*** signed8bit_ZZZzz is now known as signed8bit18:41
*** yamamoto has quit IRC18:42
*** tflynn has joined #openstack-neutron18:42
*** dannywilson has quit IRC18:44
*** madhu_ak has joined #openstack-neutron18:46
*** dannywilson has joined #openstack-neutron18:46
*** piyanai has quit IRC18:46
*** vivek-ebay has quit IRC18:49
*** vivek-ebay has joined #openstack-neutron18:49
*** paulose has quit IRC18:52
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442418:53
*** kiran-r has quit IRC18:53
*** sbelous_ has joined #openstack-neutron18:54
*** afazekas has joined #openstack-neutron18:55
*** piyanai has joined #openstack-neutron18:56
*** Samos123 has quit IRC19:00
*** cdl has joined #openstack-neutron19:00
*** Samos123 has joined #openstack-neutron19:00
regXboiquestion for the channel: where do the tempest folks hang out - infra?19:00
*** cdl has left #openstack-neutron19:01
*** azbiswas has quit IRC19:02
*** arajagopal has quit IRC19:02
*** afazekas has quit IRC19:03
*** dannywilson has quit IRC19:04
*** klindgren_ has quit IRC19:05
*** dannywilson has joined #openstack-neutron19:05
*** Aish has joined #openstack-neutron19:05
*** piyanai has quit IRC19:06
*** tiswanso has joined #openstack-neutron19:07
*** arajagopal has joined #openstack-neutron19:08
*** Samos123 has quit IRC19:08
*** achanda has joined #openstack-neutron19:08
*** sbelous_ has quit IRC19:09
*** piyanai has joined #openstack-neutron19:10
*** emagana has quit IRC19:10
*** dannywilson has quit IRC19:10
*** Samos123 has joined #openstack-neutron19:10
*** lori is now known as lori|away19:10
*** dannywilson has joined #openstack-neutron19:10
*** cbouch has quit IRC19:12
*** tfukushima has quit IRC19:12
*** amuller_afk is now known as amuller19:14
*** srilatha has quit IRC19:16
*** azbiswas has joined #openstack-neutron19:17
*** srilatha has joined #openstack-neutron19:17
*** tkatarki has quit IRC19:17
*** kbringard has quit IRC19:18
*** kbringard has joined #openstack-neutron19:18
*** pal has joined #openstack-neutron19:19
*** tfukushima has joined #openstack-neutron19:19
openstackgerritMiguel Lavalle proposed openstack/neutron: External DNS driver reference implementation  https://review.openstack.org/21221319:21
*** pc_m_ has quit IRC19:21
*** srilatha has quit IRC19:21
*** madhu_ak has quit IRC19:22
*** gangil has quit IRC19:22
*** kbringard has quit IRC19:23
*** tiswanso has quit IRC19:23
*** dims_ has joined #openstack-neutron19:26
*** vishwana_ has joined #openstack-neutron19:26
*** ivar-lazzaro has joined #openstack-neutron19:27
*** tfukushima has quit IRC19:27
*** jprovazn has quit IRC19:27
*** ijw_ has joined #openstack-neutron19:27
*** signed8bit is now known as signed8bit_ZZZzz19:27
*** rward has quit IRC19:27
*** jckasper has joined #openstack-neutron19:27
*** rex_lee has quit IRC19:27
*** jamielennox|away has quit IRC19:27
*** clev is now known as clev-away19:28
*** signed8bit_ZZZzz is now known as signed8bit19:28
*** dims has quit IRC19:28
*** ijw has quit IRC19:28
*** puck has quit IRC19:28
*** adam_g has quit IRC19:28
*** piyanai has quit IRC19:28
*** ivar-la__ has quit IRC19:28
*** r-daneel has quit IRC19:28
*** vishwanathj has quit IRC19:28
*** sbalukoff has quit IRC19:28
*** otherwiseguy has quit IRC19:28
*** dguitarbite has quit IRC19:28
*** dsneddon has quit IRC19:28
*** serverascode has quit IRC19:28
*** tiswanso has joined #openstack-neutron19:28
*** sbelous_ has joined #openstack-neutron19:29
*** irenab has quit IRC19:29
*** piyanai has joined #openstack-neutron19:29
*** dave-mccowan has joined #openstack-neutron19:30
*** pal has quit IRC19:30
*** clev-away is now known as clev19:30
*** rward has joined #openstack-neutron19:31
*** madhu_ak has joined #openstack-neutron19:31
*** serverascode has joined #openstack-neutron19:32
*** rm_work is now known as rm_work|away19:32
*** tkatarki has joined #openstack-neutron19:32
*** signed8bit is now known as signed8bit_ZZZzz19:32
*** dave-mcc_ has quit IRC19:33
*** achanda has quit IRC19:34
*** rex_lee has joined #openstack-neutron19:34
*** nlahouti has quit IRC19:35
*** puck has joined #openstack-neutron19:35
*** adam_g has joined #openstack-neutron19:36
*** adam_g has quit IRC19:36
*** adam_g has joined #openstack-neutron19:36
*** signed8bit_ZZZzz is now known as signed8bit19:36
*** padkrish has quit IRC19:36
*** ivar-lazzaro has quit IRC19:39
*** s3wong has quit IRC19:39
*** crose has joined #openstack-neutron19:39
*** vern has quit IRC19:39
*** Samos123 has quit IRC19:40
*** Marga_ has quit IRC19:40
*** moha_hunt has joined #openstack-neutron19:40
*** vern has joined #openstack-neutron19:40
*** abhiraut has quit IRC19:40
*** Marga_ has joined #openstack-neutron19:41
*** dguitarbite has joined #openstack-neutron19:41
*** salv-orlando has joined #openstack-neutron19:41
*** sbalukoff has joined #openstack-neutron19:42
*** dsneddon has joined #openstack-neutron19:43
*** breitz has quit IRC19:43
*** Samos123 has joined #openstack-neutron19:43
*** breitz has joined #openstack-neutron19:43
*** tkatarki has quit IRC19:43
*** nlahouti has joined #openstack-neutron19:44
*** r-daneel has joined #openstack-neutron19:44
*** j^2 has quit IRC19:45
*** otherwiseguy has joined #openstack-neutron19:45
*** j^2 has joined #openstack-neutron19:46
*** sudipto has quit IRC19:46
*** armax has joined #openstack-neutron19:47
*** alejandrito has quit IRC19:47
*** jamielennox|away has joined #openstack-neutron19:47
*** jamielennox|away is now known as jamielennox19:47
*** jbell8 has quit IRC19:49
*** tkatarki has joined #openstack-neutron19:49
*** alejandrito has joined #openstack-neutron19:49
*** tkatarki has quit IRC19:50
*** tkatarki has joined #openstack-neutron19:51
*** setuid has quit IRC19:52
*** nlahouti has quit IRC19:52
*** klindgren has joined #openstack-neutron19:52
*** padkrish has joined #openstack-neutron19:52
*** padkrish has quit IRC19:52
*** padkrish has joined #openstack-neutron19:52
*** ajmiller has quit IRC19:53
*** srilatha has joined #openstack-neutron19:53
*** padkrish has quit IRC19:56
*** setuid has joined #openstack-neutron19:59
*** kbringard has joined #openstack-neutron19:59
*** ushkalim has quit IRC20:00
*** neophy has quit IRC20:01
*** ronis__ has quit IRC20:01
*** annegentle has quit IRC20:04
salv-orlandothere's a weird calm in this channel today.20:04
*** annegentle has joined #openstack-neutron20:04
sbelous_  Still waters run deep20:06
*** sridhar_ram1 has joined #openstack-neutron20:07
*** sridhar_ram has quit IRC20:08
*** sridhar_ram1 is now known as sridhar_ram20:09
*** alejandrito has quit IRC20:09
*** Steap has quit IRC20:11
*** ushkalim has joined #openstack-neutron20:13
*** achanda has joined #openstack-neutron20:13
*** alejandrito has joined #openstack-neutron20:13
*** irenab has joined #openstack-neutron20:15
*** irenab has quit IRC20:15
salv-orlandozzzeek: I have a nested transaction that when run concurrently by several processes triggers this error " (1305, u'SAVEPOINT sa_savepoint_1 does not exist') [SQL: u'ROLLBACK TO SAVEPOINT sa_savepoint_1']" - I know I did something wrong, but does sqlalchemy offer some tools for understanding what caused the parent transaction to rollback as well?20:17
*** tkatarki has quit IRC20:17
*** neophy has joined #openstack-neutron20:17
zzzeeksalv-orlando: entirelyt separate python processes?20:17
salv-orlandothere are unfortunately too many events in the mysql log, so it is very hard to correlate there. These failure occurs under heavy load, with multiple workers doing neutron API operations concurrently20:18
salv-orlandozzzeek: yes20:18
salv-orlandoI mean they're forked from each other20:18
salv-orlandothe neutron API workers20:18
zzzeeksalv-orlando: um.   i would think the name of a SAVEPOINT in mysql is local to the connection20:18
zzzeeksalv-orlando: because it looks like the same savepoint is being rolled back elsewhere20:18
*** sthillma has quit IRC20:18
salv-orlandozzzeek: I thought that was not possible?20:19
*** mickeys has joined #openstack-neutron20:19
*** achanda has quit IRC20:19
zzzeeksalv-orlando: shouldnt be?   seems like this would have come up earlier if not20:19
*** emagana has joined #openstack-neutron20:20
salv-orlandozzzeek: I guess you're right. So another connection is probably rolling back the savepoint. This is consistent with what I see in the logs actually.20:20
zzzeeksalv-orlando: execpt that’s not how it works :)20:20
zzzeeksalv-orlando: *unless*.  you are sharing the connection down to the child fork20:20
salv-orlandozzzeek: that's what I'm frightened about. But I guess it's not the case20:21
zzzeeksalv-orlando: jsut tested.  they are definitely independent20:21
salv-orlandowell, let's say I hope it's not the case20:21
zzzeeksalv-orlando: I’d see if you can log the session id perhaps20:22
zzzeeksalv-orlando: that is, the MySQL session id20:22
zzzeekhttp://dev.mysql.com/doc/refman/5.1/en/information-functions.html#function_connection-id20:22
*** tidwellr has quit IRC20:22
zzzeeksalv-orlando: also is this galera multi-master ?20:22
salv-orlandozzzeek: thanks this helps a lot. Nope it's not galera.20:22
zzzeeksalv-orlando: ok20:22
*** tidwellr has joined #openstack-neutron20:23
zzzeeksalv-orlando: teh other thing that could happen is that in one process you are sharing the connection or session around to multiple greenlets20:23
zzzeeksalv-orlando: if neutron doesnt have its ducks in a row as far as how a request maps to a greenlet maps to a Session, that will wreak havoc20:23
zzzeeksalv-orlando: ive tried to follow this code in various apps like nova and i can’t follow it20:24
zzzeeksalv-orlando: e.g. the wsgi->greenlet system20:24
*** Guest100 has joined #openstack-neutron20:24
*** Guest100 has quit IRC20:24
salv-orlandozzzeek: neutron is done in a way that every API request gets its own session; unless we are so dumb that we believe we've done this and we're actually sharing sessions20:24
zzzeeksalv-orlando: I’d check on that20:25
*** crose has quit IRC20:25
zzzeeksalv-orlando: or if you are spawning greenlets in a request20:25
zzzeeksalv-orlando: or real threads20:25
*** rossella_s has quit IRC20:25
zzzeeksalv-orlando: i think connection_id() mysql function can tell all on this20:25
*** rossella_s has joined #openstack-neutron20:25
salv-orlandozzzeek: It's all sequentially processed. Yes connection_id will surely shed some light. Thanks for the advice. I have solved the issue in another way - but I still want to get to the bottom of this issue because it's not the first time that I see it20:27
zzzeeksalv-orlando: yes this is critical20:27
*** jwy has quit IRC20:27
salv-orlandoand I never managed to fully understand its cause. I always solve it by  reworking the logic so that I do not execute anymore the DB operation that triggers it20:27
*** ushkalim has quit IRC20:31
*** achanda has joined #openstack-neutron20:32
*** jwy has joined #openstack-neutron20:32
*** abhiraut has joined #openstack-neutron20:33
salv-orlandozzzeek: anyway thanks very much. I will abuse of your kindness again should I need further assistance20:34
zzzeeksalv-orlando: please do20:34
*** Sukhdev has quit IRC20:34
*** tidwellr1 has joined #openstack-neutron20:35
*** gangil has joined #openstack-neutron20:36
*** tidwellr has quit IRC20:37
*** signed8bit is now known as signed8bit_ZZZzz20:37
*** padkrish has joined #openstack-neutron20:38
*** alex_klimov has joined #openstack-neutron20:39
*** DWfuturetec has joined #openstack-neutron20:41
*** boris-42 has joined #openstack-neutron20:41
*** signed8bit_ZZZzz is now known as signed8bit20:41
*** padkrish has quit IRC20:42
*** ushkalim has joined #openstack-neutron20:43
*** mickeys has quit IRC20:45
*** mickeys has joined #openstack-neutron20:46
*** emagana has quit IRC20:47
*** armax has quit IRC20:49
*** ameybhide has joined #openstack-neutron20:51
*** regXboi has quit IRC20:55
*** srilatha has quit IRC20:57
*** achanda has quit IRC20:57
*** achanda_ has joined #openstack-neutron20:57
*** nyechiel_ has joined #openstack-neutron21:00
*** dannywil_ has joined #openstack-neutron21:00
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532921:00
*** sthillma has joined #openstack-neutron21:01
*** ivar-lazzaro has joined #openstack-neutron21:02
*** dannywilson has quit IRC21:03
*** sthillma has quit IRC21:04
*** sthillma has joined #openstack-neutron21:04
*** tiswanso has quit IRC21:05
*** dsneddon has quit IRC21:07
*** yamahata has joined #openstack-neutron21:10
*** regXboi has joined #openstack-neutron21:11
*** debedb has joined #openstack-neutron21:11
*** rm_work|away is now known as rm_work21:13
*** crose has joined #openstack-neutron21:13
*** nyechiel_ has quit IRC21:14
*** thorst has quit IRC21:16
*** annegentle has quit IRC21:20
*** annegentle has joined #openstack-neutron21:20
*** tiny-hands has left #openstack-neutron21:21
*** dsneddon has joined #openstack-neutron21:24
*** Aish has quit IRC21:24
*** padkrish has joined #openstack-neutron21:24
*** nlahouti has joined #openstack-neutron21:24
*** krtaylor has quit IRC21:26
*** armax has joined #openstack-neutron21:26
*** abhiraut has quit IRC21:27
*** sbelous_ has quit IRC21:28
*** dguerri` is now known as dguerri21:30
*** rpothier has quit IRC21:30
*** dguerri is now known as dguerri`21:31
*** crose has quit IRC21:32
*** TrevorV has quit IRC21:33
openstackgerritIvar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation  https://review.openstack.org/21442421:33
*** clev is now known as clev-away21:33
*** Aish has joined #openstack-neutron21:33
*** clev-away is now known as clev21:34
openstackgerritMerged openstack/neutron: Python 3: encode or decode i/o data of Popen.communicate()  https://review.openstack.org/20884821:34
openstackgerritMerged openstack/neutron: Add EnvironmentDescription, pass it down  https://review.openstack.org/21083021:34
*** gangil has quit IRC21:34
*** Piet has joined #openstack-neutron21:35
*** gangil has joined #openstack-neutron21:38
*** nlahouti has quit IRC21:38
*** dannywil_ has quit IRC21:40
*** nlahouti has joined #openstack-neutron21:41
*** absubram has quit IRC21:42
*** annegentle has quit IRC21:42
*** edmondsw has quit IRC21:44
*** absubram has joined #openstack-neutron21:47
*** alejandrito has quit IRC21:49
*** bapalm has quit IRC21:49
*** arajagopal has quit IRC21:51
*** bapalm has joined #openstack-neutron21:52
*** nlahouti has left #openstack-neutron21:52
*** nlahouti has joined #openstack-neutron21:52
*** alejandrito has joined #openstack-neutron21:52
openstackgerritRyan Moats proposed openstack/neutron: Introduce report_counters RPC for agents  https://review.openstack.org/21347421:53
openstackgerritRyan Moats proposed openstack/neutron: Add instrumentation devref  https://review.openstack.org/20428221:53
*** ushkalim has quit IRC21:53
*** srilatha has joined #openstack-neutron21:54
*** dannywilson has joined #openstack-neutron21:54
*** britthouser has quit IRC21:54
*** signed8bit has quit IRC21:55
*** piyanai has quit IRC21:56
*** bapalm has quit IRC21:57
Sam-I-Amkevinbenton: when you're around (maybe on this side of the world) i found some issues with the mac spoofing business on linuxbridge (might also apply to ovs)21:58
*** dmartls1 has quit IRC21:59
*** bapalm has joined #openstack-neutron22:03
*** jobewan has joined #openstack-neutron22:03
*** lindycoder has quit IRC22:04
*** claudiub has quit IRC22:05
*** kbringard has quit IRC22:05
*** bapalm has quit IRC22:08
*** s3wong has joined #openstack-neutron22:12
openstackgerritCedric Brandily proposed openstack/neutron: Move in-tree vendor VIF_TYPE_* constants  https://review.openstack.org/21480522:12
openstackgerritCedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants  https://review.openstack.org/21480622:12
openstackgerritSalvatore Orlando proposed openstack/neutron: WIP: Fix reservation engine  https://review.openstack.org/21536322:13
*** bapalm has joined #openstack-neutron22:14
*** sbelous_ has joined #openstack-neutron22:16
*** krtaylor has joined #openstack-neutron22:18
*** bapalm has quit IRC22:19
*** annegentle has joined #openstack-neutron22:20
*** chlong has quit IRC22:25
*** sbelous has quit IRC22:26
*** sbelous_ has quit IRC22:26
*** amuller has quit IRC22:27
*** sbelous has joined #openstack-neutron22:27
*** alex_klimov has quit IRC22:28
*** sbelous has quit IRC22:28
*** bapalm has joined #openstack-neutron22:29
*** thorst has joined #openstack-neutron22:32
*** annegentle has quit IRC22:33
*** annegentle has joined #openstack-neutron22:33
mlavallekevinbenton: ping22:33
*** bapalm has quit IRC22:34
*** shwetaap has quit IRC22:34
*** Aish has quit IRC22:35
openstackgerritSalvatore Orlando proposed openstack/neutron: Used namedtuple for ReservationInfo  https://review.openstack.org/21536922:35
openstackgerritPengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255]  https://review.openstack.org/21222022:35
*** bapalm has joined #openstack-neutron22:35
*** tfukushima has joined #openstack-neutron22:37
*** annegentle has quit IRC22:38
*** emagana has joined #openstack-neutron22:40
*** bapalm has quit IRC22:43
*** bapalm has joined #openstack-neutron22:43
*** bapalm has quit IRC22:48
*** dannywilson has quit IRC22:48
*** jobewan has quit IRC22:49
*** bapalm has joined #openstack-neutron22:51
*** s3wong has quit IRC22:52
*** rkukura has quit IRC22:52
*** thurloat_isgone is now known as thurloat22:52
*** tfukushima has quit IRC22:55
*** thorst has quit IRC22:55
*** dannywilson has joined #openstack-neutron22:57
*** rm_work is now known as rm_work|away23:01
*** rkukura has joined #openstack-neutron23:01
*** sridhar_ram has quit IRC23:02
*** jobewan has joined #openstack-neutron23:02
*** sridhar_ram has joined #openstack-neutron23:02
*** thurloat is now known as thurloat_isgone23:03
*** achanda_ has quit IRC23:05
*** alejandrito has quit IRC23:06
*** DWfuturetec has quit IRC23:07
*** tiny-hands has joined #openstack-neutron23:07
*** tidwellr1 has left #openstack-neutron23:09
*** emagana has quit IRC23:14
*** kutija has quit IRC23:15
*** regXboi has quit IRC23:15
*** achanda has joined #openstack-neutron23:16
*** dannywilson has quit IRC23:19
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns  https://review.openstack.org/21532923:24
*** zzzeek has quit IRC23:25
*** sridhar_ram has quit IRC23:26
*** r-daneel has quit IRC23:27
*** sridhar_ram has joined #openstack-neutron23:28
*** Fdaisuke has joined #openstack-neutron23:28
*** dannywilson has joined #openstack-neutron23:30
Fdaisukearmax: Hi,23:30
Fdaisukearmax: I commented to https://review.openstack.org/#/c/207328/23:32
*** ameybhid_ has joined #openstack-neutron23:32
FdaisukeWould you please have a look at this?23:32
*** ameybhid_ has quit IRC23:33
*** banix has quit IRC23:33
*** ameybhide has quit IRC23:34
*** s3wong has joined #openstack-neutron23:35
*** regXboi has joined #openstack-neutron23:37
*** debedb has quit IRC23:38
*** arosen has quit IRC23:40
*** sridhar_ram has quit IRC23:41
*** sridhar_ram has joined #openstack-neutron23:42
*** SourabhP has quit IRC23:43
*** Marga_ has quit IRC23:46
*** salv-orlando has quit IRC23:46
*** Marga_ has joined #openstack-neutron23:47
armaxFdaisuke: done23:48
*** Swami has quit IRC23:48
*** cdl has joined #openstack-neutron23:50
*** cdl has left #openstack-neutron23:51
*** debedb has joined #openstack-neutron23:51
Fdaisukearmax: thanks. I'll check.23:52
*** srilatha has quit IRC23:54
*** srilatha has joined #openstack-neutron23:54
*** gildub has joined #openstack-neutron23:55
*** debedb has quit IRC23:55
*** dannywilson has quit IRC23:56
*** debedb has joined #openstack-neutron23:56
*** SumitNaiksatam has quit IRC23:59
*** srilatha has quit IRC23:59

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