| *** itsuugo has joined #openstack-neutron | 00:00 | |
| *** jamielennox is now known as jamielennox|away | 00:00 | |
| *** mlavalle has quit IRC | 00:01 | |
| *** tongli has quit IRC | 00:02 | |
| *** davidlenwell has quit IRC | 00:03 | |
| *** itsuugo has quit IRC | 00:05 | |
| *** itsuugo has joined #openstack-neutron | 00:06 | |
| *** tidwellr has left #openstack-neutron | 00:06 | |
| kevinbenton | carl_baldwin: looks like the DHCP port updates are being triggered because of stale data | 00:09 |
|---|---|---|
| carl_baldwin | kevinbenton: the data probably smells bad too. | 00:10 |
| kevinbenton | carl_baldwin: the dhcp agent has port update events in it's queue | 00:10 |
| kevinbenton | carl_baldwin: one of which is its own port | 00:10 |
| kevinbenton | carl_baldwin: and if they subnet create for the new subnet comes in first | 00:10 |
| kevinbenton | carl_baldwin: well not comes in first, but gets the lock first | 00:10 |
| *** itsuugo has quit IRC | 00:11 | |
| kevinbenton | carl_baldwin: it will update the dhcp port | 00:11 |
| kevinbenton | carl_baldwin: server side | 00:11 |
| kevinbenton | carl_baldwin: and then after finishing that, process a port update from its own port | 00:11 |
| *** itsuugo has joined #openstack-neutron | 00:11 | |
| kevinbenton | carl_baldwin: from before | 00:11 |
| *** jckasper has joined #openstack-neutron | 00:12 | |
| carl_baldwin | kevinbenton: that all makes sense | 00:12 |
| kevinbenton | carl_baldwin: at least that's what i'm seeing locally | 00:12 |
| kevinbenton | carl_baldwin: my special is_stale function was broken recently (by me) because armax made me change the name of the revision number field :) | 00:13 |
| armax | I did? | 00:13 |
| kevinbenton | armax: yes | 00:13 |
| armax | you mess up as usual! | 00:13 |
| kevinbenton | armax: you are going under the bus on this one | 00:13 |
| armax | you fix 1 thing and break 10 in the process | 00:13 |
| armax | get out! | 00:13 |
| *** Alexey_Abashkin_ has quit IRC | 00:13 | |
| *** jamielennox|away is now known as jamielennox | 00:14 | |
| carl_baldwin | armax: kevinbenton: do I need to step in and help you two play nice? | 00:14 |
| *** Alexey_Abashkin_ has joined #openstack-neutron | 00:14 | |
| kevinbenton | armax: carl's going to pull the car over | 00:14 |
| kevinbenton | armax: and beat us | 00:15 |
| *** rmcall has quit IRC | 00:15 | |
| *** rmcall has joined #openstack-neutron | 00:16 | |
| carl_baldwin | kevinbenton: armax: It sounds like we've got about 4 ways to fix this bug most of which we should probably do. | 00:16 |
| *** davidlenwell has joined #openstack-neutron | 00:16 | |
| kevinbenton | carl_baldwin: it's definitely bad behavior of the DHCP agent making it worse for the server | 00:17 |
| carl_baldwin | kevinbenton: sounds like it. | 00:17 |
| *** Alexey_Abashkin_ has quit IRC | 00:18 | |
| kevinbenton | carl_baldwin: i'm seeing lock acquisistion times on the dhcp agent side that go into the hundreds of milliseconds, so i can see how we get built up messages | 00:18 |
| carl_baldwin | kevinbenton: times to aquire? I saw one yesterday at 2.5 seconds. | 00:18 |
| kevinbenton | carl_baldwin: yeah | 00:19 |
| carl_baldwin | kevinbenton: The locks were usually not held more than 50 - 100 ms from what I saw. | 00:19 |
| kevinbenton | carl_baldwin: yep, we just have way more processing now | 00:19 |
| kevinbenton | armax, carl_baldwin: i think I'm going to submit a patch to have the dhcp agent skip reloads if nothing but the port status has changed | 00:20 |
| armax | kevinbenton: sure, keep breaking the damn thing | 00:20 |
| armax | :) | 00:20 |
| carl_baldwin | kevinbenton: ok | 00:21 |
| carl_baldwin | armax: Weren't you out running? Didn't that help? ;) | 00:21 |
| armax | not really | 00:21 |
| *** jckasper has quit IRC | 00:22 | |
| *** vishwanathj has quit IRC | 00:22 | |
| openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: Fix KeyError when fw rule associated with a policy is updated https://review.openstack.org/373565 | 00:24 |
| *** itsuugo has quit IRC | 00:27 | |
| *** itsuugo has joined #openstack-neutron | 00:27 | |
| openstackgerrit | Kengo Hobo proposed openstack/neutron: Wrap ns non-existent err during updating iptables by metering-agent https://review.openstack.org/371407 | 00:28 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP agent use 'revision_numer' https://review.openstack.org/373566 | 00:29 |
| kevinbenton | armax, carl_baldwin: ^^ | 00:29 |
| *** Alexey_Abashkin_ has joined #openstack-neutron | 00:32 | |
| *** Sukhdev has joined #openstack-neutron | 00:35 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP agent use 'revision_numer' https://review.openstack.org/373566 | 00:36 |
| *** itsuugo has quit IRC | 00:39 | |
| *** itsuugo has joined #openstack-neutron | 00:40 | |
| *** tonytan4ever has joined #openstack-neutron | 00:48 | |
| *** hoangcx has joined #openstack-neutron | 00:48 | |
| *** fnaval has joined #openstack-neutron | 00:49 | |
| *** yushiro has joined #openstack-neutron | 00:51 | |
| *** tonytan4ever has quit IRC | 00:52 | |
| *** itsuugo has quit IRC | 00:54 | |
| *** oshvartz has quit IRC | 00:55 | |
| *** itsuugo has joined #openstack-neutron | 00:55 | |
| *** mriedem has joined #openstack-neutron | 00:58 | |
| *** mfuruta has joined #openstack-neutron | 01:02 | |
| *** itsuugo has quit IRC | 01:03 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Stop oslo_messaging from error logging CallbackNotFound https://review.openstack.org/373575 | 01:03 |
| *** itsuugo has joined #openstack-neutron | 01:05 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: DNM: Triaging bug 1622616 https://review.openstack.org/373536 | 01:05 |
| openstack | bug 1622616 in neutron "delete_subnet update_port appears racey with ipam" [Critical,In progress] https://launchpad.net/bugs/1622616 - Assigned to Kevin Benton (kevinbenton) | 01:06 |
| *** johnbelamaric has joined #openstack-neutron | 01:07 | |
| *** tbachman has quit IRC | 01:08 | |
| *** rkukura has quit IRC | 01:09 | |
| *** oshvartz has joined #openstack-neutron | 01:10 | |
| *** itsuugo has quit IRC | 01:14 | |
| *** bana_k has quit IRC | 01:15 | |
| *** itsuugo has joined #openstack-neutron | 01:15 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: fullstack: Add security group tests https://review.openstack.org/341328 | 01:16 |
| *** kevo has quit IRC | 01:16 | |
| *** rossella_s has quit IRC | 01:19 | |
| *** rossella_s has joined #openstack-neutron | 01:19 | |
| *** sdake_ has joined #openstack-neutron | 01:19 | |
| *** sdake has quit IRC | 01:21 | |
| *** amotoki has joined #openstack-neutron | 01:26 | |
| *** stanzgy has joined #openstack-neutron | 01:27 | |
| *** gvrangan has quit IRC | 01:30 | |
| *** itsuugo has quit IRC | 01:36 | |
| *** johnbelamaric has quit IRC | 01:36 | |
| *** johnbelamaric has joined #openstack-neutron | 01:36 | |
| *** Jeffrey4l has joined #openstack-neutron | 01:37 | |
| *** itsuugo has joined #openstack-neutron | 01:37 | |
| *** s3wong has quit IRC | 01:38 | |
| *** salv-orl_ has joined #openstack-neutron | 01:42 | |
| *** salv-orlando has quit IRC | 01:44 | |
| *** yedongcan has joined #openstack-neutron | 01:46 | |
| *** itsuugo has quit IRC | 01:46 | |
| *** johnbelamaric has quit IRC | 01:47 | |
| *** itsuugo has joined #openstack-neutron | 01:47 | |
| *** donghao has joined #openstack-neutron | 01:47 | |
| *** tonytan4ever has joined #openstack-neutron | 01:49 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [wip]: Fix Ml2 revision_number handling https://review.openstack.org/373582 | 01:49 |
| *** baoli has joined #openstack-neutron | 01:51 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [wip]: Fix Ml2 revision_number handling https://review.openstack.org/373582 | 01:53 |
| *** tonytan4ever has quit IRC | 01:54 | |
| *** itsuugo has quit IRC | 01:55 | |
| *** itsuugo has joined #openstack-neutron | 01:56 | |
| openstackgerrit | LIU Yulong proposed openstack/neutron-dynamic-routing: Gracefully withdraw the floating IP bgp route https://review.openstack.org/357716 | 01:59 |
| openstackgerrit | LIU Yulong proposed openstack/neutron-dynamic-routing: Let the bgp_plugin to query floating IP bgp next_hop https://review.openstack.org/372310 | 01:59 |
| *** lnicolas has joined #openstack-neutron | 02:03 | |
| *** ivar-lazzaro has quit IRC | 02:06 | |
| *** itsuugo has quit IRC | 02:06 | |
| *** itsuugo has joined #openstack-neutron | 02:07 | |
| *** wu_ng has joined #openstack-neutron | 02:09 | |
| *** ivar-lazzaro has joined #openstack-neutron | 02:10 | |
| *** itsuugo has quit IRC | 02:12 | |
| *** itsuugo has joined #openstack-neutron | 02:12 | |
| *** stanzgy has quit IRC | 02:15 | |
| *** ivar-lazzaro has quit IRC | 02:15 | |
| *** mriedem has quit IRC | 02:19 | |
| *** itsuugo has quit IRC | 02:19 | |
| *** itsuugo has joined #openstack-neutron | 02:21 | |
| openstackgerrit | Merged openstack/neutron: Raise level of message to info https://review.openstack.org/370964 | 02:21 |
| *** tonytan4ever has joined #openstack-neutron | 02:23 | |
| *** amotoki has quit IRC | 02:25 | |
| *** marlinc has quit IRC | 02:30 | |
| *** marlinc has joined #openstack-neutron | 02:32 | |
| openstackgerrit | Yu Jie proposed openstack/python-neutronclient: Remove duplicated code for create network https://review.openstack.org/373598 | 02:35 |
| *** itsuugo has quit IRC | 02:36 | |
| *** itsuugo has joined #openstack-neutron | 02:37 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 02:40 |
| *** amotoki has joined #openstack-neutron | 02:41 | |
| openstackgerrit | Merged openstack/neutron: Always cleanup stale devices on DHCP setup failure https://review.openstack.org/372817 | 02:44 |
| *** itsuugo has quit IRC | 02:45 | |
| *** yushiro is now known as yushiro_lunch | 02:46 | |
| *** mickeys has quit IRC | 02:46 | |
| *** itsuugo has joined #openstack-neutron | 02:47 | |
| *** amotoki has quit IRC | 02:49 | |
| *** amotoki has joined #openstack-neutron | 02:49 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 02:50 |
| *** yuanying has quit IRC | 02:52 | |
| *** amotoki has quit IRC | 02:54 | |
| *** fnaval has quit IRC | 02:55 | |
| *** tbachman has joined #openstack-neutron | 02:56 | |
| *** amotoki has joined #openstack-neutron | 02:58 | |
| *** itsuugo has quit IRC | 02:58 | |
| *** itsuugo has joined #openstack-neutron | 03:00 | |
| *** jamespage has quit IRC | 03:00 | |
| *** jamespag` has joined #openstack-neutron | 03:00 | |
| *** baoli has quit IRC | 03:01 | |
| *** xiaoyang has quit IRC | 03:02 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** yedongcan1 has joined #openstack-neutron | 03:04 | |
| *** amotoki has quit IRC | 03:04 | |
| openstackgerrit | John Perkins proposed openstack/neutron: Move OVO exceptions to neutron-lib/exceptions https://review.openstack.org/357787 | 03:05 |
| *** itsuugo has quit IRC | 03:05 | |
| *** itsuugo has joined #openstack-neutron | 03:06 | |
| *** yedongcan has quit IRC | 03:07 | |
| *** tbachman_ has joined #openstack-neutron | 03:10 | |
| *** tbachman has quit IRC | 03:12 | |
| *** tbachman_ is now known as tbachman | 03:12 | |
| openstackgerrit | lilintan proposed openstack/neutron: Adjustment method execution order https://review.openstack.org/373609 | 03:13 |
| openstackgerrit | Yang Yu proposed openstack/neutron-specs: Support agentless driver in neutron-dynamic-routing https://review.openstack.org/373029 | 03:14 |
| *** itsuugo has quit IRC | 03:23 | |
| *** armax has quit IRC | 03:23 | |
| *** tbachman_ has joined #openstack-neutron | 03:24 | |
| *** itsuugo has joined #openstack-neutron | 03:25 | |
| *** armax has joined #openstack-neutron | 03:25 | |
| *** yfried has joined #openstack-neutron | 03:25 | |
| *** tbachman has quit IRC | 03:25 | |
| *** tbachman_ is now known as tbachman | 03:25 | |
| *** itsuugo has quit IRC | 03:29 | |
| *** itsuugo has joined #openstack-neutron | 03:31 | |
| openstackgerrit | lilintan proposed openstack/python-neutronclient: Replace 'MagicMock' with 'Mock' https://review.openstack.org/373620 | 03:33 |
| *** itsuugo has quit IRC | 03:36 | |
| *** itsuugo has joined #openstack-neutron | 03:36 | |
| *** ijw has quit IRC | 03:40 | |
| *** itsuugo has quit IRC | 03:41 | |
| *** itsuugo has joined #openstack-neutron | 03:42 | |
| *** sdake_ has quit IRC | 03:42 | |
| *** wolverineav has joined #openstack-neutron | 03:43 | |
| openstackgerrit | Aradhana Singh proposed openstack/neutron: OVO for SegmentHostMapping https://review.openstack.org/359379 | 03:44 |
| *** ijw_ has joined #openstack-neutron | 03:45 | |
| *** amotoki has joined #openstack-neutron | 03:46 | |
| *** chandanc_ has joined #openstack-neutron | 03:46 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/neutron: fullstack: Add security group tests https://review.openstack.org/341328 | 03:48 |
| *** ijw_ has quit IRC | 03:49 | |
| *** wolverineav has quit IRC | 03:49 | |
| *** yuanying has joined #openstack-neutron | 03:49 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP agent use 'revision_number' https://review.openstack.org/373566 | 03:51 |
| *** itsuugo has quit IRC | 03:51 | |
| *** tbachman has quit IRC | 03:51 | |
| *** itsuugo has joined #openstack-neutron | 03:52 | |
| *** yfried has quit IRC | 03:54 | |
| *** sticker_ has joined #openstack-neutron | 03:54 | |
| *** yushiro_lunch is now known as yushiro | 03:54 | |
| *** donghao has quit IRC | 03:55 | |
| *** sticker has quit IRC | 03:58 | |
| *** itsuugo has quit IRC | 03:59 | |
| *** itsuugo has joined #openstack-neutron | 04:01 | |
| *** markvoelker has quit IRC | 04:03 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Try to reuse existing IPs when a port update specifies subnet https://review.openstack.org/372794 | 04:04 |
| *** itsuugo has quit IRC | 04:05 | |
| *** wolverineav has joined #openstack-neutron | 04:06 | |
| *** itsuugo has joined #openstack-neutron | 04:06 | |
| *** yamamoto has joined #openstack-neutron | 04:07 | |
| *** lnicolas has quit IRC | 04:09 | |
| *** markvoelker has joined #openstack-neutron | 04:10 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 04:10 |
| *** wolverineav has quit IRC | 04:10 | |
| *** itsuugo has quit IRC | 04:11 | |
| *** itsuugo has joined #openstack-neutron | 04:12 | |
| *** ijw has joined #openstack-neutron | 04:14 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: [wip]: Fix Ml2 revision_number handling https://review.openstack.org/373582 | 04:16 |
| *** ababich has quit IRC | 04:16 | |
| *** wolverineav has joined #openstack-neutron | 04:17 | |
| *** itsuugo has quit IRC | 04:17 | |
| *** ababich has joined #openstack-neutron | 04:17 | |
| *** itsuugo has joined #openstack-neutron | 04:18 | |
| *** mkolesni_ has joined #openstack-neutron | 04:18 | |
| *** ijw has quit IRC | 04:19 | |
| *** ababich has quit IRC | 04:21 | |
| *** ababich has joined #openstack-neutron | 04:21 | |
| *** itsuugo has quit IRC | 04:23 | |
| *** wolverineav has quit IRC | 04:23 | |
| *** itsuugo has joined #openstack-neutron | 04:24 | |
| *** wolverineav has joined #openstack-neutron | 04:28 | |
| *** itsuugo has quit IRC | 04:29 | |
| *** itsuugo has joined #openstack-neutron | 04:30 | |
| *** donghao has joined #openstack-neutron | 04:32 | |
| *** wolverineav has quit IRC | 04:33 | |
| *** shausy has joined #openstack-neutron | 04:34 | |
| *** oshvartz has quit IRC | 04:34 | |
| *** ababich has quit IRC | 04:35 | |
| reedip | armax : ping, you there? | 04:36 |
| *** ababich has joined #openstack-neutron | 04:36 | |
| *** vthapar has joined #openstack-neutron | 04:39 | |
| *** gvrangan has joined #openstack-neutron | 04:42 | |
| *** vthapar has quit IRC | 04:44 | |
| *** itsuugo has quit IRC | 04:45 | |
| *** itsuugo has joined #openstack-neutron | 04:46 | |
| *** itsuugo has quit IRC | 04:51 | |
| *** garyk1 has joined #openstack-neutron | 04:52 | |
| *** itsuugo has joined #openstack-neutron | 04:53 | |
| *** yamamoto has quit IRC | 04:53 | |
| *** yedongcan1 has quit IRC | 04:53 | |
| *** wolverineav has joined #openstack-neutron | 04:56 | |
| *** yfried has joined #openstack-neutron | 04:56 | |
| *** wolverineav has quit IRC | 04:57 | |
| *** wolverineav has joined #openstack-neutron | 04:57 | |
| openstackgerrit | garyk proposed openstack/neutron: Revert "Do not retry default security group creation" https://review.openstack.org/373647 | 04:59 |
| garyk1 | armax: kevinbenton: please see above - this break the API sigantures. | 05:02 |
| *** prateek has joined #openstack-neutron | 05:07 | |
| *** ratailor has joined #openstack-neutron | 05:07 | |
| *** sc68cal_ has joined #openstack-neutron | 05:09 | |
| *** yedongcan has joined #openstack-neutron | 05:09 | |
| *** karad has quit IRC | 05:11 | |
| *** sc68cal has quit IRC | 05:11 | |
| *** itsuugo has quit IRC | 05:12 | |
| *** itsuugo has joined #openstack-neutron | 05:12 | |
| *** pbandark has joined #openstack-neutron | 05:14 | |
| *** bana_k has joined #openstack-neutron | 05:15 | |
| *** claudiub|2 has joined #openstack-neutron | 05:18 | |
| *** rossella_s has quit IRC | 05:18 | |
| *** zhhuabj_ has quit IRC | 05:19 | |
| *** rossella_s has joined #openstack-neutron | 05:19 | |
| kevinbenton | garyk1: what plugins are using default_sg? | 05:20 |
| kevinbenton | garyk1: it's only for the recursive call, no? | 05:20 |
| *** moshele has joined #openstack-neutron | 05:21 | |
| kevinbenton | garyk1: ah, i see vmware is subclassing and calling super | 05:21 |
| *** wuhg has joined #openstack-neutron | 05:22 | |
| garyk1 | kevinbenton: everyone is :) | 05:24 |
| garyk1 | we need a change that will not break the world | 05:24 |
| garyk1 | of at least keep the same api signature | 05:24 |
| garyk1 | i see that dragon flow is using it too | 05:24 |
| kevinbenton | garyk1: yeah, i'm looking at the change again to see if just adding it to the signature would be enough | 05:25 |
| *** pgadiya has joined #openstack-neutron | 05:26 | |
| *** itsuugo has quit IRC | 05:26 | |
| *** ganesh has joined #openstack-neutron | 05:26 | |
| *** ganesh is now known as gkadam | 05:26 | |
| *** itsuugo has joined #openstack-neutron | 05:27 | |
| garyk1 | kevinbenton: ok. i think that we should revert in the mean time till we come up with a better solution | 05:32 |
| *** itsuugo has quit IRC | 05:32 | |
| *** itsuugo has joined #openstack-neutron | 05:33 | |
| *** adriant has quit IRC | 05:35 | |
| kevinbenton | garyk1: +2'ed. I'll have Oleg file a different fix once he's online | 05:35 |
| kevinbenton | amotoki: you around? | 05:35 |
| amotoki | kevinbenton: hi | 05:36 |
| kevinbenton | amotoki: can you +2 that revert? | 05:36 |
| kevinbenton | amotoki: https://review.openstack.org/373647 | 05:37 |
| amotoki | kevinbenton: sure | 05:37 |
| kevinbenton | amotoki: thx | 05:37 |
| *** yamamoto has joined #openstack-neutron | 05:38 | |
| garyk1 | kevinbenton: amotoki: thanks guys! | 05:38 |
| amotoki | garyk1: kevinbenton: yeah, every plugin with sg support needs default_sg of create_security_group... | 05:39 |
| kevinbenton | amotoki, garyk1: yeah, i was dreaming and thinking all plugins would have stopped subclassing and subscribed to events ;) | 05:40 |
| amotoki | kevinbenton: garyk1: we need to change plugin code first, for example to mark default_sg of create_security_group and encourage to use some submethod like the reverted patch. | 05:41 |
| *** ociuhandu has quit IRC | 05:42 | |
| *** bana_k has quit IRC | 05:45 | |
| amotoki | kevinbenton: sorry. subscribing BEFORE_CREATE event is the right approach. | 05:46 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Fix ML2 revision_number handling in port updates https://review.openstack.org/373582 | 05:47 |
| *** gongysh has joined #openstack-neutron | 05:47 | |
| kevinbenton | amotoki: yeah, or PRECOMMIT | 05:47 |
| kevinbenton | amotoki: perhaps we should discuss at the summit doing that with most of the mixins | 05:48 |
| *** yedongcan has quit IRC | 05:49 | |
| *** Jeffrey4l has quit IRC | 05:49 | |
| *** Jeffrey4l has joined #openstack-neutron | 05:51 | |
| *** itsuugo has quit IRC | 05:53 | |
| *** itsuugo has joined #openstack-neutron | 05:54 | |
| *** hynekm has joined #openstack-neutron | 05:55 | |
| *** itsuugo has quit IRC | 05:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/371108 | 05:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/371109 | 06:00 |
| *** kobis has joined #openstack-neutron | 06:00 | |
| *** ganeshnalawade has joined #openstack-neutron | 06:00 | |
| *** oanson has joined #openstack-neutron | 06:00 | |
| *** itsuugo has joined #openstack-neutron | 06:01 | |
| *** markvoelker has quit IRC | 06:01 | |
| *** jhershbe has joined #openstack-neutron | 06:01 | |
| *** markvoelker has joined #openstack-neutron | 06:03 | |
| *** rkukura has joined #openstack-neutron | 06:03 | |
| *** oanson has quit IRC | 06:05 | |
| *** itsuugo has quit IRC | 06:05 | |
| *** itsuugo has joined #openstack-neutron | 06:07 | |
| *** rcernin has joined #openstack-neutron | 06:07 | |
| *** yedongcan has joined #openstack-neutron | 06:09 | |
| *** markvoelker has quit IRC | 06:11 | |
| *** tonytan4ever has quit IRC | 06:14 | |
| *** itsuugo has quit IRC | 06:17 | |
| *** itsuugo has joined #openstack-neutron | 06:18 | |
| *** jschwarz|ooo is now known as jschwarz | 06:19 | |
| jschwarz | hello :) | 06:19 |
| *** oshvartz has joined #openstack-neutron | 06:20 | |
| *** garyk1 has quit IRC | 06:21 | |
| *** namnh has joined #openstack-neutron | 06:25 | |
| *** iranzo has joined #openstack-neutron | 06:25 | |
| *** itsuugo has quit IRC | 06:25 | |
| *** ociuhandu has joined #openstack-neutron | 06:26 | |
| *** itsuugo has joined #openstack-neutron | 06:27 | |
| *** vthapar has joined #openstack-neutron | 06:28 | |
| *** ociuhandu has quit IRC | 06:31 | |
| *** korzen_ has joined #openstack-neutron | 06:32 | |
| *** andreas_s has joined #openstack-neutron | 06:32 | |
| *** ihrachys has joined #openstack-neutron | 06:33 | |
| hynekm | ihrachys, https://review.openstack.org/#/q/topic:bug/1609184+branch:stable/mitaka | 06:33 |
| openstackgerrit | Reedip proposed openstack/neutron-fwaas: [WIP]Add firewall_policy_id in FWaaS v2 https://review.openstack.org/370731 | 06:33 |
| *** jprovazn has joined #openstack-neutron | 06:35 | |
| *** itsuugo has quit IRC | 06:37 | |
| *** karthiks has joined #openstack-neutron | 06:37 | |
| *** itsuugo has joined #openstack-neutron | 06:38 | |
| *** oanson has joined #openstack-neutron | 06:39 | |
| *** rkukura has quit IRC | 06:42 | |
| *** itsuugo has quit IRC | 06:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/371108 | 06:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-dynamic-routing: Updated from global requirements https://review.openstack.org/373765 | 06:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/371109 | 06:44 |
| *** rkukura has joined #openstack-neutron | 06:44 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lib: Updated from global requirements https://review.openstack.org/373767 | 06:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/373768 | 06:44 |
| *** itsuugo has joined #openstack-neutron | 06:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/373772 | 06:47 |
| *** wolverineav has quit IRC | 06:48 | |
| *** wolverineav has joined #openstack-neutron | 06:49 | |
| *** lihi has joined #openstack-neutron | 06:49 | |
| *** lihiwish has joined #openstack-neutron | 06:50 | |
| *** wolverineav has quit IRC | 06:53 | |
| *** ababich has quit IRC | 06:55 | |
| *** ababich has joined #openstack-neutron | 06:56 | |
| openstackgerrit | Claudiu Belu proposed openstack/os-vif: Adds NoopPlugin https://review.openstack.org/362876 | 06:58 |
| openstackgerrit | Yang Yu proposed openstack/neutron-specs: Support agentless driver in neutron-dynamic-routing https://review.openstack.org/373029 | 06:59 |
| *** wolverineav has joined #openstack-neutron | 07:00 | |
| *** tmorin has joined #openstack-neutron | 07:00 | |
| *** itsuugo has quit IRC | 07:02 | |
| *** namnh has quit IRC | 07:02 | |
| *** ratailor has quit IRC | 07:03 | |
| *** itsuugo has joined #openstack-neutron | 07:03 | |
| openstackgerrit | Yang Yu proposed openstack/neutron-dynamic-routing: Refactor the code to support agentless BGP driver https://review.openstack.org/349921 | 07:03 |
| *** namnh has joined #openstack-neutron | 07:03 | |
| *** obondarev has joined #openstack-neutron | 07:04 | |
| *** wolverineav has quit IRC | 07:05 | |
| *** Sukhdev has quit IRC | 07:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/371108 | 07:07 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/371109 | 07:07 |
| *** itsuugo has quit IRC | 07:10 | |
| *** ganeshnalawade has quit IRC | 07:11 | |
| *** markvoelker has joined #openstack-neutron | 07:11 | |
| *** ganeshnalawade has joined #openstack-neutron | 07:12 | |
| *** ababich has quit IRC | 07:12 | |
| *** wolverineav has joined #openstack-neutron | 07:12 | |
| *** armax has quit IRC | 07:12 | |
| *** itsuugo has joined #openstack-neutron | 07:12 | |
| *** pcaruana has joined #openstack-neutron | 07:13 | |
| *** ababich has joined #openstack-neutron | 07:13 | |
| *** donghao has quit IRC | 07:15 | |
| *** markvoelker has quit IRC | 07:16 | |
| *** ratailor has joined #openstack-neutron | 07:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/371108 | 07:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/371109 | 07:18 |
| *** claudiub|2 has quit IRC | 07:19 | |
| *** zhhuabj_ has joined #openstack-neutron | 07:19 | |
| *** jpena|off is now known as jpena | 07:19 | |
| *** Alexey_Abashkin_ has quit IRC | 07:20 | |
| *** wolverineav has quit IRC | 07:20 | |
| *** matrohon has joined #openstack-neutron | 07:20 | |
| *** AlexeyAbashkin has joined #openstack-neutron | 07:23 | |
| *** ociuhandu has joined #openstack-neutron | 07:23 | |
| *** itsuugo has quit IRC | 07:25 | |
| *** itsuugo has joined #openstack-neutron | 07:27 | |
| *** jlanoux has joined #openstack-neutron | 07:27 | |
| *** ociuhandu has quit IRC | 07:28 | |
| *** cryptarium has joined #openstack-neutron | 07:28 | |
| *** sridharg has joined #openstack-neutron | 07:29 | |
| *** kevo has joined #openstack-neutron | 07:32 | |
| *** itsuugo has quit IRC | 07:33 | |
| *** ababich has quit IRC | 07:33 | |
| *** ababich has joined #openstack-neutron | 07:34 | |
| *** itsuugo has joined #openstack-neutron | 07:35 | |
| *** null0 has joined #openstack-neutron | 07:38 | |
| *** korzen_ has quit IRC | 07:40 | |
| *** lihiwish has quit IRC | 07:42 | |
| *** salv-orlando has joined #openstack-neutron | 07:42 | |
| *** wolverineav has joined #openstack-neutron | 07:44 | |
| *** fzdarsky has joined #openstack-neutron | 07:44 | |
| *** tonytan4ever has joined #openstack-neutron | 07:44 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix a release note typo for implicit provider loading deprecation https://review.openstack.org/373830 | 07:45 |
| *** chandanc_ has quit IRC | 07:45 | |
| *** salv-orl_ has quit IRC | 07:45 | |
| *** wolverineav has quit IRC | 07:48 | |
| openstackgerrit | Merged openstack/neutron: Revert "Do not retry default security group creation" https://review.openstack.org/373647 | 07:49 |
| *** tonytan4ever has quit IRC | 07:50 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: Do not retry default security group creation https://review.openstack.org/373841 | 07:51 |
| *** wolverineav has joined #openstack-neutron | 07:51 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: catch eventlet.Timeout exception https://review.openstack.org/372552 | 07:52 |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/python-neutronclient: Add required_keys to --subport argument https://review.openstack.org/373847 | 07:53 |
| *** Prem_ has quit IRC | 07:53 | |
| *** roeyc has joined #openstack-neutron | 07:54 | |
| *** obondarev has quit IRC | 07:56 | |
| *** ganeshnalawade has quit IRC | 07:57 | |
| *** ganeshnalawade has joined #openstack-neutron | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-neutron | 08:00 | |
| *** fzdarsky has quit IRC | 08:01 | |
| *** gongysh has quit IRC | 08:03 | |
| *** ganeshnalawade has quit IRC | 08:04 | |
| *** itsuugo has quit IRC | 08:04 | |
| *** sleviim has joined #openstack-neutron | 08:05 | |
| *** ganeshnalawade has joined #openstack-neutron | 08:05 | |
| *** ababich has quit IRC | 08:05 | |
| *** itsuugo has joined #openstack-neutron | 08:05 | |
| *** ababich has joined #openstack-neutron | 08:05 | |
| *** gongysh has joined #openstack-neutron | 08:06 | |
| ihrachys | kevinbenton: caught an interesting test failure in std attr tests: http://paste.fedoraproject.org/431803/45342147/ | 08:09 |
| ihrachys | kevinbenton: it's because test_standard_attr_resource_model_map defines some subclasses for the model, and probably sometimes they are not garbage collected | 08:10 |
| ihrachys | when the next call to get_standard_attr_resource_model_map hits | 08:10 |
| *** ababich has quit IRC | 08:12 | |
| *** wdeclercq has quit IRC | 08:12 | |
| *** ababich has joined #openstack-neutron | 08:12 | |
| *** donghao has joined #openstack-neutron | 08:12 | |
| *** akamyshnikova_ has joined #openstack-neutron | 08:14 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Garbage collect HasStandardAttributes subclasses in StandardAttrTestCase https://review.openstack.org/373868 | 08:14 |
| ihrachys | kevinbenton: the fix ^ | 08:15 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Garbage collect HasStandardAttributes subclasses in StandardAttrTestCase https://review.openstack.org/373868 | 08:16 |
| *** null0 has quit IRC | 08:16 | |
| *** ababich has quit IRC | 08:18 | |
| *** ababich has joined #openstack-neutron | 08:18 | |
| *** korzen_ has joined #openstack-neutron | 08:19 | |
| openstackgerrit | Merged openstack/neutron: Don't immediately restart in DHCP agent on port change https://review.openstack.org/372236 | 08:20 |
| *** wu_ng has quit IRC | 08:20 | |
| openstackgerrit | Merged openstack/neutron: Make DHCP agent use 'revision_number' https://review.openstack.org/373566 | 08:21 |
| *** wu_ng has joined #openstack-neutron | 08:21 | |
| openstackgerrit | Qiang Tang proposed openstack/neutron: Replace 'MagicMock' with 'Mock' https://review.openstack.org/373877 | 08:22 |
| *** ababich has quit IRC | 08:22 | |
| *** ababich has joined #openstack-neutron | 08:22 | |
| *** gvrangan has quit IRC | 08:23 | |
| *** Alex_Stef has joined #openstack-neutron | 08:24 | |
| *** claudiub has joined #openstack-neutron | 08:28 | |
| *** tmorin has quit IRC | 08:34 | |
| *** mhickey has joined #openstack-neutron | 08:38 | |
| *** ociuhandu has joined #openstack-neutron | 08:38 | |
| *** tmorin has joined #openstack-neutron | 08:39 | |
| *** iwamoto has joined #openstack-neutron | 08:39 | |
| *** ababich has quit IRC | 08:43 | |
| *** ociuhandu has quit IRC | 08:43 | |
| *** ababich has joined #openstack-neutron | 08:43 | |
| *** chandanc_ has joined #openstack-neutron | 08:47 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add basic status transition trunk scenario test https://review.openstack.org/372424 | 08:48 |
| *** pmannidi has quit IRC | 08:48 | |
| *** ababich has quit IRC | 08:50 | |
| *** john-davidge has joined #openstack-neutron | 08:51 | |
| *** ababich has joined #openstack-neutron | 08:51 | |
| *** bfernando has joined #openstack-neutron | 08:52 | |
| *** tobias-fiberdata has quit IRC | 08:53 | |
| *** john-dav_ has joined #openstack-neutron | 08:54 | |
| *** john-davidge has quit IRC | 08:54 | |
| *** jhershbe has quit IRC | 08:54 | |
| *** jlibosva has joined #openstack-neutron | 08:57 | |
| *** mfuruta has quit IRC | 08:58 | |
| *** mvk has quit IRC | 09:00 | |
| *** korzen is now known as korzen__ | 09:01 | |
| *** iranzo has quit IRC | 09:03 | |
| *** anilvenkata has joined #openstack-neutron | 09:03 | |
| *** iranzo has joined #openstack-neutron | 09:04 | |
| *** iranzo has joined #openstack-neutron | 09:04 | |
| *** efoley_ has joined #openstack-neutron | 09:04 | |
| *** obondarev has joined #openstack-neutron | 09:05 | |
| *** roeyc has quit IRC | 09:05 | |
| *** nijaba has quit IRC | 09:05 | |
| *** nijaba has joined #openstack-neutron | 09:06 | |
| *** nijaba has quit IRC | 09:06 | |
| *** nijaba has joined #openstack-neutron | 09:06 | |
| *** efoley__ has joined #openstack-neutron | 09:06 | |
| *** ababich has quit IRC | 09:08 | |
| *** korzen_ is now known as korzen | 09:08 | |
| *** korzen__ has left #openstack-neutron | 09:08 | |
| *** ababich has joined #openstack-neutron | 09:08 | |
| *** ociuhandu has joined #openstack-neutron | 09:08 | |
| *** pmannidi has joined #openstack-neutron | 09:09 | |
| *** pmannidi has quit IRC | 09:09 | |
| *** pmannidi has joined #openstack-neutron | 09:10 | |
| *** efoley_ has quit IRC | 09:10 | |
| *** apdibbo has joined #openstack-neutron | 09:11 | |
| *** ociuhandu has quit IRC | 09:13 | |
| *** wolverin_ has joined #openstack-neutron | 09:13 | |
| *** ababich has quit IRC | 09:13 | |
| matrohon | amotoki, hi | 09:14 |
| *** ababich has joined #openstack-neutron | 09:15 | |
| *** bks has joined #openstack-neutron | 09:16 | |
| *** wolverineav has quit IRC | 09:16 | |
| *** itsuugo has quit IRC | 09:16 | |
| *** itsuugo has joined #openstack-neutron | 09:17 | |
| *** ociuhandu has joined #openstack-neutron | 09:17 | |
| *** yushiro has quit IRC | 09:20 | |
| *** itsuugo has quit IRC | 09:22 | |
| *** tobias_fiberdata has joined #openstack-neutron | 09:22 | |
| amotoki | matrohon: hi | 09:22 |
| openstackgerrit | zhangyanxian proposed openstack/neutron: Fix typos in test_ipam_pluggable_backend.py & base.py https://review.openstack.org/373949 | 09:23 |
| *** itsuugo has joined #openstack-neutron | 09:23 | |
| matrohon | amotoki : I submitted a backport in horizon : | 09:23 |
| matrohon | https://review.openstack.org/#/c/373935/ | 09:23 |
| matrohon | amotoki, I've tried to explain the reason of this backport in comments | 09:24 |
| openstackgerrit | zhangyanxian proposed openstack/neutron: Fix typos in test_ipam_pluggable_backend.py & base.py https://review.openstack.org/373949 | 09:25 |
| amotoki | matrohon: i think #-horizon is approach channel. | 09:26 |
| matrohon | amotoki, I'm not logged in #horizon, but I'm jumping in :) | 09:26 |
| amotoki | matrohon: hehe:) | 09:27 |
| *** tobias_fiberdata has quit IRC | 09:28 | |
| *** ranjithd has quit IRC | 09:28 | |
| *** mvk has joined #openstack-neutron | 09:31 | |
| *** itsuugo has quit IRC | 09:34 | |
| *** itsuugo has joined #openstack-neutron | 09:35 | |
| *** yedongcan1 has joined #openstack-neutron | 09:36 | |
| *** nplanel has joined #openstack-neutron | 09:36 | |
| *** ganeshnalawade has quit IRC | 09:38 | |
| *** yedongcan has quit IRC | 09:40 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas: Fix an argument for an exception message https://review.openstack.org/373964 | 09:41 |
| *** iwamoto has quit IRC | 09:41 | |
| ihrachys | zzzeek: ping re metaclasses for declarative models | 09:45 |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Change default exception in wait_until_true https://review.openstack.org/373973 | 09:48 |
| *** wolverin_ has quit IRC | 09:53 | |
| *** wolverineav has joined #openstack-neutron | 09:53 | |
| *** wolverin_ has joined #openstack-neutron | 09:54 | |
| *** roeyc has joined #openstack-neutron | 09:55 | |
| *** john-dav_ is now known as john-davidge | 09:55 | |
| kevinbenton | ihrachys: so you saw MyModel from the tests leak out? | 09:56 |
| ihrachys | kevinbenton: yeah. you can repro it by calling gc.disable() at the end of the test. | 09:56 |
| ihrachys | kevinbenton: I was now looking into using a 'registry' metaclass for those subclasses that would apply to HasStandardAttributes automatically, and that would provide unregister class method for tests, but I struggle to make it work with sqlalchemy metaclass magic. | 09:57 |
| openstackgerrit | garyk proposed openstack/neutron: Remove deprecated class NeutronController https://review.openstack.org/372324 | 09:57 |
| ihrachys | kevinbenton: I want to talk it thru with zzzeek if it's even possible to have custom metaclasses on models | 09:57 |
| *** wolverineav has quit IRC | 09:57 | |
| *** raorn_ is now known as raorn | 10:00 | |
| *** yedongcan has joined #openstack-neutron | 10:01 | |
| *** rdo_ has quit IRC | 10:02 | |
| kevinbenton | ihrachys: well if you disable the garbage collector lots of stuff will break with our use of sqlalchemy | 10:03 |
| kevinbenton | :) | 10:03 |
| *** rdo_ has joined #openstack-neutron | 10:04 | |
| ihrachys | kevinbenton: yeah, but apparently that's what happens there - sometimes we reach get_**_map before classes are cleaned up by the collector | 10:04 |
| ihrachys | kevinbenton: relying on collector is probably not a good idea in general | 10:04 |
| *** yedongcan1 has quit IRC | 10:05 | |
| kevinbenton | ihrachys: but we do pretty heavily in the regular code | 10:07 |
| ihrachys | kevinbenton: like?.. | 10:07 |
| kevinbenton | ihrachys: the session stores weakrefs | 10:07 |
| kevinbenton | ihrachys: if they aren't removed as expected, we end up with the stale data problem | 10:08 |
| kevinbenton | ihrachys: maybe weakrefs are more deterministic than using __subclasses__ | 10:09 |
| kevinbenton | ihrachys: and will return None even if actual gc hasn't happened | 10:09 |
| ihrachys | I don't think that's how it works | 10:10 |
| ihrachys | "when the only remaining references to a referent are weak references, garbage collection is free to destroy the referent and reuse its memory for something else" | 10:10 |
| ihrachys | 'is free to' does not sound very strict | 10:10 |
| kevinbenton | ihrachys: right, so based on the determistic behavior we've had so far, weakrefs must return None even if the garbage collector hasn't kicked in | 10:11 |
| ihrachys | I don't see it in the wording, do you? | 10:11 |
| ihrachys | "if the referent is no longer alive, calling the reference object will cause None to be returned" | 10:12 |
| ihrachys | I suspect 'alive' is not really defined | 10:12 |
| kevinbenton | should be easy to test | 10:12 |
| ihrachys | but I expect it to be ~ 'not cleaned up by gc' | 10:12 |
| *** jhershbe has joined #openstack-neutron | 10:13 | |
| ihrachys | kevinbenton: how would you use those weakrefs though? you would still need to save classes with those refs somewhere, so you still need to have a metaclass to do it, don't you? | 10:14 |
| *** obondarev has quit IRC | 10:18 | |
| *** oanson has quit IRC | 10:19 | |
| *** jamespag` is now known as jamespage | 10:19 | |
| *** obondarev has joined #openstack-neutron | 10:20 | |
| *** itsuugo has quit IRC | 10:21 | |
| *** itsuugo has joined #openstack-neutron | 10:23 | |
| *** liuyulong__ has joined #openstack-neutron | 10:25 | |
| *** wu_ng has quit IRC | 10:27 | |
| *** liuyulong has quit IRC | 10:28 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: WIP: register standard attributes aware models with a metaclass https://review.openstack.org/374012 | 10:32 |
| *** itsuugo has quit IRC | 10:33 | |
| *** anilvenkata is now known as anilvenkata_afk | 10:34 | |
| *** itsuugo has joined #openstack-neutron | 10:34 | |
| *** hoangcx has quit IRC | 10:36 | |
| ihrachys | kevinbenton: that won't work right now, but that's the path I want to take ^ | 10:37 |
| *** sdague has joined #openstack-neutron | 10:37 | |
| kevinbenton | ihrachys: yeah, you get the declarativebase mess with that | 10:38 |
| kevinbenton | ihrachys: and now that lives in neutron-lib :) | 10:39 |
| ihrachys | kevinbenton: right. it's not the first time I need something like that, so maybe zzzeek will enlighten me how to do it right | 10:39 |
| *** obondarev has quit IRC | 10:39 | |
| *** obondarev has joined #openstack-neutron | 10:40 | |
| ihrachys | kevinbenton: well even if I make my metaclass declarative.DeclarativeMeta based, and pass metaclass=StandardAttributesRegistry to declarative_base, it does not seem to work. | 10:40 |
| *** itsuugo has quit IRC | 10:41 | |
| *** itsuugo has joined #openstack-neutron | 10:43 | |
| *** obondarev has quit IRC | 10:45 | |
| *** tonytan4ever has joined #openstack-neutron | 10:46 | |
| *** yedongcan has quit IRC | 10:49 | |
| openstackgerrit | Genadi Chereshnya proposed openstack/neutron: Checking functionality of DVR https://review.openstack.org/340383 | 10:49 |
| *** Jeffrey4l has quit IRC | 10:50 | |
| *** Jeffrey4l has joined #openstack-neutron | 10:50 | |
| *** yedongcan has joined #openstack-neutron | 10:51 | |
| *** tonytan4ever has quit IRC | 10:52 | |
| kevinbenton | ihrachys: https://github.com/python/cpython/blob/2.7/Include/weakrefobject.h#L66-L76 | 10:52 |
| kevinbenton | ihrachys: it does check refcount and return None even if it hasn't been collected | 10:52 |
| kevinbenton | ihrachys: not really relevant to our particular issue here, but it makes for interesting reading :) | 10:53 |
| ihrachys | kevinbenton: you are truly dedicated :) | 10:53 |
| ihrachys | kevinbenton: still we would need to have something to collect refs, so we are back to our metaclass problem :) | 10:53 |
| kevinbenton | ihrachys: right, the issue is that __subclasses__ doesn't appear to behave like weakrefs | 10:54 |
| ihrachys | kevinbenton: of course we could go without a metaclass, requiring a registration decorator to be applied to all such classes, but I hate that | 10:54 |
| *** claudiub has quit IRC | 10:54 | |
| *** gongysh has quit IRC | 10:56 | |
| *** davidlenwell has quit IRC | 10:57 | |
| *** claudiub has joined #openstack-neutron | 10:57 | |
| ihrachys | kevinbenton: I believe we still need carl's https://review.openstack.org/#/c/372794/ to avoid that race scenario, don't we? | 10:57 |
| ihrachys | kevinbenton: probably not a critical anymore because we give some time for ipam to process its part. | 10:57 |
| *** davidlenwell has joined #openstack-neutron | 10:58 | |
| *** itsuugo has quit IRC | 10:58 | |
| kevinbenton | ihrachys: yes, but even that patch won't be enough to truly close it | 10:58 |
| kevinbenton | ihrachys: because concurrent port updates can still confuse ipam if they both ask to remove the same IP | 10:59 |
| *** itsuugo has joined #openstack-neutron | 11:00 | |
| kevinbenton | ihrachys: that's why this patch is only Partial for the critical | 11:00 |
| ihrachys | kevinbenton: well, wouldn't it be legit in that case to raise the exception? | 11:00 |
| ihrachys | one of requesters succeed, another gets a notfound exception | 11:01 |
| kevinbenton | ihrachys: i don't think so. it's not the PUT semantic of the port API | 11:01 |
| kevinbenton | ihrachys: i haven't requested to remove an IP, i've requested to set the IPs to something specific | 11:01 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron: Linux Bridge: driver support for QoS egress minimum bandwidth https://review.openstack.org/357865 | 11:02 |
| ihrachys | kevinbenton: ok, then the complete fix would be catching this error in plugin and retrying allocation (plus maybe cleaning up what we just allocated in db)? | 11:02 |
| kevinbenton | ihrachys: yeah, something along those lines | 11:03 |
| kevinbenton | ihrachys: armando had a patch that would catch and reraise as retry | 11:03 |
| kevinbenton | ihrachys: i'll let carl decide where to put it | 11:03 |
| kevinbenton | ihrachys: semantically it does make sense in ipam itself to throw that error because we ask it to deallocate something | 11:04 |
| kevinbenton | ihrachys: it just shouldn't make it out via the port API | 11:04 |
| kevinbenton | ihrachys: but you're correct that the bug can probably be significantly dropped in severity now | 11:04 |
| ihrachys | ack. though since we already made some port changes, we can't just retry, we need to revert whatever we committed the first try | 11:04 |
| *** chandanc_ has quit IRC | 11:05 | |
| kevinbenton | ihrachys: we're in a transaction a retry is going to rollback | 11:06 |
| *** lilintan has joined #openstack-neutron | 11:06 | |
| kevinbenton | ihrachys: from an IPAM driver perspective we may need to do some undo calls for non-db drivers | 11:06 |
| ihrachys | kevinbenton: are we? I somehow thought that a call to ipam subsystem is under a separate transaction. | 11:06 |
| kevinbenton | ihrachys: no, unfortunately not | 11:06 |
| * ihrachys reads code | 11:06 | |
| *** namnh has quit IRC | 11:07 | |
| kevinbenton | ihrachys: https://github.com/openstack/neutron/blob/master/neutron/db/db_base_plugin_v2.py#L1201-L1221 | 11:07 |
| ihrachys | kevinbenton: so we actually talk to external ipam providers while keeping transaction open? nice.</irony> | 11:07 |
| *** ijw has joined #openstack-neutron | 11:07 | |
| kevinbenton | ihrachys: yep :) | 11:08 |
| ihrachys | on the other side, it's easier to roll back then :) | 11:08 |
| *** rtheis has joined #openstack-neutron | 11:08 | |
| kevinbenton | ihrachys: i think carl has a TODO somewhere to capture when the whole port is rolled back outside of IPAM to notify the drivers | 11:08 |
| kevinbenton | ihrachys: there is quite a bit of work internally in ipam to capture exceptions and issue delete calls to the drivers | 11:09 |
| kevinbenton | ihrachys: (it's a NOOP for the in-tree db-based one) | 11:09 |
| *** vthapar has quit IRC | 11:09 | |
| kevinbenton | ihrachys: but that doesn't help for rollbacks triggered by something outside of ipam | 11:09 |
| kevinbenton | ihrachys: ah, subclasses does use weakrefs | 11:11 |
| kevinbenton | ihrachys: but apparently the mro stuff creates a cyclic reference | 11:11 |
| kevinbenton | ihrachys: which the garbage collector has to detect and destroy | 11:11 |
| kevinbenton | ihrachys: http://www.gossamer-threads.com/lists/python/python/626283 | 11:12 |
| *** yamamoto has quit IRC | 11:12 | |
| *** ijw has quit IRC | 11:12 | |
| kevinbenton | ihrachys: so if you can't get a metaclass fix working, i'd be okay with either the explicit gc.collect to break that cyclic ref or just putting a skip on that test | 11:13 |
| kevinbenton | ihrachys: not sure how often this is hitting in the gate | 11:14 |
| ihrachys | kevinbenton: once so far | 11:14 |
| ihrachys | kevinbenton: but I was lucky to hit it :) | 11:14 |
| kevinbenton | ihrachys: want to just propose a skip really quick with a ref to the bug? | 11:14 |
| ihrachys | kevinbenton: though the test is rather fresh, so it's probably not once in a lifetie | 11:14 |
| ihrachys | *lifetime | 11:14 |
| ihrachys | kevinbenton: well I would go with gc.collect for the start. | 11:14 |
| ihrachys | kevinbenton: https://review.openstack.org/#/c/373868/ | 11:15 |
| ihrachys | kevinbenton: I will work on a metaclass solution on top of it. | 11:15 |
| kevinbenton | ihrachys: ack | 11:15 |
| *** anilvenkata_afk is now known as anilvenkata | 11:15 | |
| *** baoli has joined #openstack-neutron | 11:15 | |
| kevinbenton | ihrachys: there is another option, i was just unfamiliar with sqlalchemy to dig into it | 11:17 |
| kevinbenton | ihrachys: there is probably already a way to as the declarative base for its children | 11:17 |
| kevinbenton | ihrachys: if we can do that, we can iterate it looking for instances of HasStandardAttributes | 11:18 |
| kevinbenton | to ask* | 11:18 |
| *** baoli_ has joined #openstack-neutron | 11:18 | |
| *** lucas-afk is now known as lucasagomes | 11:18 | |
| kevinbenton | ihrachys: http://docs.sqlalchemy.org/en/latest/orm/extensions/declarative/api.html#sqlalchemy.ext.declarative.declarative_base.params.class_registry | 11:19 |
| ihrachys | kevinbenton: hm, that's an interesting suggestion | 11:19 |
| ihrachys | kevinbenton: re link, meaning we need to modify neutron-lib first to capture that? | 11:20 |
| kevinbenton | ihrachys: unfortunately we may | 11:20 |
| *** baoli has quit IRC | 11:21 | |
| *** boden has joined #openstack-neutron | 11:22 | |
| kevinbenton | ihrachys: or we poke at _decl_class_registry ourselves... | 11:23 |
| ihrachys | kevinbenton: underscores make me not sleep well :) I will send a patch for neutron-lib | 11:24 |
| kevinbenton | ihrachys: yeah | 11:24 |
| *** rmart04 has joined #openstack-neutron | 11:26 | |
| *** itsuugo has quit IRC | 11:28 | |
| openstackgerrit | Tony Xu proposed openstack/neutron: log info is not reasonable enough https://review.openstack.org/374051 | 11:28 |
| *** nicolasbock has joined #openstack-neutron | 11:28 | |
| * ihrachys kevinbenton: hm, on second thought, wouldn't the registry still contain the test classes while they are still alive? | 11:29 | |
| *** mkolesni_ has quit IRC | 11:29 | |
| *** itsuugo has joined #openstack-neutron | 11:29 | |
| kevinbenton | ihrachys: oh, duh yeah | 11:29 |
| ihrachys | ok, my worldview is not shattered | 11:30 |
| kevinbenton | ihrachys: well if there is a dict storing this stuff we can pop the keys | 11:30 |
| *** garyk1 has joined #openstack-neutron | 11:30 | |
| garyk1 | ihrachys: can you please hit https://review.openstack.org/#/c/372324/ | 11:30 |
| kevinbenton | ihrachys: so test teardown would explicitly remove from the dict we put in class_registry | 11:30 |
| ihrachys | kevinbenton: that we could do only under assumption that the declarative_base logic does not depend on the dict contents in some way | 11:30 |
| garyk1 | kevinbenton: do you not sleep | 11:31 |
| *** ociuhandu has quit IRC | 11:32 | |
| kevinbenton | ihrachys: well we want to break anything depending on those test classes :) | 11:32 |
| kevinbenton | garyk1: i sleep in the morning :) | 11:32 |
| jschwarz | haleyb, hey | 11:33 |
| *** obondarev has joined #openstack-neutron | 11:33 | |
| kevinbenton | garyk1: pretend i live in hong kong | 11:33 |
| jschwarz | haleyb, re: https://bugs.launchpad.net/neutron/+bug/1619312 | 11:34 |
| openstack | Launchpad bug 1619312 in neutron "dvr: can't migrate legacy router to DVR" [High,Fix released] - Assigned to Brian Haley (brian-haley) | 11:34 |
| ihrachys | kevinbenton: as long as you don't break anything that does not... I mean, you can't be sure that the declarative_base is resistant against dict modifications from outside the declarative_base itself. but yeah, I would probably be surprised if it does depend on it being untouched because the dict is supposed to be passed into multiple bases. | 11:34 |
| ihrachys | I guess I will talk to zzzeek on the matter first, both on safety of the dict as well as metaclasses. | 11:35 |
| *** lilintan has quit IRC | 11:36 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Change default exception in wait_until_true https://review.openstack.org/373973 | 11:37 |
| kevinbenton | ihrachys: sounds good | 11:37 |
| kevinbenton | ihrachys: have a sec to review https://review.openstack.org/#/c/373956/ really quick? | 11:38 |
| *** ociuhandu has joined #openstack-neutron | 11:38 | |
| *** lujinluo has quit IRC | 11:39 | |
| *** itsuugo has quit IRC | 11:39 | |
| ihrachys | kevinbenton: done | 11:39 |
| kevinbenton | ihrachys: thx | 11:40 |
| *** itsuugo has joined #openstack-neutron | 11:40 | |
| *** ganeshnalawade has joined #openstack-neutron | 11:42 | |
| garyk1 | kevinbenton: :) | 11:44 |
| openstackgerrit | Merged openstack/neutron: tests: Don't raise TimeoutError when waiting for ping https://review.openstack.org/369521 | 11:45 |
| *** lilintan has joined #openstack-neutron | 11:49 | |
| *** edmondsw has joined #openstack-neutron | 11:49 | |
| *** itsuugo has quit IRC | 11:52 | |
| *** itsuugo has joined #openstack-neutron | 11:53 | |
| *** bks has quit IRC | 11:57 | |
| *** jpena is now known as jpena|lunch | 11:57 | |
| *** amotoki has quit IRC | 11:58 | |
| *** rodrigods has quit IRC | 11:59 | |
| *** rodrigods has joined #openstack-neutron | 11:59 | |
| *** ganeshnalawade has quit IRC | 11:59 | |
| *** wuhg has quit IRC | 12:01 | |
| *** vijaykc4 has joined #openstack-neutron | 12:02 | |
| *** NightKhaos has quit IRC | 12:03 | |
| *** efoley_ has joined #openstack-neutron | 12:05 | |
| *** efoley__ has quit IRC | 12:07 | |
| openstackgerrit | Inessa Vasilevskaya proposed openstack/neutron: Fix TypeError in native of_interface _get_dpid https://review.openstack.org/358706 | 12:08 |
| *** NightKhaos has joined #openstack-neutron | 12:09 | |
| *** mkolesni_ has joined #openstack-neutron | 12:09 | |
| *** ratailor has quit IRC | 12:11 | |
| *** amuller has joined #openstack-neutron | 12:11 | |
| *** hyakuhei has joined #openstack-neutron | 12:18 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: DNM: Investigate bug 1609540 and probably 1540983 https://review.openstack.org/374106 | 12:24 |
| openstack | bug 1609540 in neutron "Deleting csnat port fails due to no fixed ips" [High,In progress] https://launchpad.net/bugs/1609540 - Assigned to Carl Baldwin (carl-baldwin) | 12:24 |
| *** crose has joined #openstack-neutron | 12:25 | |
| *** yamamoto has joined #openstack-neutron | 12:26 | |
| rcernin | Hi! Is it possible to restrict neutron to check the uniqueneess on the network names ? | 12:27 |
| *** yamamoto_ has joined #openstack-neutron | 12:28 | |
| *** yfried has quit IRC | 12:28 | |
| *** zhhuabj_ has quit IRC | 12:29 | |
| *** itsuugo has quit IRC | 12:30 | |
| *** yamamoto has quit IRC | 12:32 | |
| *** itsuugo has joined #openstack-neutron | 12:32 | |
| *** markvoelker has joined #openstack-neutron | 12:33 | |
| *** efoley_ has quit IRC | 12:36 | |
| *** zkassab has joined #openstack-neutron | 12:36 | |
| *** obondarev has quit IRC | 12:38 | |
| akamyshnikova_ | rcernin, hi, the uniqueness of name is not restricted. ID for network is unique indicator. | 12:38 |
| *** janzian has joined #openstack-neutron | 12:39 | |
| *** ganeshnalawade has joined #openstack-neutron | 12:40 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Garbage collect HasStandardAttributes subclasses in StandardAttrTestCase https://review.openstack.org/373868 | 12:40 |
| rcernin | akamyshnikova_: yeah, we just discussed and found also this : https://bugs.launchpad.net/neutron/+bug/1624103/ but some third party plugin crashes with network with same name/subnet range | 12:41 |
| openstack | Launchpad bug 1624103 in neutron "Neutron accepting multiple networks/subnets to be created with same name, network address" [Undecided,Invalid] | 12:41 |
| *** yamamoto_ has quit IRC | 12:42 | |
| rcernin | akamyshnikova_: so my question was if we could restrict same names/subnets optionally. | 12:42 |
| *** obondarev has joined #openstack-neutron | 12:43 | |
| *** garyk1 has quit IRC | 12:45 | |
| akamyshnikova_ | rcernin, I don't think so, I guess the tests should be changed for checking ids instead of names. | 12:48 |
| *** yamamoto has joined #openstack-neutron | 12:49 | |
| *** anilvenkata has quit IRC | 12:49 | |
| *** zhhuabj_ has joined #openstack-neutron | 12:54 | |
| *** dronshaw has joined #openstack-neutron | 12:54 | |
| *** mriedem has joined #openstack-neutron | 12:54 | |
| *** itsuugo has quit IRC | 12:55 | |
| *** crose has quit IRC | 12:56 | |
| *** david-lyle has joined #openstack-neutron | 12:56 | |
| *** itsuugo has joined #openstack-neutron | 12:57 | |
| *** roeyc has quit IRC | 12:57 | |
| *** krtaylor has joined #openstack-neutron | 12:57 | |
| *** roeyc has joined #openstack-neutron | 12:59 | |
| *** roeyc has quit IRC | 13:00 | |
| *** jpena|lunch is now known as jpena | 13:00 | |
| *** liuyulong has joined #openstack-neutron | 13:00 | |
| *** liuyulong__ has quit IRC | 13:02 | |
| *** ranjithd has joined #openstack-neutron | 13:02 | |
| *** amotoki has joined #openstack-neutron | 13:03 | |
| *** roeyc has joined #openstack-neutron | 13:05 | |
| *** vishwanathj has joined #openstack-neutron | 13:06 | |
| *** mohankumar has joined #openstack-neutron | 13:07 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Garbage collect HasStandardAttributes subclasses in StandardAttrTestCase https://review.openstack.org/373868 | 13:07 |
| *** vijaykc4 has quit IRC | 13:07 | |
| jlibosva | ihrachys: wanted to save you some time :) | 13:09 |
| *** gkadam has quit IRC | 13:10 | |
| *** pgadiya has quit IRC | 13:10 | |
| ihrachys | jlibosva: you did!!! :))) | 13:11 |
| *** tbachman has joined #openstack-neutron | 13:11 | |
| *** liuyulong__ has joined #openstack-neutron | 13:11 | |
| *** yfried has joined #openstack-neutron | 13:11 | |
| *** wolverin_ has quit IRC | 13:16 | |
| *** tbachman has quit IRC | 13:16 | |
| *** tbachman has joined #openstack-neutron | 13:17 | |
| *** wolverineav has joined #openstack-neutron | 13:17 | |
| *** rossella_s has quit IRC | 13:19 | |
| *** rossella_s has joined #openstack-neutron | 13:19 | |
| *** tbachman has quit IRC | 13:20 | |
| *** tbachman_ has joined #openstack-neutron | 13:20 | |
| *** gouthamr has joined #openstack-neutron | 13:21 | |
| *** wolverineav has quit IRC | 13:22 | |
| *** wolverineav has joined #openstack-neutron | 13:25 | |
| hynekm | ihrachys, when you have time, could you pls re-review https://review.openstack.org/#/q/topic:bug/1609184+branch:stable/mitaka | 13:28 |
| ihrachys | hynekm: ack. I missed that you reuploaded. doing. | 13:29 |
| *** lilintan has quit IRC | 13:29 | |
| hynekm | ihrachys, thanks | 13:29 |
| *** roeyc has quit IRC | 13:33 | |
| *** wolverineav has quit IRC | 13:33 | |
| *** wolverineav has joined #openstack-neutron | 13:34 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: tests: catch eventlet.Timeout exception https://review.openstack.org/372552 | 13:34 |
| *** kb has joined #openstack-neutron | 13:35 | |
| *** ijw has joined #openstack-neutron | 13:38 | |
| *** wolverineav has quit IRC | 13:38 | |
| *** prateek has quit IRC | 13:39 | |
| *** ushkalim has quit IRC | 13:39 | |
| *** ushkalim_ has quit IRC | 13:39 | |
| haleyb | jschwarz: hey there. so that bug should be fixed | 13:40 |
| *** nplanel has quit IRC | 13:40 | |
| jschwarz | haleyb, aye | 13:40 |
| jschwarz | haleyb, I add a problem where I migrated a router from legacy to dvr and i lost connectivity | 13:40 |
| jschwarz | haleyb, alas, I blamed PEBKAC as I couldn't reproduce this anymore | 13:41 |
| jschwarz | so... :) | 13:41 |
| *** kb has quit IRC | 13:41 | |
| *** kberger has joined #openstack-neutron | 13:41 | |
| hynekm | kevinbenton, if you have time, could you please review backport to mitaka: https://review.openstack.org/#/q/topic:bug/1609184+branch:stable/mitaka ? | 13:42 |
| *** salv-orl_ has joined #openstack-neutron | 13:42 | |
| haleyb | jschwarz: well, the port has to be down to migrate, so i'd expect something to get lost | 13:42 |
| jschwarz | haleyb, even after I brought the router back up it didn't work | 13:43 |
| *** wolverineav has joined #openstack-neutron | 13:43 | |
| *** ijw has quit IRC | 13:43 | |
| jschwarz | haleyb, so I restarted the services (as the last time I messed around with them was last week) and it worked | 13:43 |
| jschwarz | probably some rouge rule or something | 13:43 |
| haleyb | or an ARP update didn't go out | 13:44 |
| *** salv-orlando has quit IRC | 13:45 | |
| kevinbenton | hynekm: what's the context of these backports? | 13:46 |
| jschwarz | haleyb, most probably | 13:47 |
| *** wolverineav has quit IRC | 13:47 | |
| hynekm | kevinbenton, the context is that the precommit should allow retries to mitigate itermittent failures. To allow the retries, the other three commits in the query need to be backported too. | 13:48 |
| *** ganeshnalawade has quit IRC | 13:48 | |
| kevinbenton | hynekm: i see, so you ran into a driver throwing db errors? | 13:48 |
| ihrachys | kevinbenton: also handling staledataerror for retries | 13:48 |
| *** baoli_ has quit IRC | 13:48 | |
| kevinbenton | ihrachys: yeah, the staledata one makes sense. i'm just curious why all of the multiple exception stuff was needed with our limited rollout of registry stuff in mitaka | 13:49 |
| *** wolverineav has joined #openstack-neutron | 13:49 | |
| kevinbenton | but a mech driver issue makes sense | 13:50 |
| ihrachys | kevinbenton: well, just catching staledataerror would not work for callbackerrors, so they compliment each other. | 13:50 |
| *** yedongcan has left #openstack-neutron | 13:50 | |
| *** yfried has quit IRC | 13:51 | |
| ihrachys | it would be probably not complete to catch just some stales | 13:51 |
| *** tri2sing has joined #openstack-neutron | 13:51 | |
| ihrachys | but I admit the changes are not of obvious class of backports. | 13:51 |
| *** tonytan4ever has joined #openstack-neutron | 13:52 | |
| kevinbenton | ihrachys: i mean in mitaka the few callback notifications that are allowed to raise errors are usually caught by the notify caller and converted to something else | 13:54 |
| kevinbenton | ihrachys: but for mech drivers i can see this being an issue | 13:54 |
| *** tongli has joined #openstack-neutron | 13:54 | |
| ihrachys | right. hynekm do you have a rhbzbug to link to? | 13:55 |
| ihrachys | just to give some context | 13:55 |
| hynekm | kevinbenton, it is beneficial to have db deadlock as well as staledata handled in the same way, which is done in this set of commits - hence I think backporting the relevant commits in (almost) full is cleaner way. | 13:55 |
| hynekm | ihrachys, kevinbenton, it is https://bugzilla.redhat.com/show_bug.cgi?id=1377518 | 13:55 |
| openstack | bugzilla.redhat.com bug 1377518 in openstack-neutron "Allow retry on DB error for precommit" [Urgent,Assigned] - Assigned to hmlnarik | 13:55 |
| ihrachys | jlibosva: have we seen such failures before? http://logs.openstack.org/68/373868/4/check/gate-neutron-dsvm-functional-ubuntu-trusty/4de275e/testr_results.html.gz | 13:55 |
| *** akshai has joined #openstack-neutron | 13:55 | |
| ihrachys | jlibosva: something crashes, then the test times out after 180 secs | 13:55 |
| kevinbenton | hynekm: yes, i understand the staledata. i was wondering about the multipleexception handling stuff | 13:56 |
| kevinbenton | hynekm: but i see it's for ML2 | 13:56 |
| *** wolverineav has quit IRC | 13:56 | |
| ihrachys | hynekm: we could in theory just make is_retriable handle two types - stale and deadlock - without handling multipleexceptions | 13:57 |
| *** lifeless has quit IRC | 13:57 | |
| ihrachys | hynekm: but I don't think it's a bad thing we do :) | 13:57 |
| ihrachys | kevinbenton: so according to the rhbz bug, cisco plugin is hit in precommit | 13:57 |
| *** r-daneel has joined #openstack-neutron | 13:58 | |
| jlibosva | ihrachys: I don't recall. We had an issue where ovsdb hangs forever, then test timed out after 180 seconds and got stuck on cleanup until global timeout expired and running tests were sigkilled | 13:58 |
| hynekm | ihrachys, kevinbenton, true, in master the multipleexception is used. If there were another type of exception raised in the code, that is already handled in master, this way makes backport more straightforward | 13:58 |
| openstackgerrit | Merged openstack/os-vif: vif: stop VIFOpenVSwitch inheriting VIFBridge https://review.openstack.org/371562 | 13:59 |
| *** sdake has joined #openstack-neutron | 13:59 | |
| *** lifeless has joined #openstack-neutron | 13:59 | |
| hynekm | ihrachys, kevinbenton, ... thus safer | 13:59 |
| kevinbenton | hynekm, ihrachys: lgtm. it's not that invasive | 13:59 |
| *** sdake has quit IRC | 13:59 | |
| hynekm | kevinbenton, thanks | 14:00 |
| *** sdake has joined #openstack-neutron | 14:00 | |
| ihrachys | jlibosva: that's db, both mysql btw | 14:00 |
| ihrachys | jlibosva: with same index tuple error | 14:01 |
| ihrachys | not sure where it comes from though | 14:01 |
| ajo | njohnston, slaweq , ihrachys did we plan QoS meeting today? | 14:02 |
| *** wolverineav has joined #openstack-neutron | 14:02 | |
| ihrachys | ajo: it's in my call, but I don't join | 14:02 |
| ajo | I guess we have nothing important, I can update on nova placement API details, but that can also happen via email :) | 14:02 |
| jschwarz | haleyb, do you have a minute to discuss our Lord and Saviour, floating IPs in DVR? | 14:03 |
| ihrachys | ajo: but generally for the cycle, I'd like to give validation enhancements a priority before we introduce more rules/apis | 14:03 |
| ajo | ihrachys, makes sense | 14:03 |
| ihrachys | ajo: it becomes insanely hard to use the thing | 14:03 |
| *** davidsha has joined #openstack-neutron | 14:03 | |
| ajo | agreed | 14:03 |
| ajo | slaweq, is aware of it | 14:03 |
| ajo | afaik | 14:03 |
| ajo | :) | 14:03 |
| ihrachys | ok cool. I just saw some patches for new api flying around, and thought it's part of your thinking. | 14:04 |
| haleyb | jschwarz: does this have to do with the failure to send ARPs sometimes seen? | 14:04 |
| jschwarz | haleyb, no? :) | 14:04 |
| ihrachys | jlibosva: in a good run, the test takes like 5 secs | 14:04 |
| jschwarz | haleyb, I have a setup working and I'm trying to understand the packet lifecycle when sending packets from "The InterWebz" to an instance through a floating IP | 14:05 |
| jlibosva | ihrachys: so it gets stuck on a call between db and neutron? | 14:05 |
| haleyb | jschwarz: ok | 14:05 |
| jlibosva | ihrachys: both exception comes from pymysql/connections.py | 14:05 |
| ihrachys | jlibosva: I don't think the test even starts any neutron code. it's alembic walking. | 14:06 |
| *** andymaier has joined #openstack-neutron | 14:06 | |
| jschwarz | haleyb, specifically looking at the fip namespace, I don't see any ip route rules directing traffic back to br-int | 14:06 |
| ihrachys | jlibosva: we don't have dedicated logs for the type of tests, so it's hard to get more details | 14:06 |
| *** jhershbe has quit IRC | 14:07 | |
| *** Swami has joined #openstack-neutron | 14:08 | |
| *** Swami_ has joined #openstack-neutron | 14:08 | |
| *** Swami_ has quit IRC | 14:08 | |
| *** Swami has quit IRC | 14:08 | |
| *** Swami has joined #openstack-neutron | 14:08 | |
| *** oshvartz has quit IRC | 14:09 | |
| haleyb | jschwarz: let me get in a devstack to follow along, but thought the routing rules would direct traffic into the fpr devices | 14:09 |
| *** marst has quit IRC | 14:09 | |
| *** wolverineav has quit IRC | 14:10 | |
| jschwarz | haleyb, ack, thanks | 14:11 |
| slaweq | ajo, ihrachys: yes, I know | 14:11 |
| electrocucaracha | korzen: ihrachys ping? | 14:12 |
| slaweq | but for now I think ajo should first make his "rename/change" of notification drivers | 14:12 |
| ajo | slaweq, ack true | 14:12 |
| ihrachys | slaweq: that one too | 14:12 |
| *** pradk_ has joined #openstack-neutron | 14:13 | |
| ihrachys | electrocucaracha: hey | 14:13 |
| electrocucaracha | korzen, ihrachys, I tried to collect all the points discussed during the last two upgrades meetings, can you take a look and complete it https://etherpad.openstack.org/p/neutron_upgrades_status | 14:13 |
| slaweq | so, I have to rebase my patch with validation on master but I was not doing it yet because I'm waiting for decisions notification driver :) | 14:13 |
| jschwarz | haleyb, I see lots of rules going outside (to 172.24.4.XXX), but none to the private network (10.0.0.XXX) | 14:13 |
| jschwarz | haleyb, also I don't see additional rule tables, hence my question | 14:14 |
| * electrocucaracha never think about how many things can be discussed in two hours :S | 14:14 | |
| *** vijaykc4 has joined #openstack-neutron | 14:15 | |
| korzen | electrocucaracha, 2 hours of summarizing 2 weeks of work | 14:15 |
| *** vikram_ has quit IRC | 14:16 | |
| *** pbandark has quit IRC | 14:16 | |
| electrocucaracha | korzen: yeah, also covering questions, concerns, discussions of a dozen of people | 14:17 |
| ihrachys | electrocucaracha: I will, but tomorrow. does it fit you? | 14:17 |
| haleyb | jschwarz: arg, this whole time i've been playing in a devstack that is legacy, not dvr | 14:17 |
| electrocucaracha | ihrachys: no problem, as long as we send it on time | 14:18 |
| electrocucaracha | ihrachys: I mean, not to wait until the next biweekly report | 14:18 |
| electrocucaracha | ihrachys: and at least all the information is collected so I think that it's easier to complete it | 14:19 |
| korzen | electrocucaracha, note that bi-weekly means once in 3/4 weeks :D | 14:19 |
| electrocucaracha | korzen: hahaha | 14:19 |
| *** vijaykc4 has quit IRC | 14:20 | |
| *** cbouch has joined #openstack-neutron | 14:21 | |
| *** shausy has quit IRC | 14:22 | |
| dasm | korzen: "every other second week"? | 14:22 |
| *** shausy has joined #openstack-neutron | 14:22 | |
| *** marst has joined #openstack-neutron | 14:22 | |
| *** pradk has quit IRC | 14:26 | |
| *** pradk_ is now known as pradk | 14:26 | |
| *** mohankumar has quit IRC | 14:27 | |
| *** shausy has quit IRC | 14:28 | |
| *** yamahata has joined #openstack-neutron | 14:28 | |
| *** iyamahat has joined #openstack-neutron | 14:29 | |
| mugsie | is there a way of listing ports that are not attached to anything in the neutron CLI ? | 14:30 |
| openstackgerrit | Bernard Cafarelli proposed openstack/neutron: Install dibbler.filters rootwrap file https://review.openstack.org/374197 | 14:30 |
| bcafarel | ^ just a one-liner in setup.cfg | 14:30 |
| ihrachys | jlibosva: fyi I also spotted a lack of isolation between ovs func tests lately https://bugs.launchpad.net/neutron/+bug/1625724 | 14:30 |
| openstack | Launchpad bug 1625724 in neutron "some functional ovs tests are not mutually isolated" [High,New] | 14:30 |
| jlibosva | ihrachys: good job, I saw the bug report though I haven't looked at it yet | 14:31 |
| jlibosva | oh I see | 14:31 |
| *** jlanoux has quit IRC | 14:32 | |
| *** caboucha has joined #openstack-neutron | 14:32 | |
| *** caboucha has quit IRC | 14:32 | |
| *** ushkalim has joined #openstack-neutron | 14:32 | |
| *** yamamoto has quit IRC | 14:32 | |
| *** nplanel has joined #openstack-neutron | 14:33 | |
| *** moshele has quit IRC | 14:33 | |
| rtheis | kevinbenton: are there still gate fixes in progress for DHCP? The networking-ovn gate is struggle with DHCP issues which may be related to force_metadata | 14:34 |
| *** jheroux has joined #openstack-neutron | 14:35 | |
| rtheis | networking-ovn sets it to True and may explain why only networking-ovn is hitting https://bugs.launchpad.net/neutron/+bug/1624079 | 14:35 |
| openstack | Launchpad bug 1624079 in neutron "KeyError on "subnet_dhcp_ip = subnet_to_interface_ip[subnet.id]"" [High,Confirmed] | 14:35 |
| *** john-davidge has quit IRC | 14:37 | |
| *** vijaykc4 has joined #openstack-neutron | 14:37 | |
| *** caboucha has joined #openstack-neutron | 14:38 | |
| *** gvrangan has joined #openstack-neutron | 14:39 | |
| ihrachys | jlibosva: oooh, that's a known bug in the db driver it seems: http://lists.openstack.org/pipermail/openstack-dev/2016-September/103665.html | 14:40 |
| *** akshai has quit IRC | 14:40 | |
| *** vijaykc4 has quit IRC | 14:41 | |
| *** sleviim has quit IRC | 14:42 | |
| *** mlavalle has joined #openstack-neutron | 14:42 | |
| *** akshai has joined #openstack-neutron | 14:43 | |
| *** crose has joined #openstack-neutron | 14:44 | |
| ihrachys | jlibosva: the last email in the thread: "tl;dr this appears to have been around forever (at least since we switched to using a pure-Python MySQL client) and is almost certainly completely unrelated to any particular release of oslo.db." | 14:44 |
| ihrachys | so we are probably 'good' | 14:44 |
| *** cleong has joined #openstack-neutron | 14:45 | |
| *** vijaykc4 has joined #openstack-neutron | 14:45 | |
| *** phil_h has joined #openstack-neutron | 14:48 | |
| *** akshai has quit IRC | 14:49 | |
| *** ihrachys has quit IRC | 14:52 | |
| *** akshai has joined #openstack-neutron | 14:52 | |
| electrocucaracha | ankur-gupta-f: ping? | 14:52 |
| *** EinstCrazy has joined #openstack-neutron | 14:52 | |
| *** jprovazn has quit IRC | 14:53 | |
| haleyb | jschwarz: so there should be an ip route for the FIP in the fip namespace sending it into the fpr, there will be no routes for any private networks since the namespace is shared | 14:53 |
| *** baoli has joined #openstack-neutron | 14:54 | |
| electrocucaracha | korzen: ping? | 14:54 |
| korzen | pong | 14:54 |
| openstackgerrit | tonytan4ever proposed openstack/neutron: DistributedVirtualRouter mac address to OVO https://review.openstack.org/304873 | 14:54 |
| electrocucaracha | korzen: I was reviewing and rebasing router ovo patch | 14:54 |
| electrocucaracha | korzen: and regarding this moment https://review.openstack.org/#/c/307964/20/neutron/objects/router.py@1 | 14:55 |
| *** jprovazn has joined #openstack-neutron | 14:55 | |
| *** fzdarsky has joined #openstack-neutron | 14:55 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Remove logging during plugin initialization https://review.openstack.org/374225 | 14:55 |
| electrocucaracha | korzen: afair, we decided to keep ovo in objects folder with few exceptions | 14:55 |
| electrocucaracha | korzen: I'm not sure if router is an exception | 14:56 |
| boden | HenryG dougwig: do we have anything to discuss for neutron-lib meeting today? | 14:56 |
| *** EinstCrazy has quit IRC | 14:56 | |
| *** mkolesni has joined #openstack-neutron | 14:57 | |
| *** obondarev has quit IRC | 14:57 | |
| *** mkolesni_ has quit IRC | 14:57 | |
| korzen | electrocucaracha, can you elaborate more? what exceptions? | 14:57 |
| *** caboucha has quit IRC | 14:57 | |
| HenryG | boden: I have nothing. | 14:57 |
| *** vijaykc4 has quit IRC | 14:58 | |
| electrocucaracha | korzen: let me try to find a patch that support this | 14:58 |
| *** openstack has joined #openstack-neutron | 15:04 | |
| korzen | electrocucaracha, so I guess we have reverted the network structure to be included in single file | 15:04 |
| korzen | electrocucaracha, I do not have any strong opinions which way we should be heading | 15:05 |
| korzen | electrocucaracha, for router patch, I would stick to the neutron/objects/router.py | 15:06 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Handle agent restart for trunks https://review.openstack.org/365176 | 15:06 |
| *** mihero has quit IRC | 15:07 | |
| *** obondarev has joined #openstack-neutron | 15:07 | |
| *** ivc_ has joined #openstack-neutron | 15:08 | |
| *** pcarver has joined #openstack-neutron | 15:09 | |
| pcarver | matrohon: are you around? | 15:11 |
| matrohon | pcarver, hi | 15:11 |
| openstackgerrit | Nakul Dahiwade proposed openstack/neutron: OVO for NetworkDhcpAgentBinding https://review.openstack.org/370452 | 15:12 |
| pcarver | matrohon: I haven't had networking-bgpvpn setup in a while. I'm trying to setup a fresh devstack and I think the instructions are missing a step. | 15:12 |
| rcernin | are there any neutronclient folks? | 15:12 |
| matrohon | pcarver : what error do you have? | 15:13 |
| pcarver | Using this http://docs.openstack.org/developer/networking-bgpvpn/bagpipe/index.html I don't think it's actually enabling networking-bgpvpn | 15:13 |
| jlibosva | kevinbenton: hi, can you confirm https://review.openstack.org/#/c/372424/2/neutron/tests/tempest/scenario/test_trunk.py@110 that my worries are too paranoid? :) | 15:13 |
| rcernin | Would you have way to hack the neutronclient check the network names on creation and do not allow the ones that already exists? | 15:13 |
| pcarver | matrohon: I don't see any BGPVPN commands in neutron help and I get a 404 when trying the API | 15:14 |
| *** andymaier has quit IRC | 15:14 | |
| *** yamamoto has joined #openstack-neutron | 15:14 | |
| russellb | rtheis: I think we should get rid of the job that uses the l3 and dhcp agents once we merge the floating ip patch | 15:14 |
| *** yamamoto has quit IRC | 15:14 | |
| pcarver | Getting networks works with this command: curl -s -X GET -H "X-Auth-Token: $ATOKEN" http://localhost:9696/v2.0/networks | 15:15 |
| pcarver | but this command gives a 404 curl -s -X GET -H "X-Auth-Token: $ATOKEN" http://localhost:9696/v2.0/bgpvpn/bgpvpns | 15:15 |
| pcarver | $ATOKEN is the keystone token for the default admin user created by devstack | 15:15 |
| matrohon | pcarver, you have enabled the plugin in devstack | 15:16 |
| rtheis | russellb: ack | 15:16 |
| russellb | rtheis: then less of this to fight ... | 15:17 |
| matrohon | pcarver, with this line in local.conf : | 15:17 |
| pcarver | matrohon: following the instructions on http://docs.openstack.org/developer/networking-bgpvpn/bagpipe/index.html I added NETWORKING_BGPVPN_DRIVER="BGPVPN:BaGPipe:networking_bgpvpn.neutron.services.service_drivers.bagpipe.bagpipe.BaGPipeBGPVPNDriver:default" to my local.conf | 15:17 |
| matrohon | enable_plugin networking-bgpvpn git://git.openstack.org/openstack/networking-bgpvpn.git | 15:17 |
| rtheis | certainly would make life easier :) | 15:17 |
| *** vijaykc4 has joined #openstack-neutron | 15:17 | |
| russellb | rtheis: sooooon | 15:18 |
| russellb | rtheis: patch seems to be passing tests, so can't be too far off :) | 15:18 |
| matrohon | pcarver : ok, I'm not sure the bagpipe plugin automatically activate the bgpvpn service plugin! | 15:19 |
| pcarver | matrohon: That seems to be missing. I thought I copied and pasted everything from that instruction page. Let me try again. | 15:19 |
| openstackgerrit | Merged openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/371109 | 15:19 |
| pcarver | matrohon: does that line go above or below the post-config line? Or does it not matter? I'm not running a separate compute node, just a single VM devstack. | 15:20 |
| *** fzdarsky has quit IRC | 15:20 | |
| openstackgerrit | Oleg Bondarev proposed openstack/neutron: DNM: Investigate bug 1609540 and probably 1540983 https://review.openstack.org/374106 | 15:20 |
| openstack | bug 1609540 in neutron "Deleting csnat port fails due to no fixed ips" [High,In progress] https://launchpad.net/bugs/1609540 - Assigned to Carl Baldwin (carl-baldwin) | 15:20 |
| matrohon | pcarver : I use to put it above | 15:21 |
| matrohon | pcarver : if you just wanto test the API, you don't need to activate bagpipe | 15:22 |
| pcarver | matrohon: Ok. I'll re-run devstack. I assume that it if works the bgpvpn URL should be on the Neutron endpoint and start with v2.0, right? | 15:22 |
| *** garyk has quit IRC | 15:23 | |
| *** vijaykc4 has quit IRC | 15:23 | |
| matrohon | pcarver, a dummy driver is embedded, it stores data in db and doesn't configure the dataplane, nodriver is enabled | 15:24 |
| matrohon | pcarver, ^right | 15:24 |
| pcarver | matrohon: Ok, so I can just leave out the bagpipe line from local.conf? | 15:24 |
| *** EinstCrazy has joined #openstack-neutron | 15:24 | |
| matrohon | pcarver, yep | 15:24 |
| *** kobis has quit IRC | 15:24 | |
| *** hieulq_ has joined #openstack-neutron | 15:25 | |
| *** EinstCrazy has quit IRC | 15:26 | |
| *** EinstCrazy has joined #openstack-neutron | 15:27 | |
| *** yamamoto has joined #openstack-neutron | 15:27 | |
| electrocucaracha | korzen: I think that it's a good topic to discuss during the next upgrades meeting | 15:27 |
| korzen | electrocucaracha, yes, I agree | 15:27 |
| matrohon | pacarver, you can also use the neutron client instead of curl cmds | 15:27 |
| matrohon | pcarver, ^ | 15:28 |
| *** Leo_ has joined #openstack-neutron | 15:28 | |
| *** sridharg has quit IRC | 15:28 | |
| *** andreas_s has quit IRC | 15:28 | |
| pcarver | matrohon: I know, but I want to verify exactly what's happening at a REST level in order to validate the accuracy of the API documentation | 15:28 |
| *** vijaykc4 has joined #openstack-neutron | 15:28 | |
| *** jamesden_ has joined #openstack-neutron | 15:28 | |
| pcarver | I don't want the neutron client to gloss over any details | 15:29 |
| *** liuyulong has quit IRC | 15:29 | |
| matrohon | pcarver, fine! don't hesitate to report bugs if needed :) | 15:29 |
| *** Leom has joined #openstack-neutron | 15:29 | |
| *** vijaykc4 has quit IRC | 15:29 | |
| *** liuyulong has joined #openstack-neutron | 15:29 | |
| pcarver | matrohon: will do. My goal is that the API doc should basically work if you copy/paste into curl except for changing tokens and UUIDs of course | 15:30 |
| matrohon | pcarver, once devstack has finished, you can check the neutron.conf and networking_bgpvpn.conf, to see if everything is in order | 15:30 |
| *** mvk has quit IRC | 15:31 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Make plugin loading more consistent with logging guidelines. https://review.openstack.org/374225 | 15:31 |
| *** yamamoto has quit IRC | 15:32 | |
| *** Leo_ has quit IRC | 15:32 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Change default exception in wait_until_true https://review.openstack.org/373973 | 15:32 |
| openstackgerrit | Merged openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/373768 | 15:32 |
| *** dolphm has joined #openstack-neutron | 15:33 | |
| *** Alex_Stef has quit IRC | 15:33 | |
| matrohon | pcarver, if you want it to go fast, you can only activate neutron in devstack with the following line in your local.conf : | 15:33 |
| matrohon | ENABLED_SERVICES=q-svc,q-agt,q-dhcp,q-l3,key,mysql,rabbit | 15:33 |
| *** oshvartz has joined #openstack-neutron | 15:33 | |
| matrohon | and activate the bgpvpn service plugi of course : | 15:34 |
| matrohon | enable_plugin networking-bgpvpn git://git.openstack.org/openstack/networking-bgpvpn.git | 15:34 |
| pcarver | matrohon: Thanks. I already started the stack process but I'll note that for the future. So basically you're saying leave out Nova, Horizon, etc and just start Neutron and Keystone? | 15:35 |
| *** jhershbe has quit IRC | 15:35 | |
| *** lucasagomes is now known as lucas-hungry | 15:35 | |
| matrohon | pcarver, yep, with mysql and rabbitmq | 15:35 |
| pcarver | matrohon: Thanks | 15:36 |
| matrohon | pcarver, it works like a charm, asa you only want to test the neutron API and DB | 15:36 |
| *** vhosakot has joined #openstack-neutron | 15:36 | |
| *** hynekm has quit IRC | 15:37 | |
| *** tri2sing has quit IRC | 15:40 | |
| *** obondarev has quit IRC | 15:40 | |
| *** tri2sing has joined #openstack-neutron | 15:40 | |
| *** fzdarsky has joined #openstack-neutron | 15:40 | |
| *** mickeys has joined #openstack-neutron | 15:41 | |
| *** garyk has joined #openstack-neutron | 15:41 | |
| *** Leom has quit IRC | 15:42 | |
| *** bks has joined #openstack-neutron | 15:45 | |
| openstackgerrit | Nakul Dahiwade proposed openstack/neutron-lib: api-ref: Fix api-ref for subnetpools https://review.openstack.org/371172 | 15:46 |
| *** ramishra has quit IRC | 15:48 | |
| *** amotoki has quit IRC | 15:49 | |
| *** fzdarsky has quit IRC | 15:49 | |
| *** kbringard has joined #openstack-neutron | 15:49 | |
| *** bana_k has joined #openstack-neutron | 15:49 | |
| *** ramishra has joined #openstack-neutron | 15:50 | |
| *** kumarb has joined #openstack-neutron | 15:51 | |
| *** andreas_s has joined #openstack-neutron | 15:52 | |
| *** yamamoto has joined #openstack-neutron | 15:52 | |
| *** bks has quit IRC | 15:52 | |
| *** sticker__ has joined #openstack-neutron | 15:53 | |
| *** itsuugo has quit IRC | 15:53 | |
| matrohon | pcarver, https://review.openstack.org/#/c/374259/ | 15:54 |
| *** itsuugo has joined #openstack-neutron | 15:55 | |
| openstackgerrit | Aradhana Singh proposed openstack/neutron: OVO for Tag https://review.openstack.org/356825 | 15:55 |
| openstackgerrit | Aradhana Singh proposed openstack/neutron: Relocate Tag DB model https://review.openstack.org/355536 | 15:55 |
| *** mkolesni has quit IRC | 15:56 | |
| *** sticker_ has quit IRC | 15:56 | |
| *** rajinir has joined #openstack-neutron | 15:58 | |
| *** EinstCrazy has quit IRC | 15:59 | |
| *** kobis has joined #openstack-neutron | 16:00 | |
| *** matrohon has quit IRC | 16:00 | |
| *** banix has joined #openstack-neutron | 16:02 | |
| *** murphy_zhao has quit IRC | 16:03 | |
| *** nplanel has quit IRC | 16:03 | |
| *** moshele has joined #openstack-neutron | 16:03 | |
| *** rcernin has quit IRC | 16:03 | |
| *** abhiraut has joined #openstack-neutron | 16:03 | |
| *** murphy_zhao has joined #openstack-neutron | 16:04 | |
| *** bfernando has quit IRC | 16:08 | |
| *** wolverineav has joined #openstack-neutron | 16:08 | |
| *** baoli has quit IRC | 16:08 | |
| *** baoli has joined #openstack-neutron | 16:09 | |
| *** jprovazn has quit IRC | 16:12 | |
| *** wolverineav has quit IRC | 16:13 | |
| *** vkmc is now known as vkmc|afk | 16:14 | |
| *** slaweq_ has joined #openstack-neutron | 16:14 | |
| *** crose has quit IRC | 16:15 | |
| *** pcaruana has quit IRC | 16:15 | |
| *** john5223 has joined #openstack-neutron | 16:17 | |
| *** oshvartz has quit IRC | 16:17 | |
| *** pcaruana has joined #openstack-neutron | 16:17 | |
| *** apdibbo has quit IRC | 16:18 | |
| *** akamyshnikova_ has quit IRC | 16:18 | |
| *** moshele has quit IRC | 16:19 | |
| *** kevo has quit IRC | 16:24 | |
| openstackgerrit | Merged openstack/neutron: Remove deprecated class NeutronController https://review.openstack.org/372324 | 16:25 |
| *** tmorin has quit IRC | 16:25 | |
| *** lucas-hungry is now known as lucasagomes | 16:29 | |
| *** davidsha has quit IRC | 16:29 | |
| *** krtaylor has quit IRC | 16:31 | |
| *** wolverineav has joined #openstack-neutron | 16:34 | |
| *** Swami has quit IRC | 16:34 | |
| *** iyamahat has quit IRC | 16:34 | |
| dougwig | boden: nothing from me. not much activity with newton happening. do you have any items? | 16:35 |
| *** mriedem is now known as mriedem_away | 16:35 | |
| *** crose has joined #openstack-neutron | 16:35 | |
| *** yamahata has quit IRC | 16:37 | |
| *** andreas_s has quit IRC | 16:37 | |
| *** mmalik4 has joined #openstack-neutron | 16:41 | |
| *** ivar-lazzaro has joined #openstack-neutron | 16:44 | |
| *** ganeshnalawade has joined #openstack-neutron | 16:44 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: fullstack: VLAN aware VMs test https://review.openstack.org/369001 | 16:45 |
| *** wolverineav has quit IRC | 16:45 | |
| *** slaweq_ has quit IRC | 16:46 | |
| *** todin has joined #openstack-neutron | 16:47 | |
| *** rmart04 has quit IRC | 16:47 | |
| *** Swami has joined #openstack-neutron | 16:48 | |
| *** abhiraut has quit IRC | 16:48 | |
| *** liuyulong__ has quit IRC | 16:49 | |
| boden | dougwig: I don’t have much; now that lib is open for ocata I’ll start pushing some patches | 16:52 |
| *** hemanthm has joined #openstack-neutron | 16:53 | |
| *** hemanthm has left #openstack-neutron | 16:54 | |
| *** rkukura has quit IRC | 16:54 | |
| *** iyamahat has joined #openstack-neutron | 16:55 | |
| *** wolverineav has joined #openstack-neutron | 16:55 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: fullstack: VLAN aware VMs test https://review.openstack.org/369001 | 16:55 |
| *** Swami_ has joined #openstack-neutron | 16:55 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron: Linux Bridge: driver support for QoS egress minimum bandwidth https://review.openstack.org/357865 | 16:56 |
| *** yamamoto has quit IRC | 16:56 | |
| *** vijaykc4 has joined #openstack-neutron | 16:57 | |
| *** oshvartz has joined #openstack-neutron | 16:58 | |
| *** korzen has quit IRC | 16:59 | |
| *** Swami has quit IRC | 16:59 | |
| *** vijaykc4 has quit IRC | 17:00 | |
| *** slaweq_ has joined #openstack-neutron | 17:00 | |
| *** wolverineav has quit IRC | 17:00 | |
| *** wolverineav has joined #openstack-neutron | 17:00 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: fullstack: VLAN aware VMs test https://review.openstack.org/369001 | 17:01 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Be more generic about the missing params in trunk invalid error message https://review.openstack.org/374297 | 17:02 |
| *** tidwellr has quit IRC | 17:02 | |
| *** kbringard has quit IRC | 17:03 | |
| *** krtaylor has joined #openstack-neutron | 17:04 | |
| *** eezhova has quit IRC | 17:04 | |
| *** kobis has quit IRC | 17:07 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: Change default exception in wait_until_true https://review.openstack.org/373973 | 17:08 |
| *** kobis has joined #openstack-neutron | 17:08 | |
| *** kumarb has quit IRC | 17:09 | |
| *** jlibosva has quit IRC | 17:09 | |
| *** wolverineav has quit IRC | 17:09 | |
| *** ociuhandu has quit IRC | 17:10 | |
| *** vijaykc4 has joined #openstack-neutron | 17:10 | |
| *** Swami_ has quit IRC | 17:11 | |
| *** kobis has quit IRC | 17:11 | |
| *** kevo has joined #openstack-neutron | 17:12 | |
| *** yamahata has joined #openstack-neutron | 17:12 | |
| *** mhickey has quit IRC | 17:13 | |
| *** jpena is now known as jpena|off | 17:14 | |
| *** wolverineav has joined #openstack-neutron | 17:16 | |
| *** ganeshnalawade has quit IRC | 17:16 | |
| dougwig | boden, HenryG: ok, let's cancel today and focus on newton. | 17:18 |
| *** rossella_s has quit IRC | 17:19 | |
| *** rossella_s has joined #openstack-neutron | 17:19 | |
| *** garyk has quit IRC | 17:20 | |
| *** slaweq_ has quit IRC | 17:20 | |
| *** itsuugo has quit IRC | 17:22 | |
| *** itsuugo has joined #openstack-neutron | 17:23 | |
| *** vijaykc4 has quit IRC | 17:23 | |
| *** wolverineav has quit IRC | 17:24 | |
| *** wolverineav has joined #openstack-neutron | 17:24 | |
| *** Swami has joined #openstack-neutron | 17:24 | |
| *** boden has quit IRC | 17:25 | |
| *** rkukura has joined #openstack-neutron | 17:27 | |
| openstackgerrit | Mohit Malik proposed openstack/neutron: [WIP] Integration of OVO for Ipam https://review.openstack.org/365155 | 17:28 |
| *** kbringard has joined #openstack-neutron | 17:28 | |
| *** trevormc has joined #openstack-neutron | 17:28 | |
| *** ivar-lazzaro has quit IRC | 17:33 | |
| *** tflynn has joined #openstack-neutron | 17:33 | |
| *** pradk has quit IRC | 17:34 | |
| *** pradk has joined #openstack-neutron | 17:35 | |
| *** jhershbe has joined #openstack-neutron | 17:36 | |
| *** jckasper has joined #openstack-neutron | 17:37 | |
| *** njohnsto_ has joined #openstack-neutron | 17:38 | |
| *** abhiraut has joined #openstack-neutron | 17:39 | |
| *** dsneddon has quit IRC | 17:40 | |
| *** iyamahat has quit IRC | 17:43 | |
| *** iyamahat has joined #openstack-neutron | 17:43 | |
| *** njohnsto_ has quit IRC | 17:43 | |
| *** ijw has joined #openstack-neutron | 17:45 | |
| *** rkukura has quit IRC | 17:45 | |
| *** haplo37__ has joined #openstack-neutron | 17:45 | |
| *** jamesden_ has quit IRC | 17:45 | |
| *** rkukura has joined #openstack-neutron | 17:47 | |
| *** abhiraut has quit IRC | 17:48 | |
| *** jamesden_ has joined #openstack-neutron | 17:49 | |
| *** bana_k has quit IRC | 17:51 | |
| *** ekuris has joined #openstack-neutron | 17:52 | |
| *** jamesden_ has quit IRC | 17:53 | |
| *** mriedem_away is now known as mriedem | 17:56 | |
| *** yamamoto has joined #openstack-neutron | 17:57 | |
| *** ekuris has quit IRC | 17:58 | |
| *** jamesden_ has joined #openstack-neutron | 17:59 | |
| *** Alex_Stef has joined #openstack-neutron | 17:59 | |
| *** hieulq_ has quit IRC | 18:00 | |
| *** john-davidge has joined #openstack-neutron | 18:00 | |
| openstackgerrit | Merged openstack/neutron-lib: Updated from global requirements https://review.openstack.org/373767 | 18:03 |
| *** abhiraut has joined #openstack-neutron | 18:03 | |
| *** rkukura has quit IRC | 18:05 | |
| *** yamamoto has quit IRC | 18:05 | |
| *** jamesden_ has quit IRC | 18:05 | |
| *** sterdnotshaken has joined #openstack-neutron | 18:06 | |
| *** rkukura has joined #openstack-neutron | 18:08 | |
| *** boden has joined #openstack-neutron | 18:09 | |
| *** vijaykc4 has joined #openstack-neutron | 18:09 | |
| *** vijaykc4 has quit IRC | 18:10 | |
| *** itsuugo has quit IRC | 18:11 | |
| *** vkmc|afk is now known as vkmc | 18:12 | |
| *** tobias_fiberdata has joined #openstack-neutron | 18:12 | |
| *** jamesden_ has joined #openstack-neutron | 18:12 | |
| *** itsuugo has joined #openstack-neutron | 18:13 | |
| *** tonytan4ever has quit IRC | 18:14 | |
| *** tonytan4ever has joined #openstack-neutron | 18:15 | |
| *** rmart04 has joined #openstack-neutron | 18:15 | |
| *** bana_k has joined #openstack-neutron | 18:17 | |
| *** baoli has quit IRC | 18:19 | |
| *** rmart04_ has joined #openstack-neutron | 18:19 | |
| *** sterdnotshaken has quit IRC | 18:19 | |
| *** baoli has joined #openstack-neutron | 18:20 | |
| openstackgerrit | Merged openstack/neutron: Do not retry default security group creation https://review.openstack.org/373841 | 18:20 |
| *** rmart04 has quit IRC | 18:21 | |
| *** rmart04_ is now known as rmart04 | 18:21 | |
| *** ihrachys has joined #openstack-neutron | 18:21 | |
| openstackgerrit | Merged openstack/neutron: Fix a release note typo for implicit provider loading deprecation https://review.openstack.org/373830 | 18:23 |
| openstackgerrit | Merged openstack/neutron: Install dibbler.filters rootwrap file https://review.openstack.org/374197 | 18:24 |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Update metadata proxy when subnet add/delete https://review.openstack.org/293237 | 18:25 |
| *** jhershbe has quit IRC | 18:26 | |
| *** sbezverk has quit IRC | 18:26 | |
| *** ihrachys has quit IRC | 18:27 | |
| *** yfried has joined #openstack-neutron | 18:29 | |
| *** vijaykc4 has joined #openstack-neutron | 18:30 | |
| *** rcernin has joined #openstack-neutron | 18:30 | |
| *** gouthamr has quit IRC | 18:31 | |
| *** bsv has joined #openstack-neutron | 18:31 | |
| *** vijaykc4 has quit IRC | 18:32 | |
| *** jprovazn has joined #openstack-neutron | 18:33 | |
| *** john-davidge has quit IRC | 18:33 | |
| *** john-davidge has joined #openstack-neutron | 18:33 | |
| *** ivar-lazzaro has joined #openstack-neutron | 18:34 | |
| *** ijw has quit IRC | 18:34 | |
| prometheanfire | carl_baldwin: thanks :D | 18:34 |
| *** oshvartz has quit IRC | 18:35 | |
| carl_baldwin | prometheanfire: glad to help. What specifically are you thanking me for? | 18:35 |
| russellb | carl_baldwin: for being awesome | 18:36 |
| prometheanfire | raising the prefix delegated firewall issue to high :P | 18:36 |
| prometheanfire | and that too | 18:36 |
| russellb | :) | 18:36 |
| *** ijw has joined #openstack-neutron | 18:36 | |
| carl_baldwin | russellb: I'm still working on that. I'll let you know when I get there. | 18:36 |
| carl_baldwin | prometheanfire: Ah, that. You're welcome. I felt badly that it had escaped attention for so long. | 18:36 |
| russellb | it's a journey | 18:36 |
| *** phil_h has quit IRC | 18:37 | |
| prometheanfire | ya | 18:37 |
| prometheanfire | the workarounds help at least | 18:37 |
| *** john-davidge has quit IRC | 18:38 | |
| carl_baldwin | prometheanfire: we'll be sure to hit that bug soon. Hopefully we can backport too. | 18:38 |
| prometheanfire | cool | 18:38 |
| *** ivar-lazzaro has quit IRC | 18:39 | |
| *** sdake has quit IRC | 18:39 | |
| *** vhoward has joined #openstack-neutron | 18:41 | |
| *** itsuugo has quit IRC | 18:42 | |
| *** itsuugo has joined #openstack-neutron | 18:44 | |
| *** vijaykc4 has joined #openstack-neutron | 18:46 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron-lib: api-ref: Update to current state of Keystone V3 support https://review.openstack.org/372857 | 18:46 |
| *** rkukura has quit IRC | 18:47 | |
| *** jhershbe has joined #openstack-neutron | 18:47 | |
| *** eezhova has joined #openstack-neutron | 18:49 | |
| *** phil_h has joined #openstack-neutron | 18:50 | |
| *** ig0r_ has joined #openstack-neutron | 18:50 | |
| *** gouthamr has joined #openstack-neutron | 18:51 | |
| *** tflynn has quit IRC | 18:51 | |
| *** tflynn has joined #openstack-neutron | 18:52 | |
| *** sdake has joined #openstack-neutron | 18:56 | |
| *** moshele has joined #openstack-neutron | 18:59 | |
| openstackgerrit | Merged openstack/neutron: Add admin api tests for project-id https://review.openstack.org/371998 | 19:00 |
| *** rkukura has joined #openstack-neutron | 19:02 | |
| *** moshele has quit IRC | 19:03 | |
| rtheis | ihrachys: looking for review help on networking-ovn stable/newton cherry-pick: https://review.openstack.org/#/c/374333/ | 19:06 |
| rtheis | thanks | 19:06 |
| *** itsuugo has quit IRC | 19:06 | |
| *** jhershbe has quit IRC | 19:06 | |
| *** ivar-lazzaro has joined #openstack-neutron | 19:07 | |
| *** mkolesni has joined #openstack-neutron | 19:07 | |
| *** itsuugo has joined #openstack-neutron | 19:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/371108 | 19:08 |
| *** vijaykc4 has quit IRC | 19:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/373772 | 19:11 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Ensure trunk status is acknowledged during OVS subport operations https://review.openstack.org/372750 | 19:11 |
| *** catintheroof has joined #openstack-neutron | 19:12 | |
| amuller | Sam-I-Am: Did you want me to look specifically at https://review.openstack.org/#/c/367691/2/doc/networking-guide/source/deploy-lb-ha-vrrp.rst? | 19:12 |
| Sam-I-Am | amuller: yessir | 19:13 |
| Sam-I-Am | amuller: and soon the ovs variant, but thats not up yet. | 19:13 |
| Sam-I-Am | also, is native ovs secgroups becoming a default-ish thing at some point? | 19:13 |
| amuller | Sam-I-Am: can you point me to what you currently have? | 19:13 |
| amuller | Sam-I-Am: not for newton | 19:13 |
| Sam-I-Am | what i have for ovs? | 19:13 |
| amuller | no for LB | 19:13 |
| Sam-I-Am | thats the link you posted | 19:14 |
| amuller | yeah I mean | 19:14 |
| amuller | what exists currently, before this patch | 19:14 |
| Sam-I-Am | ohhhh | 19:14 |
| Sam-I-Am | http://docs.openstack.org/mitaka/networking-guide/scenario-l3ha-lb.html | 19:14 |
| Sam-I-Am | keep in mind these were all restructured | 19:14 |
| Sam-I-Am | the ha arch builds on self-service nets which builds on provider nets | 19:14 |
| *** ijw has quit IRC | 19:14 | |
| *** ijw has joined #openstack-neutron | 19:15 | |
| Sam-I-Am | going with a building blocks design now... and trying to reduce all the duplication (that you always bugged me about) | 19:15 |
| *** slaweq_ has joined #openstack-neutron | 19:15 | |
| amuller | hehe yeah | 19:15 |
| Sam-I-Am | at this point of your review, i just want to make sure the l3ha stuff makes sense assuming you have a neutron deployment with provider and self-service nets before you start | 19:16 |
| Sam-I-Am | which more or less comes to adding a network node | 19:16 |
| amuller | ok | 19:16 |
| Sam-I-Am | more or less looking for technical accuracy | 19:16 |
| prometheanfire | Sam-I-Am: hi | 19:16 |
| Sam-I-Am | prometheanfire: oink | 19:16 |
| prometheanfire | moo | 19:17 |
| Sam-I-Am | amuller: i can point you at a rendered version too | 19:17 |
| amuller | Sam-I-Am: so not a bunch of grammar nits then =p | 19:17 |
| Sam-I-Am | well, there's one in the build, but i also have a local one | 19:18 |
| Sam-I-Am | amuller: eh, the docs people will shit on that enough later :) | 19:18 |
| *** vijaykc4 has joined #openstack-neutron | 19:20 | |
| *** akshai has quit IRC | 19:20 | |
| *** ivar-lazzaro has quit IRC | 19:21 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Make plugin loading more consistent with logging guidelines. https://review.openstack.org/374225 | 19:21 |
| *** rmart04 has quit IRC | 19:21 | |
| *** ijw has quit IRC | 19:25 | |
| *** akshai has joined #openstack-neutron | 19:25 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 19:26 |
| *** mkolesni has quit IRC | 19:27 | |
| *** vijaykc4 has quit IRC | 19:28 | |
| *** iyamahat has quit IRC | 19:28 | |
| *** mkolesni has joined #openstack-neutron | 19:30 | |
| *** tobias_fiberdata has quit IRC | 19:33 | |
| *** Alex_Stef has quit IRC | 19:34 | |
| *** tidwellr has joined #openstack-neutron | 19:34 | |
| *** mkolesni_ has joined #openstack-neutron | 19:35 | |
| amuller | Sam-I-Am: done | 19:35 |
| *** mkolesni has quit IRC | 19:35 | |
| Sam-I-Am | amuller: thx | 19:37 |
| *** mkolesni has joined #openstack-neutron | 19:38 | |
| *** gvrangan has quit IRC | 19:38 | |
| openstackgerrit | Genadi Chereshnya proposed openstack/neutron: Checking functionality of DVR https://review.openstack.org/340383 | 19:39 |
| *** jprovazn has quit IRC | 19:40 | |
| *** ijw has joined #openstack-neutron | 19:40 | |
| Sam-I-Am | amuller: interesting. i killed the l3 agent and had a failover. | 19:40 |
| amuller | Sam-I-Am: did you start it again, or just stopped it? | 19:41 |
| *** ijw has quit IRC | 19:41 | |
| *** ivc_ has quit IRC | 19:41 | |
| *** mkolesni_ has quit IRC | 19:41 | |
| *** ijw has joined #openstack-neutron | 19:41 | |
| *** tonytan_brb has joined #openstack-neutron | 19:41 | |
| amuller | Sam-I-Am: when you start the l3 agent it will resync and eventually SIGHUP all keepaliveds | 19:42 |
| amuller | Sam-I-Am: that should not, but can cause elections | 19:42 |
| Sam-I-Am | stopped it. master router moved. started it again. master router did not move back. | 19:42 |
| amuller | hmm, stopping the l3 agent should really not cause a failover, that's very odd | 19:42 |
| Sam-I-Am | it works as i would have expected it to work | 19:42 |
| *** salv-orlando has joined #openstack-neutron | 19:42 | |
| Sam-I-Am | wouldn't keepalived stop transmitting heartbeats? | 19:42 |
| amuller | but stopping the l3 agent should not touch keepalived | 19:43 |
| amuller | are you using packaging? | 19:43 |
| *** mkolesni_ has joined #openstack-neutron | 19:43 | |
| *** bana_k has quit IRC | 19:43 | |
| *** mkolesni has quit IRC | 19:43 | |
| Sam-I-Am | yeah. its an init script for neutron-l3-agent. | 19:43 |
| amuller | maybe that script kills the l3 agent and all of its children | 19:44 |
| amuller | it shouldn't | 19:44 |
| *** tonytan4ever has quit IRC | 19:44 | |
| amuller | Sam-I-Am: is keepalived still up after the stop the l3 agent on the master node? | 19:44 |
| amuller | and also do you see any keepalived logging in syslog? | 19:44 |
| Sam-I-Am | lemmie look | 19:45 |
| amuller | last time I checked with RDO stopping the l3 agent doesn't touch keepalived so there's no failover | 19:45 |
| *** salv-orl_ has quit IRC | 19:46 | |
| *** vhoward has quit IRC | 19:47 | |
| Sam-I-Am | amuller: stopping the l3 agent killed all the keepaliveds | 19:47 |
| amuller | Sam-I-Am: likely to do with deb/ubuntu packaging | 19:47 |
| Sam-I-Am | amuller: http://paste.openstack.org/show/582457/ | 19:48 |
| amuller | Sam-I-Am: on .rpm packaging restarting/killing the control plane will not affect the data plane | 19:48 |
| Sam-I-Am | that's the process list | 19:48 |
| *** mkolesni_ has quit IRC | 19:48 | |
| Sam-I-Am | amuller: it doesnt kill bridges or namespaces. if you stop the l3 agent without l3ha, packets still flow. | 19:49 |
| Sam-I-Am | with l3ha, it just migrates to the other network node | 19:49 |
| amuller | Sam-I-Am: which causes a short data plane interruption | 19:49 |
| Sam-I-Am | it does | 19:49 |
| amuller | Sam-I-Am: lemme ask this - if you kill the dhcp agent does it kill dnsmasqs? | 19:49 |
| amuller | (IMO it shouldn't) | 19:50 |
| *** rawlin has quit IRC | 19:50 | |
| amuller | existing VMs that renew their lease should not be affected by the dhcp agent being blown away... | 19:50 |
| Sam-I-Am | they stay there | 19:50 |
| amuller | similarly restarting the l3 agent should not blow away keepalived | 19:50 |
| *** jamesden_ has quit IRC | 19:53 | |
| *** nmagnezi has joined #openstack-neutron | 19:56 | |
| *** bana_k has joined #openstack-neutron | 19:59 | |
| *** vijaykc4 has joined #openstack-neutron | 20:01 | |
| *** prateek has quit IRC | 20:03 | |
| *** jamesden_ has joined #openstack-neutron | 20:03 | |
| *** mhickey has joined #openstack-neutron | 20:03 | |
| *** boden has quit IRC | 20:04 | |
| Sam-I-Am | amuller: so you're thinking this might be a packaging boog? | 20:04 |
| amuller | Sam-I-Am: I think so | 20:04 |
| Sam-I-Am | hmm, the init scripts look the same | 20:05 |
| Sam-I-Am | for dhcp and l3 | 20:05 |
| *** catintheroof has quit IRC | 20:08 | |
| *** zkassab has quit IRC | 20:08 | |
| *** lucasagomes is now known as lucas-afk | 20:09 | |
| *** _cjones_ has joined #openstack-neutron | 20:10 | |
| openstackgerrit | Anindita Das proposed openstack/neutron: [WIP] Integration of VxlanAllocation/Endpoints OVO https://review.openstack.org/367810 | 20:12 |
| *** mhickey has quit IRC | 20:13 | |
| bsv | what are people using to test neutron setups? :) | 20:14 |
| *** rawlin has joined #openstack-neutron | 20:16 | |
| bsv | people/devs | 20:16 |
| *** ijw has quit IRC | 20:18 | |
| *** iyamahat has joined #openstack-neutron | 20:21 | |
| *** vijaykc4 has quit IRC | 20:21 | |
| *** ivar-lazzaro has joined #openstack-neutron | 20:21 | |
| openstackgerrit | Shashank Kumar Shankar proposed openstack/neutron: Integration of OVO Provisioning Block https://review.openstack.org/361303 | 20:23 |
| *** ivar-lazzaro has quit IRC | 20:26 | |
| *** ijw has joined #openstack-neutron | 20:29 | |
| *** jckasper has quit IRC | 20:31 | |
| *** jckasper has joined #openstack-neutron | 20:32 | |
| *** baoli has quit IRC | 20:33 | |
| *** baoli has joined #openstack-neutron | 20:33 | |
| *** Jeffrey4l has quit IRC | 20:35 | |
| *** gvrangan has joined #openstack-neutron | 20:36 | |
| *** banix has quit IRC | 20:36 | |
| *** jckasper has quit IRC | 20:36 | |
| *** iranzo has quit IRC | 20:37 | |
| *** Jeffrey4l has joined #openstack-neutron | 20:37 | |
| *** eezhova has quit IRC | 20:39 | |
| *** jamesden_ has quit IRC | 20:42 | |
| *** bsv has quit IRC | 20:42 | |
| *** bana_k has quit IRC | 20:42 | |
| *** mriedem has quit IRC | 20:43 | |
| *** crose has quit IRC | 20:44 | |
| *** mriedem has joined #openstack-neutron | 20:48 | |
| *** mihero has joined #openstack-neutron | 20:48 | |
| *** john5223 has quit IRC | 20:49 | |
| *** ijw has quit IRC | 20:49 | |
| *** ijw has joined #openstack-neutron | 20:50 | |
| *** janzian has quit IRC | 20:53 | |
| *** ijw has quit IRC | 20:53 | |
| *** krtaylor has quit IRC | 20:56 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add MAC address to subports in trunk_details https://review.openstack.org/374388 | 20:57 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Allow more time for DB migration tests https://review.openstack.org/374390 | 20:58 |
| *** tobias_fiberdata has joined #openstack-neutron | 20:59 | |
| *** akshai has quit IRC | 20:59 | |
| *** bana_k has joined #openstack-neutron | 21:00 | |
| *** tongli has quit IRC | 21:00 | |
| *** baoli has quit IRC | 21:02 | |
| *** baoli has joined #openstack-neutron | 21:02 | |
| *** cleong has quit IRC | 21:04 | |
| *** abhiraut1 has joined #openstack-neutron | 21:04 | |
| openstackgerrit | Anindita Das proposed openstack/neutron: [WIP] Integration of VxlanAllocation/Endpoints OVO https://review.openstack.org/367810 | 21:06 |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Allow more time for DB migration tests https://review.openstack.org/374390 | 21:07 |
| HenryG | armax: ^^ hope this works | 21:07 |
| *** abhiraut has quit IRC | 21:08 | |
| amuller | kevinbenton: quick Q - Is there a profiler you use to profile the neutron-server process? Last I checked gevent profiler is the one that worked for me | 21:10 |
| *** marun has left #openstack-neutron | 21:11 | |
| *** abhiraut1 has quit IRC | 21:11 | |
| amuller | (or anyone else :) | 21:12 |
| *** ijw has joined #openstack-neutron | 21:14 | |
| *** itsuugo has quit IRC | 21:16 | |
| *** itsuugo has joined #openstack-neutron | 21:17 | |
| *** baoli_ has joined #openstack-neutron | 21:19 | |
| *** baoli has quit IRC | 21:19 | |
| *** ijw has quit IRC | 21:19 | |
| *** nmagnezi has quit IRC | 21:19 | |
| *** baoli_ has quit IRC | 21:19 | |
| *** dsneddon has joined #openstack-neutron | 21:19 | |
| *** baoli has joined #openstack-neutron | 21:20 | |
| *** yfried has quit IRC | 21:21 | |
| kevinbenton | amuller: i've used https://docs.python.org/2/library/profile.html#module-cProfile | 21:23 |
| amuller | kevinbenton: hmm, that gave you accurate results in spite of eventlet? | 21:24 |
| kevinbenton | amuller: but to get the results you have to send a signal to the process IIRC | 21:24 |
| *** melwitt has left #openstack-neutron | 21:24 | |
| kevinbenton | amuller: for the few things i had used it for, yeah | 21:24 |
| *** rcernin has quit IRC | 21:24 | |
| *** wasmum has joined #openstack-neutron | 21:24 | |
| *** salv-orlando has quit IRC | 21:26 | |
| *** salv-orlando has joined #openstack-neutron | 21:27 | |
| *** baoli has quit IRC | 21:28 | |
| *** baoli has joined #openstack-neutron | 21:29 | |
| *** ujjain has quit IRC | 21:29 | |
| amuller | kevinbenton: ack, thanks | 21:29 |
| *** ujjain has joined #openstack-neutron | 21:30 | |
| *** ujjain has quit IRC | 21:30 | |
| *** ujjain has joined #openstack-neutron | 21:30 | |
| kevinbenton | amuller: http://lists.openstack.org/pipermail/openstack-dev/2016-May/095707.html | 21:30 |
| *** itsuugo has quit IRC | 21:31 | |
| amuller | kevinbenton: yeah I just read that as well =p | 21:31 |
| amuller | I haven't done this for a while but I recall when using cprofiler it was producing garbage results | 21:31 |
| amuller | as a result of eventlet | 21:31 |
| amuller | eventlet profiler didn't seem to work, gevent profiler did | 21:31 |
| amuller | I was just wondering if anyone has more up to date info | 21:32 |
| kevinbenton | amuller: its quite possible that's still the case | 21:32 |
| *** itsuugo has joined #openstack-neutron | 21:32 | |
| *** banix has joined #openstack-neutron | 21:32 | |
| kevinbenton | amuller: IIRC i was profiling agent side stuff with it | 21:32 |
| kevinbenton | amuller: which wasn't impacted | 21:32 |
| amuller | ovs agent doesn't use eventlet | 21:33 |
| amuller | l3 does | 21:33 |
| *** wolverineav has quit IRC | 21:33 | |
| amuller | and dhcp it looks like | 21:33 |
| *** gouthamr has quit IRC | 21:34 | |
| *** banix_ has joined #openstack-neutron | 21:35 | |
| *** wolverineav has joined #openstack-neutron | 21:35 | |
| *** banix has quit IRC | 21:37 | |
| *** banix_ is now known as banix | 21:37 | |
| *** rtheis has quit IRC | 21:39 | |
| *** krtaylor has joined #openstack-neutron | 21:40 | |
| *** edmondsw has quit IRC | 21:40 | |
| *** itsuugo has quit IRC | 21:42 | |
| *** wolverineav has quit IRC | 21:42 | |
| *** itsuugo has joined #openstack-neutron | 21:42 | |
| *** ig0r_ has quit IRC | 21:45 | |
| *** wolverineav has joined #openstack-neutron | 21:46 | |
| *** mmalik4 has quit IRC | 21:48 | |
| *** jheroux has quit IRC | 21:48 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 21:50 |
| *** sterdnotshaken has joined #openstack-neutron | 21:50 | |
| *** amuller has quit IRC | 21:52 | |
| *** sterdnotshaken1 has joined #openstack-neutron | 21:53 | |
| *** itsuugo has quit IRC | 21:55 | |
| *** haplo37__ has quit IRC | 21:55 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: Clean-up Tempest test networks with project_id https://review.openstack.org/374413 | 21:56 |
| *** itsuugo has joined #openstack-neutron | 21:56 | |
| *** kbringard has quit IRC | 21:56 | |
| *** sterdnotshaken has quit IRC | 21:56 | |
| *** sterdnotshaken has joined #openstack-neutron | 21:57 | |
| *** mfranc213 has quit IRC | 21:58 | |
| *** Swami has quit IRC | 21:58 | |
| openstackgerrit | Mohit Malik proposed openstack/neutron: Change IPAM DB API to use context instead of session object. https://review.openstack.org/363206 | 21:59 |
| *** armax has quit IRC | 21:59 | |
| *** armax has joined #openstack-neutron | 22:00 | |
| *** sterdnotshaken1 has quit IRC | 22:00 | |
| openstackgerrit | Mohit Malik proposed openstack/neutron: OVO for Ipam(Allocation,Subnet,Pool) https://review.openstack.org/360799 | 22:00 |
| *** itsuugo has quit IRC | 22:01 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: Clean-up Tempest test networks with project_id https://review.openstack.org/374413 | 22:01 |
| *** mmalik4 has joined #openstack-neutron | 22:02 | |
| *** ijw has joined #openstack-neutron | 22:02 | |
| *** itsuugo has joined #openstack-neutron | 22:02 | |
| *** wolverineav has quit IRC | 22:02 | |
| *** mfranc213 has joined #openstack-neutron | 22:03 | |
| openstackgerrit | Mohit Malik proposed openstack/neutron: [WIP] Integration of OVO for Ipam https://review.openstack.org/365155 | 22:03 |
| *** wolverineav has joined #openstack-neutron | 22:06 | |
| *** eezhova has joined #openstack-neutron | 22:09 | |
| *** eezhova has quit IRC | 22:09 | |
| *** itsuugo has quit IRC | 22:10 | |
| *** itsuugo has joined #openstack-neutron | 22:10 | |
| *** adriant has joined #openstack-neutron | 22:12 | |
| openstackgerrit | Shashank Kumar Shankar proposed openstack/neutron: [WIP] Integration of Port Binding Level OVO. https://review.openstack.org/374418 | 22:12 |
| *** jckasper has joined #openstack-neutron | 22:13 | |
| *** wolverineav has quit IRC | 22:16 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Ensure trunk status is acknowledged during OVS subport operations https://review.openstack.org/372750 | 22:16 |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Update trunk metadata during wire/unwire operations https://review.openstack.org/373600 | 22:17 |
| *** marst has quit IRC | 22:17 | |
| *** s3wong has joined #openstack-neutron | 22:18 | |
| *** wolverineav has joined #openstack-neutron | 22:18 | |
| *** jckasper has quit IRC | 22:18 | |
| *** itsuugo has quit IRC | 22:21 | |
| *** wolverineav has quit IRC | 22:22 | |
| *** ivar-lazzaro has joined #openstack-neutron | 22:23 | |
| *** claudiub has quit IRC | 22:23 | |
| *** itsuugo has joined #openstack-neutron | 22:23 | |
| *** wolverineav has joined #openstack-neutron | 22:24 | |
| *** xinliang has quit IRC | 22:27 | |
| *** ivar-lazzaro has quit IRC | 22:27 | |
| *** wolverineav has quit IRC | 22:28 | |
| kevinbenton | armax, HenryG: wouldn't these API tests break on any translated environment? https://review.openstack.org/#/c/374289/ | 22:31 |
| *** tbachman_ has left #openstack-neutron | 22:31 | |
| armax | kevinbenton: I said it, I said it! | 22:31 |
| *** thorst has joined #openstack-neutron | 22:31 | |
| *** wolverineav has joined #openstack-neutron | 22:32 | |
| armax | kevinbenton: https://review.openstack.org/#/c/371998/3/neutron/tests/tempest/api/admin/test_networks.py@71 | 22:32 |
| *** ijw has quit IRC | 22:33 | |
| *** marst has joined #openstack-neutron | 22:36 | |
| kevinbenton | armax, HenryG: yeah, i think the issue would be running tempest tests against an API server with a different localization | 22:38 |
| *** sterdnotshaken has quit IRC | 22:39 | |
| *** banix has quit IRC | 22:39 | |
| *** xinliang has joined #openstack-neutron | 22:40 | |
| *** yamahata has quit IRC | 22:40 | |
| *** thorst has quit IRC | 22:41 | |
| HenryG | kevinbenton: armax: I thought we agreed the openstack gate doesn't do that? | 22:41 |
| *** thorst has joined #openstack-neutron | 22:41 | |
| armax | no | 22:42 |
| kevinbenton | HenryG: tempest tests aren't just for the gate | 22:42 |
| kevinbenton | HenryG: you are supposed to be able to point tempest at arbitrary openstack clouds to test them | 22:42 |
| *** tbachman has joined #openstack-neutron | 22:42 | |
| armax | kevinbenton: in this case you would need to point tempest and do tox -eall-plugin neutron | 22:42 |
| armax | but yes | 22:42 |
| kevinbenton | armax: yes | 22:43 |
| armax | yes | 22:43 |
| *** markvoelker has quit IRC | 22:43 | |
| kevinbenton | yes | 22:43 |
| armax | yes | 22:43 |
| kevinbenton | thank you for the tutorial on how to run api tests ;) | 22:43 |
| armax | you’re very welcome | 22:43 |
| openstackgerrit | Eric Fried proposed openstack/neutron: Allow neutron.tests to be imported from tests https://review.openstack.org/374425 | 22:44 |
| *** Swami has joined #openstack-neutron | 22:44 | |
| armax | thanks for the lecture on how to break out of gate things | 22:44 |
| kevinbenton | you're also very welcome | 22:44 |
| armax | thank you so very much | 22:44 |
| *** mlavalle has quit IRC | 22:45 | |
| HenryG | too much politeness | 22:45 |
| *** thorst has quit IRC | 22:46 | |
| *** Swami has quit IRC | 22:46 | |
| *** Swami has joined #openstack-neutron | 22:46 | |
| *** itsuugo has quit IRC | 22:51 | |
| *** Swami_ has joined #openstack-neutron | 22:52 | |
| *** itsuugo has joined #openstack-neutron | 22:52 | |
| *** thorst has joined #openstack-neutron | 22:54 | |
| *** Swami has quit IRC | 22:54 | |
| *** banix has joined #openstack-neutron | 22:54 | |
| *** ijw has joined #openstack-neutron | 22:54 | |
| *** yamahata has joined #openstack-neutron | 22:56 | |
| *** itsuugo has quit IRC | 22:57 | |
| *** _cjones_ has quit IRC | 22:57 | |
| *** itsuugo has joined #openstack-neutron | 22:58 | |
| *** vhosakot has quit IRC | 22:58 | |
| *** ijw has quit IRC | 22:59 | |
| *** thorst has quit IRC | 22:59 | |
| *** sdague has quit IRC | 23:00 | |
| *** banix has quit IRC | 23:00 | |
| carl_baldwin | kevinbenton: armax: that's more like it. You're getting along now. I'm so happy. | 23:01 |
| armax | carl_baldwin: I am just keeping up appeareances | 23:01 |
| armax | *appearances | 23:01 |
| carl_baldwin | armax: As long as I don't have to pull over the car again. | 23:02 |
| *** itsuugo has quit IRC | 23:02 | |
| *** mvk has joined #openstack-neutron | 23:04 | |
| *** baoli has quit IRC | 23:04 | |
| *** jamielennox is now known as jamielennox|away | 23:04 | |
| *** itsuugo has joined #openstack-neutron | 23:04 | |
| *** wolverineav has quit IRC | 23:07 | |
| *** itsuugo has quit IRC | 23:10 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron-lib: Add segments to v2 API reference https://review.openstack.org/374434 | 23:10 |
| *** hoonetorg has quit IRC | 23:10 | |
| *** tidwellr has left #openstack-neutron | 23:10 | |
| Sam-I-Am | kevinbenton: is there any way to enable l2pop on existing networks? | 23:11 |
| *** itsuugo has joined #openstack-neutron | 23:11 | |
| *** wolverineav has joined #openstack-neutron | 23:15 | |
| openstackgerrit | Merged openstack/neutron: Garbage collect HasStandardAttributes subclasses in StandardAttrTestCase https://review.openstack.org/373868 | 23:15 |
| *** itsuugo has quit IRC | 23:16 | |
| *** yamamoto has joined #openstack-neutron | 23:17 | |
| *** itsuugo has joined #openstack-neutron | 23:17 | |
| kevinbenton | Sam-I-Am: probably have to enable the driver and restart all agents | 23:18 |
| kevinbenton | Sam-I-Am: i don't think there is anything that needs to change in the DB though | 23:19 |
| Sam-I-Am | kevinbenton: the driver was always enabled. i had it disabled in the linuxbridge agent config. | 23:19 |
| *** donghao has quit IRC | 23:19 | |
| kevinbenton | Sam-I-Am: ah, and enabling and restarting the agent didn't work? | 23:19 |
| Sam-I-Am | not sure yet. | 23:19 |
| *** itsuugo has quit IRC | 23:22 | |
| *** itsuugo has joined #openstack-neutron | 23:22 | |
| kevinbenton | Sam-I-Am: yeah, unless something changed, l2pop stuff is calculated all on the fly | 23:24 |
| kevinbenton | Sam-I-Am: so having the ports exist before it was enabled shouldn't be an issue | 23:24 |
| Sam-I-Am | interesting | 23:24 |
| Sam-I-Am | i was having a weird issue with linuxbridge+l3ha+l2pop yesterday. instance could see dhcp servers and visa-versa. router could see neither. | 23:25 |
| Sam-I-Am | disabled l2pop, created a new network, and things magically worked | 23:25 |
| Sam-I-Am | https://bugs.launchpad.net/neutron/+bug/1523031 | 23:26 |
| openstack | Launchpad bug 1523031 in neutron "Neighbor table entry for router missing with Linux bridge + L3HA + L2 population" [High,New] | 23:26 |
| Sam-I-Am | pretty sure i hit that again | 23:26 |
| Sam-I-Am | seems like the can keeps getting kicked down the road | 23:27 |
| Sam-I-Am | bottom line is i dont think i can recommend enabling l2pop with lb+l3ha in the networking guide | 23:28 |
| *** trevormc has quit IRC | 23:28 | |
| *** tri2sing has quit IRC | 23:28 | |
| Sam-I-Am | kevinbenton: there's also this bug - https://bugs.launchpad.net/neutron/+bug/1590845 | 23:30 |
| openstack | Launchpad bug 1590845 in neutron "Router interfaces report being in BUILD state - l3ha vrrp+LinuxBridge" [Medium,In progress] - Assigned to venkata anil (anil-venkata) | 23:30 |
| Sam-I-Am | which seems to have a patch | 23:30 |
| Sam-I-Am | did you tag the wrong bug? | 23:31 |
| *** ijw has joined #openstack-neutron | 23:32 | |
| *** wolverineav has quit IRC | 23:32 | |
| *** itsuugo has quit IRC | 23:32 | |
| kevinbenton | Sam-I-Am: what do you mean? | 23:32 |
| Sam-I-Am | you changed it from fix-released to confirmed | 23:33 |
| kevinbenton | Sam-I-Am: because venkata's analysis was incorrect about the cause | 23:33 |
| *** wolverineav has joined #openstack-neutron | 23:33 | |
| kevinbenton | Sam-I-Am: if this exists in stable/mitaka | 23:33 |
| kevinbenton | Sam-I-Am: i think he had another patch that was supposed to address this | 23:34 |
| kevinbenton | Sam-I-Am: but realistically i think it would be safer to not recommend it for now | 23:34 |
| kevinbenton | Sam-I-Am: until we are sure it's stable | 23:34 |
| Sam-I-Am | ok. seems like your dhcp provisioning block patch tagged this bug. | 23:34 |
| *** itsuugo has joined #openstack-neutron | 23:34 | |
| kevinbenton | Sam-I-Am: yes, at the time we thought this was the issue | 23:35 |
| Sam-I-Am | openstack infra marked it back in-progress and said it was fixed in 9.0.0.0b3 | 23:35 |
| kevinbenton | Sam-I-Am: but it wasn't if this hits stable/mitaka | 23:35 |
| kevinbenton | Sam-I-Am: provisioning blocks didn't exist then | 23:35 |
| kevinbenton | let me see if i can find the patch he was working on | 23:36 |
| Sam-I-Am | cool | 23:36 |
| *** ijw has quit IRC | 23:36 | |
| kevinbenton | Sam-I-Am: https://review.openstack.org/#/c/255237/ | 23:37 |
| kevinbenton | Sam-I-Am: so that is kinda related to the status stuff | 23:37 |
| *** wolverineav has quit IRC | 23:37 | |
| Sam-I-Am | hmm. it didnt tag the bugs for it. | 23:38 |
| kevinbenton | Sam-I-Am: but i need to confirm with him if it was supposed to address that bug | 23:38 |
| kevinbenton | Sam-I-Am: i remember him saying it was needed | 23:38 |
| Sam-I-Am | that'd be cool | 23:38 |
| kevinbenton | Sam-I-Am: but maybe that wasn't quite enough | 23:38 |
| Sam-I-Am | i'm quite curious | 23:38 |
| Sam-I-Am | not using quite that new of a version of neutron | 23:39 |
| kevinbenton | Sam-I-Am: can you see if the behavior is still the same with current stable/newton branch? | 23:39 |
| kevinbenton | Sam-I-Am: or is that difficult | 23:39 |
| Sam-I-Am | not right this minute. this sort of came up during a massive rewrite of the deployment scenarios in the networking guide. | 23:40 |
| Sam-I-Am | i'd need to toss up another lab | 23:40 |
| kevinbenton | ok | 23:40 |
| kevinbenton | let me see if i can catch him tonight | 23:40 |
| Sam-I-Am | coo | 23:41 |
| *** wolverineav has joined #openstack-neutron | 23:41 | |
| Sam-I-Am | if push comes to shove i can deploy some lab bits | 23:41 |
| *** ijw has joined #openstack-neutron | 23:41 | |
| Sam-I-Am | just kind of bogged down in this patch - https://review.openstack.org/#/c/367691 | 23:42 |
| *** jamielennox|away is now known as jamielennox | 23:43 | |
| *** markvoelker has joined #openstack-neutron | 23:44 | |
| Sam-I-Am | which minds me, i need to find andreas scheuring. not sure i've seen him on irc. | 23:44 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Fix ML2 revision_number handling in port updates https://review.openstack.org/373582 | 23:44 |
| *** itsuugo has quit IRC | 23:46 | |
| *** itsuugo has joined #openstack-neutron | 23:47 | |
| *** markvoelker has quit IRC | 23:49 | |
| *** gouthamr has joined #openstack-neutron | 23:52 | |
| *** yamamoto has quit IRC | 23:53 | |
| *** itsuugo has quit IRC | 23:55 | |
| *** itsuugo has joined #openstack-neutron | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!