Tuesday, 2014-08-12

yamamotonlahouti: have you asked mark to revisit https://review.openstack.org/#/c/89211/ ?00:00
*** zarric has quit IRC00:00
nlahoutiyamamoto: Yes.00:00
nlahoutiyamamoto: I had brief discussion with him.00:00
nlahoutiyamamoto: will continue talking to him tomorrow.00:01
*** vivek-ebay has quit IRC00:01
yamamotonlahouti: does he have more reasons to keep -2 than bp approval?00:02
*** nati_ueno has quit IRC00:02
*** ianw has quit IRC00:02
*** ianw has joined #openstack-neutron00:03
nlahoutiyamamoto: his concern was to improve spec and addess how the API is not going to be broken ...00:04
nlahoutiyamamoto: I need to talk to him more to get more details and understand what his concern is00:04
nlahoutiyamamoto: he didn't have time today, I have to talk to him tomorrow.00:05
yamamotonlahouti: ok.  thank you00:06
nlahoutiyamamoto: np. thx always for your great review00:06
*** ijw_ has joined #openstack-neutron00:07
*** sbfox has quit IRC00:08
*** otherwiseguy has joined #openstack-neutron00:11
*** ijw_ has quit IRC00:11
*** flwang_ has quit IRC00:13
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: NSX: unify the two distributed routing extensions  https://review.openstack.org/10817700:15
*** _cjones_ has quit IRC00:15
*** baoli has joined #openstack-neutron00:16
*** _cjones_ has joined #openstack-neutron00:16
*** dfarrell07 has quit IRC00:16
*** tomoe_ has quit IRC00:17
*** tomoe_ has joined #openstack-neutron00:17
*** nati_ueno has joined #openstack-neutron00:18
*** hemanthravi has quit IRC00:20
*** _cjones_ has quit IRC00:20
*** xgerman has quit IRC00:22
openstackgerritAngus Lees proposed a change to openstack/neutron: Unset HTTP_PROXY before attempting to connect to test WSGI server  https://review.openstack.org/11085300:22
*** shashankhegde has quit IRC00:23
openstackgerritAngus Lees proposed a change to openstack/neutron: Ignore http_proxy while connecting to test WSGI server  https://review.openstack.org/11085300:24
openstackgerritBrandon Logan proposed a change to openstack/neutron: Implement managers for synchronous haproxy driver  https://review.openstack.org/10817400:28
openstackgerritBrandon Logan proposed a change to openstack/neutron: Plugin/DB additions for version 2 of LBaaS API  https://review.openstack.org/10560900:28
openstackgerritBrandon Logan proposed a change to openstack/neutron: Implement synchronous haproxy driver methods  https://review.openstack.org/10817300:28
openstackgerritBrandon Logan proposed a change to openstack/neutron: Tests for extension, db and plugin for LBaaS V2  https://review.openstack.org/10561000:28
openstackgerritBrandon Logan proposed a change to openstack/neutron: New extension for version 2 of LBaaS API  https://review.openstack.org/10533100:28
openstackgerritBrandon Logan proposed a change to openstack/neutron: Implement Jinja templates for haproxy config  https://review.openstack.org/10686700:28
*** zhhuabj has joined #openstack-neutron00:28
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings  https://review.openstack.org/11316600:29
*** dfarrell07 has joined #openstack-neutron00:30
*** armax has left #openstack-neutron00:32
*** a_le has quit IRC00:32
openstackgerritItsuro Oda proposed a change to openstack/neutron: Add endpoint_type parameter to MetaInterfaceDriver  https://review.openstack.org/11341100:36
*** ZZelle has quit IRC00:38
openstackgerritSwaminathan Vasudevan proposed a change to openstack/neutron: Implement migration of legacy routers to distributed  https://review.openstack.org/10585500:41
*** netuser has quit IRC00:41
*** nati_ueno has quit IRC00:42
*** SumitNaiksatam has quit IRC00:43
*** popw has joined #openstack-neutron00:44
*** banix has joined #openstack-neutron00:45
*** nlahouti has quit IRC00:46
*** marun has quit IRC00:46
*** marun has joined #openstack-neutron00:47
*** a_le has joined #openstack-neutron00:47
*** a_le has quit IRC00:56
openstackgerritRajeev Grover proposed a change to openstack/neutron: Fix interface add for dvr with gateway  https://review.openstack.org/11341200:57
*** VijayB_ has quit IRC01:00
*** a_le has joined #openstack-neutron01:02
*** SumitNaiksatam has joined #openstack-neutron01:02
*** amcrn has quit IRC01:02
*** packet has joined #openstack-neutron01:03
*** ijw has quit IRC01:09
*** moha_hunt has quit IRC01:09
*** packet has quit IRC01:10
*** mlavalle has quit IRC01:11
openstackgerritRajeev Grover proposed a change to openstack/neutron: Fix interface add for dvr with gateway  https://review.openstack.org/11341201:12
*** rudrarugge has quit IRC01:15
*** baoli has quit IRC01:15
*** a_le has quit IRC01:17
*** padkrish has quit IRC01:17
*** a_le has joined #openstack-neutron01:18
*** baoli has joined #openstack-neutron01:18
*** xianghuihui has quit IRC01:21
openstackgerritSayaji Patil proposed a change to openstack/neutron: Implements sync mechanism between Neutron and Nuage VSD  https://review.openstack.org/11341801:23
*** xianghui has joined #openstack-neutron01:26
*** SumitNaiksatam has quit IRC01:28
*** SumitNaiksatam has joined #openstack-neutron01:29
*** magesh has quit IRC01:31
*** rudrarugge has joined #openstack-neutron01:33
*** xianghui has quit IRC01:34
*** xianghuihui has joined #openstack-neutron01:34
*** sbfox has joined #openstack-neutron01:34
*** aranjan_ has joined #openstack-neutron01:35
*** carl_baldwin has quit IRC01:35
*** SumitNaiksatam has quit IRC01:36
*** yamahata has joined #openstack-neutron01:36
*** AbsinthMind has quit IRC01:38
*** SumitNaiksatam has joined #openstack-neutron01:38
*** aranjan has quit IRC01:38
*** marun has quit IRC01:38
*** aranjan_ has quit IRC01:39
*** marun has joined #openstack-neutron01:40
*** SumitNaiksatam has quit IRC01:40
*** banix has quit IRC01:42
*** nati_ueno has joined #openstack-neutron01:42
*** a_le has quit IRC01:45
openstackgerritAngus Lees proposed a change to openstack/neutron: Purge use of "PRED and A or B" poor-mans-ternary  https://review.openstack.org/11084901:48
*** dfarrell07 has quit IRC01:50
*** rook has joined #openstack-neutron01:50
openstackgerritItsuro Oda proposed a change to openstack/neutron: Add endpoint_type parameter to MetaInterfaceDriver  https://review.openstack.org/11341101:55
*** SridharR_ has joined #openstack-neutron02:00
*** trinaths has joined #openstack-neutron02:01
openstackgerritFranck Yelles proposed a change to openstack/neutron: Implements: blueprint dhcp-host-routes-and-dns-support-for-nuage-plugin  https://review.openstack.org/11342102:01
*** SridharRamaswamy has quit IRC02:04
*** yano has quit IRC02:05
*** SridharR_ has quit IRC02:05
*** leenheer has joined #openstack-neutron02:05
*** sbfox has quit IRC02:05
*** ijw has joined #openstack-neutron02:08
*** magesh_ has joined #openstack-neutron02:09
*** leenheer has quit IRC02:09
*** yano has joined #openstack-neutron02:11
openstackgerritMichael Smith proposed a change to openstack/neutron: Fixes an issue with FIP re-association  https://review.openstack.org/11242702:12
*** AbsinthMind1 has joined #openstack-neutron02:12
*** ijw has quit IRC02:13
*** flwang_ has joined #openstack-neutron02:14
*** flwang_ has quit IRC02:18
*** [1]trinaths has joined #openstack-neutron02:20
*** banix has joined #openstack-neutron02:20
*** AbsinthMind1 has quit IRC02:21
*** nlahouti has joined #openstack-neutron02:21
*** trinaths has quit IRC02:22
*** [1]trinaths is now known as trinaths02:22
openstackgerritIsaku Yamahata proposed a change to openstack/neutron: add auth token to context  https://review.openstack.org/11175602:22
openstackgerrityangxurong proposed a change to openstack/neutron: Use call to report state when ovs_agent starts up  https://review.openstack.org/10869302:22
*** zarric has joined #openstack-neutron02:23
*** _cjones_ has joined #openstack-neutron02:23
*** _cjones_ has quit IRC02:23
*** sbalukoff has quit IRC02:23
*** _cjones_ has joined #openstack-neutron02:24
*** VijayB_ has joined #openstack-neutron02:25
*** _cjones_ has quit IRC02:25
*** magesh_ has quit IRC02:25
*** trinaths has quit IRC02:25
*** _cjones_ has joined #openstack-neutron02:25
*** nlahouti_ has joined #openstack-neutron02:27
*** nlahouti has quit IRC02:28
*** vivek-ebay has joined #openstack-neutron02:28
*** nlahouti has joined #openstack-neutron02:29
*** nlahouti_ has quit IRC02:29
*** _cjones_ has quit IRC02:30
*** vivek-eb_ has joined #openstack-neutron02:35
*** vivek-ebay has quit IRC02:35
*** nlahouti has quit IRC02:36
*** magesh_ has joined #openstack-neutron02:36
*** nlahouti has joined #openstack-neutron02:43
*** nlahouti has quit IRC02:43
*** nlahouti has joined #openstack-neutron02:44
*** diegows has quit IRC02:49
*** WackoRobie has quit IRC02:53
*** WackoRobie has joined #openstack-neutron02:53
*** magesh_ has quit IRC03:01
*** magesh_ has joined #openstack-neutron03:01
*** baoli has quit IRC03:03
*** jgrimm has joined #openstack-neutron03:05
*** VijayB_ has quit IRC03:06
*** aranjan has joined #openstack-neutron03:09
openstackgerritAngus Lees proposed a change to openstack/neutron: Remove unused arg to config.setup_logging()  https://review.openstack.org/11118003:10
*** leenheer has joined #openstack-neutron03:12
*** aranjan has quit IRC03:12
*** aranjan has joined #openstack-neutron03:13
*** leenheer has quit IRC03:13
*** tomoe_ has quit IRC03:14
*** tomoe_ has joined #openstack-neutron03:14
openstackgerritA change was merged to openstack/neutron: Implement Midonet Juno Network Api calls  https://review.openstack.org/10382003:20
*** tomoe_ has quit IRC03:25
*** tomoe_ has joined #openstack-neutron03:25
*** yamamot__ has joined #openstack-neutron03:26
*** AlexF_ has joined #openstack-neutron03:27
*** yamamoto has quit IRC03:29
*** nati_ueno has quit IRC03:31
*** leenheer has joined #openstack-neutron03:31
*** AbsinthMind has joined #openstack-neutron03:33
*** rwsu has quit IRC03:38
*** SumitNaiksatam has joined #openstack-neutron03:38
*** crc32 has quit IRC03:41
*** sbfox has joined #openstack-neutron03:41
*** Xurong has quit IRC03:49
*** banix has quit IRC03:50
*** WackoRobie has quit IRC03:54
*** WackoRobie has joined #openstack-neutron03:54
*** chandankumar has joined #openstack-neutron03:55
*** zarric has quit IRC03:56
*** WackoRobie has quit IRC03:58
*** zarric has joined #openstack-neutron03:59
*** magesh_ has quit IRC03:59
*** AbsinthMind has quit IRC04:00
*** leenheer has quit IRC04:02
openstackgerritA change was merged to openstack/python-neutronclient: Provide support for nested objects  https://review.openstack.org/11147404:03
openstackgerritA change was merged to openstack/neutron: MLNX Agent: Process port_update notifications in the main agent loop  https://review.openstack.org/9488204:04
*** rook has quit IRC04:04
*** zarric has quit IRC04:04
*** popw has quit IRC04:05
*** popw has joined #openstack-neutron04:05
*** a_le has joined #openstack-neutron04:06
*** ijw has joined #openstack-neutron04:10
*** leenheer has joined #openstack-neutron04:11
*** HenryG is now known as HenryG_afk04:14
*** sgordon has quit IRC04:14
*** ijw has quit IRC04:15
*** flwang_ has joined #openstack-neutron04:15
*** a_le has quit IRC04:15
*** a_le has joined #openstack-neutron04:15
*** carl_baldwin has joined #openstack-neutron04:17
*** marun has quit IRC04:17
openstackgerritAngus Lees proposed a change to openstack/python-neutronclient: Avoid modifying default function arguments  https://review.openstack.org/11343504:18
*** magesh_ has joined #openstack-neutron04:18
*** flwang_ has quit IRC04:19
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix PortNotFound exception during sync_routers  https://review.openstack.org/11340004:20
*** chandankumar has quit IRC04:20
openstackgerritA change was merged to openstack/python-neutronclient: Fix unit tests to succeed on any PYTHONHASHSEED  https://review.openstack.org/11099104:21
*** leenheer has quit IRC04:22
*** WackoRobie has joined #openstack-neutron04:25
*** leenheer has joined #openstack-neutron04:25
*** WackoRobie has quit IRC04:26
*** WackoRobie has joined #openstack-neutron04:27
*** networkstatic has quit IRC04:27
*** nati_ueno has joined #openstack-neutron04:27
*** yfried has quit IRC04:29
*** AlexF_ has quit IRC04:30
*** sbalukoff has joined #openstack-neutron04:30
*** leenheer has quit IRC04:32
*** nati_ueno has quit IRC04:32
*** leenheer has joined #openstack-neutron04:35
*** WackoRobie has quit IRC04:37
*** leenheer has quit IRC04:38
*** trinaths has joined #openstack-neutron04:43
*** leenheer has joined #openstack-neutron04:43
*** magesh_ has quit IRC04:46
*** dfarrell07 has joined #openstack-neutron04:48
*** hajay_ has joined #openstack-neutron04:55
*** gildub has quit IRC04:56
*** gildub has joined #openstack-neutron04:58
*** kopparam has joined #openstack-neutron05:02
*** WackoRobie has joined #openstack-neutron05:02
*** rharwood has quit IRC05:07
*** Longgeek has joined #openstack-neutron05:07
*** ianw has quit IRC05:08
*** ianw has joined #openstack-neutron05:08
*** rharwood has joined #openstack-neutron05:08
*** shashankhegde has joined #openstack-neutron05:09
*** WackoRobie has quit IRC05:09
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Make plugin and l3plugin available as mixin's properties  https://review.openstack.org/11340005:09
*** enikanorov has quit IRC05:10
*** enikanorov has joined #openstack-neutron05:11
*** garyk has joined #openstack-neutron05:12
*** carl_baldwin has quit IRC05:13
*** jaypipes has joined #openstack-neutron05:14
*** garyk1 has joined #openstack-neutron05:14
*** garyk has quit IRC05:14
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Make plugin and l3plugin available as mixin's properties  https://review.openstack.org/11340005:15
openstackgerritRudra Rugge proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663005:15
*** Zemeio has quit IRC05:15
*** ramishra has joined #openstack-neutron05:15
*** wanghong has quit IRC05:15
*** chandankumar has joined #openstack-neutron05:16
*** wanghong has joined #openstack-neutron05:16
*** chandankumar has quit IRC05:17
*** chandankumar has joined #openstack-neutron05:17
*** ramishra has quit IRC05:17
*** ramishra has joined #openstack-neutron05:17
*** aleksandr_null has quit IRC05:20
*** aleksandr_null has joined #openstack-neutron05:21
*** s3wong has joined #openstack-neutron05:22
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Make plugin and l3plugin available as mixin's properties  https://review.openstack.org/11340005:22
*** seizadi has joined #openstack-neutron05:22
*** otherwiseguy has quit IRC05:22
*** otherwiseguy has joined #openstack-neutron05:23
*** WackoRobie has joined #openstack-neutron05:24
*** AlexF_ has joined #openstack-neutron05:25
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Make plugin and l3plugin available as mixin's properties  https://review.openstack.org/11340005:27
*** nati_ueno has joined #openstack-neutron05:28
*** WackoRobie has quit IRC05:29
*** ramishra has quit IRC05:29
*** rotbeard has joined #openstack-neutron05:29
*** ramishra has joined #openstack-neutron05:30
*** vivek-ebay has joined #openstack-neutron05:31
*** nati_ueno has quit IRC05:33
*** ramishra has quit IRC05:34
*** vivek-eb_ has quit IRC05:34
*** ramishra has joined #openstack-neutron05:35
*** vivek-ebay has quit IRC05:35
*** k4n0 has joined #openstack-neutron05:37
*** WackoRobie has joined #openstack-neutron05:38
*** gangil has joined #openstack-neutron05:39
*** WackoRobie has quit IRC05:40
*** ianw has quit IRC05:42
*** ianw has joined #openstack-neutron05:42
*** WackoRobie has joined #openstack-neutron05:43
*** ianw has quit IRC05:43
openstackgerritAngus Lees proposed a change to openstack/python-neutronclient: Remove incorrect super() call  https://review.openstack.org/11343905:43
*** ianw has joined #openstack-neutron05:43
*** rharwood has quit IRC05:43
*** ianw has quit IRC05:43
*** ianw has joined #openstack-neutron05:44
*** ianw has quit IRC05:44
*** yfried has joined #openstack-neutron05:44
*** ianw has joined #openstack-neutron05:44
*** ianw has quit IRC05:45
*** ianw has joined #openstack-neutron05:45
*** ianw has quit IRC05:46
*** ianw has joined #openstack-neutron05:46
*** rharwood has joined #openstack-neutron05:46
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix PortNotFound exception during sync_routers  https://review.openstack.org/11344105:48
*** Xurong has joined #openstack-neutron05:48
*** leenheer has quit IRC05:48
*** ianw has quit IRC05:50
*** ianw has joined #openstack-neutron05:51
*** leenheer has joined #openstack-neutron05:53
*** otherwiseguy has quit IRC05:53
*** ajo_ has quit IRC05:53
*** ajo__ has joined #openstack-neutron05:53
openstackgerritA change was merged to openstack/neutron: Fixes an issue with FIP re-association  https://review.openstack.org/11242705:54
*** leenheer has quit IRC06:00
*** s3wong has quit IRC06:00
*** ianw has quit IRC06:01
*** tomoe_ has quit IRC06:01
openstackgerritPreeti Mirji proposed a change to openstack/neutron: Adm policy can't use other tenants unshared rules  https://review.openstack.org/10895206:01
*** reed has quit IRC06:01
*** tomoe_ has joined #openstack-neutron06:01
*** sgordon`` has quit IRC06:01
*** rharwood has quit IRC06:01
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Improve structure of external_gateway_added  https://review.openstack.org/11238506:01
*** reed has joined #openstack-neutron06:02
openstackgerritOpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/11276106:04
*** sgordon has joined #openstack-neutron06:06
*** rharwood has joined #openstack-neutron06:07
*** SumitNaiksatam has quit IRC06:07
*** evgenyf has joined #openstack-neutron06:07
*** shashankhegde has quit IRC06:08
*** SumitNaiksatam has joined #openstack-neutron06:08
*** yamamoto has joined #openstack-neutron06:09
*** yamamo___ has joined #openstack-neutron06:10
*** ijw has joined #openstack-neutron06:12
*** oda-g has quit IRC06:12
*** yamamot__ has quit IRC06:12
*** yamamoto has quit IRC06:14
*** AlexF_ has quit IRC06:14
trinathsHi Reviewers, May I have your time for review of my new patchset at https://review.openstack.org/#/c/109659/06:15
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings  https://review.openstack.org/11316606:15
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: Allow network types overridable  https://review.openstack.org/11294706:15
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN  https://review.openstack.org/11294906:15
*** AlexF_ has joined #openstack-neutron06:15
*** flwang_ has joined #openstack-neutron06:16
*** ijw has quit IRC06:16
*** nati_ueno has joined #openstack-neutron06:17
*** Longgeek has quit IRC06:19
*** irenab has joined #openstack-neutron06:19
*** dfarrell07 has quit IRC06:20
*** flwang_ has quit IRC06:20
*** Longgeek has joined #openstack-neutron06:21
*** pradipta_away is now known as pradipta06:23
*** ollivier has joined #openstack-neutron06:26
*** yamamo___ has quit IRC06:28
*** a_le has quit IRC06:31
*** akash has joined #openstack-neutron06:31
*** gangil has left #openstack-neutron06:31
*** devvesa has joined #openstack-neutron06:33
*** atmark has joined #openstack-neutron06:33
atmarkHello, I'm having problem starting neutron-server. I'm getting this error in the log file06:35
atmarkReconnecting to AMQP server on 10.107.1.1:567206:35
trinathsatmark: check your neutron.conf whether amqp is configured correctly06:35
atmarkrabbitmq is running in the background06:35
atmark[root@controller-node ~(keystone_admin)]# lsof -i :567206:35
atmarkCOMMAND    PID     USER   FD   TYPE DEVICE SIZE/OFF NODE NAME06:36
atmarkbeam.smp 37027 rabbitmq   15u  IPv6 232688      0t0  TCP *:amqp (LISTEN)06:36
trinathsatmark: use paste.openstack.org to showcase logs06:36
*** akash has quit IRC06:37
*** lordd_ has joined #openstack-neutron06:39
atmarktrinaths: http://sprunge.us/ibYJ06:39
*** hajay_ has quit IRC06:42
*** nlahouti has quit IRC06:42
atmarktrinaths: am I missing something?06:43
openstackgerritRobin Wang proposed a change to openstack/neutron: Neutron-ovs-agent crashes when OVS is restarted/dead  https://review.openstack.org/11345006:43
*** gangil has joined #openstack-neutron06:44
*** bvandenh has joined #openstack-neutron06:44
*** gangil has quit IRC06:44
*** gangil has joined #openstack-neutron06:44
trinathsatmark: neutron.conf is okay06:45
*** gangil has quit IRC06:45
trinathsatmark: can you restart and check the server log.06:45
*** gangil has joined #openstack-neutron06:45
atmarkAMQP server on 10.107.1.1:5672 is unreachable: Socket closed. Trying again in 3 seconds.06:48
*** AlexF_ has quit IRC06:51
*** lordd_ has quit IRC06:52
trinathsatmark: where is this amqp actually running? Is on Openstack controller or some other server06:53
*** WackoRobie has quit IRC06:56
atmarkit's on the controller06:57
*** pasquier-s has joined #openstack-neutron06:57
*** yamamoto has joined #openstack-neutron06:58
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: New common util module for Barbican TLS containers  https://review.openstack.org/10984906:59
*** ivar-lazzaro has quit IRC07:00
*** ivar-lazzaro has joined #openstack-neutron07:00
*** lordd_ has joined #openstack-neutron07:01
trinathsatmark: okay. can make it localhost for rabbit_host and try it07:02
*** ivar-lazzaro has quit IRC07:02
*** ivar-lazzaro1 has joined #openstack-neutron07:02
*** a_le has joined #openstack-neutron07:02
*** ramishra has quit IRC07:02
*** ramishra has joined #openstack-neutron07:02
atmarktrinaths: jus jut a sec07:05
atmarkjust a sec*07:06
*** ramishra has quit IRC07:07
atmarktrinaths: still the same07:08
atmarkAMQP server on 127.0.0.1:5672 is unreachable: Socket closed. Trying again in 15 seconds.07:08
*** matrohon has joined #openstack-neutron07:08
*** a_le has quit IRC07:08
*** yamamoto has quit IRC07:10
*** dbite has joined #openstack-neutron07:10
*** SridharG has joined #openstack-neutron07:12
*** ijw has joined #openstack-neutron07:13
*** nlahouti has joined #openstack-neutron07:13
*** nlahouti has quit IRC07:17
*** ijw has quit IRC07:17
openstackgerritAngus Lees proposed a change to openstack/python-neutronclient: Remove incorrect super() call  https://review.openstack.org/11343907:18
trinathsatmark: can you restart amqp07:18
*** jgallard has joined #openstack-neutron07:20
openstackgerritRudra Rugge proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663007:21
*** jgallard has quit IRC07:23
*** jgallard has joined #openstack-neutron07:23
*** nagyz has joined #openstack-neutron07:24
*** ramishra has joined #openstack-neutron07:27
*** sballe has joined #openstack-neutron07:27
*** seizadi has quit IRC07:27
*** salv-orlando has quit IRC07:27
*** jschwarz has joined #openstack-neutron07:28
*** yamamoto has joined #openstack-neutron07:28
*** afazekas has joined #openstack-neutron07:31
*** yamamoto has quit IRC07:31
*** jistr has joined #openstack-neutron07:36
*** trinaths has quit IRC07:38
*** rudrarugge has quit IRC07:40
*** trinaths has joined #openstack-neutron07:42
*** AlexF_ has joined #openstack-neutron07:49
openstackgerritAngus Lees proposed a change to openstack/neutron: Raise correct NotImplementedError exception  https://review.openstack.org/11345707:50
*** flwang_ has joined #openstack-neutron07:51
*** gangil has quit IRC07:53
*** jpich has joined #openstack-neutron07:53
*** gangil has joined #openstack-neutron07:53
*** Krast has joined #openstack-neutron07:55
*** gangil has quit IRC07:58
*** gangil has joined #openstack-neutron07:58
*** gangil has quit IRC07:58
*** gangil has joined #openstack-neutron07:58
*** networkstatic has joined #openstack-neutron07:58
*** ramishra has quit IRC07:58
*** networkstatic has quit IRC07:58
*** ramishra has joined #openstack-neutron07:59
*** fifieldt__ is now known as fifieldt07:59
*** Krast has quit IRC08:02
*** yamamoto has joined #openstack-neutron08:02
openstackgerritRobin Wang proposed a change to openstack/neutron: Neutron-ovs-agent crashes when OVS is restarted/dead  https://review.openstack.org/11345008:02
*** sbfox has quit IRC08:03
*** ramishra has quit IRC08:03
*** AlexF_ has quit IRC08:04
*** dosaboy_ has joined #openstack-neutron08:05
openstackgerritChirag Shahani proposed a change to openstack/neutron: Add template attr. for subnet, router create in Nuage plugin  https://review.openstack.org/11274908:05
*** yamamoto has quit IRC08:06
*** Krast has joined #openstack-neutron08:06
*** Dafna has joined #openstack-neutron08:07
openstackgerritJohn Schwarz proposed a change to openstack/neutron: Allow filtering routers by network-ids  https://review.openstack.org/11158008:09
openstackgerritRobin Wang proposed a change to openstack/neutron: Neutron-ovs-agent crashes when OVS is restarted/dead(Alternative Patch I)  https://review.openstack.org/11053808:13
*** jlibosva has joined #openstack-neutron08:14
*** ijw has joined #openstack-neutron08:14
*** nlahouti has joined #openstack-neutron08:15
*** ihrachyshka has joined #openstack-neutron08:16
*** nlahouti has quit IRC08:19
*** ijw has quit IRC08:19
*** ihrachyshka has quit IRC08:20
*** ihrachyshka has joined #openstack-neutron08:21
openstackgerritRobin Wang proposed a change to openstack/neutron: Neutron-ovs-agent crashes when OVS is restarted/dead(Alternative Patch II)  https://review.openstack.org/11345008:22
openstackgerritKevin Benton proposed a change to openstack/neutron: External Attachment Point Extension Mixin  https://review.openstack.org/9717308:23
*** zhhuabj has quit IRC08:24
*** zhhuabj has joined #openstack-neutron08:25
openstackgerritshihanzhang proposed a change to openstack/neutron: Refactor security group rpc call  https://review.openstack.org/11187608:25
*** amuller has joined #openstack-neutron08:27
*** yfried is now known as yfried_afk08:27
*** AlexF_ has joined #openstack-neutron08:27
*** yamamoto has joined #openstack-neutron08:28
*** ramishra has joined #openstack-neutron08:29
trinathsHi Reviewers, May I have your time for review of my new patchset at https://review.openstack.org/#/c/109659/08:31
*** yamamoto has quit IRC08:33
*** nati_ueno has quit IRC08:33
*** ramishra has quit IRC08:34
*** nati_ueno has joined #openstack-neutron08:34
*** dosaboy_ has quit IRC08:36
*** drankis has quit IRC08:37
*** tomoe_ has quit IRC08:38
*** tomoe_ has joined #openstack-neutron08:38
*** Krast has quit IRC08:40
*** Krast has joined #openstack-neutron08:40
*** AlexF_ has quit IRC08:41
*** nlahouti has joined #openstack-neutron08:46
*** dosaboy_ has joined #openstack-neutron08:46
*** dosaboy_ is now known as dosaboy-alt08:46
*** nagyz has quit IRC08:46
*** dosaboy-alt has quit IRC08:47
*** nlahouti has quit IRC08:50
*** pradipta is now known as pradipta_away08:51
openstackgerritFranck Yelles proposed a change to openstack/neutron: Adding support of DNS nameserver and Host routes for the Nuage Plugin  https://review.openstack.org/11342108:53
*** AlexF_ has joined #openstack-neutron08:55
*** zhhuabj has quit IRC08:56
*** zhhuabj has joined #openstack-neutron08:56
*** gangil has quit IRC09:03
*** a_le has joined #openstack-neutron09:03
*** Hayim has quit IRC09:06
*** AlexF_ has quit IRC09:07
openstackgerritRobin Wang proposed a change to openstack/neutron: Neutron-ovs-agent crashes when OVS is restarted/dead(Alternative Patch I)  https://review.openstack.org/11053809:07
*** Hayim has joined #openstack-neutron09:07
*** a_le has quit IRC09:08
*** AlexF_ has joined #openstack-neutron09:08
*** nati_ueno has quit IRC09:12
*** nlahouti has joined #openstack-neutron09:16
*** claudiub has joined #openstack-neutron09:18
*** dosaboy has quit IRC09:20
openstackgerritMiroslav Los proposed a change to openstack/neutron: Fix performance of get_bridge_for_tap_interface and its use  https://review.openstack.org/11346709:20
*** nlahouti has quit IRC09:21
*** dosaboy has joined #openstack-neutron09:23
openstackgerritMiroslav Los proposed a change to openstack/neutron: Fix performance of get_bridge_for_tap_interface  https://review.openstack.org/11346709:24
*** aranjan has quit IRC09:24
*** evgenyf has quit IRC09:27
*** ramishra has joined #openstack-neutron09:28
*** yamamoto has joined #openstack-neutron09:29
*** ajo_ has joined #openstack-neutron09:29
openstackgerritMiroslav Los proposed a change to openstack/neutron: Fix performance of get_bridge_for_tap_interface  https://review.openstack.org/11346709:29
*** geekinut1h has joined #openstack-neutron09:30
*** AlexF_ has quit IRC09:32
*** ajo__ has quit IRC09:32
*** evgenyf has joined #openstack-neutron09:32
*** geekinutah has quit IRC09:33
*** yamamoto has quit IRC09:34
*** AlexF_ has joined #openstack-neutron09:36
*** oreillyd has joined #openstack-neutron09:37
openstackgerritIlya Shakhat proposed a change to openstack/python-neutronclient: Replace httpretty with requests_mock  https://review.openstack.org/11326509:38
trinathsHi Reviewers, May I have your time for review of my new patchset at https://review.openstack.org/#/c/109659/09:46
openstackgerritKoteswara Rao Kelam proposed a change to openstack/python-neutronclient: firewall policy update for a rule is not working  https://review.openstack.org/10413209:47
*** zhiyan has quit IRC09:47
*** jdag has quit IRC09:48
*** ctracey has quit IRC09:49
*** skath has quit IRC09:50
*** jdag_ has joined #openstack-neutron09:50
*** sweston has quit IRC09:50
*** nimrodsun has quit IRC09:52
*** zhiyan has joined #openstack-neutron09:52
openstackgerritRobin Wang proposed a change to openstack/neutron: OVS-Agent crashes when OVS restarted/dead(Alternative Patch II)  https://review.openstack.org/11345009:53
*** ctracey has joined #openstack-neutron09:53
*** sweston has joined #openstack-neutron09:54
*** skath has joined #openstack-neutron09:54
*** dgollub has joined #openstack-neutron09:54
*** jdag_ has quit IRC09:55
*** flaviof_zzz is now known as flaviof09:55
*** zhiyan has quit IRC09:57
openstackgerritJohn Davidge proposed a change to openstack/neutron: Do not assume order of new_peers list elements  https://review.openstack.org/11203409:57
openstackgerritRobin Wang proposed a change to openstack/neutron: OVS-Agent crashes when OVS dead(Alternative I)  https://review.openstack.org/11053809:57
*** nimrodsun has joined #openstack-neutron09:57
openstackgerritRobin Wang proposed a change to openstack/neutron: OVS-Agent crashes when OVS dead(Alternative II)  https://review.openstack.org/11345009:58
*** AlexF_ has quit IRC09:59
*** zhiyan has joined #openstack-neutron09:59
*** jdag_ has joined #openstack-neutron10:00
*** dave_tucker_zzz is now known as dave_tucker10:00
*** raorn_ is now known as raorn10:03
*** nati_ueno has joined #openstack-neutron10:09
*** aranjan has joined #openstack-neutron10:11
*** alexpilotti has joined #openstack-neutron10:12
*** jgallard has quit IRC10:12
*** ianw has joined #openstack-neutron10:14
*** nati_ueno has quit IRC10:14
*** nlahouti has joined #openstack-neutron10:18
*** pasquier-s has quit IRC10:20
*** AlexF_ has joined #openstack-neutron10:21
*** nlahouti has quit IRC10:22
*** aranjan has quit IRC10:24
*** ajo__ has joined #openstack-neutron10:26
*** sgordon_ has joined #openstack-neutron10:28
*** ajo_ has quit IRC10:28
*** yamamoto has joined #openstack-neutron10:29
*** ajo_ has joined #openstack-neutron10:30
*** ajo__ has quit IRC10:30
*** AlexF_ has quit IRC10:31
*** yamamoto has quit IRC10:33
*** AlexF_ has joined #openstack-neutron10:34
*** yfried_afk is now known as yfried10:35
*** dgollub has quit IRC10:43
*** sballe has quit IRC10:46
*** Krast has quit IRC10:50
*** RockKuo_Home has joined #openstack-neutron10:51
*** aranjan has joined #openstack-neutron10:51
*** ZZelle has joined #openstack-neutron10:52
jschwarzilyashakhat_, ping10:52
*** aranjan has quit IRC10:53
*** kopparam has quit IRC10:53
ZZelleyfried, hi10:53
ilyashakhat_jschwarz: hi!10:53
yfriedsbadia:  ZZelle?10:53
yfriedZZelle: sorry. hi10:53
*** aranjan has joined #openstack-neutron10:53
jschwarzilyashakhat_, can you look at https://bugs.launchpad.net/python-neutronclient/+bug/1353536 please?10:54
*** aranjan has quit IRC10:54
ilyashakhat_jschwarz: ok, I'll take it10:55
ZZelleyfried, i am trying to understand how i can address your 2nd comment in https://review.openstack.org/#/c/103498/9/tempest/api/network/admin/test_provider_extension.py10:55
jschwarzIt would appear the bug is at https://github.com/openstack/python-neutronclient/blob/master/neutronclient/neutron/v2_0/lb/healthmonitor.py L77 - the parameter 'timeout' is already defined elsewhere, causing a collision and making the parameter to always not appear10:55
*** aranjan has joined #openstack-neutron10:55
ZZelleyfried, i am not really tempest fluent10:55
jschwarzilyashakhat_, actually I don't mind taking it, just wanted to make sure I found the right cause for it :)10:55
*** ramishra has quit IRC10:56
*** ramishra has joined #openstack-neutron10:57
jschwarzIt is already defined https://github.com/openstack/python-neutronclient/blob/master/neutronclient/shell.py L59710:57
ilyashakhat_jschwarz: it was added in https://review.openstack.org/#/c/105386/10:57
*** sgordon_ has quit IRC10:58
*** reed has quit IRC10:58
yfriedZZelle: did you read the bp I linked to?10:58
jschwarzI agree10:59
*** claudiub has quit IRC11:00
ZZelleyfried, i understand the BP but not where to put things11:00
jschwarzilyashakhat_, you mind if I take the fix? I'll send you a link to the patch when it's up11:00
*** aranjan has quit IRC11:00
ilyashakhat_jschwarz: yes, please do it11:01
ilyashakhat_btw, what will be the fix? rollback of global option or rename of LB-specific?11:01
yfriedZZelle: let's move this to #openstack-qa11:01
jschwarzrename of LB-specific to something like '--monitor-timeout' and assigning it to the original variable at L9311:02
jschwarzunless you have a better idea11:03
ilyashakhat_the concern is about backward compatibility, because we change the cli11:03
*** Hayim has quit IRC11:04
jschwarzoh, I see..11:04
jschwarzI imagine a similar solution can be implemented for the global option, though I'd like to talk to Kevin about it first11:05
*** jschwarz is now known as jschwarz|away11:07
ilyashakhat_agree, renaming of global option looks better11:08
*** yamamoto has joined #openstack-neutron11:10
*** Longgeek has quit IRC11:10
*** Longgeek has joined #openstack-neutron11:10
*** nati_ueno has joined #openstack-neutron11:11
*** yamahata has quit IRC11:11
*** GonZo2000 has joined #openstack-neutron11:14
*** GonZo2000 has quit IRC11:14
*** GonZo2000 has joined #openstack-neutron11:14
*** Longgeek has quit IRC11:14
*** nati_ueno has quit IRC11:15
*** vishnoianil has joined #openstack-neutron11:16
vishnoianilHi All11:16
vishnoianilI am trying to test FWaaS service and facing issue related to iptables rule11:17
vishnoianilanybody up for taking a shot at it ?11:17
vishnoianilI am using icehouse11:17
*** vishnoianil is now known as avishnoi11:17
*** nlahouti has joined #openstack-neutron11:20
trinathsavishnoi,: what is the issue you are facing11:21
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: Removing sorted() function from assertEqual()  https://review.openstack.org/11349411:23
*** flaviof is now known as flaviof_zzz11:23
*** kopparam has joined #openstack-neutron11:24
*** nlahouti has quit IRC11:24
*** drankis has joined #openstack-neutron11:24
*** kopparam has quit IRC11:28
*** ramishra has quit IRC11:29
*** ramishra has joined #openstack-neutron11:30
*** Longgeek has joined #openstack-neutron11:34
*** ramishra has quit IRC11:34
*** gildub has quit IRC11:34
*** sgordon_ has joined #openstack-neutron11:35
*** sambetts has joined #openstack-neutron11:35
*** jab is now known as bradjones11:36
*** baoli has joined #openstack-neutron11:38
*** gildub has joined #openstack-neutron11:38
*** pasquier-s has joined #openstack-neutron11:38
*** dgollub has joined #openstack-neutron11:39
*** WackoRobie has joined #openstack-neutron11:40
*** AlexF_ has quit IRC11:40
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Import all neutron modules in autogenerate  https://review.openstack.org/11292811:40
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Change autogenerate to be unconditional  https://review.openstack.org/10870011:40
*** lordd_ has quit IRC11:42
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings  https://review.openstack.org/11316611:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Enable local arp responder for local VMs  https://review.openstack.org/11316011:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: Allow network types overridable  https://review.openstack.org/11294711:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: get_agent_ports: Don't yield (None, {})  https://review.openstack.org/11315911:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: merge br-tun into br-int  https://review.openstack.org/11248811:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN  https://review.openstack.org/11294911:45
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ML2: Make get_device_details report mac address as well  https://review.openstack.org/11294811:45
*** ramishra has joined #openstack-neutron11:48
*** jschwarz|away is now known as jschwarz11:48
jschwarzilyashakhat_, sorry I went afk all of a sudden11:48
jschwarzwill contact Kevin Benton and see what he says about it11:49
ilyashakhat_ok11:49
*** nlahouti has joined #openstack-neutron11:51
*** WackoRobie has quit IRC11:51
*** WackoRobie has joined #openstack-neutron11:52
jschwarzkevinbenton, ping11:52
avishnoitrinaths: i created two private network for a tenant11:53
avishnoiand i am using vxlan based network11:53
*** aranjan has joined #openstack-neutron11:53
avishnoitrinaths: i created router to connect both the private network11:53
avishnoitrinaths: i created firewall with rule, which blocks ICMP traffic between the two VM connected to these two networks11:54
avishnoitrinaths: but its not blocking the ICMP traffic11:55
*** nlahouti has quit IRC11:55
avishnoitrinaths: when i dump iptables rules on the router namespace, i don't see any new rules that deny the traffic11:55
avishnoitrinaths: but i observer strange behaviour, when i restart l3-agent, it installs those icmp related rules in the iptables11:56
*** diegows has joined #openstack-neutron11:57
*** aranjan has quit IRC11:57
avishnoitrinaths: i further dig into iptables_manage.py, and it seems like we call defer_apply_on in process_router and call defer_apply_off in this function when we see if there is any external gw interface is attached to the router11:58
*** flwang_ has quit IRC11:58
avishnoitrinaths: which is not the case in my setup, so i created external network and attached it to the router and now its working absolutely fine11:58
matrohonyamamoto : hi11:59
avishnoitrinaths: so just wanted to know, if its a requirement that router need to be connected to the external network?11:59
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings  https://review.openstack.org/11316612:02
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN  https://review.openstack.org/11294912:02
*** AlexF_ has joined #openstack-neutron12:04
trinathsavishnoi: Not required. but you need a router configured.12:05
*** jgallard has joined #openstack-neutron12:06
*** prad has joined #openstack-neutron12:07
jschwarzpcm_, ping12:09
*** trinaths has quit IRC12:10
*** salv-orlando has joined #openstack-neutron12:10
*** banix has joined #openstack-neutron12:17
*** AlexF_ has quit IRC12:17
openstackgerritEvgeny Fedoruk proposed a change to openstack/neutron: Removing sorted() function from assertEqual()  https://review.openstack.org/11349412:18
*** lordd_ has joined #openstack-neutron12:19
*** nlahouti has joined #openstack-neutron12:22
*** dbite has quit IRC12:24
*** kopparam has joined #openstack-neutron12:24
*** dbite has joined #openstack-neutron12:25
*** ihrachyshka_ has joined #openstack-neutron12:25
*** SridharG has quit IRC12:25
openstackgerritRobin Wang proposed a change to openstack/neutron: OVS-Agent crashes when OVS dead(Alternative I)  https://review.openstack.org/11053812:25
*** nlahouti has quit IRC12:26
*** baoli has quit IRC12:26
*** ihrachyshka has quit IRC12:28
*** yamamoto has quit IRC12:31
*** swat30 has quit IRC12:32
*** saju_m has joined #openstack-neutron12:33
*** swat30 has joined #openstack-neutron12:33
*** ramishra has quit IRC12:35
*** ramishra_ has joined #openstack-neutron12:36
*** kopparam has quit IRC12:36
*** saju_m has quit IRC12:38
*** lordd_ has quit IRC12:39
*** saju_m has joined #openstack-neutron12:39
*** zhipeng has quit IRC12:43
*** Dafna has quit IRC12:44
pcm_jschwarz: pong12:44
jschwarzwas gonna ask you if you saw the answer I wrote on your comment on my patch, but you already beat me to it :)12:45
*** gildub has quit IRC12:46
*** AlexF_ has joined #openstack-neutron12:48
openstackgerritBritt Houser proposed a change to openstack/neutron: Change nexus_dict values to be array  https://review.openstack.org/11300912:52
*** nlahouti has joined #openstack-neutron12:53
*** k4n0 has quit IRC12:53
*** aranjan has joined #openstack-neutron12:53
*** banix has quit IRC12:54
*** jistr has quit IRC12:56
*** rotbeard has quit IRC12:56
*** nlahouti has quit IRC12:57
*** jistr has joined #openstack-neutron12:57
*** heyongli has joined #openstack-neutron12:57
*** aranjan has quit IRC12:58
*** nati_ueno has joined #openstack-neutron12:58
*** AlexF_ has quit IRC13:00
*** thomasem has joined #openstack-neutron13:00
ZZelleHi, everyone13:00
mesteryZZelle: Hi my friend!13:01
mesteryZZelle: How are you doing?13:01
*** absubram_ has joined #openstack-neutron13:01
ZZelleBetter than ever, summer under the rain :s13:01
mesteryZZelle: Ha! :)13:02
*** HenryG_afk is now known as HenryG13:02
*** kbringard has joined #openstack-neutron13:02
ZZellemestery, one (culture) question, do you if there is any intentions to deprecate mlnx and bigswitch plugins?13:04
ZZellemestery, it seems their ml2 mechanisms are providing the same features13:04
*** AbsinthMind has joined #openstack-neutron13:04
openstackgerritVinod Kumar proposed a change to openstack/neutron: Fix dvr router getting scheduled on router update  https://review.openstack.org/11257613:04
*** julim has joined #openstack-neutron13:04
*** julim has quit IRC13:06
mesteryZZelle: I've not heard anything yet, but perhaps13:07
*** rpothier has joined #openstack-neutron13:07
ZZellemestery, when you look at mlnx README, it seems we should use mlnx ml2 mechanism13:08
mesteryZZelle: Interesting13:08
ZZellemestery, let me find the link13:08
*** WackoRobie has quit IRC13:08
*** WackoRobie has joined #openstack-neutron13:09
ZZellemestery, https://wiki.openstack.org/wiki/Mellanox-Neutron13:09
*** AlexF_ has joined #openstack-neutron13:10
*** SridharG has joined #openstack-neutron13:10
ZZelleirenab, hi13:11
mesteryZZelle: cool13:13
*** WackoRobie has quit IRC13:13
*** shashankhegde has joined #openstack-neutron13:13
ZZellemestery, when i look at bigswitch plugin and ml2 mechanism it seems the first one can be deprecated in favor of the mechanism13:14
mesteryZZelle: +10013:14
mesteryZZelle: Also, the Cisco standalone plugin will be deprecated this cycle as well in favor of the Nexus ML2 driver13:14
*** lordd_ has joined #openstack-neutron13:14
ZZellemestery, sure? it seems (based on module/package names) the plugin supports more cisco appliances?13:15
*** kbringard has quit IRC13:15
HenryGZZelle: You are correct13:15
*** BillTheKat has joined #openstack-neutron13:15
*** kbringard has joined #openstack-neutron13:15
HenryGZZelle: Only the Nexus sub-plugin will be deprecated. The n1kv sub-plugin will remain.13:16
ZZelleHenryG, ok13:16
ZZellekevinbenton, hi13:17
*** jgallard has quit IRC13:17
*** phil_h has joined #openstack-neutron13:18
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Plugin/DB additions for version 2 of LBaaS API  https://review.openstack.org/10560913:19
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Tests for extension, db and plugin for LBaaS V2  https://review.openstack.org/10561013:19
openstackgerritAvishay Balderman proposed a change to openstack/neutron: New extension for version 2 of LBaaS API  https://review.openstack.org/10533113:19
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Radware LBaaS v2 driver  https://review.openstack.org/11351613:19
*** ivar-lazzaro1 has quit IRC13:19
*** mrsnivvel has quit IRC13:21
*** changbl has quit IRC13:21
*** AlexF_ has quit IRC13:21
ZZellemestery, only core plugins "owners" can choose to deprecate them?13:22
mesteryZZelle: Yes13:22
*** BillTheKat has quit IRC13:22
*** nati_ueno has quit IRC13:22
ZZellemestery, so let hope we have more owners like yamamoto!13:23
*** nlahouti has joined #openstack-neutron13:23
mesteryZZelle: ++13:24
*** julim has joined #openstack-neutron13:24
*** blogan has quit IRC13:27
*** ptoohill has quit IRC13:28
*** TrevorV has quit IRC13:28
*** nlahouti has quit IRC13:28
*** evgenyf has quit IRC13:29
*** jgallard has joined #openstack-neutron13:30
*** garywils has joined #openstack-neutron13:30
*** AlexF_ has joined #openstack-neutron13:31
irenabZZelle: hi13:31
irenabZZelle: hi13:31
irenabhi13:32
*** yamamoto has joined #openstack-neutron13:32
ZZelleirenab, one question about mlnx plugin and ml2 mechanism13:32
irenabhi13:32
*** baoli has joined #openstack-neutron13:32
*** irenab has left #openstack-neutron13:32
*** irenab has joined #openstack-neutron13:32
irenabZZelle:hi13:32
*** aveiga has joined #openstack-neutron13:32
*** ptoohill has joined #openstack-neutron13:33
ZZelleirenab, when i look at the wiki page on mlnx, it seems ml2 mechanism is the prefered way to interact with mlnx13:33
*** blogan has joined #openstack-neutron13:33
ZZelleirenab, am i right?13:33
ZZelleirenab, https://wiki.openstack.org/wiki/Mellanox-Neutron13:34
*** TrevorV has joined #openstack-neutron13:34
irenabZZelle: yes. Initally in Havana there was monolitic plugin, and in Icehouse ML2  MD was added to provide same functionality13:34
*** zhipeng has joined #openstack-neutron13:34
irenabZZelle: any specific issue that concerns you?13:35
ZZelleirenab, any intention to deprecate the mlnx plugin if the ml2 mechanism provides the same feature?13:35
irenabZZelle: yes. Actually right now, looking into mail thread regarding the way to deprecate plugin :-)13:36
*** yamamoto has quit IRC13:36
ZZelleok, great13:36
ZZellemestery, ^13:36
irenabZZelle: had to reconnect and do not see previous content. Any concern that mestery has?13:37
*** claudiub has joined #openstack-neutron13:37
*** netuser has joined #openstack-neutron13:39
*** TrevorV has quit IRC13:39
*** garywils is now known as gary13:39
*** gary is now known as gaz13:39
*** ptoohill has quit IRC13:39
*** gaz is now known as gazza13:40
*** blogan has quit IRC13:40
netuserQUESTION: When I run stack.sh, in neutron only 1 network is created, no public network is created ??????13:40
*** reaper has joined #openstack-neutron13:40
*** reaper has quit IRC13:41
pcm_netuser: What's your localrc have?13:41
netuserdisable_service n-net, enable_service q-svc, enable_service q-agt, enable_service q-dhcp, enable_service q-13, enable_service q-meta, enable_service neutron13:43
*** rook has joined #openstack-neutron13:43
netuseralso13:43
netuserRECLONE=yes13:43
*** ivar-lazzaro has joined #openstack-neutron13:43
netuserI only see private network after running stack.sh, even though I'm supposed to have 2: private and public13:44
*** phil_h has quit IRC13:45
*** crose has joined #openstack-neutron13:45
*** ptoohill has joined #openstack-neutron13:45
pcm_netuser: Check lib/neutron script to see what  conditions are needed to create the second network.13:45
*** blogan has joined #openstack-neutron13:46
pcm_netuser: YOu can also look at the stack.sh.log (if set) and see what nets it is creating.13:46
*** AbsinthMind has quit IRC13:46
*** TrevorV has joined #openstack-neutron13:46
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Plugin/DB additions for version 2 of LBaaS API  https://review.openstack.org/10560913:46
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Tests for extension, db and plugin for LBaaS V2  https://review.openstack.org/10561013:46
openstackgerritAvishay Balderman proposed a change to openstack/neutron: New extension for version 2 of LBaaS API  https://review.openstack.org/10533113:46
openstackgerritAvishay Balderman proposed a change to openstack/neutron: Radware LBaaS v2 driver  https://review.openstack.org/11352513:47
pcm_netuser: There is a create_neutron_initial_network function that creates the network/subnet13:48
openstackgerritIhar Hrachyshka proposed a change to openstack/neutron: Use jsonutils instead of stdlib json  https://review.openstack.org/11270813:48
*** AbsinthMind has joined #openstack-neutron13:49
*** avishnoi has left #openstack-neutron13:49
*** jistr has quit IRC13:49
netuserpcm_:Yeah, I'm looking at it right now13:49
pcm_netuser: Should have l3 service enabled to get public net.13:49
pcm_netuser: seems like you do.13:49
*** jistr has joined #openstack-neutron13:50
*** irenab has quit IRC13:50
*** AbsinthMind has quit IRC13:50
*** jprovazn has joined #openstack-neutron13:51
netuserpcm_: It also doesn't create br-ex bridge13:51
*** WackoRobie has joined #openstack-neutron13:51
*** marun has joined #openstack-neutron13:51
netuserwhich is for public network13:51
pcm_netuser: May need 'OVS_PHYSICAL_BRIDGE=br-ex' in localrc.13:52
*** flaviof_zzz is now known as flaviof13:53
*** aranjan has joined #openstack-neutron13:53
*** evgenyf has joined #openstack-neutron13:53
*** ramishra_ has quit IRC13:54
*** ramishra has joined #openstack-neutron13:54
netuserpcm_: I ran another installation on another vm without setting OVS_PHYSICAL_BRIDGE and stack.sh created br-ex automatically13:54
*** nlahouti has joined #openstack-neutron13:54
*** FJB has joined #openstack-neutron13:55
*** aranjan has quit IRC13:57
*** julim has quit IRC13:58
*** heyongli has quit IRC13:59
*** flwang_ has joined #openstack-neutron13:59
*** xuhanp has joined #openstack-neutron13:59
*** nlahouti has quit IRC13:59
pcm_netuser: Not sure if that should be the default used. You'd have to check the scripts. However, I had one system w/o br-ex and had to set that variable.13:59
*** julim has joined #openstack-neutron14:01
*** ramishra has quit IRC14:02
*** flwang_ has quit IRC14:03
*** WackoRobie has quit IRC14:04
*** WackoRobie has joined #openstack-neutron14:04
openstackgerritSergey Shnaidman proposed a change to openstack/neutron: Add None as one of valid ipv6 attributes  https://review.openstack.org/11314514:05
*** afazekas has quit IRC14:05
pcm_netuser: You can post your whole localrc using paste.openstack.org so folks here can look at your config and comment.14:06
*** banix has joined #openstack-neutron14:06
openstackgerritSergey Shnaidman proposed a change to openstack/neutron: Add None as one of valid ipv6 attributes  https://review.openstack.org/11314514:06
*** gazza has quit IRC14:08
*** armax has joined #openstack-neutron14:10
*** phil_h has joined #openstack-neutron14:13
*** hanzhou has joined #openstack-neutron14:14
*** jistr has quit IRC14:14
*** otherwiseguy has joined #openstack-neutron14:15
*** FJB has quit IRC14:15
*** jistr has joined #openstack-neutron14:15
*** FJB has joined #openstack-neutron14:19
*** FJB has quit IRC14:19
pcm_netuser: Key will be to see what the log shows and then you can do failed commands manually and see what happens.14:19
*** FJB has joined #openstack-neutron14:19
*** afazekas has joined #openstack-neutron14:20
*** jistr has quit IRC14:21
*** ijw has joined #openstack-neutron14:21
*** jistr has joined #openstack-neutron14:21
*** gazza has joined #openstack-neutron14:22
*** jgallard has quit IRC14:25
*** nlahouti has joined #openstack-neutron14:25
*** ijw has quit IRC14:25
*** dfarrell07 has joined #openstack-neutron14:27
*** nlahouti has quit IRC14:29
*** tongli has joined #openstack-neutron14:30
*** seizadi has joined #openstack-neutron14:31
*** yamamoto has joined #openstack-neutron14:32
*** salv-orlando has quit IRC14:33
*** ivar-lazzaro has quit IRC14:35
*** Dafna has joined #openstack-neutron14:35
*** nlahouti has joined #openstack-neutron14:36
netuserpcm:_ OK, I fixed it.14:36
netuserpcm:_ I have another question14:36
pcm_netuser: What was the fix?14:37
*** yamamoto has quit IRC14:37
netuserpcm:_ I just ran stack.sh14:37
netuserand it worked for some reason14:38
*** jprovazn has quit IRC14:38
netuserpcm_: that was weird14:38
pcm_netuser: yes14:38
netuserpcm_: I have a port group/bridge in vmware vcenter which I use for vm deployment14:38
netuserpcm_: how can I use it in neutron14:39
*** mrmoje has joined #openstack-neutron14:39
pcm_netuser: Afraid I don't know about VMWare at all.14:39
netuserpcm_: I ran: sudo ovs-vsctl add-br br112 , sudo ovs-vsctl add-port br112 eth314:39
netuserpcm_: how can I tell neutron to use that bridge14:40
*** networkstatic has joined #openstack-neutron14:40
*** nlahouti has quit IRC14:40
*** gazza has quit IRC14:40
*** SridharG has quit IRC14:41
pcm_netuser: Not really sure. Maybe specifying eth3 as PUBLIC_INTERFACE (if that is what you want to use it for)? Not really sure. I usually just do a plain setup for development.14:43
pcm_netuser: You may want to ask on openstack IRC for usage Openstack questions as there may be people there better able to answer.14:43
pcm_this IRC is typically for development related questions in Neutron.14:44
*** sbfox has joined #openstack-neutron14:44
*** markmcclain has joined #openstack-neutron14:44
netuserpcm_: I'm actually doing dev as well, I just need to setup neutron14:45
larsksnetuser: You probably want to ask "how do i configure" questions on #openstack.14:45
netuserpcm_: and isn't setup a part of the devepment  ?14:45
*** vivek-ebay has joined #openstack-neutron14:46
larsksWhereas this is a good place for "I am writing a neutron plugin and I need help understanding the internal api".14:46
pcm_netuser: #openstack is for usage questions. #openstack-neutron is for Neutron coding questions.14:47
*** dfarrell07 has quit IRC14:47
*** ollivier has quit IRC14:48
pcm_netuser: Not that people here can't answer your questions, but it's not the focus of the list. Just trying to guide you to the best place to get help.14:50
netuserpcm_: ok, go tit14:51
pcm_netuser: I'm not very versed in the different config setups. Usually just have a basic config and then tweak it as needed once and a while.14:52
*** AlexF_ has quit IRC14:52
openstackgerritWei Wang proposed a change to openstack/neutron: "if dict.get(key):" over "if key in dict and dict[key]:"  https://review.openstack.org/11251114:53
*** aranjan has joined #openstack-neutron14:53
*** ihrachyshka_ has quit IRC14:55
*** jdag_ is now known as jdag14:55
*** AlexF_ has joined #openstack-neutron14:57
*** carl_baldwin has joined #openstack-neutron14:58
*** nati_ueno has joined #openstack-neutron14:58
*** yeungp has joined #openstack-neutron14:58
*** sbfox has quit IRC14:59
*** markmcclain1 has joined #openstack-neutron15:00
*** mrmoje has quit IRC15:00
*** markmcclain has quit IRC15:01
*** jlibosva has quit IRC15:01
*** devvesa has quit IRC15:02
*** nati_ueno has quit IRC15:03
*** dfarrell07 has joined #openstack-neutron15:04
*** radez_g0n3 is now known as radez15:04
openstackgerritA change was merged to openstack/neutron: Add delete operations for the ODL MechanismDriver  https://review.openstack.org/9645915:06
*** WackoRobie has quit IRC15:06
*** WackoRobie has joined #openstack-neutron15:07
*** aranjan has quit IRC15:07
openstackgerritWei Wang proposed a change to openstack/neutron: DuplicateSecurityGroupRuleInPost should point out rule id  https://review.openstack.org/11355115:07
*** gazza has joined #openstack-neutron15:08
*** afazekas has quit IRC15:08
*** nlahouti has joined #openstack-neutron15:09
*** WackoRobie has quit IRC15:09
*** WackoRobie has joined #openstack-neutron15:09
*** dbite has quit IRC15:12
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix AttributeError when setting external gateway on DVR router  https://review.openstack.org/11214615:12
*** banix has quit IRC15:12
*** amuller_ has joined #openstack-neutron15:13
*** WackoRobie has quit IRC15:14
*** amuller has quit IRC15:15
*** ramishra has joined #openstack-neutron15:15
*** absubram_ has quit IRC15:15
*** xgerman has joined #openstack-neutron15:16
*** jorgem has joined #openstack-neutron15:20
*** mlavalle has joined #openstack-neutron15:20
*** mlavalle has quit IRC15:20
openstackgerritMiguel Angel Ajo proposed a change to openstack/neutron: Watch child processes and take action when any of them die  https://review.openstack.org/11279815:20
*** ramishra has quit IRC15:21
*** ramishra has joined #openstack-neutron15:21
*** afazekas has joined #openstack-neutron15:21
*** lordd_ has quit IRC15:21
*** ijw has joined #openstack-neutron15:22
*** jckasper has quit IRC15:22
*** lordd_ has joined #openstack-neutron15:23
*** vivek-eb_ has joined #openstack-neutron15:24
*** changbl has joined #openstack-neutron15:24
*** vivek-ebay has quit IRC15:26
*** skolekonov has quit IRC15:26
*** ijw has quit IRC15:26
armaxenikanorov: ping15:27
enikanorov_armax: pong15:27
*** amotoki has joined #openstack-neutron15:27
armaxenikanorov_: re patch 113144'15:27
armaxenikanorov_: are you still working on it?15:28
*** xuhanp has quit IRC15:28
*** dbite has joined #openstack-neutron15:28
enikanorov_armax: I am. what are your suggestions?15:28
armaxenikanorov_: did you look into Rossella’s comment further?15:28
enikanorov_armax: yes, i've replied to it15:28
*** regXboi has joined #openstack-neutron15:28
*** saju_m has quit IRC15:29
armaxenikanorov_: ok, in relation to the log noise, have you had a chance to look at the NotFound errors for the firewall?15:29
armaxenikanorov_: I’ve been meaning to look at those for a while15:29
enikanorov_no, I didn't do it for firewall. we did fix similar issue for lbaas, which ended with lowering log level for such cases15:30
*** AbsinthMind has joined #openstack-neutron15:30
enikanorov_because that's the nature of API tests that delete objects before they get reflected on the backend15:31
enikanorov_so agents try to access already-deleted objects, i guess it's the same for firewall15:31
armaxenikanorov_: well, this is the same type of failure that bug #1355409 is about15:31
*** jschwarz has quit IRC15:32
enikanorov_understood. it's just strange that it started to appear a not so long ago15:32
armaxenikanorov_: you mean the firewall errors?15:32
*** FJB has quit IRC15:32
*** yamamoto has joined #openstack-neutron15:32
enikanorov_Im mean PortNotFound15:32
armaxenikanorov_: true, I tried to root cause it15:33
armaxenikanorov_: but I couldn’t find anything obvious…that code has been like that for a while15:33
armaxenikanorov_: perhaps it was a time bomb waiting to explode :)15:33
enikanorov_maybe it's something on tempest side, i don't know15:34
openstackgerritTri Hoang Vo proposed a change to openstack/neutron: Detect if iproute2 allows delete namespace  https://review.openstack.org/10636515:34
enikanorov_then it might make sense to keep those traces in the logs15:34
enikanorov_armax: just to not mask the issue15:34
*** mlavalle has joined #openstack-neutron15:35
*** afazekas has quit IRC15:35
*** rwsu has joined #openstack-neutron15:35
armaxenikanorov_: we should really cleaning up the logs from all the errors and start voting -1 on traces15:35
armaxenikanorov_: *should be cleaning15:36
*** magesh_ has joined #openstack-neutron15:36
enikanorov_armax: agree on that also15:37
armaxenikanorov_: only then it becomes more clear if it’s a regression, or an hidden timing issue15:37
*** yamamoto has quit IRC15:37
armaxenikanorov_: do you know if there’s anyone working on that?15:37
*** magesh_ has quit IRC15:38
enikanorov_armax: no, i don't know anybody looking into that except us15:39
armaxenikanorov_: ok, I guess that would be us then :)15:40
*** hanzhou has quit IRC15:41
*** ramishra_ has joined #openstack-neutron15:48
*** ramishra_ has quit IRC15:48
*** shashankhegde has quit IRC15:48
*** ramishra_ has joined #openstack-neutron15:48
*** FJB has joined #openstack-neutron15:49
*** ramishra has quit IRC15:51
*** jgallard has joined #openstack-neutron15:53
*** aranjan has joined #openstack-neutron15:53
*** ivar-lazzaro has joined #openstack-neutron15:54
*** reed has joined #openstack-neutron15:54
*** emagana has joined #openstack-neutron15:55
*** ihrachyshka has joined #openstack-neutron15:55
*** dgollub has quit IRC15:55
*** annegent_ has joined #openstack-neutron15:55
*** aranjan has quit IRC15:58
*** matrohon has quit IRC15:59
*** banix has joined #openstack-neutron15:59
*** nati_ueno has joined #openstack-neutron15:59
*** flwang_ has joined #openstack-neutron16:00
mesteryamuller_: Does this one (https://blueprints.launchpad.net/neutron/+spec/tenant-delete) still have a chance for Juno? Will you push code by FPF next Thursday?16:01
amuller_mestery: Good question... I've been working on the VRRP spec very intensely in the past few weeks16:02
openstackgerritElena Ezhova proposed a change to openstack/neutron: Send HTTP exceptions in the format expected by neutronclient  https://review.openstack.org/11357016:03
*** jgallard has quit IRC16:03
*** a_le has joined #openstack-neutron16:03
*** nati_ueno has quit IRC16:03
amuller_mestery: I have code up, one patch is missing. I'm going to try for Juno.16:04
mesteryamuller_: Awesome!16:04
*** flwang_ has quit IRC16:04
*** jckasper has joined #openstack-neutron16:04
*** ihrachyshka has quit IRC16:06
*** tongli has quit IRC16:09
*** vivek-ebay has joined #openstack-neutron16:10
*** seizadi has quit IRC16:10
*** _cjones_ has joined #openstack-neutron16:11
*** ajo__ has joined #openstack-neutron16:12
*** yfried_ has joined #openstack-neutron16:13
*** sbfox has joined #openstack-neutron16:13
*** vivek-eb_ has quit IRC16:14
*** yfried has quit IRC16:14
*** bvandenh has quit IRC16:14
*** vivek-ebay has quit IRC16:14
*** evgenyf has quit IRC16:15
*** amuller__ has joined #openstack-neutron16:15
*** yfried__ has joined #openstack-neutron16:16
*** ajo_ has quit IRC16:16
*** FJB has quit IRC16:16
*** SridharRamaswamy has joined #openstack-neutron16:16
*** WackoRobie has joined #openstack-neutron16:17
*** pasquier-s has quit IRC16:17
*** ajo_ has joined #openstack-neutron16:18
*** amuller_ has quit IRC16:19
openstackgerritWei Wang proposed a change to openstack/neutron: SecurityGroupRuleExists should point out rule id inseand of group id  https://review.openstack.org/11355116:19
*** yfried_ has quit IRC16:20
*** ajo__ has quit IRC16:20
*** yfried__ has quit IRC16:22
*** shashankhegde has joined #openstack-neutron16:22
*** hajay has joined #openstack-neutron16:23
*** bjornar_ has joined #openstack-neutron16:23
*** sbalukoff has quit IRC16:25
openstackgerritstephen-ma proposed a change to openstack/neutron: Delete DVR namespaces on node after removing last VM  https://review.openstack.org/11246516:25
*** hajay has left #openstack-neutron16:25
*** jistr has quit IRC16:25
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix KeyError during sync_routers  https://review.openstack.org/11358016:28
*** pballand has joined #openstack-neutron16:28
*** yamamoto has joined #openstack-neutron16:32
*** shashankhegde has quit IRC16:35
*** chandankumar has quit IRC16:36
*** AlexF_ has quit IRC16:37
*** yamamoto has quit IRC16:37
*** vivek-ebay has joined #openstack-neutron16:40
diegowsAny good presentation about neutron to recommend? We are going to talk about it in OpenStack Latinamerica hangout today and I don't have time to reinvent the wheel. Something short just to start the discussion, english, spanish, doesn't matter16:41
diegowsI mean, any good slide :)16:41
diegowsI'll keep the credits to the origin author of course16:41
*** ramishra_ has quit IRC16:41
*** markmcclain1 has quit IRC16:42
EmilienMdiegows: this one is good: http://www.slideshare.net/enovance/mc-clainopenstack-in-action16:43
*** sbfox has quit IRC16:45
*** thomasem_ has joined #openstack-neutron16:46
diegowsEmilienM, looks good16:46
diegowsthanks16:46
*** thomasem has quit IRC16:47
*** banix has quit IRC16:47
*** banix has joined #openstack-neutron16:48
*** aranjan has joined #openstack-neutron16:48
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Preserve link local IP allocations for DVR fip ns across restart  https://review.openstack.org/10948616:48
*** ramishra has joined #openstack-neutron16:48
*** dsneddon has joined #openstack-neutron16:49
*** seizadi has joined #openstack-neutron16:49
anteayaso actually I think mestery should weigh in on this thread first emagana: http://lists.openstack.org/pipermail/openstack-dev/2014-August/042746.html16:50
*** seizadi has quit IRC16:50
* mestery looks16:50
*** thomasem_ has quit IRC16:50
*** ramishra_ has joined #openstack-neutron16:50
emaganaanteaya: Thanks for following up16:50
*** ramishra_ has quit IRC16:51
emaganamestery: I believe we are missing the way we "audit" plugins/drivers CI16:51
*** oreillyd has quit IRC16:51
*** ramishra_ has joined #openstack-neutron16:51
anteayaemagana: good point16:51
mesteryemagana: I'm looking for someone to lead that role, interested? :)16:51
mesteryIt's a full time job.16:51
mesteryIdeally we want it automated16:52
emaganaanteaya: Is there any process in place in Infra for validation of CIs?16:52
anteayano16:52
anteayaactually it is a known pain point16:52
emaganamestery: If you want me to not have any more friends in the community, sure!16:52
anteayaand something we need some resources on16:52
anteayawhile automating is a direction16:52
mesteryemagana: hahahahahahahaha16:52
emaganamestery: ha ha ha ha16:52
anteayawe need to agree on what we are automating16:52
emaganamestery: Let's start for "manual" validation, once we have a process in place we can propose an automation16:53
anteayaso it will be largely manual for a long while until we even have an agreed upon set of data that we can collect and evaluate to demonstrate "validation"16:53
anteayaemagana: jinx16:53
emaganaanteaya: same thought!16:53
mesteryWorks for me16:53
anteaya:D16:53
anteayaso are you willing to take this on?16:53
anteayado you have time with documentation?16:54
anteayaI want to work with you but documentation can't suffer as a result16:54
*** ramishra has quit IRC16:54
emaganaanteaya: Documentation is my priority right now, specially for this cycle because Neutron will sufeer a lot of changes16:54
emaganaHowever, I am not the writer most of the time, the great docs team help me to do that16:55
*** AbsinthMind has quit IRC16:55
emaganaI mainly help the Docs team to identidy what needs to be changed what is the best way to add the new content16:56
*** SumitNaiksatam has quit IRC16:56
emaganaanteaya & mestery: I can start reviewing what plugins/drivers we have in the tree and create a matrix (if it does not exist already) that should be reviewed weekly16:57
mesteryemagana: We have that matrix, I can point you at it.16:57
emaganaI am ware of the Drivelog16:57
mesteryemagana: Before you duplicate any work, lets syncup on this. :)16:57
mesteryemagana: DriverLog is anteaya'16:57
mesteryfavorites thing16:57
mestery^^^16:57
mestery:)16:57
emaganaI thought DriveLog was a Mirantis activity  :-(16:58
emaganaMy bad!16:58
*** ihrachyshka has joined #openstack-neutron16:58
*** a_le has quit IRC16:58
*** amuller__ has quit IRC16:58
*** SridharG has joined #openstack-neutron16:59
mesteryI think the issues with DriverLog are around it's usage of certified, etc.16:59
*** amuller__ has joined #openstack-neutron16:59
mesteryThere's a thread on that, not sure how it ended up though.16:59
*** hajay__ has joined #openstack-neutron16:59
*** a_le has joined #openstack-neutron16:59
*** hajay__ has quit IRC17:00
*** arosen has joined #openstack-neutron17:00
*** nati_ueno has joined #openstack-neutron17:00
emaganamestery: I can weekly run a validation of the plugins/drivers17:00
*** hajay__ has joined #openstack-neutron17:00
*** jpich has quit IRC17:00
emaganaI can report what is missing and failing or something weird..17:01
mesteryemagana: That's what the weekly third party meeting does already, it would be good to coordinate there.17:01
mesteryMake sense?17:01
mesteryThat would be helpful though!17:01
mesteryI did that for a while, but my plate is constantly overfilled and things fall off all the time17:01
emaganamestery: Understood!17:02
*** mestery has quit IRC17:02
*** WackoRobie has quit IRC17:02
*** magesh_ has joined #openstack-neutron17:02
*** hajay__ has left #openstack-neutron17:02
*** mestery has joined #openstack-neutron17:03
emaganaSo far, it is just you coordinating that meeting, sorry I have not attend it lately17:03
*** WackoRobie has joined #openstack-neutron17:03
*** thomasem has joined #openstack-neutron17:03
emaganahappy to not support any plugin/driver anymore!17:03
emagana:-)17:03
*** shashankhegde has joined #openstack-neutron17:04
*** ijw has joined #openstack-neutron17:04
*** rudrajit has joined #openstack-neutron17:05
*** nati_ueno has quit IRC17:05
*** seizadi has joined #openstack-neutron17:06
*** a_le has quit IRC17:06
*** WackoRobie has quit IRC17:07
*** a_le has joined #openstack-neutron17:07
*** jschwarz has joined #openstack-neutron17:07
*** ianw has quit IRC17:08
emaganamestery: This is the meeting right: https://wiki.openstack.org/wiki/Meetings#Third_Party_Meeting17:08
emaganamestery: Is there one specific for Neutron? I remember you started one17:08
*** SridharG has quit IRC17:09
mesteryemagana: Nope, that's the one which is active17:12
*** SumitNaiksatam has joined #openstack-neutron17:12
emaganamestery: good!17:13
*** bjornar_ has quit IRC17:13
anteayagreat17:14
anteayaso yes a weekly update on plugin/driver data would be great17:14
*** claudiub has quit IRC17:14
anteayaif you just start by gathering any data and displaying it17:14
*** kaiserpathos has joined #openstack-neutron17:14
anteayawe can iterate on how to present it for consumption17:14
anteayaand then we can figure out what the data means17:15
anteayaand when a data point is considered actionable17:15
anteayaemagana: is that fair?17:15
*** krtaylor has quit IRC17:15
emaganaanteaya: it sounds good to me!17:15
emaganaI will have the report ready for the meeting17:15
anteayaemagana: anything you start with will be great17:16
anteayajust gather something and we can go from there17:16
anteayaand thank you17:16
emaganaanteaya: perfect!17:16
anteayathank you!17:17
* anteaya does a happy dance17:17
emaganaanteaya: On a very different topic, do you know if the multinode spec has been delayed?17:17
emaganaanteaya: I dont see any reviews on it besides Carl and Rosella's ones17:17
anteayaah17:17
anteayaokay so some info you probably don't have17:17
anteayayou did a great job on the spec in MN and I thank you17:18
anteayaI was wrong17:18
anteayasince I thought it was a seed spec for work on mulinode in germany17:18
anteayaand it wasn't17:18
anteayaso my fault there17:18
anteayasdague and attila worked on it in germany17:18
anteayaand as far as I know multinode capability is available17:19
anteayaso today is tuesday17:19
anteayaso show up at the infra meeting today17:19
anteayaand you can add a discussion item to today's agenda17:19
anteayaor hope for some open discussion time17:19
anteayaalso I suggest attending the qa meeting on thursday17:20
anteayaemagana: sorry you didn't have this info prior to now17:20
emaganaanteaya: n.p.17:21
*** igordcard has joined #openstack-neutron17:21
anteayaso does that help?17:21
emaganaSo, this spec is not required anymore? Is that correct?17:21
anteayaactually come to the infra meeting today17:21
anteayalet's get some clarity on that17:21
emaganaok.. let me chack the time..17:21
emaganacheck*17:21
anteayaclarkb has some thoughts and I would like you to hear from him17:21
emaganaanteaya: Its a 1900 UTC right?17:22
* anteaya checks too17:22
anteaya1900 yes: https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting17:22
*** ijw_ has joined #openstack-neutron17:24
mtreinisharmax: hey on https://review.openstack.org/#/c/113414/ if you update the commit msg I'll give it a +2 right now17:24
armaxmtreinish: yes, sir, right on it17:25
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove SELECT FOR UPDATE from db_base_plugin  https://review.openstack.org/10096317:25
mtreinisharmax: cool, thanks17:25
armaxmtreinish: my bad for not being more thorough17:26
*** bjornar_ has joined #openstack-neutron17:26
*** SridharRamaswamy has quit IRC17:26
*** aranjan_ has joined #openstack-neutron17:26
*** SridharRamaswamy has joined #openstack-neutron17:27
*** AbsinthMind has joined #openstack-neutron17:28
*** ijw_ has quit IRC17:28
mtreinisharmax: no worries, I'm equally guilty of nondescript commit msgs17:28
mtreinisharmax: on that second patch it needs a rebase17:28
*** SridharR_ has joined #openstack-neutron17:28
armaxmtreinish: it merges fine though17:28
mtreinishit's based on an old rev of the todo removal patch, you just need to push the rebase button17:29
armaxmtreinish: I’ll rebase it one more time once I am done with the commit message…making sure I don’t make grammar mistakes or typos17:29
armax:)17:29
*** aranjan has quit IRC17:30
*** aranjan_ has quit IRC17:31
*** SridharRamaswamy has quit IRC17:31
*** yamamoto has joined #openstack-neutron17:32
*** dsneddon has quit IRC17:32
armaxmtreinish: done17:33
*** ramishra has joined #openstack-neutron17:33
armaxmtreinish: thanks again17:33
*** yfried__ has joined #openstack-neutron17:36
*** absubram_ has joined #openstack-neutron17:36
*** ramishra_ has quit IRC17:36
*** emagana has quit IRC17:36
*** yamamoto has quit IRC17:37
*** emagana has joined #openstack-neutron17:37
mtreinisharmax: sure np. I'm glad to remove that bit. IIRC I was opposed to adding it in the first place :)17:38
armaxmtreinish: I’ll keep an eye if anything regresses17:38
armaxmtreinish: but as far as I can tell if the cleanup is not there, code would go astray, it’d be good if mlavalle could chime in too17:39
*** salv-orlando has joined #openstack-neutron17:40
*** ramishra_ has joined #openstack-neutron17:40
mtreinishthanks, but I doubt it will, the bug which caused us to add that was fixed a long time ago. I think someone else would have noticed if it were still a problem17:40
mtreinisharmax: yeah getting his input would be good17:40
*** SayajiPatil has joined #openstack-neutron17:42
*** ramishra has quit IRC17:42
*** sbalukoff has joined #openstack-neutron17:47
*** flwang_ has joined #openstack-neutron17:48
*** ramishra_ has quit IRC17:49
*** vishwanathj has joined #openstack-neutron17:49
*** spandhe_ has joined #openstack-neutron17:50
*** flwang_ has quit IRC17:52
*** kaiserpathos has quit IRC17:53
*** padkrish has joined #openstack-neutron17:53
*** dbite has quit IRC17:56
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix PortNotFound exception during sync_routers  https://review.openstack.org/11344117:58
*** mestery has quit IRC18:00
*** nati_ueno has joined #openstack-neutron18:01
*** VijayB has joined #openstack-neutron18:01
*** aveiga has quit IRC18:02
*** amuller__ has quit IRC18:03
*** jschwarz has quit IRC18:03
*** thomasem has quit IRC18:04
*** jckasper has quit IRC18:04
*** jschwarz has joined #openstack-neutron18:04
*** banix has quit IRC18:05
*** garyduan has joined #openstack-neutron18:05
*** nati_ueno has quit IRC18:05
*** jckasper has joined #openstack-neutron18:06
*** gduan has quit IRC18:06
*** jschwarz has quit IRC18:07
*** garyk1 has quit IRC18:08
*** otherwiseguy has quit IRC18:08
*** thomasem has joined #openstack-neutron18:09
*** AbsinthMind has quit IRC18:10
*** BillTheKat has joined #openstack-neutron18:10
*** AbsinthMind has joined #openstack-neutron18:11
*** k_stev has joined #openstack-neutron18:11
*** sbfox has joined #openstack-neutron18:12
*** AbsinthMind1 has joined #openstack-neutron18:13
*** gazza has quit IRC18:15
*** sbfox1 has joined #openstack-neutron18:15
*** AbsinthMind has quit IRC18:15
*** sbfox has quit IRC18:17
*** ajo__ has joined #openstack-neutron18:17
*** _cjones_ has quit IRC18:18
*** _cjones_ has joined #openstack-neutron18:19
*** padkrish has quit IRC18:19
*** krtaylor has joined #openstack-neutron18:19
*** padkrish has joined #openstack-neutron18:20
*** ajo_ has quit IRC18:20
openstackgerritRudra Rugge proposed a change to openstack/neutron: Opencontrail plug-in implementation for core resources  https://review.openstack.org/9663018:20
*** padkrish_ has joined #openstack-neutron18:20
*** marun has quit IRC18:21
*** marun has joined #openstack-neutron18:21
*** markmcclain has joined #openstack-neutron18:21
*** a_le has quit IRC18:23
*** a_le has joined #openstack-neutron18:23
*** nlahouti has quit IRC18:24
*** padkrish has quit IRC18:24
*** AbsinthMind1 has quit IRC18:27
*** a_le has quit IRC18:28
*** nlahouti has joined #openstack-neutron18:28
*** a_le has joined #openstack-neutron18:29
*** ajo_ has joined #openstack-neutron18:30
*** spandhe_ has quit IRC18:31
*** a_le has quit IRC18:31
*** ajo__ has quit IRC18:31
*** a_le has joined #openstack-neutron18:32
*** padkrish_ has quit IRC18:32
*** Longgeek has quit IRC18:32
*** yamamoto has joined #openstack-neutron18:32
*** crc32 has joined #openstack-neutron18:32
*** padkrish has joined #openstack-neutron18:32
*** crc32 has quit IRC18:33
*** crc32 has joined #openstack-neutron18:34
*** dbite has joined #openstack-neutron18:34
*** vivek-eb_ has joined #openstack-neutron18:34
*** dsneddon has joined #openstack-neutron18:35
*** vivek-eb_ has quit IRC18:35
*** dconde has joined #openstack-neutron18:35
*** spandhe_ has joined #openstack-neutron18:36
*** vivek-eb_ has joined #openstack-neutron18:36
*** rudrarugge has joined #openstack-neutron18:36
*** yamamoto has quit IRC18:36
openstackgerritSwaminathan Vasudevan proposed a change to openstack/neutron: Fix for floatingip-delete not removing fip_gw port  https://review.openstack.org/11142118:36
*** padkrish_ has joined #openstack-neutron18:37
*** padkrish has quit IRC18:37
*** vivek-ebay has quit IRC18:37
*** ivar-lazzaro has quit IRC18:37
*** alexpilotti has quit IRC18:39
mlavallearmax: did you oing me?18:40
mlavalleping me?18:40
armaxmlavalle: yeah, was looking for you to comment on: https://review.openstack.org/#/c/113414/18:40
openstackgerritBob Melander proposed a change to openstack/neutron: Adds router service plugin for CSR1kv  https://review.openstack.org/10100218:41
mlavallearmax: ok, I'll take a look later today. is that ok?18:41
armaxmlavalle: yup no worries, thanks18:41
mlavallearmax: :-)18:41
*** banix has joined #openstack-neutron18:41
*** banix has quit IRC18:43
*** banix has joined #openstack-neutron18:44
*** networkstatic has quit IRC18:45
*** catohornet has joined #openstack-neutron18:45
*** sdmonov has joined #openstack-neutron18:50
*** ivar-lazzaro has joined #openstack-neutron18:50
*** ijw_ has joined #openstack-neutron18:51
*** WackoRobie has joined #openstack-neutron18:52
*** seizadi has quit IRC18:53
*** ijw has quit IRC18:54
*** banix has quit IRC18:55
*** ianw has joined #openstack-neutron18:58
*** vivek-eb_ has quit IRC18:58
*** spandhe_ has quit IRC18:58
*** BillTheKat has quit IRC18:59
*** spandhe_ has joined #openstack-neutron19:01
*** sambetts has quit IRC19:03
openstackgerritMarga Millet proposed a change to openstack/neutron: Raise exception for network delete with subnets presents  https://review.openstack.org/11134919:03
*** reed has quit IRC19:03
netuserHow can I tell configure neutron to use a bridge ?19:05
*** rpothier has quit IRC19:06
*** igordcard has quit IRC19:06
*** annegent_ has quit IRC19:10
openstackgerritSam Betts proposed a change to openstack/neutron: Syncronize disable_dhcp_helper to avoid race condition  https://review.openstack.org/11361719:11
*** jckasper_ has joined #openstack-neutron19:11
*** nlahouti has quit IRC19:12
*** netuser has quit IRC19:13
*** jckasper has quit IRC19:14
openstackgerritSam Betts proposed a change to openstack/neutron: Syncronize disable_dhcp_helper to avoid race condition  https://review.openstack.org/11361719:15
*** otherwiseguy has joined #openstack-neutron19:16
*** phil_h has quit IRC19:17
*** ihrachyshka has quit IRC19:21
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix KeyError during sync_routers  https://review.openstack.org/11358019:22
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Refactor l2_pop code to pass mac/ip info more readably  https://review.openstack.org/11217819:24
*** ijw has joined #openstack-neutron19:25
*** magesh_ has quit IRC19:26
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix 404 error fetching metadata when using DVR  https://review.openstack.org/11237619:26
*** mestery has joined #openstack-neutron19:27
openstackgerritArmando Migliaccio proposed a change to openstack/neutron: Fix KeyError during sync_routers  https://review.openstack.org/11358019:27
*** ijw has quit IRC19:29
*** annegent_ has joined #openstack-neutron19:32
*** yamamoto has joined #openstack-neutron19:32
*** AbsinthMind has joined #openstack-neutron19:33
*** WackoRobie has quit IRC19:37
*** yamamoto has quit IRC19:37
*** shashankhegde has quit IRC19:41
*** WackoRobie has joined #openstack-neutron19:43
*** annegent_ has quit IRC19:43
*** ajo_ has quit IRC19:44
*** rook is now known as rook-out19:47
*** flwang_ has joined #openstack-neutron19:49
*** marun has quit IRC19:50
*** marun has joined #openstack-neutron19:50
*** dconde has quit IRC19:53
*** flwang_ has quit IRC19:53
*** SumitNaiksatam has quit IRC19:55
*** VijayB has quit IRC19:56
*** AlexF has joined #openstack-neutron19:57
*** shashankhegde has joined #openstack-neutron19:57
*** seizadi has joined #openstack-neutron19:58
armaxhaleyb: ping20:00
*** jckasper__ has joined #openstack-neutron20:01
*** gangil has joined #openstack-neutron20:02
*** gangil has quit IRC20:02
*** gangil has joined #openstack-neutron20:02
*** amotoki has quit IRC20:02
*** thomasem has quit IRC20:02
*** jorgem has quit IRC20:03
*** jckasper_ has quit IRC20:04
openstackgerritbadveli_vishnuus proposed a change to openstack/neutron: Add service group as a firewall customized service  https://review.openstack.org/10627420:05
*** sbfox1 has quit IRC20:05
*** dave_tucker is now known as dave_tucker_zzz20:06
*** saju_m has joined #openstack-neutron20:08
*** gildub has joined #openstack-neutron20:08
*** baoli has quit IRC20:11
*** saju_m has quit IRC20:13
*** rms_13_ has joined #openstack-neutron20:15
*** nlahouti has joined #openstack-neutron20:15
*** bjornar_ has quit IRC20:15
*** dconde has joined #openstack-neutron20:16
*** sbfox has joined #openstack-neutron20:17
*** mlavalle has quit IRC20:18
*** nlahouti has quit IRC20:19
openstackgerritAaron Rosen proposed a change to openstack/neutron: NSX: update_lswitch method can add duplicate version tags  https://review.openstack.org/11336620:22
jogoarosen: ping re nova neutron20:25
arosenjogo:  pong20:26
arosenwhat's up?20:26
*** ijw has joined #openstack-neutron20:26
jogomordred had a good idea in openstack-infra20:26
jogonova should have 'nova boot --gimme-floating-ip'20:26
*** thomasem has joined #openstack-neutron20:26
jogoas that stuff is tricky to get right as a user20:27
jogoarosen: thoughts?20:27
*** thomasem has quit IRC20:27
*** magesh_ has joined #openstack-neutron20:27
*** thomasem has joined #openstack-neutron20:27
*** nlahouti has joined #openstack-neutron20:28
arosenjogo:  Hrm, personally I'm not a big fan of moving more orchestration into nova  :/20:28
arosenI think horizon should have a checkbox allocate floating and then horizon can call out to neutron to create the floatingip20:28
*** amcrn has joined #openstack-neutron20:28
jogoarosen: sure, but we awnt to support CLI users for this as well20:29
arosenjogo:  eventually i hope that nova doesn't have to do any write operations to neutron only reads.20:29
*** mordred has joined #openstack-neutron20:29
jogoarosen: we can move things into nova client instead maybe20:29
mordredjogo: if it's just in teh client, then you'll still have the race condition20:29
arosenjogo:  one sec i actually think there is a way the novaclient can kinda do this today.20:30
arosenmordred:  i'd prefer the race condition  on the client side ;)20:30
mordredjogo: which is that if one API call creates a floating ip, and another program says "I want to see if there are any unclaimed floating ips"20:30
mordredarosen: the problem we have in nodepool is that $sometimes, floating ips don't get cleaned up when we delete servers20:31
*** emagana has quit IRC20:32
mordredthat means we either  have to have a program that cleans that and perhaps deletes an ip that was in the middle of the create/attach logic20:32
mordredor we have to have create look for unused ips and then race to attach it20:32
arosenmordred:  how are you deleting the floatingips?20:32
mordredarosen: we delete them in our delete server code20:32
mordredexplicitly20:32
*** yamamoto has joined #openstack-neutron20:32
*** emagana has joined #openstack-neutron20:32
arosenmordred:  does that call out to neutron to delete them?20:32
mordredarosen: https://github.com/openstack-infra/nodepool/blob/master/nodepool/provider_manager.py#L48320:33
mordredright now we just use the novaclient20:33
jogoso mordred: a bit of backround arosen doe a lot o fwork on the neutron integration in nova20:34
* arosen let me look at what nova does with this api call. 20:34
mordredjogo: awesome20:34
arosenmordred:  does it raise an error or just not delete the floatingip?20:34
mordredarosen: the reason we occasionally leak floating ips is unknown20:35
mordredbut at times, we'll wind up with a few hundred unattached floating ips20:35
mordredwhich we should never have20:35
mordredbut, you know, clouds :)20:36
mordredthe thing is - the _simple_ case is so hard to do and get right20:36
mordredwhich is "I want a usable server"20:36
mordredit takes a ton of steps20:36
*** julim has quit IRC20:36
*** mlavalle has joined #openstack-neutron20:36
mordredwe mostly get it right - I mean, we spin up between 10k and 20k a day ... so it's obviously possible20:36
*** emagana has quit IRC20:36
*** yamamoto has quit IRC20:37
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Preserve link local IP allocations for DVR fip ns across restart  https://review.openstack.org/10948620:37
*** networkstatic has joined #openstack-neutron20:37
jogosalv-orlando: ^20:37
*** crc32 has quit IRC20:38
salv-orlandojogo: are you pointing me to that patch or what mordred said?20:38
salv-orlandoarosen, mordred, jogo: I might be a bit more informed since I’ve had previous discussion on this with potential contributors which then found me too annoying to deal with ;)20:38
jogosalv-orlando: the arosen mordred backlog20:38
arosenmordred: looking at the TaskManager in  nodepool/task_manager.py does it not log the return of the commands it runs on the client side?20:39
salv-orlandoso about 1 year ago there was somebody (Steven_weston I think) that looked at how nova implements this feature20:39
arosenmordred:  my guess is neutron raises and error and  we never successfully delete the floatingip in neutron :(20:39
arosenmordred:  I'm not quite sure though without seeing the neutron-server logs.20:40
mordredarosen: nod. I can try to capture more information20:40
mordredoh - well, I mean, I'll never have access to those20:40
mordredbut if it's something that would return an error through to the novaclient call, I can find and log that for sure20:40
arosenmordred:  how does nova boot --gimme-floating-ip help with this?20:40
*** saju_m has joined #openstack-neutron20:40
salv-orlandoarosen: basically leaving to either neutron or nova/neutron integration ensure that there are no leaks20:41
*** seizadi has quit IRC20:41
arosenmordred:  there definitely should be some kind of error raised on the nova-client i imagine. Does nodepool retry on 500 errors?20:41
*** seizadi has joined #openstack-neutron20:41
salv-orlandoyet what I’m skeptical about that is the fact that a floating IP once allocated, in neutron logic, can be easily reused if it’s not associated20:41
mordredarosen: it's purely going to be either moving on or erroring in that case based on exceptinos thrown by novaclient20:41
mordredsalv-orlando: yah - that's the race condition that makes it hard to close the loop on my side20:42
mordredsalv-orlando: without keeping a database of allocation associations or something20:42
mordreddon't get me wrong - for the times when I want the flexibility, floating-ip concept is great ...20:43
mordredbut many times I just wanta  server with an ip, and I want the ip to go away wehn the server goes away20:43
nlahoutimarkmcclain: Hi Mark, do you have time to continue our discussion?20:44
salv-orlandomordred: understood. We’ve seen that requirement a lot of times. The fact is that if I get it right the floating IP which are “leaked” are not associated to any internal port. Is that correct?20:44
mordredyeah20:44
mordredsalv-orlando: and then, for a human it's easy20:45
*** gduan has joined #openstack-neutron20:45
mordredbut to deal with it programmatically either to reap un-associated ips, or to re-use unassociated ips on subsequent create both have race conditions20:45
*** seizadi has quit IRC20:45
*** seizadi has joined #openstack-neutron20:45
mordredof course, I could probably just put the re-use logic into a tight loop20:45
*** garyduan has quit IRC20:46
mordredso that create always first tries to use an existing one, and attach always handles failure by going back to the start of looking for an exisitng ip and maybe creating one if needed20:46
mordredbut eek, you know?20:46
salv-orlandomordred: that would work I think. But “try and use an existing one” might not be super obvious with neutron API. Mostly because when listing floating IPs *I think* we don’t have a way to return only those not associated20:48
arosenmordred:  yea i understand this definitely sucks. We definitely don't want the client to have to do so much work.20:48
salv-orlandonot that this is something hard to fix20:48
*** clarkb has joined #openstack-neutron20:48
arosensalv-orlando:  it's easy enough to filter on the client side if there isn't a port_id20:49
mordredsalv-orlando: currently when I do this by hand, I run neutron floating-ip list on the commadn line and then grep for ' \- '20:49
salv-orlandoarosen: so we fixed the thing about not being able to filter on none attributes there?20:49
jogomarun: ^ backlog20:49
*** saju_m has quit IRC20:49
clarkbmordred: that doesn't work with neutron client, it works with nova client20:50
clarkbebcause the two clients use two different output formats20:50
arosensalv-orlando:  i'm just saying if we don't support filtering for listing floatingips that aren't allocated we can just have the client list all the floating-ips and they can determine on their end.20:50
marunWhy isn't it possible to allocate a port and associate it with a floating ip in one step?20:50
salv-orlandoarosen: sure. I was just trying to remove sources for burden on the client side.20:50
marunIt's a common enough use-case.20:50
arosensalv-orlando:  gotcha20:50
*** saju_m has joined #openstack-neutron20:50
marun(i mean server-side, in neutron)20:51
arosenmarun:  the problem is actually on the delete side.20:51
marunarosen: why?20:51
arosenmarun:  delete a port and it's floatingip20:51
marunarosen: and again, that isn't reasonable to do in one step?20:51
*** ezobn__ has joined #openstack-neutron20:52
arosenmarun:  only because our api doesn't do any kind of cascading deletes between though resources20:52
arosenwe treat them independently;20:52
marunarosen: so, let's fix that.20:52
salv-orlandoarosen: I think marun is talking about binding the life cycle of floating IP to that of the port where it’s associated20:52
marunsalv-orlando: +120:52
arosenmarun:  it's possible someone doesn't want to give up the floatingip when they delete the port though.20:52
salv-orlandoso when you delete a port you just don’t disassociate the floating IPs, you delete them too.20:52
marunarosen: so make it optional20:52
marunarosen: but still support it20:52
arosenmarun:  sure we could do that but that would be an additional extension to the api right?20:53
salv-orlandomarun: technically yes. And an annoying one given how crappy is the ext framework in this case.20:53
*** kbringard has quit IRC20:54
marunFor now an extension, sure.20:54
salv-orlandoI think we should make that simpler by not providing the automation when one creates a port (ie: saying something like —auto-allocate-floatingips)20:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy API-2: Classifiers, Actions, and Rules  https://review.openstack.org/10181620:54
marunBut so long as the cli and orchestration frameworks can take advantage, I'm not sure it matters that it's not in the core API (yet).20:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Mapping Plugin-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/10195720:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver  https://review.openstack.org/10527220:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Plugin-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9639320:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy: Implicit Policy Driver  https://review.openstack.org/10472720:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy DB-3: Contracts  https://review.openstack.org/10375520:54
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy API-3: Contracts  https://review.openstack.org/10348620:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9590020:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Mapping API-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/10071620:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy DB-2: Classifiers, Actions, and Rules  https://review.openstack.org/10345620:55
salv-orlandowe should do that when we associate a floating ip. With an attribute called “bind”20:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Plugin-3: Contracts  https://review.openstack.org/10379820:55
marunHuh20:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy DB-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/9605020:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Mapping DB-1: EP, EPG, L2 Policy, L3 Policy  https://review.openstack.org/10179520:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy Plugin-2: Classifiers, Actions, and Rules  https://review.openstack.org/10346520:55
openstackgerritMagesh GV proposed a change to openstack/neutron: Group Policy: One Convergence NVSD Group Policy Driver  https://review.openstack.org/11364920:55
marunI think he's trying to tell us something20:55
salv-orlandoand then you tell me I should like group policy20:55
marunBut I'm not sure it's what he's intending.20:55
marunhah20:55
salv-orlandopossibly that all these problems would be solved if we move to group policy?20:55
marunSo I hear.20:55
salv-orlandowhere were we?20:56
*** ijw has quit IRC20:56
marunSo, implement in an extension?20:56
marunsalv-orlando: I'm not sure I understood your last point20:56
salv-orlandoyeah but not on port20:56
marunre: auto-allocate-floatingips20:56
*** ijw has joined #openstack-neutron20:56
salv-orlandoon floating ip  association.20:56
arosenIt's funny we have two different work flows in the api actually20:57
arosenOne can do this:20:57
salv-orlandothere we would pass a flag that binds the floating IP lifecycle to the port where it’s being associated20:57
arosenneutron port-create net1; neutron floatingip-create public; neutron floatingip-associate blah balh;20:57
arosenor: neutron port-create net1; neutron   floatingip-create --port-id=1f12a922-e8c9-44af-960c-f0ab958cbef9 Colo-InterConnect20:57
marunsalv-orlando: ah, gotcha.  So the associate could optionally become stickier20:57
marunarosen: so associate on create vs associate after create20:58
arosenyup20:58
arosenmy gut though tells me that the client side should be dealing with clean up here if they get an error to an API call.20:59
*** emagana has joined #openstack-neutron20:59
marunarosen: I think that might be a reasonable stop-gap20:59
arosenmarun:  what's the stop-gap? Doing it on the client side?21:00
arosenretrying the delete/21:00
marunarosen: But I think having the option of tighter binding between ports and floating ips on the server-side should be the goal21:00
marunarosen: yes21:00
marunarosen: client-side can paper over server deficiencies, and maybe that's a good short-term goal21:00
marunarosen: but long-term I think fixing the server-side makes more sense21:01
*** ijw has quit IRC21:01
salv-orlandomarun, arosen: yes, as long as we do not add a lot of orchestration among neutron resources - that would be error prone.21:01
*** saju_m has quit IRC21:01
arosenmarun:  right but i don't know if fixing the server by providing an API call to cascade the delete is the right fix. I'm guessing we'll still have the same underlying issue there somewhere for why the delete is failing.21:01
salv-orlandoorchestration like creating a floating ip as a side effect of a port-create call21:01
*** VijayB has joined #openstack-neutron21:02
salv-orlandoarosen: you are probably correct as well. Why the cascaded delete might be desirable, it makes sense to go first to the bottom of the problem is manifesting in nodepool21:02
salv-orlandoarosen: did mordred already tell you where to find the nodepools logs? I don’t know if they’re accessible through logstash.21:03
clarkbthey are not21:03
mordredthey are not21:03
clarkbwe don't put our system logs in logstash because they erally shouldn't all be public21:03
clarkbtest logs are 100% public21:03
aroseni'm digging into the python-novaclient bindings looking to see if there might be another way.21:03
clarkbbut we are happy to provide people with logs21:03
clarkbone thing (I don't know if mordred covered this before), but nova/neutron don't do a good job of keeping track of floating ip state aiui. So if you have a shared tenant/project/whateveritiscallednow you have to do all sorts of manual tracking yourself21:04
clarkbotherwise you could delete allocated floating ips because you raced the associating to a node21:05
*** sdmonov has quit IRC21:05
*** jckasper_ has joined #openstack-neutron21:05
marunclarkb: that sounds accurate21:05
salv-orlandoclarkb: do you mean tracking allocation/association state through nova compute floatingips extension?21:06
*** VijayB has quit IRC21:06
arosenclarkb:  i know this isn't a great solution but why not ensure the floatingip is deleted before we issue the call to delete the server. This way we would know the mapping.21:06
clarkbsalv-orlando: I mean that floating ip delete errored a month ago. neutron and nova don't expose that info21:06
arosensalv-orlando:  it looks like there is no way to tell which floatingips to garbage collect as they could be new ones that were just allocated21:07
salv-orlandoclarkb: I see. that’s what arosen is trying to understand then, and why we’d like to look at the logs21:07
clarkbarosen: correct21:07
*** VijayB has joined #openstack-neutron21:07
openstackgerritKevin Benton proposed a change to openstack/neutron: Big Switch: Separate L3 functions into L3 service  https://review.openstack.org/11365321:08
arosenclarkb:  alternatively if this api call was made we could also avoid that: neutron floatingip-create --port-id=port...id network ;21:08
arosenthat creates the floatingip and port binding in one call to neutron21:08
salv-orlandoarosen: putting for one second aside the fact that floating ip delete errors out and that probably goes unnoticed in nova, would you say that all floating IPs without an associate port can be garbage collected?21:08
clarkbarosen: does novaclient expose that?21:08
salv-orlandoclarkb: not.21:09
*** jckasper__ has quit IRC21:09
arosensalv-orlando: no i don't think they can be garbaged collected because i believe the workflow from the nova side is create floatingip (floating ip will have no port binding) then associate21:09
salv-orlandobecuase the nova’s floating IP extension does not do that, and the —auto-allocate-floating-ips feature is not available when using neutorn.21:09
*** bradjones has quit IRC21:10
clarkbarosen: we do delete floating ips first https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n50321:10
salv-orlandoarosen: ok let’s make the assumption we don’t want to change the current workflow then ;)21:10
arosenclarkb:  i wonder if this line is raising then: https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n50921:11
salv-orlandoclarkb: and you do that after destroyng the server, right?21:11
clarkbsalv-orlando: no server deletion is the end of that method21:11
clarkbarosen: I think the actual issue is during server creation21:11
salv-orlandoclarkb: right sorry about that. I should have scrolled a bit more!21:12
arosenclarkb:  interesting, so do you know if these leaked floatingips are all from failed jobs?21:12
clarkbarosen: I don't. last time I looked at this it apepared related to creating servers that failed. the errored server would cleanup but the floating ip wouldn't always21:13
arosenclarkb:  The workflow for creation is: launch-server; create-floatingip; associate-with-instance?21:13
salv-orlandoclarkb: here? https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n33321:13
*** vishwanathj has quit IRC21:13
arosenclarkb:  oh so we don't wait for the server to go active first either i guess?21:13
arosenbefore we associate it21:13
*** vishwanathj has joined #openstack-neutron21:13
clarkbsalv-orlando: up a call level iirc, let me find it21:13
clarkbarosen: its been a while. I am refreshing my memory21:14
arosenclarkb:  i see you do wait for it to go active here:21:15
clarkbarosen: https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/nodepool.py#n38321:15
arosenhttps://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/nodepool.py#n39221:15
clarkbya it is synchronous so maybe that wasn't it21:15
arosenclarkb:  okay that's good. If it wasn't waiting then there could definitely be a race condition were the port wasn't created in neutron21:16
*** dconde has quit IRC21:16
*** radez is now known as radez_g0n321:17
*** mlavalle has quit IRC21:17
*** yamamoto has joined #openstack-neutron21:17
salv-orlandoclarkb: let’s go back for a sec to https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n50821:18
*** vishwanathj has quit IRC21:18
salv-orlandothis spawns a task which remove (ie: disassociates) the floating ip - I think submit task does not wait for a response from the API server21:18
*** seizadi has quit IRC21:19
*** seizadi has joined #openstack-neutron21:19
salv-orlandothen the next line tries to remove that, ans does again submittask21:19
*** emagana has quit IRC21:20
openstackgerritA change was merged to openstack/neutron: MLNX Agent: ensure removed ports get treated on resyncs  https://review.openstack.org/9488821:20
*** rudrarugge has quit IRC21:21
openstackgerritChirag Shahani proposed a change to openstack/neutron: Add template attr. for subnet, router create in Nuage plugin  https://review.openstack.org/11274921:21
salv-orlandoif the task started at line 508 did not complete yet, the one at line 509 will error out… I’m not sure however if the exception raised by the task will appear in delete_server. If it does not, then the server will be deleted, the floating IP disassociated there (if it’s still available)21:21
salv-orlandobut the floating Ip, as a db resource will not be deleted.21:21
*** emagana has joined #openstack-neutron21:21
salv-orlandoclarkb: if my reasoning if correct, that might explain the leakage - maybe21:21
*** sbfox has quit IRC21:22
*** yamamoto has quit IRC21:24
*** yfried__ has quit IRC21:25
*** WackoRobie has quit IRC21:27
*** WackoRobie has joined #openstack-neutron21:28
*** padkrish_ has quit IRC21:29
nlahoutimarkmcclain: ping21:30
*** padkrish_ has joined #openstack-neutron21:31
*** WackoRobie has quit IRC21:32
*** marun has quit IRC21:33
*** ivar-lazzaro has quit IRC21:33
*** ivar-lazzaro has joined #openstack-neutron21:34
salv-orlandonlahouti: sorry I’m having a delay reviewing your patch as I had to deal with a few higher priority tasks. Will try and sort this review out tomorrow.21:34
nlahoutisalv-orlando: really appreciate it21:34
*** crc32 has joined #openstack-neutron21:36
clarkbsalv-orlando: sorry grabbed lunch, reading now21:42
salv-orlandoclarkb: stil here21:42
arosenclarkb:  salv-orlando   https://review.openstack.org/11365921:43
clarkbarosen: I am not sure that is true via novaclient? at least maybe not how we use it?21:43
clarkbarosen: oh wait its just the association that is removed21:43
clarkbI see21:43
salv-orlandoarosen: you’re the usual commit-grabber.21:44
clarkbarosen: salv-orlando 113659 can still fail like the situation salv-orlando described?21:44
arosenclarkb:  i tested it locally http://paste.openstack.org/show/94049/21:44
salv-orlandoclarkb, arosen: I just need to check one thing in niova’s floating ip extension21:45
salv-orlandohold on21:45
ZZellekevinbenton, hi21:45
clarkbsalv-orlando: ya ok, tasks are submitted and processed in order, but if one failed asynchronously the other would charge ahead and fail too21:45
salv-orlandoarosen, clarkb: cool. Delete includes disassociate21:45
kevinbentonZZelle: hey, what’s up?21:45
salv-orlandoarosen’s patch should fix the potential race21:46
arosensalv-orlando:  yup my pasted shows that :)21:46
ZZellekevinbenton, one question about bigswitch plugin/ml2 mechanism21:46
salv-orlandoarosen: I did not even bother looking at that… you could have fabricated it ;)21:46
arosen:)21:46
kevinbentonZZelle: shoot21:46
ZZellekevinbenton, am i wrong or ML2 + bigswitch mechanism provides the same feature as bigswitch plugin21:46
ZZelle?21:46
clarkbsalv-orlando: arosen: can yo udelete the floating ip before the server is deleted?21:47
salv-orlandoclarkb: yes you can.21:47
clarkbsalv-orlando: arosen: it isn't an error to delete and associated floating ip?21:47
arosenstill though 509 could raise in error i guess  https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n509 it looks like retrying on that could be helpful.21:47
kevinbentonZZelle: not quite. right now the Big Switch plugin assumes the controller owns the vswitch21:47
kevinbentonZZelle: the ML2 driver assumes the agent configures the vswitch21:47
salv-orlandoclarkb: you are right but if you novaclient the delete call will also try and disassociate21:47
salv-orlando* if you use novaclient21:48
kevinbentonZZelle: we are working on converging them, this L3 separation work that I just pushed is a first step21:48
clarkbsalv-orlando: right but the order of that code is delete floating ip, delete node21:48
arosenclarkb:  it looks like nova lets you delete a floatingip even if it's associated  to a server: see: http://paste.openstack.org/show/94049/ (but this could be fabricated ;) )21:48
salv-orlandoyou can point them to the code ;)21:48
clarkbI don't think this will fix the problem21:48
clarkbI swear neutron has errored on me trying to delete associated items21:49
arosenclarkb:  i agree i don't think this will really fix anything. Let me ask you this. If line 508 raises an exception does 509 still get executed?21:49
salv-orlandoclarkb, arosen: https://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib/floating_ips.py#n19321:49
clarkbarosen: no21:49
ZZellekevinbenton, ok just wondering if they behaves the same and if the plugin could be deprecated ==> so no21:49
clarkbsalv-orlando: yes, but nodepool is deleting the floating ip before deleting the node21:50
clarkbsalv-orlando: so delting floating ip will fail then the node won't be deleted21:50
kevinbentonZZelle: not quite yet. possibly in Kilo though21:50
*** flwang_ has joined #openstack-neutron21:50
arosenclarkb:   actually so i think this might help. Previously you were saying you would see a lot of floatingips that were not mapped to server. In this case the only way this would happen now is if there was a failure because we're deleting the floatingip directly21:50
clarkbreally it appears that nodepool needs a floatingip table to track that state and manage associations itself then do deletions on floating ips like ti delets nodes21:50
*** sdmonov has joined #openstack-neutron21:50
clarkbI think this is the bug that mordred is trying to point out when it comes to clients needing to track all this state themselves21:51
salv-orlandoclarkb: ok so that’s important - an error during any asynchronous task will raise an exception in delete_server preventing deletion of the node21:51
ZZellekevinbenton, great21:51
clarkbsalv-orlando: sort of, we will retry deletions so it is ok if we eventually succeed21:51
clarkbsalv-orlando: in this case we will never succeed21:51
clarkb(we have to retry for other raisisn)21:51
*** dconde has joined #openstack-neutron21:51
salv-orlandoclarkb, arosen: this might sound stupid at this point, but do we have a bug report where I could get more details? I don’t want to sound procedural, but I’m not understanding if we are dealing with leaked floating ips, leaked node, or both21:52
arosenclarkb:  I think with this change you will actually now be able to garbage collect safely.21:53
clarkbarosen: no, I think it will prevent any garbage collection because the delete floating ip will always fail21:53
clarkbsalv-orlando: nodepool is leaking floating ips21:53
*** armax has quit IRC21:53
clarkbsalv-orlando: this is happening because nodepool does not treat floating ips as discrete resources. instead it relies on their association to nodes whcih may be deleted to find them21:53
arosenclarkb:  you said if the delete_floatingip fails you don't continue on to delete the server?21:54
clarkbsalv-orlando: the "bug" here is that nova and neutron should handle this for us21:54
mordredsalv-orlando: mainly ... we're trying to be good citizens since we're such heavy openstack users21:54
*** flwang_ has quit IRC21:54
clarkbarosen: https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n509 is where we delete, I need to check if that is a synchronous or async task21:54
mordredsalv-orlando: in this case, I'd go so far as to say that the fact that I, as an end user, have to know ANYTHING about floating ips to accomplish the simple task of "give me a server with an IP" is a leakage of an implementation detail into an interface21:55
mordredI understand it may be hard/impossible to fix21:55
*** sgordon_ has quit IRC21:55
clarkbarosen: but #509 will fail the first tiem it runs. best case no floating ip is deleted and nova disassociates the node from the ip21:55
clarkbarosen: at that point we have leaked the floating ip21:55
clarkbworst case that method short circuits and we never delete the node21:55
mordredand I don't even know if there _is_ a fix or who would own it - but I'd happily file a bug if that's helpful21:55
*** oda-g has joined #openstack-neutron21:56
mordredI similarly, as a user, almost never want to know anything about ports or routers. in fact, I do not actually want to know anything about ports or routers.21:57
salv-orlandoclarkb: that my analysis as well, under the assumptions that the task is aunc21:57
mordredmy neutron got messed up one time and I had to learn about them ... and it was neat that they existed!21:58
*** jckasper_ has quit IRC21:58
mordredbut they don't have any bearing on me getting my workload issues taken care of day in and day out21:58
clarkbarosen: it looks like it is synchronous via https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/task_manager.py#n4321:58
clarkbso we submit the task then that thread waits for it to happen. if an exception is thrown that is reraised and nodedelete won't happen21:58
*** ijw has joined #openstack-neutron21:59
salv-orlandomordred: I *might* try and sidestep the conversation saying that then the group policy API will relieve you of that and give you the ability of handling everything with concepts such as endpoints, contracts, and policies. But I won’t dare..21:59
*** yamamoto has joined #openstack-neutron21:59
arosenclarkb: Then do you guys manually clean up the server?21:59
salv-orlandomordred: your usability concerns are correct.21:59
clarkbarosen: no, it seems to be working now because we disassociate first22:00
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Implement physical_interface_mappings  https://review.openstack.org/11316622:00
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Enable local arp responder for local VMs  https://review.openstack.org/11316022:00
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: l2pop: Allow network types overridable  https://review.openstack.org/11294722:00
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: merge br-tun into br-int  https://review.openstack.org/11248822:00
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Local arp responder for VLAN  https://review.openstack.org/11294922:00
clarkbarosen: that initial disassociate may fail, but then we will try again22:00
mordredsalv-orlando: hehe22:00
clarkbarosen: I think the issue with your patch is we will never try to disassociate first so we can't get beyond #50922:00
mordredsalv-orlando: yeah  - if you told me I'd need to thin about endpoints, contracts, and policies, I would quite diretly just start using amazon22:00
mordred:)22:00
*** sdmonov has quit IRC22:01
salv-orlandomordred: unless you’re a commercial lawyer. Then you should be comfortable with contracts.22:01
mordredsalv-orlando: if I was a commercial lawyer, I'd own many more private jets than I currently do22:01
*** SridharRamaswamy has joined #openstack-neutron22:02
salv-orlandoanyway - how to address these concerns. I think there are 2 classes of usability issues - 1) things we do wrong in the API and should be fixed 2) things that are fine for an api that exposes “primitives” but that 90% of users won’t never be bothered with22:02
*** amotoki has joined #openstack-neutron22:03
salv-orlandofor the first we’ve been looking at API changes for addressing the network/subnet dicotomy for instance, but the discussion never came to a concrete code proposal22:03
*** Sridhar__ has joined #openstack-neutron22:04
salv-orlandofor the second actually, my opinion is that we should do that outside of neutron, but I’m still battling with this as on other hand I don’t want to add another API endpoint which does orchestration. To this aim, Heat would be an option, but what about situations where it’s not dpeloyed?22:04
salv-orlandoanyway back to our problem.22:04
emaganamestery: It seems that you approve this one: https://blueprints.launchpad.net/neutron/+spec/bsn-l3-service-plugin22:04
emaganamestery: I couldn't find the spec related to this BP.22:05
emaganaI should also ask to kevinbenton22:05
salv-orlandoclarkb: let’s say that we have grounds to suspect there is a race there, and that we have an hypothesis that however cannot be confirmed until we are sure the tasks are async22:05
salv-orlandowhere there means delete_server in nodepool22:06
*** SridharR_ has quit IRC22:06
arosenclarkb: where do you try again? I'm not sure I see the benefit of putting tasks in the task queue if they are executed synchronously ?22:06
*** SridharRamaswamy has quit IRC22:06
*** banix has joined #openstack-neutron22:07
arosen(02:54:08 PM) clarkb: arosen: I think the issue with your patch is we will never try to disassociate first so we can't get beyond #50922:07
arosen^why do you say this?22:07
clarkbarosen: we retry with an internal "cron". we put them in a queue to do rate limiting22:07
emaganamestery or kevinbenton: are you around?22:07
clarkbarosen: you cannot sanely do async api calls to openstack22:07
clarkbyou get ratelimited and blacklisted22:07
*** amotoki has quit IRC22:08
kevinbentonemagana: https://review.openstack.org/#/c/106269/22:08
emaganakevinbenton: Thanks.. I could not find it in the BP in launchpad22:09
arosenclarkb:  sorry one more question. So when this occurs i guess these api calls to nova eventually return 200's because nodepool is able to delete the instance?22:09
emaganareviewing your patch right now  ;)22:09
clarkbarosen: yes, I believe that is true22:09
clarkbarosen: otherwise we would retry the failing method repeatedly22:10
mordredsalv-orlando: yah - also, needing heat for "gimme a server that works kthxbai" is a bit of overkill22:10
kevinbentonemagana: ok. i updated the whiteboard on launchpad with a link22:10
emaganakevinbenton: great, other reviewers will need it as well.22:10
kevinbentonemagana: i didn’t think reviewers read the specs :-)22:11
*** yeungp has quit IRC22:11
emaganakevinbenton: I do.. I like to amke sure that the BP is covering everything in the spec22:12
emaganamake*22:12
*** lordd_ has quit IRC22:12
*** seizadi1 has joined #openstack-neutron22:13
*** seizadi has quit IRC22:13
arosenclarkb:  hrm it seems like everything should be cleaned up fine then :(22:13
emaganakevinbenton: I know.. I am a weird guy!22:14
kevinbentonemagana: that’s the dream of the spec!22:15
nlahoutimarkmcclain: ping22:16
emaganakevinbenton: +222:16
*** lordd_ has joined #openstack-neutron22:17
*** emagana has quit IRC22:17
*** emagana has joined #openstack-neutron22:18
*** regXboi has quit IRC22:19
ZZellehi, some cores can have a look to https://review.openstack.org/105472 ?22:20
*** changbl has quit IRC22:21
*** emagana has quit IRC22:22
*** armax has joined #openstack-neutron22:22
rms_13_cores, can someone take a look at https://review.openstack.org/#/c/112171/22:26
clarkbarosen: salv-orlando: mordred: ok new theory, the disassociate succeeds, the delete floating ip fails, the node delete succeeds22:28
clarkbarosen: salv-orlando: mordred: in that situations we would never retry the floating ip delete22:28
clarkbarosen: so I think your suggestion to retry the floating ip delete may be the thing to do22:29
*** Sridhar__ has quit IRC22:29
salv-orlandoclarkb: which is my initial one - but in order to be credible we need to be sure that a failure in delete floating ip might go unnocited22:29
clarkbsalv-orlando: ya so let me go pull up logs22:29
salv-orlandoand therefore the tasks started at lines 508 and 509 should be async22:29
*** SridharRamaswamy has joined #openstack-neutron22:29
*** ijw has quit IRC22:30
arosenclarkb:  why do you say you guys aren't retrying on delete_floatingip?22:30
*** SridharR_ has joined #openstack-neutron22:32
clarkbarosen: because once the node is deleted we will never run that function for that node again22:32
*** emagana has joined #openstack-neutron22:33
*** emagana has quit IRC22:33
*** ivar-lazzaro has quit IRC22:33
*** ivar-lazzaro has joined #openstack-neutron22:34
*** emagana has joined #openstack-neutron22:34
arosenI've actually reproduced that here but i'm confused why that happends. I put a raise TypeError here: https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n164 and saw nodepool retry and retry22:34
*** SridharRamaswamy has quit IRC22:34
arosenI moved the raise TypeError to herer: https://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/provider_manager.py#n169 and it looks like it only tried trice.22:35
*** sbfox has joined #openstack-neutron22:35
clarkbhuh22:35
*** jaypipes has quit IRC22:36
arosenclarkb:  AH I completely see why this is occuring22:37
arosenIt's getting the floatingip binding on the server from list-servers22:37
arosenonce we disassociate it's no longer there.22:37
clarkbyes22:38
arosenso it no longer finds the floatingip it had22:38
arosendoh :)22:38
clarkbour resources are nodes not ips22:38
clarkband this is the impedance mismatch we are trying to explain22:38
clarkbwe don't care about managing floating ips as discrete things22:38
arosenclarkb:  in that case i think my patch should completely solve this issue for you. We'll just keep retrying to delete the floatingip22:39
lifelesssome people do sometimes22:39
clarkblifeless: yes, I think having the ability to is good22:39
lifelessbut most folk don't most of the time22:39
clarkblifeless: but making the default thing the hard thing that most people don't want is a bug22:39
lifelessclarkb: I totally agree :)22:39
clarkbarosen: oh I see, except we will need two round trips to do it22:39
lifelessthis is why I spat the dummy when nova proposed making port management for vms entirely manual22:39
*** AlexF has quit IRC22:40
clarkbarosen: actually no22:40
clarkbbecause deleteip will fail, we will then delete the node, then nothing will call deleteip again because the node is gone22:40
arosenclarkb:  two round trips to getServerFromList?22:40
arosenclarkb:  one sec let me test.22:40
clarkbarosen: your change removes line 508 so that we rely on nova to disassociate. that must happen before we delete ip22:41
clarkbso deleteip will throw and exception. if we ever manage to dlete teh server we will not come back and delete the ip22:41
clarkbI think the "proper" fix here is the thing we are tring to avoid and that is treating floating ips as a new resource that nodepool tracks22:41
*** shashankhegde has quit IRC22:42
arosenclarkb: That patch cause us to keep trying delete_floatingip22:43
arosenwe'll never delete the server22:43
*** banix has quit IRC22:44
arosenclarkb: http://paste.openstack.org/show/94062/22:44
clarkboh right, order is important. we could put a retry loop at line 50922:44
*** prad has quit IRC22:44
arosenclarkb:  if you scroll through though logs you'll see we keep raising typerror22:44
arosenMan this was a tricky bug :)22:44
clarkbya, and it never drops through to delete server22:44
clarkbso it fails again and again :)22:45
arosenright which is what we want so hopefully we'll eventually clean it up.22:45
*** hemanthravi has joined #openstack-neutron22:46
arosenclarkb:  I'll update the commit message now that we know what's going on :)22:47
clarkbarosen: maybe we should go back to -infra and fill jeblair in? he may have other ideas on fixing this22:47
clarkbarosen: wait that commit doesn't work as is I thought?22:47
*** carl_baldwin has quit IRC22:48
arosenclarkb:  no it does. I mean just to update the commit message to saw why it fixes the issue. Initially i though it just removed an unneeded call22:48
clarkbarosen: I don't think it does22:48
arosenwhy?22:48
clarkbarosen: deleteip will not succeed until after we disassociate the ip22:48
clarkbwe will never disassociate the ip because we do not delete the node22:48
clarkbbecause the function short circuits on deleteip failing22:48
clarkbit is the same situation as raising your TypeError22:49
arosenclarkb:  we'll eventually delete the node once we are able to delete the floatingip22:49
*** stack__ has joined #openstack-neutron22:49
*** banix has joined #openstack-neutron22:49
clarkbarosen: I don't think we can delete the floatingip until we disassocaite the node from the ip22:49
arosenclarkb:  https://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib/floating_ips.py#n19322:50
arosenclarkb:  i showed in my pastebin that you can ?22:50
*** thomasem_ has joined #openstack-neutron22:50
arosenclarkb:  that said I think there is another bad thing that can happen here. If you look at that link i sent you it's possible for line 196 to succeed which means we'll be in the same situation :(22:50
clarkboh it is the delete of the ip that disassociates22:51
clarkbI thought it was the delete of the node22:51
clarkbok I am on board now22:51
arosenclarkb:  the delete of the node also disassociates the floatingip but the problem is then we won't be able to find the ip anymore22:51
clarkbarosen: yup22:52
clarkbok, that is where I was stuck. I didn't realize delete of floating ip did it too22:52
*** shashankhegde has joined #openstack-neutron22:52
clarkbin this case if it fails it will make all the deletion happen again22:52
clarkband again and again so we should be good22:52
clarkboh wait22:52
clarkbthat won't fix it22:53
clarkbit has the same bug we have today22:53
*** thomasem has quit IRC22:53
clarkbarosen: if https://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib/floating_ips.py#n196 succeeds but https://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib/floating_ips.py#n203 fails we will not get the ip from the floating ip list ?22:53
arosenclarkb:  right that was what i was saying :)22:53
*** emagana has quit IRC22:53
arosenclarkb:  but  interestingly enough neutron allows us to delete the floatingip even if it associated.22:54
arosenclarkb:  so we could actually change this code to say if using neutron don't call disassociate_floating_ip22:54
*** thomasem_ has quit IRC22:54
arosenclarkb:  I got to run to a 4:00 :(. I'll try and reconnect there though.22:54
clarkband delete it directly? and only then delete the node?22:55
kevinbentonrkukura: do you have that document handy that shows what information is available during the various states of port creation/binding?22:55
*** julim has joined #openstack-neutron22:56
*** jeblair has joined #openstack-neutron22:57
salv-orlandoclarkb: I can take it from arosen22:57
salv-orlandoas he’s now afk22:57
mordredso, currently, I don't think we have any nova-network+floating-ip providers22:58
mordredso "if neutron: delete floating ip first" is probably fine, yeah?22:59
*** shashankhegde has quit IRC22:59
clarkbmordred: but only via the neutron client22:59
clarkbmordred: if you do it via nova client it will do the wrong leaky thing22:59
clarkbmordred: so we would need to maintain another task manager for neutron...22:59
mordredclarkb: well ... hrm23:00
salv-orlandouhm. I sense like you guys don’t want to use the neutroclient though...23:00
mordredsalv-orlando: it's just more complexity on our side ...23:00
clarkbsalv-orlando: well on principle no :)23:00
mordredwhat I really want is openstacksdk23:00
clarkbwe can do it23:00
salv-orlandoclarkb, mordred: understood23:00
*** julim has quit IRC23:00
mordredand to have that use the right tools as it sees fit23:00
salv-orlandoI think we can fix https://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib/floating_ips.py#n178 to do the right thing with neutron23:01
*** ijw_ has quit IRC23:01
salv-orlandoand you can still use the novaclient23:01
salv-orlandoand everybody will be happy23:01
* mordred hands salv-orlando a cheesecake23:01
clarkbya so we could go kick novaclient too23:01
salv-orlandowell not mordred as he wants the “gimme a machine with a public ip kthxbie” thing23:01
mordredsalv-orlando: one of these days I'll get my wish there!!!!23:02
clarkbsalv-orlando: that said, is it "correct" to delete a floating ip without disassociating it?23:02
clarkbto me one implies the other23:02
clarkbbecause how can you be associated with a deleted thing?23:02
salv-orlandoclarkb: uhm… so when you delete a floating ip neutron implicitly removes any association with active ports (that’s VM for you)23:03
clarkbsalv-orlando: doesn't noav need to know about that?23:04
*** crose has quit IRC23:04
clarkbotherwise the ip listing will be wrong23:04
salv-orlandoclarkb: you might be right. I forgot that other thing about ip listing.23:04
salv-orlandothat however might be a different bug...23:05
*** networkstatic has quit IRC23:07
*** _cjones_ has quit IRC23:07
*** drjones has joined #openstack-neutron23:07
*** ZZelle has quit IRC23:08
*** banix has quit IRC23:10
*** dbite has quit IRC23:10
clarkbsalv-orlando: mordred: if we use neutron to list associations we will get teh correct mapping right?23:10
clarkbsalv-orlando: mordred: so nodepool may just need to be careful to use neutronclient for those other things23:10
salv-orlandoyes but in that case you will see association by ports, not vms23:10
clarkboh23:11
salv-orlandoclarkb: let’s just go a step back. If line 196 succeeds and line 203 fails then the floating IP is never removed from the nova list - causing it to appear as stale.23:13
clarkbsalv-orlando: the association is removed so we won't find it23:14
clarkbsalv-orlando: we are looking for IPs associated to the node that is being deleted23:14
salv-orlandoclarkb: right however if we change the code there, as also arosen said, to do things differently with neutron, by not disassociating first, then the siingle delete operation would do both disassociate and delete - and that will come for “free”. In this way, if that fails the IP should not appear as disassociated and you should be able to find it.23:15
*** emagana has joined #openstack-neutron23:15
salv-orlandohowever, when then you delete the VM… this will trigger a process in neutron which disassociates the floating IP but does not delete it, so you might back at square 1 :/23:15
*** emagana has quit IRC23:20
*** AbsinthMind has quit IRC23:23
*** SridharRamaswamy has joined #openstack-neutron23:24
nlahoutimarkmcclain: ping23:25
*** sbfox has quit IRC23:25
*** dconde has quit IRC23:25
*** Sridhar__ has joined #openstack-neutron23:26
*** diegows has quit IRC23:26
*** SridharR_ has quit IRC23:27
*** emagana has joined #openstack-neutron23:28
*** SridharRamaswamy has quit IRC23:28
*** lordd_ has quit IRC23:32
*** lordd_ has joined #openstack-neutron23:33
*** reed has joined #openstack-neutron23:34
*** padkrish_ has quit IRC23:34
*** lordd_ has quit IRC23:35
*** padkrish has joined #openstack-neutron23:35
*** lordd_ has joined #openstack-neutron23:35
*** cgoncalves has quit IRC23:36
*** cgoncalves has joined #openstack-neutron23:37
*** emagana has quit IRC23:39
*** emagana has joined #openstack-neutron23:39
*** padkrish has quit IRC23:39
*** emagana has quit IRC23:42
*** emagana has joined #openstack-neutron23:43
openstackgerritDoug Wiegley proposed a change to openstack/neutron: A10 Networks LBaaS v1 Driver  https://review.openstack.org/11370223:44
*** nati_ueno has joined #openstack-neutron23:45
*** Sridhar__ has quit IRC23:46
*** SridharRamaswamy has joined #openstack-neutron23:46
*** emagana has quit IRC23:47
*** flwang_ has joined #openstack-neutron23:51
*** SridharRamaswamy has quit IRC23:51
*** otherwiseguy has quit IRC23:54
*** flwang_ has quit IRC23:56

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