*** markvoelker has quit IRC | 00:00 | |
openstackgerrit | Sofia Enriquez proposed openstack/neutron: Port allowed address pairs to OVO https://review.openstack.org/268274 | 00:03 |
---|---|---|
openstackgerrit | Sofia Enriquez proposed openstack/neutron: [WIP] Port security https://review.openstack.org/292178 | 00:03 |
*** vik_ has quit IRC | 00:04 | |
*** wolverineav has quit IRC | 00:05 | |
*** vik_ has joined #openstack-neutron | 00:09 | |
*** jckasper has joined #openstack-neutron | 00:10 | |
openstackgerrit | Sofia Enriquez proposed openstack/neutron: [WIP] Port security https://review.openstack.org/292178 | 00:12 |
*** baojg has quit IRC | 00:13 | |
openstackgerrit | Sofia Enriquez proposed openstack/neutron: [WIP] Port security https://review.openstack.org/292178 | 00:14 |
openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp to neutron ext resources. https://review.openstack.org/290234 | 00:14 |
openstackgerrit | Sofia Enriquez proposed openstack/neutron: [WIP] Port security https://review.openstack.org/292178 | 00:16 |
*** Shiv_ has joined #openstack-neutron | 00:16 | |
*** jckasper has quit IRC | 00:18 | |
*** Shiv has quit IRC | 00:19 | |
*** Shiv_ is now known as Shiv | 00:19 | |
*** jckasper has joined #openstack-neutron | 00:19 | |
*** Shiv has quit IRC | 00:21 | |
*** dims_ has joined #openstack-neutron | 00:27 | |
*** dims has quit IRC | 00:28 | |
*** jckasper has quit IRC | 00:28 | |
*** jckasper has joined #openstack-neutron | 00:29 | |
*** dims has joined #openstack-neutron | 00:33 | |
*** namnh has joined #openstack-neutron | 00:33 | |
*** jckasper has quit IRC | 00:33 | |
*** dims_ has quit IRC | 00:33 | |
openstackgerrit | fumihiko kakuma proposed openstack/neutron: DO NOT REVIEW: sandbox for Ryu-CI https://review.openstack.org/182943 | 00:34 |
*** jckasper has joined #openstack-neutron | 00:39 | |
*** EinstCrazy has quit IRC | 00:40 | |
*** vhosakot has joined #openstack-neutron | 00:41 | |
openstackgerrit | Li Xipeng proposed openstack/neutron: Add 169.254.169.254 when enable force_metadata https://review.openstack.org/286345 | 00:43 |
*** namnh has quit IRC | 00:44 | |
*** mfuruta has joined #openstack-neutron | 00:50 | |
*** chlong has quit IRC | 00:51 | |
*** zengine_ has joined #openstack-neutron | 00:53 | |
*** jckasper has quit IRC | 00:54 | |
*** david-lyle has quit IRC | 00:54 | |
*** david-lyle has joined #openstack-neutron | 00:54 | |
*** jckasper has joined #openstack-neutron | 00:55 | |
*** zengine has quit IRC | 00:55 | |
*** zengine_ is now known as zengine | 00:55 | |
*** jckasper has quit IRC | 00:58 | |
*** jckasper has joined #openstack-neutron | 00:58 | |
*** cshahani has quit IRC | 01:00 | |
*** apuimedo has quit IRC | 01:00 | |
*** dave-mccowan has quit IRC | 01:01 | |
*** djan_ has joined #openstack-neutron | 01:02 | |
*** djan has quit IRC | 01:04 | |
*** wolverineav has joined #openstack-neutron | 01:05 | |
*** chlong has joined #openstack-neutron | 01:06 | |
*** hoangcx has joined #openstack-neutron | 01:07 | |
*** wolverineav has quit IRC | 01:09 | |
*** gvrangan has quit IRC | 01:10 | |
*** akshai has quit IRC | 01:14 | |
*** jckasper has quit IRC | 01:16 | |
*** jckasper has joined #openstack-neutron | 01:17 | |
*** Linzi has joined #openstack-neutron | 01:18 | |
*** jckasper has quit IRC | 01:21 | |
*** stanzgy has joined #openstack-neutron | 01:24 | |
openstackgerrit | li,chen proposed openstack/neutron: Add RECLONE support for ovs https://review.openstack.org/282233 | 01:24 |
*** kevo has joined #openstack-neutron | 01:26 | |
kevo | Hi, question for neutron lbaas-member-update command. Is the --admin-state-down option still supported? | 01:26 |
openstackgerrit | li,chen proposed openstack/neutron: Add RECLONE support for ovs https://review.openstack.org/282233 | 01:29 |
Linzi | Hi all, I want to contribute a neutron ml2 driver in openstack newton release. | 01:35 |
Linzi | So 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 |
Linzi | Is that means there is no Neutron subproject which can be proposed in the openstack newton? | 01:36 |
Linzi | Or should I wait until the new schedule announcement after newton design summit(Apr 25-29)? Thank you. | 01:36 |
*** panda has quit IRC | 01:40 | |
*** panda has joined #openstack-neutron | 01:41 | |
*** djan has joined #openstack-neutron | 01:44 | |
*** zengine has quit IRC | 01:44 | |
*** djan_ has quit IRC | 01:45 | |
*** zengine has joined #openstack-neutron | 01:49 | |
*** Sukhdev has joined #openstack-neutron | 01:51 | |
*** Sukhdev has quit IRC | 01:53 | |
*** Sukhdev has joined #openstack-neutron | 01:54 | |
*** baojg has joined #openstack-neutron | 01:55 | |
*** markvoelker has joined #openstack-neutron | 01:56 | |
*** zhenguo_ has joined #openstack-neutron | 01:58 | |
*** thorst has joined #openstack-neutron | 02:00 | |
*** markvoelker has quit IRC | 02:00 | |
*** baoli has joined #openstack-neutron | 02:00 | |
*** thorst has quit IRC | 02:05 | |
*** wolverineav has joined #openstack-neutron | 02:07 | |
*** chlong has quit IRC | 02:07 | |
*** dims has quit IRC | 02:07 | |
*** mfuruta has quit IRC | 02:09 | |
*** mfuruta has joined #openstack-neutron | 02:09 | |
*** rkukura has quit IRC | 02:10 | |
*** mfuruta has quit IRC | 02:10 | |
*** mfuruta has joined #openstack-neutron | 02:10 | |
*** mfuruta has quit IRC | 02:11 | |
*** wolverineav has quit IRC | 02:11 | |
*** mfuruta has joined #openstack-neutron | 02:12 | |
*** tflynn has joined #openstack-neutron | 02:14 | |
*** fangzhen__ has quit IRC | 02:15 | |
*** fangzhen__ has joined #openstack-neutron | 02:15 | |
*** fzdarsky__ has joined #openstack-neutron | 02:16 | |
*** ajmiller has quit IRC | 02:18 | |
*** mfuruta has quit IRC | 02:19 | |
*** mfuruta has joined #openstack-neutron | 02:19 | |
*** mfuruta has joined #openstack-neutron | 02:19 | |
*** fzdarsky_ has quit IRC | 02:20 | |
openstackgerrit | yalei wang proposed openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once https://review.openstack.org/291603 | 02:20 |
*** mfuruta has joined #openstack-neutron | 02:20 | |
*** chlong has joined #openstack-neutron | 02:20 | |
*** mfuruta has quit IRC | 02:20 | |
*** mfuruta has joined #openstack-neutron | 02:21 | |
*** mfuruta has quit IRC | 02:21 | |
*** dims has joined #openstack-neutron | 02:21 | |
*** mfuruta has joined #openstack-neutron | 02:21 | |
*** fangzhen has joined #openstack-neutron | 02:26 | |
*** fangzhen__ has quit IRC | 02:28 | |
openstackgerrit | Lujin Luo proposed openstack/neutron: Avoid multiple dhcp-agents being auto-scheduled to one network https://review.openstack.org/288271 | 02:35 |
*** cshahani has joined #openstack-neutron | 02:38 | |
*** fangzhen has quit IRC | 02:39 | |
*** sbalukoff has joined #openstack-neutron | 02:40 | |
*** akshai has joined #openstack-neutron | 02:40 | |
*** fangzhen_ has joined #openstack-neutron | 02:40 | |
*** dims has quit IRC | 02:41 | |
*** tflynn has quit IRC | 02:43 | |
*** ljianbj has joined #openstack-neutron | 02:43 | |
openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: fullstack: use SIGTERM when stopping ovs agents https://review.openstack.org/290277 | 02:45 |
*** fangzhen_ is now known as fangzhen | 02:45 | |
*** ljianbj has left #openstack-neutron | 02:46 | |
*** haukebruno has joined #openstack-neutron | 02:46 | |
*** ljianbj has joined #openstack-neutron | 02:46 | |
*** fangzhen has quit IRC | 02:46 | |
openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: Fix a couple of problems with random hex string generation https://review.openstack.org/288224 | 02:47 |
*** hoangcx has quit IRC | 02:47 | |
*** sbalukoff has quit IRC | 02:52 | |
*** yamamoto has quit IRC | 02:55 | |
*** iwamoto has joined #openstack-neutron | 02:56 | |
*** rkukura has joined #openstack-neutron | 02:57 | |
*** akshai_ has joined #openstack-neutron | 02:59 | |
*** akshai has quit IRC | 03:02 | |
*** baoli has quit IRC | 03:03 | |
*** fangzhen_ has joined #openstack-neutron | 03:03 | |
*** baoli has joined #openstack-neutron | 03:04 | |
*** dave-mccowan has joined #openstack-neutron | 03:06 | |
*** emagana has joined #openstack-neutron | 03:13 | |
openstackgerrit | Fang Zhen proposed openstack/neutron: Switch to oslo.cache https://review.openstack.org/290305 | 03:14 |
*** kevo has quit IRC | 03:15 | |
*** emagana has quit IRC | 03:22 | |
*** chlong has quit IRC | 03:22 | |
openstackgerrit | yalei wang proposed openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once https://review.openstack.org/291603 | 03:23 |
*** dave-mccowan has quit IRC | 03:26 | |
*** banix has quit IRC | 03:26 | |
*** shivrao has joined #openstack-neutron | 03:27 | |
*** Sukhdev has quit IRC | 03:32 | |
*** akshai_ has quit IRC | 03:34 | |
*** lujinluo has quit IRC | 03:37 | |
*** chlong has joined #openstack-neutron | 03:41 | |
*** tflynn has joined #openstack-neutron | 03:41 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Call expunge_all on session before retrying requests https://review.openstack.org/291795 | 03:42 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 03:44 |
*** dkehn has quit IRC | 03:45 | |
*** sridharg has joined #openstack-neutron | 03:45 | |
*** tflynn has quit IRC | 03:47 | |
*** links has joined #openstack-neutron | 03:50 | |
*** dkehn has joined #openstack-neutron | 03:51 | |
*** Sukhdev has joined #openstack-neutron | 03:53 | |
*** yamamoto has joined #openstack-neutron | 03:56 | |
*** markvoelker has joined #openstack-neutron | 03:57 | |
*** ajmiller has joined #openstack-neutron | 04:00 | |
*** ekarlso- has quit IRC | 04:00 | |
*** baoli has quit IRC | 04:01 | |
*** lujinluo has joined #openstack-neutron | 04:01 | |
*** markvoelker has quit IRC | 04:01 | |
*** ajmiller_ has joined #openstack-neutron | 04:02 | |
*** ajmiller has quit IRC | 04:06 | |
*** andymaier has joined #openstack-neutron | 04:07 | |
*** wolverineav has joined #openstack-neutron | 04:08 | |
*** ajmiller_ has quit IRC | 04:09 | |
*** tbachman has quit IRC | 04:11 | |
*** wolverineav has quit IRC | 04:13 | |
*** networkninja has quit IRC | 04:13 | |
*** ekarlso- has joined #openstack-neutron | 04:13 | |
*** jckasper has joined #openstack-neutron | 04:19 | |
*** andymaier has quit IRC | 04:23 | |
*** hoangcx has joined #openstack-neutron | 04:24 | |
*** jckasper has quit IRC | 04:26 | |
*** jckasper has joined #openstack-neutron | 04:26 | |
*** emagana has joined #openstack-neutron | 04:27 | |
*** thorst has joined #openstack-neutron | 04:27 | |
*** jckasper has quit IRC | 04:28 | |
*** jckasper has joined #openstack-neutron | 04:28 | |
*** abregman has quit IRC | 04:31 | |
*** y_sawai has joined #openstack-neutron | 04:31 | |
*** baoli has joined #openstack-neutron | 04:32 | |
openstackgerrit | Li Xipeng proposed openstack/neutron: Add 169.254.169.254 when enable force_metadata https://review.openstack.org/286345 | 04:32 |
*** baoli has quit IRC | 04:33 | |
*** thorst has quit IRC | 04:35 | |
*** chlong has quit IRC | 04:35 | |
*** jckasper has quit IRC | 04:38 | |
*** armax has joined #openstack-neutron | 04:38 | |
*** jckasper has joined #openstack-neutron | 04:38 | |
*** jckasper has quit IRC | 04:42 | |
*** jckasper has joined #openstack-neutron | 04:42 | |
*** prapulla has joined #openstack-neutron | 04:46 | |
*** chlong has joined #openstack-neutron | 04:47 | |
*** baojg has quit IRC | 04:52 | |
*** jckasper has quit IRC | 04:52 | |
*** jckasper has joined #openstack-neutron | 04:53 | |
*** baojg has joined #openstack-neutron | 04:53 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add API test for tag https://review.openstack.org/292204 | 04:54 |
*** djan_ has joined #openstack-neutron | 04:56 | |
*** djan has quit IRC | 04:57 | |
*** jckasper has quit IRC | 04:57 | |
*** otherwiseguy has quit IRC | 05:02 | |
*** sridharg has quit IRC | 05:03 | |
*** chlong has quit IRC | 05:03 | |
*** tbh has joined #openstack-neutron | 05:05 | |
*** boris-42 has joined #openstack-neutron | 05:06 | |
*** gvrangan has joined #openstack-neutron | 05:08 | |
*** gberginc has quit IRC | 05:08 | |
*** cshahani has quit IRC | 05:09 | |
*** y_sawai has quit IRC | 05:10 | |
*** gvrangan has quit IRC | 05:12 | |
*** otherwiseguy has joined #openstack-neutron | 05:12 | |
*** ekuris_ has joined #openstack-neutron | 05:15 | |
*** chlong has joined #openstack-neutron | 05:16 | |
*** vishwana_ has joined #openstack-neutron | 05:16 | |
*** vishwanathj_away has quit IRC | 05:18 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Compute IPAvailabilityRanges in memory during IP allocation https://review.openstack.org/292207 | 05:19 |
*** chlong has quit IRC | 05:22 | |
*** emagana has quit IRC | 05:23 | |
*** lazy_prince has joined #openstack-neutron | 05:27 | |
*** EinstCrazy has joined #openstack-neutron | 05:31 | |
*** y_sawai has joined #openstack-neutron | 05:31 | |
*** thorst has joined #openstack-neutron | 05:33 | |
*** chlong has joined #openstack-neutron | 05:34 | |
*** kiran-r has joined #openstack-neutron | 05:37 | |
*** EinstCrazy has quit IRC | 05:38 | |
*** thorst has quit IRC | 05:41 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add API test for tag https://review.openstack.org/292204 | 05:42 |
*** javeriak has joined #openstack-neutron | 05:43 | |
*** gberginc has joined #openstack-neutron | 05:44 | |
*** fawadkhaliq has joined #openstack-neutron | 05:46 | |
*** tflynn has joined #openstack-neutron | 05:46 | |
*** gberginc has quit IRC | 05:48 | |
*** sridharg has joined #openstack-neutron | 05:48 | |
*** y_sawai has quit IRC | 05:48 | |
*** irenab has quit IRC | 05:51 | |
*** tflynn has quit IRC | 05:52 | |
*** Linzi has left #openstack-neutron | 05:54 | |
*** korzen has joined #openstack-neutron | 05:54 | |
*** tbh has quit IRC | 05:54 | |
*** mkolesni has joined #openstack-neutron | 05:54 | |
*** y_sawai has joined #openstack-neutron | 05:54 | |
*** dedery has joined #openstack-neutron | 05:56 | |
openstackgerrit | Merged openstack/neutron: Extend dicts with original model in create/update https://review.openstack.org/291765 | 05:57 |
openstackgerrit | Tang Chen proposed openstack/neutron: Fix dict.keys() incompatibility in py34 in dhcp test https://review.openstack.org/292048 | 05:57 |
*** markvoelker has joined #openstack-neutron | 05:58 | |
*** emagana has joined #openstack-neutron | 05:58 | |
*** vishwana_ is now known as vishwanathj_away | 05:58 | |
*** javeriak_ has joined #openstack-neutron | 05:59 | |
*** y_sawai has quit IRC | 06:01 | |
*** javeriak has quit IRC | 06:01 | |
*** dedery has quit IRC | 06:01 | |
*** ihrachys has joined #openstack-neutron | 06:02 | |
*** markvoelker has quit IRC | 06:02 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Zanata https://review.openstack.org/292214 | 06:03 |
ihrachys | armax: howdy! | 06:03 |
armax | ihrachys: yello | 06:03 |
ihrachys | armax: *now* I realize why you were talking about India. Silly me :) | 06:04 |
armax | ihrachys: there you go | 06:04 |
ihrachys | took me just three days | 06:04 |
armax | ihrachys: 3 is better than 4 | 06:05 |
armax | or 5 or 6 | 06:05 |
*** abregman has joined #openstack-neutron | 06:05 | |
*** dedery has joined #openstack-neutron | 06:06 | |
armax | ihrachys: I replied to your email | 06:06 |
*** tbh has joined #openstack-neutron | 06:06 | |
*** gvrangan has joined #openstack-neutron | 06:07 | |
armax | ihrachys: 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 point | 06:07 |
armax | I’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 land | 06:08 |
ihrachys | armax: I checked it just now, I think the plan is good. | 06:08 |
armax | ihrachys: what about this one? https://review.openstack.org/#/c/280595/ | 06:08 |
armax | ihrachys: 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 together | 06:08 |
ihrachys | armax: we could land it. not that it's actually needed to do it though: later backport would be as easy. | 06:09 |
ihrachys | armax: aye, I was in the middle of preparing the list of safe ovo patches to unblock | 06:09 |
armax | ihrachys: it depends though | 06:09 |
armax | ihrachys: by how ugly any potential follow-up is needed ;) | 06:09 |
ihrachys | armax: I mean, I am not sure what merge now buys us | 06:10 |
*** Fdaisuke has quit IRC | 06:10 | |
ihrachys | but since it's not used... it could go | 06:10 |
armax | a smaller backport later | 06:10 |
*** ircuser-1 has joined #openstack-neutron | 06:12 | |
*** kevo has joined #openstack-neutron | 06:13 | |
*** neeti has joined #openstack-neutron | 06:14 | |
*** marios_ has joined #openstack-neutron | 06:16 | |
openstackgerrit | Sreekumar S proposed openstack/neutron: SG protocol validation to allow numbers or names https://review.openstack.org/271301 | 06:16 |
*** vhosakot has quit IRC | 06:16 | |
kevinbenton | armax: ping | 06:18 |
armax | kevinbenton: pong? | 06:19 |
*** ihrachys_ has joined #openstack-neutron | 06:19 | |
kevinbenton | armax: 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 in | 06:19 |
*** sridhar_ram has joined #openstack-neutron | 06:19 | |
kevinbenton | e.g. first arg vs second arg vs inside of the request arge | 06:19 |
kevinbenton | arg* | 06:19 |
armax | kevinbenton: oh, right | 06:19 |
ihrachys_ | armax: I just dropped the list of ovo patches via email that I believe are safe to unblock | 06:19 |
armax | ihrachys_: ok, let me get to it | 06:20 |
ihrachys_ | it's not too much, should be rather innocent | 06:20 |
armax | kevinbenton: let me mull over it a bit longer | 06:20 |
armax | ihrachys_: ack | 06: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 |
armax | ihrachys_: that would be great | 06:21 |
kevinbenton | armax: i can refactor the calls to be more consistent in the positioning of the context | 06:21 |
kevinbenton | armax: then the move it to the decorator | 06:21 |
armax | ihrachys_: make sure you beat the crap out of it | 06:21 |
*** ihrachys has quit IRC | 06:21 | |
armax | kevinbenton: that would be an alternative | 06:22 |
*** yfried has joined #openstack-neutron | 06:22 | |
armax | ihrachys_: the patch I mean ;) | 06:22 |
*** kevo has quit IRC | 06:22 | |
ihrachys_ | lol | 06:22 |
ihrachys_ | I am too weak to persuade in any physical way ;) | 06:23 |
*** anilvenkata has joined #openstack-neutron | 06:23 | |
openstackgerrit | Sreekumar S proposed openstack/neutron: Avoid multiple gw ports on the same router https://review.openstack.org/274021 | 06:23 |
openstackgerrit | Sreekumar S proposed openstack/neutron: Huge mem usage during tunnel type driver initialization https://review.openstack.org/251940 | 06:26 |
*** vik_ has quit IRC | 06:28 | |
armax | kevinbenton: assumed that is always safe to call expunge? | 06:28 |
openstackgerrit | venkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface https://review.openstack.org/291641 | 06:29 |
*** itzikb has quit IRC | 06:29 | |
armax | ihrachys_: I’ll keep on cleaning up RC1 | 06:30 |
*** miyagishi_t has quit IRC | 06:30 | |
armax | I looked at bug 1365461 and bug 1375625 right now | 06:30 |
openstack | bug 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 |
openstack | bug 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 IRC | 06:31 | |
*** irenab has joined #openstack-neutron | 06: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 |
kevinbenton | armax: yes, in most cases it will be a noop | 06:35 |
openstackgerrit | Sean M. Collins proposed openstack/neutron: Consolidate the OVS DF and CSUM options into constants https://review.openstack.org/285440 | 06:35 |
kevinbenton | armax: if it's not a noop we don't want what it has anyway | 06:35 |
armax | ihrachys_: what nomination? | 06:39 |
armax | ihrachys_: who says I am running? | 06:39 |
kevinbenton | i do | 06:39 |
ihrachys_ | same here, yes | 06:39 |
kevinbenton | second term is mandatory | 06:39 |
*** pcaruana has quit IRC | 06:39 | |
armax | kevinbenton: parse error, please explain again what you mean on the expunge stuff | 06:39 |
armax | ihrachys_, kevinbenton you’ll do fine without me | 06:40 |
kevinbenton | armax: all of these options we are retrying are the start of a process. there should be nothing in the session | 06:40 |
openstackgerrit | venkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver https://review.openstack.org/281116 | 06:40 |
kevinbenton | armax: if there is something there, it's leftover garbage | 06:40 |
armax | kevinbenton: what process? | 06:40 |
*** javeriak_ has quit IRC | 06:40 | |
kevinbenton | armax: we don't use retries in the middle of an operation | 06:42 |
kevinbenton | armax: they are only at the beginning | 06:42 |
armax | or the end? | 06:42 |
armax | but what if the whole this is a composite one? | 06:43 |
openstackgerrit | venkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver https://review.openstack.org/281116 | 06:43 |
openstackgerrit | venkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports https://review.openstack.org/241227 | 06:43 |
*** djan_ has quit IRC | 06:43 | |
*** john-davidge has joined #openstack-neutron | 06:44 | |
kevinbenton | armax: i don't think that's possible, did you have something particular in mind? | 06:44 |
*** john-davidge has quit IRC | 06:44 | |
armax | kevinbenton: no, mine is an hypotetical | 06:44 |
*** ihrachys has joined #openstack-neutron | 06:46 | |
*** gberginc has joined #openstack-neutron | 06:46 | |
*** marios_ has quit IRC | 06:47 | |
kevinbenton | armax: if there is a composit operation that calls one of these with uncommited data, that's broken IMO | 06:47 |
armax | kevinbenton: fair enough, so I wonder if we are fixing a non-issue | 06:47 |
*** ihrachys_ has quit IRC | 06:47 | |
kevinbenton | armax: no, we are fixing the fact that things can be left in the session from the previous iteration | 06:48 |
*** chlong has quit IRC | 06:48 | |
ihrachys | armax: 'you’ll do fine without me ' no way. you should solve the stadium. :P | 06:48 |
ihrachys | kevinbenton: have a small question on rpc client tests. | 06:49 |
ihrachys | kevinbenton: in gerrit, I mean | 06:49 |
armax | ihrachys: I will do that regardless whether I am or not the next PTL | 06:49 |
* armax walks away for a few | 06:50 | |
ihrachys | armax: you never know. you know what always occurs after a dictator leaves his position? | 06:50 |
kevinbenton | ihrachys: ask away | 06:50 |
*** oanson has joined #openstack-neutron | 06:50 | |
ihrachys | kevinbenton: 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 IRC | 06:53 | |
*** emagana has quit IRC | 06:53 | |
*** ihrachys_ has joined #openstack-neutron | 06:53 | |
openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: Revert use of update_device_list https://review.openstack.org/277279 | 06:53 |
*** oshvartz has joined #openstack-neutron | 06:55 | |
*** ihrachys has quit IRC | 06:55 | |
*** miyagishi_t has joined #openstack-neutron | 06:56 | |
*** moshele has joined #openstack-neutron | 06:56 | |
*** djan has joined #openstack-neutron | 06:59 | |
*** wolverineav has joined #openstack-neutron | 07:01 | |
*** thorst has joined #openstack-neutron | 07:02 | |
*** hichihara has joined #openstack-neutron | 07:03 | |
*** emagana has joined #openstack-neutron | 07:03 | |
*** ihrachys_ has quit IRC | 07:04 | |
*** thorst has quit IRC | 07:07 | |
openstackgerrit | shihanzhang proposed openstack/neutron: Optimize conntrack: modify RPC method https://review.openstack.org/243994 | 07:07 |
*** shivrao has quit IRC | 07:07 | |
*** emagana has quit IRC | 07:07 | |
*** fawadkhaliq has quit IRC | 07:08 | |
*** hwoarang_ has joined #openstack-neutron | 07:09 | |
*** igordcar1_ has joined #openstack-neutron | 07:10 | |
*** hlkv6_ has joined #openstack-neutron | 07:10 | |
*** garyk has joined #openstack-neutron | 07:10 | |
*** javeriak has joined #openstack-neutron | 07:11 | |
*** korzen has joined #openstack-neutron | 07:12 | |
*** akamyshnikova has quit IRC | 07:13 | |
*** kobis has joined #openstack-neutron | 07:13 | |
*** pdr- has joined #openstack-neutron | 07:14 | |
*** miyagishi_t has quit IRC | 07:15 | |
*** gvrangan has quit IRC | 07:15 | |
*** jcook_ has quit IRC | 07:15 | |
*** Marga_ has quit IRC | 07:15 | |
*** ccesario_ has quit IRC | 07:15 | |
*** isq_ has quit IRC | 07:15 | |
*** eezhova has quit IRC | 07:15 | |
*** nplanel has quit IRC | 07:15 | |
*** lezbar has quit IRC | 07:15 | |
*** pdr_ has quit IRC | 07:15 | |
*** sgordon has quit IRC | 07:15 | |
*** bigjools has quit IRC | 07:15 | |
*** doude has quit IRC | 07:15 | |
*** hwoarang has quit IRC | 07:15 | |
*** sirushti has quit IRC | 07:15 | |
*** andyhky has quit IRC | 07:15 | |
*** hlkv6 has quit IRC | 07:15 | |
*** dasm has quit IRC | 07:15 | |
*** X123 has quit IRC | 07:15 | |
*** pmath has quit IRC | 07:15 | |
*** igordcard_ has quit IRC | 07:15 | |
*** mkoderer has quit IRC | 07:15 | |
*** bigjools has joined #openstack-neutron | 07:16 | |
*** miyagishi_t has joined #openstack-neutron | 07:16 | |
*** gvrangan has joined #openstack-neutron | 07:16 | |
*** jcook_ has joined #openstack-neutron | 07:16 | |
*** Marga_ has joined #openstack-neutron | 07:16 | |
*** ccesario_ has joined #openstack-neutron | 07:16 | |
*** isq_ has joined #openstack-neutron | 07:16 | |
*** eezhova has joined #openstack-neutron | 07:16 | |
*** nplanel has joined #openstack-neutron | 07:16 | |
*** lezbar has joined #openstack-neutron | 07:16 | |
*** andyhky has joined #openstack-neutron | 07:16 | |
*** mkoderer has joined #openstack-neutron | 07:16 | |
*** sgordon has joined #openstack-neutron | 07:16 | |
*** doude has joined #openstack-neutron | 07:16 | |
*** sirushti has joined #openstack-neutron | 07:16 | |
*** dasm has joined #openstack-neutron | 07:16 | |
*** X123 has joined #openstack-neutron | 07:16 | |
*** pmath has joined #openstack-neutron | 07:16 | |
*** itzikb has joined #openstack-neutron | 07:24 | |
openstackgerrit | venkata anil proposed openstack/neutron: use separate device owner for HA router interface https://review.openstack.org/291651 | 07:24 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Add exponential back-off RPC client https://review.openstack.org/280595 | 07:24 |
*** djan has quit IRC | 07:26 | |
openstackgerrit | venkata anil proposed openstack/neutron: l2pop fdb flows for HA router ports https://review.openstack.org/255237 | 07:26 |
*** korzen has quit IRC | 07:26 | |
*** akamyshnikova has joined #openstack-neutron | 07:27 | |
*** itzikb has quit IRC | 07:28 | |
*** ihrachys has joined #openstack-neutron | 07:30 | |
*** nmagnezi has joined #openstack-neutron | 07:31 | |
*** gvrangan has quit IRC | 07:32 | |
*** apuimedo has joined #openstack-neutron | 07:33 | |
*** jhershbe has joined #openstack-neutron | 07:38 | |
*** scheuran has joined #openstack-neutron | 07:40 | |
*** gampel has joined #openstack-neutron | 07:41 | |
*** ekuris_ has quit IRC | 07:45 | |
*** neeti has quit IRC | 07:45 | |
*** tflynn has joined #openstack-neutron | 07:50 | |
*** tesseract has joined #openstack-neutron | 07:52 | |
*** tesseract is now known as Guest59187 | 07:52 | |
*** sridharg has quit IRC | 07:52 | |
openstackgerrit | Tang Chen proposed openstack/neutron: Fix dict.keys() incompatibility in py34 in dhcp test https://review.openstack.org/292048 | 07:53 |
*** mickeys has joined #openstack-neutron | 07:54 | |
*** sridharg has joined #openstack-neutron | 07:54 | |
*** evgenyf has joined #openstack-neutron | 07:55 | |
*** hoangcx has quit IRC | 07:56 | |
*** Sukhdev has quit IRC | 07:57 | |
*** Sukhdev has joined #openstack-neutron | 07:58 | |
*** baojg has quit IRC | 07:58 | |
*** markvoelker has joined #openstack-neutron | 07:58 | |
*** emagana has joined #openstack-neutron | 07:59 | |
*** [1]evgenyf has joined #openstack-neutron | 07:59 | |
*** ildikov has quit IRC | 07:59 | |
*** javeriak_ has joined #openstack-neutron | 08:00 | |
*** evgenyf has quit IRC | 08:02 | |
*** [1]evgenyf is now known as evgenyf | 08:02 | |
*** javeriak has quit IRC | 08:02 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-specs: Provide a release postmortem https://review.openstack.org/286413 | 08:02 |
*** emagana has quit IRC | 08:03 | |
*** tflynn has quit IRC | 08:03 | |
*** markvoelker has quit IRC | 08:03 | |
*** javeriak_ has quit IRC | 08:03 | |
*** numans has joined #openstack-neutron | 08:04 | |
*** baojg has joined #openstack-neutron | 08:05 | |
*** jlibosva has joined #openstack-neutron | 08:06 | |
*** mickeys has quit IRC | 08:06 | |
*** tmorin has joined #openstack-neutron | 08:07 | |
*** thorst has joined #openstack-neutron | 08:07 | |
*** javeriak has joined #openstack-neutron | 08:10 | |
*** thorst has quit IRC | 08:11 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-specs: Follow README guidelines in preparation of RC window https://review.openstack.org/283383 | 08:11 |
*** jlibosva has quit IRC | 08:13 | |
*** tmorin has quit IRC | 08:14 | |
*** tmorin has joined #openstack-neutron | 08:15 | |
*** jlibosva has joined #openstack-neutron | 08:16 | |
*** neeti has joined #openstack-neutron | 08:18 | |
*** nyechiel_ has joined #openstack-neutron | 08:22 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron-specs: Follow README guidelines in preparation of RC window https://review.openstack.org/283383 | 08:25 |
*** rcernin has joined #openstack-neutron | 08:25 | |
*** nmagnezi has quit IRC | 08:28 | |
*** matrohon has joined #openstack-neutron | 08:29 | |
*** nmagnezi has joined #openstack-neutron | 08:29 | |
openstackgerrit | Takanori Miyagishi proposed openstack/neutron: do not create namespace if snat is disabled https://review.openstack.org/272059 | 08:29 |
*** tbh has left #openstack-neutron | 08:32 | |
*** yfried has quit IRC | 08:34 | |
*** yfried has joined #openstack-neutron | 08:34 | |
*** wolverineav has quit IRC | 08:35 | |
*** lezbar has quit IRC | 08:37 | |
*** jhershbe has quit IRC | 08:39 | |
*** Marga_ has quit IRC | 08:39 | |
*** Sukhdev has quit IRC | 08:40 | |
*** lezbar has joined #openstack-neutron | 08:42 | |
*** pcaruana has joined #openstack-neutron | 08:43 | |
*** yfried_ has joined #openstack-neutron | 08:45 | |
*** kiran-r has quit IRC | 08:46 | |
*** kiran-r has joined #openstack-neutron | 08:46 | |
*** yfried has quit IRC | 08:47 | |
*** mgoddard has joined #openstack-neutron | 08:47 | |
*** prapulla has quit IRC | 08:49 | |
*** permalac has joined #openstack-neutron | 08:50 | |
*** jhershbe has joined #openstack-neutron | 08:51 | |
*** jckasper has joined #openstack-neutron | 08:54 | |
*** emagana has joined #openstack-neutron | 08:55 | |
*** ekuris_ has joined #openstack-neutron | 08:55 | |
*** lujinluo has quit IRC | 08:56 | |
*** hichihara has quit IRC | 08:57 | |
*** ociuhandu has quit IRC | 08:57 | |
*** tflynn has joined #openstack-neutron | 08:58 | |
*** jckasper has quit IRC | 08:59 | |
*** javeriak has quit IRC | 08:59 | |
*** emagana has quit IRC | 08:59 | |
*** oshvartz has quit IRC | 09:00 | |
*** bsv has quit IRC | 09:00 | |
*** jschwarz has joined #openstack-neutron | 09:01 | |
*** safchain has joined #openstack-neutron | 09:03 | |
*** garyk has quit IRC | 09:04 | |
*** korzen has joined #openstack-neutron | 09:05 | |
*** mhickey has joined #openstack-neutron | 09:06 | |
*** neiljerram has joined #openstack-neutron | 09:06 | |
jschwarz | kevinbenton, good morning :) | 09:09 |
*** tflynn has quit IRC | 09:09 | |
*** thorst has joined #openstack-neutron | 09:09 | |
openstackgerrit | Merged openstack/neutron: SG PRECOMMIT_CREATE should be triggered only once https://review.openstack.org/291603 | 09:11 |
*** oshvartz has joined #openstack-neutron | 09:14 | |
*** jlanoux has joined #openstack-neutron | 09:14 | |
*** davidsha has joined #openstack-neutron | 09:15 | |
*** thorst has quit IRC | 09:17 | |
*** rmstar has quit IRC | 09:18 | |
*** lucas-afk is now known as lucasagomes | 09:19 | |
openstackgerrit | Victor Stinner proposed openstack/neutron: Reuse to_utf8() and jsonutils.dump_as_bytes() https://review.openstack.org/280732 | 09:19 |
*** mfuruta has quit IRC | 09:21 | |
*** dingboopt has joined #openstack-neutron | 09:21 | |
*** javeriak has joined #openstack-neutron | 09:23 | |
*** oshvartz has quit IRC | 09:28 | |
*** jistr has joined #openstack-neutron | 09:28 | |
openstackgerrit | venkata anil proposed openstack/neutron: Avoid calling _get_subnet(s) multiple times in ipam driver https://review.openstack.org/281116 | 09:33 |
*** ArchiFle1 has quit IRC | 09:36 | |
*** wolverineav has joined #openstack-neutron | 09:36 | |
*** ociuhandu has joined #openstack-neutron | 09:36 | |
*** armax has quit IRC | 09:37 | |
*** evgenyf has quit IRC | 09:37 | |
*** jlanoux_ has joined #openstack-neutron | 09:38 | |
*** ArchiFleKs has joined #openstack-neutron | 09:38 | |
*** anshul has joined #openstack-neutron | 09:39 | |
*** jlanoux has quit IRC | 09:41 | |
*** wolverineav has quit IRC | 09:41 | |
*** oshvartz has joined #openstack-neutron | 09:41 | |
*** javeriak has quit IRC | 09:41 | |
openstackgerrit | Adit Sarfaty proposed openstack/neutron: QOS: Provide get methods for policy & network/port binding https://review.openstack.org/292083 | 09:42 |
*** rmstar has joined #openstack-neutron | 09:46 | |
*** emagana has joined #openstack-neutron | 09:50 | |
*** emagana has quit IRC | 09:51 | |
*** emagana has joined #openstack-neutron | 09:51 | |
openstackgerrit | venkata anil proposed openstack/neutron: Fix prefix delegation deleting addresses for ports https://review.openstack.org/241227 | 09:52 |
*** javeriak has joined #openstack-neutron | 09:53 | |
*** emagana has quit IRC | 09:56 | |
*** tpsilva has joined #openstack-neutron | 09:56 | |
*** moshele has quit IRC | 09:59 | |
*** markvoelker has joined #openstack-neutron | 09:59 | |
*** liuyulong has joined #openstack-neutron | 10:01 | |
*** markvoelker has quit IRC | 10:03 | |
*** moshele has joined #openstack-neutron | 10:04 | |
ihrachys | jlibosva: https://review.openstack.org/#/c/277558/ | 10:04 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses https://review.openstack.org/277558 | 10:04 |
*** abregman is now known as abregman|afk | 10:05 | |
*** saggi has joined #openstack-neutron | 10:05 | |
*** evgenyf has joined #openstack-neutron | 10:08 | |
*** dims has joined #openstack-neutron | 10:09 | |
*** cristicalin has joined #openstack-neutron | 10:10 | |
*** slaweq_ has joined #openstack-neutron | 10:10 | |
*** ygbo has joined #openstack-neutron | 10:11 | |
*** sdague has joined #openstack-neutron | 10:13 | |
*** ljianbj has quit IRC | 10:14 | |
*** jlanoux_ has quit IRC | 10:15 | |
*** tbachman has joined #openstack-neutron | 10:15 | |
openstackgerrit | Federico Ressi proposed openstack/neutron: Fix wrong use of list of dict in _check_driver_to_bind https://review.openstack.org/271959 | 10:17 |
openstackgerrit | Federico Ressi proposed openstack/neutron: Write MechanismManager.bind_port unit tests https://review.openstack.org/289863 | 10:17 |
*** aswadr has joined #openstack-neutron | 10:19 | |
*** miyagishi_t has quit IRC | 10:20 | |
*** tbachman has quit IRC | 10:20 | |
*** sambetts|afk is now known as sambetts | 10:21 | |
*** dims has quit IRC | 10:21 | |
*** sridharg has quit IRC | 10:23 | |
*** rossella_s has joined #openstack-neutron | 10:23 | |
openstackgerrit | venkata anil proposed openstack/neutron: HA: Avoid multiple port binding by agents https://review.openstack.org/285773 | 10:23 |
*** mvk has joined #openstack-neutron | 10:24 | |
*** sridharg has joined #openstack-neutron | 10:24 | |
*** sridharg has quit IRC | 10:24 | |
*** kawa2014 has joined #openstack-neutron | 10:25 | |
*** sridharg has joined #openstack-neutron | 10:25 | |
openstackgerrit | yalei wang proposed openstack/neutron: Pass the complete info in sg/rules db into PRECOMMIT_XXX callback https://review.openstack.org/281693 | 10:26 |
*** sridharg has quit IRC | 10:26 | |
*** sridharg has joined #openstack-neutron | 10:26 | |
openstackgerrit | venkata anil proposed openstack/neutron: fullstack: HA failover with l2pop test https://review.openstack.org/286087 | 10:27 |
davidsha | ajo: ping | 10:29 |
*** dims has joined #openstack-neutron | 10:30 | |
*** tiagogomes has joined #openstack-neutron | 10:31 | |
*** evgenyf has quit IRC | 10:32 | |
openstackgerrit | venkata anil proposed openstack/neutron: use separate device owner for HA router interface https://review.openstack.org/291651 | 10:32 |
tiagogomes | Hello, 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 IRC | 10:37 | |
davidsha | ajo: 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 IRC | 10:38 | |
*** thorst has joined #openstack-neutron | 10:38 | |
*** abregman|afk is now known as abregman | 10:38 | |
*** dims has quit IRC | 10:39 | |
*** liuyulong has quit IRC | 10:39 | |
*** dims has joined #openstack-neutron | 10:40 | |
*** baoli has joined #openstack-neutron | 10:41 | |
*** thorst has quit IRC | 10:43 | |
*** armax has joined #openstack-neutron | 10:44 | |
*** dave-mccowan has joined #openstack-neutron | 10:45 | |
*** baoli has quit IRC | 10:46 | |
*** nyechiel_ has quit IRC | 10:47 | |
jlibosva | jschwarz: I left some nits on the custom types patches | 10:51 |
jschwarz | jlibosva, I see - will make sure I refresh | 10:51 |
*** evgenyf has joined #openstack-neutron | 10:53 | |
*** tbachman has joined #openstack-neutron | 10:53 | |
ihrachys | mhickey: https://review.openstack.org/281850 | 11:01 |
*** baojg has quit IRC | 11:03 | |
*** mdih has joined #openstack-neutron | 11:03 | |
*** numan_ has joined #openstack-neutron | 11:03 | |
*** numans has quit IRC | 11:03 | |
*** rodrigods has quit IRC | 11:04 | |
*** rodrigods has joined #openstack-neutron | 11:04 | |
*** claudiub has joined #openstack-neutron | 11:04 | |
*** kiran-r has quit IRC | 11:06 | |
*** tflynn has joined #openstack-neutron | 11:06 | |
ihrachys | armax: hi. | 11:07 |
armax | ihrachys: yo | 11:08 |
mhickey | ihrachys: thanks | 11:09 |
*** fawadkhaliq has joined #openstack-neutron | 11:09 | |
*** tflynn has quit IRC | 11:17 | |
*** wei-wang has joined #openstack-neutron | 11:18 | |
*** ihrachys has quit IRC | 11:18 | |
ajo | davidsha, 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 queue | 11:19 |
ajo | egress is more complicated | 11:19 |
ajo | since 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 |
ajo | I think nftables let's you do that now, btw | 11:20 |
*** roeyc has joined #openstack-neutron | 11:21 | |
*** korzen has quit IRC | 11:21 | |
*** gampel1 has joined #openstack-neutron | 11:22 | |
davidsha | ajo: 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 |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses https://review.openstack.org/277558 | 11:23 |
*** rtheis has joined #openstack-neutron | 11:23 | |
jschwarz | jlibosva, I've taken the liberty to modify the test code of ^, please have a look when you come back :) | 11:24 |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR. https://review.openstack.org/285349 | 11:25 |
*** gampel has quit IRC | 11:25 | |
ajo | davidsha, I believe that's the state so far, we may want to talk to sc68cal to sync on that | 11:27 |
ajo | that could be expanded to have openflow/iptables/nftables generators | 11:27 |
ajo | out of traffic filters | 11:27 |
ajo | or something like that | 11:27 |
ajo | davidsha, or even a common api / extension to declare such filters | 11:28 |
*** ericbaum has joined #openstack-neutron | 11:28 | |
davidsha | ajoi: kk, thats what I'm thinking (common api). | 11:28 |
davidsha | ajo: kk, thats what I'm thinking (common api). | 11:28 |
ajo | I find suboptimal that every feature making use of it has to declare it's own api | 11:28 |
ajo | yeah :D | 11:28 |
*** claudiub has quit IRC | 11:28 | |
davidsha | (Need to start proof reading these messages) | 11:28 |
*** claudiub has joined #openstack-neutron | 11:29 | |
davidsha | I'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-neutron | 11:33 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR. https://review.openstack.org/285349 | 11:36 |
*** ujjain- is now known as ujjain | 11:39 | |
*** buttercup has joined #openstack-neutron | 11:41 | |
*** thorst has joined #openstack-neutron | 11:42 | |
*** thorst_ has joined #openstack-neutron | 11:43 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress https://review.openstack.org/286347 | 11:43 |
johnthetubaguy | armax: 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/6 | 11:45 |
johnthetubaguy | armax: MTU and vhostuser issues | 11:45 |
armax | johnthetubaguy: looking | 11:45 |
*** dims_ has joined #openstack-neutron | 11:45 | |
* johnthetubaguy suddenly remembers armax is in india, rather than doing an all nighter | 11:46 | |
openstackgerrit | Adit 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/292315 | 11:46 |
armax | johnthetubaguy: there you go! | 11:46 |
armax | johnthetubaguy: isn’t that suddenly convenient? | 11:46 |
armax | :) | 11:47 |
*** thorst has quit IRC | 11:47 | |
johnthetubaguy | armax: :) | 11:47 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: _get_router_for_floatingip: Fix confusing indentation https://review.openstack.org/292317 | 11:47 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix validation of floating-ip association https://review.openstack.org/292318 | 11:47 |
*** dims has quit IRC | 11:47 | |
*** cristicalin has quit IRC | 11:48 | |
*** anilvenkata has quit IRC | 11:49 | |
*** anilvenkata has joined #openstack-neutron | 11:50 | |
*** yfried has joined #openstack-neutron | 11:54 | |
*** yfried_ has quit IRC | 11:54 | |
reedip | armax is actually in India? | 11:55 |
armax | reedip: yes I am | 11:56 |
reedip | armax: gr8, where in India? | 11:56 |
armax | BLR | 11:56 |
reedip | oh, the code sprint ? | 11:56 |
reedip | I hope you are enjoying your stay ... | 11:57 |
*** amuller has joined #openstack-neutron | 11:59 | |
*** yfried_ has joined #openstack-neutron | 11:59 | |
*** yfried has quit IRC | 11:59 | |
*** yamamoto has quit IRC | 12:00 | |
*** markvoelker has joined #openstack-neutron | 12:00 | |
*** ihrachys has joined #openstack-neutron | 12:01 | |
*** chlong has joined #openstack-neutron | 12:01 | |
*** korzen has joined #openstack-neutron | 12:01 | |
*** ihrachys has quit IRC | 12:03 | |
*** numan_ has quit IRC | 12:03 | |
*** ihrachys has joined #openstack-neutron | 12:03 | |
*** markvoelker has quit IRC | 12:04 | |
ihrachys | armax: 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 hooks | 12:05 |
* armax looks | 12:05 | |
armax | ihrachys: this needs a rebase though | 12:06 |
*** sayalilunkad_lap has joined #openstack-neutron | 12:06 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Create a hook in base object to modify the fields before DB operations https://review.openstack.org/281850 | 12:07 |
ihrachys | armax: I did rebase, though it was just a zuul mistake (there was a flood of fake conflicts lately in neutron) | 12:08 |
armax | ihrachys: ack | 12:08 |
*** nyechiel_ has joined #openstack-neutron | 12:14 | |
*** sayalilunkad_lap has quit IRC | 12:14 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses https://review.openstack.org/277558 | 12:15 |
*** jckasper has joined #openstack-neutron | 12:15 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add custom SQLAlchemy type for CIDR https://review.openstack.org/285349 | 12:15 |
jschwarz | ihrachys, ^ | 12:15 |
jschwarz | jlibosva, ^ | 12:15 |
*** sayalilunkad_lap has joined #openstack-neutron | 12:17 | |
ihrachys | jschwarz: ++ thanks man | 12:18 |
*** jckasper has quit IRC | 12:20 | |
jlibosva | jschwarz: did you look at my comments? | 12:20 |
jlibosva | jschwarz: on PS3 | 12:21 |
jschwarz | jlibosva, I think I did but I might have missed some | 12:21 |
jschwarz | jlibosva, on which patch? | 12:21 |
jschwarz | oh I see them now | 12:21 |
*** baoli has joined #openstack-neutron | 12:21 | |
*** javeriak has quit IRC | 12:22 | |
*** rossella_s has joined #openstack-neutron | 12:22 | |
jschwarz | jlibosva, aye, I agree with those comments but sadly I have to drop out :< | 12:22 |
armax | amuller: ping | 12:22 |
*** erlarese has joined #openstack-neutron | 12:22 | |
armax | johnthetubaguy pointed me to this issue: https://review.openstack.org/#/c/271444/ | 12:22 |
armax | I thought I’d keep you in the loop since you were working on https://blueprints.launchpad.net/neutron/+spec/vhost-ovs | 12:23 |
*** javeriak has joined #openstack-neutron | 12:23 | |
*** gampel has joined #openstack-neutron | 12:23 | |
ihrachys | rossella_s: https://review.openstack.org/#/c/270230/ | 12:24 |
openstackgerrit | Adit 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/292315 | 12:24 |
armax | ihrachys: is this relevant you think? | 12:24 |
armax | http://logs.openstack.org/50/281850/9/check/gate-neutron-dsvm-fullstack/d1e230a//testr_results.html.gz | 12:24 |
*** wei-wang has quit IRC | 12:24 | |
*** wei-wang has joined #openstack-neutron | 12:25 | |
*** gampel1 has quit IRC | 12:25 | |
*** armax has quit IRC | 12:25 | |
*** gampel1 has joined #openstack-neutron | 12:25 | |
*** baoli has quit IRC | 12:26 | |
*** wei-wang has quit IRC | 12:26 | |
ihrachys | korzen: ^ | 12:28 |
*** gampel has quit IRC | 12:28 | |
*** nikil89_ has joined #openstack-neutron | 12:31 | |
*** edmondsw has joined #openstack-neutron | 12:31 | |
*** boris-42 has quit IRC | 12:34 | |
*** enriquetaso has joined #openstack-neutron | 12:38 | |
*** javeriak_ has joined #openstack-neutron | 12:39 | |
*** javeriak_ has quit IRC | 12:39 | |
*** MVenesio has joined #openstack-neutron | 12:40 | |
*** baoli has joined #openstack-neutron | 12:42 | |
*** baoli_ has joined #openstack-neutron | 12:43 | |
*** javeriak has quit IRC | 12:43 | |
openstackgerrit | John Schwarz proposed openstack/neutron: WIP: Remove auto_schedule_routers codepath https://review.openstack.org/285480 | 12:43 |
*** jckasper has joined #openstack-neutron | 12:45 | |
*** jckasper has quit IRC | 12:45 | |
*** jckasper has joined #openstack-neutron | 12:45 | |
*** links has quit IRC | 12:46 | |
*** baoli has quit IRC | 12:46 | |
*** links has joined #openstack-neutron | 12:46 | |
*** yamamoto has joined #openstack-neutron | 12:48 | |
*** armax has joined #openstack-neutron | 12:48 | |
*** markvoelker has joined #openstack-neutron | 12:51 | |
*** jckasper has quit IRC | 12:52 | |
*** jckasper has joined #openstack-neutron | 12:53 | |
*** lucasagomes is now known as lucas-hungry | 12:54 | |
*** stanzgy has quit IRC | 12:56 | |
*** anilvenkata has quit IRC | 12:58 | |
*** asingh has joined #openstack-neutron | 12:59 | |
*** nmagnezi has quit IRC | 12:59 | |
*** mgoddard_ has joined #openstack-neutron | 12:59 | |
*** asingh has quit IRC | 13:00 | |
*** asingh has joined #openstack-neutron | 13:00 | |
*** asingh has quit IRC | 13:00 | |
*** asingh has joined #openstack-neutron | 13:00 | |
*** ferest has joined #openstack-neutron | 13:00 | |
*** ferest has quit IRC | 13:00 | |
*** mgoddard has quit IRC | 13:02 | |
*** EinstCrazy has joined #openstack-neutron | 13:04 | |
*** davidsha has quit IRC | 13:04 | |
*** ranjithd has joined #openstack-neutron | 13:04 | |
*** fangzhen_ has quit IRC | 13:05 | |
*** anilvenkata has joined #openstack-neutron | 13:08 | |
*** anilvenkata has quit IRC | 13:09 | |
*** anilvenkata has joined #openstack-neutron | 13:10 | |
*** yfried_ has quit IRC | 13:10 | |
*** yfried has joined #openstack-neutron | 13:11 | |
*** lnicolas has quit IRC | 13:12 | |
*** tflynn has joined #openstack-neutron | 13:13 | |
*** neelashah has joined #openstack-neutron | 13:14 | |
*** gberginc has quit IRC | 13:14 | |
*** roeyc has quit IRC | 13:15 | |
*** nmagnezi has joined #openstack-neutron | 13:15 | |
*** ajmiller has joined #openstack-neutron | 13:15 | |
*** roeyc has joined #openstack-neutron | 13:15 | |
*** brad_behle has joined #openstack-neutron | 13:16 | |
*** yarkot_ has joined #openstack-neutron | 13:19 | |
*** neeti has quit IRC | 13:19 | |
*** tflynn has quit IRC | 13:20 | |
*** singhj has joined #openstack-neutron | 13:22 | |
*** haukebruno has quit IRC | 13:22 | |
openstackgerrit | Andreas Scheuring proposed openstack/neutron: lb: interface name hashing for too long vlan interface names https://review.openstack.org/246954 | 13:23 |
*** zengine has quit IRC | 13:23 | |
*** haukebruno has joined #openstack-neutron | 13:24 | |
openstackgerrit | Andreas Scheuring proposed openstack/neutron: l3: Send notify on router_create when ext gw is specified https://review.openstack.org/290470 | 13:24 |
*** BigWillie has joined #openstack-neutron | 13:24 | |
*** ranjithd has quit IRC | 13:25 | |
*** ranjithd has joined #openstack-neutron | 13:25 | |
*** apuimedo has quit IRC | 13:26 | |
*** sridharg has quit IRC | 13:26 | |
*** rickyrem has joined #openstack-neutron | 13:26 | |
*** yarkot_ has quit IRC | 13:27 | |
*** akshai has joined #openstack-neutron | 13:27 | |
*** ajmiller_ has joined #openstack-neutron | 13:29 | |
*** singhj has quit IRC | 13:31 | |
*** krotscheck has joined #openstack-neutron | 13:32 | |
*** ajmiller has quit IRC | 13:32 | |
*** nyechiel_ has quit IRC | 13:32 | |
*** nyechiel has joined #openstack-neutron | 13:32 | |
*** prithiv has joined #openstack-neutron | 13:33 | |
*** singhj has joined #openstack-neutron | 13:33 | |
*** davidsha has joined #openstack-neutron | 13:35 | |
*** vhoward has joined #openstack-neutron | 13:37 | |
*** dedery has quit IRC | 13:38 | |
*** ranjithd has quit IRC | 13:39 | |
*** ajmiller_ has quit IRC | 13:40 | |
*** panda has quit IRC | 13:40 | |
*** panda has joined #openstack-neutron | 13:40 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started https://review.openstack.org/292392 | 13:40 |
ihrachys | jlibosva: hit it into the wall ^ | 13:40 |
*** ranjithd has joined #openstack-neutron | 13:41 | |
*** mvk has quit IRC | 13:41 | |
*** nplanel has quit IRC | 13:41 | |
*** nplanel has joined #openstack-neutron | 13:41 | |
*** banix has joined #openstack-neutron | 13:42 | |
*** ferest has joined #openstack-neutron | 13:42 | |
*** ferest has quit IRC | 13:42 | |
*** lazy_prince has quit IRC | 13:43 | |
sbelous | mordred: ping | 13:44 |
*** brad_behle_ has joined #openstack-neutron | 13:45 | |
*** brad_behle_ has quit IRC | 13:46 | |
*** brad_behle has quit IRC | 13:46 | |
*** javeriak has joined #openstack-neutron | 13:46 | |
*** tbachman has quit IRC | 13:47 | |
*** jpena is now known as jpena|lunch | 13:48 | |
*** irenab has quit IRC | 13:49 | |
mordred | sbelous: sup? | 13:50 |
openstackgerrit | Ryan Moats proposed openstack/neutron: Collector Proof of Concept https://review.openstack.org/213474 | 13:50 |
sbelous | mordred: Hi, Monty! Can you please review my patch to infra? | 13:51 |
*** vhoward has quit IRC | 13:51 | |
sbelous | mordred: this one https://review.openstack.org/#/c/286085/ | 13:51 |
*** reedip__ has joined #openstack-neutron | 13:51 | |
*** skamithi has joined #openstack-neutron | 13:51 | |
jschwarz | jlibosva, lmao. | 13:52 |
jschwarz | jlibosva, really? | 13:52 |
*** javeriak_ has joined #openstack-neutron | 13:52 | |
jlibosva | jschwarz: wat? | 13:53 |
jschwarz | jlibosva, that RNG seed patch | 13:53 |
jlibosva | jschwarz: you're fast man :) | 13:53 |
jschwarz | jlibosva, I get desktop notifications :) | 13:53 |
jlibosva | jschwarz: cool :) | 13:53 |
* jschwarz sees ihrachys' name on the patch, I +1 immediately | 13:53 | |
* jschwarz hopes ihrachys will +2 likewise ;-) | 13:53 | |
*** javeriak has quit IRC | 13:54 | |
*** brad_behle has joined #openstack-neutron | 13:55 | |
ihrachys | jschwarz: I don't like your exchange rate | 13:58 |
jschwarz | ihrachys, XD | 13:58 |
ihrachys | jschwarz: honestly, I haven't even executed the code | 13:58 |
jschwarz | ihrachys, it seems reasonable, mind you | 13:59 |
jschwarz | a simple 'seed' and the reason seems justified | 13:59 |
jschwarz | ihrachys, how did you encounter this? | 13:59 |
*** EkulTails has joined #openstack-neutron | 13:59 | |
*** kbringard has joined #openstack-neutron | 14:00 | |
*** dileepr has joined #openstack-neutron | 14:00 | |
*** kbringard has quit IRC | 14:01 | |
*** asingh has quit IRC | 14:01 | |
*** kbringard has joined #openstack-neutron | 14:01 | |
*** asingh has joined #openstack-neutron | 14:01 | |
*** asingh has quit IRC | 14:02 | |
*** ig0r_ has joined #openstack-neutron | 14:02 | |
*** asingh has joined #openstack-neutron | 14:02 | |
*** lucas-hungry is now known as lucasagomes | 14:03 | |
*** boris-42 has joined #openstack-neutron | 14:05 | |
*** gvrangan has joined #openstack-neutron | 14:06 | |
*** Marga_ has joined #openstack-neutron | 14:06 | |
ihrachys | jschwarz: well, just stole some ideas from https://bugs.launchpad.net/neutron/+bug/1551288 comments | 14:06 |
openstack | Launchpad 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-neutron | 14:06 | |
*** jhershbe_ has joined #openstack-neutron | 14:06 | |
jschwarz | ah | 14:06 |
jschwarz | makes sense | 14:06 |
ihrachys | jschwarz: then played with fork and random.choice locally | 14:07 |
*** jlanoux has joined #openstack-neutron | 14:07 | |
*** jhershbe has quit IRC | 14:07 | |
*** EkulTails has quit IRC | 14:08 | |
*** rpothier has joined #openstack-neutron | 14:08 | |
*** EkulTails has joined #openstack-neutron | 14:08 | |
*** salv-orlando has joined #openstack-neutron | 14:08 | |
ihrachys | jschwarz: 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 IRC | 14:10 | |
*** tongli has joined #openstack-neutron | 14:10 | |
jschwarz | ihrachys, why not seed directly in fullstack's base class? | 14:10 |
ihrachys | jschwarz: well why not? :) | 14:10 |
ihrachys | jschwarz: it may be useful for functional too | 14:10 |
ihrachys | depending on how results of generate_port used | 14:10 |
jschwarz | :) | 14:10 |
ihrachys | since the port generator is in common test code, I decided maybe it's time for global change | 14:10 |
amuller | ihrachys: I commented on the patch, fullstack use a different base class | 14:11 |
* jschwarz has to go to class and will be back in 30 minutes | 14:12 | |
*** dslevin has joined #openstack-neutron | 14:12 | |
*** mvk has joined #openstack-neutron | 14:12 | |
*** haleyb has joined #openstack-neutron | 14:12 | |
*** dslevin has quit IRC | 14:12 | |
*** rickyrem1 has joined #openstack-neutron | 14:13 | |
*** javeriak_ has quit IRC | 14:13 | |
*** dslevin has joined #openstack-neutron | 14:14 | |
*** dslevin has quit IRC | 14:14 | |
*** rickyrem has quit IRC | 14:14 | |
*** fawadkhaliq has quit IRC | 14:15 | |
*** fawadkhaliq has joined #openstack-neutron | 14:15 | |
*** dslevin has joined #openstack-neutron | 14:16 | |
*** david-lyle has quit IRC | 14:16 | |
*** jschwarz has quit IRC | 14:16 | |
*** dslevin has quit IRC | 14:16 | |
*** dslevin has joined #openstack-neutron | 14:17 | |
*** irenab has joined #openstack-neutron | 14:17 | |
*** dslevin has quit IRC | 14:17 | |
*** asingh has quit IRC | 14:18 | |
*** evgenyf has quit IRC | 14:18 | |
*** dslevin has joined #openstack-neutron | 14:19 | |
*** mgoddard_ has quit IRC | 14:19 | |
*** dslevin has quit IRC | 14:19 | |
*** david-lyle has joined #openstack-neutron | 14:19 | |
*** cbouch has joined #openstack-neutron | 14:20 | |
*** dslevin has joined #openstack-neutron | 14:20 | |
*** kawa2014 has quit IRC | 14:20 | |
*** haleyb is now known as haleyb_ | 14:20 | |
*** dslevin has quit IRC | 14:20 | |
*** haleyb has joined #openstack-neutron | 14:21 | |
*** absubram has joined #openstack-neutron | 14:21 | |
*** dslevin has joined #openstack-neutron | 14:22 | |
*** mgoddard has joined #openstack-neutron | 14:22 | |
*** absubram_ has joined #openstack-neutron | 14:22 | |
*** networkninja has joined #openstack-neutron | 14:23 | |
*** dslevin has quit IRC | 14:23 | |
*** javeriak has joined #openstack-neutron | 14:23 | |
*** ajmiller has joined #openstack-neutron | 14:24 | |
*** kawa2014 has joined #openstack-neutron | 14:24 | |
*** absubram has quit IRC | 14:25 | |
*** absubram_ is now known as absubram | 14:25 | |
*** tbachman has joined #openstack-neutron | 14:25 | |
*** yamamoto has quit IRC | 14:25 | |
*** oshvartz has joined #openstack-neutron | 14:25 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: register all objects before validating their hash versions https://review.openstack.org/270230 | 14:27 |
ihrachys | korzen: neutron/extensions/default_subnetpools.py that's where extension is | 14:27 |
ihrachys | korzen: rossella_s: ^ btw I respinned the hasher test patch moving code into test module | 14:27 |
*** pcaruana has quit IRC | 14:28 | |
ihrachys | armax: ^ can I get that patch unblocked? | 14:28 |
*** rossella_ has joined #openstack-neutron | 14:28 | |
armax | amuller: thanks for chiming in on bug 1375625, see my reply…let me know what you think | 14:28 |
openstack | bug 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 looks | 14:28 | |
*** abregman is now known as abregman|mtg | 14:28 | |
*** Leo_ has joined #openstack-neutron | 14:28 | |
amuller | armax: done | 14:29 |
armax | ihrachys: that import looks a bit hideous | 14:29 |
armax | ihrachys: do you mind expanding on the issue somewhat? | 14:30 |
armax | amuller: thank you sir | 14:30 |
*** rossella_s has quit IRC | 14:30 | |
*** crose has joined #openstack-neutron | 14:31 | |
*** yamahata has quit IRC | 14:31 | |
rossella_ | ajo, holaaaa! | 14:32 |
armax | amuller: re-replied | 14:32 |
armax | amuller: :) | 14:32 |
*** asingh has joined #openstack-neutron | 14:32 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started https://review.openstack.org/292392 | 14:34 |
*** absubram has quit IRC | 14:34 | |
ihrachys | armax: I guess if even *you* wonder, then I need to expand it in the commit message; gimme a sec | 14:35 |
*** yamamoto has joined #openstack-neutron | 14:35 | |
*** yamamoto has quit IRC | 14:36 | |
diltram | armax, amuller I also replied about that bug | 14:36 |
*** reedip__ has quit IRC | 14:38 | |
ajo | rossella_, holaaaa! ;) enjoy brno | 14:39 |
rossella_ | ajo ;) | 14:39 |
*** roeyc has quit IRC | 14:40 | |
*** jschwarz has joined #openstack-neutron | 14:40 | |
*** mlavalle has joined #openstack-neutron | 14:41 | |
skamithi | was 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-neutron | 14:41 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: register all objects before validating their hash versions https://review.openstack.org/270230 | 14:42 |
ihrachys | armax: I hope that clarifies ^ | 14:42 |
*** roeyc has joined #openstack-neutron | 14:45 | |
*** wolverineav has quit IRC | 14:46 | |
openstackgerrit | Merged openstack/neutron-vpnaas: neutron-vpnaas fix for 'tox -e api' test https://review.openstack.org/291949 | 14:47 |
*** nmagnezi has quit IRC | 14:48 | |
*** cshahani has joined #openstack-neutron | 14:49 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses https://review.openstack.org/277558 | 14:50 |
*** aswadr has quit IRC | 14:51 | |
*** fawadkhaliq has quit IRC | 14:51 | |
*** vhoward has joined #openstack-neutron | 14:51 | |
*** fawadkhaliq has joined #openstack-neutron | 14:51 | |
*** mgoddard_ has joined #openstack-neutron | 14:52 | |
*** mgoddard has quit IRC | 14:52 | |
*** yamamoto has joined #openstack-neutron | 14:52 | |
*** reedip__ has joined #openstack-neutron | 14:53 | |
*** abregman|mtg is now known as abregman | 14:53 | |
*** tflynn has joined #openstack-neutron | 14:56 | |
*** jpena|lunch is now known as jpena | 14:56 | |
ihrachys | jschwarz: bad boy :P pep8 failures for rebased patches ;) | 14:56 |
jschwarz | ihrachys, :< | 14:57 |
jschwarz | ihrachys, I saw | 14:57 |
*** tflynn has quit IRC | 14:57 | |
jschwarz | ihrachys, I'm very ashamed | 14:57 |
*** javeriak has quit IRC | 14:57 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for CIDR https://review.openstack.org/285349 | 14:58 |
*** javeriak has joined #openstack-neutron | 14:58 | |
*** feleouet has joined #openstack-neutron | 14:58 | |
openstackgerrit | Adelina Tuvenie proposed openstack/neutron: Fixes "OVS Agent doesn't start on Windows" https://review.openstack.org/292434 | 14:59 |
ihrachys | armax: may I ask you to unblock https://review.openstack.org/#/c/277558 [custom types series] | 14:59 |
ihrachys | it's completely unbound from other code, should be safe | 15:00 |
*** irenab has quit IRC | 15:02 | |
*** asingh has quit IRC | 15:02 | |
*** javeriak has quit IRC | 15:03 | |
*** jhershbe_ has quit IRC | 15:03 | |
*** irenab has joined #openstack-neutron | 15:03 | |
*** irenab has quit IRC | 15:03 | |
*** irenab has joined #openstack-neutron | 15:05 | |
*** ollivier has joined #openstack-neutron | 15:06 | |
*** oshvartz has quit IRC | 15:06 | |
*** kbringard1 has joined #openstack-neutron | 15:07 | |
*** kbringard has quit IRC | 15:07 | |
*** mkolesni has quit IRC | 15:08 | |
davidsha | ajo, ihrachys: ping | 15:08 |
ihrachys | davidsha: hi | 15:08 |
armax | ihrachys: to be honest I’d rather not rely on such a magic if I can help it | 15:08 |
*** gangil has joined #openstack-neutron | 15:09 | |
*** gangil has quit IRC | 15:09 | |
*** gangil has joined #openstack-neutron | 15:09 | |
ihrachys | armax: well otherwise we are doomed to forget updating the test sometimes and getting random failures until we add an import somewhere. | 15:09 |
davidsha | ihrachys: Hey, is it ok with you if we work on the TODOs and nits for the dscp patch as a separate patch? | 15:09 |
armax | ihrachys: but that’s at least predictable | 15:10 |
armax | ihrachys: _predictable_ | 15:10 |
*** networkninja has quit IRC | 15:10 | |
armax | ihrachys: how about doing something like we do for the db migration stuff? | 15:11 |
ihrachys | davidsha: since we are out of M rush, I would like to see them all up for review before we start landing them | 15:11 |
armax | ihrachys: we’ll do it often enough that gets imprimted into people’s minds | 15:11 |
njohnston | ihrachys: 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 |
ihrachys | armax: I personally hate manual solutions, but if you are strong on it, I can do it otherwise | 15:12 |
*** gangil has quit IRC | 15:12 | |
anteaya | hey 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.html | 15:12 |
armax | ihrachys: me too | 15:12 |
ihrachys | njohnston: I see. | 15:12 |
ihrachys | njohnston: do we have too many of those TODOs? | 15:12 |
armax | ihrachys: but if the automatic ones rely on recursive imports, then I freak out | 15:13 |
*** amotoki has quit IRC | 15:14 | |
ihrachys | armax: that's a matter of taste :) | 15:14 |
armax | and correctness | 15:14 |
ihrachys | armax: so you are strong? I can go and change it in 5 mins | 15:14 |
armax | if we can get someone else eyes on this I am happy to change my mind | 15:14 |
armax | but I would definitly not want to merge that in haste | 15:15 |
ihrachys | ok sec, let me make you happy | 15:15 |
armax | you don’t have to | 15:15 |
njohnston | ihrachys: 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 patch | 15:16 |
armax | let’s get someone else’s opinion | 15:16 |
ihrachys | well I want to land it quicker since others will need to apply a similar approach. | 15:16 |
*** jhershbe_ has joined #openstack-neutron | 15:16 | |
davidsha | ihrachys: there is about 6 TODOs associated with this dscp. (4 for ajo and 2 for me) | 15:16 |
*** tonytan4ever has quit IRC | 15:16 | |
ihrachys | njohnston: of those, I believe fullstack is the most important | 15:17 |
ihrachys | since it may indicate some issue with the patch | 15:17 |
ihrachys | others are innocent assuming we get them handled in due time | 15:17 |
ihrachys | kevinbenton: 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 |
njohnston | ihrachys: I am working on fullstack now, since it is split into a separate patch: https://review.openstack.org/#/c/288392/ | 15:18 |
ihrachys | njohnston: yeah, I mean the TODO for a test disabled | 15:19 |
*** tonytan4ever has joined #openstack-neutron | 15:19 | |
*** oshvartz has joined #openstack-neutron | 15:19 | |
armax | ihrachys: talking about Mitaka critical fixes | 15:19 |
armax | https://review.openstack.org/#/c/292434/ | 15:19 |
njohnston | ihrachys: Ah, yes, got it. | 15:19 |
*** emagana has joined #openstack-neutron | 15:23 | |
*** links has quit IRC | 15:23 | |
*** ig0r__ has joined #openstack-neutron | 15:23 | |
davidsha | ihrachys: last question, do you think it's worth back porting the L2-agent patch to stable-liberty? | 15:23 |
ihrachys | armax: aye. I posted a question there. | 15:24 |
*** jwarendt has joined #openstack-neutron | 15:24 | |
ihrachys | davidsha: which patch? for agent API? | 15:24 |
ihrachys | davidsha: absolutely not, it's a new feature :) | 15:25 |
*** regXboi has joined #openstack-neutron | 15:25 | |
*** doug-fis_ has joined #openstack-neutron | 15:26 | |
*** gangil has joined #openstack-neutron | 15:26 | |
*** gangil has quit IRC | 15:26 | |
*** gangil has joined #openstack-neutron | 15:26 | |
*** ig0r_ has quit IRC | 15:26 | |
*** Swami has joined #openstack-neutron | 15:26 | |
davidsha | ihrachys: 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 IRC | 15:27 | |
armax | ihrachys: replied | 15:28 |
*** vhosakot has joined #openstack-neutron | 15:28 | |
*** yamamoto has quit IRC | 15:30 | |
*** salv-orlando has quit IRC | 15:31 | |
*** mickeys has joined #openstack-neutron | 15:32 | |
*** salv-orlando has joined #openstack-neutron | 15:33 | |
*** armax has quit IRC | 15:33 | |
ihrachys | davidsha: which problem? | 15:35 |
openstackgerrit | venkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface https://review.openstack.org/291641 | 15:35 |
*** armax has joined #openstack-neutron | 15:35 | |
anilvenkata | pc_m, Hi Paul | 15:35 |
anilvenkata | pc_m, need help | 15:36 |
pc_m | anilvenkata: hi | 15:36 |
*** emagana has quit IRC | 15:36 | |
anilvenkata | pc_m, I added a new flag in neuton-lib, and want to use that in neutron change | 15:36 |
*** jaugustine has joined #openstack-neutron | 15:36 | |
anilvenkata | pc_m, this is neutron-lib change https://review.openstack.org/#/c/291641/ | 15:37 |
davidsha | ihrachys: 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 |
anilvenkata | pc_m, this is neutron change https://review.openstack.org/#/c/291651/ which is dependent on above neutron-lib change | 15:37 |
anilvenkata | pc_m, how to add this dependency | 15:37 |
anilvenkata | ? | 15:37 |
anilvenkata | pc_m, I added Depends-On: in commit message, still it is not picking the change | 15:38 |
anilvenkata | pc_m, I also added change suggested in wiki https://wiki.openstack.org/wiki/Neutron/Lib as "Depends-On" | 15:39 |
pc_m | anilvenkata: 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 |
amuller | pc_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 IRC | 15:41 | |
anilvenkata | dougwig, any suggestion for the above issue? | 15:41 |
pc_m | anilvenkata: Doesn't look like dougwig is online yet. | 15:42 |
openstackgerrit | Adelina Tuvenie proposed openstack/neutron: Ovs agent can't start on Windows because of validate_local_ip https://review.openstack.org/227077 | 15:42 |
amuller | pc_m: dougwig: That's what they did in Tempest... move from tempest_lib to tempest.lib in-tree. Any thoughts pc_m? | 15:42 |
anilvenkata | pc_m, Thanks Paul | 15:42 |
*** gampel1 has quit IRC | 15:42 | |
pc_m | amuller: I don't know if it was considered. Personally, I'm not well versed in the packaging stuff. | 15:43 |
*** gangil has quit IRC | 15:44 | |
*** alhu has joined #openstack-neutron | 15:44 | |
amuller | pc_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_m | anilvenkata: Didn't help much... except point you in a direction :) I'm curious as well as to the answer. | 15:45 |
ihrachys | armax: 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 IRC | 15:46 | |
pc_m | amuller: Initially, some of the neutron-lib stuff was placed in the tree to try things out | 15:46 |
*** thorst_ is now known as thorst_afk | 15:46 | |
armax | ihrachys: one of many? :) | 15:46 |
*** davideagnello has joined #openstack-neutron | 15:47 | |
ihrachys | davidsha: their extensions should still work with older agent versions, so not sure how it would help. | 15:47 |
*** moshele has quit IRC | 15:47 | |
ihrachys | davidsha: I mean, older *liberty* versions | 15:48 |
*** jprovazn has joined #openstack-neutron | 15:48 | |
ihrachys | davidsha: otherwise we start requiring a minor version of the agent, which is not really the way to go. | 15:48 |
ihrachys | amuller: ok, we need a second opinion on https://review.openstack.org/#/c/270230/ | 15:49 |
amuller | looking | 15:50 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress https://review.openstack.org/286347 | 15:50 |
davidsha | ihrachys: 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-neutron | 15:50 | |
openstackgerrit | venkata anil proposed openstack/neutron-lib: New device owner flag for HA router interface https://review.openstack.org/291641 | 15:50 |
*** tonytan4ever has quit IRC | 15:50 | |
*** tonytan4ever has joined #openstack-neutron | 15:51 | |
*** atuvenie has joined #openstack-neutron | 15:51 | |
*** mgoddard_ has quit IRC | 15:52 | |
*** mgoddard has joined #openstack-neutron | 15:52 | |
amuller | ihrachys: 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 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for IP addresses https://review.openstack.org/277558 | 15:52 |
*** jhershbe_ has quit IRC | 15:52 | |
ihrachys | amuller: importing all modules manually | 15:52 |
armax | amuller: where? | 15:52 |
ihrachys | amuller: like we do for db models in head.py | 15:53 |
*** iyamahat has joined #openstack-neutron | 15:53 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for CIDR https://review.openstack.org/285349 | 15:54 |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add custom SQLAlchemy type for MACAddress https://review.openstack.org/286347 | 15:54 |
*** tonytan_brb has joined #openstack-neutron | 15:54 | |
*** tonytan4ever has quit IRC | 15:55 | |
*** tonytan_brb has quit IRC | 15:55 | |
*** tflynn has joined #openstack-neutron | 15:55 | |
*** tflynn_ has joined #openstack-neutron | 15:56 | |
*** gvrangan has quit IRC | 15:56 | |
amuller | ihrachys: 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 proof | 15:56 |
amuller | ihrachys: armax: When new models are added (But not added to the import list), we'll start getting intermittent unit test failures again | 15:57 |
armax | amuller: it’s bugs I am worried about, but if you think there are none, feel free to go ahead | 15:57 |
amuller | I think we can stomach the 'magicness' to improve quality in this case | 15:57 |
*** tonytan4ever has joined #openstack-neutron | 15:57 | |
armax | amuller: when bugs creep in, I’d be happy to have your head on a silver plate :) | 15:57 |
armax | how about that? | 15:57 |
*** minwang2 has joined #openstack-neutron | 15:58 | |
amuller | armax: You will have both Ihar and myself, I'll stand behind that :) | 15:58 |
armax | amuller: I am in! | 15:58 |
*** emagana has joined #openstack-neutron | 15:58 | |
* armax pretends he’s looking the other way | 15:58 | |
ihrachys | armax: that's the approach! | 15:59 |
amuller | ihrachys: what do you think of iglob vs recursive function though? | 15:59 |
*** lazy_prince has joined #openstack-neutron | 15:59 | |
*** jckasper has quit IRC | 16:00 | |
*** tflynn has quit IRC | 16:00 | |
armax | ihrachys: and if you do go ahead with the magic import | 16:00 |
armax | I want a test for the test | 16:00 |
armax | :P | 16:00 |
ihrachys | armax: what amuses me lately is that I can't distinguish you joking and telling your mind | 16:01 |
*** jckasper has joined #openstack-neutron | 16:01 | |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: WIP: Introduces ovo objects for security groups https://review.openstack.org/284738 | 16:02 |
amuller | armax: ihrachys: That's not an outrageous idea, if you rename import_modules to get_modules_with_object_models, it would be easily testable | 16:02 |
*** tflynn_ has quit IRC | 16:02 | |
amuller | ihrachys: So you just test if the function detects the correct files | 16:02 |
armax | we should have that method invoked in neutron/__init__.py | 16:02 |
*** killer_prince has joined #openstack-neutron | 16:02 | |
amuller | ihrachys: then you have another method that iterates on the results and imports everything | 16:02 |
*** ekuris_ has quit IRC | 16:02 | |
*** singhj has quit IRC | 16:03 | |
* ihrachys is looking for his cyanide pills | 16:03 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time for each test started https://review.openstack.org/292392 | 16:03 |
ihrachys | amuller: with os.getpid() melded in but please let's not reset gate until we validate it ^ | 16:04 |
*** tflynn has joined #openstack-neutron | 16:04 | |
*** lazy_prince has quit IRC | 16:04 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Reset RNG seed with current time and pid for each test started https://review.openstack.org/292392 | 16:04 |
amuller | ihrachys: Since it wasn't WIP I assumed you did validate it locally | 16:04 |
ihrachys | amuller: ack. my fault. | 16:04 |
openstackgerrit | Sayali Lunkad proposed openstack/neutron: WIP: Introduces ovo objects for security groups https://review.openstack.org/284738 | 16:05 |
*** bsv has quit IRC | 16:05 | |
*** jckasper has quit IRC | 16:06 | |
*** emagana has quit IRC | 16:06 | |
*** oshvartz has quit IRC | 16:06 | |
*** absubram has joined #openstack-neutron | 16:07 | |
*** ramishra has quit IRC | 16:08 | |
*** pcaruana has joined #openstack-neutron | 16:09 | |
*** ramishra has joined #openstack-neutron | 16:10 | |
*** jaypipes has joined #openstack-neutron | 16:10 | |
*** amuller is now known as amuller_afk | 16:11 | |
*** fawadkhaliq has quit IRC | 16:11 | |
*** jschwarz has quit IRC | 16:13 | |
*** EinstCrazy has quit IRC | 16:14 | |
openstackgerrit | venkata anil proposed openstack/neutron: use separate device owner for HA router interface https://review.openstack.org/291651 | 16:15 |
ihrachys | amuller_afk: re iglob: I *think* that it would not get me into unspecified number of directories deeper | 16:16 |
ihrachys | amuller_afk: and I don't wan't to hardcode the code structure in the function | 16:16 |
*** singhj has joined #openstack-neutron | 16:16 | |
*** tidwellr has joined #openstack-neutron | 16:19 | |
*** cbouch has quit IRC | 16:19 | |
*** anilvenkata has quit IRC | 16:21 | |
*** sayalilunkad_lap has quit IRC | 16:21 | |
openstackgerrit | Martin Hickey proposed openstack/neutron: Prevent primary keys in Neutron OVOs from being updated https://review.openstack.org/292488 | 16:22 |
*** sayalilunkad_lap has joined #openstack-neutron | 16:22 | |
*** sayalilunkad_lap has quit IRC | 16:22 | |
*** slunkad has joined #openstack-neutron | 16:23 | |
*** tflynn has quit IRC | 16:25 | |
*** haplo37 has joined #openstack-neutron | 16:26 | |
*** gangil has joined #openstack-neutron | 16:26 | |
*** gangil has quit IRC | 16:26 | |
*** gangil has joined #openstack-neutron | 16:26 | |
*** scheuran has quit IRC | 16:27 | |
mhickey | ihrachys, korzen, rossella_s, jlibosva, sayalilunkad: Review if you have time ^^^^^. Go easy, I am very near you! :) | 16:27 |
*** fawadkhaliq has joined #openstack-neutron | 16:29 | |
*** dileepr has quit IRC | 16:29 | |
*** yamamoto has joined #openstack-neutron | 16:31 | |
*** abregman has quit IRC | 16:31 | |
*** Guest59187 has quit IRC | 16:31 | |
*** yfried has quit IRC | 16:33 | |
*** anshul has quit IRC | 16:33 | |
*** kobis has quit IRC | 16:34 | |
*** aswadr has joined #openstack-neutron | 16:37 | |
*** aswadr has left #openstack-neutron | 16:38 | |
*** matrohon has quit IRC | 16:39 | |
*** rpothier has quit IRC | 16:40 | |
*** slaweq_ has quit IRC | 16:40 | |
*** doug-fis_ has quit IRC | 16:41 | |
*** doug-fish has joined #openstack-neutron | 16:41 | |
*** yamamoto has quit IRC | 16:41 | |
ihrachys | armax: amuller_afk: now, just don't tell me you haven't asked a test for a test: https://review.openstack.org/270230 | 16:43 |
*** wolverineav has joined #openstack-neutron | 16:43 | |
*** jhershbe has joined #openstack-neutron | 16:44 | |
*** neelashah has quit IRC | 16:46 | |
*** armax has quit IRC | 16:46 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR: Agent side change for live migration with floatingip https://review.openstack.org/260738 | 16:46 |
*** singhj has quit IRC | 16:46 | |
*** doug-fish has quit IRC | 16:47 | |
*** wolverineav has quit IRC | 16:47 | |
*** wolverineav has joined #openstack-neutron | 16:50 | |
*** Marga_ has quit IRC | 16:51 | |
manjeets | armax: ping | 16:51 |
*** Marga_ has joined #openstack-neutron | 16:54 | |
*** jckasper has joined #openstack-neutron | 16:55 | |
*** rickyrem1 has quit IRC | 16:56 | |
*** rcernin has quit IRC | 16:56 | |
*** chlong has quit IRC | 16:56 | |
*** feleouet has quit IRC | 16:57 | |
*** jhershbe has quit IRC | 16:57 | |
*** jckasper has quit IRC | 16:59 | |
*** jckasper has joined #openstack-neutron | 16:59 | |
*** Marga_ has quit IRC | 16:59 | |
*** kevo has joined #openstack-neutron | 17:00 | |
*** killer_prince has quit IRC | 17:01 | |
*** doug-fish has joined #openstack-neutron | 17:01 | |
ihrachys | rossella_: 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 IRC | 17:03 | |
rossella_ | ihrachys, ok :) | 17:03 |
*** alhu has joined #openstack-neutron | 17:03 | |
*** singhj has joined #openstack-neutron | 17:04 | |
*** jckasper has quit IRC | 17:05 | |
*** jckasper has joined #openstack-neutron | 17:06 | |
*** mdih has quit IRC | 17:06 | |
*** iyamahat has quit IRC | 17:06 | |
*** yamahata has joined #openstack-neutron | 17:07 | |
*** emagana has joined #openstack-neutron | 17:07 | |
*** ollivier has quit IRC | 17:08 | |
*** dims has joined #openstack-neutron | 17:08 | |
jlibosva | rossella_: https://review.openstack.org/#/c/270309/5/neutron/tests/unit/api/rpc/handlers/test_resources_rpc.py | 17:08 |
*** jckasper has quit IRC | 17:09 | |
*** jckasper has joined #openstack-neutron | 17:09 | |
*** dims_ has quit IRC | 17:09 | |
*** neelashah has joined #openstack-neutron | 17:09 | |
*** jckasper has quit IRC | 17:11 | |
*** azbiswas has joined #openstack-neutron | 17:12 | |
*** jckasper has joined #openstack-neutron | 17:12 | |
*** gvrangan has joined #openstack-neutron | 17:13 | |
*** jckasper has quit IRC | 17:13 | |
*** jckasper has joined #openstack-neutron | 17:13 | |
*** shivharis has joined #openstack-neutron | 17:14 | |
*** nyechiel has quit IRC | 17:14 | |
*** jckasper has quit IRC | 17:14 | |
*** jckasper has joined #openstack-neutron | 17:14 | |
ihrachys | amuller_afk: the fullstack change for random seed passed twice. how many times do we want to validate it before pushing? | 17:14 |
openstackgerrit | Adelina Tuvenie proposed openstack/neutron: Fixes "OVS Agent doesn't start on Windows" https://review.openstack.org/292434 | 17:15 |
*** tmorin has quit IRC | 17:15 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: objects: introduce object for extra dhcp options https://review.openstack.org/273072 | 17:16 |
ihrachys | mhickey: rebased and fixed tests ^ should be good to go if passes gate. | 17:16 |
*** sc68cal has joined #openstack-neutron | 17:17 | |
*** eil397 has joined #openstack-neutron | 17:17 | |
*** jckasper has quit IRC | 17:19 | |
*** prithiv has quit IRC | 17:20 | |
*** thorst_afk is now known as thorst_ | 17:20 | |
mhickey | ihrachys: thanks | 17:23 |
*** ildikov_ has quit IRC | 17:25 | |
*** ygbo has quit IRC | 17:25 | |
*** jistr has quit IRC | 17:28 | |
*** neelashah has quit IRC | 17:29 | |
*** singhj has quit IRC | 17:29 | |
*** jckasper has joined #openstack-neutron | 17:30 | |
*** adelia has joined #openstack-neutron | 17:31 | |
*** neelashah has joined #openstack-neutron | 17:31 | |
*** wolverineav has quit IRC | 17:32 | |
*** jlanoux has quit IRC | 17:34 | |
*** SumitNaiksatam has joined #openstack-neutron | 17:35 | |
*** alhu has quit IRC | 17:35 | |
*** amuller_afk is now known as amuller | 17:36 | |
*** crose has quit IRC | 17:37 | |
amuller | ihrachys: last I checked iglob works fine, just returns all subdirs recursively | 17:37 |
amuller | ihrachys: without any assumptions | 17:37 |
ihrachys | amuller: not sure which pattern to use | 17:37 |
*** bbzhao has joined #openstack-neutron | 17:38 | |
amuller | ihrachys: if you're already 'in the dir' just '.' | 17:38 |
ihrachys | oh | 17:38 |
ihrachys | :-) | 17:38 |
*** roeyc has quit IRC | 17:38 | |
amuller | ihrachys: I was really excited about iglob when I found out about it =p | 17:38 |
ihrachys | >>> list(glob.iglob('.')) | 17:38 |
ihrachys | ['.'] | 17:38 |
jlibosva | also you need to limit it by .py file extension | 17:39 |
amuller | ihrachys: Well, I don't remember offhand, play with it =p | 17:39 |
amuller | ihrachys: it has good docs IIRC | 17:39 |
ihrachys | amuller: why do people hate recursion? :) | 17:39 |
* jlibosva played as well, didn't find the pattern | 17:39 | |
*** panda has quit IRC | 17:40 | |
*** panda has joined #openstack-neutron | 17:40 | |
*** bzhao has quit IRC | 17:41 | |
*** shivrao has joined #openstack-neutron | 17:44 | |
amuller | ihrachys: jlibosva: It seems like glob is only recursive from python 3.5 with a new parameter :( | 17:45 |
amuller | ihrachys: my memory deceived me | 17:45 |
amuller | ihrachys: so we can either use os.walk, or drop support for python < 3.5 and use glob, your call! | 17:46 |
openstackgerrit | Merged openstack/neutron: Create a hook in base object to modify the fields before DB operations https://review.openstack.org/281850 | 17:46 |
*** sridhar_ram has joined #openstack-neutron | 17:47 | |
jlibosva | amuller: or have "if version.major == 2: os.walk else: glob.iglob" :D JK | 17:47 |
ihrachys | amuller: I vote dropping support for py3 right away. py4k is the next big thing. | 17:48 |
amuller | ihrachys: only if it py4 is backwards incompatible of course | 17:48 |
ihrachys | amuller: of course. that's the spirit | 17:49 |
*** jwarendt has quit IRC | 17:50 | |
*** wolverineav has joined #openstack-neutron | 17:51 | |
*** shivharis has quit IRC | 17:51 | |
*** safchain has quit IRC | 17:51 | |
*** bdemers has quit IRC | 17:51 | |
*** alejandrito has joined #openstack-neutron | 17:51 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: secgroup: Define hybrid plug in driver itself https://review.openstack.org/284620 | 17:51 |
*** abhiraut has joined #openstack-neutron | 17:52 | |
*** bdemers has joined #openstack-neutron | 17:53 | |
*** sleviim has joined #openstack-neutron | 17:54 | |
*** sleviim has left #openstack-neutron | 17:54 | |
*** salv-orl_ has joined #openstack-neutron | 17:54 | |
*** sbalukoff has joined #openstack-neutron | 17:54 | |
*** singhj has joined #openstack-neutron | 17:55 | |
*** davidsha has quit IRC | 17:55 | |
*** anilvenkata has joined #openstack-neutron | 17:55 | |
*** tpsilva has quit IRC | 17:57 | |
*** salv-orlando has quit IRC | 17:57 | |
anilvenkata | carl_baldwin, haleyb ping, need help | 17:58 |
pc_m | ihrachys: ping | 17:59 |
ihrachys | pc_m: howdy | 17:59 |
*** salv-orl_ has quit IRC | 17:59 | |
pc_m | ihrachys: Did the changes get upstreamed for supporting constraints on cover, venv, releasenotes? | 17:59 |
ihrachys | pc_m: nope we wait for some zuul patch | 17:59 |
pc_m | ihrachys: OK. Thanks. | 17:59 |
*** Sukhdev has quit IRC | 18:00 | |
*** jckasper has quit IRC | 18:01 | |
*** kawa2014 has quit IRC | 18:01 | |
amuller | jlibosva: hey, how did the security groups scenario tests pass on https://review.openstack.org/#/c/284620 patchset 3? | 18:02 |
amuller | jlibosva: any idea? | 18:02 |
amuller | jlibosva: oh nevermind | 18:03 |
amuller | jlibosva: yeah they should have passed | 18:03 |
amuller | ignore me :) | 18:03 |
*** alhu has joined #openstack-neutron | 18:04 | |
*** jpena is now known as jpena|off | 18:05 | |
*** emagana has quit IRC | 18:06 | |
jlibosva | amuller: 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 |
amuller | jlibosva: that sounds like it would suck hard to figure out | 18:07 |
*** Sukhdev has joined #openstack-neutron | 18:09 | |
*** emagana has joined #openstack-neutron | 18:09 | |
*** mvk has quit IRC | 18:09 | |
*** jlibosva has quit IRC | 18:10 | |
*** ihrachys has quit IRC | 18:10 | |
*** Sukhdev has quit IRC | 18:11 | |
*** testestsetse has joined #openstack-neutron | 18:12 | |
*** ildikov_ has joined #openstack-neutron | 18:12 | |
*** testestsetse has quit IRC | 18:12 | |
*** rmstar has quit IRC | 18:13 | |
*** emagana has quit IRC | 18:13 | |
*** abregman has joined #openstack-neutron | 18:14 | |
*** korzen has quit IRC | 18:15 | |
*** mhickey has quit IRC | 18:15 | |
*** doug-fish has quit IRC | 18:15 | |
*** rossella_ has quit IRC | 18:15 | |
*** slunkad has quit IRC | 18:15 | |
*** singhj has quit IRC | 18:15 | |
*** ildikov has joined #openstack-neutron | 18:16 | |
dougwig | amuller, anilvenkata: still around? | 18:16 |
amuller | dougwig: aye | 18:16 |
anilvenkata | amuller, yes Assaf | 18:16 |
*** singhj has joined #openstack-neutron | 18:16 | |
dougwig | amuller: 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 |
amuller | dougwig: I understand | 18:18 |
amuller | dougwig: Is there any way we can simplify the process so that it's easier to move stuff in there? | 18:18 |
dougwig | anilvenkata: 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 IRC | 18:19 | |
amuller | dougwig: so do everything in neutron in one patch, then move crap later? that would mean that sometimes 'later' never happens | 18:19 |
*** abhiraut has quit IRC | 18:19 | |
dougwig | amuller: 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-neutron | 18:19 | |
anilvenkata | dougwig, but I have to modify constants.py which is now in neutron.lib | 18:20 |
dougwig | amuller: 'later' only needs to happen if another project wants to import something. | 18:20 |
dougwig | anilvenkata: no, it is in both places. | 18:20 |
*** abhiraut has joined #openstack-neutron | 18:20 | |
*** mickeys has quit IRC | 18:20 | |
*** mickeys has joined #openstack-neutron | 18:21 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:21 | |
*** tiagogomes has quit IRC | 18:21 | |
amuller | dougwig: Tempest doesn't have to deal with this | 18:21 |
amuller | damn them | 18:22 |
anilvenkata | dougwig, 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.py | 18:22 |
*** sleviim has joined #openstack-neutron | 18:22 | |
*** sleviim has left #openstack-neutron | 18:22 | |
dougwig | amuller: tempest doesn't have to be both a released server and a lib at the same time. this is decomp fallout, really. | 18:22 |
amuller | dougwig: yep | 18:22 |
anilvenkata | dougwig, what will be your suggestion? redefine ROUTER_INTERFACE_OWNERS in neutron/common/constants.py ? | 18:23 |
*** fzdarsky__ is now known as fzdarsky|afk | 18:23 | |
dougwig | anilvenkata: 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-neutron | 18:23 | |
dougwig | anilvenkata: or if it's trivial, ping me or armax and see if we're close to a release. | 18:23 |
dougwig | anilvenkata: 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-neutron | 18:25 | |
amuller | anilvenkata: overriding sounds ok to me | 18:25 |
*** lucasagomes is now known as lucas-dinner | 18:25 | |
anilvenkata | dougwig, ok, I will override that for now | 18:25 |
jguy_ | Anyone worked with the l2gateway-agent? | 18:25 |
*** ivar-lazzaro has quit IRC | 18:25 | |
anilvenkata | dougwig, thanks dougwig | 18:26 |
*** Marga_ has joined #openstack-neutron | 18:27 | |
*** Marga_ has quit IRC | 18:27 | |
*** doug-fish has joined #openstack-neutron | 18:27 | |
*** dslevin has joined #openstack-neutron | 18:27 | |
*** Marga_ has joined #openstack-neutron | 18:27 | |
dougwig | anilvenkata, 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-neutron | 18:28 | |
*** Marga__ has quit IRC | 18:28 | |
*** Marga__ has joined #openstack-neutron | 18:29 | |
*** Marga_ has quit IRC | 18:29 | |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 18:32 |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Avoids logging error on OVS agent start. https://review.openstack.org/281498 | 18:33 |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Add IPv6 Prefix Delegation support for DVR https://review.openstack.org/277657 | 18:33 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 18:34 |
*** madhu_ak has joined #openstack-neutron | 18:35 | |
*** john-davidge has joined #openstack-neutron | 18:35 | |
*** madhu_ak has quit IRC | 18:36 | |
*** s3wong has joined #openstack-neutron | 18:37 | |
*** emagana has joined #openstack-neutron | 18:38 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 18:38 |
*** singhj has quit IRC | 18:38 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:39 | |
*** madhu_ak has joined #openstack-neutron | 18:40 | |
*** yamahata has quit IRC | 18:40 | |
*** yamahata has joined #openstack-neutron | 18:40 | |
*** singhj has joined #openstack-neutron | 18:41 | |
*** bjornar has joined #openstack-neutron | 18:44 | |
*** abhiraut has quit IRC | 18:46 | |
*** rpothier has joined #openstack-neutron | 18:49 | |
*** abhiraut has joined #openstack-neutron | 18:51 | |
ajo | kevinbenton, ping https://review.openstack.org/#/c/243994/18//COMMIT_MSG | 18:56 |
ajo | :) | 18:56 |
ajo | shz, ^ | 18:56 |
*** akshai has quit IRC | 18:57 | |
*** gangil has quit IRC | 18:58 | |
*** gangil has joined #openstack-neutron | 18:58 | |
*** gangil has quit IRC | 18:58 | |
*** gangil has joined #openstack-neutron | 18:58 | |
*** akshai has joined #openstack-neutron | 18:59 | |
*** emagana has quit IRC | 19:00 | |
*** abhiraut has quit IRC | 19:01 | |
openstackgerrit | Dustin Lundquist proposed openstack/neutron: devref doc config option separation https://review.openstack.org/275381 | 19:02 |
*** numan_ has joined #openstack-neutron | 19:03 | |
openstackgerrit | venkata anil proposed openstack/neutron: use separate device owner for HA router interface https://review.openstack.org/291651 | 19:06 |
*** atuvenie has quit IRC | 19:11 | |
*** mvk has joined #openstack-neutron | 19:12 | |
*** puck has quit IRC | 19:15 | |
*** puck has joined #openstack-neutron | 19:15 | |
*** jhershbe has quit IRC | 19:15 | |
*** doug-fish has quit IRC | 19:17 | |
*** doug-fish has joined #openstack-neutron | 19:17 | |
*** enriquetaso has quit IRC | 19:18 | |
haleyb | anilvenkata: pong, but maybe your question was already answered? | 19:19 |
anilvenkata | haleyb, Thanks Brian, not yet :) I was asking about PD fix | 19:19 |
anilvenkata | haleyb, 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-neutron | 19:20 | |
anilvenkata | haleyb, so wanted to check with you and carl_baldwin, about Armando's comment on patchset 17 ? | 19:21 |
*** doug-fish has quit IRC | 19:21 | |
*** jckasper has joined #openstack-neutron | 19:21 | |
*** rickyrem has joined #openstack-neutron | 19:22 | |
*** claudiub|2 has joined #openstack-neutron | 19:22 | |
sc68cal | anilvenkata: It appears we need tempest scenario testing for prefix delegation. | 19:24 |
sc68cal | since bugs like this slip in | 19:24 |
sc68cal | or at least, functional tests? | 19:24 |
haleyb | anilvenkata: i'm looking, but Carl is out this week | 19:24 |
anilvenkata | haleyb, sc68cal ok | 19:25 |
*** doug-fis_ has quit IRC | 19:25 | |
anilvenkata | haleyb, sc68cal thanks Brian and Sean | 19:25 |
haleyb | sc68cal: yes, there are no tests for PD that i could find | 19:25 |
*** claudiub has quit IRC | 19:25 | |
sc68cal | I am not surprised. | 19:25 |
*** tflynn has joined #openstack-neutron | 19:25 | |
*** jckasper has quit IRC | 19:26 | |
haleyb | we 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 elsewhere | 19:26 |
*** singhj has quit IRC | 19:27 | |
*** tflynn has quit IRC | 19:27 | |
* sc68cal coughs | 19:27 | |
anilvenkata | haleyb, may fullstack test | 19:27 |
sc68cal | devstack plugin ;) | 19:27 |
*** rickyrem has quit IRC | 19:27 | |
*** tflynn has joined #openstack-neutron | 19:27 | |
*** rickyrem has joined #openstack-neutron | 19:28 | |
haleyb | sc68cal: 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 that | 19:29 |
*** jckasper has joined #openstack-neutron | 19:29 | |
sc68cal | haleyb: right. My thought is, generate a /48 and feed it into the pd server that the devstack plugin starts | 19:30 |
sc68cal | maybe do a bit of ovs wiring or netns to hook it into neutron | 19:30 |
*** jckasper has quit IRC | 19:31 | |
*** jckasper has joined #openstack-neutron | 19:31 | |
amuller | haleyb: functional tests for the agent parts would be a good start IMO | 19:31 |
haleyb | so it looks off-link, then just swallows packets | 19:31 |
amuller | haleyb: if you start from the top and abandon the foundations the building eventually crumbles and falls ;-) | 19:31 |
*** davideagnello has quit IRC | 19:32 | |
*** Sukhdev has joined #openstack-neutron | 19:32 | |
haleyb | but IPv6 has 20 years of building poor foundations :) | 19:32 |
*** gvrangan has quit IRC | 19:33 | |
haleyb | amuller: this actually sounds like good blueprint material | 19:34 |
amuller | haleyb: I don't think we've ever tracked the addition of new tests via blueprints | 19:34 |
amuller | haleyb: apart from new testing infrastructures | 19:34 |
anilvenkata | amuller, may be adding IPv6 support in fullstack tests? | 19:35 |
amuller | unless you're masochistic in some way and enjoy bureaucracy | 19:35 |
*** akshai has quit IRC | 19:35 | |
*** mickeys has quit IRC | 19:35 | |
haleyb | amuller: 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 |
amuller | anilvenkata: 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 IRC | 19:36 | |
anilvenkata | amuller, only for tenant networks | 19:36 |
amuller | anilvenkata: we don't have dhcp agent support right now, so some ipv6 address types would fail | 19:37 |
*** singhj has joined #openstack-neutron | 19:37 | |
amuller | other than that, I think slaac with radvd should work currently | 19:37 |
*** iyamahat has joined #openstack-neutron | 19:37 | |
anilvenkata | amuller, mostly I think they are using SLAAC or stateless | 19:37 |
*** yamahata has quit IRC | 19:38 | |
openstackgerrit | Merged openstack/neutron-lib: Remove unused pngmath Sphinx extension https://review.openstack.org/286212 | 19:38 |
*** akshai has joined #openstack-neutron | 19:40 | |
*** iyamahat has quit IRC | 19:42 | |
*** numan_ has quit IRC | 19:43 | |
*** ivar-lazzaro has quit IRC | 19:45 | |
*** abhiraut has joined #openstack-neutron | 19:47 | |
*** tflynn has quit IRC | 19:48 | |
*** skamithi has quit IRC | 19:48 | |
amuller | tidwellr: heya Ryan, I'm updating http://docs.openstack.org/developer/neutron/devref/testing_coverage.html | 19:50 |
amuller | looking at BGP testing | 19:50 |
*** fawadkhaliq has quit IRC | 19:52 | |
sc68cal | ah, another thing that needs test coverage ;) | 19:53 |
*** sambetts is now known as sambetts|afk | 19:53 | |
amuller | sc68cal: the doc itself? =D | 19:56 |
sc68cal | :) - no I mean the bgp feature | 19:56 |
amuller | yeah | 19:56 |
amuller | I know it was squeezed in to Mitaka at the very end | 19:57 |
amuller | shame we're still doing testing after the fact | 19:57 |
sc68cal | yerp. | 19:57 |
anilvenkata | kevinbenton, as you know, https://review.openstack.org/#/c/291651/ , changes values of device_owner column, still migration script is needed? | 19:57 |
kevinbenton | anilvenkata: how does it change the existing records? | 19:58 |
*** ivase has joined #openstack-neutron | 19:58 | |
openstackgerrit | Lubosz Kosnik proposed openstack/neutron: Add support for Keepalived VRRP health check https://review.openstack.org/273546 | 19:59 |
anilvenkata | kevinbenton, yes correct, I am going through http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html | 19:59 |
openstackgerrit | Merged openstack/neutron-lib: Add in missing constants https://review.openstack.org/286564 | 20:00 |
openstackgerrit | Merged openstack/neutron-lib: Add Collector Agent type string https://review.openstack.org/279159 | 20:00 |
anilvenkata | kevinbenton, thanks Kevin | 20:00 |
*** claudiub|2 has quit IRC | 20:01 | |
kevinbenton | anilvenkata: np | 20:01 |
*** mickeys has joined #openstack-neutron | 20:01 | |
*** skamithi has joined #openstack-neutron | 20:01 | |
*** asingh has joined #openstack-neutron | 20:01 | |
*** skamithi has quit IRC | 20:02 | |
haleyb | anilvenkata: 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 |
anilvenkata | checking | 20:02 |
*** ig0r_ has joined #openstack-neutron | 20:03 | |
anilvenkata | haleyb, it is still needed | 20:04 |
haleyb | port has multiple fixed_ips | 20:04 |
anilvenkata | haleyb, get_ports is returing a port | 20:04 |
anilvenkata | haleyb, yes, it will have other fixed_ip | 20:04 |
haleyb | yeah, i thought of that right after i asked... | 20:05 |
anilvenkata | haleyb, :) | 20:05 |
*** gvrangan has joined #openstack-neutron | 20:06 | |
*** ig0r__ has quit IRC | 20:07 | |
*** tflynn has joined #openstack-neutron | 20:07 | |
openstackgerrit | Sean M. Collins proposed openstack/neutron: Devref: Remove upgrade testing from missing tests https://review.openstack.org/292577 | 20:07 |
*** salv-orlando has joined #openstack-neutron | 20:08 | |
sc68cal | amuller: ^ is what happens when you link things and I read them :) | 20:08 |
*** rtheis has quit IRC | 20:08 | |
*** rtheis_ has joined #openstack-neutron | 20:08 | |
amuller | sc68cal: I was about to address that in my update | 20:08 |
sc68cal | :) | 20:08 |
*** tflynn has quit IRC | 20:08 | |
amuller | sc68cal: I was going to modify that bullet point to say that I want dhcp/l3 also covered | 20:08 |
*** iyamahat has joined #openstack-neutron | 20:08 | |
amuller | sc68cal: currently only the OVS agent is left on the previous version | 20:09 |
sc68cal | ah good point. | 20:09 |
*** tflynn has joined #openstack-neutron | 20:09 | |
amuller | sc68cal: we need to modify devstack-gate to install those 2 agents on the secondary node, not the primary node | 20:09 |
*** tonytan4ever has quit IRC | 20:09 | |
amuller | sc68cal: to be fair I never communicated my ask to anyone | 20:09 |
sc68cal | amuller: agreed. Also there are OVS agents on the primary node (which is upgraded) and the secondary node | 20:10 |
amuller | sc68cal: remind me is there nova-compute on the primary node also? | 20:10 |
sc68cal | I'm not 100% sure | 20:10 |
sc68cal | maybe? | 20:10 |
amuller | sc68cal: yeah =p | 20:10 |
amuller | sc68cal: if not, we can remove the ovs agent from the primary node | 20:10 |
amuller | sc68cal: but I don't think it matters? we can keep nova-compute and ovs-agent on the primary node, just have both old and new | 20:11 |
sc68cal | my concern about old and new is then we're at the mercy of the scheduler | 20:11 |
sc68cal | to make sure we don't actually break things in upgrade | 20:11 |
amuller | right | 20:11 |
amuller | if we run enough tests it shouldn't be a problem | 20:11 |
sc68cal | fair point. | 20:11 |
amuller | probability of *every* test executing only the new node is low, unless the nodes are not equal | 20:12 |
amuller | sc68cal: I don't know we'd have nova compute on the primary node though? | 20:12 |
amuller | why* | 20:12 |
sc68cal | probably test multi_host | 20:12 |
*** MVenesio has quit IRC | 20:12 | |
sc68cal | dunno | 20:13 |
*** emagana has joined #openstack-neutron | 20:14 | |
*** ociuhandu has quit IRC | 20:15 | |
*** sayalilunkad has quit IRC | 20:16 | |
*** abhiraut has quit IRC | 20:17 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Add IPAllocation object to session info to stop GC https://review.openstack.org/291795 | 20:18 |
*** claudiub has joined #openstack-neutron | 20:18 | |
*** yfried has joined #openstack-neutron | 20:19 | |
*** davideagnello has joined #openstack-neutron | 20:20 | |
*** singhj has quit IRC | 20:20 | |
*** abhiraut has joined #openstack-neutron | 20:21 | |
*** singhj has joined #openstack-neutron | 20:22 | |
*** ankurgupta1 has joined #openstack-neutron | 20:23 | |
*** sayalilunkad has joined #openstack-neutron | 20:26 | |
*** anilvenkata has quit IRC | 20:28 | |
*** gvrangan has quit IRC | 20:30 | |
*** singhj has quit IRC | 20:31 | |
*** enriquetaso has joined #openstack-neutron | 20:32 | |
*** edmondsw has quit IRC | 20:35 | |
*** gvrangan has joined #openstack-neutron | 20:36 | |
*** dims_ has joined #openstack-neutron | 20:36 | |
*** ijw has joined #openstack-neutron | 20:37 | |
*** anshul has joined #openstack-neutron | 20:37 | |
*** dims has quit IRC | 20:38 | |
*** kbringard1 has quit IRC | 20:41 | |
*** kbringard has joined #openstack-neutron | 20:41 | |
*** iyamahat has quit IRC | 20:41 | |
*** erlarese has quit IRC | 20:44 | |
*** BigWillie has quit IRC | 20:46 | |
*** ijw has quit IRC | 20:50 | |
*** gvrangan has quit IRC | 20:50 | |
*** ivar-lazzaro has joined #openstack-neutron | 20:50 | |
*** singhj has joined #openstack-neutron | 20:54 | |
*** banix has quit IRC | 20:55 | |
*** bapalm has quit IRC | 20:55 | |
*** Sukhdev has quit IRC | 20:57 | |
*** bapalm has joined #openstack-neutron | 20:58 | |
*** singhj has quit IRC | 20:58 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Update testing coverage document https://review.openstack.org/292607 | 20:58 |
*** singhj has joined #openstack-neutron | 21:00 | |
*** alhu has quit IRC | 21:00 | |
*** alhu has joined #openstack-neutron | 21:01 | |
*** eddima has joined #openstack-neutron | 21:03 | |
*** safchain has joined #openstack-neutron | 21:04 | |
*** eddima has quit IRC | 21:04 | |
*** singhj has quit IRC | 21:04 | |
*** yfried has quit IRC | 21:04 | |
amuller | oops, looks like fullstack tests use 'sudo' as a root_helper | 21:04 |
amuller | time to fix that... | 21:04 |
*** john-davidge has quit IRC | 21:05 | |
tidwellr | amuller: re test coverage, what's missing? | 21:08 |
amuller | tidwellr: need a row for BGP in http://docs.openstack.org/developer/neutron/devref/testing_coverage.html | 21:09 |
amuller | tidwellr: The goal is to document missing coverage where it's relevant | 21:09 |
tidwellr | amuller: ok, got it | 21:09 |
amuller | tidwellr: For example I believe it's relevant to have functional tests for the BGP agent, like we do for the L3/OVS/DHCP agents | 21:09 |
amuller | tidwellr: Another thing to consider is integration testing | 21:09 |
tidwellr | amuller: we have some functional tests, I have not looked that closely at them | 21:11 |
tidwellr | amuller: I'll take a look at filling out the table | 21:11 |
amuller | tidwellr: I see functional tests for the BGP scheduler | 21:11 |
tidwellr | amuller: could be that we only have tests for that | 21:11 |
amuller | tidwellr: ok | 21:11 |
amuller | tidwellr: Thank you | 21:11 |
amuller | tidwellr: I've found that accountability is the first step towards upping our gaming when it comes to testing :) | 21:12 |
tidwellr | amuller: totally | 21:12 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: (WIP)DVR: Pings to floatingip returns with fixed-ip on same network https://review.openstack.org/289172 | 21:12 |
*** tidwellr has left #openstack-neutron | 21:13 | |
*** matrohon has joined #openstack-neutron | 21:14 | |
*** chlong has joined #openstack-neutron | 21:14 | |
*** john-davidge has joined #openstack-neutron | 21:16 | |
*** anshul has quit IRC | 21:16 | |
*** ijw has joined #openstack-neutron | 21:16 | |
*** fawadkhaliq has joined #openstack-neutron | 21:17 | |
*** ijw has quit IRC | 21:17 | |
*** rtheis_ has quit IRC | 21:19 | |
*** sdague has quit IRC | 21:19 | |
*** emagana has quit IRC | 21:22 | |
*** john-davidge has quit IRC | 21:23 | |
*** jaugustine has quit IRC | 21:24 | |
*** ociuhandu has joined #openstack-neutron | 21:25 | |
*** absubram has quit IRC | 21:26 | |
*** jguy_ has quit IRC | 21:26 | |
*** rpothier has quit IRC | 21:27 | |
*** thorst_ has quit IRC | 21:28 | |
*** thorst_ has joined #openstack-neutron | 21:31 | |
*** thorst_ has quit IRC | 21:35 | |
*** yamahata has joined #openstack-neutron | 21:36 | |
*** john-davidge has joined #openstack-neutron | 21:37 | |
*** baoli_ has quit IRC | 21:37 | |
*** jprovazn has quit IRC | 21:38 | |
*** reedip__ has quit IRC | 21:39 | |
*** alhu has quit IRC | 21:39 | |
*** panda has quit IRC | 21:40 | |
*** alhu has joined #openstack-neutron | 21:40 | |
*** tflynn has quit IRC | 21:40 | |
*** regXboi has quit IRC | 21:40 | |
*** panda has joined #openstack-neutron | 21:41 | |
*** banix has joined #openstack-neutron | 21:41 | |
*** tflynn has joined #openstack-neutron | 21:42 | |
*** neelashah has quit IRC | 21:43 | |
*** ivar-lazzaro has quit IRC | 21:45 | |
*** abregman has quit IRC | 21:46 | |
*** tflynn has quit IRC | 21:46 | |
*** tflynn has joined #openstack-neutron | 21:47 | |
*** vhoward has quit IRC | 21:47 | |
*** dims_ has quit IRC | 21:53 | |
*** adelia_ has joined #openstack-neutron | 21:53 | |
*** john-davidge has quit IRC | 21:56 | |
*** safchain has quit IRC | 21:56 | |
*** adelia has quit IRC | 21:57 | |
*** adelia_ has quit IRC | 21:57 | |
*** korzen has joined #openstack-neutron | 21:59 | |
*** rickyrem has quit IRC | 21:59 | |
*** bjornar has quit IRC | 21:59 | |
*** rickyrem has joined #openstack-neutron | 21:59 | |
openstackgerrit | Victor Morales proposed openstack/neutron: SubnetPool and SubnetPoolPrefix to OVO https://review.openstack.org/275789 | 22:04 |
*** neiljerram has quit IRC | 22:06 | |
*** azbiswas has quit IRC | 22:07 | |
*** azbiswas has joined #openstack-neutron | 22:08 | |
*** tidwellr has joined #openstack-neutron | 22:09 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Skip racy BGP tests and remove unnecessary use of LockFixture https://review.openstack.org/288921 | 22:10 |
openstackgerrit | Victor Morales proposed openstack/neutron: SubnetPool and SubnetPoolPrefix to OVO https://review.openstack.org/275789 | 22:11 |
*** azbiswas has quit IRC | 22:12 | |
*** MVenesio has joined #openstack-neutron | 22:13 | |
*** EkulTails has quit IRC | 22:14 | |
*** ivar-lazzaro has joined #openstack-neutron | 22:14 | |
*** fawadkhaliq has quit IRC | 22:15 | |
*** fawadkhaliq has joined #openstack-neutron | 22:16 | |
*** krtaylor has quit IRC | 22:17 | |
*** MVenesio has quit IRC | 22:18 | |
*** asingh has quit IRC | 22:22 | |
*** haplo37 has quit IRC | 22:23 | |
*** fawadkhaliq has quit IRC | 22:23 | |
*** fawadkhaliq has joined #openstack-neutron | 22:24 | |
*** SumitNaiksatam has quit IRC | 22:24 | |
*** yamahata has quit IRC | 22:26 | |
*** dims has joined #openstack-neutron | 22:26 | |
*** claudiub has quit IRC | 22:26 | |
*** asingh has joined #openstack-neutron | 22:28 | |
*** ircuser-1 has quit IRC | 22:28 | |
*** ircuser-1 has joined #openstack-neutron | 22:29 | |
*** abhiraut has quit IRC | 22:29 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Update test coverage devref with BGP information https://review.openstack.org/292627 | 22:29 |
*** abhiraut has joined #openstack-neutron | 22:29 | |
*** mickeys has quit IRC | 22:30 | |
tidwellr | amuller: ^^^^ | 22:30 |
tidwellr | amuller: took a stab at the BGP coverage information | 22:30 |
*** reedip__ has joined #openstack-neutron | 22:30 | |
*** korzen has quit IRC | 22:32 | |
*** abhiraut has quit IRC | 22:34 | |
*** ivar-lazzaro has quit IRC | 22:35 | |
*** krtaylor has joined #openstack-neutron | 22:35 | |
*** matrohon has quit IRC | 22:37 | |
*** gvrangan has joined #openstack-neutron | 22:38 | |
*** rickyrem has quit IRC | 22:38 | |
*** iyamahat has joined #openstack-neutron | 22:39 | |
*** ivar-lazzaro has joined #openstack-neutron | 22:39 | |
*** markvoelker has quit IRC | 22:39 | |
*** lykinsbd has joined #openstack-neutron | 22:40 | |
*** ivase has quit IRC | 22:40 | |
*** jguy_ has joined #openstack-neutron | 22:40 | |
amuller | tidwellr: thanks, commented | 22:41 |
*** alejandrito has quit IRC | 22:42 | |
*** SumitNaiksatam has joined #openstack-neutron | 22:42 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: WIP: Switch fullstack tests to use rootwrap https://review.openstack.org/292632 | 22:42 |
tidwellr | amuller: 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 announcements | 22:43 |
amuller | tidwellr: yeah, I'm not familiar with the details, but do we have access to a process that can receive BGP advertisements? | 22:43 |
amuller | tidwellr: then have the test start it up? | 22:44 |
tidwellr | amuller: we have 2 options: 1. spin up bird/quagga 2. build a quick and dirty ryu speaker to peer with | 22:44 |
tidwellr | amuller: at least that's what I'm coming up with at the moment | 22:45 |
tidwellr | amuller: maybe you've got some trick up your sleeve | 22:45 |
amuller | tidwellr: I'm not familiar with linux BGP tools =/ | 22:45 |
*** iyamahat has quit IRC | 22:46 | |
tidwellr | amuller: and I'm new to writing functional test jobs :) | 22:46 |
tidwellr | amuller: I'll spend some time this week on this | 22:46 |
amuller | tidwellr: do you use quagga/bird at all in the current code base, or do you only use ryu? | 22:47 |
*** lykinsbd has quit IRC | 22:47 | |
*** krtaylor has quit IRC | 22:47 | |
*** mvk_ has joined #openstack-neutron | 22:48 | |
tidwellr | amuller: the reference impl for the BGP driver uses Ryu | 22:48 |
amuller | tidwellr: ok | 22:49 |
amuller | tidwellr: does Ryu also support a listener? | 22:49 |
amuller | tidwellr: that you could send routes to? | 22:49 |
tidwellr | amuller: I'm thinking it would be pretty easy to craft a simple BGP process with Ryu to peer with for tests | 22:49 |
amuller | tidwellr: ok | 22:49 |
amuller | tidwellr: sounds reasonable | 22:49 |
tidwellr | amuller: yes, Ryu is a fully featured BGP impl | 22:49 |
amuller | tidwellr: (just curious), the Ryu implementation is self-contained, it doesn't use quagga or something? | 22:50 |
*** dims has quit IRC | 22:50 | |
tidwellr | amuller: yes, it is self-contained which is what made it attractive as a reference impl for BGP | 22:50 |
amuller | tidwellr: it must be a lot of code to implement BGP =D | 22:50 |
tidwellr | amuller: I was surprised how little code it actually is :) | 22:51 |
amuller | tidwellr: 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 |
amuller | tidwellr: 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 IRC | 22:52 | |
amuller | tidwellr: then the router is deleted, and assertions are made again (was the namespace deleted?) | 22:52 |
*** mickeys has joined #openstack-neutron | 22:52 | |
amuller | tidwellr: it would be reasonable to follow the same model with the BGP agent | 22:52 |
amuller | tidwellr: so any RPC interactions with the server are faked | 22:53 |
amuller | tidwellr: and ideally that's the only thing you fake/mock | 22:53 |
tidwellr | amuller: sounds reasonable | 22:54 |
tidwellr | amuller: thanks for the pointer, I'll take a look and ping you as I go along :) | 22:54 |
*** tidwellr has left #openstack-neutron | 22:56 | |
*** yamamoto_ has joined #openstack-neutron | 22:56 | |
*** yamahata has joined #openstack-neutron | 22:56 | |
*** mlavalle has quit IRC | 22:57 | |
*** vhosakot has quit IRC | 22:57 | |
*** fangzhen has joined #openstack-neutron | 22:59 | |
*** yamamoto_ has quit IRC | 23:00 | |
*** krtaylor has joined #openstack-neutron | 23:00 | |
*** amuller has quit IRC | 23:02 | |
*** vhosakot has joined #openstack-neutron | 23:02 | |
*** pradk has quit IRC | 23:02 | |
*** fangzhen has quit IRC | 23:08 | |
openstackgerrit | Jamie Lennox proposed openstack/neutron: Return oslo_config Opts to config generator https://review.openstack.org/292640 | 23:09 |
*** Leo_ has quit IRC | 23:10 | |
openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Add db migrations test framework with data https://review.openstack.org/124946 | 23:16 |
*** asingh has quit IRC | 23:17 | |
*** asingh has joined #openstack-neutron | 23:18 | |
*** armax has joined #openstack-neutron | 23:21 | |
*** dims has joined #openstack-neutron | 23:21 | |
*** gongysh has joined #openstack-neutron | 23:21 | |
*** daneyon has joined #openstack-neutron | 23:23 | |
*** akshai has quit IRC | 23:26 | |
*** dims has quit IRC | 23:27 | |
*** lnicolas has joined #openstack-neutron | 23:28 | |
*** Sukhdev has joined #openstack-neutron | 23:28 | |
*** tflynn has quit IRC | 23:30 | |
*** Sukhdev has quit IRC | 23:31 | |
*** mickeys has quit IRC | 23:32 | |
*** thorst_ has joined #openstack-neutron | 23:32 | |
manjeets | armax: ping | 23:37 |
armax | manjeets: ping | 23:37 |
armax | pong | 23:37 |
*** Sukhdev has joined #openstack-neutron | 23:38 | |
manjeets | i 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.rst | 23:38 |
manjeets | should it go in a separate file you mean or something else ? | 23:39 |
armax | manjeets: that’s what I thought, like the other features that are described in the manaual | 23:39 |
*** Sukhdev has quit IRC | 23:40 | |
*** asingh has quit IRC | 23:41 | |
*** Sukhdev has joined #openstack-neutron | 23:41 | |
*** asingh has joined #openstack-neutron | 23:41 | |
*** rickyrem has joined #openstack-neutron | 23:41 | |
*** vhosakot has quit IRC | 23:41 | |
*** djan has joined #openstack-neutron | 23:42 | |
*** asingh has quit IRC | 23:43 | |
*** tflynn has joined #openstack-neutron | 23:43 | |
*** asingh has joined #openstack-neutron | 23:43 | |
manjeets | in openstack-manuals/doc/admin-guide-cloud i don't see separate files for different features since this feature is admin only | 23:43 |
*** Sukhdev has quit IRC | 23:44 | |
*** Sukhdev_ has joined #openstack-neutron | 23:44 | |
manjeets | armax: 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.rst | 23:46 |
manjeets | and I don't see a separate file for router or specific feature ? | 23:46 |
armax | manjeets: hang on, let me check | 23:47 |
*** azbiswas has joined #openstack-neutron | 23:48 | |
*** mkrcmari__ has joined #openstack-neutron | 23:48 | |
*** Sukhdev_ has quit IRC | 23:49 | |
*** brad_behle has quit IRC | 23:49 | |
*** Fdaisuke has joined #openstack-neutron | 23:49 | |
*** davideagnello has quit IRC | 23:49 | |
*** Sukhdev has joined #openstack-neutron | 23:49 | |
*** banix has quit IRC | 23:50 | |
*** ZZelle has joined #openstack-neutron | 23:50 | |
armax | manjeets: I see two places potentially | 23:51 |
armax | now entry point could be: | 23:51 |
armax | https://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/networking-guide/source/intro-os-networking.rst | 23:51 |
*** mvk_ has quit IRC | 23:52 | |
armax | or | 23:52 |
armax | https://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/networking-guide/source/adv-config.rst | 23:52 |
*** lujinluo has joined #openstack-neutron | 23:52 | |
armax | I think it’s probably the former | 23:53 |
*** yamahata has quit IRC | 23:53 | |
manjeets | ok i added in admin-guide-cloud/ dir thought its admin only feature | 23:53 |
armax | oh I see, you’re right | 23:53 |
manjeets | so should i keep in same one and address other comments ? | 23:54 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR:Pro-active router creation with live migration https://review.openstack.org/275420 | 23:54 |
*** salv-orl_ has joined #openstack-neutron | 23:54 | |
*** haukebruno has quit IRC | 23:55 | |
*** SumitNaiksatam has quit IRC | 23:55 | |
armax | manjeets: I would still think that the content should stand out somehow rather than being where it is but it can always be moved later | 23:56 |
*** eil397 has quit IRC | 23:56 | |
manjeets | ok armax: thanks will do that | 23:56 |
*** rickyrem has quit IRC | 23:57 | |
*** haukebruno has joined #openstack-neutron | 23:57 | |
*** salv-orlando has quit IRC | 23:57 | |
*** yamamoto_ has joined #openstack-neutron | 23:57 | |
*** ajmiller has quit IRC | 23:57 | |
*** davideagnello has joined #openstack-neutron | 23:57 | |
*** asingh has quit IRC | 23:58 | |
*** davideagnello has quit IRC | 23:58 | |
*** davideagnello has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!