Monday, 2014-02-17

*** sweston has joined #openstack-neutron00:11
openstackgerritRonak Shah proposed a change to openstack/neutron: Nuage Networks Plugin  https://review.openstack.org/6727200:19
*** banix has joined #openstack-neutron00:20
*** matsuhashi has joined #openstack-neutron00:23
*** networkstatic has joined #openstack-neutron00:31
*** matsuhashi has quit IRC00:33
*** matsuhashi has joined #openstack-neutron00:34
*** Sukhdev has quit IRC00:35
*** matsuhashi has quit IRC00:38
*** xuhanp has joined #openstack-neutron00:39
*** yamahata has quit IRC00:41
*** alagalah has quit IRC00:42
*** yamahata has joined #openstack-neutron00:44
*** alagalah has joined #openstack-neutron00:50
*** sweston has quit IRC00:56
*** xuhanp has quit IRC00:56
*** rossigee has quit IRC00:57
*** rossigee has joined #openstack-neutron00:58
*** sweston has joined #openstack-neutron00:58
*** yamahata has quit IRC01:00
openstackgerritRonak Shah proposed a change to openstack/neutron: Nuage Networks Plugin  https://review.openstack.org/6727201:01
*** matsuhashi has joined #openstack-neutron01:04
*** WackoRobie has joined #openstack-neutron01:04
*** dkehn__ has joined #openstack-neutron01:04
*** clev has quit IRC01:04
*** clev has joined #openstack-neutron01:04
*** dkehn has quit IRC01:05
*** dkehn__ has quit IRC01:05
*** dkehn__ has joined #openstack-neutron01:05
*** dkehn_ has quit IRC01:06
*** dkehn has joined #openstack-neutron01:07
*** dkehn__ is now known as dkehn_01:09
*** sweston has quit IRC01:14
*** oda-g has joined #openstack-neutron01:27
*** morganfainberg_Z is now known as morganfainberg01:28
*** alagalah_ has joined #openstack-neutron01:40
*** baoli has joined #openstack-neutron01:47
*** yamahata__ has quit IRC01:49
*** yamahata has joined #openstack-neutron01:51
*** yamahata_ has joined #openstack-neutron01:54
*** WackoRobie has quit IRC01:54
*** WackoRobie has joined #openstack-neutron01:55
*** Gamekiller77 has quit IRC01:55
*** ramishra has joined #openstack-neutron01:55
*** ramishra has quit IRC01:55
*** ramishra has joined #openstack-neutron01:56
*** alagalah has quit IRC01:58
*** WackoRobie has quit IRC02:00
*** WackoRob_ has joined #openstack-neutron02:00
*** yamahata has quit IRC02:01
*** xianghui has joined #openstack-neutron02:02
*** JPtheJP has quit IRC02:07
*** JPtheJP has joined #openstack-neutron02:07
*** Jianyong has joined #openstack-neutron02:14
*** liudong has joined #openstack-neutron02:16
*** matsuhashi has quit IRC02:18
*** matsuhashi has joined #openstack-neutron02:26
*** baoli has quit IRC02:27
*** baoli has joined #openstack-neutron02:37
*** baoli has quit IRC02:41
*** dave_tucker is now known as dave_tucker_zzz02:42
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add OpenDaylight ML2 MechanismDriver  https://review.openstack.org/6977502:46
*** ianw has quit IRC02:55
*** pcm_ has quit IRC03:01
*** ianw has joined #openstack-neutron03:02
*** pcm_ has joined #openstack-neutron03:02
openstackgerritlifeless proposed a change to openstack/python-neutronclient: Don't reuse 'body' for response data  https://review.openstack.org/6911003:08
*** RajeshMohan has quit IRC03:08
*** RajeshMohan has joined #openstack-neutron03:09
openstackgerritYongli He proposed a change to openstack/neutron: Use six.StringIO/BytesIO instead of StringIO.StringIO  https://review.openstack.org/7390203:10
*** baoli has joined #openstack-neutron03:12
*** networkstatic has quit IRC03:13
*** networkstatic has joined #openstack-neutron03:13
*** otherwiseguy has quit IRC03:17
*** sweston has joined #openstack-neutron03:22
*** otherwiseguy has joined #openstack-neutron03:29
*** otherwiseguy has quit IRC03:30
openstackgerritberlin proposed a change to openstack/neutron: Fix duplicate name of NVP LBaaS objs not allowed on vShield Edge  https://review.openstack.org/7390503:34
*** dkehn__ has joined #openstack-neutron03:38
*** dkehn_ has quit IRC03:41
*** dkehn has quit IRC03:41
*** dkehn has joined #openstack-neutron03:42
*** matsuhashi has quit IRC03:48
*** baoli has quit IRC04:06
openstackgerritHemanth Ravi proposed a change to openstack/neutron: One Convergence Neutron Plugin Implementation  https://review.openstack.org/6924604:06
*** WackoRob_ has quit IRC04:06
*** WackoRobie has joined #openstack-neutron04:07
*** sweston has quit IRC04:08
openstackgerritshihanzhang proposed a change to openstack/neutron: Fix problem of deleting dhcp port  https://review.openstack.org/7380204:08
*** banix has quit IRC04:09
*** liudong has quit IRC04:10
*** WackoRobie has quit IRC04:12
*** alagalah_ has quit IRC04:12
*** alagalah has joined #openstack-neutron04:12
*** morganfainberg is now known as morganfainberg_Z04:13
*** liudong has joined #openstack-neutron04:13
*** dkehn__ is now known as dkehn_04:16
*** alagalah has quit IRC04:17
*** matsuhashi has joined #openstack-neutron04:32
*** amotoki has joined #openstack-neutron04:39
*** yamahata has joined #openstack-neutron04:42
*** liudong has quit IRC04:43
*** alagalah has joined #openstack-neutron04:43
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: options: consolidate options definitions  https://review.openstack.org/7352004:51
*** alagalah has quit IRC04:51
*** ramishra has quit IRC04:53
*** yfried has quit IRC04:55
*** chandan_kumar has joined #openstack-neutron05:02
openstackgerritstephen-ma proposed a change to openstack/neutron: Delete duplicate internal devices in router namespace  https://review.openstack.org/5795405:03
*** ramishra has joined #openstack-neutron05:04
*** irenab has joined #openstack-neutron05:13
irenabamotoki: hi, are you on IRC?05:15
amotokiirenab: hi05:16
irenabhi, may I ask question on the review comment you gave for https://review.openstack.org/#/c/53609/12?05:17
amotokiirenab: let me check a bit later. I am discussing with my collegue.05:18
irenabamotoki: sure, please let me know when05:19
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: l3-agent-consolidation: framework for consolidating l3 agents  https://review.openstack.org/5762705:23
*** gdubreui has quit IRC05:27
*** RajeshMohan has quit IRC05:29
*** RajeshMohan has joined #openstack-neutron05:30
*** yfried has joined #openstack-neutron05:40
*** pcm_ has quit IRC05:45
*** rotbeard has joined #openstack-neutron06:02
*** ramishra has quit IRC06:06
*** saju_m has joined #openstack-neutron06:15
*** iwamoto has joined #openstack-neutron06:20
amotokiirenab: thanks for waiting.06:23
irenabamotoki: Can you please take a look at the comment you posted?06:24
irenabI have a problem in reworking the code the way you suggested06:24
amotokiirenab: I checked the comments on patch set 12.06:25
irenabunit test fails, since in testing environment there is no zmq06:25
amotokiI see. Are you talking about Jenkins UT failure?06:26
irenabyes06:26
irenabthere was previous comments regarding zmq, and following Mark's suggestion, I use try_import06:27
openstackgerritJenkins proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/7393206:27
irenabin real deploymnet the external service that agent talks to requires zmq on its nstalation, so there is no way zmq is not installed06:27
amotokiThe current code (patch set 14) looks good to me too. I agree with mark.06:28
irenabI am wondering if I can change the code back as it was prior to your comment or you have some other suggestion?06:28
irenabThe current code does not pass unit tests, since zmq is on in testing required packages06:29
amotokihmm.. does faking zmq work?06:29
irenabdidn't find a way how to do it....06:30
amotokiAFAIK, ryu unit test deos the similar. Let me check for a while.06:30
irenabwill be great if you cangive me some reference to look at06:31
amotokiIt is one example: https://review.openstack.org/#/c/71791/7/neutron/tests/unit/ofagent/fake_oflib.py (though it is under review)06:31
amotokiIt replaces real ryu modules with mock.06:32
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations  https://review.openstack.org/5541106:34
irenabI looked at other examples with try_import, not sure its required to raise system exit, since it cannot be in real deployment06:35
irenablike hyperv agent: https://github.com/openstack/neutron/blob/master/neutron/plugins/hyperv/agent/utils.py06:36
openstackgerritIWAMOTO Toshihiro proposed a change to openstack/neutron: LBaaS: implements LVS driver  https://review.openstack.org/6740506:36
openstackgerritYongli He proposed a change to openstack/python-neutronclient: Use six.moves cStringIO instead of cStringIO  https://review.openstack.org/7393706:39
amotokiirenab: SystemExit is not necessarily required. It depends on you, but it looks better to raise SystemExit. I think it is clearer to operators.06:40
*** rohit404 has quit IRC06:41
amotokiirenab: i can take a look at the UT in your patch later today. It seems to requires some tries.06:41
*** jaypipes has quit IRC06:42
*** afazekas_ has quit IRC06:42
*** jaypipes has joined #openstack-neutron06:42
irenabamotoki: thanks, will try to look at the code you posted. But not sure its necessary to handle missing zmq due to depedency of additional service used by the agent06:43
*** rohit404 has joined #openstack-neutron06:44
amotokiirenab: your case looks more simple. Changing the test code to the following will address the unit test failures:07:00
amotoki        with mock.patch.object(utils, 'zmq'):07:00
amotoki            self.agent = eswitch_neutron_agent.MlnxEswitchNeutronAgent({})07:00
openstackgerritGary Duan proposed a change to openstack/neutron: FWaaS integration with service type framewrok  https://review.openstack.org/6069907:01
openstackgerritXu Han Peng proposed a change to openstack/neutron: Change firewall to INACTIVE when admin state down  https://review.openstack.org/7394407:02
amotokiirenab: this is the diff: http://paste.openstack.org/show/66369/   In this case, it is okay if property _conn in utils.EswitchUtils, so mocking mlnx.agent.utils.zmq is enough.07:02
irenabamotoki: taking a look07:02
irenabamotoki: will try to run it now07:04
amotokiirenab: in my env, agent ut passed after the change.07:04
*** oda-g has quit IRC07:07
*** ajo has joined #openstack-neutron07:08
irenabamotoki: still waiting to tests results. Once completed sucessfully will upload the fix. Many thanks for your help07:14
openstackgerritLee Li proposed a change to openstack/neutron: Remove dependent module py3kcompat  https://review.openstack.org/7376607:16
openstackgerritYongli He proposed a change to openstack/python-neutronclient: Use six.moves cStringIO instead of cStringIO  https://review.openstack.org/7393707:16
amotokiirenab: do you know how to run specific unit tests? I just ran only mlnx unit tests.07:24
irenabamotoki: I am running it with testr run mlnx07:25
irenabany other way?07:25
yfriedjianingy1afk: ping07:26
amotokiirenab: i ran "tox -e py27 neutron.tests.unit.mlnx.test_mlnx_neutron_agent". python module path can be used.07:27
*** afazekas_ has joined #openstack-neutron07:27
yfriedjianingy1afk: is this patch https://review.openstack.org/#/c/60512/ still alive? is this bp - https://blueprints.launchpad.net/neutron/+spec/router-port-forwarding going to be ready for I3?07:28
irenabamotoki: it take a lot of time, not finished yet...07:29
irenabamotoki: finally over and passed! Thank you very much, Uploading the fix07:30
amotokiirenab: good to know that. you are welcome.07:31
openstackgerritIWAMOTO Toshihiro proposed a change to openstack/neutron: LBaaS: implements LVS driver  https://review.openstack.org/6740507:32
openstackgerritBerezovsky Irena proposed a change to openstack/neutron: Add update from agent to plugin on device up  https://review.openstack.org/5360907:32
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations  https://review.openstack.org/5541107:58
*** bvandenh has joined #openstack-neutron08:03
*** slawek has joined #openstack-neutron08:04
*** mflobo has joined #openstack-neutron08:04
openstackgerritIWAMOTO Toshihiro proposed a change to openstack/neutron: LBaaS: add routed-service-insertion extension to the plugin  https://review.openstack.org/7396208:06
slawekhello08:07
slawekI have question about l3-agent: is it possible to have got multiple l3-agents, for example on every compute-node instead of one main on e.g. neutron server?08:08
slawekwill it work like "vrouters" on each node?08:08
*** rohit404 has quit IRC08:09
iwamotoyou might want to check 'distributed virtual router'08:10
*** jlibosva has joined #openstack-neutron08:10
slawek<iwamoto>: but where I should "check" it?08:12
iwamotoslawek: it's under development now, iirc.08:12
slawekI see only blueprints with such feature: https://blueprints.launchpad.net/neutron/+spec/neutron-ovs-dvr but it is probably not implemented yet08:12
slawekok, is it this blueprint which I send above?08:13
iwamotoslawek: yes08:14
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: NSX plugin: Introduce operational status for security groups  https://review.openstack.org/7169308:14
*** yamahata has quit IRC08:21
*** pasquier-s has joined #openstack-neutron08:22
*** alagalah has joined #openstack-neutron08:23
*** leseb has joined #openstack-neutron08:24
*** afazekas_ is now known as afazekas08:27
openstackgerritVincent Untz proposed a change to openstack/neutron: Reset the policy after loading extensions  https://review.openstack.org/7386608:28
*** alagalah has quit IRC08:28
*** saju_m has quit IRC08:31
*** jroovers has joined #openstack-neutron08:32
*** jroovers|afk has joined #openstack-neutron08:33
*** matrohon has joined #openstack-neutron08:34
*** salv-orlando_ has joined #openstack-neutron08:35
*** salv-orlando has quit IRC08:35
*** salv-orlando_ is now known as salv-orlando08:35
*** amuller has joined #openstack-neutron08:35
*** jroovers has quit IRC08:37
*** jgallard has joined #openstack-neutron08:40
*** iwamoto has quit IRC08:40
openstackgerritA change was merged to openstack/neutron: Remove DEBUG:....nsx_cluster:Attribute is empty or null  https://review.openstack.org/7374908:44
*** amuller has quit IRC08:46
*** ygbo has joined #openstack-neutron08:53
*** amuller has joined #openstack-neutron08:55
*** saju_m has joined #openstack-neutron08:58
openstackgerritMarcos Fermín Lobo proposed a change to openstack/python-neutronclient: Unexpected response in agent-list command  https://review.openstack.org/7036308:59
*** jistr has joined #openstack-neutron09:01
*** Jianyong has quit IRC09:02
*** chandan_kumar has quit IRC09:10
*** ajo has quit IRC09:16
*** rohit404 has joined #openstack-neutron09:19
*** chandan_kumar has joined #openstack-neutron09:20
*** shausy has joined #openstack-neutron09:33
*** zenoway has joined #openstack-neutron09:33
openstackgerritberlin proposed a change to openstack/neutron: Fix duplicate name of NVP LBaaS objs not allowed on vShield Edge  https://review.openstack.org/7390509:47
*** rossella-s has joined #openstack-neutron09:50
*** matsuhashi has quit IRC09:54
*** rossella-s has quit IRC09:54
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: Implement service vm framework: REST API and DB model part  https://review.openstack.org/5689209:55
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: Implement service vm framework: drivers for load balander (WIP)  https://review.openstack.org/7206809:55
*** networkstatic has quit IRC09:56
*** networkstatic has joined #openstack-neutron09:56
*** rossella-s has joined #openstack-neutron10:00
openstackgerritTrinath Somanchi proposed a change to openstack/neutron: implements:bp/fsl-sdn-os-mech-driver::fsl mech driver  https://review.openstack.org/7398510:00
*** leseb has quit IRC10:01
*** leseb has joined #openstack-neutron10:02
*** matsuhashi has joined #openstack-neutron10:02
*** jp_at_hp has joined #openstack-neutron10:15
*** matsuhashi has quit IRC10:17
*** matsuhashi has joined #openstack-neutron10:17
openstackgerritJun Jie Nan proposed a change to openstack/neutron: Check vxlan enablement via modinfo  https://review.openstack.org/6590210:18
openstackgerritTomoko Inoue proposed a change to openstack/neutron: Fix typo in service_drivers.ipsec.  https://review.openstack.org/7384810:20
*** ygbo has quit IRC10:21
*** ygbo has joined #openstack-neutron10:21
*** matsuhashi has quit IRC10:22
*** ajo has joined #openstack-neutron10:27
*** ramishra has joined #openstack-neutron10:38
yfriedjianingy1afk: ping10:38
openstackgerritTomoko Inoue proposed a change to openstack/neutron: Fix typo in service_drivers.ipsec  https://review.openstack.org/7384810:39
*** jp_at_hp has quit IRC10:39
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: tests/service: consolidate setUp/tearDown logic  https://review.openstack.org/7066810:42
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: Implement service vm framework: REST API and DB model part  https://review.openstack.org/5689210:42
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: Implement service vm framework: drivers for load balander (WIP)  https://review.openstack.org/7206810:42
*** vkozhukalov has joined #openstack-neutron10:46
*** matsuhashi has joined #openstack-neutron10:47
openstackgerritTrinath Somanchi proposed a change to openstack/neutron: implements:bp/fsl-sdn-os-mech-driver::fsl mech driver  https://review.openstack.org/7398510:51
*** yamahata has joined #openstack-neutron10:51
openstackgerritTrinath Somanchi proposed a change to openstack/neutron: implements:bp/fsl-sdn-os-mech-driver::fsl mech driver  https://review.openstack.org/7398510:55
*** networkstatic has quit IRC10:55
*** jroovers|afk is now known as jroovers10:58
*** yamahata has quit IRC11:00
*** xianghui has quit IRC11:00
shausyI have multinode openstack installation up and running with instances launched11:01
shausybut I have issue in ping /ssh to instance from outside11:01
shausywhereas i can able to ping the public IP from controller to instance11:02
*** matrohon has quit IRC11:06
*** ramishra has quit IRC11:08
*** ramishra has joined #openstack-neutron11:09
*** djoreilly has joined #openstack-neutron11:10
*** matrohon has joined #openstack-neutron11:11
*** leseb has quit IRC11:11
*** b3nt_pin has joined #openstack-neutron11:12
*** leseb has joined #openstack-neutron11:12
*** b3nt_pin is now known as beagles11:12
*** leseb has quit IRC11:16
*** leseb has joined #openstack-neutron11:18
*** pcm_ has joined #openstack-neutron11:18
*** pcm_ has quit IRC11:20
*** pcm_ has joined #openstack-neutron11:21
*** akamyshnikova has quit IRC11:21
*** akamyshnikova has joined #openstack-neutron11:24
*** akamyshnikova has joined #openstack-neutron11:24
*** leseb has quit IRC11:28
*** leseb has joined #openstack-neutron11:29
*** leseb has quit IRC11:33
openstackgerritOleg Bondarev proposed a change to openstack/neutron: Reschedule router if new external gateway is on other network  https://review.openstack.org/5288411:34
*** jp_at_hp has joined #openstack-neutron11:37
*** matsuhashi has quit IRC11:37
ihrachysmordred: ping11:38
*** jp_at_hp has quit IRC11:38
*** jgallard has quit IRC11:38
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations  https://review.openstack.org/5541111:42
*** jp_at_hp has joined #openstack-neutron11:48
*** shausy has quit IRC11:48
*** shausy has joined #openstack-neutron11:48
*** zenoway has quit IRC11:53
*** alagalah has joined #openstack-neutron12:00
*** bvandenh has quit IRC12:01
*** alagalah has quit IRC12:04
*** yfried has quit IRC12:10
*** alexpilotti has quit IRC12:13
ihrachysok, anyone able to release neutronclient? I've asked for it for a week already here, with no result... and now we start to hit issues in latest stable release due to lack of new client: https://review.openstack.org/#/c/73904/12:14
ihrachysI think we should release it asap, otherwise we leave neutron metadata agent broken for all users that are not running client from git master [aka everyone]12:15
ihrachysmarkmcclain said before that he's going to tag and release it, but it already took too much time, so if anyone wants to step in and unbreak neutron stable, please do so12:16
amullerihrachys: Perhaps an upstream email about this would be more effective / gain more traction12:18
ihrachysamuller: actually there is already one, from Robert Collins12:19
amullerihrachys: I see12:19
ihrachysbut yeah, I've just added +1 and ASAP to the thread12:22
*** rohit404 has quit IRC12:22
*** rohit404 has joined #openstack-neutron12:22
amullerihrachys: I haven't seen the thread but maybe you can explain what bugs are popping up, why they are urgent etc12:23
*** baoli_ has joined #openstack-neutron12:23
*** yfried has joined #openstack-neutron12:26
ihrachysamuller: neutron-metadata-agent assumes Client.get_auth_info() works ok if Client is instantiated with token= predefined, there was a bug with this scenario, it was fixed in master but was not released yet. Now neutron assumes it works ok and uses API, but since new client hasn't hit PyPI, and requirements.txt were not updated (for there is no new client), we get UnexpectedError raised in12:26
ihrachysmetadata-agent, and metadata requests fail.12:26
*** yfried has quit IRC12:26
amullerihrachys: I meant in the email thread =D12:26
ihrachysamuller: ah, ok :)12:28
*** leseb has joined #openstack-neutron12:29
*** yfried1 has joined #openstack-neutron12:30
*** leseb has quit IRC12:31
*** leseb has joined #openstack-neutron12:31
yfried1jianingy1afk: ping12:32
*** IanGovett has joined #openstack-neutron12:34
*** jgallard has joined #openstack-neutron12:34
ihrachysamotoki: we should release new client asap. do you know how to achieve this?12:34
amotokiihrachys: unfortunately I don't have a power to release the client.12:35
amotokiihrachys: I think markmcclain will be online in a couple of hours. Otherwise we can request it in today's neutron team meeting.12:36
pcm_anyone: I want to update the patch for a review someone else has out so that it is updated with latest. How do I do that?12:36
pcm_(essentially rebase with master latest)12:37
ihrachysamotoki: ok, let's wait a bit more.12:38
ihrachysamotoki: but yes, the issue should be raised if not fixed12:39
ihrachysamotoki: I guess gongysh can also release new clients (since he's a Team Lead), right?12:40
*** rohit404 has quit IRC12:41
amotokiihrachys: I am not sure. neutronclient is alwasy released by neutron ptl so far (danwent and mark).12:41
amotokiihrachys: most important point is who can upload python package to PyPI.12:42
ihrachysok, I see. then all we have is patience :)12:43
*** leseb has quit IRC12:46
*** baoli_ has quit IRC12:47
*** leseb has joined #openstack-neutron12:47
*** leseb has quit IRC12:47
*** baoli has joined #openstack-neutron12:47
*** leseb has joined #openstack-neutron12:47
openstackgerritOleg Bondarev proposed a change to openstack/neutron: Delete disassociated floating ips on external network deletion  https://review.openstack.org/5336412:51
openstackgerritOleg Bondarev proposed a change to openstack/neutron: LBaaS: check for associations before deleting health monitor  https://review.openstack.org/5382112:52
pcm_amotoki: ping12:53
amotokipcm_: pong12:53
pcm_amotoki: do you know how to update someone else's review?12:53
openstackgerritenikanorov proposed a change to openstack/neutron: Validate rule uuids provided for update_policy  https://review.openstack.org/7402312:54
pcm_amotoki: Nachi has a review out and I need to push mine out dependent on his, but his is way out of date to master.12:54
amotokipcm_: we can update other's review unless it is abandoned.12:55
pcm_amotoki: haven't been able to get ahold of Nachi. Wanted to update his review with a rebase to master.12:55
pcm_it's active12:55
pcm_how do I go about that?12:55
amotokipcm_: It is really same as usual.12:55
pcm_Do I pull a new branch off master, pull in his patches, and then commit/review and uses the same chagne ID?12:56
enikanorov_yes12:56
pcm_cool. Just didn't want to muck it up.12:56
enikanorov_in fact you just pull his change12:56
enikanorov_rebase it and push back to gerrit12:56
amotokithe easiest way is to checkout his patch and then rebase it to the latest master.12:57
pcm_thanks guys, I'll give it a whirl12:58
pcm_amotoki: I used the "checkout" from github, pasted that into my workspace. It say's I'm in detached head state. Is that right?12:59
openstackgerritOleg Bondarev proposed a change to openstack/neutron: LBaaS: check for associations before deleting health monitor  https://review.openstack.org/5382113:00
amotokipcm_: "checkout" I mean is the "checkout" command line of gerrit review.13:01
*** markvoelker has joined #openstack-neutron13:01
amotokipcm_: you must see "download" link in the gerrit reivew. After that you are in detached head,13:02
amotokipcm_:  so you need to create a branch with the same name (bug/xxxx or bp/xxxx).13:03
amotokiit is recommended but i think we can update the patch even in a detached branch.13:03
pcm_so I did the checkout (pasted the link from 'checkout' and am in detached head with branch refs/changes/27/41827/2013:04
*** shausy has quit IRC13:04
pcm_So do checkout -b pcm/bp/...13:05
*** shausy has joined #openstack-neutron13:05
pcm_or do I have to match the same branch name as what Nachi had?13:05
pcm_bp/vpn-service-types-integration13:05
openstackgerritenikanorov proposed a change to openstack/neutron: LBaaS: Fix discrepancy between status and admin_state_up  https://review.openstack.org/5503213:05
amotokipcm_: either can work.. I usually the same name.13:06
amotoki  /usually/usually use/13:06
openstackgerritAnn Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations  https://review.openstack.org/5541113:07
pcm_amotoki: ok did that. Will rebase now.13:07
*** yfried1 has quit IRC13:09
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: New SSL extension  https://review.openstack.org/7403113:11
*** rohit404 has joined #openstack-neutron13:13
*** dave_tucker_zzz is now known as dave_tucker13:19
*** bvandenh has joined #openstack-neutron13:19
*** alexpilotti has joined #openstack-neutron13:22
*** alagalah has joined #openstack-neutron13:23
*** yfried has joined #openstack-neutron13:25
openstackgerritA change was merged to openstack/neutron: Skip a test for nicira service plugin  https://review.openstack.org/7383213:28
pcm_amotoki: enikanorov_: Thanks! Seems to be working. Will push up for review once I run tox13:39
pcm_mestery: ping13:40
openstackgerritenikanorov proposed a change to openstack/neutron: Validate rule uuids provided for update_policy  https://review.openstack.org/7402313:40
*** salv-orlando has quit IRC13:43
*** yamahata has joined #openstack-neutron13:47
*** jecarey has quit IRC13:47
*** clev has quit IRC13:50
*** rohit404 has quit IRC13:52
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Provide agent status via status file (WIP)  https://review.openstack.org/7404514:00
*** ramishra_ has joined #openstack-neutron14:01
*** jecarey has joined #openstack-neutron14:01
*** ramishra has quit IRC14:02
*** ramishra_ has quit IRC14:06
*** WackoRobie has joined #openstack-neutron14:06
*** alagalah has quit IRC14:08
*** julim has joined #openstack-neutron14:09
openstackgerritWill proposed a change to openstack/neutron: Add z/VM mechanism driver to support IBM z/VM agent  https://review.openstack.org/7318714:15
*** ramishra has joined #openstack-neutron14:15
*** saju_m has quit IRC14:15
*** banix has joined #openstack-neutron14:16
*** irenab has quit IRC14:20
*** feleouet has joined #openstack-neutron14:21
*** leseb has quit IRC14:27
*** ramishra has quit IRC14:31
ajothere is some way to indicate a patchset is "WIP" (work in progress) that avoids jenkins tests?,14:34
ajoI have published a very preliminary patchset minutes ago, but I realized that jenkins is testing..14:34
*** dguitarbite has joined #openstack-neutron14:38
*** alagalah has joined #openstack-neutron14:38
openstackgerritenikanorov proposed a change to openstack/neutron: Validate rule uuids provided for update_policy  https://review.openstack.org/7402314:38
enikanorov_ajo: No14:39
enikanorov_ajo: it will still run the tests14:39
openstackgerritCedric Brandily proposed a change to openstack/neutron: Allow to delegate some tunnel provider attributes choice to neutron  https://review.openstack.org/7405514:39
ajoenikanorov_, thanks, wouldn't it make sense? (for example, you want to share the code with others, to get feedback, but's in an early state)14:40
ajobut on the other hand, if there is compute power available, the tests are good, even if early14:40
*** alagalah has quit IRC14:40
*** alagalah_ has joined #openstack-neutron14:40
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: New SSL extension  https://review.openstack.org/7403114:40
*** dkehn__ has joined #openstack-neutron14:41
*** yamahata has quit IRC14:43
*** dkehn has quit IRC14:43
*** yamahata has joined #openstack-neutron14:44
*** dkehn_ has quit IRC14:44
*** aymenfrikha has joined #openstack-neutron14:45
*** dkehn has joined #openstack-neutron14:46
*** afazekas has quit IRC14:46
*** clev has joined #openstack-neutron14:46
*** dkehn__ is now known as dkehn_14:46
*** aymenfrikha has quit IRC14:50
*** peristeri has joined #openstack-neutron14:52
*** dguitarbite has quit IRC14:54
*** dguitarbite has joined #openstack-neutron14:59
*** rotbeard has quit IRC15:00
*** skraynev is now known as skraynev_afk15:02
*** julim_ has joined #openstack-neutron15:02
*** skraynev_afk is now known as skraynev_afk_goi15:03
*** skraynev_afk_goi is now known as skraynev_afk15:03
*** aymenfrikha has joined #openstack-neutron15:04
*** ygbo has quit IRC15:04
*** otherwiseguy has joined #openstack-neutron15:04
*** julim has quit IRC15:05
*** chandan_kumar has quit IRC15:07
*** yfried has quit IRC15:07
*** rustlebee is now known as russellb15:08
*** rwsu has joined #openstack-neutron15:09
*** matrohon_ has joined #openstack-neutron15:12
*** dguitarbite has quit IRC15:13
*** bada has quit IRC15:13
*** matrohon_ has quit IRC15:14
*** bada has joined #openstack-neutron15:15
openstackgerritXiang Hui proposed a change to openstack/neutron: Add options for some commands executing run_vsctl  https://review.openstack.org/5895415:15
*** ygbo has joined #openstack-neutron15:16
*** dguitarbite has joined #openstack-neutron15:18
*** bada_ has joined #openstack-neutron15:18
*** leseb has joined #openstack-neutron15:19
*** yfried has joined #openstack-neutron15:19
*** bada has quit IRC15:20
*** alagalah_ has quit IRC15:20
*** yfried has quit IRC15:22
*** yfried has joined #openstack-neutron15:23
*** dguitarbite has quit IRC15:24
*** ygbo has quit IRC15:26
*** rohit404 has joined #openstack-neutron15:26
*** shausy has quit IRC15:26
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: New SSL extension  https://review.openstack.org/7403115:27
*** dguitarbite has joined #openstack-neutron15:29
openstackgerritXiang Hui proposed a change to openstack/neutron: Add options for some commands executing run_vsctl  https://review.openstack.org/5895415:30
*** amuller has quit IRC15:30
*** enikanorov has joined #openstack-neutron15:31
*** ramishra has joined #openstack-neutron15:32
*** ramishra has quit IRC15:37
*** ygbo has joined #openstack-neutron15:38
*** rkukura has quit IRC15:38
*** cjellick has joined #openstack-neutron15:42
*** cjellick has quit IRC15:43
*** ramishra has joined #openstack-neutron15:44
*** cjellick has joined #openstack-neutron15:47
*** beagles is now known as beagles_brb15:50
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add additional unit tests for the ML2 plugin  https://review.openstack.org/7407915:52
*** ramishra has quit IRC15:55
*** alagalah has joined #openstack-neutron16:00
*** salv-orlando has joined #openstack-neutron16:00
*** amotoki has quit IRC16:03
*** ttx has joined #openstack-neutron16:18
*** ttx has left #openstack-neutron16:18
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: New SSL extension  https://review.openstack.org/7403116:20
*** jgrimm has joined #openstack-neutron16:20
*** rohit404 has quit IRC16:21
*** rohit404 has joined #openstack-neutron16:21
*** bvandenh has quit IRC16:23
*** chandan_kumar has joined #openstack-neutron16:29
*** chandan_kumar has quit IRC16:29
*** chandan_kumar has joined #openstack-neutron16:30
*** skraynev_afk is now known as skraynev16:31
*** Gamekiller77 has joined #openstack-neutron16:32
*** yfried has quit IRC16:33
*** bvandenh has joined #openstack-neutron16:35
*** jistr has quit IRC16:35
*** rohit404_ has joined #openstack-neutron16:36
*** rohit404 has quit IRC16:36
*** rohit404_ is now known as rohit40416:36
*** alexpilotti has quit IRC16:38
*** alexpilotti has joined #openstack-neutron16:38
*** cjellick1 has joined #openstack-neutron16:38
*** cjellick1 has quit IRC16:39
*** rkukura has joined #openstack-neutron16:41
*** cjellick has quit IRC16:42
pcm_enikanorov_: ping16:45
*** chandan_kumar has quit IRC16:55
*** dave_tucker is now known as dave_tucker_zzz16:55
*** vkozhukalov has quit IRC17:00
*** jlibosva has quit IRC17:06
*** jaypipes has quit IRC17:07
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS integration with service type framework  https://review.openstack.org/4182717:08
openstackgerritCedric Brandily proposed a change to openstack/neutron: Allow to delegate some tunnel provider attributes choice to neutron  https://review.openstack.org/7405517:16
*** beagles_brb is now known as beagles17:16
*** sn6i23a has joined #openstack-neutron17:17
*** clev has quit IRC17:18
*** clev has joined #openstack-neutron17:19
*** ianw has quit IRC17:23
*** ianw has joined #openstack-neutron17:24
*** clev has quit IRC17:25
*** bjornar has joined #openstack-neutron17:26
*** clev has joined #openstack-neutron17:29
*** jgallard has quit IRC17:33
openstackgerritArvind Somya proposed a change to openstack/neutron: Cisco APIC ML2 mechanism driver, part 2  https://review.openstack.org/7337217:34
openstackgerritArvind Somya proposed a change to openstack/neutron: Cisco APIC ML2 mechanism driver, part 1  https://review.openstack.org/7335517:34
*** ygbo has quit IRC17:35
*** SumitNaiksatam has quit IRC17:37
*** RajeshMohan has quit IRC17:37
*** RajeshMohan has joined #openstack-neutron17:39
*** marun has joined #openstack-neutron17:41
enikanorov_pcm_: pong17:46
*** rossella-s has quit IRC17:46
*** leseb has quit IRC17:49
*** leseb has joined #openstack-neutron17:50
openstackgerritenikanorov proposed a change to openstack/neutron: Validate rule uuids provided for update_policy  https://review.openstack.org/7402317:50
*** clev has quit IRC17:50
*** jaypipes has joined #openstack-neutron17:53
*** leseb has quit IRC17:54
*** SumitNaiksatam has joined #openstack-neutron17:58
*** mlavalle has joined #openstack-neutron17:59
*** cjellick has joined #openstack-neutron18:08
*** jp_at_hp has quit IRC18:08
*** alagalah has quit IRC18:08
*** alagalah has joined #openstack-neutron18:09
pcm_Had some database related Qs. We talked before about mapping UUIDs and I created a table to hold those mappings. I have code that queries another table and then either queries this table or inserts into it. All this is done in a single session.18:12
pcm_enikanorov_: reping18:12
pcm_enikanorov_: Sorry I missed your pong.18:13
pcm_enikanorov_: Have some questions for you regarding database (related) issue I see.18:15
*** networkstatic has joined #openstack-neutron18:16
*** alagalah has quit IRC18:17
enikanorov_pcm_: what is it?18:17
*** alagalah has joined #openstack-neutron18:17
pcm_enikanorov_: Don't know if you saw my text a few lines up?18:18
* pcm_ I can retype18:18
enikanorov_yeah18:18
enikanorov_i mean i can see it :)18:18
pcm_enikanorov_: good. So I have it working and have UTs for some lower level functions and the top level function.18:19
enikanorov_ok18:19
pcm_enikanorov_: However, when I run the full tox suite or on the module, I get errors.18:19
enikanorov_pls paste them to paste18:20
pcm_enikanorov_: Do the tests run in parallel at the per test case level or class level?18:20
pcm_enikanorov_: Sure I can paste (working on pushing for review...but the testing and some other issues are preventing me right now)18:21
pcm_enikanorov_: http://paste.openstack.org/show/66573/18:22
enikanorov_looking18:23
*** gongysh has joined #openstack-neutron18:23
enikanorov_oh, i ean can you paste those errors?18:23
pcm_enikanorov_: sorry - it's huge... the tests for the mapping are in £TestCiscoIPsecDriverMapping18:24
*** SumitNaiksatam has quit IRC18:24
pcm_enikanorov_: I was wondering if I needed to put some of the test cases in a transaction18:24
enikanorov_ok, lets wait for jenkings to run uts18:25
*** gongysh has quit IRC18:29
*** suresh12 has joined #openstack-neutron18:30
*** alagalah has quit IRC18:31
*** alagalah has joined #openstack-neutron18:31
*** markmcclain has joined #openstack-neutron18:33
*** markmcclain has quit IRC18:33
*** carl_baldwin has joined #openstack-neutron18:33
*** dguitarbite has quit IRC18:37
enikanorov_jaypipes: ping18:39
jaypipesenikanorov: pong18:39
enikanorov_oh cool you're here18:39
*** markmcclain has joined #openstack-neutron18:40
*** suresh12_ has joined #openstack-neutron18:41
*** suresh12 has quit IRC18:42
*** morganfainberg_Z is now known as morganfainberg18:42
enikanorov_markmcclain: Hi. Will you have time today to continue the discussion on lb instance? I've got wiki page & pictures ready18:43
*** vkozhukalov has joined #openstack-neutron18:44
markmcclainenikanorov_: in a bit.. say 1h15m from now?18:44
enikanorov_markmcclain: good, works for me18:44
*** carl_baldwin has quit IRC18:51
*** mlavalle has quit IRC18:53
*** mlavalle has joined #openstack-neutron18:53
*** openstackgerrit has quit IRC18:53
*** openstackgerrit has joined #openstack-neutron18:54
*** clev has joined #openstack-neutron18:54
*** dave_tucker_zzz is now known as dave_tucker18:54
*** bjornar has quit IRC18:56
*** bjornar has joined #openstack-neutron18:58
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Provide agent status via status file (WIP)  https://review.openstack.org/7404518:58
*** Gamekiller77 has quit IRC18:59
*** Gamekiller77 has joined #openstack-neutron19:00
*** dave_tucker is now known as dave_tucker_zzz19:00
*** otherwiseguy has quit IRC19:03
*** jprovazn has joined #openstack-neutron19:04
*** dguitarbite has joined #openstack-neutron19:06
*** jog0 has left #openstack-neutron19:17
openstackgerritSandhya Dasu proposed a change to openstack/neutron: Implements a ML2 mechanism driver for Cisco UCS Manager.  https://review.openstack.org/7413419:24
*** mjbot has quit IRC19:25
*** mriedem has joined #openstack-neutron19:25
mriedemsalv-orlando: ping19:25
mriedemor anyone i guess :)19:26
mriedemnewb question19:26
mriedemif i want to filter the response when calling neutronclient.list_floatingips I just pass in kwarg filters=['x','y','z']?19:27
mriedemsomething like that?19:27
mriedemsay i want to filter list_floatingips by the 'id' field19:27
mriedemoops, fields=[], not filters19:27
mriedemnevermind, answered my own questoin19:30
enikanorov_filter={'id': [ids]}19:34
mriedemyeah, i meant fields19:34
enikanorov_fields parameter filters fields returned, not rows19:34
enikanorov_ok19:34
mriedemi want all floating_ips but want to keep the response small, so only fields=['id']19:35
enikanorov_aha, fileds is what you need19:35
mriedemyup, thanks19:35
mriedemtrying to resurrect this nova patch https://review.openstack.org/#/c/43822/19:35
mriedemit's only been restored like 3 or 4 times now...19:35
*** yfried has joined #openstack-neutron19:40
mtreinishmriedem: wow >30k changes since then and it was from the summer :)19:40
mriedemmtreinish: what are you doing lurking in here?19:40
mriedemmtreinish: that patch should also fix a tempest fail with neutron too...that's why i care19:41
*** armax has joined #openstack-neutron19:42
mtreinishmriedem: don't we just skip those tests?19:42
mriedemmtreinish: yeah, because of the bug :)19:43
*** Gamekiller77 has quit IRC19:43
mriedemnova isn't getting quota information from neutron19:43
*** zzelle has joined #openstack-neutron19:43
mriedemand as long as v3 is expiremental this is something we want in nova to interact with neutron19:44
mriedemi should say, as long as nova v2 is not deprecated19:44
mtreinishyeah, it's one of several pairity issues19:44
mtreinishwell v3 is going to have nova network stuff in it too19:44
*** otherwiseguy has joined #openstack-neutron19:44
mtreinishdoesn't that mean we'll have to worry about this for v3 too?19:44
mriedemshiiii, maybe?19:44
mriedemi'd have to ask cyeoh19:45
mriedemthe v3 api king19:45
*** Gamekiller77 has joined #openstack-neutron19:45
*** suresh12_ has quit IRC19:53
openstackgerritRajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS)  Service Insertion  https://review.openstack.org/6259919:53
*** suresh12 has joined #openstack-neutron19:53
*** alagalah has quit IRC19:57
*** alagalah has joined #openstack-neutron19:57
*** djoreilly has quit IRC19:58
*** suresh12 has quit IRC19:58
enikanorov_markmcclain: ping20:00
markmcclainenikanorov_: pong20:00
enikanorov_markmcclain: so here are pictures i've prepared: https://wiki.openstack.org/wiki/Neutron/LBaaS/LoadbalancerInstance/Discussion20:01
enikanorov_i'm mostly interested in discussing (2) and (3)20:01
* markmcclain looking20:01
enikanorov_as far as i understood your proposal, it looks like (3) to me20:01
enikanorov_please take a look20:01
*** alagalah has quit IRC20:02
pcm_mestery: ping20:02
mesterypcm_: pong hi there20:02
pcm_mestery: had a question on pushing for review.20:03
mesterypcm_: Shoot!20:03
pcm_mestery: did a git review -d of another review, created a branch, patched in my changes and tested it out20:04
pcm_mestery: did a git review, and it says that I have 2 outstanding commits (one is mine and one is the dependency).20:04
mesteryOK20:04
pcm_mestery: do I just say "YES" or did I miss some step?20:04
*** suresh12 has joined #openstack-neutron20:05
mesteryI think it's fine to say yes. Worst case, it will push a new version of hte dependent patch, which is mostly harmless as the author will remain the same.20:05
*** markwash has joined #openstack-neutron20:05
pcm_I just pushed the dependent commit earlier today (Nachi's review that I need).20:06
markmcclainenikanorov_: 3 is closer20:06
*** dave_tucker_zzz is now known as dave_tucker20:06
pcm_Concerned as my next step is to do the device driver which will depend on this one.20:06
enikanorov_markmcclain: so to understand your proposal better, could you provide the workflow for the following case:20:06
enikanorov_1st vip -> 1st pool, 2nd vip -> 1st pool and 2nd vip-> 2nd pool (via L7).20:06
enikanorov_e.g. 2 vips, 2 pools, all objects are on the same logical config.20:07
pcm_mestery: Just don't want to mess this up...20:07
mesterypcm_: I think you will be ok. I've done this before, and like I said, worst case is it pushes another rev out.20:07
mesterypcm_: You are doing great stuff here, no worries my friend. :)20:07
pcm_mestery: OK here goes.20:07
* pcm_ load explosions20:07
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS Service Driver for Cisco CSR  https://review.openstack.org/7414420:08
pcm_mestery: Looks like it went out.20:08
pcm_mestery: thanks20:09
mesterypcm_: Nice work!20:09
markmcclainenikanorov_: not sure I'm following 100%20:10
markmcclainonly the 2nd case uses L7 and the first does not?20:10
enikanorov_right, one vip handles only 1 pool, another vip handles 2 pools, one default and one via L7 rules20:10
enikanorov_the question is how to tell neutron that all these objects should be on one logical instance20:11
markmcclainenikanorov_: the user does not care20:12
enikanorov_the user may not care20:13
enikanorov_but we care :)20:13
ihrachysmarkmcclain: hi. we got the first Critical bug reported due to missing new client release: https://bugs.launchpad.net/neutron/+bug/128094120:13
markmcclainthe vip defines the endpoint using the terminology from #3 the listener defines the relationship to the pool20:13
markmcclainihrachys: sorry, I've have to rekey everything I use to sign releases20:14
enikanorov_#3 solves the workflow more or less, but it still makes user to care about logical config20:14
enikanorov_the root object is now VIP20:14
enikanorov_and to create the pool, you need to pass vip_id20:14
markmcclainnot seeing how the use cares about the logical config20:15
markmcclainI'd argue we could drop that semantic20:15
markmcclainand just create a pool20:15
enikanorov_ok, so that will be pure-db object at that point20:15
enikanorov_right?20:15
markmcclainyes20:15
enikanorov_ok, then we assign it to a listener via l7 rules20:16
enikanorov_and it get associated with existing lopgical config20:16
enikanorov_?20:16
markmcclainyes20:16
markmcclainso consider the case where vip1 and vip2 get assigned to two different backends20:17
markmcclainand but share the same pool20:17
enikanorov_that means two backends will share the same pool configuration20:18
markmcclainthe backend implementation has two choices… 1) reschedule of the vips onto other's instance or 2) duplicate the pools20:18
*** carl_baldwin has joined #openstack-neutron20:18
ihrachysmarkmcclain: any eta? :) I'm keen to update requirements.txt in global and neutron... ;)20:18
enikanorov_in fact i'd say that the ability to get them deployed to single backedn is useful20:18
markmcclainihrachys: as soon as I get my pub key published20:19
*** carl_baldwin has quit IRC20:19
*** skraynev is now known as skraynev_afk20:19
markmcclainenikanorov_: it is useful20:19
markmcclainbut from a user perspective that is an implementation detail20:19
enikanorov_markmcclain: yes, it should bnot be a requirement20:19
ihrachysmarkmcclain: is that on pypi side?20:19
enikanorov_markmcclain: so this way the problems are addressed, but the biggest issue is that it's complete redesign of object model and API20:20
markmcclainihrachys: I have push signed tags to kick off the publishing20:20
enikanorov_otherwise I would be happy to go with it20:20
*** dave_tucker is now known as dave_tucker_zzz20:20
markmcclainihrachys: to multiple places20:20
markmcclainenikanorov_: we are allowed to adapt the object model over time as the understand the problem space better20:21
markmcclainenikanorov_: remember the original object model was craved out over a week last Feb20:21
enikanorov_markmcclain: well... but the public API breaks too20:22
enikanorov_is it acceptable?20:22
*** irenab has joined #openstack-neutron20:23
enikanorov_i remember tempest folks gave me no-no for changing the api...20:23
markmcclainI still don't see how the public API breaks if we make the appropriate mappings from new to old20:23
markmcclainwithin a major version (v2 in our case) we have to maintain compatibility20:23
enikanorov_because the workflow for #3 is different then how we create configuration right now20:23
markmcclainit's different under the hood20:24
markmcclainbut from the API, I don't see anything that would prevent us having a proper compatibility layer20:24
enikanorov_ok, i'll think about it more. currently i'm not so sure we can maintain compatibility going with #320:25
enikanorov_but at least it provides the same model as an 'instance' approach20:25
enikanorov_which is good20:25
*** suresh12 has quit IRC20:26
markmcclainyeah even in #3 I still think we need to look at how we express the relationships of the pools20:27
enikanorov_markmcclain: btw, looks like neither libra nor aws don't have API flexible enough to support multiple vips and multiple pools at once20:27
markmcclainI'm concerned that in some cases we'll have to know to go through l7 mappings and in others we have to detect to skip them20:27
enikanorov_markmcclain: which cases ae you thinking about?20:28
mriedemarosen1: enikanorov: ping when you have a minute20:28
markmcclainenikanorov_: yeah supporting both increases complexity20:28
*** suresh12 has joined #openstack-neutron20:29
enikanorov_markmcclain: ..and looks like both aws and libra has the 'loadbalancer' notion, but it maps to our 'pool'20:29
enikanorov_so i really didn't have the concern of letting the user to care about logical config20:29
enikanorov_because they do care already, working with the pool, rather then with instance/vip we're trying to introduce20:30
enikanorov_so i'm just saying is this concern big enough to redefing the API and find a way to make it compatible20:31
markmcclainenikanorov_: they're probably only concerned with the pool now because that is the direction we steered them20:31
*** rohit404_ has joined #openstack-neutron20:31
*** rohit404 has quit IRC20:31
*** rohit404_ is now known as rohit40420:31
enikanorov_those who work with single pool will not change the workflow and will not notice the instance20:31
enikanorov_and for those who need more than 1 pool instance_id is what attaching new object to existing conf20:32
markmcclainnot sure I'm following the last statement20:33
*** jroovers|afk has joined #openstack-neutron20:33
*** clev has quit IRC20:33
enikanorov_i mean that for the exampel above with two pools20:33
enikanorov_you create the 1st one, the instance created for it automaticaly, and then you use instance_id to create 2nd pool20:34
enikanorov_so they will be colocated on the backend20:34
enikanorov_i'm not sure why do we need to isolate the user of such knowledge20:35
enikanorov_(colocation)20:35
*** jroovers has quit IRC20:35
enikanorov_user don't care about the backend, but they know how logical configuration envelops created objects20:37
enikanorov_and, say, they can migrate configuration from flavor to flavor, router, agent, etc20:37
ihrachysmarkmcclain: ok, thanks, keep me informed :) btw there is also 'Urgentpython-neutronclientrelease needed' thread in openstack-dev.20:39
*** itzikb has joined #openstack-neutron20:40
*** bada_ has quit IRC20:41
enikanorov_markmcclain: in #3 such logical instance is simply a VIP, and it is not what current VIP is. that's why i'm not sure how we can be bw compatible20:41
*** jlibosva has joined #openstack-neutron20:42
*** jecarey has quit IRC20:43
*** jecarey has joined #openstack-neutron20:44
itzikbHi, Can someone please review this patch https://review.openstack.org/#/c/66545/ ?20:46
markmcclainenikanorov_: sorry20:47
*** irenab has quit IRC20:47
enikanorov_markmcclain: ok, the meeting is soon. I'll think about how #3 could have compatible API with existing one. Currently it seems to me that it can not. I hope to continue discussion after I get more thoughts on that20:47
markmcclainenikanorov_: ok.. I'll read the scroll back and get caught up20:48
*** jecarey has quit IRC20:49
jlibosvamarkmcclain: hello20:50
*** sweston has joined #openstack-neutron20:50
swestonbeagles: ping20:51
beaglessweston: pong20:52
*** jecarey has joined #openstack-neutron20:52
*** gongysh has joined #openstack-neutron20:52
mriedemarosen1: shouldn't 'fields' here be specying a list? https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L81820:52
mriedem*specifying20:52
*** ajo has quit IRC20:52
swestonbeagles: hey there ;-).  I finally had some time this weekend to review the NovaNeutronGap document ... it was a rough week20:53
swestonbeagles:  if you have time to chat about it soon, I am available20:53
beaglessweston: okay. I should have a bit of time right after the neutron IRC meeting20:54
swestonbeagles: excellent!20:54
beagles34290420:54
beagleswhoops20:55
gongyshmriedem: hi, L818 will be transported into  networks.json?fields=xxx&router:external=true20:57
mriedemgongysh: but i thought fields was supposed to be a list?20:57
gongyshmriedem: if it is a list, it will be ?fields=xxx&fields=xxx2&router:external=true..20:58
mriedemwhere does the client handle that?20:58
openstackgerritHareesh Puthalath proposed a change to openstack/neutron: Configuration agent for Cisco Service VMs  https://review.openstack.org/7415220:58
gongyshmriedem: the server will received it as a list anyway.20:58
mriedemgongysh: ah i see, safe_encode_dict in the client21:00
mriedemhandles strings, lists and dicts in the kwargs passed in21:00
*** emagana has joined #openstack-neutron21:00
*** amotoki has joined #openstack-neutron21:00
gongyshmriedem: yes21:00
*** SumitNaiksatam has joined #openstack-neutron21:01
*** JPtheJP has quit IRC21:03
*** nati_ueno has joined #openstack-neutron21:03
*** jecarey_ has joined #openstack-neutron21:06
*** nati_ueno has quit IRC21:06
*** nati_ueno has joined #openstack-neutron21:06
*** jecarey has quit IRC21:08
openstackgerritPaul Michali proposed a change to openstack/neutron: VPNaaS Device Driver for Cisco CSR  https://review.openstack.org/7415621:13
*** ianw has quit IRC21:14
openstackgerritVincent Untz proposed a change to openstack/neutron: Reset the policy after loading extensions  https://review.openstack.org/7386621:14
*** ianw has joined #openstack-neutron21:15
*** afazekas has joined #openstack-neutron21:21
*** suresh12 has quit IRC21:22
annegentleemagana: I know you're in the neutron meeting now, but if you could look at https://review.openstack.org/#/c/72850/ for the Operations Guide it would be much appreciated!21:22
emaganaannegentle: I am on it!21:23
*** suresh12 has joined #openstack-neutron21:23
*** salv-orlando has quit IRC21:25
*** salv-orlando has joined #openstack-neutron21:25
*** JPtheJP has joined #openstack-neutron21:31
enikanorov_SumitNaiksatam: could you please take a look at  https://review.openstack.org/#/c/74023/ a fix to a samll issue i've found today, thanks.21:35
SumitNaiksatamsure21:36
*** dims has joined #openstack-neutron21:41
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Provide agent status via status file (WIP)  https://review.openstack.org/7404521:43
*** nati_ueno has quit IRC21:49
*** nati_ueno has joined #openstack-neutron21:49
*** vkozhukalov has quit IRC21:53
*** alagalah has joined #openstack-neutron21:53
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Poll the agent child processes status (WIP)  https://review.openstack.org/7417921:54
*** jprovazn has quit IRC21:55
*** devlaps1 has joined #openstack-neutron21:58
*** alagalah has quit IRC22:00
*** sputnik13 has joined #openstack-neutron22:00
*** WackoRobie has quit IRC22:01
*** WackoRobie has joined #openstack-neutron22:01
*** alagalah has joined #openstack-neutron22:02
*** nati_ueno has quit IRC22:04
pcm_nati_ueno: ping22:04
*** dguitarbite_ has joined #openstack-neutron22:04
*** dguitarbite has quit IRC22:04
swestonbeagles: give me 5 minutes?22:05
beaglessweston, sure22:05
*** WackoRobie has quit IRC22:06
swestonbeagles: thanks, trying to finish up a bug ;-)22:06
*** ecbanks has quit IRC22:06
*** gongysh has quit IRC22:06
jlibosvamarkmcclain: hi, do you have a minute?22:07
markmcclainjlibosva: I do22:07
markmcclainwhat's up?22:07
jlibosvamarkmcclain: wrt https://review.openstack.org/#/c/69433/3 and offline migration - I added condition for offline migration. Could you please take a look and consider your -2 or discuss other options how to avoid dropping non-existing table?22:08
markmcclainjlibosva: yes I'll take a 2nd look22:08
jlibosvamarkmcclain: thanks22:08
*** pcm_ has left #openstack-neutron22:09
*** armax has left #openstack-neutron22:10
*** jecarey_ has quit IRC22:11
*** bjornar has quit IRC22:12
swestonbeagles: ok, I am ready when you are.  Wanna discuss over PM?22:13
beaglessweston, probably best to discuss here in case others want to throw in22:14
*** leseb has joined #openstack-neutron22:15
swestonbeagles: sure, that works for me.  where would you like to start/22:15
sweston?22:15
*** ecbanks has joined #openstack-neutron22:15
*** clev has joined #openstack-neutron22:16
*** markmcclain has quit IRC22:16
*** mriedem has left #openstack-neutron22:17
beaglessweston, let's start with a working definition of parity: to date you could define parity as "whatever" is required to deprecate nova-network, eventually allowing it to be removed22:17
beaglessweston, there are multiple facets to that: functionality/features, stability/performance, quality of default implementation22:18
beaglesthese are very general categories that can get pretty big when you dig into them...22:19
*** julim_ has quit IRC22:19
beaglessweston: for Icehouse a lot of progress has been made in the 2nd and 3rd items and more to come22:19
*** jlibosva has quit IRC22:19
swestonbeagles: so, then this is a question of documenting the differences, or coming up with a plan to address them, or both?22:19
beaglessweston: an implicit aspect of both of these is tempest coverage etc. ... these are all being handled22:20
beaglessweston, so the first item is left22:20
beaglessweston, yeah... so22:20
beaglessweston, really when it comes down to broad strokes, it is possible to configure neutron to behave very much like a nova deployment... let's ignore the steps for a moment, but the underlying "guts" are there22:21
beaglessweston, except...22:21
*** hemanthravi has joined #openstack-neutron22:21
swestonbeagles: is there another document with nova parity identified issues?  I have an open bug for one of them, I remember anita was going to put together a roadmap in December but I did not follow up on that yet.22:22
beaglessweston, when you compare the accessibility and ease-of-use of using floating IPs between the two, neutron is pretty hard22:22
beaglessweston: which bug are you referring to?22:22
*** clev has quit IRC22:24
swestonbeagles: yes, but I do not think it will be difficult to change the default installation of devstack to create a provider network on the same range as the eth0 interface, for example22:24
swestonbeagles: I assume there have already been discussions about that, though? ;-)22:24
beaglessweston, while you are digging that up, the other remaining hunk is multi-host22:24
*** clev has joined #openstack-neutron22:24
beaglessweston, it has more to do with how they are connected22:24
beaglessweston, e.g. in nova, the public interface may be a physical interface on the host and the floating ips are aliased to that22:25
beaglessweston, so add a floating ip and off you go22:25
swestonbeagles: I've had this one for a while ... but no progress yet https://blueprints.launchpad.net/neutron/+spec/auto-associate-floating-ip22:26
beaglessweston, in neutron, you have to pick an approach of forwarding stuff into br-ex, set up some bridging so you can add a provider network interface to a physical interface, ya-da-ya-da22:26
beaglessweston, for folks who don't do networking it is a pretty difficult pill to swallow22:27
*** banix has quit IRC22:28
beaglessweston, it isn't the assignment of the floating IPs, but what you need to do with the system to get external access working. It seems like a trivial thing from a networking point of view, but if all you want is a private/developer/tiny-poc cloud it raises the bar22:28
swestonbeagles: so why not make that the default installation?  so i.e. the default network type would be a local provider network, bridged to the eth0 interface, using the ml2 openvswitch mechanism driver22:28
*** rohit404 has quit IRC22:28
beaglessweston, as soon as you bridge an eth, it loses its IP, etc. It requires some reconfig to get things working22:29
swestonbeagles: nova network is fairly simple, i do not see why we cannot do the same for neutron.  forgive me if I am asking "naive" questions22:29
*** itzikb has quit IRC22:30
rms_13SumitNaiksatam: Thanks for reviewing and approving https://review.openstack.org/#/c/67272/.22:30
beaglessweston, for example.. all of my dev machines treat eth0 as a bridge interface and I configure a br0 as the "main interface" that gets my host's IP... I add whatever I want to that to get permit access between neutron L3 and L2 and my physical (or virtual) networks.22:30
*** leseb has quit IRC22:31
beaglessweston, basically, nova-network doesn't require bridging your public interface so it just kinda works22:31
*** leseb has joined #openstack-neutron22:31
SumitNaiksatamrms_13: hi, happy to help, it still needs one more core reviewer to approve it22:31
beaglessweston, the interesting dilemmas you run into when looking at how to mimic this behavior is that it obviates the ability to use some neutron features (not a huge deal for a transitional deployment), but also in some cases it seems like "hacking up" neutron or doing things that are against the spirit of neutron in order to get that type of thing22:33
rms_13SumitNaiksatam: I understand. Probably markmcclain can take a look as he already did some initial digging22:33
beaglessweston, a similar set of issues arise when you start looking at multi-host. Nova-network multihost comes with some compromises that a similar implementation in neutron could avoid. There are discussions about creating similar functionality, however on occasion where similar compromises appear, they block further progress along those veins.22:36
beaglessweston, thinking of some examples...22:36
beaglessweston, as multi-host is basically a nova-network controller on each compute node that manages L3 and L2 for the instances on that node...22:37
beaglessweston, that implies multiple dhcp namespaces and dnsmasq instances... consuming an IP in that address space,22:38
beaglessweston, it also implies conditional or haphazard east-west routing...22:38
zzellehi22:38
beaglessweston, in the former, nova actually has a configurable option to workaround that and the east-west routing thing is a problem in nova-network too22:39
beaglessweston, so one approach to the multi-host issue would be bite the bullet and accept those issues...22:39
beaglessweston, another approach would be to forget it and do something else.22:40
zzelleone question, does someone know why veth are used to connect physnets and br-int instead of ovs patchs when using VLAN + OVS22:40
zzelle?22:40
beaglessweston, to date, I think we are doing the latter ;)22:40
beaglessweston, I'm thinking that multi-host is "too big a fish to swallow". There are guys working on DVR and it has the floor and may address some of the issues multi-host was meant to address.22:41
swestonbeagles: ok I follow you ... but, the goal is to make this technology easily consumable, right?  yes, there are some changes that have to happen with bridging networks and manipulating routing tables, but if it's an issue that is standing in the way of neutron adoption, I think it's a small sacrafice to make22:41
beaglessweston, I do not disagree...22:41
beaglessweston, so what would be cool if there were a minimal non-bizarro way to get this to work, even if it means saying "hey you can't do XYZ if you configure it like this"22:42
beaglessweston, e.g. pretty much either your L3 couldn't use namespaces and would have to play nice with the host, or there would need to be a "simple L3" router22:43
beaglessweston, some of the other folks may have some thoughts about whether that is a direction we want to go and/or how it might be done.22:44
swestonbeagles: yes, I agree, it would be totally possible to create a poc with devstack and neutron that would bind everything together ... of course it would be limited functionality, but everything out of the box that is set up by default makes that compromise.22:45
swestonbeagles: I think it would have to be a community decision, yes, and there would probably have to be frequent discussions about the best way to do it22:47
swestonbeagles: it is always going to be a tug-of-war between functionality and simplicity22:47
beaglessweston, sure.. and to be fair, as soon as you start looking at "real world" nova-networking deployments, things get more complicated pretty quickly. It's the very simple cases where nova and neutron have a more noticeable gap22:47
beaglessweston, yup22:47
*** oda-g has joined #openstack-neutron22:49
beaglessweston, another mondo-question that comes to mind is whether these things are reasonable priorities at the moment22:49
beaglessweston, ie. is it more reasonable to focus on performance, stability and scalability, quality, etc.22:50
swestonbeagles: so possibly the next step here, if everyone agrees that this is worth moving forward, is to set up a working group, with weekly meetings for direction and organization, leadership ... I think it is necessary in order to move neutron forward22:51
beaglessweston, bodies22:53
beaglessweston, yes22:53
swestonbeagles: yes, that is a big question, it is a lot of work, and not as cool as LBaaS or DVR, but I think this effort is everyone's responsibility.  this is a community and it is important to realize that if nova network never gets deprecated, then we will never have any of the cool things happening in the subgroups right now22:54
* beagles chuckles22:54
* sweston smiles22:54
beaglessweston, I'm chuckling because it sounds pretty familiar22:54
swestonbeagles: I think we all know it's true, though ;-)22:55
*** dave_tucker_zzz is now known as dave_tucker22:55
beaglessweston, well, that actually begs the question: does nova-network *not* being deprecate obviate the existence of all of the aforementioned cool things22:56
swestonbeagles: how many thousands of times have development projects gotten stomped on and put out because all of the developers wanted to work on what was fun instead of having to drive the basic functionality22:57
*** RajeshMohan has quit IRC22:58
*** gdubreui has joined #openstack-neutron22:58
beaglessweston, :)22:59
swestonbeagles: would anything that is going on in the subgroups really be possible without ml2?  nova-network is not modular, it is stable yes, but fragile, and hard to extend.22:59
*** RajeshMohan has joined #openstack-neutron22:59
*** JPtheJP has quit IRC23:00
swestonbeagles: but I can tell that I am preaching to the choir here, so what in your opinion is the best way to proceed?23:00
swestonbeagles: ;-)23:00
beaglessweston, oh my23:00
beaglessweston, well...23:03
*** pcm_ has joined #openstack-neutron23:03
*** alagalah_ has joined #openstack-neutron23:04
beaglessweston, I think to start it needs to be established whether there is a true business case behind parity and the notion of re-casting the simplified - or in the case of multi-host- functionality using the neutron component23:07
*** alagalah has quit IRC23:07
swestonbeagles: so, I think that everyone realizes what a good thing we have going on with Neutron, and that we want to do everything we can to favor the adoption and success of the project, right?  I know that many people are working very hard on contributing, I certainly am, and I would hate to see a lot of good work go to waste because there was a lack of effort to make the code usable.23:08
beaglessweston: let me state it another way, does neutron need to close those gap items in order to continue to exist and evolve23:10
* beagles stipulates he is punching out of his weight class and wonders if the cores are watching :)23:11
swestonbeagles: I have been given the opportunity to begin working on developing this software full time, as of the beginning of the year, and now that I can be working with the community (as my job) to make this effort successful, and I am willing to help out in whatever way I can.23:12
*** ecbanks has quit IRC23:12
*** amotoki has quit IRC23:12
swestonme is chuckling.23:13
* sweston is chuckling23:13
swestonbeagles:  well, I think anyone is free to join in here, but yes, I do believe the gaps need to be closed in order to make it an easier transition.23:14
beaglessweston, the reason I'm playing devil's advocate because it is not clear to me that it is commonly viewed this way.23:16
*** yamahata has quit IRC23:17
beaglessweston, but that's enough of that.23:17
beaglessweston, there are two challenges moving forward that: one is the gap that's already been discussed and the other will be keeping up with the changes in the Openstack ecosystem.23:19
swestonbeagles: yes ;-).  I do think, though that it is reasonable to address the issues of nova-network out of the box Neutron parity, at least to some extent, in order to favor the adoption of the technology.  I know that I do not want to be the only one working on parity, though, I believe that it should be a community effort ... although I am willing to at least help facilitate some amount of compromise here.23:21
beaglessweston, e.g. changes in how nova and neutron interact - these may be significant moving forward. In some cases it may make things a lot easier/smoother in others it may "break" things or at the very least obsolete efforts that might be underway23:21
beaglessweston, understood23:21
*** otherwiseguy has quit IRC23:22
swestonbeagles: understood, as well, ;-)23:22
openstackgerritAkihiro Motoki proposed a change to openstack/neutron: NEC plugin: PFC packet fitler support  https://review.openstack.org/7419223:23
*** peristeri has quit IRC23:25
*** jgrimm has quit IRC23:26
swestonbeagles: so, what are a few things I can get started on to help?  let's pick some tasks, in your opinion, that will not hinder the current functionality and restore some of the community goodness in making Neutron more "palatable"23:26
openstackgerritRyu Ishimoto proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse  https://review.openstack.org/7419323:26
*** aymenfrikha has quit IRC23:28
openstackgerritBob Melander proposed a change to openstack/neutron: Implements Neutron routing API using Cisco appliances.  https://review.openstack.org/7419423:30
*** alexpilotti has quit IRC23:30
beaglessweston, to be honest, I think we should focus on fixing the bugs that affect common usability. If we are looking at functionality that doesn't exist yet while the key functionality that does exist, it doesn't matter if we close the remaining gap. It's gotta be ... smooooooth ...23:31
beaglessweston, there is an image problem now as well, so scrutiny is always going to be very high23:32
beaglessweston, I guess in a vague way, I feel that dilution of the resource pool doesn't get us there. If we can make neutron be the best neutron it can be, then the last bits either will be acceptable or easier to resolve23:34
beaglessweston, strong foundations and all of that.23:34
beaglessweston, and ...23:34
beaglessweston, unfortunately I have to run.. it's 2004h here and we haven't had supper yet23:35
beaglesI missed some words there somewhere ...23:35
beagles...  looking at functionality that doesn't exist yet while the key functionality that does exist doesn't work very well...23:36
*** leseb has quit IRC23:36
beaglessweston, anyways.. the natives are now hollering. Catch up tomorrow?23:37
swestonbeagles: Yes, you bring some good issues and problems to light.  Perhaps one of the best strengths of working as a community is that there are enough bright minds involved to make the vast majority happy with some common ground.  Many other open source projects have been successful in this area, there is no reason, in my opinion, why we cannot strive for the same "togetherness".  :-).23:39
*** oda-g_ has joined #openstack-neutron23:40
swestonbeagles: Yes, let's catch up tomorrow ... I have been working all night on the lbaas ovs hybrid interface driver, and I could use a break as well.  Have a good evening ;-)23:41
*** oda-g has quit IRC23:42
*** IanGovett has left #openstack-neutron23:43
*** mwagner_lap has quit IRC23:51
*** mlavalle has quit IRC23:55
*** arosen has joined #openstack-neutron23:57

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