| *** markvoelker has joined #openstack-meeting-5 | 00:05 | |
| *** mjturek has quit IRC | 00:05 | |
| *** markvoelker has quit IRC | 00:10 | |
| *** TuanVu has joined #openstack-meeting-5 | 00:16 | |
| *** TuanVu has quit IRC | 00:23 | |
| *** TuanVu has joined #openstack-meeting-5 | 00:32 | |
| *** TuanVu has quit IRC | 00:36 | |
| *** njohnston has quit IRC | 00:48 | |
| *** TuanVu has joined #openstack-meeting-5 | 00:53 | |
| *** TuanVu has quit IRC | 00:57 | |
| *** markvoelker has joined #openstack-meeting-5 | 01:05 | |
| *** markvoelker has quit IRC | 01:09 | |
| *** TuanVu has joined #openstack-meeting-5 | 01:10 | |
| *** yamahata has quit IRC | 01:14 | |
| *** TuanVu has quit IRC | 01:15 | |
| *** lujinluo has joined #openstack-meeting-5 | 01:17 | |
| *** TuanVu has joined #openstack-meeting-5 | 01:25 | |
| *** TuanVu has quit IRC | 01:29 | |
| *** TuanVu has joined #openstack-meeting-5 | 01:38 | |
| *** TuanVu has quit IRC | 01:43 | |
| *** TuanVu has joined #openstack-meeting-5 | 01:53 | |
| *** TuanVu has quit IRC | 01:58 | |
| *** TuanVu has joined #openstack-meeting-5 | 02:01 | |
| *** TuanVu has quit IRC | 02:05 | |
| *** TuanVu has joined #openstack-meeting-5 | 02:09 | |
| *** TuanVu has quit IRC | 02:14 | |
| *** TuanVu has joined #openstack-meeting-5 | 02:21 | |
| *** lujinluo has quit IRC | 02:23 | |
| *** TuanVu has quit IRC | 02:25 | |
| *** TuanVu has joined #openstack-meeting-5 | 02:27 | |
| *** lujinluo has joined #openstack-meeting-5 | 02:28 | |
| *** lujinluo has quit IRC | 02:30 | |
| *** lujinluo has joined #openstack-meeting-5 | 02:30 | |
| *** TuanVu has quit IRC | 02:32 | |
| *** TuanVu has joined #openstack-meeting-5 | 02:34 | |
| *** TuanVu has quit IRC | 04:01 | |
| *** TuanVu has joined #openstack-meeting-5 | 04:05 | |
| *** lujinluo has quit IRC | 04:07 | |
| *** TuanVu has quit IRC | 04:44 | |
| *** skazi has quit IRC | 04:58 | |
| *** lujinluo has joined #openstack-meeting-5 | 05:33 | |
| *** lujinluo has quit IRC | 05:38 | |
| *** lujinluo has joined #openstack-meeting-5 | 05:47 | |
| *** lujinluo has quit IRC | 05:52 | |
| *** lujinluo has joined #openstack-meeting-5 | 06:15 | |
| *** roman_g has joined #openstack-meeting-5 | 06:25 | |
| *** lujinluo has quit IRC | 06:32 | |
| *** lujinluo has joined #openstack-meeting-5 | 06:36 | |
| *** lujinluo has quit IRC | 06:44 | |
| *** persia has quit IRC | 07:19 | |
| *** persia has joined #openstack-meeting-5 | 07:21 | |
| *** roman_g has quit IRC | 07:22 | |
| *** ralonsoh has joined #openstack-meeting-5 | 07:26 | |
| *** ralonsoh has quit IRC | 07:27 | |
| *** ralonsoh has joined #openstack-meeting-5 | 07:28 | |
| *** roman_g has joined #openstack-meeting-5 | 07:35 | |
| *** slaweq has joined #openstack-meeting-5 | 08:02 | |
| *** roman_g has quit IRC | 08:11 | |
| *** ralonsoh_ has joined #openstack-meeting-5 | 08:29 | |
| *** ralonsoh has quit IRC | 08:31 | |
| *** derekh has joined #openstack-meeting-5 | 08:39 | |
| *** ralonsoh_ is now known as ralonsoh | 08:56 | |
| *** roman_g has joined #openstack-meeting-5 | 09:05 | |
| *** roman_g has quit IRC | 09:47 | |
| *** roman_g has joined #openstack-meeting-5 | 09:57 | |
| *** ralonsoh has quit IRC | 10:30 | |
| *** ralonsoh has joined #openstack-meeting-5 | 10:31 | |
| *** ralonsoh has quit IRC | 10:38 | |
| *** ralonsoh has joined #openstack-meeting-5 | 10:39 | |
| *** roman_g has quit IRC | 11:11 | |
| *** roman_g has joined #openstack-meeting-5 | 11:13 | |
| *** njohnston has joined #openstack-meeting-5 | 11:28 | |
| *** dims has quit IRC | 12:30 | |
| *** dims has joined #openstack-meeting-5 | 12:33 | |
| *** mjturek has joined #openstack-meeting-5 | 13:22 | |
| *** lujinluo has joined #openstack-meeting-5 | 13:41 | |
| *** mjturek has quit IRC | 13:43 | |
| *** lujinluo has quit IRC | 13:52 | |
| *** lujinluo has joined #openstack-meeting-5 | 13:53 | |
| *** annp_ has joined #openstack-meeting-5 | 13:55 | |
| lujinluo | #startmeeting neutron_upgrades | 14:00 |
|---|---|---|
| openstack | Meeting started Thu Oct 18 14:00:06 2018 UTC and is due to finish in 60 minutes. The chair is lujinluo. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: neutron_upgrades)" | 14:00 | |
| openstack | The meeting name has been set to 'neutron_upgrades' | 14:00 |
| lujinluo | o/ | 14:00 |
| njohnston | o/ | 14:01 |
| ralonsoh | o/ | 14:02 |
| slaweq | hi | 14:02 |
| lujinluo | hi njohnston ralonsoh slaweq ! | 14:02 |
| lujinluo | then let's get started | 14:02 |
| lujinluo | last week we have one action item: njohnston to enabl new_face in Agent OVO | 14:03 |
| lujinluo | #link https://review.openstack.org/#/c/609749/ | 14:03 |
| njohnston | Yes, I tried that, and it had some issues | 14:03 |
| lujinluo | *new_facade | 14:03 |
| njohnston | not as straightforward as I would have hoped | 14:03 |
| njohnston | given that the agent part of the spreadsheet was completed | 14:03 |
| lujinluo | yeah, I downloaded the patch locally and tried to test it. I think we still miss something | 14:04 |
| njohnston | I worry that it is some of the "common" code that we can't convert until all of the code has been changed to OVO | 14:05 |
| njohnston | i.e. the stuff marked as "blocked | 14:05 |
| njohnston | " on the OVO tracking spreadsheet | 14:05 |
| *** hongbin has joined #openstack-meeting-5 | 14:05 | |
| *** slaweq has quit IRC | 14:06 | |
| lujinluo | i see. | 14:07 |
| njohnston | the failures in unit tests: http://logs.openstack.org/49/609749/1/check/openstack-tox-py36/a962088/testr_results.html.gz | 14:07 |
| njohnston | and in functional: http://logs.openstack.org/49/609749/1/check/neutron-functional/d69aeae/job-output.txt.gz#_2018-10-11_19_14_37_954448 | 14:07 |
| njohnston | that "common" thing is just a hypothesis, I have no data to back it up yet | 14:07 |
| *** slaweq has joined #openstack-meeting-5 | 14:08 | |
| lujinluo | the failures are most related to router and l3 agent though | 14:08 |
| lujinluo | it could be router OVO that is blocking | 14:08 |
| njohnston | so my thought was to pick a specific test and step through with pdb and see if I can track the problem | 14:08 |
| lujinluo | i think that is what we should try at least | 14:09 |
| lujinluo | i will continue testing that patch on my side as well. i will ping you if i find anything interesting njohnston | 14:10 |
| njohnston | #action njohnston pick tests from https://review.openstack.org/609749 and step through with pdb to detect failure | 14:11 |
| njohnston | will do, thanks very much for your help lujinluo! | 14:11 |
| lujinluo | no problem! | 14:11 |
| lujinluo | then let's move to OVO patches | 14:11 |
| lujinluo | #topic OVO | 14:11 |
| *** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)" | 14:11 | |
| lujinluo | #link https://review.openstack.org/#/q/status:open+topic:bp/adopt-oslo-versioned-objects-for-db | 14:12 |
| lujinluo | #link https://review.openstack.org/#/c/608198/ | 14:12 |
| lujinluo | Use NetworkRBAC OVO in neutron/db/db_base_plugin_v2.py | 14:12 |
| lujinluo | i just have one small question in https://review.openstack.org/#/c/608198/7/neutron/db/db_base_plugin_v2.py@437 | 14:13 |
| lujinluo | other changes look good to me | 14:13 |
| ralonsoh | lujinluo: I'll answer this question today | 14:14 |
| lujinluo | thanks! ralonsoh | 14:14 |
| lujinluo | next #link https://review.openstack.org/#/c/608638/ | 14:15 |
| lujinluo | Use NetworkRBAC OVO in tests/unit/db/test_db_base_plugin_v2.py | 14:15 |
| lujinluo | it has got 3 +1 | 14:16 |
| lujinluo | slaweq: could you please take a look at it when you are available? | 14:16 |
| lujinluo | it is changes to a test file | 14:16 |
| slaweq | lujinluo: sure | 14:16 |
| slaweq | I will review it today | 14:16 |
| lujinluo | thank you slaweq ! | 14:16 |
| lujinluo | #link https://review.openstack.org/#/c/611461/ | 14:17 |
| lujinluo | WIP: Fix port status for router ha_migration | 14:17 |
| lujinluo | it is still a WIP | 14:17 |
| lujinluo | it is not a OVO transition but related to it, so i guess we can wait until CI is cool with it before we review | 14:20 |
| njohnston | +1 | 14:20 |
| lujinluo | #link https://review.openstack.org/#/c/610280/ | 14:20 |
| *** mjturek has joined #openstack-meeting-5 | 14:21 | |
| lujinluo | Use RBACBaseObject OVO in neutron/db/rbac_db_mixin.py | 14:21 |
| ralonsoh | I still need to fix the UTs | 14:21 |
| ralonsoh | I need more time | 14:21 |
| lujinluo | got it. yeah, take your time :) | 14:21 |
| lujinluo | #link https://review.openstack.org/#/c/608628/ | 14:22 |
| lujinluo | Use NetworkRBAC OVO in neutron/db/external_net_db.py | 14:22 |
| lujinluo | slawek suggeted that we wait for #link https://review.openstack.org/#/c/609692/ to be merged first | 14:23 |
| lujinluo | but the changes look good to most of us :) | 14:23 |
| lujinluo | #link https://review.openstack.org/#/c/544206/ | 14:24 |
| lujinluo | Integration of (Distributed) Port Binding OVO | 14:25 |
| lujinluo | last week, i said that i was trying to justify the code change mentioned by hongbin, but it turns that the change was not necessary, so i pushed a new patch set | 14:26 |
| lujinluo | i still have one last comment related to load relationship that i have not tested yet. | 14:26 |
| lujinluo | i will remove my -1 if i finished the testing, then reviews are much welcomed | 14:27 |
| lujinluo | #link https://review.openstack.org/#/c/602497/ | 14:28 |
| lujinluo | use ovo for L3HARouterAgentPortBinding in l3_hamode. | 14:28 |
| lujinluo | Zuul is still not happy about it | 14:29 |
| *** mjturek has quit IRC | 14:31 | |
| hongbin | o/ | 14:31 |
| lujinluo | just rechecked and hope it will make it this time | 14:31 |
| lujinluo | hi hongbin ! | 14:31 |
| * hongbin is being pulled from fwaas meeting | 14:31 | |
| hongbin | lujinluo: hi | 14:32 |
| lujinluo | aha, busy with fwaas these days? hongbin | 14:32 |
| lujinluo | next ps is from hongbin #link https://review.openstack.org/#/c/610182/ | 14:32 |
| hongbin | lujinluo: a bit, but my topic is finished there now | 14:32 |
| lujinluo | Support fetching specific db column in OVO | 14:32 |
| hongbin | for this one, i am figuring out how to pass the unit tests | 14:33 |
| lujinluo | this patch is -W by hongbin, so let's give him more time :) | 14:33 |
| hongbin | yes, more time is always needed :) | 14:33 |
| lujinluo | #link https://review.openstack.org/#/c/507772/ Use Network OVO in db_base_plugin | 14:34 |
| lujinluo | Tuan did not update this patch since last week, and boden left a comment about the compatibility layer which we once discussed before | 14:35 |
| *** lemko has joined #openstack-meeting-5 | 14:35 | |
| lujinluo | I will try to catch Tuan this week to see if he has any updates on the comments | 14:35 |
| lujinluo | #action lujin to keep an eye on Network OVO in db_base_plugin | 14:35 |
| lujinluo | #link https://review.openstack.org/#/c/609847/ Support filter with "not in" criterion | 14:36 |
| lujinluo | i think this patch would benefit us a lot, thanks hongbin! | 14:37 |
| njohnston | +1 | 14:37 |
| hongbin | np | 14:37 |
| ralonsoh | agree | 14:37 |
| hongbin | will address the comments later, good comments though | 14:37 |
| lujinluo | excellent! | 14:37 |
| hongbin | and i expect there might be a bit more criterion to add (other than "not in") | 14:38 |
| *** mjturek_ has joined #openstack-meeting-5 | 14:38 | |
| *** TuanVu has joined #openstack-meeting-5 | 14:39 | |
| *** TuanVu has quit IRC | 14:39 | |
| njohnston | yes, this builds a good base for those future operators to be developed on later | 14:39 |
| lujinluo | +1 | 14:40 |
| lujinluo | The next two patches seem to be WIP #link https://review.openstack.org/#/c/610184/ and #link https://review.openstack.org/#/c/609848/ | 14:41 |
| lujinluo | Convert Subnet to OVO in ipam_pluggable_backend.py | 14:42 |
| lujinluo | Convert Port to OVO in db/ipam_pluggable_backend.py | 14:42 |
| hongbin | the first one is for testing the feature for fetching specific column | 14:43 |
| hongbin | the second one is for testing the feature of "not in" filter | 14:43 |
| hongbin | both are small patches, i will remove the WIP marker once the depending features are landed | 14:44 |
| lujinluo | understood. | 14:44 |
| lujinluo | the next one about retry decorator, which i want to discuss about | 14:45 |
| lujinluo | #Link https://review.openstack.org/#/c/602362/ | 14:45 |
| lujinluo | and this one in neutron-lib #link https://review.openstack.org/#/c/605365/ | 14:45 |
| lujinluo | So michel would like us not to refactor old engine facade | 14:46 |
| lujinluo | and suggested to "unpack the context and call a helper function" | 14:46 |
| njohnston | yes, it seems like an interesting idea, although I have not given it enough thought to know how to go about it | 14:47 |
| lujinluo | my concern is that, if we follow this way, do we need to do it in every methods that we use retry decorator | 14:47 |
| lujinluo | Lajos said he/she would push a patch set to do it | 14:47 |
| lujinluo | but only the Tag ovo | 14:48 |
| lujinluo | ok, i think it is an interesting idea too. just want to bring it on the table to make us aware of its status :) | 14:48 |
| lujinluo | we can continue the discussion in Gerrit after Lajos pushes another ps in #link https://review.openstack.org/#/c/602362/ | 14:50 |
| lujinluo | the rest of the ovo patches are not updated since last week | 14:50 |
| lujinluo | #topic new engine facade | 14:51 |
| *** openstack changes topic to "new engine facade (Meeting topic: neutron_upgrades)" | 14:51 | |
| lujinluo | #link https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/enginefacade-switch | 14:51 |
| lujinluo | interesting enough, i was trying to switch to new engine facade following #Link https://review.openstack.org/#/c/545501/ | 14:51 |
| lujinluo | so i pushed #link https://review.openstack.org/611462 | 14:52 |
| lujinluo | but it turns out it is some cleaning work of agent ovo | 14:52 |
| lujinluo | "Remove context guard of Agent OVO" | 14:53 |
| njohnston | I'll try rebasing my agent ovo engine facade adoption change on top of this | 14:54 |
| lujinluo | yeah, that worth a try | 14:54 |
| njohnston | done | 14:54 |
| lujinluo | great! | 14:55 |
| lujinluo | ok, that's all for the patches. | 14:55 |
| lujinluo | #topic open discussion | 14:55 |
| *** openstack changes topic to "open discussion (Meeting topic: neutron_upgrades)" | 14:55 | |
| lujinluo | Does anyone have anything else wants to share? | 14:55 |
| lujinluo | if not, let's call it a day | 14:56 |
| njohnston | thanks all | 14:56 |
| lujinluo | have a great day/night everyone | 14:56 |
| ralonsoh | bye! | 14:56 |
| lujinluo | see you all next week | 14:56 |
| lujinluo | #endmeeting | 14:57 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:57 | |
| openstack | Meeting ended Thu Oct 18 14:57:10 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:57 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-18-14.00.html | 14:57 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-18-14.00.txt | 14:57 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-18-14.00.log.html | 14:57 |
| *** annp_ has quit IRC | 14:57 | |
| *** lujinluo has quit IRC | 15:07 | |
| *** lujinluo has joined #openstack-meeting-5 | 15:19 | |
| *** lujinluo has quit IRC | 15:24 | |
| *** skazi has joined #openstack-meeting-5 | 15:27 | |
| *** lujinluo has joined #openstack-meeting-5 | 15:43 | |
| *** mjturek_ has quit IRC | 15:49 | |
| *** lujinluo has quit IRC | 15:51 | |
| *** lujinluo has joined #openstack-meeting-5 | 15:55 | |
| *** lujinluo has quit IRC | 16:08 | |
| *** roman_g has quit IRC | 16:21 | |
| *** lujinluo has joined #openstack-meeting-5 | 16:31 | |
| *** lujinluo has quit IRC | 16:36 | |
| *** lujinluo has joined #openstack-meeting-5 | 16:40 | |
| *** lujinluo has quit IRC | 16:40 | |
| *** derekh has quit IRC | 16:54 | |
| *** yamahata has joined #openstack-meeting-5 | 17:04 | |
| *** mjturek has joined #openstack-meeting-5 | 17:16 | |
| *** mjturek has quit IRC | 18:35 | |
| *** mjturek has joined #openstack-meeting-5 | 18:53 | |
| *** lujinluo has joined #openstack-meeting-5 | 19:22 | |
| *** lujinluo has quit IRC | 19:30 | |
| *** lujinluo has joined #openstack-meeting-5 | 19:31 | |
| *** lujinluo has quit IRC | 19:46 | |
| *** lujinluo has joined #openstack-meeting-5 | 19:47 | |
| *** ralonsoh has quit IRC | 20:00 | |
| *** lujinluo has quit IRC | 20:00 | |
| *** mjturek has quit IRC | 20:38 | |
| *** lemko has quit IRC | 22:34 | |
| *** hongbin has quit IRC | 23:08 | |
| *** mjturek has joined #openstack-meeting-5 | 23:18 | |
| *** mjturek has quit IRC | 23:42 | |
| *** markvoelker has joined #openstack-meeting-5 | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!