*** ricolin has joined #openstack-meeting-5 | 01:35 | |
*** yamahata has quit IRC | 01:43 | |
*** yamahata has joined #openstack-meeting-5 | 03:06 | |
*** skazi has quit IRC | 05:53 | |
*** beisner has quit IRC | 06:40 | |
*** beisner has joined #openstack-meeting-5 | 06:40 | |
*** slaweq_ has joined #openstack-meeting-5 | 06:51 | |
*** slaweq_ has quit IRC | 07:00 | |
*** slaweq_ has joined #openstack-meeting-5 | 07:04 | |
*** hongbin has quit IRC | 07:23 | |
*** lemko[m] has quit IRC | 08:11 | |
*** ricolin has quit IRC | 08:12 | |
*** skazi has joined #openstack-meeting-5 | 08:24 | |
*** ianychoi_ has quit IRC | 08:42 | |
*** derekh has joined #openstack-meeting-5 | 08:43 | |
*** roman_g has joined #openstack-meeting-5 | 08:52 | |
*** skazi has quit IRC | 11:50 | |
*** lujinluo has joined #openstack-meeting-5 | 12:10 | |
*** lujinluo has quit IRC | 12:51 | |
*** mjturek has joined #openstack-meeting-5 | 12:52 | |
*** mjturek has quit IRC | 13:30 | |
*** mjturek has joined #openstack-meeting-5 | 13:37 | |
*** mjturek has quit IRC | 13:45 | |
*** mjturek has joined #openstack-meeting-5 | 13:45 | |
*** hongbin has joined #openstack-meeting-5 | 13:49 | |
*** hungpv has joined #openstack-meeting-5 | 13:54 | |
*** lujinluo has joined #openstack-meeting-5 | 13:56 | |
lujinluo | #startmeeting neutron_upgrades | 14:00 |
---|---|---|
openstack | Meeting started Thu Aug 16 14:00:11 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 |
*** annp_ has joined #openstack-meeting-5 | 14:00 | |
*** mjturek has quit IRC | 14:00 | |
lujinluo | ok, I will wait for my friends | 14:00 |
*** njohnston has joined #openstack-meeting-5 | 14:01 | |
njohnston | o/ | 14:01 |
lujinluo | o/ | 14:01 |
lujinluo | It seems it would be just the two of us today, haha | 14:01 |
lujinluo | #topic OVO | 14:02 |
*** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)" | 14:02 | |
njohnston | perhaps others will join in time :-) | 14:02 |
lujinluo | haha, then let's wait for another 3 mins | 14:02 |
lujinluo | after that, let's start ;) | 14:02 |
njohnston | seems to be a fair number of gate issues of late | 14:02 |
njohnston | many Zuul -1s | 14:03 |
lujinluo | yeah | 14:03 |
lujinluo | hopefully it won't affect releasing Rocky | 14:03 |
hungpv | yes, hope so | 14:03 |
lujinluo | oh, hey hungpv | 14:04 |
lujinluo | do you know if Tuan will come today? hungpv | 14:04 |
njohnston | sadly our Berlin talk proposal was not accepted FYI :-( | 14:04 |
hungpv | I think he will come late | 14:04 |
lujinluo | :-( so sorry to hear that | 14:05 |
hungpv | But I'm not suee | 14:05 |
lujinluo | I see. Then let's get started first | 14:05 |
hungpv | Yeah, what a loss for them :(( | 14:05 |
lujinluo | #link https://review.openstack.org/#/q/topic:bp/adopt-oslo-versioned-objects-for-db+(status:open) | 14:05 |
lujinluo | #link https://review.openstack.org/#/c/382037/ PortBinding Level | 14:05 |
lujinluo | slaweq respinned it and it hit the Zuul failure | 14:06 |
lujinluo | I just rechecked it, and I think it is very close to merge | 14:06 |
lujinluo | #link https://review.openstack.org/#/c/544206/ PortBinding OVO | 14:07 |
lujinluo | I pushed a PS about one hour ago. It should pass, so welcome reviews! | 14:07 |
lujinluo | if zuul is happy ;) | 14:07 |
njohnston | :) | 14:08 |
lujinluo | #link https://review.openstack.org/#/c/549168/ Router OVO | 14:08 |
lujinluo | i guess you want to share something about this one? hungpv | 14:08 |
lujinluo | It is still pretty red | 14:08 |
*** mjturek has joined #openstack-meeting-5 | 14:11 | |
lujinluo | a quick comment would be https://review.openstack.org/#/c/549168/21/neutron/db/l3_db.py@259 | 14:11 |
*** hungpv_ has joined #openstack-meeting-5 | 14:11 | |
lujinluo | not to change _update_router_db to _update_router_obj in one patch | 14:11 |
*** hungpv has quit IRC | 14:12 | |
lujinluo | i think you may add _update_router_obj to handle methods using objects and leave other in db, cause router objects are used in too many places | 14:12 |
lujinluo | hungpv_: oh did you lose connection? | 14:12 |
hungpv_ | Yes, thanks for suggestions | 14:13 |
hungpv_ | My network acting weird recently | 14:13 |
hungpv_ | Last week I tried to sign in but got baned for spam | 14:13 |
njohnston | I think your suggestion is a good one lujinluo, looking at errors like this it looks like router objects are getting misallocated somehow: | 14:14 |
njohnston | ERROR [neutron.db._utils] Cannot clean up created object 2a337710-5021-49fa-b1b7-7170d1bc2c49. Exception: Class 'neutron.objects.router.Router' is not mapped" | 14:14 |
*** slaweq_ is now known as slaweq | 14:14 | |
lujinluo | yes, I had same issued with Port object | 14:14 |
lujinluo | they are too many in codes | 14:14 |
lujinluo | we have to eat them bite by bite ;) | 14:14 |
njohnston | +1 | 14:15 |
hungpv_ | +1 | 14:15 |
hungpv_ | I will think about it tomorrow | 14:15 |
lujinluo | hopefully this can at least take us one step further | 14:15 |
lujinluo | #link https://review.openstack.org/#/c/579053/ OVO in subnet_alloc.py | 14:16 |
lujinluo | I checked this one njohnston | 14:16 |
lujinluo | But I am not sure if you have tried what I tried | 14:16 |
njohnston | Yes, and your comment is a good one; I will respin once my slew of meetings is done in 2h or so :-) | 14:17 |
njohnston | I have not tried that. | 14:17 |
lujinluo | great! | 14:17 |
lujinluo | #link https://review.openstack.org/#/c/580721/ Port OVO in ipam pluggable backend | 14:18 |
lujinluo | I did not have enough time for it yet, but I saw Hongbin reviewed it | 14:18 |
hongbin | o/ | 14:19 |
*** hungpv has joined #openstack-meeting-5 | 14:19 | |
lujinluo | I think it is related although I have not tested it yet | 14:19 |
lujinluo | hello hongbin! | 14:19 |
hongbin | lujinluo: hi | 14:19 |
hongbin | yes, i left 2 comments in that patch | 14:20 |
*** hungpv_ has quit IRC | 14:20 | |
hongbin | one of them might relate to the test failure | 14:20 |
njohnston | Yes. One of hongbin's comments was that to iterate through all ports to ensure they are not owned by a set of administrative users is suboptimal in situations with large numbers of ports. This is a valid point, so I will look at ways to offload that into the OVO so it can be taken care of in SQL. | 14:20 |
lujinluo | +1 | 14:21 |
hongbin | cool | 14:21 |
lujinluo | Great! | 14:21 |
lujinluo | #link https://review.openstack.org/#/c/565773/ Tag OVO | 14:22 |
njohnston | I don't see lajoskatona on, but this is a good forum to talk about one thing | 14:22 |
lujinluo | https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 this one? | 14:22 |
njohnston | line 70 here: https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 | 14:23 |
lujinluo | Do you mean you'd prefer that we move db_api.retry_if_session_inactive() to OVO as well? | 14:23 |
njohnston | It is my understanding that a full cutover to OVO would hide the details of the database interaction from the rest of the neutron code, as a side effect of upgrade compatibility | 14:24 |
njohnston | Yes, that is what I was thinking, but I am wondering if I am being excessively object oriented in my thinking on this | 14:24 |
lujinluo | Well, I actually think this is a very good point | 14:25 |
lujinluo | So far, we have not had time to consider these decorators | 14:25 |
lujinluo | but indeed they are dealing with db sessions | 14:25 |
lujinluo | I would be in favor of considering moving them into OVO objects | 14:26 |
njohnston | it seems at least that a decorator like that should be as close as possible to the code that actually interacts with the database | 14:26 |
lujinluo | agree | 14:26 |
njohnston | ok, I will come up with a specific recommendation for where it should be relocated to and comment on the change | 14:26 |
lujinluo | I will check the Google sheet Tuan created long ago and add these decorators, to keep us reminded | 14:27 |
njohnston | #action njohnston Comment on https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 with location for decorator relocation | 14:27 |
njohnston | #action lujinluo Add decorators to google tracking spreadsheet | 14:27 |
njohnston | :-) | 14:27 |
njohnston | otherwise I will forget :-) | 14:28 |
lujinluo | njohnston: thanks ;) | 14:28 |
lujinluo | #link https://review.openstack.org/#/c/561834/ segmenta model to OVO | 14:28 |
lujinluo | this one is ready to go as Stein opens | 14:29 |
njohnston | indeed | 14:29 |
njohnston | just added a couple of extra cores for review | 14:29 |
lujinluo | thanks! i had a network interruption here i believe, suddenly could not open that page :( | 14:30 |
lujinluo | well, thankfully that's all the active patches we have now | 14:30 |
lujinluo | #topic open discussion | 14:30 |
*** openstack changes topic to "open discussion (Meeting topic: neutron_upgrades)" | 14:30 | |
lujinluo | #link Stein Neutron etherpad https://etherpad.openstack.org/p/neutron-stein-ptg | 14:31 |
lujinluo | just a FYI of the PTG etherpad | 14:31 |
lujinluo | As far as I could see, we do not have any topics related to OVO/upgrades yet | 14:32 |
lujinluo | Does anyone have anything else he/she wants to share? | 14:32 |
njohnston | I will try to drum up some help for OVO/upgrades through informal discussions, but I don't have any topics for it | 14:32 |
lujinluo | I see. Then just write Help wanted? ;) | 14:34 |
lujinluo | well, let's try to gather some thoughts in our next meeting | 14:34 |
njohnston | something like that ;) | 14:34 |
lujinluo | One week for us to spot unseen things | 14:35 |
njohnston | thanks everyone | 14:35 |
lujinluo | if no one has anything else | 14:35 |
lujinluo | let's call it a day | 14:35 |
lujinluo | have a good day/night my friends | 14:36 |
njohnston | you too@ | 14:36 |
lujinluo | #endmeeting | 14:36 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:36 | |
openstack | Meeting ended Thu Aug 16 14:36:12 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:36 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.html | 14:36 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.txt | 14:36 |
openstack | Log: http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.log.html | 14:36 |
*** mjturek has quit IRC | 14:36 | |
*** mjturek has joined #openstack-meeting-5 | 14:38 | |
*** mjturek has quit IRC | 14:38 | |
*** mjturek has joined #openstack-meeting-5 | 14:40 | |
*** lujinluo has quit IRC | 14:40 | |
*** mjturek has quit IRC | 14:41 | |
*** mjturek has joined #openstack-meeting-5 | 14:42 | |
*** mjturek has quit IRC | 14:43 | |
*** mjturek has joined #openstack-meeting-5 | 14:43 | |
*** mjturek has quit IRC | 14:51 | |
*** mjturek has joined #openstack-meeting-5 | 14:52 | |
*** annp_ has quit IRC | 14:59 | |
*** ricolin has joined #openstack-meeting-5 | 15:15 | |
*** mjturek has quit IRC | 15:20 | |
*** mjturek has joined #openstack-meeting-5 | 15:22 | |
*** yamahata has quit IRC | 15:22 | |
*** mjturek has quit IRC | 15:34 | |
*** mjturek has joined #openstack-meeting-5 | 15:39 | |
*** mjturek has quit IRC | 15:39 | |
*** mjturek has joined #openstack-meeting-5 | 15:40 | |
*** hungpv has quit IRC | 15:49 | |
*** mjturek has quit IRC | 15:49 | |
*** mjturek has joined #openstack-meeting-5 | 15:52 | |
*** mjturek has quit IRC | 15:56 | |
*** mjturek has joined #openstack-meeting-5 | 16:04 | |
*** mjturek has quit IRC | 16:14 | |
*** mjturek has joined #openstack-meeting-5 | 16:16 | |
*** mjturek has quit IRC | 16:22 | |
*** mjturek has joined #openstack-meeting-5 | 16:25 | |
*** derekh has quit IRC | 16:29 | |
*** mjturek has quit IRC | 16:43 | |
*** ricolin has quit IRC | 17:33 | |
*** skazi has joined #openstack-meeting-5 | 17:37 | |
*** slaweq has quit IRC | 17:44 | |
*** skazi has quit IRC | 17:57 | |
*** skazi has joined #openstack-meeting-5 | 17:59 | |
*** yamahata has joined #openstack-meeting-5 | 18:10 | |
*** slaweq has joined #openstack-meeting-5 | 19:03 | |
*** slaweq has quit IRC | 19:19 | |
*** slaweq has joined #openstack-meeting-5 | 19:22 | |
*** slaweq has quit IRC | 19:38 | |
*** slaweq has joined #openstack-meeting-5 | 20:34 | |
*** slaweq has quit IRC | 21:49 | |
*** hongbin has quit IRC | 22:36 | |
*** slaweq has joined #openstack-meeting-5 | 23:10 | |
*** slaweq has quit IRC | 23:15 | |
*** njohnston has quit IRC | 23:27 | |
*** njohnston has joined #openstack-meeting-5 | 23:28 | |
*** slaweq has joined #openstack-meeting-5 | 23:52 | |
*** slaweq has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!