Thursday, 2018-08-16

*** ricolin has joined #openstack-meeting-501:35
*** yamahata has quit IRC01:43
*** yamahata has joined #openstack-meeting-503:06
*** skazi has quit IRC05:53
*** beisner has quit IRC06:40
*** beisner has joined #openstack-meeting-506:40
*** slaweq_ has joined #openstack-meeting-506:51
*** slaweq_ has quit IRC07:00
*** slaweq_ has joined #openstack-meeting-507:04
*** hongbin has quit IRC07:23
*** lemko[m] has quit IRC08:11
*** ricolin has quit IRC08:12
*** skazi has joined #openstack-meeting-508:24
*** ianychoi_ has quit IRC08:42
*** derekh has joined #openstack-meeting-508:43
*** roman_g has joined #openstack-meeting-508:52
*** skazi has quit IRC11:50
*** lujinluo has joined #openstack-meeting-512:10
*** lujinluo has quit IRC12:51
*** mjturek has joined #openstack-meeting-512:52
*** mjturek has quit IRC13:30
*** mjturek has joined #openstack-meeting-513:37
*** mjturek has quit IRC13:45
*** mjturek has joined #openstack-meeting-513:45
*** hongbin has joined #openstack-meeting-513:49
*** hungpv has joined #openstack-meeting-513:54
*** lujinluo has joined #openstack-meeting-513:56
lujinluo#startmeeting neutron_upgrades14:00
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: neutron_upgrades)"14:00
openstackThe meeting name has been set to 'neutron_upgrades'14:00
lujinluoo/14:00
*** annp_ has joined #openstack-meeting-514:00
*** mjturek has quit IRC14:00
lujinluook, I will wait for my friends14:00
*** njohnston has joined #openstack-meeting-514:01
njohnstono/14:01
lujinluoo/14:01
lujinluoIt seems it would be just the two of us today, haha14:01
lujinluo#topic OVO14:02
*** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)"14:02
njohnstonperhaps others will join in time :-)14:02
lujinluohaha, then let's wait for another 3 mins14:02
lujinluoafter that, let's start ;)14:02
njohnstonseems to be a fair number of gate issues of late14:02
njohnstonmany Zuul -1s14:03
lujinluoyeah14:03
lujinluohopefully it won't affect releasing Rocky14:03
hungpvyes, hope so14:03
lujinluooh, hey hungpv14:04
lujinluodo you know if Tuan will come today? hungpv14:04
njohnstonsadly our Berlin talk proposal was not accepted FYI :-(14:04
hungpvI think he will come late14:04
lujinluo:-( so sorry to hear that14:05
hungpvBut I'm not suee14:05
lujinluoI see. Then let's get started first14:05
hungpvYeah, 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 Level14:05
lujinluoslaweq respinned it and it hit the Zuul failure14:06
lujinluoI just rechecked it, and I think it is very close to merge14:06
lujinluo#link https://review.openstack.org/#/c/544206/ PortBinding OVO14:07
lujinluoI pushed a PS about one hour ago. It should pass, so welcome reviews!14:07
lujinluoif zuul is happy ;)14:07
njohnston:)14:08
lujinluo#link https://review.openstack.org/#/c/549168/ Router OVO14:08
lujinluoi guess you want to share something about this one? hungpv14:08
lujinluoIt is still pretty red14:08
*** mjturek has joined #openstack-meeting-514:11
lujinluoa quick comment would be https://review.openstack.org/#/c/549168/21/neutron/db/l3_db.py@25914:11
*** hungpv_ has joined #openstack-meeting-514:11
lujinluonot to change _update_router_db to _update_router_obj in one patch14:11
*** hungpv has quit IRC14:12
lujinluoi 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 places14:12
lujinluohungpv_: oh did you lose connection?14:12
hungpv_Yes, thanks for suggestions14:13
hungpv_My network acting weird recently14:13
hungpv_Last week I tried to sign in but got baned for spam14:13
njohnstonI think your suggestion is a good one lujinluo, looking at errors like this it looks like router objects are getting misallocated somehow:14:14
njohnstonERROR [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 slaweq14:14
lujinluoyes, I had same issued with Port object14:14
lujinluothey are too many in codes14:14
lujinluowe have to eat them bite by bite ;)14:14
njohnston+114:15
hungpv_+114:15
hungpv_I will think about it tomorrow14:15
lujinluohopefully this can at least take us one step further14:15
lujinluo#link https://review.openstack.org/#/c/579053/ OVO in subnet_alloc.py14:16
lujinluoI checked this one njohnston14:16
lujinluoBut I am not sure if you have tried what I tried14:16
njohnstonYes, and your comment is a good one; I will respin once my slew of meetings is done in 2h or so :-)14:17
njohnstonI have not tried that.14:17
lujinluogreat!14:17
lujinluo#link https://review.openstack.org/#/c/580721/ Port OVO in ipam pluggable backend14:18
lujinluoI did not have enough time for it yet, but I saw Hongbin reviewed it14:18
hongbino/14:19
*** hungpv has joined #openstack-meeting-514:19
lujinluoI think it is related although I have not tested it yet14:19
lujinluohello hongbin!14:19
hongbinlujinluo: hi14:19
hongbinyes, i left 2 comments in that patch14:20
*** hungpv_ has quit IRC14:20
hongbinone of them might relate to the test failure14:20
njohnstonYes.  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+114:21
hongbincool14:21
lujinluoGreat!14:21
lujinluo#link https://review.openstack.org/#/c/565773/ Tag OVO14:22
njohnstonI don't see lajoskatona on, but this is a good forum to talk about one thing14:22
lujinluohttps://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 this one?14:22
njohnstonline 70 here: https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@7014:23
lujinluoDo you mean you'd prefer that we move db_api.retry_if_session_inactive() to OVO as well?14:23
njohnstonIt 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 compatibility14:24
njohnstonYes, that is what I was thinking, but I am wondering if I am being excessively object oriented in my thinking on this14:24
lujinluoWell, I actually think this is a very good point14:25
lujinluoSo far, we have not had time to consider these decorators14:25
lujinluobut indeed they are dealing with db sessions14:25
lujinluoI would be in favor of considering moving them into OVO objects14:26
njohnstonit seems at least that a decorator like that should be as close as possible to the code that actually interacts with the database14:26
lujinluoagree14:26
njohnstonok, I will come up with a specific recommendation for where it should be relocated to and comment on the change14:26
lujinluoI will check the Google sheet Tuan created long ago and add these decorators, to keep us reminded14:27
njohnston#action njohnston Comment on https://review.openstack.org/#/c/565773/5/neutron/services/tag/tag_plugin.py@70 with location for decorator relocation14:27
njohnston#action lujinluo Add decorators to google tracking spreadsheet14:27
njohnston:-)14:27
njohnstonotherwise I will forget :-)14:28
lujinluonjohnston: thanks ;)14:28
lujinluo#link https://review.openstack.org/#/c/561834/ segmenta model to OVO14:28
lujinluothis one is ready to go as Stein opens14:29
njohnstonindeed14:29
njohnstonjust added a couple of extra cores for review14:29
lujinluothanks! i had a network interruption here i believe, suddenly could not open that page :(14:30
lujinluowell, thankfully that's all the active patches we have now14:30
lujinluo#topic open discussion14: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-ptg14:31
lujinluojust a FYI of the PTG etherpad14:31
lujinluoAs far as I could see, we do not have any topics related to OVO/upgrades yet14:32
lujinluoDoes anyone have anything else he/she wants to share?14:32
njohnstonI will try to drum up some help for OVO/upgrades through informal discussions, but I don't have any topics for it14:32
lujinluoI see. Then just write Help wanted? ;)14:34
lujinluowell, let's try to gather some thoughts in our next meeting14:34
njohnstonsomething like that ;)14:34
lujinluoOne week for us to spot unseen things14:35
njohnstonthanks everyone14:35
lujinluoif no one has anything else14:35
lujinluolet's call it a day14:35
lujinluohave a good day/night my friends14:36
njohnstonyou too@14:36
lujinluo#endmeeting14:36
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:36
openstackMeeting ended Thu Aug 16 14:36:12 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:36
openstackMinutes:        http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.html14:36
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.txt14:36
openstackLog:            http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-08-16-14.00.log.html14:36
*** mjturek has quit IRC14:36
*** mjturek has joined #openstack-meeting-514:38
*** mjturek has quit IRC14:38
*** mjturek has joined #openstack-meeting-514:40
*** lujinluo has quit IRC14:40
*** mjturek has quit IRC14:41
*** mjturek has joined #openstack-meeting-514:42
*** mjturek has quit IRC14:43
*** mjturek has joined #openstack-meeting-514:43
*** mjturek has quit IRC14:51
*** mjturek has joined #openstack-meeting-514:52
*** annp_ has quit IRC14:59
*** ricolin has joined #openstack-meeting-515:15
*** mjturek has quit IRC15:20
*** mjturek has joined #openstack-meeting-515:22
*** yamahata has quit IRC15:22
*** mjturek has quit IRC15:34
*** mjturek has joined #openstack-meeting-515:39
*** mjturek has quit IRC15:39
*** mjturek has joined #openstack-meeting-515:40
*** hungpv has quit IRC15:49
*** mjturek has quit IRC15:49
*** mjturek has joined #openstack-meeting-515:52
*** mjturek has quit IRC15:56
*** mjturek has joined #openstack-meeting-516:04
*** mjturek has quit IRC16:14
*** mjturek has joined #openstack-meeting-516:16
*** mjturek has quit IRC16:22
*** mjturek has joined #openstack-meeting-516:25
*** derekh has quit IRC16:29
*** mjturek has quit IRC16:43
*** ricolin has quit IRC17:33
*** skazi has joined #openstack-meeting-517:37
*** slaweq has quit IRC17:44
*** skazi has quit IRC17:57
*** skazi has joined #openstack-meeting-517:59
*** yamahata has joined #openstack-meeting-518:10
*** slaweq has joined #openstack-meeting-519:03
*** slaweq has quit IRC19:19
*** slaweq has joined #openstack-meeting-519:22
*** slaweq has quit IRC19:38
*** slaweq has joined #openstack-meeting-520:34
*** slaweq has quit IRC21:49
*** hongbin has quit IRC22:36
*** slaweq has joined #openstack-meeting-523:10
*** slaweq has quit IRC23:15
*** njohnston has quit IRC23:27
*** njohnston has joined #openstack-meeting-523:28
*** slaweq has joined #openstack-meeting-523:52
*** slaweq has quit IRC23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!