Friday, 2015-03-27

*** Aish has left #openstack-neutron00:00
*** dannywil_ has quit IRC00:02
*** markvoelker has quit IRC00:02
*** annegentle has joined #openstack-neutron00:06
*** yamamoto has joined #openstack-neutron00:06
*** xgerman_ has quit IRC00:11
*** Longgeek has quit IRC00:15
*** sbalukoff has quit IRC00:16
*** ijw_ has joined #openstack-neutron00:16
*** ijw__ has joined #openstack-neutron00:18
openstackgerritItsuro Oda proposed openstack/neutron: Enable services on agents with admin_state_up False  https://review.openstack.org/14703200:18
*** ijw has quit IRC00:19
*** dboik has joined #openstack-neutron00:21
*** ijw_ has quit IRC00:21
*** jamiem has quit IRC00:22
*** jamiem has joined #openstack-neutron00:26
*** jamiem has quit IRC00:27
*** padkrish_ has quit IRC00:28
*** padkrish has joined #openstack-neutron00:28
*** Flcn has joined #openstack-neutron00:28
*** reed has quit IRC00:29
*** padkrish_ has joined #openstack-neutron00:29
*** reed has joined #openstack-neutron00:30
*** reed has quit IRC00:30
*** reed has joined #openstack-neutron00:31
*** padkrish has quit IRC00:32
*** Flcn has quit IRC00:33
*** igordcard_ has quit IRC00:34
*** yamamoto has quit IRC00:34
*** vishwanathj has quit IRC00:37
*** tflynn has joined #openstack-neutron00:38
*** fredlhsu has joined #openstack-neutron00:39
*** baoli has joined #openstack-neutron00:41
*** pavel_bondar has quit IRC00:41
*** suresh12 has quit IRC00:41
*** pavel_bondar has joined #openstack-neutron00:42
*** tflynn has quit IRC00:43
*** catohornet has joined #openstack-neutron00:43
*** baoli has quit IRC00:48
openstackgerritArmando Migliaccio proposed openstack/neutron: Simplify base test cases  https://review.openstack.org/16704100:48
kevinbentonoda-g: sorry about the delay00:48
kevinbentonoda-g: i'm fine with cedric's suggestion to00:48
*** tiswanso has joined #openstack-neutron00:49
*** fredlhsu has quit IRC00:49
*** jamiem has joined #openstack-neutron00:53
*** rangansw has quit IRC00:56
*** nlahouti has quit IRC00:57
*** SumitNaiksatam has quit IRC00:57
*** banix has joined #openstack-neutron01:00
*** tiswanso has quit IRC01:03
*** tiswanso has joined #openstack-neutron01:03
*** Longgeek has joined #openstack-neutron01:03
openstackgerritArmando Migliaccio proposed openstack/neutron: Simplify base test cases  https://review.openstack.org/16704101:06
oda-gkevinbenton: thank you for your support. it's really help to me.01:09
kevinbentonoda-g: no prob. your fixing useful things01:09
*** salv-orlando has quit IRC01:10
*** annegentle has quit IRC01:14
*** crc32 has quit IRC01:16
*** annegentle has joined #openstack-neutron01:17
*** yamamoto has joined #openstack-neutron01:17
*** zhhuabj has quit IRC01:21
*** dims_ has quit IRC01:23
*** ChuckC has joined #openstack-neutron01:26
*** jamiem has quit IRC01:29
*** _cjones_ has quit IRC01:30
*** _cjones_ has joined #openstack-neutron01:31
*** popw has joined #openstack-neutron01:33
*** _cjones_ has quit IRC01:35
*** SumitNaiksatam has joined #openstack-neutron01:35
openstackgerritItsuro Oda proposed openstack/neutron: Enable services on agents with admin_state_up False  https://review.openstack.org/14703201:35
*** ChuckC has quit IRC01:36
*** stanzgy has joined #openstack-neutron01:38
*** tflynn has joined #openstack-neutron01:39
*** shaohe_feng has joined #openstack-neutron01:39
openstackgerritKevin Benton proposed openstack/neutron: Set floating IP port status to "N/A"  https://review.openstack.org/12226501:39
*** s3wong has quit IRC01:42
*** baoli has joined #openstack-neutron01:43
*** tflynn has quit IRC01:43
*** annegentle has quit IRC01:48
*** melwitt has quit IRC01:48
*** yamahata has quit IRC01:49
*** amotoki has quit IRC01:50
*** happyeveryday has joined #openstack-neutron01:50
*** Longgeek has quit IRC01:52
*** banix has quit IRC01:52
*** Longgeek has joined #openstack-neutron01:53
*** dank_ has quit IRC01:55
*** happyeveryday has quit IRC01:58
*** Networkn3rd has joined #openstack-neutron02:00
*** haigang has joined #openstack-neutron02:00
*** vishwanathj has joined #openstack-neutron02:05
*** mlavalle has joined #openstack-neutron02:06
*** salv-orlando has joined #openstack-neutron02:08
*** erkules_ has joined #openstack-neutron02:09
*** erkules has quit IRC02:12
*** gildub has joined #openstack-neutron02:14
openstackgerritgoocher proposed openstack/neutron: Change check_child_processes_interval default  https://review.openstack.org/16820402:16
*** oomichi_ has quit IRC02:17
*** banix has joined #openstack-neutron02:19
*** anshul has joined #openstack-neutron02:20
openstackgerritDan Prince proposed openstack/neutron: Revert "Fix validation of physical network name for flat nets"  https://review.openstack.org/16820702:21
*** achanda has quit IRC02:27
*** haigang has quit IRC02:29
*** tsg has quit IRC02:29
*** padkrish_ has quit IRC02:30
*** haigang has joined #openstack-neutron02:30
*** padkrish has joined #openstack-neutron02:30
*** padkrish has quit IRC02:35
*** jckasper has joined #openstack-neutron02:39
*** catohornet has quit IRC02:39
*** pal_ has joined #openstack-neutron02:43
*** jckasper has quit IRC02:45
*** salv-orlando has quit IRC02:45
*** annegentle has joined #openstack-neutron02:48
*** annegentle has quit IRC02:54
*** devlaps has quit IRC02:54
*** oda-g has quit IRC02:59
*** Murali has joined #openstack-neutron03:00
*** tiswanso has quit IRC03:04
*** ijw__ has quit IRC03:06
openstackgerritMerged openstack/neutron: Simplify base test cases  https://review.openstack.org/16704103:12
*** achanda has joined #openstack-neutron03:13
*** shwetaap has joined #openstack-neutron03:16
openstackgerritHenry Gessau proposed openstack/neutron: Modify a different agent in test_update_agent_description  https://review.openstack.org/16821303:16
*** emagana has joined #openstack-neutron03:18
*** anish_ has joined #openstack-neutron03:18
*** liusheng has joined #openstack-neutron03:21
*** suresh12 has joined #openstack-neutron03:23
openstackgerritKevin Benton proposed openstack/neutron: Don't eagerly load ranges from IPAllocationPool  https://review.openstack.org/16821403:25
*** atuvenie has joined #openstack-neutron03:25
*** tsg has joined #openstack-neutron03:26
*** Marga_ has quit IRC03:26
openstackgerritZengfa Gao proposed openstack/neutron: Add API tests for subnetpool allocation  https://review.openstack.org/16761103:26
HenryGkevinbenton: https://review.openstack.org/14954603:27
*** tflynn has joined #openstack-neutron03:28
*** ajmiller has joined #openstack-neutron03:28
*** Murali has quit IRC03:29
lizk_hi, guys, would you please review https://review.openstack.org/#/c/155317/ and https://review.openstack.org/#/c/166539/ thx :)03:30
*** amuller_ has quit IRC03:32
*** Longgeek has quit IRC03:34
*** suresh12 has quit IRC03:38
*** tflynn has quit IRC03:40
*** tflynn has joined #openstack-neutron03:40
*** padkrish has joined #openstack-neutron03:41
openstackgerritAndrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port  https://review.openstack.org/16778403:42
*** pal__ has joined #openstack-neutron03:44
*** pal_ has quit IRC03:45
*** padkrish has quit IRC03:46
*** salv-orlando has joined #openstack-neutron03:46
*** banix has quit IRC03:46
*** sbalukoff has joined #openstack-neutron03:46
*** _cjones_ has joined #openstack-neutron03:48
*** _cjones_ has quit IRC03:49
*** baoli has quit IRC03:49
*** annegentle has joined #openstack-neutron03:50
*** emagana has quit IRC03:51
*** emagana has joined #openstack-neutron03:52
*** baoli has joined #openstack-neutron03:52
*** salv-orlando has quit IRC03:52
*** tflynn has quit IRC03:54
*** pal__ has quit IRC03:55
*** annegentle has quit IRC03:55
*** emagana has quit IRC03:56
*** lykinsbd has quit IRC03:58
*** vishwana_ has joined #openstack-neutron03:58
*** _cjones_ has joined #openstack-neutron03:58
openstackgerritshihanzhang proposed openstack/neutron: Reduce dhcp agent calling 'reload_allocations'  https://review.openstack.org/16821603:58
*** fredlhsu has joined #openstack-neutron03:59
*** deepthi has joined #openstack-neutron04:00
*** haleyb has joined #openstack-neutron04:02
*** _cjones_ has quit IRC04:02
*** baoli has quit IRC04:02
*** _cjones_ has joined #openstack-neutron04:02
*** atuvenie has quit IRC04:06
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113004:06
*** Longgeek has joined #openstack-neutron04:08
*** annegentle has joined #openstack-neutron04:08
*** tflynn has joined #openstack-neutron04:09
*** yuanying has joined #openstack-neutron04:09
*** annegentle has quit IRC04:13
*** fredlhsu has quit IRC04:13
*** nlahouti has joined #openstack-neutron04:17
*** mlavalle has quit IRC04:19
*** _cjones_ has quit IRC04:33
*** tflynn_ has joined #openstack-neutron04:35
*** tflynn has quit IRC04:35
*** markvoelker has joined #openstack-neutron04:39
*** bquan has joined #openstack-neutron04:39
*** lizk has quit IRC04:41
*** bquan has quit IRC04:41
*** lizk_ has quit IRC04:42
*** markvoelker has quit IRC04:43
*** Networkn3rd has quit IRC04:43
*** tflynn_ has quit IRC04:44
*** tflynn has joined #openstack-neutron04:45
*** yfauser has joined #openstack-neutron04:46
*** yatin has joined #openstack-neutron04:47
*** YorikSar has quit IRC04:48
*** tflynn has quit IRC04:51
*** fredlhsu has joined #openstack-neutron04:53
*** lizk has joined #openstack-neutron04:53
*** lizk_ has joined #openstack-neutron04:54
*** fredlhsu has quit IRC04:54
*** trinaths has joined #openstack-neutron04:58
*** padkrish has joined #openstack-neutron04:58
*** padkrish has joined #openstack-neutron04:59
*** nlahouti has quit IRC05:00
*** achanda has quit IRC05:00
*** reed has quit IRC05:03
*** ajmiller has quit IRC05:05
*** shwetaap1 has joined #openstack-neutron05:09
*** shwetaap has quit IRC05:13
*** lpetrut has joined #openstack-neutron05:13
*** Marga_ has joined #openstack-neutron05:14
*** pal_ has joined #openstack-neutron05:16
*** nplanel_ has quit IRC05:19
*** annegentle has joined #openstack-neutron05:19
*** fredlhsu has joined #openstack-neutron05:19
*** yog__ has joined #openstack-neutron05:19
*** annegentle has quit IRC05:24
*** fredlhsu has quit IRC05:29
*** tsg has quit IRC05:30
*** chandankumar has joined #openstack-neutron05:36
*** achanda has joined #openstack-neutron05:37
*** markvoelker has joined #openstack-neutron05:39
openstackgerritAbishek Subramanian proposed openstack/neutron: Support IPv6 Router  https://review.openstack.org/15628305:40
*** jamielennox is now known as jamielennox|away05:41
*** markvoelker has quit IRC05:43
*** Networkn3rd has joined #openstack-neutron05:44
*** salv-orlando has joined #openstack-neutron05:44
*** lpetrut has quit IRC05:45
*** lpetrut has joined #openstack-neutron05:46
*** dims has joined #openstack-neutron05:46
*** vthapar has joined #openstack-neutron05:46
*** salv-orlando has quit IRC05:47
*** trinaths has quit IRC05:47
*** Networkn3rd has quit IRC05:48
*** anish_ has quit IRC05:51
*** anshul has quit IRC05:51
*** anshul has joined #openstack-neutron05:52
*** neeti has joined #openstack-neutron05:54
*** YorikSar has joined #openstack-neutron05:55
*** yalie has quit IRC05:57
*** yalie has joined #openstack-neutron05:58
*** vishwanathj has quit IRC05:59
*** Murali has joined #openstack-neutron06:00
*** aranjan has joined #openstack-neutron06:01
*** Marga_ has quit IRC06:02
*** trinaths has joined #openstack-neutron06:07
*** dsneddon has quit IRC06:08
*** anish_ has joined #openstack-neutron06:08
*** aranjan has quit IRC06:11
stockpiratehi! someone who is awake?06:12
*** armax has quit IRC06:13
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/16749906:15
*** aranjan has joined #openstack-neutron06:16
stockpiratei have been created a net and subnet. the neutron agent-list is down on all agents06:17
stockpirateso i will delete those net and subnet i have been created. but it hang06:18
stockpiratean example:  neutron net-delete 04797502-0629-417f-8b13-8eda888c34ac06:18
stockpiratenothing happend06:18
*** numan has joined #openstack-neutron06:18
trinathsstockpirate: is the DB active06:20
*** annegentle has joined #openstack-neutron06:20
*** SridharG has joined #openstack-neutron06:21
*** yfauser has left #openstack-neutron06:21
stockpirateit is: root@vitkenxfs03:/etc/neutron# netstat -an | grep LISTEN06:21
stockpiratetcp        0      0 0.0.0.0:35357           0.0.0.0:*               LISTEN06:21
stockpiratetcp        0      0 0.0.0.0:9696            0.0.0.0:*               LISTEN06:21
*** dims has quit IRC06:21
openstackgerritMerged openstack/neutron-fwaas: Remove reference to testlib_plugin  https://review.openstack.org/16812306:21
stockpirateand;06:22
stockpirateMariaDB [(none)]> show databases;06:22
stockpirate+--------------------+06:22
stockpirate| Database           |06:22
stockpirate+--------------------+06:22
stockpirate| information_schema |06:22
stockpirate| neutron            |06:22
stockpirate+--------------------+06:22
stockpirate2 rows in set (0.00 sec)06:22
*** shaohe_feng has quit IRC06:23
*** anish_ has quit IRC06:23
stockpiratewhen i do:06:25
stockpirateroot@vitkencas03:/etc/neutron# neutron net-list06:25
stockpirate+--------------------------------------+--------+-----------------------------------------------------+06:25
stockpirate| id                                   | name   | subnets                                             |06:25
stockpirate+--------------------------------------+--------+-----------------------------------------------------+06:25
stockpirate| 04797502-0629-417f-8b13-8eda888c34ac | test01 | 0b6f60ee-d6f9-4182-b243-3797fb943597 192.168.0.0/24 |06:25
stockpirate+--------------------------------------+--------+-----------------------------------------------------+06:25
*** annegentle has quit IRC06:25
*** afazekas is now known as __afazekas06:28
*** aranjan has quit IRC06:30
openstackgerritItsuro Oda proposed openstack/neutron: Make floatingip reachable from the same network  https://review.openstack.org/16194706:31
*** aranjan has joined #openstack-neutron06:31
*** Bhavya has joined #openstack-neutron06:32
*** achanda has quit IRC06:32
*** shaohe_feng has joined #openstack-neutron06:33
*** salv-orlando has joined #openstack-neutron06:33
*** shwetaap has joined #openstack-neutron06:33
*** shwetaap1 has quit IRC06:36
*** ishant has joined #openstack-neutron06:36
openstackgerritP. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help router-update" help info updated  https://review.openstack.org/14100606:36
*** salv-orlando has quit IRC06:37
*** aepifanov has joined #openstack-neutron06:39
trinathsstockpirate: can u check the logs in neutron server06:39
trinathsstockpirate: when u execute the same CLI neutron command.06:39
stockpirateyes. two windows up. one running neutron net-delete06:39
stockpiratethe other for logs06:40
*** markvoelker has joined #openstack-neutron06:40
trinathsstockpirate: when u post a huge logs for debugging, you may use http://paste.openstack.org so that, its easy to communicate the debug logs06:40
stockpiratefor neutron/server.log06:40
stockpirateokei thanks for tip06:40
trinathsstockpirate: look into the neutron/server.log on what wrong with the command your are executing for deleting the subnet from neutron cli.06:42
stockpiratehttp://paste.openstack.org/show/196988/06:42
stockpiratealmost nothing written in server.log  amq connected etc06:43
stockpiratesome stuff from the horizon06:43
*** haleyb has quit IRC06:43
*** anish_ has joined #openstack-neutron06:43
trinathsstockpirate: did neutron server start correctly.06:44
*** markvoelker has quit IRC06:44
*** haleyb has joined #openstack-neutron06:45
stockpiratehttp://paste.openstack.org/show/196989/06:46
stockpirateupdated what happend when im stopping and starting neutron-server service again06:46
vthaparstockpirate: one more thing you can do is give --debug [or is it -debug? check the help] option when giving neutron cli commands, gives better idea of what went wrong.06:47
trinathsstockpirate: yes, enable debug in neutron.conf06:47
trinathsvthapar: thanks for this hint :)06:48
*** yamamoto has quit IRC06:48
vthapartrinaths: you're welcome :)06:48
*** aranjan has quit IRC06:49
vthaparstockpirate: from neutron server logs looks like server did come up, confirmed by netstat output. running neutron client cli with debug option will tell what response came from neturon server, if any.06:49
stockpiratehttp://paste.openstack.org/show/196991/06:49
stockpirateupdated06:50
stockpiratethanks06:50
stockpiratewhen im running  neutron net-delete 04797502-0629-417f-8b13-8eda888c34ac --debug06:50
*** Longgeek has quit IRC06:50
stockpirateafter the end of text, it just "blinking" hang and nothing more happend.06:51
trinathsstockpirate: can u rung this command, curl -i http://10.55.0.28:9696/v2.0/networks/04797502-0629-417f-8b13-8eda888c34ac.json -X DELETE -H "X-Auth-Token: a505731b90204e5d8c4d12a5a6a5866d" -H "Content-Type: application/json" -H "Accept: application/json" -H "User-Agent: python-neutronclient"06:51
vthaparstockpirate: so you're delete request goes to neutron server, nothing after that. second response in first output confirms neutrons erver is still running and reachable so network issues also ruled out.06:52
vthaparstockpirate: next step would be to check neutron server logs, with debug enabled.06:53
*** Longgeek has joined #openstack-neutron06:53
stockpirate<trinaths> doing that command directly now. It just hang/blinking/wainting for "something".06:53
stockpiratevthapar: thanks, you can see whap happend when i do neutron net-del ****  --debud here,;  http://paste.openstack.org/show/196991/06:54
stockpiratethe logs say nothing06:54
stockpirateA stupid question. Do i need to stop a service before doing a net-del or sub-del?06:55
*** bradjones has quit IRC06:55
*** pradipta has joined #openstack-neutron06:55
*** sridhar_ram has joined #openstack-neutron06:56
*** haleyb has quit IRC06:56
vthaparstockpirate: yeah, I took a look at logs and looks like client-server communication is fine. now you need to take a look at neutron server logs and openvswitch agent logs.06:56
*** bradjones has joined #openstack-neutron06:57
*** bradjones has quit IRC06:57
*** bradjones has joined #openstack-neutron06:57
*** gildub has quit IRC06:57
openstackgerritshihanzhang proposed openstack/neutron: Reduce dhcp agent calling 'reload_allocations'  https://review.openstack.org/16821606:57
trinathsstockpirate: no need to restart for delete06:58
vthaparand you don't need to stop any service. you will need to restart a service for any conf file changes to take effect.06:58
*** ajo has quit IRC06:58
stockpirateokei thank i see06:59
stockpiratejust adding some logs06:59
*** sridhar_ram has quit IRC07:00
*** nfedotov has joined #openstack-neutron07:01
*** apuimedo has joined #openstack-neutron07:01
openstackgerritP. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help net-list" help info updated  https://review.openstack.org/14062807:02
stockpiratehttp://paste.openstack.org/show/196993/07:03
*** shwetaap has quit IRC07:04
*** bjornar_ has joined #openstack-neutron07:04
trinathsstockpirate: ovsdb error?07:05
stockpiratehttp://paste.openstack.org/show/196995/07:07
stockpirateadding ovs in the bottom07:07
stockpiratesorry missing db07:08
stockpiratea sec07:08
*** scheuran has joined #openstack-neutron07:08
stockpirateovsdb added in the bottom07:09
*** anish_ has quit IRC07:12
*** afazekas has joined #openstack-neutron07:13
*** padkrish has quit IRC07:13
openstackgerritCedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers  https://review.openstack.org/16586607:15
*** chlong has quit IRC07:16
*** anish_ has joined #openstack-neutron07:17
*** bjornar_ has quit IRC07:18
openstackgerritCedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests  https://review.openstack.org/14341207:23
stockpirateThanks for helping me !!!07:23
stockpirateim07:23
stockpiratethink it only was a reboot07:23
stockpiratethere was a hang on one service07:24
stockpiratefinally i could delete the net!!!07:24
stockpirateim rebooted everything so i dont really know what service it was. so i need to figure it out before im going in production some time07:25
*** anilvenkata has joined #openstack-neutron07:25
trinathsstockpirate: congrats :)07:25
*** Miouge has joined #openstack-neutron07:26
stockpiratethank you sir for you time!!07:26
*** Networkn3rd has joined #openstack-neutron07:28
openstackgerritCedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine  https://review.openstack.org/16029007:29
*** anish__ has joined #openstack-neutron07:30
*** shaohe_feng has quit IRC07:30
*** anish_ has quit IRC07:30
openstackgerritCedric Brandily proposed openstack/neutron: Clean neutron.tests.functional.agent.linux.base  https://review.openstack.org/16191307:30
*** shaohe_feng has joined #openstack-neutron07:31
*** angela-s has quit IRC07:32
*** Networkn3rd has quit IRC07:32
*** anish__ has quit IRC07:36
openstackgerritP. Watsalya Mishra proposed openstack/python-neutronclient: "neutron help net-update" help info updated  https://review.openstack.org/13727907:40
*** markvoelker has joined #openstack-neutron07:40
*** bquan has joined #openstack-neutron07:42
*** bobmel has joined #openstack-neutron07:42
*** lpetrut has quit IRC07:45
*** markvoelker has quit IRC07:45
*** bquan has quit IRC07:46
*** haleyb has joined #openstack-neutron07:53
openstackgerritDarragh O'Reilly proposed openstack/neutron: lb-agent: handle security group updates in main loop  https://review.openstack.org/13851207:55
*** haleyb has quit IRC07:58
*** aranjan has joined #openstack-neutron08:00
*** aranjan has quit IRC08:04
*** pradipta has quit IRC08:06
*** aleksandr_null has quit IRC08:07
*** SumitNaiksatam has quit IRC08:08
*** zhenguo has joined #openstack-neutron08:09
*** shaohe_feng has quit IRC08:10
openstackgerritenikanorov proposed openstack/neutron: OOP cleanup: start protected method names with underscore  https://review.openstack.org/16805608:11
*** erkules_ is now known as erkules08:11
*** erkules has joined #openstack-neutron08:11
*** anish_ has joined #openstack-neutron08:11
*** matrohon has joined #openstack-neutron08:12
*** lpetrut has joined #openstack-neutron08:12
*** jpena has joined #openstack-neutron08:14
*** salv-orlando has joined #openstack-neutron08:15
*** safchain has joined #openstack-neutron08:15
*** lpetrut has quit IRC08:15
*** ronis has joined #openstack-neutron08:16
*** anish_ has quit IRC08:17
*** anish_ has joined #openstack-neutron08:18
*** anilvenkata has left #openstack-neutron08:20
*** salv-orlando has quit IRC08:20
*** annegentle has joined #openstack-neutron08:22
*** aleksandr_null has joined #openstack-neutron08:23
*** sbalukoff has quit IRC08:24
*** sbalukoff has joined #openstack-neutron08:25
*** ajo has joined #openstack-neutron08:27
*** lpetrut has joined #openstack-neutron08:27
*** annegentle has quit IRC08:27
*** marun has joined #openstack-neutron08:31
*** doude has joined #openstack-neutron08:31
*** ihrachyshka has joined #openstack-neutron08:32
*** marun has quit IRC08:35
*** enikanorov has joined #openstack-neutron08:36
*** jlanoux has joined #openstack-neutron08:37
*** shaohe_feng has joined #openstack-neutron08:40
*** jistr has joined #openstack-neutron08:40
*** markvoelker has joined #openstack-neutron08:41
openstackgerritCedric Brandily proposed openstack/neutron: Allow to define enable_snat default value  https://review.openstack.org/13058808:42
*** ihrachyshka has quit IRC08:45
*** markvoelker has quit IRC08:46
stockpiratemore questions from me:  tenant router gateway will not occupy ip from floating ip list.08:50
stockpiratehow could i list ip list form router gateway... to see if not have any ip or refresh it to get an ip ?08:53
*** nyechiel has joined #openstack-neutron08:53
*** Madkiss has quit IRC08:59
*** rotbart has joined #openstack-neutron09:00
*** anilvenkata has joined #openstack-neutron09:01
*** ygbo has joined #openstack-neutron09:03
*** Madkiss has joined #openstack-neutron09:03
*** Madkiss has quit IRC09:03
*** Madkiss has joined #openstack-neutron09:03
stockpiratei find it when im clicking on router in horizon09:04
stockpirateand status down09:04
*** nfedotov has quit IRC09:05
openstackgerritRomil Gupta proposed openstack/neutron: Move values for network_type to plugins.common.constants.py  https://review.openstack.org/16687909:09
*** Madkiss has quit IRC09:09
*** atuvenie has joined #openstack-neutron09:10
*** Madkiss has joined #openstack-neutron09:10
*** Madkiss has quit IRC09:10
*** Madkiss has joined #openstack-neutron09:10
*** yatin has quit IRC09:11
*** pradipta has joined #openstack-neutron09:13
*** pradipta has quit IRC09:14
*** pradipta has joined #openstack-neutron09:14
*** yatin has joined #openstack-neutron09:15
openstackgerritMerged openstack/neutron: Enable services on agents with admin_state_up False  https://review.openstack.org/14703209:16
*** Networkn3rd has joined #openstack-neutron09:16
*** bjornar_ has joined #openstack-neutron09:20
*** Networkn3rd has quit IRC09:21
*** shaohe_feng has quit IRC09:23
*** lukasa has joined #openstack-neutron09:23
*** flaviof has joined #openstack-neutron09:26
stockpirateI followed this: http://docs.openstack.org/juno/install-guide/install/apt/content/neutron_initial-networks-verify.html10.55.0.71 network:router_gatewayDOWNUP09:32
stockpiratesorry::::  I followed this: http://docs.openstack.org/juno/install-guide/install/apt/content/neutron_initial-networks-verify.html09:32
stockpiratebut the status is down in router gateway:  10.55.0.71 network:router_gatewayDOWNUP09:32
stockpiratedoing a:: ip netns exec  qrouter-e91277b9-f0ca-4d6f-aa85-a2a97e9fd69f ping 10.55.0.7109:33
stockpirateand get answer09:33
stockpiratebut I dont get any answer when im doing only: ping 10.55.0.7109:33
gongyshmestery: can u help to have a quick review on https://review.openstack.org/#/c/160430/09:35
*** markvoelker has joined #openstack-neutron09:42
*** gongysh has quit IRC09:44
*** markvoelker has quit IRC09:46
*** virtuallynathan_ has joined #openstack-neutron09:47
*** nyechiel has quit IRC09:50
hanzhanghi all, who can help review this patch:https://review.openstack.org/#/c/151501/09:50
*** dims_ has joined #openstack-neutron09:51
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Enable Process Monitor by default.  https://review.openstack.org/16511709:52
*** neiljerram has quit IRC09:53
*** Flcn has joined #openstack-neutron09:58
*** txomon|fon_ has quit IRC09:58
*** txomon|fon_ has joined #openstack-neutron09:58
*** hichihar_ has quit IRC09:58
*** nyechiel has joined #openstack-neutron09:59
*** yalie has quit IRC10:02
openstackgerritAbhishek G M proposed openstack/neutron: Test to verify  shared attribute of network  https://review.openstack.org/16756610:05
openstackgerritAbhishek G M proposed openstack/neutron: Test to verify  shared attribute of network  https://review.openstack.org/16756610:09
*** pc_m has joined #openstack-neutron10:11
*** amotoki has joined #openstack-neutron10:13
*** nyechiel has quit IRC10:15
*** nyechiel has joined #openstack-neutron10:15
*** pradipta has quit IRC10:17
*** deepthi has quit IRC10:17
*** marun has joined #openstack-neutron10:20
*** txomon|fon_ has quit IRC10:23
*** annegentle has joined #openstack-neutron10:23
*** marun has quit IRC10:24
*** zilberstein has quit IRC10:26
*** annegentle has quit IRC10:29
*** salv-orlando has joined #openstack-neutron10:30
*** gsagie has joined #openstack-neutron10:31
*** Longgeek has quit IRC10:31
*** salv-orlando has quit IRC10:34
*** matrohon has quit IRC10:34
*** shaohe_feng has joined #openstack-neutron10:37
*** zilberstein has joined #openstack-neutron10:39
*** gildub has joined #openstack-neutron10:39
*** gsagie has quit IRC10:42
*** nyechiel has quit IRC10:42
*** markvoelker has joined #openstack-neutron10:43
openstackgerritRomil Gupta proposed openstack/neutron: Move values for network_type to plugins.common.constants.py  https://review.openstack.org/16687910:43
*** trinaths has quit IRC10:46
*** atuvenie has quit IRC10:47
*** markvoelker has quit IRC10:48
*** haigang has quit IRC10:48
*** stanzgy has quit IRC10:49
*** txomon|fon has joined #openstack-neutron10:51
*** pradip_vedams has joined #openstack-neutron10:52
*** yuanying has quit IRC10:54
*** amotoki has quit IRC10:56
*** amotoki has joined #openstack-neutron10:57
*** yuanying has joined #openstack-neutron10:57
*** flaviof has quit IRC11:03
*** txomon|fon has quit IRC11:05
*** Networkn3rd has joined #openstack-neutron11:05
*** txomon|fon has joined #openstack-neutron11:07
*** Longgeek has joined #openstack-neutron11:08
*** Networkn3rd has quit IRC11:10
*** baoli has joined #openstack-neutron11:11
*** ihrachyshka has joined #openstack-neutron11:11
*** pal_ has quit IRC11:12
*** yog__ has quit IRC11:12
*** pal_ has joined #openstack-neutron11:13
openstackgerritPaul Michali proposed openstack/neutron-fwaas: FWaaS: Refactor callback mechanism  https://review.openstack.org/16727511:15
*** flaviof has joined #openstack-neutron11:16
*** ajo has quit IRC11:18
*** gildub has quit IRC11:20
*** jamiem has joined #openstack-neutron11:21
*** baoli has quit IRC11:22
*** Flcn is now known as flcn11:24
*** annegentle has joined #openstack-neutron11:24
*** salv-orlando has joined #openstack-neutron11:24
*** shaohe_feng has quit IRC11:26
*** annegentle has quit IRC11:29
*** ociuhandu has joined #openstack-neutron11:31
*** Murali has quit IRC11:31
pc_mamotoki: ping11:34
*** rpothier has joined #openstack-neutron11:34
amotokipc_m: pong11:34
pc_mamotoki: Thanks for reviewing the VPN API documentation11:34
pc_mamotoki: Just to be sure... I see that all of the list APIs are incorrect (not plural)11:35
amotokipc_m: yeah.. all resources are in a singular case :-(11:35
pc_mamotoki: That's what you're seeing right? Nothing else?11:35
amotokipc_m: yes. I followed the result of the  Jenkins job at the bottom and then open the PDF of networking-api.11:36
amotokipc_m: other than that, i could not find a good way to check the contents.11:37
*** blues-man has joined #openstack-neutron11:37
pc_mamotoki: OK. I just saw you mentioned the show command for ipsec connection. wasn't sure if you thought something was wrong there - I thought it was OK>11:37
amotokipc_m: ah.. my comment was ambiguous.11:38
pc_mamotoki: Yeah I build the doc and access the pages locally11:38
pc_mamotoki: I got the gist.11:38
pc_mJust wanted to make sure I didn't miss anything else.11:38
*** mwagner_lap has quit IRC11:38
pc_mIf you get a chance can you look at https://review.openstack.org/#/c/164466/ and https://review.openstack.org/#/c/165226/?11:38
pc_mI'd like to get them into the RC, if we can get reviewers in time.  It is a refactoring cleanup of VPN/FW and Neutron for callbacks to the services from L3 agent.11:39
amotokipc_m: sure. actually i am off from Wed and I am online only at night :-)11:39
pc_mamotoki: thanks!11:40
*** Longgeek has quit IRC11:42
*** johnbelamaric has joined #openstack-neutron11:43
*** markvoelker has joined #openstack-neutron11:43
*** tiswanso has joined #openstack-neutron11:44
*** tiswanso has quit IRC11:45
*** neeti has quit IRC11:45
*** tiswanso has joined #openstack-neutron11:46
*** jprovazn has joined #openstack-neutron11:46
*** ferest has joined #openstack-neutron11:47
*** markvoelker has quit IRC11:48
*** neeti has joined #openstack-neutron11:50
*** claudiub has joined #openstack-neutron11:50
pc_mamotoki: FYI: I just pushed up a new version of the API doc commit. https://review.openstack.org/#/c/167609/11:54
amotokipc_m: thanks. check it after reviewing fwaas insertion in horizon11:55
pc_mamotoki: Awesome!11:55
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP)  https://review.openstack.org/16811511:57
*** tiswanso has quit IRC11:59
*** pal_ has quit IRC11:59
*** dane_leblanc has joined #openstack-neutron12:00
HenryGpc_m: Have you and Armando converged on https://review.openstack.org/164466 ?12:01
HenryGAnd do you know what dougwig tried to ask?12:02
pc_mHenryG: Last I checked :) He wanted the callback manager mocked, and that is in the latest (after several botched commits). PS 9 is essentially PS 6 with a rebase.12:03
pc_mHenryG: dougwig had a Question and marked it -1, but the question contents are blank. Will wait for him to get back online and will see if he had any concerns.12:04
HenryGpc_m: Don't worry about the botched commits, I got that :)12:04
pc_mHenryG: Overall, I think it, and the VPN and FW commits are ready to go. If cores think it is good to go and have time to review, it can go into RC, IMHO.12:05
*** atuvenie has joined #openstack-neutron12:06
pc_mHenryG: If you get  chance please take a look.12:06
HenryGpc_m: The code looks fine to me. Will wait for dougwig to clarify.12:07
pc_mHenryG: thanks!12:07
*** raildo|away is now known as raildo12:08
*** pal_ has joined #openstack-neutron12:10
amotokipc_m: re VPNaaS documentation, I still see a singular case.12:10
pc_mamotoki: Which one?12:10
amotokipc_m: for exmaple, p.204 of http://docs-draft.openstack.org/09/167609/6/check/gate-api-site-tox-doc-publish-checkbuild/0840f26//publish-docs/api-ref-guides/bk-api-ref-networking-v2.pdf12:10
amotokipc_m: for example, /v2.0/vpnaas/vpnservice/{vpnservice_id}12:10
amotokipc_m: I think "vpnservice" should be "vpnservices".12:11
*** zilberstein has quit IRC12:11
pc_mamotoki: No, I think it is correct. For the LIST API they are plural, for the rest they are singular.12:11
amotokipc_m: checking with devstack12:13
*** baoli has joined #openstack-neutron12:13
*** baoli has quit IRC12:14
amotokipc_m: for example: neutron vpn-service-delete <xxxx> send DELETE http://172.16.18.35:9696/v2.0/vpn/vpnservices/115e08af-34dd-42a4-8572-32bb458a635d.json12:14
amotokipc_m: it is tested with Juno env.12:15
*** baoli has joined #openstack-neutron12:15
*** apuimedo has quit IRC12:15
pc_mWhat option did you use to see the URL? I did with --debug and I didn't see that for IKEPolicy12:16
amotokipc_m: the command line is neutron -v vpn-service-delete 115e08af-34dd-42a4-8572-32bb458a635d12:16
*** pal_ has quit IRC12:16
*** dboik has quit IRC12:16
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323612:18
*** markvoelker has joined #openstack-neutron12:18
amotokipc_m: I will leave comments on the review.12:18
*** ociuhandu has quit IRC12:18
*** amuller_ has joined #openstack-neutron12:18
pc_mamotoki: Hmmm. I thought they were all singular, but it looks like service is plural. IKEpolicy is singular. Will check others.12:19
pc_mamotoki: Would have liked this to be consistent...12:19
*** mwagner_lap has joined #openstack-neutron12:22
*** annegentle has joined #openstack-neutron12:22
amotokipc_m: all urls have plural form. for example /v2.0/networks/<uuid> are used for create/update/delete12:23
*** zilberstein has joined #openstack-neutron12:23
*** annegentle has quit IRC12:23
pc_mamotoki: You're right. Sorry. I was looking at --debug output and I think I was looking at the wrong info.12:26
amotokipc_m: plural form is always used in URL, but the body contains a singular form for create/delete/update/show. It is common in REST API.12:26
*** rushil has joined #openstack-neutron12:26
*** yatin has quit IRC12:27
openstackgerritMerged openstack/neutron: Enable Process Monitor by default.  https://review.openstack.org/16511712:28
*** ociuhandu has joined #openstack-neutron12:29
pc_mamotoki: Will post another patch in a few mins... building and testing...12:29
amotokipc_m: no need to hurry :)12:30
pc_mamotoki: Nothing else to do on a rainy Friday :)12:30
*** atuvenie has quit IRC12:32
*** anish_ has quit IRC12:33
*** yamamoto has joined #openstack-neutron12:33
*** blues-man has quit IRC12:38
*** yamamoto has quit IRC12:40
*** yuanying has quit IRC12:42
*** rook_ has joined #openstack-neutron12:45
*** amotoki has quit IRC12:50
*** atuvenie has joined #openstack-neutron12:52
*** john-davidge has joined #openstack-neutron12:56
john-davidge#openstack-lbaas12:57
john-davidgehaha, oops12:57
*** dboik has joined #openstack-neutron12:58
*** amotoki has joined #openstack-neutron12:58
*** suresh12 has joined #openstack-neutron12:58
*** xuhanp has joined #openstack-neutron13:03
*** 32NABM7ZF has joined #openstack-neutron13:03
*** pal_ has joined #openstack-neutron13:04
*** jaypipes is now known as leakypipes13:05
*** kbringard has joined #openstack-neutron13:13
*** neeti has quit IRC13:16
*** pal_ has quit IRC13:16
*** nplanel_ has joined #openstack-neutron13:21
*** shwetaap has joined #openstack-neutron13:22
*** lykinsbd has joined #openstack-neutron13:23
*** annegentle has joined #openstack-neutron13:24
*** baoli has quit IRC13:25
*** atuvenie has quit IRC13:27
*** ihrachyshka has quit IRC13:27
*** shwetaap1 has joined #openstack-neutron13:27
*** ihrachyshka has joined #openstack-neutron13:27
*** baoli has joined #openstack-neutron13:28
*** ilyashakhat has quit IRC13:28
*** annegentle has quit IRC13:29
*** shwetaap has quit IRC13:30
*** julien_ZTE has joined #openstack-neutron13:30
*** chem has joined #openstack-neutron13:30
*** ishant has quit IRC13:31
*** chandankumar has quit IRC13:31
*** jckasper has joined #openstack-neutron13:32
*** matrohon has joined #openstack-neutron13:32
*** carlp has quit IRC13:34
*** jpena is now known as jpena|lunch13:39
*** yamamoto has joined #openstack-neutron13:40
*** suresh12 has quit IRC13:41
*** tiswanso has joined #openstack-neutron13:41
*** tiswanso has quit IRC13:42
*** tiswanso has joined #openstack-neutron13:42
*** annegentle has joined #openstack-neutron13:43
openstackgerritPavel Bondar proposed openstack/neutron: IPAM reference driver  https://review.openstack.org/15048513:44
*** shwetaap1 has quit IRC13:45
*** yamamoto has quit IRC13:46
*** apuimedo has joined #openstack-neutron13:47
*** banix has joined #openstack-neutron13:47
*** dims_ is now known as dimsum__13:49
openstackgerritPavel Bondar proposed openstack/neutron: Implement IPAM Driver loader  https://review.openstack.org/14747913:50
*** f13o has joined #openstack-neutron13:50
*** padkrish has joined #openstack-neutron13:55
*** ihrachyshka has quit IRC13:56
*** flaviof has quit IRC13:57
*** ihrachyshka has joined #openstack-neutron13:57
*** padkrish has quit IRC13:57
*** r-daneel has joined #openstack-neutron13:58
*** padkrish has joined #openstack-neutron13:58
*** suresh12 has joined #openstack-neutron13:58
*** amuller_ is now known as amuller_afk13:58
*** salv-orl_ has joined #openstack-neutron14:00
*** ferest has quit IRC14:00
*** flaviof has joined #openstack-neutron14:01
*** dimsum__ has quit IRC14:01
*** dimsum__ has joined #openstack-neutron14:02
*** tiswanso has quit IRC14:02
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323614:03
*** salv-orlando has quit IRC14:03
*** signed8bit has joined #openstack-neutron14:04
*** Marga_ has joined #openstack-neutron14:06
*** tiswanso has joined #openstack-neutron14:06
*** tiswanso has quit IRC14:06
*** tiswanso has joined #openstack-neutron14:06
*** annegentle has quit IRC14:07
*** _cjones_ has joined #openstack-neutron14:08
*** Marga_ has quit IRC14:08
*** saikrishna has joined #openstack-neutron14:09
*** Marga_ has joined #openstack-neutron14:09
*** shwetaap has joined #openstack-neutron14:09
*** baoli has quit IRC14:10
*** _cjones_ has quit IRC14:11
*** ianw has quit IRC14:12
*** ianw has joined #openstack-neutron14:12
*** suresh12 has quit IRC14:13
*** popw has quit IRC14:17
*** chandankumar has joined #openstack-neutron14:19
*** zzzeek has joined #openstack-neutron14:19
*** kodoku has joined #openstack-neutron14:20
kodokuHi, How I can configure neutron server for connect to keystone with HTTPS and with insecure option ?14:21
*** ianw has quit IRC14:21
*** ianw has joined #openstack-neutron14:21
kodokubecause I have issue " HTTP connection exception: Unable to establish connection to https://10.121.141.35:35357/"14:21
*** saikrishna has quit IRC14:27
openstackgerritAndrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports  https://review.openstack.org/15636014:28
openstackgerritKyle Mestery proposed openstack/neutron-specs: Cleanup Kilo specs  https://review.openstack.org/16835114:28
*** joshw42 has joined #openstack-neutron14:28
*** prad has joined #openstack-neutron14:29
*** kodoku has left #openstack-neutron14:29
*** bobmel_ has joined #openstack-neutron14:30
*** Networkn3rd has joined #openstack-neutron14:31
*** chandankumar has quit IRC14:31
*** annegentle has joined #openstack-neutron14:32
*** stockpirate has quit IRC14:32
*** bobmel has quit IRC14:33
*** reed has joined #openstack-neutron14:33
*** achanda has joined #openstack-neutron14:35
*** 32NABM7ZF has quit IRC14:35
*** xuhanp has quit IRC14:35
openstackgerritAndrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port  https://review.openstack.org/16778414:37
*** chandankumar has joined #openstack-neutron14:38
*** Rajeev has joined #openstack-neutron14:39
*** wznoinsk has quit IRC14:39
*** chem has quit IRC14:40
*** leenheer has joined #openstack-neutron14:41
*** zz_jgrimm is now known as jgrimm14:41
*** lizk has quit IRC14:42
*** achanda has quit IRC14:42
*** _cjones_ has joined #openstack-neutron14:43
*** wznoinsk has joined #openstack-neutron14:44
*** banix has quit IRC14:47
*** dansmith is now known as superdan14:47
pc_mdougwig: ping14:48
*** banix has joined #openstack-neutron14:48
*** rangansw has joined #openstack-neutron14:50
openstackgerritZengfa Gao proposed openstack/neutron: Add API tests for subnetpool allocation  https://review.openstack.org/16761114:52
*** svinota|work has joined #openstack-neutron14:52
*** aranjan has joined #openstack-neutron14:53
*** emagana has joined #openstack-neutron14:54
*** gsagie has joined #openstack-neutron14:54
*** jpena|lunch is now known as jpena14:55
*** Bhavya has quit IRC14:56
*** Marga_ has quit IRC14:57
*** aranjan has quit IRC14:57
*** _cjones_ has quit IRC14:58
*** bquan has joined #openstack-neutron14:58
*** _cjones_ has joined #openstack-neutron14:59
*** carl_baldwin has joined #openstack-neutron15:01
*** Rajeev has quit IRC15:02
*** bquan_ has joined #openstack-neutron15:02
*** doude has quit IRC15:04
*** annegentle has quit IRC15:04
*** dsneddon has joined #openstack-neutron15:05
*** annegentle has joined #openstack-neutron15:05
*** bquan has quit IRC15:05
*** julim has quit IRC15:07
*** julim has joined #openstack-neutron15:08
*** chem has joined #openstack-neutron15:09
*** devlaps has joined #openstack-neutron15:10
*** marun has joined #openstack-neutron15:10
*** atuvenie has joined #openstack-neutron15:12
*** julim has quit IRC15:13
*** dannywilson has joined #openstack-neutron15:14
*** julim has joined #openstack-neutron15:14
*** dannywilson has quit IRC15:14
*** ZZelle has quit IRC15:14
*** dannywilson has joined #openstack-neutron15:14
*** reed has quit IRC15:14
*** marun has quit IRC15:15
*** sbelous has quit IRC15:15
*** xgerman_ has joined #openstack-neutron15:16
*** amuller_afk is now known as amuller15:17
*** 17WABICRT has quit IRC15:20
*** chandankumar has quit IRC15:20
*** ggnel has quit IRC15:20
amullermestery: About https://review.openstack.org/#/c/164208/15:20
*** fifieldt has quit IRC15:21
mesteryamuller: Looking15:21
amullerIt looks like there's some confusion there, maybe some of it could be resolved if you split it up to 2 patches, one that enhances the text / captures what's currently going on, and the other patch is just the rename15:21
amullermestery: you're getting -1's for describing reality...15:21
amullerwhich is ridiculous15:21
mestery:)15:21
mesteryamuller: To be honest, I'd really like to converge with current neutron folks and mege it WITH those -1s15:22
amulleryeah I get that15:22
mesteryBecause I find it hilarious that people not involved with neutron, and who SUPPORTED AND ARCHITECTED THE BIG TENT MODEL, are complaining about this change15:22
mesteryPFft15:22
amullermestery: People are so afraid to make changes because 'other projects are not doing it', someone has to lead...15:23
mesteryamuller: Yes! We've done that as a team (neutron) a lot recently, so I have no fear :)15:23
mesteryamuller: We're in a much better place than we were last year for example.15:23
amullerIt's like a bunch of people looking left and right before crossing the street, but nobody crossing because no one else is15:23
mesteryamuller: Good analogy15:23
*** suresh12 has joined #openstack-neutron15:24
*** _cjones_ has quit IRC15:24
*** ajo has joined #openstack-neutron15:25
*** armax has joined #openstack-neutron15:28
*** suresh12 has quit IRC15:28
openstackgerritZengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool  https://review.openstack.org/16837115:29
*** suresh12 has joined #openstack-neutron15:29
*** garyk has joined #openstack-neutron15:29
*** baoli has joined #openstack-neutron15:30
*** baoli has quit IRC15:31
pc_mamuller: Hi. Can you look at https://review.openstack.org/#/c/164466/ again? It has what we talked about before, and a change to mock, based on what armax wanted.15:31
pc_mdougwig: ping15:32
*** rook_ has quit IRC15:33
amullerpc_m: sure15:33
pc_mamuller: Thanks. It has a -1 from doug, who had a question, but didn't put in the question. I'm trying to ping him to see what's up.15:34
*** xgerman has joined #openstack-neutron15:34
*** baoli has joined #openstack-neutron15:35
amullerpc_m: +115:36
pc_mamuller: Thanks!15:36
*** xgerman_ has quit IRC15:38
*** padkrish has quit IRC15:38
*** Shrews has joined #openstack-neutron15:39
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP)  https://review.openstack.org/16811515:39
*** zhhuabj has joined #openstack-neutron15:40
*** stockpirate has joined #openstack-neutron15:40
stockpirateHi, could i come with a question, again?15:42
stockpirateask*15:42
ShrewsHello all. I'm am having a difficult time constructing the proper JSON body to update a router through the python API. The documentation is not very helpful here. I have tried these JSON bodies: http://paste.openstack.org/show/197181/15:46
Shrewscan anyone shed some light on where I've gone wrong?15:46
*** vthapar has quit IRC15:48
ShrewsI would use the --debug option to the command line client to see what it constructs, but I can't figure out how to specify values with the router-update command.15:49
*** _cjones_ has joined #openstack-neutron15:50
*** dboik_ has joined #openstack-neutron15:51
*** Rajeev has joined #openstack-neutron15:52
ihrachyshkabaoli, fyi sent dibbler package for fedora review. now waiting for comments and merge. I guess next week is when we see it.15:52
ihrachyshkabaoli, which fedora versions do we need for the package?15:53
*** dboik has quit IRC15:54
*** lykinsbd_ has joined #openstack-neutron15:56
*** lykinsbd has quit IRC15:56
*** chachajee has joined #openstack-neutron15:56
*** dboik_ has quit IRC15:57
*** scheuran has quit IRC15:57
*** dboik has joined #openstack-neutron15:57
*** tsg has joined #openstack-neutron15:57
*** lykinsbd_ has quit IRC15:58
*** chachaji has quit IRC15:58
*** lykinsbd has joined #openstack-neutron15:58
*** YorikSar has quit IRC15:59
john-davidgeihrachyshka: That's great! Thanks. I imagine 20 and 21 will be sufficient15:59
ihrachyshkajohn-davidge, ack16:00
ihrachyshkajohn-davidge, that's basically all that is supported :)16:00
john-davidgeihrachyshka: :)16:00
ihrachyshkaI don't think there will be issues with f20 though, so fine16:00
baoliihrachyshka, thanks for doing the packaging.16:00
flcnmestery , emagana  Me, with haleyb finish review of https://review.openstack.org/#/c/143672/  Can you please review it one more time? Thanks16:00
ShrewsAh, solved my own problem. Nevermind  :)16:01
dougwigpc_m: ack16:01
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961816:01
*** _cjones_ has quit IRC16:01
*** _cjones_ has joined #openstack-neutron16:02
*** bjornar_ has quit IRC16:02
mesteryflcn: ack16:02
pc_mdougwig: You gave a -1 on https://review.openstack.org/#/c/164466/ and had a question, but the body of the question was blank...16:03
*** lykinsbd has quit IRC16:03
dougwigoh nuts, gertty ate my comment.  one sec.16:03
*** carl_baldwin has quit IRC16:03
*** nlahouti has joined #openstack-neutron16:04
dougwigre-commented.16:04
pc_mdougwig: thanks. will look16:04
*** thedodd has joined #openstack-neutron16:04
amullerdougwig: your question is answered in the commit message I think16:05
amullerdougwig: the L3 uses an event notification system for the metadata proxy, vpnaas and fwaas16:05
dougwigamuller: ahh, so that first line is the "old" mechanism?16:05
pc_mdougwig: YEah, essentially, we want to do a replacement, but because other repos use, we need to do an add then remove.16:05
amullerdougwig: yep16:05
dougwigok, got it.16:05
*** annegentle has quit IRC16:06
amullerarmax: pc_m: Armando has a point though, maybe we want to release K with all systems using the old system, then make a complete switch during L16:06
amullerpc_m: Unless there's an urgency here that I'm missing?16:06
dougwigarmax: i'm in favor of merging that refactor if we think it's low risk enough, because of the point made that if it merges early in liberty, since it's some structural change, it'll make bug fixing backports harder.16:07
emaganaflcn: Done!16:07
dougwigdepends on risk level.16:07
*** anilvenkata has quit IRC16:07
pc_mamuller: no urgency. Vyatta's out of tree drivers are waiting for this so they can update their driver, and it is pretty much done.16:08
*** Marga_ has joined #openstack-neutron16:09
*** atuvenie has quit IRC16:09
*** csoukup has joined #openstack-neutron16:09
*** suresh12 has quit IRC16:09
*** chandankumar has joined #openstack-neutron16:09
pc_mdougwig: right, backports wouldn't have to deal with the change.16:09
*** numan has quit IRC16:10
*** ChuckC has joined #openstack-neutron16:11
*** SridharG has left #openstack-neutron16:11
flcnemagana Thanks!16:13
emaganaflcn: n.p.16:13
emaganaflcn: Thanks for contributing to the code!16:13
amullerpc_m: I'd wait until L16:13
*** dave-mccowan has joined #openstack-neutron16:14
flcnemagana thanks for help!16:14
*** Miouge has quit IRC16:14
*** dave-mcc_ has joined #openstack-neutron16:15
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961816:16
*** prad has quit IRC16:17
*** Swami has joined #openstack-neutron16:17
*** padkrish has joined #openstack-neutron16:19
*** dave-mccowan has quit IRC16:19
openstackgerritMerged openstack/neutron: If configured, set the MTU for fpr/rfp intefaces  https://review.openstack.org/16231216:19
*** yeungp has joined #openstack-neutron16:19
openstackgerritShaoRan Xiao proposed openstack/neutron: Definition of the Virtual Network Performance interface Implements:blueprint virtual-network-performance-monitor  https://review.openstack.org/16481416:22
*** crc32 has joined #openstack-neutron16:22
*** aepifanov has quit IRC16:22
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool  https://review.openstack.org/16591016:24
*** leenheer has quit IRC16:25
*** tidwellr has joined #openstack-neutron16:25
stockpiratehmm need some help to understand. when im launch instance from miage in horizon i get http 500 error. And when i do:16:27
stockpirateroot@vitkencas03:/etc/neutron# nova secgroup-list16:27
stockpirateERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-d182fd73-6a33-498b-8398-60a2e47be01f)16:27
stockpiratei get exatly the same16:27
*** ijw has joined #openstack-neutron16:27
stockpiratenova-api.log ConnectionFailed: Connection to neutron failed: Maximum attempts reached16:27
*** ijw has quit IRC16:28
*** pal_ has joined #openstack-neutron16:28
openstackgerritZengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API  https://review.openstack.org/15961816:28
openstackgerritCarl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes  https://review.openstack.org/16322216:28
*** ijw has joined #openstack-neutron16:28
*** carl_baldwin has joined #openstack-neutron16:28
*** claudiub has quit IRC16:29
*** aranjan has joined #openstack-neutron16:29
carl_baldwinamuller:  https://review.openstack.org/#/c/163222/ was an interesting conflict resolution after yours merged.  I had to get a little creative.16:32
*** rangansw has quit IRC16:32
carl_baldwinamuller: I attempted to isolate state_change_callback to HA (mostly).  It is passed as an extra parameter to HaRouter when it is created.16:33
*** chem has quit IRC16:33
*** tidwellr has quit IRC16:33
*** tidwellr1 has joined #openstack-neutron16:33
*** chem has joined #openstack-neutron16:34
openstackgerritPavel Bondar proposed openstack/neutron: Refactor db_base_plugin_v2.py to use new IPAM  https://review.openstack.org/15323616:34
amullercarl_baldwin: yeah I imagined it'd be a rough merge conflict16:35
amullersorry about that =p16:35
*** SumitNaiksatam has joined #openstack-neutron16:35
*** fredlhsu has joined #openstack-neutron16:36
carl_baldwinamuller, no worries.  I just had to deal with that callback and the process_monitor.  I think what I've done better isolates that stuff to HA.16:36
openstackgerritAbishek Subramanian proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers  https://review.openstack.org/14906816:37
*** ronis has quit IRC16:38
*** suresh12 has joined #openstack-neutron16:39
*** tsg has quit IRC16:39
*** alaski is now known as lascii16:39
*** afazekas has quit IRC16:40
carl_baldwinHenryG: ping16:40
HenryGcarl_baldwin: pong16:40
amullercarl_baldwin: I'll have to look at that patch closely16:40
amullercarl_baldwin: ... after lunch :)16:40
carl_baldwinHenryG:  How are we doing on https://review.openstack.org/#/c/149068/ ?16:41
carl_baldwinamuller:  Thanks.16:41
HenryGcarl_baldwin: dane_leblanc and dboik have figured out all the tempest failures and addressed them16:42
HenryGcarl_baldwin: I'll add a comment with a summary16:43
*** gsagie has quit IRC16:44
*** yamamoto has joined #openstack-neutron16:45
carl_baldwinHenryG, thanks.  I'll watch for it.  Looks like tests are running on it too.16:47
*** toutuu has joined #openstack-neutron16:48
toutuugood evening to everyone! o/16:48
toutuuI have a quick question: what is physically an openstack router? is it a little VM? is it a collection of iptables rules?16:49
*** yamamoto has quit IRC16:49
*** alexpilotti has joined #openstack-neutron16:50
*** ygbo has quit IRC16:53
*** stockpirate has quit IRC16:54
carl_baldwintoutuu:  In the reference L3 implementation, the router is created in a Linux network namespace to isolate routing rules, conntrack tables, iptables rules, etc.16:55
carl_baldwintoutuu, this is closely related to container (e.g. LXC, docker) networking and is not a full VM.16:56
*** marun has joined #openstack-neutron16:58
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: don't rely on configuration files outside tests directory  https://review.openstack.org/16523716:58
HenryGcarl_baldwin: comment added17:00
*** matrohon has quit IRC17:00
*** signed8b_ has joined #openstack-neutron17:01
*** signed8bit has quit IRC17:02
*** seizadi has joined #openstack-neutron17:03
*** Marga_ has quit IRC17:03
openstackgerritAndrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports  https://review.openstack.org/15636017:03
carl_baldwinHenryG, ack17:03
openstackgerritAndrew Boik proposed openstack/neutron: Allow update of ext gateway IP's w/out port delete  https://review.openstack.org/16107317:04
openstackgerritAndrew Boik proposed openstack/neutron: Auto-update gateway port after subnet-create  https://review.openstack.org/16108517:04
*** annegentle has joined #openstack-neutron17:04
*** toutuu has quit IRC17:05
carl_baldwinHenryG, just posted my review.17:06
*** jlanoux has quit IRC17:06
sc68calmestery: sorry to ask you dumb questions, but my latest e-mail I notice that both mechanism drivers for openvswitch and linuxbridge are enabled in devstack by default, but the openvswitch agent is only selected to run. Does the linuxbridge agent need to be running if you want both ovs and linuxbridge?17:07
sc68calor is there something that would prevent the ovs agent and lb agent from both running?17:07
HenryGcarl_baldwin: much appreciated!17:07
*** yamahata has joined #openstack-neutron17:07
HenryGcarl_baldwin: anything left around l3agent refactoring you need eyes on?17:08
amullersc68cal: you'd run only either ovs or the lb agent on a single machine17:08
*** _cjones_ has quit IRC17:08
amullersc68cal: and having both lb and ovs mech drivers enable on your server enables you to run a mixed environment where each node would run either/or17:08
sc68calamuller: ah....17:08
*** seizadi has quit IRC17:09
*** seizadi has joined #openstack-neutron17:09
amullermestery: Are you seeing the discussion about devstack switching from OVS to LB for its default settings?17:10
amullermestery: "Status of the nova-network to Neutron migration work"17:11
amullermarun: ^17:11
amullerI made the point that we can't make that switch even if we wanted to without gating on LB17:11
amullerbut that was ignored :)17:11
*** lpetrut has left #openstack-neutron17:11
marunwho is doing the ignoring?17:11
*** nplanel_ has quit IRC17:12
*** s3wong has joined #openstack-neutron17:12
*** Aish has joined #openstack-neutron17:12
amullerTo be fair Sean Dague did not reply since I made that comment17:13
amullerSo really I'm just blowing smoke17:13
amullerbut anyway, wanted you to be aware of that dicussion17:13
*** jistr has quit IRC17:13
*** seizadi has quit IRC17:13
*** seizadi has joined #openstack-neutron17:13
marunIt's pretty embarrassing that even openstack devs seem to see ovs as scary.17:14
openstackgerritEd Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets  https://review.openstack.org/16359417:14
sc68calLet's first see how bad it explodes - https://review.openstack.org/#/c/168423/117:15
amullermarun: also they were talking about simplifying, using provider networks, well, you can do that with ovs17:15
marunurl 117:15
marunamuller: having 2 code paths to have to exercise is, frankly asinine17:15
marunwow, that comma makes that sentence all kinds of awkward17:16
russellbovs seems fairly mature at this point17:16
russellbnot sure why it's so scary17:16
marunrussellb: the unknown17:16
russellbi also wish the conversation would focus more on concrete gaps people see17:16
*** achanda has joined #openstack-neutron17:16
russellbso far it's just "too complicated"17:16
amullerthe gap that OVS has is that its not LB :)17:16
marunto be fair, debugging ovs does suck17:16
*** achanda has quit IRC17:16
russellbwhich is legit i suppose, assuming it's really just a docs issue ...17:16
marunrussellb: it kind of is, yeah.17:17
marunrussellb: lb is simple, at least17:17
* russellb nods17:17
*** achanda has joined #openstack-neutron17:17
marunovs was designed with a controller that could provide a birds-eye view of the world17:17
sc68calCheck the notes from the ops summit - https://etherpad.openstack.org/p/PHL-ops-OVS17:17
*** dboik_ has joined #openstack-neutron17:17
sc68calpeople want distributed router for linxubridge.... :-\17:17
marunin the absence of that controller, the complexity is hard to grapple with compared with lb17:17
marunsc68cal: also, pony ;)17:17
amullerDVR for LB will never happen17:18
sc68calmarun: pony with a horn17:18
russellbi think a lot of the complains on that etherpad are very ovs+neuron, not ovs itself17:18
marunheh17:18
*** rushil has quit IRC17:19
sc68calPart of it is knowledge gap - Matt (Sam-I-Am) and others have been trying to fix that with the new networking guide17:19
russellbsc68cal: yes17:19
sc68calso that people are less scared of OVS17:19
mesterymarun amuller: I'm burned out of arguing with people who don't want to broaden their horizons17:19
sdaguerussellb: too complex is valid. If you have learn a ton of new stuff all at once, that's a problem. Much like breaking up patch series into digestable chunks, these things have to be in digestable deploy chunks.17:19
mesteryI mean, LB isn't tested at all with neutron right now17:20
mesteryTo say we're going to use that is scary at best17:20
russellbsdague: yep, just worried there's more that isn't surfaced17:20
amullermestery: sc68cal pushed a devstack patch to change the default to LB. Let's see what happens17:20
*** dboik has quit IRC17:20
amuller(re: testing)17:20
*** bjornar_ has joined #openstack-neutron17:21
mesteryamuller: Again, is that tested anywhere?17:21
sc68calIf it explodes, that's a good data point to say look it's not feasible17:21
mesteryamuller: Because if it's not, we've just created a whole new mess for ourselves17:21
mesterymarun: ^^^^17:21
sdaguerussellb: if you've mentally stack overflowed people in looking at it, you aren't going to get good feedback17:21
*** leenheer has joined #openstack-neutron17:21
*** ijw has quit IRC17:22
sc68calOne thing to ask - if we aren't testing linuxbridge, why do we have it in tree?17:22
*** apuimedo has quit IRC17:22
sc68calI mean we've thrown vendors out for having no CI17:22
marunsc68cal: we've been asking the same question for a whiel17:22
russellbovs has matured *a lot* in the last few years i'd say17:22
russellbdoesn't make as much sense anymore17:23
mesterymarun: ++17:23
mesteryrussellb: Right17:23
sc68calSo is this an education issue? Everyone thinks "linuxbridge is simple, how could it possibly not work?"17:23
sdaguerussellb: that's kind of irrelevant17:23
russellb?17:23
mesterysdague: How so?17:23
*** saikrishna has joined #openstack-neutron17:23
marunI don't think we can afford to target both ovs and lb as well-tested solutions upstream.17:24
sdagueyou still have to understand the ovs model to do work completely unrelated to networking when things go wrong17:24
marunWe need a single good path17:24
marunand ovs covers more use-cases17:24
openstackgerritCarl Baldwin proposed openstack/neutron: Honor the default ipv6 pool configuration setting  https://review.openstack.org/16809817:24
openstackgerritMerged openstack/neutron: Don't eagerly load ranges from IPAllocationPool  https://review.openstack.org/16821417:24
marunSo regardless of folks concern, unless they're willing to do yeoman's work in maintaining lb, I'm not sure we should keep supporting it directly.17:24
maruni.e. it can be maintained outside of the tree by folks that care about it17:25
sdagueI realize I'm annoying tons of people, but this is the best compromise out there that I've managed to come up with. If someone wants to champion different approaches feel free17:25
sdaguebut everything else has been face plant here17:25
marunsdague: we can't make everyone happy17:25
sdagueI think this is a path to actual n-net deprecation17:25
marunsdague: we fail hard when we try17:26
dougwigsdague: are you saying we can't make OVS simple enough in the single box case for anyone to ever accept it?17:26
sdaguedougwig: yep17:26
sdagueand that many large deployers were reverting from ovs -> linuxbridge because of per reasons17:27
sdaguewhich came up at phili ops meetup17:27
dougwigsdague: that strikes me as odd, since when i started with devstack, i had no clue what underlying network packages were at play, and it didn't matter.  what is the use case stopper with OVS?17:27
sdagues/per/performance/17:27
*** jpena has quit IRC17:27
maruncan we please quantify 'many large deployers'?17:27
*** zilberstein has quit IRC17:27
sdaguepeople that spoke up17:28
marunsee -> we can't please everyone17:28
marunwe need to pick our battles on objective criteriea17:28
maruncriteria17:28
*** yamahata has quit IRC17:28
dougwigok, so performance of large delpyments is a reason.  seems like a reason for those deployers to help support linuxbridge.  but ... how does that affect the devstack issue?17:28
sdaguesure, so we have this plan and are executing it. Someone else wants to actually work on solving this problem, please do.17:28
openstackgerritMaru Newby proposed openstack/neutron: Simplify retargetable test framework  https://review.openstack.org/16707517:29
*** s3wong has quit IRC17:30
mesterySo, to me, if we are going to get LB in there a bit more, we really need to see it tested a bit more, that's the only thing that struck me in that thread.17:30
*** pal_ has quit IRC17:30
*** ijw has joined #openstack-neutron17:30
sdaguemestery: sure, testing was assumed17:30
sdaguesorry, I didn't think I needed to call that out explicitly17:31
mesterysdague: What are infras thoughts on testing LB and OVS in the gate? It may increase our jobs. /cc marun17:31
*** s3wong has joined #openstack-neutron17:31
amullerSo we start finding issues specific to the LB agent, who's going to fix those issues? What justification is there to spending more of our time fixing the LB agent when the OVS agent is more than twice as popular?17:31
*** gsagie has joined #openstack-neutron17:32
mesteryamuller: I would hope the operators who want LB would dedicate some resources to it's maintenance17:32
marunmestery: double the number of jobs?  If not, we may have a disparity in the amount of testing and a consequent lack of confidence in the one that gets less attention.17:32
*** suresh12 has quit IRC17:32
dougwigsdague: the etherpad lists deployment/performance issues.  that's certainly valid.  but back to devstack, the only relevant item is "When it breaks. Not if, when.".  any chance that can be characterized more?  i haven't had that experience.17:32
marunsee: postgres17:32
*** safchain has quit IRC17:33
*** tflynn has joined #openstack-neutron17:33
mesteryI had hoped as part of documenting the nova-net/neutron gap we could show how OVS might fit in here too, my only concern is around mainting the LB implementatino17:33
sdaguedougwig: devstack at random code points breaks all the time, and you have to go poke at the system to figure out what's going on17:33
mesteryHowever, given we're planning to spin out the in-tree reference implementation, maybe this is all moot anyways17:33
russellbmestery: yeah, part of the gap plan was supposed to be writing these use-case mapping docs17:34
*** gongysh_ has joined #openstack-neutron17:34
*** gongysh_ has quit IRC17:34
dougwigsdague: right, i'm not disputing that folks are seeing issues.  i was hoping for some specifics around 'breaks all the time'.17:35
*** Marga_ has joined #openstack-neutron17:35
*** Networkn3rd has quit IRC17:36
*** suresh12 has joined #openstack-neutron17:37
sdaguedougwig: well I'll appologize for not having every scenario in my head at this point. I've spent a bunch of time trying to synthesize information coming in and put it into something digestable and with a path forward.17:38
*** dave-mcc_ has quit IRC17:38
*** seizadi has quit IRC17:39
mesteryMaybe the plan is we see the current state of LB agent, and try to map the (hopefully) easy use cases of nova-net into it, and then try to migrate folks to OVS?17:39
mesteryToo many steps?17:39
mesteryToo complicated?17:39
mesteryThoughts?17:39
*** ajmiller has joined #openstack-neutron17:39
sc68calmestery: I think that's the plan17:39
openstackgerritRyan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas  https://review.openstack.org/16526417:39
*** shwetaap has quit IRC17:40
sc68caluse linuxbridge to help ease transition from nova-net17:40
sc68cal(if it works)17:40
mesteryAnd make it work for the use cases nova-net had17:40
mesterynot all the neutron cases17:40
sdaguemestery: correct17:40
*** Networkn3rd has joined #openstack-neutron17:40
sdagueyou saw my mixins email right17:40
mesteryThat seems like a reasonable thing to do perhaps17:40
*** YorikSar has joined #openstack-neutron17:40
mesteryYEah, slowly catching up17:41
sdagueif you want the complicated bits, you need to move to ovs17:41
sdaguebut the base starting point is lb, because it's simple17:41
openstackgerritMaru Newby proposed openstack/neutron: Remove neutron.tests.sub_base  https://review.openstack.org/16843017:41
dougwigsdague: that's great, and i appreciate it.  i was just looking for if there was some feedback we could use to improve those issues.  i'll admit to never having used LB with neutron, simply because devstack always installed OVS, and it's what I learned that way, and have grown to like it's flexibility, even if i railed at it a bit at first.  though i've17:41
dougwigrailed at most of openstack at one point or another.17:41
*** aepifanov has joined #openstack-neutron17:41
sdaguethen you can get used to neutron services, how and/when they fail, ensure they are working, get comfortable17:41
sdaguethen you flip a config to introduce ovs if you want to17:42
sdaguenow you know if it's ovs and your environment going odd, as that change is bisectable17:42
sdaguesame thing with tenant self service networks later17:42
*** sridhar_ram has joined #openstack-neutron17:43
*** banix has quit IRC17:43
*** _cjones_ has joined #openstack-neutron17:44
*** SumitNaiksatam has quit IRC17:44
*** amotoki has quit IRC17:45
*** catohornet has joined #openstack-neutron17:45
*** sbfox has joined #openstack-neutron17:46
openstackgerritMaru Newby proposed openstack/neutron: Simplify retargetable test framework  https://review.openstack.org/16707517:47
openstackgerritMichael Smith proposed openstack/neutron: L3 Agent support for routers with HA and DVR  https://review.openstack.org/13968617:48
*** mwagner_lap has quit IRC17:49
openstackgerritMaru Newby proposed openstack/neutron: Move API tests to neutron.test.api  https://review.openstack.org/16732017:51
*** seizadi has joined #openstack-neutron17:51
*** gsagie has quit IRC17:53
*** john-davidge has quit IRC17:53
*** jprovazn has quit IRC17:53
*** pritesh has joined #openstack-neutron17:54
*** Rajeev has quit IRC17:54
*** ronis has joined #openstack-neutron17:55
openstackgerritMichael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy  https://review.openstack.org/15175817:56
*** yamahata has joined #openstack-neutron17:57
openstackgerritKyle Mestery proposed openstack/neutron: Add language around re-proposing specs for new releases  https://review.openstack.org/16843418:01
*** Networkn_ has joined #openstack-neutron18:01
*** saikrishna has quit IRC18:01
*** sridhar_ram1 has joined #openstack-neutron18:02
*** sridhar_ram has quit IRC18:04
openstackgerritMaru Newby proposed openstack/neutron: Remove neutron.tests.sub_base  https://review.openstack.org/16843018:05
flcnmestery Can you please review http://review.openstack.org/#/c/143672/ one more time? I fix all small problems18:05
*** Networkn3rd has quit IRC18:05
*** leenheer has quit IRC18:06
*** leenheer has joined #openstack-neutron18:07
mesteryflcn: Looking now.18:07
*** nlahouti has quit IRC18:10
mesterymarun: Do we do any testing of the metering agent in the gate at all? I know it runs, but any idea what tests we do?18:10
mesteryThis is in relation to this one: http://review.openstack.org/#/c/143672/18:10
marunmestery: unsure18:10
mesterymarun: Just trying to understand what our test coverage is at this point.18:10
mesterymarun: YEah, me too18:10
marunmestery: It looks like unit test only18:11
mesterymarun: That's what I was afraid of. I guess we can add that to what else we should look at adding more testing to in Liberty.18:11
marunmestery: who's crazy enough to use the metering agent, anyway?18:12
mesterymarun: o_O18:12
*** rotbart has quit IRC18:12
marunmestery: it's not like it has any guarantee against failure18:12
mesterymarun: I hear you18:13
marunsee: ceilometer being an unmitigated disaster18:13
mesteryyup18:13
marunah, tgif :)18:14
mesteryrofl18:14
* sc68cal looks forward to when neutron is not the whipping boy anymore 18:15
amullermarun: I love how you use every opportunity to diss (To put it nicely...) Ceilometer18:15
marunamuller: it's such an easy target18:15
openstackgerritRyan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas  https://review.openstack.org/16526418:16
amullermarun: Maybe I should learn more about it so I can feel more comfortable dissing it18:16
amullerPretty noble reason to learn about something I think18:16
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Adds DVR functional test for multi-external networks  https://review.openstack.org/16357318:17
russellbi think uninformed dissing is quite reckless, indeed :)18:17
*** ijw has quit IRC18:17
mesteryamuller: lol18:17
*** MarceloAmaral has joined #openstack-neutron18:18
*** sridhar_ram1 has quit IRC18:19
MarceloAmaralSwami: hi18:19
MarceloAmaralSwami: I am interested in the ODL summer intership18:20
*** tidwellr1 has quit IRC18:20
*** shwetaap has joined #openstack-neutron18:20
*** sridhar_ram has joined #openstack-neutron18:20
amullerarmax: re: https://review.openstack.org/#/c/126182/ - I'm looking in to manipulating list_columns according to the response the client got from the server18:21
*** banix has joined #openstack-neutron18:22
Flcn_mestery thanks!18:22
SwamiMarceloAmaral: hi18:22
*** afazekas has joined #openstack-neutron18:23
*** seizadi has quit IRC18:23
*** seizadi has joined #openstack-neutron18:23
*** seizadi1 has joined #openstack-neutron18:24
*** seizadi has quit IRC18:24
openstackgerritMaru Newby proposed openstack/neutron: Move API tests to neutron.test.api  https://review.openstack.org/16732018:25
openstackgerritMaru Newby proposed openstack/neutron: Simplify retargetable test framework  https://review.openstack.org/16707518:25
*** svinota|work has quit IRC18:25
*** saikrishna has joined #openstack-neutron18:26
*** lukasa has quit IRC18:27
*** dkehn has quit IRC18:31
*** enikanorov has quit IRC18:32
*** Sukhdev has joined #openstack-neutron18:32
*** ZZelle has joined #openstack-neutron18:33
*** dane_leblanc has quit IRC18:33
openstackgerritCarl Baldwin proposed openstack/neutron: Implement default subnet pool configuration settings  https://review.openstack.org/16809818:33
*** dkehn has joined #openstack-neutron18:34
*** ijw has joined #openstack-neutron18:36
*** afazekas has quit IRC18:37
*** yatin has joined #openstack-neutron18:37
*** ijw_ has joined #openstack-neutron18:39
ZZellehi18:39
ZZelleit seems neutron-vpnaas code or tests are broken18:39
ZZellewho should i inform?18:39
*** seizadi has joined #openstack-neutron18:40
pc_mZZelle: me.18:40
*** seizadi1 has quit IRC18:40
pc_mZZelle: I see it already. I'm working on a patch18:40
ZZellepc_m, ok, great18:40
*** annegentle has quit IRC18:40
pc_mZZelle: I don't know what change caused the failure, and I don't know why the mock.patch is not working, but my thought for a solution is to mock it much higher (in VPN code), instead of in Neutron.18:41
*** annegentle has joined #openstack-neutron18:41
ZZellepc_m, it would be better as i would also reduce coupling with neutron18:41
pc_mZZelle: The code tries to mock neutron.agent.linux.utils.execute()18:41
pc_mZZelle: I don't know why that patch is no longer working.18:42
*** ijw has quit IRC18:42
pc_mZZelle: and only for the StrongSwan driver.18:42
*** smccully has joined #openstack-neutron18:43
*** lukasa has joined #openstack-neutron18:43
openstackgerritAndrew Boik proposed openstack/neutron: Allow only one IPv4 and one IPv6 fixed IP on router gateway port  https://review.openstack.org/16778418:43
*** crc32 has quit IRC18:46
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Revise functional test hooks (WIP)  https://review.openstack.org/16811518:47
*** yatin has quit IRC18:47
ZZellepc_m, you mock neutron.agent.common.utils.execute18:47
ZZellepc_m, you SHOULD mock neutron.agent.common.utils.execute18:47
*** ihrachyshka has quit IRC18:47
*** saikrishna has quit IRC18:48
pc_mZZelle: You mean it was doing neutron.agent.linux.utils.execute, right?18:48
pc_mZZelle: I was thinking that the conditional import in common.utils.py was not being handled as exepected, but decided to mock in VPN code instead.18:49
*** med_ has joined #openstack-neutron18:49
pc_mZZelle: neutron.agent.linux.utils.execute -> neutron.agent.common.utils.execute18:49
ZZellepc_m, yes18:50
pc_mZZelle: Just surprised me that it still didn't work. In any case, I'll do it in VPN code.18:50
*** SumitNaiksatam has joined #openstack-neutron18:50
ZZellepc_m, mock should "mock" the imported module which is neutron.agent.common.utils.execute18:51
pc_mZZelle: I'm testing now. Hope to push it out soon (trying to fix a few other things in the UT as well).18:51
ZZellepc_m, https://github.com/openstack/neutron/commit/f0d9410a8268e01369a43c5159621f3083855b5f#diff-6a3c59b7b71c12cc29eca4c85268864eL2218:51
openstackgerritAbishek Subramanian proposed openstack/neutron: Support IPv6 Router  https://review.openstack.org/15628318:53
pc_mZZelle: thanks!18:54
*** padkrish has quit IRC18:54
ZZellepc_m, it works?18:54
pc_mZZelle: I mean thanks for the link, I didn't know what the change was that altered this.18:54
*** padkrish has joined #openstack-neutron18:54
*** tsg has joined #openstack-neutron18:55
*** tsg_ has joined #openstack-neutron18:57
*** imcsk8 has quit IRC18:58
*** imcsk8 has joined #openstack-neutron18:58
*** tidwellr has joined #openstack-neutron18:58
openstackgerritAssaf Muller proposed openstack/python-neutronclient: Add HA router state to l3-agent-list-hosting-router  https://review.openstack.org/12618218:58
*** padkrish has quit IRC18:59
*** tsg has quit IRC19:00
*** fredlhsu has quit IRC19:02
*** sridhar_ram1 has joined #openstack-neutron19:03
*** seizadi has quit IRC19:03
*** seizadi1 has joined #openstack-neutron19:03
*** sridhar_ram has quit IRC19:04
*** dane_leblanc has joined #openstack-neutron19:05
*** seizadi1 has quit IRC19:05
*** seizadi has joined #openstack-neutron19:06
*** alexpilotti has quit IRC19:07
*** nlahouti has joined #openstack-neutron19:08
*** emagana has quit IRC19:08
*** chandankumar has quit IRC19:12
*** melwitt has joined #openstack-neutron19:14
*** Shrews has left #openstack-neutron19:15
openstackgerritEd Bak proposed openstack/neutron: Return from check_ports_exist_on_l3agent if no subnet found  https://review.openstack.org/15428919:17
openstackgerritMerged openstack/neutron: Send only one rule in queue on rule create/delete  https://review.openstack.org/14367219:17
*** virtuallynathan_ has quit IRC19:17
*** rangansw has joined #openstack-neutron19:19
*** afazekas has joined #openstack-neutron19:24
*** mwagner_lap has joined #openstack-neutron19:25
*** flcn has quit IRC19:28
*** flcn has joined #openstack-neutron19:30
*** afazekas has quit IRC19:32
*** igordcard_ has joined #openstack-neutron19:35
*** devlaps has quit IRC19:36
*** angela-s has joined #openstack-neutron19:39
*** emagana has joined #openstack-neutron19:41
*** kiran-r has joined #openstack-neutron19:41
*** leakypipes has quit IRC19:42
*** banix has quit IRC19:42
openstackgerritDoug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron  https://review.openstack.org/15473619:45
*** emagana has quit IRC19:46
openstackgerritDoug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron  https://review.openstack.org/15473619:47
openstackgerritDoug Wiegley proposed openstack/neutron-specs: Split neutron-lib out of neutron  https://review.openstack.org/15473619:47
*** yamamoto has joined #openstack-neutron19:47
*** kiran-r has quit IRC19:49
*** yamamoto has quit IRC19:52
*** shwetaap has quit IRC19:55
*** shwetaap has joined #openstack-neutron20:00
*** emagana has joined #openstack-neutron20:01
*** svinota|work has joined #openstack-neutron20:05
*** svinota|work has left #openstack-neutron20:05
*** baoli has quit IRC20:05
*** banix has joined #openstack-neutron20:07
*** afazekas has joined #openstack-neutron20:07
*** baoli has joined #openstack-neutron20:07
*** achanda has quit IRC20:08
*** aranjan has quit IRC20:09
*** matrohon has joined #openstack-neutron20:09
*** padkrish has joined #openstack-neutron20:10
*** fredlhsu has joined #openstack-neutron20:11
*** achanda has joined #openstack-neutron20:12
*** aranjan has joined #openstack-neutron20:14
*** ociuhandu has quit IRC20:16
openstackgerritZengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool  https://review.openstack.org/16837120:17
pc_mZZelle: ping20:18
ZZellepc_m, pong20:18
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS Fix unit test breakage  https://review.openstack.org/16847220:18
pc_mZZelle: I pushed up a simple change. Had a more extensive one, not using Neutron, but see other issues, so need more time to really fix that right.20:19
pc_mZZelle: ^^ is the simple fix.20:19
ZZellepc_m, no pb20:19
*** emagana has quit IRC20:19
*** padkrish has quit IRC20:19
pc_mZZelle: I need to head out for a bit, but will check back later tonight.20:19
*** Networkn_ has quit IRC20:20
pc_mdougwig: mestery: Can you review https://review.openstack.org/168472. Fixes unit test breakage.20:20
openstackgerritZengfa Gao proposed openstack/neutron: Add API tests for subnet-create with subnetpool  https://review.openstack.org/16837120:20
ZZellepc_m, could you remove your associated -1 on https://review.openstack.org/168169 ?20:20
*** baoli has quit IRC20:21
pc_mZZelle: sure20:21
*** pritesh has quit IRC20:26
*** pritesh_ has joined #openstack-neutron20:27
*** Sukhdev has quit IRC20:32
*** s3wong has quit IRC20:35
*** s3wong has joined #openstack-neutron20:36
*** lukasa has quit IRC20:37
*** MarceloAmaral has quit IRC20:37
*** lukasa has joined #openstack-neutron20:39
*** britthouser has quit IRC20:39
*** seizadi has quit IRC20:40
*** seizadi has joined #openstack-neutron20:40
openstackgerritHenry Gessau proposed openstack/neutron: Modify a different agent in test_update_agent_description  https://review.openstack.org/16821320:42
*** c_soukup has joined #openstack-neutron20:42
HenryGmarun: ^^ that should help with the api job stability20:43
*** achanda has quit IRC20:43
marunHenryG: has it been flaky?20:43
marunHenryG: and does that imply we shouldn't let it vote just yet?20:44
HenryGmarun: I only noticed it twice. Seems fairly rare. Details in the bug.20:44
*** raildo has quit IRC20:44
*** csoukup has quit IRC20:45
*** tiswanso has quit IRC20:45
openstackgerritDane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment  https://review.openstack.org/16494320:46
marunHenryG: Thanks!20:46
*** afazekas has quit IRC20:47
*** lukasa has quit IRC20:47
ZZelledougwig, mestery, hi20:48
mesteryZZelle: yo!20:48
ZZelledougwig, mestery, could you look at https://review.openstack.org/168164? This change avoids breaking neutron-fwaas after a rework on neutron test code20:49
*** rangansw has quit IRC20:49
*** emagana has joined #openstack-neutron20:50
*** achanda has joined #openstack-neutron20:50
mesteryZZelle: Nice work! I'll leave it to dougwig to +A20:51
ZZellemestery, there is the same change for vpnaas but i wait for pcm change merge20:51
mesteryZZelle: Thanks for fixing this! /cc pc_m20:52
ZZellemestery, we should find a way in liberty to better handle such cases through loose coupling or cross-repo tests20:53
*** emagana has quit IRC20:54
*** mfisch has quit IRC20:56
*** Guest90957 has joined #openstack-neutron20:58
-openstackstatus- NOTICE: Gerrit maintenance commences in 1 hour at 22:00 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html21:01
*** aranjan has quit IRC21:02
*** seizadi has quit IRC21:05
*** seizadi has joined #openstack-neutron21:05
openstackgerritDivya ChanneGowda proposed openstack/neutron: Update contribute.rst with Nuage plugin decomposition  https://review.openstack.org/16553621:06
*** achanda has quit IRC21:07
*** rkukura has quit IRC21:08
*** firl has joined #openstack-neutron21:08
firlanyone available to help me figure out where my problem is ? I am trying to create an external network and when I do I cannot ping on that net21:08
openstackgerritCarl Baldwin proposed openstack/neutron: Implement default subnet pool configuration settings  https://review.openstack.org/16809821:08
firlguess this should be in just the parent room of #openstack21:10
*** aepifanov has quit IRC21:10
*** annegentle has quit IRC21:10
*** padkrish has joined #openstack-neutron21:11
*** absubram has quit IRC21:11
*** absubram has joined #openstack-neutron21:12
*** chan1 has joined #openstack-neutron21:12
chan1hi! is there somebody here which could help me to understand something?21:13
chan1when im running: nova boot --flavor m1.tiny --image Udt01 --nic net-id=f10fa830-5bd2-455e-9c79-134421d76cb2   --security-group default --key-name demo-key demo-instance121:13
chan1the nova-api.log tells me: NoNetworksFound: No networks defined.21:14
chan1neutron net-list listening two networks.21:14
chan1but off course nova network-list is empty21:14
chan1my question is: Do i have to configure legacy networking also?21:16
*** bquan has joined #openstack-neutron21:16
*** pritesh_ has quit IRC21:17
chan1or is openstack networkin21:17
*** pritesh has joined #openstack-neutron21:17
chan1sorry.. **or is openstack networking (openstack) or legacy nova network like a "whether or"21:18
*** baoli has joined #openstack-neutron21:18
*** bquan_ has quit IRC21:19
chan1i read the doc: Proceed to network configuration for the example OpenStack Networking (neutron) or legacy networking (nova-network) architecture.21:20
chan1both is conflict??21:20
*** seizadi1 has joined #openstack-neutron21:20
*** Marga_ has quit IRC21:21
*** seizadi has quit IRC21:21
*** rkukura has joined #openstack-neutron21:24
*** dkehn has quit IRC21:24
chan1I can buy you a beer for an answer!!!21:24
*** yfauser has joined #openstack-neutron21:25
*** yfauser has left #openstack-neutron21:26
priteshchan1: you can't have nova-network and neutron running at same time as far as i know, but since you see the network under neutron net-list it means you have configured neutron, which means the nova-network is disabled.21:26
*** achanda has joined #openstack-neutron21:27
ZZellemarun,21:28
ZZellemarun, hi21:28
priteshchan1: here is the link to get it wokring for devstack : https://wiki.openstack.org/wiki/NeutronDevstack21:28
marunZZelle: hi21:28
ZZellemarun, about https://review.openstack.org/#/c/165866/7/neutron/tests/common/net_helpers.py21:28
marunZZelle: re: ports21:29
marunZZelle: removing an ovs bridge automatically removes ports21:29
openstackgerritMerged openstack/neutron: Use router state in get_ha_device_name and ha_network_added  https://review.openstack.org/16746221:29
chan1Pritesh! thank you!  i will remove nova-network from the node. ifconfig find a network card virbr021:29
chan1which is from nova-network21:30
openstackgerritAndrew Boik proposed openstack/neutron: Support multiple IPv6 prefixes on internal router ports  https://review.openstack.org/15636021:30
marunZZelle: so I don't see why fixture has to do it explicitly - it's unnecessary overhead21:30
marunZZelle: in the same way that manually removing rows in a table doesn't make sense if teardown removes the db21:30
marunZZelle: does that make sense?21:30
*** mlavalle has joined #openstack-neutron21:30
chan1Pritesh: nova-api-log says:  NoNetworksFound: No networks defined.21:31
ZZellemarun, but deleting a db ensures table delete, deleting a bridge does not ensure it deletes ports in general as you don't know the backend behavior21:31
marunZZelle: it does when we're talking about ovs21:31
marunZZelle: I'm not saying we never delete21:32
marunZZelle: I'm saying for ovs, we don't need to21:32
marunZZelle: For other bridge types, I'm not saying we don't have to delete21:32
ZZelleso if someone does with PortFixture(): it won't behave as expected21:32
marunZZelle: why not?21:32
ZZellebecause the port won't be deleted on port cleanup21:33
marunhmmm21:33
marunI guess I wasn't seeing that use-case represented in the patch in question21:33
priteshchan1: I am not sure why it says that but if you configure it correctly as per the link, you ca get devstack up and running with neutron.21:33
marunZZelle: Ok, that makes more sense. Is there a way to ensure that the cleanup is done only when used with 'with'?21:34
priteshchan1: you may have to tweak the localrc a bit, here is one posted by a user earlier: https://gist.github.com/amotoki/652074121:35
ZZellemarun, normally no, we must change __exit__ behavior21:35
openstackgerritCarl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes  https://review.openstack.org/16322221:35
*** baoli has quit IRC21:36
ZZellemarun, but it disallows explicit Fixture.cleanUp call21:36
*** baoli has joined #openstack-neutron21:36
*** angela-s has left #openstack-neutron21:37
marunZZelle: so either/or?21:37
chan1pritesh: thank you sir21:37
priteshnp21:37
openstackgerritCarl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes  https://review.openstack.org/16322221:38
ZZellemarun, it seems not possible if we want to support the 3 ways to use fixtures21:38
russellbgaryk: i started working on the ovn automation via devstack today, still more to go though ... https://github.com/russellb/networking-ovn/commit/171cae4b5930f73d12b98f01b99f54a6c5aec15221:39
marunZZelle: Honestly, I'm not entirely sure of the value of using 'with'21:39
marunZZelle: do you have examples in subsequent patches?21:40
ZZellemarun, ovs monitor  func tests perhaps21:40
openstackgerritCarl Baldwin proposed openstack/neutron: Move final remnants of router processing to router classes  https://review.openstack.org/16322221:40
marunZZelle: My question is whether there is a compelling reason to use 'with'21:41
marunZZelle: I'm having trouble seeing examples of where that would be preferable to simply self.useFixture21:41
marunseeing -> imagining21:41
ZZellemarun, it could typically useful for tests with the pattern without-with-without21:42
marunZZelle: I would vote for YAGNI at this point21:42
marunZZelle: we can add support for 'with' if/when we find a compelling reason for it21:43
marunZZelle: But I'm not inclined to support it just because21:43
ZZellemarun, so we implement a buggy fixture?21:43
marunZZelle: buggy how?21:43
marunZZelle: add an error message to __enter__21:44
*** leenheer has quit IRC21:44
marunadd an error message -> raise an exception21:44
*** lukasa has joined #openstack-neutron21:44
ZZellemarun, an explicit cleanUp call won't work as expected21:44
marunZZelle: :/21:44
marunZZelle: can you point me to docs on this issue?21:45
marunZZelle: tbh, it's probably not something worth blocking on21:45
marunZZelle: we can always optimize in the future21:45
marunZZelle: but I would like to understand the mechanics of fixtures that makes this an issue21:46
*** Marga_ has joined #openstack-neutron21:46
ZZellemarun, there is no issue in fixtures, but in our PortFixture if we remove the cleanup and assume BridgeFixture will do it21:47
*** seizadi1 has quit IRC21:47
chan1pritesh: yes! Need to clean up the nova-network part. This work perfectly without --nic parameter: -> nova boot --flavor m1.tiny --image cirros-0.3.3-x86_64  --security-group default --key-name demo-key demo-instance121:47
*** seizadi has joined #openstack-neutron21:47
chan1and terminated nova-network service21:47
*** matrohon has quit IRC21:47
marunZZelle: ah, I get it21:47
marunZZelle: no, I don't21:48
priteshchan1: good, so you need to get neutron part up and running and you should be able to get the network runnign as well.21:48
marunZZelle: I see 2 casees21:48
maruncases21:48
ZZellemarun, arg21:48
marunZZelle: 1 - with 2 - useFixture21:48
*** lukasa has quit IRC21:48
marunfor useFixture, there is no expectation of local cleanup, so deletion on bridge cleanup is fine21:49
ZZellemarun, 3 - use setUp/cleanUp manually21:49
marunfor with, we want local cleanup21:49
ZZellemarun, no21:49
marunZZelle: arguably we shouldn't see that21:49
*** yamamoto has joined #openstack-neutron21:49
marunsetUp/cleanUp explicitly I mean21:49
ZZellemarun, yes21:49
marunwe don't have to support every possible permutation of use21:49
*** pritesh has quit IRC21:49
marunuseFixture is the way fixture should be managed21:50
marunZZelle: so, what are the mechanics of the local cleanup required for with?21:50
marunZZelle: would addCleanup on __enter__ work, for example?21:50
* marun goes to look at fixtures.Fixture.__enter__21:51
ZZellemarun, http://bazaar.launchpad.net/~testing-cabal/python-fixtures/master/view/head:/fixtures/fixture.py21:51
marunZZelle: Ah, still in bzr :/21:52
marunhttp://bazaar.launchpad.net/~testing-cabal/python-fixtures/master/view/head:/fixtures/fixture.py#L13121:52
ZZellemarun, long long time ago, we used it :|21:52
marunZZelle: so did I, for years even. Used to work with a bzr dev21:52
marunZZelle: I liked it, frankly, but the small community was problematic21:53
marunZZelle: in any case, it looks simple enough to override __enter__21:53
marunZZelle: https://etherpad.openstack.org/p/5ZlQkQYaw721:53
*** yamamoto has quit IRC21:54
marunZZelle: does that make sense?21:54
*** dane_leblanc has quit IRC21:54
marunZZelle: so cleanup would be explicit for contextmanager use, but implicit without it21:55
*** ociuhandu has joined #openstack-neutron21:55
marunZZelle: the port would already have a reference to the bridge, and the port name could be retrieved from the port21:56
*** ChuckC has quit IRC21:56
ZZellemarun, i am trying if we can have tricky case as 1- 2- can be mixed21:56
ZZellei ma tryin to see ..21:56
*** dkehn has joined #openstack-neutron21:57
marunZZelle: Explicit cleanup adds some overhead, but doesn't harm the implicit use case21:57
ZZellemarun, ie: a fixture used a context can use useFixture in its setUp21:57
marunZZelle: This is just an optimization to avoid deleting ports explicitly21:57
marunZZelle: I don't think fixtures should be smart21:58
marunZZelle: They should know how to achieve an end-state if a certain workflow is followed21:58
marunZZelle: but we don't have to make them magical, because magical == complicated to implement and reason about21:58
openstackgerritCarl Baldwin proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool  https://review.openstack.org/16591021:58
openstackgerritCarl Baldwin proposed openstack/python-neutronclient: Allow passing None for subnetpool  https://review.openstack.org/16849921:58
*** ajmiller_ has joined #openstack-neutron22:00
ZZellemarun, let me update change i integrate your remarks22:00
amullercarl_baldwin: I'm reviewing the refactor patch, should take me 20 more mins~22:01
*** sambetts has quit IRC22:01
marunZZelle: sounds good22:01
*** kbringard has quit IRC22:02
*** ajmiller has quit IRC22:02
carl_baldwinamuller, hopefully you're on PS5 and my other botched attempts didn't trip you up.22:03
carl_baldwinamuller, thanks for reviewing.22:03
openstackgerritAndrew Boik proposed openstack/neutron: Consider all address scopes in init_l3  https://review.openstack.org/16850222:03
*** catohornet has quit IRC22:03
*** dboik_ has quit IRC22:03
*** annegentle has joined #openstack-neutron22:04
amullercarl_baldwin: indeed I'm on PS5... just ran the functional tests on your patch, completely unrelated that the process_monitor is trying to respawn neutron-keepalived-state-change after the router is deleted and before the test is over22:04
*** smccully has quit IRC22:04
*** sambetts has joined #openstack-neutron22:04
*** Swami has quit IRC22:04
amuller(The process monitor is on by default now)22:04
-openstackstatus- NOTICE: Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html22:05
*** ChanServ changes topic to "Gerrit is offline for maintenance, ETA 22:30 UTC http://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html"22:05
amullercarl_baldwin: I'll have to ask ajo to investigate if that could cause issues22:05
marunwow, is gerrit down?22:05
carl_baldwinmarun, seems to be.  I have a vague memory of a maintenance window being announced.22:06
marunhttp://lists.openstack.org/pipermail/openstack-dev/2015-March/059948.html22:06
marunoof22:06
marunnot sure why they wouldn't do this outside of working hours like any normal ops procedure22:06
amullerahh crap22:07
amullerthis kind of sucks =D22:07
marunindeed22:07
marunif only I was using gertty ;)22:07
amullerETA is 21 minutes from now22:08
*** dimsum__ has quit IRC22:09
*** dkehn has quit IRC22:10
*** banix has quit IRC22:11
*** leenheer has joined #openstack-neutron22:11
*** leenheer has quit IRC22:11
amullerreduced to reviewing via a piece of paper22:12
amullerthe humanity22:12
*** leenheer has joined #openstack-neutron22:13
*** aranjan has joined #openstack-neutron22:14
amullermarun: When do you think we could start running the functional tests only with the native ovsdb implementation?22:14
amullerever?22:15
amullermarun: Maybe we can make the tests run only with the native implementation but offer an OS ENV variable so the gate could run it both ways?22:16
marunamuller: I'm not sure what would stop us, though iirc the rootwrap daemon is actually pretty good.22:16
marunamuller: a major limitation is python-openvswitch not being on pypi22:17
amullermarun: I'm more talking about running with only one implementation, not specifically the native one22:17
amullerjust don't like doubling the run cost22:17
marunamuller: well, how much cost are we talking about?22:18
marunamuller: there's nothing wrong with a local optimization22:18
marunamuller: but we can't switch to running one or the other in the gate so long as we need to support both22:18
amullerThe L3 functional tests take exactly twice when running with both implementations22:18
amullerso I edit the code to make it run only with one then remove that modification before pushing22:18
*** pc_m has quit IRC22:18
marunamuller: I guess I'm confused, how is that enabled?22:18
amullermarun: https://github.com/openstack/neutron/blob/master/neutron/tests/functional/agent/linux/base.py#L6022:19
*** flcn has quit IRC22:19
*** c_soukup has quit IRC22:19
marunamuller: that is pretty stupid22:19
marunamuller: I'd merge a patch that made it conditional at least22:20
*** smccully has joined #openstack-neutron22:20
*** ChuckC has joined #openstack-neutron22:20
marunamuller: scenarios = get_scenarios(), which can limit to one or the other as you say via env var22:20
*** dimsum__ has joined #openstack-neutron22:20
amullermarun: I'll add that to my TO DO list22:21
amullerpeople run tests less if they take too long22:21
marunamuller: I don't see much value in running both22:21
marunamuller: it's not like a random functional tests is going to do anything interesting22:21
marunamuller: that the ovs tests don't already do22:22
amullermarun: maybe the right thing to do instead is to move the scenarios to the OVS test case instead of the OVS base class22:22
amullerso only the explicit OVS tests run with both implementations22:22
amullerbut other tests that inherit from the OVS base class only use one22:22
*** dkehn has joined #openstack-neutron22:23
marunamuller: I'd be in favor of that22:24
marungetting extra implicit coverage of ovs usage doesn't seem that valuable to me22:25
marunespecially given the potential penalty22:25
*** ChuckC has quit IRC22:25
*** ChuckC has joined #openstack-neutron22:26
*** thedodd has quit IRC22:27
*** dkehn has quit IRC22:27
*** ajmiller has joined #openstack-neutron22:29
*** ajmiller has quit IRC22:29
*** ronis has quit IRC22:30
*** ajmiller_ has quit IRC22:30
*** achanda_ has joined #openstack-neutron22:31
*** jamiem has quit IRC22:31
*** ChanServ changes topic to "Discussion of OpenStack Networking || for support join #openstack"22:33
amullermarun: Gerrit's up again22:34
*** achanda has quit IRC22:34
*** achanda_ has quit IRC22:36
*** leenheer has quit IRC22:37
*** baoli has quit IRC22:38
*** chan1 has quit IRC22:39
*** markvoelker has quit IRC22:40
*** shwetaap has quit IRC22:41
*** bquan has quit IRC22:42
*** leenheer has joined #openstack-neutron22:42
*** baoli has joined #openstack-neutron22:43
openstackgerritCedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers  https://review.openstack.org/16586622:43
ZZellemarun, ^^22:44
*** aranjan has quit IRC22:44
marunZZelle: I still don't see the point of 'get'22:45
marunZZelle: I'd rather see the port fixtures accept the necessary arguments to init22:45
marunZZelle: that is a standard and well-documented protocol22:45
ZZellemarun, we need different implementations for each bridge type as they have strongly different implementations22:47
marunZZelle: so?22:47
marunZZelle: each port fixture can define it's own init22:47
marunZZelle: we don't have to pretend we care about inheritance and make init somehow conform across implementations22:47
ZZellemarun, who chooses the right PortFixture implementation?22:48
marunZZelle: whoever is writing the test22:48
marunZZelle: I'm not seeing any justification for trying to treat port fixtures polymorphically22:49
marunZZelle: what am I missing?22:49
ZZellemarun, typically it allows to define a test scenario for checking bridgefixture/portfixture22:50
marunZZelle: example?22:50
marunZZelle: YAGNI - complicating a design for the sake of a 'might be nice' is terrible practice22:51
*** signed8b_ is now known as signed8bit_ZZZzz22:51
ZZellemarun, it's used by fake machines in follow-up change22:52
marunZZelle: so make the change in the follow-up change22:52
marunZZelle: adding code that doesn't make sense in the context of the current change is something to be avoided if at all possible22:53
marunZZelle: it effectively means the patches can't be considered in isolation22:53
marunZZelle: and what's the point of having multiple patches if that's the case?22:53
ZZellemarun, because you require smaller changes!!!22:54
marunZZelle: right22:54
marunZZelle: smaller _and_ cohesive changes22:54
marunZZelle: breaking up something arbitrarily and including stuff that doesn't make sense is not going to make for mergeable code22:54
ZZelleso in this change i add 6 times XXXPortFixture, in next change i replace them?22:54
marunZZelle: yes22:54
marunZZelle: we're trying to get away from mix-ins22:55
marunZZelle: fixtures make that possible.22:55
marunZZelle: why go backwards and tie things together with inheritance if we don't need to?22:56
HenryGamuller: ping22:56
amullerHenryG: yo22:56
ZZellemarun, it's not inheritance, it's an interface22:56
*** anshul has quit IRC22:57
ZZellemarun, how do you ensure a common behavior? with implicit duck typing?22:57
HenryGamuller: https://review.openstack.org/#/c/167462/2/neutron/tests/functional/agent/test_l3_agent.py22:57
HenryGamuller: lines 512-51522:57
*** baoli has quit IRC22:57
*** apuimedo has joined #openstack-neutron22:57
*** apuimedo has quit IRC22:58
marunZZelle: introduce the change where it is needed22:58
ZZellemarun, no22:58
marunZZelle: don't complicate a patch with 'I'll need it next patch'22:58
amullerHenryG: Yes?22:59
ZZellemarun, so i merge it with next change22:59
marunZZelle: yes, please22:59
HenryGamuller: 1 sec22:59
*** baoli has joined #openstack-neutron23:00
ZZellemarun, you will get a change twice bigger23:00
*** dane_leblanc has joined #openstack-neutron23:00
marunZZelle: really?  For the sake of a get() method?23:00
HenryGamuller: nevermind, I can't count. sorry23:01
*** salv-orl_ has quit IRC23:01
marunZZelle: the point of breaking patches up is _not_ due to some strictly mechanical judgement23:01
marunZZelle: It's to make sure that the reviewer can follow along and provide effective oversight23:01
*** tidwellr has quit IRC23:01
amullerHenryG: np I'm glad to know you're looking at that stuff :)23:01
marunZZelle: you may have the entire patch series in your head, but I don't review past the first change and it has to make sense to me23:02
ZZellemarun, each change prepares the next change, the aim is to be efficient globally not locally23:02
marunZZelle: well, if you can find a reviewer that works that way, ask them to review instead of me23:02
marunZZelle: I merge stuff I understand and that is cohesive enough to stand on its own23:02
marunZZelle: If I get far enough along the learning curve that the point is obvious, there can be exceptions23:03
*** salv-orlando has joined #openstack-neutron23:03
marunZZelle: but the point of review isn't just about merging code - it's making sure that someone else can understand what you're trying to accomplish23:03
marunZZelle: and no reviewer has as much time as the author to climb the curve.23:04
marunZZelle: so doing your best to provide a cohesive set of patches that make sense in isolation is to your benefit23:04
ZZellemarun, get() is an helper for hidding portfixture behavior for each bridge type, i don't see how it's unrelated23:05
marunZZelle: it adds abstraction to no end23:06
marunZZelle: Nothing in the change requires that abstraction23:06
amullercarl_baldwin: done23:08
*** carl_baldwin has quit IRC23:08
*** rpothier has quit IRC23:10
*** chem has quit IRC23:11
marunZZelle: I'm afraid I'm done for the day.  Will pick up next week if amuller, armax and HenryG don't beat me to it.23:12
*** marun has quit IRC23:12
*** seizadi1 has joined #openstack-neutron23:12
*** seizadi has quit IRC23:13
*** baoli has quit IRC23:13
*** tflynn has quit IRC23:13
ZZellemarun, good week end23:16
*** annegentle has quit IRC23:19
*** flcn has joined #openstack-neutron23:20
*** padkrish has quit IRC23:21
*** _cjones_ has quit IRC23:21
*** padkrish has joined #openstack-neutron23:21
*** nlahouti has quit IRC23:21
*** zhhuabj has quit IRC23:22
*** shaohe_feng has joined #openstack-neutron23:23
*** ajmiller has joined #openstack-neutron23:23
*** jckasper has quit IRC23:24
*** flcn has quit IRC23:24
*** padkrish has quit IRC23:26
*** xgerman has quit IRC23:28
*** aranjan has joined #openstack-neutron23:31
*** Aish has quit IRC23:32
openstackgerritAssaf Muller proposed openstack/neutron: Stop running L3 functional tests with both OVSDB interfaces  https://review.openstack.org/16851323:32
openstackgerritAssaf Muller proposed openstack/neutron: Stop running L3 functional tests with both OVSDB interfaces  https://review.openstack.org/16851323:33
*** dane_leblanc has quit IRC23:34
*** nlahouti has joined #openstack-neutron23:36
*** padkrish has joined #openstack-neutron23:37
openstackgerritMichael Smith proposed openstack/neutron: manual add/remove router for dvr_snat agent  https://review.openstack.org/12202423:39
*** markvoelker has joined #openstack-neutron23:41
*** ajmiller has quit IRC23:41
*** padkrish has quit IRC23:42
openstackgerritMichael Smith proposed openstack/neutron: L3 Agent support of manual dvr snat add/remove  https://review.openstack.org/12381523:43
*** vivek-ebay has joined #openstack-neutron23:46
*** markvoelker has quit IRC23:46
*** julien_ZTE has quit IRC23:47
*** shaohe_feng has quit IRC23:48
*** padkrish has joined #openstack-neutron23:48
*** sbfox has quit IRC23:48
*** signed8bit_ZZZzz is now known as signed8b_23:49
*** julien_ZTE has joined #openstack-neutron23:50
*** amuller has quit IRC23:50
*** flcn has joined #openstack-neutron23:50
*** ociuhandu has quit IRC23:52
*** julien_ZTE has quit IRC23:52
*** dannywilson has quit IRC23:53
*** flcn has quit IRC23:55
*** yeungp has quit IRC23:56
*** seizadi1 has quit IRC23:58
*** dimsum__ has quit IRC23:59

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