*** shwetaap has joined #openstack-neutron | 00:00 | |
*** dannywilson has quit IRC | 00:01 | |
*** blogan_ has joined #openstack-neutron | 00:01 | |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 00:02 |
---|---|---|
*** SourabhP has quit IRC | 00:03 | |
openstackgerrit | Merged openstack/neutron: Remove out-of-tree vendor AGENT_TYPE_* constant https://review.openstack.org/216417 | 00:03 |
openstackgerrit | Merged openstack/neutron: Process user iptables rules before INVALID https://review.openstack.org/218517 | 00:03 |
*** shwetaap1 has quit IRC | 00:03 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Added Exceptions and Logs to track FIP on DVR https://review.openstack.org/223879 | 00:04 |
openstackgerrit | Merged openstack/neutron: Allow passing arbitrary ip route parameters to add/delete_route https://review.openstack.org/219037 | 00:04 |
*** fitoduarte has quit IRC | 00:08 | |
*** shashank_hegde has quit IRC | 00:08 | |
*** yamamoto has quit IRC | 00:09 | |
*** shoutm has joined #openstack-neutron | 00:10 | |
*** Swami has quit IRC | 00:11 | |
*** shashank_hegde has joined #openstack-neutron | 00:13 | |
*** SumitNaiksatam has quit IRC | 00:15 | |
*** imcsk8_ has joined #openstack-neutron | 00:16 | |
*** hanzhang has joined #openstack-neutron | 00:17 | |
*** armax_ has joined #openstack-neutron | 00:17 | |
*** toddnni_ has joined #openstack-neutron | 00:18 | |
*** zhhuabj has quit IRC | 00:18 | |
*** thorst has joined #openstack-neutron | 00:18 | |
*** devlaps has quit IRC | 00:18 | |
*** julien_ZTE has quit IRC | 00:19 | |
*** dims_ has quit IRC | 00:20 | |
*** bapalm_ has joined #openstack-neutron | 00:20 | |
*** clayton- has joined #openstack-neutron | 00:20 | |
*** cburgess_ has joined #openstack-neutron | 00:21 | |
*** imcsk8 has quit IRC | 00:21 | |
*** shihanzhang has quit IRC | 00:21 | |
*** cburgess has quit IRC | 00:21 | |
*** shz has quit IRC | 00:21 | |
*** armax has quit IRC | 00:21 | |
*** toddnni has quit IRC | 00:21 | |
*** toddnni_ is now known as toddnni | 00:21 | |
*** shz has joined #openstack-neutron | 00:21 | |
*** bapalm has quit IRC | 00:21 | |
*** clayton has quit IRC | 00:21 | |
*** shz has quit IRC | 00:21 | |
*** clayton- is now known as clayton | 00:21 | |
*** armax_ is now known as armax | 00:21 | |
*** shihanzhang has joined #openstack-neutron | 00:21 | |
*** gildub has joined #openstack-neutron | 00:22 | |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #openstack-neutron | 00:23 | |
*** yamahata has quit IRC | 00:24 | |
*** shivrao has quit IRC | 00:25 | |
*** bradjones has quit IRC | 00:25 | |
*** tkatarki has joined #openstack-neutron | 00:26 | |
*** minwang2_ has joined #openstack-neutron | 00:27 | |
*** thorst has quit IRC | 00:28 | |
*** shoutm_ has joined #openstack-neutron | 00:28 | |
*** abhiraut has quit IRC | 00:28 | |
*** ildikov_ has quit IRC | 00:28 | |
*** Prem_ has joined #openstack-neutron | 00:28 | |
*** xgerman_ has joined #openstack-neutron | 00:28 | |
*** kgalanov_ has joined #openstack-neutron | 00:29 | |
*** tomoe_ has joined #openstack-neutron | 00:29 | |
*** shoutm has quit IRC | 00:29 | |
*** StevenK_ has joined #openstack-neutron | 00:31 | |
*** wasmum- has joined #openstack-neutron | 00:31 | |
*** raorn_ has joined #openstack-neutron | 00:32 | |
*** abhiraut has joined #openstack-neutron | 00:32 | |
*** StevenK has quit IRC | 00:32 | |
*** StevenK_ is now known as StevenK | 00:33 | |
*** tidwellr has left #openstack-neutron | 00:33 | |
*** JoshNang_ has joined #openstack-neutron | 00:34 | |
*** agireud- has joined #openstack-neutron | 00:34 | |
*** madhu_ak has quit IRC | 00:34 | |
*** rex_lee_ has joined #openstack-neutron | 00:34 | |
*** sweston_ has joined #openstack-neutron | 00:35 | |
*** minwang2 has quit IRC | 00:35 | |
*** jaypipes has quit IRC | 00:35 | |
*** agireud has quit IRC | 00:35 | |
*** JoshNang has quit IRC | 00:35 | |
*** akamyshnikova has quit IRC | 00:35 | |
*** lennyb has quit IRC | 00:35 | |
*** raorn has quit IRC | 00:35 | |
*** clarkb has quit IRC | 00:35 | |
*** fnordahl has quit IRC | 00:35 | |
*** Prem has quit IRC | 00:35 | |
*** sweston has quit IRC | 00:36 | |
*** kgalanov has quit IRC | 00:36 | |
*** xgerman has quit IRC | 00:36 | |
*** rex_lee has quit IRC | 00:36 | |
*** wasmum has quit IRC | 00:36 | |
*** agireud- is now known as agireud | 00:36 | |
*** minwang2_ is now known as minwang2 | 00:36 | |
*** JoshNang_ is now known as JoshNang | 00:36 | |
*** Prem_ is now known as Prem | 00:36 | |
*** sweston_ is now known as sweston | 00:37 | |
*** xgerman_ is now known as xgerman | 00:37 | |
*** akamyshnikova has joined #openstack-neutron | 00:37 | |
*** jaypipes has joined #openstack-neutron | 00:40 | |
*** tiswanso has joined #openstack-neutron | 00:40 | |
*** shwetaap has quit IRC | 00:40 | |
*** banix has joined #openstack-neutron | 00:41 | |
*** fnordahl has joined #openstack-neutron | 00:42 | |
*** lennyb has joined #openstack-neutron | 00:42 | |
*** clarkb has joined #openstack-neutron | 00:42 | |
*** mwagner_lap has joined #openstack-neutron | 00:42 | |
*** arajagopal has joined #openstack-neutron | 00:43 | |
*** _cjones_ has quit IRC | 00:43 | |
*** jckasper has joined #openstack-neutron | 00:45 | |
*** tiswanso_ has joined #openstack-neutron | 00:45 | |
*** shoutm_ has quit IRC | 00:46 | |
*** shashank_hegde has quit IRC | 00:48 | |
*** tiswanso has quit IRC | 00:48 | |
*** julien_ZTE has joined #openstack-neutron | 00:48 | |
*** shoutm has joined #openstack-neutron | 00:48 | |
*** moha_hunt has quit IRC | 00:49 | |
*** sthillma has quit IRC | 00:49 | |
*** sridhar_ram has quit IRC | 00:50 | |
*** salv-orl_ has joined #openstack-neutron | 00:53 | |
*** yamamoto has joined #openstack-neutron | 00:55 | |
*** salv-orlando has quit IRC | 00:56 | |
*** salv-orl_ has quit IRC | 00:57 | |
*** hoangcx has joined #openstack-neutron | 00:58 | |
*** banix has quit IRC | 01:01 | |
*** vivek-ebay has joined #openstack-neutron | 01:03 | |
*** tkatarki has quit IRC | 01:05 | |
*** dims_ has joined #openstack-neutron | 01:05 | |
*** vivek-eb_ has joined #openstack-neutron | 01:05 | |
*** tkatarki has joined #openstack-neutron | 01:06 | |
*** hanzhang has quit IRC | 01:08 | |
*** yamahata has joined #openstack-neutron | 01:08 | |
*** hanzhang has joined #openstack-neutron | 01:08 | |
*** Sukhdev has quit IRC | 01:09 | |
*** vivek-ebay has quit IRC | 01:09 | |
hoangcx | sc68cal: ping | 01:11 |
hoangcx | sc68cal: Could you please take a look on it and give me some comments to file the spec? | 01:13 |
hoangcx | sc68cal: #link https://review.openstack.org/#/c/203509/ | 01:13 |
hoangcx | sc68cal: Thanks! | 01:13 |
*** baojg has joined #openstack-neutron | 01:15 | |
*** crose has joined #openstack-neutron | 01:16 | |
*** r-daneel has quit IRC | 01:17 | |
*** lazy_prince has joined #openstack-neutron | 01:18 | |
*** stevemar has joined #openstack-neutron | 01:19 | |
*** stanzgy has joined #openstack-neutron | 01:19 | |
*** stanzgy__ has joined #openstack-neutron | 01:19 | |
*** baojg has quit IRC | 01:20 | |
*** jwy has quit IRC | 01:20 | |
*** shwetaap has joined #openstack-neutron | 01:22 | |
*** killer_prince has joined #openstack-neutron | 01:27 | |
*** stevemar has quit IRC | 01:29 | |
*** banix has joined #openstack-neutron | 01:29 | |
*** stevemar has joined #openstack-neutron | 01:30 | |
*** lazy_prince has quit IRC | 01:30 | |
*** lazy_prince has joined #openstack-neutron | 01:30 | |
*** killer_prince has quit IRC | 01:31 | |
*** dims_ has quit IRC | 01:34 | |
*** baoli has joined #openstack-neutron | 01:36 | |
*** baojg has joined #openstack-neutron | 01:38 | |
*** clev is now known as clev-away | 01:40 | |
*** ivar-laz_ has joined #openstack-neutron | 01:43 | |
*** ivar-lazzaro has quit IRC | 01:46 | |
*** sballe has quit IRC | 01:46 | |
*** ivar-laz_ has quit IRC | 01:47 | |
*** jwy has joined #openstack-neutron | 01:49 | |
*** tkatarki has quit IRC | 01:50 | |
*** nlahouti has quit IRC | 01:50 | |
*** zhenguo has joined #openstack-neutron | 01:50 | |
*** muralir has joined #openstack-neutron | 01:54 | |
*** erhudy1 has quit IRC | 01:57 | |
*** muralir has quit IRC | 01:58 | |
*** madhu_ak has joined #openstack-neutron | 02:03 | |
*** madhu_ak has joined #openstack-neutron | 02:03 | |
*** thorst has joined #openstack-neutron | 02:03 | |
*** vivek-eb_ has quit IRC | 02:04 | |
*** shoutm has quit IRC | 02:05 | |
*** abhiraut has quit IRC | 02:06 | |
*** madhu_ak has quit IRC | 02:06 | |
*** s3wong has quit IRC | 02:06 | |
*** yog_ has joined #openstack-neutron | 02:08 | |
*** armax has quit IRC | 02:13 | |
*** shoutm has joined #openstack-neutron | 02:13 | |
*** thorst has quit IRC | 02:16 | |
*** thorst has joined #openstack-neutron | 02:16 | |
*** thorst has left #openstack-neutron | 02:16 | |
*** saksham has quit IRC | 02:20 | |
*** stevemar_ has joined #openstack-neutron | 02:20 | |
*** clarkb1 has joined #openstack-neutron | 02:27 | |
*** zhenguo has quit IRC | 02:28 | |
*** baojg has quit IRC | 02:28 | |
*** stevemar has quit IRC | 02:28 | |
*** julien_ZTE has quit IRC | 02:28 | |
*** fnordahl has quit IRC | 02:28 | |
*** lennyb has quit IRC | 02:28 | |
*** clarkb has quit IRC | 02:28 | |
*** rtheis has joined #openstack-neutron | 02:29 | |
*** baojg has joined #openstack-neutron | 02:29 | |
*** lennyb has joined #openstack-neutron | 02:29 | |
*** crose has quit IRC | 02:31 | |
*** zhenguo has joined #openstack-neutron | 02:33 | |
*** lazy_prince has quit IRC | 02:34 | |
*** fnordahl has joined #openstack-neutron | 02:34 | |
*** julien_ZTE has joined #openstack-neutron | 02:34 | |
*** lazy_prince has joined #openstack-neutron | 02:34 | |
*** achanda has joined #openstack-neutron | 02:35 | |
*** bharathm has joined #openstack-neutron | 02:42 | |
*** shwetaap has quit IRC | 02:43 | |
*** gongysh has joined #openstack-neutron | 02:45 | |
*** jwy has quit IRC | 02:46 | |
*** bharathm has quit IRC | 02:47 | |
*** banix has quit IRC | 02:48 | |
*** ivar-lazzaro has joined #openstack-neutron | 02:49 | |
*** ferest has joined #openstack-neutron | 02:49 | |
*** shwetaap has joined #openstack-neutron | 02:50 | |
*** rtheis has quit IRC | 02:51 | |
*** lazy_prince has quit IRC | 02:51 | |
openstackgerrit | Yuji Azama proposed openstack/neutron: Add head option to neutron-db-manage command https://review.openstack.org/223901 | 02:52 |
*** zhhuabj has joined #openstack-neutron | 02:54 | |
*** mickeyspiegel has quit IRC | 02:54 | |
*** SourabhP has joined #openstack-neutron | 02:54 | |
*** julien_ZTE has quit IRC | 02:56 | |
*** nplanel has joined #openstack-neutron | 02:58 | |
*** SourabhP_ has joined #openstack-neutron | 02:58 | |
*** banix has joined #openstack-neutron | 02:59 | |
*** SourabhP has quit IRC | 03:00 | |
*** SourabhP_ is now known as SourabhP | 03:00 | |
*** blogan_ has quit IRC | 03:02 | |
openstackgerrit | fumihiko kakuma proposed openstack/neutron: Remove requirements.txt for the ofagent mechanism driver https://review.openstack.org/223903 | 03:04 |
*** baojg has quit IRC | 03:07 | |
*** azbiswas has joined #openstack-neutron | 03:10 | |
*** zzzeek has joined #openstack-neutron | 03:12 | |
*** ivar-lazzaro has quit IRC | 03:14 | |
*** ivar-lazzaro has joined #openstack-neutron | 03:14 | |
*** carora has joined #openstack-neutron | 03:15 | |
carora | Hi everyone, I have a question about neutron service | 03:15 |
carora | I am using juno and would like to know how can I create a dynamic domain name in all the vms created. | 03:16 |
*** zzzeek has quit IRC | 03:16 | |
carora | for example in resolve conf I want <tenant-name>.xyz (domain) | 03:16 |
*** fifieldt has joined #openstack-neutron | 03:18 | |
*** ivar-lazzaro has quit IRC | 03:18 | |
*** deepthi has joined #openstack-neutron | 03:21 | |
*** baojg has joined #openstack-neutron | 03:22 | |
*** ivar-lazzaro has joined #openstack-neutron | 03:22 | |
*** fitoduarte has joined #openstack-neutron | 03:23 | |
*** armax has joined #openstack-neutron | 03:23 | |
*** mragupat has joined #openstack-neutron | 03:24 | |
*** mragupat_ has joined #openstack-neutron | 03:25 | |
reedip | Dear All, during creation of Migration script, I am getting the following error. Can anyone suggest how to proceed forward? | 03:25 |
reedip | Command: neutron-db-manage revision -m "Description" --autogenerate | 03:26 |
reedip | Error: FAILED: Multiple heads are present; please specify the head revision on which the new revision should be based, or perform a merge. | 03:26 |
*** vivek-ebay has joined #openstack-neutron | 03:28 | |
*** mragupat has quit IRC | 03:29 | |
*** boris-42 has quit IRC | 03:30 | |
*** SumitNaiksatam has joined #openstack-neutron | 03:31 | |
*** vivek-eb_ has joined #openstack-neutron | 03:31 | |
*** vivek-ebay has quit IRC | 03:32 | |
*** tiswanso_ has quit IRC | 03:33 | |
openstackgerrit | gongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies https://review.openstack.org/217092 | 03:33 |
*** tfukushima has joined #openstack-neutron | 03:36 | |
*** banix has quit IRC | 03:37 | |
openstackgerrit | gongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies https://review.openstack.org/217092 | 03:41 |
*** david-lyle has joined #openstack-neutron | 03:41 | |
openstackgerrit | gongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies https://review.openstack.org/217092 | 03:45 |
*** shoutm has quit IRC | 03:48 | |
openstackgerrit | gongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies https://review.openstack.org/217092 | 03:48 |
*** shoutm_ has joined #openstack-neutron | 03:49 | |
*** boris-42 has joined #openstack-neutron | 03:49 | |
*** boris-42 has quit IRC | 03:49 | |
*** boris-42 has joined #openstack-neutron | 03:49 | |
*** ivar-laz_ has joined #openstack-neutron | 03:51 | |
*** ivar-laz_ has quit IRC | 03:51 | |
*** ivar-laz_ has joined #openstack-neutron | 03:52 | |
*** zhhuabj has quit IRC | 03:53 | |
*** ivar-lazzaro has quit IRC | 03:54 | |
*** tflynn_ has joined #openstack-neutron | 03:54 | |
*** amotoki has joined #openstack-neutron | 03:56 | |
*** ivar-laz_ has quit IRC | 03:56 | |
*** tflynn has quit IRC | 03:57 | |
*** minwang2 has quit IRC | 03:58 | |
*** iwamoto has joined #openstack-neutron | 03:58 | |
*** tflynn_ has quit IRC | 03:58 | |
*** carl_baldwin has joined #openstack-neutron | 03:59 | |
*** fawadkhaliq has joined #openstack-neutron | 04:00 | |
*** shivrao has joined #openstack-neutron | 04:01 | |
*** shivrao_ has joined #openstack-neutron | 04:03 | |
*** changbl has joined #openstack-neutron | 04:04 | |
*** shivrao has quit IRC | 04:05 | |
*** shivrao_ is now known as shivrao | 04:05 | |
*** sthillma has joined #openstack-neutron | 04:10 | |
*** baoli has quit IRC | 04:10 | |
*** shwetaap has quit IRC | 04:11 | |
*** tomoe_ has quit IRC | 04:12 | |
*** Marga_ has quit IRC | 04:16 | |
*** fawadkhaliq has quit IRC | 04:17 | |
*** rotbeard has joined #openstack-neutron | 04:19 | |
*** carora has quit IRC | 04:20 | |
*** arajagopal has quit IRC | 04:20 | |
*** arajagopal has joined #openstack-neutron | 04:21 | |
*** tfukushima has quit IRC | 04:21 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: ovsdb: Fix a few docstring https://review.openstack.org/223917 | 04:21 |
*** sridhar_ram has joined #openstack-neutron | 04:22 | |
*** fawadkhaliq has joined #openstack-neutron | 04:23 | |
*** ferest has quit IRC | 04:24 | |
*** arajagopal has quit IRC | 04:26 | |
*** coolsvap has joined #openstack-neutron | 04:29 | |
*** gberginc has joined #openstack-neutron | 04:30 | |
*** irenab has quit IRC | 04:30 | |
*** deepthi has quit IRC | 04:31 | |
*** sdake has joined #openstack-neutron | 04:33 | |
*** baojg has quit IRC | 04:33 | |
*** anilvenkata has joined #openstack-neutron | 04:34 | |
*** sdake_ has joined #openstack-neutron | 04:34 | |
*** yfried has quit IRC | 04:35 | |
*** jbell8 has quit IRC | 04:35 | |
*** stevemar_ has quit IRC | 04:38 | |
*** sdake has quit IRC | 04:38 | |
*** baojg has joined #openstack-neutron | 04:40 | |
*** david-lyle has quit IRC | 04:42 | |
*** arajagopal has joined #openstack-neutron | 04:42 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Always return iterables in L3 get_candidates https://review.openstack.org/222828 | 04:43 |
reedip | @ amotoki: , @hichihara: Hi there . I am facing the following problem in creating a migration script : | 04:44 |
reedip | reedip@reedip-VirtualBox:/opt/stack/neutron-fwaas/neutron_fwaas$ neutron-db-manage --subproject neutron-fwaas revision -m "description of revision" | 04:44 |
reedip | Running revision for neutron-fwaas ... | 04:44 |
reedip | FAILED: Multiple heads are present; please specify the head revision on which the new revision should be based, or perform a merge. | 04:44 |
*** SourabhP has quit IRC | 04:44 | |
reedip | Any idea in resolving the same? | 04:44 |
reedip | amotoki: , hichihara: Any idea in resolving the same? | 04:44 |
*** david-lyle has joined #openstack-neutron | 04:45 | |
*** jbell8 has joined #openstack-neutron | 04:45 | |
*** dannywilson has joined #openstack-neutron | 04:46 | |
*** achanda has quit IRC | 04:46 | |
*** deepthi has joined #openstack-neutron | 04:47 | |
hichihara | reedip: I'm not db guy but let's change "description of revision". | 04:47 |
*** fitoduarte has quit IRC | 04:48 | |
reedip | I changed it but its not helping out | 04:49 |
*** carl_baldwin has quit IRC | 04:49 | |
*** tomoe_ has joined #openstack-neutron | 04:50 | |
amotoki | reedip: hichihara: reading alembic docs | 04:50 |
hichihara | reedip: Did you try to use option "--autogenerate"? | 04:51 |
reedip | @amotoki, @hichihara: Yes, I did | 04:51 |
*** achanda has joined #openstack-neutron | 04:51 | |
*** fedexo has joined #openstack-neutron | 04:52 | |
*** tflynn has joined #openstack-neutron | 04:52 | |
*** deepthi has quit IRC | 04:54 | |
*** ferest has joined #openstack-neutron | 04:54 | |
*** tflynn_ has joined #openstack-neutron | 04:54 | |
*** fitoduarte has joined #openstack-neutron | 04:55 | |
*** sdake_ has quit IRC | 04:56 | |
reedip | amotoki:, hichihara: Something I found while looking at the various operation | 04:56 |
reedip | Amotoki:, Hichihara: reedip@reedip-VirtualBox:/opt/stack/neutron-fwaas/neutron_fwaas$ neutron-db-manage --subproject neutron-fwaas check_migration | 04:56 |
reedip | Running branches for neutron-fwaas ... | 04:56 |
reedip | kilo (branchpoint) | 04:56 |
reedip | -> c40fbb377ad (expand) | 04:56 |
reedip | -> 67c8e8d61d5 (contract) (head) | 04:56 |
reedip | OK | 04:56 |
reedip | FAILED: HEADS file does not match migration timeline heads, expected: 4b47ea298795, 67c8e8d61d5 | 04:56 |
reedip | Amotoki:, Hichihara: Seems some migration is not yet complete (??) | 04:57 |
*** tflynn has quit IRC | 04:57 | |
amotoki | reedip: hmm... something seems wrong | 04:57 |
*** fawadkhaliq has quit IRC | 04:57 | |
*** ferest has quit IRC | 04:58 | |
amotoki | reedip: checking fwaas db migratio history.. | 04:59 |
*** amit213 has quit IRC | 04:59 | |
*** sdake has joined #openstack-neutron | 04:59 | |
*** amit213 has joined #openstack-neutron | 04:59 | |
reedip | amotoki: Ok. I have it with me by the way. | 04:59 |
amotoki | reedip: I succeeded for neutron-vpnaas: neutron-db-manage --subproject neutron-vpnaas check_migration | 04:59 |
reedip | amotoki: reedip@reedip-VirtualBox:/opt/stack/neutron-fwaas/neutron_fwaas$ neutron-db-manage --subproject neutron-fwaas history | 05:00 |
reedip | Running history for neutron-fwaas ... | 05:00 |
reedip | c40fbb377ad -> 4b47ea298795 (expand) (head), add reject rule | 05:00 |
reedip | kilo -> c40fbb377ad (expand), Initial Liberty no-op script. | 05:00 |
amotoki | reedip: for neutron-fwaas, it should succeed | 05:00 |
reedip | kilo -> 67c8e8d61d5 (contract) (head), Initial Liberty no-op script. | 05:00 |
reedip | 796c68dffbb -> kilo (branchpoint), kilo | 05:00 |
reedip | 540142f314f4 -> 796c68dffbb, cisco_csr_fwaas | 05:00 |
reedip | 4202e3047e47 -> 540142f314f4, FWaaS router insertion | 05:00 |
reedip | start_neutron_fwaas -> 4202e3047e47, add_index_tenant_id | 05:00 |
reedip | <base> -> start_neutron_fwaas, start neutron-fwaas chain | 05:00 |
reedip | OK | 05:00 |
reedip | amotoki: Oh ok.. So I should contact the owner of the bug 'add reject rule' ( HEAD ID: 4b47ea298795) for this ? | 05:01 |
amotoki | reedip: let me check for a while | 05:01 |
*** hitalia has joined #openstack-neutron | 05:02 | |
reedip | amotoki: Ok | 05:03 |
*** mohankumar has joined #openstack-neutron | 05:03 | |
*** neeti has joined #openstack-neutron | 05:04 | |
*** saksham has joined #openstack-neutron | 05:05 | |
*** sridhar_ram1 has joined #openstack-neutron | 05:05 | |
*** sridhar_ram has quit IRC | 05:07 | |
*** saksham_ has joined #openstack-neutron | 05:07 | |
*** ekuris has joined #openstack-neutron | 05:07 | |
*** adduarte has joined #openstack-neutron | 05:09 | |
amotoki | reedip: I found the contents of HEADS file is not correct. | 05:10 |
amotoki | reedip: I am preparing a patch. | 05:10 |
*** sdake_ has joined #openstack-neutron | 05:10 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Deprecate drop_flows_on_start https://review.openstack.org/223921 | 05:10 |
*** saksham has quit IRC | 05:10 | |
reedip | @amotoki: Oh ok. Great Thanks :) | 05:10 |
amotoki | reedip: you can replace 4b47ea29879 with 4b47ea298795 in HEADS file | 05:11 |
*** Sukhdev_ has joined #openstack-neutron | 05:11 | |
*** fitoduarte has quit IRC | 05:12 | |
reedip | Yes I can | 05:12 |
*** nlahouti has joined #openstack-neutron | 05:12 | |
reedip | should I do it? | 05:12 |
*** arajagopal has quit IRC | 05:12 | |
Sukhdev_ | Sukhdev_:hi | 05:13 |
*** arajagopal has joined #openstack-neutron | 05:13 | |
amotoki | reedip: I am preparing a patch, but as a workaround you can go forward your wokr ahead by changing HEADS file. | 05:13 |
reedip | I changed the HEADS file | 05:13 |
Sukhdev_ | amotoki: Ping | 05:13 |
amotoki | Sukhdev_: pong | 05:13 |
reedip | but it didnot change the 'check_migration' output | 05:14 |
Sukhdev_ | amotoki: We have been missing you in neutron_ironic meetings | 05:14 |
*** sdake has quit IRC | 05:14 | |
*** nlahouti1 has joined #openstack-neutron | 05:14 | |
amotoki | Sukhdev_: really sorry _o_ | 05:14 |
Sukhdev_ | amotoki: Is everything OK? | 05:14 |
*** saksham_ has quit IRC | 05:14 | |
amotoki | Sukhdev_: fine. I need to have spent busy days to run OpenStack cloud too... | 05:15 |
amotoki | reedip: still checking | 05:15 |
HenryG | amotoki: reedip: can I help with fwaas alembic migrations? | 05:16 |
openstackgerrit | Merged openstack/neutron: Prevent full sync in dhcp_agent when possible https://review.openstack.org/222546 | 05:16 |
amotoki | HenryG: great | 05:16 |
openstackgerrit | Merged openstack/neutron: Prevent full sync in dhcp_agent when possible https://review.openstack.org/222546 | 05:16 |
reedip | @HenryG : sure :) | 05:16 |
Sukhdev_ | amotoki: Are you still planning on pushing that documentation patch? | 05:16 |
*** nlahouti has quit IRC | 05:16 | |
amotoki | Sukhdev_: I might do it, but if someone is interested in it, please take it. | 05:17 |
amotoki | Sukhdev_: I should review it instead. | 05:17 |
*** yuanying_ has joined #openstack-neutron | 05:17 | |
Sukhdev_ | amotoki: can you give me a pointer as to what to modify, I will take a stab at it and then you can review it | 05:18 |
*** arajagopal has quit IRC | 05:18 | |
HenryG | reedip: what is the issue? | 05:18 |
amotoki | HenryG: reedip: I just filed a bug: https://bugs.launchpad.net/neutron/+bug/1496239 | 05:18 |
openstack | Launchpad bug 1496239 in neutron "neutron-fwaas check_migartion fails" [High,New] - Assigned to Akihiro Motoki (amotoki) | 05:18 |
*** yuanying has quit IRC | 05:19 | |
*** yuanying_ is now known as yuanying | 05:19 | |
*** itzikb has joined #openstack-neutron | 05:19 | |
reedip | @amotoki: yes, thats bang on | 05:20 |
amotoki | HenryG: I see the latest fwaas expand migration 4b47ea298795_add_reject_rule.py has 11 chars revision | 05:20 |
reedip | @AMotoki, HenryG: I did an experiment | 05:21 |
amotoki | HenryG: and HEAD contains the first 10 chars. Even after changing HEADS to 11 char version, check_migration is still failing | 05:21 |
*** tfukushima has joined #openstack-neutron | 05:21 | |
*** fawadkhaliq has joined #openstack-neutron | 05:22 | |
HenryG | amotoki: Do you count from 0? :) | 05:23 |
HenryG | amotoki: I get 12 and 11 | 05:23 |
*** obondarev has joined #openstack-neutron | 05:23 | |
reedip | HenryG: Yes its 12 and 11 :) | 05:23 |
HenryG | amotoki: But you may be right, that this is an alembic restriction | 05:23 |
*** lazy_prince has joined #openstack-neutron | 05:24 | |
HenryG | amotoki: Looks like we need to drop a char from the revision and file name | 05:24 |
*** mragupat_ has quit IRC | 05:24 | |
amotoki | HenryG: you are right. 12 and 11 | 05:24 |
*** ferest has joined #openstack-neutron | 05:25 | |
reedip | HenryG:, amotoki: checking something here | 05:25 |
reedip | In the versions folder , following files are listed: | 05:25 |
reedip | 4202e3047e47_add_index_tenant_id.py 540142f314f4_fwaas_router_insertion.py 796c68dffbb_cisco_csr_fwaas.py | 05:25 |
reedip | Each has 12 characters | 05:25 |
amotoki | HenryG: but don't we need to consider deployments which already uses 12 char revision? | 05:25 |
*** irenab has joined #openstack-neutron | 05:25 | |
reedip | @amotoki :, @henryG: Each has 12 characters | 05:26 |
*** lajos-katona has joined #openstack-neutron | 05:26 | |
HenryG | reedip: interesting | 05:26 |
reedip | @amotoki: , @henryG: only the expand and contract have 11 characters for the initial files | 05:26 |
*** tryit has joined #openstack-neutron | 05:26 | |
*** tfukushima has quit IRC | 05:26 | |
*** baojg has quit IRC | 05:28 | |
sc68cal | amotoki: I just hopped on because I saw the bug you opened | 05:28 |
*** ferest has quit IRC | 05:29 | |
sc68cal | amotoki: i'll probably be up for another 30 mins or so, if I can help | 05:29 |
HenryG | Right. alembic branches thinks c40fbb377ad is the expand head | 05:29 |
amotoki | sc68cal: thanks. we continues the discussion here | 05:30 |
*** gberginc has quit IRC | 05:30 | |
*** kengo_sakai has joined #openstack-neutron | 05:31 | |
amotoki | It seems alembic itself has no limitation the length of revisions. | 05:32 |
*** sridhar_ram1 has quit IRC | 05:32 | |
amotoki | it seems others except check_migration is working well. | 05:33 |
*** sbalukoff has quit IRC | 05:38 | |
amotoki | HenryG: When I inserted a debug message, I got: observered_heads: ['67c8e8d61d5', '4b47ea298795'] expected_heads: ['4b47ea298795', '67c8e8d61d5'] | 05:39 |
*** sbalukoff has joined #openstack-neutron | 05:40 | |
amotoki | HenryG: it just looks a sorting problem. | 05:40 |
*** baoli has joined #openstack-neutron | 05:41 | |
*** agireud has quit IRC | 05:41 | |
*** trinaths has joined #openstack-neutron | 05:41 | |
*** agireud has joined #openstack-neutron | 05:41 | |
reedip | amotoki: I tried run the same on neutron | 05:41 |
reedip | amotoki: this is my observed output | 05:42 |
*** nlahouti1 has quit IRC | 05:42 | |
reedip | amotoki: reedip@reedip-VirtualBox:/opt/stack/neutron/neutron/db/migration/alembic_migrations/versions$ cat HEADS | 05:42 |
reedip | 34af2b5c5a59 | 05:42 |
reedip | 4af11ca47297 | 05:42 |
*** nlahouti has joined #openstack-neutron | 05:42 | |
reedip | amotoki: reedip@reedip-VirtualBox:/opt/stack/neutron/neutron/db/migration/alembic_migrations/versions$ neutron-db-manage --subproject neutron check_migration | 05:43 |
reedip | Running branches for neutron ... | 05:43 |
reedip | kilo (branchpoint) | 05:43 |
reedip | -> 354db87e3225 (expand) | 05:43 |
reedip | -> 30018084ec99 (contract) | 05:43 |
reedip | OK | 05:43 |
*** shoutm_ has quit IRC | 05:44 | |
amotoki | the first one is expand head and the secodn one is contract head. | 05:44 |
amotoki | When I swapped the two line in neutron-fwaas, it succeeded. | 05:44 |
reedip | amotoki: Ok | 05:45 |
-amotoki- $ git diff | cat | 05:45 | |
-amotoki- diff --git a/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS b/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS | 05:45 | |
-amotoki- index 5d2dc8a..ad565a8 100644 | 05:45 | |
-amotoki- --- a/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS | 05:45 | |
-amotoki- +++ b/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS | 05:45 | |
-amotoki- @@ -1,2 +1,2 @@ | 05:45 | |
-amotoki- +4b47ea298795 | 05:45 | |
-amotoki- 67c8e8d61d5 | 05:45 | |
-amotoki- -4b47ea29879 | 05:45 | |
-amotoki- $ neutron-db-manage --subproject neutron-fwaas check_migration | 05:45 | |
-amotoki- Running branches for neutron-fwaas ... | 05:45 | |
-amotoki- kilo (branchpoint) | 05:45 | |
-amotoki- -> c40fbb377ad (expand) | 05:45 | |
-amotoki- -> 67c8e8d61d5 (contract) (head) | 05:45 | |
-amotoki- OK | 05:45 | |
amotoki | I am checking the detail. | 05:45 |
reedip | amotoki: yes it worked | 05:45 |
*** arajagopal has joined #openstack-neutron | 05:45 | |
*** baoli has quit IRC | 05:46 | |
HenryG | amotoki: I thought ihar had fixed that | 05:47 |
HenryG | Looks like not | 05:47 |
amotoki | I thought so too. It looks better to wait Ihar for hours. | 05:48 |
reedip | amotoki: henryg: I changed the HEADS. Should the migration script work now? | 05:48 |
HenryG | amotoki: I think the expectation is that cli.update_heads_file should generate the HEADS sorted | 05:50 |
amotoki | reedip: not sure. | 05:50 |
amotoki | HenryG: the original problem reedip has is that how to generate a new revision file. | 05:50 |
amotoki | he tried neutron-db-manage --subproject neutron-fwaas revision -m 'xxxxx' | 05:50 |
reedip | amotoki: Its not :( It is giving the same error. I think I have to wait for your commit (??) | 05:50 |
amotoki | but it failed with FAILED: Multiple heads are present; please specify the head revision on which the new revision should be based, or perform a merge. | 05:51 |
HenryG | amotoki: yes, the 'revision' command for subprojects needs work :( | 05:51 |
reedip | @amotoki:, @henryg : so how should I proceed? | 05:52 |
*** tfukushima has joined #openstack-neutron | 05:52 | |
*** tfukushima has quit IRC | 05:53 | |
*** tfukushima has joined #openstack-neutron | 05:53 | |
*** fawadkhaliq has quit IRC | 05:55 | |
*** fawadk has joined #openstack-neutron | 05:55 | |
*** yfried has joined #openstack-neutron | 05:55 | |
amotoki | reedip: "revision" command seems not to work now for fwaas. You can prepare it manually and test it. | 05:55 |
*** carora has joined #openstack-neutron | 05:55 | |
*** tflynn_ has quit IRC | 05:55 | |
HenryG | amotoki: Patch https://review.openstack.org/148416 created HEADS in the wrong order | 05:55 |
reedip | @Amotoki : @henryG: Ok, I will create it manually | 05:55 |
*** ferest has joined #openstack-neutron | 05:56 | |
reedip | @Amotoki, @HenryG: do I need to modify the HEADS file as well , or should I ignore it? | 05:56 |
amotoki | reedip: you need to update HEADS too | 05:56 |
HenryG | reedip: and make sure it is sorted | 05:56 |
amotoki | HenryG: what is the expected order of HEADS files? | 05:56 |
HenryG | amotoki: sorted(HEADS) | 05:57 |
reedip | @Amotoki: @HenryG: and what about the random number for the file name ?? | 05:57 |
HenryG | basically | 05:57 |
amotoki | HenryG: alphabetical order? | 05:57 |
reedip | @amotoki: @henryg: how do I create it ?? :D | 05:57 |
reedip | @amotoki: @henryg: Just an Idea. For the HEADS file, I can run check_migration to see if the result is correct or not :) | 05:58 |
HenryG | reedip: sure | 05:58 |
HenryG | reedip: the new revision number is any random hex string not used before | 05:59 |
reedip | @HenryG: and for the random number | 05:59 |
amotoki | reedip: in addition ot check_migration, you need to test it against real database. | 05:59 |
reedip | @henryG: how do I generate that? | 05:59 |
*** yfried has quit IRC | 05:59 | |
*** yfried_ has joined #openstack-neutron | 05:59 | |
reedip | @amotoki: how to check against a real database? | 06:00 |
amotoki | reedip: check_migration just check if migration sequence is valid and head revisions match HEADS file. | 06:00 |
HenryG | reedip: Here is one: 8adf9d10c57 | 06:00 |
*** ferest has quit IRC | 06:00 | |
reedip | @amotoki : Ok .@HenryG : How did u create it ??? :D Please do tell. It would help me out if this issue reoccurs with some other patch | 06:00 |
HenryG | reedip: I ran "neutron-db-manage revision -m foo --autogenerate" on neutron core | 06:01 |
HenryG | reedip: It gave me an emptry script | 06:01 |
reedip | @HenryG : Oh thats a great way..... almost a hack :) | 06:01 |
reedip | @HenryG : Thanks a lot for your support. | 06:02 |
*** zhhuabj has joined #openstack-neutron | 06:02 | |
amotoki | reedip: what i usually do is to prepare a new database named "neutron" on mysql and run "neutron-db-manage upgrade heads". | 06:02 |
amotoki | reedip: let me test. | 06:03 |
reedip | @Amotoki: Wont that also work with launching devstack ? | 06:03 |
*** obondarev has quit IRC | 06:03 | |
amotoki | reedip: you need to unstack first. | 06:03 |
reedip | @Amotoki: Yes, thats part of my launch script :) | 06:04 |
HenryG | amotoki: reedip: I am going to bed now. If you need more help, akamyshnikova and Ihar will be online soon I think. | 06:04 |
amotoki | HenryG: thanks for your help. good night! | 06:04 |
amotoki | HenryG: I am catching up with the changes around db migration improvements now. | 06:05 |
*** obondarev has joined #openstack-neutron | 06:05 | |
reedip | @HenryG: Goodnight, and thank you very much for your help :) | 06:05 |
*** EinstCrazy has joined #openstack-neutron | 06:06 | |
reedip | @Amotoki: If you change the HEAD file , wont it conflict when I change my HEAD file while commiting to the master? | 06:06 |
*** kiran-r has joined #openstack-neutron | 06:06 | |
*** hitalia has quit IRC | 06:07 | |
amotoki | reedip: there wil be conflict temporarily, but you can address conflicts by rebasing if another patch is merged. | 06:07 |
amotoki | reedip: it is not a rare issue. we are developing in parallel :-) | 06:07 |
reedip | @amotoki: Yes I know :) | 06:07 |
reedip | @amotoki: Ok, i will prepare my patch now | 06:08 |
reedip | @amotoki: Thanks a lot for your help. | 06:08 |
reedip | :) | 06:08 |
amotoki | reedip: good luch | 06:08 |
amotoki | luck | 06:08 |
reedip | @Amotoki: Thanks.... definitely need it :) | 06:09 |
*** _cjones_ has joined #openstack-neutron | 06:09 | |
*** _cjones_ has quit IRC | 06:09 | |
*** _cjones_ has joined #openstack-neutron | 06:10 | |
*** diga has joined #openstack-neutron | 06:11 | |
*** obondarev has quit IRC | 06:11 | |
*** shoutm has joined #openstack-neutron | 06:12 | |
*** mkolesni has joined #openstack-neutron | 06:14 | |
*** numans has joined #openstack-neutron | 06:15 | |
*** trinaths has left #openstack-neutron | 06:15 | |
*** jbell8 has quit IRC | 06:18 | |
*** fedexo has quit IRC | 06:19 | |
amotoki | reedip: This is what I do to test db miration against real database table. http://paste.openstack.org/show/464133/ | 06:20 |
amotoki | reedip: After doing so, you can check detail schemas using mysql command or something. | 06:20 |
*** obondarev has joined #openstack-neutron | 06:20 | |
reedip | @Amotoki: You have been really helpful :) I will proceed with the given information | 06:21 |
openstackgerrit | Akihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error https://review.openstack.org/223935 | 06:24 |
*** kiran-r has quit IRC | 06:24 | |
*** markvoelker has quit IRC | 06:25 | |
*** shashank_hegde has joined #openstack-neutron | 06:26 | |
*** ferest has joined #openstack-neutron | 06:26 | |
*** ferest has quit IRC | 06:31 | |
openstackgerrit | Akihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error https://review.openstack.org/223935 | 06:33 |
amotoki | reedip: ^^ | 06:33 |
*** bharathm has joined #openstack-neutron | 06:34 | |
*** ildikov has joined #openstack-neutron | 06:36 | |
*** moshele has joined #openstack-neutron | 06:37 | |
*** obondarev has quit IRC | 06:37 | |
*** stevemar has joined #openstack-neutron | 06:38 | |
*** scheuran has joined #openstack-neutron | 06:39 | |
*** drjones has joined #openstack-neutron | 06:40 | |
*** stevemar has quit IRC | 06:42 | |
*** Murali has joined #openstack-neutron | 06:42 | |
openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 06:42 |
*** _cjones_ has quit IRC | 06:44 | |
*** nyechiel_ has joined #openstack-neutron | 06:46 | |
*** sthillma has quit IRC | 06:47 | |
*** kawa2014 has joined #openstack-neutron | 06:47 | |
*** yfried_ is now known as yfried|afk | 06:49 | |
*** abregman has joined #openstack-neutron | 06:50 | |
*** shivrao has quit IRC | 06:51 | |
*** shashank_hegde has quit IRC | 06:51 | |
*** nlahouti has quit IRC | 06:53 | |
*** ronis has joined #openstack-neutron | 06:53 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB https://review.openstack.org/183369 | 06:53 |
*** k-yasui has joined #openstack-neutron | 06:55 | |
*** fzdarsky_ has joined #openstack-neutron | 06:56 | |
*** yfried|afk is now known as yfried_ | 06:57 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB https://review.openstack.org/183369 | 06:57 |
*** sbelous has joined #openstack-neutron | 07:00 | |
*** vivek-ebay has joined #openstack-neutron | 07:00 | |
*** sbelous has quit IRC | 07:00 | |
*** marios has joined #openstack-neutron | 07:01 | |
*** marios has quit IRC | 07:01 | |
*** marios has joined #openstack-neutron | 07:01 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler https://review.openstack.org/204436 | 07:01 |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add functional test for availability_zone support https://review.openstack.org/210977 | 07:02 |
*** gampel has joined #openstack-neutron | 07:02 | |
*** tmorin has joined #openstack-neutron | 07:02 | |
*** ihrachys has joined #openstack-neutron | 07:03 | |
*** SridharG has joined #openstack-neutron | 07:03 | |
*** vivek-eb_ has quit IRC | 07:04 | |
*** gildub has quit IRC | 07:04 | |
*** rkukura_ has joined #openstack-neutron | 07:05 | |
*** achanda has quit IRC | 07:06 | |
*** nikil89_ has joined #openstack-neutron | 07:06 | |
*** rkukura has quit IRC | 07:08 | |
*** rkukura_ is now known as rkukura | 07:08 | |
*** nyechiel has joined #openstack-neutron | 07:09 | |
carora | I was wondering if someone here could help me in getting dynamic domain in resolv conf. I want my vms to show domain search <tenant-name>.xyz (domain) in resolv conf | 07:11 |
carora | which file do I need to change. | 07:11 |
*** SumitNaiksatam has quit IRC | 07:11 | |
*** SumitNaiksatam has joined #openstack-neutron | 07:12 | |
*** alex_klimov has joined #openstack-neutron | 07:12 | |
*** armax has quit IRC | 07:12 | |
openstackgerrit | Yuji Azama proposed openstack/neutron: Add head option to neutron-db-manage command https://review.openstack.org/223901 | 07:13 |
*** ihrachys has quit IRC | 07:14 | |
*** yamahata has quit IRC | 07:14 | |
shihanzhang | hi all, who can help to review this patch https://review.openstack.org/#/c/221209/ | 07:14 |
*** anshul has joined #openstack-neutron | 07:15 | |
*** matrohon has joined #openstack-neutron | 07:16 | |
*** nyechiel_ has quit IRC | 07:19 | |
*** moshele has quit IRC | 07:19 | |
*** fawadk has quit IRC | 07:19 | |
*** boris-42 has quit IRC | 07:19 | |
*** fnordahl has quit IRC | 07:19 | |
*** marios has quit IRC | 07:19 | |
*** yuanying has quit IRC | 07:19 | |
*** azbiswas has quit IRC | 07:19 | |
*** zhenguo has quit IRC | 07:19 | |
*** slaweq has joined #openstack-neutron | 07:19 | |
*** jpena has joined #openstack-neutron | 07:21 | |
*** hoangcx has quit IRC | 07:23 | |
*** mohankumar has quit IRC | 07:23 | |
*** abehl has joined #openstack-neutron | 07:24 | |
*** marios has joined #openstack-neutron | 07:24 | |
*** yuanying has joined #openstack-neutron | 07:24 | |
*** azbiswas has joined #openstack-neutron | 07:24 | |
*** zhenguo has joined #openstack-neutron | 07:24 | |
*** abehl has quit IRC | 07:24 | |
*** abehl has joined #openstack-neutron | 07:24 | |
*** moshele has joined #openstack-neutron | 07:25 | |
*** fawadk has joined #openstack-neutron | 07:25 | |
*** boris-42 has joined #openstack-neutron | 07:25 | |
*** fnordahl has joined #openstack-neutron | 07:25 | |
*** markvoelker has joined #openstack-neutron | 07:26 | |
*** EinstCrazy has quit IRC | 07:27 | |
*** ferest has joined #openstack-neutron | 07:28 | |
*** Sukhdev_ has quit IRC | 07:30 | |
*** dannywilson has quit IRC | 07:30 | |
*** markvoelker has quit IRC | 07:30 | |
*** evgenyf has joined #openstack-neutron | 07:31 | |
*** luqas has joined #openstack-neutron | 07:31 | |
*** abehl has quit IRC | 07:33 | |
*** ferest has quit IRC | 07:33 | |
*** carora has quit IRC | 07:34 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: Fix establishing UDP connection https://review.openstack.org/212425 | 07:35 |
*** luqas has quit IRC | 07:35 | |
*** [1]evgenyf has joined #openstack-neutron | 07:35 | |
*** evgenyf has quit IRC | 07:38 | |
*** [1]evgenyf is now known as evgenyf | 07:38 | |
*** hanzhang has quit IRC | 07:40 | |
*** obondarev has joined #openstack-neutron | 07:40 | |
*** ushkalim has joined #openstack-neutron | 07:40 | |
*** jlanoux has joined #openstack-neutron | 07:40 | |
*** shz has joined #openstack-neutron | 07:41 | |
*** vivek-ebay has quit IRC | 07:41 | |
openstackgerrit | Reedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn https://review.openstack.org/223951 | 07:42 |
*** zhhuabj has quit IRC | 07:44 | |
*** diga has quit IRC | 07:44 | |
*** zhhuabj has joined #openstack-neutron | 07:53 | |
*** rossella_s has joined #openstack-neutron | 07:54 | |
*** drjones has quit IRC | 07:57 | |
*** _cjones_ has joined #openstack-neutron | 07:58 | |
*** fawadkhaliq has joined #openstack-neutron | 07:58 | |
*** ferest has joined #openstack-neutron | 07:59 | |
*** fawadk has quit IRC | 07:59 | |
*** nmagnezi has joined #openstack-neutron | 08:00 | |
*** ihrachys has joined #openstack-neutron | 08:00 | |
*** ushkalim has quit IRC | 08:01 | |
*** safchain has joined #openstack-neutron | 08:01 | |
*** fawadkhaliq has quit IRC | 08:02 | |
*** ferest has quit IRC | 08:04 | |
*** pece has joined #openstack-neutron | 08:04 | |
*** thanhnt-z_ has joined #openstack-neutron | 08:05 | |
*** thanhnt-z_ has quit IRC | 08:06 | |
*** jbell8 has joined #openstack-neutron | 08:07 | |
*** gampel1 has joined #openstack-neutron | 08:08 | |
reedip | @amotoki : Reedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn https://review.openstack.org/223951 | 08:08 |
reedip | :) | 08:08 |
apuimedo | gsagie: I believe this one could be considered as "fix released" https://bugs.launchpad.net/kuryr/+bug/1492882 | 08:08 |
openstack | Launchpad bug 1492882 in kuryr "Avoid cloning of Neutron repository in CI process" [Undecided,Fix committed] - Assigned to Gal Sagie (gal-sagie) | 08:08 |
apuimedo | since it is about testing | 08:08 |
*** nmagnezi has quit IRC | 08:08 | |
apuimedo | and it was already merged | 08:08 |
*** kiran-r has joined #openstack-neutron | 08:09 | |
*** deepthi has joined #openstack-neutron | 08:09 | |
*** gampel has quit IRC | 08:10 | |
*** nmagnezi has joined #openstack-neutron | 08:10 | |
*** mickeys has joined #openstack-neutron | 08:10 | |
*** diga has joined #openstack-neutron | 08:11 | |
*** sdake has joined #openstack-neutron | 08:11 | |
*** gberginc has joined #openstack-neutron | 08:11 | |
*** hoangcx has joined #openstack-neutron | 08:11 | |
*** jbell8 has quit IRC | 08:12 | |
*** sdake_ has quit IRC | 08:12 | |
amotoki | reedip: question: does it work for postgresql? | 08:13 |
itzikb | moshele: lennyb: Have you checked the case with the pre-existing port? | 08:13 |
*** thanhnt-z has joined #openstack-neutron | 08:13 | |
*** rotbeard has quit IRC | 08:14 | |
reedip | @amotoki: Postgresql already has case sensitive columns | 08:15 |
*** yog_ has quit IRC | 08:15 | |
reedip | @amotoki: I found this information after googling. http://nandovieira.com/using-insensitive-case-columns-in-postgresql-with-citext | 08:16 |
*** mickeys has quit IRC | 08:17 | |
*** ushkalim has joined #openstack-neutron | 08:17 | |
amotoki | reedip: thanks, it would be helpful for all reviewers if you add a comment in the review or in th e commit message. | 08:18 |
reedip | @amotoki: Problem is mysql is case insensitive | 08:18 |
thanhnt-z | hi obondarev, I understand that, "device_id" of a dhcp port always start with 'dhcp'. For example: dhcp60ca53ad-97f1-51c6-86cf-94786eb89592-ae9af78f-6c1d-424c-98b8-4d201d931b22. Could you please help me to confirm about that? it's right or wrong? | 08:19 |
reedip | @Amotoki: I have added the information is specific to mysql. I will mention in the gerrit review page that postgresql is case sensitive so there is no such impact on it. Wil that be ok? ( or should I change my commit message?) | 08:19 |
*** jistr has joined #openstack-neutron | 08:19 | |
amotoki | reedip: We need to make sure that db migration works both for mysql and postgresql, so reviewers need to care if it affects postgresql or not. | 08:22 |
amotoki | reedip: I know this is specific to mysql, but we need to know/check it doesn't affect postgresql. so I think some comment on postgresql would be helpful. | 08:22 |
amotoki | reedip: does it make sense ? | 08:23 |
gsagie | apuimedo: done | 08:24 |
reedip | @amotoki: Yes, it does. I will update the review comment itself | 08:27 |
*** ygbo has joined #openstack-neutron | 08:27 | |
reedip | I mean the commit message | 08:27 |
diga | gsagie: Hi, can you look at this change - https://review.openstack.org/#/c/221640/ | 08:27 |
SridharG | kevinbenton: Hello, Can you please take a look at the patch when you get sometime. I have updated the patch to include the necessary changes for native implementation. Thank you - https://review.openstack.org/#/c/201650/ | 08:29 |
*** deepthi has quit IRC | 08:29 | |
*** ferest has joined #openstack-neutron | 08:30 | |
openstackgerrit | Reedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn https://review.openstack.org/223951 | 08:30 |
apuimedo | gsagie: thanks | 08:30 |
apuimedo | diga: hi | 08:30 |
reedip | @amotoki: Done | 08:30 |
diga | Hi | 08:31 |
*** moshele has quit IRC | 08:31 | |
diga | apuimedo: hi | 08:31 |
diga | apuimedo: have you seen the mohammed's comments | 08:31 |
*** moshele has joined #openstack-neutron | 08:32 | |
*** itzikb_ has joined #openstack-neutron | 08:32 | |
*** ekuris has quit IRC | 08:32 | |
*** shoutm has quit IRC | 08:33 | |
diga | apuimedo: I have given a reply to that | 08:34 |
*** itzikb has quit IRC | 08:34 | |
*** ferest has quit IRC | 08:34 | |
*** alexpilotti has joined #openstack-neutron | 08:35 | |
*** mohankumar has joined #openstack-neutron | 08:37 | |
gsagie | diga: will take a look now | 08:38 |
diga | gsagie: sure | 08:39 |
*** SourabhP has joined #openstack-neutron | 08:40 | |
gsagie | diga: i think you need to work with banix as you two have both implemented configuration parts: https://review.openstack.org/#/c/222960/ and https://review.openstack.org/#/c/221640/ | 08:40 |
gsagie | Are you able to catch him online and solve this as he still has -1 on your patch | 08:41 |
gsagie | ? | 08:41 |
gsagie | i can try to talk with him for you if you are unable due to time zones difference | 08:41 |
apuimedo | gsagie: diga: banix patch seems to have more things. I like its direction. Maybe diga can address banix comments and banix can put the extra stuff in a follow-up patch | 08:42 |
apuimedo | gsagie: what do you think about that? | 08:42 |
gsagie | apuimedo: sounds good to me | 08:42 |
gsagie | diga: what do you think? | 08:42 |
diga | gsagie: apuimedo : I will address comments given by banix | 08:43 |
*** obondarev has quit IRC | 08:43 | |
diga | either or is fine for me | 08:43 |
diga | if his patch is good to go then I will move my VIF work | 08:43 |
diga | apuimedo: gsagie : I dont mind | 08:44 |
*** _cjones_ has quit IRC | 08:45 | |
apuimedo | diga: gsagie: so maybe better like that. We take banix's patch as soon as he addresses the comment this afternoon | 08:45 |
apuimedo | so diga can now work on the binding stuff | 08:46 |
apuimedo | right now | 08:46 |
gsagie | diga: just make sure that he addressed everything you need for the binding | 08:46 |
gsagie | configuration wise | 08:46 |
apuimedo | gsagie: good point | 08:46 |
diga | apuimedo: gsagie : Fine! | 08:46 |
diga | gsagie: let me review his patch, then will give comment what I need extra in config for VIF | 08:47 |
*** ushkalim has quit IRC | 08:47 | |
*** ekuris has joined #openstack-neutron | 08:48 | |
*** ekuris has quit IRC | 08:48 | |
*** ekuris has joined #openstack-neutron | 08:48 | |
apuimedo | diga: gsagie: or maybe better just add the extra things as a patch that depends on banix's one | 08:49 |
apuimedo | it may be faster | 08:49 |
apuimedo | (and more correct) | 08:49 |
irenab | apuimedo: +1 | 08:49 |
diga | apuimedo: that;s good | 08:50 |
apuimedo | diga: thanks. So then abandon the config patch and do it like we just agreed :-) | 08:50 |
diga | yes | 08:50 |
diga | apuimedo: abandoned :) | 08:52 |
apuimedo | diga: thanks! | 08:52 |
*** kbyrne has joined #openstack-neutron | 08:53 | |
gsagie | thanks diga | 08:55 |
diga | apuimedo: gsagie : welcome! | 08:55 |
diga | apuimedo: gsagie : we have decided 2377 port for kuryr, right ? | 08:56 |
*** l6unchpad has joined #openstack-neutron | 08:57 | |
*** hichihara has quit IRC | 08:58 | |
*** hdaniel has joined #openstack-neutron | 08:59 | |
*** obondarev has joined #openstack-neutron | 08:59 | |
openstackgerrit | shihanzhang proposed openstack/neutron: DVR: notify specific agent when create/update port https://review.openstack.org/221209 | 08:59 |
*** luqas has joined #openstack-neutron | 09:00 | |
*** ushkalim has joined #openstack-neutron | 09:00 | |
*** ferest has joined #openstack-neutron | 09:01 | |
thanhnt-z | @amotoki: I understand that "device_id" of a dhcp port always start with 'dhcp'. For example: dhcp60ca53ad-97f1-51c6-86cf-94786eb89592-ae9af78f-6c1d-424c-98b8-4d201d931b22. Could you please help me to confirm about that? it's right or wrong? | 09:01 |
apuimedo | diga: that's right | 09:01 |
diga | apuimedo: okay | 09:01 |
*** hdaniel has quit IRC | 09:02 | |
*** hdaniel has joined #openstack-neutron | 09:02 | |
*** obondarev has quit IRC | 09:04 | |
*** obondarev has joined #openstack-neutron | 09:04 | |
*** ferest has quit IRC | 09:05 | |
*** anilvenkata has quit IRC | 09:07 | |
*** zhhuabj has quit IRC | 09:08 | |
*** l6unchpad has quit IRC | 09:11 | |
*** Prem has quit IRC | 09:14 | |
*** hoangcx has quit IRC | 09:18 | |
*** aix has joined #openstack-neutron | 09:18 | |
*** SourabhP has quit IRC | 09:20 | |
*** neeti has quit IRC | 09:20 | |
*** rotbeard has joined #openstack-neutron | 09:23 | |
*** anilvenkata has joined #openstack-neutron | 09:25 | |
*** yamamoto has quit IRC | 09:26 | |
*** markvoelker has joined #openstack-neutron | 09:26 | |
*** moshele has quit IRC | 09:27 | |
*** moshele has joined #openstack-neutron | 09:28 | |
*** l6unchpad has joined #openstack-neutron | 09:28 | |
*** sdake has quit IRC | 09:30 | |
amotoki | thanhnt-z: yes, correct. It is hard coded in dhcp-agent code. | 09:30 |
*** markvoelker has quit IRC | 09:31 | |
*** ferest has joined #openstack-neutron | 09:32 | |
*** kengo_sakai has quit IRC | 09:33 | |
*** killer_prince has joined #openstack-neutron | 09:33 | |
*** dims_ has joined #openstack-neutron | 09:35 | |
*** lazy_prince has quit IRC | 09:36 | |
*** ferest has quit IRC | 09:36 | |
ZZelle | thanhnt-z, it's dhcp<host-uuid>-<network-id> | 09:36 |
*** stevemar has joined #openstack-neutron | 09:40 | |
openstackgerrit | ZhaoBo proposed openstack/neutron-specs: Add timestamp to neutron extension resources https://review.openstack.org/223988 | 09:43 |
*** stevemar has quit IRC | 09:44 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove "ebtables_installed" useless option https://review.openstack.org/223990 | 09:45 |
*** _cjones_ has joined #openstack-neutron | 09:47 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: use safer version of configure_dhcp_for_network https://review.openstack.org/223226 | 09:47 |
thanhnt-z | amotoki, ZZelle: Thank you very much for your support | 09:47 |
*** neeti has joined #openstack-neutron | 09:47 | |
*** baojg has joined #openstack-neutron | 09:48 | |
*** bigjools has quit IRC | 09:49 | |
*** bharathm has quit IRC | 09:50 | |
*** ekuris has quit IRC | 09:50 | |
*** yfried_ is now known as yfried|afk | 09:51 | |
*** itzikb_ has quit IRC | 09:51 | |
*** _cjones_ has quit IRC | 09:52 | |
*** obondarev has quit IRC | 09:55 | |
*** fawadkhaliq has joined #openstack-neutron | 09:59 | |
*** deepthi has joined #openstack-neutron | 09:59 | |
*** moshele has quit IRC | 10:00 | |
*** neeti has quit IRC | 10:01 | |
*** sudipto has joined #openstack-neutron | 10:02 | |
*** obondarev has joined #openstack-neutron | 10:03 | |
*** ferest has joined #openstack-neutron | 10:03 | |
*** sbelous has joined #openstack-neutron | 10:03 | |
*** fawadkhaliq has quit IRC | 10:04 | |
*** tryit has quit IRC | 10:05 | |
*** pc_m has joined #openstack-neutron | 10:06 | |
openstackgerrit | Cao Xuan Hoang proposed openstack/neutron-specs: Logging API for security group and firewall rules https://review.openstack.org/203509 | 10:06 |
*** claudiub has joined #openstack-neutron | 10:07 | |
openstackgerrit | ZhaoBo proposed openstack/neutron-specs: Add timestamp to neutron extension resources https://review.openstack.org/223988 | 10:07 |
*** ferest has quit IRC | 10:07 | |
*** tomoe_ has quit IRC | 10:08 | |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/neutron: Use tempest-lib's token_client https://review.openstack.org/224006 | 10:08 |
*** yfried|afk is now known as yfried_ | 10:10 | |
*** jkraj has joined #openstack-neutron | 10:19 | |
ihrachys | ajo: pushed the refactoring patch for qos. please rebase the 'network ports' one | 10:21 |
ajo | ihrachys, I'm on it | 10:21 |
ajo | ihrachys, right writing the tests | 10:21 |
ajo | ihrachys: thanks :) | 10:21 |
ihrachys | thanks | 10:21 |
*** gberginc has quit IRC | 10:22 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: Added +x permission to gate hook scripts https://review.openstack.org/222655 | 10:22 |
*** obondarev has quit IRC | 10:23 | |
*** iwamoto has quit IRC | 10:24 | |
*** yamamoto has joined #openstack-neutron | 10:26 | |
ihrachys | ajo: "Ihar and the other DB lieutenants" WAT? :) | 10:26 |
ihrachys | I am not in db team, at least officially, neither I am a lieutenant anywhere :) | 10:27 |
ajo | ihrachys, I think you lead the DB lieutenants | 10:27 |
ihrachys | I am a poor guy in the corner writing non-sense patches :) | 10:27 |
ajo | or you're on the doc I least as far as I remember | 10:27 |
ihrachys | ajo: no way! | 10:27 |
ajo | hmmm | 10:27 |
ajo | lemme check ;) | 10:27 |
ihrachys | ajo: http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy | 10:28 |
ajo | ah no | 10:28 |
ajo | :) | 10:28 |
ajo | HenryG & amotoki :) | 10:28 |
ihrachys | see, I am kinda out of the boat :) | 10:28 |
amotoki | ajo: I think Ihar is more familiar with DB than me. I know about API but .... | 10:30 |
*** baojg has quit IRC | 10:30 | |
ajo | we need to amend that doc ;) | 10:30 |
*** alex_klimov has quit IRC | 10:31 | |
*** baojg has joined #openstack-neutron | 10:31 | |
amotoki | ajo: agree, but I would like to defer it to Kyle or the next PTL :) | 10:31 |
*** ociuhandu has quit IRC | 10:31 | |
*** yamamoto has quit IRC | 10:32 | |
amotoki | ihrachys: earlier today I investigated check_migration check failure in neutron-fwaas https://review.openstack.org/#/c/223935/ . | 10:33 |
amotoki | ihrachys: dutring the debug, one question hit me. | 10:33 |
amotoki | ihrachys: why is the contents of HEADS kept in the alphabetical order? | 10:33 |
*** ferest has joined #openstack-neutron | 10:34 | |
ihrachys | amotoki: no particular reason. maybe it was easier for me to write a unit test. | 10:34 |
ihrachys | amotoki: we actually should kill it. I want to do it in start of M | 10:34 |
amotoki | ihrachys: thanks. | 10:34 |
amotoki | ihrachys: I think it is easy to understand if both HEADS and observed revisons are sorted. | 10:35 |
*** baojg has quit IRC | 10:35 | |
*** mwagner_lap has quit IRC | 10:36 | |
openstackgerrit | Sudhakar Babu Gariganti proposed openstack/neutron: Avoid full_sync in l3_agent for router updates https://review.openstack.org/224019 | 10:36 |
ihrachys | amotoki: the only tech reason to have HEADS now is to forbid branches. I need to check whether we covered all branch cases with check_migrations, and if so, just kill it | 10:36 |
ihrachys | amotoki: currently, any new migration script merged makes all other patches conflict. | 10:36 |
amotoki | ihrachys: sounds reasonable. I know the bad world before introduing HEAD file ;-( | 10:37 |
*** itzikb_ has joined #openstack-neutron | 10:38 | |
*** gberginc has joined #openstack-neutron | 10:38 | |
*** ferest has quit IRC | 10:38 | |
ihrachys | you are more mature than me in terms of collecting bad project experience :) | 10:39 |
amotoki | :) | 10:40 |
*** anilvenkata has quit IRC | 10:40 | |
pc_m | amotoki: hi | 10:40 |
amotoki | pc_m: hi | 10:41 |
pc_m | amotoki: Can you weigh in on https://review.openstack.org/#/c/219455/ ? | 10:41 |
pc_m | amotoki: Question on naming of some of the arguments (endpoint type and endpoint values). | 10:41 |
pc_m | See commit message. | 10:42 |
*** anilvenkata has joined #openstack-neutron | 10:42 | |
*** bigjools has joined #openstack-neutron | 10:42 | |
amotoki | pc_m: sure. let me check after getting home. | 10:42 |
pc_m | amotoki: thanks! | 10:43 |
pc_m | amotoki: FYI that's the client side. The server is https://review.openstack.org/#/c/212692/ | 10:43 |
*** moshele has joined #openstack-neutron | 10:44 | |
amotoki | pc_m: I see it. I should review it too. | 10:45 |
pc_m | amotoki: thanks! | 10:45 |
*** stanzgy__ has quit IRC | 10:48 | |
*** stanzgy has quit IRC | 10:49 | |
*** ZZelle has quit IRC | 10:50 | |
*** aix has quit IRC | 10:50 | |
*** ZZelle has joined #openstack-neutron | 10:50 | |
*** fawadkhaliq has joined #openstack-neutron | 10:55 | |
*** shz has quit IRC | 10:59 | |
*** shz has joined #openstack-neutron | 11:00 | |
*** obondarev has joined #openstack-neutron | 11:01 | |
*** b3nt_pin has quit IRC | 11:01 | |
openstackgerrit | Merged openstack/neutron-vpnaas: Exclude neutron_vpnaas/tests from coverage report https://review.openstack.org/223673 | 11:02 |
*** beagles has joined #openstack-neutron | 11:02 | |
*** beagles is now known as Guest86825 | 11:02 | |
*** obondarev has quit IRC | 11:03 | |
*** aix has joined #openstack-neutron | 11:03 | |
openstackgerrit | Merged openstack/neutron: Remove requirements.txt for the ofagent mechanism driver https://review.openstack.org/223903 | 11:03 |
*** yamamoto has joined #openstack-neutron | 11:03 | |
*** yamamoto has quit IRC | 11:03 | |
*** sbelous has quit IRC | 11:04 | |
*** ferest has joined #openstack-neutron | 11:04 | |
*** itsuugo has joined #openstack-neutron | 11:04 | |
*** irenab has quit IRC | 11:05 | |
*** mohankumar has quit IRC | 11:06 | |
*** ferest has quit IRC | 11:08 | |
*** alex_klimov has joined #openstack-neutron | 11:08 | |
*** yfried_ is now known as yfried|afk | 11:11 | |
*** itsuugo has quit IRC | 11:11 | |
*** lazy_prince has joined #openstack-neutron | 11:13 | |
*** tmorin has quit IRC | 11:15 | |
*** killer_prince has quit IRC | 11:16 | |
openstackgerrit | Merged openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 11:16 |
*** _cjones_ has joined #openstack-neutron | 11:17 | |
*** itsuugo has joined #openstack-neutron | 11:19 | |
*** ociuhandu has joined #openstack-neutron | 11:21 | |
*** _cjones_ has quit IRC | 11:22 | |
*** Guest86825 has quit IRC | 11:23 | |
*** banix has joined #openstack-neutron | 11:23 | |
*** gongysh has quit IRC | 11:24 | |
*** beagles__ has joined #openstack-neutron | 11:24 | |
gsagie | ping irenab | 11:25 |
*** itsuugo has quit IRC | 11:26 | |
*** banix has quit IRC | 11:27 | |
*** markvoelker has joined #openstack-neutron | 11:27 | |
*** julim has joined #openstack-neutron | 11:28 | |
*** diga has quit IRC | 11:30 | |
*** yfried|afk is now known as yfried_ | 11:31 | |
*** markvoelker has quit IRC | 11:32 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies https://review.openstack.org/220164 | 11:32 |
ajo | ihrachys ^ | 11:32 |
*** fawadkhaliq has quit IRC | 11:33 | |
*** beagles__ has quit IRC | 11:33 | |
*** ferest has joined #openstack-neutron | 11:34 | |
*** tbachman has joined #openstack-neutron | 11:37 | |
*** ferest has quit IRC | 11:39 | |
*** yfried_ is now known as yfried|afk | 11:41 | |
*** jbell8 has joined #openstack-neutron | 11:45 | |
*** irenab has joined #openstack-neutron | 11:45 | |
*** jpena is now known as jpena|lunch | 11:45 | |
*** tomoe_ has joined #openstack-neutron | 11:45 | |
*** rotbeard has quit IRC | 11:46 | |
*** fawadkhaliq has joined #openstack-neutron | 11:46 | |
*** l6unchpad has quit IRC | 11:48 | |
*** rotbeard has joined #openstack-neutron | 11:49 | |
*** obondarev has joined #openstack-neutron | 11:49 | |
*** jbell8 has quit IRC | 11:49 | |
*** boris-42 has quit IRC | 11:50 | |
*** yfried|afk is now known as yfried_ | 11:51 | |
*** arajagopal has quit IRC | 11:52 | |
*** arajagopal has joined #openstack-neutron | 11:52 | |
*** mohankumar has joined #openstack-neutron | 11:55 | |
*** thorst has joined #openstack-neutron | 11:55 | |
*** k-yasui has quit IRC | 11:57 | |
*** arajagopal has quit IRC | 11:57 | |
*** baoli has joined #openstack-neutron | 11:57 | |
*** baoli has quit IRC | 11:57 | |
*** gampel has joined #openstack-neutron | 11:57 | |
*** baoli has joined #openstack-neutron | 11:58 | |
*** baoli_ has joined #openstack-neutron | 11:59 | |
*** gampel1 has quit IRC | 12:00 | |
*** tkatarki has joined #openstack-neutron | 12:01 | |
*** fawadkhaliq has quit IRC | 12:01 | |
*** baoli has quit IRC | 12:03 | |
*** evgenyf has quit IRC | 12:04 | |
*** yamamoto has joined #openstack-neutron | 12:04 | |
ihrachys | ajo: I left some dumb grammar nits for the qos patch, please respin and I am happy to +2 | 12:04 |
*** itsuugo has joined #openstack-neutron | 12:05 | |
*** ferest has joined #openstack-neutron | 12:05 | |
*** jckasper has quit IRC | 12:05 | |
*** yfried_ is now known as yfried|afk | 12:07 | |
*** l6unchpad has joined #openstack-neutron | 12:09 | |
*** tkatarki has quit IRC | 12:09 | |
*** evgenyf has joined #openstack-neutron | 12:09 | |
ajo | ihrachys: ack, thanks a lot, I will respin in a few minutes | 12:09 |
*** yamamoto has quit IRC | 12:09 | |
*** ferest has quit IRC | 12:09 | |
*** fawadkhaliq has joined #openstack-neutron | 12:10 | |
*** cbits has quit IRC | 12:10 | |
HenryG | ihrachys: amotoki: we need HEADS to prevent independent merges causing migrations branching. I don't think pep8 check_migrations can catch that until very late, or even reliably? | 12:11 |
ihrachys | HenryG: yes, that's why I wrote that I need to check whether we don't miss any cases that would allow it | 12:12 |
*** thanhnt-z has quit IRC | 12:12 | |
*** markvoelker has joined #openstack-neutron | 12:12 | |
*** flaviof has quit IRC | 12:12 | |
ihrachys | HenryG: btw I am not so sure it's a disaster to have branches, but I need to think about it a bit more. | 12:12 |
*** tkatarki has joined #openstack-neutron | 12:13 | |
ihrachys | HenryG: we could just allow them, and then merge on final release | 12:13 |
ihrachys | that would unblock all those conflict resolutions and all | 12:13 |
HenryG | ihrachys: Maybe | 12:14 |
*** mwagner_lap has joined #openstack-neutron | 12:16 | |
*** beagles has joined #openstack-neutron | 12:16 | |
ajo | ihrachys, irenab , moshele , gsagie today we have nova folks joining the QoS meeting for the nova/neutron coordination thing. | 12:17 |
ihrachys | ack | 12:17 |
irenab | ajo: I will join a bit later than usual | 12:17 |
ajo | irenab, ack, | 12:18 |
*** obondarev has quit IRC | 12:18 | |
*** _cjones_ has joined #openstack-neutron | 12:19 | |
*** yfried|afk is now known as yfried_ | 12:19 | |
*** tkatarki has quit IRC | 12:20 | |
*** beagles has quit IRC | 12:21 | |
*** itsuugo has quit IRC | 12:22 | |
*** EinstCrazy has joined #openstack-neutron | 12:23 | |
*** leitan has joined #openstack-neutron | 12:23 | |
*** _cjones_ has quit IRC | 12:23 | |
*** tmorin has joined #openstack-neutron | 12:24 | |
*** kiran-r has quit IRC | 12:25 | |
*** jaypipes has quit IRC | 12:26 | |
*** jaypipes_ has joined #openstack-neutron | 12:26 | |
*** MVenesio has joined #openstack-neutron | 12:28 | |
*** tkatarki has joined #openstack-neutron | 12:28 | |
*** manas has joined #openstack-neutron | 12:28 | |
*** sudipto has quit IRC | 12:30 | |
ajo | johnthetubaguy, are you able to join today the QoS meeting for nova/neutron integration?, Nikola Dipanov and Vladik Romanovsky are joining | 12:32 |
johnthetubaguy | ajo: I can lurk, what time was that again? | 12:33 |
ajo | now + 1:30h | 12:33 |
ajo | johnthetubaguy ^ | 12:33 |
*** edmondsw has joined #openstack-neutron | 12:33 | |
johnthetubaguy | ajo: OK, I will watch out for that, thanks for the ping | 12:33 |
ajo | johnthetubaguy: thanks :) | 12:33 |
ajo | johnthetubaguy, last time we talked you commented something about a new vif binding library | 12:34 |
ajo | or something like that | 12:34 |
ajo | any link/pointer to it? | 12:34 |
*** obondarev has joined #openstack-neutron | 12:35 | |
*** deepthi has quit IRC | 12:35 | |
*** yfried_ is now known as yfried|afk | 12:36 | |
*** tongli has joined #openstack-neutron | 12:38 | |
*** tkatarki has quit IRC | 12:39 | |
*** EinstCrazy has quit IRC | 12:40 | |
*** tkatarki has joined #openstack-neutron | 12:42 | |
*** yfried|afk is now known as yfried_ | 12:42 | |
*** tkatarki has quit IRC | 12:43 | |
openstackgerrit | Andreas Scheuring proposed openstack/neutron: Make OVS interface name hasing algorithm common and extend it https://review.openstack.org/224064 | 12:43 |
*** adam_g has quit IRC | 12:44 | |
*** kiran-r has joined #openstack-neutron | 12:44 | |
*** tkatarki has joined #openstack-neutron | 12:45 | |
*** jpena|lunch is now known as jpena | 12:45 | |
*** Samos123 has quit IRC | 12:47 | |
*** wendar has quit IRC | 12:47 | |
*** wendar has joined #openstack-neutron | 12:47 | |
*** dims_ has quit IRC | 12:47 | |
*** sbalukoff has quit IRC | 12:48 | |
*** dsneddon has quit IRC | 12:48 | |
*** tkatarki has quit IRC | 12:49 | |
*** lbragstad has quit IRC | 12:49 | |
*** dsneddon has joined #openstack-neutron | 12:49 | |
*** comstud has quit IRC | 12:50 | |
*** yamamoto has joined #openstack-neutron | 12:50 | |
*** sbalukoff has joined #openstack-neutron | 12:50 | |
*** adam_g has joined #openstack-neutron | 12:50 | |
*** bharathm has joined #openstack-neutron | 12:51 | |
*** adam_g has quit IRC | 12:51 | |
*** adam_g has joined #openstack-neutron | 12:51 | |
*** comstud has joined #openstack-neutron | 12:51 | |
*** lbragstad has joined #openstack-neutron | 12:51 | |
*** cbits has joined #openstack-neutron | 12:52 | |
*** Samos123 has joined #openstack-neutron | 12:52 | |
*** tbachman has quit IRC | 12:54 | |
*** annegentle has joined #openstack-neutron | 12:54 | |
*** bharathm has quit IRC | 12:55 | |
johnthetubaguy | ajo: oops, sorry got distracted by some script tweaks, will see if we have a pointer to that somewhere | 12:56 |
ajo | johnthetubaguy, thanks, no problem :) | 12:56 |
johnthetubaguy | ajo: I think this is a rough pointer: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067783.html | 12:56 |
*** aix has quit IRC | 12:56 | |
johnthetubaguy | ajo: not sure how the details of that are panning out yet, there was quite a bit of chatter at the midcycle | 12:57 |
ajo | reading :) | 12:57 |
johnthetubaguy | was hoping that will start in early mitaka | 12:57 |
*** shoutm has joined #openstack-neutron | 12:58 | |
*** rpothier has joined #openstack-neutron | 12:58 | |
*** dims_ has joined #openstack-neutron | 12:58 | |
matrohon | amuller : hi | 12:59 |
*** Murali has quit IRC | 12:59 | |
ajo | johnthetubaguy, sounds good, I failed to read the thread from MLo riginally | 12:59 |
ajo | johnthetubaguy: I need to read the spec carefully btw | 13:00 |
johnthetubaguy | not sure the spec has the latest thoughts on the subject, but those are both good places to start | 13:00 |
*** arajagopal has joined #openstack-neutron | 13:00 | |
*** jckasper has joined #openstack-neutron | 13:01 | |
*** flaviof has joined #openstack-neutron | 13:01 | |
*** tbachman has joined #openstack-neutron | 13:01 | |
*** shwetaap has joined #openstack-neutron | 13:02 | |
*** dsneddon has quit IRC | 13:02 | |
*** annemccormick has joined #openstack-neutron | 13:03 | |
*** ushkalim has quit IRC | 13:04 | |
*** amuller has joined #openstack-neutron | 13:04 | |
*** dr3662 has joined #openstack-neutron | 13:05 | |
*** itzikb_ has quit IRC | 13:05 | |
*** lindycoder has joined #openstack-neutron | 13:06 | |
*** Samos123 has quit IRC | 13:06 | |
*** gampel1 has joined #openstack-neutron | 13:06 | |
*** gampel has quit IRC | 13:07 | |
*** jkraj has quit IRC | 13:10 | |
*** aix has joined #openstack-neutron | 13:10 | |
*** dr3662__ has joined #openstack-neutron | 13:11 | |
*** dr3662__ has quit IRC | 13:12 | |
*** keopp has joined #openstack-neutron | 13:12 | |
*** MVenesio has quit IRC | 13:14 | |
*** dr3662 has quit IRC | 13:14 | |
*** jprovazn has joined #openstack-neutron | 13:14 | |
*** tomoe_ has quit IRC | 13:15 | |
*** MVenesio has joined #openstack-neutron | 13:15 | |
*** beagles has joined #openstack-neutron | 13:15 | |
*** mohankumar has quit IRC | 13:15 | |
*** Samos123 has joined #openstack-neutron | 13:15 | |
*** dsneddon has joined #openstack-neutron | 13:16 | |
*** rotbeard has quit IRC | 13:16 | |
*** abregman is now known as abregman|brb | 13:17 | |
*** itsuugo has joined #openstack-neutron | 13:17 | |
*** pece has quit IRC | 13:19 | |
*** shwetaap1 has joined #openstack-neutron | 13:19 | |
*** _cjones_ has joined #openstack-neutron | 13:19 | |
*** ushkalim has joined #openstack-neutron | 13:20 | |
*** krtaylor has quit IRC | 13:20 | |
*** gongysh has joined #openstack-neutron | 13:20 | |
*** tiswanso has joined #openstack-neutron | 13:20 | |
*** shwetaap has quit IRC | 13:22 | |
*** bdemers has joined #openstack-neutron | 13:23 | |
*** tkatarki has joined #openstack-neutron | 13:23 | |
*** _cjones_ has quit IRC | 13:24 | |
*** bdemers has quit IRC | 13:27 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-vpnaas: tox.ini: switch to --subproject for check-migration https://review.openstack.org/224082 | 13:28 |
*** yuanying_ has joined #openstack-neutron | 13:29 | |
*** yuanying has quit IRC | 13:29 | |
*** yuanying_ is now known as yuanying | 13:29 | |
*** gongysh has quit IRC | 13:29 | |
*** annegentle has quit IRC | 13:30 | |
*** annegentle has joined #openstack-neutron | 13:30 | |
*** bdemers has joined #openstack-neutron | 13:31 | |
*** salv-orlando has joined #openstack-neutron | 13:31 | |
*** kiran-r has quit IRC | 13:32 | |
*** sudipto has joined #openstack-neutron | 13:32 | |
*** fawadkhaliq has quit IRC | 13:33 | |
*** numans has quit IRC | 13:35 | |
*** itsuugo has quit IRC | 13:39 | |
*** arajagopal has quit IRC | 13:40 | |
*** achanda has joined #openstack-neutron | 13:40 | |
*** arajagopal has joined #openstack-neutron | 13:40 | |
*** achanda has quit IRC | 13:41 | |
*** amotoki has quit IRC | 13:42 | |
*** LukeHinds has joined #openstack-neutron | 13:42 | |
*** arajagopal has quit IRC | 13:42 | |
*** arajagopal has joined #openstack-neutron | 13:43 | |
*** zhenguo has quit IRC | 13:43 | |
openstackgerrit | Akihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error https://review.openstack.org/223935 | 13:45 |
*** arajagopal has quit IRC | 13:50 | |
*** arajagopal has joined #openstack-neutron | 13:50 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: add return in for-loop in unbind_router_servicenode https://review.openstack.org/223435 | 13:51 |
*** abregman|brb is now known as abregman | 13:51 | |
ihrachys | ajo: speaking of responsibilities :))) what's the qos doc story? I saw some assignment moves on the bug | 13:52 |
ajo | ihrachys, I need to sync to emagana, I know he's working on it while I finished bugs | 13:53 |
ajo | now that bugs are over, (well, he sent me an email with another one.. but that's easy) | 13:53 |
*** kbringard has joined #openstack-neutron | 13:53 | |
ihrachys | ajo: ok cool. just wanted to make sure it's under control :) | 13:53 |
ajo | ihrachys: he found that if you enable everything, but forget the ml2 extension driver... when you try to push a policy_id on a port or network.. you get a kaboom exception | 13:54 |
ihrachys | ajo: we also need that upgrade write-up, but it's fine for after L | 13:54 |
ihrachys | ajo: how kaboom is it? | 13:54 |
ajo | ihrachys: let me resend you the email | 13:54 |
ajo | ihrachys I think I will fill a bug now, and make a simple patch for it | 13:54 |
ihrachys | ajo: that should probably be in some bug tracker insteawd | 13:54 |
ajo | exchanging the kaboom exception for an explanation humans can understand | 13:54 |
ihrachys | ajo: right :) | 13:54 |
*** yamahata has joined #openstack-neutron | 13:54 | |
*** arajagopal has quit IRC | 13:55 | |
ajo | ihrachys, it's something our "move the -check port changes for extensions- in ml2/plugin.py to the extensions" would fix too | 13:55 |
ihrachys | ajo: aye, we need core resource extender manager :) | 13:56 |
ajo | ihrachys: yes | 13:56 |
ajo | ihrachys: we must fill an RFE for M | 13:56 |
ihrachys | ajo: let me handle the piece in M | 13:56 |
ajo | ihrachys++ sync with armax, he was interested in the topic | 13:56 |
ihrachys | roger | 13:57 |
ajo | QoS meeting in T+3m | 13:57 |
ajo | I almost forgot, so nice I have calendar integration back to my desktop | 13:57 |
ajo | :') | 13:57 |
*** dane_leblanc has joined #openstack-neutron | 13:58 | |
*** nate_gone is now known as njohnston | 13:58 | |
*** pradk has joined #openstack-neutron | 13:59 | |
ajo | moshele ping, could you join #openstack-meeting-3 ? :) | 14:00 |
moshele | ajo: sure | 14:01 |
*** sayali has quit IRC | 14:02 | |
*** zzzeek has joined #openstack-neutron | 14:02 | |
*** lajos-katona has left #openstack-neutron | 14:03 | |
*** wverdugo500 has quit IRC | 14:03 | |
*** sridhar_ram has joined #openstack-neutron | 14:04 | |
salv-orlando | HenryG, akamyshnikova: is it correct that each subproject from a migration perspective has a different alembic environment? | 14:04 |
*** ushkalim has quit IRC | 14:06 | |
HenryG | salv-orlando: yes, although it is 99% a clone of neutron's | 14:06 |
*** arajagopal has joined #openstack-neutron | 14:06 | |
salv-orlando | HenryG: sure. I am aiming at understanding how I can deal with dependency among subprojects | 14:06 |
*** vthapar has joined #openstack-neutron | 14:06 | |
HenryG | salv-orlando: Just the script_location and alembic_branches table is different | 14:06 |
vthapar | tmorin: ping | 14:06 |
akamyshnikova | salv-orlando, yes | 14:07 |
tmorin | vthapar: icmp echo reply | 14:07 |
salv-orlando | HenryG: sure. But if I specify in a migration for project X a depends_on with a revision for subprokect Y that won't work because the enviroment are different | 14:07 |
tmorin | vthapar: I'm reading, go ahead ;) | 14:07 |
HenryG | salv-orlando: Abhishek has asked me about this | 14:08 |
vthapar | tmorin: had couple questions on networking-bgpvpn. I'm working on ODL driver and wanted some clarity on couple things | 14:08 |
salv-orlando | as we realized openstack/vmware-nsx has a dependency on openstack/networking-l2gw I need to devise an alternative strategy | 14:08 |
salv-orlando | HenryG: yes I know... I just wanted to confirm with you that this was impossible for the foresseable future | 14:08 |
tmorin | vthapar: yes, I've seen your name on https://git.opendaylight.org/gerrit/#/c/26711/5 | 14:08 |
vthapar | tmorin: in the spec certain fields are supose to have default values. can I assume that they will always be filled in by plugin when it hits driver code? | 14:09 |
*** mragupat has joined #openstack-neutron | 14:09 | |
*** obondarev has quit IRC | 14:09 | |
vthapar | tmorin: and does it apply to both create and update? | 14:09 |
HenryG | salv-orlando: I *think* if we put all sub-project's alembic branches in one table we can support stuff like what you need | 14:09 |
*** pradk has quit IRC | 14:10 | |
*** arajagopal has quit IRC | 14:10 | |
tmorin | vthapar: the default values are filled in by the API engine for create | 14:10 |
HenryG | salv-orlando: but I am still trying to figure out how to get there | 14:10 |
*** mragupat has quit IRC | 14:10 | |
salv-orlando | HenryG: one table? what do you mean? I thought we were collecting subprojects by entrypoints | 14:10 |
*** aix has quit IRC | 14:11 | |
tmorin | vthapar: for updates, I don't know, in an update only the updated fields are specified, and by definition they do not have a default value | 14:11 |
HenryG | salv-orlando: the alembic branch heads are stored in a table | 14:11 |
vthapar | tmorin: def update_bgpvpn_postcommit(self, context, old_bgpvpn, bgpvpn): | 14:12 |
tmorin | vthapar: the resource map defines the default values here: https://github.com/openstack/networking-bgpvpn/blob/master/networking_bgpvpn/neutron/extensions/bgpvpn.py#L69 | 14:12 |
akamyshnikova | HenryG, yes, that could be solution for such cases | 14:12 |
vthapar | tmorin: will bgpvpn be the delta [new items added] or the updated object with old+changes? | 14:12 |
*** pradk has joined #openstack-neutron | 14:12 | |
*** r-daneel has joined #openstack-neutron | 14:12 | |
*** julim has quit IRC | 14:12 | |
vthapar | tomirn: yeah, I saw that. ran into some issues with update part of code, so want to understand how OpenStack works before I figure out if needs to be taken care of in my driver code or in ODL. | 14:13 |
*** mkolesni has quit IRC | 14:13 | |
tmorin | vthapar: however, the API side does not yet cover some of the attributes defined in the specs (eg. technique or vnid), so for these, no attribute will exist in the data passed to the driver | 14:13 |
akamyshnikova | HenryG, but may be zzzeek can suggest something else | 14:13 |
salv-orlando | akamyshnikova: so you're suggesting a way to specify dependencies among branches, not single revisions. Like run branch X before branch Y? | 14:13 |
*** sayali has joined #openstack-neutron | 14:14 | |
vthapar | tmorin: what I noticed was that status and admin_state were coming as null. | 14:14 |
vthapar | tmorin: ^in update. create fills everything correctly. | 14:14 |
tmorin | vthapar: old_bgpvpn is the old content, and bgpvpn is the new | 14:14 |
vthapar | tmorin: so all default values should be filled in bgpvpn, right? | 14:15 |
tmorin | vthapar: at least that's my understanding, but I haven't been deep in that to be 100% sure | 14:15 |
HenryG | akamyshnikova: zzzeek suggested to use one table for all branches to support a single env | 14:16 |
*** Marga_ has joined #openstack-neutron | 14:16 | |
tmorin | vthapar: just checked, yes, that's what it does (pass old and new) | 14:16 |
*** minwang2 has joined #openstack-neutron | 14:16 | |
*** yog_ has joined #openstack-neutron | 14:16 | |
tmorin | vthapar: indeed, admin_status is not implemented yet | 14:16 |
vthapar | tmorin: okay, that was my understanding too. in rest of ODL drivers [ML2 etc.] updated object is sent as is so I expected it to be so. will capture some logs to confirm and share. | 14:16 |
*** anilvenkata has left #openstack-neutron | 14:16 | |
zzzeek | HenryG: yes because that’s the feature I put a ton of work into making very robust, and it directly considers this use case | 14:16 |
vthapar | tmorin: aha! that explains it :) | 14:17 |
tmorin | vthapar: well, it does not explain why it would be non-null for create | 14:17 |
zzzeek | HenryG: it would make it very easy for out-of-tree plugins to declare dependences in in-tree migrations and structures for example | 14:17 |
*** lazy_prince has quit IRC | 14:17 | |
akamyshnikova | salv-orlando, no, I think about revisions, because branches can have such problems later... | 14:17 |
tmorin | vthapar: with the current code, it should be null for both | 14:17 |
zzzeek | dependencies *on* | 14:17 |
* vthapar checking logs. | 14:18 | |
HenryG | zzzeek: salv-orlando has a dependency between two out-of-tree projects | 14:18 |
vthapar | tmorin: oh yeah, my bad. it is null for create too. but my code [ODL] handles it on creation so didn't see exception. | 14:19 |
*** ushkalim has joined #openstack-neutron | 14:19 | |
tmorin | vathapar: I guess your API should be robust and default to a sane behavior if the client is incomplete -- but I guess this question must have been explored in other Neutron-related contexts : what was the decision made ? (rely on Neutron setting the defaults or fill the missing things on ODL side ?) | 14:19 |
*** dane_leblanc has quit IRC | 14:19 | |
*** dane_leblanc has joined #openstack-neutron | 14:20 | |
tmorin | tmorin: I would in fact expect the Yang datamodel to possibly define a default value for any field, a bit like Neutron resource map does | 14:20 |
*** nplanel has quit IRC | 14:20 | |
vthapar | tmoring: extra sanity checks always help. wanted to get better understanding of OpenStack side of things before bringing it up with ODL folks. thanks. I got devstack working with bgpvpn+ODL. | 14:20 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Delete unused file tests/unit/database_stubs.py https://review.openstack.org/224104 | 14:20 |
*** _cjones_ has joined #openstack-neutron | 14:21 | |
tmorin | vthapar: yes, prem told us yesterday on IRC, great news ! | 14:21 |
tmorin | vthapar: out of curiosity, what is the BGP implementation used in the backend ? (forgot to ask yesterday) | 14:21 |
*** clarkb1 is now known as clarkb | 14:21 | |
*** aix has joined #openstack-neutron | 14:22 | |
vthapar | tmorin: I think it is quagga, not sure though as we were trying with Ryu too at one point. current plan is to leverage ODL's BGP. | 14:23 |
tmorin | vthapar: yep, I was wondering if you already had the implementation with ODL BGP or if it was still quagga | 14:23 |
tmorin | vthapar: is the quagga integration also part of the recent gerrit change ? | 14:24 |
*** pradk has quit IRC | 14:24 | |
vthapar | tmorin: no. | 14:24 |
*** obondarev has joined #openstack-neutron | 14:24 | |
*** pradk has joined #openstack-neutron | 14:24 | |
tmorin | vthapar: ok, so the commit is only one part of the target solution | 14:24 |
vthapar | tmorin: yep. BGP integration will work on other side of ODL and other guys are working on it already. | 14:25 |
*** _cjones_ has quit IRC | 14:25 | |
tmorin | vthapar: excellent | 14:26 |
tmorin | vthapar: anyhow, we'll be ready to review the BGPVPN ODL driver when it comes | 14:27 |
*** sdake_ has joined #openstack-neutron | 14:27 | |
salv-orlando | HenryG, zzzeek, akamyshnikova: I'm not really up-to-date on the migration front. Is this feature zzzeek mentioned ready in alembic but not leveraged from the neutron side or still under development? | 14:28 |
*** crose has joined #openstack-neutron | 14:28 | |
*** sayali has quit IRC | 14:28 | |
*** anshul has quit IRC | 14:28 | |
HenryG | salv-orlando: can zzzeek and I get back to you in a couple of days with a plan? | 14:28 |
*** alexpilotti has quit IRC | 14:29 | |
HenryG | salv-orlando: I think I know what is needed, and zzzeek will help me | 14:29 |
salv-orlando | HenryG I'm not asking for you to add the feature for me ;) | 14:30 |
HenryG | salv-orlando: It's for neutron and openstack :) | 14:30 |
pc_m | akamyshnikova: ping | 14:30 |
salv-orlando | I need for my subproject, and I was going to do it myself if necessary! | 14:30 |
*** julien_ZTE has joined #openstack-neutron | 14:30 | |
HenryG | salv-orlando: don't be so selfish :) | 14:31 |
akamyshnikova | pc_m, hi | 14:31 |
pc_m | akamyshnikova: Do you know if there is any issue with naming a model field 'type'? | 14:31 |
*** openstackgerrit has quit IRC | 14:31 | |
salv-orlando | HenryG: surely switching from separate alembic envs to a common one is something I'd delay after Liberty | 14:31 |
pc_m | akamyshnikova: There was a suggestion to rename an enum from endpoint_type to type. | 14:31 |
pc_m | Didn't know if that would be a problem. | 14:31 |
*** hdaniel has quit IRC | 14:32 | |
HenryG | salv-orlando: of course | 14:32 |
*** openstackgerrit has joined #openstack-neutron | 14:32 | |
*** flaviof_ has joined #openstack-neutron | 14:32 | |
HenryG | salv-orlando: I need to step out for a few hours. Back later | 14:33 |
akamyshnikova | pc_m, In fact it should not, but I think it is better to name Enum endpoint_type, not just type, because there are a lot of Enums in Neutron and all names need to be different it is urgent for PostgreSQL. | 14:33 |
pc_m | akamyshnikova: thanks | 14:34 |
openstackgerrit | Merged openstack/neutron: Always return iterables in L3 get_candidates https://review.openstack.org/222828 | 14:34 |
salv-orlando | thanks HenryG you already answered all my question. If you need help with unifying alembic envs consider myself and abhraut your servants | 14:34 |
*** tongli_ has joined #openstack-neutron | 14:35 | |
*** flaviof has quit IRC | 14:36 | |
akamyshnikova | pc_m, np | 14:36 |
*** Marga_ has quit IRC | 14:36 | |
*** julim has joined #openstack-neutron | 14:37 | |
*** Marga_ has joined #openstack-neutron | 14:37 | |
*** tongli has quit IRC | 14:37 | |
*** david-lyle has quit IRC | 14:37 | |
*** minwang2 has quit IRC | 14:40 | |
*** sayali has joined #openstack-neutron | 14:41 | |
*** minwang2 has joined #openstack-neutron | 14:41 | |
openstackgerrit | Merged openstack/neutron: QoS agent extension and driver refactoring https://review.openstack.org/214218 | 14:42 |
openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Make chown rootwrap filter ipsec.secrets file specific https://review.openstack.org/224133 | 14:42 |
*** mriedem has joined #openstack-neutron | 14:45 | |
mriedem | is there a fix for this somewhere? http://logs.openstack.org/13/223713/2/gate/gate-neutron-python34-constraints/fd61c44/console.html#_2015-09-16_13_48_01_379 | 14:45 |
mriedem | those new constraints jobs are blowing up the uncategorized bugs page in the gate http://status.openstack.org/elastic-recheck/data/uncategorized.html | 14:46 |
*** charlesr has joined #openstack-neutron | 14:46 | |
amuller | mriedem: Thanks for bringing that to our attention | 14:47 |
mriedem | i'm opening a bug now | 14:47 |
amuller | mriedem: Thanks | 14:48 |
*** arajagopal has joined #openstack-neutron | 14:49 | |
*** l6unchpad has quit IRC | 14:49 | |
*** arajagopal has quit IRC | 14:49 | |
mriedem | https://bugs.launchpad.net/neutron/+bug/1496424 | 14:49 |
openstack | Launchpad bug 1496424 in neutron "gate-neutron-python27(34)-constraints jobs blowing up in gate due to misconfig" [Undecided,New] | 14:49 |
*** arajagopal has joined #openstack-neutron | 14:49 | |
mriedem | i assume this is a non-voting job? | 14:50 |
amuller | mriedem: yes | 14:50 |
mriedem | it shouldn't be in the gate queue then | 14:50 |
amuller | I agree | 14:50 |
mriedem | patch incoming to infra in a few minutes | 14:50 |
amuller | mriedem: looks like these failures started today? | 14:51 |
amuller | I wonder what changed | 14:51 |
mriedem | yesterday | 14:51 |
amuller | no new tox release | 14:51 |
amuller | must be an infra patch | 14:52 |
amuller | I don't see any obvious patch in project-config | 14:53 |
*** Prem has joined #openstack-neutron | 14:53 | |
*** annegent_ has joined #openstack-neutron | 14:54 | |
*** tidwellr has joined #openstack-neutron | 14:54 | |
*** annegentle has quit IRC | 14:54 | |
*** l6unchpad has joined #openstack-neutron | 14:55 | |
*** nmagnezi has quit IRC | 14:56 | |
*** vthapar has quit IRC | 14:56 | |
amuller | mriedem: might be something to do with branches. It looks like patches to the pecan branch are all failing that job: https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:feature/pecan,n,z | 14:56 |
mriedem | yeah | 14:57 |
*** adduarte has quit IRC | 14:57 | |
*** tkatarki has quit IRC | 14:57 | |
mriedem | that's because c029954c8ae041e5f15b14ceef0e2aa060928e05 isn't on the feature/pecan branch | 14:57 |
mriedem | gah, feature branches | 14:58 |
*** dims_ has quit IRC | 14:58 | |
*** stevemar has joined #openstack-neutron | 14:58 | |
*** carl_baldwin has joined #openstack-neutron | 14:58 | |
mriedem | i can fix that quick | 14:58 |
*** dims_ has joined #openstack-neutron | 14:58 | |
*** tkatarki has joined #openstack-neutron | 14:58 | |
*** jistr is now known as jistr|call | 14:58 | |
amuller | mriedem: Allow me :) | 14:59 |
*** sridhar_ram has quit IRC | 14:59 | |
mriedem | too late | 14:59 |
amuller | mriedem: https://review.openstack.org/#/c/224138/ | 14:59 |
amuller | bam | 14:59 |
mriedem | add bug 1496424 to that commit message | 15:00 |
openstack | bug 1496424 in neutron "gate-neutron-python27(34)-constraints jobs blowing up in gate due to misconfig" [Undecided,New] https://launchpad.net/bugs/1496424 | 15:00 |
mriedem | closes-bug 1496424 | 15:00 |
*** tomoe_ has joined #openstack-neutron | 15:00 | |
*** boris-42 has joined #openstack-neutron | 15:01 | |
amuller | done | 15:01 |
*** openstackgerrit has quit IRC | 15:01 | |
amuller | self approved | 15:01 |
amuller | alright | 15:01 |
amuller | what about stable branches | 15:01 |
*** ygbo has quit IRC | 15:01 | |
mriedem | not a problem on stable i don't think | 15:01 |
*** openstackgerrit has joined #openstack-neutron | 15:02 | |
*** tkatarki has quit IRC | 15:02 | |
mriedem | - name: ^gate-.*-py(?:thon\d\d|py)-constraints$ | 15:02 |
mriedem | branch: ^(?!stable/(?:juno|kilo)).*$ | 15:02 |
*** l6unchpad has quit IRC | 15:03 | |
*** tkatarki has joined #openstack-neutron | 15:03 | |
*** vivek-ebay has joined #openstack-neutron | 15:04 | |
amuller | alright | 15:04 |
amuller | mriedem: Thanks for highlighting this issue! | 15:04 |
*** yfried_ is now known as yfried|afk | 15:04 | |
mriedem | np | 15:04 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies https://review.openstack.org/220164 | 15:06 |
ajo | ihrachys ^there it goes, sorry, It was left on a console asking "Type 'yes' to confirm, other to cancel:".... for a couple of hours :( | 15:06 |
ihrachys | ajo: lol | 15:06 |
ajo | '':) | 15:07 |
ajo | very sad, it could have passed CI already | 15:07 |
*** banix has joined #openstack-neutron | 15:07 | |
*** kawa2014 has quit IRC | 15:08 | |
*** jckasper has quit IRC | 15:08 | |
*** mickeys has joined #openstack-neutron | 15:08 | |
*** vivek-ebay has quit IRC | 15:09 | |
*** hdaniel has joined #openstack-neutron | 15:09 | |
sc68cal | scheuran: saw your review for the hashing. Will review probably tomorrow - travel today | 15:10 |
*** yfried|afk is now known as yfried_ | 15:10 | |
scheuran | sc68cal, sure no problem. Thanks! | 15:10 |
*** emagana has joined #openstack-neutron | 15:12 | |
*** tkatarki has quit IRC | 15:12 | |
*** tkatarki has joined #openstack-neutron | 15:13 | |
ihrachys | eezhova: ping re https://bugs.launchpad.net/bugs/1276694 | 15:14 |
openstack | Launchpad bug 1276694 in Sahara "Openstack services should support SIGHUP signal" [Medium,In progress] - Assigned to Sergey Lukjanov (slukjanov) | 15:14 |
ihrachys | eezhova: it's marked for RC1, but it does not seem like the patch https://review.openstack.org/#/c/170905/ is ready for review | 15:14 |
*** slaweq has quit IRC | 15:14 | |
ihrachys | ajo: +2 | 15:16 |
*** fawadkhaliq has joined #openstack-neutron | 15:16 | |
*** pc_m is now known as pc_m_AWAY | 15:16 | |
*** jckasper has joined #openstack-neutron | 15:16 | |
*** sean-k-m1 has joined #openstack-neutron | 15:17 | |
*** tomoe_ has quit IRC | 15:17 | |
*** sean-k-mooney has quit IRC | 15:18 | |
*** gampel1 has quit IRC | 15:19 | |
*** mhickey has joined #openstack-neutron | 15:19 | |
*** ajmiller has joined #openstack-neutron | 15:21 | |
*** ronis has quit IRC | 15:21 | |
*** ijw_ has quit IRC | 15:22 | |
*** ijw has joined #openstack-neutron | 15:22 | |
*** manas has quit IRC | 15:23 | |
*** jaypipes_ is now known as jaypipes | 15:23 | |
*** pumaranikar has joined #openstack-neutron | 15:23 | |
*** Marga_ has quit IRC | 15:24 | |
emagana | ajo: Trying your suggestion for the QoS testing | 15:24 |
*** yamamoto has quit IRC | 15:24 | |
irenab | ajo: https://etherpad.openstack.org/p/qos-nova-neutron | 15:24 |
mhickey | Hi Everyone! I am a newbie to neutron!:) | 15:25 |
*** Sukhdev_ has joined #openstack-neutron | 15:26 | |
*** SridharG has quit IRC | 15:30 | |
*** shoutm has quit IRC | 15:30 | |
*** lazy_prince has joined #openstack-neutron | 15:30 | |
*** jistr|call is now known as jistr | 15:30 | |
*** scheuran has quit IRC | 15:32 | |
*** annegent_ has quit IRC | 15:32 | |
*** annegentle has joined #openstack-neutron | 15:33 | |
*** david-ly_ has joined #openstack-neutron | 15:35 | |
*** bharathm has joined #openstack-neutron | 15:35 | |
*** sudipto has quit IRC | 15:36 | |
*** vthapar has joined #openstack-neutron | 15:36 | |
vthapar | tmorin: ping | 15:36 |
*** TrevorV has joined #openstack-neutron | 15:37 | |
tmorin | vthapar: yes ? | 15:37 |
vthapar | tmorin: needed some inputs on writing associate_network function on driver. | 15:38 |
*** sudipto has joined #openstack-neutron | 15:39 | |
vthapar | tmorin: the parameters passed are network and bgpvpn ids, I need the full bgpvpn object. will service_plugin.get_bgpvpn(context, bgpvpn_id) provide me that? | 15:39 |
*** minwang2 has quit IRC | 15:39 | |
vthapar | tmorin: or should I do it differently? | 15:39 |
*** armax has joined #openstack-neutron | 15:39 | |
tmorin | vthapar: yes, that looks like a possible way | 15:39 |
tmorin | vthapar: you could also query self.bgpvpn_db.get_xxxx | 15:40 |
vthapar | tmorin: aha, that does sound a lot cleaner. thanks :) | 15:40 |
*** jbell8 has joined #openstack-neutron | 15:41 | |
*** david-ly_ is now known as david-lyle | 15:41 | |
tmorin | vthapar: not much difference in the end, but yes, cleaner :) | 15:41 |
*** ushkalim has quit IRC | 15:41 | |
*** jbell8 has quit IRC | 15:42 | |
*** Piet has joined #openstack-neutron | 15:43 | |
vthapar | tmorin: bgpvpn_db is defined in base driver? | 15:43 |
*** Sukhdev_ has quit IRC | 15:44 | |
*** yfried_ has quit IRC | 15:44 | |
*** Swami has joined #openstack-neutron | 15:44 | |
*** abregman has quit IRC | 15:46 | |
tmorin | vthapar: in BGPVPNDriverBase which the ODL driver will inherit from, so you'd call self.bgpvpn_db.get_bgpvpn | 15:46 |
*** shivrao has joined #openstack-neutron | 15:46 | |
*** minwang2 has joined #openstack-neutron | 15:46 | |
*** Sukhdev has joined #openstack-neutron | 15:47 | |
*** moshele has quit IRC | 15:47 | |
vthapar | tmorin: I am using class OpenDaylightBgpvpnDriver(driver_api.BGPVPNDriver): | 15:50 |
vthapar | tmorin: is that okay or should I replace it with BGPVPNDriverBase? | 15:50 |
*** nlahouti has joined #openstack-neutron | 15:51 | |
*** krtaylor has joined #openstack-neutron | 15:51 | |
*** pumaranikar has quit IRC | 15:51 | |
*** ByPasS has quit IRC | 15:52 | |
*** dane_leblanc has quit IRC | 15:52 | |
tmorin | vthapar: my understanding is that the ODL driver needs to rely on the Neutron-side DB to persist the BGPVPN objects, if so then you have to inherit from BGPVPNDriverBase and you can use self.bgpvpn_db.get_xxxx | 15:52 |
vthapar | tmorin: yes, it does rely on Neutron-side DB. will use DriverBase then. | 15:53 |
tmorin | vthapar: if not (no persistency Neutron-side) then you should inherit from BGPVPNDriverBase and you can't by definition use the Neutron-side DB | 15:54 |
*** jistr has quit IRC | 15:54 | |
tmorin | vthapar: yes | 15:54 |
*** ByPasS has joined #openstack-neutron | 15:54 | |
*** devlaps has joined #openstack-neutron | 15:54 | |
tmorin | vthapar: same for the bagpipe driver, but it's not yet updated (wip at https://review.openstack.org/224154) | 15:54 |
*** mriedem has left #openstack-neutron | 15:56 | |
*** pece has joined #openstack-neutron | 15:57 | |
vthapar | tmorin: aha, I used that as reference. | 15:57 |
tmorin | vthapar: arf... sorry... | 15:58 |
vthapar | tmorin: np, one of the challenges of working with wip projects. | 15:58 |
tmorin | vthapar: yes | 15:59 |
*** minwang2 has quit IRC | 15:59 | |
*** zhangjn has joined #openstack-neutron | 16:00 | |
tmorin | vthapar: sorry, I'm mixing things up | 16:01 |
tmorin | vthapar: let me recap | 16:01 |
*** evgenyf has quit IRC | 16:01 | |
matrohon | vthapar, hi | 16:01 |
*** pumaranikar has joined #openstack-neutron | 16:01 | |
tmorin | vthapar: a driver that wants Neutron-side persistency needs to inherit from BGPVPNDriver, which the bagpipe driver does already | 16:01 |
tmorin | vthapar: and what you'll do as well | 16:02 |
tmorin | hi matrohon | 16:02 |
*** l6unchpad has joined #openstack-neutron | 16:02 | |
vthapar | matrohorn, hi. | 16:02 |
matrohon | tmorin, vthapar : +1 inherit from BGPVPNDriver and overlod methods that you need | 16:02 |
matrohon | vthapar, you should inherit postcommit methods | 16:03 |
armax | amuller: ping | 16:03 |
amuller | armax: yo | 16:03 |
armax | amuller: yo yo | 16:03 |
armax | amuller: question | 16:03 |
vthapar | matrohorn, tmorin: got it. yep, am overriding only the postcommit, associate and disassociate methods. just had confusion after tmorin's comment on which class to inherit from | 16:03 |
matrohon | vthapar, I just pushed an initial implem of the bagpipe driver, adapted to last API/framework changes | 16:03 |
matrohon | vthapar, https://review.openstack.org/#/c/224154/ | 16:04 |
tmorin | matrohon: you mean overload postcommit methods | 16:04 |
armax | amuller: did you look into the most recent failure modes for the functional job? | 16:04 |
matrohon | tmorin, yep sorry | 16:04 |
vthapar | matrohorn, will take a look. I am trying to get basic driver working and once do some testing will push it for review. | 16:04 |
armax | amuller: it looks like the failure rate is still rather high | 16:04 |
tmorin | vthapar: your driver should overload only the _postcommit methods | 16:04 |
matrohon | vthapar, sounds great | 16:04 |
armax | amuller: with all the recent fixes it looks like we must have pissed it off even further somehow | 16:04 |
*** EinstCrazy has joined #openstack-neutron | 16:04 | |
vthapar | tmorin: what about associate/disassociate? need to pass that information to ODL. | 16:05 |
amuller | armax: I didn't have the chance to look in to in the last week, been in a conference then PTO | 16:05 |
armax | ok | 16:05 |
matrohon | vthapar, you have an associate_postcommit method | 16:05 |
*** hdaniel has quit IRC | 16:05 | |
*** jlanoux has quit IRC | 16:05 | |
matrohon | vthapar, same for disassociate | 16:06 |
*** EinstCrazy has quit IRC | 16:06 | |
*** sballe has joined #openstack-neutron | 16:06 | |
*** slaweq has joined #openstack-neutron | 16:06 | |
vthapar | tmorin, matrohorn: oops. yep, that is what I meant. postcommit methods for create, udpate, delete, associate and disassociate. forgot that the associate one is also postcommit | 16:06 |
Sam-I-Am | sc68cal: moo. | 16:06 |
*** minwang2 has joined #openstack-neutron | 16:07 | |
amuller | armax: wow what happened it really spiked tonight | 16:07 |
amuller | *sighs* | 16:07 |
armax | amuller: I haven’t looked at it yet | 16:07 |
armax | hence I was checking with you first | 16:07 |
*** fawadkhaliq has quit IRC | 16:08 | |
*** EinstCrazy has joined #openstack-neutron | 16:08 | |
*** EinstCrazy has quit IRC | 16:08 | |
*** Marga_ has joined #openstack-neutron | 16:09 | |
*** EinstCrazy has joined #openstack-neutron | 16:09 | |
*** mlavalle has joined #openstack-neutron | 16:09 | |
*** marios has quit IRC | 16:09 | |
*** vivek-ebay has joined #openstack-neutron | 16:09 | |
*** vivek-ebay has quit IRC | 16:09 | |
*** vivek-ebay has joined #openstack-neutron | 16:10 | |
*** saksham has joined #openstack-neutron | 16:10 | |
*** marios has joined #openstack-neutron | 16:10 | |
*** oomichi has joined #openstack-neutron | 16:10 | |
*** safchain has quit IRC | 16:11 | |
*** jbell8 has joined #openstack-neutron | 16:11 | |
vthapar | tmorin: network_id is string or list of network_ids? | 16:11 |
tmorin | vthapar: one network | 16:12 |
*** saksham_ has joined #openstack-neutron | 16:12 | |
tmorin | vthapar: multiple successive associations will result in multiple calls | 16:12 |
tmorin | vthapar: get_vpn(...)['networks'] should give you all the associated networks | 16:13 |
sc68cal | Sam-I-Am: hey dude | 16:13 |
*** vivek-ebay has quit IRC | 16:13 | |
*** saksham_ has left #openstack-neutron | 16:13 | |
*** saksham has quit IRC | 16:14 | |
vthapar | tmorin: ok. I need to append it to existing networks list and then send to ODL. | 16:15 |
*** jpena is now known as jpena|away | 16:15 | |
tmorin | vthapar: no, at the time when associate_network is called, the list already includes the last associate network | 16:15 |
tmorin | vthapar: I mean the get_vpn(...)['networks'] list | 16:16 |
Sam-I-Am | sc68cal: after writing the new content for the install guide, turns out i cant really 'build' option 2 on top of option 1. architecturally it works fine and makes sense, but it requires a lot more 're-editing' of config files thanks to things being scattered all over. | 16:17 |
vthapar | tmorin: got it. so from ODL driver I just need to get_bgpvpn and PUT it to ODL. | 16:17 |
*** minwang2 has quit IRC | 16:17 | |
tmorin | vthapar: yes think so, unless you have a way to add one association via the ODL API | 16:18 |
*** sridhar_ram has joined #openstack-neutron | 16:18 | |
vthapar | tmorin: nope. the way udpates work in ODL driver is we send the updated object in full. | 16:19 |
tmorin | tmorin: ok | 16:19 |
vthapar | tmorin: thanks. I think I got all that I needed for now. | 16:19 |
tmorin | vthapar: ok, great :) | 16:20 |
*** julien_ZTE has quit IRC | 16:20 | |
*** annegentle has quit IRC | 16:20 | |
*** ijw has quit IRC | 16:20 | |
sc68cal | Sam-I-Am: that sounds like something we should fix | 16:21 |
sc68cal | Sam-I-Am: I'm on my way to the airport, how about I ping you tomorrow and we do a hangout to discuss | 16:21 |
*** annegentle has joined #openstack-neutron | 16:21 | |
Sam-I-Am | sc68cal: long story short... i wish we didnt have to go back and edit the ml2 config file, just add l3 and call it good. | 16:21 |
*** ijw has joined #openstack-neutron | 16:21 | |
Sam-I-Am | or edit neutron.conf | 16:21 |
Sam-I-Am | we can chat tomorrow... i have a lot of meetings, but there might be time in the morning | 16:22 |
*** EinstCrazy has quit IRC | 16:22 | |
openstackgerrit | Gal Sagie proposed openstack/neutron-specs: Add Kuryr spec to Neutron https://review.openstack.org/213490 | 16:22 |
amuller | armax: build_name:"gate-neutron-dsvm-functional" AND build_status:"FAILURE" AND tags:"console" AND message:"Finished: FAILURE" | 16:22 |
amuller | armax: I think that should give you a unique list of functional test failures | 16:22 |
amuller | then looking at the last 48 hours, going to each console and seeing what tests failed | 16:23 |
*** sdake_ is now known as sdake | 16:23 | |
amuller | is there an easier test to prioritize test failures according to numbers of occurence? | 16:23 |
amuller | easier way* | 16:23 |
amuller | zuul rechecks tabs is not helping | 16:23 |
*** matrohon has quit IRC | 16:24 | |
*** yamamoto has joined #openstack-neutron | 16:25 | |
*** nikil89_ has quit IRC | 16:25 | |
sc68cal | Sam-I-Am: ok. yeah it would be nice to make config additive and straight forward | 16:25 |
Sam-I-Am | sc68cal: well, i did what i thought might work and its super confusing to a new user. the features are additive, the config not so much :/ | 16:27 |
Sam-I-Am | also, if ubuntu didnt start services by default after install :/ | 16:27 |
Sam-I-Am | i can install the l3 agent on rhel and suse, just not start it | 16:27 |
Sam-I-Am | on ubuntu, its like 'well, install this stuff, but if you want to use option 2, install this one other thing' | 16:28 |
Sam-I-Am | uggghhh | 16:28 |
Sam-I-Am | i'm more than certain the copy-paste club will miss that note | 16:28 |
Sam-I-Am | there's also issues with the ordering of administrative tasks such as database population | 16:29 |
russellb | Sam-I-Am: on a *completely* unrelated note, I've been reading green eggs and ham at least once a night, if not multiple times, for the last couple of weeks with my kid. Now I can't see your IRC nick without the book running through my head. | 16:29 |
Sam-I-Am | russellb: haha. neither can i. | 16:29 |
*** watanabe_isao has joined #openstack-neutron | 16:29 | |
*** yamamoto has quit IRC | 16:29 | |
Sam-I-Am | sc68cal: if you look at the patch now, its using the incremental-but-confusing layout | 16:30 |
Sam-I-Am | im rewriting it another way to see how it looks | 16:30 |
*** beagles is now known as beagles-brb | 16:30 | |
sc68cal | would you run neutron on your box? would you run neutron with a fox? | 16:32 |
anteaya | in a house? | 16:32 |
anteaya | with a mouse? | 16:32 |
armax | amuller: me looks | 16:33 |
Sam-I-Am | sc68cal: btw, if you enable allow_isolated_metadata in the dhcp agent, i think you can use the conventional metadata service with provider networks | 16:33 |
*** watanabe_isao has quit IRC | 16:33 | |
armax | amuller: the process of elimination doesn’t change | 16:33 |
*** breitz has joined #openstack-neutron | 16:33 | |
armax | amuller: it’s the issues that do that | 16:34 |
armax | amuller: I don’t see a consistent failure mode | 16:35 |
amuller | armax: no me neither | 16:35 |
armax | amuller: it’s like the functional job is on a rampage | 16:35 |
*** lazy_prince has quit IRC | 16:35 | |
sc68cal | Sam-I-Am: make sense, since my devstack cluster runs the metadata agent but uses provider nets | 16:35 |
amuller | armax: don't forget that it's legit failures in that graph as well | 16:35 |
armax | amuller: aye | 16:35 |
amuller | armax: anyway just asked a question in openstack-qa you might be interested in as well | 16:35 |
armax | amuller: don’t worry, this isn’t the first time I have done that | 16:35 |
*** nyechiel has quit IRC | 16:36 | |
*** gsagie_ has joined #openstack-neutron | 16:36 | |
armax | amuller: that’s a good one | 16:37 |
*** fawadkhaliq has joined #openstack-neutron | 16:37 | |
*** l6unchpad has quit IRC | 16:37 | |
*** SourabhP has joined #openstack-neutron | 16:38 | |
*** ronis has joined #openstack-neutron | 16:38 | |
*** pc_m_AWAY is now known as pc_m | 16:39 | |
*** cbit has joined #openstack-neutron | 16:40 | |
*** alex_klimov has quit IRC | 16:41 | |
*** cbit has quit IRC | 16:42 | |
*** dannywilson has joined #openstack-neutron | 16:42 | |
*** shashank_hegde has joined #openstack-neutron | 16:42 | |
*** SourabhP has quit IRC | 16:42 | |
*** Jabadia has joined #openstack-neutron | 16:43 | |
*** cbits has quit IRC | 16:43 | |
Jabadia | Hi, question for the experts - what is the behavioural in case of many L2 ( non tcp/udp ) traffic ? | 16:44 |
*** fawadkhaliq has quit IRC | 16:44 | |
Jabadia | I see many drops on the TAP | 16:44 |
*** johnbelamaric has joined #openstack-neutron | 16:45 | |
Jabadia | I assume some table getting filled up. how can I debug it ? how can I improve it ? ( rate is ~50Kpps ) | 16:45 |
*** bharathm has quit IRC | 16:46 | |
*** bharathm has joined #openstack-neutron | 16:46 | |
*** tfukushima has quit IRC | 16:47 | |
*** dannywilson has quit IRC | 16:49 | |
Sukhdev | rkukura: are you here? | 16:49 |
rkukura | Sukhdev: yes | 16:49 |
*** s3wong has joined #openstack-neutron | 16:50 | |
openstackgerrit | Sergey Belous proposed openstack/neutron: [WIP] Add functional test cases for dhcp-agent (OVS) https://review.openstack.org/206033 | 16:51 |
*** fitoduarte has joined #openstack-neutron | 16:54 | |
*** lindycoder has quit IRC | 16:54 | |
*** minwang2 has joined #openstack-neutron | 16:56 | |
*** arajagopal has quit IRC | 16:57 | |
*** arajagopal has joined #openstack-neutron | 16:57 | |
*** _cjones_ has joined #openstack-neutron | 16:58 | |
*** pumaranikar has quit IRC | 16:58 | |
*** annegentle has quit IRC | 16:59 | |
*** yamahata has quit IRC | 17:00 | |
*** pradk has quit IRC | 17:00 | |
*** annegentle has joined #openstack-neutron | 17:00 | |
*** sbalukoff has quit IRC | 17:00 | |
*** beagles-brb is now known as beagles | 17:01 | |
*** SourabhP has joined #openstack-neutron | 17:01 | |
*** Sukhdev has quit IRC | 17:01 | |
*** arajagopal has quit IRC | 17:02 | |
*** pradk has joined #openstack-neutron | 17:02 | |
*** hitalia has joined #openstack-neutron | 17:02 | |
*** l6unchpad has joined #openstack-neutron | 17:02 | |
*** mhickey has quit IRC | 17:04 | |
*** sridhar_ram has quit IRC | 17:05 | |
*** yog_ has quit IRC | 17:06 | |
*** achanda has joined #openstack-neutron | 17:09 | |
armax | amuller: ping | 17:09 |
amuller | armax: pong | 17:09 |
* armax wonders if build_name:"gate-neutron-dsvm-functional" AND build_status:"FAILURE" AND tags:"console" AND message:"Failed: 1" is a better query | 17:10 | |
armax | to weed out the geniune failures | 17:10 |
amuller | that can filter out edge cases, I think it's reasonable | 17:10 |
amuller | I'm currently playing around with the public instance of subunit2sql | 17:11 |
armax | amuller: there’s been a surge around yesterday | 17:11 |
armax | even by taking into account the single failures | 17:11 |
*** tmorin has quit IRC | 17:13 | |
*** yamamoto has joined #openstack-neutron | 17:13 | |
*** l6unchpad has quit IRC | 17:14 | |
armax | amuller: the most offender is this guy: | 17:15 |
armax | message:"in test_killed_monitor_respawns" AND build_status:"FAILURE" | 17:15 |
*** anshul has joined #openstack-neutron | 17:15 | |
armax | the surge is caused by him | 17:15 |
armax | nasty little bugger | 17:16 |
amuller | armax: https://bugs.launchpad.net/neutron/+bug/1495937 | 17:16 |
openstack | Launchpad bug 1495937 in neutron "test_killed_monitor_respawns fails with MismatchError" [Undecided,New] | 17:16 |
armax | 28 hits starting yesterday | 17:16 |
amuller | armax: I think that was fixed by https://review.openstack.org/#/c/223543/ | 17:16 |
armax | ah ah | 17:16 |
armax | something is not right then | 17:17 |
*** fitoduarte has quit IRC | 17:17 | |
armax | I wonder if that patch has had the opposite effect | 17:17 |
*** madhu_ak has joined #openstack-neutron | 17:18 | |
armax | the patch merged around 10PST | 17:18 |
armax | and that’s whe spike popped up | 17:18 |
*** shashank_hegde has quit IRC | 17:19 | |
*** yog_ has joined #openstack-neutron | 17:19 | |
*** julien_ZTE has joined #openstack-neutron | 17:20 | |
*** SourabhP has quit IRC | 17:21 | |
ihrachys | armax: had revert broke smth? | 17:21 |
armax | ihrachys: not sure yet, but something isn’t right | 17:21 |
*** dannywilson has joined #openstack-neutron | 17:22 | |
*** jckasper has quit IRC | 17:22 | |
*** abregman has joined #openstack-neutron | 17:22 | |
*** jbell8 has quit IRC | 17:22 | |
*** shivrao has quit IRC | 17:23 | |
* ihrachys needs to run. update me for tomorrow if my involvement is needed around the revert | 17:23 | |
armax | ihrachys: all I can notice is the correlation between the merge of https://review.openstack.org/#/c/223543/ and spike on test_killed_monitor_respawns fails with MismatchError | 17:23 |
*** flaviof_ is now known as flaviof | 17:24 | |
armax | ihrachys, HenryG: we didn’t have enough data to confirm that patch 223543 indeed fixed the issue | 17:24 |
armax | ihrachys, HenryG: we only ran the functional job once | 17:24 |
armax | in that patch | 17:24 |
*** fitoduarte has joined #openstack-neutron | 17:24 | |
ihrachys | armax: you can be right. but the patch does not look correct to me in the first place. | 17:25 |
ihrachys | see comments at https://review.openstack.org/#/c/223543/1/neutron/agent/linux/async_process.py | 17:25 |
*** vthapar has quit IRC | 17:25 | |
armax | ihrachys: so you’re saying that the revert has still some grounds? | 17:25 |
ihrachys | assuming I read the code correctly. | 17:26 |
ihrachys | you can check me | 17:26 |
armax | ihrachys: I am not thinking of reverting the revert..there must be something else at play | 17:26 |
*** julien_ZTE has quit IRC | 17:26 | |
ihrachys | armax: link for test_killed_monitor_respawns failure? I still have 10 mins for that. :) | 17:26 |
armax | ihrachys: https://bugs.launchpad.net/neutron/+bug/1495937 | 17:27 |
openstack | Launchpad bug 1495937 in neutron "test_killed_monitor_respawns fails with MismatchError" [Critical,Confirmed] | 17:27 |
armax | got the logstash query in tehre | 17:27 |
*** SumitNaiksatam has quit IRC | 17:27 | |
ihrachys | armax: oh I think I start to get it. so it's not that revert triggered it, but that it did not fix the issue. | 17:28 |
armax | ihrachys: aye | 17:28 |
armax | ihrachys: I mean there’s certainly a correlation between the spike and the revert | 17:28 |
*** yamahata has joined #openstack-neutron | 17:28 | |
armax | but the revert might have simply initiated a different failure mode | 17:28 |
ihrachys | armax: so my idea was that ovsdb monitor is not killed, so it does not output any new data to the stdout, hence no actual (='') | 17:29 |
*** shivrao has joined #openstack-neutron | 17:29 | |
armax | the patch that was reverted had to be reverted anyway | 17:29 |
armax | it changed the rootwrap filters | 17:29 |
armax | that at least ought be tagged with UpgradeImpact to be nice | 17:30 |
armax | but meh | 17:30 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver https://review.openstack.org/207635 | 17:30 |
*** sridhar_ram has joined #openstack-neutron | 17:30 | |
openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Make chown rootwrap filter ipsec.secrets file specific https://review.openstack.org/224133 | 17:35 |
*** tflynn has joined #openstack-neutron | 17:38 | |
*** shashank_hegde has joined #openstack-neutron | 17:41 | |
*** sridhar_ram has quit IRC | 17:42 | |
*** lindycoder has joined #openstack-neutron | 17:43 | |
*** sridhar_ram has joined #openstack-neutron | 17:43 | |
*** arajagopal has joined #openstack-neutron | 17:43 | |
*** yog_ has quit IRC | 17:43 | |
*** obondarev has quit IRC | 17:43 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: wait for ovsdb-monitor to bootstrap before reading the stdout https://review.openstack.org/224209 | 17:44 |
*** arajagopal has joined #openstack-neutron | 17:44 | |
ihrachys | armax: I would give it a shot ^^ needs recheck stats | 17:44 |
armax | ihrachys: me looks | 17:44 |
armax | ihrachys: I’ll keep an eye on it | 17:45 |
ihrachys | that sleep should also give a chance for the green thread that reads from stdout to trigger | 17:45 |
armax | ihrachys: thanks for kicking ass so quickly | 17:45 |
ihrachys | (I presume sleep means eventlet context switch) | 17:45 |
armax | ihrachys: ya | 17:45 |
ihrachys | ok, now I really run away. my wife will kill me now. :P | 17:46 |
armax | ihrachys: and no-one wants that | 17:46 |
*** l6unchpad has joined #openstack-neutron | 17:46 | |
*** yog_ has joined #openstack-neutron | 17:46 | |
*** ihrachys has quit IRC | 17:46 | |
*** tfukushima has joined #openstack-neutron | 17:48 | |
*** flaviof is now known as flaviof_ | 17:49 | |
*** pradk has quit IRC | 17:50 | |
*** ivar-lazzaro has joined #openstack-neutron | 17:50 | |
*** ushkalim has joined #openstack-neutron | 17:50 | |
*** fawadkhaliq has joined #openstack-neutron | 17:50 | |
*** ociuhandu has quit IRC | 17:52 | |
*** charlesr has quit IRC | 17:52 | |
*** crose has quit IRC | 17:52 | |
*** imcsk8_ is now known as imcsk8 | 17:52 | |
*** tfukushima has quit IRC | 17:52 | |
*** crose has joined #openstack-neutron | 17:52 | |
*** ociuhandu has joined #openstack-neutron | 17:53 | |
*** ivar-lazzaro has quit IRC | 17:53 | |
*** beagles has quit IRC | 17:53 | |
*** ivar-lazzaro has joined #openstack-neutron | 17:54 | |
*** sdake_ has joined #openstack-neutron | 17:54 | |
*** ivar-laz_ has joined #openstack-neutron | 17:55 | |
*** ociuhandu has quit IRC | 17:57 | |
*** sdake has quit IRC | 17:58 | |
*** ivar-lazzaro has quit IRC | 17:59 | |
*** padkrish has joined #openstack-neutron | 17:59 | |
*** MVenesio has quit IRC | 17:59 | |
*** abhiraut has joined #openstack-neutron | 17:59 | |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 17:59 |
ritesh | amuller: ^^^ kindly take a look | 18:00 |
*** pumaranikar has joined #openstack-neutron | 18:00 | |
*** LukeHinds has quit IRC | 18:05 | |
*** pradk has joined #openstack-neutron | 18:05 | |
*** abhiraut has quit IRC | 18:05 | |
armax | russellb: ping | 18:07 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Provide Endpoint groups capability. https://review.openstack.org/212692 | 18:08 |
*** l6unchpad has quit IRC | 18:08 | |
*** sbalukoff has joined #openstack-neutron | 18:09 | |
*** nyechiel has joined #openstack-neutron | 18:10 | |
*** ritesh has quit IRC | 18:10 | |
*** Guest91493 has joined #openstack-neutron | 18:10 | |
*** pumaranikar has quit IRC | 18:11 | |
*** thurloat_isgone is now known as thurloat | 18:12 | |
*** annegentle has quit IRC | 18:12 | |
*** annegentle has joined #openstack-neutron | 18:12 | |
*** padkrish has quit IRC | 18:14 | |
*** ZZelle_ has joined #openstack-neutron | 18:14 | |
*** openstackgerrit has quit IRC | 18:16 | |
*** pece has quit IRC | 18:16 | |
*** openstackgerrit has joined #openstack-neutron | 18:17 | |
*** nyechiel has quit IRC | 18:17 | |
*** jprovazn has quit IRC | 18:17 | |
*** padkrish_ has joined #openstack-neutron | 18:18 | |
*** tfukushima has joined #openstack-neutron | 18:18 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Provide Endpoint groups capability. https://review.openstack.org/212692 | 18:20 |
*** ritesh has joined #openstack-neutron | 18:21 | |
*** Sukhdev_ has joined #openstack-neutron | 18:22 | |
*** numans has joined #openstack-neutron | 18:22 | |
*** Guest91493 has quit IRC | 18:22 | |
*** Guest91493 has joined #openstack-neutron | 18:22 | |
*** Guest91493 is now known as arosen | 18:22 | |
*** amuller is now known as amuller_afk | 18:22 | |
apuimedo | irenab: gsagie: let's get banix's https://review.openstack.org/#/c/222960/ in | 18:22 |
*** changbl has quit IRC | 18:23 | |
*** anilvenkata has joined #openstack-neutron | 18:23 | |
*** anilvenkata has left #openstack-neutron | 18:23 | |
*** tfukushima has quit IRC | 18:23 | |
*** anilvenkata has joined #openstack-neutron | 18:23 | |
*** mickeys has quit IRC | 18:23 | |
*** achanda has quit IRC | 18:25 | |
*** saksham has joined #openstack-neutron | 18:26 | |
*** oomichi has quit IRC | 18:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Zanata https://review.openstack.org/224245 | 18:27 |
*** carl_baldwin has quit IRC | 18:27 | |
*** david-lyle has quit IRC | 18:30 | |
*** boris-42 has quit IRC | 18:30 | |
*** padkrish_ has quit IRC | 18:32 | |
*** padkrish has joined #openstack-neutron | 18:32 | |
*** padkrish has quit IRC | 18:33 | |
*** padkrish has joined #openstack-neutron | 18:34 | |
*** sthillma has joined #openstack-neutron | 18:34 | |
*** mlavalle has quit IRC | 18:34 | |
openstackgerrit | Paul Michali proposed openstack/python-neutronclient: Enable VPN test cases https://review.openstack.org/214587 | 18:35 |
*** arajagopal has quit IRC | 18:35 | |
*** arajagopal has joined #openstack-neutron | 18:36 | |
*** ivar-laz_ has quit IRC | 18:38 | |
*** jckasper has joined #openstack-neutron | 18:38 | |
*** ivar-lazzaro has joined #openstack-neutron | 18:39 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Test code added to check the ARP update failures https://review.openstack.org/224250 | 18:39 |
*** arajagopal has quit IRC | 18:41 | |
*** yfried_ has joined #openstack-neutron | 18:42 | |
*** tkatarki has quit IRC | 18:44 | |
*** aix has quit IRC | 18:46 | |
*** dannywilson has quit IRC | 18:46 | |
*** obondarev has joined #openstack-neutron | 18:47 | |
*** anilvenkata has left #openstack-neutron | 18:47 | |
*** yog_ has quit IRC | 18:48 | |
*** padkrish has quit IRC | 18:48 | |
*** matrohon has joined #openstack-neutron | 18:49 | |
*** padkrish has joined #openstack-neutron | 18:49 | |
*** tkatarki has joined #openstack-neutron | 18:50 | |
*** jwy has joined #openstack-neutron | 18:50 | |
openstackgerrit | Gal Sagie proposed openstack/neutron-specs: Add Kuryr spec to Neutron https://review.openstack.org/213490 | 18:52 |
*** l6unchpad has joined #openstack-neutron | 18:52 | |
*** padkrish has quit IRC | 18:53 | |
*** jwy has quit IRC | 18:53 | |
*** amuller_afk is now known as amuller | 18:53 | |
openstackgerrit | Paul Michali proposed openstack/python-neutronclient: CRUD for VPN endpoint group API https://review.openstack.org/219455 | 18:54 |
*** nijaba_ has joined #openstack-neutron | 18:54 | |
*** nijaba_ has quit IRC | 18:54 | |
*** nijaba_ has joined #openstack-neutron | 18:54 | |
*** nijaba has quit IRC | 18:55 | |
*** jwy has joined #openstack-neutron | 18:55 | |
*** korzen has joined #openstack-neutron | 18:56 | |
korzen | Hi, does anybody know how to do data migration while online schema migration BP was implemented in Librerty? | 18:57 |
korzen | As i understand, in first move I have to do neutron-db-manage upgrade --contract | 18:58 |
korzen | and this is only schema migration | 18:58 |
*** pradk has quit IRC | 18:58 | |
korzen | how about data migration? | 18:58 |
*** arajagopal has joined #openstack-neutron | 18:58 | |
*** arajagopal has joined #openstack-neutron | 18:59 | |
*** pradk has joined #openstack-neutron | 19:00 | |
*** bharathm has quit IRC | 19:00 | |
*** padkrish has joined #openstack-neutron | 19:02 | |
*** pumaranikar has joined #openstack-neutron | 19:03 | |
*** fawadkhaliq has quit IRC | 19:04 | |
*** nyechiel has joined #openstack-neutron | 19:08 | |
*** wasmum- has quit IRC | 19:08 | |
*** julien_ZTE has joined #openstack-neutron | 19:09 | |
azbiswas | Hi does anyone know if there is a limit of 1000 networks in devstack? I'm seeing the following error "Unable to create the network. No tenant network is available for allocation." after around 1000 networks. | 19:10 |
amuller | azbiswas: check out /etc/neutron/ml2_conf.ini | 19:10 |
*** sudipto has quit IRC | 19:10 | |
amuller | azbiswas: either the tunneling or vlan ranges must be set to a range of 1000 | 19:10 |
*** gsagie_ has quit IRC | 19:11 | |
*** l6unchpad has quit IRC | 19:12 | |
azbiswas | amuller: thanks that's what I thought | 19:12 |
*** achanda has joined #openstack-neutron | 19:12 | |
*** Leo_ has joined #openstack-neutron | 19:13 | |
azbiswas | amuller: So there no way to increase those limits? | 19:13 |
*** Leo_ has quit IRC | 19:13 | |
amuller | azbiswas: just change them and restart the neutron server | 19:13 |
*** julien_ZTE has quit IRC | 19:13 | |
*** Leo_ has joined #openstack-neutron | 19:14 | |
azbiswas | amuller: thanks | 19:14 |
*** lori|away is now known as lori | 19:15 | |
*** ihrachys has joined #openstack-neutron | 19:16 | |
*** padkrish has quit IRC | 19:18 | |
*** pumaranikar has quit IRC | 19:18 | |
korzen | anyone on db data migration? | 19:19 |
*** padkrish has joined #openstack-neutron | 19:19 | |
*** sudipto has joined #openstack-neutron | 19:21 | |
*** abregman has quit IRC | 19:22 | |
*** tbachman has quit IRC | 19:22 | |
*** padkrish has quit IRC | 19:23 | |
*** moshele has joined #openstack-neutron | 19:24 | |
*** aix has joined #openstack-neutron | 19:25 | |
*** pumaranikar has joined #openstack-neutron | 19:25 | |
*** pumaranikar has quit IRC | 19:26 | |
*** padkrish_ has joined #openstack-neutron | 19:26 | |
*** alex_klimov has joined #openstack-neutron | 19:27 | |
*** jbell8 has joined #openstack-neutron | 19:30 | |
*** jbell8 has quit IRC | 19:32 | |
*** jbell8 has joined #openstack-neutron | 19:32 | |
*** abregman has joined #openstack-neutron | 19:33 | |
*** jbell8 has quit IRC | 19:35 | |
*** tidwellr has left #openstack-neutron | 19:36 | |
*** sudipto has quit IRC | 19:36 | |
*** jbell8 has joined #openstack-neutron | 19:36 | |
*** abregman has quit IRC | 19:37 | |
*** wasmum has joined #openstack-neutron | 19:37 | |
*** mlavalle has joined #openstack-neutron | 19:38 | |
*** ihrachys has quit IRC | 19:38 | |
*** ihrachys has joined #openstack-neutron | 19:39 | |
*** achanda_ has joined #openstack-neutron | 19:40 | |
*** achanda has quit IRC | 19:40 | |
*** Leo_ has quit IRC | 19:42 | |
*** Leom has joined #openstack-neutron | 19:42 | |
*** nyechiel has quit IRC | 19:43 | |
*** saksham has left #openstack-neutron | 19:43 | |
*** nlahouti has quit IRC | 19:44 | |
*** bharathm has joined #openstack-neutron | 19:44 | |
*** blallau has joined #openstack-neutron | 19:45 | |
*** padkrish_ has quit IRC | 19:47 | |
*** jbell8 has quit IRC | 19:47 | |
*** padkrish has joined #openstack-neutron | 19:48 | |
*** sdake has joined #openstack-neutron | 19:49 | |
*** bharathm has quit IRC | 19:49 | |
*** tbachman has joined #openstack-neutron | 19:52 | |
*** sdake_ has quit IRC | 19:52 | |
*** dkehn has quit IRC | 19:53 | |
*** jbell8 has joined #openstack-neutron | 19:54 | |
*** padkrish has quit IRC | 19:55 | |
*** oomichi has joined #openstack-neutron | 19:55 | |
*** padkrish has joined #openstack-neutron | 19:55 | |
*** jbell8_ has joined #openstack-neutron | 19:57 | |
*** ihrachys has quit IRC | 19:57 | |
*** jbell8 has quit IRC | 19:57 | |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 19:58 |
*** shashank_hegde has quit IRC | 19:59 | |
*** padkrish has quit IRC | 19:59 | |
*** openstackstatus has quit IRC | 20:02 | |
*** mestery has quit IRC | 20:02 | |
*** shashank_hegde has joined #openstack-neutron | 20:03 | |
*** jpena|away is now known as jpena | 20:03 | |
*** mestery has joined #openstack-neutron | 20:03 | |
*** openstackstatus has joined #openstack-neutron | 20:04 | |
*** ChanServ sets mode: +v openstackstatus | 20:04 | |
*** sambetts has quit IRC | 20:05 | |
*** mickeys has joined #openstack-neutron | 20:05 | |
*** carl_baldwin has joined #openstack-neutron | 20:05 | |
*** dkehn has joined #openstack-neutron | 20:06 | |
*** abhiraut has joined #openstack-neutron | 20:06 | |
*** annemccormick has quit IRC | 20:06 | |
*** bharathm has joined #openstack-neutron | 20:07 | |
*** Aish has joined #openstack-neutron | 20:07 | |
madhu_ak | pc_m any idea where the tempest config file is actually picking when running vpn api tests? shouldn't it pick from /opt/stack/tempest/etc? | 20:08 |
*** stevemar has quit IRC | 20:08 | |
*** stevemar has joined #openstack-neutron | 20:09 | |
pc_m | madhu_ak: I don't know. | 20:09 |
*** sambetts has joined #openstack-neutron | 20:10 | |
madhu_ak | pc_m because, api tests doesnt run though: https://review.openstack.org/#/c/211381/ so, I thought to check with you | 20:11 |
madhu_ak | anyways, no worries | 20:11 |
*** stevemar has quit IRC | 20:11 | |
*** tkatarki has quit IRC | 20:13 | |
*** flaviof_ has quit IRC | 20:13 | |
*** Sukhdev_ has quit IRC | 20:13 | |
pc_m | madhu_ak: RE your comments on 211381 | 20:14 |
madhu_ak | amuller: checking where I might be doing wrong related to the tempest config when running vpn api tests: https://review.openstack.org/#/c/211381/ | 20:14 |
madhu_ak | pc_m yep | 20:14 |
pc_m | madhu_ak: Are you saying you're copying lib files that will eventually be in the library and then can be removed from VPN? | 20:15 |
madhu_ak | pc_m nope, I was saying, rather than relying picking up files from neutron project, I felt moving those required files called by vpn tests directly seems a good point for me..its just IMO.. | 20:17 |
madhu_ak | pc_m not sure of your intentions though.. | 20:17 |
*** alex_klimov has quit IRC | 20:17 | |
*** beagles has joined #openstack-neutron | 20:18 | |
*** tfukushima has joined #openstack-neutron | 20:19 | |
openstackgerrit | Miguel Lavalle proposed openstack/neutron: External DNS driver reference implementation https://review.openstack.org/212213 | 20:19 |
pc_m | madhu_ak: Well they're copies of neutron files. Concern of all the duplication. Are the neutron files (these common ones) moving to tempest lib? | 20:20 |
*** Piet has quit IRC | 20:20 | |
madhu_ak | pc_m not all those nuetron files, its just a subset of tempest files requied for running vpn tests.. | 20:21 |
*** tkatarki has joined #openstack-neutron | 20:21 | |
openstackgerrit | Merged openstack/neutron: Enable servicing lbaasV2 vip by DVR https://review.openstack.org/222135 | 20:22 |
pc_m | madhu_ak: But we're talking about 20-30 files that are identical to what is in neutron right? | 20:22 |
*** gberginc has quit IRC | 20:22 | |
madhu_ak | you referencing the api test file? | 20:24 |
*** tfukushima has quit IRC | 20:24 | |
*** Prem has quit IRC | 20:24 | |
openstackgerrit | Merged openstack/neutron: Add IPv6 Address Resolution protection https://review.openstack.org/201650 | 20:25 |
*** matrohon has quit IRC | 20:25 | |
openstackgerrit | Merged openstack/neutron: ovsdb: Fix a few docstring https://review.openstack.org/223917 | 20:25 |
openstackgerrit | Merged openstack/neutron: Use tempest-lib's token_client https://review.openstack.org/224006 | 20:25 |
openstackgerrit | Aaron Rosen proposed openstack/neutron: ipam: Prevent none from being passed to delete https://review.openstack.org/224289 | 20:25 |
madhu_ak | pc_m they are. but I thought, vpn tests can handle independently without relying on files present in neutron. | 20:26 |
pc_m | madhu_ak: The ones in common and services, etc. | 20:26 |
*** jbell8_ has quit IRC | 20:26 | |
madhu_ak | pc_m yes, those are common files (only the files specifically used in api tests) | 20:27 |
pc_m | I mean we're pulling in 8771 lines of code for a 317 line test module. | 20:27 |
openstackgerrit | Merged openstack/neutron: Add QoS fullstack test https://review.openstack.org/215688 | 20:27 |
madhu_ak | pc_m yes, those are copied.. I guess l am following as per the tests used in lbaas | 20:27 |
madhu_ak | pc_m https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/README.rst | 20:28 |
pc_m | I guess I don't think that was a good decision either. | 20:29 |
madhu_ak | pc_m maybe we need to have discussion in the next vpn meeting? | 20:32 |
*** korzen has quit IRC | 20:32 | |
openstackgerrit | Merged openstack/neutron: Fix establishing UDP connection https://review.openstack.org/212425 | 20:34 |
pc_m | yes. I responded in the review. Would like to see other cores' opinion here. | 20:34 |
madhu_ak | pc_m sure, thanks for your comment. sounds good | 20:35 |
*** moha_hunt has joined #openstack-neutron | 20:39 | |
*** ajmiller has quit IRC | 20:39 | |
*** ajmiller has joined #openstack-neutron | 20:40 | |
*** mwagner_lap has quit IRC | 20:44 | |
*** carora has joined #openstack-neutron | 20:47 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fixes SNAT port not found for internal port https://review.openstack.org/224250 | 20:51 |
openstackgerrit | Edgar Magana proposed openstack/neutron: In OVS agent segmentation_id can't be unicode None https://review.openstack.org/224297 | 20:51 |
Swami | carl_baldwin: ping | 20:52 |
carl_baldwin | Swami: pong | 20:52 |
Swami | carl_baldwin: I found another bug which was caused by the refactor. | 20:52 |
Swami | carl_baldwin: can you take a look at 224250 | 20:53 |
Swami | carl_baldwin: When we did a patch update last week we might have missed this use case of deleting a gateway. | 20:53 |
carl_baldwin | Looking... | 20:54 |
Swami | carl_baldwin: thanks | 20:54 |
*** leitan has quit IRC | 20:54 | |
*** leitan has joined #openstack-neutron | 20:54 | |
amuller | kevinbenton: Am I crazy or is port security not available via the CLI? | 20:55 |
*** julien_ZTE has joined #openstack-neutron | 20:55 | |
amuller | kevinbenton: oh it's --no-security-groups nm | 20:56 |
amuller | err, no it isn;t | 20:57 |
*** alex_klimov has joined #openstack-neutron | 20:57 | |
*** blallau has quit IRC | 20:58 | |
kevinbenton | amuller: --port-security-enabled=False | 20:58 |
*** crose has quit IRC | 20:58 | |
amuller | kevinbenton: why do we keep merging crap to our client without help text | 20:58 |
*** padkrish has joined #openstack-neutron | 20:59 | |
*** julien_ZTE has quit IRC | 21:00 | |
*** tiswanso has quit IRC | 21:00 | |
carora | Hi Neutron, I posted my question at #openstack, but that didn't work. So I am asking my question here. I want to use my domain name in all the VMs. When I spin the vm, I want domain name to show at /etc/resolv.conf | 21:00 |
*** dkehn has quit IRC | 21:00 | |
*** Piet has joined #openstack-neutron | 21:00 | |
kevinbenton | amuller: because help is for noobs | 21:00 |
kevinbenton | amuller: :) | 21:00 |
*** ijw_ has joined #openstack-neutron | 21:01 | |
kevinbenton | amuller: i'm not sure how that slipped through | 21:01 |
*** dkehn has joined #openstack-neutron | 21:02 | |
*** padkrish has quit IRC | 21:03 | |
*** regXboi has joined #openstack-neutron | 21:03 | |
*** flaviof has joined #openstack-neutron | 21:03 | |
*** tflynn_ has joined #openstack-neutron | 21:04 | |
*** nmagnezi has joined #openstack-neutron | 21:06 | |
*** nlahouti has joined #openstack-neutron | 21:07 | |
*** padkrish has joined #openstack-neutron | 21:08 | |
*** tflynn has quit IRC | 21:08 | |
regXboi | carl_baldwin: following up yesterday afternoon - is there any way via local.conf and/or .ini files to force the is_extension_supported condition to be false? | 21:08 |
regXboi | or am I looking at a code hack workaround? | 21:08 |
*** ushkalim has quit IRC | 21:08 | |
kevinbenton | regXboi: are you talking about the test skips? | 21:08 |
*** pc_m has quit IRC | 21:09 | |
regXboi | kevinbenton: no I'm talking about Ls 80-81 of l3_rpc.py | 21:09 |
carl_baldwin | regXboi: To what end? | 21:09 |
regXboi | carl_baldwin: so that I can test that the else branch works and has linear behavior | 21:09 |
regXboi | carl_baldwin: I've got an operator that is burning on this point and so I need a work around while the problem gets fixed | 21:10 |
regXboi | carl_baldwin: I can hack the code, but that's just plain ugly | 21:10 |
kevinbenton | regXboi: oh, you will need to hack code | 21:10 |
kevinbenton | regXboi: you can't change what a plugin supports via config | 21:10 |
*** nmagnezi has quit IRC | 21:10 | |
regXboi | kevinbenton: I was really afraid of that | 21:11 |
* regXboi wonders if the upcoming bug just went critical because of that | 21:11 | |
kevinbenton | regXboi: what version of code are you running? | 21:11 |
regXboi | the operator is running kilo | 21:12 |
regXboi | but, that specific code pattern is the same as Liberty | 21:12 |
*** jbell8 has joined #openstack-neutron | 21:12 | |
*** ociuhandu has joined #openstack-neutron | 21:12 | |
*** obondarev has quit IRC | 21:13 | |
carl_baldwin | regXboi: I had a start on a fix yesterday. Let me see if I can finish it up before I have to leave. | 21:13 |
regXboi | carl_baldwin: ack - I'm still going to need to test a workaround tomorrow morning as a fallback | 21:14 |
carl_baldwin | ok | 21:14 |
*** yamahata has quit IRC | 21:14 | |
carl_baldwin | regXboi: I think kevinbenton is right, you’ll have to change code. | 21:14 |
*** ijw_ has quit IRC | 21:14 | |
*** jbell8 has quit IRC | 21:15 | |
*** ijw_ has joined #openstack-neutron | 21:15 | |
kevinbenton | regXboi: https://review.openstack.org/#/c/174845/ | 21:15 |
regXboi | carl_baldwin, kevinbenton: ack - that is what I was afraid of looking this afternoon but wanted to confirm before I went the wrong way | 21:15 |
*** nmagnezi has joined #openstack-neutron | 21:15 | |
regXboi | kevinbenton: that doesn't help | 21:16 |
regXboi | the problem exists in tip of tree Liberty | 21:16 |
*** crose has joined #openstack-neutron | 21:16 | |
*** thorst has quit IRC | 21:16 | |
carl_baldwin | regXboi: Do you have a link to the bug? | 21:16 |
regXboi | carl_baldwin: haven't filed yet, I just got back | 21:16 |
kevinbenton | regXboi: and you're certain it's the same issue? | 21:17 |
*** TrevorV has quit IRC | 21:17 | |
regXboi | kevinbenton: so here's the issue | 21:17 |
kevinbenton | regXboi: because i spent a bunch of time profiling the l3 rpc calls | 21:17 |
kevinbenton | regXboi: and at the start of liberty merged the major issues | 21:17 |
carl_baldwin | regXboi: ack | 21:17 |
regXboi | kevinbenton: here's the issue: | 21:17 |
regXboi | when I schedule a legacy router and have DVR turned off, the scheduling code still ends up running through code on the server that asks for all of the ports on the network node | 21:18 |
regXboi | which means that as the number of routers on the node increases, the time to add another node increases | 21:18 |
regXboi | the code in question is in _get_dvr_sync_data in db/l3_dvr_db.py: | 21:19 |
regXboi | port_filter = {portbindings.HOST_ID: [host]} | 21:20 |
regXboi | ports = self._core_plugin.get_ports(context, port_filter) | 21:20 |
regXboi | kevinbenton: if you can tell me a way to modify local.conf/*.ini to avoid that path, I'm all ears | 21:20 |
*** arajagopal has quit IRC | 21:20 | |
*** mickeys has quit IRC | 21:21 | |
*** arajagopal has joined #openstack-neutron | 21:21 | |
*** mickeys has joined #openstack-neutron | 21:21 | |
regXboi | kevinbenton: sc68cal has the stack trace here: http://paste.openstack.org/show/463640/ | 21:22 |
armax | ZZelle: ping | 21:22 |
kevinbenton | regXboi: i see. yeah, i don't think you can skip | 21:22 |
ZZelle_ | armax, hi | 21:22 |
kevinbenton | regXboi: but a get_ports call shouldn't be that expensive. is this just ML2? | 21:22 |
armax | ZZelle_: I noticed you filed bug 1496586 | 21:23 |
openstack | bug 1496586 in neutron "Avoid the pattern sql select/delete" [Undecided,New] https://launchpad.net/bugs/1496586 - Assigned to Cedric Brandily (cbrandily) | 21:23 |
armax | kudos to you | 21:23 |
regXboi | kevinbenton: it scales linerally in the number of ports | 21:23 |
regXboi | so when I want 100s or more of routers on a network node, the time for the router to come up exceeds the service budget | 21:23 |
*** Aish_ has joined #openstack-neutron | 21:23 | |
regXboi | and doing that for a non-dvr node is ... well ... silly | 21:23 |
regXboi | er non-dvr router | 21:24 |
regXboi | (imho) | 21:24 |
armax | ZZelle_: I have been thinking that we should document a number of Neutron developer common pitfalls somewhere in the devref | 21:24 |
*** Aish_ has quit IRC | 21:24 | |
ZZelle_ | armax, it's based on your comment in a review :) | 21:24 |
armax | ZZelle_: and this is a classic example I have seen throughtout the codebase | 21:24 |
armax | ZZelle_: right | 21:24 |
kevinbenton | regXboi: it may scale linearly, but it should be able to retrieve hundreds of ports in < 1 sec | 21:24 |
armax | I have been meaning to stop me saying the same thing over and over and over :) | 21:24 |
regXboi | kevinbenton: doesn't matter - I'm looking to remove any O(n) I can | 21:25 |
armax | ZZelle_: I thought of creating a section in the devref where we document the most recurrent mistakes/reviewer comments | 21:25 |
*** Aish has quit IRC | 21:25 | |
armax | so that we can say to contributors: go check that out and come back when you’ve done your homework | 21:25 |
*** arajagopal has quit IRC | 21:25 | |
ZZelle_ | armax, should we document it in neutron or oslo.db as there is already some guidelines in oslo.db code (but who read them?) | 21:25 |
*** boris-42 has joined #openstack-neutron | 21:26 | |
armax | ZZelle_: well this one in particular is an anti-pattern that goes beyond the way we use oslo.db | 21:26 |
armax | or sqlalchemy for that matter | 21:26 |
kevinbenton | regXboi: it most certainly does. we don't want to waste time trying to optimize an operation that is 1 db call either way | 21:26 |
kevinbenton | regXboi: something has gone really wrong with get_ports if it's taking that long | 21:26 |
regXboi | kevinbenton: the requirement is that when an instance comes up, the network port has to be available within single seconds | 21:27 |
armax | ZZelle_: I was wondering if you were interested in drafting a document like that | 21:27 |
regXboi | today at scale, it takes anywhere from 45 - 60 seconds | 21:27 |
*** Sukhdev has joined #openstack-neutron | 21:27 | |
kevinbenton | regXboi: and it should be. if the bottleneck is the get_ports call then something is badly broken in there. | 21:28 |
regXboi | kevinbenton: I think we are talking past each other | 21:28 |
armax | I can have a first stab at it, but I think it’s only fair to start documenting these types of issues alongside the fixes | 21:28 |
ZZelle_ | armax, i am in! | 21:28 |
armax | to make sure we don’t make the same mistakes | 21:28 |
armax | ZZelle_: cool | 21:28 |
regXboi | kevinbenton: because I don't see trying to remove linear behavior as a "waste as time" I see it as "making the code scalable" | 21:28 |
armax | ZZelle_: I am sure that amuller will have plenty of best practices and anti-pattern for tests | 21:29 |
kevinbenton | regXboi: it's a waste of time because we're trying to work around a larger problem. get_ports calls need to be cheap because we use them everywhere | 21:29 |
armax | and kevinbenton can provide input on how to do things in bulk | 21:29 |
amuller | regXboi: the goal is not to make code scale one way or another with respect to N, it's to remove bottlenecks | 21:29 |
regXboi | kevinbenton: I don't think the problem is the get_ports call, I think it is the "all ports on the host" filter | 21:29 |
amuller | regXboi: you've profiled the code and identified that get_ports call to be where most of the time is spent? | 21:30 |
kevinbenton | regXboi: but my point is that all ports on a host should be retrievable in a very short amount of time. even if it's not the most optimal thing to ask for | 21:30 |
regXboi | and I question why I need to run dvr code for a non-dvr router | 21:30 |
*** changbl has joined #openstack-neutron | 21:30 | |
kevinbenton | regXboi: certainly go forward with reducing what it asks for, but i'm saying it sounds like we have a much larger lurking issue | 21:30 |
regXboi | amuller: in scheduling a router - yes | 21:30 |
regXboi | kevinbenton: ack - *that* I agree with | 21:30 |
amuller | yeah that's troubling =D | 21:30 |
regXboi | kevinbenton, amuller: in fact, I'm planning on proposing a three phase patch set: | 21:31 |
amuller | regXboi: in get_ports is there any time spent outside of the main sqlalchemy query that retrives the port? | 21:31 |
amuller | s | 21:31 |
armax | regXboi, amuller: sorry if I interject but is this discussion on how to improve performance of the dvr code path? | 21:31 |
regXboi | amuller: my memory is no, but I have to go double check the data | 21:31 |
regXboi | armax: NO | 21:31 |
* armax scrolls back even further | 21:32 | |
amuller | regXboi: This is not the only place I've seen DVR code slow down non-DVR code BTW | 21:32 |
regXboi | armax: this is a legacy router scheduling getting held up when DVR is turned off because the code forces calls to DVR code | 21:32 |
*** padkrish has quit IRC | 21:32 | |
regXboi | amuller: ack | 21:32 |
regXboi | anyway - the patch sets | 21:32 |
regXboi | (1) change non-entry/exit debug statements to info | 21:32 |
*** padkrish has joined #openstack-neutron | 21:33 | |
regXboi | (2) change default log level in neutron devstack to info level | 21:33 |
regXboi | (3) put in entry and exit statements at the debug level *everywhere* | 21:33 |
kevinbenton | info log statements need to be a single unit of work | 21:33 |
armax | regXboi: ic | 21:33 |
kevinbenton | so that won't fly | 21:33 |
regXboi | so that we don't kill infra, but can have full instrumentation at debug level if we need to profile things | 21:33 |
kevinbenton | there is some document describing what we can log at the info level | 21:34 |
kevinbenton | since it's operator facing | 21:34 |
kevinbenton | that applies to all of openstack | 21:34 |
regXboi | kevinbenton: I'd like a reference to that | 21:34 |
kevinbenton | regXboi: let me see if i can find it | 21:34 |
regXboi | kevinbenton: because the alternative is pretty ugly - hacking a higher debug level | 21:34 |
regXboi | kevinbenton: I'd actually probably be in favor of that document | 21:34 |
regXboi | because I doubt we are conforming to it today | 21:35 |
regXboi | we == neutron | 21:35 |
amuller | regXboi: try profiling update_device_up in ml2, most of the time spent is in DVR code (Even if the system doesn't have a single router defined, much less a distributed one) | 21:35 |
*** julim has quit IRC | 21:35 | |
regXboi | amuller: I suspect I'll be there after I get this settled | 21:35 |
*** slaweq has quit IRC | 21:35 | |
*** padkrish_ has joined #openstack-neutron | 21:35 | |
regXboi | amuller: unless somebody else on my team has already gotten there I don't know about it yet | 21:36 |
kevinbenton | regXboi: https://wiki.openstack.org/wiki/LoggingStandards | 21:36 |
regXboi | er there *and* I | 21:36 |
regXboi | kevinbenton: acked and bookmarked | 21:36 |
*** padkrish has quit IRC | 21:36 | |
regXboi | and I definitely think neutron doesn't currently conform to this | 21:37 |
armax | amuller: that’s true, but there are some circumstances where running dvr code is inevitable | 21:37 |
kevinbenton | regXboi: how sow? | 21:37 |
*** slaweq has joined #openstack-neutron | 21:37 | |
armax | amuller: I am sure there’s plenty of room for improvement though | 21:37 |
kevinbenton | regXboi: have you seen code logging at INFO level that violates this? | 21:37 |
regXboi | kevinbenton: no - I've seen code that *should* be logging at INFO and above level that isn't | 21:38 |
regXboi | the lack of conformity is in the absence of code, not the presence of bad code | 21:38 |
regXboi | er is *from* the absence | 21:38 |
kevinbenton | regXboi: its a restriction, not a requirement to log everything | 21:38 |
kevinbenton | regXboi: because people will have different opinions on what is useful to an operator | 21:38 |
armax | regXboi: there was a lady @Huawei who wanted to sort out logging along the same lines you’re talking about | 21:39 |
kevinbenton | regXboi: if a single create router request results in 15 INFO statements, we screwed up | 21:39 |
regXboi | kevinbenton: I postulate that if we had better logging at INFO, we'd have better luck with things like elastic recheck | 21:39 |
armax | regXboi: don’t recall her name exactly | 21:39 |
armax | regXboi: but I am pretty sure she left a digital trace somewhere | 21:39 |
kevinbenton | regXboi: we log at DEBUG for elasticrecheck | 21:39 |
regXboi | armax: neither do I | 21:39 |
*** arajagopal has joined #openstack-neutron | 21:40 | |
*** arajagopal has quit IRC | 21:40 | |
armax | regXboi: let me see what I can find | 21:40 |
regXboi | kevinbenton: but that logging is (I again postulate) not as useful as it should be | 21:40 |
*** arajagopal has joined #openstack-neutron | 21:40 | |
regXboi | but given that, then I'm looking at a logging hack to allow instrumentation to be turned on | 21:40 |
regXboi | which can be done, it's just ...well ... a hack | 21:41 |
regXboi | armax: ack | 21:41 |
kevinbenton | regXboi: then you can improve the debug statements. but we shouldn't start upgrading things to INFO level because we dump so much garbage at DEBUG that infra can't even have it on | 21:41 |
kevinbenton | regXboi: i'm all for smarter debug statements | 21:41 |
regXboi | kevinbenton: that was implied in (1) - if we can't go to INFO, then we make what we have smarter | 21:41 |
kevinbenton | regXboi: but i don't just want a debug statement for every function we call | 21:41 |
kevinbenton | regXboi: i'm completely against going to INFO | 21:42 |
regXboi | kevinbenton: I get that and I can work with the INFO piece | 21:42 |
regXboi | kevinbenton: however, I actually *do* want the ability to log entry/exit in DEBUG, but only if something special is set in local.conf | 21:42 |
regXboi | I don't want that to happen normally | 21:43 |
*** jpena is now known as jpena|zzz | 21:43 | |
regXboi | lord knows, I don't want that to happen normally | 21:43 |
*** johnbelamaric has quit IRC | 21:43 | |
regXboi | but I should be able to turn it on in my local.conf | 21:43 |
*** tkatarki has quit IRC | 21:44 | |
*** gangil has joined #openstack-neutron | 21:45 | |
*** anshul has quit IRC | 21:45 | |
*** achanda_ has quit IRC | 21:45 | |
regXboi | but we've gotten off track and my wife is giving me a *very* dirty look, since this was supposed to be a 15 minute conversation and it's now going on 45 | 21:46 |
regXboi | so in the interest of domestic harmony, I need to sign off | 21:46 |
*** tongli_ has quit IRC | 21:46 | |
*** krtaylor has quit IRC | 21:46 | |
regXboi | kevinbenton: thanks for the logging pointer - I'll read through it tomorrow morning | 21:47 |
*** slaweq has quit IRC | 21:47 | |
regXboi | kevinbenton, carl_baldwin: thanks for the confirmation on the workaround | 21:47 |
regXboi | and I'll be back tomorrow | 21:48 |
*** rpothier has quit IRC | 21:48 | |
*** regXboi has left #openstack-neutron | 21:48 | |
*** abhiraut has quit IRC | 21:48 | |
*** abhiraut has joined #openstack-neutron | 21:50 | |
*** slaweq has joined #openstack-neutron | 21:52 | |
*** beagles has quit IRC | 21:54 | |
*** flaviof has quit IRC | 21:55 | |
*** tflynn_ has quit IRC | 21:55 | |
*** emagana has quit IRC | 21:55 | |
*** changbl has quit IRC | 21:55 | |
*** tflynn has joined #openstack-neutron | 21:55 | |
*** david-lyle has joined #openstack-neutron | 21:56 | |
*** emagana has joined #openstack-neutron | 21:56 | |
*** emagana has quit IRC | 21:57 | |
*** pradk has quit IRC | 21:59 | |
*** achanda has joined #openstack-neutron | 21:59 | |
*** ijw_ has quit IRC | 22:01 | |
*** klindgren_ has joined #openstack-neutron | 22:02 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: WIP Only get host data for floating ips on DVR routers https://review.openstack.org/224323 | 22:03 |
*** klindgren has quit IRC | 22:03 | |
mlavalle | carl_baldwin: ping | 22:04 |
*** l6unchpad has joined #openstack-neutron | 22:04 | |
carl_baldwin | mlavalle: pong | 22:04 |
mlavalle | carl_baldwin: any special subjects for tomorrow's meeting? | 22:04 |
*** nmagnezi has quit IRC | 22:06 | |
*** achanda has quit IRC | 22:07 | |
*** carl_baldwin has quit IRC | 22:09 | |
*** achanda has joined #openstack-neutron | 22:09 | |
*** jbell8 has joined #openstack-neutron | 22:10 | |
*** stevemar has joined #openstack-neutron | 22:10 | |
*** thorst has joined #openstack-neutron | 22:11 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fixes SNAT port not found for internal port https://review.openstack.org/224250 | 22:11 |
*** stevemar has quit IRC | 22:13 | |
*** arajagopal has quit IRC | 22:14 | |
*** arajagopal has joined #openstack-neutron | 22:15 | |
*** annegentle has quit IRC | 22:16 | |
*** achanda_ has joined #openstack-neutron | 22:17 | |
*** achanda has quit IRC | 22:17 | |
*** arajagopal has quit IRC | 22:19 | |
*** tfukushima has joined #openstack-neutron | 22:20 | |
*** padkrish_ has quit IRC | 22:23 | |
*** padkrish has joined #openstack-neutron | 22:24 | |
*** moshele has quit IRC | 22:25 | |
*** tfukushima has quit IRC | 22:25 | |
*** julien_ZTE has joined #openstack-neutron | 22:25 | |
*** padkrish has quit IRC | 22:28 | |
*** Sukhdev has quit IRC | 22:28 | |
*** david-lyle has quit IRC | 22:29 | |
openstackgerrit | Mathieu Gagné proposed openstack/neutron: Add neutron-linuxbridge-cleanup util https://review.openstack.org/221508 | 22:29 |
*** k-yasui has joined #openstack-neutron | 22:31 | |
*** ronis has quit IRC | 22:32 | |
*** krtaylor has joined #openstack-neutron | 22:32 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver https://review.openstack.org/207635 | 22:32 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce BgpDrAgent https://review.openstack.org/207625 | 22:32 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: [WIP] Add BGP Speaker Service Plugin RPC Calls https://review.openstack.org/216994 | 22:33 |
*** Leom has quit IRC | 22:34 | |
*** lindycoder has quit IRC | 22:35 | |
*** mwagner_lap has joined #openstack-neutron | 22:37 | |
*** abhiraut has quit IRC | 22:38 | |
*** tflynn_ has joined #openstack-neutron | 22:39 | |
*** abhiraut has joined #openstack-neutron | 22:40 | |
*** padkrish has joined #openstack-neutron | 22:40 | |
*** tbachman has quit IRC | 22:41 | |
*** tflynn has quit IRC | 22:42 | |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally https://review.openstack.org/213781 | 22:42 |
*** tflynn_ has quit IRC | 22:43 | |
*** shashank_hegde has quit IRC | 22:43 | |
*** lori is now known as lori|away | 22:43 | |
openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity https://review.openstack.org/213819 | 22:44 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver https://review.openstack.org/207635 | 22:44 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: [WIP] Add BGP Speaker Service Plugin RPC Calls https://review.openstack.org/216994 | 22:45 |
*** thurloat is now known as thurloat_isgone | 22:46 | |
*** gildub has joined #openstack-neutron | 22:47 | |
*** alex_klimov has quit IRC | 22:48 | |
*** tbachman has joined #openstack-neutron | 22:48 | |
*** dims_ has quit IRC | 22:48 | |
*** banix has quit IRC | 22:48 | |
openstackgerrit | Merged openstack/neutron: No network devices on network attached qos policies https://review.openstack.org/220164 | 22:49 |
*** Sukhdev_ has joined #openstack-neutron | 22:51 | |
kevinbenton | ZZelle_: ping | 22:51 |
*** shashank_hegde has joined #openstack-neutron | 22:51 | |
openstackgerrit | Aaron Rosen proposed openstack/neutron: ipam: Prevent none from being passed to delete https://review.openstack.org/224289 | 22:51 |
kevinbenton | ZZelle_, armax: this pattern looks icky https://bugs.launchpad.net/neutron/+bug/1496586 | 22:52 |
openstack | Launchpad bug 1496586 in neutron "Avoid the pattern sql select/delete" [Wishlist,Confirmed] - Assigned to Cedric Brandily (cbrandily) | 22:52 |
*** mlavalle has quit IRC | 22:52 | |
kevinbenton | ZZelle_, armax: also. calling .delete() directly on a query prevents sqlalchemy from doing it's own cascading actions | 22:52 |
armax | kevinbenton: I think that the usage of the pattern needs to be evaluated on a case by case basis | 22:54 |
armax | kevinbenton: it cannot be applied blindly | 22:54 |
armax | that said, how is it possible that it prevents the cascading? | 22:54 |
kevinbenton | armax: in-python cascades | 22:54 |
ZZelle_ | kevinbenton, armax, good point about in-python cascades | 22:55 |
*** thurloat_isgone is now known as thurloat | 22:55 | |
*** k-yasui has quit IRC | 22:55 | |
kevinbenton | which would become relevant if we use a main neutron objects table that the ports and networks, etc are related to | 22:55 |
ZZelle_ | imo, it means we should avoid the select than delete pattern or document why we need it | 22:55 |
openstackgerrit | Aaron Rosen proposed openstack/neutron: db_base: Make delete_port raise NotFound if port is not found https://review.openstack.org/224337 | 22:56 |
armax | kevinbenton: how can you overcome that? | 22:56 |
armax | kevinbenton: I am baffled that in order to support in-python cascade we issue a sub-optimal set of sql queries | 22:56 |
kevinbenton | armax: unbaffle yourself | 22:57 |
kevinbenton | armax: it's the price of an ORM with magic :) | 22:57 |
kevinbenton | armax: http://docs.sqlalchemy.org/en/rel_1_0/orm/query.html#sqlalchemy.orm.query.Query.delete | 22:58 |
kevinbenton | armax,ZZelle_: it sounds like we can use an after_bulk_delete event to cleanup if we needed | 22:58 |
armax | kevinbenton: makes sense | 23:00 |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: Add devref for BGP Speaker https://review.openstack.org/196452 | 23:01 |
kevinbenton | armax, ZZelle_: is the issue with just catching the norecord exception that the exception invalidates the session? | 23:01 |
armax | kevinbenton: which issue are you talking about in particular? | 23:03 |
kevinbenton | armax: instead of switching to this bulk delete pattern, why can we just catch object not found exceptions on the delete | 23:04 |
armax | that’s icky too | 23:05 |
*** markvoelker has quit IRC | 23:05 | |
*** l6unchpad has quit IRC | 23:06 | |
armax | unless we can do that without issuing an extra sql statement | 23:06 |
*** ijw_ has joined #openstack-neutron | 23:07 | |
kevinbenton | armax: oh, i see. this is just for cases where nothing is done with the selected object at all other than deleteing it | 23:07 |
armax | ya | 23:07 |
armax | that’s the case I am talking about | 23:07 |
kevinbenton | armax: ok. please proceed then and ignore me | 23:07 |
armax | if I am using the object fine | 23:07 |
armax | kevinbenton: no | 23:07 |
armax | kevinbenton: you raised a good point | 23:07 |
armax | kevinbenton: keep doing that | 23:07 |
kevinbenton | armax: this is the delete_port patch that aaron proposed | 23:07 |
kevinbenton | armax: that brought this up? | 23:08 |
armax | kevinbenton: aye | 23:08 |
kevinbenton | armax: right. we don't use sql triggers for anything except for the status update i think | 23:08 |
kevinbenton | armax: is that right? | 23:08 |
armax | kevinbenton: I am sure the situation is more variagated than that | 23:09 |
kevinbenton | armax: ? | 23:09 |
armax | kevinbenton: ok, can you rephrase? I might have misunderstood you | 23:10 |
kevinbenton | armax: i just mean we don't have any delete triggers depending on that | 23:10 |
armax | depending on what? | 23:10 |
kevinbenton | armax: right? | 23:10 |
armax | you mean ports? | 23:10 |
kevinbenton | armax: yes | 23:11 |
armax | I think we do | 23:11 |
armax | we must do | 23:11 |
kevinbenton | for what? | 23:12 |
armax | I am gonna have to look I don’t remember on the top of my head | 23:13 |
*** tbachman has quit IRC | 23:14 | |
*** l6unchpad has joined #openstack-neutron | 23:15 | |
*** Sukhdev_ has quit IRC | 23:15 | |
*** carora has quit IRC | 23:16 | |
*** kbringard has quit IRC | 23:17 | |
*** padkrish has quit IRC | 23:18 | |
*** padkrish has joined #openstack-neutron | 23:19 | |
*** julien_ZTE has quit IRC | 23:21 | |
*** l6unchpad has quit IRC | 23:22 | |
*** padkrish has quit IRC | 23:23 | |
*** tbachman has joined #openstack-neutron | 23:25 | |
*** sneti has joined #openstack-neutron | 23:27 | |
*** oomichi has quit IRC | 23:30 | |
*** arajagopal has joined #openstack-neutron | 23:31 | |
*** dims_ has joined #openstack-neutron | 23:32 | |
*** arajagopal has quit IRC | 23:35 | |
*** arajagopal has joined #openstack-neutron | 23:36 | |
*** baoli_ has quit IRC | 23:37 | |
*** k-yasui has joined #openstack-neutron | 23:37 | |
*** abhiraut has quit IRC | 23:38 | |
*** yamamoto has quit IRC | 23:39 | |
*** hitalia has quit IRC | 23:40 | |
*** abhiraut has joined #openstack-neutron | 23:40 | |
*** shivrao has quit IRC | 23:41 | |
*** chlong has quit IRC | 23:41 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Avoid the select and delete sql pattern https://review.openstack.org/224351 | 23:42 |
*** ZZelle_ has quit IRC | 23:43 | |
*** k-yasui_ has joined #openstack-neutron | 23:47 | |
*** hoangcx has joined #openstack-neutron | 23:47 | |
*** salv-orlando has quit IRC | 23:48 | |
*** Swami has quit IRC | 23:50 | |
*** amuller has quit IRC | 23:50 | |
*** yamahata has joined #openstack-neutron | 23:51 | |
*** tomoe_ has joined #openstack-neutron | 23:52 | |
*** Sukhdev has joined #openstack-neutron | 23:52 | |
*** shwetaap1 has quit IRC | 23:55 | |
*** hichihara has joined #openstack-neutron | 23:56 | |
*** veena has joined #openstack-neutron | 23:57 | |
*** padkrish has joined #openstack-neutron | 23:59 | |
*** shivharis has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!