| *** alagalah has quit IRC | 00:00 | |
| *** carl_baldwin has quit IRC | 00:01 | |
| *** SumitNaiksatam has quit IRC | 00:08 | |
| *** csd has joined #openstack-neutron | 00:09 | |
| *** jorgem has joined #openstack-neutron | 00:09 | |
| *** jorgem has quit IRC | 00:13 | |
| *** dvorkinista has quit IRC | 00:16 | |
| *** thuc has quit IRC | 00:20 | |
| *** thuc has joined #openstack-neutron | 00:21 | |
| *** gdubreui has quit IRC | 00:22 | |
| *** singhs has quit IRC | 00:24 | |
| *** thuc has quit IRC | 00:26 | |
| *** matsuhashi has joined #openstack-neutron | 00:26 | |
| *** ajo has quit IRC | 00:27 | |
| *** SumitNaiksatam has joined #openstack-neutron | 00:28 | |
| *** BuSerD has quit IRC | 00:29 | |
| *** jorgem has joined #openstack-neutron | 00:30 | |
| *** openstackgerrit has joined #openstack-neutron | 00:30 | |
| *** jorgem has quit IRC | 00:37 | |
| *** manishg has quit IRC | 00:38 | |
| *** hrushig has quit IRC | 00:40 | |
| *** ajo has joined #openstack-neutron | 00:40 | |
| *** sbfox has quit IRC | 00:42 | |
| openstackgerrit | gongysh proposed a change to openstack/neutron: use floatingip's ID as key instead of itself https://review.openstack.org/81474 | 00:46 |
|---|---|---|
| *** matsuhashi has quit IRC | 00:47 | |
| *** matsuhashi has joined #openstack-neutron | 00:48 | |
| *** csd has quit IRC | 00:51 | |
| *** matsuhashi has quit IRC | 00:52 | |
| *** manishg has joined #openstack-neutron | 00:53 | |
| *** manishg_ has joined #openstack-neutron | 00:55 | |
| *** matsuhashi has joined #openstack-neutron | 00:57 | |
| *** manishg has quit IRC | 00:58 | |
| *** Sukhdev has joined #openstack-neutron | 00:58 | |
| *** manishg_ is now known as manishg | 00:58 | |
| *** baoli has joined #openstack-neutron | 01:03 | |
| *** dvorkinista has joined #openstack-neutron | 01:04 | |
| jogo | any neutron cores around? | 01:05 |
| jogo | re: https://review.openstack.org/#/c/84568/ unblocking your unit tests | 01:05 |
| *** jorgem has joined #openstack-neutron | 01:05 | |
| *** jorgem has quit IRC | 01:07 | |
| *** WackoRobie has quit IRC | 01:07 | |
| *** networkstatic has joined #openstack-neutron | 01:08 | |
| *** jaypipes has quit IRC | 01:08 | |
| *** WackoRobie has joined #openstack-neutron | 01:08 | |
| *** manishg_ has joined #openstack-neutron | 01:09 | |
| *** manishg has quit IRC | 01:10 | |
| *** alagalah has joined #openstack-neutron | 01:10 | |
| *** manishg_ has quit IRC | 01:10 | |
| *** tomoe_ has joined #openstack-neutron | 01:12 | |
| *** manishg has joined #openstack-neutron | 01:12 | |
| *** WackoRobie has quit IRC | 01:13 | |
| *** dvorkinista has quit IRC | 01:13 | |
| *** alagalah has quit IRC | 01:15 | |
| *** WackoRobie has joined #openstack-neutron | 01:15 | |
| *** sandr8 has quit IRC | 01:16 | |
| *** tomoe_ has quit IRC | 01:19 | |
| *** tomoe_ has joined #openstack-neutron | 01:19 | |
| *** ramishra has joined #openstack-neutron | 01:22 | |
| *** tomoe_ has quit IRC | 01:24 | |
| *** beagles has quit IRC | 01:28 | |
| *** BuSerD has joined #openstack-neutron | 01:28 | |
| *** yongli has joined #openstack-neutron | 01:29 | |
| *** manishg has quit IRC | 01:30 | |
| *** ramishra has quit IRC | 01:30 | |
| *** tomoe_ has joined #openstack-neutron | 01:31 | |
| *** thuc has joined #openstack-neutron | 01:32 | |
| *** tomoe_ has quit IRC | 01:33 | |
| *** networkstatic has quit IRC | 01:33 | |
| *** thuc has quit IRC | 01:36 | |
| Mario_ | hi everyone! | 01:37 |
| Mario_ | does anyone know how to add the "dns ip" in the dhcp_agent.ini, I am referring to the VMs resolv.conf using linux for example | 01:38 |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas on a network without gateway https://review.openstack.org/84058 | 01:39 |
| *** manishg has joined #openstack-neutron | 01:41 | |
| Mario_ | On my configs I am now able to add the router, but the problem on the VMs it only get the ip from the internal subnet. How do I add the my router IP to the VMs.. | 01:44 |
| Mario_ | ? | 01:44 |
| *** gdubreui has joined #openstack-neutron | 01:47 | |
| *** changbl has joined #openstack-neutron | 01:47 | |
| Mario_ | hello? | 01:47 |
| *** xuhanp has joined #openstack-neutron | 01:50 | |
| *** xianghui has joined #openstack-neutron | 01:53 | |
| Mario_ | what is this dhcp_confs? in /etc/neutron/dhcp_agent.ini. Are there any sample on this? | 01:55 |
| *** dims_ has quit IRC | 01:57 | |
| *** dims_ has joined #openstack-neutron | 01:58 | |
| openstackgerrit | Yaguang Tang proposed a change to openstack/python-neutronclient: Make neutronclient parse keystone v3 endpoints correctly https://review.openstack.org/81767 | 01:59 |
| openstackgerrit | Sukhdev Kapur proposed a change to openstack/neutron: Fixing Arista CLI command https://review.openstack.org/84603 | 02:05 |
| *** amotoki has joined #openstack-neutron | 02:11 | |
| *** sballe has joined #openstack-neutron | 02:11 | |
| *** lollipop has joined #openstack-neutron | 02:12 | |
| *** flwang has quit IRC | 02:17 | |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Add functional tests to check host VXLAN support https://review.openstack.org/84604 | 02:19 |
| mestery | marun: When you get a chance, please provide me feedback on this functional test: https://review.openstack.org/#/c/84604/ | 02:20 |
| mestery | marun: This is for the VXLAN host support issue we've talked about today. Thanks! | 02:20 |
| mestery | marun: I marked it WIP for now until you eyeball it a bit. | 02:20 |
| *** dvorkinista has joined #openstack-neutron | 02:23 | |
| *** sballe has quit IRC | 02:26 | |
| *** dvorkinista has quit IRC | 02:28 | |
| *** yamahata has joined #openstack-neutron | 02:31 | |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Return 409 for second firewall creation https://review.openstack.org/84607 | 02:33 |
| *** singhs has joined #openstack-neutron | 02:35 | |
| marun | mestery: will do | 02:35 |
| *** ramishra has joined #openstack-neutron | 02:36 | |
| *** ramishra_ has joined #openstack-neutron | 02:37 | |
| *** SumitNaiksatam has quit IRC | 02:39 | |
| *** ramishra has quit IRC | 02:41 | |
| *** singhs_ has joined #openstack-neutron | 02:42 | |
| *** singhs has quit IRC | 02:42 | |
| *** singhs_ is now known as singhs | 02:42 | |
| *** banix has joined #openstack-neutron | 02:43 | |
| *** otherwiseguy has quit IRC | 02:47 | |
| *** tomoe_ has joined #openstack-neutron | 02:48 | |
| openstackgerrit | Henry Gessau proposed a change to openstack/neutron: Clean up unused CONF.state_path overrides in tests https://review.openstack.org/84608 | 02:48 |
| openstackgerrit | A change was merged to openstack/neutron: Use a temp dir for CONF.state_path https://review.openstack.org/84568 | 02:49 |
| *** sbfox has joined #openstack-neutron | 02:51 | |
| *** flwang has joined #openstack-neutron | 02:55 | |
| *** Sukhdev has quit IRC | 02:58 | |
| *** flwang has quit IRC | 03:03 | |
| *** flwang has joined #openstack-neutron | 03:04 | |
| *** BuSerD has quit IRC | 03:08 | |
| *** singhs has quit IRC | 03:11 | |
| *** dims_ has quit IRC | 03:15 | |
| *** SumitNaiksatam has joined #openstack-neutron | 03:17 | |
| *** dims_ has joined #openstack-neutron | 03:17 | |
| *** vilobhmm has joined #openstack-neutron | 03:19 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: NSX: Ensure that DynamicLoop for sync doesn't raise https://review.openstack.org/84523 | 03:19 |
| *** matsuhashi has quit IRC | 03:21 | |
| *** yfried__ has joined #openstack-neutron | 03:22 | |
| *** harlowja is now known as harlowja_away | 03:24 | |
| *** morganfainberg is now known as morganfainberg_Z | 03:34 | |
| *** amotoki has quit IRC | 03:34 | |
| *** networkstatic has joined #openstack-neutron | 03:36 | |
| *** ramishra_ has quit IRC | 03:37 | |
| *** ramishra has joined #openstack-neutron | 03:37 | |
| *** yamamoto_ has joined #openstack-neutron | 03:51 | |
| openstackgerrit | Li Ma proposed a change to openstack/neutron: Race condition of L3-agent to add/remove routers https://review.openstack.org/73234 | 03:53 |
| *** WackoRobie has quit IRC | 03:54 | |
| *** bashok has joined #openstack-neutron | 03:55 | |
| *** WackoRobie has joined #openstack-neutron | 03:55 | |
| *** tomoe_ has quit IRC | 03:56 | |
| *** tomoe_ has joined #openstack-neutron | 03:57 | |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas on a network without gateway https://review.openstack.org/84058 | 03:59 |
| *** WackoRobie has quit IRC | 03:59 | |
| *** tomoe_ has quit IRC | 04:01 | |
| *** BuSerD has joined #openstack-neutron | 04:04 | |
| *** BuSerD has quit IRC | 04:04 | |
| *** yfried__ has quit IRC | 04:11 | |
| *** thuc has joined #openstack-neutron | 04:12 | |
| *** thuc_ has joined #openstack-neutron | 04:13 | |
| *** tomoe_ has joined #openstack-neutron | 04:17 | |
| *** thuc has quit IRC | 04:17 | |
| *** sandr8 has joined #openstack-neutron | 04:25 | |
| *** singhs has joined #openstack-neutron | 04:26 | |
| *** bashok has quit IRC | 04:28 | |
| *** WackoRobie has joined #openstack-neutron | 04:29 | |
| *** singhs_ has joined #openstack-neutron | 04:29 | |
| *** zhipeng has quit IRC | 04:29 | |
| *** manishg has quit IRC | 04:30 | |
| *** singhs has quit IRC | 04:31 | |
| *** singhs_ is now known as singhs | 04:31 | |
| *** sbfox has joined #openstack-neutron | 04:33 | |
| *** matsuhashi has joined #openstack-neutron | 04:36 | |
| *** WackoRobie has quit IRC | 04:39 | |
| *** tomoe_ has quit IRC | 04:41 | |
| *** sbfox1 has joined #openstack-neutron | 04:42 | |
| *** tomoe_ has joined #openstack-neutron | 04:42 | |
| *** tomoe__ has joined #openstack-neutron | 04:43 | |
| *** tomoe_ has quit IRC | 04:43 | |
| *** sbfox has quit IRC | 04:44 | |
| *** matsuhashi has quit IRC | 04:50 | |
| *** oda-g has joined #openstack-neutron | 04:52 | |
| *** banix has quit IRC | 04:53 | |
| *** thuc_ has quit IRC | 04:57 | |
| *** sridhar has joined #openstack-neutron | 04:57 | |
| *** thuc has joined #openstack-neutron | 04:58 | |
| *** matsuhashi has joined #openstack-neutron | 05:00 | |
| *** thuc has quit IRC | 05:02 | |
| *** ianw has quit IRC | 05:08 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Fix up usage of state_path mocking in unit tests https://review.openstack.org/84620 | 05:08 |
| *** ianw has joined #openstack-neutron | 05:08 | |
| *** ianw has quit IRC | 05:09 | |
| *** ianw has joined #openstack-neutron | 05:09 | |
| *** yfried__ has joined #openstack-neutron | 05:11 | |
| *** irenab has joined #openstack-neutron | 05:12 | |
| openstackgerrit | fumihiko kakuma proposed a change to openstack/neutron: OFA agent: use hexadecimal IP address in tunnel port name https://review.openstack.org/81436 | 05:22 |
| *** tomoe__ has quit IRC | 05:27 | |
| *** amotoki has joined #openstack-neutron | 05:31 | |
| *** lollipop has quit IRC | 05:32 | |
| *** WackoRobie has joined #openstack-neutron | 05:34 | |
| *** lollipop has joined #openstack-neutron | 05:34 | |
| *** WackoRobie has quit IRC | 05:38 | |
| openstackgerrit | A change was merged to openstack/neutron: Log configuration values for OFA agent https://review.openstack.org/78861 | 05:45 |
| *** tomoe_ has joined #openstack-neutron | 05:46 | |
| *** sridhar has quit IRC | 06:01 | |
| *** tomoe_ has quit IRC | 06:01 | |
| *** vilobhmm has quit IRC | 06:06 | |
| *** bashok has joined #openstack-neutron | 06:08 | |
| *** thuc has joined #openstack-neutron | 06:08 | |
| *** tomoe_ has joined #openstack-neutron | 06:10 | |
| *** thuc has quit IRC | 06:13 | |
| xuhanp | amotoki, ping | 06:15 |
| *** WackoRobie has joined #openstack-neutron | 06:18 | |
| *** dvorkinista has joined #openstack-neutron | 06:22 | |
| *** WackoRobie has quit IRC | 06:23 | |
| openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/84635 | 06:31 |
| *** gdubreui has quit IRC | 06:32 | |
| *** sungju_ has quit IRC | 06:34 | |
| *** pcm_ has quit IRC | 06:37 | |
| *** saju_m has joined #openstack-neutron | 06:39 | |
| *** nplanel_ has quit IRC | 06:51 | |
| *** coolsvap has joined #openstack-neutron | 06:52 | |
| *** evgenyf has joined #openstack-neutron | 06:52 | |
| *** Longgeek has joined #openstack-neutron | 06:53 | |
| amotoki | xuhanp: pong | 07:00 |
| *** ramishra has quit IRC | 07:11 | |
| xuhanp | amotoki, can I borrow some time from you to review https://review.openstack.org/#/c/72252/ again? | 07:11 |
| *** dvorkini_ has joined #openstack-neutron | 07:12 | |
| *** dvorkinista has quit IRC | 07:12 | |
| xuhanp | amotoki, for your information, I didn't totally get the problem about Darragh's -1. So hope you can help understand that as well. | 07:13 |
| *** ramishra has joined #openstack-neutron | 07:16 | |
| *** WackoRobie has joined #openstack-neutron | 07:19 | |
| amotoki | xuhanp: i will check the review later. now in the meeting. | 07:21 |
| xuhanp | amotoki, thanks a lot! | 07:22 |
| *** WackoRobie has quit IRC | 07:24 | |
| *** sandr8 has quit IRC | 07:25 | |
| *** jlibosva has joined #openstack-neutron | 07:26 | |
| *** jroovers has joined #openstack-neutron | 07:26 | |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Return 409 for second firewall creation https://review.openstack.org/84607 | 07:26 |
| *** luqas has joined #openstack-neutron | 07:32 | |
| *** tomoe__ has joined #openstack-neutron | 07:33 | |
| *** tomoe_ has quit IRC | 07:36 | |
| *** dvorkini_ has quit IRC | 07:37 | |
| openstackgerrit | Marios Andreou proposed a change to openstack/neutron: Validate CIDR given as ip-prefix in security-group-rule-create https://review.openstack.org/59212 | 07:37 |
| *** jgallard has joined #openstack-neutron | 07:42 | |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas delete and re-add same vip will make the vip can't work https://review.openstack.org/84647 | 07:42 |
| *** saju_m has quit IRC | 07:42 | |
| openstackgerrit | Rajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS) Service Insertion https://review.openstack.org/62599 | 07:43 |
| *** roeyc has joined #openstack-neutron | 07:44 | |
| *** jroovers|afk has joined #openstack-neutron | 07:46 | |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas delete and re-add same vip will make the vip can't work https://review.openstack.org/84647 | 07:46 |
| *** jroovers has quit IRC | 07:49 | |
| *** jroovers|afk has quit IRC | 07:50 | |
| *** jroovers has joined #openstack-neutron | 07:51 | |
| *** saju_m has joined #openstack-neutron | 07:52 | |
| *** jistr has joined #openstack-neutron | 07:53 | |
| openstackgerrit | Rajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS) Service Insertion https://review.openstack.org/62599 | 07:55 |
| *** Mario_ has quit IRC | 07:56 | |
| *** jpich has joined #openstack-neutron | 07:58 | |
| *** ygbo has joined #openstack-neutron | 07:58 | |
| *** safchain has joined #openstack-neutron | 08:00 | |
| openstackgerrit | Yves-Gwenael Bourhis proposed a change to openstack/neutron: Changed DictModel to dict with attribute access https://review.openstack.org/64696 | 08:00 |
| *** djoreilly has joined #openstack-neutron | 08:01 | |
| djoreilly | xuhanp: ping | 08:02 |
| *** SumitNaiksatam has quit IRC | 08:04 | |
| *** skraynev_afk is now known as skraynev | 08:08 | |
| *** leseb has joined #openstack-neutron | 08:09 | |
| *** rotbeard has joined #openstack-neutron | 08:12 | |
| *** _afazekas has joined #openstack-neutron | 08:13 | |
| ihrachys | salv-orlando: around? | 08:13 |
| salv-orlando | yes ihrachys | 08:13 |
| ihrachys | salv-orlando: https://bugs.launchpad.net/neutron/+bug/1301027 | 08:13 |
| ihrachys | salv-orlando: seems to affect stable | 08:14 |
| ihrachys | salv-orlando: http://logs.openstack.org/periodic-stable/periodic-neutron-python26-havana/e17f1e1/ | 08:14 |
| salv-orlando | it affects everything where oslo.config was upgraded to 1.3 | 08:14 |
| ihrachys | salv-orlando: but the fix went to neutron master, so probably it should be backported | 08:15 |
| ihrachys | salv-orlando: I can do it, I just need confirmation for the need | 08:15 |
| salv-orlando | yes that will surely be done; | 08:15 |
| salv-orlando | if you can do that it will be great | 08:15 |
| ihrachys | ok, on my way to cherry-pick | 08:15 |
| ihrachys | tnx salv-orlando | 08:15 |
| salv-orlando | np | 08:16 |
| xuhanp | djoreilly, pong | 08:18 |
| *** rand738 has quit IRC | 08:18 | |
| djoreilly | xuhanp: yeah please push that patch and I will test now | 08:18 |
| *** rand738 has joined #openstack-neutron | 08:18 | |
| xuhanp | djoreilly, will do. Thanks for testing this out. | 08:18 |
| djoreilly | xuhanp: it should be the last one | 08:19 |
| xuhanp | djoreilly, hope so :-) | 08:19 |
| *** WackoRobie has joined #openstack-neutron | 08:20 | |
| *** devvesa has joined #openstack-neutron | 08:20 | |
| *** pradipta_away is now known as pradipta | 08:22 | |
| *** WackoRobie has quit IRC | 08:24 | |
| openstackgerrit | Xu Han Peng proposed a change to openstack/neutron: Permit ICMPv6 RAs only from known routers https://review.openstack.org/72252 | 08:25 |
| openstackgerrit | Rajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS) Service Insertion https://review.openstack.org/62599 | 08:27 |
| *** ramishra has quit IRC | 08:27 | |
| *** wayneeseguin has quit IRC | 08:28 | |
| *** wayneeseguin has joined #openstack-neutron | 08:28 | |
| *** dvorkinista has joined #openstack-neutron | 08:41 | |
| *** sbfox1 has quit IRC | 08:42 | |
| *** Longgeek has quit IRC | 08:42 | |
| *** Longgeek has joined #openstack-neutron | 08:45 | |
| *** dvorkinista has quit IRC | 08:46 | |
| *** saju_m has quit IRC | 08:48 | |
| *** saju_m has joined #openstack-neutron | 08:49 | |
| *** lollipop has quit IRC | 08:54 | |
| *** yamamoto_ has quit IRC | 08:54 | |
| openstackgerrit | A change was merged to openstack/neutron: NSX plugin: return 400 for invalid gw certificate https://review.openstack.org/80948 | 08:56 |
| openstackgerrit | A change was merged to openstack/neutron: Removing vim header lines https://review.openstack.org/84186 | 08:57 |
| *** matrohon_ has joined #openstack-neutron | 08:57 | |
| *** matrohon_ has quit IRC | 08:57 | |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas delete and re-add same vip will make the vip can't work https://review.openstack.org/84647 | 08:59 |
| *** chandankumar_ has joined #openstack-neutron | 08:59 | |
| *** leseb has quit IRC | 09:01 | |
| *** coolsvap has quit IRC | 09:02 | |
| *** leseb has joined #openstack-neutron | 09:02 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Implement rootwrap daemon support https://review.openstack.org/84667 | 09:02 |
| *** leseb_ has joined #openstack-neutron | 09:04 | |
| *** Longgeek_ has joined #openstack-neutron | 09:04 | |
| *** leseb has quit IRC | 09:07 | |
| *** Longgeek has quit IRC | 09:07 | |
| *** flwang has quit IRC | 09:11 | |
| *** jp_at_hp has joined #openstack-neutron | 09:12 | |
| *** ramishra has joined #openstack-neutron | 09:13 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Implement rootwrap daemon support https://review.openstack.org/84667 | 09:16 |
| *** singhs has quit IRC | 09:20 | |
| *** WackoRobie has joined #openstack-neutron | 09:21 | |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas delete and re-add same vip will make the vip can't work https://review.openstack.org/84647 | 09:21 |
| *** dvorkinista has joined #openstack-neutron | 09:21 | |
| *** yamahata has quit IRC | 09:24 | |
| *** saju_m has quit IRC | 09:24 | |
| *** WackoRobie has quit IRC | 09:25 | |
| *** saju_m has joined #openstack-neutron | 09:25 | |
| *** dvorkinista has quit IRC | 09:26 | |
| ajo | arosen, salv-orlando, amotoki if you had a moment, please take a look at: https://review.openstack.org/#/c/80261/ , it fixes the netns-cleanup script which is broken | 09:26 |
| openstackgerrit | Liping Mao proposed a change to openstack/neutron: lbaas delete and re-add same vip will make the vip can't work https://review.openstack.org/84647 | 09:28 |
| ajo | also, this one is built on top of the other: and fixes a few testing issues: https://review.openstack.org/#/c/84139/ | 09:28 |
| *** saju_m has quit IRC | 09:31 | |
| *** yamamoto has joined #openstack-neutron | 09:33 | |
| *** alexpilotti has joined #openstack-neutron | 09:34 | |
| *** tomoe__ has quit IRC | 09:34 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Poll the agent child processes status (WIP) https://review.openstack.org/74179 | 09:36 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: Provide agent status via status file (WIP) https://review.openstack.org/74045 | 09:37 |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Abstract out root_helper calls to classes https://review.openstack.org/82787 | 09:39 |
| *** yamamoto has quit IRC | 09:39 | |
| *** saju_m has joined #openstack-neutron | 09:39 | |
| *** networkstatic is now known as networkstatic_zZ | 09:47 | |
| *** tomoe_ has joined #openstack-neutron | 09:47 | |
| *** matsuhashi has quit IRC | 09:50 | |
| *** matsuhashi has joined #openstack-neutron | 09:57 | |
| *** tomoe_ has quit IRC | 09:58 | |
| *** flwang has joined #openstack-neutron | 09:59 | |
| *** tomoe_ has joined #openstack-neutron | 09:59 | |
| *** arosen has quit IRC | 10:00 | |
| *** arosen has joined #openstack-neutron | 10:01 | |
| *** leseb_ has quit IRC | 10:03 | |
| *** leseb has joined #openstack-neutron | 10:04 | |
| *** jroovers has quit IRC | 10:04 | |
| *** tomoe_ has quit IRC | 10:04 | |
| *** jroovers|afk has joined #openstack-neutron | 10:04 | |
| *** tomoe__ has joined #openstack-neutron | 10:04 | |
| openstackgerrit | A change was merged to openstack/neutron: Stop using portbindings_db in BSN ML2 driver https://review.openstack.org/84377 | 10:06 |
| *** leseb has quit IRC | 10:08 | |
| *** chandankumar_ has quit IRC | 10:11 | |
| *** pcm_ has joined #openstack-neutron | 10:11 | |
| openstackgerrit | A change was merged to openstack/neutron: Return 409 for second firewall creation https://review.openstack.org/84607 | 10:13 |
| *** pcm_ has quit IRC | 10:13 | |
| *** pcm_ has joined #openstack-neutron | 10:13 | |
| *** tomoe__ has quit IRC | 10:14 | |
| *** xuhanp has quit IRC | 10:16 | |
| *** evgenyf has quit IRC | 10:17 | |
| *** leseb has joined #openstack-neutron | 10:18 | |
| *** jroovers has joined #openstack-neutron | 10:18 | |
| *** jamespage has joined #openstack-neutron | 10:21 | |
| *** WackoRobie has joined #openstack-neutron | 10:21 | |
| *** jroovers|afk has quit IRC | 10:22 | |
| *** leseb has quit IRC | 10:22 | |
| *** tomoe_ has joined #openstack-neutron | 10:24 | |
| *** WackoRobie has quit IRC | 10:26 | |
| *** jgallard has quit IRC | 10:27 | |
| *** RockKuo_iPad has joined #openstack-neutron | 10:29 | |
| *** garyk has joined #openstack-neutron | 10:29 | |
| *** RockKuo_iPad has quit IRC | 10:30 | |
| *** RockKuo_iPad has joined #openstack-neutron | 10:30 | |
| *** RockKuo_iPad has quit IRC | 10:32 | |
| *** RockKuo_iPad has joined #openstack-neutron | 10:33 | |
| *** evgenyf has joined #openstack-neutron | 10:33 | |
| *** leseb has joined #openstack-neutron | 10:34 | |
| *** leseb_ has joined #openstack-neutron | 10:36 | |
| *** leseb has quit IRC | 10:38 | |
| *** leseb_ has quit IRC | 10:41 | |
| *** RockKuo_iPad has quit IRC | 10:42 | |
| *** amuller has joined #openstack-neutron | 10:56 | |
| *** dguitarbite has joined #openstack-neutron | 10:57 | |
| openstackgerrit | A change was merged to openstack/neutron: Update BigSwitch Name to its correct name https://review.openstack.org/83855 | 10:57 |
| ajo | arosen, salv-orlando, amotoki do you think it would be possible to approve https://review.openstack.org/#/c/80261/ , (netns-cleanup script fix), I was planning to backport the fix to havana, which has the same problem too. | 10:59 |
| ajo | ? :) | 10:59 |
| amotoki | ajo: will look. | 11:00 |
| ajo | thank you very much amotoki | 11:00 |
| *** matsuhashi has quit IRC | 11:00 | |
| *** tomoe_ has quit IRC | 11:07 | |
| *** tomoe_ has joined #openstack-neutron | 11:08 | |
| *** openstackgerrit has quit IRC | 11:08 | |
| *** amuller has quit IRC | 11:08 | |
| *** openstackgerrit has joined #openstack-neutron | 11:09 | |
| *** Longgeek_ has quit IRC | 11:10 | |
| *** leseb has joined #openstack-neutron | 11:11 | |
| *** tomoe_ has quit IRC | 11:12 | |
| *** amuller has joined #openstack-neutron | 11:13 | |
| amotoki | ajo: i think it is not a good idea to call rpc method release_dhcp_port. IMO it should be faked. | 11:13 |
| *** leseb has quit IRC | 11:15 | |
| ajo | amotoki: that rpc_method goes to the dhcp agent, right? | 11:15 |
| ajo | hmm, you mean, for example, not calling the method if plugin is unavailable? | 11:15 |
| amotoki | ajo: netns_cleanup is a standalone tool and it needs to work locally (without any communication). | 11:16 |
| *** WackoRobie has joined #openstack-neutron | 11:16 | |
| ajo | yes, when the communication it's not available it just catches the exception, and goes on | 11:16 |
| *** matsuhashi has joined #openstack-neutron | 11:16 | |
| *** julim has joined #openstack-neutron | 11:16 | |
| ajo | but I'd need to recheck that, | 11:16 |
| ajo | or, as you say, fake it out | 11:17 |
| amotoki | ajo: In your patch, DhcpPluginApi is instantiated, so I think release_dhcp_port RPC method is called. | 11:17 |
| ajo | amotoki, you're right, yes | 11:17 |
| ajo | I think that was the ultimate reason to instantiate it, | 11:17 |
| ajo | because the release_dhcp_port was called on a "None" plugin, breaking up execution | 11:18 |
| ajo | but if that's not needed for a proper cleanup, we can just skip the call on "not plugin" | 11:18 |
| *** tomoe_ has joined #openstack-neutron | 11:18 | |
| openstackgerrit | Henry Gessau proposed a change to openstack/neutron: Clean up unused CONF.state_path overrides in tests https://review.openstack.org/84608 | 11:19 |
| *** WackoRobie has quit IRC | 11:21 | |
| ajo | amotoki, thanks | 11:21 |
| amotoki | ajo: I think we are trying to cleanup a device (i.e., unplug). right? | 11:21 |
| ajo | yes, I'm checking that | 11:21 |
| ajo | amotoki, you mean, in the release_dhcp_port call, right? | 11:21 |
| amotoki | ajo: no. what I mean is self.driver.unplug just before self.plugin.release_dhcp_port. | 11:22 |
| amotoki | ajo: in DeviceManager.destroy(). | 11:22 |
| ajo | yes | 11:23 |
| ajo | hmmm, I'm trying to understand the exact logic of the destroy call... | 11:25 |
| *** [1]evgenyf has joined #openstack-neutron | 11:25 | |
| *** WackoRobie has joined #openstack-neutron | 11:26 | |
| *** evgenyf has quit IRC | 11:27 | |
| *** [1]evgenyf is now known as evgenyf | 11:27 | |
| *** overlayer has joined #openstack-neutron | 11:29 | |
| ajo | amotoki, I have to leave for a while, but I'll review the logic of all this to check if it's safe to fake that call out and what's it's exact purpose | 11:29 |
| amotoki | ajo: Faking DhcpPluginApi looks straight-forward approach to me. thanks. | 11:29 |
| *** overlayer has quit IRC | 11:32 | |
| *** tomoe_ has quit IRC | 11:35 | |
| *** tomoe_ has joined #openstack-neutron | 11:36 | |
| *** xianghui has quit IRC | 11:36 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Set correct columns' length https://review.openstack.org/80539 | 11:39 |
| *** tomoe_ has quit IRC | 11:40 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Set correct nullable parameter for columns https://review.openstack.org/82089 | 11:44 |
| *** leseb has joined #openstack-neutron | 11:44 | |
| *** amotoki has quit IRC | 11:56 | |
| openstackgerrit | Berezovsky Irena proposed a change to openstack/neutron: Implements ML2 mechanism driver for SR-IOV capable NIC based switching https://review.openstack.org/74464 | 11:57 |
| *** tomoe_ has joined #openstack-neutron | 11:58 | |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Remove duplicated tests for check_ovs_vxlan_version https://review.openstack.org/84703 | 12:03 |
| *** tomoe_ has quit IRC | 12:03 | |
| *** tomoe_ has joined #openstack-neutron | 12:03 | |
| *** mengxd has joined #openstack-neutron | 12:04 | |
| *** markvoelker has joined #openstack-neutron | 12:05 | |
| *** tomoe_ has quit IRC | 12:08 | |
| *** jecarey has quit IRC | 12:10 | |
| *** Longgeek has joined #openstack-neutron | 12:10 | |
| *** overlayer has joined #openstack-neutron | 12:11 | |
| *** dims_ has quit IRC | 12:13 | |
| *** Longgeek has quit IRC | 12:16 | |
| *** Longgeek has joined #openstack-neutron | 12:16 | |
| *** saju_m has quit IRC | 12:17 | |
| *** Matt2 has quit IRC | 12:17 | |
| *** jistr is now known as jistr|english | 12:18 | |
| *** luqas has quit IRC | 12:25 | |
| *** [1]evgenyf has joined #openstack-neutron | 12:25 | |
| *** dims_ has joined #openstack-neutron | 12:27 | |
| *** evgenyf has quit IRC | 12:27 | |
| *** [1]evgenyf is now known as evgenyf | 12:27 | |
| ajo | v | 12:28 |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: Validate maximum length for string fields https://review.openstack.org/84708 | 12:31 |
| *** luqas has joined #openstack-neutron | 12:32 | |
| *** chandankumar_ has joined #openstack-neutron | 12:38 | |
| *** xuhanp has joined #openstack-neutron | 12:40 | |
| *** ramishra has quit IRC | 12:42 | |
| *** b3nt_pin has joined #openstack-neutron | 12:45 | |
| *** baoli has quit IRC | 12:45 | |
| *** b3nt_pin is now known as beagles | 12:45 | |
| *** baoli has joined #openstack-neutron | 12:46 | |
| *** saju_m has joined #openstack-neutron | 12:47 | |
| *** sphoorti has joined #openstack-neutron | 12:50 | |
| openstackgerrit | Henry Gessau proposed a change to openstack/neutron: Clean up unused CONF.state_path overrides in tests https://review.openstack.org/84608 | 12:50 |
| ihrachys | arosen: can I ask you to check https://review.openstack.org/#/c/84586/ ? this is blocking successful gate checks in stable neutron | 12:50 |
| *** chandankumar_ has quit IRC | 12:51 | |
| *** jecarey has joined #openstack-neutron | 12:53 | |
| ekarlso | hmmm, I'm using GRE and the Instance has MTU 1500 and it seems like packets are dropping due to it. Should one use a bigger MTU on the GRE interfaces or so ? | 12:53 |
| *** thuc has joined #openstack-neutron | 12:54 | |
| *** jprovazn has joined #openstack-neutron | 12:55 | |
| *** thuc_ has joined #openstack-neutron | 12:55 | |
| *** chandan_kumar has quit IRC | 12:55 | |
| *** tomoe_ has joined #openstack-neutron | 12:56 | |
| amuller | ekarlso: Yes ideally you'd use jumbo frames on NICs transporting GRE | 12:57 |
| amuller | or at least 1520 MTU | 12:58 |
| *** thuc has quit IRC | 12:59 | |
| *** ygbo has quit IRC | 12:59 | |
| *** pradipta is now known as pradipta_away | 12:59 | |
| *** _afazekas has quit IRC | 13:01 | |
| *** yamahata has joined #openstack-neutron | 13:02 | |
| *** matsuhashi has quit IRC | 13:02 | |
| *** matsuhashi has joined #openstack-neutron | 13:02 | |
| *** yfried__ has quit IRC | 13:04 | |
| *** prad_ has joined #openstack-neutron | 13:04 | |
| *** matsuhashi has quit IRC | 13:07 | |
| *** matsuhashi has joined #openstack-neutron | 13:10 | |
| *** ygbo has joined #openstack-neutron | 13:11 | |
| *** Longgeek has quit IRC | 13:14 | |
| *** Longgeek has joined #openstack-neutron | 13:14 | |
| *** Longgeek_ has joined #openstack-neutron | 13:15 | |
| *** WackoRobie has quit IRC | 13:17 | |
| *** leseb has quit IRC | 13:17 | |
| *** WackoRobie has joined #openstack-neutron | 13:17 | |
| *** leseb has joined #openstack-neutron | 13:17 | |
| *** jgallard has joined #openstack-neutron | 13:18 | |
| *** Longgeek has quit IRC | 13:18 | |
| *** WackoRobie has quit IRC | 13:22 | |
| *** leseb has quit IRC | 13:22 | |
| openstackgerrit | Henry Gessau proposed a change to openstack/neutron: Remove auto-generation of db schema from models at startup https://review.openstack.org/40296 | 13:23 |
| *** leseb has joined #openstack-neutron | 13:25 | |
| *** tomoe_ has quit IRC | 13:25 | |
| *** tomoe_ has joined #openstack-neutron | 13:26 | |
| openstackgerrit | Henry Gessau proposed a change to openstack/neutron: WIP: add dependent plugin migrations https://review.openstack.org/84519 | 13:27 |
| *** tomoe_ has quit IRC | 13:31 | |
| *** RockKuo has joined #openstack-neutron | 13:32 | |
| *** BillTheKat has joined #openstack-neutron | 13:33 | |
| *** BillTheKat is now known as Gil_McGrath | 13:34 | |
| *** irenab has quit IRC | 13:34 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup https://review.openstack.org/80261 | 13:34 |
| *** saju_m has quit IRC | 13:36 | |
| ekarlso | ok so amuller I've done that now, MTU 9000 on all eth's that are GRE endpoints and also enabled it for the vlan that has GRE traffic | 13:37 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes broken neutron-netns-cleanup https://review.openstack.org/80261 | 13:37 |
| ekarlso | still I get funky MTU issues by the loos where as when I try to do a "curl" in a vm towards a local address which is a ubuntu mirror it just stops | 13:37 |
| ekarlso | if I lower the MTU on the instance though and try again it works | 13:37 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes tests using called_once_ without assert https://review.openstack.org/84139 | 13:43 |
| *** jistr|english is now known as jistr | 13:43 | |
| *** thuc_ has quit IRC | 13:44 | |
| *** thuc has joined #openstack-neutron | 13:44 | |
| *** chandan_kumar has joined #openstack-neutron | 13:49 | |
| *** thuc has quit IRC | 13:49 | |
| *** WackoRobie has joined #openstack-neutron | 13:58 | |
| *** WackoRobie has quit IRC | 13:59 | |
| *** jgrimm has joined #openstack-neutron | 13:59 | |
| *** WackoRobie has joined #openstack-neutron | 13:59 | |
| *** ramishra has joined #openstack-neutron | 14:00 | |
| sphoorti | anteaya: hello | 14:02 |
| sphoorti | anteaya: only the RHEL7 tests fail with 'Missing packages detected' error. should that be a concern for the entire patch ? | 14:06 |
| sphoorti | and Jenkins gave a +1 | 14:08 |
| *** tomoe_ has joined #openstack-neutron | 14:08 | |
| *** mengxd has quit IRC | 14:09 | |
| anteaya | sphoorti: hi | 14:14 |
| anteaya | what patch are we discussing? | 14:14 |
| anteaya | can you link me please? | 14:15 |
| anteaya | sphoorti: are you talking about https://review.openstack.org/#/c/84452/ | 14:19 |
| anteaya | Jenkins reports the results of our internal testing system | 14:20 |
| anteaya | Red Hat CI is an external 3rd party testing system: https://review.openstack.org/#/admin/groups/91,members | 14:21 |
| anteaya | which unfortunately doesn't have an email contact address associated with it | 14:21 |
| *** tomoe_ has quit IRC | 14:21 | |
| *** tomoe_ has joined #openstack-neutron | 14:22 | |
| sphoorti | anteaya: so what should I do ? | 14:22 |
| anteaya | I'm trying to find out what the proper course of action is | 14:23 |
| *** otherwiseguy has joined #openstack-neutron | 14:23 | |
| amuller | ekarlso: the physical NICs and switches along the way need to support the MTU you configured | 14:24 |
| anteaya | if there was an email address I would encourage you to email the system in question and ask the very question you asked me | 14:24 |
| anteaya | apparently user ianw is the contact person for that account | 14:25 |
| sphoorti | okay :) . I followed your conversation on qa channel | 14:25 |
| anteaya | so I encourage you to contact that person and have them explain to you, so that you understand, what the results of their tests mean | 14:25 |
| anteaya | great | 14:25 |
| sphoorti | anteaya: should I ping him on the channel | 14:25 |
| anteaya | I have done so on both | 14:25 |
| anteaya | by typing his/her name | 14:25 |
| anteaya | so now we wait | 14:26 |
| sphoorti | okay | 14:26 |
| anteaya | if we have no response by tomorrow I will see if I can find an email address for this person | 14:26 |
| sphoorti | thank you anteaya | 14:26 |
| anteaya | the tl;dr is don't worry about it | 14:26 |
| anteaya | but I don't want to encourage the habit of people ignoring test results, since that fosters very bad habits | 14:27 |
| anteaya | you should care about test results | 14:27 |
| anteaya | but results providers should also provide results in a manner that makes caring about them reasonable | 14:27 |
| *** tomoe_ has quit IRC | 14:28 | |
| *** tomoe_ has joined #openstack-neutron | 14:28 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Fix incorrect change of Enum type https://review.openstack.org/84730 | 14:29 |
| sphoorti | sure anteaya | 14:29 |
| *** tomoe_ has quit IRC | 14:33 | |
| *** jprovazn is now known as jprovazn_afk | 14:37 | |
| *** tvardeman has joined #openstack-neutron | 14:39 | |
| *** chandankumar_ has joined #openstack-neutron | 14:40 | |
| *** coolsvap has joined #openstack-neutron | 14:49 | |
| *** carl_baldwin has joined #openstack-neutron | 14:49 | |
| *** thedodd has joined #openstack-neutron | 14:50 | |
| *** thuc_ has joined #openstack-neutron | 14:51 | |
| *** otherwiseguy has quit IRC | 14:51 | |
| *** matsuhashi has quit IRC | 14:52 | |
| *** doddstack has joined #openstack-neutron | 14:54 | |
| *** thuc has joined #openstack-neutron | 14:57 | |
| *** thedodd has quit IRC | 14:57 | |
| *** thuc_ has quit IRC | 14:58 | |
| mtreinish | salv-orlando: want to do me a favor and push this through?: https://review.openstack.org/#/c/84258/ it should be an easy win | 14:59 |
| *** Matt2 has joined #openstack-neutron | 14:59 | |
| *** Sukhdev has joined #openstack-neutron | 15:00 | |
| *** Longgeek has joined #openstack-neutron | 15:00 | |
| *** networkstatic_zZ has quit IRC | 15:01 | |
| *** yfried__ has joined #openstack-neutron | 15:01 | |
| *** banix has joined #openstack-neutron | 15:02 | |
| salv-orlando | done | 15:02 |
| mtreinish | salv-orlando: thanks, now I can vote :) | 15:03 |
| *** [1]evgenyf has joined #openstack-neutron | 15:03 | |
| *** Longgeek_ has quit IRC | 15:04 | |
| salv-orlando | mtreinish: np. always happy to hand out electoral cards | 15:05 |
| *** _TheDodd_ has joined #openstack-neutron | 15:05 | |
| *** evgenyf has quit IRC | 15:05 | |
| *** [1]evgenyf is now known as evgenyf | 15:05 | |
| *** doddstack has quit IRC | 15:07 | |
| *** garyk has quit IRC | 15:09 | |
| *** sphoorti_ has joined #openstack-neutron | 15:11 | |
| *** dvorkinista has joined #openstack-neutron | 15:12 | |
| roaet | sorry to bother people, but can I get eyes on https://review.openstack.org/#/c/57517/ need core. and yes, it is python-neutronclient. | 15:12 |
| *** skraynev is now known as skraynev_afk | 15:13 | |
| *** otherwiseguy has joined #openstack-neutron | 15:13 | |
| *** hrushig has joined #openstack-neutron | 15:13 | |
| *** thuc has quit IRC | 15:15 | |
| *** sphoorti has quit IRC | 15:15 | |
| *** jaypipes has joined #openstack-neutron | 15:15 | |
| *** thuc has joined #openstack-neutron | 15:15 | |
| *** jorgem has joined #openstack-neutron | 15:18 | |
| *** thuc has quit IRC | 15:20 | |
| *** openstackgerrit has quit IRC | 15:21 | |
| *** openstackgerrit has joined #openstack-neutron | 15:22 | |
| *** WackoRob_ has joined #openstack-neutron | 15:23 | |
| *** WackoRob_ has quit IRC | 15:23 | |
| *** networkstatic has joined #openstack-neutron | 15:24 | |
| *** WackoRob_ has joined #openstack-neutron | 15:24 | |
| *** SumitNaiksatam has joined #openstack-neutron | 15:26 | |
| *** WackoRobie has quit IRC | 15:27 | |
| *** leseb has quit IRC | 15:28 | |
| *** WackoRob_ has quit IRC | 15:28 | |
| *** leseb has joined #openstack-neutron | 15:28 | |
| *** jorgem1 has joined #openstack-neutron | 15:29 | |
| *** alagalah has joined #openstack-neutron | 15:31 | |
| *** amotoki has joined #openstack-neutron | 15:32 | |
| *** jorgem has quit IRC | 15:32 | |
| *** leseb has quit IRC | 15:33 | |
| *** WackoRobie has joined #openstack-neutron | 15:35 | |
| mestery | rkukura: This was just reported, I'm looking at it now, but it looks serious around ML2+ODL: https://bugs.launchpad.net/neutron/+bug/1301449 | 15:38 |
| pasquier-s | mestery, hi, i'm still here for a couple of minutes if you have questions ;) | 15:38 |
| mestery | pasquier-s: Hey, glad to see you here! | 15:39 |
| mestery | pasquier-s: I'll try to reproduce this today, but it appears you've found a valid bug here which we likely want to fix for Icehouse even. | 15:39 |
| mestery | pasquier-s: Thanks for reporting this! | 15:39 |
| pasquier-s | mestery, np | 15:39 |
| *** leseb has joined #openstack-neutron | 15:40 | |
| *** WackoRobie has quit IRC | 15:40 | |
| *** manishg has joined #openstack-neutron | 15:41 | |
| amotoki | mestery: indeed. it looks high priority. | 15:43 |
| mestery | amotoki: Yes, I'm bringing up the latest in my multi-node ODL setup to verify it's there, if it is, I may mark it critical in fact. | 15:43 |
| *** mlavalle has joined #openstack-neutron | 15:43 | |
| *** WackoRobie has joined #openstack-neutron | 15:44 | |
| *** jlibosva has quit IRC | 15:44 | |
| Sukhdev | mestery: Hi Kyle | 15:47 |
| amotoki | i haven't tried ODL after event-notification was implemented .... If ODL detects port plugging immediately, I think we can set the initial port status to ACTIVE. | 15:48 |
| *** WackoRob_ has joined #openstack-neutron | 15:50 | |
| mestery | amotoki: I agree on that front | 15:50 |
| mestery | Hi Sukhdev | 15:50 |
| *** WackoRob_ has quit IRC | 15:51 | |
| andyhky | mestery: is the required config there? couldn't tell from the paste. https://review.openstack.org/#/c/75253/29/etc/neutron.conf | 15:51 |
| Sukhdev | I saw your comment on https://review.openstack.org/#/c/84603/ and fixed the CI issue | 15:51 |
| *** WackoRobie has quit IRC | 15:51 | |
| mestery | Sukhdev: Cool, thanks! | 15:51 |
| Sukhdev | mestery: can you have another look? | 15:51 |
| mestery | andyhky: You mean on the nova side from devstack? Unsure. | 15:51 |
| andyhky | neutron side from devstack | 15:52 |
| mestery | Got it. That's a good question, pasquier-s, can you confir,? | 15:52 |
| *** WackoRobie has joined #openstack-neutron | 15:53 | |
| pasquier-s | mestery, checking | 15:53 |
| pasquier-s | mestery, yes, I have "notify_nova_on_port_status_change = True" and other parameters in my neutron.conf | 15:55 |
| *** devvesa has quit IRC | 15:55 | |
| mestery | thanks for confirming | 15:55 |
| mestery | I have to run the ML2 meeting in 4 minutes, so I won't be able to look at this in depth until after that is complete. | 15:56 |
| mestery | I'll get back to you after that pasquier-s. | 15:56 |
| pasquier-s | mestery, I will be gone but you can ping me tomorrow, thks | 15:56 |
| *** thuc has joined #openstack-neutron | 15:57 | |
| mestery | Will do pasquier-s | 15:57 |
| *** xuhanp has quit IRC | 15:59 | |
| *** Longgeek has quit IRC | 15:59 | |
| *** rcurran has joined #openstack-neutron | 16:00 | |
| *** irenab has joined #openstack-neutron | 16:02 | |
| carl_baldwin | marun: ping | 16:04 |
| *** pasquier-s has quit IRC | 16:04 | |
| *** yamahata has quit IRC | 16:06 | |
| *** armax has joined #openstack-neutron | 16:07 | |
| *** coolsvap has quit IRC | 16:11 | |
| *** dvorkinista has quit IRC | 16:12 | |
| *** catohornet has joined #openstack-neutron | 16:13 | |
| *** jgallard has quit IRC | 16:14 | |
| *** sbfox has joined #openstack-neutron | 16:14 | |
| *** coolsvap has joined #openstack-neutron | 16:17 | |
| *** jpich has quit IRC | 16:18 | |
| *** coolsvap has quit IRC | 16:18 | |
| *** amuller has quit IRC | 16:20 | |
| *** chandankumar_ has quit IRC | 16:20 | |
| *** dvorkinista has joined #openstack-neutron | 16:20 | |
| *** garyk has joined #openstack-neutron | 16:21 | |
| *** arosen1 has joined #openstack-neutron | 16:21 | |
| *** bashok has quit IRC | 16:23 | |
| *** WackoRobie has quit IRC | 16:23 | |
| *** sbalukoff has quit IRC | 16:26 | |
| *** [1]evgenyf has joined #openstack-neutron | 16:26 | |
| *** [2]evgenyf has joined #openstack-neutron | 16:27 | |
| *** jobewan has joined #openstack-neutron | 16:27 | |
| *** evgenyf has quit IRC | 16:28 | |
| *** [2]evgenyf is now known as evgenyf | 16:28 | |
| openstackgerrit | A change was merged to openstack/neutron: Only send notifications on uuid device_id's https://review.openstack.org/78533 | 16:29 |
| *** leseb has quit IRC | 16:30 | |
| openstackgerrit | Manish Godara proposed a change to openstack/neutron: update doc string - correct typo https://review.openstack.org/84785 | 16:30 |
| *** [1]evgenyf has quit IRC | 16:30 | |
| *** ramishra has quit IRC | 16:31 | |
| *** leseb has joined #openstack-neutron | 16:31 | |
| *** ygbo has quit IRC | 16:31 | |
| *** jistr has quit IRC | 16:33 | |
| *** WackoRobie has joined #openstack-neutron | 16:33 | |
| *** luqas has quit IRC | 16:34 | |
| *** leseb has quit IRC | 16:35 | |
| *** rand738 has quit IRC | 16:43 | |
| *** markmcclain has joined #openstack-neutron | 16:45 | |
| marun | carl_baldwin: pong | 16:45 |
| carl_baldwin | marun: Hi, checking in on that bug fix. Did you read Mike's comments? | 16:46 |
| marun | carl_baldwin: not yet | 16:46 |
| marun | carl_baldwin: just getting started | 16:46 |
| *** dvorkinista has quit IRC | 16:46 | |
| carl_baldwin | I thought that the notifier he mentions was running in the api worker process but it wasn't. So, he makes a good and valid point. | 16:46 |
| marun | carl_baldwin: I don't get what he's talking about | 16:47 |
| carl_baldwin | It may be a little complicated to fix his case in the same way as I was trying to fix the first case. | 16:47 |
| marun | carl_baldwin: we need a notifier per child | 16:47 |
| *** arosen1 has quit IRC | 16:47 | |
| carl_baldwin | marun: Right, we do need the notifier to work in all processes. | 16:47 |
| marun | carl_baldwin: why does the controller get created before the fork? | 16:47 |
| carl_baldwin | His comment about a singleton gave me another idea which will fix the problem with changes localized to the notifier. | 16:48 |
| carl_baldwin | marun: The controller gets created when the paste app is loaded. | 16:48 |
| marun | carl_baldwin: *sigh* | 16:48 |
| *** singhs has joined #openstack-neutron | 16:48 | |
| marun | carl_baldwin: Would lazy-loading be sufficient then? | 16:49 |
| marun | carl_baldwin: to ensure that it isn't created until post-fork? | 16:49 |
| *** safchain has quit IRC | 16:50 | |
| carl_baldwin | An idea I'm playing with this morning is to replace the persistent "looping call" thread with another mechanism. I'll put a quick draft up for you to look at. | 16:50 |
| *** digambar_ has joined #openstack-neutron | 16:51 | |
| digambar_ | Hello stackers | 16:52 |
| digambar_ | working on https://bugs.launchpad.net/neutron/+bug/1274682 | 16:53 |
| digambar_ | bug | 16:53 |
| marun | markmcclain, arosen: Am I correct in remembering that you two are our representatives in stable core? | 16:54 |
| marun | carl_baldwin: ok | 16:54 |
| markmcclain | marun: we have 3, arosen, garyk, and me | 16:54 |
| marun | markmcclain: garyk isn't terribly active in neutron these days | 16:54 |
| marun | markmcclain: we've had the polling minimization patches sitting in review for 2+ months | 16:55 |
| markmcclain | from what I understand he'll be back in Juno | 16:55 |
| marun | markmcclain: is there a reason they haven't been merged yet? | 16:55 |
| *** rand738 has joined #openstack-neutron | 16:55 | |
| *** dvorkinista has joined #openstack-neutron | 16:55 | |
| marun | markmcclain: they just missed the latest stable release | 16:55 |
| *** sandr8 has joined #openstack-neutron | 16:55 | |
| *** jroovers has quit IRC | 16:56 | |
| *** singhs has quit IRC | 16:57 | |
| roaet | markmcclain: howdy. I know you're busy and stuff. but since you reviewed https://review.openstack.org/#/c/57517/ earlier, could you possibly approve it? It's been awhile since implementation. | 16:57 |
| markmcclain | yeah we had problems getting patches merged due to the frequent random faults neutron stable testing | 16:57 |
| markmcclain | roaet: I'll add it to my queue | 16:58 |
| garyk | marun: markmcclain: neutron and gu is in my blood :) | 16:58 |
| roaet | thank you. I appreciate it greatly. | 16:58 |
| roaet | It is a client change, tis quite small though. | 16:58 |
| marun | markmcclain: that could be part of the reason, but the patches haven't been looked at for a month let alone tested. | 16:58 |
| *** singhs has joined #openstack-neutron | 16:58 | |
| *** csd has joined #openstack-neutron | 16:58 | |
| *** evgenyf has quit IRC | 16:58 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 16:59 |
| marun | markmcclain: I think patches that are being backported are likely to be important enough to deserve more attention. | 16:59 |
| *** rcurran has quit IRC | 16:59 | |
| marun | markmcclain: otherwise distro maintainers end up having to do maintain local backports indefinitely, and there is way more effort involved. | 16:59 |
| *** Sukhdev has quit IRC | 17:00 | |
| markmcclain | markmcclain: understood sorry they got lost in the shuffle with I-3/RC1 | 17:00 |
| amotoki | unfortunately stable release freeze is always just after milestone cut in master branch :-( | 17:01 |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 17:01 |
| marun | markmcclain: fair enough, just want to make sure we take steps to improve stable maint next cycle. | 17:01 |
| *** sandr8 has quit IRC | 17:02 | |
| *** sandr8 has joined #openstack-neutron | 17:02 | |
| *** otherwiseguy has quit IRC | 17:02 | |
| *** leseb has joined #openstack-neutron | 17:02 | |
| marun | carl_baldwin: should that patch be WIP given that it has no tests? | 17:02 |
| markmcclain | amotoki: yeah maybe we'll be able to adjust the schedule a bit for Icehouse stable branch schedule | 17:03 |
| carl_baldwin | marun: Yes, I had marked ps 1 as WIP but hadn't yet marked ps2 | 17:03 |
| marun | arosen: regarding https://review.openstack.org/#/c/84798/ | 17:04 |
| carl_baldwin | marun: It is a quick draft and I've barely tested it but it seems to get the job done. Mostly I want your feedback on the high level direction of the patch. | 17:04 |
| marun | arosen: do you think it makes sense to batch or maybe just switch to sending async? | 17:04 |
| carl_baldwin | marun: I've got to step out for a bit but please provide your feedback. We may not have found the right answer yet but I think we're getting closer. | 17:05 |
| *** tomoe_ has joined #openstack-neutron | 17:05 | |
| marun | carl_baldwin: I agree, this is a step in the right direction. | 17:05 |
| marun | carl_baldwin: I still wonder about the Controller instantiation, though. I'm wondering if we could simply avoid that pre-fork. | 17:06 |
| amotoki | banix: around? can resume your topic about vif_detail in ml2 meeting? I am still not sure what is needed. | 17:08 |
| carl_baldwin | marun: It wasn't obvious how to easily avoid it when I was looking at it. But, I will take a little more time with it. | 17:08 |
| banix | amotoki: yes hi | 17:08 |
| carl_baldwin | marun: talk to you in a bit. | 17:08 |
| banix | amotoki: Yes, let me explain | 17:08 |
| amotoki | banix: thanks. | 17:08 |
| banix | amotoki: This is rather a limitted use case so may not be of great importance. Having said that, let me explain: icehouse is the first release where we have our plugin included. | 17:10 |
| banix | amotoki: So we can use this plugin code easily with Havana as well; The only issue being the plugin code refers to VIF_DETAILS | 17:10 |
| *** nlahouti has joined #openstack-neutron | 17:10 | |
| *** roeyc has quit IRC | 17:10 | |
| amotoki | banix: my point is that binding:capabilities is not actually used in nova side. | 17:11 |
| amotoki | banix: so i think your plugin will work even if vif_details is included in a response. | 17:12 |
| banix | amotoki: sure but if we want to use our code unchanged there is a reference in our code to something that is not defined in havana. | 17:12 |
| banix | amotoki: does that make sense? | 17:13 |
| amotoki | banix: ah..perhaps i got your point. your plugin in icehouse refers something not in havana to generate vif_details. right? | 17:13 |
| *** Sukhdev has joined #openstack-neutron | 17:14 | |
| banix | amotoki: yes, that's it. I know this is not an issue of concern but was wondering what is the best way to deal with this. | 17:14 |
| banix | amotoki: hence, wondering if having vif_details just defined as capabilities and backporting it is a reasonable or viable/advisable solution. | 17:16 |
| *** tomoe_ has quit IRC | 17:17 | |
| amotoki | banix: but i am not sure why we need to backport for unused field.. | 17:17 |
| *** tomoe_ has joined #openstack-neutron | 17:17 | |
| amotoki | banix: basically API change is not allowed in stable backport. | 17:19 |
| amotoki | banix: https://wiki.openstack.org/wiki/StableBranch#Appropriate_Fixes is a good description what fixes are accepted. | 17:19 |
| banix | amotoki: Yeah, the only purpose would be having Icehouse plugin working without any changes in Havana. I realize this may not be a reasonable option | 17:19 |
| banix | yes makes sense | 17:19 |
| amotoki | banix: neutron core havana with your plugin icehouse? | 17:20 |
| banix | amotoki: yes | 17:20 |
| banix | amotoki: which may not be something that is generally done to begin with | 17:21 |
| *** dvorkinista has quit IRC | 17:21 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Fix up usage of state_path mocking in unit tests https://review.openstack.org/84620 | 17:21 |
| *** tomoe_ has quit IRC | 17:22 | |
| banix | amotoki: thanks for the pointer and the explanation | 17:22 |
| amotoki | banix: i think one possible way is not to generate binding:capability. this is all in my mind now. | 17:23 |
| amotoki | andyhky: you're welcome. | 17:23 |
| *** morganfainberg_Z is now known as morganfainberg | 17:23 | |
| andyhky | what? | 17:24 |
| banix | andyhky: think that was meant to be sent to me. | 17:24 |
| andyhky | ahh, tab completion | 17:25 |
| banix | miss a b in banix and you are andyhky | 17:25 |
| *** sandr8 has quit IRC | 17:25 | |
| amotoki | andyhky: sorry... | 17:25 |
| *** sandr8 has joined #openstack-neutron | 17:25 | |
| *** thuc_ has joined #openstack-neutron | 17:27 | |
| *** armitage81 has joined #openstack-neutron | 17:27 | |
| *** armitage81 has quit IRC | 17:28 | |
| *** armitage81 has joined #openstack-neutron | 17:28 | |
| *** amotoki has quit IRC | 17:28 | |
| *** sbfox has quit IRC | 17:28 | |
| *** digambar_ has quit IRC | 17:29 | |
| *** jorgem has joined #openstack-neutron | 17:29 | |
| *** WackoRob_ has joined #openstack-neutron | 17:29 | |
| *** sandr8_ has joined #openstack-neutron | 17:30 | |
| *** sbfox has joined #openstack-neutron | 17:30 | |
| *** networks_ has joined #openstack-neutron | 17:30 | |
| openstackgerrit | Thomas Goirand proposed a change to openstack/neutron: Better defaults in config files https://review.openstack.org/84809 | 17:30 |
| *** SumitNaiksatam_ has joined #openstack-neutron | 17:30 | |
| *** thuc has quit IRC | 17:31 | |
| *** markmcclain has quit IRC | 17:31 | |
| *** markmcclain has joined #openstack-neutron | 17:33 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: NSX: Ensure that DynamicLoop for sync doesn't raise https://review.openstack.org/84523 | 17:34 |
| *** harlowja_away is now known as harlowja | 17:35 | |
| *** evgenyf has joined #openstack-neutron | 17:35 | |
| *** jprovazn_afk is now known as jprovazn | 17:37 | |
| *** sandr8 has quit IRC | 17:38 | |
| *** WackoRobie has quit IRC | 17:38 | |
| *** jorgem1 has quit IRC | 17:38 | |
| *** SumitNaiksatam has quit IRC | 17:38 | |
| *** networkstatic has quit IRC | 17:38 | |
| *** sphoorti_ has quit IRC | 17:38 | |
| *** matrohon has quit IRC | 17:38 | |
| *** tchaypo has quit IRC | 17:38 | |
| *** aryan has quit IRC | 17:38 | |
| *** SumitNaiksatam_ is now known as SumitNaiksatam | 17:38 | |
| pcm_ | Can I get a second core review on https://review.openstack.org/#/c/82306/2 please? | 17:40 |
| *** aryan has joined #openstack-neutron | 17:44 | |
| *** tchaypo has joined #openstack-neutron | 17:44 | |
| *** sphoorti_ has joined #openstack-neutron | 17:45 | |
| *** sbalukoff has joined #openstack-neutron | 17:47 | |
| *** matrohon has joined #openstack-neutron | 17:47 | |
| *** dvorkinista has joined #openstack-neutron | 17:48 | |
| *** jroovers has joined #openstack-neutron | 17:48 | |
| *** jroovers|afk has joined #openstack-neutron | 17:50 | |
| *** bjornar has joined #openstack-neutron | 17:51 | |
| *** yfried__ has quit IRC | 17:52 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack/neutron: Abstract out root_helper calls to classes https://review.openstack.org/82787 | 17:52 |
| *** jroovers has quit IRC | 17:53 | |
| *** mlavalle has quit IRC | 17:53 | |
| *** _TheDodd_ has quit IRC | 17:55 | |
| *** leseb has quit IRC | 17:55 | |
| *** irenab has quit IRC | 17:57 | |
| *** itzikb has joined #openstack-neutron | 18:02 | |
| *** mlavalle has joined #openstack-neutron | 18:06 | |
| mestery | marun: Here? | 18:06 |
| *** vilobhmm_ has joined #openstack-neutron | 18:07 | |
| *** leseb has joined #openstack-neutron | 18:07 | |
| mestery | marun: I'm refactoring the functional tests to share code, but even your existing ovsdb ones don't run for me. | 18:07 |
| mestery | marun: http://paste.openstack.org/show/74887/ <--- See this. | 18:07 |
| mestery | marun: The problem is that the tests are trying to use BaseOVS, but they really want and OVSBridge object. | 18:08 |
| mestery | marun: I think the reason you did this is to make it easier to get a non-conflicting random name, but they don't work as is. | 18:08 |
| mestery | marun: I'm going to make these work, FYI. | 18:08 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/neutron: Remove invalid copyright headers under API module https://review.openstack.org/84825 | 18:09 |
| pcm_ | mestery: ping | 18:10 |
| marun | mestery: If the tests don't work, you've changed them. | 18:10 |
| marun | mestery: they're running in the check queue just fine | 18:10 |
| *** vilobhmm___ has joined #openstack-neutron | 18:11 | |
| mestery | marun: Weird, I don't understand how they work, because you're calling functions on OVSBridge class, but your object is BAseOVS. | 18:11 |
| mestery | marun: Can you explain? | 18:11 |
| mestery | marun: Even with a fresh checkout they fail for me. | 18:11 |
| marun | mestery: really? | 18:11 |
| mestery | marun: Yes. | 18:11 |
| mestery | marun: You see my confusion now. This is perplexing how they can run in the gate, I clearly must be missing something. :) | 18:11 |
| *** vilobhmm_ has quit IRC | 18:12 | |
| marun | mestery: their not gating yet :( | 18:12 |
| mestery | pcm: Hang in there, juggling 5 things at the moment ;) | 18:12 |
| marun | mestery: they're | 18:12 |
| mestery | marun: But how can they even run? | 18:12 |
| mestery | marun: I'm not following something here. | 18:12 |
| * mestery needs some help or he's losing his mind. :) | 18:13 | |
| *** dguitarbite has quit IRC | 18:13 | |
| * pcm_ sure... taking a ticket :) | 18:13 | |
| marun | mestery: The fact that they aren't gating means they're broken as of yesterday due to a state_path change | 18:13 |
| *** dguitarbite has joined #openstack-neutron | 18:13 | |
| * marun shakes fist at jogo | 18:13 | |
| mestery | marun: Wow! | 18:13 |
| marun | mestery: I'll try to see what you mean about the base class | 18:13 |
| mestery | marun: Please! This is driving me nuts ;) | 18:14 |
| marun | mestery: the BaseOVS is a red herring | 18:14 |
| jogo | marun: what d id I break? | 18:14 |
| mestery | marun: Please xplain | 18:14 |
| marun | jogo: not actually your fault, the functional job is non-voting right now so it can't block anything | 18:14 |
| jogo | ahh, and it uses crazy pants state_path logic? | 18:15 |
| marun | jogo: I'm just metaphorically frustrated at you, but not really | 18:15 |
| jogo | marun: heh I figured, just wanted to know for entertainment value | 18:15 |
| marun | jogo: It doesn't do anything tricky, but the base test class apparently does | 18:15 |
| * mestery is still confused. | 18:15 | |
| marun | jogo: http://logs.openstack.org/35/81535/4/check/check-neutron-dsvm-functional/e64521c/console.html | 18:15 |
| marun | jogo: I'm guessing your patches are actually going to clean this up, actually. | 18:16 |
| jogo | marun: at least partially | 18:16 |
| jogo | but it won't cover this case directly | 18:16 |
| mestery | marun: So does this explain why it's failing for me with the latest as well? Or something different? | 18:16 |
| marun | mestery: your error is different I think | 18:17 |
| mestery | marun: OK :) | 18:17 |
| marun | jogo: so long as the base test class starts doing things sanely the functional job should start working again | 18:17 |
| * mestery waits for an explanation of the Red Herring referenced earlier. | 18:17 | |
| marun | jogo: it doesn't use state path at all | 18:17 |
| marun | mestery: https://github.com/openstack/neutron/blob/master/neutron/tests/functional/agent/linux/test_ovsdb_monitor.py#L75 | 18:17 |
| jogo | marun: import_opt | 18:17 |
| *** vilobhmm___ has quit IRC | 18:17 | |
| marun | jogo: ? | 18:18 |
| jogo | I was afraid of this but didn't see it fail | 18:18 |
| jogo | will push a patch | 18:18 |
| mestery | marun: I see that. | 18:18 |
| marun | mestery: baseovs is only used to create the bridge | 18:18 |
| mestery | Yes, but add_bridge is not a member function of BaseOVS, it's a member function of OVSBridge. See my confusion? | 18:18 |
| marun | mestery: the object that gets created is OVSBridge | 18:19 |
| marun | mestery: there are two... | 18:19 |
| *** dkehn__ has joined #openstack-neutron | 18:19 | |
| marun | mestery: wait | 18:19 |
| * mestery waits. | 18:19 | |
| marun | mestery: since when? | 18:19 |
| mestery | marun: Since a long time? | 18:19 |
| mestery | Not sure. | 18:19 |
| mestery | But you see my problem now, right? | 18:19 |
| marun | mestery: I'm looking at code from yesterday, and add_bridge is on BaseOVS | 18:20 |
| *** dguitarbite has quit IRC | 18:20 | |
| marun | mestery: https://github.com/openstack/neutron/blob/master/neutron/agent/linux/ovs_lib.py#L76 | 18:20 |
| * mestery is now confused too. | 18:20 | |
| mestery | marun: Wait, add_port | 18:21 |
| marun | mestery: you're talking about add_port | 18:21 |
| mestery | and for me, add_tunnel_port | 18:21 |
| mestery | marun: yes | 18:21 |
| mestery | sorry for confusion | 18:21 |
| *** sandr8 has joined #openstack-neutron | 18:21 | |
| *** sandr8_ has quit IRC | 18:21 | |
| marun | mestery: uh | 18:21 |
| marun | mestery: the errors you pasted indicate that the bridge is None | 18:21 |
| mestery | marun: Exactly | 18:22 |
| marun | mestery: so of course it wouldn't have any attributes | 18:22 |
| marun | mestery: that isn't a problem in the ovsdb tests | 18:22 |
| mestery | Crap. So that failed then. :( | 18:22 |
| jogo | marun: trying to reproduce the bug locallly | 18:22 |
| marun | mestery: wait | 18:23 |
| marun | mestery: now I'm starting to see what you mean | 18:23 |
| mestery | marun: :P | 18:23 |
| marun | mestery: add_bridge doesn't return anything | 18:23 |
| jogo | in short functiaonal tests are started a different way that didn't load neutron/common/config.py: | 18:23 |
| mestery | marun: Yes. | 18:23 |
| jogo | which defines state_path | 18:23 |
| marun | mestery: Now I'm really confused | 18:23 |
| *** dkehn_ has quit IRC | 18:23 | |
| mestery | marun: There is no object returneD! | 18:23 |
| mestery | marun: How did this ever work? | 18:23 |
| marun | mestery: I really don't know. | 18:24 |
| mestery | marun: We need to use the OVSBridge class, which in fact means we get the OVSBridge object to make subsequent calls. | 18:24 |
| mestery | marun: OK | 18:24 |
| mestery | marun: I'll clean all this up then I guess :) | 18:24 |
| mestery | marun: I'm in this code deeply now anyways. | 18:24 |
| marun | mestery: separate patches, please. | 18:24 |
| mestery | marun: I have a first patch which makes this more modular, and one for the VXLAN test. | 18:24 |
| mestery | marun: Are you proposing a third? | 18:24 |
| mestery | marun: Or can I fold this cleanup into the first. | 18:24 |
| marun | mestery: there should be a fix for the ovsdb monitor tests | 18:25 |
| mestery | marun: OK, three patches then. :) | 18:25 |
| marun | mestery: separate from the others | 18:25 |
| mestery | marun: Agreed. | 18:25 |
| mestery | marun: OK, I'll see what I can do here, thanks for the consult! | 18:25 |
| marun | mestery: I suggest Related-Bug rather than filing a new one | 18:25 |
| mestery | marun: Agreed. | 18:25 |
| marun | jogo: where is neutron/common/config loaded? | 18:25 |
| jogo | marun: not sure, but there is a better solution | 18:27 |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Explicitly import state_path opt in tests.base https://review.openstack.org/84832 | 18:27 |
| jogo | funny enough when doing 'tox -epy27 neutron.tests.functional' it worked | 18:27 |
| jogo | marun: ^ is your fix | 18:27 |
| jogo | see the branch name :) | 18:27 |
| marun | heh | 18:27 |
| marun | jogo: you've got my vote! | 18:28 |
| marun | jogo: what does import_opt do | 18:29 |
| marun | jogo: I had conflicts when neutron loaded its config ahead of tempest (in the api job), so I'd like to make sure this doesn't introduce a similar problem. | 18:29 |
| jogo | marun: I think it loads a module and looks for the definition | 18:30 |
| jogo | marun: so this may have the same problem | 18:30 |
| marun | jogo: oy | 18:31 |
| marun | jogo: well, more important to get the functional job running for now. the api job is still wip | 18:31 |
| jogo | marun: haha baby steps | 18:33 |
| marun | jogo: btw, thank you for the pointer to devpi server. pip caching is great and all, but being able to cut out unnecessary pypi querying is loads better | 18:33 |
| jogo | marun: np, its really amazing I stumbled on it myself | 18:34 |
| jogo | it means a 'tox -r' takes an extra minute at most | 18:35 |
| marun | jogo: I've tried things like eggproxy in the past but never had much luck. | 18:35 |
| marun | jogo: nice to see one that's actively maintained | 18:35 |
| jogo | yeah I thought it would be hard to set up, but it was trivial | 18:37 |
| *** dvorkinista has quit IRC | 18:37 | |
| HenryG | jogo: 'tox -epy27 neutron.tests.functional' doesn't actually run any tests for me? It just says PASSED. | 18:38 |
| jogo | HenryG: heh that may be why it worked | 18:38 |
| marun | mestery: oh for petes sake | 18:38 |
| jogo | HenryG: yyup | 18:38 |
| mestery | marun: ??? | 18:38 |
| marun | mestery: it was skipping the damn tests on my box. but i didn't notice because the 'slowest' report had the tests listed anyway. | 18:39 |
| mestery | marun: Ha! | 18:39 |
| mestery | marun: So this hasn't been working for a while then, isn't it? | 18:39 |
| marun | mestery: apparently | 18:39 |
| marun | mestery: I swear they were running at one point | 18:39 |
| mestery | marun: OK, I should have my 3-patch set to fix this and add a new functional test out later tonight, is that timeframe ok? | 18:39 |
| marun | mestery: because I use nose in my editor and skipping is more obvious | 18:39 |
| mestery | marun: Currently also lookign at this one: https://bugs.launchpad.net/neutron/+bug/1301449 | 18:40 |
| pcm_ | anyone: I want to make some Neutron related changes to DevStack, do I create a bug? under Neutron? | 18:40 |
| *** pradipta_away has quit IRC | 18:40 | |
| *** rand738 has quit IRC | 18:41 | |
| *** rand738 has joined #openstack-neutron | 18:41 | |
| *** chandankumar_ has joined #openstack-neutron | 18:41 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Delete routers that are requested but not reported as active (WIP) https://review.openstack.org/84575 | 18:41 |
| marun | mestery: ffs | 18:41 |
| mestery | pcm_: Create a devstack bug, there is a launchpad project for it. | 18:41 |
| marun | mestery: https://github.com/openstack/neutron/blob/5b5ba869aa4680bcfa7544c4e718cb5a5dc9c168/neutron/agent/linux/ovs_lib.py#L76 | 18:41 |
| pcm_ | mestery: thanks. | 18:42 |
| marun | mestery: someone broke it | 18:42 |
| marun | mestery: I'm going to hunt them down | 18:42 |
| mestery | marun: Christ, when did that happen? | 18:42 |
| mestery | marun: #sadpanda | 18:42 |
| *** pradipta_away has joined #openstack-neutron | 18:42 | |
| *** evgenyf has quit IRC | 18:42 | |
| marun | mestery: this is my first bisection point, feb 11 | 18:42 |
| mestery | marun: That explains all the issues here. | 18:42 |
| marun | mestery: so between then and now | 18:42 |
| * mestery nods. | 18:42 | |
| marun | mestery: it is likely really recent | 18:42 |
| marun | mestery: although I need to add that flag that raises an error if check conditions aren't met | 18:43 |
| mestery | marun: I feel much better now and happy I reached out to you, this has saved me lots of time. :) | 18:43 |
| *** otherwiseguy has joined #openstack-neutron | 18:43 | |
| marun | mestery: otherwise things like this are easily masked | 18:43 |
| marun | mestery: sorry for doubting you :/ | 18:43 |
| mestery | marun: To be honest, I can refactor this to work with the existing lib code | 18:43 |
| mestery | marun: No worries man. :) | 18:43 |
| marun | mestery: please fix the lib code | 18:43 |
| *** dvorkinista has joined #openstack-neutron | 18:44 | |
| *** tvardeman has quit IRC | 18:44 | |
| marun | mestery: add_bridge *should* return the damn thing | 18:44 |
| mestery | marun: Add that back in? That's an easy patch, I can submit that ASAP. | 18:44 |
| mestery | marun: I think so too. | 18:44 |
| mestery | I'll submit a patch for that and all my troubles will likely go away :) | 18:44 |
| Sukhdev | markmcclain: ping | 18:44 |
| marun | mestery: although, the re compile should be module-level | 18:44 |
| marun | mestery: it's stupid to do it on bridge init, or even as part of the bridge class | 18:44 |
| *** sphoorti_ has quit IRC | 18:45 | |
| mestery | marun: true | 18:45 |
| *** thuc_ has quit IRC | 18:45 | |
| markmcclain | marun: add_bridge's return value was removed because nothing was consuming it | 18:45 |
| marun | markmcclain: …except the functional tests | 18:45 |
| mestery | markmcclain: The functional tests were (or at least are) now. :) | 18:45 |
| marun | markmcclain: *sigh* | 18:45 |
| *** thuc has joined #openstack-neutron | 18:46 | |
| mestery | markmcclain: Any problems with adding it back? | 18:46 |
| marun | markmcclain: when the tests moved from unit to functional they were no longer gating | 18:46 |
| mestery | marun: https://review.openstack.org/#/c/71142/ | 18:47 |
| mestery | marun: It was me who approved the commit! :) | 18:47 |
| marun | *sigh* | 18:47 |
| marun | -1 to you all | 18:47 |
| mestery | marun: I'm going to add it back in since the functional tests use it clearly. | 18:47 |
| openstackgerrit | A change was merged to openstack/neutron: NSX: Fix pagination support https://review.openstack.org/82218 | 18:47 |
| markmcclain | so the question is why do the functional test need it? | 18:48 |
| *** thuc has quit IRC | 18:48 | |
| mestery | To make calls on the OVSBridge objects themselves. | 18:48 |
| *** thuc has joined #openstack-neutron | 18:48 | |
| markmcclain | there's nothing special about the instance returned | 18:48 |
| markmcclain | the test could easily build its own | 18:48 |
| marun | Because OO is better than the alternative? | 18:48 |
| mestery | Yes, that's true I guess. | 18:48 |
| marun | Because we're going to do a lot more of this in functional testing? | 18:48 |
| *** thuc has quit IRC | 18:48 | |
| marun | The fact that we don't have a good oo model around os-level constructs like bridges, ports, etc is silly | 18:49 |
| mestery | +1 to that marun as I'm already doing with the new tests I'm adding. | 18:49 |
| marun | mestery: danek | 18:49 |
| marun | danke | 18:49 |
| mestery | :) | 18:49 |
| marun | :) | 18:49 |
| *** thuc has joined #openstack-neutron | 18:49 | |
| *** alexpilotti has quit IRC | 18:50 | |
| mestery | marun markmcclain: Did we decide we are ok with adding the return back based on the functional testing needs? | 18:50 |
| *** rkukura has quit IRC | 18:51 | |
| markmcclain | honestly I'm a bit mixed on it mainly because I don't like adding random return values that aren't actually used by production code | 18:51 |
| marun | markmcclain: That's just silly | 18:52 |
| marun | markmcclain: it's an api | 18:52 |
| mestery | Ha! | 18:52 |
| *** sbfox has quit IRC | 18:52 | |
| marun | markmcclain: need to to be useful for all uses, not just production. | 18:52 |
| marun | markmcclain: and functional testing is a valid use case. | 18:52 |
| marun | markmcclain: it's not like there is a performance impact here. | 18:52 |
| markmcclain | marun: understand my reservation is that we're adding return values with only a little bit of a real plan | 18:53 |
| marun | markmcclain: I have a plan | 18:53 |
| markmcclain | blueprint? | 18:53 |
| *** rotbeard has quit IRC | 18:53 | |
| marun | markmcclain: ffs | 18:53 |
| marun | markmcclain: I hope you're joking :p | 18:54 |
| markmcclain | marun: only a little :) | 18:54 |
| markmcclain | so while I do think adding this back makes sense | 18:54 |
| markmcclain | my concern is that this module obviously needs a refactor to make it more coherent | 18:55 |
| *** chandankumar_ has quit IRC | 18:55 | |
| mestery | ovs_lib needs some tender loving care, agreed. | 18:55 |
| markmcclain | I do have plans on how to fix too but have to get API stuff out first | 18:56 |
| *** _TheDodd_ has joined #openstack-neutron | 18:56 | |
| mestery | markmcclain: Nice! Lets refactor so it makes sense for production use as well as functional testing. | 18:56 |
| marun | markmcclain: +1 to refactor | 18:56 |
| marun | mestery: +1 to multiple use cases | 18:57 |
| marun | have to run, back in a few hours.l | 18:57 |
| *** marun is now known as maru_afk | 18:57 | |
| mestery | Bye marun | 18:57 |
| mestery | markmcclain: I may just hold back on this functional testing stuff for now, I've got 2 other High bugs to look at :P | 18:57 |
| *** Sukhdev has quit IRC | 18:58 | |
| *** dvorkinista has quit IRC | 18:58 | |
| *** WackoRob_ has quit IRC | 19:00 | |
| markmcclain | yeah makes sense | 19:00 |
| markmcclain | I would like to get functional testing voting sooner rather than later | 19:01 |
| mestery | Agreed, and the work I'm doing now will be beneficial there. | 19:01 |
| *** rwsu has quit IRC | 19:01 | |
| *** ramishra has joined #openstack-neutron | 19:01 | |
| *** overlayer has quit IRC | 19:02 | |
| *** jorgem has quit IRC | 19:05 | |
| *** sphoorti has joined #openstack-neutron | 19:05 | |
| *** jaypipes has quit IRC | 19:05 | |
| *** sphoorti has quit IRC | 19:05 | |
| *** jorgem has joined #openstack-neutron | 19:06 | |
| *** jorgem has quit IRC | 19:06 | |
| *** jorgem has joined #openstack-neutron | 19:06 | |
| *** rwsu has joined #openstack-neutron | 19:07 | |
| roaet | markmcclain: thank you so much for the review. | 19:09 |
| *** mwagner_afk has quit IRC | 19:10 | |
| *** jroovers|afk has quit IRC | 19:10 | |
| *** TrevorV has joined #openstack-neutron | 19:11 | |
| *** TrevorV has quit IRC | 19:14 | |
| *** TrevorV has joined #openstack-neutron | 19:14 | |
| *** evgenyf has joined #openstack-neutron | 19:16 | |
| *** sbfox has joined #openstack-neutron | 19:18 | |
| *** garyk has quit IRC | 19:19 | |
| *** jlibosva has joined #openstack-neutron | 19:19 | |
| *** tvardeman has joined #openstack-neutron | 19:19 | |
| *** Sukhdev has joined #openstack-neutron | 19:20 | |
| *** networks_ is now known as networkstatic | 19:21 | |
| *** leseb has quit IRC | 19:21 | |
| Sukhdev | markmcclain: Hi Mark | 19:21 |
| *** leseb has joined #openstack-neutron | 19:22 | |
| *** overlayer has joined #openstack-neutron | 19:22 | |
| openstackgerrit | A change was merged to openstack/neutron: update doc string - correct typo https://review.openstack.org/84785 | 19:22 |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 19:22 |
| *** dvorkinista has joined #openstack-neutron | 19:23 | |
| *** leseb has quit IRC | 19:26 | |
| *** jlibosva has quit IRC | 19:26 | |
| *** rkukura has joined #openstack-neutron | 19:27 | |
| markmcclain | Sukhdev: hi | 19:27 |
| Sukhdev | markmcclain: I have a very minor change that I would like to get into RC2 - https://review.openstack.org/#/c/84603/ | 19:28 |
| Sukhdev | markmcclain: wondering if I can bother you with this request | 19:29 |
| markmcclain | it's tagged correctly | 19:29 |
| Sukhdev | markmcclain: Yes, I believe so...https://bugs.launchpad.net/neutron/+bug/1301093 | 19:30 |
| markmcclain | once it merges we'll backport to RC2 | 19:30 |
| Sukhdev | markmcclain: Cool - when is the deadline for RC2? | 19:30 |
| markmcclain | Sukhdev: it hasn't opened yet | 19:31 |
| markmcclain | I'm guessing we'll cut it friday/monday | 19:31 |
| Sukhdev | markmcclain: Also, can you give the patch a quick review? if you can spare few minutes | 19:31 |
| markmcclain | mestery: looks like your -1 has be addresses for this one: https://review.openstack.org/#/c/84603/ | 19:31 |
| Sukhdev | Cool - hopefully, this will merge by Friday. | 19:32 |
| markmcclain | Sukhdev: already reviewed :) | 19:32 |
| mestery | markmcclain Sukhdev: Yes, I'll change that now, thanks for taking care of that Sukhdev! | 19:32 |
| mestery | +2 Sukhdev | 19:32 |
| Sukhdev | markmcclain: cool - thanks | 19:32 |
| Sukhdev | mestery: thanks | 19:33 |
| Sukhdev | so, what does it take to get it approved now? | 19:33 |
| Sukhdev | markmcclain: I see it approved - thanks a bunch…. | 19:34 |
| markmcclain | Sukhdev: you're welcome | 19:35 |
| *** gaud has joined #openstack-neutron | 19:35 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: fixes tests using called_once_ without assert https://review.openstack.org/84139 | 19:36 |
| gaud | hi, I am wondering why there is no cisco plugin for catalyst, is there something that catalyst doesn't support that prevent it from having a plugin for neutron ? | 19:36 |
| mestery | gaud: No one has written one yet. | 19:37 |
| *** leseb has joined #openstack-neutron | 19:37 | |
| gaud | i'm wondering why, it's quite a popular device.. | 19:38 |
| mestery | gaud: I don't know what external programmatic interfaces catalyst has. That's likely the sticking point. | 19:39 |
| pcm_ | mestery: probably netconf, no? | 19:39 |
| mestery | pcm_: catalyst supports netconf? | 19:40 |
| openstackgerrit | Rajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS) Service Insertion https://review.openstack.org/62599 | 19:40 |
| *** jorgem has quit IRC | 19:40 | |
| pcm_ | mestery: isn't netconf just an I/F into the CLI? | 19:40 |
| pcm_ | mestery: I thought it was equivalent to doing telnet into the box and running IOS CLI commands. | 19:41 |
| pcm_ | mestery: that was my impression - other's can correct. | 19:42 |
| mestery | pcm_: It is, but each box has it's own level of "support" of netconf. | 19:42 |
| mestery | Or rather, each version of the OS | 19:42 |
| *** jp_at_hp has quit IRC | 19:42 | |
| gaud | so basically, it's only a matter of creating vlans and adding them as a member of a port ? | 19:42 |
| *** jorgem has joined #openstack-neutron | 19:43 | |
| HenryG | gaud: You can just use it as a switch, no? Set it and forget it. You don't need a plugin for that. Not optimal, but it should work. | 19:43 |
| gaud | well, I still want to isolate tenant network | 19:43 |
| gaud | and I don't want all the traffic to go through a neutron l3 node | 19:47 |
| gaud | is there a way to do that without using a vendor plugin ? | 19:47 |
| HenryG | gaud: do you have a router somewhere else? (outside neutron) | 19:49 |
| *** leseb has quit IRC | 19:51 | |
| gaud | I have access to 2 stacked extremenetwork switches | 19:51 |
| *** ramishra has quit IRC | 19:51 | |
| gaud | we would like to use cisco nexus but we would prefer catalyst as we have some already available. | 19:52 |
| *** csd has quit IRC | 19:52 | |
| *** jorgem has quit IRC | 19:53 | |
| *** jorgem has joined #openstack-neutron | 19:53 | |
| banix | mestery: Just saw the convrsation regarding ovs-lib patch and removal of a return statement… Still not sure about the OO rational for having this looking at the BaseOVS and OVSBridge classes and how they are related but that is a discussion for a different time. | 19:53 |
| banix | mestery: Was planning to add methods for set-controller, etc but seeing you are looking at refactoring that code will hold on to it but when you get to doing the refactoring you may want to consider adding a few new methods. | 19:53 |
| mestery | banix: Agree 100% on your second point, those are good methos to have. | 19:54 |
| mestery | *methods | 19:54 |
| *** leseb has joined #openstack-neutron | 19:54 | |
| mestery | banix: I've put the functional test code I have on hold until I can talk to marun a bit more for now, working on my other 2 bugs. :) | 19:54 |
| *** nlahouti has quit IRC | 19:57 | |
| *** otherwiseguy has quit IRC | 20:00 | |
| gaud | so basically, there is a virtual network that is created on the router itself, corresponding to what there is in openvswitch ? | 20:01 |
| *** dfarrell07 has joined #openstack-neutron | 20:03 | |
| markmcclain | banix: the plan is to add those methods | 20:04 |
| banix | markmcclain: great. Was thinking that if refactoring of this code is going to happen then adding the methods should perhaps happen then rather than now. Right? | 20:05 |
| *** ajo has quit IRC | 20:08 | |
| openstackgerrit | A change was merged to openstack/python-neutronclient: Adds support for os-auth-strategy=noauth https://review.openstack.org/57517 | 20:08 |
| mestery | banix: Agreed | 20:11 |
| markmcclain | banix: correct | 20:11 |
| *** otherwiseguy has joined #openstack-neutron | 20:13 | |
| *** markmcclain has quit IRC | 20:15 | |
| *** Gil_McGrath has quit IRC | 20:18 | |
| *** manishg has quit IRC | 20:20 | |
| *** manishg has joined #openstack-neutron | 20:21 | |
| *** ajo has joined #openstack-neutron | 20:23 | |
| *** dave_tucker_zzz is now known as dave_tucker | 20:24 | |
| *** dkehn__ is now known as dkehn_ | 20:29 | |
| maru_afk | mestery: ping | 20:30 |
| mestery | maru_afk: pong! | 20:30 |
| *** maru_afk is now known as marun | 20:31 | |
| marun | mestery: you have questions? | 20:31 |
| mestery | marun: The only question was around bringing back the return of the object. :) | 20:31 |
| marun | mestery: do tell | 20:31 |
| *** jprovazn has quit IRC | 20:32 | |
| mestery | marun: No, really that's it: I'm confused about what we agreed with regards to reverting the change banix made. That fix unbreaks all of this and is quite simple frankly. | 20:32 |
| marun | mestery: I think we agreed that the change could be reverted. | 20:32 |
| mestery | marun: Wonderful! I'll go ahead and do that then. | 20:32 |
| *** markmcclain has joined #openstack-neutron | 20:32 | |
| marun | mestery: great! | 20:33 |
| marun | banix: so you're the responsible party? | 20:33 |
| * marun shakes fist at banix | 20:33 | |
| mestery | marun: Cool, then I can go and propose theo ther fixes. | 20:33 |
| jogo | marun: is anyone working on making neutron unit tests significantly faster? | 20:34 |
| banix | marun: Yes I think if this is an urgent matter reverting of the patch makes sense. Otherwise, I do not see why the code in ovs-lib is the way it is but that is a different discussion. | 20:34 |
| marun | banix: See the discussion on the original merge: https://review.openstack.org/#/c/45677/1..11/neutron/agent/linux/ovs_lib.py | 20:34 |
| marun | banix: Removing code without understanding why it's there in the first place is simply bad practice. | 20:35 |
| marun | banix: a simple 'git blame' will allow one to figure out the 'why' | 20:35 |
| *** RockKuo_TW has joined #openstack-neutron | 20:35 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 20:37 |
| marun | banix: but given that the functional job wasn't voting, it's understandable that nobody noticed that removal would be an issue. | 20:37 |
| *** RockKuo has quit IRC | 20:38 | |
| marun | jogo: I am | 20:38 |
| mestery | marun: With that reverted, it's now failing on jobo's state_path issue :P | 20:38 |
| openstackgerrit | A change was merged to openstack/neutron: Fixing Arista CLI command https://review.openstack.org/84603 | 20:38 |
| mestery | marun: I'll push the patch now. | 20:39 |
| jogo | marun: sweet, I had a hunch | 20:39 |
| marun | jogo: it's supposed to be my primary focus but there's always something coming up. | 20:39 |
| openstackgerrit | A change was merged to openstack/neutron: Enable hacking H301 check https://review.openstack.org/84258 | 20:39 |
| marun | jogo: sadly there is no one culprit. we have 12k tests, most of which take multiple seconds. | 20:39 |
| marun | jogo: unit tests should be subsecond :( | 20:39 |
| carl_baldwin | marun: I have just pushed up a new patch set on the fix that proposes removing the looping call. I think I like this direction over the first direction for the short term. It is easier to wrap one's head around and localized. | 20:40 |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Fix up usage of state_path mocking in unit tests https://review.openstack.org/84620 | 20:40 |
| jogo | marun: amen | 20:40 |
| openstackgerrit | Kyle Mestery proposed a change to openstack/neutron: Revert changes removing OVSBridge return https://review.openstack.org/84856 | 20:41 |
| jogo | marun: when working on ^ I don't run the full test suite locally because it takes to long so I just push patches up for review | 20:41 |
| *** yfried__ has joined #openstack-neutron | 20:41 | |
| marun | jogo: as per the etherpad, I'm going to try to figure out which mixin tests cumulatively are taking the most time and work on those first | 20:41 |
| marun | jogo: https://etherpad.openstack.org/p/neutron-testing-refactor | 20:41 |
| mestery | marun banix: ^^^^ | 20:41 |
| *** leseb has quit IRC | 20:41 | |
| marun | jogo: no problem | 20:41 |
| mestery | marun: Once that merges, we'll be in much better shape for my functional test refactoring. | 20:41 |
| marun | jogo: i actually built a new 6 core server so I could iterate faster on speeding the tests | 20:41 |
| jogo | haha damn | 20:41 |
| marun | jogo: can run in 10m instead of 90 | 20:41 |
| marun | jogo: hopefully I can go back to laptop when I'm done :) | 20:42 |
| mestery | marun: Now you're just showing off. :) | 20:42 |
| jogo | I have a rax instance for that when things get bad | 20:42 |
| *** jecarey has quit IRC | 20:42 | |
| marun | mestery: it's a terrible state of affairs. we should be able to run the tests in 10m on a laptop vm | 20:42 |
| mestery | marun: Your eyes on this are much appreciated: https://review.openstack.org/84856 | 20:42 |
| marun | mestery: so desperate times times call for desperate measures | 20:42 |
| marun | mestery: ok | 20:42 |
| mestery | marun: True dat. | 20:42 |
| *** dfarrell07 has quit IRC | 20:42 | |
| marun | mestery: great job on that, and making sure the unit test catches the issue. | 20:43 |
| mestery | marun: Thanks. Now to find another core to push it over the edge. :) | 20:44 |
| *** networkstatic has quit IRC | 20:44 | |
| *** dfarrell07 has joined #openstack-neutron | 20:45 | |
| mestery | markmcclain: Any chance you could give this one some love? It's the revert we spoke about earlier: https://review.openstack.org/#/c/84856/ | 20:45 |
| marun | jogo: I'll be happy to +2 https://review.openstack.org/#/c/84620 once the tests pass | 20:45 |
| marun | tests -> checks | 20:45 |
| jogo | marun: cool, hopefully I cought everything this time | 20:46 |
| *** evgenyf has quit IRC | 20:47 | |
| *** leseb has joined #openstack-neutron | 20:47 | |
| marun | carl_baldwin: I have to step out for an hour, the one comment on the patch is that the send interval should probably remain configurable instead of hard-coded. I'll review in depth when I get back. | 20:48 |
| *** marun is now known as maru_afk | 20:48 | |
| *** csd has joined #openstack-neutron | 20:50 | |
| markmcclain | mestery:+2/A | 20:54 |
| mestery | markmcclain: Thanks man! | 20:54 |
| banix | marun: obviously a change that break other parts of the code shouldn't get through (merged on February 21st). That part aside, the way the code is organized in ovs-lib does not make sense to me but I haven't had the time to go through the review comments on the original patch where you made significant changes to that file. | 20:56 |
| openstackgerrit | Rajesh Mohan proposed a change to openstack/neutron: Firewall as a Service (FWaaS) Service Insertion https://review.openstack.org/62599 | 20:57 |
| *** Gil_McGrath has joined #openstack-neutron | 20:57 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 20:58 |
| *** dvorkinista has quit IRC | 20:58 | |
| *** sbfox1 has joined #openstack-neutron | 20:59 | |
| *** sbfox has quit IRC | 21:00 | |
| *** sbfox1 has quit IRC | 21:01 | |
| *** _TheDodd_ has quit IRC | 21:09 | |
| *** dvorkinista has joined #openstack-neutron | 21:11 | |
| *** pcm_ has quit IRC | 21:12 | |
| *** dave_tucker is now known as dave_tucker_zzz | 21:12 | |
| *** bvandenh has joined #openstack-neutron | 21:12 | |
| *** tvardeman has quit IRC | 21:13 | |
| *** Sukhdev has quit IRC | 21:20 | |
| *** sbfox has joined #openstack-neutron | 21:20 | |
| *** bvandenh has quit IRC | 21:20 | |
| *** Sukhdev has joined #openstack-neutron | 21:21 | |
| HenryG | jogo: we overlapped :) https://review.openstack.org/84608 | 21:22 |
| HenryG | jogo: I left a comment on yours | 21:22 |
| jogo | HenryG: saw, looking now | 21:25 |
| jogo | HenryG: strange thing is the tests work locally for me | 21:27 |
| HenryG | jogo: hmmm, I got assert_called_with errors because paths did not match | 21:30 |
| jogo | when you tried locally? | 21:30 |
| HenryG | yes | 21:31 |
| jogo | HenryG: what version of oslo.config? | 21:31 |
| *** sbfox has quit IRC | 21:31 | |
| * HenryG is double-checking ... | 21:31 | |
| jogo | I am gonna wait for jenkins to finish running on my patch | 21:32 |
| *** bjornar has quit IRC | 21:32 | |
| HenryG | 1.3.0 | 21:33 |
| jogo | HenryG: hmm now i am confused | 21:34 |
| jogo | can you pastebin me your errors? | 21:34 |
| *** sungju_ has joined #openstack-neutron | 21:36 | |
| HenryG | jogo: hmmm, it's not quite what I remember seeing before, but: http://paste.openstack.org/show/74904/ | 21:36 |
| jogo | hmm I got the fail to this time | 21:39 |
| *** sbfox has joined #openstack-neutron | 21:46 | |
| *** sbfox1 has joined #openstack-neutron | 21:47 | |
| *** sc68cal has quit IRC | 21:48 | |
| jogo | HenryG: the solution is to just delete more code :) | 21:50 |
| *** sbfox has quit IRC | 21:50 | |
| *** sbfox1 has quit IRC | 21:50 | |
| *** sbfox has joined #openstack-neutron | 21:50 | |
| *** sungju_ has quit IRC | 21:51 | |
| *** sungju_ has joined #openstack-neutron | 21:52 | |
| *** singhs has quit IRC | 21:53 | |
| *** dfarrell07 has quit IRC | 21:56 | |
| *** sungju_ has quit IRC | 21:56 | |
| *** sbfox has quit IRC | 21:56 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Fix up usage of state_path mocking in unit tests https://review.openstack.org/84620 | 21:59 |
| openstackgerrit | Joe Gordon proposed a change to openstack/neutron: Explicitly import state_path opt in tests.base https://review.openstack.org/84832 | 21:59 |
| jogo | HenryG: I think there is still another bug in there somewhere but cannot reproduce locally | 21:59 |
| *** thuc has quit IRC | 22:00 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Delete routers that are requested but not reported as active https://review.openstack.org/84575 | 22:00 |
| *** sbfox has joined #openstack-neutron | 22:00 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Clean out namespaces even if we don't delete namespaces https://review.openstack.org/84570 | 22:00 |
| *** thuc has joined #openstack-neutron | 22:01 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Split API and RPC workers in to one separate process each (WIP) https://review.openstack.org/76021 | 22:02 |
| *** thuc has quit IRC | 22:05 | |
| *** overlayer has quit IRC | 22:06 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Rename workers to api_workers and simplify code https://review.openstack.org/72564 | 22:09 |
| *** gdubreui has joined #openstack-neutron | 22:10 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Replace loopingcall in notifier with a delayed send https://review.openstack.org/84798 | 22:10 |
| *** Gil_McGrath has quit IRC | 22:12 | |
| *** ajo has quit IRC | 22:15 | |
| *** leseb has quit IRC | 22:17 | |
| *** leseb has joined #openstack-neutron | 22:18 | |
| *** zhenjie has joined #openstack-neutron | 22:21 | |
| *** leseb has quit IRC | 22:22 | |
| *** zhenjie has quit IRC | 22:23 | |
| *** Jesse22 has joined #openstack-neutron | 22:24 | |
| *** djoreilly has quit IRC | 22:25 | |
| *** Jesse22 has left #openstack-neutron | 22:26 | |
| *** ajo has joined #openstack-neutron | 22:26 | |
| *** banix has quit IRC | 22:30 | |
| *** ajo has quit IRC | 22:31 | |
| *** enikanorov_ has joined #openstack-neutron | 22:32 | |
| *** tomoe_ has joined #openstack-neutron | 22:34 | |
| *** enikanorov has quit IRC | 22:35 | |
| *** harlowja has quit IRC | 22:35 | |
| *** tomoe__ has joined #openstack-neutron | 22:36 | |
| *** ajo has joined #openstack-neutron | 22:36 | |
| *** harlowja has joined #openstack-neutron | 22:36 | |
| *** Jesse22 has joined #openstack-neutron | 22:37 | |
| *** tomoe__ has quit IRC | 22:38 | |
| *** tomoe_ has quit IRC | 22:39 | |
| *** alagalah_ has joined #openstack-neutron | 22:39 | |
| *** ajo has quit IRC | 22:40 | |
| *** dims_ has quit IRC | 22:40 | |
| *** tchaypo has quit IRC | 22:40 | |
| *** ajo has joined #openstack-neutron | 22:41 | |
| *** armitage81 has quit IRC | 22:41 | |
| *** alagalah has quit IRC | 22:41 | |
| *** armitage81 has joined #openstack-neutron | 22:42 | |
| openstackgerrit | A change was merged to openstack/neutron: Revert changes removing OVSBridge return https://review.openstack.org/84856 | 22:44 |
| *** tomoe_ has joined #openstack-neutron | 22:45 | |
| *** tomoe_ has quit IRC | 22:46 | |
| *** tomoe_ has joined #openstack-neutron | 22:47 | |
| *** prad_ has quit IRC | 22:48 | |
| *** ajo has quit IRC | 22:48 | |
| *** tomoe_ has quit IRC | 22:51 | |
| *** jorgem has quit IRC | 22:53 | |
| *** tchaypo has joined #openstack-neutron | 22:54 | |
| *** harlowja has quit IRC | 22:54 | |
| *** harlowja has joined #openstack-neutron | 22:55 | |
| *** ajo has joined #openstack-neutron | 22:55 | |
| *** dims_ has joined #openstack-neutron | 22:56 | |
| *** WackoRobie has joined #openstack-neutron | 22:56 | |
| *** networkstatic has joined #openstack-neutron | 23:00 | |
| *** WackoRobie has quit IRC | 23:01 | |
| *** yamahata has joined #openstack-neutron | 23:01 | |
| *** sbfox has quit IRC | 23:07 | |
| *** jobewan has quit IRC | 23:08 | |
| *** yamahata has quit IRC | 23:11 | |
| *** thuc has joined #openstack-neutron | 23:11 | |
| *** dave_tucker_zzz is now known as dave_tucker | 23:15 | |
| *** thuc has quit IRC | 23:16 | |
| *** ajo has quit IRC | 23:16 | |
| *** beagles has quit IRC | 23:17 | |
| *** thuc has joined #openstack-neutron | 23:18 | |
| *** gaud has quit IRC | 23:19 | |
| *** thuc has quit IRC | 23:19 | |
| *** ajo has joined #openstack-neutron | 23:19 | |
| *** thuc has joined #openstack-neutron | 23:19 | |
| HenryG | jogo: sorry, was afk. What other bug? | 23:20 |
| *** thuc has quit IRC | 23:20 | |
| *** thuc has joined #openstack-neutron | 23:20 | |
| *** rand738 has quit IRC | 23:21 | |
| *** Sukhdev has quit IRC | 23:21 | |
| *** sc68cal has joined #openstack-neutron | 23:23 | |
| *** rand738 has joined #openstack-neutron | 23:24 | |
| jogo | HenryG: not sure will see what jenkins says | 23:24 |
| *** otherwiseguy has quit IRC | 23:25 | |
| *** bada has quit IRC | 23:26 | |
| HenryG | jogo: We'll see, but your latest patch looks good to me. | 23:29 |
| *** thuc has quit IRC | 23:33 | |
| *** thuc has joined #openstack-neutron | 23:33 | |
| *** blogan has joined #openstack-neutron | 23:34 | |
| *** thuc_ has joined #openstack-neutron | 23:35 | |
| *** thuc_ has quit IRC | 23:36 | |
| *** thuc has quit IRC | 23:36 | |
| *** thuc has joined #openstack-neutron | 23:36 | |
| *** ajo has quit IRC | 23:37 | |
| *** Jesse22 has left #openstack-neutron | 23:37 | |
| *** ajo has joined #openstack-neutron | 23:50 | |
| *** manishg has quit IRC | 23:53 | |
| *** carl_baldwin has quit IRC | 23:58 | |
| *** blogan has quit IRC | 23:58 | |
| HenryG | jogo: jenkins failed. Running individual test modules works fine, but running in parallel ... kablooie. Test isolation is broken. No fun to debug. :( | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!