Wednesday, 2015-09-16

*** shwetaap has joined #openstack-neutron00:00
*** dannywilson has quit IRC00:01
*** blogan_ has joined #openstack-neutron00:01
openstackgerritRitesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers.  https://review.openstack.org/18815700:02
*** SourabhP has quit IRC00:03
openstackgerritMerged openstack/neutron: Remove out-of-tree vendor AGENT_TYPE_* constant  https://review.openstack.org/21641700:03
openstackgerritMerged openstack/neutron: Process user iptables rules before INVALID  https://review.openstack.org/21851700:03
*** shwetaap1 has quit IRC00:03
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Added Exceptions and Logs to track FIP on DVR  https://review.openstack.org/22387900:04
openstackgerritMerged openstack/neutron: Allow passing arbitrary ip route parameters to add/delete_route  https://review.openstack.org/21903700:04
*** fitoduarte has quit IRC00:08
*** shashank_hegde has quit IRC00:08
*** yamamoto has quit IRC00:09
*** shoutm has joined #openstack-neutron00:10
*** Swami has quit IRC00:11
*** shashank_hegde has joined #openstack-neutron00:13
*** SumitNaiksatam has quit IRC00:15
*** imcsk8_ has joined #openstack-neutron00:16
*** hanzhang has joined #openstack-neutron00:17
*** armax_ has joined #openstack-neutron00:17
*** toddnni_ has joined #openstack-neutron00:18
*** zhhuabj has quit IRC00:18
*** thorst has joined #openstack-neutron00:18
*** devlaps has quit IRC00:18
*** julien_ZTE has quit IRC00:19
*** dims_ has quit IRC00:20
*** bapalm_ has joined #openstack-neutron00:20
*** clayton- has joined #openstack-neutron00:20
*** cburgess_ has joined #openstack-neutron00:21
*** imcsk8 has quit IRC00:21
*** shihanzhang has quit IRC00:21
*** cburgess has quit IRC00:21
*** shz has quit IRC00:21
*** armax has quit IRC00:21
*** toddnni has quit IRC00:21
*** toddnni_ is now known as toddnni00:21
*** shz has joined #openstack-neutron00:21
*** bapalm has quit IRC00:21
*** clayton has quit IRC00:21
*** shz has quit IRC00:21
*** clayton- is now known as clayton00:21
*** armax_ is now known as armax00:21
*** shihanzhang has joined #openstack-neutron00:21
*** gildub has joined #openstack-neutron00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-neutron00:23
*** yamahata has quit IRC00:24
*** shivrao has quit IRC00:25
*** bradjones has quit IRC00:25
*** tkatarki has joined #openstack-neutron00:26
*** minwang2_ has joined #openstack-neutron00:27
*** thorst has quit IRC00:28
*** shoutm_ has joined #openstack-neutron00:28
*** abhiraut has quit IRC00:28
*** ildikov_ has quit IRC00:28
*** Prem_ has joined #openstack-neutron00:28
*** xgerman_ has joined #openstack-neutron00:28
*** kgalanov_ has joined #openstack-neutron00:29
*** tomoe_ has joined #openstack-neutron00:29
*** shoutm has quit IRC00:29
*** StevenK_ has joined #openstack-neutron00:31
*** wasmum- has joined #openstack-neutron00:31
*** raorn_ has joined #openstack-neutron00:32
*** abhiraut has joined #openstack-neutron00:32
*** StevenK has quit IRC00:32
*** StevenK_ is now known as StevenK00:33
*** tidwellr has left #openstack-neutron00:33
*** JoshNang_ has joined #openstack-neutron00:34
*** agireud- has joined #openstack-neutron00:34
*** madhu_ak has quit IRC00:34
*** rex_lee_ has joined #openstack-neutron00:34
*** sweston_ has joined #openstack-neutron00:35
*** minwang2 has quit IRC00:35
*** jaypipes has quit IRC00:35
*** agireud has quit IRC00:35
*** JoshNang has quit IRC00:35
*** akamyshnikova has quit IRC00:35
*** lennyb has quit IRC00:35
*** raorn has quit IRC00:35
*** clarkb has quit IRC00:35
*** fnordahl has quit IRC00:35
*** Prem has quit IRC00:35
*** sweston has quit IRC00:36
*** kgalanov has quit IRC00:36
*** xgerman has quit IRC00:36
*** rex_lee has quit IRC00:36
*** wasmum has quit IRC00:36
*** agireud- is now known as agireud00:36
*** minwang2_ is now known as minwang200:36
*** JoshNang_ is now known as JoshNang00:36
*** Prem_ is now known as Prem00:36
*** sweston_ is now known as sweston00:37
*** xgerman_ is now known as xgerman00:37
*** akamyshnikova has joined #openstack-neutron00:37
*** jaypipes has joined #openstack-neutron00:40
*** tiswanso has joined #openstack-neutron00:40
*** shwetaap has quit IRC00:40
*** banix has joined #openstack-neutron00:41
*** fnordahl has joined #openstack-neutron00:42
*** lennyb has joined #openstack-neutron00:42
*** clarkb has joined #openstack-neutron00:42
*** mwagner_lap has joined #openstack-neutron00:42
*** arajagopal has joined #openstack-neutron00:43
*** _cjones_ has quit IRC00:43
*** jckasper has joined #openstack-neutron00:45
*** tiswanso_ has joined #openstack-neutron00:45
*** shoutm_ has quit IRC00:46
*** shashank_hegde has quit IRC00:48
*** tiswanso has quit IRC00:48
*** julien_ZTE has joined #openstack-neutron00:48
*** shoutm has joined #openstack-neutron00:48
*** moha_hunt has quit IRC00:49
*** sthillma has quit IRC00:49
*** sridhar_ram has quit IRC00:50
*** salv-orl_ has joined #openstack-neutron00:53
*** yamamoto has joined #openstack-neutron00:55
*** salv-orlando has quit IRC00:56
*** salv-orl_ has quit IRC00:57
*** hoangcx has joined #openstack-neutron00:58
*** banix has quit IRC01:01
*** vivek-ebay has joined #openstack-neutron01:03
*** tkatarki has quit IRC01:05
*** dims_ has joined #openstack-neutron01:05
*** vivek-eb_ has joined #openstack-neutron01:05
*** tkatarki has joined #openstack-neutron01:06
*** hanzhang has quit IRC01:08
*** yamahata has joined #openstack-neutron01:08
*** hanzhang has joined #openstack-neutron01:08
*** Sukhdev has quit IRC01:09
*** vivek-ebay has quit IRC01:09
hoangcxsc68cal: ping01:11
hoangcxsc68cal: Could you please take a look on it and give me some comments to file the spec?01:13
hoangcxsc68cal: #link https://review.openstack.org/#/c/203509/01:13
hoangcxsc68cal: Thanks!01:13
*** baojg has joined #openstack-neutron01:15
*** crose has joined #openstack-neutron01:16
*** r-daneel has quit IRC01:17
*** lazy_prince has joined #openstack-neutron01:18
*** stevemar has joined #openstack-neutron01:19
*** stanzgy has joined #openstack-neutron01:19
*** stanzgy__ has joined #openstack-neutron01:19
*** baojg has quit IRC01:20
*** jwy has quit IRC01:20
*** shwetaap has joined #openstack-neutron01:22
*** killer_prince has joined #openstack-neutron01:27
*** stevemar has quit IRC01:29
*** banix has joined #openstack-neutron01:29
*** stevemar has joined #openstack-neutron01:30
*** lazy_prince has quit IRC01:30
*** lazy_prince has joined #openstack-neutron01:30
*** killer_prince has quit IRC01:31
*** dims_ has quit IRC01:34
*** baoli has joined #openstack-neutron01:36
*** baojg has joined #openstack-neutron01:38
*** clev is now known as clev-away01:40
*** ivar-laz_ has joined #openstack-neutron01:43
*** ivar-lazzaro has quit IRC01:46
*** sballe has quit IRC01:46
*** ivar-laz_ has quit IRC01:47
*** jwy has joined #openstack-neutron01:49
*** tkatarki has quit IRC01:50
*** nlahouti has quit IRC01:50
*** zhenguo has joined #openstack-neutron01:50
*** muralir has joined #openstack-neutron01:54
*** erhudy1 has quit IRC01:57
*** muralir has quit IRC01:58
*** madhu_ak has joined #openstack-neutron02:03
*** madhu_ak has joined #openstack-neutron02:03
*** thorst has joined #openstack-neutron02:03
*** vivek-eb_ has quit IRC02:04
*** shoutm has quit IRC02:05
*** abhiraut has quit IRC02:06
*** madhu_ak has quit IRC02:06
*** s3wong has quit IRC02:06
*** yog_ has joined #openstack-neutron02:08
*** armax has quit IRC02:13
*** shoutm has joined #openstack-neutron02:13
*** thorst has quit IRC02:16
*** thorst has joined #openstack-neutron02:16
*** thorst has left #openstack-neutron02:16
*** saksham has quit IRC02:20
*** stevemar_ has joined #openstack-neutron02:20
*** clarkb1 has joined #openstack-neutron02:27
*** zhenguo has quit IRC02:28
*** baojg has quit IRC02:28
*** stevemar has quit IRC02:28
*** julien_ZTE has quit IRC02:28
*** fnordahl has quit IRC02:28
*** lennyb has quit IRC02:28
*** clarkb has quit IRC02:28
*** rtheis has joined #openstack-neutron02:29
*** baojg has joined #openstack-neutron02:29
*** lennyb has joined #openstack-neutron02:29
*** crose has quit IRC02:31
*** zhenguo has joined #openstack-neutron02:33
*** lazy_prince has quit IRC02:34
*** fnordahl has joined #openstack-neutron02:34
*** julien_ZTE has joined #openstack-neutron02:34
*** lazy_prince has joined #openstack-neutron02:34
*** achanda has joined #openstack-neutron02:35
*** bharathm has joined #openstack-neutron02:42
*** shwetaap has quit IRC02:43
*** gongysh has joined #openstack-neutron02:45
*** jwy has quit IRC02:46
*** bharathm has quit IRC02:47
*** banix has quit IRC02:48
*** ivar-lazzaro has joined #openstack-neutron02:49
*** ferest has joined #openstack-neutron02:49
*** shwetaap has joined #openstack-neutron02:50
*** rtheis has quit IRC02:51
*** lazy_prince has quit IRC02:51
openstackgerritYuji Azama proposed openstack/neutron: Add head option to neutron-db-manage command  https://review.openstack.org/22390102:52
*** zhhuabj has joined #openstack-neutron02:54
*** mickeyspiegel has quit IRC02:54
*** SourabhP has joined #openstack-neutron02:54
*** julien_ZTE has quit IRC02:56
*** nplanel has joined #openstack-neutron02:58
*** SourabhP_ has joined #openstack-neutron02:58
*** banix has joined #openstack-neutron02:59
*** SourabhP has quit IRC03:00
*** SourabhP_ is now known as SourabhP03:00
*** blogan_ has quit IRC03:02
openstackgerritfumihiko kakuma proposed openstack/neutron: Remove requirements.txt for the ofagent mechanism driver  https://review.openstack.org/22390303:04
*** baojg has quit IRC03:07
*** azbiswas has joined #openstack-neutron03:10
*** zzzeek has joined #openstack-neutron03:12
*** ivar-lazzaro has quit IRC03:14
*** ivar-lazzaro has joined #openstack-neutron03:14
*** carora has joined #openstack-neutron03:15
caroraHi everyone, I have a question about neutron service03:15
caroraI 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 IRC03:16
carorafor example in resolve conf I want <tenant-name>.xyz (domain)03:16
*** fifieldt has joined #openstack-neutron03:18
*** ivar-lazzaro has quit IRC03:18
*** deepthi has joined #openstack-neutron03:21
*** baojg has joined #openstack-neutron03:22
*** ivar-lazzaro has joined #openstack-neutron03:22
*** fitoduarte has joined #openstack-neutron03:23
*** armax has joined #openstack-neutron03:23
*** mragupat has joined #openstack-neutron03:24
*** mragupat_ has joined #openstack-neutron03:25
reedipDear All, during creation of Migration script, I am getting the following error. Can anyone suggest how to proceed forward?03:25
reedipCommand:     neutron-db-manage  revision -m "Description" --autogenerate03:26
reedipError:     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-neutron03:28
*** mragupat has quit IRC03:29
*** boris-42 has quit IRC03:30
*** SumitNaiksatam has joined #openstack-neutron03:31
*** vivek-eb_ has joined #openstack-neutron03:31
*** vivek-ebay has quit IRC03:32
*** tiswanso_ has quit IRC03:33
openstackgerritgongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies  https://review.openstack.org/21709203:33
*** tfukushima has joined #openstack-neutron03:36
*** banix has quit IRC03:37
openstackgerritgongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies  https://review.openstack.org/21709203:41
*** david-lyle has joined #openstack-neutron03:41
openstackgerritgongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies  https://review.openstack.org/21709203:45
*** shoutm has quit IRC03:48
openstackgerritgongysh proposed openstack/neutron: Do not allow tenants to detach enforced QoS policies  https://review.openstack.org/21709203:48
*** shoutm_ has joined #openstack-neutron03:49
*** boris-42 has joined #openstack-neutron03:49
*** boris-42 has quit IRC03:49
*** boris-42 has joined #openstack-neutron03:49
*** ivar-laz_ has joined #openstack-neutron03:51
*** ivar-laz_ has quit IRC03:51
*** ivar-laz_ has joined #openstack-neutron03:52
*** zhhuabj has quit IRC03:53
*** ivar-lazzaro has quit IRC03:54
*** tflynn_ has joined #openstack-neutron03:54
*** amotoki has joined #openstack-neutron03:56
*** ivar-laz_ has quit IRC03:56
*** tflynn has quit IRC03:57
*** minwang2 has quit IRC03:58
*** iwamoto has joined #openstack-neutron03:58
*** tflynn_ has quit IRC03:58
*** carl_baldwin has joined #openstack-neutron03:59
*** fawadkhaliq has joined #openstack-neutron04:00
*** shivrao has joined #openstack-neutron04:01
*** shivrao_ has joined #openstack-neutron04:03
*** changbl has joined #openstack-neutron04:04
*** shivrao has quit IRC04:05
*** shivrao_ is now known as shivrao04:05
*** sthillma has joined #openstack-neutron04:10
*** baoli has quit IRC04:10
*** shwetaap has quit IRC04:11
*** tomoe_ has quit IRC04:12
*** Marga_ has quit IRC04:16
*** fawadkhaliq has quit IRC04:17
*** rotbeard has joined #openstack-neutron04:19
*** carora has quit IRC04:20
*** arajagopal has quit IRC04:20
*** arajagopal has joined #openstack-neutron04:21
*** tfukushima has quit IRC04:21
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovsdb: Fix a few docstring  https://review.openstack.org/22391704:21
*** sridhar_ram has joined #openstack-neutron04:22
*** fawadkhaliq has joined #openstack-neutron04:23
*** ferest has quit IRC04:24
*** arajagopal has quit IRC04:26
*** coolsvap has joined #openstack-neutron04:29
*** gberginc has joined #openstack-neutron04:30
*** irenab has quit IRC04:30
*** deepthi has quit IRC04:31
*** sdake has joined #openstack-neutron04:33
*** baojg has quit IRC04:33
*** anilvenkata has joined #openstack-neutron04:34
*** sdake_ has joined #openstack-neutron04:34
*** yfried has quit IRC04:35
*** jbell8 has quit IRC04:35
*** stevemar_ has quit IRC04:38
*** sdake has quit IRC04:38
*** baojg has joined #openstack-neutron04:40
*** david-lyle has quit IRC04:42
*** arajagopal has joined #openstack-neutron04:42
openstackgerritKevin Benton proposed openstack/neutron: Always return iterables in L3 get_candidates  https://review.openstack.org/22282804:43
reedip@ amotoki: , @hichihara:  Hi there . I am facing the following problem in creating a migration script :04:44
reedipreedip@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 IRC04:44
reedipAny idea in resolving the same?04:44
reedipamotoki: , hichihara: Any idea in resolving the same?04:44
*** david-lyle has joined #openstack-neutron04:45
*** jbell8 has joined #openstack-neutron04:45
*** dannywilson has joined #openstack-neutron04:46
*** achanda has quit IRC04:46
*** deepthi has joined #openstack-neutron04:47
hichiharareedip: I'm not db guy but let's change "description of revision".04:47
*** fitoduarte has quit IRC04:48
reedipI changed it but its not helping out04:49
*** carl_baldwin has quit IRC04:49
*** tomoe_ has joined #openstack-neutron04:50
amotokireedip: hichihara: reading alembic docs04:50
hichiharareedip: Did you try to use option "--autogenerate"?04:51
reedip@amotoki, @hichihara: Yes, I did04:51
*** achanda has joined #openstack-neutron04:51
*** fedexo has joined #openstack-neutron04:52
*** tflynn has joined #openstack-neutron04:52
*** deepthi has quit IRC04:54
*** ferest has joined #openstack-neutron04:54
*** tflynn_ has joined #openstack-neutron04:54
*** fitoduarte has joined #openstack-neutron04:55
*** sdake_ has quit IRC04:56
reedipamotoki:, hichihara: Something I found while looking at the various operation04:56
reedip Amotoki:, Hichihara: reedip@reedip-VirtualBox:/opt/stack/neutron-fwaas/neutron_fwaas$ neutron-db-manage --subproject neutron-fwaas check_migration04:56
reedip  Running branches for neutron-fwaas ...04:56
reedipkilo (branchpoint)04:56
reedip     -> c40fbb377ad (expand)04:56
reedip     -> 67c8e8d61d5 (contract) (head)04:56
reedip  OK04:56
reedip  FAILED: HEADS file does not match migration timeline heads, expected: 4b47ea298795, 67c8e8d61d504:56
reedipAmotoki:, Hichihara: Seems some migration is not yet complete (??)04:57
*** tflynn has quit IRC04:57
amotokireedip: hmm... something seems wrong04:57
*** fawadkhaliq has quit IRC04:57
*** ferest has quit IRC04:58
amotokireedip: checking fwaas db migratio history..04:59
*** amit213 has quit IRC04:59
*** sdake has joined #openstack-neutron04:59
*** amit213 has joined #openstack-neutron04:59
reedipamotoki: Ok. I have it with me by the way.04:59
amotokireedip: I succeeded for neutron-vpnaas: neutron-db-manage --subproject neutron-vpnaas check_migration04:59
reedipamotoki: reedip@reedip-VirtualBox:/opt/stack/neutron-fwaas/neutron_fwaas$ neutron-db-manage --subproject neutron-fwaas history05:00
reedip  Running history for neutron-fwaas ...05:00
reedipc40fbb377ad -> 4b47ea298795 (expand) (head), add reject rule05:00
reedipkilo -> c40fbb377ad (expand), Initial Liberty no-op script.05:00
amotokireedip: for neutron-fwaas, it should succeed05:00
reedipkilo -> 67c8e8d61d5 (contract) (head), Initial Liberty no-op script.05:00
reedip796c68dffbb -> kilo (branchpoint), kilo05:00
reedip540142f314f4 -> 796c68dffbb, cisco_csr_fwaas05:00
reedip4202e3047e47 -> 540142f314f4, FWaaS router insertion05:00
reedipstart_neutron_fwaas -> 4202e3047e47, add_index_tenant_id05:00
reedip<base> -> start_neutron_fwaas, start neutron-fwaas chain05:00
reedip  OK05:00
reedipamotoki: Oh ok.. So I should contact the owner of the bug 'add reject rule' ( HEAD ID: 4b47ea298795) for this ?05:01
amotokireedip: let me check for a while05:01
*** hitalia has joined #openstack-neutron05:02
reedipamotoki: Ok05:03
*** mohankumar has joined #openstack-neutron05:03
*** neeti has joined #openstack-neutron05:04
*** saksham has joined #openstack-neutron05:05
*** sridhar_ram1 has joined #openstack-neutron05:05
*** sridhar_ram has quit IRC05:07
*** saksham_ has joined #openstack-neutron05:07
*** ekuris has joined #openstack-neutron05:07
*** adduarte has joined #openstack-neutron05:09
amotokireedip: I found the contents of HEADS file is not correct.05:10
amotokireedip: I am preparing a patch.05:10
*** sdake_ has joined #openstack-neutron05:10
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Deprecate drop_flows_on_start  https://review.openstack.org/22392105:10
*** saksham has quit IRC05:10
reedip@amotoki: Oh ok. Great Thanks :)05:10
amotokireedip: you can replace 4b47ea29879 with 4b47ea298795 in HEADS file05:11
*** Sukhdev_ has joined #openstack-neutron05:11
*** fitoduarte has quit IRC05:12
reedipYes I can05:12
*** nlahouti has joined #openstack-neutron05:12
reedipshould I do it?05:12
*** arajagopal has quit IRC05:12
Sukhdev_Sukhdev_:hi05:13
*** arajagopal has joined #openstack-neutron05:13
amotokireedip: I am preparing a patch, but as a workaround you can go forward your wokr ahead by changing HEADS file.05:13
reedipI changed the HEADS file05:13
Sukhdev_amotoki: Ping05:13
amotokiSukhdev_: pong05:13
reedipbut it didnot change the 'check_migration' output05:14
Sukhdev_amotoki: We have been missing you in neutron_ironic meetings05:14
*** sdake has quit IRC05:14
*** nlahouti1 has joined #openstack-neutron05:14
amotokiSukhdev_: really sorry _o_05:14
Sukhdev_amotoki: Is everything OK?05:14
*** saksham_ has quit IRC05:14
amotokiSukhdev_: fine. I need to have spent busy days to run OpenStack cloud too...05:15
amotokireedip: still checking05:15
HenryGamotoki: reedip: can I help with fwaas alembic migrations?05:16
openstackgerritMerged openstack/neutron: Prevent full sync in dhcp_agent when possible  https://review.openstack.org/22254605:16
amotokiHenryG: great05:16
openstackgerritMerged openstack/neutron: Prevent full sync in dhcp_agent when possible  https://review.openstack.org/22254605:16
reedip@HenryG : sure :)05:16
Sukhdev_amotoki: Are you still planning on pushing that documentation patch?05:16
*** nlahouti has quit IRC05:16
amotokiSukhdev_: I might do it, but if someone is interested in it, please take it.05:17
amotokiSukhdev_: I should review it instead.05:17
*** yuanying_ has joined #openstack-neutron05: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 it05:18
*** arajagopal has quit IRC05:18
HenryGreedip: what is the issue?05:18
amotokiHenryG: reedip: I just filed a bug: https://bugs.launchpad.net/neutron/+bug/149623905:18
openstackLaunchpad bug 1496239 in neutron "neutron-fwaas check_migartion fails" [High,New] - Assigned to Akihiro Motoki (amotoki)05:18
*** yuanying has quit IRC05:19
*** yuanying_ is now known as yuanying05:19
*** itzikb has joined #openstack-neutron05:19
reedip@amotoki: yes, thats bang on05:20
amotokiHenryG: I see the latest fwaas expand migration 4b47ea298795_add_reject_rule.py has 11 chars revision05:20
reedip@AMotoki, HenryG:  I did an experiment05:21
amotokiHenryG: and HEAD contains the first 10 chars. Even after changing HEADS to 11 char version, check_migration is still failing05:21
*** tfukushima has joined #openstack-neutron05:21
*** fawadkhaliq has joined #openstack-neutron05:22
HenryGamotoki: Do you count from 0? :)05:23
HenryGamotoki: I get 12 and 1105:23
*** obondarev has joined #openstack-neutron05:23
reedipHenryG: Yes its 12 and 11 :)05:23
HenryGamotoki: But you may be right, that this is an alembic restriction05:23
*** lazy_prince has joined #openstack-neutron05:24
HenryGamotoki: Looks like we need to drop a char from the revision and file name05:24
*** mragupat_ has quit IRC05:24
amotokiHenryG: you are right. 12 and 1105:24
*** ferest has joined #openstack-neutron05:25
reedipHenryG:, amotoki: checking something here05:25
reedipIn the versions folder , following files are listed:05:25
reedip4202e3047e47_add_index_tenant_id.py   540142f314f4_fwaas_router_insertion.py   796c68dffbb_cisco_csr_fwaas.py05:25
reedipEach has 12 characters05:25
amotokiHenryG: but don't we need to consider deployments which already uses 12 char revision?05:25
*** irenab has joined #openstack-neutron05:25
reedip@amotoki :, @henryG: Each has 12 characters05:26
*** lajos-katona has joined #openstack-neutron05:26
HenryGreedip: interesting05:26
reedip@amotoki: , @henryG: only the expand and contract have 11 characters for the initial files05:26
*** tryit has joined #openstack-neutron05:26
*** tfukushima has quit IRC05:26
*** baojg has quit IRC05:28
sc68calamotoki: I just hopped on because I saw the bug you opened05:28
*** ferest has quit IRC05:29
sc68calamotoki: i'll probably be up for another 30 mins or so, if I can help05:29
HenryGRight. alembic branches thinks c40fbb377ad is the expand head05:29
amotokisc68cal: thanks. we continues the discussion here05:30
*** gberginc has quit IRC05:30
*** kengo_sakai has joined #openstack-neutron05:31
amotokiIt seems alembic itself has no limitation the length of revisions.05:32
*** sridhar_ram1 has quit IRC05:32
amotokiit seems others except check_migration is working well.05:33
*** sbalukoff has quit IRC05:38
amotokiHenryG: When I inserted a debug message, I got: observered_heads: ['67c8e8d61d5', '4b47ea298795'] expected_heads: ['4b47ea298795', '67c8e8d61d5']05:39
*** sbalukoff has joined #openstack-neutron05:40
amotokiHenryG: it just looks a sorting problem.05:40
*** baoli has joined #openstack-neutron05:41
*** agireud has quit IRC05:41
*** trinaths has joined #openstack-neutron05:41
*** agireud has joined #openstack-neutron05:41
reedipamotoki: I tried run the same on neutron05:41
reedipamotoki: this is my observed output05:42
*** nlahouti1 has quit IRC05:42
reedipamotoki: reedip@reedip-VirtualBox:/opt/stack/neutron/neutron/db/migration/alembic_migrations/versions$ cat HEADS05:42
reedip34af2b5c5a5905:42
reedip4af11ca4729705:42
*** nlahouti has joined #openstack-neutron05:42
reedipamotoki: reedip@reedip-VirtualBox:/opt/stack/neutron/neutron/db/migration/alembic_migrations/versions$ neutron-db-manage --subproject neutron check_migration05:43
reedip  Running branches for neutron ...05:43
reedipkilo (branchpoint)05:43
reedip     -> 354db87e3225 (expand)05:43
reedip     -> 30018084ec99 (contract)05:43
reedip  OK05:43
*** shoutm_ has quit IRC05:44
amotokithe first one is expand head and the secodn one is contract head.05:44
amotokiWhen I swapped the two line in neutron-fwaas, it succeeded.05:44
reedipamotoki: Ok05:45
-amotoki- $ git diff | cat05:45
-amotoki- diff --git a/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS b/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS05:45
-amotoki- index 5d2dc8a..ad565a8 10064405:45
-amotoki- --- a/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS05:45
-amotoki- +++ b/neutron_fwaas/db/migration/alembic_migrations/versions/HEADS05:45
-amotoki- @@ -1,2 +1,2 @@05:45
-amotoki- +4b47ea29879505:45
-amotoki- 67c8e8d61d505:45
-amotoki- -4b47ea2987905:45
-amotoki- $ neutron-db-manage --subproject neutron-fwaas check_migration05: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- OK05:45
amotokiI am checking the detail.05:45
reedipamotoki: yes it worked05:45
*** arajagopal has joined #openstack-neutron05:45
*** baoli has quit IRC05:46
HenryGamotoki: I thought ihar had fixed that05:47
HenryGLooks like not05:47
amotokiI thought so too. It looks better to wait Ihar for hours.05:48
reedipamotoki: henryg: I changed the HEADS. Should the migration script work now?05:48
HenryGamotoki: I think the expectation is that cli.update_heads_file should generate the HEADS sorted05:50
amotokireedip: not sure.05:50
amotokiHenryG: the original problem reedip has is that how to generate a new revision file.05:50
amotokihe tried neutron-db-manage --subproject neutron-fwaas revision -m 'xxxxx'05:50
reedipamotoki:  Its not :( It is giving the same error. I think I have to wait for your commit (??)05:50
amotokibut 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
HenryGamotoki: yes, the 'revision' command for subprojects needs work :(05:51
reedip@amotoki:, @henryg : so how should I proceed?05:52
*** tfukushima has joined #openstack-neutron05:52
*** tfukushima has quit IRC05:53
*** tfukushima has joined #openstack-neutron05:53
*** fawadkhaliq has quit IRC05:55
*** fawadk has joined #openstack-neutron05:55
*** yfried has joined #openstack-neutron05:55
amotokireedip: "revision" command seems not to work now for fwaas. You can prepare it manually and test it.05:55
*** carora has joined #openstack-neutron05:55
*** tflynn_ has quit IRC05:55
HenryGamotoki: Patch https://review.openstack.org/148416 created HEADS in the wrong order05:55
reedip@Amotoki : @henryG: Ok, I will create it manually05:55
*** ferest has joined #openstack-neutron05:56
reedip@Amotoki, @HenryG:  do I need to modify the HEADS file as well , or should I ignore it?05:56
amotokireedip: you need to update HEADS too05:56
HenryGreedip: and make sure it is sorted05:56
amotokiHenryG: what is the expected order of HEADS files?05:56
HenryGamotoki: sorted(HEADS)05:57
reedip@Amotoki: @HenryG: and what about the random number for the file name ??05:57
HenryGbasically05:57
amotokiHenryG: alphabetical order?05:57
reedip@amotoki: @henryg: how do I create it ?? :D05: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
HenryGreedip: sure05:58
HenryGreedip: the new revision number is any random hex string not used before05:59
reedip@HenryG: and for the random number05:59
amotokireedip: 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 IRC05:59
*** yfried_ has joined #openstack-neutron05:59
reedip@amotoki: how to check against a real database?06:00
amotokireedip: check_migration just check if migration sequence is valid and head revisions match HEADS file.06:00
HenryGreedip: Here is one: 8adf9d10c5706:00
*** ferest has quit IRC06: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 patch06:00
HenryGreedip: I ran "neutron-db-manage revision -m foo --autogenerate" on neutron core06:01
HenryGreedip: It gave me an emptry script06: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-neutron06:02
amotokireedip: what i usually do is to prepare a new database named "neutron" on mysql and run "neutron-db-manage upgrade heads".06:02
amotokireedip: let me test.06:03
reedip@Amotoki:  Wont that also work with launching devstack ?06:03
*** obondarev has quit IRC06:03
amotokireedip: you need to unstack first.06:03
reedip@Amotoki: Yes, thats part of my launch script :)06:04
HenryGamotoki: reedip: I am going to bed now. If you need more help, akamyshnikova and Ihar will be online soon I think.06:04
amotokiHenryG: thanks for your help. good night!06:04
amotokiHenryG: I am catching up with the changes around db migration improvements now.06:05
*** obondarev has joined #openstack-neutron06:05
reedip@HenryG: Goodnight, and thank you very much for your help :)06:05
*** EinstCrazy has joined #openstack-neutron06: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-neutron06:06
*** hitalia has quit IRC06:07
amotokireedip: there wil be conflict temporarily, but you can address conflicts by rebasing if another patch is merged.06:07
amotokireedip: 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 now06:08
reedip@amotoki: Thanks a lot for your help.06:08
reedip:)06:08
amotokireedip: good luch06:08
amotokiluck06:08
reedip@Amotoki: Thanks.... definitely need it :)06:09
*** _cjones_ has joined #openstack-neutron06:09
*** _cjones_ has quit IRC06:09
*** _cjones_ has joined #openstack-neutron06:10
*** diga has joined #openstack-neutron06:11
*** obondarev has quit IRC06:11
*** shoutm has joined #openstack-neutron06:12
*** mkolesni has joined #openstack-neutron06:14
*** numans has joined #openstack-neutron06:15
*** trinaths has left #openstack-neutron06:15
*** jbell8 has quit IRC06:18
*** fedexo has quit IRC06:19
amotokireedip: This is what I do to test db miration against real database table. http://paste.openstack.org/show/464133/06:20
amotokireedip: After doing so, you can check detail schemas using mysql command or something.06:20
*** obondarev has joined #openstack-neutron06:20
reedip@Amotoki: You have been really helpful  :) I will proceed with the given information06:21
openstackgerritAkihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error  https://review.openstack.org/22393506:24
*** kiran-r has quit IRC06:24
*** markvoelker has quit IRC06:25
*** shashank_hegde has joined #openstack-neutron06:26
*** ferest has joined #openstack-neutron06:26
*** ferest has quit IRC06:31
openstackgerritAkihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error  https://review.openstack.org/22393506:33
amotokireedip: ^^06:33
*** bharathm has joined #openstack-neutron06:34
*** ildikov has joined #openstack-neutron06:36
*** moshele has joined #openstack-neutron06:37
*** obondarev has quit IRC06:37
*** stevemar has joined #openstack-neutron06:38
*** scheuran has joined #openstack-neutron06:39
*** drjones has joined #openstack-neutron06:40
*** stevemar has quit IRC06:42
*** Murali has joined #openstack-neutron06:42
openstackgerritZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources  https://review.openstack.org/21358606:42
*** _cjones_ has quit IRC06:44
*** nyechiel_ has joined #openstack-neutron06:46
*** sthillma has quit IRC06:47
*** kawa2014 has joined #openstack-neutron06:47
*** yfried_ is now known as yfried|afk06:49
*** abregman has joined #openstack-neutron06:50
*** shivrao has quit IRC06:51
*** shashank_hegde has quit IRC06:51
*** nlahouti has quit IRC06:53
*** ronis has joined #openstack-neutron06:53
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB  https://review.openstack.org/18336906:53
*** k-yasui has joined #openstack-neutron06:55
*** fzdarsky_ has joined #openstack-neutron06:56
*** yfried|afk is now known as yfried_06:57
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for API and DB  https://review.openstack.org/18336906:57
*** sbelous has joined #openstack-neutron07:00
*** vivek-ebay has joined #openstack-neutron07:00
*** sbelous has quit IRC07:00
*** marios has joined #openstack-neutron07:01
*** marios has quit IRC07:01
*** marios has joined #openstack-neutron07:01
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler  https://review.openstack.org/20443607:01
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add functional test for availability_zone support  https://review.openstack.org/21097707:02
*** gampel has joined #openstack-neutron07:02
*** tmorin has joined #openstack-neutron07:02
*** ihrachys has joined #openstack-neutron07:03
*** SridharG has joined #openstack-neutron07:03
*** vivek-eb_ has quit IRC07:04
*** gildub has quit IRC07:04
*** rkukura_ has joined #openstack-neutron07:05
*** achanda has quit IRC07:06
*** nikil89_ has joined #openstack-neutron07:06
*** rkukura has quit IRC07:08
*** rkukura_ is now known as rkukura07:08
*** nyechiel has joined #openstack-neutron07: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 conf07:11
carorawhich file do I need to change.07:11
*** SumitNaiksatam has quit IRC07:11
*** SumitNaiksatam has joined #openstack-neutron07:12
*** alex_klimov has joined #openstack-neutron07:12
*** armax has quit IRC07:12
openstackgerritYuji Azama proposed openstack/neutron: Add head option to neutron-db-manage command  https://review.openstack.org/22390107:13
*** ihrachys has quit IRC07:14
*** yamahata has quit IRC07:14
shihanzhanghi all, who can help to review this patch https://review.openstack.org/#/c/221209/07:14
*** anshul has joined #openstack-neutron07:15
*** matrohon has joined #openstack-neutron07:16
*** nyechiel_ has quit IRC07:19
*** moshele has quit IRC07:19
*** fawadk has quit IRC07:19
*** boris-42 has quit IRC07:19
*** fnordahl has quit IRC07:19
*** marios has quit IRC07:19
*** yuanying has quit IRC07:19
*** azbiswas has quit IRC07:19
*** zhenguo has quit IRC07:19
*** slaweq has joined #openstack-neutron07:19
*** jpena has joined #openstack-neutron07:21
*** hoangcx has quit IRC07:23
*** mohankumar has quit IRC07:23
*** abehl has joined #openstack-neutron07:24
*** marios has joined #openstack-neutron07:24
*** yuanying has joined #openstack-neutron07:24
*** azbiswas has joined #openstack-neutron07:24
*** zhenguo has joined #openstack-neutron07:24
*** abehl has quit IRC07:24
*** abehl has joined #openstack-neutron07:24
*** moshele has joined #openstack-neutron07:25
*** fawadk has joined #openstack-neutron07:25
*** boris-42 has joined #openstack-neutron07:25
*** fnordahl has joined #openstack-neutron07:25
*** markvoelker has joined #openstack-neutron07:26
*** EinstCrazy has quit IRC07:27
*** ferest has joined #openstack-neutron07:28
*** Sukhdev_ has quit IRC07:30
*** dannywilson has quit IRC07:30
*** markvoelker has quit IRC07:30
*** evgenyf has joined #openstack-neutron07:31
*** luqas has joined #openstack-neutron07:31
*** abehl has quit IRC07:33
*** ferest has quit IRC07:33
*** carora has quit IRC07:34
openstackgerritJakub Libosvar proposed openstack/neutron: Fix establishing UDP connection  https://review.openstack.org/21242507:35
*** luqas has quit IRC07:35
*** [1]evgenyf has joined #openstack-neutron07:35
*** evgenyf has quit IRC07:38
*** [1]evgenyf is now known as evgenyf07:38
*** hanzhang has quit IRC07:40
*** obondarev has joined #openstack-neutron07:40
*** ushkalim has joined #openstack-neutron07:40
*** jlanoux has joined #openstack-neutron07:40
*** shz has joined #openstack-neutron07:41
*** vivek-ebay has quit IRC07:41
openstackgerritReedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn  https://review.openstack.org/22395107:42
*** zhhuabj has quit IRC07:44
*** diga has quit IRC07:44
*** zhhuabj has joined #openstack-neutron07:53
*** rossella_s has joined #openstack-neutron07:54
*** drjones has quit IRC07:57
*** _cjones_ has joined #openstack-neutron07:58
*** fawadkhaliq has joined #openstack-neutron07:58
*** ferest has joined #openstack-neutron07:59
*** fawadk has quit IRC07:59
*** nmagnezi has joined #openstack-neutron08:00
*** ihrachys has joined #openstack-neutron08:00
*** ushkalim has quit IRC08:01
*** safchain has joined #openstack-neutron08:01
*** fawadkhaliq has quit IRC08:02
*** ferest has quit IRC08:04
*** pece has joined #openstack-neutron08:04
*** thanhnt-z_ has joined #openstack-neutron08:05
*** thanhnt-z_ has quit IRC08:06
*** jbell8 has joined #openstack-neutron08:07
*** gampel1 has joined #openstack-neutron08:08
reedip@amotoki : Reedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn  https://review.openstack.org/22395108:08
reedip:)08:08
apuimedogsagie: I believe this one could be considered as "fix released" https://bugs.launchpad.net/kuryr/+bug/149288208:08
openstackLaunchpad bug 1492882 in kuryr "Avoid cloning of Neutron repository in CI process" [Undecided,Fix committed] - Assigned to Gal Sagie (gal-sagie)08:08
apuimedosince it is about testing08:08
*** nmagnezi has quit IRC08:08
apuimedoand it was already merged08:08
*** kiran-r has joined #openstack-neutron08:09
*** deepthi has joined #openstack-neutron08:09
*** gampel has quit IRC08:10
*** nmagnezi has joined #openstack-neutron08:10
*** mickeys has joined #openstack-neutron08:10
*** diga has joined #openstack-neutron08:11
*** sdake has joined #openstack-neutron08:11
*** gberginc has joined #openstack-neutron08:11
*** hoangcx has joined #openstack-neutron08:11
*** jbell8 has quit IRC08:12
*** sdake_ has quit IRC08:12
amotokireedip: question: does it work for postgresql?08:13
itzikbmoshele: lennyb: Have you checked the case with the pre-existing port?08:13
*** thanhnt-z has joined #openstack-neutron08:13
*** rotbeard has quit IRC08:14
reedip@amotoki: Postgresql already has case sensitive columns08:15
*** yog_ has quit IRC08:15
reedip@amotoki: I found this information after googling. http://nandovieira.com/using-insensitive-case-columns-in-postgresql-with-citext08:16
*** mickeys has quit IRC08:17
*** ushkalim has joined #openstack-neutron08:17
amotokireedip: 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 insensitive08:18
thanhnt-zhi 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-neutron08:19
amotokireedip: 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
amotokireedip: 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
amotokireedip: does it make sense ?08:23
gsagieapuimedo: done08:24
reedip@amotoki: Yes, it does. I will update the review comment itself08:27
*** ygbo has joined #openstack-neutron08:27
reedipI mean the commit message08:27
digagsagie: Hi, can you look at this change - https://review.openstack.org/#/c/221640/08:27
SridharGkevinbenton: 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 IRC08:29
*** ferest has joined #openstack-neutron08:30
openstackgerritReedip proposed openstack/neutron-fwaas: Change fw rules table for case-sensitive name coulmn  https://review.openstack.org/22395108:30
apuimedogsagie: thanks08:30
apuimedodiga: hi08:30
reedip@amotoki: Done08:30
digaHi08:31
*** moshele has quit IRC08:31
digaapuimedo: hi08:31
digaapuimedo: have you seen the mohammed's comments08:31
*** moshele has joined #openstack-neutron08:32
*** itzikb_ has joined #openstack-neutron08:32
*** ekuris has quit IRC08:32
*** shoutm has quit IRC08:33
digaapuimedo: I have given a reply to that08:34
*** itzikb has quit IRC08:34
*** ferest has quit IRC08:34
*** alexpilotti has joined #openstack-neutron08:35
*** mohankumar has joined #openstack-neutron08:37
gsagiediga: will take a look now08:38
digagsagie: sure08:39
*** SourabhP has joined #openstack-neutron08:40
gsagiediga: 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
gsagieAre you able to catch him online and solve this as he still has -1 on your patch08:41
gsagie?08:41
gsagiei can try to talk with him for you if you are unable due to time zones difference08:41
apuimedogsagie: 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 patch08:42
apuimedogsagie: what do you think about that?08:42
gsagieapuimedo: sounds good to me08:42
gsagiediga: what do you think?08:42
digagsagie: apuimedo : I will address comments given by banix08:43
*** obondarev has quit IRC08:43
digaeither or is fine for me08:43
digaif his patch is good to go then I will move my VIF work08:43
digaapuimedo: gsagie : I dont mind08:44
*** _cjones_ has quit IRC08:45
apuimedodiga: gsagie: so maybe better like that. We take banix's patch as soon as he addresses the comment this afternoon08:45
apuimedoso diga can now work on the binding stuff08:46
apuimedoright now08:46
gsagiediga: just make sure that he addressed everything you need for the binding08:46
gsagieconfiguration wise08:46
apuimedogsagie: good point08:46
digaapuimedo: gsagie : Fine!08:46
digagsagie: let me review his patch, then will give comment what I need extra in config for VIF08:47
*** ushkalim has quit IRC08:47
*** ekuris has joined #openstack-neutron08:48
*** ekuris has quit IRC08:48
*** ekuris has joined #openstack-neutron08:48
apuimedodiga: gsagie: or maybe better just add the extra things as a patch that depends on banix's one08:49
apuimedoit may be faster08:49
apuimedo(and more correct)08:49
irenabapuimedo: +108:49
digaapuimedo: that;s good08:50
apuimedodiga: thanks. So then abandon the config patch and do it like we just agreed :-)08:50
digayes08:50
digaapuimedo: abandoned :)08:52
apuimedodiga: thanks!08:52
*** kbyrne has joined #openstack-neutron08:53
gsagiethanks diga08:55
digaapuimedo: gsagie : welcome!08:55
digaapuimedo: gsagie : we have decided 2377 port for kuryr, right ?08:56
*** l6unchpad has joined #openstack-neutron08:57
*** hichihara has quit IRC08:58
*** hdaniel has joined #openstack-neutron08:59
*** obondarev has joined #openstack-neutron08:59
openstackgerritshihanzhang proposed openstack/neutron: DVR: notify specific agent when create/update port  https://review.openstack.org/22120908:59
*** luqas has joined #openstack-neutron09:00
*** ushkalim has joined #openstack-neutron09:00
*** ferest has joined #openstack-neutron09: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
apuimedodiga: that's right09:01
digaapuimedo: okay09:01
*** hdaniel has quit IRC09:02
*** hdaniel has joined #openstack-neutron09:02
*** obondarev has quit IRC09:04
*** obondarev has joined #openstack-neutron09:04
*** ferest has quit IRC09:05
*** anilvenkata has quit IRC09:07
*** zhhuabj has quit IRC09:08
*** l6unchpad has quit IRC09:11
*** Prem has quit IRC09:14
*** hoangcx has quit IRC09:18
*** aix has joined #openstack-neutron09:18
*** SourabhP has quit IRC09:20
*** neeti has quit IRC09:20
*** rotbeard has joined #openstack-neutron09:23
*** anilvenkata has joined #openstack-neutron09:25
*** yamamoto has quit IRC09:26
*** markvoelker has joined #openstack-neutron09:26
*** moshele has quit IRC09:27
*** moshele has joined #openstack-neutron09:28
*** l6unchpad has joined #openstack-neutron09:28
*** sdake has quit IRC09:30
amotokithanhnt-z: yes, correct. It is hard coded in dhcp-agent code.09:30
*** markvoelker has quit IRC09:31
*** ferest has joined #openstack-neutron09:32
*** kengo_sakai has quit IRC09:33
*** killer_prince has joined #openstack-neutron09:33
*** dims_ has joined #openstack-neutron09:35
*** lazy_prince has quit IRC09:36
*** ferest has quit IRC09:36
ZZellethanhnt-z, it's dhcp<host-uuid>-<network-id>09:36
*** stevemar has joined #openstack-neutron09:40
openstackgerritZhaoBo proposed openstack/neutron-specs: Add timestamp to neutron extension resources  https://review.openstack.org/22398809:43
*** stevemar has quit IRC09:44
openstackgerritCedric Brandily proposed openstack/neutron: Remove "ebtables_installed" useless option  https://review.openstack.org/22399009:45
*** _cjones_ has joined #openstack-neutron09:47
openstackgerritSudhakar Babu Gariganti proposed openstack/neutron: use safer version of configure_dhcp_for_network  https://review.openstack.org/22322609:47
thanhnt-zamotoki, ZZelle: Thank you very much for your support09:47
*** neeti has joined #openstack-neutron09:47
*** baojg has joined #openstack-neutron09:48
*** bigjools has quit IRC09:49
*** bharathm has quit IRC09:50
*** ekuris has quit IRC09:50
*** yfried_ is now known as yfried|afk09:51
*** itzikb_ has quit IRC09:51
*** _cjones_ has quit IRC09:52
*** obondarev has quit IRC09:55
*** fawadkhaliq has joined #openstack-neutron09:59
*** deepthi has joined #openstack-neutron09:59
*** moshele has quit IRC10:00
*** neeti has quit IRC10:01
*** sudipto has joined #openstack-neutron10:02
*** obondarev has joined #openstack-neutron10:03
*** ferest has joined #openstack-neutron10:03
*** sbelous has joined #openstack-neutron10:03
*** fawadkhaliq has quit IRC10:04
*** tryit has quit IRC10:05
*** pc_m has joined #openstack-neutron10:06
openstackgerritCao Xuan Hoang proposed openstack/neutron-specs: Logging API for security group and firewall rules  https://review.openstack.org/20350910:06
*** claudiub has joined #openstack-neutron10:07
openstackgerritZhaoBo proposed openstack/neutron-specs: Add timestamp to neutron extension resources  https://review.openstack.org/22398810:07
*** ferest has quit IRC10:07
*** tomoe_ has quit IRC10:08
openstackgerritKen'ichi Ohmichi proposed openstack/neutron: Use tempest-lib's token_client  https://review.openstack.org/22400610:08
*** yfried|afk is now known as yfried_10:10
*** jkraj has joined #openstack-neutron10:19
ihrachysajo: pushed the refactoring patch for qos. please rebase the 'network ports' one10:21
ajoihrachys, I'm on it10:21
ajoihrachys, right writing the tests10:21
ajoihrachys: thanks :)10:21
ihrachysthanks10:21
*** gberginc has quit IRC10:22
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: Added +x permission to gate hook scripts  https://review.openstack.org/22265510:22
*** obondarev has quit IRC10:23
*** iwamoto has quit IRC10:24
*** yamamoto has joined #openstack-neutron10:26
ihrachysajo: "Ihar and the other DB lieutenants" WAT? :)10:26
ihrachysI am not in db team, at least officially, neither I am a lieutenant anywhere :)10:27
ajoihrachys, I think you lead the DB lieutenants10:27
ihrachysI am a poor guy in the corner writing non-sense patches :)10:27
ajoor you're on the doc I least as far as I remember10:27
ihrachysajo: no way!10:27
ajohmmm10:27
ajolemme check ;)10:27
ihrachysajo: http://docs.openstack.org/developer/neutron/policies/core-reviewers.html#core-review-hierarchy10:28
ajoah no10:28
ajo:)10:28
ajoHenryG & amotoki  :)10:28
ihrachyssee, I am kinda out of the boat :)10:28
amotokiajo: I think Ihar is more familiar  with DB than me. I know about API but ....10:30
*** baojg has quit IRC10:30
ajowe need to amend that doc ;)10:30
*** alex_klimov has quit IRC10:31
*** baojg has joined #openstack-neutron10:31
amotokiajo: agree, but I would like to defer it to Kyle or the next PTL :)10:31
*** ociuhandu has quit IRC10:31
*** yamamoto has quit IRC10:32
amotokiihrachys: earlier today I investigated check_migration check failure in neutron-fwaas https://review.openstack.org/#/c/223935/ .10:33
amotokiihrachys: dutring the debug, one question hit me.10:33
amotokiihrachys: why is the contents of HEADS kept in the alphabetical order?10:33
*** ferest has joined #openstack-neutron10:34
ihrachysamotoki: no particular reason. maybe it was easier for me to write a unit test.10:34
ihrachysamotoki: we actually should kill it. I want to do it in start of M10:34
amotokiihrachys: thanks.10:34
amotokiihrachys: I think it is easy to understand if both HEADS and observed revisons are sorted.10:35
*** baojg has quit IRC10:35
*** mwagner_lap has quit IRC10:36
openstackgerritSudhakar Babu Gariganti proposed openstack/neutron: Avoid full_sync in l3_agent for router updates  https://review.openstack.org/22401910:36
ihrachysamotoki: 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 it10:36
ihrachysamotoki: currently, any new migration script merged makes all other patches conflict.10:36
amotokiihrachys: sounds reasonable. I know the bad world before introduing HEAD file ;-(10:37
*** itzikb_ has joined #openstack-neutron10:38
*** gberginc has joined #openstack-neutron10:38
*** ferest has quit IRC10:38
ihrachysyou are more mature than me in terms of collecting bad project experience :)10:39
amotoki:)10:40
*** anilvenkata has quit IRC10:40
pc_mamotoki: hi10:40
amotokipc_m: hi10:41
pc_mamotoki: Can you weigh in on https://review.openstack.org/#/c/219455/ ?10:41
pc_mamotoki: Question on naming of some of the arguments (endpoint type and endpoint values).10:41
pc_mSee commit message.10:42
*** anilvenkata has joined #openstack-neutron10:42
*** bigjools has joined #openstack-neutron10:42
amotokipc_m: sure. let me check after getting home.10:42
pc_mamotoki: thanks!10:43
pc_mamotoki: FYI that's the client side. The server is https://review.openstack.org/#/c/212692/10:43
*** moshele has joined #openstack-neutron10:44
amotokipc_m: I see it. I should review it too.10:45
pc_mamotoki: thanks!10:45
*** stanzgy__ has quit IRC10:48
*** stanzgy has quit IRC10:49
*** ZZelle has quit IRC10:50
*** aix has quit IRC10:50
*** ZZelle has joined #openstack-neutron10:50
*** fawadkhaliq has joined #openstack-neutron10:55
*** shz has quit IRC10:59
*** shz has joined #openstack-neutron11:00
*** obondarev has joined #openstack-neutron11:01
*** b3nt_pin has quit IRC11:01
openstackgerritMerged openstack/neutron-vpnaas: Exclude neutron_vpnaas/tests from coverage report  https://review.openstack.org/22367311:02
*** beagles has joined #openstack-neutron11:02
*** beagles is now known as Guest8682511:02
*** obondarev has quit IRC11:03
*** aix has joined #openstack-neutron11:03
openstackgerritMerged openstack/neutron: Remove requirements.txt for the ofagent mechanism driver  https://review.openstack.org/22390311:03
*** yamamoto has joined #openstack-neutron11:03
*** yamamoto has quit IRC11:03
*** sbelous has quit IRC11:04
*** ferest has joined #openstack-neutron11:04
*** itsuugo has joined #openstack-neutron11:04
*** irenab has quit IRC11:05
*** mohankumar has quit IRC11:06
*** ferest has quit IRC11:08
*** alex_klimov has joined #openstack-neutron11:08
*** yfried_ is now known as yfried|afk11:11
*** itsuugo has quit IRC11:11
*** lazy_prince has joined #openstack-neutron11:13
*** tmorin has quit IRC11:15
*** killer_prince has quit IRC11:16
openstackgerritMerged openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445511:16
*** _cjones_ has joined #openstack-neutron11:17
*** itsuugo has joined #openstack-neutron11:19
*** ociuhandu has joined #openstack-neutron11:21
*** _cjones_ has quit IRC11:22
*** Guest86825 has quit IRC11:23
*** banix has joined #openstack-neutron11:23
*** gongysh has quit IRC11:24
*** beagles__ has joined #openstack-neutron11:24
gsagieping irenab11:25
*** itsuugo has quit IRC11:26
*** banix has quit IRC11:27
*** markvoelker has joined #openstack-neutron11:27
*** julim has joined #openstack-neutron11:28
*** diga has quit IRC11:30
*** yfried|afk is now known as yfried_11:31
*** markvoelker has quit IRC11:32
openstackgerritMiguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies  https://review.openstack.org/22016411:32
ajoihrachys ^11:32
*** fawadkhaliq has quit IRC11:33
*** beagles__ has quit IRC11:33
*** ferest has joined #openstack-neutron11:34
*** tbachman has joined #openstack-neutron11:37
*** ferest has quit IRC11:39
*** yfried_ is now known as yfried|afk11:41
*** jbell8 has joined #openstack-neutron11:45
*** irenab has joined #openstack-neutron11:45
*** jpena is now known as jpena|lunch11:45
*** tomoe_ has joined #openstack-neutron11:45
*** rotbeard has quit IRC11:46
*** fawadkhaliq has joined #openstack-neutron11:46
*** l6unchpad has quit IRC11:48
*** rotbeard has joined #openstack-neutron11:49
*** obondarev has joined #openstack-neutron11:49
*** jbell8 has quit IRC11:49
*** boris-42 has quit IRC11:50
*** yfried|afk is now known as yfried_11:51
*** arajagopal has quit IRC11:52
*** arajagopal has joined #openstack-neutron11:52
*** mohankumar has joined #openstack-neutron11:55
*** thorst has joined #openstack-neutron11:55
*** k-yasui has quit IRC11:57
*** arajagopal has quit IRC11:57
*** baoli has joined #openstack-neutron11:57
*** baoli has quit IRC11:57
*** gampel has joined #openstack-neutron11:57
*** baoli has joined #openstack-neutron11:58
*** baoli_ has joined #openstack-neutron11:59
*** gampel1 has quit IRC12:00
*** tkatarki has joined #openstack-neutron12:01
*** fawadkhaliq has quit IRC12:01
*** baoli has quit IRC12:03
*** evgenyf has quit IRC12:04
*** yamamoto has joined #openstack-neutron12:04
ihrachysajo: I left some dumb grammar nits for the qos patch, please respin and I am happy to +212:04
*** itsuugo has joined #openstack-neutron12:05
*** ferest has joined #openstack-neutron12:05
*** jckasper has quit IRC12:05
*** yfried_ is now known as yfried|afk12:07
*** l6unchpad has joined #openstack-neutron12:09
*** tkatarki has quit IRC12:09
*** evgenyf has joined #openstack-neutron12:09
ajoihrachys: ack, thanks a lot, I will respin in a few minutes12:09
*** yamamoto has quit IRC12:09
*** ferest has quit IRC12:09
*** fawadkhaliq has joined #openstack-neutron12:10
*** cbits has quit IRC12:10
HenryGihrachys: 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
ihrachysHenryG: yes, that's why I wrote that I need to check whether we don't miss any cases that would allow it12:12
*** thanhnt-z has quit IRC12:12
*** markvoelker has joined #openstack-neutron12:12
*** flaviof has quit IRC12:12
ihrachysHenryG: 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-neutron12:13
ihrachysHenryG: we could just allow them, and then merge on final release12:13
ihrachysthat would unblock all those conflict resolutions and all12:13
HenryGihrachys: Maybe12:14
*** mwagner_lap has joined #openstack-neutron12:16
*** beagles has joined #openstack-neutron12:16
ajoihrachys, irenab , moshele , gsagie  today we have nova folks joining the QoS meeting for the nova/neutron coordination thing.12:17
ihrachysack12:17
irenabajo: I will join a bit later than usual12:17
ajoirenab, ack,12:18
*** obondarev has quit IRC12:18
*** _cjones_ has joined #openstack-neutron12:19
*** yfried|afk is now known as yfried_12:19
*** tkatarki has quit IRC12:20
*** beagles has quit IRC12:21
*** itsuugo has quit IRC12:22
*** EinstCrazy has joined #openstack-neutron12:23
*** leitan has joined #openstack-neutron12:23
*** _cjones_ has quit IRC12:23
*** tmorin has joined #openstack-neutron12:24
*** kiran-r has quit IRC12:25
*** jaypipes has quit IRC12:26
*** jaypipes_ has joined #openstack-neutron12:26
*** MVenesio has joined #openstack-neutron12:28
*** tkatarki has joined #openstack-neutron12:28
*** manas has joined #openstack-neutron12:28
*** sudipto has quit IRC12:30
ajojohnthetubaguy, are you able to join today the QoS meeting for nova/neutron integration?, Nikola Dipanov and Vladik Romanovsky are joining12:32
johnthetubaguyajo: I can lurk, what time was that again?12:33
ajonow + 1:30h12:33
ajojohnthetubaguy ^12:33
*** edmondsw has joined #openstack-neutron12:33
johnthetubaguyajo: OK, I will watch out for that, thanks for the ping12:33
ajojohnthetubaguy: thanks :)12:33
ajojohnthetubaguy, last time we talked you commented something about a new vif binding library12:34
ajoor something like that12:34
ajoany link/pointer to it?12:34
*** obondarev has joined #openstack-neutron12:35
*** deepthi has quit IRC12:35
*** yfried_ is now known as yfried|afk12:36
*** tongli has joined #openstack-neutron12:38
*** tkatarki has quit IRC12:39
*** EinstCrazy has quit IRC12:40
*** tkatarki has joined #openstack-neutron12:42
*** yfried|afk is now known as yfried_12:42
*** tkatarki has quit IRC12:43
openstackgerritAndreas Scheuring proposed openstack/neutron: Make OVS interface name hasing algorithm common and extend it  https://review.openstack.org/22406412:43
*** adam_g has quit IRC12:44
*** kiran-r has joined #openstack-neutron12:44
*** tkatarki has joined #openstack-neutron12:45
*** jpena|lunch is now known as jpena12:45
*** Samos123 has quit IRC12:47
*** wendar has quit IRC12:47
*** wendar has joined #openstack-neutron12:47
*** dims_ has quit IRC12:47
*** sbalukoff has quit IRC12:48
*** dsneddon has quit IRC12:48
*** tkatarki has quit IRC12:49
*** lbragstad has quit IRC12:49
*** dsneddon has joined #openstack-neutron12:49
*** comstud has quit IRC12:50
*** yamamoto has joined #openstack-neutron12:50
*** sbalukoff has joined #openstack-neutron12:50
*** adam_g has joined #openstack-neutron12:50
*** bharathm has joined #openstack-neutron12:51
*** adam_g has quit IRC12:51
*** adam_g has joined #openstack-neutron12:51
*** comstud has joined #openstack-neutron12:51
*** lbragstad has joined #openstack-neutron12:51
*** cbits has joined #openstack-neutron12:52
*** Samos123 has joined #openstack-neutron12:52
*** tbachman has quit IRC12:54
*** annegentle has joined #openstack-neutron12:54
*** bharathm has quit IRC12:55
johnthetubaguyajo: oops, sorry got distracted by some script tweaks, will see if we have a pointer to that somewhere12:56
ajojohnthetubaguy, thanks, no problem :)12:56
johnthetubaguyajo: I think this is a rough pointer: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067783.html12:56
*** aix has quit IRC12:56
johnthetubaguyajo: not sure how the details of that are panning out yet, there was quite a bit of chatter at the midcycle12:57
ajoreading :)12:57
johnthetubaguywas hoping that will start in early mitaka12:57
*** shoutm has joined #openstack-neutron12:58
*** rpothier has joined #openstack-neutron12:58
*** dims_ has joined #openstack-neutron12:58
matrohonamuller : hi12:59
*** Murali has quit IRC12:59
ajojohnthetubaguy, sounds good, I failed to read the thread from MLo riginally12:59
ajojohnthetubaguy: I need to read the spec carefully btw13:00
johnthetubaguynot sure the spec has the latest thoughts on the subject, but those are both good places to start13:00
*** arajagopal has joined #openstack-neutron13:00
*** jckasper has joined #openstack-neutron13:01
*** flaviof has joined #openstack-neutron13:01
*** tbachman has joined #openstack-neutron13:01
*** shwetaap has joined #openstack-neutron13:02
*** dsneddon has quit IRC13:02
*** annemccormick has joined #openstack-neutron13:03
*** ushkalim has quit IRC13:04
*** amuller has joined #openstack-neutron13:04
*** dr3662 has joined #openstack-neutron13:05
*** itzikb_ has quit IRC13:05
*** lindycoder has joined #openstack-neutron13:06
*** Samos123 has quit IRC13:06
*** gampel1 has joined #openstack-neutron13:06
*** gampel has quit IRC13:07
*** jkraj has quit IRC13:10
*** aix has joined #openstack-neutron13:10
*** dr3662__ has joined #openstack-neutron13:11
*** dr3662__ has quit IRC13:12
*** keopp has joined #openstack-neutron13:12
*** MVenesio has quit IRC13:14
*** dr3662 has quit IRC13:14
*** jprovazn has joined #openstack-neutron13:14
*** tomoe_ has quit IRC13:15
*** MVenesio has joined #openstack-neutron13:15
*** beagles has joined #openstack-neutron13:15
*** mohankumar has quit IRC13:15
*** Samos123 has joined #openstack-neutron13:15
*** dsneddon has joined #openstack-neutron13:16
*** rotbeard has quit IRC13:16
*** abregman is now known as abregman|brb13:17
*** itsuugo has joined #openstack-neutron13:17
*** pece has quit IRC13:19
*** shwetaap1 has joined #openstack-neutron13:19
*** _cjones_ has joined #openstack-neutron13:19
*** ushkalim has joined #openstack-neutron13:20
*** krtaylor has quit IRC13:20
*** gongysh has joined #openstack-neutron13:20
*** tiswanso has joined #openstack-neutron13:20
*** shwetaap has quit IRC13:22
*** bdemers has joined #openstack-neutron13:23
*** tkatarki has joined #openstack-neutron13:23
*** _cjones_ has quit IRC13:24
*** bdemers has quit IRC13:27
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: tox.ini: switch to --subproject for check-migration  https://review.openstack.org/22408213:28
*** yuanying_ has joined #openstack-neutron13:29
*** yuanying has quit IRC13:29
*** yuanying_ is now known as yuanying13:29
*** gongysh has quit IRC13:29
*** annegentle has quit IRC13:30
*** annegentle has joined #openstack-neutron13:30
*** bdemers has joined #openstack-neutron13:31
*** salv-orlando has joined #openstack-neutron13:31
*** kiran-r has quit IRC13:32
*** sudipto has joined #openstack-neutron13:32
*** fawadkhaliq has quit IRC13:33
*** numans has quit IRC13:35
*** itsuugo has quit IRC13:39
*** arajagopal has quit IRC13:40
*** achanda has joined #openstack-neutron13:40
*** arajagopal has joined #openstack-neutron13:40
*** achanda has quit IRC13:41
*** amotoki has quit IRC13:42
*** LukeHinds has joined #openstack-neutron13:42
*** arajagopal has quit IRC13:42
*** arajagopal has joined #openstack-neutron13:43
*** zhenguo has quit IRC13:43
openstackgerritAkihiro Motoki proposed openstack/neutron-fwaas: Fix check_migration error  https://review.openstack.org/22393513:45
*** arajagopal has quit IRC13:50
*** arajagopal has joined #openstack-neutron13:50
openstackgerritZongKai LI proposed openstack/neutron: add return in for-loop in unbind_router_servicenode  https://review.openstack.org/22343513:51
*** abregman|brb is now known as abregman13:51
ihrachysajo: speaking of responsibilities :))) what's the qos doc story? I saw some assignment moves on the bug13:52
ajoihrachys, I need to sync to emagana, I know he's working on it while I finished bugs13:53
ajonow that bugs are over, (well, he sent me an email with another one.. but that's easy)13:53
*** kbringard has joined #openstack-neutron13:53
ihrachysajo: ok cool. just wanted to make sure it's under control :)13:53
ajoihrachys: 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 exception13:54
ihrachysajo: we also need that upgrade write-up, but it's fine for after L13:54
ihrachysajo: how kaboom is it?13:54
ajoihrachys: let me resend you the email13:54
ajoihrachys I think I will fill a bug now, and make a simple patch for it13:54
ihrachysajo: that should probably be in some bug tracker insteawd13:54
ajoexchanging the kaboom exception for an explanation humans can understand13:54
ihrachysajo: right :)13:54
*** yamahata has joined #openstack-neutron13:54
*** arajagopal has quit IRC13:55
ajoihrachys, it's something our "move the -check port changes for extensions- in ml2/plugin.py to the extensions" would fix too13:55
ihrachysajo: aye, we need core resource extender manager :)13:56
ajoihrachys: yes13:56
ajoihrachys: we must fill an RFE for M13:56
ihrachysajo: let me handle the piece in M13:56
ajoihrachys++ sync with armax, he was interested in the topic13:56
ihrachysroger13:57
ajoQoS meeting in T+3m13:57
ajoI almost forgot, so nice I have calendar integration back to my desktop13:57
ajo:')13:57
*** dane_leblanc has joined #openstack-neutron13:58
*** nate_gone is now known as njohnston13:58
*** pradk has joined #openstack-neutron13:59
ajomoshele ping, could you join #openstack-meeting-3 ?  :)14:00
mosheleajo: sure14:01
*** sayali has quit IRC14:02
*** zzzeek has joined #openstack-neutron14:02
*** lajos-katona has left #openstack-neutron14:03
*** wverdugo500 has quit IRC14:03
*** sridhar_ram has joined #openstack-neutron14:04
salv-orlandoHenryG, akamyshnikova: is it correct that each subproject from a migration perspective has a different alembic environment?14:04
*** ushkalim has quit IRC14:06
HenryGsalv-orlando: yes, although it is 99% a clone of neutron's14:06
*** arajagopal has joined #openstack-neutron14:06
salv-orlandoHenryG: sure. I am aiming at understanding how I can deal with dependency among subprojects14:06
*** vthapar has joined #openstack-neutron14:06
HenryGsalv-orlando: Just the script_location and alembic_branches table is different14:06
vthapartmorin: ping14:06
akamyshnikovasalv-orlando, yes14:07
tmorinvthapar: icmp echo reply14:07
salv-orlandoHenryG: 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 different14:07
tmorinvthapar: I'm reading, go ahead ;)14:07
HenryGsalv-orlando: Abhishek has asked me about this14:08
vthapartmorin: had couple questions on networking-bgpvpn. I'm working on ODL driver and wanted some clarity on couple things14:08
salv-orlandoas we realized openstack/vmware-nsx has a dependency on openstack/networking-l2gw I need to devise an alternative strategy14:08
salv-orlandoHenryG: yes I know... I just wanted to confirm with you that this was impossible for the foresseable future14:08
tmorinvthapar: yes, I've seen your name on https://git.opendaylight.org/gerrit/#/c/26711/514:08
vthapartmorin: 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-neutron14:09
*** obondarev has quit IRC14:09
vthapartmorin: and does it apply to both create and update?14:09
HenryGsalv-orlando: I *think* if we put all sub-project's alembic branches in one table we can support stuff like what you need14:09
*** pradk has quit IRC14:10
*** arajagopal has quit IRC14:10
tmorinvthapar: the default values are filled in by the API engine for create14:10
HenryGsalv-orlando: but I am still trying to figure out how to get there14:10
*** mragupat has quit IRC14:10
salv-orlandoHenryG: one table? what do you mean? I thought we were collecting subprojects by entrypoints14:10
*** aix has quit IRC14:11
tmorinvthapar: for updates, I don't know, in an update only the updated fields are specified, and by definition they do not have a default value14:11
HenryGsalv-orlando: the alembic branch heads are stored in a table14:11
vthapartmorin: def update_bgpvpn_postcommit(self, context, old_bgpvpn, bgpvpn):14:12
tmorinvthapar: the resource map defines the default values here: https://github.com/openstack/networking-bgpvpn/blob/master/networking_bgpvpn/neutron/extensions/bgpvpn.py#L6914:12
akamyshnikovaHenryG, yes, that could be solution for such cases14:12
vthapartmorin: will bgpvpn be the delta [new items added] or the updated object with old+changes?14:12
*** pradk has joined #openstack-neutron14:12
*** r-daneel has joined #openstack-neutron14:12
*** julim has quit IRC14:12
vthapartomirn: 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 IRC14:13
tmorinvthapar: 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 driver14:13
akamyshnikovaHenryG, but may be zzzeek can suggest something else14:13
salv-orlandoakamyshnikova: 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-neutron14:14
vthapartmorin: what I noticed was that status and admin_state were coming as null.14:14
vthapartmorin: ^in update. create fills everything correctly.14:14
tmorinvthapar: old_bgpvpn is the old content, and bgpvpn is the new14:14
vthapartmorin: so all default values should be filled in bgpvpn, right?14:15
tmorinvthapar: at least that's my understanding, but I haven't been deep in that to be 100% sure14:15
HenryGakamyshnikova: zzzeek suggested to use one table for all branches to support a single env14:16
*** Marga_ has joined #openstack-neutron14:16
tmorinvthapar: just checked, yes, that's what it does (pass old and new)14:16
*** minwang2 has joined #openstack-neutron14:16
*** yog_ has joined #openstack-neutron14:16
tmorinvthapar: indeed, admin_status is not implemented yet14:16
vthapartmorin: 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-neutron14:16
zzzeekHenryG: yes because that’s the feature I put a ton of work into making very robust, and it directly considers this use case14:16
vthapartmorin: aha! that explains it :)14:17
tmorinvthapar: well, it does not explain why it would be non-null for create14:17
zzzeekHenryG: it would make it very easy for out-of-tree plugins to declare dependences in in-tree migrations and structures for example14:17
*** lazy_prince has quit IRC14:17
akamyshnikovasalv-orlando, no, I think about revisions, because branches can have such problems later...14:17
tmorinvthapar: with the current code, it should be null for both14:17
zzzeekdependencies *on*14:17
* vthapar checking logs.14:18
HenryGzzzeek: salv-orlando has a dependency between two out-of-tree projects14:18
vthapartmorin: 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-neutron14:19
tmorinvathapar: 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 IRC14:19
*** dane_leblanc has joined #openstack-neutron14:20
tmorintmorin: I would in fact expect the Yang datamodel to possibly define a default value for any field, a bit like Neutron resource map does14:20
*** nplanel has quit IRC14:20
vthapartmoring: 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
openstackgerritAssaf Muller proposed openstack/neutron: Delete unused file tests/unit/database_stubs.py  https://review.openstack.org/22410414:20
*** _cjones_ has joined #openstack-neutron14:21
tmorinvthapar: yes, prem told us yesterday on IRC, great news !14:21
tmorinvthapar: out of curiosity, what is the BGP implementation used in the backend  ? (forgot to ask yesterday)14:21
*** clarkb1 is now known as clarkb14:21
*** aix has joined #openstack-neutron14:22
vthapartmorin: 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
tmorinvthapar: yep, I was wondering if you already had the implementation with ODL BGP or if it was still quagga14:23
tmorinvthapar: is the quagga integration also part of the recent gerrit change ?14:24
*** pradk has quit IRC14:24
vthapartmorin: no.14:24
*** obondarev has joined #openstack-neutron14:24
*** pradk has joined #openstack-neutron14:24
tmorinvthapar: ok, so the commit is only one part of the target solution14:24
vthapartmorin: yep. BGP integration will work on other side of ODL and other guys are working on it already.14:25
*** _cjones_ has quit IRC14:25
tmorinvthapar: excellent14:26
tmorinvthapar: anyhow, we'll be ready to review the BGPVPN ODL driver when it comes14:27
*** sdake_ has joined #openstack-neutron14:27
salv-orlandoHenryG, 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-neutron14:28
*** sayali has quit IRC14:28
*** anshul has quit IRC14:28
HenryGsalv-orlando: can zzzeek and I get back to you in a couple of days with a plan?14:28
*** alexpilotti has quit IRC14:29
HenryGsalv-orlando: I think I know what is needed, and zzzeek will help me14:29
salv-orlandoHenryG I'm not asking for you to add the feature for me ;)14:30
HenryGsalv-orlando: It's for neutron and openstack :)14:30
pc_makamyshnikova: ping14:30
salv-orlandoI need for my subproject, and I was going to do it myself if necessary!14:30
*** julien_ZTE has joined #openstack-neutron14:30
HenryGsalv-orlando: don't be so selfish :)14:31
akamyshnikovapc_m, hi14:31
pc_makamyshnikova: Do you know if there is any issue with naming a model field 'type'?14:31
*** openstackgerrit has quit IRC14:31
salv-orlandoHenryG: surely switching from separate alembic envs to a common one is something I'd delay after Liberty14:31
pc_makamyshnikova: There was a suggestion to rename an enum from endpoint_type to type.14:31
pc_mDidn't know if that would be a problem.14:31
*** hdaniel has quit IRC14:32
HenryGsalv-orlando: of course14:32
*** openstackgerrit has joined #openstack-neutron14:32
*** flaviof_ has joined #openstack-neutron14:32
HenryGsalv-orlando: I need to step out for a few hours. Back later14:33
akamyshnikovapc_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_makamyshnikova: thanks14:34
openstackgerritMerged openstack/neutron: Always return iterables in L3 get_candidates  https://review.openstack.org/22282814:34
salv-orlandothanks HenryG you already answered all my question. If you need help with unifying alembic envs consider myself and abhraut your servants14:34
*** tongli_ has joined #openstack-neutron14:35
*** flaviof has quit IRC14:36
akamyshnikovapc_m, np14:36
*** Marga_ has quit IRC14:36
*** julim has joined #openstack-neutron14:37
*** Marga_ has joined #openstack-neutron14:37
*** tongli has quit IRC14:37
*** david-lyle has quit IRC14:37
*** minwang2 has quit IRC14:40
*** sayali has joined #openstack-neutron14:41
*** minwang2 has joined #openstack-neutron14:41
openstackgerritMerged openstack/neutron: QoS agent extension and driver refactoring  https://review.openstack.org/21421814:42
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Make chown rootwrap filter ipsec.secrets file specific  https://review.openstack.org/22413314:42
*** mriedem has joined #openstack-neutron14:45
mriedemis 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_37914:45
mriedemthose new constraints jobs are blowing up the uncategorized bugs page in the gate http://status.openstack.org/elastic-recheck/data/uncategorized.html14:46
*** charlesr has joined #openstack-neutron14:46
amullermriedem: Thanks for bringing that to our attention14:47
mriedemi'm opening a bug now14:47
amullermriedem: Thanks14:48
*** arajagopal has joined #openstack-neutron14:49
*** l6unchpad has quit IRC14:49
*** arajagopal has quit IRC14:49
mriedemhttps://bugs.launchpad.net/neutron/+bug/149642414:49
openstackLaunchpad 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-neutron14:49
mriedemi assume this is a non-voting job?14:50
amullermriedem: yes14:50
mriedemit shouldn't be in the gate queue then14:50
amullerI agree14:50
mriedempatch incoming to infra in a few minutes14:50
amullermriedem: looks like these failures started today?14:51
amullerI wonder what changed14:51
mriedemyesterday14:51
amullerno new tox release14:51
amullermust be an infra patch14:52
amullerI don't see any obvious patch in project-config14:53
*** Prem has joined #openstack-neutron14:53
*** annegent_ has joined #openstack-neutron14:54
*** tidwellr has joined #openstack-neutron14:54
*** annegentle has quit IRC14:54
*** l6unchpad has joined #openstack-neutron14:55
*** nmagnezi has quit IRC14:56
*** vthapar has quit IRC14:56
amullermriedem: 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,z14:56
mriedemyeah14:57
*** adduarte has quit IRC14:57
*** tkatarki has quit IRC14:57
mriedemthat's because c029954c8ae041e5f15b14ceef0e2aa060928e05 isn't on the feature/pecan branch14:57
mriedemgah, feature branches14:58
*** dims_ has quit IRC14:58
*** stevemar has joined #openstack-neutron14:58
*** carl_baldwin has joined #openstack-neutron14:58
mriedemi can fix that quick14:58
*** dims_ has joined #openstack-neutron14:58
*** tkatarki has joined #openstack-neutron14:58
*** jistr is now known as jistr|call14:58
amullermriedem: Allow me :)14:59
*** sridhar_ram has quit IRC14:59
mriedemtoo late14:59
amullermriedem: https://review.openstack.org/#/c/224138/14:59
amullerbam14:59
mriedemadd bug 1496424 to that commit message15:00
openstackbug 1496424 in neutron "gate-neutron-python27(34)-constraints jobs blowing up in gate due to misconfig" [Undecided,New] https://launchpad.net/bugs/149642415:00
mriedemcloses-bug 149642415:00
*** tomoe_ has joined #openstack-neutron15:00
*** boris-42 has joined #openstack-neutron15:01
amullerdone15:01
*** openstackgerrit has quit IRC15:01
amullerself approved15:01
amulleralright15:01
amullerwhat about stable branches15:01
*** ygbo has quit IRC15:01
mriedemnot a problem on stable i don't think15:01
*** openstackgerrit has joined #openstack-neutron15:02
*** tkatarki has quit IRC15:02
mriedem  - name: ^gate-.*-py(?:thon\d\d|py)-constraints$15:02
mriedem    branch: ^(?!stable/(?:juno|kilo)).*$15:02
*** l6unchpad has quit IRC15:03
*** tkatarki has joined #openstack-neutron15:03
*** vivek-ebay has joined #openstack-neutron15:04
amulleralright15:04
amullermriedem: Thanks for highlighting this issue!15:04
*** yfried_ is now known as yfried|afk15:04
mriedemnp15:04
openstackgerritMiguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies  https://review.openstack.org/22016415:06
ajoihrachys ^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
ihrachysajo: lol15:06
ajo'':)15:07
ajovery sad, it could have passed CI already15:07
*** banix has joined #openstack-neutron15:07
*** kawa2014 has quit IRC15:08
*** jckasper has quit IRC15:08
*** mickeys has joined #openstack-neutron15:08
*** vivek-ebay has quit IRC15:09
*** hdaniel has joined #openstack-neutron15:09
sc68calscheuran: saw your review for the hashing. Will review probably tomorrow - travel today15:10
*** yfried|afk is now known as yfried_15:10
scheuransc68cal, sure no problem. Thanks!15:10
*** emagana has joined #openstack-neutron15:12
*** tkatarki has quit IRC15:12
*** tkatarki has joined #openstack-neutron15:13
ihrachyseezhova: ping re https://bugs.launchpad.net/bugs/127669415:14
openstackLaunchpad bug 1276694 in Sahara "Openstack services should support SIGHUP signal" [Medium,In progress] - Assigned to Sergey Lukjanov (slukjanov)15:14
ihrachyseezhova: it's marked for RC1, but it does not seem like the patch https://review.openstack.org/#/c/170905/ is ready for review15:14
*** slaweq has quit IRC15:14
ihrachysajo: +215:16
*** fawadkhaliq has joined #openstack-neutron15:16
*** pc_m is now known as pc_m_AWAY15:16
*** jckasper has joined #openstack-neutron15:16
*** sean-k-m1 has joined #openstack-neutron15:17
*** tomoe_ has quit IRC15:17
*** sean-k-mooney has quit IRC15:18
*** gampel1 has quit IRC15:19
*** mhickey has joined #openstack-neutron15:19
*** ajmiller has joined #openstack-neutron15:21
*** ronis has quit IRC15:21
*** ijw_ has quit IRC15:22
*** ijw has joined #openstack-neutron15:22
*** manas has quit IRC15:23
*** jaypipes_ is now known as jaypipes15:23
*** pumaranikar has joined #openstack-neutron15:23
*** Marga_ has quit IRC15:24
emaganaajo: Trying your suggestion for the QoS testing15:24
*** yamamoto has quit IRC15:24
irenabajo: https://etherpad.openstack.org/p/qos-nova-neutron15:24
mhickeyHi Everyone! I am a newbie to neutron!:)15:25
*** Sukhdev_ has joined #openstack-neutron15:26
*** SridharG has quit IRC15:30
*** shoutm has quit IRC15:30
*** lazy_prince has joined #openstack-neutron15:30
*** jistr|call is now known as jistr15:30
*** scheuran has quit IRC15:32
*** annegent_ has quit IRC15:32
*** annegentle has joined #openstack-neutron15:33
*** david-ly_ has joined #openstack-neutron15:35
*** bharathm has joined #openstack-neutron15:35
*** sudipto has quit IRC15:36
*** vthapar has joined #openstack-neutron15:36
vthapartmorin: ping15:36
*** TrevorV has joined #openstack-neutron15:37
tmorinvthapar: yes ?15:37
vthapartmorin: needed some inputs on writing associate_network function on driver.15:38
*** sudipto has joined #openstack-neutron15:39
vthapartmorin: 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 IRC15:39
vthapartmorin: or should I do it differently?15:39
*** armax has joined #openstack-neutron15:39
tmorinvthapar: yes, that looks like a possible way15:39
tmorinvthapar: you could also query self.bgpvpn_db.get_xxxx15:40
vthapartmorin: aha, that does sound a lot cleaner. thanks :)15:40
*** jbell8 has joined #openstack-neutron15:41
*** david-ly_ is now known as david-lyle15:41
tmorinvthapar: not much difference in the end, but yes, cleaner :)15:41
*** ushkalim has quit IRC15:41
*** jbell8 has quit IRC15:42
*** Piet has joined #openstack-neutron15:43
vthapartmorin: bgpvpn_db is defined in base driver?15:43
*** Sukhdev_ has quit IRC15:44
*** yfried_ has quit IRC15:44
*** Swami has joined #openstack-neutron15:44
*** abregman has quit IRC15:46
tmorinvthapar: in BGPVPNDriverBase which the ODL driver will inherit from, so you'd call self.bgpvpn_db.get_bgpvpn15:46
*** shivrao has joined #openstack-neutron15:46
*** minwang2 has joined #openstack-neutron15:46
*** Sukhdev has joined #openstack-neutron15:47
*** moshele has quit IRC15:47
vthapartmorin: I am using class OpenDaylightBgpvpnDriver(driver_api.BGPVPNDriver):15:50
vthapartmorin: is that okay or should I replace it with BGPVPNDriverBase?15:50
*** nlahouti has joined #openstack-neutron15:51
*** krtaylor has joined #openstack-neutron15:51
*** pumaranikar has quit IRC15:51
*** ByPasS has quit IRC15:52
*** dane_leblanc has quit IRC15:52
tmorinvthapar: 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_xxxx15:52
vthapartmorin: yes, it does rely on Neutron-side DB. will use DriverBase then.15:53
tmorinvthapar: if not (no persistency Neutron-side) then you should inherit from BGPVPNDriverBase and you can't by definition use the Neutron-side DB15:54
*** jistr has quit IRC15:54
tmorinvthapar: yes15:54
*** ByPasS has joined #openstack-neutron15:54
*** devlaps has joined #openstack-neutron15:54
tmorinvthapar: same for the bagpipe driver, but it's not yet updated (wip at https://review.openstack.org/224154)15:54
*** mriedem has left #openstack-neutron15:56
*** pece has joined #openstack-neutron15:57
vthapartmorin: aha, I used that as reference.15:57
tmorinvthapar: arf... sorry...15:58
vthapartmorin: np, one of the challenges of working with wip projects.15:58
tmorinvthapar: yes15:59
*** minwang2 has quit IRC15:59
*** zhangjn has joined #openstack-neutron16:00
tmorinvthapar: sorry, I'm mixing things up16:01
tmorinvthapar: let me recap16:01
*** evgenyf has quit IRC16:01
matrohonvthapar, hi16:01
*** pumaranikar has joined #openstack-neutron16:01
tmorinvthapar: a driver that wants Neutron-side persistency needs to inherit from BGPVPNDriver, which the bagpipe driver does already16:01
tmorinvthapar: and what you'll do as well16:02
tmorinhi matrohon16:02
*** l6unchpad has joined #openstack-neutron16:02
vthaparmatrohorn, hi.16:02
matrohontmorin, vthapar : +1 inherit from BGPVPNDriver and overlod methods that you need16:02
matrohonvthapar, you should inherit postcommit methods16:03
armaxamuller: ping16:03
amullerarmax: yo16:03
armaxamuller: yo yo16:03
armaxamuller: question16:03
vthaparmatrohorn, 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 from16:03
matrohonvthapar, I just pushed an initial implem of the bagpipe driver, adapted to last API/framework changes16:03
matrohonvthapar, https://review.openstack.org/#/c/224154/16:04
tmorinmatrohon: you mean overload postcommit methods16:04
armaxamuller: did you look into the most recent failure modes for the functional job?16:04
matrohontmorin, yep sorry16:04
vthaparmatrohorn, will take a look. I am trying to get basic driver working and once do some testing will push it for review.16:04
armaxamuller: it looks like the failure rate is still rather high16:04
tmorinvthapar: your driver should overload only the _postcommit methods16:04
matrohonvthapar, sounds great16:04
armaxamuller: with all the recent fixes it looks like we must have pissed it off even further somehow16:04
*** EinstCrazy has joined #openstack-neutron16:04
vthapartmorin: what about associate/disassociate? need to pass that information to ODL.16:05
amullerarmax: I didn't have the chance to look in to in the last week, been in a conference then PTO16:05
armaxok16:05
matrohonvthapar, you have an associate_postcommit method16:05
*** hdaniel has quit IRC16:05
*** jlanoux has quit IRC16:05
matrohonvthapar, same  for disassociate16:06
*** EinstCrazy has quit IRC16:06
*** sballe has joined #openstack-neutron16:06
*** slaweq has joined #openstack-neutron16:06
vthapartmorin, matrohorn: oops. yep, that is what I meant. postcommit methods for create, udpate, delete, associate and disassociate. forgot that the associate one is also postcommit16:06
Sam-I-Amsc68cal: moo.16:06
*** minwang2 has joined #openstack-neutron16:07
amullerarmax: wow what happened it really spiked tonight16:07
amuller*sighs*16:07
armaxamuller: I haven’t looked at it yet16:07
armaxhence I was checking with you first16:07
*** fawadkhaliq has quit IRC16:08
*** EinstCrazy has joined #openstack-neutron16:08
*** EinstCrazy has quit IRC16:08
*** Marga_ has joined #openstack-neutron16:09
*** EinstCrazy has joined #openstack-neutron16:09
*** mlavalle has joined #openstack-neutron16:09
*** marios has quit IRC16:09
*** vivek-ebay has joined #openstack-neutron16:09
*** vivek-ebay has quit IRC16:09
*** vivek-ebay has joined #openstack-neutron16:10
*** saksham has joined #openstack-neutron16:10
*** marios has joined #openstack-neutron16:10
*** oomichi has joined #openstack-neutron16:10
*** safchain has quit IRC16:11
*** jbell8 has joined #openstack-neutron16:11
vthapartmorin: network_id is string or list of network_ids?16:11
tmorinvthapar: one network16:12
*** saksham_ has joined #openstack-neutron16:12
tmorinvthapar: multiple successive associations will result in multiple calls16:12
tmorinvthapar: get_vpn(...)['networks'] should give you all the associated networks16:13
sc68calSam-I-Am: hey dude16:13
*** vivek-ebay has quit IRC16:13
*** saksham_ has left #openstack-neutron16:13
*** saksham has quit IRC16:14
vthapartmorin: ok. I need to append it to existing networks list and then send to ODL.16:15
*** jpena is now known as jpena|away16:15
tmorinvthapar: no, at the time when associate_network is called, the list already includes the last associate network16:15
tmorinvthapar: I mean the get_vpn(...)['networks'] list16:16
Sam-I-Amsc68cal: 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
vthapartmorin: got it. so from ODL driver I just need to get_bgpvpn and PUT it to ODL.16:17
*** minwang2 has quit IRC16:17
tmorinvthapar: yes think so, unless you have a way to add one association via the ODL API16:18
*** sridhar_ram has joined #openstack-neutron16:18
vthapartmorin: nope. the way udpates work in ODL driver is we send the updated object in full.16:19
tmorintmorin: ok16:19
vthapartmorin: thanks. I think I got all that I needed for now.16:19
tmorinvthapar: ok, great :)16:20
*** julien_ZTE has quit IRC16:20
*** annegentle has quit IRC16:20
*** ijw has quit IRC16:20
sc68calSam-I-Am: that sounds like something we should fix16:21
sc68calSam-I-Am: I'm on my way to the airport, how about I ping you tomorrow and we do a hangout to discuss16:21
*** annegentle has joined #openstack-neutron16:21
Sam-I-Amsc68cal: 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-neutron16:21
Sam-I-Amor edit neutron.conf16:21
Sam-I-Amwe can chat tomorrow... i have a lot of meetings, but there might be time in the morning16:22
*** EinstCrazy has quit IRC16:22
openstackgerritGal Sagie proposed openstack/neutron-specs: Add Kuryr spec to Neutron  https://review.openstack.org/21349016:22
amullerarmax: build_name:"gate-neutron-dsvm-functional" AND build_status:"FAILURE" AND tags:"console" AND message:"Finished: FAILURE"16:22
amullerarmax: I think that should give you a unique list of functional test failures16:22
amullerthen looking at the last 48 hours, going to each console and seeing what tests failed16:23
*** sdake_ is now known as sdake16:23
amulleris there an easier test to prioritize test failures according to numbers of occurence?16:23
amullereasier way*16:23
amullerzuul rechecks tabs is not helping16:23
*** matrohon has quit IRC16:24
*** yamamoto has joined #openstack-neutron16:25
*** nikil89_ has quit IRC16:25
sc68calSam-I-Am: ok. yeah it would be nice to make config additive and straight forward16:25
Sam-I-Amsc68cal: 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-Amalso, if ubuntu didnt start services by default after install :/16:27
Sam-I-Ami can install the l3 agent on rhel and suse, just not start it16:27
Sam-I-Amon ubuntu, its like 'well, install this stuff, but if you want to use option 2, install this one other thing'16:28
Sam-I-Amuggghhh16:28
Sam-I-Ami'm more than certain the copy-paste club will miss that note16:28
Sam-I-Amthere's also issues with the ordering of administrative tasks such as database population16:29
russellbSam-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-Amrussellb: haha. neither can i.16:29
*** watanabe_isao has joined #openstack-neutron16:29
*** yamamoto has quit IRC16:29
Sam-I-Amsc68cal: if you look at the patch now, its using the incremental-but-confusing layout16:30
Sam-I-Amim rewriting it another way to see how it looks16:30
*** beagles is now known as beagles-brb16:30
sc68calwould you run neutron on your box? would you run neutron with a fox?16:32
anteayain a house?16:32
anteayawith a mouse?16:32
armaxamuller: me looks16:33
Sam-I-Amsc68cal: btw, if you enable allow_isolated_metadata in the dhcp agent, i think you can use the conventional metadata service with provider networks16:33
*** watanabe_isao has quit IRC16:33
armaxamuller: the process of elimination doesn’t change16:33
*** breitz has joined #openstack-neutron16:33
armaxamuller: it’s the issues that do that16:34
armaxamuller: I don’t see a consistent failure mode16:35
amullerarmax: no me neither16:35
armaxamuller: it’s like the functional job is on a rampage16:35
*** lazy_prince has quit IRC16:35
sc68calSam-I-Am: make sense, since my devstack cluster runs the metadata agent but uses provider nets16:35
amullerarmax: don't forget that it's legit failures in that graph as well16:35
armaxamuller: aye16:35
amullerarmax: anyway just asked a question in openstack-qa you might be interested in as well16:35
armaxamuller: don’t worry, this isn’t the first time I have done that16:35
*** nyechiel has quit IRC16:36
*** gsagie_ has joined #openstack-neutron16:36
armaxamuller: that’s a good one16:37
*** fawadkhaliq has joined #openstack-neutron16:37
*** l6unchpad has quit IRC16:37
*** SourabhP has joined #openstack-neutron16:38
*** ronis has joined #openstack-neutron16:38
*** pc_m_AWAY is now known as pc_m16:39
*** cbit has joined #openstack-neutron16:40
*** alex_klimov has quit IRC16:41
*** cbit has quit IRC16:42
*** dannywilson has joined #openstack-neutron16:42
*** shashank_hegde has joined #openstack-neutron16:42
*** SourabhP has quit IRC16:42
*** Jabadia has joined #openstack-neutron16:43
*** cbits has quit IRC16:43
JabadiaHi, question for the experts - what is the behavioural in case of many L2 ( non tcp/udp ) traffic ?16:44
*** fawadkhaliq has quit IRC16:44
JabadiaI see many drops on the TAP16:44
*** johnbelamaric has joined #openstack-neutron16:45
JabadiaI assume some table getting filled up. how can I debug it ? how can I improve it ? ( rate is ~50Kpps )16:45
*** bharathm has quit IRC16:46
*** bharathm has joined #openstack-neutron16:46
*** tfukushima has quit IRC16:47
*** dannywilson has quit IRC16:49
Sukhdevrkukura: are you here?16:49
rkukuraSukhdev: yes16:49
*** s3wong has joined #openstack-neutron16:50
openstackgerritSergey Belous proposed openstack/neutron: [WIP] Add functional test cases for dhcp-agent (OVS)  https://review.openstack.org/20603316:51
*** fitoduarte has joined #openstack-neutron16:54
*** lindycoder has quit IRC16:54
*** minwang2 has joined #openstack-neutron16:56
*** arajagopal has quit IRC16:57
*** arajagopal has joined #openstack-neutron16:57
*** _cjones_ has joined #openstack-neutron16:58
*** pumaranikar has quit IRC16:58
*** annegentle has quit IRC16:59
*** yamahata has quit IRC17:00
*** pradk has quit IRC17:00
*** annegentle has joined #openstack-neutron17:00
*** sbalukoff has quit IRC17:00
*** beagles-brb is now known as beagles17:01
*** SourabhP has joined #openstack-neutron17:01
*** Sukhdev has quit IRC17:01
*** arajagopal has quit IRC17:02
*** pradk has joined #openstack-neutron17:02
*** hitalia has joined #openstack-neutron17:02
*** l6unchpad has joined #openstack-neutron17:02
*** mhickey has quit IRC17:04
*** sridhar_ram has quit IRC17:05
*** yog_ has quit IRC17:06
*** achanda has joined #openstack-neutron17:09
armaxamuller: ping17:09
amullerarmax: pong17:09
* armax wonders if build_name:"gate-neutron-dsvm-functional" AND build_status:"FAILURE" AND tags:"console" AND message:"Failed: 1" is a better query17:10
armaxto weed out the geniune failures17:10
amullerthat can filter out edge cases, I think it's reasonable17:10
amullerI'm currently playing around with the public instance of subunit2sql17:11
armaxamuller: there’s been a surge around yesterday17:11
armaxeven by taking into account the single failures17:11
*** tmorin has quit IRC17:13
*** yamamoto has joined #openstack-neutron17:13
*** l6unchpad has quit IRC17:14
armaxamuller: the most offender is this guy:17:15
armaxmessage:"in test_killed_monitor_respawns" AND build_status:"FAILURE"17:15
*** anshul has joined #openstack-neutron17:15
armaxthe surge is caused by him17:15
armaxnasty little bugger17:16
amullerarmax: https://bugs.launchpad.net/neutron/+bug/149593717:16
openstackLaunchpad bug 1495937 in neutron "test_killed_monitor_respawns fails with MismatchError" [Undecided,New]17:16
armax28 hits starting yesterday17:16
amullerarmax: I think that was fixed by https://review.openstack.org/#/c/223543/17:16
armaxah ah17:16
armaxsomething is not right then17:17
*** fitoduarte has quit IRC17:17
armaxI wonder if that patch has had the opposite effect17:17
*** madhu_ak has joined #openstack-neutron17:18
armaxthe patch merged around 10PST17:18
armaxand that’s whe spike popped up17:18
*** shashank_hegde has quit IRC17:19
*** yog_ has joined #openstack-neutron17:19
*** julien_ZTE has joined #openstack-neutron17:20
*** SourabhP has quit IRC17:21
ihrachysarmax: had revert broke smth?17:21
armaxihrachys: not sure yet, but something isn’t right17:21
*** dannywilson has joined #openstack-neutron17:22
*** jckasper has quit IRC17:22
*** abregman has joined #openstack-neutron17:22
*** jbell8 has quit IRC17:22
*** shivrao has quit IRC17:23
* ihrachys needs to run. update me for tomorrow if my involvement is needed around the revert17:23
armaxihrachys: 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 MismatchError17:23
*** flaviof_ is now known as flaviof17:24
armaxihrachys, HenryG: we didn’t have enough data to confirm that patch 223543 indeed fixed the issue17:24
armaxihrachys, HenryG: we only ran the functional job once17:24
armaxin that patch17:24
*** fitoduarte has joined #openstack-neutron17:24
ihrachysarmax: you can be right. but the patch does not look correct to me in the first place.17:25
ihrachyssee comments at https://review.openstack.org/#/c/223543/1/neutron/agent/linux/async_process.py17:25
*** vthapar has quit IRC17:25
armaxihrachys: so you’re saying that the revert has still some grounds?17:25
ihrachysassuming I read the code correctly.17:26
ihrachysyou can check me17:26
armaxihrachys: I am not thinking of reverting the revert..there must be something else at play17:26
*** julien_ZTE has quit IRC17:26
ihrachysarmax: link for test_killed_monitor_respawns failure? I still have 10 mins for that. :)17:26
armaxihrachys: https://bugs.launchpad.net/neutron/+bug/149593717:27
openstackLaunchpad bug 1495937 in neutron "test_killed_monitor_respawns fails with MismatchError" [Critical,Confirmed]17:27
armaxgot the logstash query in tehre17:27
*** SumitNaiksatam has quit IRC17:27
ihrachysarmax: 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
armaxihrachys: aye17:28
armaxihrachys: I mean there’s certainly a correlation between the spike and the revert17:28
*** yamahata has joined #openstack-neutron17:28
armaxbut the revert might have simply initiated a different failure mode17:28
ihrachysarmax: 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-neutron17:29
armaxthe patch that was reverted had to be reverted anyway17:29
armaxit changed the rootwrap filters17:29
armaxthat at least ought be tagged with UpgradeImpact to be nice17:30
armaxbut meh17:30
openstackgerritRyan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver  https://review.openstack.org/20763517:30
*** sridhar_ram has joined #openstack-neutron17:30
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Make chown rootwrap filter ipsec.secrets file specific  https://review.openstack.org/22413317:35
*** tflynn has joined #openstack-neutron17:38
*** shashank_hegde has joined #openstack-neutron17:41
*** sridhar_ram has quit IRC17:42
*** lindycoder has joined #openstack-neutron17:43
*** sridhar_ram has joined #openstack-neutron17:43
*** arajagopal has joined #openstack-neutron17:43
*** yog_ has quit IRC17:43
*** obondarev has quit IRC17:43
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: wait for ovsdb-monitor to bootstrap before reading the stdout  https://review.openstack.org/22420917:44
*** arajagopal has joined #openstack-neutron17:44
ihrachysarmax: I would give it a shot ^^ needs recheck stats17:44
armaxihrachys: me looks17:44
armaxihrachys: I’ll keep an eye on it17:45
ihrachysthat sleep should also give a chance for the green thread that reads from stdout to trigger17:45
armaxihrachys: thanks for kicking ass so quickly17:45
ihrachys(I presume sleep means eventlet context switch)17:45
armaxihrachys: ya17:45
ihrachysok, now I really run away. my wife will kill me now. :P17:46
armaxihrachys: and no-one wants that17:46
*** l6unchpad has joined #openstack-neutron17:46
*** yog_ has joined #openstack-neutron17:46
*** ihrachys has quit IRC17:46
*** tfukushima has joined #openstack-neutron17:48
*** flaviof is now known as flaviof_17:49
*** pradk has quit IRC17:50
*** ivar-lazzaro has joined #openstack-neutron17:50
*** ushkalim has joined #openstack-neutron17:50
*** fawadkhaliq has joined #openstack-neutron17:50
*** ociuhandu has quit IRC17:52
*** charlesr has quit IRC17:52
*** crose has quit IRC17:52
*** imcsk8_ is now known as imcsk817:52
*** tfukushima has quit IRC17:52
*** crose has joined #openstack-neutron17:52
*** ociuhandu has joined #openstack-neutron17:53
*** ivar-lazzaro has quit IRC17:53
*** beagles has quit IRC17:53
*** ivar-lazzaro has joined #openstack-neutron17:54
*** sdake_ has joined #openstack-neutron17:54
*** ivar-laz_ has joined #openstack-neutron17:55
*** ociuhandu has quit IRC17:57
*** sdake has quit IRC17:58
*** ivar-lazzaro has quit IRC17:59
*** padkrish has joined #openstack-neutron17:59
*** MVenesio has quit IRC17:59
*** abhiraut has joined #openstack-neutron17:59
openstackgerritRitesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers.  https://review.openstack.org/18815717:59
riteshamuller: ^^^ kindly take a look18:00
*** pumaranikar has joined #openstack-neutron18:00
*** LukeHinds has quit IRC18:05
*** pradk has joined #openstack-neutron18:05
*** abhiraut has quit IRC18:05
armaxrussellb: ping18:07
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Provide Endpoint groups capability.  https://review.openstack.org/21269218:08
*** l6unchpad has quit IRC18:08
*** sbalukoff has joined #openstack-neutron18:09
*** nyechiel has joined #openstack-neutron18:10
*** ritesh has quit IRC18:10
*** Guest91493 has joined #openstack-neutron18:10
*** pumaranikar has quit IRC18:11
*** thurloat_isgone is now known as thurloat18:12
*** annegentle has quit IRC18:12
*** annegentle has joined #openstack-neutron18:12
*** padkrish has quit IRC18:14
*** ZZelle_ has joined #openstack-neutron18:14
*** openstackgerrit has quit IRC18:16
*** pece has quit IRC18:16
*** openstackgerrit has joined #openstack-neutron18:17
*** nyechiel has quit IRC18:17
*** jprovazn has quit IRC18:17
*** padkrish_ has joined #openstack-neutron18:18
*** tfukushima has joined #openstack-neutron18:18
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Provide Endpoint groups capability.  https://review.openstack.org/21269218:20
*** ritesh has joined #openstack-neutron18:21
*** Sukhdev_ has joined #openstack-neutron18:22
*** numans has joined #openstack-neutron18:22
*** Guest91493 has quit IRC18:22
*** Guest91493 has joined #openstack-neutron18:22
*** Guest91493 is now known as arosen18:22
*** amuller is now known as amuller_afk18:22
apuimedoirenab: gsagie: let's get banix's https://review.openstack.org/#/c/222960/ in18:22
*** changbl has quit IRC18:23
*** anilvenkata has joined #openstack-neutron18:23
*** anilvenkata has left #openstack-neutron18:23
*** tfukushima has quit IRC18:23
*** anilvenkata has joined #openstack-neutron18:23
*** mickeys has quit IRC18:23
*** achanda has quit IRC18:25
*** saksham has joined #openstack-neutron18:26
*** oomichi has quit IRC18:27
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Zanata  https://review.openstack.org/22424518:27
*** carl_baldwin has quit IRC18:27
*** david-lyle has quit IRC18:30
*** boris-42 has quit IRC18:30
*** padkrish_ has quit IRC18:32
*** padkrish has joined #openstack-neutron18:32
*** padkrish has quit IRC18:33
*** padkrish has joined #openstack-neutron18:34
*** sthillma has joined #openstack-neutron18:34
*** mlavalle has quit IRC18:34
openstackgerritPaul Michali proposed openstack/python-neutronclient: Enable VPN test cases  https://review.openstack.org/21458718:35
*** arajagopal has quit IRC18:35
*** arajagopal has joined #openstack-neutron18:36
*** ivar-laz_ has quit IRC18:38
*** jckasper has joined #openstack-neutron18:38
*** ivar-lazzaro has joined #openstack-neutron18:39
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Test code added to check the ARP update failures  https://review.openstack.org/22425018:39
*** arajagopal has quit IRC18:41
*** yfried_ has joined #openstack-neutron18:42
*** tkatarki has quit IRC18:44
*** aix has quit IRC18:46
*** dannywilson has quit IRC18:46
*** obondarev has joined #openstack-neutron18:47
*** anilvenkata has left #openstack-neutron18:47
*** yog_ has quit IRC18:48
*** padkrish has quit IRC18:48
*** matrohon has joined #openstack-neutron18:49
*** padkrish has joined #openstack-neutron18:49
*** tkatarki has joined #openstack-neutron18:50
*** jwy has joined #openstack-neutron18:50
openstackgerritGal Sagie proposed openstack/neutron-specs: Add Kuryr spec to Neutron  https://review.openstack.org/21349018:52
*** l6unchpad has joined #openstack-neutron18:52
*** padkrish has quit IRC18:53
*** jwy has quit IRC18:53
*** amuller_afk is now known as amuller18:53
openstackgerritPaul Michali proposed openstack/python-neutronclient: CRUD for VPN endpoint group API  https://review.openstack.org/21945518:54
*** nijaba_ has joined #openstack-neutron18:54
*** nijaba_ has quit IRC18:54
*** nijaba_ has joined #openstack-neutron18:54
*** nijaba has quit IRC18:55
*** jwy has joined #openstack-neutron18:55
*** korzen has joined #openstack-neutron18:56
korzenHi, does anybody know how to do data migration while online schema migration BP was implemented in Librerty?18:57
korzenAs i understand, in first move I have to do neutron-db-manage upgrade --contract18:58
korzenand this is only schema migration18:58
*** pradk has quit IRC18:58
korzenhow about data migration?18:58
*** arajagopal has joined #openstack-neutron18:58
*** arajagopal has joined #openstack-neutron18:59
*** pradk has joined #openstack-neutron19:00
*** bharathm has quit IRC19:00
*** padkrish has joined #openstack-neutron19:02
*** pumaranikar has joined #openstack-neutron19:03
*** fawadkhaliq has quit IRC19:04
*** nyechiel has joined #openstack-neutron19:08
*** wasmum- has quit IRC19:08
*** julien_ZTE has joined #openstack-neutron19:09
azbiswasHi 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
amullerazbiswas: check out /etc/neutron/ml2_conf.ini19:10
*** sudipto has quit IRC19:10
amullerazbiswas: either the  tunneling or vlan ranges must be set to a range of 100019:10
*** gsagie_ has quit IRC19:11
*** l6unchpad has quit IRC19:12
azbiswasamuller: thanks that's what I thought19:12
*** achanda has joined #openstack-neutron19:12
*** Leo_ has joined #openstack-neutron19:13
azbiswasamuller: So there no way to increase those limits?19:13
*** Leo_ has quit IRC19:13
amullerazbiswas: just change them and restart the neutron server19:13
*** julien_ZTE has quit IRC19:13
*** Leo_ has joined #openstack-neutron19:14
azbiswasamuller: thanks19:14
*** lori|away is now known as lori19:15
*** ihrachys has joined #openstack-neutron19:16
*** padkrish has quit IRC19:18
*** pumaranikar has quit IRC19:18
korzenanyone on db data migration?19:19
*** padkrish has joined #openstack-neutron19:19
*** sudipto has joined #openstack-neutron19:21
*** abregman has quit IRC19:22
*** tbachman has quit IRC19:22
*** padkrish has quit IRC19:23
*** moshele has joined #openstack-neutron19:24
*** aix has joined #openstack-neutron19:25
*** pumaranikar has joined #openstack-neutron19:25
*** pumaranikar has quit IRC19:26
*** padkrish_ has joined #openstack-neutron19:26
*** alex_klimov has joined #openstack-neutron19:27
*** jbell8 has joined #openstack-neutron19:30
*** jbell8 has quit IRC19:32
*** jbell8 has joined #openstack-neutron19:32
*** abregman has joined #openstack-neutron19:33
*** jbell8 has quit IRC19:35
*** tidwellr has left #openstack-neutron19:36
*** sudipto has quit IRC19:36
*** jbell8 has joined #openstack-neutron19:36
*** abregman has quit IRC19:37
*** wasmum has joined #openstack-neutron19:37
*** mlavalle has joined #openstack-neutron19:38
*** ihrachys has quit IRC19:38
*** ihrachys has joined #openstack-neutron19:39
*** achanda_ has joined #openstack-neutron19:40
*** achanda has quit IRC19:40
*** Leo_ has quit IRC19:42
*** Leom has joined #openstack-neutron19:42
*** nyechiel has quit IRC19:43
*** saksham has left #openstack-neutron19:43
*** nlahouti has quit IRC19:44
*** bharathm has joined #openstack-neutron19:44
*** blallau has joined #openstack-neutron19:45
*** padkrish_ has quit IRC19:47
*** jbell8 has quit IRC19:47
*** padkrish has joined #openstack-neutron19:48
*** sdake has joined #openstack-neutron19:49
*** bharathm has quit IRC19:49
*** tbachman has joined #openstack-neutron19:52
*** sdake_ has quit IRC19:52
*** dkehn has quit IRC19:53
*** jbell8 has joined #openstack-neutron19:54
*** padkrish has quit IRC19:55
*** oomichi has joined #openstack-neutron19:55
*** padkrish has joined #openstack-neutron19:55
*** jbell8_ has joined #openstack-neutron19:57
*** ihrachys has quit IRC19:57
*** jbell8 has quit IRC19:57
openstackgerritRitesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers.  https://review.openstack.org/18815719:58
*** shashank_hegde has quit IRC19:59
*** padkrish has quit IRC19:59
*** openstackstatus has quit IRC20:02
*** mestery has quit IRC20:02
*** shashank_hegde has joined #openstack-neutron20:03
*** jpena|away is now known as jpena20:03
*** mestery has joined #openstack-neutron20:03
*** openstackstatus has joined #openstack-neutron20:04
*** ChanServ sets mode: +v openstackstatus20:04
*** sambetts has quit IRC20:05
*** mickeys has joined #openstack-neutron20:05
*** carl_baldwin has joined #openstack-neutron20:05
*** dkehn has joined #openstack-neutron20:06
*** abhiraut has joined #openstack-neutron20:06
*** annemccormick has quit IRC20:06
*** bharathm has joined #openstack-neutron20:07
*** Aish has joined #openstack-neutron20:07
madhu_akpc_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 IRC20:08
*** stevemar has joined #openstack-neutron20:09
pc_mmadhu_ak: I don't know.20:09
*** sambetts has joined #openstack-neutron20:10
madhu_akpc_m because, api tests doesnt run though: https://review.openstack.org/#/c/211381/ so, I thought to check with you20:11
madhu_akanyways, no worries20:11
*** stevemar has quit IRC20:11
*** tkatarki has quit IRC20:13
*** flaviof_ has quit IRC20:13
*** Sukhdev_ has quit IRC20:13
pc_mmadhu_ak: RE your comments on 21138120:14
madhu_akamuller: 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_akpc_m yep20:14
pc_mmadhu_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_akpc_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_akpc_m not sure of your intentions though..20:17
*** alex_klimov has quit IRC20:17
*** beagles has joined #openstack-neutron20:18
*** tfukushima has joined #openstack-neutron20:19
openstackgerritMiguel Lavalle proposed openstack/neutron: External DNS driver reference implementation  https://review.openstack.org/21221320:19
pc_mmadhu_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 IRC20:20
madhu_akpc_m not all those nuetron files, its just a subset of tempest files requied for running vpn tests..20:21
*** tkatarki has joined #openstack-neutron20:21
openstackgerritMerged openstack/neutron: Enable servicing lbaasV2 vip by DVR  https://review.openstack.org/22213520:22
pc_mmadhu_ak: But we're talking about 20-30 files that are identical to what is in neutron right?20:22
*** gberginc has quit IRC20:22
madhu_akyou referencing the api test file?20:24
*** tfukushima has quit IRC20:24
*** Prem has quit IRC20:24
openstackgerritMerged openstack/neutron: Add IPv6 Address Resolution protection  https://review.openstack.org/20165020:25
*** matrohon has quit IRC20:25
openstackgerritMerged openstack/neutron: ovsdb: Fix a few docstring  https://review.openstack.org/22391720:25
openstackgerritMerged openstack/neutron: Use tempest-lib's token_client  https://review.openstack.org/22400620:25
openstackgerritAaron Rosen proposed openstack/neutron: ipam: Prevent none from being passed to delete  https://review.openstack.org/22428920:25
madhu_akpc_m they are. but I thought, vpn tests can handle independently without relying on files present in neutron.20:26
pc_mmadhu_ak: The ones in common and services, etc.20:26
*** jbell8_ has quit IRC20:26
madhu_akpc_m yes, those are common files (only the files specifically used in api tests)20:27
pc_mI mean we're pulling in 8771 lines of code for a 317 line test module.20:27
openstackgerritMerged openstack/neutron: Add QoS fullstack test  https://review.openstack.org/21568820:27
madhu_akpc_m yes, those are copied.. I guess l am following as per the tests used in lbaas20:27
madhu_akpc_m https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/tests/tempest/README.rst20:28
pc_mI guess I don't think that was a good decision either.20:29
madhu_akpc_m maybe we need to have discussion in the next vpn meeting?20:32
*** korzen has quit IRC20:32
openstackgerritMerged openstack/neutron: Fix establishing UDP connection  https://review.openstack.org/21242520:34
pc_myes. I responded in the review. Would like to see other cores' opinion here.20:34
madhu_akpc_m sure, thanks for your comment. sounds good20:35
*** moha_hunt has joined #openstack-neutron20:39
*** ajmiller has quit IRC20:39
*** ajmiller has joined #openstack-neutron20:40
*** mwagner_lap has quit IRC20:44
*** carora has joined #openstack-neutron20:47
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fixes SNAT port not found for internal port  https://review.openstack.org/22425020:51
openstackgerritEdgar Magana proposed openstack/neutron: In OVS agent segmentation_id can't be unicode None  https://review.openstack.org/22429720:51
Swamicarl_baldwin: ping20:52
carl_baldwinSwami: pong20:52
Swamicarl_baldwin: I found another bug which was caused by the refactor.20:52
Swamicarl_baldwin: can you take a look at 22425020:53
Swamicarl_baldwin: When we did a patch update last week we might have missed this use case of deleting a gateway.20:53
carl_baldwinLooking...20:54
Swamicarl_baldwin: thanks20:54
*** leitan has quit IRC20:54
*** leitan has joined #openstack-neutron20:54
amullerkevinbenton: Am I crazy or is port security not available via the CLI?20:55
*** julien_ZTE has joined #openstack-neutron20:55
amullerkevinbenton: oh it's --no-security-groups nm20:56
amullererr, no it isn;t20:57
*** alex_klimov has joined #openstack-neutron20:57
*** blallau has quit IRC20:58
kevinbentonamuller: --port-security-enabled=False20:58
*** crose has quit IRC20:58
amullerkevinbenton: why do we keep merging crap to our client without help text20:58
*** padkrish has joined #openstack-neutron20:59
*** julien_ZTE has quit IRC21:00
*** tiswanso has quit IRC21:00
caroraHi 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.conf21:00
*** dkehn has quit IRC21:00
*** Piet has joined #openstack-neutron21:00
kevinbentonamuller: because help is for noobs21:00
kevinbentonamuller: :)21:00
*** ijw_ has joined #openstack-neutron21:01
kevinbentonamuller: i'm not sure how that slipped through21:01
*** dkehn has joined #openstack-neutron21:02
*** padkrish has quit IRC21:03
*** regXboi has joined #openstack-neutron21:03
*** flaviof has joined #openstack-neutron21:03
*** tflynn_ has joined #openstack-neutron21:04
*** nmagnezi has joined #openstack-neutron21:06
*** nlahouti has joined #openstack-neutron21:07
*** padkrish has joined #openstack-neutron21:08
*** tflynn has quit IRC21:08
regXboicarl_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
regXboior am I looking at a code hack workaround?21:08
*** ushkalim has quit IRC21:08
kevinbentonregXboi: are you talking about the test skips?21:08
*** pc_m has quit IRC21:09
regXboikevinbenton: no I'm talking about Ls 80-81 of l3_rpc.py21:09
carl_baldwinregXboi: To what end?21:09
regXboicarl_baldwin: so that I can test that the else branch works and has linear behavior21:09
regXboicarl_baldwin: I've got an operator that is burning on this point and so I need a work around while the problem gets fixed21:10
regXboicarl_baldwin: I can hack the code, but that's just plain ugly21:10
kevinbentonregXboi: oh, you will need to hack code21:10
kevinbentonregXboi: you can't change what a plugin supports via config21:10
*** nmagnezi has quit IRC21:10
regXboikevinbenton: I was really afraid of that21:11
* regXboi wonders if the upcoming bug just went critical because of that21:11
kevinbentonregXboi: what version of code are you running?21:11
regXboithe operator is running kilo21:12
regXboibut, that specific code pattern is the same as Liberty21:12
*** jbell8 has joined #openstack-neutron21:12
*** ociuhandu has joined #openstack-neutron21:12
*** obondarev has quit IRC21:13
carl_baldwinregXboi: I had a start on a fix yesterday.  Let me see if I can finish it up before I have to leave.21:13
regXboicarl_baldwin: ack - I'm still going to need to test a workaround tomorrow morning as a fallback21:14
carl_baldwinok21:14
*** yamahata has quit IRC21:14
carl_baldwinregXboi: I think kevinbenton is right, you’ll have to change code.21:14
*** ijw_ has quit IRC21:14
*** jbell8 has quit IRC21:15
*** ijw_ has joined #openstack-neutron21:15
kevinbentonregXboi: https://review.openstack.org/#/c/174845/21:15
regXboicarl_baldwin, kevinbenton: ack - that is what I was afraid of looking this afternoon but wanted to confirm before I went the wrong way21:15
*** nmagnezi has joined #openstack-neutron21:15
regXboikevinbenton: that doesn't help21:16
regXboithe problem exists in tip of tree Liberty21:16
*** crose has joined #openstack-neutron21:16
*** thorst has quit IRC21:16
carl_baldwinregXboi: Do you have a link to the bug?21:16
regXboicarl_baldwin: haven't filed yet, I just got back21:16
kevinbentonregXboi: and you're certain it's the same issue?21:17
*** TrevorV has quit IRC21:17
regXboikevinbenton: so here's the issue21:17
kevinbentonregXboi: because i spent a bunch of time profiling the l3 rpc calls21:17
kevinbentonregXboi: and at the start of liberty merged the major issues21:17
carl_baldwinregXboi: ack21:17
regXboikevinbenton: here's the issue:21:17
regXboiwhen 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 node21:18
regXboiwhich means that as the number of routers on the node increases, the time to add another node increases21:18
regXboithe 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
regXboikevinbenton: if you can tell me a way to modify local.conf/*.ini to avoid that path, I'm all ears21:20
*** arajagopal has quit IRC21:20
*** mickeys has quit IRC21:21
*** arajagopal has joined #openstack-neutron21:21
*** mickeys has joined #openstack-neutron21:21
regXboikevinbenton: sc68cal has  the stack trace here: http://paste.openstack.org/show/463640/21:22
armaxZZelle: ping21:22
kevinbentonregXboi: i see. yeah, i don't think you can skip21:22
ZZelle_armax, hi21:22
kevinbentonregXboi: but a get_ports call shouldn't be that expensive. is this just ML2?21:22
armaxZZelle_: I noticed you filed bug 149658621:23
openstackbug 1496586 in neutron "Avoid the pattern sql select/delete" [Undecided,New] https://launchpad.net/bugs/1496586 - Assigned to Cedric Brandily (cbrandily)21:23
armaxkudos to you21:23
regXboikevinbenton: it scales linerally in the number of ports21:23
regXboiso when I want 100s or more of routers on a network node, the time for the router to come up exceeds the service budget21:23
*** Aish_ has joined #openstack-neutron21:23
regXboiand doing that for a non-dvr node is ... well ... silly21:23
regXboier non-dvr router21:24
regXboi(imho)21:24
armaxZZelle_: I have been thinking that we should document a number of Neutron developer common pitfalls somewhere in the devref21:24
*** Aish_ has quit IRC21:24
ZZelle_armax, it's based on your comment in a review :)21:24
armaxZZelle_: and this is a classic example I have seen throughtout the codebase21:24
armaxZZelle_: right21:24
kevinbentonregXboi: it may scale linearly, but it should be able to retrieve hundreds of ports in < 1 sec21:24
armaxI have been meaning to stop me saying the same thing over and over and over :)21:24
regXboikevinbenton: doesn't matter - I'm looking to remove any O(n) I can21:25
armaxZZelle_: I thought of creating a section in the devref where we document the most recurrent mistakes/reviewer comments21:25
*** Aish has quit IRC21:25
armaxso that we can say to contributors: go check that out and come back when you’ve done your homework21:25
*** arajagopal has quit IRC21: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-neutron21:26
armaxZZelle_: well this one in particular is an anti-pattern that goes beyond the way we use oslo.db21:26
armaxor sqlalchemy for that matter21:26
kevinbentonregXboi: it most certainly does. we don't want to waste time trying to optimize an operation that is 1 db call either way21:26
kevinbentonregXboi: something has gone really wrong with get_ports if it's taking that long21:26
regXboikevinbenton: the requirement is that when an instance comes up, the network port has to be available within single seconds21:27
armaxZZelle_: I was wondering if you were interested in drafting a document like that21:27
regXboitoday at scale, it takes anywhere from 45 - 60 seconds21:27
*** Sukhdev has joined #openstack-neutron21:27
kevinbentonregXboi: and it should be. if the bottleneck is the get_ports call then something is badly broken in there.21:28
regXboikevinbenton: I think we are talking past each other21:28
armaxI can have a first stab at it, but I think it’s only fair to start documenting these types of issues alongside the fixes21:28
ZZelle_armax, i am in!21:28
armaxto make sure we don’t make the same mistakes21:28
armaxZZelle_: cool21:28
regXboikevinbenton: 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
armaxZZelle_: I am sure that amuller will have plenty of best practices and anti-pattern for tests21:29
kevinbentonregXboi: 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 everywhere21:29
armaxand kevinbenton can provide input on how to do things in bulk21:29
amullerregXboi: the goal is not to make code scale one way or another with respect to N, it's to remove bottlenecks21:29
regXboikevinbenton: I don't think the problem is the get_ports call, I think it is the "all ports on the host" filter21:29
amullerregXboi: you've profiled the code and identified that get_ports call to be where most of the time is spent?21:30
kevinbentonregXboi: 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 for21:30
regXboiand I question why I need to run dvr code for a non-dvr router21:30
*** changbl has joined #openstack-neutron21:30
kevinbentonregXboi: certainly go forward with reducing what it asks for, but i'm saying it sounds like we have a much larger lurking issue21:30
regXboiamuller: in scheduling a router - yes21:30
regXboikevinbenton: ack - *that* I agree with21:30
amulleryeah that's troubling =D21:30
regXboikevinbenton, amuller: in fact, I'm planning on proposing a three phase patch set:21:31
amullerregXboi: in get_ports is there any time spent outside of the main sqlalchemy query that retrives the port?21:31
amullers21:31
armaxregXboi, amuller: sorry if I interject but is this discussion on how to improve performance of the dvr code path?21:31
regXboiamuller: my memory is no, but I have to go double check the data21:31
regXboiarmax: NO21:31
* armax scrolls back even further21:32
amullerregXboi: This is not the only place I've seen DVR code slow down non-DVR code BTW21:32
regXboiarmax: this is a legacy router scheduling getting held up when DVR is turned off because the code forces calls to DVR code21:32
*** padkrish has quit IRC21:32
regXboiamuller: ack21:32
regXboianyway - the patch sets21:32
regXboi(1) change non-entry/exit debug statements to info21:32
*** padkrish has joined #openstack-neutron21:33
regXboi(2) change default log level in neutron devstack to info level21:33
regXboi(3) put in entry and exit statements at the debug level *everywhere*21:33
kevinbentoninfo log statements need to be a single unit of work21:33
armaxregXboi: ic21:33
kevinbentonso that won't fly21:33
regXboiso that we don't kill infra, but can have full instrumentation at debug level if we need to profile things21:33
kevinbentonthere is some document describing what we can log at the info level21:34
kevinbentonsince it's operator facing21:34
kevinbentonthat applies to all of openstack21:34
regXboikevinbenton: I'd like a reference to that21:34
kevinbentonregXboi: let me see if i can find it21:34
regXboikevinbenton: because the alternative is pretty ugly - hacking a higher debug level21:34
regXboikevinbenton: I'd actually probably be in favor of that document21:34
regXboibecause I doubt we are conforming to it today21:35
regXboiwe == neutron21:35
amullerregXboi: 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 IRC21:35
regXboiamuller: I suspect I'll be there after I get this settled21:35
*** slaweq has quit IRC21:35
*** padkrish_ has joined #openstack-neutron21:35
regXboiamuller: unless somebody else on my team has already gotten there I don't know about it yet21:36
kevinbentonregXboi: https://wiki.openstack.org/wiki/LoggingStandards21:36
regXboier there *and* I21:36
regXboikevinbenton: acked and bookmarked21:36
*** padkrish has quit IRC21:36
regXboiand I definitely think neutron doesn't currently conform to this21:37
armaxamuller: that’s true, but there are some circumstances where running dvr code is inevitable21:37
kevinbentonregXboi: how sow?21:37
*** slaweq has joined #openstack-neutron21:37
armaxamuller: I am sure there’s plenty of room for improvement though21:37
kevinbentonregXboi: have you seen code logging at INFO level that violates this?21:37
regXboikevinbenton: no - I've seen code that *should* be logging at INFO and above level that isn't21:38
regXboithe lack of conformity is in the absence of code, not the presence of bad code21:38
regXboier is *from* the absence21:38
kevinbentonregXboi: its a restriction, not a requirement to log everything21:38
kevinbentonregXboi: because people will have different opinions on what is useful to an operator21:38
armaxregXboi: there was a lady @Huawei who wanted to sort out logging along the same lines you’re talking about21:39
kevinbentonregXboi: if a single create router request results in 15 INFO statements, we screwed up21:39
regXboikevinbenton: I postulate that if we had better logging at INFO, we'd have better luck with things like elastic recheck21:39
armaxregXboi: don’t recall her name exactly21:39
armaxregXboi: but I am pretty sure she left a digital trace somewhere21:39
kevinbentonregXboi: we log at DEBUG for elasticrecheck21:39
regXboiarmax: neither do I21:39
*** arajagopal has joined #openstack-neutron21:40
*** arajagopal has quit IRC21:40
armaxregXboi: let me see what I can find21:40
regXboikevinbenton: but that logging is (I again postulate) not as useful as it should be21:40
*** arajagopal has joined #openstack-neutron21:40
regXboibut given that, then I'm looking at a logging hack to allow instrumentation to be turned on21:40
regXboiwhich can be done, it's just ...well ... a hack21:41
regXboiarmax: ack21:41
kevinbentonregXboi: 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 on21:41
kevinbentonregXboi: i'm all for smarter debug statements21:41
regXboikevinbenton: that was implied in (1) - if we can't go to INFO, then we make what we have smarter21:41
kevinbentonregXboi: but i don't just want a debug statement for every function we call21:41
kevinbentonregXboi: i'm completely against going to INFO21:42
regXboikevinbenton: I get that and I can work with the INFO piece21:42
regXboikevinbenton: however, I actually *do* want the ability to log entry/exit in DEBUG, but only if something special is set in local.conf21:42
regXboiI don't want that to happen normally21:43
*** jpena is now known as jpena|zzz21:43
regXboilord knows, I don't want that to happen normally21:43
*** johnbelamaric has quit IRC21:43
regXboibut I should be able to turn it on in my local.conf21:43
*** tkatarki has quit IRC21:44
*** gangil has joined #openstack-neutron21:45
*** anshul has quit IRC21:45
*** achanda_ has quit IRC21:45
regXboibut 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 4521:46
regXboiso in the interest of domestic harmony, I need to sign off21:46
*** tongli_ has quit IRC21:46
*** krtaylor has quit IRC21:46
regXboikevinbenton: thanks for the logging pointer - I'll read through it tomorrow morning21:47
*** slaweq has quit IRC21:47
regXboikevinbenton, carl_baldwin: thanks for the confirmation on the workaround21:47
regXboiand I'll be back tomorrow21:48
*** rpothier has quit IRC21:48
*** regXboi has left #openstack-neutron21:48
*** abhiraut has quit IRC21:48
*** abhiraut has joined #openstack-neutron21:50
*** slaweq has joined #openstack-neutron21:52
*** beagles has quit IRC21:54
*** flaviof has quit IRC21:55
*** tflynn_ has quit IRC21:55
*** emagana has quit IRC21:55
*** changbl has quit IRC21:55
*** tflynn has joined #openstack-neutron21:55
*** david-lyle has joined #openstack-neutron21:56
*** emagana has joined #openstack-neutron21:56
*** emagana has quit IRC21:57
*** pradk has quit IRC21:59
*** achanda has joined #openstack-neutron21:59
*** ijw_ has quit IRC22:01
*** klindgren_ has joined #openstack-neutron22:02
openstackgerritCarl Baldwin proposed openstack/neutron: WIP Only get host data for floating ips on DVR routers  https://review.openstack.org/22432322:03
*** klindgren has quit IRC22:03
mlavallecarl_baldwin: ping22:04
*** l6unchpad has joined #openstack-neutron22:04
carl_baldwinmlavalle: pong22:04
mlavallecarl_baldwin: any special subjects for tomorrow's meeting?22:04
*** nmagnezi has quit IRC22:06
*** achanda has quit IRC22:07
*** carl_baldwin has quit IRC22:09
*** achanda has joined #openstack-neutron22:09
*** jbell8 has joined #openstack-neutron22:10
*** stevemar has joined #openstack-neutron22:10
*** thorst has joined #openstack-neutron22:11
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fixes SNAT port not found for internal port  https://review.openstack.org/22425022:11
*** stevemar has quit IRC22:13
*** arajagopal has quit IRC22:14
*** arajagopal has joined #openstack-neutron22:15
*** annegentle has quit IRC22:16
*** achanda_ has joined #openstack-neutron22:17
*** achanda has quit IRC22:17
*** arajagopal has quit IRC22:19
*** tfukushima has joined #openstack-neutron22:20
*** padkrish_ has quit IRC22:23
*** padkrish has joined #openstack-neutron22:24
*** moshele has quit IRC22:25
*** tfukushima has quit IRC22:25
*** julien_ZTE has joined #openstack-neutron22:25
*** padkrish has quit IRC22:28
*** Sukhdev has quit IRC22:28
*** david-lyle has quit IRC22:29
openstackgerritMathieu Gagné proposed openstack/neutron: Add neutron-linuxbridge-cleanup util  https://review.openstack.org/22150822:29
*** k-yasui has joined #openstack-neutron22:31
*** ronis has quit IRC22:32
*** krtaylor has joined #openstack-neutron22:32
openstackgerritRyan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver  https://review.openstack.org/20763522:32
openstackgerritRyan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce BgpDrAgent  https://review.openstack.org/20762522:32
openstackgerritRyan Tidwell proposed openstack/neutron: [WIP] Add BGP Speaker Service Plugin RPC Calls  https://review.openstack.org/21699422:33
*** Leom has quit IRC22:34
*** lindycoder has quit IRC22:35
*** mwagner_lap has joined #openstack-neutron22:37
*** abhiraut has quit IRC22:38
*** tflynn_ has joined #openstack-neutron22:39
*** abhiraut has joined #openstack-neutron22:40
*** padkrish has joined #openstack-neutron22:40
*** tbachman has quit IRC22:41
*** tflynn has quit IRC22:42
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally  https://review.openstack.org/21378122:42
*** tflynn_ has quit IRC22:43
*** shashank_hegde has quit IRC22:43
*** lori is now known as lori|away22:43
openstackgerritAishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity  https://review.openstack.org/21381922:44
openstackgerritRyan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce driver  https://review.openstack.org/20763522:44
openstackgerritRyan Tidwell proposed openstack/neutron: [WIP] Add BGP Speaker Service Plugin RPC Calls  https://review.openstack.org/21699422:45
*** thurloat is now known as thurloat_isgone22:46
*** gildub has joined #openstack-neutron22:47
*** alex_klimov has quit IRC22:48
*** tbachman has joined #openstack-neutron22:48
*** dims_ has quit IRC22:48
*** banix has quit IRC22:48
openstackgerritMerged openstack/neutron: No network devices on network attached qos policies  https://review.openstack.org/22016422:49
*** Sukhdev_ has joined #openstack-neutron22:51
kevinbentonZZelle_: ping22:51
*** shashank_hegde has joined #openstack-neutron22:51
openstackgerritAaron Rosen proposed openstack/neutron: ipam: Prevent none from being passed to delete  https://review.openstack.org/22428922:51
kevinbentonZZelle_, armax: this pattern looks icky https://bugs.launchpad.net/neutron/+bug/149658622:52
openstackLaunchpad bug 1496586 in neutron "Avoid the pattern sql select/delete" [Wishlist,Confirmed] - Assigned to Cedric Brandily (cbrandily)22:52
*** mlavalle has quit IRC22:52
kevinbentonZZelle_, armax: also. calling .delete() directly on a query prevents sqlalchemy from doing it's own cascading actions22:52
armaxkevinbenton: I think that the usage of the pattern needs to be evaluated on a case by case basis22:54
armaxkevinbenton: it cannot be applied blindly22:54
armaxthat said, how is it possible that it prevents the cascading?22:54
kevinbentonarmax: in-python cascades22:54
ZZelle_kevinbenton, armax, good point about in-python cascades22:55
*** thurloat_isgone is now known as thurloat22:55
*** k-yasui has quit IRC22:55
kevinbentonwhich would become relevant if we use a main neutron objects table that the ports and networks, etc are related to22:55
ZZelle_imo, it means we should avoid the select than delete pattern or document why we need it22:55
openstackgerritAaron Rosen proposed openstack/neutron: db_base: Make delete_port raise NotFound if port is not found  https://review.openstack.org/22433722:56
armaxkevinbenton: how can you overcome that?22:56
armaxkevinbenton: I am baffled that in order to support in-python cascade we issue a sub-optimal set of sql queries22:56
kevinbentonarmax: unbaffle yourself22:57
kevinbentonarmax: it's the price of an ORM with magic :)22:57
kevinbentonarmax: http://docs.sqlalchemy.org/en/rel_1_0/orm/query.html#sqlalchemy.orm.query.Query.delete22:58
kevinbentonarmax,ZZelle_: it sounds like we can use an after_bulk_delete event to cleanup if we needed22:58
armaxkevinbenton: makes sense23:00
openstackgerritRyan Tidwell proposed openstack/neutron: Add devref for BGP Speaker  https://review.openstack.org/19645223:01
kevinbentonarmax, ZZelle_: is the issue with just catching the norecord exception that the exception invalidates the session?23:01
armaxkevinbenton: which issue are you talking about in particular?23:03
kevinbentonarmax: instead of switching to this bulk delete pattern, why can we just catch object not found exceptions on the delete23:04
armaxthat’s icky too23:05
*** markvoelker has quit IRC23:05
*** l6unchpad has quit IRC23:06
armaxunless we can do that without issuing an extra sql statement23:06
*** ijw_ has joined #openstack-neutron23:07
kevinbentonarmax: oh, i see. this is just for cases where nothing is done with the selected object at all other than deleteing it23:07
armaxya23:07
armaxthat’s the case I am talking about23:07
kevinbentonarmax: ok. please proceed then and ignore me23:07
armaxif I am using the object fine23:07
armaxkevinbenton: no23:07
armaxkevinbenton: you raised a good point23:07
armaxkevinbenton: keep doing that23:07
kevinbentonarmax: this is the delete_port patch that aaron proposed23:07
kevinbentonarmax: that brought this up?23:08
armaxkevinbenton: aye23:08
kevinbentonarmax: right. we don't use sql triggers for anything except for the status update i think23:08
kevinbentonarmax: is that right?23:08
armaxkevinbenton: I am sure the situation is more variagated than that23:09
kevinbentonarmax: ?23:09
armaxkevinbenton: ok, can you rephrase? I might have misunderstood you23:10
kevinbentonarmax: i just mean we don't have any delete triggers depending on that23:10
armaxdepending on what?23:10
kevinbentonarmax: right?23:10
armaxyou mean ports?23:10
kevinbentonarmax: yes23:11
armaxI think we do23:11
armaxwe must do23:11
kevinbentonfor what?23:12
armaxI am gonna have to look I don’t remember on the top of my head23:13
*** tbachman has quit IRC23:14
*** l6unchpad has joined #openstack-neutron23:15
*** Sukhdev_ has quit IRC23:15
*** carora has quit IRC23:16
*** kbringard has quit IRC23:17
*** padkrish has quit IRC23:18
*** padkrish has joined #openstack-neutron23:19
*** julien_ZTE has quit IRC23:21
*** l6unchpad has quit IRC23:22
*** padkrish has quit IRC23:23
*** tbachman has joined #openstack-neutron23:25
*** sneti has joined #openstack-neutron23:27
*** oomichi has quit IRC23:30
*** arajagopal has joined #openstack-neutron23:31
*** dims_ has joined #openstack-neutron23:32
*** arajagopal has quit IRC23:35
*** arajagopal has joined #openstack-neutron23:36
*** baoli_ has quit IRC23:37
*** k-yasui has joined #openstack-neutron23:37
*** abhiraut has quit IRC23:38
*** yamamoto has quit IRC23:39
*** hitalia has quit IRC23:40
*** abhiraut has joined #openstack-neutron23:40
*** shivrao has quit IRC23:41
*** chlong has quit IRC23:41
openstackgerritCedric Brandily proposed openstack/neutron: Avoid the select and delete sql pattern  https://review.openstack.org/22435123:42
*** ZZelle_ has quit IRC23:43
*** k-yasui_ has joined #openstack-neutron23:47
*** hoangcx has joined #openstack-neutron23:47
*** salv-orlando has quit IRC23:48
*** Swami has quit IRC23:50
*** amuller has quit IRC23:50
*** yamahata has joined #openstack-neutron23:51
*** tomoe_ has joined #openstack-neutron23:52
*** Sukhdev has joined #openstack-neutron23:52
*** shwetaap1 has quit IRC23:55
*** hichihara has joined #openstack-neutron23:56
*** veena has joined #openstack-neutron23:57
*** padkrish has joined #openstack-neutron23:59
*** shivharis has joined #openstack-neutron23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!