*** dave-mccowan has quit IRC | 00:00 | |
*** shaohe_feng has joined #openstack-neutron | 00:01 | |
*** debedb has joined #openstack-neutron | 00:02 | |
HenryG | sure, understood | 00:03 |
---|---|---|
openstackgerrit | Merged openstack/neutron: DVR: fix router rescheduling on agent side https://review.openstack.org/211939 | 00:03 |
*** rushil has quit IRC | 00:03 | |
HenryG | we could submit a patch for rally to chew on | 00:04 |
abhiraut | we could reuse my dummy patch again to add a new dependency | 00:06 |
*** debedb has quit IRC | 00:06 | |
*** clev is now known as clev-away | 00:07 | |
*** mickeys has quit IRC | 00:09 | |
*** mickeys has joined #openstack-neutron | 00:09 | |
*** banix has quit IRC | 00:10 | |
*** srilatha has quit IRC | 00:10 | |
*** srilatha has joined #openstack-neutron | 00:11 | |
*** achanda has quit IRC | 00:11 | |
*** dave-mccowan has joined #openstack-neutron | 00:14 | |
*** padkrish has quit IRC | 00:14 | |
*** padkrish has joined #openstack-neutron | 00:15 | |
*** srilatha has quit IRC | 00:15 | |
openstackgerrit | Shraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent https://review.openstack.org/212836 | 00:16 |
*** Aish has joined #openstack-neutron | 00:16 | |
*** Aish has left #openstack-neutron | 00:16 | |
*** madhu_ak has quit IRC | 00:16 | |
*** moha_hunt has quit IRC | 00:16 | |
*** shaohe_feng has quit IRC | 00:17 | |
openstackgerrit | Shraddha Pandhe proposed openstack/neutron: Propose ISC-DHCPD Driver for Neutron DHCP Agent https://review.openstack.org/212836 | 00:17 |
*** shoutm has quit IRC | 00:18 | |
*** padkrish has quit IRC | 00:19 | |
*** armax has quit IRC | 00:20 | |
*** nlahouti has quit IRC | 00:20 | |
*** moha_hunt has joined #openstack-neutron | 00:20 | |
*** shoutm has joined #openstack-neutron | 00:21 | |
*** nlahouti has joined #openstack-neutron | 00:21 | |
openstackgerrit | Ritesh Anand proposed openstack/neutron: Added test cases for DVR L3 schedulers. https://review.openstack.org/188157 | 00:22 |
*** armax has joined #openstack-neutron | 00:22 | |
*** padkrish has joined #openstack-neutron | 00:22 | |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #openstack-neutron | 00:23 | |
*** minwang2 has quit IRC | 00:23 | |
*** armax has quit IRC | 00:24 | |
*** padkrish has quit IRC | 00:27 | |
*** dannywil_ has quit IRC | 00:28 | |
*** salv-orl_ has quit IRC | 00:29 | |
*** garyk1 has quit IRC | 00:30 | |
*** dims has joined #openstack-neutron | 00:30 | |
*** padkrish has joined #openstack-neutron | 00:30 | |
*** tiny-hands has joined #openstack-neutron | 00:31 | |
*** azbiswas has quit IRC | 00:32 | |
*** redbeard has joined #openstack-neutron | 00:32 | |
*** azbiswas has joined #openstack-neutron | 00:32 | |
*** padkrish has quit IRC | 00:34 | |
openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255] https://review.openstack.org/212220 | 00:35 |
*** azbiswas has quit IRC | 00:36 | |
*** dave-mccowan has quit IRC | 00:37 | |
*** padkrish has joined #openstack-neutron | 00:37 | |
*** thorst has joined #openstack-neutron | 00:38 | |
*** piyanai has quit IRC | 00:38 | |
*** hoangcx has joined #openstack-neutron | 00:42 | |
*** reedip has joined #openstack-neutron | 00:42 | |
*** aranjan has quit IRC | 00:43 | |
openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 00:48 |
*** tiswanso has joined #openstack-neutron | 00:48 | |
*** shivrao has quit IRC | 00:48 | |
*** achanda has joined #openstack-neutron | 00:48 | |
*** hoangcx has quit IRC | 00:49 | |
*** hoangcx has joined #openstack-neutron | 00:50 | |
*** reedip_ has joined #openstack-neutron | 00:50 | |
*** tiswanso has quit IRC | 00:50 | |
*** reedip has left #openstack-neutron | 00:51 | |
*** tiswanso has joined #openstack-neutron | 00:51 | |
*** shivrao has joined #openstack-neutron | 00:56 | |
*** zhhuabj has quit IRC | 00:57 | |
*** vishwanathj has quit IRC | 00:58 | |
*** yalie has joined #openstack-neutron | 00:58 | |
*** dave-mccowan has joined #openstack-neutron | 00:59 | |
*** shivrao_ has joined #openstack-neutron | 00:59 | |
*** Piet has quit IRC | 01:00 | |
*** liudong has joined #openstack-neutron | 01:00 | |
*** shivrao has quit IRC | 01:00 | |
*** shivrao_ is now known as shivrao | 01:00 | |
*** shaohe_feng has joined #openstack-neutron | 01:02 | |
*** yanxb_ has joined #openstack-neutron | 01:02 | |
*** yanxb__ has quit IRC | 01:04 | |
abhiraut | ping: HenryG .. if you could validate this one https://review.openstack.org/#/c/214413/ | 01:05 |
*** baoli has joined #openstack-neutron | 01:07 | |
hoangcx | sc68cal: ping at this channel | 01:07 |
HenryG | abhiraut: you have good copying skillz ;) | 01:07 |
*** zhhuabj has joined #openstack-neutron | 01:08 | |
openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 01:08 |
*** liangbo has joined #openstack-neutron | 01:08 | |
*** nlahouti has quit IRC | 01:08 | |
*** tfukushima has quit IRC | 01:08 | |
*** tfukushima has joined #openstack-neutron | 01:09 | |
openstackgerrit | yalei wang proposed openstack/neutron: Add missing tenant_id validation in RESOURCE_ATTRIBUTE_MAP https://review.openstack.org/214506 | 01:09 |
sc68cal | hoangcx: pong | 01:09 |
hoangcx | sc68cal: Logging API for SG and FW rules for reviewing here. | 01:10 |
*** thorst has quit IRC | 01:10 | |
abhiraut | aw well there wasnt much to do in it anyway :) | 01:10 |
hoangcx | sc68cal: RFE #link https://bugs.launchpad.net/neutron/+bug/1468366 | 01:10 |
openstack | Launchpad bug 1468366 in neutron "RFE - Logging API for security group and firewall rules" [Undecided,Confirmed] - Assigned to Yushiro FURUKAWA (y-furukawa-2) | 01:10 |
hoangcx | sc68cal: Specification #link https://review.openstack.org/#/c/203509 | 01:10 |
*** regXboi has quit IRC | 01:11 | |
hoangcx | sc68cal: Source codes #link https://review.openstack.org/#/c/204481/ | 01:11 |
hoangcx | and here #link https://review.openstack.org/#/c/204484/ | 01:11 |
abhiraut | HenryG: I shouldnt be screwing that up at least ;) thanks | 01:11 |
*** baoli has quit IRC | 01:12 | |
*** yamamoto has joined #openstack-neutron | 01:12 | |
*** nlahouti has joined #openstack-neutron | 01:15 | |
*** lazy_prince has joined #openstack-neutron | 01:15 | |
*** haigang has joined #openstack-neutron | 01:16 | |
sc68cal | hoangcx: understood - however I don't know if that'll make FFE | 01:16 |
*** yalie1 has joined #openstack-neutron | 01:17 | |
hoangcx | sc68cal: That information from weekly meeting in "Announcements / Reminders" section #link https://wiki.openstack.org/wiki/Network/Meetings | 01:18 |
*** yalie1 has quit IRC | 01:19 | |
*** yalie1 has joined #openstack-neutron | 01:19 | |
*** yalie has quit IRC | 01:19 | |
hoangcx | sc68cal: There are current targeted for L-3 #link https://launchpad.net/neutron/+milestone/liberty-3 | 01:19 |
hoangcx | sc68cal: would love to moving on Logging API to the Milestone because it can be reviewed and handled by FWaaS core team as we discussed in last FWaaS weekly meeting | 01:21 |
*** sdake_ has joined #openstack-neutron | 01:21 | |
*** agireud has joined #openstack-neutron | 01:22 | |
*** yalie1 has quit IRC | 01:23 | |
*** jwy has quit IRC | 01:24 | |
*** yalie has joined #openstack-neutron | 01:25 | |
*** sdake has quit IRC | 01:25 | |
*** dims has quit IRC | 01:25 | |
*** abhiraut has quit IRC | 01:27 | |
*** amuller has joined #openstack-neutron | 01:32 | |
*** sthillma has quit IRC | 01:33 | |
*** padkrish has quit IRC | 01:33 | |
*** s3wong has quit IRC | 01:35 | |
*** stanzgy has joined #openstack-neutron | 01:35 | |
*** shivrao has quit IRC | 01:36 | |
*** liangbo has quit IRC | 01:37 | |
*** killer_prince has joined #openstack-neutron | 01:38 | |
*** erkules_ has joined #openstack-neutron | 01:38 | |
*** yamamoto has quit IRC | 01:40 | |
*** gangil has quit IRC | 01:40 | |
*** erkules has quit IRC | 01:41 | |
*** lazy_prince has quit IRC | 01:42 | |
*** britthouser has joined #openstack-neutron | 01:45 | |
*** mickeys has quit IRC | 01:45 | |
*** lyanchih has joined #openstack-neutron | 01:46 | |
*** britthou_ has joined #openstack-neutron | 01:46 | |
*** britthouser has quit IRC | 01:50 | |
*** boris-42 has quit IRC | 01:50 | |
*** liangbo has joined #openstack-neutron | 01:50 | |
*** piyanai has joined #openstack-neutron | 01:51 | |
*** britthou_ has quit IRC | 01:52 | |
*** shashank_hegde has quit IRC | 01:52 | |
*** blogan_ has quit IRC | 01:54 | |
*** britthouser has joined #openstack-neutron | 01:54 | |
*** outworlder has joined #openstack-neutron | 01:54 | |
*** britthou_ has joined #openstack-neutron | 01:55 | |
*** jwy has joined #openstack-neutron | 01:55 | |
*** agireud has quit IRC | 01:58 | |
*** SourabhP has quit IRC | 01:59 | |
*** britthouser has quit IRC | 01:59 | |
*** ajmiller has joined #openstack-neutron | 01:59 | |
*** saksham has quit IRC | 02:00 | |
*** agireud has joined #openstack-neutron | 02:00 | |
*** neiljerram_bb has quit IRC | 02:01 | |
*** ajmiller_ has joined #openstack-neutron | 02:02 | |
*** _cjones_ has quit IRC | 02:02 | |
*** shwetaap has joined #openstack-neutron | 02:02 | |
*** killer_prince has quit IRC | 02:03 | |
*** zhhuabj has quit IRC | 02:03 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Add object status transition barrier mechanism https://review.openstack.org/181674 | 02:05 |
*** ajmiller has quit IRC | 02:05 | |
*** banix has joined #openstack-neutron | 02:06 | |
*** rynn has joined #openstack-neutron | 02:07 | |
*** baoli has joined #openstack-neutron | 02:08 | |
*** sridhar_ram1 has quit IRC | 02:10 | |
*** baoli has quit IRC | 02:12 | |
*** yalie1 has joined #openstack-neutron | 02:14 | |
*** zhhuabj has joined #openstack-neutron | 02:14 | |
*** baojg has joined #openstack-neutron | 02:16 | |
*** yalie has quit IRC | 02:16 | |
openstackgerrit | Merged openstack/neutron: Fix tenant access to qos policies https://review.openstack.org/214030 | 02:17 |
*** dims has joined #openstack-neutron | 02:21 | |
*** lyanchih has quit IRC | 02:21 | |
*** vivek-ebay has quit IRC | 02:22 | |
*** vivek-ebay has joined #openstack-neutron | 02:28 | |
*** vivek-ebay has quit IRC | 02:28 | |
*** tiny-hands has quit IRC | 02:29 | |
*** sthillma has joined #openstack-neutron | 02:29 | |
*** dims has quit IRC | 02:29 | |
*** dims has joined #openstack-neutron | 02:30 | |
*** shoutm_ has joined #openstack-neutron | 02:30 | |
*** shoutm has quit IRC | 02:31 | |
*** sthillma_ has joined #openstack-neutron | 02:33 | |
*** dims has quit IRC | 02:34 | |
*** tiny-hands has joined #openstack-neutron | 02:35 | |
*** sthillma has quit IRC | 02:35 | |
*** sthillma_ is now known as sthillma | 02:35 | |
*** agireud has quit IRC | 02:35 | |
*** yamamoto has joined #openstack-neutron | 02:40 | |
*** agireud has joined #openstack-neutron | 02:43 | |
*** salv-orlando has joined #openstack-neutron | 02:44 | |
*** markvoelker has quit IRC | 02:46 | |
*** ivar-laz_ has joined #openstack-neutron | 02:46 | |
*** sbalukoff has quit IRC | 02:46 | |
openstackgerrit | ShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly https://review.openstack.org/214930 | 02:47 |
*** iwamoto has joined #openstack-neutron | 02:48 | |
*** ivar-lazzaro has quit IRC | 02:49 | |
openstackgerrit | ShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly https://review.openstack.org/214930 | 02:49 |
*** ivar-laz_ has quit IRC | 02:50 | |
*** britthou_ has quit IRC | 02:51 | |
*** salv-orlando has quit IRC | 02:51 | |
*** hakimo has joined #openstack-neutron | 02:52 | |
*** yamahata has quit IRC | 02:53 | |
*** agireud has quit IRC | 02:53 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specify host in DVR create/update port https://review.openstack.org/214933 | 02:53 |
*** piyanai has quit IRC | 02:54 | |
*** hakimo_ has quit IRC | 02:54 | |
*** jwy has quit IRC | 02:55 | |
*** salv-orlando has joined #openstack-neutron | 02:55 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specify host in DVR create/update port https://review.openstack.org/214933 | 02:55 |
*** lyanchih has joined #openstack-neutron | 03:01 | |
*** salv-orlando has quit IRC | 03:02 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Do not query reservations table when counting resources https://review.openstack.org/214602 | 03:02 |
openstackgerrit | Henry Gessau proposed openstack/neutron: Fix _ensure_default_security_group logic https://review.openstack.org/203384 | 03:02 |
*** gongysh has joined #openstack-neutron | 03:04 | |
*** shoutm has joined #openstack-neutron | 03:06 | |
*** y_sawai has joined #openstack-neutron | 03:06 | |
*** gildub has quit IRC | 03:07 | |
*** shoutm_ has quit IRC | 03:07 | |
*** hoangcx has quit IRC | 03:08 | |
*** hoangcx has joined #openstack-neutron | 03:09 | |
*** shashank_hegde has joined #openstack-neutron | 03:12 | |
*** tidwellr has joined #openstack-neutron | 03:13 | |
*** jwy has joined #openstack-neutron | 03:14 | |
*** armax has joined #openstack-neutron | 03:15 | |
*** amuller has quit IRC | 03:16 | |
HenryG | kevinbenton: ping | 03:17 |
HenryG | kevinbenton: can you throw an eye on https://review.openstack.org/211963 ? | 03:17 |
*** y_sawai has quit IRC | 03:18 | |
*** y_sawai has joined #openstack-neutron | 03:19 | |
*** agireud has joined #openstack-neutron | 03:20 | |
*** shivrao has joined #openstack-neutron | 03:21 | |
kevinbenton | HenryG: looking | 03:23 |
*** baoli has joined #openstack-neutron | 03:23 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific host in DVR create/update port https://review.openstack.org/214933 | 03:24 |
Fdaisuke | armax: Hi, | 03:24 |
Fdaisuke | armax: I commented to https://review.openstack.org/#/c/207328/ | 03:25 |
Fdaisuke | Would you please have a look at this? | 03:26 |
*** saksham has joined #openstack-neutron | 03:28 | |
*** baoli has quit IRC | 03:28 | |
*** saksham_ has joined #openstack-neutron | 03:30 | |
*** piyanai has joined #openstack-neutron | 03:32 | |
*** vikram has joined #openstack-neutron | 03:33 | |
*** saksham has quit IRC | 03:33 | |
*** saksham_ is now known as saksham | 03:33 | |
*** y_sawai has quit IRC | 03:35 | |
*** rushil has joined #openstack-neutron | 03:36 | |
*** Kennan2 is now known as Kennan | 03:36 | |
*** jwy has left #openstack-neutron | 03:37 | |
*** vivek-ebay has joined #openstack-neutron | 03:37 | |
*** diga has joined #openstack-neutron | 03:38 | |
*** jianyong has quit IRC | 03:39 | |
*** saksham has left #openstack-neutron | 03:39 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add document for entry point in setup.cfg https://review.openstack.org/214945 | 03:40 |
*** tiny-hands has quit IRC | 03:42 | |
*** vivek-eb_ has joined #openstack-neutron | 03:43 | |
*** vivek-ebay has quit IRC | 03:43 | |
*** rushil_ has joined #openstack-neutron | 03:44 | |
*** rushil_ has quit IRC | 03:46 | |
*** markvoelker has joined #openstack-neutron | 03:46 | |
*** gildub has joined #openstack-neutron | 03:46 | |
*** baojg has quit IRC | 03:47 | |
*** rushil has quit IRC | 03:48 | |
*** y_sawai has joined #openstack-neutron | 03:48 | |
*** banix has quit IRC | 03:49 | |
*** lpetrut has joined #openstack-neutron | 03:51 | |
*** markvoelker has quit IRC | 03:51 | |
*** baojg has joined #openstack-neutron | 03:54 | |
*** changzhi has quit IRC | 03:54 | |
*** pal has joined #openstack-neutron | 03:55 | |
*** klindgren has joined #openstack-neutron | 03:55 | |
*** azbiswas has joined #openstack-neutron | 03:56 | |
*** y_sawai has quit IRC | 03:56 | |
*** piyanai has quit IRC | 03:58 | |
*** shoutm_ has joined #openstack-neutron | 04:00 | |
*** shoutm has quit IRC | 04:01 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add document for entry point in setup.cfg https://review.openstack.org/214945 | 04:02 |
*** mkolesni_ has joined #openstack-neutron | 04:03 | |
*** tiswanso has quit IRC | 04:05 | |
*** achanda has quit IRC | 04:05 | |
*** y_sawai has joined #openstack-neutron | 04:06 | |
openstackgerrit | Vadivel Poonathan proposed openstack/neutron: Adding Ale Omniswitch to sub_projects document https://review.openstack.org/214001 | 04:08 |
*** irenab has quit IRC | 04:09 | |
openstackgerrit | Vadivel Poonathan proposed openstack/neutron: Adding Ale Omniswitch to sub_projects document https://review.openstack.org/214001 | 04:10 |
*** gberginc has quit IRC | 04:10 | |
*** irenab has joined #openstack-neutron | 04:11 | |
openstackgerrit | Kenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS https://review.openstack.org/148416 | 04:16 |
*** minwang2 has joined #openstack-neutron | 04:17 | |
*** dave-mccowan has quit IRC | 04:22 | |
*** lyanchih has quit IRC | 04:22 | |
*** zhhuabj has quit IRC | 04:24 | |
*** baoli has joined #openstack-neutron | 04:24 | |
*** jianyong has joined #openstack-neutron | 04:24 | |
*** jianyong has quit IRC | 04:24 | |
*** zhhuabj has joined #openstack-neutron | 04:24 | |
*** diga has quit IRC | 04:25 | |
*** pal has quit IRC | 04:27 | |
*** minwang2 has quit IRC | 04:28 | |
*** baoli has quit IRC | 04:29 | |
*** lpetrut has quit IRC | 04:31 | |
*** fawadkhaliq has joined #openstack-neutron | 04:32 | |
*** salv-orlando has joined #openstack-neutron | 04:32 | |
*** agireud has quit IRC | 04:33 | |
*** shivrao_ has joined #openstack-neutron | 04:34 | |
*** shivrao has quit IRC | 04:34 | |
*** shivrao_ is now known as shivrao | 04:34 | |
*** agireud has joined #openstack-neutron | 04:35 | |
*** vivek-ebay has joined #openstack-neutron | 04:35 | |
*** salv-orlando has quit IRC | 04:35 | |
*** shaohe_feng has quit IRC | 04:37 | |
*** baojg has quit IRC | 04:37 | |
*** vivek-eb_ has quit IRC | 04:38 | |
*** shivrao_ has joined #openstack-neutron | 04:38 | |
*** baojg has joined #openstack-neutron | 04:38 | |
*** shivrao has quit IRC | 04:39 | |
*** shivrao_ is now known as shivrao | 04:39 | |
*** vikram has quit IRC | 04:41 | |
*** lyanchih has joined #openstack-neutron | 04:42 | |
openstackgerrit | Merged openstack/neutron: Final decomposition of ML2 Cisco UCSM driver https://review.openstack.org/213668 | 04:42 |
*** Sukhdev has joined #openstack-neutron | 04:43 | |
*** jianyong has joined #openstack-neutron | 04:45 | |
*** jianyong has quit IRC | 04:45 | |
*** salv-orlando has joined #openstack-neutron | 04:47 | |
*** Sukhdev has quit IRC | 04:47 | |
*** srilatha has joined #openstack-neutron | 04:48 | |
*** jianyong has joined #openstack-neutron | 04:49 | |
*** jianyong has quit IRC | 04:49 | |
*** shashank_hegde has quit IRC | 04:50 | |
*** azbiswas has quit IRC | 04:51 | |
*** salv-orlando has quit IRC | 04:51 | |
*** azbiswas has joined #openstack-neutron | 04:52 | |
*** vikram has joined #openstack-neutron | 04:54 | |
*** azbiswas has quit IRC | 04:56 | |
vikram | ajo: ping | 04:58 |
*** Sukhdev has joined #openstack-neutron | 04:58 | |
*** ajmiller_ has quit IRC | 05:02 | |
ajo | hi mr vikram , morning | 05:05 |
*** jianyong has joined #openstack-neutron | 05:05 | |
ajo | kevinbenton ping: https://review.openstack.org/#/c/182920/ | 05:05 |
ajo | yamamoto ^ akamyshnikova | 05:06 |
ajo | akamyshnikova, I guess you're reaching exhaustion trying to get that done, it's usually a PITA to merge big patches | 05:07 |
*** sdake has joined #openstack-neutron | 05:07 | |
ajo | there's always something to nitpick about, or that reviewers didn't see before | 05:07 |
*** achanda has joined #openstack-neutron | 05:07 | |
*** shwetaap has quit IRC | 05:08 | |
ajo | akamyshnikova, do you think it could take too long / too much to address the current comments? | 05:08 |
*** afazekas has joined #openstack-neutron | 05:08 | |
*** shwetaap has joined #openstack-neutron | 05:08 | |
ajo | I guess the complicated part is that testing br-tun needs manual testing | 05:08 |
*** shwetaap has quit IRC | 05:09 | |
*** y_sawai has quit IRC | 05:10 | |
*** sdake_ has quit IRC | 05:11 | |
vikram | ajo: Good Morning | 05:13 |
vikram | ajo: I have few more ideas about classifier | 05:13 |
ajo | vikram: and good afternoon for you :) | 05:13 |
ajo | hi, what did you think about? :) | 05:13 |
vikram | ajo: mentioned over the review links.. | 05:13 |
vikram | can you please check once | 05:13 |
vikram | :) | 05:13 |
ajo | ah, 1 sec vikram , I'm going over all my open reviews | 05:14 |
vikram | ajo: thanks.. | 05:14 |
vikram | hope you will like it | 05:14 |
vikram | :) | 05:14 |
*** haigang has quit IRC | 05:14 | |
ajo | vikram: which line? | 05:14 |
vikram | https://review.openstack.org/#/c/190463/ | 05:14 |
ajo | yeah, yeah, there I am | 05:15 |
vikram | 15 | 05:15 |
ajo | but I'm not locating your comment | 05:15 |
ajo | aahh | 05:15 |
ajo | up ther | 05:15 |
ajo | there | 05:15 |
ajo | vikram: I think that's quite firewall specific (permit/deny?) | 05:15 |
*** afazekas has quit IRC | 05:15 | |
ajo | vikram, just a first impression not very meditated | 05:16 |
vikram | ajo: I was thinking that just specify classifer won't convey any meaning.. | 05:16 |
ajo | vikram, if it's a common resources to many other services, it should not be a separate service | 05:16 |
vikram | it could be permit / deny | 05:16 |
ajo | otherwise the inter dependencies are going to be a mess | 05:16 |
vikram | i agree... | 05:16 |
vikram | I was thinking how about grouping classifier's and the application can just use the group | 05:17 |
yamamoto | ajo: do you think merging it as-is saves efforts? follow-ups need testing as well... | 05:17 |
ajo | vikram: that's interesting, what do you mean by grouping classifier? | 05:17 |
*** liangbo has quit IRC | 05:18 | |
ajo | yamamoto: it provides some emotional relief I guess, I changed my vote to let them do it | 05:18 |
vikram | i mean let's say i need 3 classifiers... C1, C2, C3 | 05:18 |
ajo | yamamoto: I understand your point | 05:18 |
vikram | then one API can group them and application can use group instead of C1, C2 and C3 | 05:18 |
*** emagana has joined #openstack-neutron | 05:18 | |
*** kiran-r has joined #openstack-neutron | 05:18 | |
ajo | yamamoto: btw, a clean history is more important if we were thinking of backports, but I don't believe it's backportable, (I didn't try to see if codebases are compatible, but 400Ls make me guess it won't be) | 05:19 |
vikram | ajo: this will give more flexibility and convenience to the user/application | 05:19 |
ajo | vikram, I see | 05:20 |
ajo | vikram, grouping can become complicated | 05:20 |
ajo | you could want to do | 05:20 |
ajo | (C1 and C2) or C3 | 05:20 |
ajo | ;) | 05:20 |
ajo | vikram, I'd leave that to a 2nd step :) | 05:20 |
yamamoto | ajo: i guess this patch made me insensitive of that kind of emotion... https://review.openstack.org/#/c/153946/ | 05:20 |
ajo | in a future RFE | 05:20 |
vikram | ajo: ok | 05:20 |
ajo | vikram: I believe it's a good idea, but may be we should do things in smaller steps :), and think grouping wisely :D | 05:21 |
vikram | ajo: we can atleast mention that as a potential work in the spec | 05:21 |
*** irenab has quit IRC | 05:21 | |
ajo | yamamoto: reading | 05:21 |
ajo | yamamoto: oouch | 05:21 |
iwamoto | hello | 05:21 |
ajo | 128 patchsets | 05:21 |
vikram | ajo: regarding the action.. I was thinking if the user want to deny a flow then how to do? then probably it has to define the action of it's own.. | 05:22 |
*** rynn has quit IRC | 05:22 | |
vikram | ajo: which could be also repetitive. | 05:22 |
ajo | yamamoto: big patches are a PITA | 05:22 |
*** azbiswas has joined #openstack-neutron | 05:22 | |
*** gsagie has joined #openstack-neutron | 05:23 | |
ajo | yamamoto: how's that going, is it ready? | 05:24 |
ajo | yamamoto: btw, I wanted to chat with you one day about some sort of ToR switch control for the openvswitch ml2... | 05:25 |
ajo | I still don't know how to model it, or do it, having ryu on board I'm sure it will help | 05:25 |
ajo | yamamoto: because, basically we will need it for NFV, since they're needing to grab "whole physical cards" for some specific apps | 05:26 |
ajo | where we loose the ability to tag via SR-IOV or ovs.. and it can be done via an OF compatible ToR switch | 05:26 |
*** Sukhdev has quit IRC | 05:26 | |
*** srilatha has quit IRC | 05:26 | |
*** srilatha has joined #openstack-neutron | 05:27 | |
ajo | I guess we may need an OF controller to handle the switches | 05:27 |
*** neophy has joined #openstack-neutron | 05:31 | |
yamamoto | ajo: the big patch of mine is almost always kept ready. (thank you iwamoto) | 05:31 |
ajo | yamamoto: I'm reading it | 05:31 |
*** srilatha has quit IRC | 05:31 | |
ajo | yamamoto: looking good | 05:31 |
ajo | yamamoto: I guess it intersects with akamyshnikova patch a bit... | 05:31 |
yamamoto | i suppose ovs-agent restart patch will go first and it requires major update though. | 05:32 |
yamamoto | sure. | 05:32 |
*** irenab has joined #openstack-neutron | 05:32 | |
ajo | yamamoto: any testing plans for this? I mean | 05:32 |
ajo | what are we going to do in the gate? :) | 05:32 |
ajo | may be we can make it available this cycle | 05:33 |
ajo | and enable by default starting next | 05:33 |
ajo | to make sure it's heavily tested on gate | 05:33 |
yamamoto | a plan is to have a separate job with it enabled | 05:33 |
ajo | yamamoto, I guess we need both tested, but | 05:33 |
ajo | once this is proven to work well | 05:34 |
ajo | I'd get rid of the ovs-vsctl version, | 05:34 |
ajo | or there's something we can't do through ryu? | 05:34 |
yamamoto | yes it's the plan | 05:34 |
iwamoto | ovs-vsctl will not go away with that patch | 05:34 |
ajo | I guess, the only issue with that is new features need to be supported in ryu first, but I know you'd be fast then ;) | 05:34 |
iwamoto | only ovs-ofctl | 05:34 |
ajo | sorry | 05:34 |
ajo | ovs-ofctl, I was meaning that | 05:34 |
ajo | :) | 05:34 |
*** mlavalle has quit IRC | 05:35 | |
ajo | we already have the native ovsdb driver... | 05:35 |
ajo | for ovs-vsctl | 05:35 |
ajo | may be we should plan that too... | 05:35 |
iwamoto | current status -> https://review.openstack.org/#/c/206329/ | 05:35 |
iwamoto | but, yes, I should add some jenkins jobs | 05:36 |
ajo | iwamoto: niiice, all green ;) | 05:36 |
yamamoto | ajo: look at "Plan:" https://github.com/openstack/neutron-specs/blob/master/specs/liberty/ovs-ofctl-to-python.rst#proposed-change | 05:36 |
ajo | good job iwamoto , yamamoto ! | 05:36 |
*** vivek-ebay has quit IRC | 05:37 | |
ajo | yamamoto: I'd say, instead of experimental, a separate job running all the time | 05:37 |
ajo | so we can make sure it's stable | 05:37 |
yamamoto | i agree | 05:37 |
*** azbiswas has quit IRC | 05:37 | |
*** Marga_ has joined #openstack-neutron | 05:38 | |
*** azbiswas has joined #openstack-neutron | 05:38 | |
*** amotoki has quit IRC | 05:39 | |
*** azbiswas_ has joined #openstack-neutron | 05:39 | |
*** dasp has quit IRC | 05:40 | |
*** jreeves has quit IRC | 05:41 | |
*** tflynn has joined #openstack-neutron | 05:41 | |
ajo | yamamoto, iwamoto , can I ask you, what do you think about the ToR switch control needs I was talking about before? | 05:41 |
*** jprovazn has joined #openstack-neutron | 05:41 | |
*** nate_gone has quit IRC | 05:42 | |
*** mfranc213 has quit IRC | 05:42 | |
ajo | (NFV grabbing a whole physical card requirement) | 05:42 |
yamamoto | ajo: i'm not familiar with such a hardware | 05:42 |
*** azbiswas has quit IRC | 05:42 | |
*** vichoward has quit IRC | 05:42 | |
ajo | yamamoto: NFV you mean? | 05:42 |
yamamoto | ToR switch | 05:42 |
*** aveiga has quit IRC | 05:42 | |
ajo | yamamoto: I mean top of rack switch | 05:42 |
*** neophy has quit IRC | 05:43 | |
ajo | any top of rack switch openflow capable | 05:43 |
yamamoto | is it an ordinary OF switch? | 05:43 |
ajo | I was meaning that, not a branch :) | 05:43 |
ajo | branch->brand | 05:43 |
*** tflynn_ has joined #openstack-neutron | 05:43 | |
*** shausy has joined #openstack-neutron | 05:44 | |
ajo | yamamoto, I guess, we'd need some agent/s to act as controllers of the switches | 05:44 |
yamamoto | probably. | 05:44 |
ajo | and modeling the network to handle cases where an ext-net is then connected to a switch port.. | 05:44 |
yamamoto | in your model leafs are handled by ovs-agent? | 05:44 |
ajo | yamamoto: I have no model yet | 05:45 |
*** tflynn has quit IRC | 05:45 | |
openstackgerrit | yalei wang proposed openstack/neutron: Add support for unaddressed port https://review.openstack.org/200809 | 05:45 |
ajo | yamamoto: just a lot of NFV crying out loud because they want to grab whole physical cards in some cases | 05:45 |
ajo | yamamoto: I guess I need to sit down, and think, | 05:46 |
iwamoto | it sounds a bit like vlan-aware-vms and the ongoing ironic+neutron tenant isolation effort | 05:46 |
*** dasp has joined #openstack-neutron | 05:46 | |
*** aveiga has joined #openstack-neutron | 05:46 | |
*** mfranc213 has joined #openstack-neutron | 05:46 | |
yamamoto | i thought NFV is something virtual. i'm not sure why it want to grab hardware. for performance reasons? | 05:46 |
*** agireud has quit IRC | 05:47 | |
ajo | yamamoto: yeah, I had the same question for them a few times | 05:47 |
*** yog_ has joined #openstack-neutron | 05:47 | |
*** nate_gone has joined #openstack-neutron | 05:47 | |
*** nate_gone is now known as njohnston | 05:47 | |
ajo | yamamoto: basically it seems that if they grab a single VF from the SR-IOV NIC PF (physical port)... | 05:47 |
*** zhhuabj has quit IRC | 05:48 | |
ajo | they get limited to only 1 queue of the hardware | 05:48 |
*** vichoward has joined #openstack-neutron | 05:48 | |
ajo | while if they grab the whole card they can use several queues at the same time | 05:48 |
ajo | also | 05:48 |
yamamoto | doesn't it depend on nic? | 05:48 |
ajo | the cards have some limitations when handling promiscous unicast | 05:48 |
*** jreeves has joined #openstack-neutron | 05:48 | |
ajo | yamamoto: totally, yes... | 05:49 |
ajo | but they seem to be standarizing VNFs for SR-IOV cards like intel or mellanox, or stuff like that | 05:49 |
ajo | the VNF has to have all the drivers... | 05:49 |
ajo | "drivers" | 05:49 |
*** bharath_ has joined #openstack-neutron | 05:49 | |
ajo | sometimes they go via DPDK to the card | 05:49 |
ajo | so as long DPDK supports the nic, they are happy | 05:50 |
*** SridharG has joined #openstack-neutron | 05:50 | |
yamamoto | i have to admit i can't distinguish NFV and VNF | 05:50 |
ajo | NFV is the general concept of virtualizing network functions | 05:50 |
ajo | and VNF is an specific nework function... | 05:50 |
ajo | like an IMS, DHCP server, DNS, SIP server, | 05:51 |
ajo | IDSs, ... | 05:51 |
ajo | they seem to have many more than I could imagine... | 05:52 |
*** vernhart is now known as vern | 05:52 | |
*** shoutm_ has quit IRC | 05:52 | |
ajo | yamamoto, iwamoto : I saw one telco which added a mechanism driver of their own to ml2 | 05:53 |
ajo | and in combination with floodlight, | 05:53 |
ajo | they were controlling the top of rack switches, to do vlan tagging /untagging | 05:53 |
yamamoto | is it opensource? | 05:53 |
*** vthapar has joined #openstack-neutron | 05:53 | |
ajo | yamamoto: I asked them to opensource it | 05:54 |
*** y_sawai has joined #openstack-neutron | 05:54 | |
*** anilvenkata has joined #openstack-neutron | 05:54 | |
yamamoto | i've heard cisco does security group on non-leaf switches | 05:54 |
ajo | yamamoto, what do you mean by non-leaf? | 05:54 |
*** numan has joined #openstack-neutron | 05:55 | |
ajo | leaf = top of rack? | 05:55 |
yamamoto | leaf == hypervisor switch | 05:55 |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line https://review.openstack.org/214961 | 05:55 |
ajo | ah, ok | 05:55 |
ajo | yamamoto: yes, that could be an advantage of controlling the top of rack | 05:55 |
*** trinaths has joined #openstack-neutron | 05:55 | |
ajo | we could offload some work if we wanted | 05:56 |
ajo | but, then the switch has to be powerful enough :) | 05:56 |
*** itzikb has joined #openstack-neutron | 05:56 | |
ajo | may be the current design distributes the work better, but loads the hypervisor | 05:56 |
*** aranjan has joined #openstack-neutron | 05:56 | |
*** pal has joined #openstack-neutron | 05:57 | |
ajo | and definitely, we should not force openstack users to have openflow switches, | 05:57 |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line https://review.openstack.org/214961 | 05:57 |
ajo | ping gongysh :) | 05:57 |
openstackgerrit | Kevin Benton proposed openstack/neutron: l2pop: check port mac in pre-commit to stop change https://review.openstack.org/214474 | 05:57 |
ajo | gongysh: I saw you took a qos bug, thanks, will you work on it soon? :) we need to clear it out before the end of the cycle | 05:58 |
openstackgerrit | Merged openstack/neutron-fwaas: Reduce contention between update and delete https://review.openstack.org/212811 | 05:58 |
ajo | merges++ | 05:58 |
*** kiran-r has quit IRC | 05:59 | |
*** armax has quit IRC | 06:00 | |
*** zhhuabj has joined #openstack-neutron | 06:00 | |
*** y_sawai has quit IRC | 06:00 | |
gsagie | Hello, i am trying to understand something, i see that in db_base_plugin_v2 , create_network can recieve 'id' from the caller (if not it generates one), but from the CLI when i try to create network with specific id i get this:"Attribute 'id' not allowed in POST" where is this blocked? and why? | 06:02 |
gsagie | is this just for internal networks being created in the DB layer? | 06:02 |
*** lukasa has joined #openstack-neutron | 06:03 | |
*** azbiswas_ has quit IRC | 06:04 | |
iwamoto | gsagie: neutron/api/v2/attributes.py RESOURCE_ATTRIBUTE_MAP ? | 06:04 |
*** azbiswas has joined #openstack-neutron | 06:05 | |
gsagie | iwamoto: thanks, does anyone know why creating network with specific 'id' is blocked from the API level but allowed in the DB layer? i am interested to know why its blocked :) | 06:05 |
*** salv-orlando has joined #openstack-neutron | 06:05 | |
*** outworlder has quit IRC | 06:05 | |
*** y_sawai has joined #openstack-neutron | 06:05 | |
*** shoutm has joined #openstack-neutron | 06:07 | |
irenab | ajo: with PF per VNF, how PF is supposed to be managed? | 06:08 |
ajo | irenab, they're working with the nova guys and the scheduler to handle that | 06:09 |
*** moha_hunt has quit IRC | 06:09 | |
fawadkhaliq | gsagie: don't know the reason why Neutron initial design took this approach but normally for POST, server decides the location of the resource and returns the address (uuid) and let the server manage consistent way of doing addresses. | 06:09 |
*** m0h4_hun7 has joined #openstack-neutron | 06:09 | |
ajo | irenab: they will need to allocate only PFs with no used VFs | 06:09 |
* iwamoto scared by a lot of NFV acronyms | 06:09 | |
*** azbiswas has quit IRC | 06:09 | |
*** amotoki has joined #openstack-neutron | 06:09 | |
ajo | iwamoto: yes, hard to get... I know :/ | 06:09 |
irenab | ajo: I understand this, but once you give PF for VNF, host does not have control over it | 06:10 |
ajo | iwamoto: in SR-IOV: PF = physical port of a card, VF= virtual network card (with pci mem map) that goes to a PF | 06:10 |
irenab | I think you will have SR-IOV disabled to support PF=> VNF | 06:10 |
gsagie | fawadkhaliq: yeah, but its already allowed in the DB layer, so you could call create_network with specific 'id' internally but you cant do it from the API, but just wondering i do want to propose allowing adding metadata for networks | 06:10 |
ajo | iwamoto: so a card can be composed of n PF's (so it has n physical ports) , and each PF has y VFs ( y virtual network cards a VM/instance can grab) | 06:10 |
fawadkhaliq | gsagie: but preferred way should if location of resource is unknown, let the server decides the URL | 06:11 |
ajo | irenab: that's the thing | 06:11 |
fawadkhaliq | s/decides/decide/ | 06:11 |
ajo | irenab: host does not have any control, so you could only control segmentation if you can control the top of rack switch | 06:11 |
ajo | irenab: you say the switch: tag this port as VLANX, ... or send this port to tunnel Y | 06:11 |
ajo | irenab: that's what they were doing in their mechanism driver, via floodlight, and floodlight talked to their switches | 06:12 |
yamamoto | gsagie: it's this way so that plugin can provide id. eg taken from its backend. | 06:12 |
ajo | irenab: but their thing is just a POC, they're looking for upstream support on that | 06:12 |
gsagie | yamamoto: yeah, i am wondering why not to allow it in the API level aswell | 06:12 |
ajo | irenab: I will try to get their code open so we can all read and think | 06:12 |
*** reedip_ has quit IRC | 06:13 | |
*** yanxb has joined #openstack-neutron | 06:15 | |
openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 06:15 |
gsagie | why not to allow an orchestration system that wants to pick its own network id's to be able to do it from the API level | 06:15 |
*** salv-orlando has quit IRC | 06:16 | |
irenab | ajo: management of segmentations via ToR makes sense in such case | 06:16 |
ajo | irenab: otherwise you make all the VNFs vendors to accept your "segmentation parameters" somehow | 06:17 |
yamamoto | gsagie: no idea. it would simplify kuryr a bit. | 06:17 |
ajo | I guess for VLANs that could be ok, but other types of segmentations could get trickier | 06:17 |
*** aranjan has quit IRC | 06:18 | |
gsagie | yamamoto: yes, Kuryr might need an additional metadata as well, but it can also help multi cloud solutions | 06:18 |
*** neophy has joined #openstack-neutron | 06:18 | |
irenab | ajo: agree | 06:19 |
gsagie | ajo: thats what hierarchical port binding is for i believe | 06:20 |
openstackgerrit | ShaoHe Feng proposed openstack/neutron: Loads service plugins fault-tolerantly https://review.openstack.org/214930 | 06:20 |
gsagie | you dont need to change the mechanism driver for OVS, just add the additional binding for the ToR | 06:20 |
*** njohnston has quit IRC | 06:20 | |
ajo | gsagie: yeah, we could leverage that, afaik hierarchical port binding is only supported by cisco :) | 06:21 |
ajo | gsagie: I need to dive into hierarchical port binding a bit btw... | 06:21 |
*** afazekas has joined #openstack-neutron | 06:21 | |
vikram | ajo: I registered a spec for that log back:) | 06:21 |
*** shashank_hegde has joined #openstack-neutron | 06:21 | |
ajo | but that was an impression I had too | 06:21 |
vikram | *long | 06:21 |
ajo | vikram++ | 06:21 |
ajo | :) | 06:21 |
gsagie | ajo: it doesnt matter, its the fact that you can do it in ML2, all you need is first bind OVS plugin then just communicate with the ToR the mapping of the segmentations | 06:21 |
*** vichoward has quit IRC | 06:21 | |
*** dasp has quit IRC | 06:21 | |
*** nyechiel_ has joined #openstack-neutron | 06:22 | |
vikram | ajo: involve me as well ++ | 06:22 |
*** baojg has quit IRC | 06:22 | |
gsagie | so you just need (as far as i understand) add another mechanism driver that can communicate with the ToR | 06:22 |
*** jreeves has quit IRC | 06:22 | |
*** mfranc213 has quit IRC | 06:22 | |
ajo | gsagie: I will re-read hierarchical port binding again, I need to refresh my knowledge | 06:22 |
openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 06:22 |
gsagie | ajo: np, let me know if you need any help | 06:22 |
*** aveiga has quit IRC | 06:23 | |
ajo | gsagie: I won't tackle it now, not enough bandwidth | 06:23 |
ajo | but may be something I will look in next cycle | 06:23 |
gsagie | ok, thats an area i am also looking at right now | 06:23 |
ajo | gsagie, for OVN, right? :) | 06:23 |
ajo | or dragonflow? | 06:23 |
*** pal has quit IRC | 06:23 | |
*** tidwellr has quit IRC | 06:23 | |
gsagie | because thats getting to be a very common deployment these days | 06:23 |
*** baojg has joined #openstack-neutron | 06:24 | |
gsagie | ajo: Dragonflow, OVN since its not a mechanism driver can support these kind of deployments using VTEP schema only, cant leverage hierarchical port binding | 06:24 |
* ajo watches: hierarchical port binding presentation: https://www.youtube.com/watch?v=Pk5bN5Fs_Vk | 06:24 | |
*** Murali has joined #openstack-neutron | 06:26 | |
openstackgerrit | venkata anil proposed openstack/neutron-vpnaas: Support VPNaaS with L3 HA https://review.openstack.org/200636 | 06:27 |
openstackgerrit | Liang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents https://review.openstack.org/214969 | 06:27 |
*** pal has joined #openstack-neutron | 06:27 | |
*** gangil has joined #openstack-neutron | 06:27 | |
*** dasp has joined #openstack-neutron | 06:28 | |
*** nate_gone has joined #openstack-neutron | 06:28 | |
*** aveiga has joined #openstack-neutron | 06:28 | |
*** nate_gone is now known as njohnston | 06:28 | |
*** vichoward has joined #openstack-neutron | 06:29 | |
*** jreeves has joined #openstack-neutron | 06:29 | |
*** vikram has quit IRC | 06:30 | |
*** mfranc213 has joined #openstack-neutron | 06:31 | |
ajo | gsagie: now that I see, it's a similar thing, but it's not the same | 06:31 |
ajo | they dynamically allocate vlans on the ToR switch to handle a tenant net traffic | 06:31 |
ajo | and then interconnect on the upper layer of switches | 06:32 |
ajo | here | 06:32 |
*** salv-orlando has joined #openstack-neutron | 06:32 | |
ajo | they just want to make sure a physical port is connected to a specific network, that's all | 06:32 |
*** shaohe_feng has joined #openstack-neutron | 06:32 | |
ajo | I'm probably still missing details of this hierarchical port binding thing | 06:33 |
* ajo keeps watching video | 06:33 | |
openstackgerrit | Mohankumar proposed openstack/python-neutronclient: Adding registration interface for non_admin_status_resources https://review.openstack.org/214193 | 06:34 |
*** vikram has joined #openstack-neutron | 06:35 | |
*** stanzgy has quit IRC | 06:35 | |
*** mohankumar has joined #openstack-neutron | 06:35 | |
*** jreeves has quit IRC | 06:35 | |
ajo | gsagie: It's very similar, I need to dig my head into it deeper... | 06:35 |
openstackgerrit | Tu Hong Jun proposed openstack/neutron: Changed filter field to router_id https://review.openstack.org/214974 | 06:36 |
*** aveiga has quit IRC | 06:36 | |
*** stanzgy has joined #openstack-neutron | 06:36 | |
*** dasp has quit IRC | 06:36 | |
*** haigang has joined #openstack-neutron | 06:37 | |
*** mfranc213 has quit IRC | 06:37 | |
*** vichoward has quit IRC | 06:37 | |
*** njohnston has quit IRC | 06:37 | |
*** vikram has quit IRC | 06:37 | |
*** afazekas has quit IRC | 06:37 | |
*** liangbo has joined #openstack-neutron | 06:38 | |
*** nlahouti has joined #openstack-neutron | 06:39 | |
*** nlahouti1 has joined #openstack-neutron | 06:41 | |
gsagie | ajo: what are you trying to do? what is the use case? from what i know these deployments usualy connect the host to the ToR with VLAN and do the encapsulation to VXLAN in the ToR (so you save encapsulation peformance) | 06:41 |
*** reedip has joined #openstack-neutron | 06:41 | |
*** jreeves has joined #openstack-neutron | 06:42 | |
*** aveiga has joined #openstack-neutron | 06:42 | |
*** mfranc213 has joined #openstack-neutron | 06:42 | |
*** dasp has joined #openstack-neutron | 06:42 | |
ajo | gsagie: VNF grabs a whole card (PF) | 06:42 |
*** nate_gone has joined #openstack-neutron | 06:43 | |
*** nate_gone is now known as njohnston | 06:43 | |
ajo | gsagie: we need to tag/tunnel untag/untunnel the specific port of the switch to still provide segmentation to that VNF interface | 06:43 |
*** vichoward has joined #openstack-neutron | 06:43 | |
*** vikram has joined #openstack-neutron | 06:43 | |
ajo | gsagie: we can only do it via the top of rack switch, or with VNF cooperation.. | 06:43 |
*** nlahouti has quit IRC | 06:44 | |
ajo | a hierarchy supporting hierarchical port binding, probably could do it easily, I guess, I need to look into that, | 06:44 |
*** ddepaoli has joined #openstack-neutron | 06:44 | |
ajo | but for other types of network arrangements, if you want to do this, you couldn't | 06:45 |
*** fawadkhaliq has quit IRC | 06:45 | |
*** y_sawai has quit IRC | 06:45 | |
*** nlahouti1 has quit IRC | 06:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/214470 | 06:45 |
gsagie | ajo : so you tackling the mixed setups with SR-IOV if i understand correctly | 06:46 |
gsagie | (which i am doing as well :)) | 06:46 |
*** y_sawai has joined #openstack-neutron | 06:47 | |
*** vikram has quit IRC | 06:48 | |
*** moshele has joined #openstack-neutron | 06:48 | |
ajo | gsagie: yep | 06:48 |
*** vikram has joined #openstack-neutron | 06:48 | |
ajo | SR-IOV -> grab a whole PF | 06:48 |
ajo | nova is working on it already | 06:48 |
ajo | on our side, is just adding another type of interface | 06:49 |
ajo | but then, the traffic would be unsegmented unless your plugin is able to do that | 06:49 |
ajo | :) | 06:49 |
*** nlahouti has joined #openstack-neutron | 06:49 | |
*** nlahouti has quit IRC | 06:49 | |
*** gangil has quit IRC | 06:50 | |
gsagie | ajo: why cant you use VLANS? | 06:50 |
ajo | gsagie: because the VM grabs the PF, so host loses control over setting a vlan to the PF | 06:51 |
ajo | gsagie: the VM grabs the whole PCI map for the card port... even if you set it before handing it down to the VM, it will be reset by the VM driver initialization... | 06:52 |
ajo | and also, only nova could do that, | 06:52 |
ajo | once the pci address space is mapped to the VM, we (neutron) cannot touch it anymore | 06:52 |
*** scheuran has joined #openstack-neutron | 06:52 | |
*** anshul has joined #openstack-neutron | 06:53 | |
*** jbell8 has quit IRC | 06:53 | |
*** vikram is now known as vikschw | 06:55 | |
*** vikschw is now known as vikram | 06:55 | |
*** jprovazn has quit IRC | 06:56 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Clear the extension requirement https://review.openstack.org/214981 | 06:56 |
*** y_sawai has quit IRC | 06:57 | |
*** vikram is now known as vikschw | 06:57 | |
*** vikschw is now known as vikram_choudhary | 06:58 | |
*** Fdaisuke has quit IRC | 06:58 | |
*** Fdaisuke has joined #openstack-neutron | 06:59 | |
*** vikram_choudhary is now known as vikschw | 06:59 | |
*** vikschw has quit IRC | 07:00 | |
*** achanda has quit IRC | 07:00 | |
*** ronis_ has joined #openstack-neutron | 07:01 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line https://review.openstack.org/214961 | 07:01 |
*** outworlder has joined #openstack-neutron | 07:01 | |
*** klindgren has quit IRC | 07:03 | |
*** ronis__ has joined #openstack-neutron | 07:03 | |
*** ronis_ has quit IRC | 07:06 | |
*** y_sawai has joined #openstack-neutron | 07:07 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Clear the extension requirement https://review.openstack.org/214981 | 07:07 |
*** vikram_choudhary has joined #openstack-neutron | 07:07 | |
*** m0h4_hun7 has quit IRC | 07:08 | |
*** fawadkhaliq has joined #openstack-neutron | 07:09 | |
*** vikram_choudhary is now known as vikram | 07:10 | |
*** vikram has left #openstack-neutron | 07:10 | |
*** outworlder has quit IRC | 07:10 | |
*** vikram_choudhary has joined #openstack-neutron | 07:15 | |
*** tflynn_ has quit IRC | 07:17 | |
*** vikram_choudhary has quit IRC | 07:18 | |
*** claudiub has joined #openstack-neutron | 07:19 | |
*** kawa2014 has joined #openstack-neutron | 07:19 | |
*** yamamoto has quit IRC | 07:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/204422 | 07:20 |
*** vikram_choudhary has joined #openstack-neutron | 07:22 | |
*** Murali_ has joined #openstack-neutron | 07:23 | |
*** vikram_choudhary is now known as Vikram | 07:23 | |
*** Vikram is now known as vikram | 07:23 | |
*** jschwarz has joined #openstack-neutron | 07:23 | |
openstackgerrit | Liang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents https://review.openstack.org/214969 | 07:25 |
*** Murali has quit IRC | 07:25 | |
*** Murali_ is now known as Murali | 07:25 | |
*** SourabhP has joined #openstack-neutron | 07:26 | |
*** gardenshed has joined #openstack-neutron | 07:28 | |
*** SourabhP_ has joined #openstack-neutron | 07:29 | |
*** Murali has quit IRC | 07:30 | |
*** Murali_ has joined #openstack-neutron | 07:30 | |
*** Murali_ is now known as Murali | 07:30 | |
*** neophy has quit IRC | 07:31 | |
*** SourabhP has quit IRC | 07:32 | |
*** SourabhP_ is now known as SourabhP | 07:32 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Fix _ensure_default_security_group logic https://review.openstack.org/203384 | 07:32 |
*** pal has quit IRC | 07:32 | |
openstackgerrit | Moshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy https://review.openstack.org/214212 | 07:33 |
jschwarz | ajo, will rebase asap :) | 07:33 |
ajo | jschwarz: I'm re-reviewing to make sure I don't come with more nitpicking later | 07:34 |
*** jlibosva has joined #openstack-neutron | 07:34 | |
jschwarz | ajo, ack. so I'll wait until your go-ahead so I don't do double the job :) | 07:34 |
ajo | ack jschwarz :) | 07:35 |
*** nyechiel_ has quit IRC | 07:36 | |
*** nyechiel_ has joined #openstack-neutron | 07:36 | |
*** tfukushima has quit IRC | 07:38 | |
ajo | jschwarz: 1st patch looks good now | 07:40 |
ajo | re-reading 2nd | 07:40 |
*** salv-orlando has quit IRC | 07:40 | |
*** yamamoto has joined #openstack-neutron | 07:40 | |
*** mohankumar has quit IRC | 07:41 | |
*** sthillma has quit IRC | 07:42 | |
*** ushkalim has joined #openstack-neutron | 07:43 | |
jschwarz | ajo, ack | 07:43 |
jschwarz | btw it would seem that the gate is clogged again | 07:43 |
*** SourabhP has quit IRC | 07:44 | |
jschwarz | ajo, rebased locally, waiting for your go-ahead for submission | 07:46 |
*** jpena has joined #openstack-neutron | 07:48 | |
ajo | jschwarz: ok, reviewed | 07:48 |
ajo | jschwarz, go go :) | 07:48 |
jschwarz | ajo, done | 07:48 |
openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2). https://review.openstack.org/198277 | 07:48 |
openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2). https://review.openstack.org/189655 | 07:48 |
ajo | thanks jschwarz | 07:48 |
jschwarz | you're most welcome ajo | 07:48 |
*** markvoelker has joined #openstack-neutron | 07:48 | |
*** fedexo has quit IRC | 07:48 | |
openstackgerrit | Liang Bo proposed openstack/neutron: Fixed broken link in neutron-server's documents https://review.openstack.org/214969 | 07:49 |
Fdaisuke | exit | 07:52 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 07:52 |
*** Fdaisuke has quit IRC | 07:52 | |
*** markvoelker has quit IRC | 07:53 | |
*** salv-orlando has joined #openstack-neutron | 07:55 | |
ajo | jschwarz: https://review.openstack.org/#/c/189655/30/neutronclient/v2_0/client.py | 07:56 |
ajo | can you check a comment | 07:56 |
ajo | it's a nitpick btw, may be I'm wrong, probably needed for dict extraction? | 07:56 |
*** gberginc has joined #openstack-neutron | 07:57 | |
jschwarz | ajo, aye we do | 07:57 |
jschwarz | since the API returns the resource as 'policies' and not 'qos_policies' | 07:57 |
jlibosva | ajo: o/ | 07:58 |
ajo | yep, jschwarz, you're right | 07:58 |
ajo | jschwarz++ | 07:58 |
*** ihrachyshka has joined #openstack-neutron | 07:58 | |
jschwarz | XD | 07:58 |
ihrachyshka | ajo, jlibosva, remember that glitch in alembic when our functional test failed to detect difference in foreign key constraint in terms of cascade delete? well it's fixed in 0.8.1: http://alembic.readthedocs.org/en/latest/changelog.html#change-0fbe5d57da5d0fc69adb44615b46a83f | 07:59 |
*** y_sawai has quit IRC | 07:59 | |
ihrachyshka | zzzeek is freaking quick | 07:59 |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Clear the extension requirement https://review.openstack.org/214981 | 07:59 |
ihrachyshka | I suspect we may be broken now in gate?.. | 07:59 |
ihrachyshka | let me check :) | 07:59 |
jlibosva | great! | 07:59 |
jlibosva | yeah, I wanted to ask :D | 07:59 |
ajo | ihrachyshka: I was just suspecting that | 07:59 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart https://review.openstack.org/182920 | 08:00 |
ihrachyshka | ah well, it's not released. good. | 08:00 |
ajo | probably lots of bad FKs around models/migrations | 08:00 |
*** yamahata has joined #openstack-neutron | 08:00 | |
ajo | ihrachyshka, what's the plan, checking locally by getting it ? | 08:00 |
ihrachyshka | I guess it would be great to get the version from git and check it | 08:00 |
jlibosva | I'll do that | 08:00 |
*** tfukushima has joined #openstack-neutron | 08:00 | |
ihrachyshka | jlibosva, tnx! | 08:00 |
ajo | jlibosva++ | 08:00 |
ajo | one day I'm going to spend 1h setting up a karma bot here | 08:00 |
ajo | if freenode allows that | 08:01 |
jschwarz | ihrachyshka, can you check the comment I made here: https://review.openstack.org/#/c/210830/5/neutron/tests/fullstack/resources/environment.py | 08:01 |
jschwarz | and tell me if it makes sense? :) | 08:01 |
jschwarz | ajo, why wouldn't it allow a karma bot? | 08:02 |
jschwarz | they already have bots such as openstackgerrit ... | 08:02 |
*** diga has joined #openstack-neutron | 08:02 | |
ihrachyshka | jschwarz, agreed and I wanted to make similar point yesterday when I saw smth like l2_agent=False in test_connectivity | 08:02 |
ihrachyshka | le* | 08:02 |
ihrachyshka | l3* | 08:02 |
jschwarz | ihrachyshka, :) | 08:02 |
jschwarz | I'll add it to the patch then | 08:03 |
ihrachyshka | ajo, it should be infra wide bot ;) | 08:03 |
ihrachyshka | jschwarz, cool | 08:03 |
*** shivrao has quit IRC | 08:04 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Add EnvironmentDescription, pass it down https://review.openstack.org/210830 | 08:05 |
jschwarz | ihrachyshka, so ^ is the next patch up the fullstack chain... hopefully you'll find the time to review it and +2 it ;-) | 08:05 |
ajo | ihrachyshka: let's test something locally, then let's suggest it to infra | 08:05 |
ajo | ihrachyshka: so they can see "the fun of it" ;) | 08:06 |
ajo | akamyshnikova: https://review.openstack.org/#/c/182920/56/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py <-- question | 08:06 |
*** lori|away is now known as lori | 08:06 | |
*** outworlder has joined #openstack-neutron | 08:06 | |
*** iwamoto has quit IRC | 08:06 | |
ajo | what compatibility reasons? | 08:06 |
ajo | may be some cooperating ovs-daemons install canary rules to check when we restarted?, but those would be gone by not having the latest cookie | 08:07 |
ihrachyshka | ajo, makes sense, just note that it should probably be based on the bot code they already have | 08:10 |
ajo | ihrachyshka, what's the bot they use? | 08:10 |
*** sridhar_ram has joined #openstack-neutron | 08:11 | |
ajo | I think they have several | 08:12 |
ajo | meetbot, openstackstatus... | 08:12 |
ajo | -openstackstatus CTCP- VERSION Python irc.bot (8.5.4) | 08:13 |
*** yalie1 has quit IRC | 08:13 | |
*** DWfuturetec has joined #openstack-neutron | 08:15 | |
*** outworlder has quit IRC | 08:15 | |
jschwarz | ajo can write a bot with python code on his free time ;) | 08:15 |
*** sridhar_ram has quit IRC | 08:15 | |
*** devvesa has joined #openstack-neutron | 08:16 | |
ajo | jschwarz: I'd rather start by using an existing one :) | 08:16 |
ajo | jschwarz: I had time when I was young to write my own IRC bot in C.... | 08:16 |
jschwarz | pss | 08:17 |
ajo | If i was able to read such code now, I would certainly puke :D | 08:17 |
jschwarz | that ought to have taken time | 08:17 |
*** yalie has joined #openstack-neutron | 08:17 | |
jschwarz | I once wrote a tetris in C++ and OpenGL | 08:17 |
akamyshnikova | ajo, see enikanorov respond in https://review.openstack.org/#/c/182920/56/neutron/plugins/ml2/drivers/openvswitch/agent/common/config.py | 08:17 |
ajo | jschwarz: lots of learnings :) | 08:17 |
*** afazekas has joined #openstack-neutron | 08:18 | |
ajo | akamyshnikova: thanks :) | 08:18 |
ajo | akamyshnikova, I guess it makes sense, I just trust the approach quite much I didn't thought that was necessary | 08:18 |
ajo | I'm fine with it | 08:19 |
mattt | is there a straightforward way of knowing, after upgrading neutron, if there are pending db migrations to apply? | 08:19 |
ajo | ihrachyshka ^ | 08:19 |
jschwarz | moshele, reviewed https://review.openstack.org/#/c/214212/ - sorry for the -1 :( | 08:20 |
moshele | jschwarz: you are a human pep8 :) | 08:21 |
ihrachyshka | mattt, compare neutron-db-manage current output with neutron-db-manage branches | 08:21 |
jschwarz | moshele, like you wouldn't believe ;-) | 08:21 |
moshele | jschwarz: ok fixing | 08:21 |
jschwarz | ty | 08:22 |
*** lyanchih has quit IRC | 08:22 | |
mattt | ihrachyshka: not seeing anything for 'branches', https://gist.github.com/mattt416/89abd52677381ff4e93d | 08:23 |
*** lyanchih has joined #openstack-neutron | 08:23 | |
*** agireud has joined #openstack-neutron | 08:23 | |
*** fawadkhaliq has quit IRC | 08:23 | |
*** fawadkhaliq has joined #openstack-neutron | 08:26 | |
*** gildub has quit IRC | 08:26 | |
openstackgerrit | Moshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy https://review.openstack.org/214212 | 08:27 |
*** yalie has quit IRC | 08:28 | |
*** jistr has joined #openstack-neutron | 08:28 | |
mattt | ihrachyshka: i'd love to be able to determine if i have outstanding expand/contract migrations, so i can either press ahead and do expand migrations w/ neutron running and then shut down neutron and do contract ones | 08:29 |
ihrachyshka | mattt, I am looking into why it may not work | 08:29 |
*** shoutm has quit IRC | 08:29 | |
mattt | ihrachyshka: thank you, let me know if i can provide you any information, i'm using neutron from commit da1ea9a45595c8170e908fc5533a0dace997ed33 | 08:30 |
*** l6unchpad has joined #openstack-neutron | 08:30 | |
*** l6unchpad has left #openstack-neutron | 08:31 | |
*** l6unchpad has joined #openstack-neutron | 08:31 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos https://review.openstack.org/213194 | 08:31 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py https://review.openstack.org/215012 | 08:31 |
ihrachyshka | mattt, I think I am either wrong about what branches is expected to do, or there is a bug | 08:32 |
ihrachyshka | mattt, for the time being, you can compare 'current' output with neutron/db/migration/alembic_migrations/versions/HEADS contents (but yeah, it sucks. it also seams that 'current' does not output 'kilo' head) | 08:32 |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted https://review.openstack.org/214215 | 08:32 |
*** agireud has quit IRC | 08:33 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted https://review.openstack.org/214215 | 08:33 |
ihrachyshka | mattt, ok, fo the issue of no kilo head in 'current' output, we have a patch: https://review.openstack.org/#/c/204551/ | 08:34 |
*** yalie has joined #openstack-neutron | 08:34 | |
ihrachyshka | mattt, it failed in gate, I pushed it now. | 08:35 |
*** sayali has quit IRC | 08:35 | |
ihrachyshka | mattt, still looking at why branches does not do what I thought it does | 08:35 |
*** SourabhP has joined #openstack-neutron | 08:35 | |
*** sayali has joined #openstack-neutron | 08:37 | |
*** pal has joined #openstack-neutron | 08:37 | |
*** sambetts has quit IRC | 08:38 | |
openstackgerrit | Merged openstack/python-neutronclient: Add document for entry point in setup.cfg https://review.openstack.org/214945 | 08:38 |
mattt | ihrachyshka: ok, thank you ... a side question, if i 'upgrade heads', do i actually have to stamp or does that happen as part of the upgrade? | 08:38 |
ihrachyshka | mattt, it should stamp. jlibosva ^^ correct? | 08:39 |
ihrachyshka | mattt, btw to upgrade just expand, do upgrade liberty_expand@head | 08:39 |
*** sambetts has joined #openstack-neutron | 08:39 | |
mattt | ihrachyshka: ah! cool, thank you | 08:39 |
ihrachyshka | mattt, some docs: http://docs.openstack.org/developer/neutron/devref/alembic_migrations.html#applying-database-migration-rules | 08:41 |
*** shashank_hegde has quit IRC | 08:41 | |
*** aix has joined #openstack-neutron | 08:41 | |
jlibosva | I confirm | 08:41 |
ihrachyshka | mattt, it's in my todo for L to implement expand and contract commands to avoid liberty/head notions | 08:42 |
*** scheuran has quit IRC | 08:44 | |
*** shoutm has joined #openstack-neutron | 08:44 | |
jlibosva | mattt: you can verify by looking into alembic_versions table in neutron database | 08:44 |
jlibosva | there is also a command for neutron-db-manage which I forgot | 08:44 |
*** DWfuturetec has quit IRC | 08:45 | |
ihrachyshka | jlibosva, it's current, but it is broken a bit now | 08:45 |
jlibosva | yes! that one | 08:45 |
*** apuimedo has joined #openstack-neutron | 08:46 | |
*** vikram has quit IRC | 08:46 | |
*** rossella_s has joined #openstack-neutron | 08:52 | |
*** shoutm has quit IRC | 08:54 | |
*** shoutm has joined #openstack-neutron | 08:55 | |
*** jpena_ has joined #openstack-neutron | 08:55 | |
jlibosva | ihrachyshka: ajo about what was said yesterday in qos mtg - should we mark qos bugs with bp/<topic> ? Wouldn't be better to use bug/<number> and prioritize bug to be in L-# ? | 08:55 |
jlibosva | if I understood that correctly | 08:55 |
*** jpena has quit IRC | 08:56 | |
ihrachyshka | jlibosva, well, it depends on whether it's critical for l | 08:57 |
ihrachyshka | jlibosva, if it is, I think it's better to solve in scope of bp | 08:57 |
salv-orlando | ihrachyshka is right as usual. The bottom line imho is that you should ask yourself whether this break or not some fundamental QoS API workflow | 08:59 |
salv-orlando | if not -> bp scope, if yes -> bug and prioritize | 08:59 |
*** nyechiel_ has quit IRC | 08:59 | |
ihrachyshka | HenryG, I try to understand what is the intent of 'branches' command in db cli. seems like it will always show us nothing (since we don't allow branch splits) | 09:00 |
ihrachyshka | salv-orlando, I think it's vice versa: if it breaks, then bp scope, otherwise a bug? | 09:01 |
jlibosva | got it :) | 09:01 |
*** vikram has joined #openstack-neutron | 09:01 | |
salv-orlando | ihrachyshka: I am seeing the world upside down maybe, but why would you not prioritize fixing something broken, moving the patch in the bp scope? | 09:02 |
*** jamie_h has joined #openstack-neutron | 09:02 | |
ihrachyshka | salv-orlando, I will do it. I think we are actually on common ground, though I read your 'if not -> bp scope, if yes -> bug and prioritize' as you suggesting to fix major issues as bugs | 09:03 |
*** yamamoto has quit IRC | 09:04 | |
*** 7GHAAZ22K has joined #openstack-neutron | 09:04 | |
ihrachyshka | I would expect that bp has a higher prio than a bug, since it shows up in reviewer dashboard etc. But I may be wrong. Anyway, seems like in the end I don't mind how we marked a patch as long as it's merged in time | 09:05 |
ihrachyshka | akamyshnikova, hi | 09:05 |
salv-orlando | ihrachyshka: I see... that's the reviewer perspective then ;) | 09:05 |
*** SourabhP has quit IRC | 09:05 | |
salv-orlando | when I was one of those +2 people, it would put high/critical bugs on top of my review list | 09:05 |
ihrachyshka | akamyshnikova, in db cli, we have 'branches' command. but it does not do anything because we don't use branch points | 09:05 |
*** jpena_ is now known as jpena | 09:06 | |
salv-orlando | ihrachyshka: followed by essential/high blueprints | 09:06 |
ihrachyshka | salv-orlando, aye, I guess it's my reviewer glitch, sorry | 09:06 |
salv-orlando | and then everything else - ie: the stuff I ended up not reviewing at all | 09:06 |
*** nyechiel_ has joined #openstack-neutron | 09:08 | |
*** fawadkhaliq has quit IRC | 09:08 | |
*** sbalukoff has joined #openstack-neutron | 09:09 | |
*** apuimedo is now known as apuimedo|away | 09:10 | |
*** shaohe_feng has quit IRC | 09:10 | |
*** l6unchpad has quit IRC | 09:11 | |
*** l6unchpad has joined #openstack-neutron | 09:12 | |
*** john-davidge has joined #openstack-neutron | 09:13 | |
*** shoutm has quit IRC | 09:15 | |
*** john-davidge_ has joined #openstack-neutron | 09:15 | |
*** john-davidge has quit IRC | 09:17 | |
*** john-davidge_ is now known as john-davidge | 09:17 | |
*** sdake_ has joined #openstack-neutron | 09:18 | |
ihrachyshka | salv-orlando, hm, quick question re https://review.openstack.org/#/c/204551/ | 09:18 |
ihrachyshka | salv-orlando, now that I started to dig thru branches/current code, it occurs to me that we actually may redefine the command meaning here. | 09:18 |
ihrachyshka | salv-orlando, alembic current shows the db state, while alembic heads show the state of migration tree, no? | 09:19 |
*** jistr has quit IRC | 09:19 | |
akamyshnikova | ihrachyshka, seems it became useless now | 09:20 |
akamyshnikova | ihrachyshka, where is it used? | 09:20 |
salv-orlando | akamyshnikova: yes the patch might have become useless if the rest of the migration CLI evolved in the meanwhile | 09:20 |
ihrachyshka | akamyshnikova, yeah, I think we should kill the cli and instead make another validation that will walk thru revisions and make sure we have no branch points. | 09:20 |
*** jistr has joined #openstack-neutron | 09:20 | |
ihrachyshka | salv-orlando, I think akamyshnikova referred to branches cli, not the patch :) | 09:21 |
salv-orlando | but maybe I'm not following | 09:21 |
salv-orlando | ihrachyshka: ah I see | 09:21 |
salv-orlando | the 'heads' command is still useufl | 09:21 |
*** sdake has quit IRC | 09:21 | |
salv-orlando | because it tells you the head for each branch | 09:21 |
salv-orlando | and ihrachyshka you're right - current is supposed to show the state in the DB | 09:22 |
*** bharath_ has quit IRC | 09:22 | |
ihrachyshka | salv-orlando, it is, I mostly wonder whether we actually break current | 09:22 |
ihrachyshka | meh. | 09:22 |
salv-orlando | so perhaps renaming current -> heads is a terrible idea | 09:22 |
ihrachyshka | aye, I was a bad reviewer who just assumed it's what it looked like to me instead of checking myself. sigh. | 09:23 |
ihrachyshka | let's block it in gate now and think how to proceed. | 09:23 |
salv-orlando | ihrachyshka: I think there is a chance current might be broken in alembic as it's not ready to deal with multiple heads? what do you reckon akamyshnikova? | 09:23 |
ihrachyshka | salv-orlando, yeah, I see 'current' does not show 'kilo' in output | 09:24 |
*** y_sawai has joined #openstack-neutron | 09:24 | |
salv-orlando | ihrachyshka: because the db table alembic_version does not contain that unless you have a continuos deployment that was first upgraded to kilo and then to head | 09:24 |
salv-orlando | if you start a devstack deployment post-kilo you never see a "kilo' revision, so you totally ignore the kilo branch | 09:25 |
salv-orlando | which is fine. | 09:25 |
salv-orlando | It's just that I find weird that it's not easy to associate a head id to its branch | 09:25 |
*** neophy has joined #openstack-neutron | 09:25 | |
ihrachyshka | salv-orlando, every revision has branch_label | 09:26 |
ihrachyshka | that is inherited from any other revision in the branch that has it set | 09:26 |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Fix _ensure_default_security_group logic https://review.openstack.org/203384 | 09:27 |
vikram | ihrachyshka: ping | 09:27 |
vikram | ihrachyshka: what changes are required for "https://bugs.launchpad.net/neutron/+bug/1485537" | 09:27 |
openstack | Launchpad bug 1485537 in neutron "adopt oslo_config.fixture.Config" [Undecided,New] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 09:27 |
ihrachyshka | salv-orlando, if I upgrade to kilo, then to all heads, I still miss kilo | 09:27 |
ihrachyshka | vikram, I think it will belong to neutron/tests/base.py mostly. we have config_parse and friends there that may need modification to use the fixture | 09:28 |
*** mohankumar has joined #openstack-neutron | 09:28 | |
vikram | ihrachyshka: ok | 09:29 |
ihrachyshka | currently we just reuse cfg.CONF there, but we need to use the fixture that is based on the cfg.CONF | 09:29 |
salv-orlando | ihrachyshka: let me check a bit the branch history | 09:30 |
akamyshnikova | salv-orlando, we need to check this with Mike, as it seems expected to work | 09:31 |
ihrachyshka | salv-orlando, I guess it's alembic that should have put kilo in the table? | 09:31 |
salv-orlando | ihrachyshka: so both liberty/contract and liberty/expand branch off kilo. So when you run the first migration for any one of these branches I think alembic overwrite kilo in the table alembic_revision | 09:31 |
ihrachyshka | it drops it once we apply a revision that depends_on kilo | 09:31 |
salv-orlando | because as a matter of fact you have no element of your schema that belongs anymore to the kilo branch... | 09:32 |
salv-orlando | at least that's my interpretation | 09:32 |
ihrachyshka | salv-orlando, in fact, there is no kilo branch | 09:32 |
*** yamahata has quit IRC | 09:32 | |
ihrachyshka | well, there is one, but it's not even labeled | 09:32 |
*** britthouser has joined #openstack-neutron | 09:32 | |
salv-orlando | if you added a migration in this kilo branch then probably alembic_revision would list it once you upgraded the schema | 09:33 |
salv-orlando | but I'm speculating | 09:33 |
salv-orlando | are we ever going to add anything to that kilo branch? Can we consider kilo a branchpoint, and the kilo branch dead with kilo revision id itself? | 09:33 |
*** britthou_ has joined #openstack-neutron | 09:33 | |
ihrachyshka | I guess you are right. but it's weird that script.get_heads() return kilo too | 09:34 |
ihrachyshka | salv-orlando, you mean, we switch expand/contract roots from depends_on to down_revision? | 09:35 |
salv-orlando | no not really | 09:35 |
salv-orlando | I was just saying whether kilo for us is an active branch or just a branch representing the sequence of reevision until the expand/contract branchpoint | 09:36 |
salv-orlando | ihrachyshka: I do not think we consider it an active branch, do we? | 09:36 |
*** jpena has quit IRC | 09:36 | |
ihrachyshka | salv-orlando, we don't | 09:36 |
ihrachyshka | salv-orlando, so once I switch roots to down_revision = 'kilo', both current and script.get_heads() that we put into HEADS file now do not contain kilo | 09:37 |
salv-orlando | ihrachyshka: but we have no way to tell that alembic. Because if I wanted I could go ahead and add a new migration in that branch | 09:37 |
*** openkarma has joined #openstack-neutron | 09:37 | |
*** britthouser has quit IRC | 09:37 | |
openstackgerrit | John Davidge proposed openstack/neutron: L3 agent changes and reference implementation for IPv6 PD https://review.openstack.org/185977 | 09:37 |
ihrachyshka | salv-orlando, you could do it, yes. in fact, it could be a way to backport db migrations | 09:37 |
ihrachyshka | though it would require conditionals in master | 09:38 |
*** openkarma has quit IRC | 09:38 | |
ihrachyshka | if we are afraid of branching, we may just forbid branch points expect for specific release revisions | 09:38 |
ihrachyshka | *except | 09:38 |
jlibosva | moshele: I'm looking at https://review.openstack.org/#/c/214212/5/neutron/tests/api/test_qos.py | 09:39 |
salv-orlando | ihrachyshka: so by using down_revision = kilo for the root branch you can actually tell alembic that the head you're branching off is not anymore an active branch? | 09:39 |
ihrachyshka | I actually now think that down_revision = 'kilo' is the way to go. | 09:39 |
jlibosva | moshele: have you experienced inconsistency between what is returned by create_qos_bw_limit_rule and list_bw_limit_rules? | 09:39 |
ihrachyshka | salv-orlando, well, not really. alembic will still allow to branch off kilo | 09:39 |
jlibosva | moshele: I did in fullstack job and I'm surprised the api test works | 09:39 |
ihrachyshka | salv-orlando, but we can add a validation to check_migration to forbid it | 09:39 |
moshele | jlibosva: why? | 09:40 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest https://review.openstack.org/209943 | 09:40 |
jlibosva | moshele: I saw list_bw_limit_rules returns 'type' in the response while create doesn't | 09:40 |
jlibosva | moshele: ohh | 09:40 |
jlibosva | moshele: sorry, I mixed that | 09:40 |
jlibosva | moshele: disregard, sorry for bothering | 09:41 |
moshele | jlibosva: np | 09:41 |
* jlibosva hides | 09:41 | |
*** gberginc has quit IRC | 09:41 | |
*** y_sawai has quit IRC | 09:42 | |
salv-orlando | ihrachyshka: yes that makes sense | 09:42 |
*** vikram has quit IRC | 09:42 | |
*** openkarma has joined #openstack-neutron | 09:44 | |
*** openkarma has quit IRC | 09:44 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py https://review.openstack.org/215012 | 09:44 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos https://review.openstack.org/213194 | 09:49 |
*** jpena has joined #openstack-neutron | 09:49 | |
*** markvoelker has joined #openstack-neutron | 09:49 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos https://review.openstack.org/213194 | 09:50 |
*** sudipto has joined #openstack-neutron | 09:50 | |
*** aix has quit IRC | 09:51 | |
*** gardenshed has quit IRC | 09:51 | |
*** vikram has joined #openstack-neutron | 09:54 | |
*** markvoelker has quit IRC | 09:54 | |
*** mickeys has joined #openstack-neutron | 09:56 | |
*** dims has joined #openstack-neutron | 09:56 | |
*** yog_ has quit IRC | 09:57 | |
*** fawadkhaliq has joined #openstack-neutron | 09:58 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: neutron-db-manage: sync HEADS file with 'current' output https://review.openstack.org/215048 | 09:59 |
ihrachyshka | salv-orlando, akamyshnikova, HenryG ^^ take a look | 09:59 |
*** DWfuturetec has joined #openstack-neutron | 09:59 | |
ihrachyshka | hm, it though opens a question of what we do when mitaka_expand comes in. if we down_revision, it will get liberty_expand branch label due to inheritance | 10:00 |
ihrachyshka | oh, and I think kilo may actually now get liberty_expand label. hm. | 10:01 |
ihrachyshka | yeah, with the patch 'upgrade liberty_expand@kilo' is a valid command. meh. | 10:02 |
*** Kennan2 has joined #openstack-neutron | 10:03 | |
*** Kennan has quit IRC | 10:04 | |
salv-orlando | ihrachyshka: an loosely related question. Why did you ended up having release specific branches, and not just "expand" and "contract" branches? | 10:04 |
salv-orlando | 0" | 10:05 |
ihrachyshka | salv-orlando, honestly, mostly because it is as in zzzeek's spec. | 10:05 |
ihrachyshka | we may actually want it to be expand/contract, yeah | 10:05 |
salv-orlando | ihrachyshka: I'm ok with it, it's just that we need to handle branching at each release | 10:06 |
*** tiny-hands has joined #openstack-neutron | 10:06 | |
salv-orlando | it complicates mgmt but gives us leeway to backport migrations | 10:06 |
*** gampel has joined #openstack-neutron | 10:06 | |
*** tiny-hands has quit IRC | 10:08 | |
*** gardenshed has joined #openstack-neutron | 10:08 | |
*** amotoki has quit IRC | 10:08 | |
salv-orlando | ihrachyshka: now a related question. with your patch "upgrade liberty_expand@kilo" is valid, but what are its semantics? | 10:08 |
*** jprovazn has joined #openstack-neutron | 10:09 | |
ihrachyshka | salv-orlando, I don't like it now that I think about it, I will probably WIP-1 it. but alembic wise, it means 'upgrade to the kilo revision that belongs to liberty_expand branch. now that it's down_revision, kilo belongs to both expand and contract bracnhes. | 10:10 |
*** vikram has quit IRC | 10:10 | |
salv-orlando | ihrachyshka: and it most cases that would not work because it would be a downgrade. If you think about it, it makes sense | 10:10 |
salv-orlando | liberty_{expand|contract} start with kilo | 10:11 |
salv-orlando | mitaka_expand starts with liberty and so on | 10:11 |
ihrachyshka | salv-orlando, but then if you do upgrade liberty_expand@head on mitaka, if mitaka root is using down_revision and not depends_on, it will upgrade till mitaka | 10:12 |
ihrachyshka | while someone may expect it to upgrade till last revision in liberty only | 10:12 |
*** fawadkhaliq has quit IRC | 10:13 | |
*** nyechiel_ has quit IRC | 10:13 | |
*** bharath_ has joined #openstack-neutron | 10:13 | |
*** sdake_ is now known as sdake | 10:15 | |
*** lyanchih has quit IRC | 10:16 | |
salv-orlando | ihrachyshka: yes unfortunately this is true. But this also means that we're actually not terminating the X branch but contuining it | 10:18 |
salv-orlando | and therefore the mitaka_expand branch revisions, will belong also to the liberty_expand one. And that's not correct | 10:18 |
salv-orlando | sounds like down_revision does not work either | 10:18 |
ihrachyshka | I guess renaming labels not to include release name would avoid confusion. | 10:19 |
salv-orlando | ihrachyshka: do you mean the branch labels or the revision ids? | 10:20 |
*** britthou_ has quit IRC | 10:20 | |
ihrachyshka | branch labels. they would be just expand and contract, so you would do 'upgrade expand@head' instead of liberty_expand@head, so it would not confuse you if you get mitaka expansion scripts too | 10:21 |
*** vikram has joined #openstack-neutron | 10:22 | |
salv-orlando | ihrachyshka: correct. This will limit the ability to backport migrations though. If we were planning to provide that capability - regardless of stable backport rules - it might be a problem | 10:23 |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: DVR: make sure snat portion is always scheduled when needed https://review.openstack.org/215055 | 10:23 |
ihrachyshka | salv-orlando, you mean, for downstream consumers in case they want a backport? | 10:23 |
ihrachyshka | it would still require modifications both in stable release (the backport) and the next one (modifying the script that you've backported to not apply if we see the changes are already there) | 10:24 |
*** 7GHAAZ22K has quit IRC | 10:25 | |
ihrachyshka | with single chain, it's pretty the same: you break the migration chain and modify two scripts: one that you move to the top of stable branch, to point to stable head as down_revision, and then the next script after the script you move: to point to backported script ancestor | 10:25 |
*** mickeys has quit IRC | 10:26 | |
salv-orlando | ihrachyshka: sure. I was suggesting that with distinct branches might be easier because if trunk is X you have a branch {X-1}_expand with a {X-1} release identifier (the one with the depends on for the X_expand branch | 10:28 |
salv-orlando | and you can add backported revision to X-1_expand | 10:28 |
salv-orlando | ihrachyshka: but then upgrading to X will be a funny activity | 10:29 |
salv-orlando | that will probably end up in tears | 10:29 |
ihrachyshka | salv-orlando, then in X, you should kill the migration rule from the X_expand chain | 10:29 |
salv-orlando | so yeah the backport thing does not really work and is always a non non | 10:29 |
salv-orlando | so I agree we can just drop the release names in branch labels | 10:30 |
salv-orlando | and simplify things | 10:30 |
ihrachyshka | salv-orlando, well, unless you consider master chasing (downstreams usually don't) it mostly works, except that you need to notify users that they need to upgrade after a minor update | 10:30 |
ihrachyshka | which is PITA but may be a better option from plenty of bad ones | 10:31 |
*** DWfuturetec has quit IRC | 10:31 | |
*** vthapar has quit IRC | 10:31 | |
salv-orlando | yes, but as you too wrote, then you need to kill the backported migrations first when upgrading to a new major release | 10:31 |
salv-orlando | and we have no way to automate this at the moment | 10:32 |
*** baojg has quit IRC | 10:33 | |
*** baojg has joined #openstack-neutron | 10:34 | |
*** gberginc has joined #openstack-neutron | 10:34 | |
openstackgerrit | neethi shashidhar proposed openstack/neutron-specs: CLI implementation for listing and deleting tunnel endpoints https://review.openstack.org/215058 | 10:35 |
*** vikram has quit IRC | 10:37 | |
*** baojg has quit IRC | 10:38 | |
*** fawadkhaliq has joined #openstack-neutron | 10:38 | |
*** vikram_choudhary has joined #openstack-neutron | 10:38 | |
*** lpetrut has joined #openstack-neutron | 10:38 | |
*** hoangcx has quit IRC | 10:41 | |
*** hoangcx has joined #openstack-neutron | 10:41 | |
openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 10:43 |
HenryG | salv-orlando: ping | 10:43 |
salv-orlando | good morning HenryG | 10:43 |
salv-orlando | our lord of the database and saviour of the gate | 10:44 |
HenryG | salv-orlando: that can't be me | 10:44 |
HenryG | salv-orlando: Sorry for barging in on your reservations patch yesterday | 10:45 |
*** cbouch has joined #openstack-neutron | 10:45 | |
HenryG | salv-orlando: any thoughts on it? | 10:46 |
*** stanzgy__ has joined #openstack-neutron | 10:47 | |
*** gardenshed has quit IRC | 10:48 | |
*** DWfuturetec has joined #openstack-neutron | 10:50 | |
*** stanzgy has quit IRC | 10:50 | |
*** mohankumar has quit IRC | 10:51 | |
*** lazy_prince has joined #openstack-neutron | 10:52 | |
salv-orlando | HenryG: that issue was already knownn. It's not very worrying because: | 10:52 |
salv-orlando | 1) ihrachyshka pushed a patch for it which is not yet approved | 10:52 |
*** shaohe_feng has joined #openstack-neutron | 10:53 | |
salv-orlando | 2) I am trying to get rid of the lock altogether | 10:53 |
diga | hi tfukushima | 10:54 |
diga | hi irenab | 10:54 |
tfukushima | Hi diga, | 10:54 |
irenab | hi | 10:54 |
salv-orlando | HenryG: if this occurs in master branch with rally then I guess you can merge my patch as it is now | 10:54 |
salv-orlando | the fix is good anyway | 10:54 |
ihrachyshka | HenryG, https://review.openstack.org/#/c/214133/ | 10:55 |
diga | veth pair is not the interface but by the way I understood now how to do that | 10:55 |
*** sudipto has quit IRC | 10:55 | |
*** sudipto has joined #openstack-neutron | 10:56 | |
diga | we need to create port using ovs on host & that should get added to veth | 10:56 |
diga | correct me if I am wrong | 10:56 |
HenryG | salv-orlando: I think ihrachyshka's patch is safer | 10:57 |
HenryG | At least until you get rid of the locking | 10:57 |
salv-orlando | HenryG: ok, let's merge it then. | 10:57 |
salv-orlando | HenryG: I am still finishing up the patch. It might take the rest of the day | 10:57 |
*** hoangcx has quit IRC | 10:58 | |
salv-orlando | because I am dealing with the most annoying of the sqlalchemy errors: "This Session's transaction has been rolled back by a nested rollback() call" | 10:58 |
HenryG | ouch | 10:58 |
openstackgerrit | vikram.choudhary proposed openstack/neutron: BGP Dynamic Routing: introduce entities and model https://review.openstack.org/201621 | 10:58 |
tfukushima | diga, I only know the OVS datapath but we can bind a "link" to the datapath and the link is turned into a "port" of the datapath. | 10:58 |
diga | tfukushima, can you share docs related to this, let me go through it | 11:00 |
irenab | diga: here comes the difference between different vif types binding. For OVS/OVN it will ad port on ovs bridge | 11:01 |
*** mwagner_lap has quit IRC | 11:01 | |
HenryG | salv-orlando: fyi, I approved the "less red in rally" fix, https://review.openstack.org/203384 | 11:01 |
irenab | diga: for Midonet it will do what tfukushima said | 11:01 |
salv-orlando | HenryG: Yes that seems to work for the default sec group issue | 11:02 |
diga | irenab: okay | 11:02 |
salv-orlando | the reservation work will still have red, but not failures. | 11:02 |
HenryG | salv-orlando: what do you think about making the rally job voting soon? | 11:02 |
salv-orlando | rally as a fairly high contention model, so deadlock will happen. But the decorator will sort out the situation. | 11:02 |
salv-orlando | HenryG: it's a dual-edged sword | 11:02 |
salv-orlando | because on the one hand every developer will be forced to look at it. | 11:03 |
diga | irenab: I need to read about this ovs datapath | 11:03 |
openstackgerrit | Merged openstack/neutron: Minor typo fix https://review.openstack.org/213019 | 11:03 |
salv-orlando | but on the other hand expect an increase in time the gate is wedged | 11:03 |
salv-orlando | I think most, if not all, core reviewers look at it. | 11:04 |
*** yalie has quit IRC | 11:04 | |
*** alex_klimov has joined #openstack-neutron | 11:04 | |
salv-orlando | The issues we had happened because the pymysql and workers switch merged asynchronously with the quota reservation patch | 11:04 |
salv-orlando | on devstack-gate - which does not run the neutron rally test | 11:04 |
*** apuimedo|away is now known as apuimedo | 11:04 | |
apuimedo | diga: https://github.com/stackforge/nova-docker/blob/master/novadocker/virt/docker/vifs.py | 11:06 |
apuimedo | look at this handling | 11:06 |
akamyshnikova | ihrachyshka, about your letter, can you send me what difference was discovered as http://pastebin.test.redhat.com/306865 does not opened for me. | 11:06 |
apuimedo | this is a superset of what the executables need to do | 11:06 |
apuimedo | note that for ovs it does: linux_net.create_ovs_vifg_port | 11:07 |
diga | sure apuimedo | 11:07 |
ihrachyshka | akamyshnikova, ^^ oh right, it's an internal thing. jlibosva, can you help akamyshnikova with seeing what the new alembic caught? | 11:07 |
apuimedo | that is practically the only thing that goes into the ovs executable | 11:07 |
apuimedo | and for midonet it is going to be the mm-ctl --bind-port... | 11:07 |
diga | Here wht I have implemented according to comments received from irenab | 11:07 |
HenryG | salv-orlando: just feels to me that currently rally is the only job that decently catches DB issues | 11:08 |
diga | apuimedo: you can take a look at this - https://review.openstack.org/#/c/214002/ | 11:08 |
jlibosva | akamyshnikova: there you go :) http://paste.openstack.org/show/422619/ | 11:08 |
apuimedo | thanks diga. I'll take a look | 11:08 |
diga | sure apuimedo | 11:09 |
HenryG | salv-orlando: so I guess it will be my job to always look at it | 11:09 |
jlibosva | akamyshnikova: I pasted the same to bug I opened: https://bugs.launchpad.net/neutron/+bug/1486936 | 11:09 |
openstack | Launchpad bug 1486936 in neutron "Migration scripts and db models are not in sync" [Undecided,New] | 11:09 |
*** stanzgy has joined #openstack-neutron | 11:10 | |
cbouch | core reviewers: I have a couple cherry-picks pending approval for kilo. Please help. They are https://review.openstack.org/#/c/214784/ and https://review.openstack.org/#/c/213327/ | 11:10 |
*** ushkalim has quit IRC | 11:11 | |
*** ushkalim has joined #openstack-neutron | 11:11 | |
*** britthouser has joined #openstack-neutron | 11:12 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Dropped release name from migration branch labels https://review.openstack.org/215076 | 11:12 |
*** yalie has joined #openstack-neutron | 11:12 | |
*** stanzgy__ has quit IRC | 11:13 | |
akamyshnikova | jlibosva, thanks! | 11:13 |
openstackgerrit | Gal Sagie proposed openstack/neutron-specs: Create resource with fixed id spec https://review.openstack.org/215079 | 11:13 |
salv-orlando | HenryG: yours because you have a duty as a core reviewer, but it should be everyone's. due to its nature I am bit wary of making the rally job voring. | 11:14 |
salv-orlando | *voting | 11:14 |
*** yamamoto_ has joined #openstack-neutron | 11:14 | |
*** jamie_h has quit IRC | 11:14 | |
salv-orlando | HenryG: but still you could define a lighter version of the job by trimming down tasks, and have that job vote | 11:15 |
*** vikram_choudhary has quit IRC | 11:15 | |
*** dims has quit IRC | 11:15 | |
*** dims has joined #openstack-neutron | 11:16 | |
*** aix has joined #openstack-neutron | 11:16 | |
*** shaohe_feng has quit IRC | 11:17 | |
HenryG | ihrachyshka: Are you going make some alembic branch labels change? | 11:20 |
*** dims has quit IRC | 11:20 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Split DRIVER_TABLES in external.py https://review.openstack.org/215012 | 11:20 |
ihrachyshka | HenryG, we discussed it with salv-orlando today, you may catch up in irc logs. currently I try to see whether it will be easy now that we have other projects that use the labels | 11:22 |
ihrachyshka | the rationale is in the commit message | 11:22 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/215086 | 11:22 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/215087 | 11:22 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/215089 | 11:22 |
HenryG | ihrachyshka: yes, I read the irc backlog | 11:22 |
HenryG | ihrachyshka: which commit msg? | 11:23 |
ihrachyshka | HenryG, https://review.openstack.org/#/c/215076/ | 11:23 |
ihrachyshka | HenryG, also see the parent that switches to down_revision to make current output in sync with alembic_version in db | 11:24 |
ihrachyshka | meh, sorry... alembic heads vs alembic_version | 11:25 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/204422 | 11:25 |
*** baoli has joined #openstack-neutron | 11:25 | |
HenryG | ihrachyshka: thanks, reading | 11:26 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Common l2_extension test code moved to the right directory https://review.openstack.org/214610 | 11:26 |
*** vikram_choudhary has joined #openstack-neutron | 11:27 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Common l2_extension test code moved to the right directory https://review.openstack.org/214610 | 11:28 |
*** pc_m has joined #openstack-neutron | 11:28 | |
HenryG | ihrachyshka: I guess we'll have to see if we can undo the external projects that have initialized their alembic branches | 11:29 |
*** baoli has quit IRC | 11:30 | |
*** DWfuturetec has quit IRC | 11:31 | |
jlibosva | ihrachyshka: HenryG can you confirm please that neutron-db-manage --service <adv_service> upgrade head is not needed once there is entrypoint in system (from adv service egg file)? And it's executed automatically without --service | 11:31 |
*** scheuran has joined #openstack-neutron | 11:32 | |
ihrachyshka | HenryG, or temporarily support their schema | 11:33 |
ihrachyshka | jlibosva, it should | 11:33 |
jlibosva | ihrachyshka: thanks | 11:33 |
*** pal has quit IRC | 11:33 | |
*** dims has joined #openstack-neutron | 11:33 | |
*** markvoelker has joined #openstack-neutron | 11:35 | |
*** Murali has quit IRC | 11:35 | |
*** pal has joined #openstack-neutron | 11:39 | |
*** markvoelker has quit IRC | 11:40 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Dropped release name from migration branch labels https://review.openstack.org/215076 | 11:40 |
ihrachyshka | HenryG, I think it should work for all subprojects ^^ | 11:40 |
*** Marga_ has quit IRC | 11:42 | |
*** Marga_ has joined #openstack-neutron | 11:42 | |
HenryG | ihrachyshka: cool hack | 11:42 |
*** thorst has joined #openstack-neutron | 11:43 | |
*** l6unchpad has quit IRC | 11:43 | |
ihrachyshka | HenryG, I hate how I manage the feature that it needs those hacks. seems like I didn't think about it for long enough. | 11:43 |
ihrachyshka | another can of worms that I left to solve for later is splitting HEADS file | 11:43 |
*** yamamoto_ has quit IRC | 11:44 | |
ajo | ihrachyshka, kevinbenton : https://review.openstack.org/#/c/214608/1/neutron/objects/base.py | 11:44 |
HenryG | ihrachyshka: I am equally to blame for not catching the issues | 11:44 |
*** l6unchpad has joined #openstack-neutron | 11:45 | |
*** salv-orlando has quit IRC | 11:45 | |
*** afazekas has quit IRC | 11:46 | |
HenryG | ihrachyshka: splitting HEADS can now be simplified, no? contract/HEAD, expand/HEAD ? | 11:46 |
ihrachyshka | HenryG, yes, that's the idea. but since, again, projects caught HEADS, we need compatibility | 11:46 |
ihrachyshka | for a cycle | 11:47 |
*** dansmith has quit IRC | 11:47 | |
*** dansmith has joined #openstack-neutron | 11:48 | |
*** dansmith is now known as Guest95328 | 11:48 | |
apuimedo | tfukushima: https://review.openstack.org/#/c/208566/ | 11:49 |
apuimedo | are you going to make a new version of it with the imports fixed so we can get this merged? | 11:49 |
HenryG | ihrachyshka: at least for the project I manage no one is trunk chasing so I can revert everything before L is released | 11:49 |
openstackgerrit | Moshe Levi proposed openstack/neutron: QoS: fix get bandwidth limit rules to filter them per policy https://review.openstack.org/214212 | 11:53 |
*** britthouser has quit IRC | 11:55 | |
HenryG | akamyshnikova: hi | 11:55 |
*** vikram_choudhary is now known as vikram | 11:55 | |
ihrachyshka | HenryG, you think no one does, yes. But I believe I should suffer thru the pain to be more wise in the future :) | 11:55 |
akamyshnikova | HenryG, hi | 11:56 |
*** emagana has quit IRC | 11:56 | |
HenryG | ihrachyshka: :) | 11:56 |
HenryG | akamyshnikova: bug #1486077 | 11:56 |
openstack | bug 1486077 in neutron "VPNAAS: dsvm-functional doesn't have installed database backends" [Undecided,Invalid] https://launchpad.net/bugs/1486077 | 11:56 |
akamyshnikova | HenryG, I forgot to include deps in tox.ini :( | 11:57 |
akamyshnikova | HenryG, so mysal and postgres are installed | 11:57 |
HenryG | akamyshnikova: aha! no worries | 11:58 |
*** gardenshed has joined #openstack-neutron | 11:59 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Make NeutronObjectDuplicateEntry exception more verbose https://review.openstack.org/214608 | 11:59 |
HenryG | akamyshnikova: but functional-sswan is still not working? | 12:01 |
akamyshnikova | HenryG, no the same, it has separate section in tox.ini | 12:02 |
*** mohankumar has joined #openstack-neutron | 12:02 | |
akamyshnikova | HenryG, I was going to update it, but I need to update depend change | 12:02 |
*** lpetrut has quit IRC | 12:02 | |
HenryG | akamyshnikova: I will leave a comment on how to make common deps for functional | 12:03 |
*** gardenshed has quit IRC | 12:03 | |
akamyshnikova | pc_m, https://review.openstack.org/213174 - should I push new patch removing tenant_id in cisco_csr_identifier_map table? Is it really unneeded? | 12:04 |
akamyshnikova | HenryG, great, thanks! | 12:04 |
akamyshnikova | pc_m, or we need some other people opinion on that? | 12:05 |
pc_m | akamyshnikova: I don't think it is needed for that table (it maps OS IDs to CSR IDs and is supposed to be globally unique). I'd suggest committing a patch to remove and we can make sure that CSR team reviews it (as they are doing active support of the drivers). | 12:07 |
*** b3nt_pin is now known as beagles | 12:07 | |
pc_m | akamyshnikova: That's Tom Holtzen and Yanping Qu. | 12:08 |
akamyshnikova | pc_m, OK, thanks! | 12:09 |
*** piyanai has joined #openstack-neutron | 12:10 | |
*** jpena is now known as jpena|lunch | 12:10 | |
*** azbiswas has joined #openstack-neutron | 12:11 | |
*** salv-orlando has joined #openstack-neutron | 12:11 | |
salv-orlando | ihrachyshka: why would you split it? I was thinking we might just add branch labels to it | 12:12 |
*** bharath_ has quit IRC | 12:13 | |
ihrachyshka | salv-orlando, HEADS file? mostly to avoid half of git conflicts due to merge of a new script | 12:13 |
salv-orlando | ihrachyshka: I see that's a good point inded. | 12:13 |
ihrachyshka | yeah, it's painful to rebase, so if we can help to avoid it in some cases... | 12:14 |
*** trinaths has quit IRC | 12:14 | |
salv-orlando | ihrachyshka: rebasing by itself is not painful. It takes <1 minute. the problem are the patches going into merge conflict and needing a respin | 12:15 |
*** gildub has joined #openstack-neutron | 12:15 | |
salv-orlando | but with 90% of migrations being expand ones perhaps we'd still have those conflicts even after splitting HEADS | 12:15 |
ihrachyshka | yes, that's what I meant. and vote reset. | 12:15 |
ihrachyshka | salv-orlando, well ok, not half :) | 12:16 |
*** azbiswas has quit IRC | 12:16 | |
ihrachyshka | jschwarz, can you check the silly comment at https://review.openstack.org/#/c/210830/ ? | 12:17 |
*** lpetrut has joined #openstack-neutron | 12:17 | |
jschwarz | ihrachyshka, sure, in a few minutes | 12:17 |
*** sdake has quit IRC | 12:17 | |
*** haigang has quit IRC | 12:18 | |
pc_m | Does anyone know how networking-* projects handle CLI support? Do they have their own client code, use neutronclient, or something else? | 12:21 |
*** edmondsw has joined #openstack-neutron | 12:23 | |
*** pal has quit IRC | 12:23 | |
*** vikram has quit IRC | 12:23 | |
*** fawadkhaliq has quit IRC | 12:24 | |
*** gberginc has quit IRC | 12:24 | |
*** stanzgy has quit IRC | 12:25 | |
*** rpothier has quit IRC | 12:27 | |
*** mkolesni_ has quit IRC | 12:31 | |
*** emagana has joined #openstack-neutron | 12:32 | |
*** banix has joined #openstack-neutron | 12:32 | |
jschwarz | ihrachyshka, gah, missed that comment | 12:33 |
jschwarz | it doesn't apply, will resubmit | 12:33 |
openstackgerrit | John Schwarz proposed openstack/neutron: Add EnvironmentDescription, pass it down https://review.openstack.org/210830 | 12:34 |
*** britthouser has joined #openstack-neutron | 12:36 | |
*** signed8bit has joined #openstack-neutron | 12:36 | |
*** mwagner_lap has joined #openstack-neutron | 12:37 | |
*** vikram has joined #openstack-neutron | 12:38 | |
*** lpetrut has quit IRC | 12:39 | |
*** markvoelker has joined #openstack-neutron | 12:39 | |
*** gongysh has quit IRC | 12:39 | |
*** banix has quit IRC | 12:40 | |
*** neophy has quit IRC | 12:41 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table https://review.openstack.org/213174 | 12:42 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest https://review.openstack.org/209943 | 12:42 |
*** diga has quit IRC | 12:44 | |
*** yamamoto has joined #openstack-neutron | 12:44 | |
*** chlong has joined #openstack-neutron | 12:46 | |
HenryG | salv-orlando: can I trouble you for an IPAM code question? | 12:46 |
salv-orlando | sure | 12:46 |
HenryG | salv-orlando: https://review.openstack.org/#/c/212688/4/neutron/ipam/drivers/neutrondb_ipam/driver.py | 12:47 |
*** jpena|lunch is now known as jpena | 12:47 | |
*** garyk has joined #openstack-neutron | 12:49 | |
HenryG | salv-orlando: I guess my question is, is there a better way to get self._cidr to create_allocation_pools() than passing it through a bunch of classmethods? | 12:49 |
*** yamamoto has quit IRC | 12:49 | |
*** annegentle has joined #openstack-neutron | 12:50 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Validate router admin_state_up on upgrade to distributed https://review.openstack.org/214097 | 12:51 |
*** yamamoto has joined #openstack-neutron | 12:51 | |
*** yamamoto has quit IRC | 12:51 | |
*** yamamoto has joined #openstack-neutron | 12:52 | |
*** yamamoto has quit IRC | 12:52 | |
*** yamamoto has joined #openstack-neutron | 12:52 | |
*** dave-mccowan has joined #openstack-neutron | 12:52 | |
*** jistr is now known as jistr|mtg | 12:53 | |
*** vthapar has joined #openstack-neutron | 12:53 | |
salv-orlando | HenryG: moving those method to instance methods might not be thread safe. So you'd need also to ensure that every request get a different instance of the driver | 12:53 |
*** aix has quit IRC | 12:54 | |
*** yalie has quit IRC | 12:54 | |
*** tkatarki has joined #openstack-neutron | 12:55 | |
*** signed8bit is now known as signed8bit_ZZZzz | 12:55 | |
*** shausy has quit IRC | 12:56 | |
*** shausy has joined #openstack-neutron | 12:56 | |
*** emagana has quit IRC | 12:57 | |
*** liangbo has quit IRC | 12:57 | |
HenryG | salv-orlando: If there is a good reason for the classmethods, that's fine. I had not considered the threads aspect. | 12:57 |
*** shausy has quit IRC | 12:58 | |
ihrachyshka | ajo, ok, so in the end we merged a qos patch that broke tests. nice | 12:58 |
salv-orlando | HenryG: the reason mostly is not that classmehtod are important. You can use instance methods but then there will be other changes needed to avoid races | 12:59 |
*** pc_m has quit IRC | 12:59 | |
*** lpetrut has joined #openstack-neutron | 12:59 | |
*** pc_m has joined #openstack-neutron | 12:59 | |
*** aix has joined #openstack-neutron | 13:00 | |
irenab | pc_m: hey | 13:00 |
*** chlong has quit IRC | 13:00 | |
*** gongysh has joined #openstack-neutron | 13:00 | |
*** signed8bit_ZZZzz is now known as signed8bit | 13:00 | |
ajo | ihrachyshka, really? | 13:01 |
*** gongysh has quit IRC | 13:01 | |
ihrachyshka | ajo, well http://logs.openstack.org/49/214249/3/gate/gate-neutron-dsvm-api/6d486ff/testr_results.html.gz | 13:01 |
ajo | ihrachyshka, which one?, I saw your api-enabling one passing. | 13:01 |
ihrachyshka | ajo, that's the devstack plugin | 13:01 |
ihrachyshka | I guess the policy.json patch you pushed | 13:02 |
*** amuller has joined #openstack-neutron | 13:02 | |
ajo | ok, let me fix that | 13:02 |
ajo | I will depend on your qos-enable first | 13:02 |
ajo | and then we can put it in front via rebase | 13:02 |
ajo | ihrachyshka: sounds like a plan? | 13:02 |
ihrachyshka | ajo, yeap. please rebase the plugin on top of a fix | 13:03 |
ihrachyshka | then we'll retain votes there | 13:03 |
ajo | ack | 13:03 |
pc_m | irenab: hi | 13:03 |
ihrachyshka | ajo, tnx | 13:03 |
*** chlong has joined #openstack-neutron | 13:03 | |
ajo | ihrachyshka, you're welcome, sorry for the breakage.. | 13:04 |
*** tiny-hands has joined #openstack-neutron | 13:05 | |
tfukushima | apuimedo: I'm fixing the issues and adding unit tests. Although I'm experiencing some conflicts, I'll push them soon. | 13:06 |
irenab | pc_m: I have few questions regarding neutron-vpnaas. Would you be the right person to ask? | 13:06 |
apuimedo | tfukushima: thanks! | 13:06 |
*** amuller is now known as amuller_afk | 13:06 | |
pc_m | irenab: yes | 13:06 |
irenab | pc_m: I am looking into adding support for VPNaaS support for midonet which has its own L3 implementation without L3agents involved. | 13:08 |
irenab | pc_m: It looks like current implementation assumes L3 (vpn) agent is always deployed. Am I right? | 13:09 |
*** gardenshed has joined #openstack-neutron | 13:09 | |
pc_m | irenab: not necessarily. The reference implementation defines a VPN agent, which is a subclass of the L3 agent (so when VPN is enabled, you have L3). | 13:12 |
irenab | pc_m: I am looking at CSRDriver, but it looks like it also assumes agent is deployed | 13:12 |
pc_m | irenab: L3 agent? | 13:12 |
*** kbringard has joined #openstack-neutron | 13:12 | |
irenab | pc_m: vpn agent | 13:12 |
jschwarz | ajo, -1'd your agents patch | 13:13 |
jschwarz | ajo, it breaks fullstack | 13:13 |
ajo | jschwarz, lol, then jenkins -1'd it I hope :) | 13:13 |
jschwarz | ajo, fullstack is non-voting | 13:13 |
ajo | aah, well done then, I will check the fullstack log | 13:13 |
jschwarz | :) | 13:13 |
ajo | ajo-- | 13:13 |
ajo | ! | 13:13 |
ajo | jschwarz | 13:14 |
ajo | question about tox -eapi | 13:14 |
jschwarz | yes | 13:14 |
ajo | jschwarz: first time it started runnig, I CTRL+C'd | 13:14 |
ajo | now I try to start it... and it get's stuck at start | 13:14 |
jschwarz | ajo, erm. what does neutron-server say? | 13:15 |
ajo | jschwarz: seems quiet | 13:15 |
jschwarz | erm | 13:15 |
jschwarz | restart httpd? | 13:15 |
ajo | hmmm | 13:15 |
ajo | there's something wrong | 13:15 |
ajo | that tab in screen is locked.. | 13:15 |
ajo | :D | 13:15 |
pc_m | irenab: maybe there is some confusion with terms. | 13:16 |
ajo | jschwarz: you nailed it | 13:16 |
ajo | ;) | 13:16 |
irenab | pc_m: there is an RPC channel between server driver and device driver. device driver calls Csr API. Any idea why its not done at server side? | 13:16 |
jschwarz | ajo, :) | 13:16 |
ajo | neutron server was stuck somehow | 13:16 |
*** pc_m_ has joined #openstack-neutron | 13:16 | |
pc_m_ | irenab: Lost my connection for a min. | 13:16 |
pc_m_ | irenab: For VPN, there is a plugin for the service. A service driver is specified by the plugin, and one or more device drivers are loaded as well. | 13:17 |
*** vikram has quit IRC | 13:17 | |
pc_m_ | irenab: The drivers are paired, so for example the CSR service and device driver will work together to handle VPN requests. | 13:17 |
irenab | pc_m_: so for CSR, device driver is server side component? | 13:18 |
*** krtaylor has quit IRC | 13:18 | |
pc_m_ | irenab: The service driver is on server side. Called by plugin. Talks to device driver via RPC. | 13:19 |
ihrachyshka | ajo, should we finally get it in: https://review.openstack.org/#/c/182920/ ? | 13:19 |
ihrachyshka | what's your take? | 13:19 |
* ihrachyshka has not read the patch with due dilligence to do it himself | 13:20 | |
*** pc_m has quit IRC | 13:20 | |
ajo | ihrachyshka: IMHO yes, I'm glad to +A, but I'd like aarosen to come back and agree | 13:20 |
ajo | (or disagree) :) | 13:21 |
irenab | pc_m: if I understand correctly, device drivers are loaded by vpn agent. Am I right? vpn agent is subclass of L3 agent. | 13:21 |
pc_m_ | irenab: Yes, for reference drivers. | 13:21 |
*** lindycoder has joined #openstack-neutron | 13:21 | |
*** liangbo has joined #openstack-neutron | 13:21 | |
ihrachyshka | ajo, meh. well ok. | 13:22 |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: Sync FK constraints in db models with migration scripts https://review.openstack.org/215129 | 13:22 |
ajo | ihrachyshka; I believe the patch is good, fixes the issue, they reduced some unnecesary complexity about vlan id recovery... | 13:22 |
ajo | ihrachyshka: how about backport? :P :-) | 13:23 |
ajo | it's just a tiny patch (400L) :) | 13:23 |
openstackgerrit | sean mooney proposed openstack/neutron: Adds configurable agent type https://review.openstack.org/197210 | 13:23 |
ihrachyshka | ajo, nah, let's wait if you think aaron can stay disagreed | 13:23 |
openstackgerrit | sean mooney proposed openstack/neutron: Add config option to specify ovs datapath. https://review.openstack.org/197209 | 13:23 |
pc_m_ | irenab: IIRC, CSR does not use the L3 agent (the service is not enabled). | 13:23 |
ajo | (I'm joking) | 13:23 |
ihrachyshka | ajo, it's good for your health you do! | 13:23 |
pc_m_ | irenab: Trying to recall how the device drivers are loaded in that case. | 13:23 |
*** fawadkhaliq has joined #openstack-neutron | 13:24 | |
irenab | pc_m: So if I am looking for guidence to add new vpnaas driver which just need to propagate requests for vpnservice/connection to the Controller, I need to add service driver (server component), and device driver is not required, right? Or its still needed? I am not sure need the overhead of RPC | 13:25 |
*** neophy has joined #openstack-neutron | 13:26 | |
*** fawadkhaliq has quit IRC | 13:28 | |
irenab | pc_m: If you can share details about CSR integraton model, this can be very helpful. I think my case is similar. | 13:28 |
pc_m_ | irenab: In the reference drivers, there is one service driver, but may be multiple device drivers, if >1 host. | 13:28 |
*** vikram has joined #openstack-neutron | 13:29 | |
pc_m_ | irenab: So I don't know the answer to your question. Maybe if your controller can handle talking to different hosts to do the underlying work, then I guess maybe not. | 13:29 |
*** jamielennox is now known as jamielennox|away | 13:30 | |
pc_m_ | irenab: You can look at the Yvatta drivers. I think they use a controller, but am not too sure of how it is all set up. Sridhar R (not on right now) would know. | 13:30 |
*** krtaylor has joined #openstack-neutron | 13:31 | |
ajo | jschwarz: question, neutron.tests.api.test_qos.QosBandwidthLimitRuleTestJSON.test_rule_association_nonshared_policy shouldn't be doing both things as admin ? (policy creation, and rule creation?) | 13:31 |
irenab | pc_m_: More general question, do you see any potential problem if there is no device _driver at all? | 13:32 |
* jschwarz is looking | 13:32 | |
ajo | i see is_admin=False | 13:32 |
jschwarz | no | 13:32 |
ajo | jschwarz, ok, new policies disallow rule creation and policy creation for non-admins | 13:33 |
jschwarz | ajo, if create_bandwidth_limit_rule is done as admin, then it should see the policy even if the policy is not shared (because the user asking is admin and it ignores 'shared') | 13:33 |
*** nyechiel_ has joined #openstack-neutron | 13:33 | |
irenab | pc_m_: and thanks for the pointer, I will check Yvattta case | 13:33 |
pc_m_ | irenab: Not sure how you handle multiple nodes. | 13:33 |
Sam-I-Am | anyone here working on the ebtables anti-spoofing stuff? | 13:33 |
ajo | jschwarz: I guess some tests wouldn't apply in this new context, but I'm checking now how is it possible it's creating a policy as non-admin | 13:33 |
ajo | that fails here locally | 13:34 |
jschwarz | ajo, so... I guess 'shared=False' means it's supposed to be accessible only to admins | 13:34 |
Sam-I-Am | sc68cal: ping | 13:34 |
ajo | jschwarz: yep | 13:34 |
pc_m_ | irenab: service driver resides with plugin on Controller, where as agent(s)/device_drivers(s) are on network node(s). | 13:34 |
jschwarz | ajo, self.create_qos_policy is creating with an admin user | 13:34 |
irenab | pc_m_: by nodes, you mean the ones with Routers ? | 13:34 |
ajo | ahh | 13:34 |
pc_m_ | irenab: hosts | 13:34 |
ajo | ok, that explains the thing | 13:34 |
ajo | so, create is alwais done as admin | 13:34 |
ajo | always | 13:34 |
jschwarz | ajo, no | 13:34 |
ajo | ok, we cannot create rules as non admins now | 13:35 |
jschwarz | ajo, specifically self.create_qos_policy | 13:35 |
ajo | or policies | 13:35 |
jschwarz | ajo, and self.create_qos_bandwidth_limit_rules | 13:35 |
jschwarz | ajo, have a look at neutron/tests/api/base.py line 445 | 13:35 |
ajo | ok, that's not clear when reading the tests | 13:35 |
ajo | ack | 13:35 |
*** liangbo has quit IRC | 13:35 | |
*** dims_ has joined #openstack-neutron | 13:35 | |
irenab | pc_m_: in my case Router is a distributed entity, so its up to Controller to handle it. | 13:35 |
*** liangbo has joined #openstack-neutron | 13:36 | |
pc_m_ | irenab: So if multiple Network nodes, each with routers, there would be agent/device driver on each to handle VPN. | 13:36 |
*** amuller_afk is now known as amuller | 13:36 | |
*** prad_ has joined #openstack-neutron | 13:36 | |
jschwarz | ajo, the API client we're using atm is pretty implicit | 13:36 |
*** liangbo has quit IRC | 13:36 | |
*** clev-away is now known as clev | 13:36 | |
jschwarz | I was thinking of revamping it but that's a lot of work | 13:36 |
*** zzzeek has joined #openstack-neutron | 13:36 | |
irenab | pc_m_: got it, I think I should look into integration with server side component only. Thanks a lot for your help. | 13:37 |
ajo | jschwarz: but bandwidth limit rule create ... seems to be admin too | 13:37 |
ajo | isn't it? | 13:37 |
pc_m_ | irenab: So I guess there is an L3 agent running. I guess with CSR it talks with with their L3Router plugin, instead of the q-l3 service (I know for sure there is no q-l3 servcie running, when using CSR) | 13:38 |
jschwarz | ajo, yes | 13:38 |
jschwarz | ajo, is that a problem? | 13:38 |
ajo | jschwarz, the policy check says it's not | 13:38 |
jschwarz | ajo, link? | 13:38 |
ajo | http://fpaste.org/257148/77923144/ | 13:38 |
ajo | look for is_admin | 13:38 |
pc_m_ | irenab: yw. Afraid I don't know much about how to connect with a separate controller... hopefully some useful info though. | 13:38 |
*** dims has quit IRC | 13:39 | |
*** tiswanso has joined #openstack-neutron | 13:40 | |
ajo | jschwarz: but create policy works... | 13:40 |
jschwarz | ajo, hold on, was multi tasking | 13:40 |
jschwarz | ajo, looking now | 13:40 |
jschwarz | ajo, I need some context - what are you trying to do? | 13:40 |
ajo | jschwarz: we introduced the final policies.json | 13:41 |
ajo | in this patch: | 13:41 |
ajo | https://review.openstack.org/#/c/214030/ | 13:41 |
ajo | but now, | 13:41 |
* jschwarz when did that happen? :O | 13:41 | |
ajo | https://review.openstack.org/#/c/214249/ api tests fail here | 13:41 |
jschwarz | OK now I understand | 13:41 |
amuller | jschwarz: how's the local_ip fix coming along? | 13:42 |
jschwarz | amuller, it's not... will do it shortly | 13:42 |
jschwarz | sorry :< | 13:42 |
ihrachyshka | I told you guys we should keep the order | 13:42 |
jschwarz | ihrachyshka, everything is in order | 13:42 |
ihrachyshka | :) | 13:42 |
jschwarz | I was out of touch for a bit ;-) | 13:42 |
ajo | jschwarz: he's telling me ;) | 13:42 |
jschwarz | ah | 13:43 |
ihrachyshka | yeah, I brag a bit about us missing opportunity to get api tests on today ;) | 13:43 |
jschwarz | ajo, so according to the code it IS creating it as admin | 13:43 |
*** gildub has quit IRC | 13:44 | |
ajo | jschwarz, but policy says it's not | 13:44 |
ajo | and test_rule_create works | 13:44 |
ajo | no exception thrown... | 13:44 |
* ajo doesn't understand | 13:44 | |
*** dims_ has quit IRC | 13:44 | |
ajo | jschwarz, no worries | 13:44 |
ajo | I will debug what's going on | 13:44 |
ajo | jschwarz: go back to the local_ip thing ;) | 13:44 |
jschwarz | ajo, the gate is clogged anyway ;-( | 13:45 |
ajo | jschwarz: I can reproduce locally. | 13:45 |
jschwarz | ajo, where is the exception thrown from on the server's side? | 13:45 |
ajo | jschwarz: yep | 13:46 |
*** dims has joined #openstack-neutron | 13:46 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Clear the extension requirement https://review.openstack.org/214981 | 13:46 |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific agent when update floatingip https://review.openstack.org/215136 | 13:46 |
ajo | I wonder if creating the policy with tenant_id=.... in that specific test breaks the admin client | 13:46 |
*** julim has joined #openstack-neutron | 13:46 | |
ajo | is the only difference to other tests | 13:46 |
jschwarz | ajo, that might be | 13:46 |
*** signed8bit is now known as signed8bit_ZZZzz | 13:46 | |
jschwarz | ohhhhhhhhhh | 13:46 |
jschwarz | ajo, look | 13:46 |
* ajo smiles | 13:46 | |
*** prad_ is now known as pradk | 13:47 | |
jschwarz | ajo, it's running "self.client.create_bandwidth_limit_rule" | 13:47 |
jschwarz | ajo, note the "self.client.XXX" | 13:47 |
ajo | ahhh | 13:47 |
ajo | crap :D | 13:47 |
jschwarz | it's not using the self.admin_client | 13:47 |
ajo | ok, that's it | 13:47 |
jschwarz | so it's in user mode | 13:47 |
jschwarz | it SHOULD be throwing 403 :) | 13:47 |
ajo | ok, I will remove that test it doesn't make sense anymore with the default policies :) | 13:47 |
jschwarz | glad to be of service | 13:47 |
jschwarz | onward, to local_ip! | 13:47 |
ajo | I will revamp into two: tenant can't create policies, tenant can't create rules.. | 13:47 |
*** fawadkhaliq has joined #openstack-neutron | 13:48 | |
jschwarz | ajo, make sure to add me as a reviewer to that patch that has the change and I'll have a look | 13:48 |
*** pal has joined #openstack-neutron | 13:48 | |
ajo | jschwarz, about idempotent_id's | 13:49 |
*** signed8bit_ZZZzz is now known as signed8bit | 13:49 | |
*** rpothier has joined #openstack-neutron | 13:49 | |
ajo | should I create random ones each time I write a new test? | 13:49 |
jschwarz | ajo, yes | 13:49 |
ajo | ack | 13:49 |
*** nyechiel_ has quit IRC | 13:49 | |
*** signed8bit is now known as signed8bit_ZZZzz | 13:50 | |
openstackgerrit | Marga Millet proposed openstack/neutron: Support dhcp metadata service for all networks https://review.openstack.org/211963 | 13:51 |
openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 13:51 |
*** neophy has quit IRC | 13:52 | |
jlibosva | lifeless: hi - can you please revisit https://review.openstack.org/#/c/180587/6 ? You asked for Doug's input and it's been a while since his reply | 13:53 |
*** piyanai has quit IRC | 13:53 | |
*** neiljerram has quit IRC | 13:53 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific agent when update floatingip https://review.openstack.org/215136 | 13:54 |
openstackgerrit | Henry Gessau proposed openstack/neutron: Do not query reservations table when counting resources https://review.openstack.org/214602 | 13:56 |
*** pal has quit IRC | 13:56 | |
*** njohnston is now known as nate_gone | 13:56 | |
*** regXboi has joined #openstack-neutron | 13:56 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line https://review.openstack.org/214961 | 13:56 |
*** mohankumar has quit IRC | 13:56 | |
*** obondarev_ has joined #openstack-neutron | 13:56 | |
*** tfukushima has quit IRC | 13:57 | |
openstackgerrit | gongysh proposed openstack/python-neutronclient: Add extension name to extension's command help text line https://review.openstack.org/214961 | 13:57 |
*** pal has joined #openstack-neutron | 13:57 | |
*** sbelous has quit IRC | 13:58 | |
*** sbelous has joined #openstack-neutron | 13:58 | |
*** nate_gone is now known as njohnston | 13:58 | |
*** obondarev has quit IRC | 13:59 | |
*** eezhova has quit IRC | 13:59 | |
*** skraynev has quit IRC | 13:59 | |
ihrachyshka | ajo, I see you avoid +2'ing qos patches. I wonder wy | 13:59 |
ihrachyshka | *why | 13:59 |
*** eezhova has joined #openstack-neutron | 13:59 | |
*** skraynev has joined #openstack-neutron | 14:00 | |
ajo | ihrachyshka: db is not in my area of expertise, | 14:00 |
ajo | ok, I have some expertise | 14:00 |
ajo | but not supossedly in neutron :) | 14:00 |
*** Guest95328 is now known as dansmith | 14:00 | |
ihrachyshka | ajo, well then I wonder if you are not capable of reviewing qos stuff then who is | 14:00 |
ajo | it's in a fine line, I guess I'm an expert in the qos service, I know quite a bit about DBs, but for example , I didn't know that we recreated models from sqlalchemy models during testing... | 14:01 |
ajo | I guess I should raise the point with mestery, it felt right to me | 14:01 |
ajo | but I didn't want to step on the bad side ;) | 14:01 |
ihrachyshka | ajo, I guess few people knew it. well, you decide, but I just wondered. | 14:01 |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted https://review.openstack.org/214215 | 14:01 |
ihrachyshka | it's all about trust man! everyone trusts you know qos! :) | 14:02 |
jlibosva | akamyshnikova: Thanks for testing on postgres! I didn't do that :) | 14:03 |
*** aix has quit IRC | 14:04 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 14:04 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes https://review.openstack.org/215138 | 14:04 |
*** lukasa has quit IRC | 14:04 | |
ajo | ihrachyshka, there it goes ^ | 14:04 |
ajo | jschwarz ^ | 14:05 |
jschwarz | ajo, ack | 14:05 |
*** signed8bit_ZZZzz has quit IRC | 14:06 | |
akamyshnikova | jlibosva, np :) | 14:07 |
*** afazekas has joined #openstack-neutron | 14:07 | |
*** anilvenkata has quit IRC | 14:08 | |
*** jistr|mtg is now known as jistr | 14:08 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling https://review.openstack.org/215142 | 14:09 |
jschwarz | amuller, ^ | 14:09 |
*** aix has joined #openstack-neutron | 14:09 | |
amuller | jschwarz: are we aiming for a PS1 merge? | 14:09 |
amuller | jschwarz: aww, no bug | 14:09 |
jschwarz | amuller, my code is always perfect on first try ;-) | 14:10 |
amuller | jschwarz: better luck next time =D | 14:10 |
jschwarz | amuller, bug? | 14:10 |
amuller | jschwarz: yeah, need a bug report | 14:10 |
jschwarz | amuller, peh. | 14:10 |
jschwarz | amuller, this is a commit-message change | 14:10 |
amuller | jschwarz: still counts | 14:10 |
jschwarz | amuller, your will shall be done, my sire | 14:11 |
ihrachyshka | ajo++ | 14:12 |
*** ajmiller has joined #openstack-neutron | 14:13 | |
*** lyanchih has joined #openstack-neutron | 14:13 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling https://review.openstack.org/215142 | 14:14 |
jschwarz | amuller, ^ | 14:14 |
*** signed8bit has joined #openstack-neutron | 14:14 | |
*** kiran-r has joined #openstack-neutron | 14:14 | |
*** kiran-r has quit IRC | 14:15 | |
*** garyk has quit IRC | 14:15 | |
*** tkatarki has quit IRC | 14:16 | |
*** yamamoto has quit IRC | 14:17 | |
*** yamamoto has joined #openstack-neutron | 14:19 | |
*** yamamoto has quit IRC | 14:19 | |
*** yamamoto has joined #openstack-neutron | 14:19 | |
amuller | jschwarz: thanks I'll review after meetings | 14:20 |
jschwarz | hopefully | 14:20 |
*** dmartls1 has joined #openstack-neutron | 14:20 | |
ihrachyshka | ajo, some nits | 14:21 |
regXboi | Swami: ping | 14:23 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes https://review.openstack.org/215138 | 14:24 |
*** drankis has joined #openstack-neutron | 14:25 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 14:25 |
*** dsneddon is now known as dsneddon_afk | 14:26 | |
*** tkatarki has joined #openstack-neutron | 14:26 | |
*** pal_ has joined #openstack-neutron | 14:27 | |
*** pal has quit IRC | 14:27 | |
*** vishwanathj has joined #openstack-neutron | 14:27 | |
*** sayali has quit IRC | 14:27 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table https://review.openstack.org/213174 | 14:27 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest https://review.openstack.org/209943 | 14:27 |
*** tidwellr has joined #openstack-neutron | 14:28 | |
jlibosva | ajo: does https://review.openstack.org/#/c/215138/ fix the recent qos api failures we saw on the gate? I'm not sure if there was a bug reported | 14:28 |
jschwarz | ajo, so shouldn't https://review.openstack.org/#/c/214249/ fail at the gate? | 14:29 |
jschwarz | ajo, because the api test will fail? | 14:30 |
*** banix has joined #openstack-neutron | 14:30 | |
*** fawadkhaliq has quit IRC | 14:33 | |
*** shwetaap has joined #openstack-neutron | 14:36 | |
amuller | jschwarz: can you respin https://review.openstack.org/#/c/215142/ and I'll +2? | 14:36 |
jschwarz | amuller, I will in a few | 14:36 |
jschwarz | amuller, in the middle of some heavy-weight DVR debugging ;-) | 14:36 |
amuller | jschwarz: ok talk to you in a month then | 14:37 |
*** lpetrut has quit IRC | 14:37 | |
jschwarz | amuller, lol | 14:37 |
jschwarz | amuller, actually in 23 minutes | 14:37 |
*** lpetrut has joined #openstack-neutron | 14:38 | |
openstackgerrit | Vlad Gridin proposed openstack/neutron: This changeset removes ML2 Nuage driver vendor code, currently present in neutron core to vendor repo. https://review.openstack.org/215155 | 14:39 |
openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [OpenSwan] Enable usage of the MTU value of an IPSec connection https://review.openstack.org/206539 | 14:40 |
clarkb | 14:42 | |
clarkb | whoops | 14:42 |
*** signed8bit has quit IRC | 14:43 | |
*** signed8bit has joined #openstack-neutron | 14:44 | |
openstackgerrit | vikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce dragent scheduler model https://review.openstack.org/207607 | 14:47 |
openstackgerrit | John Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling https://review.openstack.org/215142 | 14:48 |
jschwarz | amuller, jlibosva, ^ | 14:48 |
*** pal_ has quit IRC | 14:48 | |
*** vikram has quit IRC | 14:48 | |
*** piyanai has joined #openstack-neutron | 14:49 | |
*** john-davidge has quit IRC | 14:50 | |
*** john-davidge has joined #openstack-neutron | 14:51 | |
amuller | yamamoto: heya | 14:51 |
amuller | yamamoto: wanna check out https://review.openstack.org/#/c/215142/ ? | 14:51 |
jlibosva | jschwarz: yet another questions :) | 14:51 |
*** neiljerram has joined #openstack-neutron | 14:52 | |
jschwarz | jlibosva, yet another question? :< | 14:52 |
yamamoto | amuller: sure | 14:52 |
jschwarz | jlibosva, meh you're really making me sad :< | 14:52 |
*** minwang2 has joined #openstack-neutron | 14:53 | |
jlibosva | jschwarz: it's a question(s)! | 14:53 |
*** SridharG has left #openstack-neutron | 14:54 | |
jschwarz | jlibosva, replied | 14:54 |
jschwarz | good questions :) | 14:54 |
*** signed8bit has quit IRC | 14:55 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific agent when update floatingip https://review.openstack.org/215136 | 14:55 |
*** signed8bit has joined #openstack-neutron | 14:55 | |
*** mlavalle has joined #openstack-neutron | 14:55 | |
*** garyk has joined #openstack-neutron | 14:56 | |
garyk | ihrachyshka: around? | 14:57 |
ihrachyshka | garyk, almost running away, but shoot | 14:57 |
garyk | what was the conclusion for the tagging? | 14:57 |
garyk | i have a few more backports and then i can do the tag | 14:57 |
*** annegentle has quit IRC | 14:58 | |
*** cbouch has quit IRC | 14:58 | |
*** annegentle has joined #openstack-neutron | 14:58 | |
openstackgerrit | Michael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters https://review.openstack.org/213303 | 14:59 |
openstackgerrit | Michael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension https://review.openstack.org/203842 | 14:59 |
ihrachyshka | garyk, I don't think we had time to discuss it the previous meeting | 14:59 |
ihrachyshka | garyk, I believe we need to follow up with mestery though he is pretty unavailable these week | 14:59 |
*** itzikb has quit IRC | 15:01 | |
*** ddepaoli has quit IRC | 15:01 | |
ihrachyshka | garyk, ok, I need to run now. I'll be avail in an hour | 15:01 |
*** alex_klimov has quit IRC | 15:01 | |
garyk | ihrachyshka: ok, np. guess it is not that urgent - lets talk tomorrow - have a good evening | 15:02 |
*** vikram_choudhary has joined #openstack-neutron | 15:02 | |
openstackgerrit | Michael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters https://review.openstack.org/213303 | 15:03 |
openstackgerrit | Michael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension https://review.openstack.org/203842 | 15:03 |
openstackgerrit | Pengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db https://review.openstack.org/213563 | 15:04 |
*** shoutm has joined #openstack-neutron | 15:05 | |
*** fawadkhaliq has joined #openstack-neutron | 15:06 | |
gridinv | HenryG: wrt https://review.openstack.org/215155, entry point should be removed from setup.cfg and move to vendor repo? | 15:06 |
*** l6unchpad has quit IRC | 15:07 | |
HenryG | gridinv: Yep, you got it | 15:07 |
gridinv | HenryG: thanks | 15:07 |
*** signed8bit has quit IRC | 15:08 | |
*** alejandrito has joined #openstack-neutron | 15:08 | |
*** pal has joined #openstack-neutron | 15:09 | |
openstackgerrit | John Schwarz proposed openstack/neutron: Only validate local_ip if using tunneling https://review.openstack.org/215142 | 15:10 |
amuller | yamamoto: https://review.openstack.org/#/c/215142/ | 15:10 |
*** ihrachyshka has quit IRC | 15:11 | |
yamamoto | amuller: lgtm | 15:13 |
*** ronis__ has quit IRC | 15:13 | |
openstackgerrit | Vlad Gridin proposed openstack/neutron: This changeset removes ML2 Nuage driver vendor code, currently present in neutron core to vendor repo. https://review.openstack.org/215155 | 15:13 |
amuller | yamamoto: thanks | 15:14 |
*** jbell8 has joined #openstack-neutron | 15:16 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific agent when update floatingip https://review.openstack.org/215136 | 15:16 |
amuller | jschwarz: https://review.openstack.org/#/c/215055/ | 15:16 |
*** anshul has quit IRC | 15:17 | |
*** gardenshed has quit IRC | 15:17 | |
*** vthapar has quit IRC | 15:17 | |
*** kfox1111 has quit IRC | 15:19 | |
*** shoutm has quit IRC | 15:20 | |
*** dannywilson has joined #openstack-neutron | 15:21 | |
*** shaohe_feng has joined #openstack-neutron | 15:21 | |
*** jistr is now known as jistr|mtg | 15:22 | |
*** pal has quit IRC | 15:24 | |
*** gberginc has joined #openstack-neutron | 15:24 | |
*** pal has joined #openstack-neutron | 15:24 | |
openstackgerrit | Pengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db https://review.openstack.org/213563 | 15:25 |
HenryG | gridinv: ping | 15:25 |
*** moshele has quit IRC | 15:28 | |
*** scheuran has quit IRC | 15:28 | |
*** Swami has joined #openstack-neutron | 15:31 | |
*** piyanai has quit IRC | 15:33 | |
*** piyanai has joined #openstack-neutron | 15:34 | |
*** pal has quit IRC | 15:35 | |
*** r-daneel has joined #openstack-neutron | 15:36 | |
*** TrevorV has joined #openstack-neutron | 15:37 | |
*** TrevorV has quit IRC | 15:41 | |
*** TrevorV has joined #openstack-neutron | 15:43 | |
*** armax has joined #openstack-neutron | 15:44 | |
*** mickeys has joined #openstack-neutron | 15:46 | |
openstackgerrit | Merged openstack/neutron: Add logging to debug oslo.messaging failure https://review.openstack.org/213360 | 15:48 |
*** kbringard has quit IRC | 15:48 | |
openstackgerrit | Pengtao Huang proposed openstack/neutron: ignore the repeated value of service_plugins https://review.openstack.org/212597 | 15:49 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 15:49 |
*** vikram_choudhary has quit IRC | 15:49 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 15:49 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 15:49 |
*** mwagner_lap has quit IRC | 15:50 | |
*** crose has joined #openstack-neutron | 15:51 | |
*** salv-orlando has quit IRC | 15:52 | |
*** vikram_ has joined #openstack-neutron | 15:52 | |
*** neophy has joined #openstack-neutron | 15:53 | |
*** hogepodge has quit IRC | 15:53 | |
*** mkolesni_ has joined #openstack-neutron | 15:54 | |
openstackgerrit | Zhi Chang proposed openstack/neutron: Notify specific agent when update floatingip https://review.openstack.org/215136 | 15:55 |
*** devvesa has quit IRC | 15:56 | |
*** liudong has quit IRC | 15:56 | |
*** armax has quit IRC | 15:56 | |
*** redbeard has quit IRC | 15:56 | |
*** AlexeyElagin has joined #openstack-neutron | 15:57 | |
*** lazy_prince has quit IRC | 15:59 | |
*** salv-orlando has joined #openstack-neutron | 15:59 | |
*** tflynn has joined #openstack-neutron | 16:01 | |
openstackgerrit | Paul Michali proposed openstack/neutron: TEST COMMIT - DO NOT REVIEW/APPROVE https://review.openstack.org/215208 | 16:01 |
*** ihrachyshka has joined #openstack-neutron | 16:02 | |
*** azbiswas has joined #openstack-neutron | 16:02 | |
*** SourabhP has joined #openstack-neutron | 16:02 | |
*** arajagopal has joined #openstack-neutron | 16:05 | |
*** DWfuturetec has joined #openstack-neutron | 16:05 | |
*** SourabhP_ has joined #openstack-neutron | 16:05 | |
*** sdake has joined #openstack-neutron | 16:06 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron: fullstack: use migration scripts to create db schema https://review.openstack.org/215213 | 16:06 |
*** SourabhP has quit IRC | 16:08 | |
*** SourabhP_ is now known as SourabhP | 16:08 | |
*** jproulx has joined #openstack-neutron | 16:08 | |
*** yamahata has joined #openstack-neutron | 16:09 | |
*** piyanai has quit IRC | 16:09 | |
*** jistr|mtg is now known as jistr | 16:10 | |
*** crose has quit IRC | 16:10 | |
*** kbringard has joined #openstack-neutron | 16:11 | |
openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 16:12 |
jproulx | Hi All having trouble with production DB migration (juno->kilo) & of course didn't happen in test with same DB content | 16:12 |
jproulx | Basically this part of a table creation is failing: | 16:12 |
jproulx | ALTER TABLE nsxv_internal_networks ADD FOREIGN KEY(network_id) REFERENCES networks (id) ON DELETE CASCADE; | 16:12 |
jproulx | seems like trying to reference a nonexistant foreign key but the 'networks' table definitely has an 'id' field, help? | 16:12 |
*** tflynn_ has joined #openstack-neutron | 16:12 | |
*** ihrachyshka has quit IRC | 16:13 | |
*** _cjones_ has joined #openstack-neutron | 16:13 | |
*** piyanai has joined #openstack-neutron | 16:13 | |
*** shaohe_feng has quit IRC | 16:13 | |
*** SourabhP has quit IRC | 16:15 | |
*** tflynn has quit IRC | 16:15 | |
*** paulose has joined #openstack-neutron | 16:15 | |
*** minwang2 has quit IRC | 16:16 | |
*** piyanai has quit IRC | 16:18 | |
*** krtaylor has quit IRC | 16:18 | |
*** dsneddon_afk is now known as dsneddon | 16:19 | |
*** moshele has joined #openstack-neutron | 16:20 | |
*** kiran-r has joined #openstack-neutron | 16:20 | |
*** Marga_ has quit IRC | 16:22 | |
*** hogepodge has joined #openstack-neutron | 16:23 | |
*** gberginc has quit IRC | 16:23 | |
*** ihrachyshka has joined #openstack-neutron | 16:25 | |
*** kawa2014 has quit IRC | 16:25 | |
*** rossella_s has quit IRC | 16:25 | |
*** kbringard has quit IRC | 16:26 | |
*** rossella_s has joined #openstack-neutron | 16:26 | |
*** lpetrut has quit IRC | 16:26 | |
*** crose has joined #openstack-neutron | 16:26 | |
*** kawa2014 has joined #openstack-neutron | 16:26 | |
*** vivek-ebay has joined #openstack-neutron | 16:27 | |
*** mwagner_lap has joined #openstack-neutron | 16:28 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Fix qos api-tests after policy changes https://review.openstack.org/215138 | 16:29 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 16:29 |
*** kbringard has joined #openstack-neutron | 16:29 | |
ajo | jschwarz ^ :) | 16:29 |
*** Marga_ has joined #openstack-neutron | 16:29 | |
*** klindgren has joined #openstack-neutron | 16:29 | |
ajo | un-nitted ;) | 16:29 |
*** piyanai has joined #openstack-neutron | 16:29 | |
*** nlahouti has joined #openstack-neutron | 16:30 | |
*** sdake_ has joined #openstack-neutron | 16:30 | |
*** salv-orlando has quit IRC | 16:30 | |
*** _kiran_ has joined #openstack-neutron | 16:30 | |
*** kiran-r has quit IRC | 16:31 | |
*** pradk has quit IRC | 16:31 | |
*** gampel has quit IRC | 16:32 | |
*** jckasper has quit IRC | 16:32 | |
jschwarz | ajo, it wasn't a nit though | 16:32 |
jschwarz | it was a needed test ;-) | 16:32 |
*** matrohon has quit IRC | 16:32 | |
*** cdl has joined #openstack-neutron | 16:32 | |
*** jckasper has joined #openstack-neutron | 16:33 | |
*** vikram_ has quit IRC | 16:33 | |
*** sdake has quit IRC | 16:33 | |
*** julim has quit IRC | 16:34 | |
*** piyanai has quit IRC | 16:34 | |
*** klindgren_ has joined #openstack-neutron | 16:34 | |
*** ivar-lazzaro has joined #openstack-neutron | 16:34 | |
*** sbelous_ has joined #openstack-neutron | 16:34 | |
*** cdl has left #openstack-neutron | 16:34 | |
*** sdake_ is now known as sdake | 16:34 | |
*** numan has quit IRC | 16:35 | |
*** SridharGaddam has quit IRC | 16:35 | |
*** klindgren has quit IRC | 16:35 | |
*** SridharG has joined #openstack-neutron | 16:36 | |
*** numan has joined #openstack-neutron | 16:36 | |
*** piyanai has joined #openstack-neutron | 16:37 | |
*** shwetaap has quit IRC | 16:37 | |
*** fawadk has joined #openstack-neutron | 16:37 | |
*** gangil has joined #openstack-neutron | 16:37 | |
*** fawadkhaliq has quit IRC | 16:37 | |
*** jckasper has quit IRC | 16:38 | |
*** diga has joined #openstack-neutron | 16:39 | |
haleyb | neiljerram: i'm not avoiding you, i'm like and old mainframe and have a bad time-slicing algorithm | 16:39 |
*** numan has quit IRC | 16:39 | |
*** vikram_ has joined #openstack-neutron | 16:41 | |
*** armax has joined #openstack-neutron | 16:43 | |
*** drankis has quit IRC | 16:43 | |
*** ivar-laz_ has joined #openstack-neutron | 16:45 | |
*** clev is now known as clev-away | 16:45 | |
*** gberginc has joined #openstack-neutron | 16:45 | |
openstackgerrit | Sergey Belous proposed openstack/neutron: Adds base in-tree functional testing of the dhcp agent (OVS) https://review.openstack.org/136834 | 16:45 |
*** DWfuturetec has quit IRC | 16:46 | |
*** jistr has quit IRC | 16:46 | |
*** pradk has joined #openstack-neutron | 16:46 | |
*** ivar-lazzaro has quit IRC | 16:46 | |
openstackgerrit | Merged openstack/neutron: Add a fullstack fake VM, basic connectivity test https://review.openstack.org/191197 | 16:47 |
sbelous_ | amuller, ajo, hi, guys, can you take a look at this patch: https://review.openstack.org/#/c/136834/ please? I updated functional tests of dhcp-agent and solved the problem with overriding system resolv.conf | 16:48 |
ajo | sbelous_!!, nice :) | 16:48 |
ajo | on mtg, but I will look as soon as we finish | 16:48 |
*** gangil has quit IRC | 16:49 | |
*** piyanai has quit IRC | 16:49 | |
sbelous_ | ajo, thanks :) | 16:49 |
*** tflynn_ has quit IRC | 16:49 | |
*** tflynn has joined #openstack-neutron | 16:50 | |
*** Kennan2 has quit IRC | 16:50 | |
ajo | sbelous_: niiice ;) it looks good, so it will take the netns resolv.conf if existing | 16:51 |
*** kawa2014 has quit IRC | 16:51 | |
*** kawa2014 has joined #openstack-neutron | 16:51 | |
sbelous_ | ajo, yep. I think, that it's good way | 16:52 |
*** park_ has quit IRC | 16:52 | |
sbelous_ | but it's requires strict filters settings | 16:53 |
*** minwang2 has joined #openstack-neutron | 16:53 | |
*** shwetaap has joined #openstack-neutron | 16:53 | |
*** piyanai has joined #openstack-neutron | 16:54 | |
*** SourabhP has joined #openstack-neutron | 16:55 | |
*** kbringard has quit IRC | 16:56 | |
*** piyanai has quit IRC | 16:56 | |
diga | apuimedo: irenab : when do we have container networking meeting ? | 16:57 |
paulose | is vxlan in openvswitch implemented in userspace | 16:57 |
*** armax has quit IRC | 16:57 | |
*** gangil has joined #openstack-neutron | 16:58 | |
*** piyanai has joined #openstack-neutron | 16:59 | |
sc68cal | paulose: https://github.com/openvswitch/ovs#what-is-open-vswitch | 16:59 |
*** tkatarki has quit IRC | 17:00 | |
sc68cal | paulose: last paragraph discusses userspace vs kernel module | 17:00 |
jlibosva | ajo: ihrachyshka https://review.openstack.org/#/c/215138/3/neutron/tests/api/test_qos.py - sorry I'm dumb, does it mean regular tenant cannot create qos bw rules? | 17:00 |
*** kawa2014 has quit IRC | 17:02 | |
*** armax has joined #openstack-neutron | 17:02 | |
*** afazekas has quit IRC | 17:02 | |
*** sbelous_ has quit IRC | 17:03 | |
*** mohankumar has joined #openstack-neutron | 17:03 | |
*** mkolesni_ has quit IRC | 17:03 | |
*** sbelous_ has joined #openstack-neutron | 17:03 | |
*** padkrish has joined #openstack-neutron | 17:04 | |
ihrachyshka | jlibosva, by default, I think it should not be allowed to create neither policies nor rules in them | 17:04 |
ihrachyshka | just attach, if it's shared | 17:04 |
*** sbelous_ has quit IRC | 17:05 | |
*** achanda has joined #openstack-neutron | 17:05 | |
*** jschwarz has quit IRC | 17:05 | |
*** Kennan has joined #openstack-neutron | 17:05 | |
*** diga has quit IRC | 17:06 | |
ihrachyshka | amuller, https://review.openstack.org/#/c/211166/2/neutron/api/rpc/handlers/l3_rpc.py line 162: is there space for a race there? let's say we got active, it's none, so we set a random one, but the latter hits db after master was elected. | 17:06 |
jlibosva | ihrachyshka: aha, thanks for explanation | 17:06 |
*** park_ has joined #openstack-neutron | 17:06 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/215086 | 17:07 |
*** SourabhP has quit IRC | 17:08 | |
*** abhiraut has joined #openstack-neutron | 17:09 | |
*** jlibosva has quit IRC | 17:10 | |
*** kbringard has joined #openstack-neutron | 17:10 | |
*** DWfuturetec has joined #openstack-neutron | 17:12 | |
paulose | sc68cal since there is standard port for vxlan won't that be listening in userspace? | 17:12 |
*** piyanai has quit IRC | 17:13 | |
*** piyanai has joined #openstack-neutron | 17:13 | |
sc68cal | paulose: I am not very familiar with the userspace implementation - this is a question that is probably meant for #openvswitch or ovs-discuss | 17:14 |
*** Kennan has quit IRC | 17:14 | |
*** park_ has quit IRC | 17:14 | |
*** absubram has joined #openstack-neutron | 17:14 | |
*** jpena has quit IRC | 17:15 | |
*** lyanchih has quit IRC | 17:16 | |
*** ivar-la__ has joined #openstack-neutron | 17:16 | |
paulose | sc68cal i dont have any experience using vxlan with neutron,i believe in GRE implementation there wasn't any user space | 17:17 |
*** s3wong has joined #openstack-neutron | 17:17 | |
*** crose has quit IRC | 17:17 | |
*** ivar-laz_ has quit IRC | 17:17 | |
sc68cal | paulose: I believe that most distros install the openvswitch kernel module | 17:18 |
*** ronis__ has joined #openstack-neutron | 17:19 | |
*** Kennan has joined #openstack-neutron | 17:20 | |
*** park_ has joined #openstack-neutron | 17:20 | |
*** julim has joined #openstack-neutron | 17:20 | |
*** aix has quit IRC | 17:22 | |
*** tkatarki has joined #openstack-neutron | 17:22 | |
*** neophy has quit IRC | 17:22 | |
*** Kennan has quit IRC | 17:24 | |
*** park_ has quit IRC | 17:25 | |
*** madhu_ak has joined #openstack-neutron | 17:26 | |
*** vikram_ has quit IRC | 17:26 | |
*** signed8bit has joined #openstack-neutron | 17:26 | |
*** achanda has quit IRC | 17:27 | |
*** amuller is now known as amuller_afk | 17:27 | |
*** tfukushima has joined #openstack-neutron | 17:31 | |
*** ihrachyshka has quit IRC | 17:31 | |
paulose | sc68cal do you have a running neutron setup with vxlan network_type? | 17:31 |
*** piyanai has quit IRC | 17:32 | |
sc68cal | paulose: why do you ask? | 17:33 |
*** yamahata has quit IRC | 17:34 | |
*** smccully has joined #openstack-neutron | 17:34 | |
*** neophy has joined #openstack-neutron | 17:36 | |
*** yamamoto has quit IRC | 17:36 | |
*** park_ has joined #openstack-neutron | 17:37 | |
paulose | sc68cal never mind i got the answer from #openvswitch | 17:39 |
*** jproulx has left #openstack-neutron | 17:39 | |
*** piyanai has joined #openstack-neutron | 17:39 | |
*** john-davidge has quit IRC | 17:40 | |
*** gardenshed has joined #openstack-neutron | 17:40 | |
*** Kennan has joined #openstack-neutron | 17:40 | |
*** saksham has joined #openstack-neutron | 17:40 | |
*** piyanai has quit IRC | 17:41 | |
*** salv-orlando has joined #openstack-neutron | 17:42 | |
*** DWfuturetec has quit IRC | 17:42 | |
*** emagana has joined #openstack-neutron | 17:43 | |
*** shivrao has joined #openstack-neutron | 17:43 | |
*** salv-orlando has quit IRC | 17:46 | |
*** krtaylor has joined #openstack-neutron | 17:47 | |
*** tidwellr has left #openstack-neutron | 17:47 | |
*** clev-away is now known as clev | 17:48 | |
*** DWfuturetec has joined #openstack-neutron | 17:48 | |
*** sridhar_ram has joined #openstack-neutron | 17:48 | |
*** pradk has quit IRC | 17:48 | |
*** bharath has joined #openstack-neutron | 17:48 | |
*** SumitNaiksatam has joined #openstack-neutron | 17:48 | |
*** prad_ has joined #openstack-neutron | 17:49 | |
*** _kiran_ has quit IRC | 17:50 | |
*** padkrish has quit IRC | 17:51 | |
*** breitz has quit IRC | 17:52 | |
*** breitz has joined #openstack-neutron | 17:52 | |
*** tflynn has quit IRC | 17:52 | |
*** nlahouti has quit IRC | 17:54 | |
*** achanda has joined #openstack-neutron | 17:55 | |
*** sthillma has joined #openstack-neutron | 17:56 | |
*** signed8bit is now known as signed8bit_ZZZzz | 17:56 | |
*** pc_m_ has quit IRC | 17:57 | |
*** jwy has joined #openstack-neutron | 17:57 | |
*** pc_m has joined #openstack-neutron | 17:57 | |
*** pc_m has quit IRC | 17:58 | |
*** pc_m has joined #openstack-neutron | 17:59 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove VIF_TYPES constant https://review.openstack.org/213905 | 18:00 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor VIF_TYPE_* constants https://review.openstack.org/214805 | 18:00 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants https://review.openstack.org/214806 | 18:00 |
*** piyanai has joined #openstack-neutron | 18:00 | |
*** gardenshed has quit IRC | 18:01 | |
*** gberginc has quit IRC | 18:01 | |
openstackgerrit | James Arendt proposed openstack/python-neutronclient: WIP: Add Neutron flavor framework CLI https://review.openstack.org/214776 | 18:02 |
*** arajagopal has quit IRC | 18:03 | |
*** DWfuturetec has quit IRC | 18:03 | |
*** prad_ is now known as pradk | 18:04 | |
*** padkrish has joined #openstack-neutron | 18:04 | |
*** arajagopal has joined #openstack-neutron | 18:04 | |
*** nlahouti has joined #openstack-neutron | 18:05 | |
*** signed8bit_ZZZzz is now known as signed8bit | 18:06 | |
*** tiswanso has quit IRC | 18:06 | |
*** pc_m has quit IRC | 18:10 | |
*** SourabhP has joined #openstack-neutron | 18:10 | |
*** yfried|afk has joined #openstack-neutron | 18:11 | |
*** pc_m has joined #openstack-neutron | 18:11 | |
*** bapalm has quit IRC | 18:11 | |
*** tiswanso has joined #openstack-neutron | 18:11 | |
*** minwang2 has quit IRC | 18:12 | |
*** bapalm has joined #openstack-neutron | 18:14 | |
*** mohankumar has quit IRC | 18:14 | |
*** padkrish has quit IRC | 18:16 | |
*** padkrish has joined #openstack-neutron | 18:17 | |
*** armax has quit IRC | 18:18 | |
*** moshele has quit IRC | 18:21 | |
*** padkrish has quit IRC | 18:21 | |
openstackgerrit | Merged openstack/neutron: Fix _ensure_default_security_group logic https://review.openstack.org/203384 | 18:21 |
*** padkrish has joined #openstack-neutron | 18:21 | |
openstackgerrit | Merged openstack/neutron: Unskip firewall test https://review.openstack.org/212145 | 18:21 |
*** tkatarki has quit IRC | 18:22 | |
*** madhu_ak has quit IRC | 18:22 | |
*** tkatarki has joined #openstack-neutron | 18:22 | |
*** sthillma_ has joined #openstack-neutron | 18:23 | |
*** madhu_ak has joined #openstack-neutron | 18:24 | |
*** achanda has quit IRC | 18:24 | |
*** sthillma has quit IRC | 18:25 | |
*** arosen has joined #openstack-neutron | 18:25 | |
*** sthillma has joined #openstack-neutron | 18:26 | |
*** mickeys has quit IRC | 18:26 | |
*** pc_m has quit IRC | 18:27 | |
*** fawadk has quit IRC | 18:27 | |
*** signed8bit is now known as signed8bit_ZZZzz | 18:27 | |
*** pc_m has joined #openstack-neutron | 18:27 | |
*** signed8bit_ZZZzz is now known as signed8bit | 18:28 | |
*** sthillma_ has quit IRC | 18:28 | |
*** lindycoder has quit IRC | 18:28 | |
*** tkatarki has quit IRC | 18:29 | |
*** cbouch has joined #openstack-neutron | 18:31 | |
*** tiswanso has quit IRC | 18:32 | |
*** kiran-r has joined #openstack-neutron | 18:33 | |
*** tkatarki has joined #openstack-neutron | 18:33 | |
*** srilatha has joined #openstack-neutron | 18:33 | |
*** tidwellr has joined #openstack-neutron | 18:34 | |
*** pc_m_ has joined #openstack-neutron | 18:34 | |
*** Sukhdev has joined #openstack-neutron | 18:34 | |
*** dave-mcc_ has joined #openstack-neutron | 18:36 | |
*** yamamoto has joined #openstack-neutron | 18:37 | |
*** lindycoder has joined #openstack-neutron | 18:38 | |
*** pc_m has quit IRC | 18:38 | |
*** Samos123 has quit IRC | 18:38 | |
*** dave-mccowan has quit IRC | 18:39 | |
*** signed8bit is now known as signed8bit_ZZZzz | 18:39 | |
*** madhu_ak has quit IRC | 18:40 | |
*** Samos123 has joined #openstack-neutron | 18:41 | |
*** signed8bit_ZZZzz is now known as signed8bit | 18:41 | |
*** yamamoto has quit IRC | 18:42 | |
*** tflynn has joined #openstack-neutron | 18:42 | |
*** dannywilson has quit IRC | 18:44 | |
*** madhu_ak has joined #openstack-neutron | 18:46 | |
*** dannywilson has joined #openstack-neutron | 18:46 | |
*** piyanai has quit IRC | 18:46 | |
*** vivek-ebay has quit IRC | 18:49 | |
*** vivek-ebay has joined #openstack-neutron | 18:49 | |
*** paulose has quit IRC | 18:52 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 18:53 |
*** kiran-r has quit IRC | 18:53 | |
*** sbelous_ has joined #openstack-neutron | 18:54 | |
*** afazekas has joined #openstack-neutron | 18:55 | |
*** piyanai has joined #openstack-neutron | 18:56 | |
*** Samos123 has quit IRC | 19:00 | |
*** cdl has joined #openstack-neutron | 19:00 | |
*** Samos123 has joined #openstack-neutron | 19:00 | |
regXboi | question for the channel: where do the tempest folks hang out - infra? | 19:00 |
*** cdl has left #openstack-neutron | 19:01 | |
*** azbiswas has quit IRC | 19:02 | |
*** arajagopal has quit IRC | 19:02 | |
*** afazekas has quit IRC | 19:03 | |
*** dannywilson has quit IRC | 19:04 | |
*** klindgren_ has quit IRC | 19:05 | |
*** dannywilson has joined #openstack-neutron | 19:05 | |
*** Aish has joined #openstack-neutron | 19:05 | |
*** piyanai has quit IRC | 19:06 | |
*** tiswanso has joined #openstack-neutron | 19:07 | |
*** arajagopal has joined #openstack-neutron | 19:08 | |
*** Samos123 has quit IRC | 19:08 | |
*** achanda has joined #openstack-neutron | 19:08 | |
*** sbelous_ has quit IRC | 19:09 | |
*** piyanai has joined #openstack-neutron | 19:10 | |
*** emagana has quit IRC | 19:10 | |
*** dannywilson has quit IRC | 19:10 | |
*** Samos123 has joined #openstack-neutron | 19:10 | |
*** lori is now known as lori|away | 19:10 | |
*** dannywilson has joined #openstack-neutron | 19:10 | |
*** cbouch has quit IRC | 19:12 | |
*** tfukushima has quit IRC | 19:12 | |
*** amuller_afk is now known as amuller | 19:14 | |
*** srilatha has quit IRC | 19:16 | |
*** azbiswas has joined #openstack-neutron | 19:17 | |
*** srilatha has joined #openstack-neutron | 19:17 | |
*** tkatarki has quit IRC | 19:17 | |
*** kbringard has quit IRC | 19:18 | |
*** kbringard has joined #openstack-neutron | 19:18 | |
*** pal has joined #openstack-neutron | 19:19 | |
*** tfukushima has joined #openstack-neutron | 19:19 | |
openstackgerrit | Miguel Lavalle proposed openstack/neutron: External DNS driver reference implementation https://review.openstack.org/212213 | 19:21 |
*** pc_m_ has quit IRC | 19:21 | |
*** srilatha has quit IRC | 19:21 | |
*** madhu_ak has quit IRC | 19:22 | |
*** gangil has quit IRC | 19:22 | |
*** kbringard has quit IRC | 19:23 | |
*** tiswanso has quit IRC | 19:23 | |
*** dims_ has joined #openstack-neutron | 19:26 | |
*** vishwana_ has joined #openstack-neutron | 19:26 | |
*** ivar-lazzaro has joined #openstack-neutron | 19:27 | |
*** tfukushima has quit IRC | 19:27 | |
*** jprovazn has quit IRC | 19:27 | |
*** ijw_ has joined #openstack-neutron | 19:27 | |
*** signed8bit is now known as signed8bit_ZZZzz | 19:27 | |
*** rward has quit IRC | 19:27 | |
*** jckasper has joined #openstack-neutron | 19:27 | |
*** rex_lee has quit IRC | 19:27 | |
*** jamielennox|away has quit IRC | 19:27 | |
*** clev is now known as clev-away | 19:28 | |
*** signed8bit_ZZZzz is now known as signed8bit | 19:28 | |
*** dims has quit IRC | 19:28 | |
*** ijw has quit IRC | 19:28 | |
*** puck has quit IRC | 19:28 | |
*** adam_g has quit IRC | 19:28 | |
*** piyanai has quit IRC | 19:28 | |
*** ivar-la__ has quit IRC | 19:28 | |
*** r-daneel has quit IRC | 19:28 | |
*** vishwanathj has quit IRC | 19:28 | |
*** sbalukoff has quit IRC | 19:28 | |
*** otherwiseguy has quit IRC | 19:28 | |
*** dguitarbite has quit IRC | 19:28 | |
*** dsneddon has quit IRC | 19:28 | |
*** serverascode has quit IRC | 19:28 | |
*** tiswanso has joined #openstack-neutron | 19:28 | |
*** sbelous_ has joined #openstack-neutron | 19:29 | |
*** irenab has quit IRC | 19:29 | |
*** piyanai has joined #openstack-neutron | 19:29 | |
*** dave-mccowan has joined #openstack-neutron | 19:30 | |
*** pal has quit IRC | 19:30 | |
*** clev-away is now known as clev | 19:30 | |
*** rward has joined #openstack-neutron | 19:31 | |
*** madhu_ak has joined #openstack-neutron | 19:31 | |
*** serverascode has joined #openstack-neutron | 19:32 | |
*** rm_work is now known as rm_work|away | 19:32 | |
*** tkatarki has joined #openstack-neutron | 19:32 | |
*** signed8bit is now known as signed8bit_ZZZzz | 19:32 | |
*** dave-mcc_ has quit IRC | 19:33 | |
*** achanda has quit IRC | 19:34 | |
*** rex_lee has joined #openstack-neutron | 19:34 | |
*** nlahouti has quit IRC | 19:35 | |
*** puck has joined #openstack-neutron | 19:35 | |
*** adam_g has joined #openstack-neutron | 19:36 | |
*** adam_g has quit IRC | 19:36 | |
*** adam_g has joined #openstack-neutron | 19:36 | |
*** signed8bit_ZZZzz is now known as signed8bit | 19:36 | |
*** padkrish has quit IRC | 19:36 | |
*** ivar-lazzaro has quit IRC | 19:39 | |
*** s3wong has quit IRC | 19:39 | |
*** crose has joined #openstack-neutron | 19:39 | |
*** vern has quit IRC | 19:39 | |
*** Samos123 has quit IRC | 19:40 | |
*** Marga_ has quit IRC | 19:40 | |
*** moha_hunt has joined #openstack-neutron | 19:40 | |
*** vern has joined #openstack-neutron | 19:40 | |
*** abhiraut has quit IRC | 19:40 | |
*** Marga_ has joined #openstack-neutron | 19:41 | |
*** dguitarbite has joined #openstack-neutron | 19:41 | |
*** salv-orlando has joined #openstack-neutron | 19:41 | |
*** sbalukoff has joined #openstack-neutron | 19:42 | |
*** dsneddon has joined #openstack-neutron | 19:43 | |
*** breitz has quit IRC | 19:43 | |
*** Samos123 has joined #openstack-neutron | 19:43 | |
*** breitz has joined #openstack-neutron | 19:43 | |
*** tkatarki has quit IRC | 19:43 | |
*** nlahouti has joined #openstack-neutron | 19:44 | |
*** r-daneel has joined #openstack-neutron | 19:44 | |
*** j^2 has quit IRC | 19:45 | |
*** otherwiseguy has joined #openstack-neutron | 19:45 | |
*** j^2 has joined #openstack-neutron | 19:46 | |
*** sudipto has quit IRC | 19:46 | |
*** armax has joined #openstack-neutron | 19:47 | |
*** alejandrito has quit IRC | 19:47 | |
*** jamielennox|away has joined #openstack-neutron | 19:47 | |
*** jamielennox|away is now known as jamielennox | 19:47 | |
*** jbell8 has quit IRC | 19:49 | |
*** tkatarki has joined #openstack-neutron | 19:49 | |
*** alejandrito has joined #openstack-neutron | 19:49 | |
*** tkatarki has quit IRC | 19:50 | |
*** tkatarki has joined #openstack-neutron | 19:51 | |
*** setuid has quit IRC | 19:52 | |
*** nlahouti has quit IRC | 19:52 | |
*** klindgren has joined #openstack-neutron | 19:52 | |
*** padkrish has joined #openstack-neutron | 19:52 | |
*** padkrish has quit IRC | 19:52 | |
*** padkrish has joined #openstack-neutron | 19:52 | |
*** ajmiller has quit IRC | 19:53 | |
*** srilatha has joined #openstack-neutron | 19:53 | |
*** padkrish has quit IRC | 19:56 | |
*** setuid has joined #openstack-neutron | 19:59 | |
*** kbringard has joined #openstack-neutron | 19:59 | |
*** ushkalim has quit IRC | 20:00 | |
*** neophy has quit IRC | 20:01 | |
*** ronis__ has quit IRC | 20:01 | |
*** annegentle has quit IRC | 20:04 | |
salv-orlando | there's a weird calm in this channel today. | 20:04 |
*** annegentle has joined #openstack-neutron | 20:04 | |
sbelous_ | Still waters run deep | 20:06 |
*** sridhar_ram1 has joined #openstack-neutron | 20:07 | |
*** sridhar_ram has quit IRC | 20:08 | |
*** sridhar_ram1 is now known as sridhar_ram | 20:09 | |
*** alejandrito has quit IRC | 20:09 | |
*** Steap has quit IRC | 20:11 | |
*** ushkalim has joined #openstack-neutron | 20:13 | |
*** achanda has joined #openstack-neutron | 20:13 | |
*** alejandrito has joined #openstack-neutron | 20:13 | |
*** irenab has joined #openstack-neutron | 20:15 | |
*** irenab has quit IRC | 20:15 | |
salv-orlando | zzzeek: I have a nested transaction that when run concurrently by several processes triggers this error " (1305, u'SAVEPOINT sa_savepoint_1 does not exist') [SQL: u'ROLLBACK TO SAVEPOINT sa_savepoint_1']" - I know I did something wrong, but does sqlalchemy offer some tools for understanding what caused the parent transaction to rollback as well? | 20:17 |
*** tkatarki has quit IRC | 20:17 | |
*** neophy has joined #openstack-neutron | 20:17 | |
zzzeek | salv-orlando: entirelyt separate python processes? | 20:17 |
salv-orlando | there are unfortunately too many events in the mysql log, so it is very hard to correlate there. These failure occurs under heavy load, with multiple workers doing neutron API operations concurrently | 20:18 |
salv-orlando | zzzeek: yes | 20:18 |
salv-orlando | I mean they're forked from each other | 20:18 |
salv-orlando | the neutron API workers | 20:18 |
zzzeek | salv-orlando: um. i would think the name of a SAVEPOINT in mysql is local to the connection | 20:18 |
zzzeek | salv-orlando: because it looks like the same savepoint is being rolled back elsewhere | 20:18 |
*** sthillma has quit IRC | 20:18 | |
salv-orlando | zzzeek: I thought that was not possible? | 20:19 |
*** mickeys has joined #openstack-neutron | 20:19 | |
*** achanda has quit IRC | 20:19 | |
zzzeek | salv-orlando: shouldnt be? seems like this would have come up earlier if not | 20:19 |
*** emagana has joined #openstack-neutron | 20:20 | |
salv-orlando | zzzeek: I guess you're right. So another connection is probably rolling back the savepoint. This is consistent with what I see in the logs actually. | 20:20 |
zzzeek | salv-orlando: execpt that’s not how it works :) | 20:20 |
zzzeek | salv-orlando: *unless*. you are sharing the connection down to the child fork | 20:20 |
salv-orlando | zzzeek: that's what I'm frightened about. But I guess it's not the case | 20:21 |
zzzeek | salv-orlando: jsut tested. they are definitely independent | 20:21 |
salv-orlando | well, let's say I hope it's not the case | 20:21 |
zzzeek | salv-orlando: I’d see if you can log the session id perhaps | 20:22 |
zzzeek | salv-orlando: that is, the MySQL session id | 20:22 |
zzzeek | http://dev.mysql.com/doc/refman/5.1/en/information-functions.html#function_connection-id | 20:22 |
*** tidwellr has quit IRC | 20:22 | |
zzzeek | salv-orlando: also is this galera multi-master ? | 20:22 |
salv-orlando | zzzeek: thanks this helps a lot. Nope it's not galera. | 20:22 |
zzzeek | salv-orlando: ok | 20:22 |
*** tidwellr has joined #openstack-neutron | 20:23 | |
zzzeek | salv-orlando: teh other thing that could happen is that in one process you are sharing the connection or session around to multiple greenlets | 20:23 |
zzzeek | salv-orlando: if neutron doesnt have its ducks in a row as far as how a request maps to a greenlet maps to a Session, that will wreak havoc | 20:23 |
zzzeek | salv-orlando: ive tried to follow this code in various apps like nova and i can’t follow it | 20:24 |
zzzeek | salv-orlando: e.g. the wsgi->greenlet system | 20:24 |
*** Guest100 has joined #openstack-neutron | 20:24 | |
*** Guest100 has quit IRC | 20:24 | |
salv-orlando | zzzeek: neutron is done in a way that every API request gets its own session; unless we are so dumb that we believe we've done this and we're actually sharing sessions | 20:24 |
zzzeek | salv-orlando: I’d check on that | 20:25 |
*** crose has quit IRC | 20:25 | |
zzzeek | salv-orlando: or if you are spawning greenlets in a request | 20:25 |
zzzeek | salv-orlando: or real threads | 20:25 |
*** rossella_s has quit IRC | 20:25 | |
zzzeek | salv-orlando: i think connection_id() mysql function can tell all on this | 20:25 |
*** rossella_s has joined #openstack-neutron | 20:25 | |
salv-orlando | zzzeek: It's all sequentially processed. Yes connection_id will surely shed some light. Thanks for the advice. I have solved the issue in another way - but I still want to get to the bottom of this issue because it's not the first time that I see it | 20:27 |
zzzeek | salv-orlando: yes this is critical | 20:27 |
*** jwy has quit IRC | 20:27 | |
salv-orlando | and I never managed to fully understand its cause. I always solve it by reworking the logic so that I do not execute anymore the DB operation that triggers it | 20:27 |
*** ushkalim has quit IRC | 20:31 | |
*** achanda has joined #openstack-neutron | 20:32 | |
*** jwy has joined #openstack-neutron | 20:32 | |
*** abhiraut has joined #openstack-neutron | 20:33 | |
salv-orlando | zzzeek: anyway thanks very much. I will abuse of your kindness again should I need further assistance | 20:34 |
zzzeek | salv-orlando: please do | 20:34 |
*** Sukhdev has quit IRC | 20:34 | |
*** tidwellr1 has joined #openstack-neutron | 20:35 | |
*** gangil has joined #openstack-neutron | 20:36 | |
*** tidwellr has quit IRC | 20:37 | |
*** signed8bit is now known as signed8bit_ZZZzz | 20:37 | |
*** padkrish has joined #openstack-neutron | 20:38 | |
*** alex_klimov has joined #openstack-neutron | 20:39 | |
*** DWfuturetec has joined #openstack-neutron | 20:41 | |
*** boris-42 has joined #openstack-neutron | 20:41 | |
*** signed8bit_ZZZzz is now known as signed8bit | 20:41 | |
*** padkrish has quit IRC | 20:42 | |
*** ushkalim has joined #openstack-neutron | 20:43 | |
*** mickeys has quit IRC | 20:45 | |
*** mickeys has joined #openstack-neutron | 20:46 | |
*** emagana has quit IRC | 20:47 | |
*** armax has quit IRC | 20:49 | |
*** ameybhide has joined #openstack-neutron | 20:51 | |
*** regXboi has quit IRC | 20:55 | |
*** srilatha has quit IRC | 20:57 | |
*** achanda has quit IRC | 20:57 | |
*** achanda_ has joined #openstack-neutron | 20:57 | |
*** nyechiel_ has joined #openstack-neutron | 21:00 | |
*** dannywil_ has joined #openstack-neutron | 21:00 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 21:00 |
*** sthillma has joined #openstack-neutron | 21:01 | |
*** ivar-lazzaro has joined #openstack-neutron | 21:02 | |
*** dannywilson has quit IRC | 21:03 | |
*** sthillma has quit IRC | 21:04 | |
*** sthillma has joined #openstack-neutron | 21:04 | |
*** tiswanso has quit IRC | 21:05 | |
*** dsneddon has quit IRC | 21:07 | |
*** yamahata has joined #openstack-neutron | 21:10 | |
*** regXboi has joined #openstack-neutron | 21:11 | |
*** debedb has joined #openstack-neutron | 21:11 | |
*** rm_work|away is now known as rm_work | 21:13 | |
*** crose has joined #openstack-neutron | 21:13 | |
*** nyechiel_ has quit IRC | 21:14 | |
*** thorst has quit IRC | 21:16 | |
*** annegentle has quit IRC | 21:20 | |
*** annegentle has joined #openstack-neutron | 21:20 | |
*** tiny-hands has left #openstack-neutron | 21:21 | |
*** dsneddon has joined #openstack-neutron | 21:24 | |
*** Aish has quit IRC | 21:24 | |
*** padkrish has joined #openstack-neutron | 21:24 | |
*** nlahouti has joined #openstack-neutron | 21:24 | |
*** krtaylor has quit IRC | 21:26 | |
*** armax has joined #openstack-neutron | 21:26 | |
*** abhiraut has quit IRC | 21:27 | |
*** sbelous_ has quit IRC | 21:28 | |
*** dguerri` is now known as dguerri | 21:30 | |
*** rpothier has quit IRC | 21:30 | |
*** dguerri is now known as dguerri` | 21:31 | |
*** crose has quit IRC | 21:32 | |
*** TrevorV has quit IRC | 21:33 | |
openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 21:33 |
*** clev is now known as clev-away | 21:33 | |
*** Aish has joined #openstack-neutron | 21:33 | |
*** clev-away is now known as clev | 21:34 | |
openstackgerrit | Merged openstack/neutron: Python 3: encode or decode i/o data of Popen.communicate() https://review.openstack.org/208848 | 21:34 |
openstackgerrit | Merged openstack/neutron: Add EnvironmentDescription, pass it down https://review.openstack.org/210830 | 21:34 |
*** gangil has quit IRC | 21:34 | |
*** Piet has joined #openstack-neutron | 21:35 | |
*** gangil has joined #openstack-neutron | 21:38 | |
*** nlahouti has quit IRC | 21:38 | |
*** dannywil_ has quit IRC | 21:40 | |
*** nlahouti has joined #openstack-neutron | 21:41 | |
*** absubram has quit IRC | 21:42 | |
*** annegentle has quit IRC | 21:42 | |
*** edmondsw has quit IRC | 21:44 | |
*** absubram has joined #openstack-neutron | 21:47 | |
*** alejandrito has quit IRC | 21:49 | |
*** bapalm has quit IRC | 21:49 | |
*** arajagopal has quit IRC | 21:51 | |
*** bapalm has joined #openstack-neutron | 21:52 | |
*** nlahouti has left #openstack-neutron | 21:52 | |
*** nlahouti has joined #openstack-neutron | 21:52 | |
*** alejandrito has joined #openstack-neutron | 21:52 | |
openstackgerrit | Ryan Moats proposed openstack/neutron: Introduce report_counters RPC for agents https://review.openstack.org/213474 | 21:53 |
openstackgerrit | Ryan Moats proposed openstack/neutron: Add instrumentation devref https://review.openstack.org/204282 | 21:53 |
*** ushkalim has quit IRC | 21:53 | |
*** srilatha has joined #openstack-neutron | 21:54 | |
*** dannywilson has joined #openstack-neutron | 21:54 | |
*** britthouser has quit IRC | 21:54 | |
*** signed8bit has quit IRC | 21:55 | |
*** piyanai has quit IRC | 21:56 | |
*** bapalm has quit IRC | 21:57 | |
Sam-I-Am | kevinbenton: when you're around (maybe on this side of the world) i found some issues with the mac spoofing business on linuxbridge (might also apply to ovs) | 21:58 |
*** dmartls1 has quit IRC | 21:59 | |
*** bapalm has joined #openstack-neutron | 22:03 | |
*** jobewan has joined #openstack-neutron | 22:03 | |
*** lindycoder has quit IRC | 22:04 | |
*** claudiub has quit IRC | 22:05 | |
*** kbringard has quit IRC | 22:05 | |
*** bapalm has quit IRC | 22:08 | |
*** s3wong has joined #openstack-neutron | 22:12 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor VIF_TYPE_* constants https://review.openstack.org/214805 | 22:12 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants https://review.openstack.org/214806 | 22:12 |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: WIP: Fix reservation engine https://review.openstack.org/215363 | 22:13 |
*** bapalm has joined #openstack-neutron | 22:14 | |
*** sbelous_ has joined #openstack-neutron | 22:16 | |
*** krtaylor has joined #openstack-neutron | 22:18 | |
*** bapalm has quit IRC | 22:19 | |
*** annegentle has joined #openstack-neutron | 22:20 | |
*** chlong has quit IRC | 22:25 | |
*** sbelous has quit IRC | 22:26 | |
*** sbelous_ has quit IRC | 22:26 | |
*** amuller has quit IRC | 22:27 | |
*** sbelous has joined #openstack-neutron | 22:27 | |
*** alex_klimov has quit IRC | 22:28 | |
*** sbelous has quit IRC | 22:28 | |
*** bapalm has joined #openstack-neutron | 22:29 | |
*** thorst has joined #openstack-neutron | 22:32 | |
*** annegentle has quit IRC | 22:33 | |
*** annegentle has joined #openstack-neutron | 22:33 | |
mlavalle | kevinbenton: ping | 22:33 |
*** bapalm has quit IRC | 22:34 | |
*** shwetaap has quit IRC | 22:34 | |
*** Aish has quit IRC | 22:35 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Used namedtuple for ReservationInfo https://review.openstack.org/215369 | 22:35 |
openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255] https://review.openstack.org/212220 | 22:35 |
*** bapalm has joined #openstack-neutron | 22:35 | |
*** tfukushima has joined #openstack-neutron | 22:37 | |
*** annegentle has quit IRC | 22:38 | |
*** emagana has joined #openstack-neutron | 22:40 | |
*** bapalm has quit IRC | 22:43 | |
*** bapalm has joined #openstack-neutron | 22:43 | |
*** bapalm has quit IRC | 22:48 | |
*** dannywilson has quit IRC | 22:48 | |
*** jobewan has quit IRC | 22:49 | |
*** bapalm has joined #openstack-neutron | 22:51 | |
*** s3wong has quit IRC | 22:52 | |
*** rkukura has quit IRC | 22:52 | |
*** thurloat_isgone is now known as thurloat | 22:52 | |
*** tfukushima has quit IRC | 22:55 | |
*** thorst has quit IRC | 22:55 | |
*** dannywilson has joined #openstack-neutron | 22:57 | |
*** rm_work is now known as rm_work|away | 23:01 | |
*** rkukura has joined #openstack-neutron | 23:01 | |
*** sridhar_ram has quit IRC | 23:02 | |
*** jobewan has joined #openstack-neutron | 23:02 | |
*** sridhar_ram has joined #openstack-neutron | 23:02 | |
*** thurloat is now known as thurloat_isgone | 23:03 | |
*** achanda_ has quit IRC | 23:05 | |
*** alejandrito has quit IRC | 23:06 | |
*** DWfuturetec has quit IRC | 23:07 | |
*** tiny-hands has joined #openstack-neutron | 23:07 | |
*** tidwellr1 has left #openstack-neutron | 23:09 | |
*** emagana has quit IRC | 23:14 | |
*** kutija has quit IRC | 23:15 | |
*** regXboi has quit IRC | 23:15 | |
*** achanda has joined #openstack-neutron | 23:16 | |
*** dannywilson has quit IRC | 23:19 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Router Interface delete fails to remove interface from snat ns https://review.openstack.org/215329 | 23:24 |
*** zzzeek has quit IRC | 23:25 | |
*** sridhar_ram has quit IRC | 23:26 | |
*** r-daneel has quit IRC | 23:27 | |
*** sridhar_ram has joined #openstack-neutron | 23:28 | |
*** Fdaisuke has joined #openstack-neutron | 23:28 | |
*** dannywilson has joined #openstack-neutron | 23:30 | |
Fdaisuke | armax: Hi, | 23:30 |
Fdaisuke | armax: I commented to https://review.openstack.org/#/c/207328/ | 23:32 |
*** ameybhid_ has joined #openstack-neutron | 23:32 | |
Fdaisuke | Would you please have a look at this? | 23:32 |
*** ameybhid_ has quit IRC | 23:33 | |
*** banix has quit IRC | 23:33 | |
*** ameybhide has quit IRC | 23:34 | |
*** s3wong has joined #openstack-neutron | 23:35 | |
*** regXboi has joined #openstack-neutron | 23:37 | |
*** debedb has quit IRC | 23:38 | |
*** arosen has quit IRC | 23:40 | |
*** sridhar_ram has quit IRC | 23:41 | |
*** sridhar_ram has joined #openstack-neutron | 23:42 | |
*** SourabhP has quit IRC | 23:43 | |
*** Marga_ has quit IRC | 23:46 | |
*** salv-orlando has quit IRC | 23:46 | |
*** Marga_ has joined #openstack-neutron | 23:47 | |
armax | Fdaisuke: done | 23:48 |
*** Swami has quit IRC | 23:48 | |
*** cdl has joined #openstack-neutron | 23:50 | |
*** cdl has left #openstack-neutron | 23:51 | |
*** debedb has joined #openstack-neutron | 23:51 | |
Fdaisuke | armax: thanks. I'll check. | 23:52 |
*** srilatha has quit IRC | 23:54 | |
*** srilatha has joined #openstack-neutron | 23:54 | |
*** gildub has joined #openstack-neutron | 23:55 | |
*** debedb has quit IRC | 23:55 | |
*** dannywilson has quit IRC | 23:56 | |
*** debedb has joined #openstack-neutron | 23:56 | |
*** SumitNaiksatam has quit IRC | 23:59 | |
*** srilatha has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!