| *** vishy has quit IRC | 00:02 | |
| *** mlavalle has joined #openstack-neutron | 00:03 | |
| *** annegent_ has joined #openstack-neutron | 00:03 | |
| *** r-daneel has quit IRC | 00:03 | |
| *** jdag__ has quit IRC | 00:03 | |
| *** r-daneel has joined #openstack-neutron | 00:04 | |
| *** nimrodsun_ has quit IRC | 00:04 | |
| *** xuhanp has joined #openstack-neutron | 00:04 | |
| *** devvesa has joined #openstack-neutron | 00:05 | |
| *** vishy has joined #openstack-neutron | 00:05 | |
| *** ZZelle_ has quit IRC | 00:06 | |
| *** jdag___ has joined #openstack-neutron | 00:06 | |
| *** nimrodsun_ has joined #openstack-neutron | 00:06 | |
| *** emagana has joined #openstack-neutron | 00:07 | |
| *** yamamoto_ has joined #openstack-neutron | 00:07 | |
| *** a_le_ has quit IRC | 00:07 | |
| *** a_le has joined #openstack-neutron | 00:07 | |
| *** rook_ has joined #openstack-neutron | 00:07 | |
| *** signed8b_ has joined #openstack-neutron | 00:08 | |
| *** annegent_ has quit IRC | 00:08 | |
| *** baoli_ has joined #openstack-neutron | 00:08 | |
| *** a_le has quit IRC | 00:09 | |
| *** feleouet has joined #openstack-neutron | 00:09 | |
| *** a_le has joined #openstack-neutron | 00:09 | |
| *** devvesa has quit IRC | 00:09 | |
| *** cshahani has quit IRC | 00:10 | |
| *** swat30_ has joined #openstack-neutron | 00:11 | |
| *** ekarlso- has joined #openstack-neutron | 00:11 | |
| *** stenchlarge has quit IRC | 00:12 | |
| *** baoli has quit IRC | 00:12 | |
| *** emagana_ has quit IRC | 00:12 | |
| *** signed8bit has quit IRC | 00:12 | |
| *** yamamoto has quit IRC | 00:12 | |
| *** francois_eleouet has quit IRC | 00:12 | |
| *** swat30 has quit IRC | 00:12 | |
| *** ekarlso has quit IRC | 00:12 | |
| *** smcquaid has quit IRC | 00:12 | |
| *** asadoughi has quit IRC | 00:12 | |
| *** xmltok_ has quit IRC | 00:12 | |
| *** rook has quit IRC | 00:12 | |
| *** swat30_ is now known as swat30 | 00:12 | |
| *** annegent_ has joined #openstack-neutron | 00:12 | |
| *** annegent_ has quit IRC | 00:12 | |
| *** asadoughi has joined #openstack-neutron | 00:13 | |
| *** claudiub has quit IRC | 00:13 | |
| *** xmltok has joined #openstack-neutron | 00:13 | |
| *** xgerman_ has quit IRC | 00:14 | |
| *** emagana has quit IRC | 00:16 | |
| *** emagana has joined #openstack-neutron | 00:16 | |
| *** packet has quit IRC | 00:18 | |
| *** emagana has quit IRC | 00:21 | |
| *** SumitNaiksatam has joined #openstack-neutron | 00:21 | |
| *** otherwiseguy has quit IRC | 00:23 | |
| *** sballe has quit IRC | 00:23 | |
| *** karimb has quit IRC | 00:29 | |
| *** jckasper has joined #openstack-neutron | 00:29 | |
| *** yamahata has joined #openstack-neutron | 00:33 | |
| *** SumitNaiksatam has quit IRC | 00:33 | |
| *** signed8b_ has quit IRC | 00:37 | |
| *** a_le has quit IRC | 00:38 | |
| *** yamahata has quit IRC | 00:39 | |
| *** yamahata has joined #openstack-neutron | 00:39 | |
| *** AbsinthMind has quit IRC | 00:41 | |
| *** r-daneel has quit IRC | 00:43 | |
| *** popw has joined #openstack-neutron | 00:48 | |
| *** dvorkinista has quit IRC | 00:49 | |
| *** dvorkinista has joined #openstack-neutron | 00:50 | |
| *** gildub has joined #openstack-neutron | 00:50 | |
| *** peoplemerge has quit IRC | 00:51 | |
| *** SumitNaiksatam has joined #openstack-neutron | 00:51 | |
| *** harmw has quit IRC | 00:54 | |
| *** mlavalle has quit IRC | 00:55 | |
| *** xuhanp has quit IRC | 00:57 | |
| *** tgohad has joined #openstack-neutron | 01:00 | |
| *** AbsinthMind has joined #openstack-neutron | 01:01 | |
| *** tgohad__ has joined #openstack-neutron | 01:03 | |
| *** tgohad has quit IRC | 01:03 | |
| *** tsg has quit IRC | 01:03 | |
| *** dvorkinista has quit IRC | 01:05 | |
| *** drjones has quit IRC | 01:06 | |
| *** harmw has joined #openstack-neutron | 01:06 | |
| *** _cjones_ has joined #openstack-neutron | 01:06 | |
| *** _cjones_ has quit IRC | 01:11 | |
| *** tgohad__ has quit IRC | 01:13 | |
| *** stanzgy has joined #openstack-neutron | 01:17 | |
| *** liusheng has joined #openstack-neutron | 01:21 | |
| *** JayJ has joined #openstack-neutron | 01:24 | |
| *** nlahouti has quit IRC | 01:26 | |
| *** ramishra has joined #openstack-neutron | 01:30 | |
| *** tsg has joined #openstack-neutron | 01:32 | |
| *** diegows has joined #openstack-neutron | 01:32 | |
| *** mlavalle has joined #openstack-neutron | 01:33 | |
| *** signed8bit has joined #openstack-neutron | 01:36 | |
| *** popw has quit IRC | 01:40 | |
| *** tsg has quit IRC | 01:41 | |
| *** tsg has joined #openstack-neutron | 01:41 | |
| *** signed8bit has quit IRC | 01:41 | |
| *** signed8bit has joined #openstack-neutron | 01:41 | |
| *** nati_ueno has joined #openstack-neutron | 01:41 | |
| *** baohua has joined #openstack-neutron | 01:42 | |
| *** xuhanp has joined #openstack-neutron | 01:43 | |
| *** pitterfumped has joined #openstack-neutron | 01:44 | |
| *** pitterfumped has quit IRC | 01:45 | |
| *** JayJ has quit IRC | 01:49 | |
| *** mlavalle has quit IRC | 01:49 | |
| *** JayJ has joined #openstack-neutron | 01:50 | |
| *** yamamoto_ has quit IRC | 01:51 | |
| *** HenryG has quit IRC | 01:51 | |
| *** ramishra has quit IRC | 01:53 | |
| *** ramishra has joined #openstack-neutron | 01:53 | |
| *** devvesa has joined #openstack-neutron | 01:54 | |
| *** manishg has joined #openstack-neutron | 01:55 | |
| *** vivek-ebay has quit IRC | 01:57 | |
| *** manishg_ has joined #openstack-neutron | 01:58 | |
| *** devvesa has quit IRC | 01:59 | |
| *** manishg has quit IRC | 01:59 | |
| *** manishg has joined #openstack-neutron | 02:01 | |
| *** manishg_ has quit IRC | 02:02 | |
| *** manishg_ has joined #openstack-neutron | 02:04 | |
| *** manishg has quit IRC | 02:06 | |
| *** dkehnx1 has joined #openstack-neutron | 02:06 | |
| *** manishg has joined #openstack-neutron | 02:07 | |
| *** manishg_ has quit IRC | 02:08 | |
| *** dvorkinista has joined #openstack-neutron | 02:10 | |
| *** manishg_ has joined #openstack-neutron | 02:10 | |
| *** manishg has quit IRC | 02:11 | |
| *** popw has joined #openstack-neutron | 02:12 | |
| *** vivek-ebay has joined #openstack-neutron | 02:14 | |
| *** manishg has joined #openstack-neutron | 02:14 | |
| *** manishg_ has quit IRC | 02:14 | |
| *** manishg_ has joined #openstack-neutron | 02:17 | |
| *** manishg has quit IRC | 02:18 | |
| *** JayJ has quit IRC | 02:19 | |
| *** ramishra has quit IRC | 02:20 | |
| *** manishg has joined #openstack-neutron | 02:21 | |
| *** manishg_ has quit IRC | 02:21 | |
| *** cshahani has joined #openstack-neutron | 02:22 | |
| *** manishg_ has joined #openstack-neutron | 02:23 | |
| *** oda-g has quit IRC | 02:24 | |
| *** ramishra has joined #openstack-neutron | 02:25 | |
| *** manishg has quit IRC | 02:25 | |
| *** vivek-ebay has quit IRC | 02:25 | |
| *** prasoon has joined #openstack-neutron | 02:25 | |
| *** vivek-ebay has joined #openstack-neutron | 02:26 | |
| *** manishg has joined #openstack-neutron | 02:27 | |
| *** oda-g has joined #openstack-neutron | 02:27 | |
| *** manishg_ has quit IRC | 02:27 | |
| *** diegows has quit IRC | 02:28 | |
| *** prasoon has quit IRC | 02:30 | |
| *** manishg has quit IRC | 02:31 | |
| *** manishg has joined #openstack-neutron | 02:31 | |
| *** yamamoto_ has joined #openstack-neutron | 02:32 | |
| *** nati_ueno has quit IRC | 02:32 | |
| *** nati_ueno has joined #openstack-neutron | 02:33 | |
| *** vivek-ebay has quit IRC | 02:33 | |
| *** vivek-ebay has joined #openstack-neutron | 02:34 | |
| *** manishg_ has joined #openstack-neutron | 02:34 | |
| *** AbsinthMind1 has joined #openstack-neutron | 02:35 | |
| openstackgerrit | shihanzhang proposed a change to openstack/neutron: Add ipset support for Security Groups https://review.openstack.org/111877 | 02:35 |
|---|---|---|
| *** manishg has quit IRC | 02:35 | |
| *** dougwig has quit IRC | 02:36 | |
| *** AbsinthMind has quit IRC | 02:36 | |
| *** manishg has joined #openstack-neutron | 02:38 | |
| *** nlahouti has joined #openstack-neutron | 02:38 | |
| *** manishg_ has quit IRC | 02:38 | |
| *** dougwig_ has joined #openstack-neutron | 02:39 | |
| *** nati_uen_ has joined #openstack-neutron | 02:39 | |
| *** HenryG has joined #openstack-neutron | 02:40 | |
| *** tflynn_ has quit IRC | 02:41 | |
| *** banix has quit IRC | 02:41 | |
| *** manishg_ has joined #openstack-neutron | 02:42 | |
| *** manishg has quit IRC | 02:42 | |
| *** nati_ueno has quit IRC | 02:42 | |
| *** AbsinthMind has joined #openstack-neutron | 02:42 | |
| *** AbsinthMind1 has quit IRC | 02:45 | |
| *** manishg_ has quit IRC | 02:46 | |
| *** manishg has joined #openstack-neutron | 02:46 | |
| *** ianw has quit IRC | 02:46 | |
| *** ianw has joined #openstack-neutron | 02:46 | |
| hanzhang | hi all! | 02:50 |
| *** manishg has quit IRC | 02:50 | |
| *** manishg has joined #openstack-neutron | 02:51 | |
| *** sbfox has quit IRC | 02:52 | |
| *** manishg_ has joined #openstack-neutron | 02:53 | |
| *** networkstatic has quit IRC | 02:54 | |
| *** manishg has quit IRC | 02:55 | |
| *** manishg has joined #openstack-neutron | 02:56 | |
| *** dvorkinista has quit IRC | 02:56 | |
| *** manishg_ has quit IRC | 02:57 | |
| *** tsg has quit IRC | 02:58 | |
| *** networkstatic has joined #openstack-neutron | 02:58 | |
| *** manishg_ has joined #openstack-neutron | 02:59 | |
| *** yfried has joined #openstack-neutron | 02:59 | |
| *** sgordon_ has joined #openstack-neutron | 03:00 | |
| *** manishg has quit IRC | 03:00 | |
| *** mitarun has joined #openstack-neutron | 03:01 | |
| *** manishg has joined #openstack-neutron | 03:01 | |
| *** manishg_ has quit IRC | 03:03 | |
| *** manishg_ has joined #openstack-neutron | 03:04 | |
| *** zz_naotok is now known as naotok | 03:04 | |
| *** manishg has quit IRC | 03:05 | |
| *** tongli has quit IRC | 03:05 | |
| *** manishg has joined #openstack-neutron | 03:07 | |
| *** yfried has quit IRC | 03:07 | |
| *** manishg_ has quit IRC | 03:08 | |
| *** k_stev has quit IRC | 03:09 | |
| *** manishg has quit IRC | 03:11 | |
| *** a_le has joined #openstack-neutron | 03:12 | |
| *** sgordon_ has quit IRC | 03:14 | |
| *** baoli_ has quit IRC | 03:15 | |
| *** sgordon_ has joined #openstack-neutron | 03:15 | |
| *** suresh12 has quit IRC | 03:17 | |
| *** JayJ has joined #openstack-neutron | 03:20 | |
| *** phil_h has quit IRC | 03:20 | |
| *** baoli has joined #openstack-neutron | 03:20 | |
| *** prasoon has joined #openstack-neutron | 03:20 | |
| openstackgerrit | shihanzhang proposed a change to openstack/neutron: Add ipset support for Security Groups https://review.openstack.org/111877 | 03:23 |
| *** ivar-lazzaro1 has quit IRC | 03:23 | |
| *** JayJ has quit IRC | 03:24 | |
| *** EttoreZugliani has quit IRC | 03:33 | |
| *** AbsinthMind has quit IRC | 03:33 | |
| *** manishg has joined #openstack-neutron | 03:34 | |
| *** signed8bit has quit IRC | 03:35 | |
| *** amcrn_ has quit IRC | 03:36 | |
| *** manishg has quit IRC | 03:36 | |
| *** phil_h has joined #openstack-neutron | 03:37 | |
| *** manishg has joined #openstack-neutron | 03:37 | |
| openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Remove @author tags and update copyright notices https://review.openstack.org/118951 | 03:40 |
| *** manishg has quit IRC | 03:41 | |
| *** manishg_ has joined #openstack-neutron | 03:41 | |
| *** LiberTerra_away is now known as LiberTerra | 03:42 | |
| *** devvesa has joined #openstack-neutron | 03:43 | |
| *** manishg has joined #openstack-neutron | 03:45 | |
| *** tomoe_ has joined #openstack-neutron | 03:45 | |
| *** _cjones_ has joined #openstack-neutron | 03:46 | |
| *** manishg_ has quit IRC | 03:46 | |
| *** zhhuabj has quit IRC | 03:47 | |
| *** zhhuabj has joined #openstack-neutron | 03:47 | |
| *** devvesa has quit IRC | 03:48 | |
| *** prasoon_ has joined #openstack-neutron | 03:49 | |
| *** prasoon has quit IRC | 03:49 | |
| *** manishg has quit IRC | 03:49 | |
| *** zilberstein_ has quit IRC | 03:51 | |
| nunizacu | hi, i cant get metadata proxy to work | 03:53 |
| *** a_le has quit IRC | 03:53 | |
| *** xgerman_ has joined #openstack-neutron | 03:55 | |
| *** phil_h has quit IRC | 03:55 | |
| nunizacu | proxy dont work at all and dhcp is slow as hell | 03:56 |
| nunizacu | i dont even know what exactly configs/logs i should paste | 03:57 |
| nunizacu | or if someone is willing to help | 03:57 |
| nunizacu | i guess not, another dead channel | 04:00 |
| *** sgordon_ has quit IRC | 04:02 | |
| *** zhhuabj has quit IRC | 04:04 | |
| *** nati_uen_ has quit IRC | 04:05 | |
| *** zhhuabj has joined #openstack-neutron | 04:08 | |
| *** signed8bit has joined #openstack-neutron | 04:08 | |
| *** nati_ueno has joined #openstack-neutron | 04:09 | |
| *** signed8bit has quit IRC | 04:09 | |
| *** signed8bit has joined #openstack-neutron | 04:10 | |
| *** mitarun has quit IRC | 04:11 | |
| *** mitarun has joined #openstack-neutron | 04:12 | |
| *** phil_h has joined #openstack-neutron | 04:12 | |
| openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: ofagent: Fix a possible crash in arp responder https://review.openstack.org/118958 | 04:15 |
| *** skhandel has joined #openstack-neutron | 04:16 | |
| *** mitarun has quit IRC | 04:17 | |
| *** JayJ has joined #openstack-neutron | 04:20 | |
| *** vivek-eb_ has joined #openstack-neutron | 04:21 | |
| *** vivek-ebay has quit IRC | 04:24 | |
| *** JayJ has quit IRC | 04:25 | |
| *** suresh12 has joined #openstack-neutron | 04:28 | |
| *** morganfainberg_Z is now known as morganfainberg | 04:29 | |
| *** tflynn has joined #openstack-neutron | 04:29 | |
| *** suresh12 has quit IRC | 04:32 | |
| *** vivek-eb_ has quit IRC | 04:35 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/python-neutronclient: Leverage openstack.common.jsonutils https://review.openstack.org/118962 | 04:38 |
| *** skhandel has quit IRC | 04:42 | |
| *** ramishra has quit IRC | 04:44 | |
| *** ramishra has joined #openstack-neutron | 04:51 | |
| *** cshahani has quit IRC | 04:51 | |
| *** cshahani has joined #openstack-neutron | 04:52 | |
| *** mitarun has joined #openstack-neutron | 04:53 | |
| *** trinaths has joined #openstack-neutron | 04:55 | |
| *** ramishra_ has joined #openstack-neutron | 04:56 | |
| *** ramishra has quit IRC | 04:57 | |
| *** ramishra_ has quit IRC | 04:58 | |
| *** ramishra has joined #openstack-neutron | 04:59 | |
| *** ramishra has quit IRC | 04:59 | |
| *** xgerman_ has quit IRC | 05:00 | |
| *** ramishra has joined #openstack-neutron | 05:00 | |
| *** tflynn has quit IRC | 05:02 | |
| *** tflynn has joined #openstack-neutron | 05:03 | |
| *** saju_m has joined #openstack-neutron | 05:04 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/python-neutronclient: Leverage neutronclient.openstack.common.importutils import_class https://review.openstack.org/118969 | 05:05 |
| *** _cjones_ has quit IRC | 05:05 | |
| *** _cjones_ has joined #openstack-neutron | 05:05 | |
| nunizacu | FYI there is a bug with causes problems when your hostname has capital letters | 05:10 |
| *** trinaths has quit IRC | 05:10 | |
| *** a_le has joined #openstack-neutron | 05:12 | |
| *** phil_h has quit IRC | 05:13 | |
| *** trinaths has joined #openstack-neutron | 05:13 | |
| *** rotbeard has joined #openstack-neutron | 05:14 | |
| *** a_le has quit IRC | 05:17 | |
| *** nlahouti has quit IRC | 05:20 | |
| *** JayJ has joined #openstack-neutron | 05:21 | |
| *** cipcosma has joined #openstack-neutron | 05:22 | |
| *** baoli has quit IRC | 05:22 | |
| *** JayJ has quit IRC | 05:26 | |
| *** suresh12 has joined #openstack-neutron | 05:27 | |
| *** s0mik has joined #openstack-neutron | 05:27 | |
| trinaths | Hi All. Very Good Morning | 05:29 |
| kevinbenton | nunizacu: bug with what? | 05:29 |
| *** devvesa has joined #openstack-neutron | 05:32 | |
| *** nlahouti has joined #openstack-neutron | 05:33 | |
| *** dbite has joined #openstack-neutron | 05:33 | |
| *** tflynn has quit IRC | 05:34 | |
| *** killer_prince is now known as lazy_prince | 05:34 | |
| *** tflynn has joined #openstack-neutron | 05:34 | |
| *** phil_h has joined #openstack-neutron | 05:35 | |
| *** devvesa has quit IRC | 05:37 | |
| *** Longgeek has joined #openstack-neutron | 05:44 | |
| *** signed8bit has quit IRC | 05:44 | |
| openstackgerrit | Koteswara Rao Kelam proposed a change to openstack/neutron: shared policy shouldn't have unshared rules https://review.openstack.org/115912 | 05:45 |
| *** a_le has joined #openstack-neutron | 05:46 | |
| *** lori|away is now known as lori | 05:46 | |
| *** neeti has joined #openstack-neutron | 05:48 | |
| *** trinaths has quit IRC | 05:49 | |
| *** mitarun has quit IRC | 05:49 | |
| *** mitarun has joined #openstack-neutron | 05:50 | |
| *** harlowja is now known as harlowja_away | 05:50 | |
| *** mitarun has quit IRC | 05:50 | |
| *** mitarun has joined #openstack-neutron | 05:50 | |
| *** afazekas has joined #openstack-neutron | 05:51 | |
| *** dvorkinista has joined #openstack-neutron | 05:52 | |
| *** linuxgeek_ has joined #openstack-neutron | 05:56 | |
| openstackgerrit | garyk proposed a change to openstack/neutron: Remove @author from copyright statements https://review.openstack.org/112329 | 05:58 |
| *** JayJ has joined #openstack-neutron | 06:00 | |
| *** _cjones_ has quit IRC | 06:00 | |
| *** _cjones_ has joined #openstack-neutron | 06:01 | |
| openstackgerrit | Preeti Mirji proposed a change to openstack/neutron: Adm policy can't use other tenants unshared rules https://review.openstack.org/108952 | 06:02 |
| *** JayJ has quit IRC | 06:05 | |
| *** dbite has quit IRC | 06:05 | |
| *** _cjones_ has quit IRC | 06:05 | |
| *** dbite has joined #openstack-neutron | 06:05 | |
| *** cipcosma has quit IRC | 06:06 | |
| openstackgerrit | Preeti Mirji proposed a change to openstack/neutron: Adm policy can't use other tenants unshared rules https://review.openstack.org/108952 | 06:07 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/118259 | 06:07 |
| *** SridharG has joined #openstack-neutron | 06:11 | |
| *** saju_m has quit IRC | 06:11 | |
| *** emagana has joined #openstack-neutron | 06:12 | |
| *** Longgeek has quit IRC | 06:12 | |
| *** emagana has quit IRC | 06:12 | |
| *** yfried has joined #openstack-neutron | 06:12 | |
| *** lazy_prince is now known as killer_prince | 06:14 | |
| *** nlahouti1 has joined #openstack-neutron | 06:16 | |
| *** killer_prince is now known as lazy_prince | 06:17 | |
| *** Longgeek has joined #openstack-neutron | 06:18 | |
| *** nlahouti has quit IRC | 06:18 | |
| *** trinaths has joined #openstack-neutron | 06:19 | |
| *** yfried_ has joined #openstack-neutron | 06:19 | |
| *** seizadi has joined #openstack-neutron | 06:19 | |
| *** nlahouti1 has quit IRC | 06:20 | |
| *** saju_m has joined #openstack-neutron | 06:20 | |
| *** yfried has quit IRC | 06:20 | |
| *** yamamoto_ has quit IRC | 06:21 | |
| *** Longgeek_ has joined #openstack-neutron | 06:21 | |
| *** Longgeek_ has quit IRC | 06:22 | |
| *** Longgeek_ has joined #openstack-neutron | 06:23 | |
| *** akamyshnikova has quit IRC | 06:24 | |
| *** Longgeek has quit IRC | 06:24 | |
| *** simon-AS559 has joined #openstack-neutron | 06:24 | |
| *** Longgeek has joined #openstack-neutron | 06:24 | |
| mitarun | Hello, Please review this change: Fixes port deletion failure if port missing in NVP https://review.openstack.org/#/c/116640/ | 06:24 |
| *** Youcef has joined #openstack-neutron | 06:25 | |
| *** amotoki has joined #openstack-neutron | 06:25 | |
| *** Longgeek_ has quit IRC | 06:26 | |
| *** suresh12 has quit IRC | 06:32 | |
| *** bvandenh has joined #openstack-neutron | 06:33 | |
| *** a_le has quit IRC | 06:34 | |
| *** kopparam has joined #openstack-neutron | 06:34 | |
| *** seizadi1 has joined #openstack-neutron | 06:34 | |
| *** seizadi has quit IRC | 06:34 | |
| *** a_le has joined #openstack-neutron | 06:34 | |
| *** seizadi1 has quit IRC | 06:35 | |
| *** a_le has joined #openstack-neutron | 06:39 | |
| openstackgerrit | A change was merged to openstack/neutron: Don't spawn metadata-proxy for non-isolated nets https://review.openstack.org/116832 | 06:40 |
| *** amaretskiy has joined #openstack-neutron | 06:45 | |
| *** tflynn has quit IRC | 06:47 | |
| *** a_le has quit IRC | 06:48 | |
| *** saju_m has quit IRC | 06:49 | |
| *** marun_ has joined #openstack-neutron | 06:49 | |
| *** nati_ueno has quit IRC | 06:53 | |
| *** skolekonov has joined #openstack-neutron | 06:54 | |
| *** yamamoto_ has joined #openstack-neutron | 06:55 | |
| *** sunrenjie6 has joined #openstack-neutron | 07:01 | |
| *** yfauser has joined #openstack-neutron | 07:01 | |
| *** JayJ has joined #openstack-neutron | 07:01 | |
| *** yfauser has left #openstack-neutron | 07:01 | |
| *** pasquier-s has joined #openstack-neutron | 07:02 | |
| openstackgerrit | Vivekanandan Narasimhan proposed a change to openstack/neutron: DVR to delete router namespaces for service ports https://review.openstack.org/118580 | 07:03 |
| *** flaviof_zzz has quit IRC | 07:03 | |
| *** roaet_ has quit IRC | 07:03 | |
| *** adam_g has quit IRC | 07:04 | |
| *** dguitarbite has quit IRC | 07:04 | |
| *** wendar has quit IRC | 07:04 | |
| *** wendar_ has joined #openstack-neutron | 07:04 | |
| *** JayJ has quit IRC | 07:05 | |
| *** roaet_ has joined #openstack-neutron | 07:05 | |
| *** sunrenjie6 has quit IRC | 07:05 | |
| *** jpich has joined #openstack-neutron | 07:05 | |
| *** adam_g has joined #openstack-neutron | 07:06 | |
| *** adam_g has quit IRC | 07:06 | |
| *** adam_g has joined #openstack-neutron | 07:06 | |
| *** sunrenjie6 has joined #openstack-neutron | 07:07 | |
| *** luqas has joined #openstack-neutron | 07:07 | |
| *** linuxgeek_ has quit IRC | 07:08 | |
| *** jistr has joined #openstack-neutron | 07:09 | |
| *** yamamoto_ has quit IRC | 07:10 | |
| *** sunrenjie6 has quit IRC | 07:11 | |
| *** evgenyf has joined #openstack-neutron | 07:11 | |
| *** saju_m has joined #openstack-neutron | 07:12 | |
| openstackgerrit | liusheng proposed a change to openstack/python-neutronclient: Correct the bash completion of CLI https://review.openstack.org/107226 | 07:12 |
| *** sunrenjie6 has joined #openstack-neutron | 07:14 | |
| *** saju_m has quit IRC | 07:16 | |
| *** neeti has quit IRC | 07:18 | |
| *** dvorkinista has quit IRC | 07:18 | |
| *** sunrenjie6 has quit IRC | 07:19 | |
| *** jgallard has joined #openstack-neutron | 07:20 | |
| *** Longgeek has quit IRC | 07:20 | |
| *** neeti has joined #openstack-neutron | 07:21 | |
| *** devvesa has joined #openstack-neutron | 07:21 | |
| *** Longgeek has joined #openstack-neutron | 07:21 | |
| *** dvorkinista has joined #openstack-neutron | 07:22 | |
| *** ramishra has quit IRC | 07:24 | |
| *** saju_m has joined #openstack-neutron | 07:25 | |
| *** ramishra has joined #openstack-neutron | 07:25 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/95900 | 07:25 |
| *** devvesa has quit IRC | 07:25 | |
| *** ramishra has quit IRC | 07:25 | |
| *** ramishra has joined #openstack-neutron | 07:26 | |
| *** Longgeek_ has joined #openstack-neutron | 07:26 | |
| *** Longgeek_ has quit IRC | 07:27 | |
| *** jlibosva has joined #openstack-neutron | 07:27 | |
| *** Longgeek_ has joined #openstack-neutron | 07:27 | |
| *** flaviof_zzz has joined #openstack-neutron | 07:28 | |
| *** karimb has joined #openstack-neutron | 07:28 | |
| *** Longgeek has quit IRC | 07:29 | |
| *** doude has joined #openstack-neutron | 07:29 | |
| *** mitarun has quit IRC | 07:30 | |
| *** mitarun has joined #openstack-neutron | 07:31 | |
| *** sunrenjie6 has joined #openstack-neutron | 07:32 | |
| *** Altran has joined #openstack-neutron | 07:32 | |
| *** Longgeek has joined #openstack-neutron | 07:33 | |
| *** mitarun has quit IRC | 07:36 | |
| *** sunrenjie6 has quit IRC | 07:36 | |
| *** Longgeek_ has quit IRC | 07:36 | |
| *** Longgeek has quit IRC | 07:37 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy DB-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/96050 | 07:38 |
| *** ihrachyshka has joined #openstack-neutron | 07:38 | |
| *** Longgeek has joined #openstack-neutron | 07:38 | |
| *** mspreitz has quit IRC | 07:40 | |
| *** karimb has quit IRC | 07:42 | |
| *** ramishra has quit IRC | 07:42 | |
| *** suresh12 has joined #openstack-neutron | 07:42 | |
| *** jschwarz has joined #openstack-neutron | 07:42 | |
| *** ihrachyshka has quit IRC | 07:42 | |
| *** gildub has quit IRC | 07:43 | |
| *** dvorkinista has quit IRC | 07:44 | |
| *** devvesa has joined #openstack-neutron | 07:44 | |
| *** ygbo has joined #openstack-neutron | 07:46 | |
| *** crc32 has quit IRC | 07:47 | |
| *** suresh12 has quit IRC | 07:47 | |
| jschwarz | Morning all :) | 07:47 |
| *** prasoon has joined #openstack-neutron | 07:49 | |
| *** hanzhang has quit IRC | 07:49 | |
| *** prasoon_ has quit IRC | 07:50 | |
| *** hanzhang has joined #openstack-neutron | 07:50 | |
| *** dguitarbite has joined #openstack-neutron | 07:50 | |
| openstackgerrit | Angus Lees proposed a change to openstack/neutron: Add pylint tox environment and disable all existing warnings https://review.openstack.org/115486 | 07:51 |
| *** jp_at_hp has joined #openstack-neutron | 07:53 | |
| *** yamamoto_ has joined #openstack-neutron | 07:54 | |
| *** prasoon has quit IRC | 07:54 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/96393 | 07:54 |
| *** prasoon has joined #openstack-neutron | 07:55 | |
| *** jgallard has quit IRC | 07:55 | |
| *** jgallard has joined #openstack-neutron | 07:55 | |
| ajo | jschwarz, morning | 07:58 |
| jschwarz | ajo, morning | 07:58 |
| *** yamamoto_ has quit IRC | 07:58 | |
| jschwarz | ajo, my first patch went through the gate this morning :D | 07:59 |
| *** yamamoto_ has joined #openstack-neutron | 07:59 | |
| jschwarz | ajo, https://review.openstack.org/116832 :) | 07:59 |
| ajo | congrats ;) | 07:59 |
| jschwarz | ty, ty | 07:59 |
| ajo | :D | 07:59 |
| jschwarz | ajo, question regarding https://review.openstack.org/#/c/114927/ | 07:59 |
| jschwarz | I'm working on fixing that bug we talked about yesterday | 08:00 |
| ajo | jschwarz, brb, hanzhang is waiting for me | 08:00 |
| ajo | I'll read you in a few minutes | 08:00 |
| jschwarz | was wondering when the 'respawn_interval' is used and what for | 08:00 |
| jschwarz | what's the normal value I should expect? I can change 10 to 120 but if respawn_interval is used a lot in the code... | 08:01 |
| jschwarz | ajo, ^ when you get back :) | 08:01 |
| trinaths | jschwarz: Hello.. | 08:01 |
| *** yamamoto_ has quit IRC | 08:01 | |
| *** JayJ has joined #openstack-neutron | 08:02 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Mapping API-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/100716 | 08:02 |
| jschwarz | trinaths, good morning friend | 08:02 |
| jschwarz | how are you? | 08:03 |
| *** dbite_ has joined #openstack-neutron | 08:06 | |
| *** JayJ has quit IRC | 08:06 | |
| *** dbite has quit IRC | 08:06 | |
| *** dbite_ has quit IRC | 08:08 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Mapping DB-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/101795 | 08:09 |
| *** beagles has quit IRC | 08:13 | |
| *** safchain has joined #openstack-neutron | 08:13 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Mapping Plugin-1: EP, EPG, L2 Policy, L3 Policy https://review.openstack.org/101957 | 08:14 |
| *** yfried_ has quit IRC | 08:16 | |
| *** stack__ has quit IRC | 08:16 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy: Implicit Policy Driver https://review.openstack.org/104727 | 08:16 |
| openstackgerrit | A change was merged to openstack/neutron: Refactor security group rpc call https://review.openstack.org/111876 | 08:18 |
| *** garyk has joined #openstack-neutron | 08:18 | |
| openstackgerrit | Chengli Xu proposed a change to openstack/neutron: Fix metadata agent's auth info caching https://review.openstack.org/118996 | 08:18 |
| *** ZZelle has quit IRC | 08:18 | |
| *** ZZelle has joined #openstack-neutron | 08:19 | |
| ajo | jschwarz, I'll be busy until 12:30 +/-, if HenryG is around I believe he knows the topic, may be I'm wrong, look for the reviews of my original timeout patch | 08:20 |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver https://review.openstack.org/105272 | 08:20 |
| jschwarz | ajo, aye, already looked through the code and figured it out :) | 08:21 |
| jschwarz | ajo, thanks anyways :) see you tonight? | 08:21 |
| ajo | jschwarz, sure :) | 08:21 |
| ajo | jschwarz, I'll ask you later about what you found, I'm curious :) | 08:22 |
| *** matrohon has joined #openstack-neutron | 08:22 | |
| jschwarz | ajo, np, look me up :) | 08:22 |
| ajo | jschwarz, I'm trying to find out why an unit test fails @ CI checks, but, works locally | 08:23 |
| *** ramishra has joined #openstack-neutron | 08:23 | |
| ajo | jschwarz: https://review.openstack.org/#/c/111877/ + https://jenkins04.openstack.org/job/gate-neutron-python26/3224/console | 08:23 |
| *** ramishra has quit IRC | 08:23 | |
| *** ramishra has joined #openstack-neutron | 08:24 | |
| jschwarz | ajo, oh, ouch :< | 08:24 |
| jschwarz | ajo, I'll look at it after finishing the current patchset | 08:25 |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy API-2: Classifiers, Actions, and Rules https://review.openstack.org/101816 | 08:25 |
| *** garyk has quit IRC | 08:26 | |
| ajo | jschwarz, thanks :) | 08:27 |
| *** networkstatic has quit IRC | 08:28 | |
| jschwarz | sure thing | 08:28 |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy DB-2: Classifiers, Actions, and Rules https://review.openstack.org/103456 | 08:28 |
| Altran | ajo: which CI tests fail because I don't see any failed builds | 08:28 |
| ajo | Altran, still not posted to the review by jenkins (is still testing) | 08:29 |
| ajo | Altran, but the 2nd link is the result of the current run | 08:29 |
| ajo | for py26 | 08:29 |
| ajo | hmmm, let me check the py27 part | 08:29 |
| Altran | ajo: oh okay didn't open it, probably should have :P | 08:30 |
| jschwarz | hey Altran :) | 08:30 |
| Altran | jschwarz: morning :) | 08:30 |
| jschwarz | gah, I hate errors like "False is not True" | 08:30 |
| jschwarz | :< | 08:30 |
| *** rotbeard has quit IRC | 08:30 | |
| ajo | Altran: py27 fails too https://jenkins05.openstack.org/job/gate-neutron-python27/3062/console | 08:31 |
| *** prasoon has quit IRC | 08:31 | |
| *** beagles has joined #openstack-neutron | 08:31 | |
| Altran | ajo: yep - I've looked at zuul and found it myself | 08:31 |
| *** beagles is now known as Guest53651 | 08:31 | |
| *** mitarun has joined #openstack-neutron | 08:31 | |
| *** prasoon has joined #openstack-neutron | 08:32 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Plugin-2: Classifiers, Actions, and Rules https://review.openstack.org/103465 | 08:35 |
| *** mitarun has quit IRC | 08:36 | |
| *** karimb has joined #openstack-neutron | 08:37 | |
| *** dave_tucker_zzz is now known as dave_tucker | 08:37 | |
| *** dave_tucker is now known as dave_tucker_zzz | 08:38 | |
| *** gildub has joined #openstack-neutron | 08:38 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy API-3: Contracts https://review.openstack.org/103486 | 08:40 |
| *** saju_m has quit IRC | 08:41 | |
| openstackgerrit | Chengli Xu proposed a change to openstack/neutron: Fix metadata agent's auth info caching https://review.openstack.org/118996 | 08:43 |
| Altran | ajo: O | 08:43 |
| Altran | ajo: I*'ve managed to reproduce it | 08:44 |
| ajo | Altran++ | 08:44 |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy DB-3: Contracts https://review.openstack.org/103755 | 08:44 |
| ajo | how did you do?, does it fail in your system? | 08:44 |
| Altran | normally not - but I suspected some race etc. and run hashtest few times | 08:44 |
| Altran | tox --hashseed 3315667620 -ehashtest -- neutron.tests.unit.test_security_groups_rpc.TestSecurityGroupAgentEnhancedRpcWithIptables.test_security_group_rule_updated | 08:45 |
| Altran | ajo: this should (I hope) fail on all systems | 08:45 |
| *** prasoon has quit IRC | 08:45 | |
| *** prasoon has joined #openstack-neutron | 08:45 | |
| *** yamamoto_ has joined #openstack-neutron | 08:46 | |
| openstackgerrit | John Schwarz proposed a change to openstack/neutron: Fix leftover Timeout effecting most eventlet calls https://review.openstack.org/119001 | 08:47 |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy Plugin-3: Contracts https://review.openstack.org/103798 | 08:47 |
| trinaths | jschwarz: Very Good Afternoon. Me doing gud.. came ack from lunch | 08:47 |
| jschwarz | nice :> going out to eat mine soonish | 08:47 |
| ajo | hmm my tox doesn't accept those parameters | 08:48 |
| Altran | ajo: do You have new tox ? | 08:48 |
| trinaths | jschwarz: :-) | 08:48 |
| ajo | Altran, which tox version do I need? | 08:48 |
| Altran | tox>=1.7.2 | 08:48 |
| ajo | hmmm, 1.6.1 here | 08:49 |
| *** amuller has joined #openstack-neutron | 08:49 | |
| ajo | ok, upgraded tox | 08:50 |
| *** mitarun has joined #openstack-neutron | 08:50 | |
| *** yamamoto_ has quit IRC | 08:50 | |
| Altran | ajo: hmm but I'm not sure now if those fails are related - the same test fails but on different thing :P | 08:51 |
| Altran | ajo: will dig into in a bit | 08:51 |
| *** lordd has joined #openstack-neutron | 08:51 | |
| ajo | Altran, what does the parameters change? | 08:51 |
| ajo | random names, uuids , or something like that? | 08:51 |
| Altran | ajo: things like that | 08:52 |
| ajo | hmmm | 08:52 |
| Altran | ajo: also order in dictionaries and sets | 08:52 |
| ajo | the failure I see is different to the one on the gate too | 08:52 |
| Altran | ajo: maybe something more I'm not sure | 08:52 |
| *** ramishra has quit IRC | 08:52 | |
| *** ramishra has joined #openstack-neutron | 08:53 | |
| Altran | ajo: iirc its target to run hastest on CI so in the long run this problem should be fixed but for now the jenkins fail is more important to merge | 08:54 |
| ajo | Altran, master branch also fails with those parameters | 08:54 |
| *** yamamoto_ has joined #openstack-neutron | 08:54 | |
| Altran | ajo: okay so thats unrelated | 08:54 |
| *** moha_hunt has joined #openstack-neutron | 08:55 | |
| Altran | ajo: will try to find 8 != 4 then :) | 08:55 |
| ajo | Altran, thank you very much, I'm looking for it too | 08:56 |
| ajo | hanzhang, ^ | 08:56 |
| ajo | your help is really appreciated | 08:56 |
| *** ramishra has quit IRC | 08:57 | |
| openstackgerrit | YAMAMOTO Takashi proposed a change to openstack/neutron: Make l2pop UT independent from OVS https://review.openstack.org/118620 | 08:58 |
| *** yamamoto_ has quit IRC | 08:59 | |
| *** JayJ has joined #openstack-neutron | 09:02 | |
| *** luqas has quit IRC | 09:03 | |
| *** m1dev has quit IRC | 09:04 | |
| *** yfauser1 has joined #openstack-neutron | 09:04 | |
| *** yfauser1 has left #openstack-neutron | 09:04 | |
| *** m1dev has joined #openstack-neutron | 09:04 | |
| *** yfauser2 has joined #openstack-neutron | 09:05 | |
| *** yfauser2 has left #openstack-neutron | 09:05 | |
| Altran | ajo: too bad the order of asserts isn't inverted - this way we would see which 4 calls were not executed :/ | 09:05 |
| *** JayJ has quit IRC | 09:07 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Group Policy: Resource Mapping Driver (Security Groups) https://review.openstack.org/113775 | 09:08 |
| ajo | Altran, may be we can introduce a patchset printing the actual calls | 09:08 |
| ajo | and compare to what we see locally | 09:08 |
| *** lordd has quit IRC | 09:10 | |
| ajo | Altran, I'm going to do that to gather more debug info | 09:11 |
| Altran | ajo: okay | 09:11 |
| *** sambetts has joined #openstack-neutron | 09:12 | |
| *** Longgeek_ has joined #openstack-neutron | 09:13 | |
| *** Longgeek has quit IRC | 09:15 | |
| openstackgerrit | Zang MingJie proposed a change to openstack/neutron: Implement neutron guru meditation reports https://review.openstack.org/119008 | 09:15 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Fix UT for L2pop test_get_agent_ports_no_data() https://review.openstack.org/119009 | 09:16 |
| *** sridhar has joined #openstack-neutron | 09:17 | |
| *** Longgeek_ has quit IRC | 09:17 | |
| *** Longgeek has joined #openstack-neutron | 09:18 | |
| *** dave_tucker_zzz is now known as dave_tucker | 09:18 | |
| *** Guest41143 is now known as atmark | 09:20 | |
| *** atmark has quit IRC | 09:20 | |
| *** atmark has joined #openstack-neutron | 09:20 | |
| *** lordd has joined #openstack-neutron | 09:21 | |
| *** luqas has joined #openstack-neutron | 09:22 | |
| *** garyk has joined #openstack-neutron | 09:23 | |
| *** cipcosma has joined #openstack-neutron | 09:23 | |
| *** evgenyf has quit IRC | 09:24 | |
| *** LiberTerra has quit IRC | 09:24 | |
| Altran | ajo: I've noticed on logstash that there was also the same fail on 111876 change so underlying problem could be introduced there, but I've got no idea what could it be so far | 09:25 |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Throw exception instances instead of classes https://review.openstack.org/118338 | 09:25 |
| *** claudiub has joined #openstack-neutron | 09:25 | |
| ajo | Altran, same fail? | 09:26 |
| ajo | could you point me to it so I can check the specific patchset? | 09:26 |
| Altran | ajo: http://logs.openstack.org/76/111876/27/check/gate-neutron-python27/9773074/testr_results.html.gz | 09:26 |
| ajo | thanks | 09:26 |
| *** yfauser has joined #openstack-neutron | 09:26 | |
| *** yfauser has left #openstack-neutron | 09:27 | |
| *** evgenyf has joined #openstack-neutron | 09:27 | |
| ajo | hmm, let's try that patchset locally | 09:28 |
| *** mitarun has quit IRC | 09:28 | |
| ajo | I'll commit the debug patchset, and then let's try that | 09:28 |
| *** oreillyd has joined #openstack-neutron | 09:29 | |
| *** akamyshnikova has joined #openstack-neutron | 09:29 | |
| *** yamamoto_ has joined #openstack-neutron | 09:30 | |
| *** geekinut1h has joined #openstack-neutron | 09:31 | |
| *** geekinutah has quit IRC | 09:31 | |
| *** mitarun has joined #openstack-neutron | 09:34 | |
| *** yamamoto_ has quit IRC | 09:34 | |
| *** yfauser1 has joined #openstack-neutron | 09:37 | |
| *** yfauser1 has left #openstack-neutron | 09:37 | |
| *** saju_m has joined #openstack-neutron | 09:41 | |
| *** saju_m has quit IRC | 09:42 | |
| *** Longgeek_ has joined #openstack-neutron | 09:43 | |
| *** saju_m has joined #openstack-neutron | 09:43 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 09:44 |
| ajo | Altran, ^ let's see what happens | 09:44 |
| ajo | I added a bunch of debug output | 09:44 |
| *** dvorkinista has joined #openstack-neutron | 09:44 | |
| Altran | ajo: lets hope zuul will be nice and run at least one of py26/7 in reasonable time | 09:45 |
| *** Longgeek has quit IRC | 09:46 | |
| *** yfauser has joined #openstack-neutron | 09:48 | |
| *** dvorkinista has quit IRC | 09:49 | |
| ajo | Altran, 111876,27 passes locally too...!!! | 09:49 |
| *** yfauser has left #openstack-neutron | 09:49 | |
| ajo | bad, this is not going to help the gate | 09:50 |
| Altran | ajo: yeah | 09:50 |
| ajo | hanzhang, ^ http://logs.openstack.org/76/111876/27/check/gate-neutron-python27/9773074/testr_results.html.gz | 09:50 |
| ajo | Altran, it was the only ocurrence found? | 09:50 |
| *** jdag___ has quit IRC | 09:51 | |
| *** haruka has joined #openstack-neutron | 09:51 | |
| Altran | ajo: yes it only found fails from this and 11 and 12 of 111877 | 09:51 |
| ajo | ok, I'm going to read the test again, and compare to any other similar test | 09:51 |
| ajo | and keep an eye on the gate, not sure if changes on the RPC would trigger it, | 09:52 |
| ajo | or just any random run would trigger it | 09:52 |
| Altran | ajo: uh | 09:52 |
| Altran | ajo: I've checked it again because this change was just merged | 09:52 |
| ajo | Altran, it is | 09:52 |
| Altran | ajo: and new tests with it in master trigered | 09:52 |
| *** jdag___ has joined #openstack-neutron | 09:53 | |
| Altran | ajo: and one unrelated change failed on this | 09:53 |
| *** baohua has quit IRC | 09:53 | |
| ajo | Altran, may be we need to either revert the commit, or skip that test until we find the reason | 09:53 |
| ajo | mestery, salv-orlando ^ | 09:53 |
| *** yamamoto_ has joined #openstack-neutron | 09:54 | |
| Altran | ajo: yeah another nondeterministic fail won't help gate even a bit | 09:56 |
| ajo | Altran, it wont :( | 09:56 |
| *** yamamoto_ has quit IRC | 09:59 | |
| *** foexle has joined #openstack-neutron | 09:59 | |
| *** Altran is now known as jswiders | 10:00 | |
| ajo | Altran | 10:01 |
| ajo | my logstash search only finds the ipset change , and the SG RPC change | 10:01 |
| ajo | ah | 10:01 |
| ajo | no | 10:01 |
| ajo | wrong | 10:01 |
| jswiders | ajo: I'm here now - logged on old nick - but it doesn't reflect my real name ^^ | 10:02 |
| jswiders | ajo: i forget about using new all the time :P | 10:02 |
| *** xuhanp has quit IRC | 10:02 | |
| ajo | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiTWlzbWF0Y2hFcnJvcjogOCAhPSA0XCIgQU5EIHRhZ3M6XCJjb25zb2xlXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6ImN1c3RvbSIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJmcm9tIjoiMjAxNC0wOC0wMVQwMDowMDowMCswMDowMCIsInRvIjoiMjAxNC0wOS0wNFQwOTo1NzoyMSswMDowMCIsInVzZXJfaW50ZXJ2YWwiOiIwIn0sInN0YW1wIjoxNDA5ODI0ODE2NjUxLCJtb2RlIjoiIiwiYW5hbHl6ZV9maWVsZCI6IiJ9 | 10:03 |
| *** JayJ has joined #openstack-neutron | 10:03 | |
| jswiders | ajo: yeah | 10:03 |
| *** xuhanp has joined #openstack-neutron | 10:05 | |
| ajo | mestery, salv-orlando ,marun_ , amotoki , ^ the merged SG RPC security_group_info_for_devices change is producing a non deterministic unit test failure | 10:06 |
| ajo | I believe we should either revert it, or temporarily skip the test until I found the reason, | 10:06 |
| ajo | trying to debug it here: https://review.openstack.org/119018 | 10:06 |
| ajo | as it doesn't reproduce locally | 10:06 |
| ajo | I found->we find | 10:07 |
| *** pasquier-s has quit IRC | 10:07 | |
| ajo | Altran is helping with diagnosis | 10:07 |
| *** JayJ has quit IRC | 10:08 | |
| amuller | marun_: Why does moving away from select for update make any sense? | 10:09 |
| *** cshahani has quit IRC | 10:09 | |
| amuller | I don't understand why one DB's fault means that we move away from a proven, stable solution and replace it with weird app level solutions | 10:10 |
| amuller | retries everywhere, new tables, custom code in every query | 10:10 |
| amuller | It's too much app level complexity | 10:10 |
| amuller | amotoki: safchain: Sylvain will move away from select for update on the VRRP patch | 10:12 |
| amuller | but generally speaking I disagree with this move in Neutron to remove usages of select for update | 10:12 |
| *** jgallard has quit IRC | 10:12 | |
| *** matrohon has quit IRC | 10:13 | |
| jswiders | ajo: does zuul react to recheck/reverify before it finishes run ? or do You need to push another PS to retriger cause it didn't fail this time ? | 10:13 |
| ajo | hmm jswiders going | 10:13 |
| amotoki | amuller: I understand avoiding usage of select for update is sometimes a bit pain for us. it is an input from ops-meetup in Atlanta. | 10:14 |
| amuller | amotoki: Isn't it possible to fix it in Galera? | 10:14 |
| amuller | I mean seriously does every app in the world today going to move from select for update to app level locking solutions? That's insane | 10:15 |
| amotoki | amuller: honestly i am not sure. | 10:15 |
| *** saju_m has quit IRC | 10:16 | |
| *** yamahata has quit IRC | 10:16 | |
| amotoki | I think we need to clarify the requirement on this. If neutron removed select for update completely, there is no meaning. we have several select for udpate already. | 10:16 |
| jschwarz | marun_, ping | 10:16 |
| jschwarz | ajo, did you find the cause for that bug you mentioned earlier? | 10:18 |
| ajo | jschwarz, no, and worse yet, it was introduced by the merged change on the enhanced SG RPC | 10:18 |
| ajo | as a non deterministic failure | 10:18 |
| jschwarz | ouch | 10:18 |
| ajo | ouch! | 10:18 |
| jswiders | ^^ | 10:18 |
| ajo | see log lines around 12:06 | 10:18 |
| jschwarz | aye, that logstash? | 10:19 |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 10:19 |
| ajo | jswiders, ^ retriggered | 10:19 |
| *** mitarun has quit IRC | 10:19 | |
| ajo | ah jswiders , we still had the py27 chance | 10:20 |
| ajo | for the first patchset | 10:20 |
| *** mitarun has joined #openstack-neutron | 10:20 | |
| jswiders | ajo: yeah but jenkins that runs 27 is overloaded and starts after ~2h into queue | 10:20 |
| ajo | ouch | 10:21 |
| jswiders | ajo: while 26 starts after few minutes | 10:21 |
| jschwarz | ajo, what can I do to help? | 10:21 |
| openstackgerrit | John Davidge proposed a change to openstack/neutron: Do not assume order of report list elements https://review.openstack.org/112836 | 10:21 |
| ajo | jschwarz, think about why neutron.tests.unit.test_security_groups_rpc.TestSecurityGroupAgentEnhancedIpsetWithIptables.test_security_group_rule_updated | 10:21 |
| ajo | could be failing non deterministcally... http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiTWlzbWF0Y2hFcnJvcjogOCAhPSA0XCIgQU5EIHRhZ3M6XCJjb25zb2xlXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6ImN1c3RvbSIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJmcm9tIjoiMjAxNC0wOC0wMVQwMDowMDowMCswMDowMCIsInRvIjoiMjAxNC0wOS0wNFQwOTo1NzoyMSswMDowMCIsInVzZXJfaW50ZXJ2YWwiOiIwIn0sInN0YW1wIjoxNDA5ODI0ODE2NjUxLCJtb2RlIjoiIiwiYW5hbHl6ZV9maWVs | 10:21 |
| ajo | ZCI6IiJ9 | 10:21 |
| kevinbenton | marun_: ping | 10:21 |
| jswiders | jschwarz: patch we think introduced it is https://review.openstack.org/#/c/111876 | 10:22 |
| ajo | correct | 10:22 |
| ajo | jswiders, could it be related to different execution order in tests? | 10:23 |
| ajo | may be one test modifies data or constants other test is using later... | 10:23 |
| jswiders | ajo: hmm if they aren't separated correctly then it might be the case | 10:23 |
| jschwarz | let me clear out some windows and i'll get right on it | 10:23 |
| jswiders | ajo: but I'm not sure about how would that happen | 10:24 |
| *** mitarun has quit IRC | 10:25 | |
| ajo | jswiders, I saw it before, when some method was modifying the structure it received | 10:25 |
| ajo | so, if you called it twice under the same test, original parameters were destroyed | 10:26 |
| jswiders | ajo: mhm | 10:26 |
| ajo | I'm wondering if here this could be happening, since we're reusing constants that we pass for the "enhanced RPC" and "not enhanced RPC" | 10:26 |
| ajo | so | 10:27 |
| *** xuhanp has quit IRC | 10:27 | |
| ajo | may be one order breaks it, the other doesn't | 10:27 |
| jschwarz | right, starting to check it out | 10:29 |
| *** k4n0_ has quit IRC | 10:33 | |
| *** prasoon has quit IRC | 10:34 | |
| *** k4n0 has joined #openstack-neutron | 10:35 | |
| *** xuhanp has joined #openstack-neutron | 10:40 | |
| HenryG | jschwarz: ajo: Good morning. Were you looking for me? | 10:40 |
| *** moha_hunt has quit IRC | 10:41 | |
| jschwarz | HenryG, not me :) | 10:41 |
| *** dvorkinista has joined #openstack-neutron | 10:41 | |
| *** aix has joined #openstack-neutron | 10:42 | |
| marun_ | kevinbenton: pong | 10:43 |
| marun_ | amuller: select for update is 'pessimistic locking' | 10:43 |
| marun_ | amuller: it limits scalability and is therefor bad in its own right | 10:43 |
| kevinbenton | marun_: nevermind, was going to ask a question but i found the answer | 10:44 |
| marun_ | amuller: it also restricts neutron to being deployed to single master db configurations | 10:44 |
| marun_ | amotoki: multi-master is another way to ensure scalability | 10:44 |
| marun_ | oops, sorry amotoki, meant for amuller | 10:44 |
| salv-orlando | ajo: well, that’s my fault as well. I should have caught the ut failures your patch exhibited on friday 29/8. I missed that. | 10:45 |
| jschwarz | marun_, please see comment on the change request | 10:45 |
| marun_ | amuller: I'm not saying that your patch _must_ avoid use of lockmode. Just that it would be preferable to do so. | 10:45 |
| jschwarz | tell me what you think :) | 10:45 |
| marun_ | jschwarz: will do | 10:45 |
| *** naotok is now known as zz_naotok | 10:45 | |
| *** dvorkinista has quit IRC | 10:45 | |
| ajo | salv-orlando, trying to find the reason, I have a guess | 10:46 |
| marun_ | jschwarz: I was not suggesting avoiding increasing the timeout | 10:46 |
| kevinbenton | marun_: which patch was the one you are referring to? | 10:46 |
| marun_ | kevinbenton: https://review.openstack.org/#/c/64553/ | 10:46 |
| marun_ | kevinbenton: I see you've commented as well | 10:46 |
| jschwarz | marun_, very well, then I'm increasing the default 5 (in the function's positional argument) to 20 | 10:46 |
| salv-orlando | ajo: we’re seeing quite a lot of hits. This is neutron only thankfully so the other projects are not affected. | 10:47 |
| jschwarz | marun_, I've been running this for about 20 minutes and it didn't fail once | 10:47 |
| marun_ | jschwarz: what I was asking was that you increase it _Only in the test_ | 10:47 |
| jschwarz | marun_, that I understand, but the original code has a default timeout of 5 seconds | 10:47 |
| ajo | salv-orlando, yes, I saw, | 10:47 |
| marun_ | jschwarz: the default does not matter | 10:47 |
| *** sijo_ has joined #openstack-neutron | 10:47 | |
| marun_ | jschwarz: because the only place it is used in the test and we can explictly set it to whatever we want | 10:48 |
| ajo | salv-orlando, question: within a test class, the execution order of the internal tests, can change? | 10:48 |
| jschwarz | marun_, you're right | 10:48 |
| jschwarz | marun_, will upload a new patchset in a set | 10:48 |
| jschwarz | sec* | 10:48 |
| marun_ | jschwarz++ | 10:48 |
| salv-orlando | ajo: it can whenever you add/remove tests to the class. | 10:48 |
| ajo | hmm, salv-orlando but not without adding/removing | 10:49 |
| marun_ | ajo: which change added the failing test? | 10:50 |
| ajo | marun_, the enhanced SG RPC refactor | 10:50 |
| marun_ | ajo: link? | 10:50 |
| sijo_ | Hi Everyone, I'm a openstack and network newbie, I'm following installation of openstack with three node architecture, controller, Network and Compute on Ubuntu 12.04 server. But I'm stuck up on 7th module Creating initial networks (http://docs.openstack.org/icehouse/install-guide/install/apt/content/neutron-initial-networks.html). for setting up external network do i need a separate network connection or can I use the existing one...? | 10:50 |
| jschwarz | marun_, https://review.openstack.org/#/c/111876 | 10:50 |
| amuller | sijo_: Please try ask.openstack.org | 10:51 |
| marun_ | danke | 10:51 |
| amuller | It's very active | 10:51 |
| ajo | http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiTWlzbWF0Y2hFcnJvcjogOCAhPSA0XCIgQU5EIHRhZ3M6XCJjb25zb2xlXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6ImN1c3RvbSIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJmcm9tIjoiMjAxNC0wOC0wMVQwMDowMDowMCswMDowMCIsInRvIjoiMjAxNC0wOS0wNFQwOTo1NzoyMSswMDowMCIsInVzZXJfaW50ZXJ2YWwiOiIwIn0sInN0YW1wIjoxNDA5ODI0ODE2NjUxLCJtb2RlIjoiIiwiYW5hbHl6ZV9maWVs | 10:51 |
| ajo | marun ^ this is the search for the kind of failures it's generating | 10:51 |
| marun_ | ajo: and what is the name of the test? | 10:51 |
| openstackgerrit | Juergen Brendel proposed a change to openstack/neutron: Fix ARP cache poisoning vulnerability https://review.openstack.org/119031 | 10:51 |
| *** dave_tucker is now known as dave_tucker_zzz | 10:52 | |
| ajo | marun_ neutron.tests.unit.test_security_groups_rpc.TestSecurityGroupAgentEnhancedIpsetWithIptables.test_security_group_rule_updated | 10:52 |
| marun_ | crazy that it's a _unit_ test that is non-determinstic :/ | 10:52 |
| ajo | yes | 10:52 |
| ajo | there must be something really wrong with the test for this to happen | 10:52 |
| jswiders | marun_: and so far we had no luck trying to reproduce it locally - only fails on gate | 10:52 |
| *** ramishra has joined #openstack-neutron | 10:53 | |
| ajo | marun_, I submitted a patchset with extra debug info, trying to get more information https://review.openstack.org/#/c/119018/ | 10:53 |
| *** yamamoto_ has joined #openstack-neutron | 10:54 | |
| *** jp_at_hp has quit IRC | 10:55 | |
| ajo | marun_, at 111876 the test class was different, let me get you a link, sorry | 10:56 |
| ajo | but the failure was the same, and also, it seems to be reproduced locally | 10:56 |
| openstackgerrit | John Schwarz proposed a change to openstack/neutron: Fix leftover Timeout effecting most eventlet calls https://review.openstack.org/119001 | 10:56 |
| jschwarz | marun_, done ^ | 10:56 |
| jswiders | ajo: You reproduced it locally ? | 10:57 |
| kevinbenton | marun_: oh man, my comment got discarded… | 10:57 |
| ajo | jswiders, nope | 10:57 |
| jschwarz | I'm also unable to reproduce | 10:58 |
| jschwarz | ajo, did you find out what are the 8 and 4 referenced by that bug? | 10:58 |
| ajo | marun_, neutron.tests.unit.test_security_groups_rpc.TestSecurityGroupAgentEnhancedRpcWithIptables.test_security_group_rule_updated is the right name | 10:58 |
| *** xuhanp has quit IRC | 10:58 | |
| ajo | jschwarz, yes, | 10:58 |
| jschwarz | ? | 10:58 |
| *** ramishra has quit IRC | 10:59 | |
| ajo | from the security group info result we mock, there is an expected amount of calls to iptables | 10:59 |
| ajo | sg_info -> ip tables calls | 10:59 |
| ajo | the test checks that | 10:59 |
| ajo | so for the fed sg_info ther must be 8 calls, but there are only 4 | 10:59 |
| *** yamamoto_ has quit IRC | 10:59 | |
| ajo | jschwarz, check "def _replay_iptables" | 11:00 |
| jschwarz | ajo, line 2320 is 'self._replay_iptables(2, V6_2)', right? I don't see a unittest assertion there | 11:00 |
| jschwarz | ajo, ah, the _register_mock_call... | 11:01 |
| ajo | yes, then it's checked all together at the end. | 11:01 |
| *** JayJ has joined #openstack-neutron | 11:01 | |
| jschwarz | I'm not looking at the same code | 11:02 |
| marun_ | jschwarz: reviewed, commit message needs cleanup but otherwise looks good | 11:02 |
| jschwarz | ajo, can you point me to a specific revision? | 11:02 |
| jschwarz | marun_, thanks :) | 11:02 |
| jswiders | jschwarz: its on master already so TestSecurityGroupAgentEnhancedRpcWithIptables.test_security_group_rule_updated | 11:03 |
| jswiders | jschwarz: from what I understand both self.agent.prepare_devices_filter(['tap_port1', 'tap_port3']) and self.agent.security_groups_rule_updated(['security_group1']) should cause 4 calls each | 11:03 |
| jswiders | so 8 in total - and defer_refresh_firewall is set to false | 11:03 |
| jswiders | so they shouldn't be defered | 11:04 |
| openstackgerrit | John Schwarz proposed a change to openstack/neutron: Fix leftover Timeout effecting most eventlet calls https://review.openstack.org/119001 | 11:04 |
| jschwarz | marun_, le wild fix appears :) | 11:04 |
| ajo | jschwarz, jswiders I start to think that's actually something broken within the code, not the UT itself, making that the security_groups_rule_updated doesn't actually trigger an iptables updated under some condition, | 11:05 |
| marun_ | jschwarz: oh, one more thing... | 11:05 |
| jschwarz | jswiders, I'm looking at the code but the traceback I'm looking doesn't match my code | 11:05 |
| jschwarz | marun_, lol :) ok | 11:05 |
| marun_ | jschwarz: you can submit a patch to revert the skip of the l3 agent functional test that depends on your patch | 11:06 |
| jswiders | jschwarz: http://logs.openstack.org/75/113775/5/check/gate-neutron-python26/0ab85fe/console.html is fail from master | 11:06 |
| jswiders | jschwarz: it should match Your lines numbers in master | 11:06 |
| marun_ | salv-orlando: Please take a look at this when you get a chance: https://review.openstack.org/#/c/119001/ | 11:06 |
| jschwarz | jswiders, can you run git log --oneline | head -n 1 | 11:07 |
| jschwarz | just so we're sure | 11:07 |
| marun_ | salv-orlando: it fixes the problem that was being masked by the lack of output earlier this week | 11:07 |
| jswiders | jschwarz: e04e67a Merge "Refactor security group rpc call" | 11:07 |
| marun_ | salv-orlando: and, naturally, was my fault! :) | 11:07 |
| marun_ | salv-orlando: it had been sitting there latent since I added the ovsdb monitor, just waiting for the functional job to creep above 10s in execution time | 11:08 |
| salv-orlando | ah ok makes sense | 11:08 |
| jschwarz | marun_, ok :) | 11:08 |
| *** JayJ has quit IRC | 11:09 | |
| salv-orlando | marun_, jschwarz: cool but we need to wait for jenkins to +1 before approving as infra is not forcing anymore a step through the check queue. so if I approve it will go straight in the gate queue. | 11:10 |
| *** JayJ_ has joined #openstack-neutron | 11:12 | |
| jschwarz | salv-orlando, ok :) | 11:12 |
| *** dave_tucker_zzz is now known as dave_tucker | 11:12 | |
| *** lordd has quit IRC | 11:13 | |
| *** lordd has joined #openstack-neutron | 11:14 | |
| jswiders | ajo: debug run again succeeded :( | 11:14 |
| *** jp_at_hp has joined #openstack-neutron | 11:15 | |
| *** JayJ_ has quit IRC | 11:15 | |
| openstackgerrit | John Schwarz proposed a change to openstack/neutron: Revert "Skip functional l3 agent test" https://review.openstack.org/119034 | 11:16 |
| jschwarz | marun_, Also done: ^ | 11:16 |
| jschwarz | jswiders, thanks, will look at it now | 11:18 |
| *** jab has joined #openstack-neutron | 11:18 | |
| *** jab has quit IRC | 11:18 | |
| *** jab has joined #openstack-neutron | 11:18 | |
| ajo | jswiders, let me re-send, | 11:19 |
| *** saju_m has joined #openstack-neutron | 11:19 | |
| *** xuhanp has joined #openstack-neutron | 11:20 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 11:20 |
| ajo | I need to eat something, will be right back into this | 11:20 |
| jswiders | okay :) | 11:21 |
| amuller | marun_: safchain: I forgot if I updated you, Sylvain is working on the server side patches, including the select for update debacle | 11:21 |
| amuller | I'm reporting bugs and afterwards I'll work on the agent patches | 11:21 |
| *** jab is now known as bradjones | 11:21 | |
| amuller | failover and such | 11:21 |
| *** ramishra has joined #openstack-neutron | 11:22 | |
| jswiders | jschwarz: what is even more bizzare about this fail is that it seems to always fail on both py26 and py27 - it only hit one patch unrelated to ajo but it hit both py26 and py27 | 11:23 |
| jschwarz | jswiders, what's so bizarre about it? | 11:24 |
| jswiders | jschwarz: well what are chances that it won't fail anywhere else but 2 times on the same change ? | 11:25 |
| jschwarz | jswiders, can you link me the unrelated-to-ajo time it hit? | 11:25 |
| jswiders | jschwarz: or wait maybe its not related | 11:25 |
| marun_ | amuller: ack | 11:26 |
| jswiders | jschwarz: I've only looked at it for a second but it does indeed change something in Security Groups but not in the same files | 11:26 |
| jswiders | jschwarz: https://review.openstack.org/#/c/113775/5 | 11:26 |
| *** ramishra has quit IRC | 11:27 | |
| jschwarz | ajo, I have a question regarding the code | 11:27 |
| jswiders | jschwarz: he went for a snack | 11:27 |
| *** banix has joined #openstack-neutron | 11:28 | |
| jschwarz | ajo, in line 2228-2233 you create self.iptables_execute_return_values and attach it as a side_effect which is later checked | 11:28 |
| jschwarz | but in L2218 you call to the super.setUp(), which does exactly that | 11:28 |
| *** saurabh_ has joined #openstack-neutron | 11:28 | |
| jschwarz | so the side effect is actually registered twice | 11:28 |
| *** saurabh_ has quit IRC | 11:28 | |
| *** saurabh_ has joined #openstack-neutron | 11:28 | |
| *** saurabh_ has joined #openstack-neutron | 11:28 | |
| jswiders | jschwarz: yeah I've also noticed that there is some duplication in those setups | 11:28 |
| jswiders | jschwarz: is registering side effect not idempotent ? | 11:29 |
| jschwarz | jswiders, I think it is, but go figure | 11:30 |
| *** pcm_ has joined #openstack-neutron | 11:30 | |
| jschwarz | jswiders, you linked me to here: https://review.openstack.org/#/c/113775/5 but I don't see that error reproduce there? | 11:31 |
| jswiders | jschwarz: run in still in zuul | 11:32 |
| jschwarz | ahhh | 11:32 |
| *** banix has quit IRC | 11:32 | |
| jswiders | jschwarz: i.e. http://logs.openstack.org/75/113775/5/check/gate-neutron-python27/698f366/console.html | 11:32 |
| jschwarz | link to that? ;-) | 11:32 |
| jschwarz | ty | 11:32 |
| jschwarz | yes, I see that now | 11:32 |
| *** john_mac has joined #openstack-neutron | 11:32 | |
| jswiders | jschwarz: hmm lines 2226-7 are also duplicated | 11:34 |
| jswiders | jschwarz: do You think it could be possible that 4 calls from agent go to one iptables and 4 to other ? | 11:34 |
| jswiders | jschwarz: so the one we check only has 4 calls instead of expected 8 ? | 11:34 |
| jschwarz | jswiders, no.. there aren't 2 objects, only one | 11:35 |
| jschwarz | I agree with ajo - I think it's a bug in the code itself. Somehow execute is ran 4 times and not 8 | 11:36 |
| jschwarz | maybe some error is raised and silently ignored causing iptables.execute to only run 4 times | 11:36 |
| *** xuhanp has quit IRC | 11:36 | |
| jswiders | jschwarz: ah yes nvm - self.iptables is the same in both cases | 11:36 |
| jswiders | jschwarz: I've tried to look for such case but so far didn't find one | 11:37 |
| jschwarz | jswiders, do you happen to know which "execute" function is being referred here? | 11:37 |
| *** nunizacu has quit IRC | 11:37 | |
| jschwarz | there are 2 different execute functions in the neutron/agent/linux lib | 11:38 |
| saurabh_ | Hi there, | 11:38 |
| saurabh_ | I am facing a problem with DHCP discover packet (vlan tagged), when it leaves the machine, it's vlan tag get removed by ethernet interface of the machine, I am using rhel 6, do I need to change some configuration, because no problem with Ubuntu | 11:38 |
| *** nunizacu has joined #openstack-neutron | 11:38 | |
| *** mwagner_lap has quit IRC | 11:40 | |
| *** baoli has joined #openstack-neutron | 11:41 | |
| *** dvorkinista has joined #openstack-neutron | 11:42 | |
| *** luqas has quit IRC | 11:43 | |
| jschwarz | ajo, I'm really getting the sense that the bug is somewhere in the code | 11:43 |
| jswiders | jschwarz: yeah it seems like this - about 2 executes - where do You see 2 ? I see one in utils.py | 11:44 |
| jschwarz | ajo, I'd suggest, instead of mocking "execute" - override it with a simple implementation (like "print what i'm executing + traceback") and send it to jenkins | 11:44 |
| jschwarz | ajo, the traceback will let you see where it's being called from and should narrow it down | 11:44 |
| jschwarz | jswiders, neither | 11:45 |
| jschwarz | jswiders, the object is IptablesManager | 11:45 |
| *** sijo_ has quit IRC | 11:45 | |
| jschwarz | jswiders, see neutron/agent/linux/iptables_manager.py L287 | 11:45 |
| jswiders | jschwarz: yes but under the hood it uses linux_utils.execute if none was suplied | 11:46 |
| jschwarz | hope I helped but really not much use without reproduction abilities :< | 11:46 |
| ajo | jschwarz, yes, same conclusion here, If it fails, more or less is what the debug patch I sent would provide: https://review.openstack.org/#/c/119018/2/neutron/tests/unit/test_security_groups_rpc.py | 11:46 |
| ajo | may be I understood you wrong | 11:46 |
| *** rkukura_ has joined #openstack-neutron | 11:46 | |
| jschwarz | ajo, I meant the same, only for execute | 11:46 |
| *** dvorkinista has quit IRC | 11:47 | |
| jschwarz | ajo, ie. in neutron/tests/unit/test_security_groups_rpc.py L2226, manually override execute to do some printage work | 11:47 |
| jschwarz | that should give you a better inside as to what's actually going on IMO | 11:47 |
| jschwarz | ajo, how was the snack? :D | 11:47 |
| ajo | jschwarz, still in the middle, almost finished | 11:49 |
| *** atmark1 has joined #openstack-neutron | 11:49 | |
| jschwarz | ajo, care to share a bite? :p | 11:49 |
| jschwarz | You should come visit. We have snacks here too xD | 11:50 |
| *** jroll|dupe has joined #openstack-neutron | 11:50 | |
| *** insequen1 has joined #openstack-neutron | 11:50 | |
| *** juice_ has joined #openstack-neutron | 11:50 | |
| *** rkukura has quit IRC | 11:50 | |
| *** greghaynes has quit IRC | 11:50 | |
| *** insequent has quit IRC | 11:50 | |
| *** jroll has quit IRC | 11:50 | |
| *** xianghui has quit IRC | 11:51 | |
| *** atmark has quit IRC | 11:51 | |
| *** juice has quit IRC | 11:51 | |
| *** juice_ is now known as juice | 11:51 | |
| *** jroll|dupe is now known as jroll | 11:51 | |
| *** greghayn1 has joined #openstack-neutron | 11:51 | |
| *** rkukura_ is now known as rkukura | 11:51 | |
| *** xianghui has joined #openstack-neutron | 11:51 | |
| *** sgordon_ has joined #openstack-neutron | 11:52 | |
| *** yamamoto has joined #openstack-neutron | 11:54 | |
| *** tongli has joined #openstack-neutron | 11:55 | |
| *** diegows has joined #openstack-neutron | 11:57 | |
| *** bjornar has joined #openstack-neutron | 11:58 | |
| *** yamamoto has quit IRC | 11:59 | |
| *** amaretskiy has quit IRC | 12:00 | |
| *** trinaths has left #openstack-neutron | 12:01 | |
| *** amaretskiy has joined #openstack-neutron | 12:01 | |
| jswiders | ajo: until we get more info there seems to be no way to know what is going on - but the change we found in logstash that also failed this test is related to security groups so even though its bad to have bug in code it might not cause totally unrelated patches to fail, and now I'm going to eat something :) | 12:01 |
| *** mwagner_lap has joined #openstack-neutron | 12:01 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add AsyncNsProcess https://review.openstack.org/118308 | 12:04 |
| *** jckasper has quit IRC | 12:04 | |
| ajo | jswiders, this is good news, should give us enough room to find the reason | 12:06 |
| *** prasoon has joined #openstack-neutron | 12:06 | |
| jschwarz | ajo, jswiders, and also food is always good news | 12:10 |
| jschwarz | ;-) | 12:10 |
| *** baoli has quit IRC | 12:11 | |
| *** baoli has joined #openstack-neutron | 12:12 | |
| *** Guest53651 is now known as beagles | 12:12 | |
| *** nunizacu has quit IRC | 12:14 | |
| *** nunizacu has joined #openstack-neutron | 12:14 | |
| *** pasquier-s has joined #openstack-neutron | 12:15 | |
| *** Zemeio has quit IRC | 12:15 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add AsyncNsProcess https://review.openstack.org/118308 | 12:17 |
| *** baoli has quit IRC | 12:19 | |
| amuller | marun_: safchain: https://bugs.launchpad.net/neutron/+bugs?field.tag=l3-ha | 12:22 |
| amuller | Known L3 HA bugs for now | 12:22 |
| *** yamamoto has joined #openstack-neutron | 12:22 | |
| safchain | amuller, k, thx | 12:23 |
| amuller | not including what Darragh pointed out, that every time a FIP is added the keepalived grows and grows | 12:24 |
| amuller | I think I'll fix that in the agent patch itself | 12:24 |
| *** dave_tucker is now known as dave_tucker_zzz | 12:26 | |
| *** rpothier has joined #openstack-neutron | 12:26 | |
| openstackgerrit | Miguel Angel Ajo proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 12:28 |
| ajo | jswiders, jschwarz retrying, it didn't fail | 12:28 |
| *** nati_ueno has joined #openstack-neutron | 12:28 | |
| *** nunizacu has quit IRC | 12:29 | |
| *** nati_ueno has quit IRC | 12:30 | |
| *** nati_ueno has joined #openstack-neutron | 12:31 | |
| *** mrsnivvel has joined #openstack-neutron | 12:31 | |
| *** nati_ueno has quit IRC | 12:32 | |
| *** xianghui has quit IRC | 12:32 | |
| *** dave_tucker_zzz is now known as dave_tucker | 12:36 | |
| openstackgerrit | A change was merged to openstack/neutron: Revert "Cisco DFA ML2 Mechanism Driver" https://review.openstack.org/118456 | 12:36 |
| *** rook_ is now known as jtaleric | 12:36 | |
| *** kickinz1|afk has joined #openstack-neutron | 12:37 | |
| *** john_mac has quit IRC | 12:37 | |
| *** jckasper has joined #openstack-neutron | 12:37 | |
| *** igordcard has joined #openstack-neutron | 12:37 | |
| *** dosaboy_ is now known as dosaboy | 12:38 | |
| *** nati_ueno has joined #openstack-neutron | 12:39 | |
| *** gildub has quit IRC | 12:39 | |
| *** saju_m has quit IRC | 12:40 | |
| *** john_mac has joined #openstack-neutron | 12:41 | |
| *** dvorkinista has joined #openstack-neutron | 12:43 | |
| jschwarz | ajo, :O it didn't | 12:43 |
| *** jschwarz has quit IRC | 12:43 | |
| amuller | ajo: I have a setup here where I couldn't ping a VM from the DHCP namespace | 12:43 |
| amuller | I then added a security group rule that allows all incoming ICMP | 12:43 |
| amuller | and then the ping worked | 12:43 |
| amuller | but, the DHCP port and the VM port are members of the same security group | 12:44 |
| amuller | so why didn't it allow the ping in the first place? | 12:44 |
| ajo | can you dump your security group and your iptables? | 12:44 |
| ajo | when no ICMP rule added | 12:44 |
| *** luqas has joined #openstack-neutron | 12:44 | |
| ajo | amuller, default security group should allow any ingress from its same group | 12:45 |
| amuller | that's how I understand it | 12:45 |
| *** kickinz1|afk is now known as kickinz1 | 12:45 | |
| amuller | this is on stable/icehouse btw | 12:45 |
| ajo | ah | 12:45 |
| *** xianghui has joined #openstack-neutron | 12:45 | |
| ajo | amuller, I was fearing we broke something else with the enhanced rpc | 12:45 |
| amuller | nono :) | 12:46 |
| amuller | John will come back in a sec I'll ask him to send you iptables and sg | 12:46 |
| *** dvorkinista has quit IRC | 12:47 | |
| *** baoli has joined #openstack-neutron | 12:47 | |
| *** flaviof_zzz is now known as flaviof | 12:47 | |
| ajo | thanks amuller | 12:48 |
| ajo | I will check it | 12:48 |
| *** jschwarz has joined #openstack-neutron | 12:49 | |
| *** nati_ueno has quit IRC | 12:50 | |
| *** kickinz1 has quit IRC | 12:51 | |
| *** kickinz1 has joined #openstack-neutron | 12:51 | |
| *** stanzgy has quit IRC | 12:53 | |
| *** baohua has joined #openstack-neutron | 12:54 | |
| *** kopparam has quit IRC | 12:55 | |
| openstackgerrit | shihanzhang proposed a change to openstack/neutron: Add ipset support for Security Groups https://review.openstack.org/111877 | 12:56 |
| *** baohua has quit IRC | 12:57 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Implement ModelsMigrationsSync test from oslo.db https://review.openstack.org/76520 | 12:57 |
| *** yamahata has joined #openstack-neutron | 12:58 | |
| jschwarz | ajo, got a sec? | 12:58 |
| *** JayJ_ has joined #openstack-neutron | 12:58 | |
| *** baohua has joined #openstack-neutron | 12:59 | |
| ajo | jschwarz, do you have the dump for me? :) | 12:59 |
| jschwarz | ajo, not yet | 12:59 |
| *** tsg has joined #openstack-neutron | 12:59 | |
| jschwarz | ajo, can you look at https://review.openstack.org/119001 ? | 12:59 |
| jschwarz | Specifically the part where I explicitly set the timeout in the test to 60 seconds | 13:00 |
| ajo | ah sure, let me check | 13:00 |
| *** foexle_ has joined #openstack-neutron | 13:00 | |
| jschwarz | I think the current code I have for review is buggy in that the timeout for the check exceed the interval | 13:01 |
| *** kbringard has joined #openstack-neutron | 13:01 | |
| jschwarz | I think you also ran into that working on your patch a few days ago... what do you think? | 13:02 |
| jschwarz | aji: ^ | 13:02 |
| *** radez_g0n3 is now known as radez | 13:02 | |
| jschwarz | ajo: ^ | 13:03 |
| jschwarz | marun_, ^ | 13:03 |
| *** foexle has quit IRC | 13:04 | |
| ajo | yes, but raising to 60 seconds should work | 13:05 |
| jschwarz | marun_ doesn't want to raise the timeout | 13:06 |
| jschwarz | sorry, the interval | 13:06 |
| *** marun_ has quit IRC | 13:07 | |
| *** banix has joined #openstack-neutron | 13:08 | |
| ajo | jschwarz, raising the test should be ok | 13:09 |
| ajo | but not the other | 13:09 |
| jschwarz | ajo, I'm really not that sure | 13:09 |
| jschwarz | ajo, I remember I changed the timeout to be exactly like the interval and got sporadic errors all over the place that were gone when i lowered the timeout | 13:10 |
| *** tsg has quit IRC | 13:10 | |
| HenryG | akamyshnikova: Hi. Any new info about the db migration tests? I see your latest patch, but I still get the same error. | 13:11 |
| *** neeti has quit IRC | 13:11 | |
| *** k4n0 has quit IRC | 13:12 | |
| *** mitarun has joined #openstack-neutron | 13:13 | |
| ajo | jschwarz, HenryG ^ jschwarz talk :) | 13:13 |
| akamyshnikova | HenryG, yes, I just decided to make it looks better. In fact rpodolyaka is running my test right now, so I hope will come up with something :) | 13:13 |
| jschwarz | wah? sorry? | 13:13 |
| ajo | I believe HenrryG commented me to avoid respawn_interval = timeout | 13:13 |
| *** baoli has quit IRC | 13:13 | |
| ajo | this is what you mean, right jschwarz ? | 13:13 |
| jschwarz | yes | 13:14 |
| jschwarz | here: https://review.openstack.org/#/c/114927/2/neutron/agent/linux/ovsdb_monitor.py | 13:14 |
| *** mestery has quit IRC | 13:17 | |
| HenryG | jschwarz: ajo: Well, the real issue was the timeout was not getting cleared. With that fixed, I don't think the respawn interval is an issue? | 13:17 |
| jschwarz | might be, but i remember running into problems when I set timeout = interval | 13:18 |
| jschwarz | I need to check this again on my setup so if I can't reverify this I'll let this go :) | 13:19 |
| *** baoli has joined #openstack-neutron | 13:19 | |
| HenryG | akamyshnikova: After running the tests, there are a number of databases with random names still undeleted. | 13:19 |
| HenryG | jschwarz: It might be better to check the timeout before trying to respawn? | 13:20 |
| *** mestery has joined #openstack-neutron | 13:21 | |
| *** mestery has quit IRC | 13:21 | |
| *** mestery has joined #openstack-neutron | 13:22 | |
| *** bvandenh has quit IRC | 13:23 | |
| *** bvandenh has joined #openstack-neutron | 13:23 | |
| jschwarz | HenryG, might be | 13:23 |
| jschwarz | HenryG, I think it's doing it now though, since the spawn is inside the 'with Timeout()' block | 13:24 |
| *** marun_ has joined #openstack-neutron | 13:25 | |
| openstackgerrit | Kevin Bringard proposed a change to openstack/neutron: Block external access directly to qg devices in logical routers. https://review.openstack.org/119061 | 13:27 |
| *** julim has joined #openstack-neutron | 13:27 | |
| jschwarz | HenryG, ajo, marun_, false alarm, it times out well | 13:28 |
| *** lazy_prince is now known as killer_prince | 13:28 | |
| *** baoli has quit IRC | 13:28 | |
| jschwarz | produces an error such as "AssertionError: Execution of this test timed out", so I'm satisfied with the current code | 13:28 |
| *** baoli has joined #openstack-neutron | 13:28 | |
| openstackgerrit | Rossella Sblendido proposed a change to openstack/neutron: Update DVR Binding when router_id changes https://review.openstack.org/117285 | 13:28 |
| *** ramishra has joined #openstack-neutron | 13:29 | |
| *** skolekonov has quit IRC | 13:34 | |
| *** ramishra has quit IRC | 13:34 | |
| *** ramishra has joined #openstack-neutron | 13:34 | |
| *** JayJ_ has quit IRC | 13:35 | |
| *** amotoki_ has joined #openstack-neutron | 13:35 | |
| jschwarz | marun_, can you +2 https://review.openstack.org/#/c/119034/ so it's not waiting for more approvals when jenkins +1s it? | 13:37 |
| marun_ | jschwarz: only cores can +2 and +A, so whoever is around to approve will +2 as well | 13:38 |
| *** thomasem has joined #openstack-neutron | 13:38 | |
| marun_ | jschwarz: as per salv-orlando, we can't approve until +1 because that would bypass the checks | 13:38 |
| jschwarz | marun_, only +2. That way when it does get +1, others will see it already has 2 +2s and have no problem approving ;-) | 13:39 |
| marun_ | jschwarz: it's a one-liner for skip removal, any core is going to be able to approve | 13:40 |
| marun_ | jschwarz: also, any good core will not rubber stamp | 13:41 |
| marun_ | jschwarz: they need to assure themselves they are approving something sane | 13:41 |
| *** ramishra has quit IRC | 13:42 | |
| jschwarz | marun_, ack | 13:43 |
| *** ramishra has joined #openstack-neutron | 13:43 | |
| *** dvorkinista has joined #openstack-neutron | 13:43 | |
| amotoki_ | marun_: re: the review you are discussing, what is the background of "revert"? You disabled L3 func tests yesterday and now back. I don't fully understand the situation. | 13:44 |
| *** JayJ_ has joined #openstack-neutron | 13:44 | |
| marun_ | amotoki_: we submitted a disable days ago, it only just merged yesterday because of the gate logjam | 13:44 |
| marun_ | amotoki_: this is the fix: https://review.openstack.org/#/c/119001/ | 13:44 |
| *** insequen1 is now known as insequent | 13:44 | |
| marun_ | amotoki_: once that merges the skip can merge | 13:45 |
| marun_ | sorry, skip removal | 13:45 |
| marun_ | amotoki_: it wasn't actually the l3 agent test that was at fault | 13:45 |
| marun_ | amotoki_: the latent eventlet timeout just happened to trigger during it because it's a time-consuming test | 13:45 |
| *** tsg has joined #openstack-neutron | 13:46 | |
| jschwarz | happens mostly during high-load periods of time | 13:46 |
| amotoki_ | marun_: i see. there are many thing going and i wan't aware of it. | 13:46 |
| jschwarz | I'll write the ML soonish explaining how to reproduce | 13:46 |
| ajo | jswiders, jschwarz, after inspecting the code, I found that the agent.security_groups_rule_updated won't generate system calls if defer_refresh_firewall = True during init_firewall... | 13:46 |
| jschwarz | ajo, :O | 13:47 |
| ajo | if I force it to True, when False is expected during testing, I get the same exact behaviour | 13:47 |
| ajo | so | 13:47 |
| ajo | it's like... if during the False testing, it was set to True somehow | 13:47 |
| jschwarz | awesome, well done | 13:47 |
| ajo | reanalyzing from the same perspective now | 13:47 |
| marun_ | ajo: so it was interacting with the system? | 13:47 |
| ajo | it's only an step, I need to find why does that happen | 13:47 |
| ajo | marun_, not exactly | 13:47 |
| ajo | interaction is cut before the system | 13:47 |
| ajo | and they analize the iptables rules that are sent to save/restore with regular expressions | 13:48 |
| jswiders | ajo: I've assumed that nothing else can change it to True after we set it to False - but if there is a case where it happens it would explain it | 13:48 |
| ajo | well, I agree it's not the best way to do it | 13:48 |
| *** dvorkinista has quit IRC | 13:48 | |
| *** alexpilotti has joined #openstack-neutron | 13:48 | |
| ajo | that'd need a refactor to avoid mocking at that level and do it one above (jlibosva agrees per coments in the code) | 13:48 |
| *** crc32 has joined #openstack-neutron | 13:49 | |
| jlibosva | ajo: where? I don't have a context | 13:50 |
| ajo | jlibosva, : of mocking iptables execute and mock out | 13:50 |
| ajo | # firewall instead | 13:50 |
| ajo | 13:50 | |
| ajo | # TODO(jlibosva) Get rid of mocking iptables execute and mock out | 13:50 |
| ajo | # firewall instead | 13:50 |
| ajo | neutron/tests/unit/test_security_groups_rpc.py | 13:50 |
| *** sbfox has joined #openstack-neutron | 13:50 | |
| ajo | jlibosva, I agree with that TODO :) | 13:51 |
| ajo | but now I'm looking at something else | 13:51 |
| ajo | jswiders, jschwarz : http://ur1.ca/i4crr local reproduction, by forcing the defer_refresh_firewall=True always | 13:51 |
| jlibosva | ajo: ah, yeah. :) Thanks, seems like the guy who made the comments knows what he's talking about | 13:52 |
| jlibosva | s/comments/comment/ | 13:52 |
| ajo | :D | 13:52 |
| akamyshnikova | HenryG, strange I don't see this, but I don't think that it can cause this problem. Seems that problems is with connections. It is not simple thing to investigate, but I'm in process of debugging it... | 13:52 |
| *** r-daneel has joined #openstack-neutron | 13:53 | |
| *** sballe has joined #openstack-neutron | 13:55 | |
| *** cnesa10 has quit IRC | 13:55 | |
| marun_ | ajo | 13:56 |
| ajo | pong | 13:56 |
| marun_ | ajo: verify_mocked_calls -> antipattern | 13:56 |
| marun_ | That to say I would agree with your and jlibosva's suggestion that this is not the place to test iptables application | 13:57 |
| ajo | Yeah, I'm realizing it's not a good pattern, comes with lots of maintenance | 13:57 |
| marun_ | braindead :( | 13:57 |
| HenryG | akamyshnikova: I see it only for PG. I run psql and do "\list". You don't have any random databases left over? | 13:57 |
| jschwarz | long day :( | 13:57 |
| marun_ | jschwarz: I was talking about the test strategy, but yes, it has been a long day. :) | 13:57 |
| jschwarz | :) | 13:58 |
| marun_ | everyone is so used to seeing these terrible patterns in the code that they've become acceptable and merge without comment | 13:59 |
| marun_ | I'm hoping that incidents like this can contribute to change in a more useful direction | 13:59 |
| *** xgerman has joined #openstack-neutron | 13:59 | |
| marun_ | salv-orlando, obondarev: ^^ | 13:59 |
| ajo | marun_, I think of that every time I have to maintain mock call checking everywhere after a change | 14:00 |
| marun_ | (picking on you since you technically merged it, but we're all in this boat together) | 14:00 |
| ajo | just because you added an extra parameter, for example | 14:00 |
| *** radez is now known as radez_g0n3 | 14:00 | |
| marun_ | ajo: right | 14:00 |
| *** dougwig_ is now known as dougwig | 14:00 | |
| jschwarz | marun_, I might get some time doing some rewriting during next semester | 14:00 |
| *** killer_prince is now known as lazy_prince | 14:00 | |
| jschwarz | marun_, I agree that there are some parts in the code which can be hard to handle | 14:00 |
| marun_ | jschwarz: 'some' lol | 14:01 |
| *** jorgem has joined #openstack-neutron | 14:01 | |
| ajo | marun_, may be it could be a good exercise, to pick up bad examples of testing, and the corresponding better suggestions | 14:01 |
| marun_ | jschwarz: but if you're game there is certainly an unending amount of stuff to do. | 14:01 |
| jschwarz | marun_, there exists some good code I bleieve | 14:01 |
| jschwarz | bleieve is now a word. | 14:01 |
| ajo | may be turn that into a wiki page | 14:01 |
| marun_ | jschwarz: I'd love to see it. :) | 14:01 |
| marun_ | ajo: that's definitely worth doing | 14:01 |
| marun_ | maybe even worth a summit session | 14:01 |
| ajo | marun_, *yes* | 14:02 |
| jschwarz | marun_, I try writing my code fairly well so there's that ;-) | 14:02 |
| *** yamahata has quit IRC | 14:02 | |
| jschwarz | I mean, I dedicated quite a bit of time on those refactoring changes | 14:02 |
| *** yamahata has joined #openstack-neutron | 14:02 | |
| jschwarz | surely there are others who attempt to be as righteous as me ;-p | 14:03 |
| marun_ | jschwarz: no code stands alone, though, and if the foundation is rotten the building will stillfall | 14:03 |
| marun_ | no matter how nicely the kitchen is appointed | 14:03 |
| ajo | marun_, the next moment I feel times aren't very pressing, I'll try to put things together, and may be we can look at that if you're able to come to Madrid before the summit | 14:03 |
| jswiders | ajo: I think there might be a small hole in Your explanation of this error | 14:03 |
| akamyshnikova | HenryG, oh.. yes, thanks... I have a lot of database that were not dropped. In fact test fails on dropping table, so this is the thing | 14:03 |
| ajo | jswiders, I know, I was looking at it... nothing changes the defer_apply | 14:04 |
| marun_ | jschwarz: all that to say that we need to prioritize cleanup so that foundational stuff gets attention first. | 14:04 |
| marun_ | ajo++ | 14:04 |
| *** snowgust has joined #openstack-neutron | 14:04 | |
| marun_ | ajo: so, why not fix the tests and stop mocking the wrong thing? | 14:05 |
| jschwarz | mind you, I think that a large portion of the next cycle should be dedicated to cleaning up the code and working on the nova-network parity | 14:05 |
| *** rkukura_ has joined #openstack-neutron | 14:05 | |
| ajo | marun_, because it's huge and we want the FFE items done soon | 14:05 |
| jschwarz | but i'm new here so that's me | 14:05 |
| *** pasquier-s_ has joined #openstack-neutron | 14:05 | |
| HenryG | akamyshnikova: Thanks for confirming. Let me know if any problem is found with the oslo.db unit test fixtures. | 14:06 |
| ajo | marun_, but I could look at that afterwards, may be with jlibosva's help ;) | 14:06 |
| jswiders | ajo: (had a phone call so couldn't finish thought) another think that makes me think its something other - if You look at LOG.info output | 14:06 |
| *** yamamoto_ has joined #openstack-neutron | 14:07 | |
| ajo | hmmm jswiders let me look | 14:07 |
| *** garyduan has joined #openstack-neutron | 14:07 | |
| jswiders | ajo: there is log at security_groups_rule_updated, then in _security_group_updated defer is checked and then and only if it was False | 14:07 |
| ajo | hmmm | 14:07 |
| *** ramishra_ has joined #openstack-neutron | 14:07 | |
| jswiders | ajo: refresh_firewall is called which logs another message which is present in jenkins runs | 14:07 |
| ajo | jswiders, could you link me back to the console log? | 14:07 |
| jswiders | ajo: but not in Your local run | 14:08 |
| ajo | aha | 14:08 |
| ajo | good point | 14:08 |
| jswiders | ajo: so whatever goes wrong probably goes wrong after this point and not before | 14:08 |
| *** annegent_ has joined #openstack-neutron | 14:09 | |
| jschwarz | jswiders++ | 14:09 |
| *** oreillyd_ has joined #openstack-neutron | 14:09 | |
| jswiders | but still best if we had something more to work with :/ | 14:10 |
| jswiders | w/o it its really hard to do anything other than guess | 14:10 |
| *** markmcclain has joined #openstack-neutron | 14:10 | |
| marun_ | ajo: Rewriting the tests would have taken less time than the debugging exercise has consumed thus far. | 14:10 |
| marun_ | ajo: At some point it may be worth calling it and writing the test properly rather than continuing to pick at something that is broken by design. | 14:11 |
| ajo | marun_, correct, but I believe it's something broken in the code and not the UT actually | 14:11 |
| marun_ | ajo: which code? | 14:11 |
| ajo | the code invoked by the test itself | 14:11 |
| ajo | but, guessing yet | 14:12 |
| marun_ | ajo: So can you write a test that validates what you think is broken? | 14:12 |
| marun_ | ajo: not just debugging, but validate your assumptions | 14:12 |
| ajo | marun_, when I have a valid guess, I will write it, good point | 14:12 |
| marun_ | ajo: it's a guess - anything is valid :) | 14:12 |
| jswiders | marun_: supporting for ajo statement is that there is base class from which he derives and test the same but w/o enchanced rpc with the same method | 14:12 |
| ajo | jswiders, just destroyed my guess | 14:12 |
| ajo | :) | 14:12 |
| jswiders | marun_: and it doesn't fail | 14:13 |
| marun_ | jswiders: sure, and the testing is bs in either case | 14:13 |
| jswiders | marun_: yup I agree | 14:13 |
| marun_ | jswiders: we don't know 'it doesn't fail' | 14:13 |
| ajo | or it's just it didn't fail yet | 14:13 |
| * marun_ takes a valium | 14:13 | |
| ajo | or we didn't notice | 14:13 |
| ajo | ':D | 14:13 |
| jswiders | well that a posibility | 14:13 |
| jschwarz | marun_, morphine is better | 14:13 |
| *** lordd has quit IRC | 14:14 | |
| jswiders | jschwarz: talking out of personal experience :P ? | 14:14 |
| jschwarz | jswiders, yes sirry | 14:14 |
| *** lordd has joined #openstack-neutron | 14:14 | |
| ajo | marun_, I'm all for rewriting those tests into something easier to maintain, | 14:14 |
| ajo | but it's a complex piece of code, and probably that'd require 1-2 weeks we don't have atm | 14:14 |
| marun_ | ajo: easier to maintain is part of it. I was hoping for something that actually did a decent job of testing. But yeah, a hard slog. :/ | 14:15 |
| ajo | jlibosva, when do we start the refactor? | 14:15 |
| jschwarz | marun_, btw me and Assaf was talking earlier about how functional tests might cause problems to one another with more than one worker | 14:16 |
| jschwarz | marun_, ie 4 workers doing functional testings, creating routers and settings, etc, might cause false positive | 14:16 |
| jlibosva | ajo: when we're done with current tasks? :) | 14:16 |
| jschwarz | maybe want to consider in the future to start running functional tests on a single worker | 14:16 |
| *** yamahata has quit IRC | 14:17 | |
| *** ramishra has quit IRC | 14:17 | |
| *** yamamoto has quit IRC | 14:17 | |
| *** pasquier-s has quit IRC | 14:17 | |
| *** diegows has quit IRC | 14:17 | |
| *** atmark1 has quit IRC | 14:17 | |
| *** rkukura has quit IRC | 14:17 | |
| *** oreillyd has quit IRC | 14:17 | |
| *** cipcosma has quit IRC | 14:17 | |
| *** dkehnx1 has quit IRC | 14:17 | |
| *** ekarlso- has quit IRC | 14:17 | |
| *** gduan has quit IRC | 14:17 | |
| *** harlowja_away has quit IRC | 14:17 | |
| *** rkukura_ is now known as rkukura | 14:17 | |
| marun_ | jschwarz: I don't agree with that, actually | 14:17 |
| ajo | jlibosva, lest talk about it one week from now :) | 14:17 |
| jlibosva | ajo: ack | 14:17 |
| marun_ | jschwarz: I think we need to ensure test isolation | 14:17 |
| marun_ | jschwarz: regardless of the number of workers | 14:17 |
| ajo | I agree marun_ on that | 14:17 |
| *** cipcosma has joined #openstack-neutron | 14:17 | |
| marun_ | jschwarz: that may mean modifying the code under test to support the isolation we need. | 14:17 |
| ajo | that ensures better test design, and also saves testing time | 14:17 |
| marun_ | jschwarz: certainly none of the code as it exist today was designed to be functionally testable | 14:18 |
| jschwarz | marun_, might, but otherwise we're in a collision course in the current state | 14:18 |
| marun_ | jschwarz: disagree | 14:18 |
| marun_ | jschwarz: fixing problems at the root is the best path forward | 14:18 |
| marun_ | jschwarz: ducking around them just ensures more problems that are harder to diagnose | 14:18 |
| *** lordd has quit IRC | 14:19 | |
| jschwarz | marun_, I meant, if we do nothing now and just keep on writing functional tests (as we should) will eventually cause problems IMO | 14:19 |
| marun_ | jschwarz: We will run into problems | 14:19 |
| marun_ | jschwarz: and we will fix those problems | 14:19 |
| jschwarz | marun_, look at what a single unclosed timeout caused us ;-) | 14:19 |
| marun_ | jschwarz: learning is hard | 14:19 |
| marun_ | jschwarz: but still worth it | 14:19 |
| jschwarz | agreed | 14:20 |
| marun_ | we're going to have teething pains, no question | 14:20 |
| jschwarz | marun_, what kind of test isolation is there for functional testing? | 14:20 |
| jschwarz | for unit tests for example, the SQLDB is wiped between each test | 14:20 |
| jschwarz | anything similar for functional? | 14:20 |
| marun_ | jschwarz: we already do a pretty good job of avoiding fs collisions with tempdir | 14:20 |
| *** yamahata has joined #openstack-neutron | 14:20 | |
| marun_ | jschwarz: we can isolate things like netfilter interaction with netns | 14:21 |
| *** diegows has joined #openstack-neutron | 14:21 | |
| marun_ | jschwarz: devices and ports are a bit harder, but nonetheless achievable | 14:21 |
| *** otherwiseguy has joined #openstack-neutron | 14:21 | |
| ajo | jswiders, INFO [neutron.agent.securitygroups_rpc] Refresh firewall rules not in my failure induction... yikes! | 14:21 |
| marun_ | ajo++ | 14:21 |
| ajo | marun_, what for? :) | 14:21 |
| marun_ | ajo: is that a direct log message? | 14:21 |
| *** dkehnx1 has joined #openstack-neutron | 14:21 | |
| jschwarz | marun_, better start thinking about it :) | 14:22 |
| marun_ | test isolation :) | 14:22 |
| *** atmark1 has joined #openstack-neutron | 14:22 | |
| ajo | marun_, it's a log message from the tested module | 14:22 |
| marun_ | jschwarz: we'll get to it | 14:22 |
| marun_ | jschwarz: one step at a time | 14:22 |
| jschwarz | marun_, :) | 14:22 |
| *** ekarlso- has joined #openstack-neutron | 14:23 | |
| *** Zemeio has joined #openstack-neutron | 14:23 | |
| jschwarz | marun_, writing the ML now... want to preview it before I send it? | 14:23 |
| ajo | marun_, jlibosva and I set a re-check in a week to see if we can start tackling the change to that testing | 14:23 |
| marun_ | jschwarz: if you want me to edit, feel free. You don't need my vetting, though. | 14:23 |
| jschwarz | marun_, would appreciate vetting :) | 14:24 |
| ajo | ahaaa jswiders , but the refresh_firewall has another theck to "use_enhanced_rpc" talking to one RPC or the other | 14:26 |
| ajo | theck->check | 14:26 |
| marun_ | amotoki_: we were waiting on jenkins for https://review.openstack.org/#/c/119001/ | 14:27 |
| jswiders | ajo: yes - if fore some reason the code under that check would return empty devices then it would explain | 14:27 |
| ajo | marun_, is LOG.debug shown when a unit test fails? | 14:27 |
| marun_ | amotoki_: salv-orlando mentioned that approving pre-check could cause problems, though I'm not sure why | 14:27 |
| jswiders | ajo: but I don't see why it would happen | 14:27 |
| marun_ | ajo: you have to enable debug logging | 14:27 |
| ajo | hmmm | 14:27 |
| ajo | even for UT? | 14:27 |
| *** jistr has quit IRC | 14:27 | |
| jswiders | marun_: why is this not on by default ? | 14:27 |
| marun_ | https://github.com/openstack/neutron/blob/master/neutron/tests/base.py#L79 | 14:28 |
| ajo | too bad, it would have helped debugging this | 14:28 |
| marun_ | jswiders: because the log volume is insane with debug on | 14:28 |
| jswiders | marun_: I think it would be really helpful overall in tackling test failures | 14:28 |
| jswiders | marun_: mhm | 14:28 |
| ajo | but may be increases testing time | 14:28 |
| *** ajo is now known as ajo|call | 14:28 | |
| ajo|call | I have to jump into a call, | 14:28 |
| ajo|call | I'll go back to this later | 14:28 |
| marun_ | jswiders: we can always submit a patch with debug enabled and then look for the failures | 14:28 |
| ajo|call | I will change those into .info for my try-to-reproduce patch | 14:28 |
| marun_ | ajo|call: ^ you may want to do just this | 14:29 |
| ajo|call | that's an option too | 14:29 |
| ajo|call | probably better | 14:29 |
| *** jistr has joined #openstack-neutron | 14:29 | |
| ajo|call | jswiders, if you have time, feel free to resubmit the patch with debug=True | 14:29 |
| jschwarz | marun_, btw I think that the patch won't go through to the gate until jenkins finishes the +1... wonder why salv didn't want this | 14:29 |
| marun_ | jschwarz: there is some detail we are missing | 14:30 |
| *** xgerman has quit IRC | 14:30 | |
| jswiders | jschwarz: iirc salv said that infra disabled need for +1 temporarily because of queues | 14:30 |
| marun_ | jschwarz: I think (hazy) that it might be that it doesn't wait and a gate failure can cause a gate pipeline reset | 14:30 |
| amotoki_ | marun_: thanks. In my understanding, 119001 is ready to go if Jenkins succeeds and we don't need to wait for a result of depedning patch. | 14:30 |
| *** AbsinthMind has joined #openstack-neutron | 14:31 | |
| *** xgerman has joined #openstack-neutron | 14:31 | |
| amotoki_ | marun_: +A means it can go to the gate if Jenkins check job succeeds. | 14:31 |
| marun_ | amotoki_: but waiting for jenkins +1 is currently disabled, so if the patch fails even for non-related reason maybe a gate reset will happen? | 14:31 |
| *** xgerman has quit IRC | 14:32 | |
| *** mitarun has quit IRC | 14:32 | |
| marun_ | amotoki_: my understanding is that right now (salv-orlando suggested I think) A+ will bypass check | 14:32 |
| amotoki_ | marun_: so, I revert my +A to clarify the situation. | 14:32 |
| marun_ | (if it hasn't returned already) | 14:32 |
| *** xgerman has joined #openstack-neutron | 14:32 | |
| jschwarz | The queue is long and full of terrors | 14:32 |
| *** xgerman has quit IRC | 14:32 | |
| *** mitarun has joined #openstack-neutron | 14:32 | |
| jschwarz | I'll check back tomorrow morning and make sure they get +2 | 14:32 |
| amotoki_ | Does +A bypass check? | 14:32 |
| *** xgerman has joined #openstack-neutron | 14:33 | |
| *** dkehnx1 is now known as 6A4AAKD1O | 14:33 | |
| *** rkukura is now known as 6A4AAKDJG | 14:33 | |
| *** 17SAA4MFR has joined #openstack-neutron | 14:33 | |
| *** ramishra has joined #openstack-neutron | 14:33 | |
| *** yamamoto has joined #openstack-neutron | 14:33 | |
| *** pasquier-s has joined #openstack-neutron | 14:33 | |
| *** 17SAA4J9Y has joined #openstack-neutron | 14:33 | |
| *** 17SAA4J1W has joined #openstack-neutron | 14:33 | |
| *** rkukura has joined #openstack-neutron | 14:33 | |
| *** oreillyd has joined #openstack-neutron | 14:33 | |
| *** 17SAA4HWG has joined #openstack-neutron | 14:33 | |
| *** dkehnx1 has joined #openstack-neutron | 14:33 | |
| *** 17SAA4BEV has joined #openstack-neutron | 14:33 | |
| *** gduan has joined #openstack-neutron | 14:33 | |
| *** 17SAA4BEV has quit IRC | 14:33 | |
| *** 17SAA4J9Y has quit IRC | 14:33 | |
| *** 17SAA4MFR has quit IRC | 14:33 | |
| *** ramishra has quit IRC | 14:33 | |
| *** yamamoto has quit IRC | 14:33 | |
| *** pasquier-s has quit IRC | 14:33 | |
| *** 17SAA4J1W has quit IRC | 14:33 | |
| *** rkukura has quit IRC | 14:33 | |
| amotoki_ | perhaps you and salv-orlando are correct :-) | 14:33 |
| *** oreillyd has quit IRC | 14:33 | |
| *** 17SAA4HWG has quit IRC | 14:33 | |
| *** dkehnx1 has quit IRC | 14:33 | |
| *** gduan has quit IRC | 14:34 | |
| jswiders | marun_: so what exactly do we need to do to enable debug logging ? cause I'm not sure how to do that in remote check job | 14:34 |
| *** jaypipes has joined #openstack-neutron | 14:34 | |
| marun_ | jswiders: https://github.com/openstack/neutron/blob/master/neutron/tests/base.py#L79 change this code to always enable debug logging regardless of the env state | 14:34 |
| jswiders | marun_: oh as simple as this :P I thought about some way to setting os.environ :D | 14:35 |
| marun_ | jswiders: no such luck | 14:35 |
| *** 6A4AAKD1O is now known as dkehnx1 | 14:35 | |
| *** mitarun_ has joined #openstack-neutron | 14:35 | |
| *** signed8bit has joined #openstack-neutron | 14:36 | |
| *** mitarun has quit IRC | 14:37 | |
| amotoki_ | marun_: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n39 means the gate job requires both +1 from jenkins and +A. but I would say my +A is NOT a good manner (I thought Jenkins succeed...) | 14:37 |
| amotoki_ | marun_: thanks for noticing me. | 14:37 |
| *** sbfox has quit IRC | 14:37 | |
| *** annegent_ has quit IRC | 14:37 | |
| marun_ | amotoki_: jschwarz is the one to thank :) | 14:38 |
| jschwarz | did someone call? | 14:38 |
| jschwarz | :) | 14:38 |
| marun_ | amotoki_: I haven't looked a the infra code before | 14:38 |
| *** annegentle has joined #openstack-neutron | 14:38 | |
| marun_ | amotoki_: what does the config with a check requirement look like? | 14:38 |
| jschwarz | http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n39 | 14:38 |
| jschwarz | oops :< | 14:38 |
| *** jistr has quit IRC | 14:39 | |
| openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 14:39 |
| *** jistr has joined #openstack-neutron | 14:39 | |
| *** jistr has quit IRC | 14:39 | |
| *** jistr has joined #openstack-neutron | 14:40 | |
| *** radez_g0n3 is now known as radez | 14:40 | |
| *** mitarun_ has quit IRC | 14:40 | |
| *** lordd has joined #openstack-neutron | 14:41 | |
| *** JayJ_ has quit IRC | 14:41 | |
| *** AbsinthMind1 has joined #openstack-neutron | 14:42 | |
| *** watrous has joined #openstack-neutron | 14:42 | |
| *** AbsinthMind1 has quit IRC | 14:43 | |
| *** SridharG has left #openstack-neutron | 14:44 | |
| *** dvorkinista has joined #openstack-neutron | 14:44 | |
| *** mitarun has joined #openstack-neutron | 14:45 | |
| *** nlahouti has joined #openstack-neutron | 14:45 | |
| *** AbsinthMind has quit IRC | 14:45 | |
| *** JayJ_ has joined #openstack-neutron | 14:46 | |
| *** sridhar has quit IRC | 14:47 | |
| *** kopparam has joined #openstack-neutron | 14:48 | |
| *** nlahouti has quit IRC | 14:49 | |
| *** dvorkinista has quit IRC | 14:49 | |
| *** igordcard has quit IRC | 14:50 | |
| *** carl_baldwin has joined #openstack-neutron | 14:50 | |
| *** nlahouti has joined #openstack-neutron | 14:51 | |
| *** kopparam has quit IRC | 14:51 | |
| *** Dafna has quit IRC | 14:52 | |
| *** kopparam has joined #openstack-neutron | 14:54 | |
| *** clev has joined #openstack-neutron | 14:56 | |
| *** tsg has quit IRC | 14:56 | |
| *** clev has left #openstack-neutron | 14:56 | |
| *** baohua has quit IRC | 14:56 | |
| openstackgerrit | John Schwarz proposed a change to openstack/neutron: Revert "Skip functional l3 agent test" https://review.openstack.org/119034 | 14:59 |
| *** JayJ_ has quit IRC | 14:59 | |
| jschwarz | yamamoto_, thanks for bringing this to my attention | 15:00 |
| jschwarz | was too focused on what happened here today that I didn't pay attention to the commit message | 15:00 |
| *** skolekonov has joined #openstack-neutron | 15:01 | |
| *** baoli has quit IRC | 15:02 | |
| *** nati_ueno has joined #openstack-neutron | 15:02 | |
| *** seizadi has joined #openstack-neutron | 15:02 | |
| yamamoto_ | jschwarz: np. thank you for fixing the problem. | 15:02 |
| jschwarz | :) | 15:03 |
| *** baoli has joined #openstack-neutron | 15:03 | |
| *** nati_ueno has quit IRC | 15:03 | |
| *** jorgem has quit IRC | 15:03 | |
| *** JayJ_ has joined #openstack-neutron | 15:04 | |
| *** jobewan has joined #openstack-neutron | 15:06 | |
| *** JayJ_ has quit IRC | 15:08 | |
| openstackgerrit | Jacek Swiderski proposed a change to openstack/neutron: DO NOT MERGE, debug for unit tests (ipset patch) https://review.openstack.org/119018 | 15:09 |
| *** zsl6658 has joined #openstack-neutron | 15:10 | |
| *** pballand has joined #openstack-neutron | 15:12 | |
| salv-orlando | marun_, amotoki_ : the reason is just that a patch will go directly through the gate queue. This means that if we missed some stupid thing like a pep8 error, we will end up causing a gate reset | 15:12 |
| *** pasquier-s__ has joined #openstack-neutron | 15:12 | |
| *** zsl6658 has quit IRC | 15:13 | |
| *** rpothier has quit IRC | 15:13 | |
| jswiders | salv-orlando: as a (relatively) newcomer - what is gate reset ? | 15:13 |
| *** amotoki has quit IRC | 15:14 | |
| *** amotoki_ is now known as amotoki | 15:14 | |
| salv-orlando | jswiders: it’s the end of the world ;) | 15:14 |
| salv-orlando | jswiders: not really. | 15:14 |
| jswiders | salv-orlando: hehe | 15:14 |
| salv-orlando | patches into the queue are stacked. This means that if patch A is top of the queue and patch B #2, the tests for patch B include also the changes for patch A | 15:15 |
| salv-orlando | and so on. | 15:15 |
| salv-orlando | so if patch A fails, all the tests which are in progress on patch B and subsequent will be reset | 15:15 |
| jswiders | salv-orlando: mhm | 15:16 |
| *** pasquier-s_ has quit IRC | 15:16 | |
| amotoki | jswiders: have you see gate queue in zuul status page http://status.openstack.org/zuul/ ? | 15:16 |
| jswiders | amotoki: yes | 15:16 |
| *** mitarun has quit IRC | 15:16 | |
| jswiders | amotoki: but didn't know until now why is there 1 day queue | 15:16 |
| salv-orlando | and resetting tests in progress wastes a lot of time. | 15:16 |
| salv-orlando | because there are several jobs with take 45~60 minutes | 15:16 |
| *** Longgeek has joined #openstack-neutron | 15:17 | |
| jschwarz | we'll manage with this for another day, since the test is disabled anyway | 15:17 |
| *** mitarun has joined #openstack-neutron | 15:17 | |
| jswiders | salv-orlando: so looking that there is horizon change now at the top which failed py26/7 - all 100 changes will reset and start over now ? | 15:17 |
| jschwarz | salv-orlando, amotoki, I'll ping tomorrow after the +1s are given to ask for +A | 15:17 |
| jswiders | salv-orlando: (in horizon) | 15:18 |
| *** dvorkinista has joined #openstack-neutron | 15:18 | |
| salv-orlando | I think zuul is smart enough to not cause resets on single-project jobs like that. But when that happens in integrated gate tests, like gate-tempest-dsvm-neutron-full | 15:19 |
| salv-orlando | that is a big deal. see the failure in the gate queue for openstack/nova 109755,12 | 15:19 |
| *** Longgeek_ has quit IRC | 15:20 | |
| amotoki | salv-orlando: one question on your comment "a patch will go directly through the gate queue". in my understanding all patches into the gate queue passes the check job. am i missing something? | 15:20 |
| jswiders | salv-orlando: mhm | 15:20 |
| jswiders | salv-orlando: thanks for explanation :) | 15:20 |
| *** mitarun has quit IRC | 15:21 | |
| salv-orlando | amotoki: it was this way, but the infra team then relaxe this constraint because the test node pool was starving | 15:21 |
| salv-orlando | amotoki: idk if they restored this constraint for the juno-3 crunch | 15:21 |
| amotoki | salv-orlando: I see. very useful information. | 15:22 |
| amotoki | salv-orlando: i just see zuul layout.yaml and it seems "gate" requires jenkins +1. | 15:22 |
| *** dvorkinista has quit IRC | 15:23 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Replace try-except with condition https://review.openstack.org/119089 | 15:23 |
| salv-orlando | amotoki: it could be they reverted it… that can be checked in two ways - asking in #openstack-infra or approving a patch before check test complete! | 15:23 |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Replace try-except with condition in AsyncProcess https://review.openstack.org/119089 | 15:23 |
| chuckC | salv-orlando: nlahouti ran a reverify and saw it go through the check queue first | 15:24 |
| *** popw has quit IRC | 15:24 | |
| *** skolekonov has quit IRC | 15:24 | |
| amotoki | salv-orlando: thanks. I should be more conservative :-) | 15:24 |
| *** jswiders has quit IRC | 15:24 | |
| *** kickinz1 is now known as kickinz1|afk | 15:25 | |
| *** ramishra_ has quit IRC | 15:26 | |
| salv-orlando | chuckC: it could be. and it could also be that reverify action has a different workflow than +A. As I said, one should probably ask infra first. Besides, is this discussion because we have some super urgent patch we don’t want to wait to pass check jobs first? | 15:26 |
| *** yeungp has joined #openstack-neutron | 15:26 | |
| jschwarz | salv-orlando, everything is urgent ;-) | 15:26 |
| chuckC | salv-orlando: not on my side, just offering a piece of data | 15:27 |
| * salv-orlando thinks there are actually some employers paying developers on a per-commit basis | 15:27 | |
| salv-orlando | that would be an interesting way of encouraging active contributions | 15:27 |
| jschwarz | salv-orlando, I should be happy then - my first patch went through today so money's due ;-D | 15:27 |
| * chuckC is in a lot of trouble ;) | 15:28 | |
| salv-orlando | like fix spelling here, add a comment there, sort this list in alphabetical order | 15:28 |
| -amotoki- interesting thought :) | 15:28 | |
| *** crose has joined #openstack-neutron | 15:32 | |
| *** roeyc has joined #openstack-neutron | 15:33 | |
| *** networkstatic has joined #openstack-neutron | 15:35 | |
| *** a_le has joined #openstack-neutron | 15:35 | |
| *** mlavalle has joined #openstack-neutron | 15:36 | |
| *** evgenyf has quit IRC | 15:36 | |
| *** nunizacu has joined #openstack-neutron | 15:36 | |
| *** nunizacu has quit IRC | 15:41 | |
| *** jistr has quit IRC | 15:41 | |
| *** tsg has joined #openstack-neutron | 15:44 | |
| mgagne | marun_: ping | 15:46 |
| marun_ | mgagne: pong | 15:46 |
| mgagne | marun_: I have a problem/question regarding the ml2 migration script | 15:47 |
| *** dvorkinista has joined #openstack-neutron | 15:47 | |
| mgagne | marun_: migration script runs fine. But when I try to migrate/resize an instance, I get a port binding failed error. I narrowed down the issue to the vnic_type field in ml2_port_bindings not being properly populated after the migration | 15:48 |
| marun_ | mgagne: hmmm | 15:49 |
| mgagne | marun_: when creating ML2 tables in the migration script, I see that the default value should be set to "normal" as per server_default='normal'. But SHOW CREATE TABLE (on MySQL) shows that the default value is not properly set. | 15:49 |
| *** pckizer_ is now known as pckizer | 15:50 | |
| mgagne | marun_: the question: should the migration script populate the field or should it rely on the default value of the column? If the later, it's not working as planned. | 15:50 |
| marun_ | mgagne: looking.. | 15:50 |
| *** dfarrell07 has joined #openstack-neutron | 15:50 | |
| *** _cjones_ has joined #openstack-neutron | 15:51 | |
| *** doude has quit IRC | 15:51 | |
| marun_ | mgagne: can you dump the table definition? | 15:51 |
| mgagne | marun_: the values are actually an empty string, not NULL or 'normal'. | 15:51 |
| marun_ | mgagne: I'd like to determine whether the table was created correctly or not | 15:51 |
| mgagne | marun_: http://paste.openstack.org/show/105856/ | 15:51 |
| marun_ | mgagne: or did it exist pre-migration? | 15:51 |
| *** dvorkinista has quit IRC | 15:52 | |
| mgagne | marun_: it did not exist before the migration | 15:52 |
| mgagne | marun_: I'm migrating from linuxbridge to ml2 | 15:52 |
| *** packet has joined #openstack-neutron | 15:53 | |
| *** markmcclain has quit IRC | 15:53 | |
| marun_ | mgagne: I'm guessing there should be a sqlalchemy default set too. | 15:53 |
| mgagne | marun_: default='normal' ? | 15:54 |
| marun_ | mgagne: i.e. default='normal' | 15:54 |
| marun_ | mgagne: yes | 15:54 |
| marun_ | mgagne: the code I'm looking at always does both default and server_default | 15:54 |
| mgagne | marun_: I was wondering about the difference between the 2 statements :-/ | 15:54 |
| marun_ | mgagne: I'm pretty sure default sets it in code that sqlalchemy sees | 15:54 |
| marun_ | mgagne: server_default would presumably set the ddl (but is apparently failing to do so) | 15:54 |
| mgagne | marun_: http://docs.sqlalchemy.org/en/rel_0_8/core/defaults.html#server-side-defaults | 15:55 |
| mgagne | marun_: which looks to not have the expected result in my case... | 15:55 |
| marun_ | mgagne: grrr.... | 15:55 |
| marun_ | mgagne: is that the version of sqla you are using? | 15:56 |
| mgagne | 0.8.2-1~cloud1 from ubuntu cloud archive | 15:56 |
| mgagne | marun_: I can try to manually patch the migration script with default='normal' and rerun the migration to see if default value is set correctly | 15:58 |
| *** jlibosva has quit IRC | 16:01 | |
| *** garyk has quit IRC | 16:01 | |
| *** seizadi has quit IRC | 16:02 | |
| *** simon-AS5591 has joined #openstack-neutron | 16:02 | |
| *** jlibosva has joined #openstack-neutron | 16:03 | |
| *** jschwarz has quit IRC | 16:03 | |
| marun_ | mgagne: better would be to enable sqla debugging so that you can see the table creation statements that sqla emits | 16:03 |
| *** mitarun has joined #openstack-neutron | 16:03 | |
| mgagne | marun_: alright, will take some time to test, will be back later | 16:03 |
| marun_ | mgagne: appreciated | 16:04 |
| marun_ | mgagne: I'm in utc+2 right now so won't be around much longer. | 16:04 |
| mgagne | marun_: np | 16:06 |
| *** yamahata has quit IRC | 16:06 | |
| *** simon-AS559 has quit IRC | 16:06 | |
| marun_ | mgagne: as per zzzeek's suggestion, add checkFirst to make sure the tables are created as expected | 16:06 |
| mgagne | marun_: thanks | 16:06 |
| *** marun_ is now known as marun_afk | 16:06 | |
| *** sbfox has joined #openstack-neutron | 16:07 | |
| *** sbfox has quit IRC | 16:08 | |
| *** roeyc has quit IRC | 16:09 | |
| *** Snivilis has joined #openstack-neutron | 16:11 | |
| *** jschwarz has joined #openstack-neutron | 16:12 | |
| *** sballe has quit IRC | 16:12 | |
| *** sbalukoff has quit IRC | 16:12 | |
| *** afazekas has quit IRC | 16:13 | |
| *** Snivilis is now known as m|k|e | 16:13 | |
| *** sambetts has quit IRC | 16:14 | |
| *** amaretskiy has quit IRC | 16:16 | |
| *** banix has quit IRC | 16:17 | |
| salv-orlando | mestery: are you preparing a message like this -> http://www.mail-archive.com/openstack-dev@lists.openstack.org/msg34024.html ? | 16:17 |
| * mestery looks | 16:17 | |
| mestery | salv-orlando: That's the tl;dr of what I'm preparing, yes. | 16:18 |
| mestery | salv-orlando: If people want their drivers/plugins in stackforge, great! If not, I'm going to propose a drivers repository for them all. | 16:18 |
| salv-orlando | ah if that’s the tl;dr I think you’re not writing a message, but an essay | 16:19 |
| salv-orlando | or a paper | 16:19 |
| mestery | salv-orlando: Ha! | 16:20 |
| *** JayJ_ has joined #openstack-neutron | 16:21 | |
| *** devvesa has quit IRC | 16:21 | |
| *** JayJ_ has quit IRC | 16:22 | |
| *** pasquier-s__ has quit IRC | 16:27 | |
| *** mlavalle has quit IRC | 16:27 | |
| *** absubram_ has joined #openstack-neutron | 16:29 | |
| *** mlavalle has joined #openstack-neutron | 16:29 | |
| *** dvorkinista has joined #openstack-neutron | 16:30 | |
| *** jlibosva has quit IRC | 16:31 | |
| *** greghayn1 is now known as greghaynes | 16:33 | |
| *** jpich has quit IRC | 16:33 | |
| *** JayJ_ has joined #openstack-neutron | 16:33 | |
| *** radez is now known as radez_g0n3 | 16:34 | |
| *** amcrn has joined #openstack-neutron | 16:34 | |
| *** lordd has quit IRC | 16:35 | |
| *** karimb has quit IRC | 16:36 | |
| *** annegent_ has joined #openstack-neutron | 16:37 | |
| *** radez_g0n3 is now known as radez | 16:38 | |
| *** annegentle has quit IRC | 16:38 | |
| *** ygbo has quit IRC | 16:39 | |
| *** annegent_ has quit IRC | 16:39 | |
| *** pradk has joined #openstack-neutron | 16:40 | |
| *** JayJ_ has quit IRC | 16:41 | |
| *** yfried_ has joined #openstack-neutron | 16:43 | |
| *** JayJ_ has joined #openstack-neutron | 16:45 | |
| *** sballe has joined #openstack-neutron | 16:45 | |
| *** s3wong has joined #openstack-neutron | 16:45 | |
| *** banix has joined #openstack-neutron | 16:49 | |
| *** kopparam has quit IRC | 16:56 | |
| *** kopparam has joined #openstack-neutron | 16:57 | |
| openstackgerrit | Claudiu Belu proposed a change to openstack/neutron: Fixes Hyper-V agent stateful security group rules https://review.openstack.org/117543 | 16:57 |
| *** manishg has joined #openstack-neutron | 16:57 | |
| *** diga has joined #openstack-neutron | 17:03 | |
| *** vivek-ebay has joined #openstack-neutron | 17:06 | |
| *** lazy_prince is now known as killer_prince | 17:06 | |
| *** tsg has quit IRC | 17:07 | |
| *** jlibosva has joined #openstack-neutron | 17:08 | |
| *** tsg has joined #openstack-neutron | 17:09 | |
| *** sbfox has joined #openstack-neutron | 17:09 | |
| *** JayJ_ has quit IRC | 17:10 | |
| *** banix has quit IRC | 17:10 | |
| *** kopparam has quit IRC | 17:10 | |
| *** bvandenh has quit IRC | 17:10 | |
| *** kopparam has joined #openstack-neutron | 17:11 | |
| *** oreillyd_ has quit IRC | 17:12 | |
| *** banix has joined #openstack-neutron | 17:14 | |
| *** kopparam has quit IRC | 17:15 | |
| *** Longgeek_ has joined #openstack-neutron | 17:16 | |
| *** diga has quit IRC | 17:17 | |
| *** jlibosva has quit IRC | 17:18 | |
| *** luqas has quit IRC | 17:18 | |
| *** suresh12 has joined #openstack-neutron | 17:19 | |
| *** Longgeek has quit IRC | 17:19 | |
| *** aix has quit IRC | 17:19 | |
| *** harlowja has joined #openstack-neutron | 17:20 | |
| *** luqas has joined #openstack-neutron | 17:20 | |
| *** luqas has quit IRC | 17:21 | |
| *** SumitNaiksatam has quit IRC | 17:22 | |
| *** annegentle has joined #openstack-neutron | 17:23 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:24 | |
| *** pcm_ has quit IRC | 17:27 | |
| *** pballand has quit IRC | 17:28 | |
| *** jschwarz has quit IRC | 17:28 | |
| *** claudiub has quit IRC | 17:28 | |
| *** amotoki has quit IRC | 17:29 | |
| *** pballand has joined #openstack-neutron | 17:29 | |
| *** reed has joined #openstack-neutron | 17:30 | |
| *** lordd has joined #openstack-neutron | 17:30 | |
| *** pballand has quit IRC | 17:30 | |
| *** pcm_ has joined #openstack-neutron | 17:31 | |
| *** smcquaid has joined #openstack-neutron | 17:32 | |
| *** JayJ_ has joined #openstack-neutron | 17:32 | |
| *** dsneddon has joined #openstack-neutron | 17:34 | |
| *** JayJ_ has quit IRC | 17:34 | |
| *** ajo has joined #openstack-neutron | 17:34 | |
| *** marun_afk has quit IRC | 17:36 | |
| *** SumitNaiksatam has joined #openstack-neutron | 17:36 | |
| *** SridharRamaswamy has joined #openstack-neutron | 17:36 | |
| *** dbite has joined #openstack-neutron | 17:37 | |
| *** k_stev has joined #openstack-neutron | 17:38 | |
| *** doude has joined #openstack-neutron | 17:39 | |
| *** lordd has quit IRC | 17:39 | |
| *** tflynn has joined #openstack-neutron | 17:40 | |
| *** rms_13 has joined #openstack-neutron | 17:40 | |
| *** JayJ_ has joined #openstack-neutron | 17:40 | |
| *** tflynn has quit IRC | 17:41 | |
| *** JayJ_ has quit IRC | 17:41 | |
| *** lordd has joined #openstack-neutron | 17:42 | |
| *** tflynn has joined #openstack-neutron | 17:42 | |
| *** annegentle has quit IRC | 17:42 | |
| *** amuller has quit IRC | 17:42 | |
| *** annegentle has joined #openstack-neutron | 17:43 | |
| *** packet has quit IRC | 17:45 | |
| *** packet has joined #openstack-neutron | 17:46 | |
| *** annegentle has quit IRC | 17:47 | |
| *** markmcclain has joined #openstack-neutron | 17:48 | |
| *** Youcef has quit IRC | 17:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/neutron: Updated from global requirements https://review.openstack.org/117949 | 17:50 |
| *** lordd has quit IRC | 17:51 | |
| *** lordd has joined #openstack-neutron | 17:52 | |
| *** BillTheKat has joined #openstack-neutron | 17:52 | |
| *** sbfox has quit IRC | 17:52 | |
| *** lordd has quit IRC | 17:53 | |
| *** BillTheKat has quit IRC | 17:53 | |
| *** BillTheKat has joined #openstack-neutron | 17:53 | |
| *** moha_hunt has joined #openstack-neutron | 17:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/119148 | 17:55 |
| *** tflynn has quit IRC | 17:55 | |
| *** tflynn has joined #openstack-neutron | 17:56 | |
| *** annegentle has joined #openstack-neutron | 17:56 | |
| *** emagana has joined #openstack-neutron | 17:58 | |
| *** networkstatic has quit IRC | 18:01 | |
| *** 6A4AAKDJG has quit IRC | 18:02 | |
| *** rkukura has joined #openstack-neutron | 18:02 | |
| *** vivek-ebay has quit IRC | 18:02 | |
| *** vivek-ebay has joined #openstack-neutron | 18:03 | |
| *** Youcef has joined #openstack-neutron | 18:05 | |
| *** annegentle has quit IRC | 18:06 | |
| *** dvorkinista has quit IRC | 18:12 | |
| *** sbfox has joined #openstack-neutron | 18:14 | |
| *** otherwiseguy has quit IRC | 18:15 | |
| *** dvorkinista has joined #openstack-neutron | 18:16 | |
| *** dvorkinista has quit IRC | 18:16 | |
| *** emagana has quit IRC | 18:16 | |
| *** emagana has joined #openstack-neutron | 18:17 | |
| *** pcm_ has quit IRC | 18:17 | |
| *** baoli has quit IRC | 18:18 | |
| *** pcm_ has joined #openstack-neutron | 18:18 | |
| *** yeungp has left #openstack-neutron | 18:18 | |
| *** annegentle has joined #openstack-neutron | 18:18 | |
| *** otherwiseguy has joined #openstack-neutron | 18:18 | |
| *** roeyc has joined #openstack-neutron | 18:20 | |
| *** emagana has quit IRC | 18:21 | |
| *** banix has quit IRC | 18:21 | |
| *** ajo has quit IRC | 18:21 | |
| *** crose has quit IRC | 18:21 | |
| *** crose has joined #openstack-neutron | 18:22 | |
| *** vivek-ebay has quit IRC | 18:22 | |
| *** roeyc has quit IRC | 18:24 | |
| *** ZZelle_ has joined #openstack-neutron | 18:25 | |
| *** otherwiseguy has quit IRC | 18:25 | |
| *** amcrn has quit IRC | 18:25 | |
| *** yeungp has joined #openstack-neutron | 18:26 | |
| *** JayJ_ has joined #openstack-neutron | 18:26 | |
| *** dvorkinista has joined #openstack-neutron | 18:26 | |
| *** amuller has joined #openstack-neutron | 18:26 | |
| *** doude has quit IRC | 18:29 | |
| *** emagana has joined #openstack-neutron | 18:29 | |
| *** sbfox has quit IRC | 18:30 | |
| *** JayJ_ has quit IRC | 18:30 | |
| *** cshahani has joined #openstack-neutron | 18:30 | |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Remove subnet_id from check_ports_exist_on_l3agent https://review.openstack.org/118705 | 18:31 |
| *** amuller has quit IRC | 18:32 | |
| *** JayJ_ has joined #openstack-neutron | 18:35 | |
| *** crc32 has quit IRC | 18:36 | |
| *** tomoe_ has quit IRC | 18:36 | |
| *** sbfox has joined #openstack-neutron | 18:38 | |
| *** padkrish has joined #openstack-neutron | 18:38 | |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Add scheduler unit tests to enable bug fixes and refactoring https://review.openstack.org/118704 | 18:39 |
| *** markmcclain has quit IRC | 18:41 | |
| *** markmcclain has joined #openstack-neutron | 18:41 | |
| *** dvorkinista has quit IRC | 18:41 | |
| *** ianw has quit IRC | 18:42 | |
| *** aleksandr_null has quit IRC | 18:43 | |
| *** jp_at_hp has quit IRC | 18:44 | |
| *** jp_at_hp has joined #openstack-neutron | 18:44 | |
| *** jp_at_hp has quit IRC | 18:45 | |
| *** jdag___ is now known as jdaggett | 18:45 | |
| *** a_le has quit IRC | 18:45 | |
| *** a_le has joined #openstack-neutron | 18:46 | |
| ZZelle_ | SumitNaiksatam, hi | 18:47 |
| *** JayJ_ has quit IRC | 18:47 | |
| *** JayJ_ has joined #openstack-neutron | 18:49 | |
| *** phil_h has quit IRC | 18:54 | |
| *** suresh12 has quit IRC | 18:54 | |
| *** hanzhang has quit IRC | 18:55 | |
| *** hanzhang has joined #openstack-neutron | 18:55 | |
| *** crose has quit IRC | 18:56 | |
| *** markmcclain has quit IRC | 19:04 | |
| *** tflynn has quit IRC | 19:04 | |
| *** tflynn has joined #openstack-neutron | 19:05 | |
| *** shivharis has joined #openstack-neutron | 19:06 | |
| *** otherwiseguy has joined #openstack-neutron | 19:06 | |
| *** dbite has quit IRC | 19:08 | |
| *** snowgust has quit IRC | 19:08 | |
| *** terryw has joined #openstack-neutron | 19:09 | |
| *** dbite has joined #openstack-neutron | 19:10 | |
| *** sgordon_ has quit IRC | 19:10 | |
| *** otherwiseguy has quit IRC | 19:11 | |
| *** s3wong has quit IRC | 19:13 | |
| *** s3wong has joined #openstack-neutron | 19:13 | |
| *** mitarun has quit IRC | 19:15 | |
| *** emagana has quit IRC | 19:16 | |
| *** mitarun has joined #openstack-neutron | 19:16 | |
| *** emagana has joined #openstack-neutron | 19:16 | |
| *** mitarun has quit IRC | 19:20 | |
| *** emagana has quit IRC | 19:21 | |
| SumitNaiksatam | ZZelle_: hi | 19:21 |
| *** padkrish has quit IRC | 19:22 | |
| *** Sukhdev has joined #openstack-neutron | 19:22 | |
| ZZelle_ | SumitNaiksatam, i was wondering if there is a reason why it's not possible to assign a firewall policy to a specific router? is it intended? | 19:22 |
| SumitNaiksatam | ZZelle_: the firewall object model was meant to be independent of how the firewall is realized | 19:23 |
| *** suresh12 has joined #openstack-neutron | 19:23 | |
| *** phil_h has joined #openstack-neutron | 19:24 | |
| SumitNaiksatam | ZZelle_: the fact that the firewall is associated with a router in the reference implementation is an artifact of the reference implementation and deployment | 19:24 |
| SumitNaiksatam | ZZelle_: however there are other cases when it might not be realized on a router | 19:24 |
| *** jasondotstar is now known as jasondotstar|afk | 19:24 | |
| *** packet has quit IRC | 19:24 | |
| ZZelle_ | SumitNaiksatam, so we could imagine to apply the firewall on a router? | 19:25 |
| *** banix has joined #openstack-neutron | 19:25 | |
| SumitNaiksatam | ZZelle_: yes that is one of the possibilities, but not the only one | 19:25 |
| SumitNaiksatam | ZZelle_: so this was meant to be addressed by the following #link https://review.openstack.org/#/c/93128 | 19:26 |
| *** ajo has joined #openstack-neutron | 19:26 | |
| SumitNaiksatam | ZZelle_: the above would have incorporated the router deployment case as well | 19:26 |
| ZZelle_ | SumitNaiksatam, so we could imagine also to a firewall deployment using security group backend? (checking i fully understand your first sentence) | 19:27 |
| ZZelle_ | s/to a/to/ | 19:28 |
| SumitNaiksatam | ZZelle_: well, i think the question you are asking is it possible to deploy iptables rules on hosts, to realize the firewall policies | 19:29 |
| SumitNaiksatam | ZZelle_: and the answer would be yes | 19:29 |
| *** Longgeek_ has quit IRC | 19:29 | |
| *** lori has quit IRC | 19:29 | |
| ZZelle_ | SumitNaiksatam, ok | 19:29 |
| SumitNaiksatam | ZZelle_: in fact for the FWaaS DVR support we are doing something similar | 19:29 |
| *** hanzhang has quit IRC | 19:30 | |
| *** hanzhang has joined #openstack-neutron | 19:30 | |
| SumitNaiksatam | ZZelle_: #link https://review.openstack.org/#/c/113359/ | 19:31 |
| ZZelle_ | SumitNaiksatam, indeed, firewalling at the edge | 19:31 |
| *** markmcclain has joined #openstack-neutron | 19:32 | |
| *** roeyc has joined #openstack-neutron | 19:32 | |
| SumitNaiksatam | ZZelle_: okay ttyl, got to step away | 19:32 |
| ZZelle_ | SumitNaiksatam, bye | 19:33 |
| *** roeyc has quit IRC | 19:33 | |
| *** mwagner_lap has quit IRC | 19:33 | |
| *** ZangMingJie has quit IRC | 19:34 | |
| *** ZangMingJie has joined #openstack-neutron | 19:35 | |
| *** StevenK has quit IRC | 19:38 | |
| anteaya | who is here for brocade ci? | 19:39 |
| anteaya | why is brocade adx ci commenting on tempest? https://review.openstack.org/#/c/119060/ | 19:40 |
| mestery | anteaya: You want to talk to shivharis I think. | 19:40 |
| *** banix has quit IRC | 19:41 | |
| anteaya | shivharis: why is brocade adx ci commenting on tempest? | 19:41 |
| anteaya | qa ptl is asking me, and I don't have an answer | 19:41 |
| anteaya | mestery: thanks | 19:41 |
| *** spandhe has joined #openstack-neutron | 19:41 | |
| *** emagana has joined #openstack-neutron | 19:42 | |
| *** xgerman has quit IRC | 19:43 | |
| *** StevenK has joined #openstack-neutron | 19:43 | |
| *** networkstatic has joined #openstack-neutron | 19:47 | |
| *** sgordon_ has joined #openstack-neutron | 19:48 | |
| *** prasoon has quit IRC | 19:49 | |
| *** phil_h has quit IRC | 19:49 | |
| *** aleksandr_null has joined #openstack-neutron | 19:50 | |
| openstackgerrit | Chengli Xu proposed a change to openstack/neutron: Fix metadata agent's auth info caching https://review.openstack.org/118996 | 19:51 |
| kevinbenton | anteaya: in theory, shouldn’t a CI system try out tempest changes that will affect it? | 19:51 |
| anteaya | in practice, ci operators should ask the ptl of the program prior to comment on patches | 19:52 |
| *** banix has joined #openstack-neutron | 19:52 | |
| *** padkrish has joined #openstack-neutron | 19:52 | |
| anteaya | if the qa ptl is consulted about what systems are testing which repos, then the answer to the question "why is this system doing this?" is answered | 19:52 |
| anteaya | as to whether or not a system should be commenting on tempest is a question for the qa ptl to answer | 19:53 |
| *** ajo has quit IRC | 19:53 | |
| *** terryw has quit IRC | 19:54 | |
| *** johnbelamaric has joined #openstack-neutron | 19:54 | |
| anteaya | you will note in the above patch the citrix xenserver ci is commenting | 19:54 |
| anteaya | the operator of that system, bobball has talked with the qa ptl, mtreinish and they each know why that ci is commenting | 19:55 |
| anteaya | noone knows why brocade adx ci is | 19:55 |
| mtreinish | anteaya: commenting on tempest changes is fine, but it's more that it's spamming and not leaving links to logs... | 19:56 |
| *** a_le has quit IRC | 19:56 | |
| anteaya | mtreinish: well not leaving links to logs is a requirement, which gets them disabled | 19:56 |
| *** a_le has joined #openstack-neutron | 19:56 | |
| kevinbenton | that makes sense | 19:56 |
| anteaya | mestery: I am about to disable brocade adx ci for not having links to logs, in case you wish to advocate on their behalf | 19:57 |
| anteaya | it being ff day and all | 19:57 |
| kevinbenton | i was just implying that every CI system relies on tempest so they should be testing tempest changes too. :-) | 19:57 |
| anteaya | kevinbenton: as long as they aren't spamming and have met requirements, that is up to mtreinish | 19:57 |
| kevinbenton | right | 19:58 |
| anteaya | and early indications are he feels that is fine | 19:58 |
| *** baoli has joined #openstack-neutron | 19:58 | |
| mtreinish | kevinbenton: yeah I'm fine with that, I actually wish more people would run against tempest changes | 19:58 |
| * anteaya waits a few minutes for mestery, in case he feels strongly | 19:58 | |
| * mtreinish feels like he just shot himself in the foot... | 19:58 | |
| kevinbenton | mtreinish: what’s the patch frequency for tempest? I would have our system do it but it can hardly keep up with neutron during busy times | 19:59 |
| anteaya | mtreinish: why shouldn't you have some of the fun | 19:59 |
| anteaya | :D | 19:59 |
| mestery | anteaya: Did you try pinging sweston? I think he's from Brocade like shivharis is as well. | 19:59 |
| mestery | sweston: See conversation above from anteaya and myself about Brocade CI | 19:59 |
| anteaya | I do think he used to be from brocade | 19:59 |
| mestery | anteaya: Ah, he left? | 20:00 |
| mtreinish | kevinbenton: hmm, you know I've never really looked at the patch frequency for tempest. Let me check | 20:00 |
| openstackgerrit | pritesh proposed a change to openstack/neutron: Remove the Cisco Nexus monolithic plugin https://review.openstack.org/116028 | 20:00 |
| anteaya | mestery: any strong feelings trying to persuading me to from disabling? they are not meeting requirements, no links to logs | 20:00 |
| anteaya | mestery: according to my most recent information, he did | 20:00 |
| mestery | anteaya: Did you send a mail to the list? Even after disabling is fine, but I'd send an email as well. | 20:00 |
| *** baoli has quit IRC | 20:00 | |
| anteaya | I'll send an email after disabling | 20:00 |
| *** baoli has joined #openstack-neutron | 20:01 | |
| *** liusheng has quit IRC | 20:01 | |
| mestery | anteaya: Ack | 20:01 |
| *** liusheng has joined #openstack-neutron | 20:01 | |
| *** padkrish has quit IRC | 20:01 | |
| *** hanzhang has quit IRC | 20:02 | |
| *** liusheng has quit IRC | 20:02 | |
| *** dbite has quit IRC | 20:02 | |
| *** hanzhang has joined #openstack-neutron | 20:02 | |
| *** pcm_ has quit IRC | 20:03 | |
| *** otherwiseguy has joined #openstack-neutron | 20:04 | |
| *** lori has joined #openstack-neutron | 20:05 | |
| *** emagana has quit IRC | 20:05 | |
| *** emagana has joined #openstack-neutron | 20:05 | |
| *** emagana has quit IRC | 20:05 | |
| *** emagana has joined #openstack-neutron | 20:06 | |
| *** banix has quit IRC | 20:06 | |
| *** otherwiseguy has quit IRC | 20:09 | |
| *** tflynn_ has joined #openstack-neutron | 20:09 | |
| *** lori is now known as lori|away | 20:09 | |
| *** bjornar_ has joined #openstack-neutron | 20:10 | |
| *** amuller has joined #openstack-neutron | 20:11 | |
| *** padkrish has joined #openstack-neutron | 20:12 | |
| *** tflynn has quit IRC | 20:13 | |
| *** sgordon_ has quit IRC | 20:13 | |
| *** banix has joined #openstack-neutron | 20:14 | |
| *** hanzhang has quit IRC | 20:14 | |
| *** hanzhang has joined #openstack-neutron | 20:15 | |
| haleyb | safchain: ping, had question about https://review.openstack.org/#/c/70090/ - didn't know if you could answer a question i just posted regarding amotoki's -1 | 20:15 |
| *** johnbelamaric has quit IRC | 20:18 | |
| *** cburgess_ is now known as cburgess | 20:19 | |
| *** dkehnx1 has quit IRC | 20:19 | |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Changes to support FWaaS in a DVR based environment https://review.openstack.org/113359 | 20:20 |
| shivharis | mestery: hi | 20:21 |
| *** tsg has quit IRC | 20:22 | |
| *** moha_hunt has quit IRC | 20:23 | |
| *** julim has quit IRC | 20:28 | |
| *** yamamoto_ has quit IRC | 20:28 | |
| *** tsg has joined #openstack-neutron | 20:29 | |
| shivharis | anteaya: hi | 20:32 |
| *** jorgem has joined #openstack-neutron | 20:32 | |
| anteaya | hello | 20:33 |
| anteaya | brocade adx ci is disabled | 20:33 |
| anteaya | you read the email to the -announce mailing list? | 20:33 |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Call unbind_snat_servicenode from schedule router https://review.openstack.org/118706 | 20:33 |
| *** igordcard has joined #openstack-neutron | 20:35 | |
| shivharis | anteaya: that is fine. i checked with the ADX team and they mentioned that they have disabled the CI for now. | 20:36 |
| *** doude has joined #openstack-neutron | 20:37 | |
| shivharis | btw, sweston is not with brocade anymore, you can contact me in the mean time | 20:37 |
| shivharis | i will get somebody in the adx team to coordinate the CI to get it re-enabled. | 20:38 |
| anteaya | shivharis: great | 20:38 |
| anteaya | shivharis: I left instructions in the email, if they have questions have them contact me on irc | 20:38 |
| shivharis | btw, kevinbentons comment makes sense. we should try tempest changes as well. however this was not reasoning in this specific case | 20:39 |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Remove hints from schedule_router https://review.openstack.org/118707 | 20:39 |
| shivharis | anteaya: thanks. ttyl. | 20:39 |
| *** cipcosma has quit IRC | 20:40 | |
| *** tgohad has joined #openstack-neutron | 20:41 | |
| anteaya | shivharis: kk | 20:42 |
| *** tsg has quit IRC | 20:43 | |
| *** tflynn_ has quit IRC | 20:44 | |
| *** tflynn has joined #openstack-neutron | 20:45 | |
| *** tflynn has quit IRC | 20:45 | |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Avoid an extra database query in schedule_snat_router https://review.openstack.org/114410 | 20:45 |
| shivharis | markmcclain: hi | 20:45 |
| *** hanzhang has quit IRC | 20:46 | |
| *** tflynn has joined #openstack-neutron | 20:46 | |
| markmcclain | shivharis: hi | 20:46 |
| *** hanzhang has joined #openstack-neutron | 20:46 | |
| shivharis | markmcclain: this is regarding the patch:https://review.openstack.org/#/c/114801/ which was given FFE | 20:46 |
| shivharis | I have some minor comments on the patch set, nothing major, how should i handle this? | 20:47 |
| *** xgerman has joined #openstack-neutron | 20:47 | |
| shivharis | markmcclain: suggestions? | 20:47 |
| markmcclain | shivharis: lets wait until tomorrow when FFE status will be finalized. It is possible the release management may pushback once all FFEs across all projects are determined | 20:48 |
| shivharis | so you suggest not to respin, right, just confirming.. | 20:49 |
| markmcclain | correct.. let's no respin yet | 20:49 |
| shivharis | ok, thanks. | 20:49 |
| markmcclain | once we get the go ahead we can respin and clear up the nits | 20:49 |
| shivharis | ok, got it. thanks much. | 20:50 |
| markmcclain | shivharis: you're welcome | 20:50 |
| shivharis | markmcclain: ttyl | 20:50 |
| *** sbalukoff has joined #openstack-neutron | 20:51 | |
| *** phil_h has joined #openstack-neutron | 20:53 | |
| jckasper | the ovs and linuxbridge monolith plugins. weren't they going to be removed from source tree in juno-3 ? | 20:53 |
| *** mlavalle has quit IRC | 20:54 | |
| openstackgerrit | Divya ChanneGowda proposed a change to openstack/neutron: Use dict_extend_functions to populate provider network attributes https://review.openstack.org/118911 | 20:54 |
| *** jasondotstar|afk is now known as jasondotstar | 20:56 | |
| *** yfauser has joined #openstack-neutron | 20:57 | |
| *** yfauser has left #openstack-neutron | 20:57 | |
| *** mlavalle has joined #openstack-neutron | 20:58 | |
| *** emagana has quit IRC | 20:58 | |
| *** cshahani has quit IRC | 20:59 | |
| *** emagana has joined #openstack-neutron | 21:00 | |
| *** catohornet has quit IRC | 21:02 | |
| ZZelle_ | jckasper, u should ask markmcclain for ovs/linuxbridge removals | 21:04 |
| jckasper | markmcclain: hi | 21:04 |
| *** annegentle has quit IRC | 21:05 | |
| markmcclain | jckasper: hi | 21:05 |
| jckasper | markmcclain: the ovs and linuxbridge monolith plugins. weren't they going to be removed from source tree in juno-3 ? | 21:05 |
| markmcclain | jckasper: yes we're removing them once the gate clears up | 21:05 |
| jckasper | ah, ok. | 21:06 |
| jckasper | thanks | 21:06 |
| markmcclain | jckasper: we're tracking it with this bug: https://bugs.launchpad.net/neutron/+bug/1323729 | 21:06 |
| *** networkstatic has quit IRC | 21:08 | |
| *** dave_tucker is now known as dave_tucker_zzz | 21:08 | |
| *** networkstatic has joined #openstack-neutron | 21:08 | |
| *** drankis has quit IRC | 21:08 | |
| *** cshahani has joined #openstack-neutron | 21:09 | |
| *** sbfox has quit IRC | 21:09 | |
| *** johnbelamaric has joined #openstack-neutron | 21:12 | |
| *** emagana has quit IRC | 21:12 | |
| *** emagana has joined #openstack-neutron | 21:12 | |
| *** dave_tucker_zzz is now known as dave_tucker | 21:13 | |
| *** dave_tucker is now known as dave_tucker_zzz | 21:13 | |
| *** marun_afk has joined #openstack-neutron | 21:14 | |
| *** sbfox has joined #openstack-neutron | 21:15 | |
| openstackgerrit | A change was merged to openstack/neutron: Fixes Hyper-V agent issue on Hyper-V 2008 R2 https://review.openstack.org/116505 | 21:15 |
| *** emagana has quit IRC | 21:17 | |
| *** Sukhdev has quit IRC | 21:22 | |
| *** thomasem has quit IRC | 21:23 | |
| *** yamamoto has joined #openstack-neutron | 21:29 | |
| *** bjornar_ has quit IRC | 21:33 | |
| *** yamamoto has quit IRC | 21:34 | |
| *** sbfox has quit IRC | 21:34 | |
| *** johnbelamaric has quit IRC | 21:35 | |
| *** jckasper has quit IRC | 21:35 | |
| *** johnbelamaric has joined #openstack-neutron | 21:35 | |
| *** johnbelamaric has quit IRC | 21:37 | |
| *** johnbelamaric has joined #openstack-neutron | 21:37 | |
| *** dkehnx1 has joined #openstack-neutron | 21:37 | |
| *** sbfox has joined #openstack-neutron | 21:38 | |
| *** mlavalle has quit IRC | 21:38 | |
| *** annegentle has joined #openstack-neutron | 21:38 | |
| *** annegentle has quit IRC | 21:40 | |
| *** johnbelamaric has quit IRC | 21:41 | |
| *** Youcef has quit IRC | 21:43 | |
| *** JayJ_ has quit IRC | 21:44 | |
| *** BillTheKat has quit IRC | 21:44 | |
| *** markmcclain has quit IRC | 21:45 | |
| *** JayJ_ has joined #openstack-neutron | 21:46 | |
| *** sbfox has quit IRC | 21:47 | |
| *** sbfox has joined #openstack-neutron | 21:47 | |
| *** hanzhang has quit IRC | 21:50 | |
| *** mlavalle has joined #openstack-neutron | 21:51 | |
| *** hanzhang has joined #openstack-neutron | 21:51 | |
| *** ivar-lazzaro1 has joined #openstack-neutron | 21:52 | |
| *** ivar-lazzaro has quit IRC | 21:53 | |
| *** JayJ_ has quit IRC | 21:55 | |
| *** dvorkinista has joined #openstack-neutron | 21:56 | |
| *** ianw has joined #openstack-neutron | 21:57 | |
| *** m|k|e has quit IRC | 21:58 | |
| *** tgohad has quit IRC | 21:58 | |
| *** tongli has quit IRC | 21:59 | |
| *** cshahani has quit IRC | 22:02 | |
| *** Zemeio has quit IRC | 22:04 | |
| *** watrous has quit IRC | 22:08 | |
| *** padkrish has quit IRC | 22:09 | |
| *** shivharis has quit IRC | 22:11 | |
| *** tsg has joined #openstack-neutron | 22:12 | |
| *** padkrish has joined #openstack-neutron | 22:12 | |
| *** emagana has joined #openstack-neutron | 22:13 | |
| *** jorgem has quit IRC | 22:13 | |
| *** arborism has joined #openstack-neutron | 22:14 | |
| *** absubram_ has quit IRC | 22:14 | |
| *** a_le has quit IRC | 22:15 | |
| *** a_le has joined #openstack-neutron | 22:16 | |
| *** emagana has quit IRC | 22:18 | |
| *** chuckC has quit IRC | 22:18 | |
| *** signed8bit has quit IRC | 22:20 | |
| *** doude has quit IRC | 22:22 | |
| *** pradk has quit IRC | 22:22 | |
| *** yamamoto has joined #openstack-neutron | 22:24 | |
| *** Youcef has joined #openstack-neutron | 22:24 | |
| *** sweston has quit IRC | 22:25 | |
| *** alexpilotti has quit IRC | 22:25 | |
| *** sweston has joined #openstack-neutron | 22:25 | |
| *** foexle_ has quit IRC | 22:26 | |
| openstackgerrit | Saksham Varma proposed a change to openstack/neutron: Tenants shouldn't access unshared network profiles https://review.openstack.org/119236 | 22:28 |
| *** yamamoto has quit IRC | 22:29 | |
| *** padkrish has quit IRC | 22:30 | |
| *** sbfox has quit IRC | 22:31 | |
| *** emagana has joined #openstack-neutron | 22:33 | |
| *** jobewan has quit IRC | 22:34 | |
| *** emagana has quit IRC | 22:34 | |
| *** emagana has joined #openstack-neutron | 22:34 | |
| *** arborism is now known as amcrn | 22:35 | |
| *** cshahani has joined #openstack-neutron | 22:35 | |
| *** hanzhang has quit IRC | 22:36 | |
| *** hanzhang has joined #openstack-neutron | 22:36 | |
| *** xgerman has quit IRC | 22:37 | |
| *** sbfox has joined #openstack-neutron | 22:37 | |
| openstackgerrit | Angus Lees proposed a change to openstack/neutron: Add pylint tox environment and disable all existing warnings https://review.openstack.org/115486 | 22:37 |
| *** baoli has quit IRC | 22:38 | |
| *** emagana has quit IRC | 22:39 | |
| *** baoli has joined #openstack-neutron | 22:39 | |
| *** dirtyob has joined #openstack-neutron | 22:39 | |
| *** cshahani has quit IRC | 22:40 | |
| *** dguitarbite has quit IRC | 22:42 | |
| *** flaviof has quit IRC | 22:42 | |
| *** tsg has quit IRC | 22:42 | |
| *** emagana has joined #openstack-neutron | 22:42 | |
| *** tflynn_ has joined #openstack-neutron | 22:43 | |
| *** nati_ueno has joined #openstack-neutron | 22:43 | |
| *** dfarrell07 has quit IRC | 22:44 | |
| *** nati_ueno has quit IRC | 22:46 | |
| *** nati_ueno has joined #openstack-neutron | 22:46 | |
| *** tflynn_ has quit IRC | 22:46 | |
| *** tflynn has quit IRC | 22:47 | |
| *** emagana has quit IRC | 22:47 | |
| *** tflynn has joined #openstack-neutron | 22:47 | |
| *** ZZelle_ has quit IRC | 22:48 | |
| *** cshahani has joined #openstack-neutron | 22:48 | |
| *** tflynn_ has joined #openstack-neutron | 22:52 | |
| *** tflynn has quit IRC | 22:52 | |
| *** sbfox has quit IRC | 22:53 | |
| *** dirtyob has quit IRC | 22:54 | |
| *** emagana has joined #openstack-neutron | 22:55 | |
| *** baoli has quit IRC | 22:55 | |
| *** tflynn_ has quit IRC | 22:55 | |
| *** valiantfirkin has joined #openstack-neutron | 22:56 | |
| *** tflynn has joined #openstack-neutron | 22:56 | |
| *** tflynn_ has joined #openstack-neutron | 22:57 | |
| *** tflynn has quit IRC | 22:57 | |
| *** tflynn_ has quit IRC | 22:59 | |
| *** tflynn has joined #openstack-neutron | 23:00 | |
| *** carl_baldwin has quit IRC | 23:00 | |
| *** valiantfirkin has quit IRC | 23:01 | |
| *** nati_uen_ has joined #openstack-neutron | 23:02 | |
| *** nlahouti has quit IRC | 23:03 | |
| *** nlahouti has joined #openstack-neutron | 23:06 | |
| *** nati_ueno has quit IRC | 23:06 | |
| *** padkrish has joined #openstack-neutron | 23:06 | |
| *** chuckC has joined #openstack-neutron | 23:07 | |
| *** smcquaid has quit IRC | 23:07 | |
| *** r-daneel has quit IRC | 23:07 | |
| *** ivar-lazzaro1 has quit IRC | 23:09 | |
| *** Zemeio has joined #openstack-neutron | 23:09 | |
| *** Youcef has quit IRC | 23:12 | |
| *** hanzhang has quit IRC | 23:16 | |
| *** hanzhang has joined #openstack-neutron | 23:17 | |
| *** yamamoto has joined #openstack-neutron | 23:24 | |
| *** mlavalle_ has joined #openstack-neutron | 23:24 | |
| *** mlavalle has quit IRC | 23:25 | |
| *** yamamoto has quit IRC | 23:29 | |
| *** hanzhang has quit IRC | 23:29 | |
| *** hanzhang has joined #openstack-neutron | 23:29 | |
| *** jckasper has joined #openstack-neutron | 23:31 | |
| *** jckasper has quit IRC | 23:31 | |
| openstackgerrit | Michael Smith proposed a change to openstack/neutron: Updates to reschedule_router() for DVR https://review.openstack.org/119253 | 23:34 |
| openstackgerrit | A change was merged to openstack/neutron: Fix a bug in Mellanox plugin RPC caused by secgroup RPC refactoring https://review.openstack.org/118165 | 23:41 |
| *** emagana has quit IRC | 23:42 | |
| *** emagana has joined #openstack-neutron | 23:42 | |
| openstackgerrit | Sridar Kandaswamy proposed a change to openstack/neutron: Changes to support FWaaS in a DVR based environment https://review.openstack.org/113359 | 23:44 |
| *** emagana has quit IRC | 23:47 | |
| *** amuller has quit IRC | 23:47 | |
| *** yamamoto has joined #openstack-neutron | 23:54 | |
| *** tomoe_ has joined #openstack-neutron | 23:54 | |
| *** sbalukoff has quit IRC | 23:56 | |
| *** linuxgeek_ has joined #openstack-neutron | 23:57 | |
| *** yamamoto has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!