| *** openstack has quit IRC | 04:42 | |
| *** openstack has joined #openstack-dragonflow | 04:46 | |
| *** yuval has joined #openstack-dragonflow | 04:48 | |
| *** dims has joined #openstack-dragonflow | 04:49 | |
| *** dims is now known as Guest58397 | 04:49 | |
| dimak | Good morning | 05:16 |
|---|---|---|
| lihi | Good morning | 05:51 |
| oanson | Morning | 06:12 |
| leyal | Morning | 06:13 |
| oanson | I'm preparing a patch to add kuryr-dragonflow and OSA-dragonflow gate jobs to our check pipeline (Ci / Jenkins) | 06:16 |
| oanson | Any objections before I do? | 06:16 |
| oanson | lihi, dimak, leyal, irenab ^^^ ? | 06:17 |
| dimak | +1 | 06:17 |
| leyal | sounds-good.. | 06:17 |
| oanson | Done https://review.openstack.org/494388 | 06:18 |
| lihi | +1 | 06:19 |
| oanson | lihi, great. I wasn't sure how you'd feel about the OSA gate yet :) | 06:19 |
| irenab | +1, start as not voting | 06:21 |
| oanson | Of course :) | 06:21 |
| dimak | oanson, wanna take a look at https://review.openstack.org/#/c/489003/ ? There's some strange error in q-svc log, a port did not get retrieved from the database though it should have been there | 06:24 |
| dimak | (in the tempest job) | 06:25 |
| dimak | Causes an error in BGP plugin | 06:25 |
| oanson | tempest/BGP will fail until https://review.openstack.org/494155 and https://review.openstack.org/494154 are merged | 06:46 |
| leyal | irenab , dimak - following https://bugs.launchpad.net/dragonflow/+bug/1708178 - it's look that the issue is that lport of the router created in the compute node - before it's lswitch. Omer suggested to try to disable the selective-proactive in DF - and it's seems to fix the issue. | 07:10 |
| openstack | Launchpad bug 1708178 in DragonFlow "LBaaSv2 with 3rd party provider does not work if L3agent is disabled" [Critical,New] | 07:10 |
| leyal | so it's can be a temporary workaround - and i think the issue should be solved when https://review.openstack.org/#/c/480196 will be ready .. | 07:11 |
| *** NatanBro has joined #openstack-dragonflow | 07:22 | |
| *** openstackgerrit has joined #openstack-dragonflow | 07:25 | |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add fullstack tests for DB API https://review.openstack.org/494247 | 07:25 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add an in-memory NB database driver https://review.openstack.org/494248 | 07:25 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: OvsPort to reference lport, not just hold its id https://review.openstack.org/480195 | 07:37 |
| *** zenoway has joined #openstack-dragonflow | 07:55 | |
| *** openstackgerrit has quit IRC | 08:17 | |
| *** openstackgerrit has joined #openstack-dragonflow | 08:20 | |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add an in-memory NB database driver https://review.openstack.org/494248 | 08:20 |
| *** kkxue has joined #openstack-dragonflow | 08:54 | |
| openstackgerrit | Merged openstack/dragonflow master: tox: Add rm to whitelist_externals https://review.openstack.org/494245 | 10:13 |
| *** zenoway has quit IRC | 10:34 | |
| *** zenoway has joined #openstack-dragonflow | 10:34 | |
| *** zenoway has quit IRC | 10:41 | |
| *** zenoway has joined #openstack-dragonflow | 10:42 | |
| *** kkxue has quit IRC | 10:43 | |
| *** kkxue has joined #openstack-dragonflow | 10:44 | |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add fullstack tests for DB API https://review.openstack.org/494247 | 11:01 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add an in-memory NB database driver https://review.openstack.org/494248 | 11:01 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: [WIP] Replace usage of tenant_id with project_id https://review.openstack.org/493537 | 11:21 |
| *** Guest58397 is now known as dims | 11:37 | |
| oanson | dimak, Are you in? | 11:55 |
| dimak | oanson, yes | 11:57 |
| oanson | Subnet is embedded in lswitch, but referenced in lport. db_store will return the subnet if it was already handled, right? | 11:57 |
| oanson | Come to think of it, that doesn't matter... | 11:58 |
| dimak | Not sure I understand | 11:58 |
| oanson | Anyway, I think referencing non-first-class models should be forbidden. | 11:58 |
| dimak | Oh | 11:58 |
| oanson | If we try to fire events by order, lport references subnets, so it tries to look them up | 11:58 |
| oanson | But they are not first order. They have to be found via the lswitch | 11:59 |
| dimak | Or maynbe embedding first ordre objects should be removed ;) | 11:59 |
| oanson | That's a lot of background magic to avoid the simpler (and probably more correct) solution: Make subnet a first-class model | 11:59 |
| oanson | If it's a first-order object, there's not need for it to be embedded. | 12:00 |
| dimak | yeah | 12:00 |
| oanson | Glad we caught it before the end of the month :) | 12:00 |
| dimak | What aboud deteting referenced objects? | 12:02 |
| oanson | post-delete hook? | 12:02 |
| dimak | Floating IP's floating port is removed before the floating IP itself | 12:02 |
| dimak | so once the patch that marks stale on delete is merged, DNAT will break (the new version) | 12:03 |
| dimak | :( | 12:03 |
| oanson | Two options: 1. update the floating ip object during port delette 2. The relevant apps should know this 'inconsistent' state is legal | 12:03 |
| oanson | The first is better software-engineering wise, the second better for performance (probably) and stability | 12:04 |
| openstackgerrit | Merged openstack/dragonflow master: [05/xx] Move port events handling into standalone app https://review.openstack.org/493560 | 12:12 |
| *** NatanBro has quit IRC | 12:30 | |
| *** zenoway has quit IRC | 13:01 | |
| *** zenoway has joined #openstack-dragonflow | 13:02 | |
| *** Natanbro has joined #openstack-dragonflow | 13:04 | |
| *** zenoway has quit IRC | 13:06 | |
| * dims waves to lihi | 13:34 | |
| *** kkxue has quit IRC | 13:39 | |
| *** mlavalle has joined #openstack-dragonflow | 13:40 | |
| lihi | dims o/ | 13:44 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: [WIP] Make Subnet a first-order model https://review.openstack.org/494557 | 13:52 |
| *** Natanbro has quit IRC | 14:10 | |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Update etcd standalone example local.conf to work out of the box https://review.openstack.org/494676 | 18:49 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Mark objects deleted from db_store as stale https://review.openstack.org/494153 | 20:02 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: Make Subnet a first-order model https://review.openstack.org/494557 | 20:02 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: OvsPort to reference lport, not just hold its id https://review.openstack.org/480195 | 20:02 |
| openstackgerrit | Omer Anson proposed openstack/dragonflow master: An instance update also sends an update on all referred instances https://review.openstack.org/480196 | 20:02 |
| *** oanson has quit IRC | 21:42 | |
| *** oanson has joined #openstack-dragonflow | 21:44 | |
| *** openstackgerrit has quit IRC | 22:33 | |
| *** mlavalle has quit IRC | 23:35 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!