Monday, 2016-03-14

*** markvoelker has quit IRC00:00
openstackgerritSofia Enriquez proposed openstack/neutron: Port allowed address pairs to OVO  https://review.openstack.org/26827400:03
openstackgerritSofia Enriquez proposed openstack/neutron: [WIP] Port security  https://review.openstack.org/29217800:03
*** vik_ has quit IRC00:04
*** wolverineav has quit IRC00:05
*** vik_ has joined #openstack-neutron00:09
*** jckasper has joined #openstack-neutron00:10
openstackgerritSofia Enriquez proposed openstack/neutron: [WIP] Port security  https://review.openstack.org/29217800:12
*** baojg has quit IRC00:13
openstackgerritSofia Enriquez proposed openstack/neutron: [WIP] Port security  https://review.openstack.org/29217800:14
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp to neutron ext resources.  https://review.openstack.org/29023400:14
openstackgerritSofia Enriquez proposed openstack/neutron: [WIP] Port security  https://review.openstack.org/29217800:16
*** Shiv_ has joined #openstack-neutron00:16
*** jckasper has quit IRC00:18
*** Shiv has quit IRC00:19
*** Shiv_ is now known as Shiv00:19
*** jckasper has joined #openstack-neutron00:19
*** Shiv has quit IRC00:21
*** dims_ has joined #openstack-neutron00:27
*** dims has quit IRC00:28
*** jckasper has quit IRC00:28
*** jckasper has joined #openstack-neutron00:29
*** dims has joined #openstack-neutron00:33
*** namnh has joined #openstack-neutron00:33
*** jckasper has quit IRC00:33
*** dims_ has quit IRC00:33
openstackgerritfumihiko kakuma proposed openstack/neutron: DO NOT REVIEW: sandbox for Ryu-CI  https://review.openstack.org/18294300:34
*** jckasper has joined #openstack-neutron00:39
*** EinstCrazy has quit IRC00:40
*** vhosakot has joined #openstack-neutron00:41
openstackgerritLi Xipeng proposed openstack/neutron: Add 169.254.169.254 when enable force_metadata  https://review.openstack.org/28634500:43
*** namnh has quit IRC00:44
*** mfuruta has joined #openstack-neutron00:50
*** chlong has quit IRC00:51
*** zengine_ has joined #openstack-neutron00:53
*** jckasper has quit IRC00:54
*** david-lyle has quit IRC00:54
*** david-lyle has joined #openstack-neutron00:54
*** jckasper has joined #openstack-neutron00:55
*** zengine has quit IRC00:55
*** zengine_ is now known as zengine00:55
*** jckasper has quit IRC00:58
*** jckasper has joined #openstack-neutron00:58
*** cshahani has quit IRC01:00
*** apuimedo has quit IRC01:00
*** dave-mccowan has quit IRC01:01
*** djan_ has joined #openstack-neutron01:02
*** djan has quit IRC01:04
*** wolverineav has joined #openstack-neutron01:05
*** chlong has joined #openstack-neutron01:06
*** hoangcx has joined #openstack-neutron01:07
*** wolverineav has quit IRC01:09
*** gvrangan has quit IRC01:10
*** akshai has quit IRC01:14
*** jckasper has quit IRC01:16
*** jckasper has joined #openstack-neutron01:17
*** Linzi has joined #openstack-neutron01:18
*** jckasper has quit IRC01:21
*** stanzgy has joined #openstack-neutron01:24
openstackgerritli,chen proposed openstack/neutron: Add RECLONE support for ovs  https://review.openstack.org/28223301:24
*** kevo has joined #openstack-neutron01:26
kevoHi, question for neutron lbaas-member-update command. Is the --admin-state-down option still supported?01:26
openstackgerritli,chen proposed openstack/neutron: Add RECLONE support for ovs  https://review.openstack.org/28223301:29
LinziHi all, I want to contribute a neutron ml2 driver in openstack newton release.01:35
LinziSo I look into the schedlue of the openstack mitaka to find some clues, however, there is no information for proposing the bluepront and freature of neutron.01:35
LinziIs that means there is no Neutron subproject which can be proposed in the openstack newton?01:36
LinziOr should I wait until the new schedule announcement after newton design summit(Apr 25-29)? Thank you.01:36
*** panda has quit IRC01:40
*** panda has joined #openstack-neutron01:41
*** djan has joined #openstack-neutron01:44
*** zengine has quit IRC01:44
*** djan_ has quit IRC01:45
*** zengine has joined #openstack-neutron01:49
*** Sukhdev has joined #openstack-neutron01:51
*** Sukhdev has quit IRC01:53
*** Sukhdev has joined #openstack-neutron01:54
*** baojg has joined #openstack-neutron01:55
*** markvoelker has joined #openstack-neutron01:56
*** zhenguo_ has joined #openstack-neutron01:58
*** thorst has joined #openstack-neutron02:00
*** markvoelker has quit IRC02:00
*** baoli has joined #openstack-neutron02:00
*** thorst has quit IRC02:05
*** wolverineav has joined #openstack-neutron02:07
*** chlong has quit IRC02:07
*** dims has quit IRC02:07
*** mfuruta has quit IRC02:09
*** mfuruta has joined #openstack-neutron02:09
*** rkukura has quit IRC02:10
*** mfuruta has quit IRC02:10
*** mfuruta has joined #openstack-neutron02:10
*** mfuruta has quit IRC02:11
*** wolverineav has quit IRC02:11
*** mfuruta has joined #openstack-neutron02:12
*** tflynn has joined #openstack-neutron02:14
*** fangzhen__ has quit IRC02:15
*** fangzhen__ has joined #openstack-neutron02:15
*** fzdarsky__ has joined #openstack-neutron02:16
*** ajmiller has quit IRC02:18
*** mfuruta has quit IRC02:19
*** mfuruta has joined #openstack-neutron02:19
*** mfuruta has joined #openstack-neutron02:19
*** fzdarsky_ has quit IRC02:20
openstackgerrityalei wang proposed openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once  https://review.openstack.org/29160302:20
*** mfuruta has joined #openstack-neutron02:20
*** chlong has joined #openstack-neutron02:20
*** mfuruta has quit IRC02:20
*** mfuruta has joined #openstack-neutron02:21
*** mfuruta has quit IRC02:21
*** dims has joined #openstack-neutron02:21
*** mfuruta has joined #openstack-neutron02:21
*** fangzhen has joined #openstack-neutron02:26
*** fangzhen__ has quit IRC02:28
openstackgerritLujin Luo proposed openstack/neutron: Avoid multiple dhcp-agents being auto-scheduled to one network  https://review.openstack.org/28827102:35
*** cshahani has joined #openstack-neutron02:38
*** fangzhen has quit IRC02:39
*** sbalukoff has joined #openstack-neutron02:40
*** akshai has joined #openstack-neutron02:40
*** fangzhen_ has joined #openstack-neutron02:40
*** dims has quit IRC02:41
*** tflynn has quit IRC02:43
*** ljianbj has joined #openstack-neutron02:43
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: fullstack: use SIGTERM when stopping ovs agents  https://review.openstack.org/29027702:45
*** fangzhen_ is now known as fangzhen02:45
*** ljianbj has left #openstack-neutron02:46
*** haukebruno has joined #openstack-neutron02:46
*** ljianbj has joined #openstack-neutron02:46
*** fangzhen has quit IRC02:46
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: Fix a couple of problems with random hex string generation  https://review.openstack.org/28822402:47
*** hoangcx has quit IRC02:47
*** sbalukoff has quit IRC02:52
*** yamamoto has quit IRC02:55
*** iwamoto has joined #openstack-neutron02:56
*** rkukura has joined #openstack-neutron02:57
*** akshai_ has joined #openstack-neutron02:59
*** akshai has quit IRC03:02
*** baoli has quit IRC03:03
*** fangzhen_ has joined #openstack-neutron03:03
*** baoli has joined #openstack-neutron03:04
*** dave-mccowan has joined #openstack-neutron03:06
*** emagana has joined #openstack-neutron03:13
openstackgerritFang Zhen proposed openstack/neutron: Switch to oslo.cache  https://review.openstack.org/29030503:14
*** kevo has quit IRC03:15
*** emagana has quit IRC03:22
*** chlong has quit IRC03:22
openstackgerrityalei wang proposed openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once  https://review.openstack.org/29160303:23
*** dave-mccowan has quit IRC03:26
*** banix has quit IRC03:26
*** shivrao has joined #openstack-neutron03:27
*** Sukhdev has quit IRC03:32
*** akshai_ has quit IRC03:34
*** lujinluo has quit IRC03:37
*** chlong has joined #openstack-neutron03:41
*** tflynn has joined #openstack-neutron03:41
openstackgerritKevin Benton proposed openstack/neutron: Call expunge_all on session before retrying requests  https://review.openstack.org/29179503:42
openstackgerritKevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe  https://review.openstack.org/28287603:44
*** dkehn has quit IRC03:45
*** sridharg has joined #openstack-neutron03:45
*** tflynn has quit IRC03:47
*** links has joined #openstack-neutron03:50
*** dkehn has joined #openstack-neutron03:51
*** Sukhdev has joined #openstack-neutron03:53
*** yamamoto has joined #openstack-neutron03:56
*** markvoelker has joined #openstack-neutron03:57
*** ajmiller has joined #openstack-neutron04:00
*** ekarlso- has quit IRC04:00
*** baoli has quit IRC04:01
*** lujinluo has joined #openstack-neutron04:01
*** markvoelker has quit IRC04:01
*** ajmiller_ has joined #openstack-neutron04:02
*** ajmiller has quit IRC04:06
*** andymaier has joined #openstack-neutron04:07
*** wolverineav has joined #openstack-neutron04:08
*** ajmiller_ has quit IRC04:09
*** tbachman has quit IRC04:11
*** wolverineav has quit IRC04:13
*** networkninja has quit IRC04:13
*** ekarlso- has joined #openstack-neutron04:13
*** jckasper has joined #openstack-neutron04:19
*** andymaier has quit IRC04:23
*** hoangcx has joined #openstack-neutron04:24
*** jckasper has quit IRC04:26
*** jckasper has joined #openstack-neutron04:26
*** emagana has joined #openstack-neutron04:27
*** thorst has joined #openstack-neutron04:27
*** jckasper has quit IRC04:28
*** jckasper has joined #openstack-neutron04:28
*** abregman has quit IRC04:31
*** y_sawai has joined #openstack-neutron04:31
*** baoli has joined #openstack-neutron04:32
openstackgerritLi Xipeng proposed openstack/neutron: Add 169.254.169.254 when enable force_metadata  https://review.openstack.org/28634504:32
*** baoli has quit IRC04:33
*** thorst has quit IRC04:35
*** chlong has quit IRC04:35
*** jckasper has quit IRC04:38
*** armax has joined #openstack-neutron04:38
*** jckasper has joined #openstack-neutron04:38
*** jckasper has quit IRC04:42
*** jckasper has joined #openstack-neutron04:42
*** prapulla has joined #openstack-neutron04:46
*** chlong has joined #openstack-neutron04:47
*** baojg has quit IRC04:52
*** jckasper has quit IRC04:52
*** jckasper has joined #openstack-neutron04:53
*** baojg has joined #openstack-neutron04:53
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add API test for tag  https://review.openstack.org/29220404:54
*** djan_ has joined #openstack-neutron04:56
*** djan has quit IRC04:57
*** jckasper has quit IRC04:57
*** otherwiseguy has quit IRC05:02
*** sridharg has quit IRC05:03
*** chlong has quit IRC05:03
*** tbh has joined #openstack-neutron05:05
*** boris-42 has joined #openstack-neutron05:06
*** gvrangan has joined #openstack-neutron05:08
*** gberginc has quit IRC05:08
*** cshahani has quit IRC05:09
*** y_sawai has quit IRC05:10
*** gvrangan has quit IRC05:12
*** otherwiseguy has joined #openstack-neutron05:12
*** ekuris_ has joined #openstack-neutron05:15
*** chlong has joined #openstack-neutron05:16
*** vishwana_ has joined #openstack-neutron05:16
*** vishwanathj_away has quit IRC05:18
openstackgerritRyan Tidwell proposed openstack/neutron: Compute IPAvailabilityRanges in memory during IP allocation  https://review.openstack.org/29220705:19
*** chlong has quit IRC05:22
*** emagana has quit IRC05:23
*** lazy_prince has joined #openstack-neutron05:27
*** EinstCrazy has joined #openstack-neutron05:31
*** y_sawai has joined #openstack-neutron05:31
*** thorst has joined #openstack-neutron05:33
*** chlong has joined #openstack-neutron05:34
*** kiran-r has joined #openstack-neutron05:37
*** EinstCrazy has quit IRC05:38
*** thorst has quit IRC05:41
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add API test for tag  https://review.openstack.org/29220405:42
*** javeriak has joined #openstack-neutron05:43
*** gberginc has joined #openstack-neutron05:44
*** fawadkhaliq has joined #openstack-neutron05:46
*** tflynn has joined #openstack-neutron05:46
*** gberginc has quit IRC05:48
*** sridharg has joined #openstack-neutron05:48
*** y_sawai has quit IRC05:48
*** irenab has quit IRC05:51
*** tflynn has quit IRC05:52
*** Linzi has left #openstack-neutron05:54
*** korzen has joined #openstack-neutron05:54
*** tbh has quit IRC05:54
*** mkolesni has joined #openstack-neutron05:54
*** y_sawai has joined #openstack-neutron05:54
*** dedery has joined #openstack-neutron05:56
openstackgerritMerged openstack/neutron: Extend dicts with original model in create/update  https://review.openstack.org/29176505:57
openstackgerritTang Chen proposed openstack/neutron: Fix dict.keys() incompatibility in py34 in dhcp test  https://review.openstack.org/29204805:57
*** markvoelker has joined #openstack-neutron05:58
*** emagana has joined #openstack-neutron05:58
*** vishwana_ is now known as vishwanathj_away05:58
*** javeriak_ has joined #openstack-neutron05:59
*** y_sawai has quit IRC06:01
*** javeriak has quit IRC06:01
*** dedery has quit IRC06:01
*** ihrachys has joined #openstack-neutron06:02
*** markvoelker has quit IRC06:02
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Zanata  https://review.openstack.org/29221406:03
ihrachysarmax: howdy!06:03
armaxihrachys: yello06:03
ihrachysarmax: *now* I realize why you were talking about India. Silly me :)06:04
armaxihrachys: there you go06:04
ihrachystook me just three days06:04
armaxihrachys: 3 is better than 406:05
armaxor 5 or 606:05
*** abregman has joined #openstack-neutron06:05
*** dedery has joined #openstack-neutron06:06
armaxihrachys: I replied to your email06:06
*** tbh has joined #openstack-neutron06:06
*** gvrangan has joined #openstack-neutron06:07
armaxihrachys: let me know if you want to discuss this further, at the end of the day…if there’s nothing worth merging…we’ll have to pull the trigger at some point06:07
armaxI’d say let’s give the opportunity to the US folks to go through their Monday to refresh/rebase some of the patches that have a chance to land06:08
ihrachysarmax: I checked it just now, I think the plan is good.06:08
armaxihrachys: what about this one? https://review.openstack.org/#/c/280595/06:08
armaxihrachys: in the meantime if there’s anything low risk that is ovo related that you want to unblock, let me know and we’ll vet it together06:08
ihrachysarmax: we could land it. not that it's actually needed to do it though: later backport would be as easy.06:09
ihrachysarmax: aye, I was in the middle of preparing the list of safe ovo patches to unblock06:09
armaxihrachys: it depends though06:09
armaxihrachys: by how ugly any potential follow-up is needed ;)06:09
ihrachysarmax: I mean, I am not sure what merge now buys us06:10
*** Fdaisuke has quit IRC06:10
ihrachysbut since it's not used... it could go06:10
armaxa smaller backport later06:10
*** ircuser-1 has joined #openstack-neutron06:12
*** kevo has joined #openstack-neutron06:13
*** neeti has joined #openstack-neutron06:14
*** marios_ has joined #openstack-neutron06:16
openstackgerritSreekumar S proposed openstack/neutron: SG protocol validation to allow numbers or names  https://review.openstack.org/27130106:16
*** vhosakot has quit IRC06:16
kevinbentonarmax: ping06:18
armaxkevinbenton: pong?06:19
*** ihrachys_ has joined #openstack-neutron06:19
kevinbentonarmax: i didn't put the expunge in the decorator because of the ugliness that previous decorator approach required to deal with all of the different positions the context could land in06:19
*** sridhar_ram has joined #openstack-neutron06:19
kevinbentone.g. first arg vs second arg vs inside of the request arge06:19
kevinbentonarg*06:19
armaxkevinbenton: oh, right06:19
ihrachys_armax: I just dropped the list of ovo patches via email that I believe are safe to unblock06:19
armaxihrachys_: ok, let me get to it06:20
ihrachys_it's not too much, should be rather innocent06:20
armaxkevinbenton: let me mull over it a bit longer06:20
armaxihrachys_: ack06:21
ihrachys_armax: I will work with hmlnarik to get https://review.openstack.org/#/c/284639/ in shape today. it's quite a simple change, would be a shame not to land it.06:21
armaxihrachys_: that would be great06:21
kevinbentonarmax: i can refactor the calls to be more consistent in the positioning of the context06:21
kevinbentonarmax: then the move it to the decorator06:21
armaxihrachys_: make sure you beat the crap out of it06:21
*** ihrachys has quit IRC06:21
armaxkevinbenton: that would be an alternative06:22
*** yfried has joined #openstack-neutron06:22
armaxihrachys_: the patch I mean ;)06:22
*** kevo has quit IRC06:22
ihrachys_lol06:22
ihrachys_I am too weak to persuade in any physical way ;)06:23
*** anilvenkata has joined #openstack-neutron06:23
openstackgerritSreekumar S proposed openstack/neutron: Avoid multiple gw ports on the same router  https://review.openstack.org/27402106:23
openstackgerritSreekumar S proposed openstack/neutron: Huge mem usage during tunnel type driver initialization  https://review.openstack.org/25194006:26
*** vik_ has quit IRC06:28
armaxkevinbenton: assumed that is always safe to call expunge?06:28
openstackgerritvenkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface  https://review.openstack.org/29164106:29
*** itzikb has quit IRC06:29
armaxihrachys_: I’ll keep on cleaning up RC106:30
*** miyagishi_t has quit IRC06:30
armaxI looked at bug 1365461 and bug 1375625 right now06:30
openstackbug 1365461 in neutron "HA router should failover if GW address is not reachable" [Medium,In progress] https://launchpad.net/bugs/1365461 - Assigned to Lubosz Kosnik (diltram)06:30
openstackbug 1375625 in neutron "Problem in l3-agent tenant-network interface would cause split-brain in HA router" [Medium,In progress] https://launchpad.net/bugs/1375625 - Assigned to Lubosz Kosnik (diltram)06:30
*** sridhar_ram has quit IRC06:31
*** irenab has joined #openstack-neutron06:33
ihrachys_armax: btw I don't see your nomination on -dev@ yet. I presume you plan to push it after closing gaps for RC.06:33
kevinbentonarmax: yes, in most cases it will be a noop06:35
openstackgerritSean M. Collins proposed openstack/neutron: Consolidate the OVS DF and CSUM options into constants  https://review.openstack.org/28544006:35
kevinbentonarmax: if it's not a noop we don't want what it has anyway06:35
armaxihrachys_: what nomination?06:39
armaxihrachys_: who says I am running?06:39
kevinbentoni do06:39
ihrachys_same here, yes06:39
kevinbentonsecond term is mandatory06:39
*** pcaruana has quit IRC06:39
armaxkevinbenton: parse error, please explain again what you mean on the expunge stuff06:39
armaxihrachys_, kevinbenton you’ll do fine without me06:40
kevinbentonarmax: all of these options we are retrying are the start of a process. there should be nothing in the session06:40
openstackgerritvenkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver  https://review.openstack.org/28111606:40
kevinbentonarmax: if there is something there, it's leftover garbage06:40
armaxkevinbenton: what process?06:40
*** javeriak_ has quit IRC06:40
kevinbentonarmax: we don't use retries in the middle of an operation06:42
kevinbentonarmax: they are only at the beginning06:42
armaxor the end?06:42
armaxbut what if the whole this is a composite one?06:43
openstackgerritvenkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver  https://review.openstack.org/28111606:43
openstackgerritvenkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports  https://review.openstack.org/24122706:43
*** djan_ has quit IRC06:43
*** john-davidge has joined #openstack-neutron06:44
kevinbentonarmax: i don't think that's possible, did you have something particular in mind?06:44
*** john-davidge has quit IRC06:44
armaxkevinbenton: no, mine is an hypotetical06:44
*** ihrachys has joined #openstack-neutron06:46
*** gberginc has joined #openstack-neutron06:46
*** marios_ has quit IRC06:47
kevinbentonarmax: if there is a composit operation that calls one of these with uncommited data, that's broken IMO06:47
armaxkevinbenton: fair enough, so I wonder if we are fixing a non-issue06:47
*** ihrachys_ has quit IRC06:47
kevinbentonarmax: no, we are fixing the fact that things can be left in the session from the previous iteration06:48
*** chlong has quit IRC06:48
ihrachysarmax: 'you’ll do fine without me ' no way. you should solve the stadium. :P06:48
ihrachyskevinbenton: have a small question on rpc client tests.06:49
ihrachyskevinbenton: in gerrit, I mean06:49
armaxihrachys: I will do that regardless whether I am or not the next PTL06:49
* armax walks away for a few06:50
ihrachysarmax: you never know. you know what always occurs after a dictator leaves his position?06:50
kevinbentonihrachys: ask away06:50
*** oanson has joined #openstack-neutron06:50
ihrachyskevinbenton: I mean, I posted it there. but basically, can we access timeout a bit higher up the stack before we get into oslo internals?06:51
*** korzen has quit IRC06:53
*** emagana has quit IRC06:53
*** ihrachys_ has joined #openstack-neutron06:53
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: Revert use of update_device_list  https://review.openstack.org/27727906:53
*** oshvartz has joined #openstack-neutron06:55
*** ihrachys has quit IRC06:55
*** miyagishi_t has joined #openstack-neutron06:56
*** moshele has joined #openstack-neutron06:56
*** djan has joined #openstack-neutron06:59
*** wolverineav has joined #openstack-neutron07:01
*** thorst has joined #openstack-neutron07:02
*** hichihara has joined #openstack-neutron07:03
*** emagana has joined #openstack-neutron07:03
*** ihrachys_ has quit IRC07:04
*** thorst has quit IRC07:07
openstackgerritshihanzhang proposed openstack/neutron: Optimize conntrack: modify RPC method  https://review.openstack.org/24399407:07
*** shivrao has quit IRC07:07
*** emagana has quit IRC07:07
*** fawadkhaliq has quit IRC07:08
*** hwoarang_ has joined #openstack-neutron07:09
*** igordcar1_ has joined #openstack-neutron07:10
*** hlkv6_ has joined #openstack-neutron07:10
*** garyk has joined #openstack-neutron07:10
*** javeriak has joined #openstack-neutron07:11
*** korzen has joined #openstack-neutron07:12
*** akamyshnikova has quit IRC07:13
*** kobis has joined #openstack-neutron07:13
*** pdr- has joined #openstack-neutron07:14
*** miyagishi_t has quit IRC07:15
*** gvrangan has quit IRC07:15
*** jcook_ has quit IRC07:15
*** Marga_ has quit IRC07:15
*** ccesario_ has quit IRC07:15
*** isq_ has quit IRC07:15
*** eezhova has quit IRC07:15
*** nplanel has quit IRC07:15
*** lezbar has quit IRC07:15
*** pdr_ has quit IRC07:15
*** sgordon has quit IRC07:15
*** bigjools has quit IRC07:15
*** doude has quit IRC07:15
*** hwoarang has quit IRC07:15
*** sirushti has quit IRC07:15
*** andyhky has quit IRC07:15
*** hlkv6 has quit IRC07:15
*** dasm has quit IRC07:15
*** X123 has quit IRC07:15
*** pmath has quit IRC07:15
*** igordcard_ has quit IRC07:15
*** mkoderer has quit IRC07:15
*** bigjools has joined #openstack-neutron07:16
*** miyagishi_t has joined #openstack-neutron07:16
*** gvrangan has joined #openstack-neutron07:16
*** jcook_ has joined #openstack-neutron07:16
*** Marga_ has joined #openstack-neutron07:16
*** ccesario_ has joined #openstack-neutron07:16
*** isq_ has joined #openstack-neutron07:16
*** eezhova has joined #openstack-neutron07:16
*** nplanel has joined #openstack-neutron07:16
*** lezbar has joined #openstack-neutron07:16
*** andyhky has joined #openstack-neutron07:16
*** mkoderer has joined #openstack-neutron07:16
*** sgordon has joined #openstack-neutron07:16
*** doude has joined #openstack-neutron07:16
*** sirushti has joined #openstack-neutron07:16
*** dasm has joined #openstack-neutron07:16
*** X123 has joined #openstack-neutron07:16
*** pmath has joined #openstack-neutron07:16
*** itzikb has joined #openstack-neutron07:24
openstackgerritvenkata anil proposed openstack/neutron: use separate device owner for HA router interface  https://review.openstack.org/29165107:24
openstackgerritKevin Benton proposed openstack/neutron: Add exponential back-off RPC client  https://review.openstack.org/28059507:24
*** djan has quit IRC07:26
openstackgerritvenkata anil proposed openstack/neutron: l2pop fdb flows for HA router ports  https://review.openstack.org/25523707:26
*** korzen has quit IRC07:26
*** akamyshnikova has joined #openstack-neutron07:27
*** itzikb has quit IRC07:28
*** ihrachys has joined #openstack-neutron07:30
*** nmagnezi has joined #openstack-neutron07:31
*** gvrangan has quit IRC07:32
*** apuimedo has joined #openstack-neutron07:33
*** jhershbe has joined #openstack-neutron07:38
*** scheuran has joined #openstack-neutron07:40
*** gampel has joined #openstack-neutron07:41
*** ekuris_ has quit IRC07:45
*** neeti has quit IRC07:45
*** tflynn has joined #openstack-neutron07:50
*** tesseract has joined #openstack-neutron07:52
*** tesseract is now known as Guest5918707:52
*** sridharg has quit IRC07:52
openstackgerritTang Chen proposed openstack/neutron: Fix dict.keys() incompatibility in py34 in dhcp test  https://review.openstack.org/29204807:53
*** mickeys has joined #openstack-neutron07:54
*** sridharg has joined #openstack-neutron07:54
*** evgenyf has joined #openstack-neutron07:55
*** hoangcx has quit IRC07:56
*** Sukhdev has quit IRC07:57
*** Sukhdev has joined #openstack-neutron07:58
*** baojg has quit IRC07:58
*** markvoelker has joined #openstack-neutron07:58
*** emagana has joined #openstack-neutron07:59
*** [1]evgenyf has joined #openstack-neutron07:59
*** ildikov has quit IRC07:59
*** javeriak_ has joined #openstack-neutron08:00
*** evgenyf has quit IRC08:02
*** [1]evgenyf is now known as evgenyf08:02
*** javeriak has quit IRC08:02
openstackgerritArmando Migliaccio proposed openstack/neutron-specs: Provide a release postmortem  https://review.openstack.org/28641308:02
*** emagana has quit IRC08:03
*** tflynn has quit IRC08:03
*** markvoelker has quit IRC08:03
*** javeriak_ has quit IRC08:03
*** numans has joined #openstack-neutron08:04
*** baojg has joined #openstack-neutron08:05
*** jlibosva has joined #openstack-neutron08:06
*** mickeys has quit IRC08:06
*** tmorin has joined #openstack-neutron08:07
*** thorst has joined #openstack-neutron08:07
*** javeriak has joined #openstack-neutron08:10
*** thorst has quit IRC08:11
openstackgerritArmando Migliaccio proposed openstack/neutron-specs: Follow README guidelines in preparation of RC window  https://review.openstack.org/28338308:11
*** jlibosva has quit IRC08:13
*** tmorin has quit IRC08:14
*** tmorin has joined #openstack-neutron08:15
*** jlibosva has joined #openstack-neutron08:16
*** neeti has joined #openstack-neutron08:18
*** nyechiel_ has joined #openstack-neutron08:22
openstackgerritArmando Migliaccio proposed openstack/neutron-specs: Follow README guidelines in preparation of RC window  https://review.openstack.org/28338308:25
*** rcernin has joined #openstack-neutron08:25
*** nmagnezi has quit IRC08:28
*** matrohon has joined #openstack-neutron08:29
*** nmagnezi has joined #openstack-neutron08:29
openstackgerritTakanori Miyagishi proposed openstack/neutron: do not create namespace if snat is disabled  https://review.openstack.org/27205908:29
*** tbh has left #openstack-neutron08:32
*** yfried has quit IRC08:34
*** yfried has joined #openstack-neutron08:34
*** wolverineav has quit IRC08:35
*** lezbar has quit IRC08:37
*** jhershbe has quit IRC08:39
*** Marga_ has quit IRC08:39
*** Sukhdev has quit IRC08:40
*** lezbar has joined #openstack-neutron08:42
*** pcaruana has joined #openstack-neutron08:43
*** yfried_ has joined #openstack-neutron08:45
*** kiran-r has quit IRC08:46
*** kiran-r has joined #openstack-neutron08:46
*** yfried has quit IRC08:47
*** mgoddard has joined #openstack-neutron08:47
*** prapulla has quit IRC08:49
*** permalac has joined #openstack-neutron08:50
*** jhershbe has joined #openstack-neutron08:51
*** jckasper has joined #openstack-neutron08:54
*** emagana has joined #openstack-neutron08:55
*** ekuris_ has joined #openstack-neutron08:55
*** lujinluo has quit IRC08:56
*** hichihara has quit IRC08:57
*** ociuhandu has quit IRC08:57
*** tflynn has joined #openstack-neutron08:58
*** jckasper has quit IRC08:59
*** javeriak has quit IRC08:59
*** emagana has quit IRC08:59
*** oshvartz has quit IRC09:00
*** bsv has quit IRC09:00
*** jschwarz has joined #openstack-neutron09:01
*** safchain has joined #openstack-neutron09:03
*** garyk has quit IRC09:04
*** korzen has joined #openstack-neutron09:05
*** mhickey has joined #openstack-neutron09:06
*** neiljerram has joined #openstack-neutron09:06
jschwarzkevinbenton, good morning :)09:09
*** tflynn has quit IRC09:09
*** thorst has joined #openstack-neutron09:09
openstackgerritMerged openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once  https://review.openstack.org/29160309:11
*** oshvartz has joined #openstack-neutron09:14
*** jlanoux has joined #openstack-neutron09:14
*** davidsha has joined #openstack-neutron09:15
*** thorst has quit IRC09:17
*** rmstar has quit IRC09:18
*** lucas-afk is now known as lucasagomes09:19
openstackgerritVictor Stinner proposed openstack/neutron: Reuse to_utf8() and jsonutils.dump_as_bytes()  https://review.openstack.org/28073209:19
*** mfuruta has quit IRC09:21
*** dingboopt has joined #openstack-neutron09:21
*** javeriak has joined #openstack-neutron09:23
*** oshvartz has quit IRC09:28
*** jistr has joined #openstack-neutron09:28
openstackgerritvenkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver  https://review.openstack.org/28111609:33
*** ArchiFle1 has quit IRC09:36
*** wolverineav has joined #openstack-neutron09:36
*** ociuhandu has joined #openstack-neutron09:36
*** armax has quit IRC09:37
*** evgenyf has quit IRC09:37
*** jlanoux_ has joined #openstack-neutron09:38
*** ArchiFleKs has joined #openstack-neutron09:38
*** anshul has joined #openstack-neutron09:39
*** jlanoux has quit IRC09:41
*** wolverineav has quit IRC09:41
*** oshvartz has joined #openstack-neutron09:41
*** javeriak has quit IRC09:41
openstackgerritAdit Sarfaty proposed openstack/neutron: QOS: Provide get methods for policy & network/port binding  https://review.openstack.org/29208309:42
*** rmstar has joined #openstack-neutron09:46
*** emagana has joined #openstack-neutron09:50
*** emagana has quit IRC09:51
*** emagana has joined #openstack-neutron09:51
openstackgerritvenkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports  https://review.openstack.org/24122709:52
*** javeriak has joined #openstack-neutron09:53
*** emagana has quit IRC09:56
*** tpsilva has joined #openstack-neutron09:56
*** moshele has quit IRC09:59
*** markvoelker has joined #openstack-neutron09:59
*** liuyulong has joined #openstack-neutron10:01
*** markvoelker has quit IRC10:03
*** moshele has joined #openstack-neutron10:04
ihrachysjlibosva: https://review.openstack.org/#/c/277558/10:04
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses  https://review.openstack.org/27755810:04
*** abregman is now known as abregman|afk10:05
*** saggi has joined #openstack-neutron10:05
*** evgenyf has joined #openstack-neutron10:08
*** dims has joined #openstack-neutron10:09
*** cristicalin has joined #openstack-neutron10:10
*** slaweq_ has joined #openstack-neutron10:10
*** ygbo has joined #openstack-neutron10:11
*** sdague has joined #openstack-neutron10:13
*** ljianbj has quit IRC10:14
*** jlanoux_ has quit IRC10:15
*** tbachman has joined #openstack-neutron10:15
openstackgerritFederico Ressi proposed openstack/neutron: Fix wrong use of list of dict in _check_driver_to_bind  https://review.openstack.org/27195910:17
openstackgerritFederico Ressi proposed openstack/neutron: Write MechanismManager.bind_port unit tests  https://review.openstack.org/28986310:17
*** aswadr has joined #openstack-neutron10:19
*** miyagishi_t has quit IRC10:20
*** tbachman has quit IRC10:20
*** sambetts|afk is now known as sambetts10:21
*** dims has quit IRC10:21
*** sridharg has quit IRC10:23
*** rossella_s has joined #openstack-neutron10:23
openstackgerritvenkata anil proposed openstack/neutron: HA: Avoid multiple port binding by agents  https://review.openstack.org/28577310:23
*** mvk has joined #openstack-neutron10:24
*** sridharg has joined #openstack-neutron10:24
*** sridharg has quit IRC10:24
*** kawa2014 has joined #openstack-neutron10:25
*** sridharg has joined #openstack-neutron10:25
openstackgerrityalei wang proposed openstack/neutron: Pass the complete info in sg/rules db into PRECOMMIT_XXX callback  https://review.openstack.org/28169310:26
*** sridharg has quit IRC10:26
*** sridharg has joined #openstack-neutron10:26
openstackgerritvenkata anil proposed openstack/neutron: fullstack: HA failover with l2pop test  https://review.openstack.org/28608710:27
davidshaajo: ping10:29
*** dims has joined #openstack-neutron10:30
*** tiagogomes has joined #openstack-neutron10:31
*** evgenyf has quit IRC10:32
openstackgerritvenkata anil proposed openstack/neutron: use separate device owner for HA router interface  https://review.openstack.org/29165110:32
tiagogomesHello, is it possible to create an external network that can only be used by a given tenant? Even with --shared=false and passing --tenant-id when creating the network, the network can be used by other tenants...10:32
*** rossella_s has quit IRC10:37
davidshaajo: Morning, I'm looking into neutron-classifier atm, I'm just wondering how will we rate limit based on the classifier though? The only thing I could think of is make multiple interfaces to use ingress_policer and redirect using flows.10:38
*** iwamoto has quit IRC10:38
*** thorst has joined #openstack-neutron10:38
*** abregman|afk is now known as abregman10:38
*** dims has quit IRC10:39
*** liuyulong has quit IRC10:39
*** dims has joined #openstack-neutron10:40
*** baoli has joined #openstack-neutron10:41
*** thorst has quit IRC10:43
*** armax has joined #openstack-neutron10:44
*** dave-mccowan has joined #openstack-neutron10:45
*** baoli has quit IRC10:46
*** nyechiel_ has quit IRC10:47
jlibosvajschwarz: I left some nits on the custom types patches10:51
jschwarzjlibosva, I see - will make sure I refresh10:51
*** evgenyf has joined #openstack-neutron10:53
*** tbachman has joined #openstack-neutron10:53
ihrachysmhickey: https://review.openstack.org/28185011:01
*** baojg has quit IRC11:03
*** mdih has joined #openstack-neutron11:03
*** numan_ has joined #openstack-neutron11:03
*** numans has quit IRC11:03
*** rodrigods has quit IRC11:04
*** rodrigods has joined #openstack-neutron11:04
*** claudiub has joined #openstack-neutron11:04
*** kiran-r has quit IRC11:06
*** tflynn has joined #openstack-neutron11:06
ihrachysarmax: hi.11:07
armaxihrachys: yo11:08
mhickeyihrachys: thanks11:09
*** fawadkhaliq has joined #openstack-neutron11:09
*** tflynn has quit IRC11:17
*** wei-wang has joined #openstack-neutron11:18
*** ihrachys has quit IRC11:18
ajodavidsha, hmm, for vm-ingress policing (at low level) it would be just creating several queues and OF rules to redirect each kind of traffic to the specific queue11:19
ajoegress is more complicated11:19
ajosince we don't have (currently) queues at switch-port-ingress... (VM port egress), you need to catch it at the next port hop, and that only works if your port is linux kernel, and not patch-port internal.11:20
ajoI think nftables let's you do that now, btw11:20
*** roeyc has joined #openstack-neutron11:21
*** korzen has quit IRC11:21
*** gampel1 has joined #openstack-neutron11:22
davidshaajo: egress is what I'm thinking of, I'll look into it (nftables). There isn't any implementation behind neutron classifier is there? I'm only really seeing database tables being created but nothing using them.11:23
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses  https://review.openstack.org/27755811:23
*** rtheis has joined #openstack-neutron11:23
jschwarzjlibosva, I've taken the liberty to modify the test code of ^, please have a look when you come back :)11:24
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR.  https://review.openstack.org/28534911:25
*** gampel has quit IRC11:25
ajodavidsha, I believe that's the state so far, we may want to talk to sc68cal to sync on that11:27
ajothat could be expanded to have openflow/iptables/nftables generators11:27
ajoout of traffic filters11:27
ajoor something like that11:27
ajodavidsha, or even a common api / extension to declare such filters11:28
*** ericbaum has joined #openstack-neutron11:28
davidshaajoi: kk, thats what I'm thinking (common api).11:28
davidshaajo: kk, thats what I'm thinking (common api).11:28
ajoI find suboptimal that every feature making use of it has to declare it's own api11:28
ajoyeah :D11:28
*** claudiub has quit IRC11:28
davidsha(Need to start proof reading these messages)11:28
*** claudiub has joined #openstack-neutron11:29
davidshaI'll work on it for the moment and see what I can come up with, it will probably be worth using it with the l2-agent to stop flow conflicts.11:31
*** bsv has joined #openstack-neutron11:33
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR.  https://review.openstack.org/28534911:36
*** ujjain- is now known as ujjain11:39
*** buttercup has joined #openstack-neutron11:41
*** thorst has joined #openstack-neutron11:42
*** thorst_ has joined #openstack-neutron11:43
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress  https://review.openstack.org/28634711:43
johnthetubaguyarmax: had an possible RC1 blocker in Nova that is relating to neutron, would be awesome if you can find folks to take a peak at that for us: https://review.openstack.org/#/c/271444/611:45
johnthetubaguyarmax: MTU and vhostuser issues11:45
armaxjohnthetubaguy: looking11:45
*** dims_ has joined #openstack-neutron11:45
* johnthetubaguy suddenly remembers armax is in india, rather than doing an all nighter11:46
openstackgerritAdit Sarfaty proposed openstack/neutron: QOs: TestQosPlugin receives the plugin name dynamically The neutron QoS plugin test class 'TestQosPlugin' used the hard coded name 'qos' as the plugin name (alias, in this case). In order to inherit from this test class, and use it outside, we now set the  https://review.openstack.org/29231511:46
armaxjohnthetubaguy: there you go!11:46
armaxjohnthetubaguy: isn’t that suddenly convenient?11:46
armax:)11:47
*** thorst has quit IRC11:47
johnthetubaguyarmax: :)11:47
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: _get_router_for_floatingip: Fix confusing indentation  https://review.openstack.org/29231711:47
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: Fix validation of floating-ip association  https://review.openstack.org/29231811:47
*** dims has quit IRC11:47
*** cristicalin has quit IRC11:48
*** anilvenkata has quit IRC11:49
*** anilvenkata has joined #openstack-neutron11:50
*** yfried has joined #openstack-neutron11:54
*** yfried_ has quit IRC11:54
reediparmax is actually in India?11:55
armaxreedip: yes I am11:56
reediparmax: gr8, where in India?11:56
armaxBLR11:56
reedipoh, the code sprint ?11:56
reedipI hope you are enjoying your stay ...11:57
*** amuller has joined #openstack-neutron11:59
*** yfried_ has joined #openstack-neutron11:59
*** yfried has quit IRC11:59
*** yamamoto has quit IRC12:00
*** markvoelker has joined #openstack-neutron12:00
*** ihrachys has joined #openstack-neutron12:01
*** chlong has joined #openstack-neutron12:01
*** korzen has joined #openstack-neutron12:01
*** ihrachys has quit IRC12:03
*** numan_ has quit IRC12:03
*** ihrachys has joined #openstack-neutron12:03
*** markvoelker has quit IRC12:04
ihrachysarmax: there is one patch that is both safe and ready to go forward with ovo: https://review.openstack.org/#/c/281850/ it touches base class that in theory could influence qos, but code inspection shows that it's no-op there unless you have your objects opt in using the hooks12:05
* armax looks12:05
armaxihrachys: this needs a rebase though12:06
*** sayalilunkad_lap has joined #openstack-neutron12:06
openstackgerritIhar Hrachyshka proposed openstack/neutron: Create a hook in base object to modify the fields before DB operations  https://review.openstack.org/28185012:07
ihrachysarmax: I did rebase, though it was just a zuul mistake (there was a flood of fake conflicts lately in neutron)12:08
armaxihrachys: ack12:08
*** nyechiel_ has joined #openstack-neutron12:14
*** sayalilunkad_lap has quit IRC12:14
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses  https://review.openstack.org/27755812:15
*** jckasper has joined #openstack-neutron12:15
openstackgerritJohn Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR  https://review.openstack.org/28534912:15
jschwarzihrachys, ^12:15
jschwarzjlibosva, ^12:15
*** sayalilunkad_lap has joined #openstack-neutron12:17
ihrachysjschwarz: ++ thanks man12:18
*** jckasper has quit IRC12:20
jlibosvajschwarz: did you look at my comments?12:20
jlibosvajschwarz: on PS312:21
jschwarzjlibosva, I think I did but I might have missed some12:21
jschwarzjlibosva, on which patch?12:21
jschwarzoh I see them now12:21
*** baoli has joined #openstack-neutron12:21
*** javeriak has quit IRC12:22
*** rossella_s has joined #openstack-neutron12:22
jschwarzjlibosva, aye, I agree with those comments but sadly I have to drop out :<12:22
armaxamuller: ping12:22
*** erlarese has joined #openstack-neutron12:22
armaxjohnthetubaguy pointed me to this issue: https://review.openstack.org/#/c/271444/12:22
armaxI thought I’d keep you in the loop since you were working on https://blueprints.launchpad.net/neutron/+spec/vhost-ovs12:23
*** javeriak has joined #openstack-neutron12:23
*** gampel has joined #openstack-neutron12:23
ihrachysrossella_s: https://review.openstack.org/#/c/270230/12:24
openstackgerritAdit Sarfaty proposed openstack/neutron: QOs: TestQosPlugin receives the plugin name dynamically The neutron QoS plugin test class 'TestQosPlugin' used the hard coded name 'qos' as the plugin name (alias, in this case), as well as hard coded core plugin name. In order to inherit from this test c  https://review.openstack.org/29231512:24
armaxihrachys: is this relevant you think?12:24
armaxhttp://logs.openstack.org/50/281850/9/check/gate-neutron-dsvm-fullstack/d1e230a//testr_results.html.gz12:24
*** wei-wang has quit IRC12:24
*** wei-wang has joined #openstack-neutron12:25
*** gampel1 has quit IRC12:25
*** armax has quit IRC12:25
*** gampel1 has joined #openstack-neutron12:25
*** baoli has quit IRC12:26
*** wei-wang has quit IRC12:26
ihrachyskorzen: ^12:28
*** gampel has quit IRC12:28
*** nikil89_ has joined #openstack-neutron12:31
*** edmondsw has joined #openstack-neutron12:31
*** boris-42 has quit IRC12:34
*** enriquetaso has joined #openstack-neutron12:38
*** javeriak_ has joined #openstack-neutron12:39
*** javeriak_ has quit IRC12:39
*** MVenesio has joined #openstack-neutron12:40
*** baoli has joined #openstack-neutron12:42
*** baoli_ has joined #openstack-neutron12:43
*** javeriak has quit IRC12:43
openstackgerritJohn Schwarz proposed openstack/neutron: WIP: Remove auto_schedule_routers codepath  https://review.openstack.org/28548012:43
*** jckasper has joined #openstack-neutron12:45
*** jckasper has quit IRC12:45
*** jckasper has joined #openstack-neutron12:45
*** links has quit IRC12:46
*** baoli has quit IRC12:46
*** links has joined #openstack-neutron12:46
*** yamamoto has joined #openstack-neutron12:48
*** armax has joined #openstack-neutron12:48
*** markvoelker has joined #openstack-neutron12:51
*** jckasper has quit IRC12:52
*** jckasper has joined #openstack-neutron12:53
*** lucasagomes is now known as lucas-hungry12:54
*** stanzgy has quit IRC12:56
*** anilvenkata has quit IRC12:58
*** asingh has joined #openstack-neutron12:59
*** nmagnezi has quit IRC12:59
*** mgoddard_ has joined #openstack-neutron12:59
*** asingh has quit IRC13:00
*** asingh has joined #openstack-neutron13:00
*** asingh has quit IRC13:00
*** asingh has joined #openstack-neutron13:00
*** ferest has joined #openstack-neutron13:00
*** ferest has quit IRC13:00
*** mgoddard has quit IRC13:02
*** EinstCrazy has joined #openstack-neutron13:04
*** davidsha has quit IRC13:04
*** ranjithd has joined #openstack-neutron13:04
*** fangzhen_ has quit IRC13:05
*** anilvenkata has joined #openstack-neutron13:08
*** anilvenkata has quit IRC13:09
*** anilvenkata has joined #openstack-neutron13:10
*** yfried_ has quit IRC13:10
*** yfried has joined #openstack-neutron13:11
*** lnicolas has quit IRC13:12
*** tflynn has joined #openstack-neutron13:13
*** neelashah has joined #openstack-neutron13:14
*** gberginc has quit IRC13:14
*** roeyc has quit IRC13:15
*** nmagnezi has joined #openstack-neutron13:15
*** ajmiller has joined #openstack-neutron13:15
*** roeyc has joined #openstack-neutron13:15
*** brad_behle has joined #openstack-neutron13:16
*** yarkot_ has joined #openstack-neutron13:19
*** neeti has quit IRC13:19
*** tflynn has quit IRC13:20
*** singhj has joined #openstack-neutron13:22
*** haukebruno has quit IRC13:22
openstackgerritAndreas Scheuring proposed openstack/neutron: lb: interface name hashing for too long vlan interface names  https://review.openstack.org/24695413:23
*** zengine has quit IRC13:23
*** haukebruno has joined #openstack-neutron13:24
openstackgerritAndreas Scheuring proposed openstack/neutron: l3: Send notify on router_create when ext gw is specified  https://review.openstack.org/29047013:24
*** BigWillie has joined #openstack-neutron13:24
*** ranjithd has quit IRC13:25
*** ranjithd has joined #openstack-neutron13:25
*** apuimedo has quit IRC13:26
*** sridharg has quit IRC13:26
*** rickyrem has joined #openstack-neutron13:26
*** yarkot_ has quit IRC13:27
*** akshai has joined #openstack-neutron13:27
*** ajmiller_ has joined #openstack-neutron13:29
*** singhj has quit IRC13:31
*** krotscheck has joined #openstack-neutron13:32
*** ajmiller has quit IRC13:32
*** nyechiel_ has quit IRC13:32
*** nyechiel has joined #openstack-neutron13:32
*** prithiv has joined #openstack-neutron13:33
*** singhj has joined #openstack-neutron13:33
*** davidsha has joined #openstack-neutron13:35
*** vhoward has joined #openstack-neutron13:37
*** dedery has quit IRC13:38
*** ranjithd has quit IRC13:39
*** ajmiller_ has quit IRC13:40
*** panda has quit IRC13:40
*** panda has joined #openstack-neutron13:40
openstackgerritIhar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started  https://review.openstack.org/29239213:40
ihrachysjlibosva: hit it into the wall ^13:40
*** ranjithd has joined #openstack-neutron13:41
*** mvk has quit IRC13:41
*** nplanel has quit IRC13:41
*** nplanel has joined #openstack-neutron13:41
*** banix has joined #openstack-neutron13:42
*** ferest has joined #openstack-neutron13:42
*** ferest has quit IRC13:42
*** lazy_prince has quit IRC13:43
sbelousmordred: ping13:44
*** brad_behle_ has joined #openstack-neutron13:45
*** brad_behle_ has quit IRC13:46
*** brad_behle has quit IRC13:46
*** javeriak has joined #openstack-neutron13:46
*** tbachman has quit IRC13:47
*** jpena is now known as jpena|lunch13:48
*** irenab has quit IRC13:49
mordredsbelous: sup?13:50
openstackgerritRyan Moats proposed openstack/neutron: Collector Proof of Concept  https://review.openstack.org/21347413:50
sbelousmordred: Hi, Monty! Can you please review my patch to infra?13:51
*** vhoward has quit IRC13:51
sbelousmordred: this one https://review.openstack.org/#/c/286085/13:51
*** reedip__ has joined #openstack-neutron13:51
*** skamithi has joined #openstack-neutron13:51
jschwarzjlibosva, lmao.13:52
jschwarzjlibosva, really?13:52
*** javeriak_ has joined #openstack-neutron13:52
jlibosvajschwarz: wat?13:53
jschwarzjlibosva, that RNG seed patch13:53
jlibosvajschwarz: you're fast man :)13:53
jschwarzjlibosva, I get desktop notifications :)13:53
jlibosvajschwarz: cool :)13:53
* jschwarz sees ihrachys' name on the patch, I +1 immediately13:53
* jschwarz hopes ihrachys will +2 likewise ;-)13:53
*** javeriak has quit IRC13:54
*** brad_behle has joined #openstack-neutron13:55
ihrachysjschwarz: I don't like your exchange rate13:58
jschwarzihrachys, XD13:58
ihrachysjschwarz: honestly, I haven't even executed the code13:58
jschwarzihrachys, it seems reasonable, mind you13:59
jschwarza simple 'seed' and the reason seems justified13:59
jschwarzihrachys, how did you encounter this?13:59
*** EkulTails has joined #openstack-neutron13:59
*** kbringard has joined #openstack-neutron14:00
*** dileepr has joined #openstack-neutron14:00
*** kbringard has quit IRC14:01
*** asingh has quit IRC14:01
*** kbringard has joined #openstack-neutron14:01
*** asingh has joined #openstack-neutron14:01
*** asingh has quit IRC14:02
*** ig0r_ has joined #openstack-neutron14:02
*** asingh has joined #openstack-neutron14:02
*** lucas-hungry is now known as lucasagomes14:03
*** boris-42 has joined #openstack-neutron14:05
*** gvrangan has joined #openstack-neutron14:06
*** Marga_ has joined #openstack-neutron14:06
ihrachysjschwarz: well, just stole some ideas from https://bugs.launchpad.net/neutron/+bug/1551288 comments14:06
openstackLaunchpad bug 1551288 in neutron "Fullstack native tests sometimes fail with an OVS agent failing to start with 'Address already in use' error" [High,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)14:06
*** tonytan4ever has joined #openstack-neutron14:06
*** jhershbe_ has joined #openstack-neutron14:06
jschwarzah14:06
jschwarzmakes sense14:06
ihrachysjschwarz: then played with fork and random.choice locally14:07
*** jlanoux has joined #openstack-neutron14:07
*** jhershbe has quit IRC14:07
*** EkulTails has quit IRC14:08
*** rpothier has joined #openstack-neutron14:08
*** EkulTails has joined #openstack-neutron14:08
*** salv-orlando has joined #openstack-neutron14:08
ihrachysjschwarz: I suspect that e.g. testr imports random before forking, or smth along these lines. still not sure about the approach though.14:09
*** oshvartz has quit IRC14:10
*** tongli has joined #openstack-neutron14:10
jschwarzihrachys, why not seed directly in fullstack's base class?14:10
ihrachysjschwarz: well why not? :)14:10
ihrachysjschwarz: it may be useful for functional too14:10
ihrachysdepending on how results of generate_port used14:10
jschwarz:)14:10
ihrachyssince the port generator is in common test code, I decided maybe it's time for global change14:10
amullerihrachys: I commented on the patch, fullstack use a different base class14:11
* jschwarz has to go to class and will be back in 30 minutes14:12
*** dslevin has joined #openstack-neutron14:12
*** mvk has joined #openstack-neutron14:12
*** haleyb has joined #openstack-neutron14:12
*** dslevin has quit IRC14:12
*** rickyrem1 has joined #openstack-neutron14:13
*** javeriak_ has quit IRC14:13
*** dslevin has joined #openstack-neutron14:14
*** dslevin has quit IRC14:14
*** rickyrem has quit IRC14:14
*** fawadkhaliq has quit IRC14:15
*** fawadkhaliq has joined #openstack-neutron14:15
*** dslevin has joined #openstack-neutron14:16
*** david-lyle has quit IRC14:16
*** jschwarz has quit IRC14:16
*** dslevin has quit IRC14:16
*** dslevin has joined #openstack-neutron14:17
*** irenab has joined #openstack-neutron14:17
*** dslevin has quit IRC14:17
*** asingh has quit IRC14:18
*** evgenyf has quit IRC14:18
*** dslevin has joined #openstack-neutron14:19
*** mgoddard_ has quit IRC14:19
*** dslevin has quit IRC14:19
*** david-lyle has joined #openstack-neutron14:19
*** cbouch has joined #openstack-neutron14:20
*** dslevin has joined #openstack-neutron14:20
*** kawa2014 has quit IRC14:20
*** haleyb is now known as haleyb_14:20
*** dslevin has quit IRC14:20
*** haleyb has joined #openstack-neutron14:21
*** absubram has joined #openstack-neutron14:21
*** dslevin has joined #openstack-neutron14:22
*** mgoddard has joined #openstack-neutron14:22
*** absubram_ has joined #openstack-neutron14:22
*** networkninja has joined #openstack-neutron14:23
*** dslevin has quit IRC14:23
*** javeriak has joined #openstack-neutron14:23
*** ajmiller has joined #openstack-neutron14:24
*** kawa2014 has joined #openstack-neutron14:24
*** absubram has quit IRC14:25
*** absubram_ is now known as absubram14:25
*** tbachman has joined #openstack-neutron14:25
*** yamamoto has quit IRC14:25
*** oshvartz has joined #openstack-neutron14:25
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: register all objects before validating their hash versions  https://review.openstack.org/27023014:27
ihrachyskorzen: neutron/extensions/default_subnetpools.py that's where extension is14:27
ihrachyskorzen: rossella_s: ^ btw I respinned the hasher test patch moving code into test module14:27
*** pcaruana has quit IRC14:28
ihrachysarmax: ^ can I get that patch unblocked?14:28
*** rossella_ has joined #openstack-neutron14:28
armaxamuller: thanks for chiming in on bug 1375625, see my reply…let me know what you think14:28
openstackbug 1375625 in neutron "Problem in l3-agent tenant-network interface would cause split-brain in HA router" [High,In progress] https://launchpad.net/bugs/1375625 - Assigned to Lubosz Kosnik (diltram)14:28
* armax looks14:28
*** abregman is now known as abregman|mtg14:28
*** Leo_ has joined #openstack-neutron14:28
amullerarmax: done14:29
armaxihrachys: that import looks a bit hideous14:29
armaxihrachys: do you mind expanding on the issue somewhat?14:30
armaxamuller: thank you sir14:30
*** rossella_s has quit IRC14:30
*** crose has joined #openstack-neutron14:31
*** yamahata has quit IRC14:31
rossella_ajo, holaaaa!14:32
armaxamuller: re-replied14:32
armaxamuller: :)14:32
*** asingh has joined #openstack-neutron14:32
openstackgerritIhar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started  https://review.openstack.org/29239214:34
*** absubram has quit IRC14:34
ihrachysarmax: I guess if even *you* wonder, then I need to expand it in the commit message; gimme a sec14:35
*** yamamoto has joined #openstack-neutron14:35
*** yamamoto has quit IRC14:36
diltramarmax, amuller I also replied about that bug14:36
*** reedip__ has quit IRC14:38
ajorossella_,  holaaaa! ;) enjoy brno14:39
rossella_ajo ;)14:39
*** roeyc has quit IRC14:40
*** jschwarz has joined #openstack-neutron14:40
*** mlavalle has joined #openstack-neutron14:41
skamithiwas listening to a podcast about openstack networking. http://packetpushers.net/podcast/podcasts/show-279-real-world-openstack-networking/  they mentioned that there is a neutron replacement in the works, but I can't find any info about it. What is the neutron replacement project called?14:41
*** wolverineav has joined #openstack-neutron14:41
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: register all objects before validating their hash versions  https://review.openstack.org/27023014:42
ihrachysarmax: I hope that clarifies ^14:42
*** roeyc has joined #openstack-neutron14:45
*** wolverineav has quit IRC14:46
openstackgerritMerged openstack/neutron-vpnaas: neutron-vpnaas fix for 'tox -e api' test  https://review.openstack.org/29194914:47
*** nmagnezi has quit IRC14:48
*** cshahani has joined #openstack-neutron14:49
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses  https://review.openstack.org/27755814:50
*** aswadr has quit IRC14:51
*** fawadkhaliq has quit IRC14:51
*** vhoward has joined #openstack-neutron14:51
*** fawadkhaliq has joined #openstack-neutron14:51
*** mgoddard_ has joined #openstack-neutron14:52
*** mgoddard has quit IRC14:52
*** yamamoto has joined #openstack-neutron14:52
*** reedip__ has joined #openstack-neutron14:53
*** abregman|mtg is now known as abregman14:53
*** tflynn has joined #openstack-neutron14:56
*** jpena|lunch is now known as jpena14:56
ihrachysjschwarz: bad boy :P pep8 failures for rebased patches ;)14:56
jschwarzihrachys, :<14:57
jschwarzihrachys, I saw14:57
*** tflynn has quit IRC14:57
jschwarzihrachys, I'm very ashamed14:57
*** javeriak has quit IRC14:57
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for CIDR  https://review.openstack.org/28534914:58
*** javeriak has joined #openstack-neutron14:58
*** feleouet has joined #openstack-neutron14:58
openstackgerritAdelina Tuvenie proposed openstack/neutron: Fixes "OVS Agent doesn't start on Windows"  https://review.openstack.org/29243414:59
ihrachysarmax: may I ask you to unblock https://review.openstack.org/#/c/277558 [custom types series]14:59
ihrachysit's completely unbound from other code, should be safe15:00
*** irenab has quit IRC15:02
*** asingh has quit IRC15:02
*** javeriak has quit IRC15:03
*** jhershbe_ has quit IRC15:03
*** irenab has joined #openstack-neutron15:03
*** irenab has quit IRC15:03
*** irenab has joined #openstack-neutron15:05
*** ollivier has joined #openstack-neutron15:06
*** oshvartz has quit IRC15:06
*** kbringard1 has joined #openstack-neutron15:07
*** kbringard has quit IRC15:07
*** mkolesni has quit IRC15:08
davidshaajo, ihrachys: ping15:08
ihrachysdavidsha: hi15:08
armaxihrachys: to be honest I’d rather not rely on such a magic if I can help it15:08
*** gangil has joined #openstack-neutron15:09
*** gangil has quit IRC15:09
*** gangil has joined #openstack-neutron15:09
ihrachysarmax: well otherwise we are doomed to forget updating the test sometimes and getting random failures until we add an import somewhere.15:09
davidshaihrachys: Hey, is it ok with you if we work on the TODOs and nits for the dscp patch as a separate patch?15:09
armaxihrachys: but that’s at least predictable15:10
armaxihrachys: _predictable_15:10
*** networkninja has quit IRC15:10
armaxihrachys: how about doing something like we do for the db migration stuff?15:11
ihrachysdavidsha: since we are out of M rush, I would like to see them all up for review before we start landing them15:11
armaxihrachys: we’ll do it often enough that gets imprimted into people’s minds15:11
njohnstonihrachys: Well, speaking for Comcast, the "Mitaka" Neutron we use will be the Neutron that exists immediately after https://review.openstack.org/#/c/251738 is merged, so I have a vested interest in making sure it is at the head of the line for merging, which is why I was hoping we could handle the TODOs separately and avoid a round of re-nitting the existing code.15:12
ihrachysarmax: I personally hate manual solutions, but if you are strong on it, I can do it otherwise15:12
*** gangil has quit IRC15:12
anteayahey neutron folks, in conversations with third party folks who might be accessing gerrit from behind a firewall, the gerrit ip is changing next month: http://lists.openstack.org/pipermail/openstack-dev/2016-March/088985.html15:12
armaxihrachys: me too15:12
ihrachysnjohnston: I see.15:12
ihrachysnjohnston: do we have too many of those TODOs?15:12
armaxihrachys: but if the automatic ones rely on recursive imports, then I freak out15:13
*** amotoki has quit IRC15:14
ihrachysarmax: that's a matter of taste :)15:14
armaxand correctness15:14
ihrachysarmax: so you are strong? I can go and change it in 5 mins15:14
armaxif we can get someone else eyes on this I am happy to change my mind15:14
armaxbut I would definitly not want to merge that in haste15:15
ihrachysok sec, let me make you happy15:15
armaxyou don’t have to15:15
njohnstonihrachys: There is https://review.openstack.org/#/c/251738/54/neutron/objects/qos/rule_type.py@35 and https://review.openstack.org/#/c/251738/54/neutron/tests/fullstack/test_qos.py@171 and https://review.openstack.org/#/c/251738/54/neutron/tests/unit/objects/test_base.py@149; the one at https://review.openstack.org/#/c/251738/54/neutron/services/qos/qos_plugin.py@81 predates the DSCP patch15:16
armaxlet’s get someone else’s opinion15:16
ihrachyswell I want to land it quicker since others will need to apply a similar approach.15:16
*** jhershbe_ has joined #openstack-neutron15:16
davidshaihrachys: there is about 6 TODOs associated with this dscp. (4 for ajo and 2 for me)15:16
*** tonytan4ever has quit IRC15:16
ihrachysnjohnston: of those, I believe fullstack is the most important15:17
ihrachyssince it may indicate some issue with the patch15:17
ihrachysothers are innocent assuming we get them handled in due time15:17
ihrachyskevinbenton: what if I ask you to take a look at https://review.openstack.org/#/c/270230/ since armax and I have somewhat different stand on automagic imports?15:18
njohnstonihrachys: I am working on fullstack now, since it is split into a separate patch: https://review.openstack.org/#/c/288392/15:18
ihrachysnjohnston: yeah, I mean the TODO for a test disabled15:19
*** tonytan4ever has joined #openstack-neutron15:19
*** oshvartz has joined #openstack-neutron15:19
armaxihrachys: talking about Mitaka critical fixes15:19
armaxhttps://review.openstack.org/#/c/292434/15:19
njohnstonihrachys: Ah, yes, got it.15:19
*** emagana has joined #openstack-neutron15:23
*** links has quit IRC15:23
*** ig0r__ has joined #openstack-neutron15:23
davidshaihrachys: last question, do you think it's worth back porting the L2-agent patch to stable-liberty?15:23
ihrachysarmax: aye. I posted a question there.15:24
*** jwarendt has joined #openstack-neutron15:24
ihrachysdavidsha: which patch? for agent API?15:24
ihrachysdavidsha: absolutely not, it's a new feature :)15:25
*** regXboi has joined #openstack-neutron15:25
*** doug-fis_ has joined #openstack-neutron15:26
*** gangil has joined #openstack-neutron15:26
*** gangil has quit IRC15:26
*** gangil has joined #openstack-neutron15:26
*** ig0r_ has quit IRC15:26
*** Swami has joined #openstack-neutron15:26
davidshaihrachys: kk, just hearing patches on stable-liberty still have that problem, is there anything we can do about it?15:27
*** doug-fish has quit IRC15:27
armaxihrachys: replied15:28
*** vhosakot has joined #openstack-neutron15:28
*** yamamoto has quit IRC15:30
*** salv-orlando has quit IRC15:31
*** mickeys has joined #openstack-neutron15:32
*** salv-orlando has joined #openstack-neutron15:33
*** armax has quit IRC15:33
ihrachysdavidsha: which problem?15:35
openstackgerritvenkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface  https://review.openstack.org/29164115:35
*** armax has joined #openstack-neutron15:35
anilvenkatapc_m, Hi Paul15:35
anilvenkatapc_m, need help15:36
pc_manilvenkata: hi15:36
*** emagana has quit IRC15:36
anilvenkatapc_m, I added a new flag in neuton-lib, and want to use that in neutron change15:36
*** jaugustine has joined #openstack-neutron15:36
anilvenkatapc_m, this is neutron-lib change https://review.openstack.org/#/c/291641/15:37
davidshaihrachys: flow cleanup, most projects have ways around it, but I was wondering should we put something in to give them a better way.15:37
anilvenkatapc_m, this is neutron change https://review.openstack.org/#/c/291651/ which is dependent on above neutron-lib change15:37
anilvenkatapc_m, how to add this dependency15:37
anilvenkata?15:37
anilvenkatapc_m, I added Depends-On: in commit message, still it is not picking the change15:38
anilvenkatapc_m, I also added change suggested in wiki https://wiki.openstack.org/wiki/Neutron/Lib as "Depends-On"15:39
pc_manilvenkata: Not positive. dougwig will know for sure, but I think we'd have to push a new version of neutron-lib to PyPI, so that it could be picked up in requirements (or constraints)15:39
amullerpc_m: dougwig: Did we ever consider using neutron.lib instead of neutron_lib? This would be so much faster if it were in-repo...15:40
*** davideagnello has quit IRC15:41
anilvenkatadougwig, any suggestion for the above issue?15:41
pc_manilvenkata: Doesn't look like dougwig is online yet.15:42
openstackgerritAdelina Tuvenie proposed openstack/neutron: Ovs agent can't start on Windows because of validate_local_ip  https://review.openstack.org/22707715:42
amullerpc_m: dougwig: That's what they did in Tempest... move from tempest_lib to tempest.lib in-tree. Any thoughts pc_m?15:42
anilvenkatapc_m, Thanks Paul15:42
*** gampel1 has quit IRC15:42
pc_mamuller: I don't know if it was considered. Personally, I'm not well versed in the packaging stuff.15:43
*** gangil has quit IRC15:44
*** alhu has joined #openstack-neutron15:44
amullerpc_m: me neither :) But imagine if neutron_lib was in the neutron repo. You could just move stuff around in a single patch.15:44
pc_manilvenkata: Didn't help much... except point you in a direction :)  I'm curious as well as to the answer.15:45
ihrachysarmax: oh yeah, you are right. that's a huge hackaround. I requested some comments in the code describing why it works.15:46
*** nikil89_ has quit IRC15:46
pc_mamuller: Initially, some of the neutron-lib stuff was placed in the tree to try things out15:46
*** thorst_ is now known as thorst_afk15:46
armaxihrachys: one of many? :)15:46
*** davideagnello has joined #openstack-neutron15:47
ihrachysdavidsha: their extensions should still work with older agent versions, so not sure how it would help.15:47
*** moshele has quit IRC15:47
ihrachysdavidsha: I mean, older *liberty* versions15:48
*** jprovazn has joined #openstack-neutron15:48
ihrachysdavidsha: otherwise we start requiring a minor version of the agent, which is not really the way to go.15:48
ihrachysamuller: ok, we need a second opinion on https://review.openstack.org/#/c/270230/15:49
amullerlooking15:50
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress  https://review.openstack.org/28634715:50
davidshaihrachys: kk, was just reminded that clean up is still a problem in liberty and thought I'd ask. Thanks!15:50
*** Sukhdev has joined #openstack-neutron15:50
openstackgerritvenkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface  https://review.openstack.org/29164115:50
*** tonytan4ever has quit IRC15:50
*** tonytan4ever has joined #openstack-neutron15:51
*** atuvenie has joined #openstack-neutron15:51
*** mgoddard_ has quit IRC15:52
*** mgoddard has joined #openstack-neutron15:52
amullerihrachys: armax: Armando I'm not clear on your -1, are you saying not to fix this issue, or what is the alternative you're suggesting?15:52
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses  https://review.openstack.org/27755815:52
*** jhershbe_ has quit IRC15:52
ihrachysamuller: importing all modules manually15:52
armaxamuller: where?15:52
ihrachysamuller: like we do for db models in head.py15:53
*** iyamahat has joined #openstack-neutron15:53
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for CIDR  https://review.openstack.org/28534915:54
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress  https://review.openstack.org/28634715:54
*** tonytan_brb has joined #openstack-neutron15:54
*** tonytan4ever has quit IRC15:55
*** tonytan_brb has quit IRC15:55
*** tflynn has joined #openstack-neutron15:55
*** tflynn_ has joined #openstack-neutron15:56
*** gvrangan has quit IRC15:56
amullerihrachys: armax: Ignoring the possibility of bugs with Ihar's approach I like it better, lower load on developers having to manually add more models, and it's actually future proof15:56
amullerihrachys: armax: When new models are added (But not added to the import list), we'll start getting intermittent unit test failures again15:57
armaxamuller: it’s bugs I am worried about, but if you think there are none, feel free to go ahead15:57
amullerI think we can stomach the 'magicness' to improve quality in this case15:57
*** tonytan4ever has joined #openstack-neutron15:57
armaxamuller: when bugs creep in, I’d be happy to have your head on a silver plate :)15:57
armaxhow about that?15:57
*** minwang2 has joined #openstack-neutron15:58
amullerarmax: You will have both Ihar and myself, I'll stand behind that :)15:58
armaxamuller: I am in!15:58
*** emagana has joined #openstack-neutron15:58
* armax pretends he’s looking the other way15:58
ihrachysarmax: that's the approach!15:59
amullerihrachys: what do you think of iglob vs recursive function though?15:59
*** lazy_prince has joined #openstack-neutron15:59
*** jckasper has quit IRC16:00
*** tflynn has quit IRC16:00
armaxihrachys: and if you do go ahead with the magic import16:00
armaxI want a test for the test16:00
armax:P16:00
ihrachysarmax: what amuses me lately is that I can't distinguish you joking and telling your mind16:01
*** jckasper has joined #openstack-neutron16:01
openstackgerritSayali Lunkad proposed openstack/neutron: WIP: Introduces ovo objects for security groups  https://review.openstack.org/28473816:02
amullerarmax: ihrachys: That's not an outrageous idea, if you rename import_modules to get_modules_with_object_models, it would be easily testable16:02
*** tflynn_ has quit IRC16:02
amullerihrachys: So you just test if the function detects the correct files16:02
armaxwe should have that method invoked in neutron/__init__.py16:02
*** killer_prince has joined #openstack-neutron16:02
amullerihrachys: then you have another method that iterates on the results and imports everything16:02
*** ekuris_ has quit IRC16:02
*** singhj has quit IRC16:03
* ihrachys is looking for his cyanide pills16:03
openstackgerritIhar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started  https://review.openstack.org/29239216:03
ihrachysamuller: with os.getpid() melded in but please let's not reset gate until we validate it ^16:04
*** tflynn has joined #openstack-neutron16:04
*** lazy_prince has quit IRC16:04
openstackgerritIhar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time and pid for each test started  https://review.openstack.org/29239216:04
amullerihrachys: Since it wasn't WIP I assumed you did validate it locally16:04
ihrachysamuller: ack. my fault.16:04
openstackgerritSayali Lunkad proposed openstack/neutron: WIP: Introduces ovo objects for security groups  https://review.openstack.org/28473816:05
*** bsv has quit IRC16:05
*** jckasper has quit IRC16:06
*** emagana has quit IRC16:06
*** oshvartz has quit IRC16:06
*** absubram has joined #openstack-neutron16:07
*** ramishra has quit IRC16:08
*** pcaruana has joined #openstack-neutron16:09
*** ramishra has joined #openstack-neutron16:10
*** jaypipes has joined #openstack-neutron16:10
*** amuller is now known as amuller_afk16:11
*** fawadkhaliq has quit IRC16:11
*** jschwarz has quit IRC16:13
*** EinstCrazy has quit IRC16:14
openstackgerritvenkata anil proposed openstack/neutron: use separate device owner for HA router interface  https://review.openstack.org/29165116:15
ihrachysamuller_afk: re iglob: I *think* that it would not get me into unspecified number of directories deeper16:16
ihrachysamuller_afk: and I don't wan't to hardcode the code structure in the function16:16
*** singhj has joined #openstack-neutron16:16
*** tidwellr has joined #openstack-neutron16:19
*** cbouch has quit IRC16:19
*** anilvenkata has quit IRC16:21
*** sayalilunkad_lap has quit IRC16:21
openstackgerritMartin Hickey proposed openstack/neutron: Prevent primary keys in Neutron OVOs from being updated  https://review.openstack.org/29248816:22
*** sayalilunkad_lap has joined #openstack-neutron16:22
*** sayalilunkad_lap has quit IRC16:22
*** slunkad has joined #openstack-neutron16:23
*** tflynn has quit IRC16:25
*** haplo37 has joined #openstack-neutron16:26
*** gangil has joined #openstack-neutron16:26
*** gangil has quit IRC16:26
*** gangil has joined #openstack-neutron16:26
*** scheuran has quit IRC16:27
mhickeyihrachys, korzen, rossella_s, jlibosva, sayalilunkad: Review if you have time ^^^^^. Go easy, I am very near you! :)16:27
*** fawadkhaliq has joined #openstack-neutron16:29
*** dileepr has quit IRC16:29
*** yamamoto has joined #openstack-neutron16:31
*** abregman has quit IRC16:31
*** Guest59187 has quit IRC16:31
*** yfried has quit IRC16:33
*** anshul has quit IRC16:33
*** kobis has quit IRC16:34
*** aswadr has joined #openstack-neutron16:37
*** aswadr has left #openstack-neutron16:38
*** matrohon has quit IRC16:39
*** rpothier has quit IRC16:40
*** slaweq_ has quit IRC16:40
*** doug-fis_ has quit IRC16:41
*** doug-fish has joined #openstack-neutron16:41
*** yamamoto has quit IRC16:41
ihrachysarmax: amuller_afk: now, just don't tell me you haven't asked a test for a test: https://review.openstack.org/27023016:43
*** wolverineav has joined #openstack-neutron16:43
*** jhershbe has joined #openstack-neutron16:44
*** neelashah has quit IRC16:46
*** armax has quit IRC16:46
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Agent side change for live migration with floatingip  https://review.openstack.org/26073816:46
*** singhj has quit IRC16:46
*** doug-fish has quit IRC16:47
*** wolverineav has quit IRC16:47
*** wolverineav has joined #openstack-neutron16:50
*** Marga_ has quit IRC16:51
manjeetsarmax: ping16:51
*** Marga_ has joined #openstack-neutron16:54
*** jckasper has joined #openstack-neutron16:55
*** rickyrem1 has quit IRC16:56
*** rcernin has quit IRC16:56
*** chlong has quit IRC16:56
*** feleouet has quit IRC16:57
*** jhershbe has quit IRC16:57
*** jckasper has quit IRC16:59
*** jckasper has joined #openstack-neutron16:59
*** Marga_ has quit IRC16:59
*** kevo has joined #openstack-neutron17:00
*** killer_prince has quit IRC17:01
*** doug-fish has joined #openstack-neutron17:01
ihrachysrossella_: https://review.openstack.org/#/c/270230/ you could take a look at the updated hasher fix that imports all objects ;)17:03
*** alhu has quit IRC17:03
rossella_ihrachys, ok :)17:03
*** alhu has joined #openstack-neutron17:03
*** singhj has joined #openstack-neutron17:04
*** jckasper has quit IRC17:05
*** jckasper has joined #openstack-neutron17:06
*** mdih has quit IRC17:06
*** iyamahat has quit IRC17:06
*** yamahata has joined #openstack-neutron17:07
*** emagana has joined #openstack-neutron17:07
*** ollivier has quit IRC17:08
*** dims has joined #openstack-neutron17:08
jlibosvarossella_: https://review.openstack.org/#/c/270309/5/neutron/tests/unit/api/rpc/handlers/test_resources_rpc.py17:08
*** jckasper has quit IRC17:09
*** jckasper has joined #openstack-neutron17:09
*** dims_ has quit IRC17:09
*** neelashah has joined #openstack-neutron17:09
*** jckasper has quit IRC17:11
*** azbiswas has joined #openstack-neutron17:12
*** jckasper has joined #openstack-neutron17:12
*** gvrangan has joined #openstack-neutron17:13
*** jckasper has quit IRC17:13
*** jckasper has joined #openstack-neutron17:13
*** shivharis has joined #openstack-neutron17:14
*** nyechiel has quit IRC17:14
*** jckasper has quit IRC17:14
*** jckasper has joined #openstack-neutron17:14
ihrachysamuller_afk: the fullstack change for random seed passed twice. how many times do we want to validate it before pushing?17:14
openstackgerritAdelina Tuvenie proposed openstack/neutron: Fixes "OVS Agent doesn't start on Windows"  https://review.openstack.org/29243417:15
*** tmorin has quit IRC17:15
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: introduce object for extra dhcp options  https://review.openstack.org/27307217:16
ihrachysmhickey: rebased and fixed tests ^ should be good to go if passes gate.17:16
*** sc68cal has joined #openstack-neutron17:17
*** eil397 has joined #openstack-neutron17:17
*** jckasper has quit IRC17:19
*** prithiv has quit IRC17:20
*** thorst_afk is now known as thorst_17:20
mhickeyihrachys: thanks17:23
*** ildikov_ has quit IRC17:25
*** ygbo has quit IRC17:25
*** jistr has quit IRC17:28
*** neelashah has quit IRC17:29
*** singhj has quit IRC17:29
*** jckasper has joined #openstack-neutron17:30
*** adelia has joined #openstack-neutron17:31
*** neelashah has joined #openstack-neutron17:31
*** wolverineav has quit IRC17:32
*** jlanoux has quit IRC17:34
*** SumitNaiksatam has joined #openstack-neutron17:35
*** alhu has quit IRC17:35
*** amuller_afk is now known as amuller17:36
*** crose has quit IRC17:37
amullerihrachys: last I checked iglob works fine, just returns all subdirs recursively17:37
amullerihrachys: without any assumptions17:37
ihrachysamuller: not sure which pattern to use17:37
*** bbzhao has joined #openstack-neutron17:38
amullerihrachys: if you're already 'in the dir' just '.'17:38
ihrachysoh17:38
ihrachys:-)17:38
*** roeyc has quit IRC17:38
amullerihrachys: I was really excited about iglob when I found out about it =p17:38
ihrachys>>> list(glob.iglob('.'))17:38
ihrachys['.']17:38
jlibosvaalso you need to limit it by .py file extension17:39
amullerihrachys: Well, I don't remember offhand, play with it =p17:39
amullerihrachys: it has good docs IIRC17:39
ihrachysamuller: why do people hate recursion? :)17:39
* jlibosva played as well, didn't find the pattern17:39
*** panda has quit IRC17:40
*** panda has joined #openstack-neutron17:40
*** bzhao has quit IRC17:41
*** shivrao has joined #openstack-neutron17:44
amullerihrachys: jlibosva: It seems like glob is only recursive from python 3.5 with a new parameter :(17:45
amullerihrachys: my memory deceived me17:45
amullerihrachys: so we can either use os.walk, or drop support for python < 3.5 and use glob, your call!17:46
openstackgerritMerged openstack/neutron: Create a hook in base object to modify the fields before DB operations  https://review.openstack.org/28185017:46
*** sridhar_ram has joined #openstack-neutron17:47
jlibosvaamuller: or have "if version.major == 2: os.walk else: glob.iglob" :D JK17:47
ihrachysamuller: I vote dropping support for py3 right away. py4k is the next big thing.17:48
amullerihrachys: only if it py4 is backwards incompatible of course17:48
ihrachysamuller: of course. that's the spirit17:49
*** jwarendt has quit IRC17:50
*** wolverineav has joined #openstack-neutron17:51
*** shivharis has quit IRC17:51
*** safchain has quit IRC17:51
*** bdemers has quit IRC17:51
*** alejandrito has joined #openstack-neutron17:51
openstackgerritJakub Libosvar proposed openstack/neutron: secgroup: Define hybrid plug in driver itself  https://review.openstack.org/28462017:51
*** abhiraut has joined #openstack-neutron17:52
*** bdemers has joined #openstack-neutron17:53
*** sleviim has joined #openstack-neutron17:54
*** sleviim has left #openstack-neutron17:54
*** salv-orl_ has joined #openstack-neutron17:54
*** sbalukoff has joined #openstack-neutron17:54
*** singhj has joined #openstack-neutron17:55
*** davidsha has quit IRC17:55
*** anilvenkata has joined #openstack-neutron17:55
*** tpsilva has quit IRC17:57
*** salv-orlando has quit IRC17:57
anilvenkatacarl_baldwin, haleyb ping, need help17:58
pc_mihrachys: ping17:59
ihrachyspc_m: howdy17:59
*** salv-orl_ has quit IRC17:59
pc_mihrachys: Did the changes get upstreamed for supporting constraints on cover, venv, releasenotes?17:59
ihrachyspc_m: nope we wait for some zuul patch17:59
pc_mihrachys: OK. Thanks.17:59
*** Sukhdev has quit IRC18:00
*** jckasper has quit IRC18:01
*** kawa2014 has quit IRC18:01
amullerjlibosva: hey, how did the security groups scenario tests pass on https://review.openstack.org/#/c/284620 patchset 3?18:02
amullerjlibosva: any idea?18:02
amullerjlibosva: oh nevermind18:03
amullerjlibosva: yeah they should have passed18:03
amullerignore me :)18:03
*** alhu has joined #openstack-neutron18:04
*** jpena is now known as jpena|off18:05
*** emagana has quit IRC18:06
jlibosvaamuller: btw I found out why the related connection test was failing. I learned that if I want to use ip forwarding then I need to turn on ip forwarding.18:06
amullerjlibosva: that sounds like it would suck hard to figure out18:07
*** Sukhdev has joined #openstack-neutron18:09
*** emagana has joined #openstack-neutron18:09
*** mvk has quit IRC18:09
*** jlibosva has quit IRC18:10
*** ihrachys has quit IRC18:10
*** Sukhdev has quit IRC18:11
*** testestsetse has joined #openstack-neutron18:12
*** ildikov_ has joined #openstack-neutron18:12
*** testestsetse has quit IRC18:12
*** rmstar has quit IRC18:13
*** emagana has quit IRC18:13
*** abregman has joined #openstack-neutron18:14
*** korzen has quit IRC18:15
*** mhickey has quit IRC18:15
*** doug-fish has quit IRC18:15
*** rossella_ has quit IRC18:15
*** slunkad has quit IRC18:15
*** singhj has quit IRC18:15
*** ildikov has joined #openstack-neutron18:16
dougwigamuller, anilvenkata: still around?18:16
amullerdougwig: aye18:16
anilvenkataamuller, yes Assaf18:16
*** singhj has joined #openstack-neutron18:16
dougwigamuller: morning. the short answer is the separate repos were to enable separate release cycles. right now, all neutron stadium projects have to release in utter and total lockstep, and if we did neutron.lib, that'd remain true, since we can't both put it in requirements.txt/pypi *AND* mandate a specific version as "mitaka".18:17
amullerdougwig: I understand18:18
amullerdougwig: Is there any way we can simplify the process so that it's easier to move stuff in there?18:18
dougwiganilvenkata: there's an experimental job to do what you want, but in general, make all of yours changes in *neutron*, don't try to split them. i never intended for devs to have to make multi-repo changes in order to get things done.  i'd go crazy, personally, with that workflow.18:18
*** permalac has quit IRC18:19
amullerdougwig: so do everything in neutron in one patch, then move crap later? that would mean that sometimes 'later' never happens18:19
*** abhiraut has quit IRC18:19
dougwigamuller: how so? it's somewhat a pain on purpose, to make sure we vet that only stable interfaces that we want to support long-term are released. but i don't want it to slow anyone down.18:19
*** rmstar has joined #openstack-neutron18:19
anilvenkatadougwig, but I have to modify constants.py which is now in neutron.lib18:20
dougwigamuller: 'later' only needs to happen if another project wants to import something.18:20
dougwiganilvenkata: no, it is in both places.18:20
*** abhiraut has joined #openstack-neutron18:20
*** mickeys has quit IRC18:20
*** mickeys has joined #openstack-neutron18:21
*** ivar-lazzaro has joined #openstack-neutron18:21
*** tiagogomes has quit IRC18:21
amullerdougwig: Tempest doesn't have to deal with this18:21
amullerdamn them18:22
anilvenkatadougwig, I am modifying ROUTER_INTERFACE_OWNERS which is present in neutron-lib/constants.py and it is now not present in https://github.com/openstack/neutron/blob/master/neutron/common/constants.py18:22
*** sleviim has joined #openstack-neutron18:22
*** sleviim has left #openstack-neutron18:22
dougwigamuller: tempest doesn't have to be both a released server and a lib at the same time. this is decomp fallout, really.18:22
amullerdougwig: yep18:22
anilvenkatadougwig, what will be your suggestion? redefine ROUTER_INTERFACE_OWNERS in neutron/common/constants.py ?18:23
*** fzdarsky__ is now known as fzdarsky|afk18:23
dougwiganilvenkata: two options: you can wait weeks/months for the lib, or *override* it in neutron.common and add a TODO.   ROUTER_INTERFACE_OWNERS = lib.ROUTER_INTERFACES_OWNERS + ...18:23
*** jhershbe has joined #openstack-neutron18:23
dougwiganilvenkata: or if it's trivial, ping me or armax and see if we're close to a release.18:23
dougwiganilvenkata: i can likely push a constant change today if it's really getting in the way, but the general process shouldn't block around that, or the lib is just a failed experiment in pissing us all off.18:24
*** jguy_ has joined #openstack-neutron18:25
amulleranilvenkata: overriding sounds ok to me18:25
*** lucasagomes is now known as lucas-dinner18:25
anilvenkatadougwig, ok, I will override that for now18:25
jguy_Anyone worked with the l2gateway-agent?18:25
*** ivar-lazzaro has quit IRC18:25
anilvenkatadougwig, thanks dougwig18:26
*** Marga_ has joined #openstack-neutron18:27
*** Marga_ has quit IRC18:27
*** doug-fish has joined #openstack-neutron18:27
*** dslevin has joined #openstack-neutron18:27
*** Marga_ has joined #openstack-neutron18:27
dougwiganilvenkata, amuller: the other basic takeaway here is that if we have an interface that requires churning an array to extend, that's likely not a great interface.18:27
*** Marga__ has joined #openstack-neutron18:28
*** Marga__ has quit IRC18:28
*** Marga__ has joined #openstack-neutron18:29
*** Marga_ has quit IRC18:29
openstackgerritRitesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers.  https://review.openstack.org/18815718:32
openstackgerritRitesh Anand proposed openstack/neutron: Avoids logging error on OVS agent start.  https://review.openstack.org/28149818:33
openstackgerritRitesh Anand proposed openstack/neutron: Add IPv6 Prefix Delegation support for DVR  https://review.openstack.org/27765718:33
openstackgerritKevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe  https://review.openstack.org/28287618:34
*** madhu_ak has joined #openstack-neutron18:35
*** john-davidge has joined #openstack-neutron18:35
*** madhu_ak has quit IRC18:36
*** s3wong has joined #openstack-neutron18:37
*** emagana has joined #openstack-neutron18:38
openstackgerritKevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe  https://review.openstack.org/28287618:38
*** singhj has quit IRC18:38
*** ivar-lazzaro has joined #openstack-neutron18:39
*** madhu_ak has joined #openstack-neutron18:40
*** yamahata has quit IRC18:40
*** yamahata has joined #openstack-neutron18:40
*** singhj has joined #openstack-neutron18:41
*** bjornar has joined #openstack-neutron18:44
*** abhiraut has quit IRC18:46
*** rpothier has joined #openstack-neutron18:49
*** abhiraut has joined #openstack-neutron18:51
ajokevinbenton, ping https://review.openstack.org/#/c/243994/18//COMMIT_MSG18:56
ajo:)18:56
ajoshz, ^18:56
*** akshai has quit IRC18:57
*** gangil has quit IRC18:58
*** gangil has joined #openstack-neutron18:58
*** gangil has quit IRC18:58
*** gangil has joined #openstack-neutron18:58
*** akshai has joined #openstack-neutron18:59
*** emagana has quit IRC19:00
*** abhiraut has quit IRC19:01
openstackgerritDustin Lundquist proposed openstack/neutron: devref doc config option separation  https://review.openstack.org/27538119:02
*** numan_ has joined #openstack-neutron19:03
openstackgerritvenkata anil proposed openstack/neutron: use separate device owner for HA router interface  https://review.openstack.org/29165119:06
*** atuvenie has quit IRC19:11
*** mvk has joined #openstack-neutron19:12
*** puck has quit IRC19:15
*** puck has joined #openstack-neutron19:15
*** jhershbe has quit IRC19:15
*** doug-fish has quit IRC19:17
*** doug-fish has joined #openstack-neutron19:17
*** enriquetaso has quit IRC19:18
haleybanilvenkata: pong, but maybe your question was already answered?19:19
anilvenkatahaleyb, Thanks Brian, not yet :) I was asking about PD fix19:19
anilvenkatahaleyb, as you know, this fix https://review.openstack.org/#/c/241227/ is needed to make prefix delegation to work,19:20
*** doug-fis_ has joined #openstack-neutron19:20
anilvenkatahaleyb, so wanted to check with you and carl_baldwin, about Armando's comment on patchset 17 ?19:21
*** doug-fish has quit IRC19:21
*** jckasper has joined #openstack-neutron19:21
*** rickyrem has joined #openstack-neutron19:22
*** claudiub|2 has joined #openstack-neutron19:22
sc68calanilvenkata: It appears we need tempest scenario testing for prefix delegation.19:24
sc68calsince bugs like this slip in19:24
sc68calor at least, functional tests?19:24
haleybanilvenkata: i'm looking, but Carl is out this week19:24
anilvenkatahaleyb,  sc68cal ok19:25
*** doug-fis_ has quit IRC19:25
anilvenkatahaleyb, sc68cal thanks Brian and Sean19:25
haleybsc68cal: yes, there are no tests for PD that i could find19:25
*** claudiub has quit IRC19:25
sc68calI am not surprised.19:25
*** tflynn has joined #openstack-neutron19:25
*** jckasper has quit IRC19:26
haleybwe should at least have a test that configures the agent (unit) and one that starts it (func), but making sure it works is harder since it requires an agent elsewhere19:26
*** singhj has quit IRC19:27
*** tflynn has quit IRC19:27
* sc68cal coughs19:27
anilvenkatahaleyb, may fullstack test19:27
sc68caldevstack plugin ;)19:27
*** rickyrem has quit IRC19:27
*** tflynn has joined #openstack-neutron19:27
*** rickyrem has joined #openstack-neutron19:28
haleybsc68cal: hmm, is there an example of a plugin that starts a daemon?  the PD server is typically run on the upstream router, we'd have to simulate that19:29
*** jckasper has joined #openstack-neutron19:29
sc68calhaleyb: right. My thought is, generate a /48 and feed it into the pd server that the devstack plugin starts19:30
sc68calmaybe do a bit of ovs wiring or netns to hook it into neutron19:30
*** jckasper has quit IRC19:31
*** jckasper has joined #openstack-neutron19:31
amullerhaleyb: functional tests for the agent parts would be a good start IMO19:31
haleybso it looks off-link, then just swallows packets19:31
amullerhaleyb: if you start from the top and abandon the foundations the building eventually crumbles and falls ;-)19:31
*** davideagnello has quit IRC19:32
*** Sukhdev has joined #openstack-neutron19:32
haleybbut IPv6 has 20 years of building poor foundations :)19:32
*** gvrangan has quit IRC19:33
haleybamuller: this actually sounds like good blueprint material19:34
amullerhaleyb: I don't think we've ever tracked the addition of new tests via blueprints19:34
amullerhaleyb: apart from new testing infrastructures19:34
anilvenkataamuller, may be adding IPv6 support in fullstack tests?19:35
amullerunless you're masochistic in some way and enjoy bureaucracy19:35
*** akshai has quit IRC19:35
*** mickeys has quit IRC19:35
haleybamuller: i was thinking there's almost an additional VM here which can be the upstream router, and i won't vote for more government in my IPv6 :)19:36
amulleranilvenkata: that'd be interesting, it *shouldn't* be too much work, I guess depending in how far you want to take it (only neutron tenant networks, or the openstack infrastructure as well?)19:36
*** ericbaum has quit IRC19:36
anilvenkataamuller, only for tenant networks19:36
amulleranilvenkata: we don't have dhcp agent support right now, so some ipv6 address types would fail19:37
*** singhj has joined #openstack-neutron19:37
amullerother than that, I think slaac with radvd should work currently19:37
*** iyamahat has joined #openstack-neutron19:37
anilvenkataamuller, mostly I think they are using SLAAC or stateless19:37
*** yamahata has quit IRC19:38
openstackgerritMerged openstack/neutron-lib: Remove unused pngmath Sphinx extension  https://review.openstack.org/28621219:38
*** akshai has joined #openstack-neutron19:40
*** iyamahat has quit IRC19:42
*** numan_ has quit IRC19:43
*** ivar-lazzaro has quit IRC19:45
*** abhiraut has joined #openstack-neutron19:47
*** tflynn has quit IRC19:48
*** skamithi has quit IRC19:48
amullertidwellr: heya Ryan, I'm updating http://docs.openstack.org/developer/neutron/devref/testing_coverage.html19:50
amullerlooking at BGP testing19:50
*** fawadkhaliq has quit IRC19:52
sc68calah, another thing that needs test coverage ;)19:53
*** sambetts is now known as sambetts|afk19:53
amullersc68cal: the doc itself? =D19:56
sc68cal:) - no I mean the bgp feature19:56
amulleryeah19:56
amullerI know it was squeezed in to Mitaka at the very end19:57
amullershame we're still doing testing after the fact19:57
sc68calyerp.19:57
anilvenkatakevinbenton,  as you know, https://review.openstack.org/#/c/291651/ ,  changes values of device_owner column, still migration script is needed?19:57
kevinbentonanilvenkata: how does it change the existing records?19:58
*** ivase has joined #openstack-neutron19:58
openstackgerritLubosz Kosnik proposed openstack/neutron: Add support for Keepalived VRRP health check  https://review.openstack.org/27354619:59
anilvenkatakevinbenton, yes correct, I am going through http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html19:59
openstackgerritMerged openstack/neutron-lib: Add in missing constants  https://review.openstack.org/28656420:00
openstackgerritMerged openstack/neutron-lib: Add Collector Agent type string  https://review.openstack.org/27915920:00
anilvenkatakevinbenton, thanks Kevin20:00
*** claudiub|2 has quit IRC20:01
kevinbentonanilvenkata: np20:01
*** mickeys has joined #openstack-neutron20:01
*** skamithi has joined #openstack-neutron20:01
*** asingh has joined #openstack-neutron20:01
*** skamithi has quit IRC20:02
haleybanilvenkata: i did have a Q on https://review.openstack.org/#/c/241227/18/neutron/db/db_base_plugin_v2.py - why is L795 required still?  Won't the filters passed to get_port() do that for us?20:02
anilvenkatachecking20:02
*** ig0r_ has joined #openstack-neutron20:03
anilvenkatahaleyb, it is still needed20:04
haleybport has multiple fixed_ips20:04
anilvenkatahaleyb, get_ports is returing a port20:04
anilvenkatahaleyb, yes, it will have other fixed_ip20:04
haleybyeah, i thought of that right after i asked...20:05
anilvenkatahaleyb, :)20:05
*** gvrangan has joined #openstack-neutron20:06
*** ig0r__ has quit IRC20:07
*** tflynn has joined #openstack-neutron20:07
openstackgerritSean M. Collins proposed openstack/neutron: Devref: Remove upgrade testing from missing tests  https://review.openstack.org/29257720:07
*** salv-orlando has joined #openstack-neutron20:08
sc68calamuller: ^ is what happens when you link things and I read them :)20:08
*** rtheis has quit IRC20:08
*** rtheis_ has joined #openstack-neutron20:08
amullersc68cal: I was about to address that in my update20:08
sc68cal:)20:08
*** tflynn has quit IRC20:08
amullersc68cal: I was going to modify that bullet point to say that I want dhcp/l3 also covered20:08
*** iyamahat has joined #openstack-neutron20:08
amullersc68cal: currently only the OVS agent is left on the previous version20:09
sc68calah good point.20:09
*** tflynn has joined #openstack-neutron20:09
amullersc68cal: we need to modify devstack-gate to install those 2 agents on the secondary node, not the primary node20:09
*** tonytan4ever has quit IRC20:09
amullersc68cal: to be fair I never communicated my ask to anyone20:09
sc68calamuller: agreed. Also there are OVS agents on the primary node (which is upgraded) and the secondary node20:10
amullersc68cal: remind me is there nova-compute on the primary node also?20:10
sc68calI'm not 100% sure20:10
sc68calmaybe?20:10
amullersc68cal: yeah =p20:10
amullersc68cal: if not, we can remove the ovs agent from the primary node20:10
amullersc68cal: but I don't think it matters? we can keep nova-compute and ovs-agent on the primary node, just have both old and new20:11
sc68calmy concern about old and new is then we're at the mercy of the scheduler20:11
sc68calto make sure we don't actually break things in upgrade20:11
amullerright20:11
amullerif we run enough tests it shouldn't be a problem20:11
sc68calfair point.20:11
amullerprobability of *every* test executing only the new node is low, unless the nodes are not equal20:12
amullersc68cal: I don't know we'd have nova compute on the primary node though?20:12
amullerwhy*20:12
sc68calprobably test multi_host20:12
*** MVenesio has quit IRC20:12
sc68caldunno20:13
*** emagana has joined #openstack-neutron20:14
*** ociuhandu has quit IRC20:15
*** sayalilunkad has quit IRC20:16
*** abhiraut has quit IRC20:17
openstackgerritKevin Benton proposed openstack/neutron: Add IPAllocation object to session info to stop GC  https://review.openstack.org/29179520:18
*** claudiub has joined #openstack-neutron20:18
*** yfried has joined #openstack-neutron20:19
*** davideagnello has joined #openstack-neutron20:20
*** singhj has quit IRC20:20
*** abhiraut has joined #openstack-neutron20:21
*** singhj has joined #openstack-neutron20:22
*** ankurgupta1 has joined #openstack-neutron20:23
*** sayalilunkad has joined #openstack-neutron20:26
*** anilvenkata has quit IRC20:28
*** gvrangan has quit IRC20:30
*** singhj has quit IRC20:31
*** enriquetaso has joined #openstack-neutron20:32
*** edmondsw has quit IRC20:35
*** gvrangan has joined #openstack-neutron20:36
*** dims_ has joined #openstack-neutron20:36
*** ijw has joined #openstack-neutron20:37
*** anshul has joined #openstack-neutron20:37
*** dims has quit IRC20:38
*** kbringard1 has quit IRC20:41
*** kbringard has joined #openstack-neutron20:41
*** iyamahat has quit IRC20:41
*** erlarese has quit IRC20:44
*** BigWillie has quit IRC20:46
*** ijw has quit IRC20:50
*** gvrangan has quit IRC20:50
*** ivar-lazzaro has joined #openstack-neutron20:50
*** singhj has joined #openstack-neutron20:54
*** banix has quit IRC20:55
*** bapalm has quit IRC20:55
*** Sukhdev has quit IRC20:57
*** bapalm has joined #openstack-neutron20:58
*** singhj has quit IRC20:58
openstackgerritAssaf Muller proposed openstack/neutron: Update testing coverage document  https://review.openstack.org/29260720:58
*** singhj has joined #openstack-neutron21:00
*** alhu has quit IRC21:00
*** alhu has joined #openstack-neutron21:01
*** eddima has joined #openstack-neutron21:03
*** safchain has joined #openstack-neutron21:04
*** eddima has quit IRC21:04
*** singhj has quit IRC21:04
*** yfried has quit IRC21:04
amulleroops, looks like fullstack tests use 'sudo' as a root_helper21:04
amullertime to fix that...21:04
*** john-davidge has quit IRC21:05
tidwellramuller: re test coverage, what's missing?21:08
amullertidwellr: need a row for BGP in http://docs.openstack.org/developer/neutron/devref/testing_coverage.html21:09
amullertidwellr: The goal is to document missing coverage where it's relevant21:09
tidwellramuller: ok, got it21:09
amullertidwellr: For example I believe it's relevant to have functional tests for the BGP agent, like we do for the L3/OVS/DHCP agents21:09
amullertidwellr: Another thing to consider is integration testing21:09
tidwellramuller: we have some functional tests, I have not looked that closely at them21:11
tidwellramuller: I'll take a look at filling out the table21:11
amullertidwellr: I see functional tests for the BGP scheduler21:11
tidwellramuller: could be that we only have tests for that21:11
amullertidwellr: ok21:11
amullertidwellr: Thank you21:11
amullertidwellr: I've found that accountability is the first step towards upping our gaming when it comes to testing :)21:12
tidwellramuller: totally21:12
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: (WIP)DVR: Pings to floatingip returns with fixed-ip on same network  https://review.openstack.org/28917221:12
*** tidwellr has left #openstack-neutron21:13
*** matrohon has joined #openstack-neutron21:14
*** chlong has joined #openstack-neutron21:14
*** john-davidge has joined #openstack-neutron21:16
*** anshul has quit IRC21:16
*** ijw has joined #openstack-neutron21:16
*** fawadkhaliq has joined #openstack-neutron21:17
*** ijw has quit IRC21:17
*** rtheis_ has quit IRC21:19
*** sdague has quit IRC21:19
*** emagana has quit IRC21:22
*** john-davidge has quit IRC21:23
*** jaugustine has quit IRC21:24
*** ociuhandu has joined #openstack-neutron21:25
*** absubram has quit IRC21:26
*** jguy_ has quit IRC21:26
*** rpothier has quit IRC21:27
*** thorst_ has quit IRC21:28
*** thorst_ has joined #openstack-neutron21:31
*** thorst_ has quit IRC21:35
*** yamahata has joined #openstack-neutron21:36
*** john-davidge has joined #openstack-neutron21:37
*** baoli_ has quit IRC21:37
*** jprovazn has quit IRC21:38
*** reedip__ has quit IRC21:39
*** alhu has quit IRC21:39
*** panda has quit IRC21:40
*** alhu has joined #openstack-neutron21:40
*** tflynn has quit IRC21:40
*** regXboi has quit IRC21:40
*** panda has joined #openstack-neutron21:41
*** banix has joined #openstack-neutron21:41
*** tflynn has joined #openstack-neutron21:42
*** neelashah has quit IRC21:43
*** ivar-lazzaro has quit IRC21:45
*** abregman has quit IRC21:46
*** tflynn has quit IRC21:46
*** tflynn has joined #openstack-neutron21:47
*** vhoward has quit IRC21:47
*** dims_ has quit IRC21:53
*** adelia_ has joined #openstack-neutron21:53
*** john-davidge has quit IRC21:56
*** safchain has quit IRC21:56
*** adelia has quit IRC21:57
*** adelia_ has quit IRC21:57
*** korzen has joined #openstack-neutron21:59
*** rickyrem has quit IRC21:59
*** bjornar has quit IRC21:59
*** rickyrem has joined #openstack-neutron21:59
openstackgerritVictor Morales proposed openstack/neutron: SubnetPool and SubnetPoolPrefix to OVO  https://review.openstack.org/27578922:04
*** neiljerram has quit IRC22:06
*** azbiswas has quit IRC22:07
*** azbiswas has joined #openstack-neutron22:08
*** tidwellr has joined #openstack-neutron22:09
openstackgerritRyan Tidwell proposed openstack/neutron: Skip racy BGP tests and remove unnecessary use of LockFixture  https://review.openstack.org/28892122:10
openstackgerritVictor Morales proposed openstack/neutron: SubnetPool and SubnetPoolPrefix to OVO  https://review.openstack.org/27578922:11
*** azbiswas has quit IRC22:12
*** MVenesio has joined #openstack-neutron22:13
*** EkulTails has quit IRC22:14
*** ivar-lazzaro has joined #openstack-neutron22:14
*** fawadkhaliq has quit IRC22:15
*** fawadkhaliq has joined #openstack-neutron22:16
*** krtaylor has quit IRC22:17
*** MVenesio has quit IRC22:18
*** asingh has quit IRC22:22
*** haplo37 has quit IRC22:23
*** fawadkhaliq has quit IRC22:23
*** fawadkhaliq has joined #openstack-neutron22:24
*** SumitNaiksatam has quit IRC22:24
*** yamahata has quit IRC22:26
*** dims has joined #openstack-neutron22:26
*** claudiub has quit IRC22:26
*** asingh has joined #openstack-neutron22:28
*** ircuser-1 has quit IRC22:28
*** ircuser-1 has joined #openstack-neutron22:29
*** abhiraut has quit IRC22:29
openstackgerritRyan Tidwell proposed openstack/neutron: Update test coverage devref with BGP information  https://review.openstack.org/29262722:29
*** abhiraut has joined #openstack-neutron22:29
*** mickeys has quit IRC22:30
tidwellramuller: ^^^^22:30
tidwellramuller: took a stab at the BGP coverage information22:30
*** reedip__ has joined #openstack-neutron22:30
*** korzen has quit IRC22:32
*** abhiraut has quit IRC22:34
*** ivar-lazzaro has quit IRC22:35
*** krtaylor has joined #openstack-neutron22:35
*** matrohon has quit IRC22:37
*** gvrangan has joined #openstack-neutron22:38
*** rickyrem has quit IRC22:38
*** iyamahat has joined #openstack-neutron22:39
*** ivar-lazzaro has joined #openstack-neutron22:39
*** markvoelker has quit IRC22:39
*** lykinsbd has joined #openstack-neutron22:40
*** ivase has quit IRC22:40
*** jguy_ has joined #openstack-neutron22:40
amullertidwellr: thanks, commented22:41
*** alejandrito has quit IRC22:42
*** SumitNaiksatam has joined #openstack-neutron22:42
openstackgerritAssaf Muller proposed openstack/neutron: WIP: Switch fullstack tests to use rootwrap  https://review.openstack.org/29263222:42
tidwellramuller: I've been thinking about functional tests for BGP, they seem a little tricky to me in that we need a BGP process to peer with to validate announcements22:43
amullertidwellr: yeah, I'm not familiar with the details, but do we have access to a process that can receive BGP advertisements?22:43
amullertidwellr: then have the test start it up?22:44
tidwellramuller: we have 2 options: 1. spin up bird/quagga 2. build  a quick and dirty ryu speaker to peer with22:44
tidwellramuller: at least that's what I'm coming up with at the moment22:45
tidwellramuller: maybe you've got some trick up your sleeve22:45
amullertidwellr: I'm not familiar with linux BGP tools =/22:45
*** iyamahat has quit IRC22:46
tidwellramuller: and I'm new to writing functional test jobs :)22:46
tidwellramuller: I'll spend some time this week on this22:46
amullertidwellr: do you use quagga/bird at all in the current code base, or do you only use ryu?22:47
*** lykinsbd has quit IRC22:47
*** krtaylor has quit IRC22:47
*** mvk_ has joined #openstack-neutron22:48
tidwellramuller: the reference impl for the BGP driver uses Ryu22:48
amullertidwellr: ok22:49
amullertidwellr: does Ryu also support a listener?22:49
amullertidwellr: that you could send routes to?22:49
tidwellramuller: I'm thinking it would be pretty easy to craft a simple BGP process with Ryu to peer with for tests22:49
amullertidwellr: ok22:49
amullertidwellr: sounds reasonable22:49
tidwellramuller: yes, Ryu is a fully featured BGP impl22:49
amullertidwellr: (just curious), the Ryu implementation is self-contained, it doesn't use quagga or something?22:50
*** dims has quit IRC22:50
tidwellramuller: yes, it is self-contained which is what made it attractive as a reference impl for BGP22:50
amullertidwellr: it must be a lot of code to implement BGP =D22:50
tidwellramuller: I was surprised how little code it actually is :)22:51
amullertidwellr: with the L3 agent the first test was a router lifecycle test: It creates a router using a L3 agent method (Note the test doesn't actually start a L3 agent process, it instantiates the L3 agent class then manually calls 'create_router')22:52
amullertidwellr: then a bunch of assertions are made to see if the host changed its state (for example if a namespace was created)22:52
*** mvk has quit IRC22:52
amullertidwellr: then the router is deleted, and assertions are made again (was the namespace deleted?)22:52
*** mickeys has joined #openstack-neutron22:52
amullertidwellr: it would be reasonable to follow the same model with the BGP agent22:52
amullertidwellr: so any RPC interactions with the server are faked22:53
amullertidwellr: and ideally that's the only thing you fake/mock22:53
tidwellramuller: sounds reasonable22:54
tidwellramuller: thanks for the pointer, I'll take a look and ping you as I go along :)22:54
*** tidwellr has left #openstack-neutron22:56
*** yamamoto_ has joined #openstack-neutron22:56
*** yamahata has joined #openstack-neutron22:56
*** mlavalle has quit IRC22:57
*** vhosakot has quit IRC22:57
*** fangzhen has joined #openstack-neutron22:59
*** yamamoto_ has quit IRC23:00
*** krtaylor has joined #openstack-neutron23:00
*** amuller has quit IRC23:02
*** vhosakot has joined #openstack-neutron23:02
*** pradk has quit IRC23:02
*** fangzhen has quit IRC23:08
openstackgerritJamie Lennox proposed openstack/neutron: Return oslo_config Opts to config generator  https://review.openstack.org/29264023:09
*** Leo_ has quit IRC23:10
openstackgerritManjeet Singh Bhatia proposed openstack/neutron: Add db migrations test framework with data  https://review.openstack.org/12494623:16
*** asingh has quit IRC23:17
*** asingh has joined #openstack-neutron23:18
*** armax has joined #openstack-neutron23:21
*** dims has joined #openstack-neutron23:21
*** gongysh has joined #openstack-neutron23:21
*** daneyon has joined #openstack-neutron23:23
*** akshai has quit IRC23:26
*** dims has quit IRC23:27
*** lnicolas has joined #openstack-neutron23:28
*** Sukhdev has joined #openstack-neutron23:28
*** tflynn has quit IRC23:30
*** Sukhdev has quit IRC23:31
*** mickeys has quit IRC23:32
*** thorst_ has joined #openstack-neutron23:32
manjeetsarmax: ping23:37
armaxmanjeets: ping23:37
armaxpong23:37
*** Sukhdev has joined #openstack-neutron23:38
manjeetsi had a question regarding manuals your comment after line 696 https://review.openstack.org/#/c/278779/12/doc/admin-guide-cloud/source/networking_adv-features.rst23:38
manjeetsshould it go in a separate file you mean or something else ?23:39
armaxmanjeets: that’s what I thought, like the other features that are described in the manaual23:39
*** Sukhdev has quit IRC23:40
*** asingh has quit IRC23:41
*** Sukhdev has joined #openstack-neutron23:41
*** asingh has joined #openstack-neutron23:41
*** rickyrem has joined #openstack-neutron23:41
*** vhosakot has quit IRC23:41
*** djan has joined #openstack-neutron23:42
*** asingh has quit IRC23:43
*** tflynn has joined #openstack-neutron23:43
*** asingh has joined #openstack-neutron23:43
manjeetsin openstack-manuals/doc/admin-guide-cloud i don't see separate files for different features since this feature is admin only23:43
*** Sukhdev has quit IRC23:44
*** Sukhdev_ has joined #openstack-neutron23:44
manjeetsarmax: I see advance features throught api extension in this https://github.com/openstack/openstack-manuals/blob/master/doc/admin-guide-cloud/source/networking_adv-features.rst23:46
manjeetsand I don't see a separate file for router or specific feature ?23:46
armaxmanjeets: hang on, let me check23:47
*** azbiswas has joined #openstack-neutron23:48
*** mkrcmari__ has joined #openstack-neutron23:48
*** Sukhdev_ has quit IRC23:49
*** brad_behle has quit IRC23:49
*** Fdaisuke has joined #openstack-neutron23:49
*** davideagnello has quit IRC23:49
*** Sukhdev has joined #openstack-neutron23:49
*** banix has quit IRC23:50
*** ZZelle has joined #openstack-neutron23:50
armaxmanjeets: I see two places potentially23:51
armaxnow entry point could be:23:51
armaxhttps://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/networking-guide/source/intro-os-networking.rst23:51
*** mvk_ has quit IRC23:52
armaxor23:52
armaxhttps://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/networking-guide/source/adv-config.rst23:52
*** lujinluo has joined #openstack-neutron23:52
armaxI think it’s probably the former23:53
*** yamahata has quit IRC23:53
manjeetsok i added in admin-guide-cloud/ dir thought its admin only feature23:53
armaxoh I see, you’re right23:53
manjeetsso should i keep in same one and address other comments ?23:54
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR:Pro-active router creation with live migration  https://review.openstack.org/27542023:54
*** salv-orl_ has joined #openstack-neutron23:54
*** haukebruno has quit IRC23:55
*** SumitNaiksatam has quit IRC23:55
armaxmanjeets: I would still think that the content should stand out somehow rather than being where it is but it can always be moved later23:56
*** eil397 has quit IRC23:56
manjeetsok armax: thanks will do that23:56
*** rickyrem has quit IRC23:57
*** haukebruno has joined #openstack-neutron23:57
*** salv-orlando has quit IRC23:57
*** yamamoto_ has joined #openstack-neutron23:57
*** ajmiller has quit IRC23:57
*** davideagnello has joined #openstack-neutron23:57
*** asingh has quit IRC23:58
*** davideagnello has quit IRC23:58
*** davideagnello has joined #openstack-neutron23:59

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