Friday, 2014-04-04

*** singhs has quit IRC00:03
*** sandr8 has joined #openstack-neutron00:03
*** dims_ has quit IRC00:07
*** hrushig has quit IRC00:08
openstackgerritA change was merged to openstack/neutron: Add a note that rpc_workers option is experimental  https://review.openstack.org/8516900:08
*** sbfox has quit IRC00:08
openstackgerritA change was merged to openstack/neutron: Permit ICMPv6 RAs only from known routers  https://review.openstack.org/7225200:11
*** gdubreui has quit IRC00:11
*** thuc has quit IRC00:12
*** thuc has joined #openstack-neutron00:12
*** networkstatic has quit IRC00:13
*** thuc has quit IRC00:16
*** dims_ has joined #openstack-neutron00:21
*** stmi has left #openstack-neutron00:23
*** dave_tucker is now known as dave_tucker_zzz00:24
*** tomoe_ has joined #openstack-neutron00:24
*** stmi has joined #openstack-neutron00:24
*** stmi has quit IRC00:25
*** manishg has quit IRC00:28
*** tomoe_ has quit IRC00:28
*** jaypipes has quit IRC00:30
*** manishg_ has quit IRC00:32
*** lollipop has quit IRC00:35
*** gaud has joined #openstack-neutron00:36
*** xuhanp has quit IRC00:41
*** _cjones_ has quit IRC00:45
*** _cjones_ has joined #openstack-neutron00:46
*** manishg has joined #openstack-neutron00:47
*** manishg_ has joined #openstack-neutron00:48
*** thesanchez has quit IRC00:49
*** manishg_ has quit IRC00:50
*** manishg has quit IRC00:50
*** _cjones_ has quit IRC00:50
openstackgerritRobert Kukura proposed a change to openstack/neutron: ML2: ODL driver sets port status  https://review.openstack.org/8517600:51
*** manishg has joined #openstack-neutron00:54
*** WackoRob_ has quit IRC00:54
*** WackoRobie has joined #openstack-neutron00:54
*** singhs has joined #openstack-neutron00:58
*** manishg has quit IRC01:00
*** singhs has quit IRC01:03
*** thesanchez has joined #openstack-neutron01:05
*** lollipop has joined #openstack-neutron01:09
*** sandr8 has quit IRC01:10
*** singhs_ has joined #openstack-neutron01:11
*** crc32 has quit IRC01:13
*** banix has joined #openstack-neutron01:17
*** zhenjie has quit IRC01:18
*** sandr8 has joined #openstack-neutron01:20
*** thuc has joined #openstack-neutron01:23
openstackgerritBrian Haley proposed a change to openstack/neutron: Improve iptables_manager _modify_rules() method  https://review.openstack.org/7754901:26
*** thuc has quit IRC01:27
*** xuhanp has joined #openstack-neutron01:29
*** sandr8 has quit IRC01:29
*** ctooley has joined #openstack-neutron01:33
*** ctooley has left #openstack-neutron01:34
*** Sukhdev has quit IRC01:34
*** gdubreui has joined #openstack-neutron01:35
openstackgerritBrian Haley proposed a change to openstack/neutron: Fix test MAC addresses to be valid  https://review.openstack.org/8521601:46
*** yfujioka has joined #openstack-neutron01:50
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests  https://review.openstack.org/8492301:59
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add functional tests to test detection of VXLAN support  https://review.openstack.org/8460401:59
mesterymarun: See the patches above ^^^02:01
mesterymarun: Thank you so much for your guidance, I have learned so much while doing this. These are now ready to merge!02:01
mesterymarun: And now I can work on even more functional tests! Yay!02:01
*** sbfox has joined #openstack-neutron02:03
*** tomoe_ has joined #openstack-neutron02:07
*** sbfox has quit IRC02:07
*** yamahata has joined #openstack-neutron02:09
*** xianghui has joined #openstack-neutron02:15
*** obondarev has quit IRC02:16
*** evilstephen has joined #openstack-neutron02:16
*** obondarev has joined #openstack-neutron02:17
*** ilyashakhat has quit IRC02:17
*** ilyashakhat has joined #openstack-neutron02:18
*** evilstephen has quit IRC02:20
*** changbl has joined #openstack-neutron02:22
*** sandr8 has joined #openstack-neutron02:22
*** morganfainberg is now known as morganfainberg_Z02:22
*** sandr8 has quit IRC02:23
*** HenryG has joined #openstack-neutron02:23
*** manishg has joined #openstack-neutron02:25
*** dims_ has quit IRC02:29
*** thurloat has quit IRC02:30
*** evilstephen has joined #openstack-neutron02:34
*** manishg has quit IRC02:41
openstackgerritXu Han Peng proposed a change to openstack/neutron: Trigger provider security group update for RA  https://review.openstack.org/8093202:41
*** jp_at_hp1 has quit IRC02:42
*** matsuhas_ has joined #openstack-neutron02:53
*** matsuhashi has quit IRC02:53
*** matsuhas_ has quit IRC02:54
*** matsuhashi has joined #openstack-neutron02:56
openstackgerritHenry Gessau proposed a change to openstack/neutron: WIP: add dependent plugin migrations  https://review.openstack.org/8451902:57
openstackgerritHenry Gessau proposed a change to openstack/neutron: Remove auto-generation of db schema from models at startup  https://review.openstack.org/4029602:57
*** WackoRobie has quit IRC03:01
*** WackoRobie has joined #openstack-neutron03:01
*** matsuhashi has quit IRC03:02
*** WackoRob_ has joined #openstack-neutron03:04
*** matsuhashi has joined #openstack-neutron03:04
openstackgerritHenry Gessau proposed a change to openstack/neutron: Move bash whitelisting to pep8 testenv  https://review.openstack.org/8522303:05
*** WackoRobie has quit IRC03:05
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885303:11
*** catohornet has joined #openstack-neutron03:15
*** ramishra has joined #openstack-neutron03:23
*** carl_baldwin has joined #openstack-neutron03:33
*** WackoRob_ has quit IRC03:34
openstackgerritHenry Gessau proposed a change to openstack/neutron: WIP: add dependent plugin migrations  https://review.openstack.org/8451903:37
*** bashok has joined #openstack-neutron03:37
HenryGoda-g: ping (about migrations for metaplugin)03:38
oda-gHenryG: pong03:38
HenryGoda-g: I started fixing from other plugin migrations, but not sure about meta. See here --> https://review.openstack.org/8451903:40
oda-gHenryG: I just noticed your comment of the bug report.03:40
HenryGoda-g: If you want to work separately on metaplugin migrations, I will remove it from my work.03:41
*** gaud has quit IRC03:42
oda-gHenryG: ah, yes. There are issues which are peculiar to metaplugin and they are not fixed by your patch.03:44
HenryGoda-g: That's what I suspected03:45
HenryGoda-g: So I can leave metaplugin migrations to you?03:45
oda-gHenryG: yes. I will do for metaplugin.03:46
HenryGoda-g: Thanks!03:46
oda-gHenryG: no problem.03:46
HenryGAny brocade plugin experts around?03:47
*** matsuhashi has quit IRC04:01
*** harlowja is now known as harlowja_away04:03
mesteryHenryG: For brocade, you want shivharris.04:03
HenryGmestery: Thanks!04:04
*** matsuhashi has joined #openstack-neutron04:05
*** catohornet has quit IRC04:14
*** chandan_kumar has joined #openstack-neutron04:16
*** sbfox has joined #openstack-neutron04:19
*** banix has quit IRC04:29
openstackgerritHenry Gessau proposed a change to openstack/neutron: WIP: fix brocade and embrane plugin migrations  https://review.openstack.org/8451904:32
*** geekinutah has quit IRC04:43
*** bashok has quit IRC04:43
*** geekinutah has joined #openstack-neutron04:44
*** carl_baldwin has quit IRC04:45
*** carl_baldwin has joined #openstack-neutron04:47
*** skraynev_afk is now known as skraynev04:48
*** Longgeek has joined #openstack-neutron04:50
*** yfujioka has quit IRC05:01
*** Longgeek has quit IRC05:05
*** chandan_kumar has quit IRC05:06
*** ben00b has joined #openstack-neutron05:06
*** Longgeek has joined #openstack-neutron05:06
*** ben00b has quit IRC05:08
*** chandan_kumar has joined #openstack-neutron05:08
*** lollipop has quit IRC05:25
*** harlowja_away has quit IRC05:29
*** Longgeek has quit IRC05:33
*** lollipop has joined #openstack-neutron05:41
*** Longgeek has joined #openstack-neutron05:51
*** Longgeek has quit IRC05:58
*** carl_baldwin has quit IRC06:11
openstackgerritAaron Rosen proposed a change to openstack/neutron: Test  https://review.openstack.org/8450106:22
*** Longgeek has joined #openstack-neutron06:23
openstackgerritberlin proposed a change to openstack/neutron: Fix LBaaS Haproxy occurs error if no member is added  https://review.openstack.org/8525106:26
openstackgerritfumihiko kakuma proposed a change to openstack/neutron: OFAgent: Process port_update notifications in the main agent loop  https://review.openstack.org/8086406:34
*** bashok has joined #openstack-neutron06:35
openstackgerritJenkins proposed a change to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/8525406:35
*** bvandenh has joined #openstack-neutron06:38
*** jlibosva has joined #openstack-neutron06:51
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885306:54
*** chandankumar_ has joined #openstack-neutron06:59
*** singhs_ has quit IRC07:03
*** rotbeard has joined #openstack-neutron07:05
*** sandr8 has joined #openstack-neutron07:06
*** yamahata has quit IRC07:06
*** matsuhashi has quit IRC07:07
*** yfried__ has quit IRC07:09
*** saju_m has joined #openstack-neutron07:12
*** gdubreui has quit IRC07:15
*** sandr8_ has joined #openstack-neutron07:16
*** sandr8 has quit IRC07:16
*** Longgeek_ has joined #openstack-neutron07:21
openstackgerritLiping Mao proposed a change to openstack/neutron: LBaaS VIP doesn't work after delete and re-add  https://review.openstack.org/8464707:22
*** leseb has joined #openstack-neutron07:23
*** chandankumar_ has quit IRC07:23
*** Longgeek has quit IRC07:24
*** yamahata has joined #openstack-neutron07:24
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885307:32
*** liudong has joined #openstack-neutron07:33
openstackgerritThomas Herve proposed a change to openstack/python-neutronclient: Return response status reason on error  https://review.openstack.org/7755907:34
*** vthapar has joined #openstack-neutron07:35
*** sandr8_ has quit IRC07:36
*** jgallard has joined #openstack-neutron07:38
*** ramishra has quit IRC07:43
matrohonamotoki : hi07:46
*** Longgeek_ has quit IRC07:59
*** matrohon has quit IRC08:00
*** jpich has joined #openstack-neutron08:00
*** Longgeek has joined #openstack-neutron08:01
*** matrohon has joined #openstack-neutron08:02
*** ygbo has joined #openstack-neutron08:04
*** matsuhashi has joined #openstack-neutron08:04
*** jistr has joined #openstack-neutron08:10
*** safchain has joined #openstack-neutron08:10
matrohonrkukura : hi08:11
openstackgerritAkihiro Motoki proposed a change to openstack/neutron: Validate maximum length for string fields  https://review.openstack.org/8470808:14
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator  https://review.openstack.org/8121108:14
*** Longgeek_ has joined #openstack-neutron08:14
*** Longgeek has quit IRC08:16
*** yfried has joined #openstack-neutron08:23
*** Longgeek_ has quit IRC08:31
openstackgerritLiping Mao proposed a change to openstack/neutron: LBaaS VIP doesn't work after delete and re-add  https://review.openstack.org/8464708:32
*** matsuhashi has quit IRC08:35
*** Longgeek has joined #openstack-neutron08:36
openstackgerritberlin proposed a change to openstack/neutron: Fix LBaaS Haproxy occurs error if no member is added  https://review.openstack.org/8525108:37
*** matrohon has quit IRC08:50
*** xuhanp has quit IRC08:50
*** matrohon has joined #openstack-neutron08:51
*** matsuhashi has joined #openstack-neutron08:53
*** jp_at_hp has joined #openstack-neutron09:01
*** mjbot has joined #openstack-neutron09:03
*** mjbot has quit IRC09:03
*** mjbot has joined #openstack-neutron09:05
*** matrohon has quit IRC09:07
*** enikanorov__ has quit IRC09:13
*** liudong has quit IRC09:25
*** Longgeek has quit IRC09:27
*** devvesa has joined #openstack-neutron09:34
*** Longgeek has joined #openstack-neutron09:35
openstackgerritA change was merged to openstack/neutron: ML2: ODL driver sets port status  https://review.openstack.org/8517609:36
ekarlsoSo I have agents running (I see the processes) but I see neutron are marking them as offline / online all the time on one host. Clues to what that can be ?09:36
ekarlsonothing in the logs to indicate it09:37
*** iwi has joined #openstack-neutron09:39
*** Longgeek has quit IRC09:39
openstackgerritA change was merged to openstack/neutron: Set ns_name in RouterInfo as attribute  https://review.openstack.org/8505109:40
*** Longgeek has joined #openstack-neutron09:41
openstackgerritAkihiro Motoki proposed a change to openstack/neutron: Move whitelist_externals = bash to pep8 section in tox.ini  https://review.openstack.org/8527909:41
*** Longgeek has quit IRC09:43
*** Longgeek has joined #openstack-neutron09:43
*** matsuhashi has quit IRC09:47
*** matsuhashi has joined #openstack-neutron09:47
*** Longgeek_ has joined #openstack-neutron09:50
*** lollipop has quit IRC09:52
*** Longgeek has quit IRC09:53
*** jprovazn has joined #openstack-neutron09:53
openstackgerritA change was merged to openstack/neutron: Advanced Services documentation  https://review.openstack.org/8082509:55
*** enikanorov has joined #openstack-neutron09:55
iwihi there, is it possible to share a network only to certain tenants ?09:56
*** saju_m has quit IRC09:56
*** jamespage_ has joined #openstack-neutron09:58
*** Longgeek has joined #openstack-neutron09:58
*** Longgeek_ has quit IRC09:59
*** Longgeek_ has joined #openstack-neutron10:00
*** Longgeek has quit IRC10:03
*** leseb has quit IRC10:04
*** tomoe_ has quit IRC10:06
*** jamespage_ has quit IRC10:06
*** tomoe_ has joined #openstack-neutron10:07
*** Longgeek_ has quit IRC10:08
*** saju_m has joined #openstack-neutron10:09
*** overlayer has joined #openstack-neutron10:11
*** tomoe_ has quit IRC10:11
*** jgallard has quit IRC10:12
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Metadata agent caches networks for routers  https://review.openstack.org/7068910:12
*** dave_tucker_zzz is now known as dave_tucker10:13
*** saju_m has quit IRC10:14
*** Longgeek has joined #openstack-neutron10:16
*** leseb has joined #openstack-neutron10:17
openstackgerritJakub Libosvar proposed a change to openstack/neutron: Sync service and systemd modules from oslo-incubator  https://review.openstack.org/8121110:17
*** pcm_ has joined #openstack-neutron10:21
*** Longgeek has quit IRC10:22
*** pcm_ has quit IRC10:22
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove device_exists in LinuxBridgeManager  https://review.openstack.org/8529710:22
*** pcm_ has joined #openstack-neutron10:23
*** matsuhashi has quit IRC10:23
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove device_exists in LinuxBridgeManager  https://review.openstack.org/8529710:25
*** Longgeek has joined #openstack-neutron10:28
*** saju_m has joined #openstack-neutron10:30
*** matsuhashi has joined #openstack-neutron10:30
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: Don't emit log for missing attribute check policy  https://review.openstack.org/8529910:33
*** saju_m has quit IRC10:35
*** terrylhowe has joined #openstack-neutron10:35
*** saju_m has joined #openstack-neutron10:35
*** jistr has quit IRC10:37
*** matsuhashi has quit IRC10:38
*** chandankumar_ has joined #openstack-neutron10:39
*** saju_m has quit IRC10:40
*** saju_m has joined #openstack-neutron10:40
*** matsuhas_ has joined #openstack-neutron10:41
*** matsuhas_ has quit IRC10:52
*** matsuhashi has joined #openstack-neutron10:52
*** terrylhowe has left #openstack-neutron10:53
*** chandankumar_ has quit IRC10:55
*** matsuhashi has quit IRC10:57
*** jistr has joined #openstack-neutron10:58
*** Longgeek has quit IRC11:00
*** dave_tucker is now known as dave_tucker_zzz11:00
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885311:02
*** xianghui has quit IRC11:05
*** dave_tucker_zzz is now known as dave_tucker11:06
*** dave_tucker is now known as dave_tucker_zzz11:09
kevinbentonenikanorov: ping11:10
*** dave_tucker_zzz is now known as dave_tucker11:10
*** chandankumar_ has joined #openstack-neutron11:13
*** b3nt_pin is now known as beagles11:19
*** mwagner_lap has quit IRC11:22
*** yamahata has quit IRC11:36
*** dave_tucker is now known as dave_tucker_zzz11:37
*** rdo has quit IRC11:37
*** saju_m has quit IRC11:38
*** Longgeek has joined #openstack-neutron11:40
*** dave_tucker_zzz is now known as dave_tucker11:41
*** rdo has joined #openstack-neutron11:41
*** matrohon has joined #openstack-neutron11:51
*** ihrachys has quit IRC11:55
*** Longgeek has quit IRC12:01
*** mwagner_lap has joined #openstack-neutron12:03
*** vthapar has quit IRC12:04
*** dave_tucker is now known as dave_tucker_zzz12:09
enikanorovkevinbenton: pong12:09
*** saju_m has joined #openstack-neutron12:10
*** linuxaddicts has quit IRC12:12
*** jlibosva is now known as jlibosva|afk12:14
*** Zhou_Yu has joined #openstack-neutron12:14
*** saju_m has quit IRC12:14
*** saju_m has joined #openstack-neutron12:15
enikanorovkevinbenton: i was expecting PBDB_MODULE to act like a singleton, when module is imported you set PBDB_MODULE to that module, instead of calling import each time. because that's what you currently do12:16
*** WackoRobie has joined #openstack-neutron12:18
*** WackoRobie has quit IRC12:20
*** WackoRobie has joined #openstack-neutron12:21
*** jgallard has joined #openstack-neutron12:21
*** WackoRobie has quit IRC12:26
*** eezhova has quit IRC12:32
*** enikanorov has quit IRC12:34
matrohonrkukura : are you here?12:35
openstackgerritYAMAMOTO Takashi proposed a change to openstack/neutron: ofa_neutron_agent: Fix _phys_br_block_untranslated_traffic  https://review.openstack.org/8532912:35
*** skraynev is now known as skraynev_afk12:35
*** dave_tucker_zzz is now known as dave_tucker12:35
*** linuxaddicts has joined #openstack-neutron12:36
*** evilstephen has quit IRC12:37
*** evilstephen has joined #openstack-neutron12:38
*** amotoki has joined #openstack-neutron12:39
*** gaud has joined #openstack-neutron12:43
*** jlibosva|afk is now known as jlibosva12:45
*** alexpilotti has joined #openstack-neutron12:46
*** sballe has quit IRC12:46
*** sballe has joined #openstack-neutron12:47
*** chandankumar_ has quit IRC12:47
*** enikanorov has joined #openstack-neutron12:51
*** julim has joined #openstack-neutron12:55
*** thuc has joined #openstack-neutron12:55
*** thuc_ has joined #openstack-neutron12:56
*** saju_m has quit IRC12:57
*** jecarey has joined #openstack-neutron12:58
*** enikanorov has quit IRC12:58
*** thuc has quit IRC13:00
*** baoli has joined #openstack-neutron13:02
*** julim has quit IRC13:03
*** alexpilotti_ has joined #openstack-neutron13:03
*** julim has joined #openstack-neutron13:05
*** alexpilotti has quit IRC13:06
*** alexpilotti_ has quit IRC13:06
*** banix has joined #openstack-neutron13:08
*** baoli has quit IRC13:14
*** baoli has joined #openstack-neutron13:14
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Check bridge exists and is UP in ensure_bridge  https://review.openstack.org/7630913:17
*** sballe has quit IRC13:18
*** sballe has joined #openstack-neutron13:18
*** nplanel has quit IRC13:19
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Check if bridge exists and make sure it's UP in ensure_bridge  https://review.openstack.org/7630913:19
*** Longgeek has joined #openstack-neutron13:20
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests  https://review.openstack.org/8492313:23
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add functional tests to test detection of VXLAN support  https://review.openstack.org/8460413:23
*** krtaylor has joined #openstack-neutron13:24
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Check if bridge exists and make sure it's UP in ensure_bridge  https://review.openstack.org/7630913:25
*** zana has joined #openstack-neutron13:28
zanahello13:28
zanaI want to take a look into sources of neutron13:29
zanaso could anybody give me a guidance as a starting point13:29
*** ihrachyshka has joined #openstack-neutron13:32
*** thuc_ has quit IRC13:40
*** thuc has joined #openstack-neutron13:41
pcm_zana: Do you know about DevStack?13:44
*** thuc has quit IRC13:45
*** thuc has joined #openstack-neutron13:46
*** bashok has quit IRC13:49
*** nplanel has joined #openstack-neutron13:50
*** carl_baldwin has joined #openstack-neutron13:52
*** carl_baldwin has quit IRC13:52
*** alagalah has joined #openstack-neutron13:53
*** prad has joined #openstack-neutron13:54
*** WackoRobie has joined #openstack-neutron13:56
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: Call policy.init() once per API request  https://review.openstack.org/8535413:58
*** nplanel_ has joined #openstack-neutron13:59
*** lollipop has joined #openstack-neutron14:01
openstackgerritfumihiko kakuma proposed a change to openstack/neutron: OFAgent: Avoid re-wiring ports unnecessarily  https://review.openstack.org/8535614:01
*** thuc has quit IRC14:04
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Check if bridge exists and make sure it's UP in ensure_bridge  https://review.openstack.org/7630914:04
*** thuc has joined #openstack-neutron14:04
rkukuramatrohon: Still looking for me?14:06
*** sgran has quit IRC14:07
*** thuc has quit IRC14:08
*** sgran has joined #openstack-neutron14:08
matrohonrkukura : hi, I was wondering how the process of nova waiting for neutron to tell it that the port was active before booting a VM, was working exactly with a agent base MD14:09
matrohonrkukura : dansmith answered me on openstack-dev14:09
rkukuramatrohon: All set? Is it working OK?14:10
rkukuramatrohon: The get_device_details RPC currently sets status to ACTIVE as a side-effect, right?14:10
*** djoreilly has joined #openstack-neutron14:11
*** armax has joined #openstack-neutron14:11
matrohonrkukura : yes, and vm are started but not booted, and when it get started, port are created so neutron gcan do its job14:11
openstackgerritA change was merged to openstack/neutron: Redundant SG rule create calls in unit tests  https://review.openstack.org/8512714:13
*** Longgeek has quit IRC14:13
dansmithmatrohon: note that there is not full support for this in all the neutron plugins,14:14
dansmithmatrohon: which means some of them currently "lie" about being done, which causes nova to continue almost immediately14:14
matrohonrkukura : the same process is not used for live-migration, vm are only booted. I think we will have to use such a process to live-migrate to another MD14:14
matrohonbut nova will also have to deal with another vif_type14:15
*** jgrimm has joined #openstack-neutron14:15
*** alexpilotti has joined #openstack-neutron14:15
*** Longgeek_ has joined #openstack-neutron14:17
matrohondansmith : I recall that the NSX patch call the NSX controller, the patch proposed by rkukura lie about the status of the port, am I wrong :14:17
matrohonhttps://review.openstack.org/#/c/85176/14:18
dansmithyeah, I really know very little about the neutron side, so I can't offer much help :)14:19
mesterymatrohon: How is it lieing about the state of the port? For ODL, we don't do anything, so once libvirt brings the VM up the port is effectively up.14:20
mesterymatrohon: Am I missing something?14:20
*** markmcclain has joined #openstack-neutron14:21
*** evgenyf has joined #openstack-neutron14:24
matrohonmestery : i'm not an ODL specialist, but you might have to configure the OVS flows running on the compute-node before booting the VM. That is probably the reason why NSX is calling his controller before setting the status of the port.14:24
matrohonmestery : but if you don't need to, i'm fine with that14:24
mesterymatrohon: I think the ODL guys default to dropping everything with the OpenFlow 1.3 integration, but you're right about this. I'll talk to them and get back to you.14:25
mesterymatrohon: In the meantime, this was a decent workaround to unblock Icehouse.14:25
matrohonmestery : I understand this point of course14:25
mesterymarkmcclain: If there is an RC2, can we get this ODL bug included? https://launchpad.net/bugs/1301449 https://review.openstack.org/#/c/85176/14:25
markmcclainmestery: there will be definitely be an RC2 I'll make sure those get backported14:26
*** thedodd has joined #openstack-neutron14:26
dansmithmestery: the only question is "will the guest be able to get dhcp" to determine whether you're firing the event early or at the right time14:27
mesterymarkmcclain: Thanks! The bug is marked "icehouse-rc-potential", should I mark it backport potentail too?14:27
matrohonmarkmclain : I hope this one will be in RC2 too : https://review.openstack.org/#/c/52884/14:27
matrohonmarkmcclain: ^^14:27
markmcclainmestery: rc-potential is fine since those are the bugs I'm going to backport14:27
mesterymarkmcclain: Thank you!14:27
mesterydansmith: That's right, my testing has shown this works so far, but I think a more elegant solution in Juno is in order here. Thanks!14:28
markmcclainmatrohon: 52884 is bit of a different case because it is large for an RC backport14:28
dansmithmestery: well, it was extremely racy before, so just "it worked in my testing" isn't an airtight answer, but ... yeah :)14:28
mesterydansmith: I hear you, no doubt. And since I'm striving to make ODL more bulletproof here, we'll fix this soon.14:29
dansmithmestery: excellent :)14:29
matrohonmarkmcclain : it has been hanging around since a while and has been reviewed by a lot of people. hope you will consider it14:30
*** saju_m has joined #openstack-neutron14:31
markmcclainmatrohon: yeah the real trick managing risk14:32
markmcclainand this does a fairly common code path which is why I'm not sure it has enough tests to make sure we haven't introduced any regressions14:33
*** saju_m has quit IRC14:34
*** saju_m has joined #openstack-neutron14:35
*** tvardeman has joined #openstack-neutron14:39
*** jistr is now known as jistr|biab14:40
*** tvardeman has quit IRC14:40
*** tvardeman has joined #openstack-neutron14:47
*** mlavalle has joined #openstack-neutron14:51
*** WackoRobie has quit IRC14:51
*** WackoRobie has joined #openstack-neutron14:52
mlavalleuma: can we talk a few minutes over TS?14:53
*** carl_baldwin has joined #openstack-neutron14:54
mlavalleTeam the team14:54
openstackgerritSalvatore Orlando proposed a change to openstack/neutron: Call policy.init() once per API request  https://review.openstack.org/8535414:54
*** yamahata has joined #openstack-neutron14:55
*** yfried has quit IRC14:55
*** WackoRobie has quit IRC14:56
*** yfried has joined #openstack-neutron14:56
*** matrohon has quit IRC14:59
*** thuc_ has joined #openstack-neutron15:01
*** thuc_ has quit IRC15:03
*** thuc_ has joined #openstack-neutron15:04
*** feleouet has quit IRC15:05
*** WackoRobie has joined #openstack-neutron15:06
*** rotbeard has quit IRC15:12
*** jlibosva has quit IRC15:18
*** evgenyf has quit IRC15:20
*** tvardeman has quit IRC15:24
*** nplanel_ has quit IRC15:27
*** lollipop has quit IRC15:28
*** pasquier-s has quit IRC15:31
*** ihrachyshka has quit IRC15:31
*** WackoRob_ has joined #openstack-neutron15:33
openstackgerritYves-Gwenael Bourhis proposed a change to openstack/neutron: Changed DictModel to dict with attribute access  https://review.openstack.org/6469615:34
*** saju_m has quit IRC15:35
*** WackoRobie has quit IRC15:35
*** dave_tucker is now known as dave_tucker_zzz15:36
*** tvardeman has joined #openstack-neutron15:36
*** thuc_ has quit IRC15:38
*** jprovazn has quit IRC15:41
*** sbfox1 has joined #openstack-neutron15:42
*** sbfox has quit IRC15:43
*** yfried has quit IRC15:44
openstackgerritYves-Gwenael Bourhis proposed a change to openstack/neutron: Changed DictModel to dict with attribute access  https://review.openstack.org/6469615:47
*** markmcclain has quit IRC15:48
*** thesanchez has quit IRC15:48
*** coolsvap has joined #openstack-neutron15:54
*** jistr|biab is now known as jistr15:55
HenryGmlavalle: ping? You mentioned at the last neutron meeting that you might assign me some tempest tests that have been abandoned?15:55
mlavalleHenryG: Thanks for following up. I found out that the one I had in mind wasn't abandoned. It was just replaced15:56
HenryGmlavalle: phew. :)15:56
mlavalleHenryG: There is another one for FWaaS. Wopuld you be interested on working that one?15:57
*** Zhou_Yu has quit IRC15:58
*** baoli has quit IRC15:59
HenryGmlavalle: point me to it so I can scope it. I can maybe work on it, or maybe find someone to work on it.15:59
openstackgerritA change was merged to openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/8525416:00
mlavalleHenryG: this is it. Take a look and let me know if you want to work it: https://review.openstack.org/#/c/65744.16:00
mlavalleHenryG: If you don't like it, I am willing to do it16:00
*** RockKuo_TW has quit IRC16:01
*** ygbo has quit IRC16:01
*** ihrachyshka has joined #openstack-neutron16:02
mlavalleHenryG: either way, thanks for the interest and follow up :-)16:02
*** singhs has joined #openstack-neutron16:03
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Check if bridge exists and make sure it's UP in ensure_bridge  https://review.openstack.org/7630916:03
openstackgerritRossella Sblendido proposed a change to openstack/neutron: Remove device_exists in LinuxBridgeManager  https://review.openstack.org/8529716:03
*** thesanchez has joined #openstack-neutron16:04
*** dave_tucker_zzz is now known as dave_tucker16:06
*** comstud is now known as bearhands16:06
*** russellb is now known as rustlebee16:06
*** dansmith is now known as damnsmith16:06
HenryGmlavalle: I will work on it.16:08
HenryGmlavalle: wasn't there another firewall review that did much of the same thing?16:09
*** baoli has joined #openstack-neutron16:11
mlavalleSumitNaiksatam: HenryG is willing to continue the work on a FWaaS api test. He is planning to continue working https://review.openstack.org/#/c/65744 So we don't duplicate efforts, are you aware of any other effort in this direction?16:12
*** beagles is now known as seagulls16:13
SumitNaiksatammlavalle HenryG: awesome!16:13
*** ihrachyshka has quit IRC16:13
*** _cjones_ has joined #openstack-neutron16:13
SumitNaiksatammlavalle: not that i am aware of…earlier yair and EmilienM seem to have been working on it16:13
openstackgerritCarl Baldwin proposed a change to openstack/neutron: Clean out namespaces even if we don't delete namespaces  https://review.openstack.org/8457016:14
SumitNaiksatamHenryG: the first order of business, i believe is to get fwaas enabled at the gate16:14
mlavalleSumitNaiksatam: OK, let's assume this patchset is the good one, so HenryG can continue the work16:15
SumitNaiksatammlavalle: yeah sure16:15
SumitNaiksatamHenryG: SridarK had done local testing to test that the tempest tests would not blow up if fwaas was enabled16:15
*** thuc has joined #openstack-neutron16:16
HenryGmlavalle: SumitNaiksatam: great. I also asked SridarK to check for other fwaas tempest reviews. He'll get back to me shortly.16:16
*** sbfox1 has quit IRC16:16
SumitNaiksatamHenryG: https://wiki.openstack.org/wiki/Quantum/FWaaS/Testing16:16
mlavalleHenryG: Now you are in trouble, your name is in my tracking spreadsheet next to "fwaas" ;-)16:16
* HenryG is always getting himself into trouble16:17
*** thuc_ has joined #openstack-neutron16:17
SumitNaiksatamHenryG: if you have time, it will be great if you can stop by at the fwaas meetings16:17
SumitNaiksatamHenryG: i will add you to the list as well if we are having meetings16:17
*** thuc_ has quit IRC16:17
*** thuc_ has joined #openstack-neutron16:18
*** jpich has quit IRC16:18
*** thuc_ has quit IRC16:19
HenryGSumitNaiksatam: ok, will add it to my calendar16:19
SumitNaiksatamHenryG: thanks16:19
*** thuc_ has joined #openstack-neutron16:19
*** thuc has quit IRC16:20
HenryGoda-g: ping (about migrations for metaplugin)16:21
*** singhs has quit IRC16:21
openstackgerritHenry Gessau proposed a change to openstack/neutron: WIP: fix brocade and embrane plugin migrations  https://review.openstack.org/8451916:23
*** marun has quit IRC16:23
*** manishg has joined #openstack-neutron16:23
*** htruta has joined #openstack-neutron16:25
*** WackoRob_ has quit IRC16:28
htrutahello, guys! does anyone know how neutron uses ceilometer data?16:28
*** csd_ has joined #openstack-neutron16:29
*** jp_at_hp1 has joined #openstack-neutron16:31
*** leseb has quit IRC16:31
*** tomoe_ has joined #openstack-neutron16:32
*** jp_at_hp has quit IRC16:33
openstackgerritDarragh O'Reilly proposed a change to openstack/neutron: lb-agent: reduce delete_vlan_bridge() logging severity  https://review.openstack.org/7599316:36
*** Longgeek_ has quit IRC16:36
*** baoli has quit IRC16:41
*** baoli has joined #openstack-neutron16:41
*** chandankumar_ has joined #openstack-neutron16:42
*** tomoe_ has quit IRC16:42
*** jgallard has quit IRC16:46
*** jogo is now known as flashgordon16:46
*** larsks has joined #openstack-neutron16:47
larsksIn neutron/db/l3_db.py, in disassociate_floatingips(), there is a comment that says, "should never happen" w/r/t multiple floating ips being found associated with the instance being deleted...but it's clearly possible for an instance to have multiple floating ips associate with different fixed ip addresses.16:49
larsksSo, is multiple floating ips "possible but wrong"?  Or does this code need to be udpated?16:49
*** crc32 has joined #openstack-neutron16:50
*** dave_tucker is now known as dave_tucker_zzz16:51
*** dave_tucker_zzz is now known as dave_tucker16:53
*** markmcclain has joined #openstack-neutron16:55
*** markwash has joined #openstack-neutron16:55
*** markmcclain1 has joined #openstack-neutron16:57
*** catohornet has joined #openstack-neutron16:57
*** jistr is now known as jistr|afk16:58
*** csd_ has quit IRC16:59
*** markmcclain has quit IRC16:59
*** markmcclain1 has quit IRC17:01
*** csd_ has joined #openstack-neutron17:01
openstackgerritDarragh O'Reilly proposed a change to openstack/neutron: lb-agent: fix get_interfaces_on_bridge returning None  https://review.openstack.org/8541917:02
htrutadoes anyone know?17:03
*** harlowja has joined #openstack-neutron17:04
*** SumitNaiksatam has quit IRC17:06
*** jobewan has joined #openstack-neutron17:07
*** chandankumar_ has quit IRC17:09
overlayerhtruta, explain17:09
htrutadoes neutron collect and use any ceilometer information?17:10
overlayerhtruta, try #openstack17:10
htrutai've heard something that it collects and makes some kind of analysys to improve the bandwith usage17:10
htrutaoverlayer: ok, i'll try it! tks17:12
*** TrevorV has joined #openstack-neutron17:14
*** luqas has quit IRC17:17
*** networkstatic has joined #openstack-neutron17:19
*** SumitNaiksatam has joined #openstack-neutron17:21
*** jp_at_hp1 has quit IRC17:22
*** csd has quit IRC17:23
*** ijw has joined #openstack-neutron17:24
*** safchain has quit IRC17:24
*** tvardeman has quit IRC17:27
*** TrevorV has quit IRC17:32
*** leseb has joined #openstack-neutron17:32
*** TrevorV has joined #openstack-neutron17:32
*** sandr8 has joined #openstack-neutron17:34
*** leseb has quit IRC17:37
*** TrevorV has quit IRC17:37
*** TrevorV has joined #openstack-neutron17:38
*** prad has quit IRC17:39
*** prad has joined #openstack-neutron17:39
*** bjornar has joined #openstack-neutron17:42
*** morganfainberg_Z is now known as morganfainberg17:45
*** dave_tucker is now known as dave_tucker_zzz17:46
*** dave_tucker_zzz is now known as dave_tucker17:47
*** thuc_ has quit IRC17:50
openstackgerritLars Kellogg-Stedman proposed a change to openstack/neutron: allow delete_port to work when there are multiple floating ips  https://review.openstack.org/8543217:52
*** djoreilly has quit IRC17:54
*** thuc has joined #openstack-neutron17:57
*** thuc has quit IRC17:57
arosenmestery:  ping17:58
*** thuc has joined #openstack-neutron17:58
arosenJust curious why https://review.openstack.org/#/c/75253/ broke ODL?17:59
*** openstackgerrit has quit IRC18:01
mesteryarosen: pong18:01
arosenyo18:02
*** openstackgerrit has joined #openstack-neutron18:02
mesteryarosen: It was because ODL has no agent, and was assuming when it got the bind call the port was up.18:02
*** marun has joined #openstack-neutron18:02
mesteryarosen: rkukura submitted a fix for this which was merged last night, so we're good for RC2 now.18:02
mesteryI'll fix this up to be cleaner in Juno.18:02
mesteryFrom the ODL side.18:02
arosenmestery:  When someone does a port list and you're using ODL what does the port status show?18:03
mesteryarosen: Before this patch it showed UP18:03
arosenThat event stuff relies on the port status going active at some point.18:03
mesteryOr ACTIVe18:03
mesterysorry18:03
arosenmestery:  from bobs patch it looks like the port status was always down.18:04
mesterymarun: HenryG and I had one question for you in my functional test reviews, otherwise they are ready to merge!18:04
arosenThe agent is what sets it ACTIVE.18:04
marun mestery: if you do say so yourself ;)18:05
mesterymarun: Seriously, they are awesome!18:05
mesteryarosen: ODL has no agent :)18:05
marun*sigh*18:05
mesterymarun: ???18:05
arosenmestery:  right; i understand :). I believe the port status when using ODL was always DOWN since nothing every set it ACTIVE.18:06
mesteryarosen: Makes sense, so your change made it so nova wouldn't bring things up. :)18:06
arosenmestery: anyways, no worries. Glad you guys got it sorted out. I was worried that patch could break some peoples plugin if they didn't handle port status correctly.18:06
arosenmarun: had pinged the list about this plugin status requirement before we made the change fwiw18:07
marunarosen: silly rabbit, nobody actually reads the list! ;)18:07
arosenHa yea i know was just saying we didn't blindly make that change.18:08
mesteryarosen marun: It's all good.18:08
*** WackoRobie has joined #openstack-neutron18:08
arosenmestery:  I'll be happy to +2 that backport to icehouse when we can start backporting stuff.18:09
mesteryarosen: Thanks man!18:09
arosenrossella_s:  ping18:09
arosenrossella_s:  you already working on this one: https://bugs.launchpad.net/neutron/+bug/1302312 ?18:10
openstackgerritA change was merged to openstack/neutron: Cisco VPN driver correct reporting for admin state chg  https://review.openstack.org/8230618:13
*** morganfainberg is now known as morganfainberg_Z18:14
*** markmcclain has joined #openstack-neutron18:16
*** yamahata has quit IRC18:19
rossella_sarosen: I was about to start, it was unassigned I thought you weren't working on it18:21
rossella_sarosen: shall I leave it to you?18:21
arosenrossella_s: nah go for it.18:21
arosenrossella_s:  seems like dhcp in havana/icehouse has regressed a lot18:21
rossella_sarosen: here it's kind of late, I will work on it on monday, is that ok?18:22
arosenwe just deployed havana in our cloud and i  had to make several changes to make it even work.18:22
roaetmestery: ping18:22
mesteryroaet: Pong18:22
arosenrossella_s:  sure no worries. If i get to it today i might steal it back but i'll leave  it to you for now.18:23
arosenrossella_s:  where are you located again?18:23
rossella_sarosen: italy18:23
arosenah a fellow comrade of salv-orlando and armax :)18:23
rossella_sarosen: anyway what kind of problems are you facing with your cloud? we deploy havana and it's working without any change18:23
rossella_sarosen: yes :)18:23
roaetmestery: mind if I send you a query? had a question for you18:23
mesteryroaet: Sure, please do!18:24
arosenrossella_s:  it works but it's crazy chatty with the neutron-server18:24
marunmestery, HenryG: The scope for save-and-reraise is extremely specific18:24
arosenrossella_s: if you have 5k networks it can't nearly handle that.18:24
rossella_sarosen: oh yes...but you can increase the value for the interval of the report_state18:24
arosenrossella_s:  i'll upstream all my changes.18:24
marunIt's not needed unless a) eventlet is in play or b) something unexpected is likely in the exception handler18:24
marunNeither is the case in the functional test example18:25
mesterymarun: Thanks for clarifying.18:25
HenryGmarun: thanks18:25
rossella_sarosen: yes, thanks, let me know if you get to work on the bug, otherwise I will take it on monday18:25
arosenrossella_s:  even still... It does a lot of unnecessary queries for get_dhcp_port that it doesn't need to do.18:25
marunFeel free to review the docs for save_and_reraise to confirm this for yourself.18:25
rossella_sarosen: I know...we can improve it18:25
*** TrevorV has quit IRC18:25
rossella_sarosen: I can work on it, I have spare cycles next week18:25
mesteryThanks marun.18:25
arosenrossella_s:  cool. Ciao!18:25
mesterymarun: I'm going to change that back and repost both (rosella_s had some good comments on the second patch).18:25
mesterymarun: But they are in great shape thanks to you and HenryG and rosella_s!18:26
HenryGmarun: So functional tests will never run with eventlet monkeypatched in?18:26
marunregarding rossella_s's comments, I don't agree18:26
rossella_sarosen: bye! have a nice day! I try to look into the chatty dhcp...will ping you if I have some question if you don't mind18:26
marunmestery: ^18:26
mesterymarun: Why?18:26
arosensure18:26
rossella_sarosen: thanks :)18:27
marunmestery: My previous comments were not to remove all args18:27
*** WackoRobie has quit IRC18:27
mesterymarun: Just to add *args, **kwargs, right?18:27
*** julim has quit IRC18:27
marunmestery: but the arguments for the create_func *should* be dynamic18:27
marunmestery: otherwise we end up doing way too much work for each subsequent resource that needs to be created18:27
marunmestery: we'd be duplicating the signature/docs/etc18:28
mesterymarun: Right, so I can add back "prefix, create_func" to create_resource() and we're good, right?18:28
marunmestery: I think that's a good place to be for now18:28
mesterymarun: Because it fits that description once I do that, or even before if you're ok with argv[0] argv[1]18:28
mesterymarun: Cool18:28
mesterymarun: Give me a few minutes and I'll send out the new patch.18:28
rossella_smarun, mestery: if the idea is to make it as dynamic as possible I agree with you...I don't have the big picture18:29
marunmestery, rossella_s: fyi, the model for what I was envisioning was similar to unittest's assertRaises18:29
marunpass the function and its arguments and have assertRaises do the calling18:29
marunthe funky bit is how the name is applied to the argument list18:29
marunfor now we'll just prepend, but at some point it should be more explicit18:29
rossella_smarun: got it18:30
maruni.e the caller should pass a callable that accepts only a name, or maybe just a callable that knows how to create the name itself.18:30
*** luqas has joined #openstack-neutron18:30
mesterymarun: nice18:30
marunbut, baby steps :)18:30
rossella_smarun, mestery I have to go now...I will read when I am back if the conversation goes on. Happy weekend everybody18:30
marunrossella_s: take care!18:31
mesterybye rossella_s! Have a nice weekend!18:31
*** jistr|afk has quit IRC18:32
*** leseb has joined #openstack-neutron18:33
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests  https://review.openstack.org/8492318:34
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add functional tests to test detection of VXLAN support  https://review.openstack.org/8460418:34
mesterymarun HenryG: ^^^^ Reviews please. :)18:35
*** morganfainberg_Z is now known as morganfainberg18:37
*** leseb has quit IRC18:37
*** ijw has quit IRC18:38
*** WackoRobie has joined #openstack-neutron18:39
*** ihrachyshka has joined #openstack-neutron18:39
*** devvesa has quit IRC18:40
*** coolsvap has quit IRC18:42
*** thuc has quit IRC18:43
*** thuc has joined #openstack-neutron18:43
*** yamahata has joined #openstack-neutron18:44
*** thuc_ has joined #openstack-neutron18:46
*** thuc has quit IRC18:46
*** thuc_ has quit IRC18:46
*** thuc has joined #openstack-neutron18:47
*** csd_ has quit IRC18:48
*** thuc has joined #openstack-neutron18:48
*** thuc_ has joined #openstack-neutron18:49
*** ihrachyshka has quit IRC18:50
marunmestery: done18:52
mesterymarun: Thank you for your support and guidance! I certainly learned a lot here. :)18:52
*** thuc has quit IRC18:52
* mestery checks out marun's latest comments.18:53
*** thuc_ has quit IRC18:53
*** SumitNaiksatam has quit IRC18:53
*** thuc has joined #openstack-neutron18:54
*** overlayer has quit IRC18:54
*** csd_ has joined #openstack-neutron18:54
*** SumitNaiksatam has joined #openstack-neutron18:54
*** luqas has quit IRC18:56
*** jecarey has quit IRC18:57
*** gaud has quit IRC18:57
HenryGmlavalle: just want to confirm, is this truly abandoned? --> https://review.openstack.org/6436218:57
*** rwsu has quit IRC18:57
mlavalleHenryG: Yeah, it's abandoned. Emilien based his patchset on this onw. Emilien's is the one I pointed you to a little while ago18:58
HenryGmlavalle: got it. Thanks. I was just making sure.18:59
*** banix has quit IRC18:59
HenryGmlavalle: it's too easy for duplicated efforts to go unnoticed in tempest :(18:59
mlavalleHenryG: On the contrary, THANK YOU19:00
*** Sukhdev has joined #openstack-neutron19:04
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add functional tests to verify ovs_lib VXLAN detection  https://review.openstack.org/8460419:04
mesteryMarun: ^^^ Thanks for the comments!19:05
*** catohornet has quit IRC19:06
*** rwsu has joined #openstack-neutron19:13
*** baoli has quit IRC19:22
*** baoli has joined #openstack-neutron19:23
*** baoli has quit IRC19:23
*** baoli has joined #openstack-neutron19:23
*** leseb has joined #openstack-neutron19:24
*** pcm_ has quit IRC19:25
EmilienMmlavalle: hello19:25
mlavalleEmilienM: hi19:25
*** lipinski has joined #openstack-neutron19:26
lipinskiTrying to spin up a stack with heat and I'm getting a neutronclient problem19:26
lipinskiConnectionFailed: Connection to neutron failed: __init__() got an unexpected keyword argument 'set_nonblocking'19:26
openstackgerritA change was merged to openstack/neutron: Move bash whitelisting to pep8 testenv  https://review.openstack.org/8522319:26
lipinskiany ideas?19:26
*** leseb has quit IRC19:28
EmilienMmlavalle: my patch on FWaaS was taken by jroovers in the last Montreal sprint19:28
*** carl_baldwin has quit IRC19:28
mlavalleEmilenM: I know, but he doesn't have time anymore19:28
EmilienMmlavalle: i see19:28
mlavalleI checked with him19:29
EmilienMmlavalle: so if I understand, HenryG is taking it?19:29
mlavalleEmilienM: correct, HenryG is taking it19:29
EmilienMmlavalle: ack19:29
htrutaguys... do the lbaas/health monitor use ceilometer?19:30
mlavalleEmilenM: thanks for the follow up19:30
mlavalle:-)19:30
*** banix has joined #openstack-neutron19:30
EmilienM:)19:30
htrutawhere can i find information about them?19:30
*** carl_baldwin has joined #openstack-neutron19:30
HenryGEmilienM: do you want it back? :)19:31
EmilienMHenryG: if you don't have time, I'll take it but it's not on my schedule for now. So please :-)19:31
EmilienMis it me or it's hard to avoir overlap on tempest? so many patches in progress19:32
HenryGEmilienM: yes, I am taking it, don't worry. :)19:32
openstackgerritKyle Mestery proposed a change to openstack/neutron: Add functional tests to verify ovs_lib VXLAN detection  https://review.openstack.org/8460419:32
HenryGEmilienM: I have the same observation about overlap in tempest19:32
openstackgerritFawad Khaliq proposed a change to openstack/neutron: PLUMgrid CI Testing Commit; Please do not review  https://review.openstack.org/8545919:32
openstackgerritA change was merged to openstack/neutron: fixes broken neutron-netns-cleanup  https://review.openstack.org/8026119:33
EmilienMHenryG: now, I'm scared to make effort on tempest since I don't want to overlap19:33
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885319:33
*** chadix has joined #openstack-neutron19:33
enikanorov_kevinbenton: ping19:35
kevinbentonenikanorov_: pong19:37
kevinbentonenikanorov_: did you see my reply to your comment on https://review.openstack.org/#/c/84983/19:37
enikanorov_yes i did19:38
enikanorov_i still don't get why you need PBDB_MODULE var19:39
kevinbentonenikanorov_: to prevent an import every time the function is called19:40
enikanorov_ok, but how'r you preventing that if PBDB_MODULE is always None?19:40
kevinbentonenikanorov_: oh shoot! i see19:41
kevinbentonwhoops :-)19:41
openstackgerritLars Kellogg-Stedman proposed a change to openstack/neutron: allow delete_port to work when there are multiple floating ips  https://review.openstack.org/8543219:41
enikanorov_ook :)19:41
*** morganfainberg is now known as morganfainberg_Z19:42
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Don't import portbindings_db until use  https://review.openstack.org/8498319:43
kevinbentonenikanorov_: ^^ see if that looks better :-)19:43
*** networkstatic has quit IRC19:43
enikanorov_yeah19:44
EmilienMarosen: hello!  when you are around I would like to talk about notifiers/nova.py19:47
*** dkehn_ has joined #openstack-neutron19:50
openstackgerritA change was merged to openstack/neutron: Fix test MAC addresses to be valid  https://review.openstack.org/8521619:52
arosenEmilienM: what's up?19:53
*** amotoki has quit IRC19:54
*** dkehnx has quit IRC19:54
EmilienMarosen: I was wondering why you used nova_admin_tenant_id in the notifiers/nova.py for the admin auth against nova API, it's a pain for configurations management. I'm looking now at using admin tenant name and I'm reading python-novaclient. Maybe projectid could do the job19:55
arosenEmilienM: i agree it's a pain X.x....19:55
EmilienMarosen: I'm not a guru at all, but I think projectid is actually the tenant name19:56
EmilienMin novaclient19:56
arosenEmilienM:  the reason i did that because otherwise if i passed username/password to the nova-pythonclient it would query keystone each time to look up the tenant_id which is required to make the nova api call.19:56
arosenso each notification neutron sent would require hitting keystone (which would be done internally in the nova-pythonclient.19:57
EmilienMarmax: projectid is the name of the project, which would help guys like me when deploying OpenStack to give a tenant name19:57
*** sballe has quit IRC19:57
*** lipinski has left #openstack-neutron19:57
*** sballe has joined #openstack-neutron19:57
EmilienMarosen: ^19:58
EmilienMarosen: https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L47319:58
EmilienMarmax: sorry19:58
armaxEmilienM: that's okay19:59
armax:)19:59
openstackgerritFawad Khaliq proposed a change to openstack/neutron: PLUMgrid CI Testing Commit; Please do not review  https://review.openstack.org/8545919:59
arosenEmilienM: i agree it's a pain to have to loop up the ID.19:59
EmilienMarosen: so, I'm preparing a patch and I wanted to ensure I'm going to the right direction.20:00
arosenEmilienM: If we don't have the id then the python-novaclient will need to hit keystone each time to get it.20:00
arosenI'm not sure how we can avoid that.20:00
*** morganfainberg_Z is now known as morganfainberg20:00
*** tvardeman has joined #openstack-neutron20:00
tvardeman20:00
arosenthe tenant_id is needed as that's part of the URL that nova needs /v2/<tenant_id>/os-server-external-events/20:01
tvardemansorry....20:01
*** TrevorV has joined #openstack-neutron20:01
EmilienMarosen: huh ? what about sending projectid instead (btw it's not project_id like you wrote in notifiers/nova.py L48)20:01
EmilienMok20:01
*** ram5391 has quit IRC20:02
*** Sukhdev has quit IRC20:02
EmilienMarosen: I understand your point. But we need to find a way to deal with that, people can't set an ID in configuration files.20:02
arosenEmilienM: i'm talking about the id in keystone tenant-list; it's not a string like 'admin' unfortinately.20:02
arosenEmilienM:  I agree. I know that the tripleo people are feeling this pain too: https://review.openstack.org/#/c/81120/20:03
*** ijw has joined #openstack-neutron20:04
EmilienMarosen: i'm going to look at the reverse, how nova contact neutron api20:05
arosenEmilienM: sounds good. Keep me posted. I'd definitely love to make that easier for people!20:06
*** erecio has joined #openstack-neutron20:07
openstackgerritFawad Khaliq proposed a change to openstack/neutron: PLUMgrid CI Testing Commit; Please do not review  https://review.openstack.org/8545920:11
HenryGEmilienM: can you restore https://review.openstack.org/6574420:11
EmilienMHenryG: done20:11
HenryGEmilienM: thanks!20:12
EmilienMHenryG: thx for taking care if it20:13
*** erecio has quit IRC20:14
*** erecio has joined #openstack-neutron20:14
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Don't import portbindings_db until use  https://review.openstack.org/8498320:15
*** morganfainberg is now known as morganfainberg_Z20:15
EmilienMarmax: I filed a but in case of: https://bugs.launchpad.net/neutron/+bug/130281420:15
EmilienMarosen: ^20:15
*** erecio has quit IRC20:20
*** markmcclain has quit IRC20:20
*** erecio has joined #openstack-neutron20:21
*** TrevorV has quit IRC20:23
*** WackoRobie has quit IRC20:24
*** carl_baldwin has quit IRC20:24
*** leseb has joined #openstack-neutron20:24
*** csd_ has quit IRC20:26
*** Sukhdev has joined #openstack-neutron20:26
openstackgerritKevin Benton proposed a change to openstack/neutron: BigSwitch: Improves server manager UT coverage  https://review.openstack.org/7885320:28
*** TrevorV has joined #openstack-neutron20:29
*** leseb has quit IRC20:29
*** csd_ has joined #openstack-neutron20:32
openstackgerritSridhar Gaddam proposed a change to openstack/neutron: Security Group rule validation for ICMP rules  https://review.openstack.org/8502620:35
*** htruta has quit IRC20:37
*** julim has joined #openstack-neutron20:39
*** markmcclain has joined #openstack-neutron20:41
*** tvardeman has quit IRC20:44
*** TrevorV_ has joined #openstack-neutron20:44
*** TrevorV_ has quit IRC20:44
*** jobewan has quit IRC20:44
*** TrevorV_ has joined #openstack-neutron20:44
*** TrevorV_ has quit IRC20:45
*** TrevorV has quit IRC20:45
*** erecio has quit IRC20:54
*** WackoRobie has joined #openstack-neutron21:01
*** WackoRobie has quit IRC21:03
*** WackoRobie has joined #openstack-neutron21:13
openstackgerritA change was merged to openstack/neutron: Don't emit log for missing attribute check policy  https://review.openstack.org/8529921:15
kevinbentonarosen: ping21:24
*** csd_ has quit IRC21:25
*** leseb has joined #openstack-neutron21:25
kevinbentonarosen, armax, salv-orlando: looking for someone familiar with the vmware unit tests.21:26
*** leseb has quit IRC21:29
*** mwagner_lap has quit IRC21:38
openstackgerritEmilien Macchi proposed a change to openstack/neutron: nova notifications: use admin tenant name  https://review.openstack.org/8549221:38
openstackgerritEmilien Macchi proposed a change to openstack/neutron: nova notifications: use admin tenant name  https://review.openstack.org/8549221:43
*** WackoRobie has quit IRC21:47
*** WackoRobie has joined #openstack-neutron21:47
*** amotoki has joined #openstack-neutron21:49
EmilienMarosen: due ton Gerrit error, I can't add you as reviewer, so here is the first patch: https://review.openstack.org/#/c/8549221:51
*** WackoRobie has quit IRC21:51
*** amotoki has quit IRC21:53
*** csd_ has joined #openstack-neutron22:00
*** armax has left #openstack-neutron22:06
*** prad has quit IRC22:06
*** thuc has quit IRC22:08
*** thuc has joined #openstack-neutron22:08
*** thuc has quit IRC22:13
*** thedodd has quit IRC22:22
*** banix has quit IRC22:22
*** jgrimm has quit IRC22:24
*** csd_ is now known as csd22:26
*** leseb has joined #openstack-neutron22:26
*** Sukhdev has quit IRC22:26
*** manishg_ has joined #openstack-neutron22:27
*** manishg has quit IRC22:27
*** ijw has quit IRC22:30
*** leseb has quit IRC22:30
openstackgerritA change was merged to openstack/neutron: BigSwitch: Don't import portbindings_db until use  https://review.openstack.org/8498322:45
*** WackoRobie has joined #openstack-neutron22:53
openstackgerritEmilien Macchi proposed a change to openstack/neutron: nova notifications: use admin tenant name  https://review.openstack.org/8549223:07
*** iwi has quit IRC23:10
*** Sukhdev has joined #openstack-neutron23:11
*** leseb has joined #openstack-neutron23:27
*** manishg__ has joined #openstack-neutron23:27
*** leseb has quit IRC23:31
openstackgerritfumihiko kakuma proposed a change to openstack/neutron: OFAgent: Avoid re-wiring ports unnecessarily  https://review.openstack.org/8535623:31
*** manishg_ has quit IRC23:31
*** manishg__ has quit IRC23:32
*** Sukhdev has quit IRC23:36
*** Sukhdev has joined #openstack-neutron23:37
openstackgerritfumihiko kakuma proposed a change to openstack/neutron: OFAgent: Process port_update notifications in the main agent loop  https://review.openstack.org/8086423:40
*** thuc has joined #openstack-neutron23:42
*** amotoki has joined #openstack-neutron23:49
*** SumitNaiksatam has quit IRC23:50
openstackgerritEmilien Macchi proposed a change to openstack/neutron: nova notifications: use admin tenant name  https://review.openstack.org/8549223:51
*** amotoki has quit IRC23:54
*** WackoRobie has quit IRC23:58
*** WackoRobie has joined #openstack-neutron23:59

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