| *** WackoRobie has joined #openstack-neutron | 00:01 | |
| *** hrushig has quit IRC | 00:04 | |
| *** blogan has joined #openstack-neutron | 00:12 | |
| *** singhs has joined #openstack-neutron | 00:13 | |
| *** ajo has quit IRC | 00:13 | |
| *** SumitNaiksatam has quit IRC | 00:15 | |
| *** manishg has joined #openstack-neutron | 00:16 | |
| *** manishg_ has joined #openstack-neutron | 00:17 | |
| *** sbalukoff has quit IRC | 00:18 | |
| *** dave_tucker is now known as dave_tucker_zzz | 00:18 | |
| *** csd has quit IRC | 00:20 | |
| *** maru_afk is now known as marun | 00:20 | |
| *** networkstatic has quit IRC | 00:20 | |
| *** manishg has quit IRC | 00:21 | |
| *** manishg_ is now known as manishg | 00:21 | |
| *** yamahata has joined #openstack-neutron | 00:22 | |
| mestery | jogo: I'm still seeing state_path errors even after pulling in your change here: https://review.openstack.org/#/c/84568/ | 00:22 |
|---|---|---|
| *** mlavalle has quit IRC | 00:22 | |
| mestery | jogo: For reference ---> http://paste.openstack.org/show/74917/ | 00:23 |
| *** sbfox has joined #openstack-neutron | 00:25 | |
| *** ajo has joined #openstack-neutron | 00:25 | |
| *** gdubreui has quit IRC | 00:27 | |
| *** matsuhashi has joined #openstack-neutron | 00:28 | |
| *** thuc has quit IRC | 00:30 | |
| *** prad has joined #openstack-neutron | 00:30 | |
| *** csd has joined #openstack-neutron | 00:30 | |
| *** prad has quit IRC | 00:30 | |
| *** thuc has joined #openstack-neutron | 00:31 | |
| *** gaud has joined #openstack-neutron | 00:32 | |
| mestery | marun: See ^^^, still seeing issues with state_path even with jogo's patch: http://paste.openstack.org/show/74917/ | 00:32 |
| marun | mestery: gah | 00:34 |
| mestery | marun: I'm a #sadpanda for sure :( | 00:34 |
| marun | mestery: wait, there's another patch | 00:34 |
| mestery | marun: Looking | 00:34 |
| *** thuc has quit IRC | 00:35 | |
| marun | https://review.openstack.org/#/c/84832/ | 00:35 |
| marun | the tests pass, please approve | 00:35 |
| *** alagalah_ has quit IRC | 00:35 | |
| marun | mestery: ^ | 00:35 |
| mestery | marun: Done! Thanks, I had missed that one because I was only looking at merged patches :( | 00:35 |
| mestery | marun: Will wait for this to merge and then come back and hopefully push out my two functional test patches for your review. :) | 00:36 |
| marun | mestery: hah, he changed the topic. at first it was 'sorrymarun' | 00:36 |
| *** markmcclain has quit IRC | 00:36 | |
| mestery | marun: :P | 00:36 |
| marun | mestery: great! I'll stick around to review. | 00:37 |
| mestery | marun: Ok, cool! They all pass pep8 and fail on state_path, so I'm hopeful once this merges it'll be a quick rebase and "git review". :) | 00:37 |
| *** banix has joined #openstack-neutron | 00:38 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow tenant to set router external IP https://review.openstack.org/83664 | 00:39 |
| *** thuc has joined #openstack-neutron | 00:39 | |
| *** thuc_ has joined #openstack-neutron | 00:40 | |
| *** thuc_ has quit IRC | 00:41 | |
| *** manishg has quit IRC | 00:41 | |
| *** thuc_ has joined #openstack-neutron | 00:41 | |
| *** Adri2000 has quit IRC | 00:41 | |
| *** armitage81 has quit IRC | 00:42 | |
| *** thuc__ has joined #openstack-neutron | 00:43 | |
| *** rand738 has quit IRC | 00:43 | |
| *** thuc__ has quit IRC | 00:43 | |
| *** thuc has quit IRC | 00:43 | |
| *** thuc has joined #openstack-neutron | 00:44 | |
| *** thuc__ has joined #openstack-neutron | 00:44 | |
| *** thuc__ has quit IRC | 00:45 | |
| *** thuc__ has joined #openstack-neutron | 00:45 | |
| *** rand738 has joined #openstack-neutron | 00:45 | |
| *** thuc_ has quit IRC | 00:46 | |
| *** thuc__ has quit IRC | 00:46 | |
| *** thuc_ has joined #openstack-neutron | 00:47 | |
| *** thuc has quit IRC | 00:48 | |
| *** gdubreui has joined #openstack-neutron | 00:49 | |
| *** ajo has quit IRC | 00:54 | |
| *** julim has quit IRC | 00:55 | |
| *** julim has joined #openstack-neutron | 00:56 | |
| *** sbfox has quit IRC | 00:56 | |
| *** dvorkinista has quit IRC | 00:56 | |
| *** thuc_ has quit IRC | 00:57 | |
| *** csd has quit IRC | 00:59 | |
| *** julim has quit IRC | 01:00 | |
| *** sbfox has joined #openstack-neutron | 01:01 | |
| *** harlowja is now known as harlowja_away | 01:01 | |
| *** arosen1 has joined #openstack-neutron | 01:04 | |
| openstackgerrit | A change was merged to openstack/neutron: Remove last parts of Quantum compatibility shim https://review.openstack.org/84367 | 01:04 |
| *** matsuhashi has quit IRC | 01:05 | |
| *** blogan has quit IRC | 01:06 | |
| *** matsuhashi has joined #openstack-neutron | 01:06 | |
| *** julim has joined #openstack-neutron | 01:07 | |
| *** sbfox has quit IRC | 01:07 | |
| *** yamahata has quit IRC | 01:11 | |
| jogo | mestery: there is s second patch | 01:11 |
| jogo | mestery: https://review.openstack.org/#/c/84832/ | 01:11 |
| openstackgerrit | oda-g proposed a change to openstack/neutron: Fix to work multiple RPC workers under Metaplugin https://review.openstack.org/84914 | 01:11 |
| *** julim has quit IRC | 01:12 | |
| *** adam_g has joined #openstack-neutron | 01:13 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/python-neutronclient: Add floating-ip-address to floatingip-create https://review.openstack.org/84915 | 01:13 |
| mestery | jogo: marun pointed me at that and I +2/+A it as wlel. | 01:15 |
| *** ajo has joined #openstack-neutron | 01:17 | |
| openstackgerrit | Yuuichi Fujioka proposed a change to openstack/python-neutronclient: Add floating-ip-address to floatingip-create https://review.openstack.org/84915 | 01:19 |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow tenant to set router external IP https://review.openstack.org/83664 | 01:19 |
| *** gaud has quit IRC | 01:21 | |
| openstackgerrit | Sourabh Patwardhan proposed a change to openstack/neutron: Remove List events API from Cisco N1kv Neutron https://review.openstack.org/80411 | 01:25 |
| *** _cjones_ has quit IRC | 01:28 | |
| *** _cjones_ has joined #openstack-neutron | 01:29 | |
| *** _cjones_ has quit IRC | 01:33 | |
| openstackgerrit | A change was merged to openstack/neutron: Explicitly import state_path opt in tests.base https://review.openstack.org/84832 | 01:35 |
| *** ajo has quit IRC | 01:36 | |
| *** xuhanp has joined #openstack-neutron | 01:36 | |
| *** julim has joined #openstack-neutron | 01:37 | |
| *** ajo has joined #openstack-neutron | 01:38 | |
| *** Adri2000 has joined #openstack-neutron | 01:42 | |
| *** alagalah has joined #openstack-neutron | 01:45 | |
| *** alagalah has quit IRC | 01:50 | |
| *** ajo has quit IRC | 01:51 | |
| *** dvorkinista has joined #openstack-neutron | 01:54 | |
| *** dvorkinista has quit IRC | 01:54 | |
| *** dvorkinista has joined #openstack-neutron | 01:55 | |
| *** xianghui has joined #openstack-neutron | 02:02 | |
| *** ajo has joined #openstack-neutron | 02:03 | |
| *** dvorkinista has quit IRC | 02:06 | |
| *** dvorkinista has joined #openstack-neutron | 02:06 | |
| *** thuc has joined #openstack-neutron | 02:08 | |
| *** HenryG has quit IRC | 02:08 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests https://review.openstack.org/84923 | 02:09 |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add functional tests to check host VXLAN support https://review.openstack.org/84604 | 02:09 |
| mestery | marun: See the two reviews here for your viewing pleasure: https://review.openstack.org/84923 https://review.openstack.org/84604 | 02:10 |
| mestery | marun: functional tests #ftw!!! | 02:10 |
| *** dvorkinista has quit IRC | 02:11 | |
| *** thuc has quit IRC | 02:13 | |
| *** ajo has quit IRC | 02:13 | |
| openstackgerrit | Abhishek Raut proposed a change to openstack/neutron: Segregate the VSM calls from database calls in N1kv plugin https://review.openstack.org/82266 | 02:17 |
| *** sandr8 has quit IRC | 02:18 | |
| *** otherwiseguy has joined #openstack-neutron | 02:23 | |
| *** oda-g has quit IRC | 02:24 | |
| *** _cjones_ has joined #openstack-neutron | 02:24 | |
| *** ajo has joined #openstack-neutron | 02:32 | |
| *** _cjones_ has quit IRC | 02:32 | |
| *** ajo has quit IRC | 02:36 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow tenant to set router external IP https://review.openstack.org/83664 | 02:40 |
| *** julim has quit IRC | 02:50 | |
| *** baoli_ has joined #openstack-neutron | 02:51 | |
| *** thuc has joined #openstack-neutron | 02:52 | |
| *** thuc_ has joined #openstack-neutron | 02:53 | |
| *** baoli has quit IRC | 02:53 | |
| *** oda-g has joined #openstack-neutron | 02:53 | |
| *** thuc has quit IRC | 02:57 | |
| *** matsuhashi has quit IRC | 03:05 | |
| *** thuc_ has quit IRC | 03:08 | |
| *** thuc has joined #openstack-neutron | 03:08 | |
| *** thuc has quit IRC | 03:11 | |
| *** rand738 has quit IRC | 03:12 | |
| *** angryjesters has joined #openstack-neutron | 03:16 | |
| *** rand738 has joined #openstack-neutron | 03:17 | |
| *** armax has quit IRC | 03:22 | |
| *** otherwiseguy has quit IRC | 03:28 | |
| *** coolsvap has joined #openstack-neutron | 03:32 | |
| *** thuc has joined #openstack-neutron | 03:39 | |
| *** networkstatic has joined #openstack-neutron | 03:41 | |
| *** jecarey has joined #openstack-neutron | 03:41 | |
| *** bran3993 has joined #openstack-neutron | 03:48 | |
| *** blogan has joined #openstack-neutron | 03:49 | |
| *** _cjones_ has joined #openstack-neutron | 03:57 | |
| *** harlowja_away is now known as harlowja | 04:00 | |
| *** WackoRobie has quit IRC | 04:01 | |
| *** SumitNaiksatam has joined #openstack-neutron | 04:02 | |
| *** WackoRobie has joined #openstack-neutron | 04:02 | |
| *** _cjones_ has quit IRC | 04:02 | |
| *** lollipop has joined #openstack-neutron | 04:02 | |
| *** thuc has quit IRC | 04:03 | |
| *** thuc has joined #openstack-neutron | 04:04 | |
| *** WackoRobie has quit IRC | 04:06 | |
| *** thuc has quit IRC | 04:07 | |
| *** ramishra has joined #openstack-neutron | 04:09 | |
| *** ramishra_ has joined #openstack-neutron | 04:09 | |
| *** thuc has joined #openstack-neutron | 04:11 | |
| *** ramishra has quit IRC | 04:13 | |
| *** matsuhashi has joined #openstack-neutron | 04:15 | |
| *** dvorkinista has joined #openstack-neutron | 04:19 | |
| *** rand738 has quit IRC | 04:20 | |
| *** hrushig has joined #openstack-neutron | 04:21 | |
| *** rand738 has joined #openstack-neutron | 04:22 | |
| *** yfried__ has quit IRC | 04:24 | |
| *** ramishra_ has quit IRC | 04:29 | |
| *** dvorkinista has quit IRC | 04:33 | |
| *** blogan has joined #openstack-neutron | 04:34 | |
| *** ramishra_ has joined #openstack-neutron | 04:34 | |
| *** banix has quit IRC | 04:38 | |
| *** dave_tucker_zzz is now known as dave_tucker | 04:39 | |
| *** matsuhashi has quit IRC | 04:40 | |
| *** matsuhashi has joined #openstack-neutron | 04:46 | |
| *** hrushig_ has joined #openstack-neutron | 04:46 | |
| *** hrushig has quit IRC | 04:47 | |
| *** ramishra_ has quit IRC | 04:49 | |
| *** ramishra has joined #openstack-neutron | 04:51 | |
| openstackgerrit | Ian Wienand proposed a change to openstack/neutron: Record and log reason for dhcp agent resync https://review.openstack.org/81173 | 04:51 |
| *** sbalukoff has joined #openstack-neutron | 04:51 | |
| *** skraynev_afk is now known as skraynev | 04:52 | |
| *** Swami has joined #openstack-neutron | 04:56 | |
| *** sridhar has joined #openstack-neutron | 04:58 | |
| *** thuc has quit IRC | 05:01 | |
| *** yamahata has joined #openstack-neutron | 05:02 | |
| *** thuc has joined #openstack-neutron | 05:02 | |
| *** thuc has quit IRC | 05:06 | |
| *** scotty916 has joined #openstack-neutron | 05:08 | |
| scotty916 | \help | 05:11 |
| *** yamahata has quit IRC | 05:12 | |
| *** yamahata has joined #openstack-neutron | 05:12 | |
| *** pradipta_away is now known as pradipta | 05:27 | |
| *** catohornet has quit IRC | 05:30 | |
| *** irenab has joined #openstack-neutron | 05:30 | |
| *** harlowja is now known as harlowja_away | 05:32 | |
| *** bvandenh has joined #openstack-neutron | 05:33 | |
| *** scotty916 has quit IRC | 05:34 | |
| *** dave_tucker is now known as dave_tucker_zzz | 05:35 | |
| *** yfried__ has joined #openstack-neutron | 05:36 | |
| *** arosen1 has quit IRC | 05:42 | |
| *** rand738 has quit IRC | 05:46 | |
| *** rand738 has joined #openstack-neutron | 05:46 | |
| *** hrushig_ has quit IRC | 05:50 | |
| *** hrushig_ has joined #openstack-neutron | 05:52 | |
| *** Longgeek has joined #openstack-neutron | 05:53 | |
| *** jprovazn has joined #openstack-neutron | 05:53 | |
| *** rand738 has quit IRC | 05:55 | |
| *** rand738 has joined #openstack-neutron | 05:56 | |
| *** jlibosva has joined #openstack-neutron | 06:04 | |
| *** Chinorro has joined #openstack-neutron | 06:05 | |
| *** Longgeek has quit IRC | 06:09 | |
| *** arosen1 has joined #openstack-neutron | 06:09 | |
| *** thuc has joined #openstack-neutron | 06:13 | |
| *** Longgeek has joined #openstack-neutron | 06:15 | |
| *** thuc has quit IRC | 06:17 | |
| *** evgenyf has joined #openstack-neutron | 06:21 | |
| *** singhs has quit IRC | 06:22 | |
| *** rand738 has quit IRC | 06:31 | |
| openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/84635 | 06:32 |
| *** rand738 has joined #openstack-neutron | 06:33 | |
| *** evgenyf has quit IRC | 06:33 | |
| *** matsuhashi has quit IRC | 06:36 | |
| *** matsuhashi has joined #openstack-neutron | 06:39 | |
| *** garyk has joined #openstack-neutron | 06:39 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Unique_constraint names should be equal for all dialects https://review.openstack.org/55411 | 06:41 |
| *** gdubreui has quit IRC | 06:49 | |
| *** jpich has joined #openstack-neutron | 06:52 | |
| *** Swami has quit IRC | 06:55 | |
| *** afazekas has joined #openstack-neutron | 06:56 | |
| tchaypo | marun: <3 | 06:58 |
| marun | tchaypo: ?: | 06:58 |
| *** luqas has joined #openstack-neutron | 06:58 | |
| tchaypo | Bug 1290486 :) | 06:58 |
| openstackgerrit | oda-g proposed a change to openstack/neutron: Fix to work multiple RPC workers under Metaplugin https://review.openstack.org/84914 | 06:59 |
| marun | tchaypo: ah, ok :) | 06:59 |
| *** Longgeek has quit IRC | 07:00 | |
| *** luqas has quit IRC | 07:00 | |
| *** Longgeek has joined #openstack-neutron | 07:01 | |
| *** luqas has joined #openstack-neutron | 07:01 | |
| *** sridhar has quit IRC | 07:03 | |
| *** luqas has quit IRC | 07:03 | |
| openstackgerrit | Xu Han Peng proposed a change to openstack/python-neutronclient: Create new IPv6 attributes for Subnets by client https://review.openstack.org/75871 | 07:04 |
| *** thurloat has quit IRC | 07:04 | |
| *** zigo has quit IRC | 07:04 | |
| openstackgerrit | oda-g proposed a change to openstack/neutron: Fix to work multiple RPC workers under Metaplugin https://review.openstack.org/84914 | 07:04 |
| *** sdague has quit IRC | 07:05 | |
| *** sdague has joined #openstack-neutron | 07:05 | |
| *** markwash_ has joined #openstack-neutron | 07:07 | |
| *** zigo has joined #openstack-neutron | 07:08 | |
| *** markwash has quit IRC | 07:08 | |
| *** markwash_ is now known as markwash | 07:08 | |
| *** thurloat has joined #openstack-neutron | 07:11 | |
| *** rotbeard has joined #openstack-neutron | 07:12 | |
| *** stefanb has quit IRC | 07:12 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Fix difference in index creation for foreign key in MySQL and Postgres https://review.openstack.org/80518 | 07:16 |
| *** stefanb has joined #openstack-neutron | 07:17 | |
| *** evgenyf has joined #openstack-neutron | 07:19 | |
| *** sridhar has joined #openstack-neutron | 07:20 | |
| *** dave_tucker_zzz is now known as dave_tucker | 07:21 | |
| *** jgallard has joined #openstack-neutron | 07:22 | |
| *** gangur__ has joined #openstack-neutron | 07:23 | |
| *** hrushig_ has quit IRC | 07:23 | |
| openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Allow LLA as router interface of IPv6 subnet https://review.openstack.org/76125 | 07:23 |
| *** Longgeek has quit IRC | 07:25 | |
| *** ramishra has quit IRC | 07:26 | |
| *** marun is now known as marun_afk | 07:29 | |
| openstackgerrit | A change was merged to openstack/neutron: Use os.uname() instead of calling uname in subprocess https://review.openstack.org/84462 | 07:31 |
| *** roeyc has joined #openstack-neutron | 07:35 | |
| openstackgerrit | A change was merged to openstack/neutron: Changed the message line of RouterInUse class https://review.openstack.org/83374 | 07:35 |
| *** Longgeek has joined #openstack-neutron | 07:37 | |
| *** ajo_ has joined #openstack-neutron | 07:41 | |
| *** ygbo has joined #openstack-neutron | 07:43 | |
| *** luqas has joined #openstack-neutron | 07:45 | |
| *** yfujioka has joined #openstack-neutron | 07:47 | |
| *** luqas has quit IRC | 07:47 | |
| *** ramishra has joined #openstack-neutron | 07:48 | |
| *** Longgeek has quit IRC | 07:50 | |
| *** luqas has joined #openstack-neutron | 07:51 | |
| *** safchain has joined #openstack-neutron | 07:53 | |
| *** rotbeard has quit IRC | 07:56 | |
| *** blueyes has joined #openstack-neutron | 07:58 | |
| blueyes | Hi. I want to configure transparent proxy in the neutron - each request which is comming from Nova instance will need to go through this proxy. Can you please help me to create such configuration? | 07:58 |
| *** sbfox has joined #openstack-neutron | 07:58 | |
| *** jistr has joined #openstack-neutron | 08:01 | |
| *** leseb has joined #openstack-neutron | 08:04 | |
| *** blueyes has quit IRC | 08:05 | |
| *** gangur__ has quit IRC | 08:06 | |
| *** hrushig has joined #openstack-neutron | 08:06 | |
| *** bashok has joined #openstack-neutron | 08:07 | |
| *** rotbeard has joined #openstack-neutron | 08:11 | |
| *** lollipop has quit IRC | 08:11 | |
| *** Longgeek has joined #openstack-neutron | 08:12 | |
| *** Longgeek has quit IRC | 08:14 | |
| *** francois_eleouet has quit IRC | 08:14 | |
| *** Longgeek has joined #openstack-neutron | 08:14 | |
| openstackgerrit | A change was merged to openstack/neutron: Remove invalid copyright headers under API module https://review.openstack.org/84825 | 08:15 |
| *** [1]evgenyf has joined #openstack-neutron | 08:17 | |
| *** feleouet has joined #openstack-neutron | 08:18 | |
| *** evgenyf has quit IRC | 08:19 | |
| *** [1]evgenyf is now known as evgenyf | 08:19 | |
| *** sbfox has quit IRC | 08:21 | |
| *** djoreilly has joined #openstack-neutron | 08:21 | |
| *** arosen1 has quit IRC | 08:22 | |
| *** amuller has joined #openstack-neutron | 08:29 | |
| *** rotbeard has quit IRC | 08:37 | |
| *** rotbeard has joined #openstack-neutron | 08:38 | |
| *** devvesa has joined #openstack-neutron | 08:42 | |
| *** jp_at_hp has joined #openstack-neutron | 08:46 | |
| *** roeyc has quit IRC | 08:48 | |
| *** Longgeek has quit IRC | 08:54 | |
| *** _cjones_ has joined #openstack-neutron | 09:01 | |
| *** dave_tucker is now known as dave_tucker_zzz | 09:02 | |
| *** dave_tucker_zzz is now known as dave_tucker | 09:05 | |
| *** _cjones_ has quit IRC | 09:05 | |
| *** matsuhashi has quit IRC | 09:07 | |
| *** matsuhashi has joined #openstack-neutron | 09:12 | |
| *** ramishra has quit IRC | 09:13 | |
| *** morganfainberg is now known as morganfainberg_Z | 09:15 | |
| *** luqas has quit IRC | 09:23 | |
| *** Longgeek_ has joined #openstack-neutron | 09:24 | |
| *** hrushig_ has joined #openstack-neutron | 09:25 | |
| *** hrushig has quit IRC | 09:28 | |
| *** xuhanp has quit IRC | 09:29 | |
| openstackgerrit | Édouard Thuleau proposed a change to openstack/neutron: OVS lib defer apply doesn't handle concurrency https://review.openstack.org/63917 | 09:31 |
| *** coolsvap has quit IRC | 09:35 | |
| *** veena_ has joined #openstack-neutron | 09:35 | |
| veena_ | Hi | 09:36 |
| veena_ | How to measure the number of packets transmitted by a VM to another VM? | 09:36 |
| openstackgerrit | Édouard Thuleau proposed a change to openstack/neutron: OVS lib defer apply doesn't handle concurrency https://review.openstack.org/63917 | 09:36 |
| *** ramishra has joined #openstack-neutron | 09:37 | |
| *** veena_ has quit IRC | 09:45 | |
| *** luqas has joined #openstack-neutron | 09:45 | |
| *** garyk1 has joined #openstack-neutron | 09:47 | |
| *** garyk has quit IRC | 09:47 | |
| *** garyk1 has quit IRC | 09:47 | |
| *** leseb has quit IRC | 09:51 | |
| *** leseb has joined #openstack-neutron | 09:51 | |
| openstackgerrit | A change was merged to openstack/neutron: Restore NOT NULL constraint lost by earlier migrations https://review.openstack.org/78591 | 09:52 |
| *** yfujioka has quit IRC | 09:53 | |
| *** Longgeek has joined #openstack-neutron | 09:54 | |
| *** leseb has quit IRC | 09:55 | |
| *** Longgeek_ has quit IRC | 09:56 | |
| *** sphoorti has joined #openstack-neutron | 09:58 | |
| *** matsuhashi has quit IRC | 09:59 | |
| *** matsuhashi has joined #openstack-neutron | 09:59 | |
| *** leseb has joined #openstack-neutron | 10:00 | |
| *** _cjones_ has joined #openstack-neutron | 10:02 | |
| *** sphoorti has quit IRC | 10:02 | |
| *** Longgeek has quit IRC | 10:05 | |
| *** pasquier-s has joined #openstack-neutron | 10:06 | |
| *** _cjones_ has quit IRC | 10:06 | |
| *** rotbeard has quit IRC | 10:08 | |
| *** Longgeek has joined #openstack-neutron | 10:09 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Set bridge UP in device_exists if force_up is True https://review.openstack.org/76309 | 10:13 |
| *** rotbeard has joined #openstack-neutron | 10:13 | |
| *** Longgeek has quit IRC | 10:14 | |
| *** Longgeek has joined #openstack-neutron | 10:15 | |
| *** ramishra has quit IRC | 10:17 | |
| *** alexpilotti has joined #openstack-neutron | 10:17 | |
| *** ramishra has joined #openstack-neutron | 10:17 | |
| *** chandankumar_ has joined #openstack-neutron | 10:22 | |
| *** coolsvap has joined #openstack-neutron | 10:25 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: BigSwitch: Don't import portbindings_db until use https://review.openstack.org/84983 | 10:28 |
| *** leseb has quit IRC | 10:29 | |
| *** leseb has joined #openstack-neutron | 10:29 | |
| kevinbenton | amotoki: ping | 10:29 |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Fix incorrect change of Enum type https://review.openstack.org/84730 | 10:31 |
| *** rotbeard has quit IRC | 10:33 | |
| *** leseb has quit IRC | 10:33 | |
| lifeless | arosen: around ? | 10:34 |
| lifeless | arosen: I'm fairly sure https://review.openstack.org/#/q/Ia9e71364bca6cbc24ebc1c234e6a5af14f51cd62,n,z is still broken | 10:34 |
| lifeless | arosen: we've got it deployed in the ci-overcloud where we boot with net-id=xxx for two nics and they are sometimes (but not always) out of order) | 10:35 |
| *** pcm_ has joined #openstack-neutron | 10:35 | |
| *** pcm_ has quit IRC | 10:36 | |
| *** jgallard has quit IRC | 10:37 | |
| *** pcm_ has joined #openstack-neutron | 10:37 | |
| *** pcm_ has quit IRC | 10:38 | |
| *** coolsvap has quit IRC | 10:38 | |
| *** ramishra has quit IRC | 10:41 | |
| *** ramishra has joined #openstack-neutron | 10:42 | |
| *** Longgeek_ has joined #openstack-neutron | 10:43 | |
| *** Longgeek has quit IRC | 10:47 | |
| *** Longgeek has joined #openstack-neutron | 10:47 | |
| *** matsuhashi has quit IRC | 10:48 | |
| *** blueyes has joined #openstack-neutron | 10:50 | |
| *** Longgeek_ has quit IRC | 10:50 | |
| *** obondarev1 has joined #openstack-neutron | 10:50 | |
| *** obondarev has quit IRC | 10:50 | |
| *** rotbeard has joined #openstack-neutron | 10:50 | |
| *** roeyc has joined #openstack-neutron | 10:52 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Common decorator for caching methods https://review.openstack.org/72680 | 10:55 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Metadata agent caches networks for routers https://review.openstack.org/70689 | 10:55 |
| *** matsuhashi has joined #openstack-neutron | 10:55 | |
| *** blueyes has quit IRC | 10:59 | |
| *** leseb has joined #openstack-neutron | 11:04 | |
| *** WackoRobie has joined #openstack-neutron | 11:08 | |
| *** afazekas has quit IRC | 11:09 | |
| *** leseb has quit IRC | 11:09 | |
| *** djoreilly has quit IRC | 11:10 | |
| *** matsuhashi has quit IRC | 11:15 | |
| *** matsuhashi has joined #openstack-neutron | 11:15 | |
| *** leseb has joined #openstack-neutron | 11:18 | |
| *** matsuhashi has quit IRC | 11:19 | |
| *** matsuhashi has joined #openstack-neutron | 11:21 | |
| *** baoli_ has quit IRC | 11:22 | |
| *** afazekas has joined #openstack-neutron | 11:24 | |
| *** jgallard has joined #openstack-neutron | 11:35 | |
| *** luqas has quit IRC | 11:36 | |
| *** jgallard has quit IRC | 11:36 | |
| *** jgallard has joined #openstack-neutron | 11:36 | |
| *** pradipta is now known as pradipta_away | 11:40 | |
| *** pcm_ has joined #openstack-neutron | 11:40 | |
| *** HenryG has joined #openstack-neutron | 11:47 | |
| *** xianghui has quit IRC | 11:47 | |
| *** itzikb1 has joined #openstack-neutron | 11:48 | |
| *** itzikb has quit IRC | 11:51 | |
| *** matsuhashi has quit IRC | 11:52 | |
| *** itzikb has joined #openstack-neutron | 11:53 | |
| *** itzikb1 has quit IRC | 11:55 | |
| *** WackoRobie has quit IRC | 11:55 | |
| *** itzikb has quit IRC | 11:55 | |
| *** WackoRobie has joined #openstack-neutron | 11:56 | |
| *** itzikb has joined #openstack-neutron | 11:56 | |
| *** b3nt_pin has joined #openstack-neutron | 11:57 | |
| *** bjornar has joined #openstack-neutron | 12:00 | |
| *** WackoRobie has quit IRC | 12:00 | |
| *** _cjones_ has joined #openstack-neutron | 12:03 | |
| *** gaud has joined #openstack-neutron | 12:04 | |
| *** xuhanp has joined #openstack-neutron | 12:05 | |
| *** _cjones_ has quit IRC | 12:08 | |
| *** yamahata has quit IRC | 12:10 | |
| *** luqas has joined #openstack-neutron | 12:29 | |
| *** sballe has joined #openstack-neutron | 12:31 | |
| *** gangur__ has joined #openstack-neutron | 12:32 | |
| *** sbalukoff1 has joined #openstack-neutron | 12:33 | |
| *** hrushig_ has quit IRC | 12:34 | |
| *** sbalukoff has quit IRC | 12:35 | |
| *** irenab has quit IRC | 12:40 | |
| *** lollipop has joined #openstack-neutron | 12:46 | |
| *** ramishra has quit IRC | 12:50 | |
| *** ramishra has joined #openstack-neutron | 12:53 | |
| *** thuc has joined #openstack-neutron | 12:54 | |
| *** thuc_ has joined #openstack-neutron | 12:54 | |
| *** thuc has quit IRC | 12:58 | |
| *** djoreilly has joined #openstack-neutron | 12:58 | |
| *** xianghui has joined #openstack-neutron | 13:01 | |
| *** mancdaz has quit IRC | 13:01 | |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/neutron: Security Group rule validation for ICMP when ICMP code alone is provided https://review.openstack.org/85026 | 13:01 |
| *** mancdaz has joined #openstack-neutron | 13:01 | |
| *** tvardeman has joined #openstack-neutron | 13:02 | |
| *** sridhar is now known as SridharG | 13:03 | |
| *** samuelbercovici has joined #openstack-neutron | 13:05 | |
| *** dims_ has quit IRC | 13:07 | |
| *** ajo_ is now known as ajo | 13:10 | |
| *** changbl has quit IRC | 13:11 | |
| *** julim has joined #openstack-neutron | 13:15 | |
| *** baoli has joined #openstack-neutron | 13:17 | |
| *** samuelbercovici1 has joined #openstack-neutron | 13:18 | |
| *** samuelbercovici1 has quit IRC | 13:18 | |
| *** samuelbercovici1 has joined #openstack-neutron | 13:19 | |
| *** chandankumar_ has quit IRC | 13:19 | |
| *** skraynev is now known as skraynev_afk | 13:19 | |
| *** dims_ has joined #openstack-neutron | 13:20 | |
| *** samuelbercovici has quit IRC | 13:20 | |
| *** samuelbercovici1 is now known as samuelbercovici | 13:20 | |
| *** itzikb has quit IRC | 13:21 | |
| *** itzikb has joined #openstack-neutron | 13:22 | |
| *** Longgeek has quit IRC | 13:25 | |
| *** jecarey has quit IRC | 13:30 | |
| *** nplanel_ has joined #openstack-neutron | 13:32 | |
| *** WackoRobie has joined #openstack-neutron | 13:34 | |
| *** ramishra has quit IRC | 13:35 | |
| *** banix has joined #openstack-neutron | 13:35 | |
| enikanorov_ | samuelbercovici: ping | 13:35 |
| *** BillTheKat has joined #openstack-neutron | 13:36 | |
| *** BillTheKat is now known as Gil_McGrath | 13:36 | |
| *** thuc_ has quit IRC | 13:40 | |
| mestery | Neutron folks, looks like some recent commit is causing all kinds of fun flake8 errors: http://paste.openstack.org/show/74968/ | 13:41 |
| *** thuc has joined #openstack-neutron | 13:41 | |
| mestery | I've cleaned out .tox/pep8 and still seeing these, this is with the latest master. | 13:41 |
| *** samuelbercovici1 has joined #openstack-neutron | 13:43 | |
| *** thuc has quit IRC | 13:45 | |
| *** jgrimm has quit IRC | 13:45 | |
| *** armax has joined #openstack-neutron | 13:46 | |
| *** samuelbercovici has quit IRC | 13:47 | |
| *** samuelbercovici1 is now known as samuelbercovici | 13:47 | |
| *** networkstatic is now known as networkstatic_zZ | 13:50 | |
| *** crc32 has joined #openstack-neutron | 13:51 | |
| sdague | mestery: build/ should be ignored | 13:52 |
| mestery | sdague: So does this mean we need to add that to some tox config file somewhere? Apologies for my ignorance in this area. :) | 13:52 |
| sdague | yeh, one sec | 13:52 |
| mestery | thanks! | 13:53 |
| openstackgerrit | Sean Dague proposed a change to openstack/neutron: ignore build directory for pep8 https://review.openstack.org/85034 | 13:54 |
| sdague | there are 2 later on that this won't address | 13:55 |
| *** otherwiseguy has joined #openstack-neutron | 13:55 | |
| sdague | but that will get rid of most of your output | 13:55 |
| mestery | sdague: Thanks! | 13:56 |
| mestery | sdague: +2 from me | 13:56 |
| openstackgerrit | Sean Dague proposed a change to openstack/neutron: ignore build directory for pep8 https://review.openstack.org/85034 | 13:56 |
| *** blogan has joined #openstack-neutron | 13:56 | |
| sdague | cool. Messed up email on that the first time. So rev2 should be right | 13:56 |
| *** xianghui has quit IRC | 13:57 | |
| *** jorgem has joined #openstack-neutron | 13:59 | |
| *** obondarev1 is now known as obondarev | 13:59 | |
| mestery | Looks right to me, thanks for the quick patch! | 13:59 |
| *** sbalukoff1 is now known as sbalukoff | 14:00 | |
| *** mwagner_lap has joined #openstack-neutron | 14:01 | |
| *** jecarey has joined #openstack-neutron | 14:01 | |
| *** networkstatic_zZ has quit IRC | 14:04 | |
| *** _cjones_ has joined #openstack-neutron | 14:05 | |
| *** rm_work|away is now known as rm_work | 14:06 | |
| *** thuc has joined #openstack-neutron | 14:06 | |
| *** aveiga has joined #openstack-neutron | 14:06 | |
| *** _cjones_ has quit IRC | 14:09 | |
| *** bjornar has quit IRC | 14:16 | |
| *** bjornar has joined #openstack-neutron | 14:17 | |
| *** thedodd has joined #openstack-neutron | 14:18 | |
| *** Longgeek has joined #openstack-neutron | 14:23 | |
| *** thuc_ has joined #openstack-neutron | 14:24 | |
| *** markmcclain has joined #openstack-neutron | 14:24 | |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/neutron: Security Group rule validation for ICMP rules https://review.openstack.org/85026 | 14:24 |
| *** jgrimm has joined #openstack-neutron | 14:24 | |
| *** blogan_ has joined #openstack-neutron | 14:26 | |
| *** chandan_kumar has quit IRC | 14:27 | |
| *** blogan has quit IRC | 14:27 | |
| *** chandan_kumar has joined #openstack-neutron | 14:27 | |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Replace HTTPSConnection in NEC plugin https://review.openstack.org/77414 | 14:27 |
| *** thuc has quit IRC | 14:28 | |
| *** thuc_ has quit IRC | 14:28 | |
| *** tomoe_ has joined #openstack-neutron | 14:28 | |
| *** chandan_kumar has quit IRC | 14:30 | |
| *** afazekas has quit IRC | 14:30 | |
| *** rand738 has quit IRC | 14:32 | |
| *** rand738 has joined #openstack-neutron | 14:33 | |
| *** tomoe_ has quit IRC | 14:37 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Set ns_name in RouterInfo as attribute https://review.openstack.org/85051 | 14:38 |
| *** markwash has quit IRC | 14:39 | |
| *** carl_baldwin has joined #openstack-neutron | 14:43 | |
| *** Longgeek has quit IRC | 14:44 | |
| *** g3rms has quit IRC | 14:49 | |
| *** roeyc has quit IRC | 14:49 | |
| *** g3rms has joined #openstack-neutron | 14:49 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Set correct columns' length https://review.openstack.org/80539 | 14:50 |
| *** Longgeek has joined #openstack-neutron | 14:55 | |
| *** baoli has quit IRC | 14:57 | |
| *** thuc has joined #openstack-neutron | 14:58 | |
| *** thuc_ has joined #openstack-neutron | 14:58 | |
| *** baoli has joined #openstack-neutron | 14:58 | |
| *** bjornar has quit IRC | 15:01 | |
| *** blogan_ has quit IRC | 15:01 | |
| *** thuc has quit IRC | 15:02 | |
| *** nplanel_ has quit IRC | 15:03 | |
| enikanorov_ | sballe: hi | 15:05 |
| sballe | enikanorov_, hi great meeting :-) | 15:06 |
| sballe | we are following up to get you the % you need for the spreadsheet | 15:06 |
| enikanorov_ | yeah, is there something left that you might want to discuss? | 15:06 |
| *** tomoe_ has joined #openstack-neutron | 15:06 | |
| aveiga | enikanorov_: wanted to let you know my team is requesting the ability to publish our data | 15:07 |
| *** tomoe_ has quit IRC | 15:07 | |
| enikanorov_ | aveiga: the doc is public, so please do! | 15:07 |
| sballe | I am stil trying ot figure out how to best implement our resiliency... | 15:07 |
| *** tomoe_ has joined #openstack-neutron | 15:07 | |
| *** tomoe_ has quit IRC | 15:07 | |
| sballe | and the idea of a self managed service... | 15:07 |
| *** g3rms has quit IRC | 15:07 | |
| sballe | enikanorov_, please remember to give the blueprint for the admin api framework at the plugin level | 15:08 |
| enikanorov_ | yeah... | 15:09 |
| *** gaud has quit IRC | 15:09 | |
| enikanorov_ | sballe: https://blueprints.launchpad.net/neutron/+spec/lbaas-extensions | 15:09 |
| enikanorov_ | sballe: it doesn't have any details, trying to find etherpad page that we used at the summit | 15:10 |
| sballe | enikanorov_, ok thanks. | 15:11 |
| *** dguerri_ is now known as _dguerri | 15:11 | |
| *** tomoe_ has joined #openstack-neutron | 15:11 | |
| *** _dguerri is now known as dguerri_ | 15:12 | |
| enikanorov_ | sballe: https://etherpad.openstack.org/p/icehouse-neutron-vendor-extension | 15:12 |
| sballe | enikanorov_, Let me ponder a little and I'll get back to the group. Also I am still trying to understand if we can integrate with Heat and do all this without confusing the user | 15:13 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Set ns_name in RouterInfo as attribute https://review.openstack.org/85051 | 15:13 |
| *** alagalah has joined #openstack-neutron | 15:13 | |
| enikanorov_ | sballe: sure | 15:14 |
| sballe | enikanorov_, thanks for the two bp | 15:14 |
| arosen | lifeless: crap really.. I'll take another look at it X.x | 15:14 |
| *** tongli has joined #openstack-neutron | 15:15 | |
| *** yamahata has joined #openstack-neutron | 15:28 | |
| *** leseb has quit IRC | 15:30 | |
| *** leseb has joined #openstack-neutron | 15:30 | |
| *** leseb has quit IRC | 15:34 | |
| *** alagalah has quit IRC | 15:36 | |
| *** leseb has joined #openstack-neutron | 15:47 | |
| *** WackoRobie has quit IRC | 15:48 | |
| *** yfried__ has quit IRC | 15:48 | |
| *** yfried__ has joined #openstack-neutron | 15:48 | |
| *** WackoRobie has joined #openstack-neutron | 15:48 | |
| *** gangur__ has quit IRC | 15:48 | |
| *** mlavalle has joined #openstack-neutron | 15:50 | |
| *** xuhanp has quit IRC | 15:50 | |
| *** sandr8 has joined #openstack-neutron | 15:50 | |
| *** rdo has quit IRC | 15:50 | |
| *** rdo has joined #openstack-neutron | 15:51 | |
| *** sandr8 has quit IRC | 15:51 | |
| *** samuelbercovici has quit IRC | 15:51 | |
| *** devvesa has left #openstack-neutron | 15:52 | |
| *** jlibosva has quit IRC | 15:52 | |
| pcm_ | can I get another core reviewer to look at https://review.openstack.org/#/c/82306? | 15:52 |
| *** dguerri_ is now known as _dguerri | 15:53 | |
| *** WackoRobie has quit IRC | 15:53 | |
| *** blogan has joined #openstack-neutron | 15:54 | |
| *** yfried__ has quit IRC | 15:54 | |
| *** jgallard has quit IRC | 15:55 | |
| *** _dguerri is now known as dguerri_ | 15:56 | |
| *** bjornar has joined #openstack-neutron | 15:56 | |
| *** _cjones_ has joined #openstack-neutron | 15:57 | |
| *** changbl has joined #openstack-neutron | 15:59 | |
| *** gaud has joined #openstack-neutron | 16:00 | |
| carl_baldwin | arosen: ping | 16:01 |
| *** jlibosva has joined #openstack-neutron | 16:01 | |
| *** evgenyf has quit IRC | 16:01 | |
| marun_afk | carl_baldwin: I'm afraid my review is in error | 16:01 |
| *** marun_afk is now known as marun | 16:01 | |
| marun | carl_baldwin: Late night review :/ | 16:01 |
| carl_baldwin | marun: I actually just posted my reply. I don't know if you just saw that. | 16:01 |
| marun | marun: No, just starting | 16:02 |
| *** thuc_ has quit IRC | 16:02 | |
| carl_baldwin | No worries. I've made a few of my own botched reviews this week. So, I completely understand. | 16:02 |
| *** thuc has joined #openstack-neutron | 16:02 | |
| marun | carl_baldwin: I'm still not sure about the wisdom of replacing looping call, though. | 16:03 |
| marun | carl_baldwin: despite there not being the issue I claimed in my review | 16:03 |
| marun | carl_baldwin: was your intent to prevent multiple calls from occurring at the same time? | 16:04 |
| *** baoli has quit IRC | 16:05 | |
| *** csd has joined #openstack-neutron | 16:05 | |
| carl_baldwin | marun: Not exacty. Here was my thinking. I actually started thinking down the lines of lazily configuring the looping call. When I went to do that, it hit me that every time an event is queued, it needs to check to see if it needs to trigger the looping call. So, we've got to trigger something at that point. | 16:05 |
| carl_baldwin | Then, I thought, why not cut out the middle man and just trigger the send directly. | 16:06 |
| *** thuc has quit IRC | 16:06 | |
| *** alagalah has joined #openstack-neutron | 16:07 | |
| *** tomoe_ has quit IRC | 16:07 | |
| *** tomoe_ has joined #openstack-neutron | 16:07 | |
| carl_baldwin | marun: Spawning a thread and delaying the send allows events to accumulate during the interval in the same way that the looping call does. | 16:08 |
| *** tomoe_ has quit IRC | 16:12 | |
| *** hrushig has joined #openstack-neutron | 16:12 | |
| *** jistr has quit IRC | 16:13 | |
| *** krtaylor has quit IRC | 16:13 | |
| *** ygbo has quit IRC | 16:14 | |
| *** bashok has quit IRC | 16:15 | |
| *** sandr8 has joined #openstack-neutron | 16:15 | |
| *** rand738 has quit IRC | 16:15 | |
| *** rand738 has joined #openstack-neutron | 16:17 | |
| *** baoli has joined #openstack-neutron | 16:18 | |
| *** csd_ has joined #openstack-neutron | 16:20 | |
| *** csd has quit IRC | 16:20 | |
| marun | carl_baldwin: The advantage of looping call is that it's well-documented - anyone familiar with eventlet will know what it is doing. | 16:20 |
| *** csd has joined #openstack-neutron | 16:21 | |
| marun | carl_baldwin: unless we need to do something that loopingcall doesn't support, I'm not sure there's a good justification to reinvent the wheel | 16:21 |
| *** csd has quit IRC | 16:22 | |
| *** csd_ is now known as csd | 16:23 | |
| *** rotbeard has quit IRC | 16:23 | |
| *** sphoorti has joined #openstack-neutron | 16:24 | |
| *** tvardeman has quit IRC | 16:24 | |
| *** sandr8 has quit IRC | 16:26 | |
| *** jobewan has joined #openstack-neutron | 16:26 | |
| *** jprovazn has quit IRC | 16:27 | |
| *** jpich has quit IRC | 16:27 | |
| carl_baldwin | marun: I see your point. Let me give it some thought. I've got a few things to do today that have piled up over the last couple of days. I'll revisit it later today. | 16:28 |
| *** csd_ has joined #openstack-neutron | 16:29 | |
| *** thuc_ has joined #openstack-neutron | 16:30 | |
| *** csd has quit IRC | 16:30 | |
| *** csd_ has quit IRC | 16:30 | |
| *** csd has joined #openstack-neutron | 16:30 | |
| *** WackoRobie has joined #openstack-neutron | 16:30 | |
| *** sphoorti has quit IRC | 16:30 | |
| marun | carl_baldwin: sounds good :) | 16:31 |
| *** sbfox has joined #openstack-neutron | 16:32 | |
| *** rm_work is now known as rm_you | 16:36 | |
| jorgem | enikanorov_: Hey I added separate sheets for particular features | 16:36 |
| enikanorov_ | jorgem: yep, see that, cool!! | 16:36 |
| jorgem | enikanorov_: K added a protocols one. I see you adding them on the first sheet | 16:37 |
| enikanorov_ | i was copying stuff from requirements wiki | 16:37 |
| *** blogan has quit IRC | 16:37 | |
| enikanorov_ | and wondering why nobody is filling the table :) | 16:37 |
| *** thuc has joined #openstack-neutron | 16:38 | |
| jorgem | enikanorov_: lol, I'm going to copy over the overview that I added | 16:39 |
| *** leseb has quit IRC | 16:40 | |
| *** leseb has joined #openstack-neutron | 16:40 | |
| *** thuc_ has quit IRC | 16:41 | |
| *** leseb has quit IRC | 16:42 | |
| *** leseb has joined #openstack-neutron | 16:42 | |
| *** sphoorti has joined #openstack-neutron | 16:43 | |
| *** sphoorti has quit IRC | 16:44 | |
| sballe | enikanorov_, jorgem Are you move the requirements into the spreasheet? | 16:45 |
| *** alagalah has quit IRC | 16:47 | |
| enikanorov_ | sballe: jorgem has created pages in the spreadsheet for particular features | 16:48 |
| *** singhs has joined #openstack-neutron | 16:48 | |
| sballe | yeah I see now. I am assuming the features here are only user facing? | 16:49 |
| *** krtaylor has joined #openstack-neutron | 16:50 | |
| *** leseb has quit IRC | 16:51 | |
| *** leseb has joined #openstack-neutron | 16:51 | |
| jorgem | sballe: not requirements, just operator data | 16:52 |
| *** alexpilotti has quit IRC | 16:53 | |
| mestery | sc68cal: Can you shoot me the link to that awesome ASCII art creating website? Thanks! | 16:53 |
| sballe | jorgem, make sense | 16:53 |
| mestery | sc68cal: I know you're not here, but I found it, posting for reference in case you skim the logs: asciiflow.com | 16:54 |
| *** sbfox has quit IRC | 16:55 | |
| sc68cal | haha cool | 16:55 |
| sc68cal | yeah asciiflow is the bomb | 16:55 |
| *** leseb has quit IRC | 16:57 | |
| *** alagalah has joined #openstack-neutron | 16:58 | |
| mestery | sc68cal: Thanks man! It is totally awesome, agreed. | 17:00 |
| *** dvorkinista has joined #openstack-neutron | 17:03 | |
| *** prad has joined #openstack-neutron | 17:04 | |
| *** sbfox has joined #openstack-neutron | 17:05 | |
| *** alagalah has quit IRC | 17:08 | |
| *** jobewan has quit IRC | 17:09 | |
| *** alagalah has joined #openstack-neutron | 17:09 | |
| *** singhs has quit IRC | 17:09 | |
| *** dvorkinista has quit IRC | 17:10 | |
| *** dvorkinista has joined #openstack-neutron | 17:11 | |
| *** singhs has joined #openstack-neutron | 17:11 | |
| *** harlowja_away is now known as harlowja | 17:14 | |
| *** nlahouti has joined #openstack-neutron | 17:15 | |
| *** Longgeek has quit IRC | 17:18 | |
| *** Longgeek has joined #openstack-neutron | 17:19 | |
| *** nlahouti has quit IRC | 17:20 | |
| *** luqas has quit IRC | 17:21 | |
| *** luqas has joined #openstack-neutron | 17:21 | |
| *** mutex has joined #openstack-neutron | 17:24 | |
| *** lollipop has quit IRC | 17:25 | |
| *** safchain has quit IRC | 17:25 | |
| mutex | does this behavior sound familiar to anyone ? | 17:30 |
| mutex | http://paste.openstack.org/show/74994/ | 17:30 |
| mutex | trying to track down what causes this, I'm sure it has already been fixed, and I am currently going through the bug history | 17:30 |
| *** manishg has joined #openstack-neutron | 17:30 | |
| *** Longgeek has quit IRC | 17:32 | |
| *** morganfainberg_Z is now known as morganfainberg | 17:34 | |
| *** packet has joined #openstack-neutron | 17:36 | |
| *** packet has quit IRC | 17:36 | |
| *** ptoohill_ has joined #openstack-neutron | 17:37 | |
| *** sandr8 has joined #openstack-neutron | 17:37 | |
| *** amuller has quit IRC | 17:38 | |
| *** networkstatic has joined #openstack-neutron | 17:39 | |
| *** otherwiseguy has quit IRC | 17:42 | |
| *** yfried__ has joined #openstack-neutron | 17:42 | |
| *** jobewan has joined #openstack-neutron | 17:43 | |
| *** thedodd has quit IRC | 17:45 | |
| *** jlibosva has quit IRC | 17:45 | |
| *** Longgeek has joined #openstack-neutron | 17:45 | |
| *** jlibosva has joined #openstack-neutron | 17:46 | |
| *** sbfox has quit IRC | 17:46 | |
| *** rms_13 has quit IRC | 17:46 | |
| *** sbfox has joined #openstack-neutron | 17:47 | |
| *** carl_baldwin_m has joined #openstack-neutron | 17:49 | |
| mestery | marun: You here and have a few minutes to eyeball something for me? | 17:51 |
| marun | mestery: yes | 17:51 |
| mestery | marun: Great! OK, more refactoring based on your comments, but I'm seeing the following odd error after doing this. Suspect it's due to inheritance, any ideas? http://paste.openstack.org/show/74999/ | 17:51 |
| mestery | marun: I can grab my refactored code as well, that may help. | 17:51 |
| marun | mestery: yeah, I'll need to look at the code too | 17:52 |
| mestery | marun: Here ---> http://paste.openstack.org/show/75000/ | 17:53 |
| mestery | marun: Any advice appreciated! I think I'm close. ;) | 17:53 |
| *** carl_baldwin_m has quit IRC | 17:53 | |
| marun | mestery: Regardless of error, the BaseMonitorTest shouldn't be in base.py since it's only use is to validate the ovsdb monitor | 17:54 |
| mestery | marun: Sorry, my references to which code is in which file was wrong in the paste. BaseMonitorTest and TestOvsdbMonitor are in the same file. | 17:54 |
| marun | mestery: ah, ok | 17:54 |
| mestery | marun: :) | 17:54 |
| mestery | marun: I'm having BaseLinuxTestCase create the OVSBridge object and the bridge in the construtor now | 17:55 |
| marun | mestery: is there a reason no to post the whole patch? Maybe to github if not to gerrit? the error indicates a problem with the 'test name' but there are no tests defined in the paste you provided | 17:55 |
| mestery | marun: Ok, let me push this to my github, one minute. | 17:56 |
| marun | mestery: I wasn't suggesting that the base class should create the bridge | 17:56 |
| marun | mestery: i was suggesting adding a helper method so the creation could be common | 17:56 |
| mestery | marun: I think it made sense to do that since this is specific to these types of tests (at the moment at least) | 17:56 |
| marun | mestery: I don't think it should be an ovs-specific base class, thoughn. | 17:56 |
| marun | mestery: Being able to check commands/sudo/create resources are useful for other things. | 17:57 |
| mestery | marun: Yeah, I wasn't sure on that either, perhaps I'll refactor a bit again and just provide a common method for bridge creation. | 17:57 |
| mestery | marun: understood | 17:57 |
| marun | mestery: so, I'd suggest changing __init__ to 'create_ovs_bridge' and have it return the created bridge | 17:57 |
| marun | mestery: oooh, __init__ | 17:58 |
| mestery | marun: OK, makes sense. | 17:58 |
| mestery | marun: That's the source of my issues, right? | 17:58 |
| marun | mestery: defining the root_helper makes sense in setUp, not init | 17:58 |
| marun | mestery: and it has to call the parent class'es setUp | 17:58 |
| marun | mestery: the likely cause of the error is that __init__ is not calling the parent __init__ | 17:58 |
| mestery | marun: https://github.com/CiscoSystems/neutron/tree/functional/vxlan/neutron/tests/functional/agent/linux | 17:58 |
| mestery | marun: Makes sense to me, I'll refactor and remove the __init__ then per suggestion | 17:59 |
| *** evgenyf has joined #openstack-neutron | 17:59 | |
| marun | mestery: +1 | 17:59 |
| *** sbfox has quit IRC | 18:00 | |
| openstackgerrit | Sridhar Gaddam proposed a change to openstack/neutron: Redundant SG rule create calls in unit tests https://review.openstack.org/85127 | 18:02 |
| *** rwsu has quit IRC | 18:03 | |
| openstackgerrit | A change was merged to openstack/neutron: ignore build directory for pep8 https://review.openstack.org/85034 | 18:04 |
| *** jprovazn has joined #openstack-neutron | 18:04 | |
| *** evgenyf has quit IRC | 18:06 | |
| *** overlayer has joined #openstack-neutron | 18:06 | |
| *** Longgeek_ has joined #openstack-neutron | 18:07 | |
| openstackgerrit | A change was merged to openstack/neutron: Fix quota_health_monitor opt name in neutron.conf https://review.openstack.org/84003 | 18:07 |
| *** Longgeek has quit IRC | 18:07 | |
| openstackgerrit | A change was merged to openstack/neutron: fixes tests using called_once_ without assert https://review.openstack.org/84139 | 18:08 |
| openstackgerrit | A change was merged to openstack/neutron: De-duplicate unit tests for ports in Big Switch https://review.openstack.org/81957 | 18:08 |
| *** hrushig has quit IRC | 18:09 | |
| *** jobewan has quit IRC | 18:10 | |
| openstackgerrit | A change was merged to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/84635 | 18:10 |
| *** jobewan has joined #openstack-neutron | 18:10 | |
| *** thedodd has joined #openstack-neutron | 18:11 | |
| *** hrushig has joined #openstack-neutron | 18:11 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Implement rootwrap daemon support https://review.openstack.org/84667 | 18:12 |
| *** Longgeek_ has quit IRC | 18:12 | |
| *** ptoohill_ has quit IRC | 18:13 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/neutron: Fix Jenkins translation jobs https://review.openstack.org/84234 | 18:17 |
| *** alagalah has quit IRC | 18:20 | |
| openstackgerrit | A change was merged to openstack/neutron: use floatingip's ID as key instead of itself https://review.openstack.org/81474 | 18:22 |
| *** dvorkinista has quit IRC | 18:23 | |
| *** harmw has quit IRC | 18:25 | |
| *** Longgeek has joined #openstack-neutron | 18:27 | |
| *** mwagner_lap has quit IRC | 18:27 | |
| *** jobewan has quit IRC | 18:28 | |
| mestery | marun: Seeing this error now (http://paste.openstack.org/show/75004/). I pushed the code here (https://github.com/CiscoSystems/neutron/tree/functional/vxlan/neutron/tests/functional/agent/linux), any ideas? | 18:28 |
| *** jobewan has joined #openstack-neutron | 18:28 | |
| *** alexpilotti has joined #openstack-neutron | 18:30 | |
| *** carl_baldwin_ has joined #openstack-neutron | 18:31 | |
| *** carl_baldwin_ has quit IRC | 18:32 | |
| *** harmw has joined #openstack-neutron | 18:33 | |
| *** manishg_ has joined #openstack-neutron | 18:33 | |
| *** manishg has quit IRC | 18:37 | |
| *** manishg_ is now known as manishg | 18:37 | |
| *** tvardeman has joined #openstack-neutron | 18:39 | |
| marun | mestery: when you get that kind of useless error, you'll want to run the test with testtools.run or nose to get a useful error message | 18:40 |
| marun | mestery: what editor do you use? | 18:40 |
| mestery | marun: vi (don't judge) | 18:40 |
| mestery | marun: How do I run it with testtools.run? | 18:41 |
| marun | look at TESTING in root of neutron repo, near the end | 18:41 |
| mestery | marun: checking | 18:41 |
| marun | mestery: honestly, you're hurting yourself coding in vi | 18:41 |
| marun | mestery: being able to properly symbolically debug is a huge productivity win | 18:41 |
| marun | mestery: if you intend to stick with vi, I'd recommend finding a separate tool for symbolic debugging at least. | 18:42 |
| mestery | marun: So I've been told, but old habits die hard. What editor do you use, either on Linux or Mac? | 18:42 |
| marun | mestery: I use emacs | 18:42 |
| mestery | marun: egads! | 18:42 |
| marun | mestery: you don't have to switch from vi, but symbolic debugging makes iterative development so much easier | 18:43 |
| *** ctooley has joined #openstack-neutron | 18:43 | |
| marun | mestery: when i'm writing a new test, i have a hotkey that runs the last test and post-mortem debugs on failure | 18:43 |
| mestery | Very nice | 18:43 |
| marun | mestery: tox is great for a pre-merge check, but I don't use it during development | 18:44 |
| ctooley | I'm setting up a PoC with some spare hardware that only has one NIC per box. I'm looking for a write up that talks about how to configure the networking for a single NIC per box multi-host environment. | 18:44 |
| marun | mestery: it takes too long and doesn't provide good feedback | 18:44 |
| *** blogan has joined #openstack-neutron | 18:45 | |
| *** blogan has quit IRC | 18:45 | |
| *** blogan has joined #openstack-neutron | 18:45 | |
| mestery | marun: Agreed on that, and now I'm sturgglign to get functional tests to run with your testools.run example ... | 18:45 |
| *** blogan has quit IRC | 18:45 | |
| *** blogan has joined #openstack-neutron | 18:45 | |
| *** blogan has joined #openstack-neutron | 18:45 | |
| marun | mestery: you need to activate the functional env first, exactly as the instructions describe for the unit env | 18:46 |
| *** sbfox has joined #openstack-neutron | 18:46 | |
| mestery | marun: Ah, that's the key, thanks! | 18:46 |
| marun | mestery: I typically pip install --user -e ./ and pip install --user -r test-requirements.txt | 18:47 |
| marun | mestery: I don't use the tox-created env for development at all. | 18:47 |
| *** itzikb has quit IRC | 18:47 | |
| marun | mestery: if you are using the tox env, make sure to pass the --develop flag to tox so that edits to the source tree are reflected in the env | 18:48 |
| mestery | Got it | 18:48 |
| marun | mestery: back in a few | 18:50 |
| *** marun is now known as marun_afk | 18:50 | |
| mestery | Thanks marun_afk. | 18:51 |
| mestery | marun_afk: I found this for vi (https://github.com/ameade/qtpy-vim), had nose running found my error in 1 minute. Slick! Thanks for the pointers! | 18:57 |
| *** WackoRob_ has joined #openstack-neutron | 19:00 | |
| *** WackoRob_ has quit IRC | 19:01 | |
| *** otherwiseguy has joined #openstack-neutron | 19:02 | |
| *** WackoRob_ has joined #openstack-neutron | 19:02 | |
| *** WackoRobie has quit IRC | 19:03 | |
| *** thedodd has quit IRC | 19:06 | |
| *** mwagner_lap has joined #openstack-neutron | 19:08 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests https://review.openstack.org/84923 | 19:10 |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add functional tests to test detection of VXLAN support https://review.openstack.org/84604 | 19:10 |
| *** csd_ has joined #openstack-neutron | 19:12 | |
| markmcclain | mestery: qtpy is is based on some work by Alfredo who has a couple of other cool vim plugins: https://github.com/alfredodeza | 19:13 |
| mestery | markmcclain: Cool, thanks for sharing! qtpy is AWESOME! | 19:14 |
| *** csd has quit IRC | 19:15 | |
| *** thedodd has joined #openstack-neutron | 19:21 | |
| *** jprovazn has quit IRC | 19:21 | |
| *** jlibosva has quit IRC | 19:23 | |
| *** networkstatic is now known as networkstatic_zZ | 19:25 | |
| *** jp_at_hp has quit IRC | 19:26 | |
| *** overlayer has quit IRC | 19:29 | |
| *** dvorkinista has joined #openstack-neutron | 19:29 | |
| lifeless | arosen: yeah | 19:30 |
| *** alagalah has joined #openstack-neutron | 19:31 | |
| *** jlibosva has joined #openstack-neutron | 19:31 | |
| *** alagalah_ has joined #openstack-neutron | 19:34 | |
| *** jp_at_hp has joined #openstack-neutron | 19:35 | |
| *** alagalah has quit IRC | 19:35 | |
| *** dvorkinista has quit IRC | 19:36 | |
| *** thuc has quit IRC | 19:40 | |
| *** leseb has joined #openstack-neutron | 19:41 | |
| *** jlibosva has quit IRC | 19:42 | |
| *** sbfox has quit IRC | 19:43 | |
| *** thuc_ has joined #openstack-neutron | 19:44 | |
| *** thuc_ has quit IRC | 19:44 | |
| *** thuc has joined #openstack-neutron | 19:45 | |
| *** djoreilly has quit IRC | 19:46 | |
| *** jp_at_hp1 has joined #openstack-neutron | 19:47 | |
| *** markwash has joined #openstack-neutron | 19:48 | |
| *** jp_at_hp has quit IRC | 19:48 | |
| *** alagalah_ has quit IRC | 19:50 | |
| *** crc32 has quit IRC | 19:52 | |
| *** alagalah has joined #openstack-neutron | 19:52 | |
| *** evgenyf has joined #openstack-neutron | 19:53 | |
| *** manishg has quit IRC | 19:53 | |
| *** baoli has quit IRC | 19:54 | |
| *** baoli has joined #openstack-neutron | 19:56 | |
| *** blogan has quit IRC | 19:56 | |
| *** otherwiseguy has quit IRC | 19:56 | |
| *** dvorkinista has joined #openstack-neutron | 19:56 | |
| *** rwsu has joined #openstack-neutron | 19:56 | |
| *** Longgeek has quit IRC | 19:56 | |
| *** blogan has joined #openstack-neutron | 19:57 | |
| *** baoli has quit IRC | 20:00 | |
| *** baoli has joined #openstack-neutron | 20:01 | |
| *** Gil_McGrath has quit IRC | 20:02 | |
| *** harlowja has quit IRC | 20:02 | |
| *** harlowja has joined #openstack-neutron | 20:03 | |
| *** networkstatic has joined #openstack-neutron | 20:05 | |
| *** HenryG has quit IRC | 20:06 | |
| *** hrushig_ has joined #openstack-neutron | 20:06 | |
| *** pcm_ has quit IRC | 20:06 | |
| *** evgenyf has quit IRC | 20:07 | |
| *** rwsu has quit IRC | 20:07 | |
| *** rwsu has joined #openstack-neutron | 20:09 | |
| *** hrushig has quit IRC | 20:09 | |
| *** ctooley has quit IRC | 20:09 | |
| *** ctooley has joined #openstack-neutron | 20:09 | |
| *** manishg has joined #openstack-neutron | 20:10 | |
| *** rwsu has quit IRC | 20:11 | |
| *** rwsu has joined #openstack-neutron | 20:11 | |
| *** armax has left #openstack-neutron | 20:13 | |
| *** gdubreui has joined #openstack-neutron | 20:13 | |
| *** WackoRobie has joined #openstack-neutron | 20:16 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add common base class for agent functional tests https://review.openstack.org/84923 | 20:18 |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add functional tests to test detection of VXLAN support https://review.openstack.org/84604 | 20:18 |
| *** matrohon_ has joined #openstack-neutron | 20:19 | |
| *** WackoRob_ has quit IRC | 20:19 | |
| *** julim has quit IRC | 20:19 | |
| mestery | HenryG: Thanks for the reviews of my functional test patches! | 20:19 |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Implement rootwrap daemon support https://review.openstack.org/84667 | 20:20 |
| *** leseb has quit IRC | 20:20 | |
| *** WackoRobie has quit IRC | 20:20 | |
| *** leseb has joined #openstack-neutron | 20:21 | |
| *** leseb has quit IRC | 20:26 | |
| *** lollipop has joined #openstack-neutron | 20:27 | |
| *** rwsu has quit IRC | 20:27 | |
| *** manishg has quit IRC | 20:30 | |
| *** rwsu has joined #openstack-neutron | 20:31 | |
| *** manishg has joined #openstack-neutron | 20:32 | |
| *** lyxus has joined #openstack-neutron | 20:33 | |
| *** baoli has quit IRC | 20:33 | |
| *** sbfox has joined #openstack-neutron | 20:33 | |
| lyxus | Hello Folks, I want to write some test unit for neutron. Is there a doc on how to write a decent test ? I saw only references to tempest | 20:34 |
| *** thuc has quit IRC | 20:37 | |
| *** thuc has joined #openstack-neutron | 20:37 | |
| *** thuc_ has joined #openstack-neutron | 20:39 | |
| haleyb | lyxus: there's lots of unit tests in neutron/tests/unit - is there anything in particular you're looking to do? | 20:39 |
| *** thuc__ has joined #openstack-neutron | 20:40 | |
| lyxus | I just want to create networks, attach subnet, add remove gateway etc... | 20:41 |
| *** thuc has quit IRC | 20:41 | |
| lyxus | haleyb, do you run them with tox ? | 20:43 |
| *** csd_ is now known as csd | 20:43 | |
| haleyb | yes, you can run the unit tests with tox | 20:44 |
| *** thuc_ has quit IRC | 20:44 | |
| haleyb | but creating networks, etc does sound more like tempest | 20:45 |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Add a note that rpc_workers option is experimental https://review.openstack.org/85169 | 20:45 |
| lyxus | haleyb, what is the *best* way to run them | 20:45 |
| *** marun_afk is now known as marun | 20:45 | |
| marun | mestery: you're halfway there, now you just need pm debugging :) | 20:46 |
| *** blogan has quit IRC | 20:46 | |
| mestery | marun: qtpy ROCKS! | 20:46 |
| mestery | marun: Yes, debugging is next. Thanks for hte pointers! Your advice has been very helpful! | 20:46 |
| haleyb | lyxus: you can run them all with ./run_tests.sh, or individually with tox, for example, tox -e py27 neutron.tests.unit.test_iptables_firewall | 20:47 |
| marun | mestery: btw, nose is great but will become less useful over time as we start to use testscenarios more. nose simply doesn't support it. testtools.run is the next best thing though it's not as featureful. | 20:47 |
| mestery | marun: OK, good to know! I'll have to see if someone has implemented something similar to nose with qtpy then. :) | 20:48 |
| marun | mestery: for simple test invocation I don't think it would be too hard to support testtools.run with qtpy if it doesn't already | 20:48 |
| mestery | marun: Agreed after noodling in the code a bit. | 20:48 |
| *** WackoRobie has joined #openstack-neutron | 20:48 | |
| mestery | marun: Pushed new versions of my functional test patches addressing some comments from HenryG while you were gone. | 20:49 |
| marun | I'm afraid some of his comments are not desirable | 20:50 |
| marun | raising exceptions is preferable to failing | 20:50 |
| *** dkehn__ has joined #openstack-neutron | 20:50 | |
| marun | otherwise it's not possible to differentiate between errors (code is wrong) and failures (test is wrong) | 20:50 |
| marun | mestery: ^ | 20:51 |
| marun | mestery: I'm going to countermand some of his suggestions | 20:51 |
| marun | mestery: they are not wise | 20:51 |
| *** dkehn has quit IRC | 20:51 | |
| mestery | marun: It's a learning experience no doubt. :) Thanks for the comments! | 20:52 |
| *** SridharG has left #openstack-neutron | 20:52 | |
| *** dkehn_ has quit IRC | 20:52 | |
| *** alagalah has quit IRC | 20:52 | |
| *** dkehn has joined #openstack-neutron | 20:53 | |
| *** alagalah has joined #openstack-neutron | 20:54 | |
| *** dkehn__ is now known as dkehn_ | 20:54 | |
| *** djoreilly has joined #openstack-neutron | 20:57 | |
| *** djoreilly has quit IRC | 20:58 | |
| lyxus | haleyb, thanks ! I also had a problem with tox (tox.ConfigError: ConfigError: substitution key 'posargs' not found) I had to downgrade tox, for reference pip install --upgrade "tox>=1.6,<1.7" | 20:59 |
| *** alagalah has quit IRC | 21:02 | |
| openstackgerrit | Robert Kukura proposed a change to openstack/neutron: ML2: ODL driver sets port status https://review.openstack.org/85176 | 21:04 |
| *** dvorkinista has quit IRC | 21:04 | |
| *** gaud has quit IRC | 21:05 | |
| *** leseb has joined #openstack-neutron | 21:07 | |
| *** dkehn__ has joined #openstack-neutron | 21:15 | |
| *** dkehn__ has quit IRC | 21:16 | |
| *** gaud has joined #openstack-neutron | 21:16 | |
| *** dkehn__ has joined #openstack-neutron | 21:16 | |
| *** dkehn has quit IRC | 21:16 | |
| *** dkehn_ has quit IRC | 21:16 | |
| jogo | whoa neutrons use of oslo.config is all screwy | 21:17 |
| *** dkehn has joined #openstack-neutron | 21:18 | |
| lifeless | all I heard was whoa neutron screwy :P | 21:21 |
| *** rwsu has quit IRC | 21:22 | |
| *** dkehn_ has joined #openstack-neutron | 21:23 | |
| *** bjornar has quit IRC | 21:24 | |
| *** dkehn__ has quit IRC | 21:25 | |
| *** dkehn has quit IRC | 21:25 | |
| jogo | lifeless: they redefine conf options all over the place | 21:25 |
| jogo | and double load them etc | 21:25 |
| *** jecarey has quit IRC | 21:25 | |
| lifeless | jogo: don't look at their schema management stuff | 21:26 |
| *** dkehn has joined #openstack-neutron | 21:26 | |
| jogo | haha | 21:26 |
| markmcclain | jogo: redefine within plugins or in different agents? | 21:27 |
| jogo | markmcclain: in tests | 21:30 |
| jogo | and you load them inside of classes | 21:30 |
| jogo | as far as I can tell your usage of oslo.config is very different then nova and very screwy | 21:30 |
| jogo | you have to instantiate classes to load config options etc | 21:30 |
| *** WackoRobie has quit IRC | 21:31 | |
| jogo | markmcclain: giving up on https://review.openstack.org/#/c/84620/ because the fix is massive | 21:31 |
| *** WackoRobie has joined #openstack-neutron | 21:32 | |
| markmcclain | jogo: thanks for trying | 21:32 |
| *** tvardeman has quit IRC | 21:32 | |
| *** dkehn__ has joined #openstack-neutron | 21:32 | |
| markmcclain | we should be refactoring many of the tests in Juno, so hopefully we can begin to clean up issue | 21:33 |
| *** dkehn has quit IRC | 21:33 | |
| *** dkehn_ has quit IRC | 21:34 | |
| jogo | markmcclain: unfortunately oslo.config isn't well documented with best practices either | 21:35 |
| jogo | so the problem with my patch is I cannot reproduce the failure locally | 21:35 |
| *** dkehn has joined #openstack-neutron | 21:36 | |
| jogo | you guys don't seem to use http://docs.openstack.org/developer/oslo.config/configopts.html?highlight=import_opt at all | 21:36 |
| *** sandr8_ has joined #openstack-neutron | 21:36 | |
| *** WackoRobie has quit IRC | 21:37 | |
| marun | jogo: I'm kind of swamped right now but if you could knowledge transfer next week I could take a stab | 21:37 |
| markmcclain | yeah import_opt isn't used too much | 21:37 |
| *** dkehn__ is now known as dkehn-x | 21:37 | |
| jogo | marun: sure, although markmc would be even better | 21:37 |
| marun | jogo: ahah | 21:37 |
| markmcclain | so of the setup is also a bit a reflection of how were were using the cfg lib before it was spun out to into a separate lib | 21:37 |
| jogo | marun in short you are using the global config thing very oddly | 21:38 |
| marun | jogo: re: import_opt, I think we need to use that | 21:38 |
| jogo | marun: yes, or import_group | 21:38 |
| *** sandr8__ has joined #openstack-neutron | 21:38 | |
| marun | jogo: i filed a bug because i couldn't run the ml2 plugin tests in isolation due to how ml2 drivers define options that are only loaded by stevadore | 21:38 |
| markmcclain | we also have a few tests that setup their own config object | 21:38 |
| marun | jogo: I'm betting import_opt should be used somewhere in there | 21:38 |
| *** sandr8_ has quit IRC | 21:39 | |
| jogo | marun: every time you need a config option that isn't defined in your file | 21:39 |
| *** sandr8_ has joined #openstack-neutron | 21:39 | |
| *** HenryG has joined #openstack-neutron | 21:39 | |
| marun | jogo: so every module needs to do that? | 21:39 |
| marun | jogo: not sure that question makes sense actually | 21:40 |
| *** sandr8 has quit IRC | 21:40 | |
| marun | jogo: so to confirm, if a module uses a config value, it needs to ensure it is imported explicitly via import_opt/import_group rather than relying on importing the module directly? | 21:40 |
| *** sandr8 has joined #openstack-neutron | 21:41 | |
| jogo | marun: correct | 21:41 |
| jogo | for example nova uses import_opt 302 times! | 21:41 |
| marun | jogo: or is it mix/match - use import_{opt|group} iff the defining module isn't already being importedc? | 21:41 |
| jogo | and we have ~700 conf options | 21:41 |
| jogo | so you only *need* import_opt if the module isn't loaded but its good to be safe | 21:41 |
| jogo | also you define opts inside of classes | 21:42 |
| jogo | which breaks import_opts | 21:42 |
| marun | jogo yeah, that sounds dumb even without knowing the specifics | 21:42 |
| jogo | marun: no comment ;) | 21:42 |
| marun | jogo: what do you think of adding a hacking check that validates that import_* is always used? | 21:43 |
| marun | jogo: it would take some cleanup, but then we'd prevent regressions | 21:43 |
| *** sandr8_ has quit IRC | 21:43 | |
| marun | jogo: (not sure how complex that would be to implement tho) | 21:43 |
| *** sandr8__ has quit IRC | 21:43 | |
| *** jorgem has quit IRC | 21:44 | |
| jogo | hmm actaully you can just ban register_opt inside of tests | 21:44 |
| jogo | anyway baby steps | 21:44 |
| jogo | gonna try a POC patch to show some cleanups | 21:44 |
| marun | jogo: ok | 21:45 |
| *** stmi has joined #openstack-neutron | 21:48 | |
| *** alexpilotti has quit IRC | 21:49 | |
| mestery | marun: QQ: For the refactoring of create_resource, if I'm to use **kwargs, this assumes the ovs_lib functions use **kwargs, which they don't. | 21:55 |
| *** thuc__ has quit IRC | 21:55 | |
| marun | mestery: you can pass **kwargs to a function that doesn't accept them | 21:55 |
| mestery | marun: They seem to all take positional arguments, which means create_resource will need to be conditional to handle multiple different functions. Am I missing something? | 21:55 |
| *** leseb has quit IRC | 21:55 | |
| mestery | marun: Ah, I got it now! | 21:56 |
| mestery | marun: Let me try that. | 21:56 |
| *** thuc has joined #openstack-neutron | 21:56 | |
| marun | mestery: def foo(): pass; foo(*[], **{}) | 21:56 |
| arosen | https://twitter.com/AccordionGuy/status/451353797827518465/photo/1 | 21:58 |
| arosen | enjoy :) | 21:58 |
| mestery | marun: Got it! Nice! | 21:59 |
| mestery | arosen: :P | 21:59 |
| marun | arosen: hah, I know that guy. he's from toronto. | 21:59 |
| *** sbfox has quit IRC | 22:00 | |
| arosen | marun: Joey DdeVilla ? | 22:00 |
| *** thuc has quit IRC | 22:00 | |
| marun | arosen: si | 22:00 |
| marun | arosen: ah, formerly of toronto | 22:01 |
| jogo | gah this is even funnier then I thought, you commonly defie an opt but don't load it in the same file | 22:01 |
| * jogo walks away for now | 22:01 | |
| marun | jogo: my condolences. i think it's crazy too. | 22:01 |
| anteaya | Joey DeVilla? | 22:01 |
| anteaya | haven't seen his name mentioned in a while | 22:02 |
| marun | anteaya: cats ^^ | 22:02 |
| anteaya | yeah he finds good pics | 22:03 |
| *** marun is now known as marun_afk | 22:03 | |
| *** thuc has joined #openstack-neutron | 22:04 | |
| anteaya | I actually wondered how long it would be before he was invoked in openstack | 22:05 |
| anteaya | figured it would at least take him to show up somewhere | 22:05 |
| *** gaud has quit IRC | 22:05 | |
| HenryG | mestery: marun_afk: sorry for the fail() suggestion in the review. :( I didn't think it through. | 22:07 |
| *** banix has quit IRC | 22:09 | |
| *** WackoRobie has joined #openstack-neutron | 22:11 | |
| *** dvorkinista has joined #openstack-neutron | 22:12 | |
| *** sandr8 has quit IRC | 22:14 | |
| *** HenryG_ has joined #openstack-neutron | 22:14 | |
| *** hrushig_ has quit IRC | 22:14 | |
| *** HenryG has quit IRC | 22:15 | |
| *** HenryG_ has quit IRC | 22:15 | |
| *** HenryG has joined #openstack-neutron | 22:16 | |
| *** dvorkinista has quit IRC | 22:16 | |
| *** hrushig has joined #openstack-neutron | 22:17 | |
| *** krtaylor has quit IRC | 22:22 | |
| *** WackoRob_ has joined #openstack-neutron | 22:22 | |
| *** networkstatic has quit IRC | 22:22 | |
| *** sandr8 has joined #openstack-neutron | 22:23 | |
| *** changbl has quit IRC | 22:23 | |
| *** jobewan has quit IRC | 22:24 | |
| *** gaud has joined #openstack-neutron | 22:24 | |
| *** WackoRobie has quit IRC | 22:25 | |
| *** rwsu has joined #openstack-neutron | 22:26 | |
| *** sandr8_ has joined #openstack-neutron | 22:26 | |
| *** sandr8 has quit IRC | 22:30 | |
| *** prad has quit IRC | 22:32 | |
| *** thuc has quit IRC | 22:32 | |
| *** thuc has joined #openstack-neutron | 22:32 | |
| openstackgerrit | A change was merged to openstack/neutron: Replace HTTPSConnection in NEC plugin https://review.openstack.org/77414 | 22:33 |
| openstackgerrit | A change was merged to openstack/neutron: Fix Jenkins translation jobs https://review.openstack.org/84234 | 22:33 |
| *** dkehn has quit IRC | 22:35 | |
| *** dkehn-x has quit IRC | 22:36 | |
| *** thuc has quit IRC | 22:37 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Allow tenant to set router external IP https://review.openstack.org/83664 | 22:41 |
| mestery | HenryG: No worries at all my friend. | 22:43 |
| *** yamahata has quit IRC | 22:44 | |
| HenryG | mestery: You are blazing a trail here with the first useful functional tests! Good stuff. | 22:45 |
| mestery | HenryG: Thanks to marun for the coaching! I should have new patches up later tonight, appreciate your review then! | 22:45 |
| *** jaypipes has joined #openstack-neutron | 22:45 | |
| HenryG | mestery: count on it | 22:46 |
| *** marun_afk is now known as marun | 22:52 | |
| *** dave_tucker is now known as dave_tucker_zzz | 22:57 | |
| *** bvandenh has quit IRC | 23:00 | |
| *** yamahata has joined #openstack-neutron | 23:04 | |
| *** matsuhashi has joined #openstack-neutron | 23:05 | |
| *** dkehn has joined #openstack-neutron | 23:06 | |
| *** dkehn is now known as dkehnx | 23:07 | |
| *** mlavalle has quit IRC | 23:07 | |
| *** dkehn has joined #openstack-neutron | 23:09 | |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Embrane plugin lacks db migration https://review.openstack.org/85196 | 23:09 |
| *** carl_baldwin has quit IRC | 23:10 | |
| *** thedodd has quit IRC | 23:12 | |
| *** jesusaurus has joined #openstack-neutron | 23:13 | |
| *** gaud has quit IRC | 23:14 | |
| openstackgerrit | Abhishek Raut proposed a change to openstack/neutron: Segregate the VSM calls from database calls in N1kv plugin https://review.openstack.org/82266 | 23:15 |
| *** manishg_ has joined #openstack-neutron | 23:18 | |
| *** jgrimm has quit IRC | 23:18 | |
| *** dave_tucker_zzz is now known as dave_tucker | 23:22 | |
| openstackgerrit | oda-g proposed a change to openstack/neutron: Fix to work multiple RPC workers under Metaplugin https://review.openstack.org/84914 | 23:26 |
| *** tongli has quit IRC | 23:26 | |
| *** sandr8_ has quit IRC | 23:27 | |
| *** sbfox has joined #openstack-neutron | 23:29 | |
| *** markwash has quit IRC | 23:29 | |
| *** HenryG_ has joined #openstack-neutron | 23:30 | |
| *** markmcclain has quit IRC | 23:31 | |
| *** jesusaurus has left #openstack-neutron | 23:32 | |
| *** HenryG has quit IRC | 23:32 | |
| *** Sukhdev has joined #openstack-neutron | 23:34 | |
| *** aveiga has quit IRC | 23:37 | |
| *** thuc has joined #openstack-neutron | 23:37 | |
| *** HenryG has joined #openstack-neutron | 23:38 | |
| *** thuc has quit IRC | 23:38 | |
| *** yamahata has quit IRC | 23:38 | |
| *** ctooley has quit IRC | 23:39 | |
| *** thuc has joined #openstack-neutron | 23:39 | |
| *** thuc has quit IRC | 23:39 | |
| *** thuc has joined #openstack-neutron | 23:40 | |
| *** HenryG_ has quit IRC | 23:41 | |
| *** HenryG has quit IRC | 23:42 | |
| *** crc32 has joined #openstack-neutron | 23:44 | |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Embrane plugin lacks db migration https://review.openstack.org/85196 | 23:45 |
| *** networkstatic has joined #openstack-neutron | 23:50 | |
| *** zhenjie has joined #openstack-neutron | 23:58 | |
| *** xuhanp has joined #openstack-neutron | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!