| *** gouthamr has quit IRC | 00:00 | |
| *** fragatina has quit IRC | 00:03 | |
| *** stanzgy has joined #openstack-neutron | 00:04 | |
| *** crushil has joined #openstack-neutron | 00:09 | |
| *** neiljerram has quit IRC | 00:10 | |
| *** pmannidi has joined #openstack-neutron | 00:11 | |
| *** fnaval has joined #openstack-neutron | 00:14 | |
| *** fragatina has joined #openstack-neutron | 00:16 | |
| *** karthikpr has quit IRC | 00:24 | |
| *** seanatcisco has joined #openstack-neutron | 00:25 | |
| *** ircuser-1 has quit IRC | 00:29 | |
| *** fnaval has quit IRC | 00:31 | |
| *** zhurong has joined #openstack-neutron | 00:33 | |
| *** thorst has quit IRC | 00:33 | |
| *** markstur has quit IRC | 00:35 | |
| *** fnaval has joined #openstack-neutron | 00:35 | |
| *** markstur has joined #openstack-neutron | 00:36 | |
| *** hoangcx has joined #openstack-neutron | 00:39 | |
| openstackgerrit | Ruijing proposed openstack/networking-odl master: enable new netvirt https://review.openstack.org/453386 | 00:40 |
|---|---|---|
| *** markstur has quit IRC | 00:40 | |
| *** seanatcisco has quit IRC | 00:42 | |
| *** seanatcisco has joined #openstack-neutron | 00:43 | |
| *** mickeys has quit IRC | 00:50 | |
| *** kiseok7 has joined #openstack-neutron | 00:53 | |
| *** limao has joined #openstack-neutron | 00:54 | |
| *** pckizer has quit IRC | 00:55 | |
| *** mohankumar has joined #openstack-neutron | 01:05 | |
| *** yedongcan has joined #openstack-neutron | 01:08 | |
| *** iwamoto has joined #openstack-neutron | 01:09 | |
| *** nicolasbock has joined #openstack-neutron | 01:10 | |
| *** mohankumar_ has joined #openstack-neutron | 01:10 | |
| *** mohankumar has quit IRC | 01:10 | |
| *** NikhilS has joined #openstack-neutron | 01:11 | |
| *** radhikam has joined #openstack-neutron | 01:14 | |
| *** sam_nowitzki has joined #openstack-neutron | 01:16 | |
| *** masaki has joined #openstack-neutron | 01:18 | |
| *** radhikam has quit IRC | 01:24 | |
| *** seanatcisco has quit IRC | 01:24 | |
| *** longlq_ has joined #openstack-neutron | 01:25 | |
| *** longlq_ has quit IRC | 01:26 | |
| *** longlq_ has joined #openstack-neutron | 01:27 | |
| *** sam_nowitzki has quit IRC | 01:28 | |
| *** seanatcisco has joined #openstack-neutron | 01:28 | |
| *** longlq_ has quit IRC | 01:28 | |
| *** longlq_ has joined #openstack-neutron | 01:29 | |
| *** longlq_ has quit IRC | 01:30 | |
| *** longlq_ has joined #openstack-neutron | 01:30 | |
| *** radhikam has joined #openstack-neutron | 01:31 | |
| *** longlq_ has quit IRC | 01:32 | |
| *** cuongnv has joined #openstack-neutron | 01:32 | |
| *** radhikam has quit IRC | 01:33 | |
| *** thorst has joined #openstack-neutron | 01:34 | |
| *** thorst has quit IRC | 01:37 | |
| *** thorst has joined #openstack-neutron | 01:37 | |
| *** thorst has quit IRC | 01:37 | |
| *** pck has joined #openstack-neutron | 01:38 | |
| *** mickeys has joined #openstack-neutron | 01:50 | |
| *** nplanel has joined #openstack-neutron | 01:52 | |
| *** fnaval has quit IRC | 01:56 | |
| *** mickeys has quit IRC | 01:56 | |
| *** zhurong has quit IRC | 01:58 | |
| *** tuanluong has joined #openstack-neutron | 01:59 | |
| *** tuanluong_ has joined #openstack-neutron | 02:02 | |
| *** tuanluong has quit IRC | 02:05 | |
| *** gouthamr has joined #openstack-neutron | 02:06 | |
| *** zhurong has joined #openstack-neutron | 02:10 | |
| *** aojea has joined #openstack-neutron | 02:13 | |
| *** lujinluo has joined #openstack-neutron | 02:17 | |
| *** aojea has quit IRC | 02:18 | |
| *** mohankumar_ has quit IRC | 02:20 | |
| *** markstur has joined #openstack-neutron | 02:26 | |
| *** hjensas has joined #openstack-neutron | 02:33 | |
| *** hjensas has quit IRC | 02:33 | |
| *** hjensas has joined #openstack-neutron | 02:33 | |
| *** masaki has quit IRC | 02:34 | |
| *** hjensas has quit IRC | 02:37 | |
| *** thorst has joined #openstack-neutron | 02:38 | |
| *** ramishra has joined #openstack-neutron | 02:39 | |
| *** armax has quit IRC | 02:42 | |
| *** armax has joined #openstack-neutron | 02:42 | |
| *** armax has quit IRC | 02:43 | |
| *** markstur has quit IRC | 02:44 | |
| *** yamamoto has joined #openstack-neutron | 02:45 | |
| *** tuanluong has joined #openstack-neutron | 02:47 | |
| *** tuanluong_ has quit IRC | 02:49 | |
| *** wu_ng has joined #openstack-neutron | 02:50 | |
| *** mickeys has joined #openstack-neutron | 02:53 | |
| *** nicolasbock has quit IRC | 02:53 | |
| *** mickeys has quit IRC | 02:57 | |
| *** thorst has quit IRC | 02:57 | |
| *** masaki has joined #openstack-neutron | 02:59 | |
| *** vj3k has joined #openstack-neutron | 02:59 | |
| *** mohankumar_ has joined #openstack-neutron | 03:01 | |
| *** yamamoto has quit IRC | 03:01 | |
| *** masaki has quit IRC | 03:02 | |
| *** masaki has joined #openstack-neutron | 03:03 | |
| *** mriedem has quit IRC | 03:06 | |
| *** markstur has joined #openstack-neutron | 03:14 | |
| *** gouthamr has quit IRC | 03:16 | |
| *** markstur has quit IRC | 03:19 | |
| *** ramishra has quit IRC | 03:19 | |
| *** ramishra has joined #openstack-neutron | 03:21 | |
| *** jhershbe has joined #openstack-neutron | 03:21 | |
| *** yamahata has joined #openstack-neutron | 03:21 | |
| *** gkadam has joined #openstack-neutron | 03:24 | |
| *** mohankumar_ has quit IRC | 03:26 | |
| *** vj3k has quit IRC | 03:30 | |
| *** zhhuabj has quit IRC | 03:35 | |
| *** psahoo has joined #openstack-neutron | 03:35 | |
| *** limao has quit IRC | 03:40 | |
| *** john-davidge has joined #openstack-neutron | 03:41 | |
| *** catintheroof has joined #openstack-neutron | 03:43 | |
| *** limao has joined #openstack-neutron | 03:43 | |
| *** john-davidge has quit IRC | 03:46 | |
| *** limao has quit IRC | 03:48 | |
| *** markstur has joined #openstack-neutron | 03:50 | |
| *** zhhuabj has joined #openstack-neutron | 03:50 | |
| openstackgerrit | Merged openstack/networking-ovn master: Updated from global requirements https://review.openstack.org/454504 | 03:51 |
| *** catintheroof has quit IRC | 03:51 | |
| *** markstur has quit IRC | 03:53 | |
| *** mickeys has joined #openstack-neutron | 03:53 | |
| *** thorst has joined #openstack-neutron | 03:55 | |
| *** mickeys has quit IRC | 03:58 | |
| *** jhershbe has quit IRC | 03:58 | |
| *** suresh12 has joined #openstack-neutron | 03:59 | |
| *** thorst has quit IRC | 03:59 | |
| *** yamamoto has joined #openstack-neutron | 04:01 | |
| *** radhikam has joined #openstack-neutron | 04:03 | |
| *** anilvenkata has joined #openstack-neutron | 04:04 | |
| *** markvoelker has joined #openstack-neutron | 04:05 | |
| *** mdnadeem has joined #openstack-neutron | 04:06 | |
| *** limao has joined #openstack-neutron | 04:11 | |
| *** vks1 has joined #openstack-neutron | 04:17 | |
| *** udesale has joined #openstack-neutron | 04:20 | |
| *** Dinesh_Bhor has joined #openstack-neutron | 04:22 | |
| *** markstur has joined #openstack-neutron | 04:24 | |
| *** nplanel has quit IRC | 04:30 | |
| *** nplanel has joined #openstack-neutron | 04:32 | |
| *** seanatcisco has quit IRC | 04:32 | |
| *** nplanel has quit IRC | 04:33 | |
| *** nplanel has joined #openstack-neutron | 04:34 | |
| openstackgerrit | Merged openstack/networking-ovn master: Allow tuning the probe interval from the IDL session https://review.openstack.org/454163 | 04:35 |
| *** zhhuabj has quit IRC | 04:36 | |
| *** zhhuabj has joined #openstack-neutron | 04:41 | |
| *** gcheresh has joined #openstack-neutron | 04:42 | |
| *** Jack_Iv has joined #openstack-neutron | 04:43 | |
| *** mickeys has joined #openstack-neutron | 04:54 | |
| *** thorst has joined #openstack-neutron | 04:55 | |
| *** Jack_Iv has quit IRC | 04:58 | |
| *** Jack_Iv has joined #openstack-neutron | 04:58 | |
| *** mickeys has quit IRC | 04:58 | |
| *** ociuhandu has quit IRC | 05:00 | |
| *** thorst has quit IRC | 05:00 | |
| *** ociuhandu has joined #openstack-neutron | 05:01 | |
| *** ratailor has joined #openstack-neutron | 05:02 | |
| *** Jack_Iv has quit IRC | 05:02 | |
| *** ayogi has joined #openstack-neutron | 05:04 | |
| *** crushil has quit IRC | 05:07 | |
| *** Jack_Iv has joined #openstack-neutron | 05:13 | |
| *** iranzo has joined #openstack-neutron | 05:17 | |
| *** iranzo has quit IRC | 05:17 | |
| *** iranzo has joined #openstack-neutron | 05:17 | |
| *** ayogi has quit IRC | 05:21 | |
| *** sridharg has joined #openstack-neutron | 05:21 | |
| *** ayogi has joined #openstack-neutron | 05:24 | |
| gkadam | @openstack-neutron, ping : Can we add more than one external network to the OpenStack environment along with the existing single external flat network? | 05:24 |
| *** rcernin has joined #openstack-neutron | 05:25 | |
| *** Jack_Iv has quit IRC | 05:26 | |
| *** Jack_Iv has joined #openstack-neutron | 05:27 | |
| *** mosulica has joined #openstack-neutron | 05:30 | |
| *** trinaths has joined #openstack-neutron | 05:32 | |
| *** markstur has quit IRC | 05:38 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron master: extraroute_db: Clean up update_router https://review.openstack.org/410793 | 05:40 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron master: extraroute_db: Remove _get_extra_routes_dict_by_router_id https://review.openstack.org/410794 | 05:40 |
| *** vijaykc4 has joined #openstack-neutron | 05:42 | |
| *** Jack_Iv has quit IRC | 05:42 | |
| *** john-davidge has joined #openstack-neutron | 05:42 | |
| *** yamamoto_ has joined #openstack-neutron | 05:42 | |
| *** Jack_Iv has joined #openstack-neutron | 05:42 | |
| *** yamamoto has quit IRC | 05:45 | |
| *** john-davidge has quit IRC | 05:46 | |
| *** mickeys has joined #openstack-neutron | 05:50 | |
| *** ociuhandu has quit IRC | 05:55 | |
| *** thorst has joined #openstack-neutron | 05:56 | |
| *** gcb has joined #openstack-neutron | 05:57 | |
| *** cshastri has joined #openstack-neutron | 05:58 | |
| *** tuanluong has quit IRC | 06:00 | |
| *** thorst has quit IRC | 06:00 | |
| *** pcaruana has joined #openstack-neutron | 06:01 | |
| *** mosulica has quit IRC | 06:02 | |
| *** ociuhandu has joined #openstack-neutron | 06:05 | |
| *** dsneddon has quit IRC | 06:06 | |
| *** tbh has joined #openstack-neutron | 06:07 | |
| *** sticker has joined #openstack-neutron | 06:11 | |
| sticker | Hey, whats the best way to suggest another entry in the l3.filters file? I had an issue recently on my CentOS setup that was caused by rootwrap being unable to kill a process and a line in l3.filters fixed it | 06:13 |
| sticker | it's not really code so not sure if it is really a *bug* per se | 06:13 |
| *** pgadiya has joined #openstack-neutron | 06:13 | |
| *** ratailor is now known as Guest5432 | 06:14 | |
| *** tuanluong has joined #openstack-neutron | 06:14 | |
| *** sridharg has quit IRC | 06:17 | |
| *** amotoki has joined #openstack-neutron | 06:23 | |
| *** mosulica has joined #openstack-neutron | 06:24 | |
| *** radhikam has quit IRC | 06:24 | |
| *** ramishra has quit IRC | 06:25 | |
| *** ramishra has joined #openstack-neutron | 06:27 | |
| *** g3ek has quit IRC | 06:27 | |
| *** manheim has joined #openstack-neutron | 06:29 | |
| *** ociuhandu has quit IRC | 06:31 | |
| *** janki has joined #openstack-neutron | 06:33 | |
| *** rossella_s has quit IRC | 06:33 | |
| *** rossella_s has joined #openstack-neutron | 06:34 | |
| *** andreas_s has joined #openstack-neutron | 06:35 | |
| *** janki has quit IRC | 06:35 | |
| *** janki has joined #openstack-neutron | 06:36 | |
| *** suresh12 has quit IRC | 06:36 | |
| *** g3ek has joined #openstack-neutron | 06:36 | |
| *** zhhuabj has quit IRC | 06:41 | |
| *** jlinkes has joined #openstack-neutron | 06:41 | |
| *** fragatina has quit IRC | 06:42 | |
| *** ociuhandu has joined #openstack-neutron | 06:44 | |
| *** tesseract has joined #openstack-neutron | 06:48 | |
| openstackgerrit | Merged openstack/neutron-fwaas master: Added 'shared' attribute for firewall resources https://review.openstack.org/451705 | 06:48 |
| *** jaosorior has joined #openstack-neutron | 06:49 | |
| *** vijaykc4 has quit IRC | 06:51 | |
| *** ramishra has quit IRC | 06:51 | |
| *** ramishra has joined #openstack-neutron | 06:53 | |
| *** mosulica has quit IRC | 06:54 | |
| *** voelzmo has joined #openstack-neutron | 06:54 | |
| *** amarao has joined #openstack-neutron | 06:56 | |
| *** thorst has joined #openstack-neutron | 06:57 | |
| *** mosulica has joined #openstack-neutron | 06:57 | |
| *** obondarev has joined #openstack-neutron | 06:58 | |
| *** voelzmo has quit IRC | 06:59 | |
| *** zhhuabj has joined #openstack-neutron | 06:59 | |
| *** thorst has quit IRC | 07:01 | |
| *** voelzmo has joined #openstack-neutron | 07:01 | |
| *** dmellado has joined #openstack-neutron | 07:08 | |
| *** mickeys has quit IRC | 07:08 | |
| *** jpena|off has joined #openstack-neutron | 07:08 | |
| *** reedip_ has quit IRC | 07:09 | |
| *** vijaykc4 has joined #openstack-neutron | 07:09 | |
| *** markvoelker has quit IRC | 07:12 | |
| *** ltomasbo|away has joined #openstack-neutron | 07:14 | |
| *** markstur has joined #openstack-neutron | 07:14 | |
| *** markvoelker has joined #openstack-neutron | 07:15 | |
| *** voelzmo has quit IRC | 07:15 | |
| *** nplanel has quit IRC | 07:16 | |
| *** markstur has quit IRC | 07:19 | |
| *** gcheresh has quit IRC | 07:22 | |
| *** korzen has joined #openstack-neutron | 07:22 | |
| *** ltomasbo|away is now known as ltomasbo | 07:22 | |
| *** vijaykc4 has quit IRC | 07:27 | |
| *** vijaykc4 has joined #openstack-neutron | 07:27 | |
| *** vijaykc4 has quit IRC | 07:28 | |
| *** vijaykc4 has joined #openstack-neutron | 07:28 | |
| *** vijaykc4 has quit IRC | 07:28 | |
| *** vijaykc4 has joined #openstack-neutron | 07:29 | |
| *** nplanel has joined #openstack-neutron | 07:29 | |
| *** sridharg has joined #openstack-neutron | 07:29 | |
| *** vijaykc4 has quit IRC | 07:30 | |
| *** vijaykc4 has joined #openstack-neutron | 07:30 | |
| *** vijaykc4 has quit IRC | 07:31 | |
| *** reedip has joined #openstack-neutron | 07:31 | |
| *** vijaykc4 has joined #openstack-neutron | 07:31 | |
| *** vijaykc4 has quit IRC | 07:32 | |
| *** hjensas has joined #openstack-neutron | 07:32 | |
| *** reedip has quit IRC | 07:32 | |
| *** vijaykc4 has joined #openstack-neutron | 07:32 | |
| *** vijaykc4 has quit IRC | 07:32 | |
| *** oreillyd has joined #openstack-neutron | 07:33 | |
| *** oreillyd_ has joined #openstack-neutron | 07:33 | |
| *** brault has joined #openstack-neutron | 07:35 | |
| *** adriant has quit IRC | 07:35 | |
| *** mosulica has quit IRC | 07:36 | |
| *** reedip has joined #openstack-neutron | 07:36 | |
| *** brault has quit IRC | 07:36 | |
| *** neiljerram has joined #openstack-neutron | 07:37 | |
| openstackgerrit | Ann Taraday proposed openstack/neutron master: New enginefacade for ports and sg groups https://review.openstack.org/404715 | 07:40 |
| *** manheim has quit IRC | 07:42 | |
| *** manheim has joined #openstack-neutron | 07:43 | |
| *** voelzmo has joined #openstack-neutron | 07:43 | |
| *** john-davidge has joined #openstack-neutron | 07:43 | |
| *** gcb has quit IRC | 07:46 | |
| *** brault has joined #openstack-neutron | 07:48 | |
| *** john-davidge has quit IRC | 07:48 | |
| *** lpetrut has joined #openstack-neutron | 07:48 | |
| *** mosulica has joined #openstack-neutron | 07:48 | |
| *** rmart04 has joined #openstack-neutron | 07:49 | |
| openstackgerrit | zhichao zhu proposed openstack/neutron master: Fix some reST field lists in docstrings https://review.openstack.org/449882 | 07:51 |
| *** jlibosva has joined #openstack-neutron | 07:54 | |
| *** anilvenkata is now known as anilvenkata_afk | 07:57 | |
| *** thorst has joined #openstack-neutron | 07:58 | |
| *** TMM has quit IRC | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-neutron | 08:01 | |
| *** Jack_Iv has quit IRC | 08:02 | |
| *** Jack_Iv has joined #openstack-neutron | 08:02 | |
| *** rossella_s has quit IRC | 08:02 | |
| *** Jack_Iv has quit IRC | 08:03 | |
| *** Jack_Iv_ has joined #openstack-neutron | 08:03 | |
| *** Jack_Iv_ has quit IRC | 08:03 | |
| *** xinliang has joined #openstack-neutron | 08:03 | |
| *** xinliang has quit IRC | 08:03 | |
| *** xinliang has joined #openstack-neutron | 08:03 | |
| *** Jack_Iv has joined #openstack-neutron | 08:03 | |
| *** vijaykc4 has joined #openstack-neutron | 08:06 | |
| *** rossella_s has joined #openstack-neutron | 08:06 | |
| *** tmorin has joined #openstack-neutron | 08:06 | |
| *** Jack_Iv has quit IRC | 08:08 | |
| *** rossella_s has quit IRC | 08:08 | |
| *** mickeys has joined #openstack-neutron | 08:09 | |
| *** efoley has joined #openstack-neutron | 08:12 | |
| *** gcb has joined #openstack-neutron | 08:12 | |
| *** mickeys has quit IRC | 08:14 | |
| *** bfernando has joined #openstack-neutron | 08:15 | |
| *** efoley_ has joined #openstack-neutron | 08:17 | |
| *** rossella_s has joined #openstack-neutron | 08:17 | |
| *** thorst has quit IRC | 08:17 | |
| *** MoguCloud1 has joined #openstack-neutron | 08:17 | |
| *** Jack_Iv has joined #openstack-neutron | 08:18 | |
| *** MoguCloud has quit IRC | 08:19 | |
| *** MoguCloud1 is now known as MoguCloud | 08:19 | |
| openstackgerrit | David Shaughnessy proposed openstack/neutron master: Implement QoS OpenFlowFunction for DSCP https://review.openstack.org/399680 | 08:20 |
| *** efoley has quit IRC | 08:20 | |
| *** ralonsoh has joined #openstack-neutron | 08:21 | |
| *** aojea has joined #openstack-neutron | 08:26 | |
| *** davidsha has joined #openstack-neutron | 08:33 | |
| *** Jack_Iv has quit IRC | 08:33 | |
| *** Jack_Iv has joined #openstack-neutron | 08:34 | |
| *** suresh12 has joined #openstack-neutron | 08:37 | |
| *** Jack_Iv has quit IRC | 08:39 | |
| *** suresh12 has quit IRC | 08:41 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron master: DO NOT MERGE: flush_on_subtransaction=True https://review.openstack.org/455159 | 08:42 |
| *** tritri has joined #openstack-neutron | 08:42 | |
| *** john-davidge has joined #openstack-neutron | 08:42 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/networking-midonet master: TEST FOR CI DO NOT REVIEW https://review.openstack.org/199387 | 08:42 |
| tritri | Hi there, I'm looking for someone that could help with the network part of my openstack ocata installation, I can't figure out how to correctly create the network provider and selfservice, please ping me... I don't know if I can bother this chan with that though ^^ | 08:44 |
| *** iranzo has quit IRC | 08:47 | |
| *** gcheresh has joined #openstack-neutron | 08:49 | |
| *** pjm6 has quit IRC | 08:49 | |
| *** ociuhandu has quit IRC | 08:51 | |
| *** janki has quit IRC | 08:53 | |
| *** iranzo has joined #openstack-neutron | 08:54 | |
| *** iranzo has quit IRC | 08:54 | |
| *** iranzo has joined #openstack-neutron | 08:54 | |
| *** ircuser-1 has joined #openstack-neutron | 08:57 | |
| davidsha | reedip: ping | 08:58 |
| *** TMM has joined #openstack-neutron | 08:59 | |
| *** ociuhandu has joined #openstack-neutron | 09:00 | |
| reedip | davidsha : hi | 09:00 |
| reedip | whazzup ? | 09:00 |
| davidsha | reedip: hey how are you keeping? | 09:01 |
| reedip | davidsha : just came back ... I am fine, how are u ? | 09:01 |
| reedip | :) | 09:01 |
| davidsha | reedip: good, just checking emails and reviewing patches! | 09:01 |
| reedip | davidsha : oh good .. I am just starting my monday :) ... have some patches to push | 09:02 |
| davidsha | reedip: You attend the Tap-aas meetings? I'm just wondering when there on. | 09:02 |
| *** masaki has quit IRC | 09:03 | |
| davidsha | reedip: Same had things to rebase, not too bad just waiting on jenkins to get back now... | 09:03 |
| reedip | davidsha : its on Wednesday . Its at about 0630 UTC | 09:04 |
| davidsha | reedip: as opposed to 1830 UTC? | 09:04 |
| reedip | yeah ., 0630 UTC | 09:05 |
| reedip | too early for you I guess davidsha | 09:05 |
| davidsha | kk, No I'll be up just wanted to make sure. Just wanted to plug the Flow manager spec, but a rebase up last week but no reviews so far. | 09:06 |
| reedip | davidsha :ohh , wrt TaaS ? | 09:07 |
| reedip | I have the spec in my ToDo list, will review tonight | 09:07 |
| davidsha | reedip: Thanks, The spec had more or less been approved it was just waiting for armando to review but it went into merge conflict and was forgotten. | 09:08 |
| reedip | oh ok | 09:09 |
| *** mickeys has joined #openstack-neutron | 09:10 | |
| reedip | davidsha : so you are looking to use TaaS in the spec? | 09:10 |
| *** tuanluong has quit IRC | 09:10 | |
| davidsha | reedip: Taas is mentioned in the spec and was one of the use cases. | 09:13 |
| reedip | yeah, we discussed the same in the PTG | 09:13 |
| *** thorst has joined #openstack-neutron | 09:14 | |
| *** ociuhandu has quit IRC | 09:14 | |
| *** mickeys has quit IRC | 09:14 | |
| tritri | sorry to bother you, but would any of you guys have time to help me understand how to configure openstack's network they way I would like to ? I can't seem to figure it out | 09:15 |
| davidsha | tritri: Try #openstack , this irc is for developers. | 09:16 |
| tritri | oh ok, I was wondering about that =p | 09:16 |
| tritri | good day =) | 09:16 |
| *** tritri has left #openstack-neutron | 09:16 | |
| reedip | ralonsoh : ping | 09:17 |
| *** pjm6 has joined #openstack-neutron | 09:17 | |
| reedip | davidsha : some patches in neutron-lib and neutron-fwaas are not showing Jenkins results | 09:17 |
| reedip | and the patches were updated 3-4 days ago | 09:17 |
| *** ociuhandu has joined #openstack-neutron | 09:18 | |
| reedip | is there any known issue? | 09:18 |
| davidsha | reedip: give me a sec I'll try and check | 09:18 |
| *** thorst has quit IRC | 09:18 | |
| ralonsoh | reedip: pong | 09:19 |
| reedip | ralonsoh : regarding https://review.openstack.org/#/c/453110/2 | 09:19 |
| reedip | ralonsoh : Firewall just needs the keys, they would be converted into key-value pair by the agent in neutron-fwaas | 09:20 |
| reedip | thats why we do not need the dict | 09:20 |
| ralonsoh | reedip: I know. list(constants.IPTABLES_PROTOCOL_MAP.keys()) | 09:20 |
| ralonsoh | reedip: what I'm saying is to use this instead the generator you are using | 09:21 |
| ralonsoh | reedip: that's all | 09:21 |
| reedip | and regarding movement to neutron-lib/constants, I am not sure if they should be moved as they are not global constants but local to firewall. So I thinok we can move the constants to neutron-lib/api/definitions/constants.py | 09:21 |
| ralonsoh | reedip: I agree. If there is other extension using it, maybe you can propose it. But now keep it there | 09:22 |
| reedip | oh , ok ralonsoh, I would also need the None, but I guess I can just do list(constants.IPTABLES_PROTOCOL_MAP.keys()) + [None] | 09:22 |
| ralonsoh | reedip: yes, that was in my comment | 09:22 |
| ralonsoh | reedip: PROTOCOL_VALUES = list(constants.IPTABLES_PROTOCOL_MAP.keys()) + [None] | 09:22 |
| reedip | ralonsoh : by the way, are we getting any advantage of using the method you stated over my method? I would appreciate if you can guide me | 09:22 |
| *** limao has quit IRC | 09:23 | |
| ralonsoh | reedip: faster and cleaner? But I need to check if this is faster | 09:23 |
| reedip | davidsha : thanks for the recheck but I cant see it on Zuul.... | 09:23 |
| ralonsoh | reedip: give me 2 mins | 09:23 |
| reedip | ralonsoh : sure | 09:23 |
| reedip | something is wrong ralonsoh ( maybe my patches arent going to Zuul :P ) | 09:24 |
| reedip | sorry that was for davidsha ^^^\ | 09:24 |
| ralonsoh | reedip: which one? | 09:24 |
| davidsha | reedip: Looks like the tests all stopped running sometime 2 days ago http://grafana.openstack.org/dashboard/db/neutron-lib-failure-rate | 09:25 |
| reedip | davidsha : thats weird ... havent any bug been logged for this ? | 09:27 |
| davidsha | reedip: doesn't look like it | 09:28 |
| davidsha | kevinbenton: ping | 09:28 |
| reedip | davidsha, kevinbenton : last patch to have been addressed by neutron-lib is https://review.openstack.org/#/c/404488/ | 09:29 |
| *** sambetts|afk is now known as sambetts | 09:29 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Docs: Do not turn warnings into errors https://review.openstack.org/455182 | 09:32 |
| openstackgerrit | Lujin Luo proposed openstack/neutron master: [WIP] Integration of Router OVO https://review.openstack.org/421863 | 09:34 |
| openstackgerrit | Reedip proposed openstack/neutron-lib master: Add new protocols in Firewalls https://review.openstack.org/453110 | 09:35 |
| reedip | davidsha : should we log a bug ? | 09:35 |
| *** janki has joined #openstack-neutron | 09:42 | |
| reedip | ralonsoh : one more query related to https://review.openstack.org/#/c/442375/ | 09:43 |
| *** Jack_Iv has joined #openstack-neutron | 09:44 | |
| reedip | ralonsoh : I think there is no additional TC to be added as per your last comment | 09:44 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/networking-ovn master: Docs: Uses DevStack script to create the user https://review.openstack.org/455186 | 09:44 |
| *** suresh12 has joined #openstack-neutron | 09:45 | |
| *** suresh12 has quit IRC | 09:45 | |
| *** suresh12 has joined #openstack-neutron | 09:45 | |
| ralonsoh | reedip: you are modifying update_policy_rule and create_policy_rule, but you are only testing the update function | 09:46 |
| *** jhershbe has joined #openstack-neutron | 09:46 | |
| davidsha | reedip: sorry, I'd wait till kb responds. | 09:48 |
| *** trinaths has left #openstack-neutron | 09:51 | |
| *** bzhao has quit IRC | 09:55 | |
| *** bzhao has joined #openstack-neutron | 09:56 | |
| *** aarefiev_afk is now known as aarefiev | 09:59 | |
| *** vijaykc4 has quit IRC | 10:00 | |
| *** john-dav_ has joined #openstack-neutron | 10:00 | |
| *** john-davidge has quit IRC | 10:00 | |
| *** john-dav_ is now known as john-davidge | 10:01 | |
| *** iwamoto has quit IRC | 10:01 | |
| *** obondarev has quit IRC | 10:02 | |
| *** boden has joined #openstack-neutron | 10:05 | |
| reedip | ralonsoh : okay , so I need to add the create_policy , right? | 10:10 |
| *** sdague has joined #openstack-neutron | 10:10 | |
| reedip | davidsha : ok, no issues | 10:10 |
| *** mickeys has joined #openstack-neutron | 10:11 | |
| *** vijaykc4 has joined #openstack-neutron | 10:11 | |
| ralonsoh | reedip: I think so. You need to add those tests too | 10:11 |
| reedip | ralonsoh : but one question. | 10:11 |
| reedip | ralonsoh : can we link a QoS rule during QoS policy creation ? | 10:11 |
| reedip | I mean can we create a QoS policy while passing the rule in it? the test cases dont seem to do so as of now. | 10:12 |
| ralonsoh | reedip: no, you need to create the QoS policy first. Then you can create a rule associated to this policy | 10:14 |
| *** hoangcx has quit IRC | 10:15 | |
| reedip | ralonsoh : but thats like Updating the QoS policy , right :) ( In a way you are updating it ) | 10:15 |
| *** mickeys has quit IRC | 10:16 | |
| *** gcb has quit IRC | 10:16 | |
| reedip | ralonsoh : by the way, I have already updated the test_create_qos_policy funtion in the current patch | 10:17 |
| *** bobmel_ has quit IRC | 10:18 | |
| ralonsoh | reedip: let me check the tests again. But I think you are never using QosPlugin.create_policy_rule | 10:18 |
| reedip | ralonsoh : you can check test_create_policy_rule | 10:18 |
| *** abregman has quit IRC | 10:19 | |
| reedip | I am mocking check_bandwidth_limit_rule and returning None , because there are no rules associuated to the the Policy at creation time | 10:19 |
| reedip | s/check_bandwidth_limit_rule/check_bandwidth_rule_conflict | 10:19 |
| *** vijaykc4 has quit IRC | 10:21 | |
| *** nicolasbock has joined #openstack-neutron | 10:22 | |
| *** reedip has quit IRC | 10:23 | |
| *** reedip has joined #openstack-neutron | 10:23 | |
| *** bobmel has joined #openstack-neutron | 10:24 | |
| ralonsoh | reedip: you can add a new rule (e.g. max) to a policy with an existing rule (e.g. min). In this case, you can check your rule validator | 10:24 |
| slaweq | ajo: hello | 10:25 |
| reedip | ralonsoh : but isnt it updating the policy ? | 10:25 |
| reedip | I am doing this in the update_policy tests | 10:25 |
| *** ociuhandu has quit IRC | 10:25 | |
| slaweq | ajo: do You remember about review patch with "improved rules validation"? | 10:25 |
| *** cuongnv has quit IRC | 10:26 | |
| ralonsoh | reedip: yes, you are right. My point is you never use QosPlugin.create_policy_rule. Don't worry, I think this is better with an example. Can I submit a new PS with some examples in you patch? | 10:28 |
| *** jhershbe has quit IRC | 10:28 | |
| ralonsoh | reedip: if you don't mind | 10:28 |
| openstackgerrit | Reedip proposed openstack/neutron master: Check DefaultPrefixLength in Subnetpools https://review.openstack.org/447983 | 10:28 |
| reedip | ralonsoh : no I dont mind. Please do it, will help me understand what I am missing out | 10:28 |
| reedip | :) | 10:28 |
| ralonsoh | reedip: thanks! | 10:28 |
| reedip | thanks for your help | 10:28 |
| *** bobmel has quit IRC | 10:29 | |
| *** bobmel has joined #openstack-neutron | 10:29 | |
| openstackgerrit | Reedip proposed openstack/neutron-lib master: Add CIDR for canonical format conversion https://review.openstack.org/432521 | 10:32 |
| openstackgerrit | Reedip proposed openstack/neutron master: Add ipv6 canonical address converter to API https://review.openstack.org/267283 | 10:32 |
| *** NikhilS has quit IRC | 10:33 | |
| openstackgerrit | Reedip proposed openstack/neutron master: Pass the complete info in sg/rules db into PRECOMMIT_XXX callback https://review.openstack.org/448420 | 10:33 |
| *** rossella_s has quit IRC | 10:33 | |
| *** rossella_s has joined #openstack-neutron | 10:34 | |
| *** udesale has quit IRC | 10:37 | |
| *** psahoo has quit IRC | 10:38 | |
| *** Jack_Iv has quit IRC | 10:39 | |
| *** Jack_Iv has joined #openstack-neutron | 10:39 | |
| *** gcheresh has quit IRC | 10:40 | |
| *** psahoo has joined #openstack-neutron | 10:41 | |
| *** obondarev has joined #openstack-neutron | 10:42 | |
| *** vijaykc4 has joined #openstack-neutron | 10:44 | |
| *** wu_ng has quit IRC | 10:46 | |
| *** masaki has joined #openstack-neutron | 10:48 | |
| *** mdnadeem has quit IRC | 10:48 | |
| *** masaki has quit IRC | 10:48 | |
| *** markstur has joined #openstack-neutron | 10:50 | |
| *** Jack_Iv has quit IRC | 10:51 | |
| *** Jack_Iv has joined #openstack-neutron | 10:52 | |
| *** mdnadeem has joined #openstack-neutron | 10:53 | |
| *** markstur has quit IRC | 10:55 | |
| *** ociuhandu has joined #openstack-neutron | 10:55 | |
| *** Jack_Iv has quit IRC | 10:56 | |
| *** vijaykc4 has quit IRC | 10:56 | |
| *** jhershbe has joined #openstack-neutron | 10:58 | |
| *** bobmel has quit IRC | 11:04 | |
| *** sdague_ has joined #openstack-neutron | 11:04 | |
| *** ZZelle_ has quit IRC | 11:06 | |
| *** jamesdenton has joined #openstack-neutron | 11:07 | |
| openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Use neutron-lib definition of neutron-fwaas API https://review.openstack.org/421472 | 11:07 |
| *** jhershbe has quit IRC | 11:08 | |
| *** venki1 has joined #openstack-neutron | 11:08 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bagpipe master: Add fullstack tests framework https://review.openstack.org/408008 | 11:11 |
| *** suresh12 has quit IRC | 11:11 | |
| *** suresh12 has joined #openstack-neutron | 11:12 | |
| venki1 | hi guys, has anyone tried android as instance in Openstack ? | 11:12 |
| *** jamesdenton has quit IRC | 11:12 | |
| *** Jack_Iv has joined #openstack-neutron | 11:13 | |
| *** Jack_Iv has quit IRC | 11:13 | |
| *** Jack_Iv has joined #openstack-neutron | 11:13 | |
| *** Jack_Iv has quit IRC | 11:15 | |
| *** lucasagomes is now known as lucas-hungry | 11:15 | |
| *** Jack_Iv has joined #openstack-neutron | 11:15 | |
| *** ZZelle has joined #openstack-neutron | 11:15 | |
| *** thorst has joined #openstack-neutron | 11:15 | |
| *** ZZelle is now known as Guest85867 | 11:16 | |
| *** suresh12 has quit IRC | 11:16 | |
| *** thorst has quit IRC | 11:20 | |
| *** jamielennox|away is now known as jamielennox | 11:25 | |
| *** gcheresh has joined #openstack-neutron | 11:26 | |
| *** alegacy has quit IRC | 11:26 | |
| *** jprovazn has joined #openstack-neutron | 11:32 | |
| *** nplanel has quit IRC | 11:33 | |
| *** vijaykc4 has joined #openstack-neutron | 11:34 | |
| openstackgerrit | Thomas Morin proposed openstack/networking-bagpipe master: bagpipe-bgp: follow exabgp peer down change https://review.openstack.org/455273 | 11:38 |
| *** yedongcan has left #openstack-neutron | 11:38 | |
| *** Guest70770 is now known as rook | 11:39 | |
| *** anilvenkata_afk is now known as anilvenkata | 11:43 | |
| *** venki1 has quit IRC | 11:53 | |
| *** janki has quit IRC | 11:55 | |
| *** thorst has joined #openstack-neutron | 11:55 | |
| *** stevemar has joined #openstack-neutron | 11:57 | |
| *** mickeys has joined #openstack-neutron | 12:05 | |
| *** venki has joined #openstack-neutron | 12:08 | |
| *** mickeys has quit IRC | 12:09 | |
| *** markvoelker has quit IRC | 12:10 | |
| *** vijaykc4 has quit IRC | 12:12 | |
| *** lucas-hungry is now known as lucasagomes | 12:12 | |
| *** vijaykc4 has joined #openstack-neutron | 12:13 | |
| *** ranjithd has joined #openstack-neutron | 12:17 | |
| *** ltomasbo is now known as ltomasbo|away | 12:19 | |
| *** ltomasbo|away is now known as ltomasbo | 12:21 | |
| *** vks1 has quit IRC | 12:23 | |
| *** markvoelker has joined #openstack-neutron | 12:25 | |
| *** zhurong has quit IRC | 12:26 | |
| *** hjensas has quit IRC | 12:26 | |
| *** pgadiya has quit IRC | 12:26 | |
| *** lujinluo has quit IRC | 12:27 | |
| *** gouthamr has joined #openstack-neutron | 12:28 | |
| *** haleyb has quit IRC | 12:32 | |
| slaweq | ralonsoh: hello | 12:32 |
| slaweq | ralonsoh: I have question about https://review.openstack.org/#/c/449831/7/neutron/objects/qos/policy.py | 12:32 |
| slaweq | and Your comment there | 12:32 |
| slaweq | do You think it is worth to make this small fix in separate patch to have it merger faster? | 12:33 |
| slaweq | I can do such small patch | 12:33 |
| *** mriedem has joined #openstack-neutron | 12:35 | |
| ralonsoh | slaweq: if you prefer that, you can create a bug for this. For sure this small patch will merge fast. The bug is evident | 12:36 |
| *** catintheroof has joined #openstack-neutron | 12:36 | |
| *** ayogi has quit IRC | 12:37 | |
| openstackgerrit | Ann Taraday proposed openstack/neutron master: [WIP]Use reader and writer for routers and agents https://review.openstack.org/396121 | 12:37 |
| *** tbh has quit IRC | 12:37 | |
| *** radhikam has joined #openstack-neutron | 12:38 | |
| *** tbachman has quit IRC | 12:38 | |
| *** tbachman has joined #openstack-neutron | 12:38 | |
| slaweq | ralonsoh: ok, thx. I will do it today | 12:38 |
| ralonsoh | slaweq: thank you! | 12:38 |
| openstackgerrit | Merged openstack/ovsdbapp master: Refactor to give other schemas a place to live https://review.openstack.org/452924 | 12:44 |
| *** psahoo has quit IRC | 12:49 | |
| *** gcb has joined #openstack-neutron | 12:50 | |
| *** Guest5432 has quit IRC | 12:53 | |
| *** rmart04_ has joined #openstack-neutron | 12:55 | |
| *** rmart04 has quit IRC | 12:56 | |
| *** rmart04_ is now known as rmart04 | 12:56 | |
| *** gcheresh has quit IRC | 12:57 | |
| *** gcheresh has joined #openstack-neutron | 12:58 | |
| openstackgerrit | Daniel Mellado proposed openstack/networking-ovn master: Enhance local.conf.sample documentation https://review.openstack.org/455311 | 13:01 |
| *** fragatina has joined #openstack-neutron | 13:02 | |
| *** rmart04 has quit IRC | 13:05 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn master: docs: Update distro version note. https://review.openstack.org/455314 | 13:08 |
| *** lyan has joined #openstack-neutron | 13:08 | |
| *** rmart04 has joined #openstack-neutron | 13:08 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn master: docs: Remove dpdk-snapshot repo. https://review.openstack.org/455315 | 13:10 |
| *** rmart04 has quit IRC | 13:10 | |
| *** gkadam has quit IRC | 13:10 | |
| *** lyan has quit IRC | 13:12 | |
| *** lyan has joined #openstack-neutron | 13:12 | |
| *** tonytan4ever has joined #openstack-neutron | 13:14 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [WIP] Add QoS bandwidth limit for instance ingress traffic https://review.openstack.org/449831 | 13:15 |
| *** claudiub has joined #openstack-neutron | 13:15 | |
| *** haleyb has joined #openstack-neutron | 13:15 | |
| openstackgerrit | Carlos Goncalves proposed openstack/neutron master: Port data plane status extension implementation https://review.openstack.org/424340 | 13:16 |
| *** mdnadeem has quit IRC | 13:18 | |
| *** baoli has joined #openstack-neutron | 13:20 | |
| *** swebster has quit IRC | 13:20 | |
| openstackgerrit | Russell Bryant proposed openstack/networking-ovn master: Support connecting OVN DB over SSL https://review.openstack.org/424969 | 13:22 |
| *** rmart04 has joined #openstack-neutron | 13:23 | |
| *** Administrator_ has quit IRC | 13:23 | |
| *** Administrator_ has joined #openstack-neutron | 13:24 | |
| *** stanzgy has quit IRC | 13:25 | |
| *** rmart04 has quit IRC | 13:25 | |
| *** Guest85867 is now known as ZZelle_ | 13:26 | |
| *** ZZelle_ is now known as ZZelle | 13:26 | |
| *** rmart04 has joined #openstack-neutron | 13:28 | |
| boden | FYI: neutron-lib 1.4.0 was recently released: https://pypi.python.org/pypi/neutron-lib/1.4.0 | 13:30 |
| *** cleong has joined #openstack-neutron | 13:30 | |
| *** lewo has quit IRC | 13:30 | |
| *** lewo has joined #openstack-neutron | 13:30 | |
| *** chlong has joined #openstack-neutron | 13:31 | |
| openstackgerrit | Merged openstack/networking-ovn master: Docs: Uses DevStack script to create the user https://review.openstack.org/455186 | 13:31 |
| *** tbachman_ has joined #openstack-neutron | 13:31 | |
| *** rmart04 has quit IRC | 13:32 | |
| *** rmart04 has joined #openstack-neutron | 13:32 | |
| *** tbachman has quit IRC | 13:33 | |
| *** tbachman_ is now known as tbachman | 13:33 | |
| *** openstackgerrit has quit IRC | 13:33 | |
| *** erolg has joined #openstack-neutron | 13:34 | |
| *** janzian has joined #openstack-neutron | 13:34 | |
| *** gcheresh has quit IRC | 13:35 | |
| *** rmart04_ has joined #openstack-neutron | 13:37 | |
| *** rmart04 has quit IRC | 13:38 | |
| *** rmart04_ is now known as rmart04 | 13:38 | |
| slaweq | ralonsoh: https://bugs.launchpad.net/neutron/+bug/1681440 | 13:48 |
| openstack | Launchpad bug 1681440 in neutron "QoS policy object can't be suitable with 1.2 version of object" [Undecided,New] - Assigned to Slawek Kaplonski (slaweq) | 13:48 |
| *** crushil has joined #openstack-neutron | 13:51 | |
| ralonsoh | slaweq: I'll review the patch as soon as you submit it | 13:51 |
| *** trevormc has joined #openstack-neutron | 13:52 | |
| *** sdague has quit IRC | 13:52 | |
| *** baoli has quit IRC | 13:52 | |
| *** xinliang has quit IRC | 13:52 | |
| *** xinliang has joined #openstack-neutron | 13:53 | |
| *** vijaykc4 has quit IRC | 13:54 | |
| *** kbringard has joined #openstack-neutron | 13:55 | |
| *** sdague has joined #openstack-neutron | 13:56 | |
| *** sdague__ has joined #openstack-neutron | 13:56 | |
| *** MoguCloud1 has joined #openstack-neutron | 14:02 | |
| *** MoguCloud has quit IRC | 14:02 | |
| *** MoguCloud1 is now known as MoguCloud | 14:02 | |
| *** baoli has joined #openstack-neutron | 14:03 | |
| *** dave-mccowan has joined #openstack-neutron | 14:04 | |
| slaweq | ralonsoh: thx | 14:04 |
| slaweq | I will send this patch today for sure | 14:05 |
| *** baoli has quit IRC | 14:08 | |
| *** radhikam has quit IRC | 14:08 | |
| *** dave-mccowan has quit IRC | 14:09 | |
| *** openstackgerrit has joined #openstack-neutron | 14:10 | |
| openstackgerrit | Terry Wilson proposed openstack/ovsdbapp master: Add unit tests from Neutron https://review.openstack.org/454361 | 14:11 |
| *** baoli has joined #openstack-neutron | 14:12 | |
| *** suresh12 has joined #openstack-neutron | 14:12 | |
| *** sdague has quit IRC | 14:12 | |
| *** sdague__ is now known as sdague | 14:13 | |
| trevormc | is there any way to do traffic-protection, like allowing or disabling unicast, multicast, or broadcast? I saw this wiki but found it wasn't implemented in neutron, https://wiki.openstack.org/wiki/Neutron/TrafficProtection | 14:14 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Make QoS policy object compatible with versions 1.2 and higher https://review.openstack.org/455338 | 14:15 |
| slaweq | ralonsoh: ^^ here You go :) | 14:15 |
| ralonsoh | slaweq: perfect | 14:16 |
| *** ankur-gupta-f1 has joined #openstack-neutron | 14:17 | |
| *** tidwellr has joined #openstack-neutron | 14:17 | |
| *** ataraday_ has joined #openstack-neutron | 14:22 | |
| *** zkassab has joined #openstack-neutron | 14:22 | |
| *** MoguCloud has quit IRC | 14:22 | |
| *** MoguCloud has joined #openstack-neutron | 14:23 | |
| *** Drankis has joined #openstack-neutron | 14:23 | |
| *** jlinkes has quit IRC | 14:26 | |
| *** markstur has joined #openstack-neutron | 14:27 | |
| *** rkukura has joined #openstack-neutron | 14:27 | |
| *** lucasxu has joined #openstack-neutron | 14:27 | |
| *** ihrachys has joined #openstack-neutron | 14:27 | |
| *** jlinkes has joined #openstack-neutron | 14:28 | |
| *** fragatina has quit IRC | 14:29 | |
| *** Drankis has quit IRC | 14:29 | |
| *** ihrachys has quit IRC | 14:30 | |
| *** ihrachys has joined #openstack-neutron | 14:30 | |
| *** markstur has quit IRC | 14:31 | |
| *** rossella_s has quit IRC | 14:33 | |
| *** rossella_s has joined #openstack-neutron | 14:34 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Add check for Bandwidth Limit Rules https://review.openstack.org/442375 | 14:35 |
| *** Jack_Iv has quit IRC | 14:38 | |
| *** Jack_Iv has joined #openstack-neutron | 14:38 | |
| ralonsoh | reedip: ^^ | 14:39 |
| *** hjensas has joined #openstack-neutron | 14:40 | |
| *** suresh12 has quit IRC | 14:41 | |
| *** vks1 has joined #openstack-neutron | 14:46 | |
| ralonsoh | trevormc: I think this is an old spec (feb 14), not accepted | 14:46 |
| trevormc | true, but I was wondering if it was implemented through some other agent, like firewall or l2pop | 14:46 |
| ralonsoh | trevormc: not exactly. You have some protections in firewall (arp, mac-spoofing, etc) | 14:48 |
| ralonsoh | trevormc: but not this parameters (unicast, multicast, broadcast) | 14:48 |
| ralonsoh | trevormc: I suppose this if for the VF agent | 14:48 |
| trevormc | ralonsoh, thanks yeah it is. I was in the last drivers meeting and they said I should derive my API from existing ones. | 14:49 |
| trevormc | so for those rules, they could go into secgroups or firewall I suppose. | 14:49 |
| ralonsoh | firewall will read from secgroups | 14:50 |
| *** vks1 has quit IRC | 14:50 | |
| *** baoli has quit IRC | 14:51 | |
| *** armax has joined #openstack-neutron | 14:52 | |
| *** amarao has quit IRC | 14:55 | |
| *** korzen has quit IRC | 14:55 | |
| jlibosva | ihrachys: hi, do you have a minute to consult the dhcp rpc notification ignored? You said something about ignoring message if host is different? | 14:56 |
| *** scottda has joined #openstack-neutron | 14:57 | |
| *** rmart04 has quit IRC | 14:58 | |
| *** yamahata has quit IRC | 15:00 | |
| *** yamahata has joined #openstack-neutron | 15:00 | |
| *** suresh12 has joined #openstack-neutron | 15:00 | |
| ihrachys | jlibosva, I think things like https://github.com/openstack/neutron/blob/master/neutron/agent/dhcp/agent.py#L365 were skipping ports if not hosted or smth | 15:01 |
| ihrachys | would need time to dig | 15:01 |
| ihrachys | in a meeting now | 15:01 |
| *** nplanel has joined #openstack-neutron | 15:01 | |
| *** blogan has joined #openstack-neutron | 15:01 | |
| *** manheim has quit IRC | 15:01 | |
| erolg | hi everyone, i am try to run brocade vrouter neutron plugin. When I try to start, i am getting "no such option in group DEFAULT: nova_admin_auth_url" from oslo_config. I would say it is deprecated parameter but how can i reuse it? | 15:01 |
| *** fnaval has joined #openstack-neutron | 15:02 | |
| *** rcernin has quit IRC | 15:03 | |
| *** fnaval_ has joined #openstack-neutron | 15:04 | |
| *** baoli has joined #openstack-neutron | 15:04 | |
| *** fnaval__ has joined #openstack-neutron | 15:06 | |
| *** vks1 has joined #openstack-neutron | 15:06 | |
| *** fnaval has quit IRC | 15:08 | |
| ThiagoCMC | Hey guys, anyone using Ocata's Networking OVN with both GENEVE and Provider Networks on top of DPDK? | 15:08 |
| *** aojea has quit IRC | 15:09 | |
| *** udesale has joined #openstack-neutron | 15:09 | |
| *** nplanel has quit IRC | 15:09 | |
| *** fnaval_ has quit IRC | 15:10 | |
| *** dosaboy has quit IRC | 15:10 | |
| *** baoli has quit IRC | 15:10 | |
| *** voelzmo has quit IRC | 15:13 | |
| davidsha | ThiagoCMC: might be better to ask this in #openstack , This chat is for developers. | 15:13 |
| *** crushil has quit IRC | 15:13 | |
| *** voelzmo has joined #openstack-neutron | 15:13 | |
| *** chyka has joined #openstack-neutron | 15:14 | |
| davidsha | erolg: try asking in #openstack , this seems like a deployment issue rather than development. | 15:16 |
| *** mosulica has quit IRC | 15:16 | |
| ZZelle | #lxcontainers | 15:16 |
| erolg | davidsha, thanks. | 15:17 |
| davidsha | erolg: no problem. | 15:17 |
| ralonsoh | yamamoto_: hi | 15:18 |
| *** voelzmo has quit IRC | 15:18 | |
| *** udesale has quit IRC | 15:18 | |
| *** grantma__ has joined #openstack-neutron | 15:19 | |
| *** wolverineav has joined #openstack-neutron | 15:19 | |
| *** xiaohhui has quit IRC | 15:20 | |
| *** anilvenkata has quit IRC | 15:21 | |
| *** ventifus has joined #openstack-neutron | 15:22 | |
| *** sridharg has quit IRC | 15:22 | |
| *** dosaboy has joined #openstack-neutron | 15:23 | |
| *** lyan has quit IRC | 15:23 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Fix duplicated sg rules check for IPv6 ICMP https://review.openstack.org/427670 | 15:23 |
| *** chyka has quit IRC | 15:24 | |
| *** chyka has joined #openstack-neutron | 15:25 | |
| *** vijaykc4 has joined #openstack-neutron | 15:26 | |
| vks1 | hi folks, I am getting this error with ovs-agent: Exception: Could not retrieve schema from tcp:127.0.0.1:6640: Connection refused | 15:26 |
| vks1 | Unable to execute ['ovs-vsctl', '--timeout=10', '--oneline', '--format=json', '--', '--id=@manager', 'create', 'Manager', 'target="ptcp:6640:127.0.0.1"', '--', 'add', 'Open_vSwitch', '.', 'manager_options', '@manager', '--', 'set', 'Manager', 'ptcp:6640:127.0.0.1', 'inactivity_probe=10000']. Exception: Exit code: -14; Stdin: ; Stdout: ; Stderr: 2017-04-10T15:24:35Z|00002|fatal_signal|WARN|terminating with signal 14 (Alarm clock) | 15:27 |
| *** aarefiev is now known as aarefiev_afk | 15:27 | |
| *** vijaykc4 has quit IRC | 15:30 | |
| *** fnaval has joined #openstack-neutron | 15:31 | |
| davidsha | vks1: Is the agent crashing after the error or does it just keep going? | 15:31 |
| vks1 | davidsha: I am using devstack. Its crashing and agent is also not running | 15:32 |
| davidsha | vks1: The problem is you can only have one openvswitch manager and you're old one seems to have not been deleted, try unstacking and see if that helps | 15:33 |
| vks1 | davidsha: I have tried that but of no use | 15:34 |
| *** fnaval__ has quit IRC | 15:35 | |
| davidsha | vks1: try sudo ovs-vsctl del-manager | 15:35 |
| *** thorst is now known as thorst_afk | 15:35 | |
| davidsha | when you try and start the agent it should create a new manager now. | 15:36 |
| *** crushil has joined #openstack-neutron | 15:36 | |
| vks1 | davidsha: let me try | 15:37 |
| *** mlavalle has joined #openstack-neutron | 15:37 | |
| *** cshastri has quit IRC | 15:38 | |
| *** vijaykc4 has joined #openstack-neutron | 15:38 | |
| vks1 | davidsha: deleted , will rebuild | 15:38 |
| davidsha | kk | 15:38 |
| *** TMM has quit IRC | 15:40 | |
| *** baoli has joined #openstack-neutron | 15:40 | |
| *** jlinkes has quit IRC | 15:41 | |
| *** oreillyd has quit IRC | 15:42 | |
| *** oreillyd_ has quit IRC | 15:42 | |
| vks1 | davidsha: this is weird , same error. error logs http://paste.openstack.org/show/606059/ | 15:45 |
| *** lpetrut has quit IRC | 15:47 | |
| *** obondarev has quit IRC | 15:48 | |
| *** obondarev has joined #openstack-neutron | 15:49 | |
| *** manheim has joined #openstack-neutron | 15:50 | |
| *** tidwellr has quit IRC | 15:52 | |
| *** tidwellr has joined #openstack-neutron | 15:53 | |
| davidsha | vks1: Are you working inside a proxy? | 15:53 |
| vks1 | davidsha: no | 15:54 |
| davidsha | can you paste "ovs-vsctl get-manager"? | 15:54 |
| *** manheim has quit IRC | 15:55 | |
| ralonsoh | ihrachys: hi | 15:55 |
| ihrachys | ralonsoh, hi | 15:56 |
| ralonsoh | ihrachys: I saw the comment in https://review.openstack.org/#/c/454220/ | 15:56 |
| ihrachys | oh right. and I always mix your nick with rodrigods so I pinged a wrong person in a meeting ;) | 15:56 |
| ralonsoh | ihrachys: sorry for that. I don't understand you question | 15:57 |
| vks1 | davidsha: it looks OK, ptcp:6640:127.0.0.1 | 15:57 |
| ihrachys | ralonsoh, first step would be updating the bug report to talk in terms of use cases and what you try to solve | 15:57 |
| *** seanatcisco has joined #openstack-neutron | 15:57 | |
| *** iyamahat has joined #openstack-neutron | 15:57 | |
| *** sterdnotshaken has joined #openstack-neutron | 15:58 | |
| ihrachys | atm it's worded in such a way that puzzles some folks (including me) about what's the end result from user perspective | 15:58 |
| ralonsoh | ihrachys: ok, mainly to fix the problem we had with QoS objects | 15:58 |
| ihrachys | ralonsoh, a patch showing the issue may have been handy | 15:59 |
| *** marst has joined #openstack-neutron | 15:59 | |
| davidsha | vks1: thats the problem, try "ovs-vsctl del-manager ptcp:6640:127.0.0.1" | 15:59 |
| ralonsoh | ihrachys: ? but you make a comment in the patch | 15:59 |
| ralonsoh | ihrachys: https://review.openstack.org/#/c/454220/ | 15:59 |
| davidsha | and make sure it deleted with "ovs-vsctl get-manager" | 15:59 |
| vks1 | davidsha: ok, but I deleted | 15:59 |
| davidsha | Does it show there is no manager? | 16:00 |
| *** lucasxu has quit IRC | 16:00 | |
| ihrachys | ralonsoh, yes I do. I ask for explanation of what you try to solve (in commit message or in the bug report) | 16:00 |
| ralonsoh | ihrachys: ok, I'll amend the bug description | 16:01 |
| *** mickeys has joined #openstack-neutron | 16:01 | |
| ihrachys | ralonsoh, oh you mean that's the patch that exposes the error? I am not sure I understand. this is the patch that refactors the code | 16:01 |
| ihrachys | but not that shows there is anything wrong with current code | 16:01 |
| ihrachys | ralonsoh, one thing that I don't get about the bug report, it talks in terms of micro-versioning that allegedly is used in neutron, but there is no such thing in neutron (assuming we talk about the same - api micro-versioning) | 16:02 |
| ralonsoh | ihrachys: I know neutron doesn't use it. So, so you think the bug is valid? | 16:03 |
| *** sterdnotshaken1 has joined #openstack-neutron | 16:03 | |
| *** XinBai has joined #openstack-neutron | 16:03 | |
| ihrachys | ralonsoh, I don't understand the use case that fails, so I can't judge right away | 16:04 |
| *** Jack_Iv has quit IRC | 16:04 | |
| *** Jack_Iv has joined #openstack-neutron | 16:04 | |
| ralonsoh | ihrachys: ok. If I'm not able to find this case, I'll mark this bug as invalid. | 16:04 |
| *** mickeys has quit IRC | 16:05 | |
| *** lucasagomes is now known as lucas-afk | 16:05 | |
| ihrachys | ok. don't get me wrong, I just want to understand what spurred davidsha reporting it and you coding :) | 16:05 |
| *** sterdnotshaken has quit IRC | 16:06 | |
| davidsha | ihrachys: Hey, just on this bug, someone was telling me that OvO should be used to modify the field parameter rather than throwing exceptions. an example being: https://github.com/openstack/nova/blob/c9eb9530314d047f5013941ebcfd5ef0192a9dc3/nova/objects/network.py#L90 | 16:06 |
| vks1 | davidsha: It failed I checked "get-manager" and there was nothing | 16:06 |
| *** janzian has quit IRC | 16:07 | |
| vks1 | davidsha: it failed with same error | 16:07 |
| ihrachys | davidsha, nova uses the library in a different way | 16:07 |
| ihrachys | I think | 16:07 |
| davidsha | vks1: I'm not sure then I'm afraid. | 16:07 |
| *** andreas_s has quit IRC | 16:07 | |
| ihrachys | hm, sec, actually... | 16:07 |
| davidsha | ihrachys: kk | 16:07 |
| vks1 | davidsha: thanks I will debug further | 16:08 |
| davidsha | vks1: kk, there is also #openstack that might be of some help. good luck! | 16:08 |
| rodrigods | ihrachys, ralonsoh :) | 16:08 |
| ihrachys | davidsha, ok probably the main piece of the patch is https://review.openstack.org/#/c/454220/2/neutron/objects/qos/policy.py@240? | 16:08 |
| ihrachys | yes, it probably makes sense to be more liberal there | 16:09 |
| *** mickeys has joined #openstack-neutron | 16:09 | |
| ralonsoh | ihrachys, davidsha: yes, and the rule append process | 16:09 |
| ralonsoh | ihrachys, davidsha: to remove rules not supported by the current backend | 16:09 |
| *** dsneddon has joined #openstack-neutron | 16:09 | |
| ihrachys | for rules, we need to think if that's ok to leave part of policy rules not enforced on a port | 16:10 |
| davidsha | ihrachys: When I was reporting the bug I was referring to https://review.openstack.org/#/c/454220/2/neutron/objects/qos/rule.py however ralonsoh said that it can't be modified in the way I was thinking. | 16:10 |
| davidsha | When I reported the bug. | 16:10 |
| *** lyan has joined #openstack-neutron | 16:11 | |
| *** suresh12 has quit IRC | 16:12 | |
| ihrachys | I guess the patch is actually fine (maybe not so much for ruletype but that's details); so maybe ralonsoh just reword the bug report to talk about how neutron fails to apply a qos policy with null description for old agents | 16:12 |
| ihrachys | that would be a clear use case | 16:12 |
| ralonsoh | ihrachys: perfect! | 16:13 |
| *** tidwellr has quit IRC | 16:13 | |
| ihrachys | data plane should not silently break on description being set to null | 16:13 |
| ihrachys | for rules, we will assess separately | 16:13 |
| ihrachys | I would go as far as to say those are separate bugs | 16:13 |
| ihrachys | but that's just me | 16:13 |
| ralonsoh | ihrachys: ok, to treat rules I'll open other bug | 16:14 |
| ihrachys | perfect | 16:14 |
| ihrachys | and then we can speed up landing of description change since it should be very simple | 16:14 |
| *** reedip_ has joined #openstack-neutron | 16:15 | |
| *** ankur-gupta-f1 has quit IRC | 16:17 | |
| *** obondarev has quit IRC | 16:21 | |
| *** ltomasbo is now known as ltomasbo|away | 16:22 | |
| *** tmorin has quit IRC | 16:22 | |
| *** tidwellr has joined #openstack-neutron | 16:23 | |
| openstackgerrit | Octave Orgeron proposed openstack/neutron master: Enables MySQL Cluster Support for Neutron https://review.openstack.org/446136 | 16:24 |
| *** vijaykc4 has quit IRC | 16:26 | |
| *** davidsha has quit IRC | 16:26 | |
| openstackgerrit | Igor Duarte Cardoso proposed openstack/networking-sfc master: Replace space with underscore in doc file https://review.openstack.org/455381 | 16:28 |
| *** janzian has joined #openstack-neutron | 16:29 | |
| *** vijaykc4 has joined #openstack-neutron | 16:31 | |
| *** jlibosva has quit IRC | 16:31 | |
| *** bcafarel has quit IRC | 16:32 | |
| *** tonytan4ever has quit IRC | 16:35 | |
| *** grantma__ has quit IRC | 16:35 | |
| *** ataraday_ has quit IRC | 16:41 | |
| *** blogan has quit IRC | 16:43 | |
| *** john-davidge has quit IRC | 16:44 | |
| *** john-dav_ has joined #openstack-neutron | 16:44 | |
| *** m1dev has joined #openstack-neutron | 16:44 | |
| *** g3ek has quit IRC | 16:47 | |
| *** jaosorior is now known as jaosorior_away | 16:51 | |
| *** aojea has joined #openstack-neutron | 16:51 | |
| *** tesseract has quit IRC | 16:52 | |
| *** markstur has joined #openstack-neutron | 16:53 | |
| *** g3ek has joined #openstack-neutron | 16:56 | |
| *** ataraday_ has joined #openstack-neutron | 16:56 | |
| *** sterdnotshaken has joined #openstack-neutron | 16:57 | |
| *** markstur has quit IRC | 16:58 | |
| *** ramishra has quit IRC | 16:58 | |
| *** rossella_s has quit IRC | 16:59 | |
| *** tonytan4ever has joined #openstack-neutron | 17:00 | |
| *** sterdnotshaken1 has quit IRC | 17:00 | |
| *** lucasxu has joined #openstack-neutron | 17:02 | |
| *** john-dav_ is now known as john-davidge | 17:02 | |
| *** fragatina has joined #openstack-neutron | 17:02 | |
| *** Jack_Iv has quit IRC | 17:08 | |
| *** vks1 has quit IRC | 17:08 | |
| *** suresh12 has joined #openstack-neutron | 17:08 | |
| *** amotoki has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack/neutron master: Move conntrack zones to IPTablesFirewall https://review.openstack.org/441353 | 17:09 |
| *** debedb has joined #openstack-neutron | 17:10 | |
| debedb | question about plugin driver - anyone? | 17:11 |
| debedb | ipam plugin that is | 17:11 |
| *** suresh12 has quit IRC | 17:12 | |
| *** sterdnotshaken has quit IRC | 17:14 | |
| *** sterdnotshaken has joined #openstack-neutron | 17:15 | |
| *** debedb has left #openstack-neutron | 17:15 | |
| *** thorst_afk is now known as thorst | 17:16 | |
| *** debedb37 has joined #openstack-neutron | 17:16 | |
| *** debedb has joined #openstack-neutron | 17:16 | |
| *** debedb37 has quit IRC | 17:16 | |
| *** bfernando has quit IRC | 17:19 | |
| *** john-davidge has quit IRC | 17:20 | |
| *** erolg has quit IRC | 17:20 | |
| reedip_ | kevinbenton : ping | 17:21 |
| reedip_ | ihrachys: ping | 17:21 |
| kevinbenton | reedip_: pong | 17:24 |
| reedip_ | kevinbenton : sorry, but did u get our earlier pings? about neutron-lib | 17:25 |
| *** TMM has joined #openstack-neutron | 17:25 | |
| *** TMM has joined #openstack-neutron | 17:25 | |
| ihrachys | kevinbenton, https://review.openstack.org/#/q/topic:bug/1672701 the reverts, let's make them in quick, we btw also just hit it in RDO | 17:25 |
| *** TMM has quit IRC | 17:26 | |
| *** efoley_ has quit IRC | 17:27 | |
| kevinbenton | ihrachys: done | 17:30 |
| kevinbenton | reedip_: are you saying no jobs at all are running when you propose a patch to neutron-lib? | 17:30 |
| reedip_ | kevinbenton : yeah, I couldnt see any job running for 2 days as did davidsha | 17:31 |
| kevinbenton | reedip_: did you propose a patch to neutron-lib? | 17:31 |
| *** ralonsoh has quit IRC | 17:31 | |
| *** Jack_Iv has joined #openstack-neutron | 17:32 | |
| reedip_ | kevinbenton: https://review.openstack.org/#/c/453110/ | 17:32 |
| kevinbenton | reedip_: ack | 17:33 |
| kevinbenton | reedip_: looking | 17:33 |
| reedip_ | kevinbenton: however, https://review.openstack.org/#/c/432521/ was proposed 2 hours later, and now I see that it had some tests executed. so maybe there is some other issue | 17:33 |
| reedip_ | kevinbenton : thanks for your help | 17:33 |
| debedb | oops something with my client.. I’ll try again | 17:34 |
| debedb | have a q about pluggable ipam in newton | 17:34 |
| debedb | seems the information about instance (Device_id) and host that the port belongs to is no longer available at that stage somehow - but it was there in liberty… | 17:35 |
| *** TMM has joined #openstack-neutron | 17:36 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron-lib master: Add new protocols in Firewalls https://review.openstack.org/453110 | 17:36 |
| *** ociuhandu has quit IRC | 17:39 | |
| *** fragatina has quit IRC | 17:42 | |
| *** KurtB has joined #openstack-neutron | 17:44 | |
| *** gonzalo2kx has joined #openstack-neutron | 17:45 | |
| *** gonzalo2kx has quit IRC | 17:46 | |
| *** mriedem1 has joined #openstack-neutron | 17:50 | |
| *** mriedem has quit IRC | 17:51 | |
| haleyb | reedip: ping regarding https://review.openstack.org/#/c/427670/ (ipv6 icmp change) and tempest code | 17:51 |
| *** s3wong has joined #openstack-neutron | 17:52 | |
| haleyb | reedip_: ^^ if you didn't see the ping to your other nick | 17:52 |
| reedip_ | hi haleyb | 17:52 |
| haleyb | hi reedip_ - so the update I made to the neutron code causes (at least) one failure in tempest with a duplicate SG error | 17:53 |
| reedip_ | ok haleyb | 17:54 |
| reedip_ | haleyb should we push the tempest patch, in such a case? | 17:56 |
| haleyb | so i'm thinking that test needs to change to not try all three. i didn't know if we could make a separate test that would do the duplicates, since i'm remembering tempest runs against older releases too right? | 17:56 |
| reedip_ | yes, tempest runs against previous versions, AFAIK | 17:56 |
| haleyb | so if we merge the tempest patch we won't be able to merge the neutron one, since tempest is gating | 17:57 |
| *** radhikam has joined #openstack-neutron | 17:59 | |
| *** baoli has quit IRC | 17:59 | |
| haleyb | reedip_: so we need to make the tempest test smarter - i.e. it should handle either 'icmp' or 'ipv6-icmp' coming back from the neutron API | 18:00 |
| reedip_ | haleyb : the current tempest patch takes all 3 values, but considers them to be ipv6icmp | 18:00 |
| *** Jack_Iv has quit IRC | 18:00 | |
| *** sambetts is now known as sambetts|afk | 18:00 | |
| *** mriedem1 is now known as mriedem | 18:00 | |
| *** lyan has quit IRC | 18:01 | |
| reedip_ | ok, got it, as your current patch can return either icmp or ipv6-icmp, so change is required in line#178 of tempest/api/network/test_security_groups.py | 18:01 |
| reedip_ | haleyb: in https://review.openstack.org/#/c/431276/16/tempest/api/network/test_security_groups.py | 18:01 |
| *** obondarev has joined #openstack-neutron | 18:02 | |
| haleyb | reedip_: i think my current patch will always return ipv6-icmp for IPv6, which is a change from what i had proposed - it seemed a better value after i looked into it friday | 18:02 |
| *** baoli has joined #openstack-neutron | 18:02 | |
| reedip_ | haleyb: and thats what the tempest patch is doing. For IPv6, it always expects ipv6-icmp | 18:03 |
| reedip_ | haleyb: https://review.openstack.org/#/c/431276 | 18:03 |
| haleyb | reedip_: yes, but when all those protocols are listed, the second and third icmp generate a duplicate error | 18:03 |
| haleyb | before the neutron patch they would all succeed | 18:04 |
| *** crushil has quit IRC | 18:04 | |
| reedip_ | haleyb: right, and thats what tempest reports as a problem. We should not report an error for duplicate rules, but we can log a message and proceed forward | 18:04 |
| *** ociuhandu has joined #openstack-neutron | 18:04 | |
| reedip_ | haleyb: I mean if we do not want to break API between 2 releases, and if duplicate rules are not something which we need to report an Exception for, then just having a Warning Log is enough | 18:05 |
| reedip_ | haleyb : this would ensure we do not have any corner case in Rolling Upgrades as well | 18:06 |
| haleyb | reedip_: right, i haven't even gotten to de-duping exising rules yet though, that could confuse users who see rules go away, even if they are duplicates | 18:07 |
| reedip_ | haleyb: this was something I discussed with ihrachys some time earlier, and he had an opinion that the existing rules can exist, because they are not causing any major hurdles ( ideally the first rule would match any requirement and similar rules placed below wont be in action ever). | 18:09 |
| *** dsneddon has quit IRC | 18:09 | |
| reedip_ | however, future rules can be curtailed, to avoid further duplication | 18:09 |
| haleyb | reedip_: regarding breaking the API, we could probably create a tempest test that creates duplicates, since it can detect when neutron is normalizing the icmp protocol values. i know it's a hack | 18:09 |
| haleyb | s/creates/tries to create | 18:09 |
| reedip_ | haleyb : tempest might not accept it :) | 18:10 |
| reedip_ | they are pretty strict with API right now :) | 18:10 |
| haleyb | reedip_: yeah, because we'd have to create one with icmp and see the returned value was ipv6-icmp | 18:10 |
| reedip_ | haleyb : yeah... but I think not reporting an error and just logging a message wont be an issue, because | 18:11 |
| reedip_ | haleyb : (a) User still has the rule | 18:11 |
| reedip_ | even though its an existing rule | 18:11 |
| reedip_ | haleyb (b) : user doesnt get an error so the API/tempest doesnt break | 18:11 |
| reedip_ | so user doesnt need to worry either | 18:12 |
| reedip_ | haleyb : and Reno should be enough for the user after this dedup | 18:12 |
| haleyb | reedip_: so you mean neutron wouldn't complain? i.e. it would return the previous icmp rule? | 18:13 |
| reedip_ | haleyb: I mean, if user enter a duplicate rule, then the error exception and 400 error code need not be returned to the user | 18:14 |
| reedip_ | haleyb : because it is not harming the system , as a similar rule already exists | 18:14 |
| haleyb | reedip_: i think we're in-sync. the neutron patch would catch the duplicate, but have to check this case, and just return the existing | 18:15 |
| reedip_ | haleyb : yeah, that works as well | 18:15 |
| * haleyb wonders what reedip_ was thinking if that wasn't it :) | 18:15 | |
| reedip_ | hehe | 18:16 |
| *** aojea has quit IRC | 18:16 | |
| reedip_ | haleyb : yeah, we are in sync... : | 18:17 |
| reedip_ | :) | 18:17 |
| haleyb | reedip_: ok, let me try and change the neutron code to do that, without breaking anything. if done right then the tempest code should be happy | 18:18 |
| reedip_ | haleyb : ok :) | 18:18 |
| *** debedb has quit IRC | 18:21 | |
| haleyb | reedip_: hmm, but then we won't detect duplicate icmpv6 rules | 18:21 |
| reedip_ | haleyb : we wont explicitly shout it out, but internally we would be preventing duplicate rules. | 18:21 |
| kevinbenton | reedip_: i don't see any relevant changes in project-config that would cause it to skip all jobs | 18:22 |
| reedip_ | the catch is with third party driver though, like ODL | 18:22 |
| *** Jack_Iv has joined #openstack-neutron | 18:23 | |
| *** crushil has joined #openstack-neutron | 18:23 | |
| reedip_ | kevinbenton : yeah I know, I was just wondering why suddenly the patch doesnt execute anything even though its not dependent on any commit | 18:23 |
| reedip_ | kevinbenton : but as I said earlier, I pushed another patch and its job executed, so this patch may have some other issues. I will check it once again. thanks | 18:24 |
| kevinbenton | reedip_: well the difference is that this patch wasn't a 'doc only' change | 18:25 |
| kevinbenton | reedip_: https://review.openstack.org/#/c/432521/ | 18:25 |
| reedip_ | haleyb : as I was saying earlier, if third parties convert ipv6-icmp to something they undertand ( ODL is changing to icmpv6 ) , then everything should be ok. | 18:25 |
| kevinbenton | reedip_: because it modified the tests directory | 18:25 |
| haleyb | reedip_: perhaps ipv6-icmp/ipv6-icmp raises, but ipv6-icmp/icmp and ipv6-icmp/icmpv6 doesn't (if two calls happen) | 18:25 |
| *** dsneddon has joined #openstack-neutron | 18:25 | |
| reedip_ | kevinbenton ; ok, so you mean that as the tests were changed, so tests executed ... and in the firewall patch, there were no tests, so no tests executed (???) | 18:26 |
| openstackgerrit | Terry Wilson proposed openstack/neutron master: Use the ovsdbapp library https://review.openstack.org/453014 | 18:26 |
| *** baoli has quit IRC | 18:26 | |
| kevinbenton | reedip_: right, see we have this logic | 18:26 |
| kevinbenton | reedip_: https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1549-L1557 | 18:26 |
| kevinbenton | reedip_: but that regex is only supposed to skip some jobs, not all of them | 18:27 |
| reedip_ | kevinbenton : intresting :) | 18:27 |
| kevinbenton | ihrachys: do you know what could cause a complete job skip for jenkins in neutron-lib? | 18:27 |
| kevinbenton | ihrachys: that regex seems sane and hasn't had changes in a while | 18:28 |
| *** sterdnotshaken has quit IRC | 18:29 | |
| kevinbenton | reedip_: oh, i wonder if it's because of the change-id being the same in fwaas | 18:29 |
| kevinbenton | reedip_: with a depends-on to the same change-id | 18:29 |
| kevinbenton | reedip_: zuul might not like change-id duplication when using depends-on | 18:30 |
| reedip_ | kevinbenton : ohh.. damn it | 18:30 |
| ihrachys | kevinbenton, which patch\ | 18:30 |
| reedip_ | yeah, got it kevinbenton : I hacked zuul to skip tests | 18:30 |
| ihrachys | ok seems like you solved the issue, so I get back to my lair | 18:31 |
| kevinbenton | ihrachys: yes, please return :) | 18:31 |
| reedip_ | sorry guys :) | 18:31 |
| kevinbenton | reedip_: no prob | 18:31 |
| reedip_ | haleyb : interesting scenario , so ipv6-icmp /ipv6-icmp raises duplicate error | 18:32 |
| reedip_ | haleyb :thats because the same was implemented earlier | 18:32 |
| *** baoli has joined #openstack-neutron | 18:32 | |
| kevinbenton | ihrachys: is there a bug tracking that linux bridge job instability? | 18:32 |
| reedip_ | haleyb : but now, if we do ipv6-icmp/icmpv6 then no error is generated | 18:32 |
| *** vijaykc4 has quit IRC | 18:32 | |
| reedip_ | haleyb : but then we also have this scenario , icmpv6/icmpv6. this will currently generate error | 18:33 |
| haleyb | reedip_: right, that second wouldn't generate an error, but would return the exisitng rule | 18:33 |
| reedip_ | haleyb: but now if we convert icmpv6 to ipv6-icmp . then the above sencario would be ipv6-icmp/icmpv6 and it doesnt return an error | 18:34 |
| haleyb | icmpv6/icmpv6 wouldn't in this case since both the first and second didn't match | 18:34 |
| reedip_ | haleyb :yeah, so... this is getting a bit crazy | 18:34 |
| haleyb | yup | 18:34 |
| reedip_ | not raising the error might break some tempest tests.... raising the error breaks tempest tests :D | 18:35 |
| *** seanatcisco has quit IRC | 18:35 | |
| ihrachys | kevinbenton, which one | 18:36 |
| kevinbenton | ihrachys: the higher failure rate for the linux bridge job | 18:37 |
| *** ataraday_ has quit IRC | 18:37 | |
| ihrachys | grenade? | 18:38 |
| haleyb | reedip_: yes, and it could be confusing for API users | 18:39 |
| ihrachys | if that's the one, there was https://bugs.launchpad.net/neutron/+bug/1674443 but it's now incomplete because it lacks details | 18:39 |
| openstack | Launchpad bug 1674443 in neutron "gate-grenade-dsvm-neutron-linuxbridge-multinode fails on smoke test sometimes " [Undecided,Incomplete] | 18:39 |
| ihrachys | so you could as well create a new one for specific issue if you see one | 18:39 |
| reedip_ | haleyb : so I think raising the error for each duplicate entry seems more appropriate | 18:41 |
| openstackgerrit | Reedip proposed openstack/neutron-lib master: Add new protocols in Firewalls https://review.openstack.org/455422 | 18:41 |
| haleyb | reedip_: so we'd have to change the tempest test, and possibly add one to cover this case that only runs Pike+ | 18:42 |
| reedip_ | haleyb : yeah, but I dont know how to run a version dependent tempest test | 18:43 |
| *** sterdnotshaken has joined #openstack-neutron | 18:43 | |
| haleyb | welcome to the club :) | 18:43 |
| openstackgerrit | Reedip proposed openstack/neutron-fwaas master: Add new protocols in Firewalls https://review.openstack.org/440331 | 18:44 |
| reedip_ | hehehe :) | 18:44 |
| reedip_ | kevinbenton; do you have any idea of running a version dependent test for tempest ? | 18:45 |
| kevinbenton | reedip_: no, it doesn't support that intentionally | 18:46 |
| kevinbenton | reedip_: a new behavior should be exposed with a new extension | 18:46 |
| kevinbenton | reedip_: and then you use required-ext decorator | 18:46 |
| haleyb | kevinbenton: in this case we're not adding functionality, we're better detecting SG rules are duplicates | 18:47 |
| reedip_ | kevinbenton : but what haleyb and I am proposing is not related to an extension, it basically catches duplicate SG rules for icmp, IP version 6 | 18:47 |
| kevinbenton | reedip_: does it start throwing errors or something? | 18:47 |
| reedip_ | kevinbenton : earlier, ipv6-icmp, icmp and icmpv6 all were treated different | 18:48 |
| haleyb | and they shouldn't have been | 18:48 |
| reedip_ | kevinbenton : but they internally are creating the same set of rules for IPV6 , ICMP protocol | 18:48 |
| openstackgerrit | Felipe Monteiro proposed openstack/neutron master: Adding missing neutron policies to policy.json https://review.openstack.org/455423 | 18:49 |
| reedip_ | kevinbenton : what haleyb and I am working on, is to generate an exception in case of such dulicacy | 18:49 |
| reedip_ | s/dulicacy/duplicacy | 18:49 |
| *** jprovazn has quit IRC | 18:50 | |
| *** ralonsoh has joined #openstack-neutron | 18:50 | |
| haleyb | kevinbenton: basically, standardize on one value, 'ipv6-icmp' for IPv6 ICMP security group rules, instead of allowing three | 18:50 |
| reedip_ | kevinbenton :but tempest right now accepts these 3 protocols separately and returns 204 ( IIRC ) | 18:50 |
| haleyb | i can find the ML pointer | 18:51 |
| *** debedb_ has joined #openstack-neutron | 18:51 | |
| haleyb | http://lists.openstack.org/pipermail/openstack-dev/2017-March/114570.html | 18:52 |
| kevinbenton | reedip_, haleyb: sounds like a backwards incompatible change then | 18:52 |
| kevinbenton | can we silently de-dup instead? | 18:52 |
| reedip_ | kevinbenton : it kind of is | 18:52 |
| reedip_ | kevinbenton : silent dedup creates more issues | 18:52 |
| reedip_ | kevinbenton : if we internally change , for ex. icmpv6 to ipv6-icmp | 18:53 |
| reedip_ | kevinbenton : then previously, icmpv6/icmpv6 would have been generating an error, now it wont :) | 18:53 |
| reedip_ | kevinbenton : because of the silent dedup | 18:53 |
| *** debedb_ is now known as debedb | 18:53 | |
| *** lyan has joined #openstack-neutron | 18:54 | |
| *** seanatcisco has joined #openstack-neutron | 18:54 | |
| kevinbenton | reedip_: i mean just that particular case | 18:55 |
| *** sdague_ has quit IRC | 18:55 | |
| *** iranzo has quit IRC | 18:55 | |
| kevinbenton | hmm | 18:56 |
| kevinbenton | let me think about it :) | 18:56 |
| *** suresh12 has joined #openstack-neutron | 18:56 | |
| *** crose has joined #openstack-neutron | 18:56 | |
| haleyb | kevinbenton: you are now where i was 30 minutes ago, seemed simple at first but what will the API user notice and care about | 18:57 |
| *** suresh12 has quit IRC | 18:57 | |
| *** suresh12 has joined #openstack-neutron | 18:57 | |
| *** sterdnotshaken1 has joined #openstack-neutron | 19:06 | |
| *** sterdnotshaken has quit IRC | 19:06 | |
| reedip_ | sorry, was on a call ... | 19:08 |
| reedip_ | kevinbenton : yeah , take ur time. patches are https://review.openstack.org/427670 , and https://review.openstack.org/431276 | 19:09 |
| reedip_ | haleyb, kevinbenton : I would be taking a leave now, good night.. let me know your thoughts though, my primary acccount will be logged in ( reedip) :) | 19:10 |
| *** reedip_ is now known as outofmemory | 19:10 | |
| *** fragatina has joined #openstack-neutron | 19:10 | |
| *** sterdnotshaken has joined #openstack-neutron | 19:13 | |
| ihrachys | kevinbenton, https://review.openstack.org/#/c/454870/ probably a gate fix for another functional failure | 19:14 |
| *** edmondsw_ has joined #openstack-neutron | 19:15 | |
| *** edmondsw_ has quit IRC | 19:15 | |
| ihrachys | I am not sure if that's the one, since in newton where I saw that I don't have full names, just lengths (that is solved by backports https://review.openstack.org/#/q/Ic5a3b347bea7e5aa8a5caee5035568e5954f58dc,n,z) | 19:16 |
| *** sterdnotshaken1 has quit IRC | 19:16 | |
| openstackgerrit | Reedip proposed openstack/neutron-lib master: Add new protocols in Firewalls https://review.openstack.org/455422 | 19:18 |
| *** obondarev has quit IRC | 19:18 | |
| *** john-davidge has joined #openstack-neutron | 19:20 | |
| openstackgerrit | Reedip proposed openstack/neutron master: Add check for Bandwidth Limit Rules https://review.openstack.org/442375 | 19:22 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron master: ip_lib: ignore gre and lo devices in get_devices by default https://review.openstack.org/455431 | 19:24 |
| *** john-davidge has quit IRC | 19:25 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron master: ip_lib: ignore gre and lo devices in get_devices by default https://review.openstack.org/455431 | 19:26 |
| *** marst_ has joined #openstack-neutron | 19:27 | |
| *** manheim has joined #openstack-neutron | 19:28 | |
| *** marst has quit IRC | 19:30 | |
| *** manheim has quit IRC | 19:33 | |
| *** iranzo has joined #openstack-neutron | 19:34 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: DNM: test for infra DVR+HA multinode change https://review.openstack.org/455433 | 19:34 |
| openstackgerrit | Paul Carver proposed openstack/networking-bgpvpn master: Move API definition out of n8g-bgpvpn into neutron-lib https://review.openstack.org/395585 | 19:36 |
| *** tonytan4ever has quit IRC | 19:36 | |
| *** tonytan4ever has joined #openstack-neutron | 19:40 | |
| *** iranzo has quit IRC | 19:44 | |
| *** suresh12 has quit IRC | 19:44 | |
| *** sterdnotshaken1 has joined #openstack-neutron | 19:50 | |
| *** sterdnotshaken has quit IRC | 19:53 | |
| *** gouthamr has quit IRC | 19:54 | |
| *** achadha has joined #openstack-neutron | 19:58 | |
| *** namnh has joined #openstack-neutron | 19:59 | |
| *** gcheresh has joined #openstack-neutron | 20:00 | |
| *** n0cat has joined #openstack-neutron | 20:02 | |
| *** yamahata has quit IRC | 20:03 | |
| n0cat | hello | 20:04 |
| *** grantma__ has joined #openstack-neutron | 20:09 | |
| *** Jack_Iv has quit IRC | 20:15 | |
| *** Jack_Iv has joined #openstack-neutron | 20:15 | |
| *** n0cat has quit IRC | 20:15 | |
| *** g3ek has quit IRC | 20:16 | |
| openstackgerrit | Merged openstack/networking-ovn master: Enhance local.conf.sample documentation https://review.openstack.org/455311 | 20:16 |
| openstackgerrit | Merged openstack/networking-ovn master: Docs: Do not turn warnings into errors https://review.openstack.org/455182 | 20:16 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Make QoS policy object compatible with versions 1.2 and higher https://review.openstack.org/455338 | 20:17 |
| *** Jack_Iv has quit IRC | 20:20 | |
| *** john-davidge has joined #openstack-neutron | 20:21 | |
| *** arbaindu has joined #openstack-neutron | 20:23 | |
| arbaindu | Anyone know how exactly what triggers the l2pop driver / OVS agent to re-add fdb flows for tunnels? | 20:24 |
| arbaindu | All we did was restart the ovs agent on hypervisors, noticed that the l2pop flows on br-tun are lost | 20:24 |
| arbaindu | this kills traffic, specifically because the multidest flows are lost. VMs can no longer ARP | 20:25 |
| arbaindu | unicast entries for remote MACs are lost as well | 20:25 |
| *** g3ek has joined #openstack-neutron | 20:26 | |
| *** rkukura has quit IRC | 20:26 | |
| *** john-davidge has quit IRC | 20:26 | |
| *** rkukura has joined #openstack-neutron | 20:26 | |
| openstackgerrit | Merged openstack/networking-odl master: Fixing a typo in function and variable name https://review.openstack.org/454868 | 20:27 |
| *** john-davidge has joined #openstack-neutron | 20:29 | |
| *** markstur has joined #openstack-neutron | 20:29 | |
| *** fragatin_ has joined #openstack-neutron | 20:31 | |
| *** ralonsoh has quit IRC | 20:31 | |
| *** fragatina has quit IRC | 20:34 | |
| *** markstur has quit IRC | 20:34 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: DNM: IPv6 ICMP API hack https://review.openstack.org/455444 | 20:34 |
| *** armax has quit IRC | 20:34 | |
| *** irenab has quit IRC | 20:36 | |
| *** irenab has joined #openstack-neutron | 20:36 | |
| *** john-davidge has quit IRC | 20:37 | |
| openstackgerrit | Felipe Monteiro proposed openstack/neutron master: Adding missing neutron policies to policy.json https://review.openstack.org/455423 | 20:38 |
| *** grantma__ has quit IRC | 20:40 | |
| openstackgerrit | Trevor McCasland proposed openstack/neutron master: WIP: Manage BUM traffic with secgroups https://review.openstack.org/455445 | 20:40 |
| *** tonytan4ever has quit IRC | 20:41 | |
| *** cleong has quit IRC | 20:41 | |
| *** tonytan4ever has joined #openstack-neutron | 20:41 | |
| *** tonytan4ever has quit IRC | 20:46 | |
| *** lihi has quit IRC | 20:47 | |
| *** lihi has joined #openstack-neutron | 20:48 | |
| *** aojea has joined #openstack-neutron | 20:51 | |
| *** namnh has quit IRC | 20:52 | |
| kevinbenton | ihrachys: done | 20:52 |
| kevinbenton | haleyb: ping | 20:52 |
| kevinbenton | haleyb: confused about .exists() | 20:53 |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Add sanity check for conntrack https://review.openstack.org/455450 | 20:53 |
| *** karthikpr has joined #openstack-neutron | 20:53 | |
| kevinbenton | haleyb: https://review.openstack.org/#/c/283757/50/neutron/agent/l3/dvr_fip_ns.py@403 | 20:53 |
| *** karthikpr has quit IRC | 20:53 | |
| *** karthikpr has joined #openstack-neutron | 20:54 | |
| haleyb | kevinbenton: looking | 20:54 |
| *** gcheresh has quit IRC | 20:55 | |
| *** crose has quit IRC | 20:58 | |
| *** suresh12 has joined #openstack-neutron | 20:59 | |
| kevinbenton | ihrachys: you still around for a few mins to talk about sanity checks? | 20:59 |
| *** karthikpr has quit IRC | 20:59 | |
| ihrachys | kevinbenton, yea | 20:59 |
| kevinbenton | ihrachys: we seem to have a large chunk of people that don't know sanity checks exist, let alone run them | 21:00 |
| kevinbenton | ihrachys: what do you think about running the easy ones at agent startup? | 21:00 |
| kevinbenton | ihrachys: (e.g. the one brian proposed right above) | 21:00 |
| kevinbenton | ihrachys: then we bail with systemexit on failure and have a sane failure mode | 21:00 |
| kevinbenton | ihrachys: rather than something subtle later | 21:00 |
| ihrachys | kevinbenton, will the agent crash without it? | 21:01 |
| ihrachys | ok I see the answer | 21:01 |
| ihrachys | I think there is place for mixed usage of some checks | 21:01 |
| ihrachys | not all of them | 21:01 |
| kevinbenton | ihrachys: yeah, would definitely have to be evaluated one-by-one | 21:01 |
| ihrachys | would make sense to have some kind of a decorator that we could use to tag checks for runtime invokation | 21:01 |
| kevinbenton | ihrachys: and only invoke when certain options are enabled, etc | 21:01 |
| *** suresh12 has quit IRC | 21:01 | |
| haleyb | kevinbenton: fns_ip.device is of class IPDevice, which has an exists() | 21:02 |
| *** manheim has joined #openstack-neutron | 21:02 | |
| ihrachys | I am fine with it. I would ask otherwiseguy for his thoughts on the matter, he was opinionated before. | 21:02 |
| kevinbenton | haleyb: ack, not sure how i missed that | 21:03 |
| *** tonytan4ever has joined #openstack-neutron | 21:03 | |
| kevinbenton | otherwiseguy: why do you hate sanity checks at startups and thus hate our users? :) | 21:03 |
| *** thorst has quit IRC | 21:03 | |
| ihrachys | I think he hated version checks for the most part | 21:03 |
| kevinbenton | haleyb: i thought it did, but then was looking at the code to leave another comment | 21:03 |
| ihrachys | at least initially | 21:03 |
| haleyb | kevinbenton: after that i get a free pass on the change, right? :) | 21:04 |
| kevinbenton | ihrachys: version checks make sense, particularily when certain companies backport features to older versions :) | 21:04 |
| kevinbenton | haleyb: no, it just means my next comment i leave will be valid :P | 21:04 |
| arbaindu | kevinbenton: Hey, had question regarding the conntrack issue we were discussing last week | 21:04 |
| arbaindu | Ended up disabling security groups altogether and that fixed this up too right away | 21:05 |
| otherwiseguy | kevinbenton, I only hate our users who don't know how to properly set up their system. :p | 21:05 |
| arbaindu | But, i was wondering what the downside would be to not doing a conntrack delete altogether upon port delete? | 21:05 |
| ihrachys | kevinbenton, you mean to piss off those companies? | 21:05 |
| ihrachys | then yes, agreed | 21:05 |
| arbaindu | this would only cause problems in case of a fixed IP being moved to a new security group, right? (VM gets deleted, new VM comes up with recycled fixed IP, but on diff sec group) | 21:06 |
| otherwiseguy | kevinbenton, I just don't like doing things every startup that only need to be done once at setup. Kind of a separation of concerns. "That's the installer's job." :p | 21:06 |
| ihrachys | we could probably have it as an opt-in (or opt-out) to make both camps happy | 21:07 |
| *** seanatcisco has quit IRC | 21:07 | |
| *** grantma__ has joined #openstack-neutron | 21:08 | |
| kevinbenton | arbaindu: it doesn't even cause a bad connectivity problem | 21:08 |
| kevinbenton | arbaindu: it's just that existing established connections won't be cleared | 21:08 |
| kevinbenton | arbaindu: that's the only downside | 21:08 |
| kevinbenton | arbaindu: so if you have a rule that allows HTTP for example | 21:08 |
| kevinbenton | arbaindu: and a client starts a download from your instance | 21:09 |
| otherwiseguy | Also, if every bit of software at runtime decided to make system-level configuration changes, they'd step all over each other. | 21:09 |
| kevinbenton | arbaindu: then you delete the SG rule | 21:09 |
| arbaindu | right, the http connection would still be going through | 21:09 |
| kevinbenton | arbaindu: without the conntrack call, the connection will remain active | 21:09 |
| kevinbenton | arbaindu: yep | 21:09 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [WIP] Add QoS bandwidth limit for instance ingress traffic https://review.openstack.org/449831 | 21:09 |
| kevinbenton | arbaindu: that's the only downside, which is pretty minor | 21:09 |
| openstackgerrit | boden proposed openstack/neutron-lib master: Introduce service definitions https://review.openstack.org/455451 | 21:10 |
| haleyb | kevinbenton: i would call that a security issue if removing a SG rule doesn't stop traffic | 21:10 |
| arbaindu | What about in case where the VM/port is deleted, but then -re-assigned? | 21:10 |
| *** zkassab has quit IRC | 21:10 | |
| arbaindu | sorry i am not too familiar with conntrack - how long does it keep up established connections? | 21:10 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [WIP] Add QoS bandwidth limit for instance ingress traffic https://review.openstack.org/449831 | 21:10 |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [WIP] Add QoS bandwidth limit for instance ingress traffic https://review.openstack.org/449831 | 21:11 |
| arbaindu | say a VM gets deleted, and we removed the conntrack delete. Some time later VM comes up - possibly on a different security group. would http connection still be allowed? | 21:11 |
| kevinbenton | arbaindu: well if it's re-assigned you can have leftover connections that might result in open TCP connections flooding to your instance | 21:11 |
| kevinbenton | arbaindu: only an existing HTTP connection | 21:11 |
| kevinbenton | arbaindu: which that other VM would reject anyway since it's mid TCP stream | 21:11 |
| kevinbenton | ihrachys: no, i mean i understand not liking strict version checks | 21:12 |
| *** boden has quit IRC | 21:12 | |
| kevinbenton | otherwiseguy: a sanity check doesn't make runtime configuration changes | 21:12 |
| haleyb | arbaindu: there is a sysctl that controls it, but it's more than a few minutes without seeing a packet, if traffic is flowing it will stay open until removed | 21:12 |
| arbaindu | yea that's what I guessed. gotcha, didnt know there was a sysctl parameter for it. | 21:12 |
| arbaindu | so in that case, seems like conntrack is causing more problems than solving? | 21:13 |
| arbaindu | at least in slowness - it seems to take up the bulk of the rpc_loop duration | 21:13 |
| kevinbenton | haleyb: well in TCP case it will also close immediately on RST or FIN | 21:13 |
| haleyb | kevinbenton: yes, at end of connection. I just remember if you have a ping going, without running conntrack -D it will continue | 21:14 |
| kevinbenton | haleyb: yeah, ping and UDP are different | 21:14 |
| arbaindu | given that ovs agent only scans for ports once per loop,OVS gets blocked on this step quite a bit | 21:14 |
| kevinbenton | haleyb: they are just timing criteria | 21:14 |
| kevinbenton | arbaindu: it's becomes a big issue in pretty narrow scenarios | 21:15 |
| kevinbenton | arbaindu: lots of port deletes on a single agent for ports that all belong to the same security group | 21:15 |
| *** pcaruana has quit IRC | 21:15 | |
| kevinbenton | arbaindu: if they weren't in the same security group it wouldn't be an issue | 21:15 |
| kevinbenton | otherwiseguy: " I only hate our users who don't know how to properly set up their system. :p" is how you end up having software that nobody likes to use :/ | 21:18 |
| otherwiseguy | ok, so you are talking about running them and then just logging, not fixing? Then my main argument against it is just that it leads to ever increasing startup times. Also, sanity checks were initially designed to do relatively expensive checks instead of relying on simple things like version number checks. | 21:18 |
| kevinbenton | otherwiseguy: or making them fatal | 21:18 |
| otherwiseguy | kevinbenton, no, you just make the write the right tool to help them. :) Like an installer. | 21:18 |
| kevinbenton | otherwiseguy: yeah, no fixing | 21:19 |
| kevinbenton | otherwiseguy: i.e. the agent immediately dies and says conntrack must be installed if they are using the iptables driver | 21:19 |
| kevinbenton | otherwiseguy: rather than vomit exceptions on every port delete or security group update | 21:19 |
| *** crushil has quit IRC | 21:19 | |
| openstackgerrit | Brian Haley proposed openstack/neutron master: Add sanity check for conntrack https://review.openstack.org/455450 | 21:23 |
| otherwiseguy | kevinbenton, real users will be using an installer and config management stuff would by my argument. doing the checks at runtime seems like developing for the wrong audience. | 21:23 |
| *** catintheroof has quit IRC | 21:25 | |
| kevinbenton | otherwiseguy: so are you opposed to config validation then as well? | 21:26 |
| *** trevormc has quit IRC | 21:30 | |
| *** achadha has quit IRC | 21:30 | |
| *** karthikpr has joined #openstack-neutron | 21:31 | |
| *** karthikpr has quit IRC | 21:31 | |
| *** pramodrj07 has joined #openstack-neutron | 21:32 | |
| *** MasterOfBugs has joined #openstack-neutron | 21:32 | |
| otherwiseguy | kevinbenton, Depends. If it is "this config makes no sense, you can't have these mutually exclusive options" then it seems reasonable to check it. But when you start getting into things like, "you have this configured, but it requires a feature in this external software", meh. I hoped sanity check would be a tool to enable admins to verify their configs at config time. | 21:32 |
| otherwiseguy | And installer's could run it when making config changes if they really wanted to. | 21:33 |
| otherwiseguy | s/'// | 21:33 |
| kevinbenton | otherwiseguy: but i don't understand the distinction other than the runtime cost. putting an invalid IP in the 'tunnel_ip' seems just as invalid as not having dnsmasq installed | 21:33 |
| otherwiseguy | er, management software I suppose. | 21:34 |
| *** haleyb has quit IRC | 21:35 | |
| kevinbenton | otherwiseguy: what i want to avoid is this | 21:36 |
| kevinbenton | otherwiseguy: http://lists.openstack.org/pipermail/openstack/2017-April/019125.html | 21:36 |
| kevinbenton | first sentence | 21:36 |
| kevinbenton | whether or not installers exist that handle these things | 21:36 |
| kevinbenton | we have an official deployment guide | 21:36 |
| kevinbenton | that results in users ending up with errors | 21:37 |
| otherwiseguy | Does the official deployment guide mention running sanity check? | 21:37 |
| otherwiseguy | If not, it should. And then problem solved. :D | 21:37 |
| kevinbenton | otherwiseguy: i don't think it does | 21:37 |
| kevinbenton | otherwiseguy: right, either we continually keep fixing all of the dozens of ways to deploy openstack | 21:38 |
| kevinbenton | otherwiseguy: the "you're holding it wrong" approach | 21:38 |
| kevinbenton | otherwiseguy: or just run the things at startup to save us all mailing list messages | 21:38 |
| *** tidwellr has quit IRC | 21:39 | |
| *** lucasxu has quit IRC | 21:40 | |
| *** karthikpr has joined #openstack-neutron | 21:40 | |
| *** tidwellr has joined #openstack-neutron | 21:40 | |
| *** armax has joined #openstack-neutron | 21:40 | |
| *** janzian has quit IRC | 21:41 | |
| *** Sukhdev has joined #openstack-neutron | 21:41 | |
| *** baoli has quit IRC | 21:43 | |
| otherwiseguy | or make sure the documentation is right so people don't have search through log files to notice something is wrong. That's the other nice thing about sanity-check. You run it, it tells you what's wrong. | 21:44 |
| *** baoli has joined #openstack-neutron | 21:44 | |
| *** tidwellr has quit IRC | 21:44 | |
| otherwiseguy | I'm not going to go mental or anything if we have some runtime checks. I just think there are better ways most of the time to let users know about config problems. | 21:44 |
| otherwiseguy | like that one "well, it runs fine. i just happened to look in the log and saw this" | 21:45 |
| otherwiseguy | if they are fatal errors, I'm less irritated by them. | 21:46 |
| *** baoli has quit IRC | 21:48 | |
| *** fragatin_ has quit IRC | 21:54 | |
| *** fragatina has joined #openstack-neutron | 21:54 | |
| *** karthikpr has quit IRC | 21:55 | |
| *** KurtB has quit IRC | 21:56 | |
| otherwiseguy | kevinbenton, as in, if we are doing (cheap) checks at runtime (please no version number checks), and failing to load...it isn't the end of the world, anyway. :p | 21:57 |
| otherwiseguy | though i'd still prefer updated documentation that mentions using sanity-check. | 21:58 |
| *** grantma__ has quit IRC | 22:01 | |
| *** fragatin_ has joined #openstack-neutron | 22:01 | |
| *** fragatina has quit IRC | 22:03 | |
| *** mickeys has quit IRC | 22:04 | |
| *** liangy has joined #openstack-neutron | 22:04 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Add QoS bandwidth limit for instance ingress traffic https://review.openstack.org/449831 | 22:05 |
| *** lyan has quit IRC | 22:07 | |
| *** manheim has quit IRC | 22:08 | |
| *** mlavalle has quit IRC | 22:15 | |
| *** m1dev has quit IRC | 22:15 | |
| *** hoonetorg has quit IRC | 22:16 | |
| *** manheim has joined #openstack-neutron | 22:19 | |
| *** ihrachys has quit IRC | 22:25 | |
| *** nplanel has joined #openstack-neutron | 22:26 | |
| *** armax has quit IRC | 22:26 | |
| *** catintheroof has joined #openstack-neutron | 22:27 | |
| *** kbringard has quit IRC | 22:29 | |
| *** gouthamr has joined #openstack-neutron | 22:32 | |
| *** seanatcisco has joined #openstack-neutron | 22:33 | |
| *** coreycb is now known as coreycb_vaca | 22:35 | |
| *** hoonetorg has joined #openstack-neutron | 22:39 | |
| arbaindu | What causes l2pop mech driver to send add_fdb_entries? Seems like just update port status changes to ACTIVE? | 22:47 |
| arbaindu | we restart ovs agent, and br-tun does not add any flooding flow rules | 22:47 |
| arbaindu | but we never receive update from l2pop | 22:47 |
| arbaindu | is there known issues with this? | 22:47 |
| *** aojea has quit IRC | 22:48 | |
| arbaindu | we see the setup_tunnel_port skips adding the install_flood_to_tun if l2pop is enabled... won't this break connectivity? | 22:49 |
| *** john-davidge has joined #openstack-neutron | 22:49 | |
| *** fnaval has quit IRC | 22:49 | |
| *** john-davidge has quit IRC | 22:53 | |
| *** kbaegis has joined #openstack-neutron | 22:54 | |
| *** manheim has quit IRC | 23:00 | |
| *** manheim has joined #openstack-neutron | 23:01 | |
| *** thorst has joined #openstack-neutron | 23:02 | |
| *** adriant has joined #openstack-neutron | 23:03 | |
| *** mickeys has joined #openstack-neutron | 23:05 | |
| *** mickeys has quit IRC | 23:09 | |
| *** sdague has quit IRC | 23:12 | |
| *** mickeys has joined #openstack-neutron | 23:12 | |
| *** seanatcisco has quit IRC | 23:13 | |
| *** scottda has quit IRC | 23:15 | |
| *** kbaegis has left #openstack-neutron | 23:17 | |
| *** tonytan4ever has quit IRC | 23:20 | |
| *** liangy has quit IRC | 23:21 | |
| *** chyka has quit IRC | 23:25 | |
| *** chyka has joined #openstack-neutron | 23:25 | |
| *** mordred has joined #openstack-neutron | 23:25 | |
| *** marst_ has quit IRC | 23:29 | |
| *** crushil has joined #openstack-neutron | 23:32 | |
| *** karthikpr has joined #openstack-neutron | 23:34 | |
| *** karthikpr has quit IRC | 23:38 | |
| openstackgerrit | Isaku Yamahata proposed openstack/networking-odl master: devstack: add boron SR3 release definition https://review.openstack.org/452839 | 23:41 |
| *** g3ek has quit IRC | 23:51 | |
| *** lujinluo has joined #openstack-neutron | 23:53 | |
| *** karthikpr has joined #openstack-neutron | 23:56 | |
| *** manheim has quit IRC | 23:58 | |
| *** marst has joined #openstack-neutron | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!